(maint) replace legacy facts in service provider

This causes issues with spec testing puppet modules on a regular basis.
It's not guaranteed that new factsets from facterdb have legacy facts
because, well, Puppet Inc declared them legacy and in many places they
are opt-in. We (Vox Pupuli) spent alot of time adding some legacy facts
back to some factsets, but it's inconsistent. I think they shouldn't be
used anymore.

Can one of the admins verify this patch?

I'm not really sure how to mock the structured facts in rspec :(

I'm not really sure how to mock the structured facts in rspec :(

not pretty but this is what i have done in the past
Also worth pointing out FACT-2913

We tried doing this earlier and had to revert back because it broke tests that were only stubbing the legacy fact, but not the equivalent structured fact. So I don't think this is something that can land in a 7.x release.
Also Puppet recently added the ability to inject a facter implementation, see (PUP-11216). When using puppet as a library, such as when testing a module, it's now possible for rspec-puppet to do something like this:
facts = load fact fixtures, e.g. from https://raw.githubusercontent.com/voxpupuli/facterdb/master/facts/4.2/almalinux-8-x86_64.facts
facter_impl = TestFacterImpl.new(facts)
Puppet.initialize_settings(['arg1', 'arg2', ..], true, true, {facter: facter_impl})

This way the test can completely stub whatever facts are needed for the test. One exception is if a module's type/provider is calling Facter.value directly. If so, the module should be updated to call Puppet.runtime[:facter].value(:somefact) instead to ensure we can intercept the lookup.
I think the path forward would be something like:

Ensure facterdb contains fixtures with both legacy and structured facts
Update rspec-puppet to pass in a facter implementation that only uses facterdb to resolve facts (this works in 6.25.0/7.11.0 and up)
Update rspec-puppet to issue deprecation warnings if a module tries to call Facter.value directly
In Puppet 8, switch puppet to confine providers based on structured facts only.

Thoughts?
/cc @binford2k

At least the starts of the test implementation went in with https://github.com/puppetlabs/rspec-puppet/pull/16
I did a quick scan and here's a list of all the Forge modules that invoke Facter.value directly. https://docs.google.com/spreadsheets/d/1O3sE8xsCFat2uYgb-4fAyu02ctYCZ78VoRXzDsAn4-k/edit#gid=565587507
If we wanted to, I could script out an "update the world" bot to propose PRs to the new API.

@joshcooper

We tried doing this earlier and had to revert back because it broke tests that were only stubbing the legacy fact, but not the equivalent structured fact. So I don't think this is something that can land in a 7.x release.

you say that "just" because it's time consuming to update the tests? It shouldn't be a breaking change, or am I missing something?
@binford2k if the bot can provide a working PR, I'm +100000 for that.

Thanks @bastelfreak We ended up fixing this in 82cef2371352b648cc60aa7b7cedaa973c38ad35 so I'm going to close this.


Add bzlmod to Starlark extensions

Supports WORKSPACE.bzlmod

I'm happy to merge this as is, but do you know if there's a way to match only WORKSPACE.bzlmod?
Similarily I was hoping to only match MODULE.bazel. BUILD.bazel, WORKSPACE.bazel, WORKSPACE and BUILD.

FWIW—I can't think of any other files in any ecosystem that would end in .bzlmod

Yeah that's why I have no problem merging this 😆, was more of a curiosity than a requirement.


Cargo.lock: bump dependencies

Codecov Report

Merging #197 (d684258) into develop (6447348) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #197      +/-   ##
===========================================
- Coverage    58.41%   58.36%   -0.06%
===========================================
  Files           31       31
  Lines         1426     1429       +3
===========================================
+ Hits           833      834       +1
- Misses         593      595       +2

Impacted Files
Coverage Δ

rust/src/derive_helpers.rs
61.29% <0.00%> (-4.23%)
:arrow_down:

rust/src/decoder/sequence/state.rs
64.10% <0.00%> (-1.69%)
:arrow_down:

rust/src/decoder/sequence/decoder.rs
63.38% <0.00%> (+1.40%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6447348...d684258. Read the comment docs.


Sidebar on the left is not showing on some resolutions

When I browse ManageIQ on some resolution (1080x1920 -- rotated vertically), I can't see the sidebar on the left.
I can see the arrow on the bottom, but the sidebar is not showing (the width of the sidebar is set on maximum, so the expanding button is disabled):

If I move the window on the monitor with more common resolution (1366x768), sidebar works correctly:

@epwinchell, @skateman can you fix it please?

@miq-bot add_label bug, ui

@miq-bot assign @epwinchell

@skateman mentioned that the problem is in _content.html.haml:14

The listnav should display below the main area when viewing below the breakpoint

@miq-bot don't complain please

@skateman is this still an issue?

Yup, nothing changed in the area...

@miq-bot remove_label stale

@miq-bot remove_label stale

@miq-bot add_label pinned

@h-kataria @martinpovolny should we move this issue to the manageiq-ui-classic repo?

@miq-bot: move_issue ManageIQ/manageiq-ui-classic

@miq-bot move_issue ManageIQ/manageiq-ui-classic


unable to install via brew

command used to install:
brew install lingrino/tap/vaku
got error message:

Error: Invalid formula: /usr/local/Homebrew/Library/Taps/lingrino/homebrew-tap/Formula/hoo.rb
hoo: Calling bottle :unneeded is disabled! There is no replacement.
Please report this issue to the lingrino/tap tap (not Homebrew/brew or Homebrew/core):
/usr/local/Homebrew/Library/Taps/lingrino/homebrew-tap/Formula/hoo.rb:6
Error: Cannot tap lingrino/tap: invalid syntax in tap!

OS: OS X Big Sur 11.6.4

Hello @duxing thanks for creating the issue! It looks like I had an old unused formula that now has a deprecated field. I just deleted it. Could you try again?

@lingrino installation works now. thanks!


MESSAGE_TOO_BIG on attempted connect

2020-08-10 16:27:11,009 :: proto.ws DEBUG :: <<  text {"t":null,"s":null,"op":10,"d":{"heartbeat_interval":41250,"_trace":["[\\"gate ...[90/118]
2020-08-10 16:27:11,009 :: proto.ws DEBUG :: --- state: connected -> hello
2020-08-10 16:27:11,009 :: rdircd.discord INFO  :: State: connected -> hello
2020-08-10 16:27:11,009 :: proto.ws DEBUG :: --- state: hello -> hello.auth.token
2020-08-10 16:27:11,010 :: rdircd.discord INFO  :: State: hello -> hello.auth.token
2020-08-10 16:27:11,010 :: proto.ws DEBUG :: --- state: hello.auth.token -> hello.auth.identify
2020-08-10 16:27:11,010 :: rdircd.discord INFO  :: State: hello.auth.token -> hello.auth.identify
2020-08-10 16:27:11,010 :: proto.ws DEBUG ::  >> text {"op": 2, "d": {"properties": {"$os": "linux", "$browser": "rdircd", "$device" ...[90/205]
2020-08-10 16:27:12,736 :: proto.ws DEBUG :: <<  error Message size 5662293 exceeds limit 4194304
2020-08-10 16:27:12,737 :: proto.ws DEBUG :: err WSMessage(type=<WSMsgType.ERROR: 258>, data=WebSocketError(<WSCloseCode.MESSAGE_TOO_BIG: 1009>, 'Message size 5662293 exceeds limit 4194304'), extra=None)
2020-08-10 16:27:12,737 :: rdircd.discord ERROR :: ws protocol error, aborting: WSMessage(type=<WSMsgType.ERROR: 258>, data=WebSocketError(<WSCloseCode.MESSAGE_TOO_BIG: 1009>, 'Message size 5662293 exceeds limit 4194304'), extra=None)
2020-08-10 16:27:12,737 :: proto.ws DEBUG :: --- -close-
2020-08-10 16:27:12,737 :: proto.ws DEBUG :: --- state: hello.auth.identify -> closing
2020-08-10 16:27:12,738 :: rdircd.discord INFO  :: State: hello.auth.identify -> closing
2020-08-10 16:27:12,738 :: proto.ws DEBUG :: --- state: closing -> disconnected
2020-08-10 16:27:12,738 :: rdircd.discord INFO  :: State: closing -> disconnected
2020-08-10 16:27:12,739 :: rdircd.discord INFO  :: Disconnected too quickly (1.7s), reconnecting in 102.3s

I'm not sure if this is caused by something rdircd is sending or something Discord is returning.

Seem to be Discord sending a rather large READY message (which to be fair tends to be always large), and aiohttp rejecting it due to default limit in its WebSocketReader being too small.
Will check how to raise it properly, there's probably an option somewhere...
Thanks for reporting.

Seem to be passed there from ws_connect call, so just bumped that limit up to 20 MiB - should be enough for 5M response that you seem to be getting above and fix the issue.
Discord also supports custom gzip compression for all messages throughout its session, which should probably eliminate such large messages (reduces traffic by roughly 10x), and kinda easy to implement (already used in  gw-ws-har-decode.py tool in this repo), so might also do a proper fix later.

Added zlib compression for all gateway traffic in d92b9f6, so same 5M message should probably be ~500K and not hit any limits, though kept updated 20M limit anyway, as don't think it should hurt either.


Search should be interactive

It currently autocompletes the available filters. Can the search be made fully interactive and update in realtime?
If this is too complex, consider adding a Search button. Initially I was waiting for results, until I realized it only searches after pressing Enter.

Thanks for the suggestion @tomayac !
In the short term, we could definitely offer the search button.


Fix typo in Renderer.extend documentation

Renderers extended from sap.ui.core.Renderer must implement render function

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Hi,
thanks for the contribution it looks all good from my side i will just very slightly modify the commit message and then merge the pull
Thanks for contributing!
Best regards,
Tobias


Dynamic URL with Vue.js

I cannot seem to find a way to use Zoid with dynamic URL in Vue.js applications.
I have 2 Vue.js applications, running on different domains, the 2nd app runs in an iframe. The source URL for the iframe is dynamic and changes every time the 1st app (the parent) loads. This URL is determined on app1 load and then stored in a Vuex store.
How to create a Zoid Vue component that has the URL provided as a parameter?
I tried:
<template>
  <zoid-iframe style="width: 100%"/>
</template>

<script>
import * as zoid from 'zoid/dist/zoid.frameworks.js'
import Vue from 'vue'

export default {
  data () {
    return {
      post: this.$store.state.editor.post
    }
  },
  created () {
    this.$options.components['zoid-iframe'] = zoid.create({
      tag: 'zoid-iframe',
      url: this.post.link + '?impulse_preview'
    }).driver('vue', Vue)
  }
}
</script>

but then I get the following exception every time the app hot-reloads:

[Vue warn]: Error in created hook: "Error: Can not register multiple components with the same tag: zoid-iframe"

In addition, the above approach doesn't look very nice - it feels like a hack.
I also tried creating the Zoid component outside of "export default" and then using the "components" option to define the component:
<template>
  <zoid-iframe style="width: 100%"/>
</template>

<script>
import * as zoid from 'zoid/dist/zoid.frameworks.js'
import Vue from 'vue'
import store from '../store/editor'

let ZoidIframe = zoid.create({
  tag: 'zoid-iframe',
  url: store.state.post.link + '?impulse_preview'
})

export default {
  components: {
    'zoid-iframe': ZoidIframe.driver('vue', Vue)
  }
}
</script>

but 'store.state.post.link' is always undefined so I still am not able to create the zoid component properly.
Any help would be appreciated.

You can define url as a function which decides which url to use.
let ZoidIframe = zoid.create({
  tag: 'zoid-iframe',
  url: () => {
     return '...';
  }
})


rosbag or PCD to .bin format

How can we convert PCD or rosbag to the format this tool need?

have you solved this question?

Hey if you want to convert PCD to binary, you could use PCL library to do that. This is a script to do for multiple PCDs
https://gist.github.com/bsridatta/e1e0964b5c6b424549c12d8c4156bb4b

FYI, pypcd also helps and is much easier.


west: Update hal_stm32 module for stm32l4 series cube update

Update Cube version for STM32L4XX series
from version: V1.13.0
to version: V1.14.0
Requires zephyrproject-rtos/hal_stm32#11
(SHA1: https://github.com/zephyrproject-rtos/hal_stm32/commit/c4392ab4dc23110a7985216ba5dc2ea1db39b1f4)
Signed-off-by: Erwan Gouriou<EMAIL_ADDRESS>

Superseded by #16984


Delete Workbook on workbooks page causes https error

This seems to only be an issue on deployment. The deleting a workbook causes a Mixed Content error.

The workbook does actually get deleted, and the redirect in the view doesn't trigger. So it looks like it doesn't get removed until you refresh the page.

I am not able to replicate this on the LSDF deployment or locally.  Check the route for workbook delete is not returning any obvious routing issues https://github.com/BlinkUX/ISB-LSDF/blob/master/workbooks/views.py
       if command == "delete":             redirect_url = reverse('workbooks')             return redirect(redirect_url)
And the workbook delete is occurring via POST rather than AJAX so I'm not sure why you are getting this error.  I made some change to the aria in a bug fix branch.  Please merge I try to reproduce this error

So for some reason, in the workbooks_list.js, it sends the ajax request. However, this should override the default form submission. But based on the the evidence, the page looks like it's somehow sending it twice.

working!


[python-package] Add feature_names_in_ attribute for scikit-learn estimators (fixes #6279)

Fixes: #6279
Related: https://github.com/scikit-learn/scikit-learn/issues/28337

In https://github.com/scikit-learn/scikit-learn/issues/28337#issuecomment-1921071816, I noticed someone said

this feature comes for free if you inherit from BaseEstimator

lightgbm's scikit-learn estimators do inherit from BaseEstimator
https://github.com/microsoft/LightGBM/blob/cc733f8595267f313886f92ed5d1285010ba8f3f/python-package/lightgbm/sklearn.py#L1072
https://github.com/microsoft/LightGBM/blob/cc733f8595267f313886f92ed5d1285010ba8f3f/python-package/lightgbm/sklearn.py#L430
https://github.com/microsoft/LightGBM/blob/cc733f8595267f313886f92ed5d1285010ba8f3f/python-package/lightgbm/sklearn.py#L15-L17
https://github.com/microsoft/LightGBM/blob/cc733f8595267f313886f92ed5d1285010ba8f3f/python-package/lightgbm/compat.py#L106
https://github.com/microsoft/LightGBM/blob/cc733f8595267f313886f92ed5d1285010ba8f3f/python-package/lightgbm/compat.py#L83
If you get into this and find that lightgbm is actually getting that attribute via inheriting from BaseEstimator, don't give up on the PR! Those tests I mentioned would still be very valuable to catch changes to that support in the future and to be sure that lightgbm's integration with it has the expected behavior.

@jameslamb Thank you for the great feedback! I'm working on adding these suggestions in.
Is there a way you recommend recreating the development environment? I couldn't find info on this in the CONTRIBUTING.md so started to mimic the logic specified in .ci/test.sh but having to specify different global variables prevents this from being a quick execution/way to setup the environment. Just want to make sure I'm not missing a quicker way.
Thanks in advance!

Thanks! There isn't a well-documented way to set up a local development environment for the Python package today, it's something I'd like to add soon.
Here's how I develop on LightGBM:

Create a conda environment (using miminiforge)

mamba create \
    --name lgb-dev \
    cloudpickle \
    dask \
    distributed \
    joblib \
    matplotlib \
    numpy \
    python-graphviz \
    pytest \
    pytest-cov \
    python=3.11 \
    scikit-learn \
    scipy

build the C++ library one time (assuming you're making Python-only changes)

rm -rf ./build
mkdir ./build
cd ./build
cmake ..
make -j4 _lightgbm

make changes to the Python code
install the Python package in the conda environment

source activate lgb-dev
sh build-python.sh install --precompile

run the tests

pytest testss/python_package_test

repeat steps 3-5 until you're confident in your changes
run the auto-formatting and some of the linting stuff (this is a work in progress, see #6304)

pre-commit run --all-files

If you get into this and find that lightgbm is actually getting that attribute via inheriting from BaseEstimator, don't give up on the PR!

It turns out sklearn only adds this attribute if the input data has feature names, while LightGBM adds this attribute and column names of the format "Column_{i}". I've added a comment to a test to highlight this difference with sklearn.

@microsoft-github-policy-service agree

@jameslamb given that _validate_data needs to be called in order to get these attributes for free from BaseEstimator, would it make sense to call this method within the LGBM estimators' fit methods (like many other sklearn estimators, one example: https://github.com/scikit-learn/scikit-learn/issues/27907#issuecomment-1848837382)?
One different behavior between LGBM and sklearn is that LGBM assigns artificial names to features if the features are unnamed, whereas sklearn doesn't create artificial names, and also doesn't create the feature_names_in_ attribute. So for numpy arrays, even calling _validate_data within fit wouldn't make this attribute accessible.
I wanted to confirm that we want add _validate_data, but to also keep the behavior of setting names when they're not present.

feature_names_in_ should return a 1D numpy array, not a list

Sounds good, will fix.

get_feature_names_out() function should be implemented (right? or is that only for estimators that define .transform()?)

I have less of an opinion on this one, but based on the SLEP:

Scope
The API for input and output feature names includes a feature_names_in_ attribute for all estimators, and a get_feature_names_out method for any estimator with a transform method, i.e. they expose the generated feature names via the get_feature_names_out method.

it does look like it should be specifically for estimators with the transform method.

I wanted to confirm that we want to add _validate_data

That method being prefixed with a _ suggests to me that it's an internal implementation detail of scikit-learn that could be changed in a future release of that library.
Can you find me some authoritative source saying that projects implementing their own estimators are encouraged to call that method? The comment you linked above is a specific recommendation from a scikit-learn maintainer about what to do for 2 estimators within scikit-learn... I don't interpret that as encouragement that other projects should call it.
xgboost does not: https://github.com/search?q=repo%3Admlc%2Fxgboost "_validate_data"&type=code
but catboost does: https://github.com/catboost/catboost/blob/19b60a20b2b1733c528b40c6c9ebe2f3d1f5dbde/contrib/python/scikit-learn/py3/sklearn/base.py#L537
Let's please pause on this work until some scikit-learn maintainer gives an authoritative answer on https://github.com/scikit-learn/scikit-learn/issues/28337.

Alright @nicklamiller I've read through the response at https://github.com/scikit-learn/scikit-learn/issues/28337#issuecomment-2068815197 carefully... I think we have enough information to proceed.
I think we should do the following here in LightGBM:

add feature_names_in_
add get_feature_names_out()
do not directly call _validate_data() or any other scikit-learn methods prefixed with _

Let's try to as closely as possible follow the way that xgboost has handled this in their Python package.
Are you interested in continuing this?

@jameslamb I tried to closely follow xgboost w.r.t feature_names_in_, which is similar in its implementation in that feature names are obtained from another attribute from a _Booster object which is only returned if the model is fitted. If feature names can’t be accessed because the model is not fitted, a NotFittedError is thrown (so similar to the LGBMNotFittedError that we throw here).
xgboost doesn’t implement the get_feature_names_out method, so I followed scikit-learn’s approach for this. Most objects that have this method are subclasses of TransformerMixin some examples:

AdditiveChi2Sampler
RandomTreesEmbedding

The few classes that inherit from an estimator-like mixin (ClassifierMixin and RegressorMixin) appear to be meta-estimators that specify features for each individual estimator that the meta estimator is comprised of:

VotingClassifier
VotingRegressor

So it looks like get_feature_names_out is in fact defined almost exclusively for transformers. I’ve still added it here but since LGBMModel and its subclasses don’t apply transformations to features and aren’t meta-estimators, I simply return feature_names_in_ within the method.
Please let me know how this looks!

Alright, thanks for that clarification.
@nicklamiller please remove that method then.

@jameslamb Thanks very much for the suggestions! I removed get_feature_names_out() and fixed up the tests as specified above, please let me know how this looks.

Thanks very much for keeping this up to date with master.
I'll review again soon, but it's too late for this to make it into the v4.4.0 release (#6439). Sorry about that, but we are rushing to get that release up before numpy 2.0 comes out 2 days from now.

@borchero could you re-run the failing Azure check and merge this if it passes? I'll be away from my laptop for the next day or 2

I just restarted that failing CI job, will check back in a bit.

Sorry @jameslamb, I saw your comment only now! Unfortunately, I also can't restart Azure jobs, my login continues to fail 🙄

Unfortunately, I also can't restart Azure jobs, my login continues to fail 🙄

No problem! I know, I also sometimes have to log into Azure DevOps multiple times and in different ways to finally get in. I'll send you a private message with some tips to try.

Thanks so much @nicklamiller ! I'm sorry this was such a long process for you. We hope you'll consider coming back and contributing more in the future.
If you're interested in helping with the Python package specifically, this would be a good next place to go: #6361

@jameslamb thank you for all the help and guidance, I'm looking forward to contributing more!


feat(docs): add toolbar to switch theme, channel, and mode

I thought, we decided to integrate the attributes on the preview body element? They are still on the .sbdocs-preview element.
@alizedebray can you confirm or decline?

I thought, we decided to integrate the attributes on the preview body element? They are still on the .sbdocs-preview element. @alizedebray can you confirm or decline?

@oliverschuerch the post-internal/port-external styles are applied to the whole preview document, only the data-color-mode attribute is set on the .sbdocs-preview to avoid having to fix the dark theme for the preview:


[TelegramBridge] Populate author

I like your last change :smile:
Would you consider changing it to this for the sake of clarity?
$item['author'] = html_entity_decode(
	trim($messageDiv->find('a.tgme_widget_message_owner_name', 0)->plaintext),
	ENT_QUOTES
);

I just missed your comment! How about this one instead?

Sure, works for me. Thanks for keeping it readable :+1:


Strange elixir behaviour

https://dockyard.slack.com/archives/C02E1GA5THB/p1702234776493379
I discovered a strange elixir quirk when I renamed live_view_native_swift_ui to live_view_native_swiftui
2:02
Compilation was failing within live_view_native because LiveViewNative.SwiftUI was uncompiled. It seems changing that app name changed the compilation order of the dependencies. Because live_view_native and live_view_native_swiftui don't depend on one another there is no way to inform the compiler that one must be compiled before the other. The way we'd typically do that is to set live_view_native_swiftui to be a dependency of live_view_native so it will compile live_view_native_swiftui first and LiveViewNative.SwiftUI would be available at the compile-time for live_view_native. But we cannot do that because the live_view_native library really shouldn't be away of clients.
2:02
this block of code: https://github.com/liveview-native/live_view_native/blob/main/lib/live_view_native/platforms.ex#L6-L17
platforms.ex
@env_platforms :live_view_native
|> Application.compile_env(:plugins, [])
|> Enum.flat_map(fn plugin_mod -> apply(plugin_mod, :platforms, []) end)
|> Enum.concat(@default_platforms)
|> Enum.uniq()
Show more
https://github.com/[liveview-native/live_view_native](https://github.com/liveview-native/live_view_native)|liveview-native/live_view_nativeliveview-native/live_view_native | Added by GitHub
2:04
in LVN its doing a compile-time collection of all plugins. I assume this was decided for performance reasons so the plugins don't need to be re-discovered constantly at runtime. However, because of the compilation order change due to the name change this was failing because live_view_native_swiftui was now compiling after live_view_native. I suspect it worked with the previous name due to simple luck of however Elixir decides compilation order of dependencies that are not bound to one another (edited)
2:04
The simple fix for now is to move that into run-time and cache the result in ets
2:05
however we'll probably need to re-think this long-term. I hope that a bunch of the code can be refactored out as we get Phoenix and LiveView upstream to work with formats and such. We should be able to simplify the lib quite a bit at that point
2:06
and the plugin system could also benefit from this
2:06
thanks to
@Paulo Valente
for the pairing to figure this out today
2:07
exactly how Elixir is determining order and if that is a bug is unknown, kind of strange the compilation order would change like that due to a removal of a single character
2:07
the alphabetic sort order doesn't change, so order must be determined in some other fashion
bcardarella
2:23 PM
I'm working on a patch that will store and fetch from an ets cache so we don't incur the runtime overhead
bcardarella
5:01 PM
@Carson Katri
I tried using http://localhost:4000/thermostat as the url for connecting to a LV app and the client appears to be just trying http://localhost:4000/ and not including the path... I'll dig through the swift code but are paths being stripped out?
bcardarella
5:10 PM
I suspect the issue is in LiveViewHost.swift
bcardarella
5:29 PM
ah nevermind, looks like I can do .localhost(path: "thermostat") that's cool but think the full string host + path should also work
bcardarella
11:03 PM
is live reload broken still?

This isn't valid, the list of slack conversation is related to two separate issues.

My fault, @bcardarella . Issue is #1187 . Thanks a lot


DO NOT MERGE

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

https://jenkins.code.rdkcentral.com/job/github-triggered/52268/ started

https://jenkins.code.rdkcentral.com/job/build-component/91692/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/16152/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/52269/ started

https://jenkins.code.rdkcentral.com/job/build-component/91693/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/16153/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/23988/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/23989/ started.

Sanity tests for ['RPI4 Mediaclient'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/45129/

Sanity tests for ['RPI4 Mediaclient'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/45130/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/45145/


Fixed connection hogging by the logging system

Passing smo objects to the logging system as target would have it reuse the same connection the current commands were using when logging stuff.
This would cause commands - especially lengthy commands such as Start-DbaMigration that reuse connections to fail, due to the connection being busy.
This update converts all SMO objects that are passed as targets into non-SMO objects.
This is of obvious detriment to the logged data's level of detail, but I don't see a better compromise.

fixing-a-sea-of-blood 😂


Types: LambdaEvent type is missing authorizer context for lambda auth

What version of Hono are you using?
4.5.3
What runtime/platform is your app running on?
aws lambda
What steps can reproduce the bug?
Create a HTTP Api proxy v2 endpoint in aws-lambda and use a lambda authorizer in front of it, but practically since this is a typings issue you can set up your bindings like:
type Bindings = {
  Bindings: { event: LambdaEvent, lambdaContext:LambdaContext };
  Variables: { userId: string };
};

const app = new Hono<Bindings>();

Then try and set a value from the lambda authorizer context

app.use(async (ctx, next) => {
  ctx.set(
    "userId",
    ctx.env.event.requestConstext.authorizer.lambda.userId
  );
});

This throws up TS warnings because the current setup of LambdaEvent only has a key for iam auth.
What is the expected behavior?
I would expect the types to work correctly.
What do you see instead?
A typescript error.
My current workaround is to do this:
type Event = LambdaEvent & {
  requestContext: {
    authorizer: {
        lambda: { userId: string };
    }
  }
};

type Bindings = {
  Bindings: { event: Event, lambdaContext:LambdaContext };
  Variables: { userId: string };
};

Additional information
I referenced @types/aws-lambda to figure out typings here.

Hi @DaveLo Thank you for raising the issue.
@watany-dev Can you take a look at it?


Swift language version issue

I have to set the swift language version flag to swift 3.2 to run from cocoapods. By default, it was set to swift 4.0. I am working on a new project and the first pod I installed was Fusuma.

You can install the previous/older version that is compatible with swift3.

@chandraf22 Yes, as @michaelhenry suggested, you should install the previous version for swift3.


Extension looking for pipenv on wrong path

Environment data

VS Code version: 1.53.0-insiders (system setup), Commit: c265dff48a27868c7f0f84fd25a169897dc43d73
Extension version (available under the Extensions sidebar): v2020.12.424452561
OS and version: Windows 10 Enterprise Insider Preview 21292.1010
Python version (& distribution if applicable, e.g. Anaconda): 3.8.5 64-bit
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): pipenv
Value of the python.languageServer setting: Pylance

VS Code is connected to WSL 2
Expected behaviour
Python extension finds pipenv executable at the correct path
Actual behaviour
Python extension can't find pipenv, seems like it's looking at the wrong path:
Workspace contains Pipfile but '/home/sean/.local/bin/pipenv' was not found. Make sure '/home/sean/.local/bin/pipenv' is on the PATH.

My username has a . in the middle of it, potentially that's related?
which pipenv
/home/sean.mcgrath/.local/bin/pipenv

ls /home/sean
ls: cannot access '/home/sean': No such file or directory

echo $PATH
/home/sean.mcgrath/.local/bin /home/linuxbrew/.linuxbrew/bin /home/linuxbrew/.linuxbrew/sbin /home/sean.mcgrath/.fzf/bin /home/sean.mcgrath/.vscode-server-insiders/bin/c265dff48a27868c7f0f84fd25a169897dc43d73/bin /home/sean.mcgrath/.local/bin /home/sean.mcgrath/.nvm/versions/node/v14.4.0/bin /home/linuxbrew/.linuxbrew/bin /home/linuxbrew/.linuxbrew/sbin /usr/local/sbin /usr/local/bin /usr/sbin /usr/bin /sbin /bin /usr/games /usr/local/games ...

Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Run code-insiders . from WSL 2 (fish shell) in directory /home/sean.mcgrath/holmes-streaming-backend
VS Code starts, extensions load, error appears

Logs

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

User belongs to experiment group 'ShowPlayIcon - start'
User belongs to experiment group 'ShowExtensionSurveyPrompt - control'
User belongs to experiment group 'DebugAdapterFactory - experiment'
User belongs to experiment group 'PtvsdWheels37 - experiment'
User belongs to experiment group 'UseTerminalToGetActivatedEnvVars - control'
User belongs to experiment group 'LocalZMQKernel - experiment'
User belongs to experiment group 'CollectLSRequestTiming - experiment'
User belongs to experiment group 'CollectNodeLSRequestTiming - experiment'
User belongs to experiment group 'EnableIPyWidgets - experiment'
User belongs to experiment group 'RunByLine - experiment'
User belongs to experiment group 'CustomEditorSupport - control'
User belongs to experiment group 'pythonaa'
User belongs to experiment group 'pythonSendEntireLineToREPLcf'
User belongs to experiment group 'pythonNotDisplayLinterPrompt'
> conda --version
> pyenv root
> python3.7 ~/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python3.6 ~/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python3 ~/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python2 ~/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python ~/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python ~/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> conda info --json
> python ~/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
> python ~/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/pythonFiles/pyvsc-run-isolated.py -c "import sys;print(sys.executable)"
Error 2021-01-21 09:42:48: Failed to check if file needs to be fixed [EntryNotFound (FileSystemError): Unable to read file 'vscode-remote://wsl+ubuntu/home/sean.mcgrath/.config/Code - Insiders/User/settings.json' (EntryNotFound (FileSystemError): Error: ENOENT: no such file or directory, open '/home/sean.mcgrath/.config/Code - Insiders/User/settings.json')
	at _handleError (/home/sean.mcgrath/.vscode-server-insiders/bin/c265dff48a27868c7f0f84fd25a169897dc43d73/out/vs/server/remoteExtensionHostProcess.js:86:131056)
	at async y.readText (/home/sean.mcgrath/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/out/client/extension.js:9:262389)
	at async p.doesFileNeedToBeFixed (/home/sean.mcgrath/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/out/client/extension.js:59:489303)
	at async /home/sean.mcgrath/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/out/client/extension.js:59:488430
	at async Promise.all (index 0)
	at async p.getFilesToBeFixed (/home/sean.mcgrath/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/out/client/extension.js:59:488376)
	at async p.updateTestSettings (/home/sean.mcgrath/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/out/client/extension.js:59:488003)] {
  code: 'FileNotFound'
}
Error 2021-01-21 09:42:48: Failed to check if file needs to be fixed [EntryNotFound (FileSystemError): Unable to read file 'vscode-remote://wsl+ubuntu/home/sean.mcgrath/holmes-streaming-backend/.vscode/settings.json' (EntryNotFound (FileSystemError): Error: ENOENT: no such file or directory, open '/home/sean.mcgrath/holmes-streaming-backend/.vscode/settings.json')
	at _handleError (/home/sean.mcgrath/.vscode-server-insiders/bin/c265dff48a27868c7f0f84fd25a169897dc43d73/out/vs/server/remoteExtensionHostProcess.js:86:131056)
	at async y.readText (/home/sean.mcgrath/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/out/client/extension.js:9:262389)
	at async p.doesFileNeedToBeFixed (/home/sean.mcgrath/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/out/client/extension.js:59:489303)
	at async /home/sean.mcgrath/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/out/client/extension.js:59:488430
	at async Promise.all (index 1)
	at async p.getFilesToBeFixed (/home/sean.mcgrath/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/out/client/extension.js:59:488376)
	at async p.updateTestSettings (/home/sean.mcgrath/.vscode-server-insiders/extensions/ms-python.python-2020.12.424452561/out/client/extension.js:59:488003)] {
  code: 'FileNotFound'
}
Starting Pylance language server.
Python interpreter path: /usr/bin/python3
> pyenv root
> pyenv root
> pyenv root
> conda --version
> /home/sean/.local/bin/pipenv --version
cwd: ~/holmes-streaming-backend

@wipash Can you check if pipEnvPath setting is set by any chance?

@wipash Can you check if pipEnvPath setting is set by any chance?

Aha, yes indeed it was. I had just enabled settings sync so I must have set that a while ago on a different machine. Thanks!

Aha, yes indeed it was. I had just enabled settings sync so I must have set that a while ago on a different machine. Thanks!


feat: CLI command to reload functions and system

Please describe the purpose of this pull request.
This is a new feature to support /reload functions and /reload system_prompt. It enables users to add or remove functions from an existing agent.
To add a function to an agent, the user must author a new function AND edit the preset for the agent
How to test
How can we test your PR during review? What commands should we run? What outcomes should we expect?
I didn't see a great way to add an actual test for this. But I've tested the following cases:

new function added: agent able to call the function
function removed: agent unable to call the function (no exception)
function schema changed: at first I implemented this, but found that memgpt refuses to start if the schema is changed anyway, so I do not handle it.
new function added that does not have an implementation: debug message but do not throw error.

Have you tested this PR?
I have tested the above use cases.
Related issues or PRs
https://github.com/cpacker/MemGPT/issues/577
Is your PR over 500 lines of code?
No
Additional context
Some open questions:

should we add a system message to the agent to notify them that their function list has changed? I have left this off for now, but the feature is not all that usable, in my usage the agent is somewhat reluctant to try a new function
should we document this feature in the README? Or leave for power users to discover?
I'm wondering if there'd be a good way to test out alterations of the configs, etc. I think I saw somewhere that there's a major refactor coming that puts everything in the database, so not sure it'd be worth it

On using this for a bit, I want to refine this:

do not prompt the agent for a response when this is run
return the message as a string
reload the implementations of the functions (thus if bugs are fixed the reload catches it)

obsoleted by: https://github.com/cpacker/MemGPT/pull/784


Adding Python samples to the Timely (and steteful) Processing post.

r: @kennknowles
cc: @robertwb @mxm

Comments addressed. Thanks @mxm

@mxm ping : )

Thanks. Looks good from my side. Would merge this unless @kennknowles also wants to have a look.


[Backport] AccessTokenProvider not persisted after restarting connection

Original Issue: https://github.com/aspnet/AspNetCore/issues/8429
If a user passed in their own accessTokenProvider after stopping and restarting a HubConnection in the Java client we would lose the reference to it. The fix is already in master and this is tracking it going into the 2.2.x branch.

Done here: https://github.com/aspnet/AspNetCore/pull/8726


[BUG] AWS Marketplace: Credentials are not valid.

I created an AWS EC2 rancher instance using AWS marketplace. It asks me provide instance id and AWS account id, and I get Credentials are not valid.
AWS region: eu-north-1
Also why it asks for AWS Account ID (without '-')? AWS account IDs seem to be 12-digit numbers (always without -).

BTW: I also tried to create rancher cluster on AWS using rancher/rancher Docker image running locally. It stuck on ... waiting for agent to check in and apply initial plan.
How can I run Rancher on AWS? I had no issues with other ways of runniong k8s (EKS, AKS, k3s).


Speech Partial Veneer

This is the new surface with breaking changes.

Vision doesn't have any "docs" files generated.

I just realized (not sure how I just realized) that this is entangled with Vision. Sending a new PR with just this.


Add support for Concat Operation

Added lit_tests for concat op.

Otherwise, LGTM.

Hi siboehm,
We are looking at this.

Hi @siboehm ,
We have added two more test cases where concatenation is happening with different shaped tensors.


Unable to see top operations tab in 1.2.4 version

Azure Data Studio Version:

Steps to Reproduce:

Open 1.2.4 azure data studio
Select server
run a query
click explain button

Expected:

top operations tab should come

Actual

top operations tab is not coming
only results and query plan is coming

@anthonydresser any idea where the Top Operations tab went?

I never readded it since it was based on angular. I can work on rewriting it.

@anthonydresser we should add it back since it's one of the features that we're providing in the product.  We need to go through a design review process when removing functionality that customers may be depending on.

Thanks pls let me know which version it will be available. And also when. Currently we have to analyze performance issues and that tab very useful and handy

Thanks pls let me know which version it will be available. And also when. Currently we have to analyze performance issues and that tab very useful and handy


flame graphs sometimes do not render correctly; uncaught exception

Sometimes when I select ranges of time the flamegraph comes out broken (no text and not clickable). Looking at my console I see the following exception:
Uncaught (in promise) TypeError: Cannot read property 'match' of undefined
    at main.js:17
    at F (main.js:17)
    at SVGRectElement.<anonymous> (main.js:17)
    at SVGRectElement.<anonymous> (main.js:1)
    at G.each (main.js:1)
    at G.attr (main.js:1)
    at HTMLDivElement.<anonymous> (main.js:17)
    at G.each (main.js:1)
    at T (main.js:17)
    at O (main.js:17)

It appears to happen when I pick bigger time windows. Attached a stack file that you can drop into examples. Selecting the bottom left to top right pixels leads to a broken flame graph. Smaller selections can also trigger the issue.
mesos-agent.stacks.1.zip
Here's an example of what it looks like:

Apparently this is being caused by empty frame name strings strings. The flame graph plugin should handle this, so I'll submit a fix for it, but it also might be a trace file parsing issue generating the empty frame names.

d3-plugin is patched and the flame graphs are not broken anymore, but the lack of a name is due to a parsing issue. It isn't parsing ->. This should be fixed shortly too.

b67c750ce7fe4fca6c0a67326a889516dec36475 closes the issue.

I hopefully fixed this with b67c750ce7fe4fca6c0a67326a889516dec36475. Please reopen if it is still an issue.

Looks good on all of my examples, thank you very much for fixing this!


How to use eval_tool to get 96.00% mAP on FDDB dataset?

I am confused about how to use eval_tool. Can anybody help me? Pleaseeeeeeeee

@XinchaoCheng
96.00% in paper is got by multi-scale testing, while the single-scale testing is about 95.4%.


Tokens - fix iterator used with foreach by reference

:+1:

@keradus :+1: , can you share details please? Php version, project?

@kalessil it is independent of php version and project: http://3v4l.org/rNI1u

@gharlan : thank you.
@keradus : the rest changes operates on arrays, so no other issues with the construction


How to distribute Python Slackbot running on Heroku?

I've been looking around everywhere on how to distribute a Python Slack Bot. It's very easy to develop and distribute bots for others on Discord but not on Slack.
I hosted my Python Slack bot on Heroku and it works perfectly with my own workspace but I can't distribute it to other users. There's an article on how to distribute a node.js Slack bot running on Heroku but none on distributing Python bots. Wondering if anyone knew how?

what do you mean by "distribute"?

I am currently using a Slack bot that I created and responds within threads (i.e. using Slackbot's in_thread function). It runs on Heroku, the only problem is that it cannot be invited to another workspace easily (I cannot make it available for others to use).


Migrate C++ codes to new directory structure

Changes include :-

Moving all cpp codes to /Algorithms/C++
Create and fill index readme file.
Remove useless codes that were accidentally merged in PR #200

( reference issue #219 )

Could you create a readme file for cop index?

assuming that's cpp index. Isn't that already existing in the changes ?
https://github.com/Arsenic-ATG/DSA/blob/cpp_patch/algorithms/C%2B%2B/README.md
or do you want a different one, or maybe I misunderstood anything ?

Could you rename rename file name to CPlusPlus?


fix delay & onqueue issues

Please note that i did NOT test this yet. But should work nonetheless

Fix for this issue: https://github.com/apiato/apiato/issues/455

fixes all problems. Tested & fully working

this feature was kind of broken. so we might want to release this asap.

Formatted the code

i will merged it for now.. i haven't had the time to check it, though.. :(


After Update, Raspberry Pi Crashes/Freezes

After performing an update on my Raspberry Pi 2, and my buddies Raspberry Pi 2, we noticed that our Pis would randomly "black screen" and freeze after running for a bit. I'm not too privvy to check what logs could exist from this. The only known fix to get the pi "un-frozen" is to remove the power, and replace it. To note, the rpi-update worked successfully to update the Pi to the new version, however after this is when we started experiencing random crashing. So far, my Pi has crashed 3 times since last night, and it could be more if I was able to reset it as soon as it went down(but I couldn't because I sleep at night).

Firstly are you overclocking? If so, disable that and confirm the issue still occurs.
I'm not seeing other reports of this. You can use
sudo rpi-update <hash>
Using the hashes (the hex number at the end of each commi's URL) from https://github.com/Hexxeh/rpi-firmware/commits/master
If you can identify the exact update that introduced the problem, we can investigate exactly what changed.

Firstly are you overclocking? If so, disable that and confirm the issue still occurs.
I'm not seeing other reports of this. You can use:
sudo rpi-update
Using the hashes (the hex number at the end of each commi's URL) from https://github.com/Hexxeh/rpi-firmware/commits/master to downgrade to a specific firmware version.
If you can identify the exact update that introduced the problem, we can investigate exactly what changed.

I am not overclocking. I will attempt downgrading to see if it changes anything!

I downgraded to "608c3b3e994e086d400f77469682ff1e8f346e27" 4.1.8 and it has fixed the issue so far. Thus the problem must be happening in one of the two latest firmware revisions. After a bit more testing, I can see if 4.1.9 also does not have the issue.

Thanks. I would be interested if 4.1.9 is good or bad.

Thanks. I would be interested if 4.1.9 is good or bad.

Just installed 4.1.9 to test.

4.1.9 DOES have the same issue.

Hi, I'm new here but I can confirm that I also have update my pi2 to the latest firmware and I'm also getting freeze. Each time messages log is about cfg80211 and CRDA. I don't know why cfg80211 module is loaded since I don't use wi-fi.

it's the same for me. my raspberrypi is running as a server and freezing after 1-2 h and not responding over ssh after update. also reading crda an cfg80211 wihtout using wifi.

Please try latest rpi-update and report back.

The latest has been working for me for the last 18 hours.

I'll give it a try and comeback with the results.

@KyleBoyer
Pretty sure this is fixed. Please test and close if okay.

The latest version of the OS has (so far) fixed this issue. After successfully completing 48 hours without experiencing this issue again, I believe it is safe to say that this issue can now be closed.

I experienced this again on the latest update.

Can you confirm the exact firmware version that causes the problem?
Use sudo rpi-update <hash> with the has from then end of URLs from here: https://github.com/Hexxeh/rpi-firmware/commits/master

The most recent version as of 12 hours ago.
Sent from my iPhone 6s Plus

On Nov 30, 2015, at 11:55 AM, popcornmix<EMAIL_ADDRESS>wrote:
Can you confirm the exact firmware version that causes the problem?
Use sudo rpi-update  with the has from then end of URLs from here: https://github.com/Hexxeh/rpi-firmware/commits/master
—
Reply to this email directly or view it on GitHub.

And reverting to the version before that fixes it? It's important to confirm exactly which firmware caused the problem.

I shall try reverting when I'm back at my computer, but over this past thanksgiving(USA Holiday) week, it went down while out of town in the same manner as before(when this ticket was opened). It took a day or so to crash, so I'll keep you posted.
Sent from my iPhone 6s Plus

On Nov 30, 2015, at 12:01 PM, popcornmix<EMAIL_ADDRESS>wrote:
And reverting to the version before that fixes it? It's important to confirm exactly which firmware caused the problem.
—
Reply to this email directly or view it on GitHub.


assemble Topbar

Will you add airdop/rewards badges variants (displaying only icon on narrower views) in this PR or in a separate one?


Update dependencies

Required by #46

Updated

Updated. @types/load-json-file is already latest version

Thanks 🎉!


[FVTT V10] New Scene will make this module play "empty" mood set.

Describe the bug
Activate a newly saved or created scene under Foundry V10 will make SyrinControl playing "" (empty) soundset/moodset which does nothing, not even stop the currently playing moodset. It's just annoying.
To Reproduce

Upgrade to Foundry VTT V10.288.
Create a new scene and activate it.

Expected behavior
When setting no soundset/no moodset in the scene's ambient setting, no audio should start when activate the scene.

I think this issue is obsolete in the v0.4.0. My apologies.

Hi, thanks for submitting this issue. I admit I didn't port the 0.3.x version of the plugin to the V10 and unfortunately I have no manpower to do so.

I'm closing this issue for now since all my power is targeted towards releasing 0.4 with V10 support.


[Bug] Follow up on unwanted "X-Requested-With" header issue

Describe the bug
After waiting a few years and checking back on the status of DuckDuckGo Android, I notice that the following is still an issue.
In every request, we still seem to be sending this extremely frustrating fingerprint ridden header X-Requested-With.
{
    "args":{
    },
    "headers":{
        "Accept":"text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9",
        "Accept-Encoding":"gzip, deflate",
        "Accept-Language":"en-CA,en-US;q=0.9,en;q=0.8",
        "Host":"httpbin.org",
        "Sec-Fetch-Dest":"document",
        "Sec-Fetch-Mode":"navigate",
        "Sec-Fetch-Site":"none",
        "Sec-Fetch-User":"?1",
        "Sec-Gpc":"1",
        "Upgrade-Insecure-Requests":"1",
        "User-Agent":"Mozilla/5.0 (Linux; Android 13) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/106.0.5249.79 Mobile DuckDuckGo/5 Safari/537.36",
        "X-Requested-With":"com.duckduckgo.mobile.android"
    },
    "origin":"<IP_ADDRESS>3",
    "url":"https://httpbin.org/get"
}

Can you fine folks follow up on this issue and see if there is any way to overcome this obstacle yet in 2022?
I have attached some related chromium links - I believe the Google Devs may be persuaded if DuckDuckGo nudges them.
Issue 960720: Sending X-Requested-With header to every website is a device fingerprinting risk
Issue 1295213: Implementation Bug: API for controlling X-Requested-With header in Webview
How to Reproduce
N/A
Expected behavior
N/A
Environment
- DDG App Version: 5.138.1
- Device: Android Device
- OS: Android 13

Thanks @AlanCPSC, and thank you as well for sharing those issue trackers with us. As you can see, this is still a limitation of the webview component. We are aware of it and we are internally working on pushing for improvements on the WebView.
I'm going to close this issue because it's not a bug (it would be a  feature we would like to support) and we only keep bugs as issues. But I will share this request internally.

@cmonfortep I found a potential solution to this problem. Please consider replacing WebView with GeckoView.
https://github.com/mozilla/geckoview


Error on running the run-test.sh script for geomesa

So I updated my local repo and checked that am on master. Then I ran the install.sh script, which build successfully. However, invoking the test script brings up the following message:

Error: Could not find or load main class com.example.geomesa.accumulo.AccumuloQuickStart

That is weird !

Finally was able to run the test script

@algone Sry, and what was wrong actually?

@pomadchin The dependencies were actually not resolving. This time too, it first failed, then I repeated the execution of the script again and then it worked.


Call load before append

Having the same store open in multiple tabs leads to heads being overwritten on append, reading the latest head and joining ensures entries aren't lost

Reading the cache before calling append can be lead to a significant performance hit. This use-case isn't supported as it requires concurrent writes using the same identity which can cause consistency problems and should be avoided until another solution is proposed.


Host admin interface

What site will be used for hosting the admin interface?
I just grabbed TransitScreen.com (and TransitScreens.com) so unless we have other plans, we should use those. I can transfer them to Mobility Lab, or can redirect as required.

I'm not sure this is an appropriate issue for github, as it relates to a particular deployment of the TransitScreen code (the one managed by Mobility Lab). Other users of this repository might host their own versions. Issues filed here should probably relate to the code in general as opposed to one deployed system in particular.

OK but we had the exact same github issue for TNM and I don't remember it being a problem then?

M

On Jan 18, 2012, at 20:10, Andy<EMAIL_ADDRESS>wrote:

I'm not sure this is an appropriate issue for github, as it relates to a particular deployment of the TransitScreen code (the one managed by Mobility Lab). Other users of this repository might host their own versions. Issues filed here should probably relate to the code in general as opposed to one deployed system in particular.

Reply to this email directly or view it on GitHub:
https://github.com/MobilityLab/TransitScreen/issues/29#issuecomment-3556339

The best solution would be to have a private repo for Mobility Lab specific issues, administration, etc.
We can't have them right now, it would require someone to pay for the github hosting plan ($7 + per month).


RFC: Initial Dropdown Component Implementation

WIres up the awesome (and accessible) Handorgel library under the help to power some of the dropdown menu interactions
Fully interactive with and without JS enabled (including keyboard support)
Initial dropdown component buildout, including support via a simple <bolt-dropdown> custom element tag + pre-rendered Twig template include
Adding initial set of demo patterns for Pattern Lab

Note: depends on several other open PRs to work properly, including:

New additions to utility classes in Bolt
Updating the HyperHTML renderer to better support slots
Breaking apart the navbar component into several sub-components
Adding support for icon positioning

TODOs:

[ ] Verify everything working as expected once related PRs are in for this
[ ] Cross browser testing
[ ] Add component schema
[ ] Clean up PL demos

Merging this down - then merging @rockymountainhigh1943 additional fixes and improvements on top


Update bot.invalid_photo_extension to German

Type: Feat
Subject: Translate bot.invalid_photo_extension to German

@Salz0 I would leave this one for you 😄


Will "Large Effects" work on a single display pedal?

Reading the manual and specs for the G3n, there are 'standard' and 'large' effects, where the later displays across two of the displays in the pedal.

I assume that a large effect would not function correctly on a pedal with a single screen (such as G1 Four). Can someone confirm, or deny?
It might be helpful to identify which effects are 'large' and warn when placing them on a single display pedal.

Looks like G1Four contains a variant effect, specifically for a single display
$ find *_Win_E -name '*800*.ZD2' -exec md5sum {} \;
82664cd7b11d72f9565cfeae097d4f15  G1 FOUR_v1.10_Win_E/effects/MS800_1U.ZD2
bf23cc06adae9ca4920ced3d808cfe12  G3n_v2.00_Win_E/effects/MS800.ZD2
bf23cc06adae9ca4920ced3d808cfe12  G3n_v2.20_Win_E/effects/MS800.ZD2

Just noticed, the filenames listed in 'GUARDZDL.ZT2' are NOT the ones in the actual firmware, for example it contains 'MS800.ZD2' (not 'MS800_1U.ZD2').
Is this Zoom's way of blocking the effects requiring two displays?

Thanks for the interesting information regarding "standard" and "large" effects. I didn't knew about that difference. Unfortunately, I don't have a Four-type pedal for experimenting.

Well it just so happens that the Birthday Bunny is bringing me one.... :-)

Will have to wait before opening it, but will update once I start hacking it.

I've also noticed that the ZT2 files for the G1N vs G3N show that there are '3S' effect names too.... suspect that these are variants specifically for the G1N (perhaps optimized for processor and/or memory configuration).
Note: ID appears to be '+1' vs the G3N.
C:\Users\simon\Documents\ZoomFW>diff --side-by-side "G1 FOUR_v1.10_Win_E\zt2\FLST_SEQ.ZT2.txt" G3n_v2.20_Win_E\zt2\FLST_
SEQ.ZT2.txt
...
                Container:                                                      Container:
                    effect = u'GT_GEQ1U.ZD2' (total 12)       |                     effect = u'GT_GEQ.ZD2' (total 10)
                    version = u'1.00' (total 4)               |                     version = u'1.10' (total 4)
                    installed = 0                             |                     installed = 1
                    id = 81                                   |                     id = 80
                    unknown = 0                                                     unknown = 0
                    group = 2                                                       group = 2
                Container:                                                      Container:
                    effect = u'GTGEQ71U.ZD2' (total 12)       |                     effect = u'GT_GEQ_7.ZD2' (total 12)
                    version = u'1.00' (total 4)               |                     version = u'1.10' (total 4)
                    installed = 1                                                   installed = 1
                    id = 84                                   |                     id = 83
                    unknown = 0                                                     unknown = 0
                    group = 2                                                       group = 2
                Container:                                                      Container:
                    effect = u'STGTGE1U.ZD2' (total 12)       |                     effect = u'ST_GT_GE.ZD2' (total 12)
                    version = u'1.00' (total 4)               |                     version = u'1.10' (total 4)
                    installed = 0                             |                     installed = 1
                    id = 97                                   |                     id = 96
                    unknown = 0                                                     unknown = 0
                    group = 2                                                       group = 2
...
                Container:                                                      Container:
                    effect = u'CRNTRI3S.ZD2' (total 12)       |                     effect = u'CRN_TRI.ZD2' (total 11)
                    version = u'1.00' (total 4)               |                     version = u'1.20' (total 4)
                    installed = 1                                                   installed = 1
                    id = 305                                  |                     id = 304
                    unknown = 0                                                     unknown = 0
                    group = 6                                                       group = 6
...
                Container:                                                      Container:
                    effect = u'LPROLL3S.ZD2' (total 12)       |                     effect = u'LOOPROLL.ZD2' (total 12)
                    version = u'1.00' (total 4)               |                     version = u'1.10' (total 4)
                    installed = 0                             |                     installed = 1
                    id = 65                                   |                     id = 64
                    unknown = 0                                                     unknown = 0
                    group = 7                                                       group = 7

diff.txt

Closing this out as this seems to be confirm. Specifically ZOOM have blacklisted some effects on the G1Four/G1Four-X/B1Four/B1Four-X.


Add versioning strategy for JSON Schema

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/39
Originally created by @tayloramurphy on 2021-05-10 16:09:26

We'll need to come up with some version strategy for the JSON schema and for the YAML files we're generating.
Within the YAML we could follow a similar pattern we have with discovery.yml, i.e. version: 1. The problem with this is if we increment the version we'd have to do it for a ton of files.
For the JSON schema we could use the id field I think. We'd still have to update the ref for each YAML file.
@DouweM @aaronsteers I have less experience with versioning something like this - thoughts on how we should go about it?

View 7 previous comments from the original issue on GitLab


bernburg area

Subscribe to all media in a geographic area
api.create_subscription(object='geography', lat=52.137044, lng=11.670066, radius=2000, aspect='media', callback_url='http://mydomain.com/hook/instagram')

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!


How do I enable the .jaeger-compact.server-host-port 6831 for the collector in streaming strategy

Hi, is there any example to enable port 6831 in the streaming strategy? And is it possible to expose the collector service through "NodePort" as I'd like to have some external applications send trace to the collector? Thanks.
In general, I feel the documentation for configuring the operator is inadequate. Thanks.

The port 6831 is a UDP port that is opened by the agent, not by the collector. The agent is typically deployed as either a sidecar to your pod, or as a daemonset. In general, flags can be set via the Options properties:
spec:
  collector:
    options:
      some.flag.for.the.collector.binary: value

Besides this specific part, what could we improve in the official documentation? We would love your PRs (here's the source: https://github.com/jaegertracing/documentation/blob/master/content/docs/next-release/operator.md)

Feel free to reopen if you still have questions.

@jpkrohling , thanks for the clarification. So I will deploy a jaeger agent to accept trace at its port 6831. The second question in the post is about exposing the port of the collector of streaming strategy.
I am asking because the application and agent are deployed out of the k8s cluster, so I need to expose port 14250 of the collector so that it can accept traces from the agent. Is there a way to configure this? Thanks.

The typical way of achieving this is by creating an Ingress resource that will expose the -collector service.

My understanding is that Ingress operates above layer 4. So will that work with the exposing port 14250 for the traffic between agent and collecot?
BTW, could you please take a look at another question at https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/jaeger-tracing/rNU1xKVZuGI/2yX1RfT1AgAJ
Thanks.

So will that work with the exposing port 14250 for the traffic between agent and collecot?

It should. But open another issue if you see a concrete problem.


Usage of nanoTime

The recommended usage to compare two instances of nanoTime is t1 - t0 < 0 instead of t1 < t0.  It has been suggested there may be instances where we are not using the recommended approach and may be vulnerable to overflow.  Usage of nanoTime needs broader investigation to determine if this is an issue.

@normanmaurer - FYI.

@Scottmitch thanks will take care!

@Scottmitch ahh that's a very good point. I have just this week been bitten by this and wondered why I got negative deltas :-)

@buchgr You can also have negative deltas because you're measuring start on one core, and end on another one (typically in a callback running in another thread) and cores are not always sync'ed. This is more likely to happen under load.

I think we fixed everything here. Lets close this


Auto split bullet list into separate cards

Hi, on top of my previous suggestion, have you considered splitting the list into cards, i.e. as many cards as bullets in the list? That would be a huge time saver :)

Sorry for replying so late.
Do you mean using the command to split the list into cards or by the drag action?

@cycsd automatically :) that could work for the lists initially (ordered or unordered, or maby paragraphs to start with :)

It could either be here

a checkbox, or an option of the plugin in settings to "split into separate cards" by paragraphs/list
I would go for simple result, i.e name the cards as number 1, 2, 3.. depending on how many paragraphs/bullets are  included

I have the same requirement as you. Cuz sometimes I collect many cards from other software like Zotero in markdown format, and then I'd like to break this md file into cards to re-organize them on a whiteboad.

@macin I know another plugin called "cardify" can realize what you want, but it just doesn't work on my ob and I couldn't figure out why. Maybe you can try throug this link https://github.com/joshuakto/obsidian-cardify or directly serach "cardify" in your ob.

@tociyou1995 sorry for the delayed reply. I tested out cardify and it only works on files. I'd prefer to work on the canvas instead of creating files for every piece of note. So I'd have to wait for @cycsd solution on this :)

Hi @cycsd, just wanted to check how things are going. Did you have a chance to look into this one?


feat: Integrate ocp-base with SLZ

Description
Please include a summary of the change and which issue is fixed. Also, please include the motivation and context for the change. List any dependencies that are required for this change.
Issue : https://github.com/terraform-ibm-modules/terraform-ibm-landing-zone/issues/324
Check the relevant boxes:

[ ] Bug fix (nonbreaking change that fixes an issue)
[x] New feature (nonbreaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI related update (pipeline, etc.)

Checklist

[ ] If relevant, a test for the change has been added or updated as part of this PR.
[x] If relevant, documentation for the change has been added or updated as part of this PR.

Merge

Merge using "Squash and merge".
Make sure to use a relevant conventional commit message that is based on the PR contents. The commit message determines whether a new version of the modules needs to be released, and if so, which semver number to use).

@Aashiq-J Why was upgrade test skipped? I think we need to be able to support the upgrade case to ensure current consumers clusters do not get destroyed. I imagine we will need to use "moved" blocks for the cluster resources changing here

Hi @ocofaigh , Initially there was many changes including many required variables and variable name changes, so had to skip upgrade test. But, later most of the changes were pushed to later phases and changed variable names were reverted.
Is there a way to run the upgrade test now? I tried creating a different branch and pull request but it took the all the commit messages from this branch which caused it to skip the upgrade test in the new pull request as well.

@Aashiq-J I actually created an internal issue to allow the ability to re-enable the upgrade test, but its not done yet. I haven't tried this, but maybe you can try squashing all the commits into one commit (that does not have skip upgrade test in it). something like this (not tested): git reset $(git merge-base main $(git rev-parse --abbrev-ref HEAD))

@ocofaigh
I ran the upgrade test and it will destroy the cluster and recreate it.

Hi @ocofaigh ,
I also tried to use the moved block, but moved block can only be used when we have a A single static variable reference is required. If we dynamically try to create the cluster name for the moved block it gives the following error.
│ Error: Invalid expression
│
│   on ../../moved.tf line 7, in moved:
│    7:   from = ibm_container_vpc_cluster.cluster["${var.prefix}-management-cluster"]
│
│ A single static variable reference is required: only attribute access and indexing with constant keys. No calculations, function calls, template expressions, etc are allowed here.

Only when we statically mention the cluster name in the moved block we get 0 destroy.

I deployed it and doesn't look like we have access to the master nodes -

ACLs

@vburckhardt

There is a related issues in this area https://github.com/terraform-ibm-modules/terraform-ibm-landing-zone/issues/393

On the backward compatibility aspect - it is possible to move forward to this version without destroying the cluster by doing as follow. This would be easily scriptable:
1. terraform state list | grep "ibm_container_vpc_cluster.cluster"
module.landing_zone.ibm_container_vpc_cluster.cluster["debug-ocp-management-cluster"]
module.landing_zone.ibm_container_vpc_cluster.cluster["debug-ocp-workload-cluster"]
2. Take the cluster name from the list - eg:
debug-ocp-management-cluster
debug-ocp-workload-cluster
3. For each element in the list:
terraform state mv module.landing_zone.ibm_container_vpc_cluster.cluster["<cluster name>"] module.landing_zone.module.cluster["<cluster name"].ibm_container_vpc_cluster.cluster[0]

I think we should discuss / brainstorm about the best way to package this logic for the consumer. Technical consumer can run the script directly during upgrade - but this won't work for users using the logic from IBM Cloud Schematics.
Ideally this would come as a pre-apply run (terragrunt has got that concept). Perhaps using a data external block to run the script but this may have deep side effects to evaluate further as changing the state during plan evaluation.

FYI, IBM schematics has a terraform state mv command too -> https://cloud.ibm.com/docs/schematics?topic=schematics-schematics-cli-reference#schematics-wks_statemv
ibmcloud schematics workspace state mv --id WORKSPACE_ID --source SOURCE  --destination DESTINATION

@ocofaigh - Right, most terraform cli commands are exposed through ibmcloud cli schematics. This issue here is not the commands themselves, but how do we get non-tech savvy people to run those automatically - eg: there is no equivalent of terragrunt hooks in schematics afaik.

Hi @vburckhardt,
the pod_subnet and service_subnets are optional inputs for the iks cluster creation.

/run pipeline

/run pipeline

@SirSpidey, @vburckhardt
Please review this documentation change for preventing re-creation of cluster during the major version change.
https://github.com/terraform-ibm-modules/terraform-ibm-landing-zone/pull/253/commits/26a41db0bd7bcd06c9b5347e78c4d33f5ccf6335

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

@Aashiq-J I created a PR #487 for the new readme that merges into this pr. If the content makes sense, merge it and then we can continue with this PR.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

If we get this working end to end, it would be a great thing to show on next weeks playback or derp dive


Opti sync: elaborate on why sync optimistically

This PR provides more details on the rationale to have optimistic sync as a generalized sync solution for consensus engines.
cc @tersec @paulhauner

Seems to reasonably articulate the general consensus on this point, as a descriptive matter.


Null check operator used on a null value

It looks like in the js_handle.dart we're casting a null object to non-null. Which leads to the error below.
Unhandled exception:
Null check operator used on a null value
#0      ElementHandle.$.<anonymous closure>
package:puppeteer/…/page/js_handle.dart:574
#1      _RootZone.runUnary (dart:async/zone.dart:1612:54)
#2      _FutureListener.handleValue (dart:async/future_impl.dart:152:18)
#3      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:704:45)
#4      Future._propagateToListeners (dart:async/future_impl.dart:733:32)
#5      Future._completeWithValue (dart:async/future_impl.dart:539:5)
#6      _completeOnAsyncReturn (dart:async-patch/async_patch.dart:254:13)
#7      ElementHandle.$OrNull (package:puppeteer/src/page/js_handle.dart)
package:puppeteer/…/page/js_handle.dart:1
<asynchronous suspension>

Hi, thanks for the report.
Do you have a small example to reproduce this error?

Yes! Here you go.
This is running on dart 2.12.1
void main() async {
  var browser = await puppeteer.launch();
  var myPage = await browser.newPage();

  await myPage.goto('https://dart.dev', wait: Until.networkIdle);

  // If you try to select for an element that isn't available it should return null
  // however the method returns a non-null response.
  await myPage.$('.dash-element-that-isnt-there');

  await browser.close();
}

As a temporary fix I wrapped the selector in a try/catch.

ok, thanks.
The API has changed a bit with null-safety, there is a method $OrNull for cases where the element is not there.
  Future<ElementHandle> $(String selector);
  Future<ElementHandle?> $OrNull(String selector);

We can return to the old API if I have feedback that this is hard to use.

That makes perfect sense. I'm going to open a PR to add a comment that clarifies that distinction. I wouldn't have known to use that other call.


RuntimeError: Can't add row, already generated!

Here is my code:
def self.generate_files
    Region.all.each do |region|
      doc = XlsxWriter.new
      doc.quiet_booleans!
      sheet1 = doc.add_sheet("#{region.name}")
      sheet1.add_row([
        "Here is the title that needed"
      ])
      region.rows.each do |r|
          sheet1.add_row([
            "",
            r.self_number,
            r.direction,
            r.service,
            394,
            118
          ])
      end
      require 'fileutils'
      FileUtils.mv doc.path, "/home/artyom/detail/#{region.name}.xlsx"

      doc.cleanup
   end
end

And I get:
RuntimeError: Can't add row, already generated!

Sorry, my mistake


command not found: foundation-apps

I am not able to run the foundation-apps via command line. Ran the following with no luck.

npm install -g foundation-cli bower gulp
gem install bundler
tried to create project on desktop via foundation-apps new myApp
Lost some CLI love so gave the Yeti Launch a go and got the following error on startup:
Uncaught Exception:
Error: The resource could not be loaded because the App Transport Security policy requires the use of a secure connection.: (null) at Error (native)

I was able to run the project through the yeti launcher despite the error but on my system can not run foundation-apps from CLI. Any advice would be great.
OS X El Capitan 10.11
Node Version 4.2.2
Npm Version 3.4.1
Screen shots:

Same issue on Windows 7 x64

Everything installed (and accessible via CMD) on system as follows:
<EMAIL_ADDRESS>bundler 1.10.6
sass 3.4.19
node v0.10.36
npm 1.4.28
ruby 2.1.7p400

Same in windows x64, installed all dependencies, foundation upgraded and followed instructions to letter in readme, makes software useless as can't even start a project!
[tom@server ~]$ foundation-apps new myapp
bash: foundation-apps: command not found
[tom@server ~]$

@dredgerman @LittleHendrix  @jessepinuelas  Try including an alias for  ' foundation-apps ' to the bash(CLI) config ,Do a simple google search on how to !

@dredgerman @LittleHendrix @jessepinuelas
If you're on a mac, do this
cd /usr/local/bin
ln -s /usr/local/lib/node_modules/foundation-cli/bin/cli.js foundation-apps

Update the paths so they're appropriate for your system

Hi,
Thanks,
After tearing my hair out for a number of days am moving to meteor instead,
Appreciate the help,
On 26 November 2015 at 02:08, jimmycliff obonyo<EMAIL_ADDRESS>wrote:

@dredgerman https://github.com/dredgerman @LittleHendrix
https://github.com/LittleHendrix @jessepinuelas
https://github.com/jessepinuelas Try including an alias for '
foundation-apps ' to the bash(CLI) config ,Do a simple google search on how
to !
—
Reply to this email directly or view it on GitHub
https://github.com/zurb/foundation-apps/issues/718#issuecomment-159781867
.

Hi guys,
I think you just need to type `foundation new'  then you'll choose the one you need App, Site or Mail.
![foundation-cli]
(https://dl.dropboxusercontent.com/u/25503484/foundationApp.png)

@obonyojimmy @etcook creating the alias to foundation-apps did the trick.

Ok, I get the menu. But arrow keys don't allow to select an option, they work on the underlying command line instead (bring up previously entered commands)
This is on Windows 10

@ansorg you need to build you project before you see any changes on your app , did you do gulp in CLI ?

I face the same problem like ansorg's one where I cannot select the second option "A web app" with arrow keys. It is on Windows 8. Do you guys have any tip for me?

Same arrow key problem here, windows 10.  not a good first impression....

I am having the same silly problem, I have the foundation cli installed globally, but when I try to run the command foundation-apps new, to create a new project, the command line tells me the command cannot be found, this is on Windows 10 too

the command changed to foundation not foundation-app

Just installed node 6.11 and did a npm install --global foundation-cli````. The foundation``` command is not found.


PyPi update?

Hi,
I noticed that this repo's version is at 2.0 in this file. But when I check PyPi  I see it's stuck to 1.3.3.
When do you guys plan on updating the PyPi repo?
Thanks,

For now I'll use pip install git+https://github.com/tomturner/django-tenants but I would prefer to have it available in PyPi as soon as possible.

Massive +1

I have been testing it before I release it. I will try and upload to pypi within a few days

Done


Skip auth parameters for sub-client

Resolves https://github.com/microsoft/typespec/issues/5439

API change check
API changes are not detected in this pull request.


Sentry Performance page is empty

Platform
Flutter Mobile
Obfuscation
Enabled
Debug Info
Enabled
Doctor
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
• Android SDK at C:\Users\Ahmed\AppData\Local\Android\Sdk
• Platform android-34, build-tools 34.0.0
• ANDROID_HOME = C:\Users\Ahmed\AppData\Local\Android\Sdk
• Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
• Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[X] Visual Studio - develop Windows apps
X Visual Studio not installed; this is necessary to develop Windows apps.
Download at https://visualstudio.microsoft.com/downloads/.
Please install the "Desktop development with C++" workload, including all of its default components
[√] Android Studio (version 2022.3)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• android-studio-dir = C:\Program Files\Android\Android Studio
• Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)
[√] Connected device (4 available)
• sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 14 (API 34) (emulator)
• Windows (desktop)            • windows       • windows-x64    • Microsoft Windows [Version 10.0.19045.3570]
• Chrome (web)                 • chrome        • web-javascript • Google Chrome 119.0.6045.124
• Edge (web)                   • edge          • web-javascript • Microsoft Edge 119.0.2151.58
[√] Network resources
• All expected network resources are available.
Version
7.10.1
Steps to Reproduce
Hi, I'm trying to set up the Sentry performance, so I've gone through the performance documentation, and as far as I understood, all the features are enabled by default, including, for example, App Start Instrumentation, Slow and Frozen Frames, all I need to do is to set the sample rate like this options.tracesSampleRate = 1.0; and add SentryNavigationObserver to the list of observers. Am I missing something? Because the performance page is empty for me. Thanks.
Expected Result
See some data on the performance page on my Sentry dashboard.
Actual Result
The performance page is empty.
Are you willing to submit a PR?
No

Hi, it could be that it would be dropped because the transaction has no span. You could try out creating some spans manually and go from there.

Also please see my comment here

Feel free to comment if this still an issue


Make in-memory caching opt-in for unsafe block numbers

Problem
We are caching on two levels in the EDR: in memory and on disk.  The on disk cache is handled by the RPC client and then RemoteBlockchain and CachedRemoteState handle in-memory caching on top of it.
We are going to prevent on-disk caching of requests for unsafe block numbers (blocks that can be reorged) in https://github.com/NomicFoundation/rethnet/issues/88, but question remains what do about the in-memory cache. We currently log a warning in RemoteBlockchain for unsafe blocks, but not in CachedRemoteState.
Expected behavior
After discussion with @Wodann  we came to the conclusion that the safest option here is to make in-memory caching opt in for unsafe block numbers and disable by default.
Alternatively we could just disable caching for unsafe block numbers without the opt in as a quickfix. Disabling caching for unsafe block numbers matches Hardhat Network's behavior.
Definition of Done
To be determined whether we want opt in in the first version.

Disabling caching for unsafe block numbers matches Hardhat Network's behavior.

Does this work like this for the in-memory cache, @fvictorio ?

Yes:
https://github.com/NomicFoundation/hardhat/blob/858cee43ff88ba624343a001817fada7864a49ec/packages/hardhat-core/src/internal/hardhat-network/jsonrpc/client.ts#L502-L514
And we do that for both the in-memory and on-disk caches. There is no option to enable it caching for potentially unsafe blocks.

(I'm not saying that this is the best possible behavior; in fact, I'm almost sure it's not. But this is the way Hardhat seems to work right now.)

For now we'll always disable caching for unsafe blocks. If a user requests it, we can add an override


Add support for HTML comment syntax

<EMAIL_ADDRESS>HTML comment syntax is not part of ES5 but is an optional part of ES6:

http://www.ecma-international.org/ecma-262/6.0/#sec-html-like-comments

Huh, I didn't even know this was a thing.  You have this tagged realworld - Is there really JS code in the wild that uses HTML comments?

Yes, this came up in NetSurf testing.

Old advice was to encapsulate everything in
If you want a real world example, there are a couple of scripts on http://www.terrestrialtv.uk/dtt.cgi which do exactly this.

Hm.  Although if the JS parser treats  as a normal comment (instead of ignoring it), wouldn't that then turn the entire block of code into a comment?

@fatcerberus The semantics are a bit interesting:
<!-- starts a line comment so it doesn't extend to the --> end marker (!)

Ah.  That makes more sense then.

Is this bug ever going to be fixed? It has been almost a year and it seems silly to suggest that 1/4 of inline javascript can't even be parsed because of some practice from 20 years ago.

@andwhyisit I'm not sure what you mean by a bug - but HTML syntax is not a feature of ES5 or core ES6, but rather part of ES6 Annex B of optional, legacy features. Duktape is not a full ES6 engine (yet) so several ES6 features are not yet implemented.
In any case, HTML comment syntax is now supported in master. Closing this FIXME pull as unmerged.


Add methods for raising if there are incorrect field mappings

Run LuckyRecord::Model.ensure_correct_field_mappings!
Raises if:

There is a field defined in the model doesn't have a matching column on the table
The table is missing
The field is marked as nilable with ? but the column on the table is NON NULL
A field is marked as non-nilable (no ?), but the column allows NULL

Ideally it should use https://crystal-lang.org/api/0.23.1/Levenshtein.html to check for similar named tables/field and suggest corrections. You can see how this is used for missing assets in Lucky: https://github.com/luckyframework/lucky/blob/e6c5efc578b9753816985db4fc44fae589b7573f/src/run_macros/missing_asset.cr#L10
It should also give helpful suggestions when it raises:
The field is marked as nilable (nickname : String?), but the database column does not allow nils.

Try this...

  * Mark the field as non-nilable in your model: field nickname : String
  * Or, change the column in a migration to allow nils: make_optional :users, :nickname

It does not need to find and display all problems at once. So if there is a missing table and a model has a missing field, just display the error for a missing table. Then once the table is added and they run it again show the error for a missing field.

It might be better to split these into separate PRs. e.g. one for checking for table names. One for checking if a field exists, and another for checking whether a field allow null or not

@mikeeus I'm not sure if you have time, but if you're looking for something to work on for Lucky, this would be really awesome. Of course up to you though :)

Sure, this looks really interesting! I'll see what I can do


Attempting to open a .DSK that is read-only results in an error

I'm using: CiderPress II v1.0.0
If I right-click the .DSK and make it read-only then I get this error:
---------------------------
File error
---------------------------
Error: Access to the path 'C:\Users\tomch\Downloads\mb-audit.dsk' is denied.
---------------------------
OK
---------------------------

Unchecking the read-only file property allows me to open the file.

Looks like I'm trapping the wrong exception (should be UnauthorizedAccessException rather than IOException) in the "retry as read-only" code. Easy to fix.

Fix is available in https://github.com/fadden/CiderPress2/releases/tag/v1.0.1-dev1

Available in v1.0.1.


add Red Hat Satellite product

Hello community,
this PR adds release dates for the Red Hat Satellite product. It is used to enable patch and lifecycle management for bigger environments such as a datacenter or cloud setups. I added release information from https://access.redhat.com/articles/1365633 and https://access.redhat.com/support/policy/updates/satellite/.
I hope you'll find this useful. Thanks for the great project!

Hi @stdevel, thanks for this contribution but Red Hat Satellite is already documented on https://endoflife.date/redhat-satellite. Maybe 6.0 to 6.8 releases can be moved to this page ?

Hi @marcwrobel,
you're so right - how could I miss that? 👀
I adjusted the PR to add the older versions - what do you think?


Revert "move js modules -> components + gulpfile [ci skip]"

Reverts divio/django-cms#4505
unforeseen ckeditor issue

LGTM


Wrong SQL generated because of positions of parameters within LINQ

Hello everyone. While working with LINQ I noticed something weird. I am using simple example to report the bug that is happening when trying to compare dates in Where statement.
var limit = DateTime.Now;
            try
            {
                var session = Store.OpenSession();
                var users = await session.Query<User>()
                    .Where(x => limit <= x.Birthday)
                    .ToListAsync();

                var sql = session.Query<User>()
                    .Where(x => limit <= x.Birthday)
                    .ToCommand(FetchType.FetchMany);
                Console.WriteLine(sql.CommandText);

                var sql2 = session.Query<User>()
                    .Where(x => x.Birthday >= limit)
                    .ToCommand(FetchType.FetchMany);
                Console.WriteLine(sql2.CommandText);

                session.Dispose();
            }
            catch
            {
                throw new Exception("Error while working with the db");
            }

The SQL generated from the LINQ is different in the two cases even though it should be the same.
My comparison is basically A >= B which is the same as B <= A.
The first SQL which is wrong is: select d.data, d.id, d.mt_version from public.mt_doc_user as d where public.mt_immutable_timestamp(d.data ->> 'Birthday') <= :arg0
The second SQL which is correct is: select d.data, d.id, d.mt_version from public.mt_doc_user as d where public.mt_immutable_timestamp(d.data ->> 'Birthday') >= :arg0

@StefanGeorgievski Could you confirm this is still an issue with the latest master as I think this was addressed in #869
If so could you close, thx

@jimgolfgti I had older version of Marten and updated it to Marten 2.2.0 but the update did not solve the problem. The queries were still wrong.

PR was merged 12 days ago, 2.2 shipped about 22 days ago.
Fix will be out in 2.3

Okay than, I am closing this one.
Thanks.


fix documentation bug: max length of Group Short Name should be 12

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-libraries-for-java

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

fix for issue: https://github.com/Azure/azure-rest-api-specs/issues/2840

Initial swagger had a wrong max field cap size and was wrong anyway. This PR does not introduce a breaking change.


Return Electrode - x location

Simple problem with how the x location of the return electrode is found - the code looks for cells between 90 and 110% of the x location, but this is mesh dependent.
I used a simple but inelegant fix, I'm sure there's other ways:
xtmp=np.cumsum(mesh.hx)
hxtmp=mesh.hx[np.where(np.absolute(xtmp-src_b[0])==np.absolute(xtmp-src_b[0]).min()]
surface_electrodex=(
    (mesh.gridFz[:,0] > src_b[0] - (0.5*hxtmp)) &
    (mesh.gridFz[:,0] < src_b[0] + hxtmp))
)

Thanks @gnieuwenhuis! I recently noticed this as well, what I am thinking to do is actually specify a src_b_closest on the source that just uses the closestPoints util in discretize. I have a first-pass locally. I will put it on a pr shortly!

Actually, I have pushed this change, see what you think: https://github.com/lheagy/casingResearch/blob/master/casingSimulations/sources.py#L52
(P.S. if you are starting to use this repo a bit - I will try to follow better development practices in terms of using pull requests instead of just pushing to master :) )

Looks good - also having the separate class allowing line sources without coupling to the casing is also handy for playing around with the models.
(I wouldn't worry too much about better development practice here - I'm using the files as a sandbox and basically have my own (very messy) local branch which I'm playing around with to get certain models run ... not very conducive to good development practice anyways!)


Fprats new golden metrics

Relevant information
changes to enable goldenmetrics, goldentags and some modifications to summary metrics of the following entities:
GCPSPANNERINSTANCE
GCPSTORAGEBUCKET
GCPAPPENGINESERVICE
GCPBIGQUERYDATASET
GCPBIGQUERYTABLE
GCPCLOUDFUNCTION
GCPCLOUDSQL
GCPHTTPLOADBALANCER
GCPINTERNALLOADBALANCER
Checklist

[ ] I've read the guidelines and understand the acceptance criteria.
[ ] The value of the attribute marked as identifier will be unique and valid.
[ ] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.fprats seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Could not determine artifacts error

Hello,
Today I tried to sync your library source via gradle and failed. It showed this error.
Could not determine artifacts for com.github.indexOS.media-for-mobile:android:master-SNAPSHOT:43a9026f09-1: Skipped due to earlier error

I am using gradle version 4.10.1. I didn't watch this error yesterday. Is there something wrong with jitpack server?

I checked it more deeply and found that it is causing socket timeout error

Caused by: java.net.SocketTimeoutException: Read timed out
	at java.net.SocketInputStream.socketRead0(Native Method)
	at java.net.SocketInputStream.socketRead(SocketInputStream.java:116)
	at java.net.SocketInputStream.read(SocketInputStream.java:171)
	at java.net.SocketInputStream.read(SocketInputStream.java:141)

I could fix this error with an workaround patch by extending socket timeout.
org.gradle.jvmargs= -Dorg.gradle.internal.http.connectionTimeout=120000 -Dorg.gradle.internal.http.socketTimeout=120000