Ouroboros incompatible with custom Rc-like type? (hybrid_rc crate)

The error is pretty inscrutable, so I'm not sure how to solve it, but it looks like ouroboros is incompatible with hybrid_rc?
error[E0599]: no function or associated item named `is_std_rc_type` found for struct `ouroboros::macro_help::CheckIfTypeIsStd<hybrid_rc::HybridRc<Inner, hybrid_rc::state::Local>>` in the current scope
  --> file.rs:48:1
   |
48 | #[self_referencing]
   | ^^^^^^^^^^^^^^^^^^^ function or associated item not found in `CheckIfTypeIsStd<HybridRc<Inner, Local>>`
   |
   = note: the function or associated item was found for
           - `ouroboros::macro_help::CheckIfTypeIsStd<std::rc::Rc<T>>`
   = note: this error originates in the attribute macro `self_referencing` (in Nightly builds, run with -Z macro-backtrace for more info)

This probably happened if you aliased Rc to be something other than std::rc::Rc. You can manually put a #[covariant] or #[not_covariant] annotation above the field to specify the missing information. If that doesn't work, you may need to add a type alias or use the original type name for the field so that ouroboros doesn't assume it's std::rc::Rc. (I forget exactly what triggers the special logic for Rcs)
Unfortunately since this error happens when the code is type checked and is not detectable when the macro is run, I cannot put in a better error message.

Nope, Rc isn't aliased to anything. I already have covariance specified on another field so the macro fails with TODO: Nice error, covariance specified twice. I have the type fully specified as hybrid_rc::Rc<Inner> within the wrapped struct.

That error only shows up if you specify covariance on the same field twice

I'm guessing it's erroneously thinking that hybrid_rc::Rc is std's Rc
type (it likely ignores the path and just looks at the type name.
Unfortunately macros have no access to semantic information like which
exact type a path refers to; instead it works entirely on raw syntax) and
concludes that it's covariant. It then sees your annotation and thinks the
covariance has been specified twice. The solution to this would be to make
a type alias like type HybridRc<T> = hybrid_rc::Rc<T>; and use that so
the type name doesn't overlap with the type in std.

On Wed, May 1, 2024 at 7:57 PM Vitali Lovich @.***>
wrote:

Nope, Rc isn't aliased to anything. I already have covariance specified on
another field so the macro fails with TODO: Nice error, covariance
specified twice if I also add either annotation you mentioned to this
one. I have the type fully specified as hybrid_rc::Rc within the
wrapped struct.
—
Reply to this email directly, view it on GitHub
https://github.com/someguynamedjosh/ouroboros/issues/113#issuecomment-2089357121,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOL2YRN7F4F5HSWH5TJJPSTZAGFP3AVCNFSM6AAAAABHCSBCK6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBZGM2TOMJSGE
.
You are receiving this because you commented.Message ID:
@.***>

That error only shows up if you specify covariance on the same field twice - I'm guessing it's erroneously thinking that hybrid_rc::Rc is std's Rc type (it likely ignores the path and just looks at the type name. Unfortunately macros have no access to semantic information like which exact type a path refers to; instead it works entirely on raw syntax) and concludes that it's covariant. It then sees your annotation and thinks the covariance has been specified twice. The solution to this would be to make a type alias like type HybridRc<T> = hybrid_rc::Rc<T>; and use that so the type name doesn't overlap with the type in std.


MAINT: Update for new copy kwarg

Fix for https://app.circleci.com/pipelines/github/mne-tools/mne-python/21813/workflows/d8396d6c-3548-4ea5-919c-35835d7fcbdf/jobs/61343 / epochs.get_data(..., copy=None) deprecation

Will mark as merge-when green and cut a bugfix release so that this can make it into the 1.6 installer


Jira/osst 1106

This PR
a) makes sure that what we get from daysAgoCreated is handled correctly
b) updates the README for clarity and with info about the config.js
Questions for @kevindigo :

There may be better instructions for how to export the Github token?
I believe config.json and exampleData.json have to be in a folder called examples for the paths to work (so I added that to the instructions) - is that correct?
A config.json question: What happens if there are no labels specified? What if they want all open issues, regardless of label? Is that possible?

Changes made. Let me know what you think.

Nice stuff! Does the path have to be "example"? Or could that be /docs/ or /.github/ or /config/mariner? Shooting from the hip -- I don't know anything more on this than that it's nice to not litter the root folder of all repos that consume a package. (Yes, looking at prettier/babel/all of them, really). The less things beginning with "." the happier are your users, on all platforms.

Nice stuff! Does the path have to be "example"?
@sesam : Thanks for asking! We realize it's pretty messy right now. Our plan is to merge and publish it this way. Then we will update the sample app (https://github.com/indeedeng/Mariner-Issue-Collector) to use this new package. After that, we'll come back and remove the entire step-by-step section from this README, instead pointing people to that app.

Basically, an app that uses oss-mariner can store the input and output files in any directory it wants, including outside of the project tree. And a config file won't even be required, as an app could hard-code it instead of having it in a file. This library doesn't care. It's just hard to explain all of that in what we're hoping is a very concise "getting started" section.


health check return 401 when MELTANO_AUTHENTICATION is set to true

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2783
Originally created by @ollivierj on 2021-06-04 16:17:12

What is the current bug behavior?
What is happening now?
health check (/api/v1/health) return 401 when MELTANO_AUTHENTICATION is set to true.
What is the expected correct behavior?
What should be happening?
it should return a 200 without being authentificated.
Steps to reproduce
How one can reproduce the issue?
set MELTANO_AUTHENTICATION to true, run meltano ui and then check the response at http://host:port/api/v1/health
Relevant logs and/or screenshots
Please use code blocks (```) to format console output
i will add it later.
Possible fixes
If you can, link to the line of code that might be responsible for the problem or suggest a fix
Further regression test
Ensure we automatically catch similar issues in the future

[ ] Write additional adequate test cases and submit test results
[ ] Test results should be reviewed by a person from the team

View 6 previous comments from the original issue on GitLab


json load

Thanks a lot for your great work!
I have a question  about the json load.
Since the dict in python has no order, when I try to load a dataset, e.g, UCF101, using the ucf101_01.json generated from the train/testlist01.txt, the order of the dataset is not the same as the traininglist01.txt, it just shuffle it.
Do you think it matters?
Thanks for your answer!

I think it is not any problem.
Because the order of training data is shuffled in data_loader,
even you use the ordered format.


[GSSOC'23] Create a login page

I want to update the Login page under GSSOC'23. Can you please assign me this project?

I want to work on this issue as a contributor.
Pleasure Assign so that I can show my skills under GSSoC'23.

actually one is already working on it so kindly wait for the response. If they are not able to solve this issue then I will assign it to you.


chore(CI): add CircleCI config

Installs, tests, builds and releases (semantic-release) if necessary.
Linting is commented out at the moment (too many failures). It should be uncommented later when the code base is compliant.
Addresses #1.
PS @mojoaxel: I can't set this (or any other repo) up on CircleCI. It says: “You must be an admin to add a project on CircleCI”.

I can't set this (or any other repo) up on CircleCI. It says: “You must be an admin to add a project on CircleCI”.

 Please be very responsible! (One time a contributor deleted the whole almende/vis repository from github, by mistake.)

Thanks, I'll be very careful.

:tada: This PR is included in version 5.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Style in the creating-a-team.md page

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/organizations/organizing-members-into-teams/creating-a-team#
What part(s) of the article would you like to see updated?
Switch ", to ," the way English works.
Additional information
I am PRing a fix.

@TheGiraffe3 Thank you for opening an issue and linking it to your PR! ✨


[RFE] Allow to remove default capabilities/routes

Current situation
We allow to create and even replace routes through plugins, but we don't allow to remove them.
Impact
There may be vendors that do not want to offer a certain route or entry in the sidebar.
Ideal future situation
We have a way to remove a route and entry in the sidebar. Maybe also other features, like default header actions, sections, etc.
Implementation options
One way would be to allow "deregistering" routes/sections/etc. But this is TBD. Suggestions are welcome.

I have been thinking about how to accomplish this and would like to share a quick RFC for my ideas.
Generally instead of having a "registerSidebarEntryEraser" or "removeRoute", I was thinking we should offer the ability to set up filters for sidebar entries + routes. A filter as a name and a function, the function will receive the item in question and return either null or the item (modified or not).
E.g.
   registerSidebarEntryFilter(filterName: string, filterfunc: null | ((entry: SidebarEntry) => SidebarEntry | null));
   registerRouteFilter(filterName: string, filterfunc: null | ((entry: Route) => Route | null));

This allows us to e.g. modify what a route's path is, or what a sidebar name is, without having to override them with the regular register function (thought that functionality remains, this just makes it simpler).
If the filter function return null, then those routes or sidebar entries will not be considered anymore when rendering. i.e.
// Remove namespaces sidebar
registry.registerSidebarEntryFilter('no-namespaces', (entry) => entry.name === 'namespaces' ? null : entry)
registry.registerRouteFilter('no-namespaces', (route) => route.path === '/namespaces' ? null : route)

The filter function can be null so we effectively remove the filter (which may be needed).
// Remove namespaces route filter (leave the sidebar one though)
registry.registerRouteFilter('no-namespaces', null)

Of course, returning just the argument in the filter means also not filtering it at all, but I like the idea of actually dropping the filter from memory...
I wonder what we should do for things like e.g. removing the cluster selector button, or the theme change button, if we need that too. Should we have that as a filter function for other parts of the UI/functionality, or should we have a settings object and we allow plugins to manipulate that for turning off (or on) parts of the UI.
Comments are welcome.

Took me a little while to grok what filter did here.
What happens when multiple plugins are doing their business?

Plugin A, adds a filter.
Plugin B, adds a filter.

plugin B will consume the routes that have been already filtered by plugin A. Otherwise I don't think the functionality would make much sense.
This gives a lot of power to plugins, but arguably just slightly more than what they can already do (since they can override routes already).


Current ortho fix

This PR fixes an issue with the current_ortho function which causes the alternating_update interface to break on more generic trees (current tests appeared to only check more standard cases like the comb tree so missed the bug). It also changes a test in test_dmrg to use a tree for which the code was previously breaking.
The issue appeared to be in the overlapping_vertex clause when at the end of a sweep which was assuming that the last vertex in the sweep was a neighbor of the previous vertex --- which may not always be true for the 'default_region_plan'. Removing the clause fixes the issue as the else clause always works anyway.
@mtfishman I feel like the "current_ortho" should either i) be a trait carried around by the state (by having, e.g., an orthogonal_tensornetwork type that wraps an ITensorNetwork and a v_orthocentre) or ii) something that the alternating update interface keeps track of as a variable and just re-assigns it each time a la
  psi = orthogonalize(psi, v, v_orthocentre)
  v_orthocentre = v

where orthogonalize can utilize the idea that if psi is a tree it just finds the shortest path between v and v_centre and does a sequence of QRs along that path for efficiency. If psi is not a tree it could run BP to put it in the Vidal gauge and then absorb square root lambdas everywhere except at v where it absorbs the neighboring lambda (making the BP environment identity at that site).

@mtfishman once you merge https://github.com/ITensor/ITensorNetworks.jl/pull/207 I will update the formatting to be in line with main. I won't until then just to keep this PR easier to read.

@JoeyT1994 regarding the formatting, see https://github.com/ITensor/ITensors.jl/pull/1536#issuecomment-2418350000.

@mtfishman I feel like the "current_ortho" should either i) be a trait carried around by the state (by having, e.g., an orthogonal_tensornetwork type that wraps an ITensorNetwork and a v_orthocentre) or ii) something that the alternating update interface keeps track of as a variable and just re-assigns it each time a la

The TTN type is designed like i): https://github.com/ITensor/ITensorNetworks.jl/blob/v0.11.21/src/treetensornetworks/treetensornetwork.jl#L11-L12, do you mean you also want that for non-tree networks?

Ah I see okay I will leave the formatting as is then and avoid v2.0 of JuliaFormatter

@mtfishman I feel like the "current_ortho" should either i) be a trait carried around by the state (by having, e.g., an orthogonal_tensornetwork type that wraps an ITensorNetwork and a v_orthocentre) or ii) something that the alternating update interface keeps track of as a variable and just re-assigns it each time a la

The TTN type is designed like i): https://github.com/ITensor/ITensorNetworks.jl/blob/v0.11.21/src/treetensornetworks/treetensornetwork.jl#L11-L12, do you mean you also want that for non-tree networks?

I see I didn't know the TTN type had that. I guess I was just thinking more generally, we could have an OrthogonalITensornetwork <: ITensorNetwork type instead which carries a v_centre around (and if it's a tree it means the environment for that region is exactly identity, and if it's loopy it means the environment on that site/region is, under the BP approximation, identity). That would be in-line with the VidalITensorNetwork type we already have and the ITensorNetwork type which is just in an arbitrary gauge.
If the TTN type carries the centre-site information what is the purpose of the current_ortho function here? Can't it just be pulled from the state the alternating_update routine is working with?

If the TTN type carries the centre-site information what is the purpose of the current_ortho function here? Can't it just be pulled from the state the alternating_update routine is working with instead of the sweep plan?

That's a good question, I don't remember why it was designed that way. It looks like current_ortho was introduced by @b-kloss in #143, @b-kloss do you remember the logic behind that design vs. just using the orthogonality center information stored in the TTN struct?

@JoeyT1994 if it seems easier to just use the orthogonality center stored in the TTN, feel free to try to refactor it in that way.

If the TTN type carries the centre-site information what is the purpose of the current_ortho function here? Can't it just be pulled from the state the alternating_update routine is working with instead of the sweep plan?

That's a good question, I don't remember why it was designed that way. It looks like current_ortho was introduced by @b-kloss in #143, @b-kloss do you remember the logic behind that design vs. just using the orthogonality center information stored in the TTN struct?

The function  current_ortho always returns a single vertex, while the orthogonality center of a TTN is a (contiguous on the graph) set of vertices.  They are not equivalent, and the TTN by itself does not provide enough information to determine what the return of current_ortho would be inside alternating_update.
For nsite=1, i.e. single-site alternating update, current_ortho will return the orthogonality center.
For nsite=2, i.e. two-site alternating update, current_ortho will return the site that overlaps with the previous 2-site region in the sweep plan (which is the one which won't overlap with next 2-site region). For example, the default_inserter moves the orthogonality center to the vertex that's not current_ortho in the 2-site region. The logic inside current_ortho is convoluted because it becomes more complicated than what I described on general trees.
This logic may not generalize to nsite>2, and the whole mechanism can certainly be improved.

Thanks for the explanation @b-kloss !
So in the code current_ortho always returns a single vertex ortho_centre which is not actually the current orthogonality centre but is what you want the orthogonality centre to be? Then that current_ortho is used in a call to orthogonalize(ttn, ortho_centre) via the default_extracter?
Why does current_ortho need to be a single vertex anyway? Could we not just keep track of the ortho_region as the region on which current_ortho was last called (and thus it can be obtained as ortho_region(ttn)) and then during the extractor stage of the region_update we call
ttn = orthogonalize(ttn, region)

to orthogonalize onto the new region?
I'll give a refactor a go with this plan in mind as it should be possible to simplify the code along these lines, unless I'm missing something.

@mtfishman @b-kloss I have done a refactor of the code now. The changes done are as follows:

I refactored orthogonalize so that the functionality is almost entirely written in abstractitensornetwork.jl and is wrapped around a base function with takes a vector of edges and performs a sequence of QR decompositions.
The orthogonalize functionality in abstracttreetensornetwork.jl  is now just a function orthogonalize(ttn::AbstractTreeTensorNetwork, region::Vector) which works out the sequence of edges needed to perform orthogonalize on to move the orthogonality region of a ttn from where it currently is to region. Crucially, this is done via namedgraph_astar and so should be O(1) time complexity assuming nsites is finite and the region plan always involves a continuous path through the graph (versus edge_path before which I think is O(L) time complexity with L the size of the network). The path finding logic here could probably be improved further to reduce some overheads but I wrote up a rudimentary one-line version for now.
region_update now no longer needs to work out any orthogonality information via ortho_centre, it can just be pulled from the ttn directly when needed and orthogonalize(ttn, region) can just be called in the extracter. The ortho_centre function is now no longer needed and the bug this PR aimed to fix is gone.

I think this simplifies the logic of things quite a bit and should, in principle, actually speed the code up (because it was O(L^{2}) before due to use of edge_path in orthogonalize

@JoeyT1994 this definitely looks like a nice simplification of the code, though I'm having trouble picturing how the old code and new code are working diagrammatically. Let's meet to discuss just so I make sure I understand what's going on and can be convinced that the new code and old code are doing the same thing.
This could help with something I was trying in the tensor network impurity solver. I was trying to automate finding the tree structure but for tree structures I was generating I was hitting a bug in the solver code, hopefully it was this bug that you're fixing here!

Thanks @mtfishman . I incorporated some of your comments and yes let's meet to discuss how the logic works here.
One slight flag here (which was also in the previous version) is that the edge_sequence to move the ortho_region is computed via an O(L) time complexity function with L the number of tensors in the network.
Hence this actually makes the alternating_update sweeping on trees have an O(L^{2}) time complexity term (the sequence is computed every region_update). Although this is likely much smaller than the O(L Poly(chi)) scaling it is not really optimal and we should be able to write a more sophisticated function to move the ortho_region that is O(1) if the current ortho_region and new ortho_region are not extensive and are not separated by an extensive path.

Looks good, thanks!


UserFunction(abc.ABC) -> UserFunction(abc.ABC, Callable)

Issue #, if available:
NA
Description of changes:
On top of the context-in-OuterLoop::get_next_points() PR, this implements the Callable mixin for UserFunction. It means you can do user_function(x) in addition to user_function.evaluate(x)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Can you fix the commit history here, please? It includes changes from another PR

git rebase -i e514476fdf5e39f322ae6a6c7730102e6865f529~
Change bad commits to drop from pick
git push --force
^ in case anyone else needs the git recipe

I should have mentioned earlier, but can you please add a unit test to cover this?

Unrelated, but thoughts on adding box/flaky to pytest?

Emukit's test runs are normally consistent, so not yet. But yes, sometimes we do run into transient failures, so flaky might come handy. I don't know what's box.

Ah, box as in github.com/box/flaky. And, unit test ahoy.

Codecov Report

Merging #298 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #298      +/-   ##
==========================================
+ Coverage   89.77%   89.79%   +0.01%
==========================================
  Files         119      119
  Lines        3844     3851       +7
  Branches      429      429
==========================================
+ Hits         3451     3458       +7
  Misses        312      312
  Partials       81       81

Impacted Files
Coverage Δ

emukit/core/loop/user_function.py
98.73% <100.00%> (+0.03%)
:arrow_up:

emukit/core/loop/user_function_result.py
100.00% <100.00%> (ø)

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 b550b96...60ca55c. Read the comment docs.

Looks good to me! We will need either @MashaNaslidnyk or @mmahsereci to merge this though, so let's wait for them to chime in.


Add notes on pre/post-generate hooks

Added in a reference to pre/post-generate documentation to the info.md page.
Closes #91

Great, thanks. LGTM.


Implement cluster heatlh computation projection

Hey folks! This is the improvement for the cluster health computation. It is based on projections in the discovery side and the checks results event. Hopefully it makes the work.
I have written some E2E tests and all goes green (they are not uploaded yet).
Some more details:
The partial healths fields is a map which stores all the partial healths of a resource, cluster in our case (but easy extendable for other resources like sap systems, hosts, etc). So the partial health is updated in the different events (now, when a new cluster discovery is done and the cluster checks results are executed). The 2 healths are stored, and the overall health computed.
I have a doubt with the new table though (health_states). I thinking if we should truncate the values of this table on web service boot up. This would have 2 benefits:

If for some reason we need the change the partial healths labels (hana_sr_health and config_checks by now), we would get rid off them, and we wouldn't have zombie entries.
We wouldn't have "old" health states. Maybe it is safe to restore the health to nothing on boot up, to not show old values, as it might be misleading.

LGTM as well. Generally speaking I'd strive to find a clearer naming for things like HANASrHealth4 smile but it is good enough here.
Not sure also about truncating health on boot, but let's discuss on that.

Update at: https://github.com/trento-project/trento/pull/817/commits/1f668a8354918764584f667af76598114a8e1662


Future must be_>(0).await error: await not a member of Match[Int]

Trying to await on a future does not seem to work (or, I'm doing it wrong). Example:
//			val list = Await.result(db.run(q), Duration.Inf).toSeq
//			list.length > 0 must beTrue

			db.run(q).map(_.length) must be_>(0).await

The commented lines work fine. The uncommented line results in the following error:
[error] /Users/zbeckman/Projects/Glimpulse/Server/project/glimpulse-server/test/models/TestPersonRelationDataModel.scala:34: value await is not a member of org.specs2.matcher.Matcher[Int]
[error] 			db.run(q).map(_.length) must be_>(0).await

However, db.run() very definitely returns a Future. This, for example, works fine:
			val x: Future[Int] = db.run(q).map(_.length)
			val y: Int = Await.result(x, Duration.Inf)
			y > 0 must beTrue

@zbeckman This doesn't seem related to ScalaCheck?

Oops. Should have created on the specs2 repo... sorry.


Notebook output elements should be able to honor prefers-color-scheme

Notebook output elements are commonly HTML/CSS/JS and should be able to listen to the VS Code host theme and the CSS prefers-color-scheme media query.
E.g.,

Install the .NET Interactive extension.
Ctrl+Shift+P, ".NET Interactive: Create new blank notebook".
Change the first cell's language to "HTML (.NET Interactive)".
Set the contents to this:

This is regular HTML

This is theme-aware HTML

Execute that cell.

RESULT:
The output cell always applies the dark theme, e.g., a red background, regardless of the theme set in the parent VS Code.
EXPECTED:
Switching the host VS Code between light and dark themes should also trigger the appropriate change in the notebook output webview so the theme information can be honored.
From a dark theme:

From a light theme:

@mjbvz any idea what extra we need to do to support prefers-color-scheme  in the notebook webview?


Document parachain runtime upgrades

See original issue in the cumulus repo: https://github.com/paritytech/cumulus/issues/764 made by @apopiak.
We should find the right place to put docs on how to upgrade parachains. This serves as a placeholder for now.
Preparation
Build compact.compressed.wasm runtime.
Parachain Sudo
If the parachain in question has sudo, you can just send the extrinsic:
sudo.sudoUncheckedWeight(system.setCode(code))
Relay Chain Sudo
If it doesn't but the relay chain has sudo:

Get the Blake-2_256 hash (you may also toggle “hash a file” in the Apps UI and the Blake2_256 code_hash will be computed for you)
Construct parachainSystem.authorizeUpgrade(hash) call on the parachain (e.g. Statemine) and grab the encoded call
Dispatch XCM call from Relay Chain using sudo.sudoUncheckedWeight(palletXcm.sendXcm(Transact(encodedAuthorizeUpgradeCall)))
After dispatch, submit the compact.compressed.wasm file using the unsigned transaction parachainSystem.enactAuthorizedUpgrade. Anyone can submit this extrinsic.

Relay Governance
Same as relay chain sudo, but dispatch the XCM call via governance instead.

Closing this should include srtool per https://github.com/substrate-developer-hub/substrate-docs/issues/479 (as a stretch goal)

Should update:
https://github.com/substrate-developer-hub/substrate-docs/tree/main/v3/how-to-guides/07-parachains/d-runtime-upgrades
and reference in:
https://github.com/substrate-developer-hub/substrate-docs/tree/main/v3/tutorials/09-cumulus/c-polkadot-launch


LEGO: Pull request from lego/hb_839_20170517204137952 to master

LEGO: Pull request from lego/hb_839_20170517204137952 to master with localized lcls

@huguesv If you get a chance, this needs to be pulled and merged locally so we can commit.

Going to resubmit myself


feat: app content

Description

Dispatch redux action on app init
Fetch featured blog content via ContentService
Display featured content via FeaturedContentList

On what platform have you tested the change?

[ ] Android - Emulator
[ ] Android - Physical device
[x] iOS - Emulator
[ ] iOS - Physical device

Screenshots
Please attach screenshots showing the change if relevant
Checklist

[ ] I have updated mockServer
[ ] I've added analytics as relevent
[x] I have run npm test
[x] I have run npm run test:i18n

Out of scope and potential follow-up
Is there any related changes that you plan to do in a follow-up PR or voluntarily excluded from the scope?

@snack-0verflow Are the changes to es.json purely formatting?

@snack-0verflow Are the changes to es.json purely formatting?

@snack-0verflow Are the changes to es.json purely formatting?

Yes

@snack-0verflow Are the changes to es.json purely formatting?

Yes


collect docs from correspondent github repos

we can make to deploy website from ci. it could collect all docs from correspondent github repos instead of dublicating documentation both on repo and website. it should simplify support docs in actual state.
see bem.info for example

Great suggestion, just have to find the time here :)


Renamed emit_particle to emit_subparticle shader function

This name is better to reflect its nature and not confuse users - they need to create sub-emitters to make this function actually works. This was discussed with @reduz a few months ago, and he said it makes sense.

Thanks!

Thanks!


[Business Approvals Kit - Feature]: Notify user option for each node

Is your feature request related to a problem? Please describe.
Currently, there is no option to notify specific users for each node in the Business Approvals Kit. This can be frustrating for users who want to keep certain individuals informed about the approval process but do not want them to be actual approvers.
Describe the solution you'd like
Would like to have an option to notify specific users for each node in the Business Approvals Kit, separate from the actual approvers. This would allow users to keep certain individuals informed about the approval process without requiring them to be actual approvers. The option to set notification users should be similar to the option to set approvers.
Describe alternatives you've considered
No response
Additional context?
No response
AB#695

Suggested approach to consider:
here is a suggested approach to complete the tasks:

Modify the current Power Automate cloud flow to include a step that retrieves the necessary information about the event that occurred.
Add a step to the flow that checks the user's notification preferences in the Dataverse tables. This step should retrieve the user's preferred notification method and the necessary parameters for that method.
Based on the user's notification preferences, add a step to the flow that sends the appropriate notification using one of the following methods: email, Teams message, push notification, or SMS message.
If the user's notification preferences are not set up in the Dataverse tables, add a step to the flow that sends a default notification using one of the above methods.
Update the Power App to allow the user to specify and save changes to their notification preferences in the Dataverse tables.


Package BiblioPixelAnimations so that it is installed with BP.

See https://groups.google.com/forum/#!topic/bibliopixel-dev/HFNUthTAvNk

@rec - Since I'm going to keep BP2 around for a little bit I need to resolve that with BPA.
Well, actually... ok... a couple options:
A) BP2 just dies completely. BP3 or GTFO. BPA is completely updated to match.
B) BP3 gets a new name as does BPA.
C) BP2 becomes BP3 but I package bibliopixel2 on PyPi for those that want to install the old stuff.
IDK... point being I know some people will complain and moan but option A is the easiest for us. Thoughts?

A) is very plausible.  If we never break anything, we can't move forward?
But - isn't BPA completely compatible with 2 and 3?
On 3 March 2017 at 04:13, Adam Haile<EMAIL_ADDRESS>wrote:

@rec https://github.com/rec - Since I'm going to keep BP2 around for a
little bit I need to resolve that with BPA.
Well, actually... ok... a couple options:
A) BP2 just dies completely. BP3 or GTFO. BPA is completely updated to
match.
B) BP3 gets a new name as does BPA.
C) BP2 becomes BP3 but I package bibliopixel2 on PyPi for those that want
to install the old stuff.
IDK... point being I know some people will complain and moan but option A
is the easiest for us. Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-283853655,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsn7szp48b1bxwPPAmT2YBhoU36Qgks5rh4VWgaJpZM4MQBD0
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000

Not really... some imports got moved and last I checked some of the
animations are broken. Honestly, pulling in BPA and running them against
BP3 would be a fantastic test suite to run in Travis.
Here's my current thoughts...

setup.py is updated to actively reject Python 2 entirely. With a message
that says "If you need python 2 support please run 'pip install bibliopixel
< 3.x'" or something like that. That way we don't need to maintain a diff
repo for old or new BP.
BPA goes BP3 only.

On Fri, Mar 3, 2017 at 3:14 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

A) is very plausible. If we never break anything, we can't move forward?
But - isn't BPA completely compatible with 2 and 3?
On 3 March 2017 at 04:13, Adam Haile<EMAIL_ADDRESS>wrote:

@rec https://github.com/rec - Since I'm going to keep BP2 around for a
little bit I need to resolve that with BPA.
Well, actually... ok... a couple options:
A) BP2 just dies completely. BP3 or GTFO. BPA is completely updated to
match.
B) BP3 gets a new name as does BPA.
C) BP2 becomes BP3 but I package bibliopixel2 on PyPi for those that want
to install the old stuff.
IDK... point being I know some people will complain and moan but option A
is the easiest for us. Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-
283853655>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsn7szp48b1bxwPPAmT2YBhoU36Qgks5rh4VWgaJpZM4MQBD0>
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-283892838,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6rmHp_xAUJDfVxZrAbcZt7MrCA8vks5rh8vTgaJpZM4MQBD0
.

Very convincing solution.  Also, not huge qtties of work.  Laten we het
doen!
On 3 March 2017 at 14:06, Adam Haile<EMAIL_ADDRESS>wrote:

Not really... some imports got moved and last I checked some of the
animations are broken. Honestly, pulling in BPA and running them against
BP3 would be a fantastic test suite to run in Travis.
Here's my current thoughts...

setup.py is updated to actively reject Python 2 entirely. With a message
that says "If you need python 2 support please run 'pip install bibliopixel
< 3.x'" or something like that. That way we don't need to maintain a diff
repo for old or new BP.
BPA goes BP3 only.

On Fri, Mar 3, 2017 at 3:14 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

A) is very plausible. If we never break anything, we can't move forward?
But - isn't BPA completely compatible with 2 and 3?
On 3 March 2017 at 04:13, Adam Haile<EMAIL_ADDRESS>wrote:

@rec https://github.com/rec - Since I'm going to keep BP2 around
for a
little bit I need to resolve that with BPA.
Well, actually... ok... a couple options:
A) BP2 just dies completely. BP3 or GTFO. BPA is completely updated to
match.
B) BP3 gets a new name as does BPA.
C) BP2 becomes BP3 but I package bibliopixel2 on PyPi for those that
want
to install the old stuff.
IDK... point being I know some people will complain and moan but
option A
is the easiest for us. Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-
283853655>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsn7szp48b1bxwPPAmT2YBhoU36Qgks5rh4VWgaJpZM4MQBD0>
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-
283892838>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA6a6rmHp_
xAUJDfVxZrAbcZt7MrCA8vks5rh8vTgaJpZM4MQBD0>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-283948124,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdskg8A4WzvRPlbEVhSHe07wFqkhnQks5riBBEgaJpZM4MQBD0
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000

Awesome. Should be an easy change. Will do.
On Fri, Mar 3, 2017 at 9:25 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

Very convincing solution. Also, not huge qtties of work. Laten we het
doen!
On 3 March 2017 at 14:06, Adam Haile<EMAIL_ADDRESS>wrote:

Not really... some imports got moved and last I checked some of the
animations are broken. Honestly, pulling in BPA and running them against
BP3 would be a fantastic test suite to run in Travis.
Here's my current thoughts...

setup.py is updated to actively reject Python 2 entirely. With a
message
that says "If you need python 2 support please run 'pip install
bibliopixel
< 3.x'" or something like that. That way we don't need to maintain a diff
repo for old or new BP.
BPA goes BP3 only.

On Fri, Mar 3, 2017 at 3:14 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

A) is very plausible. If we never break anything, we can't move
forward?
But - isn't BPA completely compatible with 2 and 3?
On 3 March 2017 at 04:13, Adam Haile<EMAIL_ADDRESS>wrote:

@rec https://github.com/rec - Since I'm going to keep BP2 around
for a
little bit I need to resolve that with BPA.
Well, actually... ok... a couple options:
A) BP2 just dies completely. BP3 or GTFO. BPA is completely updated
to
match.
B) BP3 gets a new name as does BPA.
C) BP2 becomes BP3 but I package bibliopixel2 on PyPi for those that
want
to install the old stuff.
IDK... point being I know some people will complain and moan but
option A
is the easiest for us. Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/
issues/121#issuecomment-
283853655>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsn7szp48b1bxwPPAmT2YBhoU36Qgks5rh4VWgaJpZM4MQBD0>
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-
283892838>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA6a6rmHp_
xAUJDfVxZrAbcZt7MrCA8vks5rh8vTgaJpZM4MQBD0>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-
283948124>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdskg8A4WzvRPlbEVhSHe07wFqkhnQks5riBBEgaJpZM4MQBD0>
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-283966178,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6k97Nxd3JQZcwN2ZWIG_dlFpdyt9ks5riCLtgaJpZM4MQBD0
.

 I'm gonna solve it with Travis :P
I'll just have travis auto push a new version to pypi automatically on merges to master. Shouldn't be too bad. But then we can just put BPA in "install_requires"

Ah, p00p. I was hoping to do that also with SimpleWebSocketServer.
If setup.py won't let you link a raw github repo, then that won't work.
 I'm
gonna solve it with Travis :P
I'll just have travis auto push a new version to pypi automatically on
merges to master. Shouldn't be too bad. But then we can just put BPA in
"install_requires"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-284107670,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsoMZYxiSJrmVUnU6gAdu_A0RMEuIks5riK_NgaJpZM4MQBD0
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000

Nope. It's theoretically possible to do, but I'm just having a lot of
trouble with it. I think partly because BPA doesn't really have a version
number. And let's worry about the websocket thing AFTER 3.0 is out.
On Fri, Mar 3, 2017 at 7:59 PM, Tom Swirly<EMAIL_ADDRESS>wrote:

Ah, p00p. I was hoping to do that also with SimpleWebSocketServer.
If setup.py won't let you link a raw github repo, then that won't work.
 I'm
gonna solve it with Travis :P
I'll just have travis auto push a new version to pypi automatically on
merges to master. Shouldn't be too bad. But then we can just put BPA in
"install_requires"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-
284107670>,
or mute the thread
<https://github.com/notifications/unsubscribe-
auth/AAPdsoMZYxiSJrmVUnU6gAdu_A0RMEuIks5riK_NgaJpZM4MQBD0>
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-284111657,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6sDX3k2tNjILF-wv397iFIe-cDLHks5riLdrgaJpZM4MQBD0
.

Note that it isn't even filed as a bug.  :-D
On 4 March 2017 at 02:17, Adam Haile<EMAIL_ADDRESS>wrote:

Nope. It's theoretically possible to do, but I'm just having a lot of
trouble with it. I think partly because BPA doesn't really have a version
number. And let's worry about the websocket thing AFTER 3.0 is out.
On Fri, Mar 3, 2017 at 7:59 PM, Tom Swirly<EMAIL_ADDRESS>wrote:

Ah, p00p. I was hoping to do that also with SimpleWebSocketServer.
If setup.py won't let you link a raw github repo, then that won't work.
 I'm
gonna solve it with Travis :P
I'll just have travis auto push a new version to pypi automatically on
merges to master. Shouldn't be too bad. But then we can just put BPA in
"install_requires"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-
284107670>,
or mute the thread
<https://github.com/notifications/unsubscribe-
auth/AAPdsoMZYxiSJrmVUnU6gAdu_A0RMEuIks5riK_NgaJpZM4MQBD0>
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-
284111657>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA6a6sDX3k2tNjILF-
wv397iFIe-cDLHks5riLdrgaJpZM4MQBD0>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-284113686,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsuSivc6UOjunWwvAlEL4Q1dl--Olks5riLuggaJpZM4MQBD0
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000

lol... :)
BPA is now packaged on PyPi and should only be uploaded there via travis. Anything that makes it to master will be pushed and the version is basically just v3.<year_month_day>.
I know, I know... that's not an ideal versioning scheme but it's better than the old one which was just the commit hash :P I don't ever want to have to manage BPA versions as I never intend to do proper releases. However, that means that all updates should now always be done on another branch and the code reviewed into master.
I'll update the rest of the BP stuff now to use this.

That should be fine.  If anyone complains, they can fix it.  :-D
On 4 March 2017 at 03:08, Adam Haile<EMAIL_ADDRESS>wrote:

lol... :)
BPA is now packaged on PyPi and should only be uploaded there via travis.
Anything that makes it to master will be pushed and the version is
basically just v3.<year_month_day>.
I know, I know... that's not an ideal versioning scheme but it's better
than the old one which was just the commit hash :P I don't ever want to
have to manage BPA versions as I never intend to do proper releases.
However, that means that all updates should now always be done on another
branch and the code reviewed into master.
I'll update the rest of the BP stuff now to use this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/121#issuecomment-284118557,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdssr-O5OHHQLNJ2ueR6rHm1GH26S4ks5riMelgaJpZM4MQBD0
.

--
/t
http://radio.swirly.com - art music radio 24/7 366/1000

Complete


Is it possible to publish the assets?

My Nginx config automatically caches and optimises assets, such as livewire/livewire.js?id=e2b302e9.
This means a route which fakes the assets being available there, such as the one pulse uses, returns a 404.
In other projects I've been able to run php artisan livewire:publish to work around this, but in Pulse this does nothing. Would this be an option going forward?

This is a dynamic route that is registered by Livewire and serves the livewire.js script. You may want to update your Nginx script to ignore this script from being cached / optimized.

Thanks @jbrooksuk

There is guide here: https://benjamincrozat.com/livewire-js-404-not-found

Thanks @jbrooksuk and @sinnbeck  - it's set by my hosting company so I was looking for a workaround to save having to go to them to start changing things. Thanks anyway.

@sunscreem Just ran the publish method and it does not seem to work in livewire 3. So maybe ask on the livewire repo or make a pr to add it back?

Opened a PR to address this moving forward: https://github.com/laravel/pulse/pull/266


Day441

Resolves #896.

Pull Request Test Coverage Report for Build 1671

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 1669:
0.0%

Covered Lines:
5

Relevant Lines:
5

💛  - Coveralls


zero env config fails in acl module.

Without PRIVATE or PUBLIC env the connection will be undefined and fail to look for the first array entry.
const connection = process.env.PRIVATE?.split('|') || process.env.PUBLIC?.split('|')

const acl_table = connection[1]?.split('.').pop()

Hello, I tried to fix the issue
This is what I did:
Fix the issue where undefined connection causes errors when accessing connection[1]. Implement early return if connection is undefined.
I also created pull request: https://github.com/GEOLYTIX/xyz/pull/1688

[!CAUTION]
Disclaimer: This fix was created by Latta AI and you should never merge before you check the correctness of generated code!
The fix provided by Latta AI might not be complete and it can serve as an inspiration.

This bug was fixed for free by Latta AI - https://latta.ai/ourmission
If you no longer want Latta AI to attempt fixing issues on your repository, you can block this account.


Add output of  output_hidden_states for onnx model export.

Feature request
Currently, the merged decoder in  ORTModelForCausalLM only output a List of 25 OrtNDArrays, 1 for the output logits + 24 from past keys and values.
Could you add the output of hidden_states? It doesn't look like much work; only need to set the key output_hidden_states=True in GPT2LMHeadModel. But this is a very important feature, which enables the SOTA contrastive search: https://huggingface.co/blog/introducing-csearch
Thanks!
Motivation
This is a very important feature, which enables the SOTA contrastive search: https://huggingface.co/blog/introducing-csearch.
Your contribution
I don't know how this works yet. But it doesn't seems hard.
In the existing contrastive search implementation, this output of hidden_states is simply enabled by setting a key output_hidden_states=True in
/transformers/generation/utils.py L1834
Relavant past PRs https://github.com/huggingface/optimum/pull/647/files, https://github.com/huggingface/optimum/pull/930/files.

Thank you, I will add an option for this!


Fix Typo in InVesselBuild and Incorporate CAD-to-DAGMC in Testing

Fixes a typo in the export_cad_to_dagmc method of InVesselBuild and incorporates that method in test_invessel_build.py.
Closes #87.

Thanks @connoramoreno


Text color scheme

Feature request
Implementation of color scheme in cmder and/or other windows shells.
Purpose
My feature request is just cosmetic, infact, I would like to ask if it is possbile to implement color schemes to make it more eye friendly and fun to read or check. I use the cmder shell. Thanks.

https://github.com/JustArchi/ArchiSteamFarm/wiki/Logging


Adding validation to help page on title and description

Fix for the help page when title and description are not provided.

merging and closing PR


Stop duplicate samples being created within a single sample summary

Motivation and Context

Stops a sample unit being created twice within a sample summary
What has changed

How to test?

Links

/deploy baileyw

/deploy baileyw

/deploy baileyw

/deploy baileyw


(IOS)can not disconnect devices when i run ios sample

in this case,i can not scan this devices,and can not reconnect this devices.

I will solve this problem for the time being.
if([@"connect" isEqualToString:call.method]) {
FlutterStandardTypedData *data = [call arguments];
ProtosConnectRequest *request = [[ProtosConnectRequest alloc] initWithData:[data data] error:nil];
NSString *remoteId = [request remoteId];
@try {
CBPeripheral *peripheral = [self findPeripheral:remoteId];
//CBPeripheral *peripheral = [_scannedPeripherals objectForKey:remoteId];
NSLog(@"connect findPeripheral peripheral ");
if(peripheral == nil) {
NSLog(@"connect _scannedPeripherals peripheral ");
peripheral = [_scannedPeripherals objectForKey:remoteId];
}
if(peripheral == nil) {
@throw [FlutterError errorWithCode:@"connect"
message:@"Peripheral not found"
details:nil];
}
// TODO: Implement Connect options (#36)
self.peripheral = peripheral;
[_centralManager connectPeripheral:self.peripheral options:nil];
result(nil);
} @catch(FlutterError *e) {
result(e);
}


KeyError: 'sdk_moniker'  while fetching parquet from ADLS Gen2 using Padas API via abfs protocol

adlfs:
2023.8.0:
Linux:
3.9:

Describe the bug
I am reading parquet from ADLS Gen2 via Pandas API. Everything was working fine till this morning. Today, when I am running the code to fetch a parquet from ADLS, I am getting below error
2023-09-12 21:42:51           df = pd.read_parquet(f"abfs://{container_name}@{adls_account_name}.dfs.core.windows.net/{get_ds_prefix()}/{schema_name}/{table_name}",
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/site-packages/pandas/io/parquet.py", line 493, in read_parquet
2023-09-12 21:42:51           return impl.read(
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/site-packages/pandas/io/parquet.py", line 233, in read
2023-09-12 21:42:51           path_or_handle, handles, kwargs["filesystem"] = _get_path_or_handle(
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/site-packages/pandas/io/parquet.py", line 84, in _get_path_or_handle
2023-09-12 21:42:51           fs, path_or_handle = fsspec.core.url_to_fs(
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/site-packages/fsspec/core.py", line 377, in url_to_fs
2023-09-12 21:42:51           chain = _un_chain(url, kwargs)
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/site-packages/fsspec/core.py", line 325, in _un_chain
2023-09-12 21:42:51           cls = get_filesystem_class(protocol)
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/site-packages/fsspec/registry.py", line 234, in get_filesystem_class
2023-09-12 21:42:51           register_implementation(protocol, _import_class(bit["class"]))
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/site-packages/fsspec/registry.py", line 269, in _import_class
2023-09-12 21:42:51           mod = importlib.import_module(mod)
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/importlib/init.py", line 127, in import_module
2023-09-12 21:42:51           return _bootstrap._gcd_import(name[level:], package, level)
2023-09-12 21:42:51         File "", line 1030, in _gcd_import
2023-09-12 21:42:51         File "", line 1007, in _find_and_load
2023-09-12 21:42:51         File "", line 986, in _find_and_load_unlocked
2023-09-12 21:42:51         File "", line 680, in _load_unlocked
2023-09-12 21:42:51         File "", line 850, in exec_module
2023-09-12 21:42:51         File "", line 228, in _call_with_frames_removed
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/site-packages/adlfs/init.py", line 3, in
2023-09-12 21:42:51           from .spec import AzureBlobFile, AzureBlobFileSystem
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/site-packages/adlfs/spec.py", line 115, in
2023-09-12 21:42:51           class AzureBlobFileSystem(AsyncFileSystem):
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/site-packages/adlfs/spec.py", line 213, in AzureBlobFileSystem
2023-09-12 21:42:51           blocksize: int = create_configuration(storage_sdk="blob").max_block_size,
2023-09-12 21:42:51         File "/usr/local/lib/python3.9/site-packages/azure/storage/blob/_shared/base_client.py", line 415, in create_configuration
2023-09-12 21:42:51           config.user_agent_policy = UserAgentPolicy(sdk_moniker=kwargs.pop('sdk_moniker'), **kwargs)
2023-09-12 21:42:51       KeyError: 'sdk_moniker'
I did some research on my side to see which package is causing this issue. I found adlfs package was updated on 8th August. and azure-storage-blob (this package is the prereq for adlfs) package was updated today (12th September). My guess was, that I am having updates from either of the two packages. So I explicitly specified the older version (azure-storage-file-datalake==12.12.0, adlfs==2023.4.0), but I am still having the same issue.
I looked at the PRs to get more context on the changes made to the package. and I found UserAgent was modified for the requests which are using client-side encryption.
To Reproduce
Steps to reproduce the behavior:

df = pd.read_parquet(f"abfs://{container_name}@{adls_account_name}.dfs.core.windows.net/{get_ds_prefix()}/{schema_name}/{table_name}", storage_options={'account_key': adls_account_key})

Expected behavior
It should work with the old code.
Additional context
The changes to the Package which is causing the issue were made at https://github.com/Azure/azure-sdk-for-python/pull/31333

I have added a temporary fix by using the old version of azure-storage-blob (azure-storage-blob==12.17.0). The current version is 12.18.0

I think I found the issue.
https://github.com/fsspec/adlfs/blob/4b704b1a5f39ec2350a8be44d9cb117aee9b1719/adlfs/spec.py#L222
In the above line, we are calling the create_configuration function, but we are not passing sdk_moniker. I checked the git blame, the line was written 3 years ago. So seems like we missed passing the sdk_moniker value here.
I also looked for other references for the create_configuration. and found one where it is called correctly.
https://github.com/Azure/azure-sdk-for-python/blob/bd006c2e68d2419776fa3997848f72e4263e4009/sdk/storage/azure-storage-blob/azure/storage/blob/_shared/base_client.py#L232 . In the constructor of the class which is calling this function we have set the value sdk_moniker and also it is being passed as a named parameter.
So, if in this line https://github.com/fsspec/adlfs/blob/4b704b1a5f39ec2350a8be44d9cb117aee9b1719/adlfs/spec.py#L222, we can pass the default value, we should be good. And for the default value, since we know, the service type is a blob, we can explicitly set the default value to  f"storage-blob/{VERSION}"

Is it possible to quickly yank 12.18.0 release of azure-storage-blob from PyPi?

Looks like this is a breaking change for users since, azure-storage-blob pre 12.18.0 looks like this:
config.user_agent_policy = UserAgentPolicy(
    sdk_moniker=(f"storage-{kwargs.pop('storage_skd')}/{VERSION}", **kwargs)
)

and, it now look like this:
config.user_agent_policy = UserAgentPolicy(sdk_moniker=kwargs.pop('sdk_moniker'), **kwargs)

Effectively, if you pass in sdk_moniker in a version before 12.18.0 you'll get both a dynamically generated skd_moniker, and the one you've specified, throwing an error.
If you don't pass one in for version 12.18.0, you'll get an error.
This means without doing some hula-hooping of "what version of azure-storage-blob is this?" conditionals you can't use create_configuration in a way that runs without error on version 12.18.0 and previous versions.
I don't know this project very well in terms of what the goal is for whether sdk_moniker should be a required keyword argument or not going forwards, but the fix (unless a breaking change is deliberate) seems to be to patch in "sdk_moniker" if it isn't in kwargs, and then just use "**kwargs".
Putting a PR in over the next few minutes.

I am also affected by this issue. Downgrading to 12.17.0 makes the issue disappear. I hope the patch will be released asap.

Hi all, thanks for the reports and the investigation. This was indeed an unintentional breaking change introduced in the latest release however, this method is intended to be private and not called by users. adlfs really should not be reaching into private package imports to grab and use this:
https://github.com/fsspec/adlfs/blob/4b704b1a5f39ec2350a8be44d9cb117aee9b1719/adlfs/spec.py#L26
However, given the adlfs package is very popular and this did unintentionally break them, I will work on getting a patch release out for this today/tomorrow. After which, I will open an issue to adlfs to address this. Thanks.

Update: Just created #32071 to fix the issue on the SDK side and prepare for patch releases. The patch release should be out later today/tomorrow. Thanks.

azure-storage-blob 12.18.1 was just released which should contain the fix for this. Closing this issue. Thanks again for everyone's help.


RFE: Allow defining namespaces in the configuration

When writing a large number of e.g. Selenium tests it becomes rather cumbersome to need to write the fully qualified name of each keyword. It would be great to be able to define a namespace selenium like selenium: org.getopentest.selenium. Then can one can simply call selenium.NavigateTo.

We're trying to be very cautious when introducing new features, because they need to be properly designed, to avoid having to change things later on, break backward compatibility and force people to make changes to fix their test suite. I thought about a feature like this too, but I'm afraid it might introduce problems with large codebases/teams, since it introduces one level of abstraction into the test.
One way to alleviate the problem of having to type boilerplate code is to use code snippets. All good text editors have this feature. Here's how that looks like for VS Code: link.

Alright. Java does not provide a way to assign aliases to classes either.
With good editor support reading and writing the full qualified names shouldn't be a problem.
It is better to do it like this and not like Robot Framework, where keywords can be shadowed depending on the order in which the keyword libraries are loaded.
Am 13. Februar 2020 18:38:48 MEZ schrieb Adrian Theodorescu<EMAIL_ADDRESS>
We're trying to be very cautious when introducing new features, because
they need to be properly designed, to avoid having to change things
later on, break backward compatibility and force people to make changes
to fix their test suite. I thought about a feature like this too, but
I'm afraid it might introduce problems with large codebases/teams,
since it introduces one level of abstraction into the test.
One way to alleviate the problem of having to type boilerplate code is
to use code snippets. All good text editors have this feature. Here's
how that looks like for VS Code:
link.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/mcdcorp/opentest/issues/270#issuecomment-585880495


[ISSUE #1435]Fix discarded mqadmin subcommand document description

What is the purpose of the change
Fix issue #1435
Brief changelog
modify docs/cn/operation.md
Verifying this change
check the doc https://github.com/apache/rocketmq/blob/master/docs/cn/operation.md
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.2%) to 49.474% when pulling 5850af4274799e5d434b2b58a011b0125fa8c59e on liuruiyiyang:fix_issue1435 into 74ffae6dc276f06e6c5710b73f3ea3ce5babd536 on apache:develop.

Plz, rebase the commit log.

@ShannonDing rebased.


[CARBONDATA-3193] Added support to compile carbon CDH spark distribution

Please use spark-2.2-cdh profile to compile cdh.
example:
mvn  -DskipTests -Pspark-2.2-cdh package

CDH uses twitter parquet instead of Apache Parquet so packages of the classes are different. Carbon uses parquet Dictionary interface in Spark 2.2 and below version vector integratiion with Spark. To make work for both Apache and CDH distribution , using reflection to invoke the respective packages.
Note that there is no changes required for Spark 2.3 and above as Spark has opened its own interfaces for Vector and Dictionary instead of depending on Parquet.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1947/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10200/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2156/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1948/

Does carbon not support Cdh using -Pspark-2.2 ?
Does CDH change the spark interface ,so that carbon can't run succesfully ?

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10201/

@qiuchenjian Please check the  PR description for why carbon need changes for Spark 2.2 CDH

@ravipesala  sorry,i didn't notice it, now i know the purpose

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2224/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2050/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10302/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2139/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2345/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2365/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2151/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10406/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/38/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/37/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12602/job/ApacheCarbonPRBuilder2.3/6028/


[Master Branch] Text Refactor broke TextInput Functionality

Is this a bug report?
Yes
Have you read the Contributing Guidelines?
Yes
Environment
Environment:
OS:  macOS Sierra 10.12.6
Node:  9.3.0
Yarn:  1.3.2
npm:  5.6.0
Watchman:  4.7.0
Xcode:  Xcode 9.0 Build version 9A235
Android Studio:  2.3 AI-162.3764568
Packages: (wanted => installed)
react: 16.2.0 => 16.2.0
react-native: github:facebook/react-native#master => 1000.0.0
Background
We are exploring the idea of supporting data detection for Text in addition to text input. In our situation, we have rich text that is supposed to detect data patterns in addition to different styling changes. This means we need data detection to work with nested Text components and for all of it to be inline.
Problem
First off, love the work @shergin did on this commit. Our text components are rendering ~30% faster!
Unfortunately, TextInput seems to have lost some functionality.

Clear button doesn't do anything while editing.
Data Detection does not work anymore.
When you delete all existing values via keyboard backspace, the text styling defaults to italics.

Expected Behavior
All mentioned issues above should be working as expected from previous versions of react native.
Reproducible Demo
This is broken for any project based on master.

Oh my, thank you for reporting this!
I will try to repro and fix it asap. I know it annoying but... if you can provide a simple Snack which illustrates the problem, it will speed up the fixing drastically.

Well, I could not repro all of this. So, @kpink224, could you please provide exact repro cases?

Hey @Shergin. So update on this issue:

The clear button not working was on our end in consequence to the file name changes. This is nothing you need to worry about.

When I said that styling defaults to italics, that was our style change. Unfortunately at the time, the style wasn't getting updated when the value was changing.

I made a project to reproduce everything today and didn't see the remaining issues anymore -- I could get data detection working again and the styling was changing as expected.

I suppose at this point, this issue is good to close! Thanks for you help and awesome improvement of Text

closing as per above comment. Let us know if this comes up again. 👍


[XLA:GPU] BatchNorm lowering overhaul for cudnn fast fp16 kernels and act+side input fusion

Problem Definition:

Currently, BatchNorm, when calling into cuDNN, forces the data layout to be NCHW (not by transforms but by squashing physical dimensions) irrespective of their original layout. This leads to calls into suboptimal BatchNorm kernels in cuDNN resulting in drastic performance regression for NHWC models. Note that latest cuDNN versions have capability of calling into fast semi-persistent NHWC BatchNorm fp16 kernels.
In addition, cuDNN allows fusion of BN+Act and BN+Add+Act (implemented in this PR) that improves BN Training performance (over BatchNorm expansion).  BN Backward fusion has not been implemented yet.
Note, whether or not XLA calls into fused cuDNN BN depends on whether Grappler remaps FusedBatchNormV3 to FusedBatchNormEx. CuDNN FusedBatchNormEx API (which allows fusion)  is only valid
a) NHWC datatype,
b) DT_HALF,
c) only ReLU activation,
d) Channel dim has to be a multiple of 4, and
e) BatchNormSpatial Persistent mode.

Implementation:
Calling into the fast semi-persistent  and fused kernels require an overhaul of the BatchNorm lowering since these require additional reserve space (transfer bits from the forward to the backward BatchNorm op) and work space (scratch space).
This PR modifies BatchNorm lowering from TF2XLA bridge to rewriter optimization to thunks in order to make sure that the performant kernels get called.

@AyanmoI  This PR is in draft, any update on this? Please. Thanks!

@AyanmoI  Can you please resolve conflicts? Thanks!

@AyanmoI Any update on this PR? Please. Thanks!

@AyanmoI Any update on this PR? Please. Thanks!

@AyanmoI  Any update on this PR? Please. Thanks!


Slow order by primary key with small limit on big data

Hello. I have such table:
CREATE TABLE logs.logs ( timestamp DateTime,  nsec UInt32,  `string_fields.names` Array(String),  `string_fields.values` Array(String),  `number_fields.names` Array(String),  `number_fields.values` Array(Float64),  `boolean_fields.names` Array(String),  `boolean_fields.values` Array(Float64),  `null_fields.names` Array(String),  date Date MATERIALIZED toDate(timestamp)) ENGINE = MergeTree(date, (timestamp, nsec), 32768)

It has 4,3 billion records:
:) select count() from logs;

SELECT count()
FROM logs

┌────count()─┐
│<PHONE_NUMBER> │
└────────────┘

1 rows in set. Elapsed: 1.565 sec. Processed 4.28 billion rows, 17.11 GB (2.73 billion rows/s., 10.93 GB/s.)

When I just select with small limit, everything works perfect:
:) select * from logs limit 5;

SELECT *
FROM logs
LIMIT 5

...

5 rows in set. Elapsed: 0.034 sec.

But it returns first five logs, but what if I want to get last five logs? I do this by ordering by primary key:

:) select * from logs order by timestamp desc, nsec desc limit 5;

SELECT *
FROM logs
ORDER BY
    timestamp DESC,
    nsec DESC
LIMIT 5

↙ Progress: 111.15 million rows, 58.55 GB (8.27 million rows/s., 4.36 GB/s.) ██▍                                                                                2%^← Progress: 111.97 million rows, 58.98 GB (8.27 million rows/s., 4.36 GB/s.) ██▍                                                                                2%Cancelling query.
Ok.
Query was cancelled.

0 rows in set. Elapsed: 13.642 sec. Processed 111.97 million rows, 58.98 GB (8.21 million rows/s., 4.32 GB/s.)

It starts processing all 4.3 billion rows, so it's slow.
Even if I use WHERE to reduce number of rows it's not so fast:
:) select count() from logs where timestamp > '2017-10-09 08:00:00';

SELECT count()
FROM logs
WHERE timestamp > '2017-10-09 08:00:00'

┌──count()─┐
│ 43025631 │
└──────────┘

1 rows in set. Elapsed: 0.036 sec. Processed 44.83 million rows, 179.31 MB (1.26 billion rows/s., 5.04 GB/s.)

:) select * from logs where timestamp > '2017-10-09 08:00:00' order by timestamp desc, nsec desc limit 5;

SELECT *
FROM logs
WHERE timestamp > '2017-10-09 08:00:00'
ORDER BY
    timestamp DESC,
    nsec DESC
LIMIT 5

...

5 rows in set. Elapsed: 5.054 sec. Processed 44.83 million rows, 23.60 GB (8.87 million rows/s., 4.67 GB/s.)

5 seconds on selecting 5 rows from 45 million rows when order by primary key looks slow.
Is this OK? What am I doing wrong in getting last N rows (by primary key)? Or maybe table is configured wrong?
PS. Versions is:
ClickHouse client version 1.1.54289.
Connecting to clickhouse:9000.
Connected to ClickHouse server version 1.1.54289.

Hello.
ClickHouse don't have optimization which allows skipping rows while reading first (or last) N rows ordered by primary key. So, results you see are expected.
Even when you need to read first N rows, ordered by primary key, it's necessary to merge data from different parts, which is full scan if your query doesn't have limitation on date column or primary key.
However, it's possible to implement more efficiently: read first (or last) N rows from each block. (Also, there are some other technical details).

Hi!
If you store (structured) logs, it's a frequent need to see "last logs". In this scenario, table primary key is timestamp (or, timestamp + nanoseconds, as shown above).
Time-to-time it could be "last logs" without any filter, or filtering by couple columns. But it always last logs. By last I mean last 500-1000 results, it's usually enough.
Will this optimisation be implemented in some near future? Or maybe you could suggest some workaround for this case?

If it's possible in clichouse internals to just get last N elements without full scan, maybe there shoud be special construction  like LIMIT -10 or something like that? It always possible to make count first and then do LIMIT (COUNT - N), N, but this realisation seems a little weird.

Just tried, this limit-with-offset solution also does not work, it does full scan:
:) select count() from logs;

SELECT count()
FROM logs

┌────count()─┐
│<PHONE_NUMBER> │
└────────────┘

1 rows in set. Elapsed: 1.362 sec. Processed 4.28 billion rows, 17.11 GB (3.14 billion rows/s., 12.56 GB/s.)

:) select * from logs limit<PHONE_NUMBER>, 10;

SELECT *
FROM logs
LIMIT<PHONE_NUMBER>, 10

↖ Progress: 216.17 million rows, 113.77 GB (10.33 million rows/s., 5.44 GB/s.) ████▋                                                                                4%^↑ Progress: 217.02 million rows, 114.22 GB (10.32 million rows/s., 5.43 GB/s.) ████▋                                                                                5%Cancelling query.
Ok.
Query was cancelled.

0 rows in set. Elapsed: 21.151 sec. Processed 217.02 million rows, 114.22 GB (10.26 million rows/s., 5.40 GB/s.)

:)

To improve performance, you can add limits on primary key. For example, use
SELECT * from logs where date = today()
or
SELECT * from logs where dateTime > now() - 60 * 60

Hello, does the ClickHouse team have any plans regarding on improving this query pattern?

This is planned to do after more fundamental modifications in query pipeline.
These modifications are planned for Q4. You may look at the prototype here: https://github.com/yandex/ClickHouse/compare/processors#diff-d84e5dc28812accede7d6803ffb73be9R13

Yes, that will be a great feature, Alexey, could you share when it will be available.
Meanwhile I came up with the following workaround:
`select * from rc where created_utc > (select max(created_utc) from rc) - 606024 and subreddit='programming' order by created_utc desc limit 10;
10 rows in set. Elapsed: 4.768 sec. Processed 3.05 million rows, 57.34 MB (638.84 thousand rows/s., 12.03 MB/s.)
`
It may require an iterative process -
1.
`:) select count() from rc where created_utc > (select max(created_utc) from rc) - 606024 and subreddit='programming';
SELECT count()
FROM rc
WHERE (created_utc > (
(
SELECT max(created_utc)
FROM rc
) - ((60 * 60) * 24))) AND (subreddit = 'programming')
┌─count()─┐
│    1248 │
└─────────┘
1 rows in set. Elapsed: 4.510 sec. Processed 3.05 million rows, 56.83 MB (675.38 thousand rows/s., 12.60 MB/s.) `

select * ...

It will be fixed by this PR: #5042

Fixed in master.

Some optimizations still remain: exponential growth of read window from some small value.


Migrate from PyYAML to ruamel.yaml

As agreed at the hacking session, lets move to ruamel.yaml for more flexibility and also comment preserving. This PR:

adds ruamel.yaml as a dependency
replaces yaml completely with ruamel.yaml (which also removes the need for some hack-arounds)
improves formatting of output YAML files as described in https://github.com/psss/tmt/issues/695 (comments are also kept)
only YAML 1.2 is now supported

Resolves: https://github.com/psss/tmt/issues/695

Added discuss label to bring some awareness to the fact that we are switching from YAML 1.1. to 1.2. Just some brainstorming: what about some "smooth" deprecation? Load with YAML 1.1 and check if the result is the same as would be with 1.2 and if it isn't, warn the user so that we give some notice that this is happening

Rebased onto latest master (import merge conflicts). I tried to implement a smooth deprecation as described above. I see one slight problem - plans, stories and also usually tests are loaded through fmf which still uses PyYAML... So realistically, in tmt we can only check it during test lint. However that's probably the main place because it can modify the YAML... I think we can merge it without waiting for 2.0 now and then just drop YAML 1.1 support in 2.0

Seems there is a problem with creating plans with custom config from command line. Simple reproducer:
tmt plan create problem -t mini --discover "{url: 'https://example/'}"

The plan is created as expected with the old implementation:
summary: Basic smoke test
execute:
    script: tmt --help
discover:
    url: https://example/

But looks like this with the new one:
summary: Basic smoke test
execute:
    script: tmt --help
discover: {url: https://example/}

Which is not supported by older PyYAML. However we need to fix this to make the plan more readable.

Thanks for the reproducer, good call to postpone for 1.7, there may be some other small things that are changed so lets play around with this a bit more

Hm, so this issue was caused by the fact that round-trip loader/dumper somehow keeps track of the formatting that was used and hence when outputting the edited yaml, it kept the custom dict formatting the same as it was on the command-line. I tried searching around but didn't find a very elegant solution, however what worked for me was using safe loader to load the data from the command line (it doesn't remember the formatting).
@psss could you please take a look at this PR again and if it seems OK to you, I will rebase with autosquash and we can merge

Looks like there is an old version of ruamel.yaml on EPEL causing comments not to be preserved correctly. As far as I can see the comment being removed is the only problem there. I don't feel like that's a breaking problem and we could disable the test for EPEL. Opinions @psss ?

Completely agreed. Definitely better than before. Let's disable the test for #comment and merge. I've also commented in BZ#1700334 and asked if the epel8 package could be updated.

I've adjusted the test and rebased on the latest master. Let's see what tests will say...

@psss Thanks, the tests now seem to pass. F34 failure is our good old fail during install and rawhide is sphinx-related so I guess we can merge.

Yes, hooray! :)


one last fix! (hopefully)

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


v10 roadmap

Want to formulate my ideas

prefer resize-observer-polyfill to use chrome 64 feature and remove nonce prop
find a way to not use findDOMNode maybe via ref callbacks in render props
make WindowScroller universal for any component
use WindowScroller in MultiGrid implementation and ScrollSync example
find different examples of using ScrollSync. Freezing colums is bad case because of lagging and wheel blocking.
remove scrollToRow, scrollToColumn, scrollToIndex in favor of ref public methods, since representing events/commands as state changing introduces a lot of bugs and inconsistences.
Finish flow types
removing proptypes babel plugin which generates untreeshakable commonjs

Hey 😄 Want to combine this with #658? We can close either one. Just doesn't seem great to have both open.

Sure


vflow don't work with sFlow from Cisco Nexus 3000

Hello!
I'm trying to collect sFlow from Cisco Nexus 3000 and it's doesn't work.
vflow received packages, it can be seen in verbose log, but queue is empty.
>$ ./vflow -verbose
...
[vflow] 2018/02/22 11:30:35 rcvd sflow data from: <IP_ADDRESS>:62255, size: 1312 bytes
[vflow] 2018/02/22 11:30:35 rcvd sflow data from: <IP_ADDRESS>:62255, size: 572 bytes
[vflow] 2018/02/22 11:30:35 rcvd sflow data from: <IP_ADDRESS>:62255, size: 1368 bytes

I added debug log and found that processing packages stop here
https://github.com/VerizonDigital/vflow/blob/master/sflow/decoder.go#L137-L139
In my case datagram.Version = 1 but my Cisco Nexus 3000 exactly sends sFlow v5.
I tried to use this https://github.com/Cistern/sflow go-library and it's doesn't work on my sFlow traffic too.
But this tool https://github.com/sflow/sflowtool works perfectly. Because I guess that bug in vflow.
I save my sflow traffic to pcap over tcpdump, may be it will be useful.

@AlexAkulov no idea why version assigns w/ one as pcap shows it sends five but the main issue that I'm seeing at pcap is sample flow data type is 3 (ipv4 data) and vFlow supports "raw packet header"


CIF-1498 - Setup IT project for CIF components

This PR adds some basic integration tests for the CIF components based on the CIF components library.
The tests are rather basic and only check that each component renders some "basic" elements like for example the product name, the products of a category, etc, without going into all the possible details. The scope of each test could be extended if needed, the main goal here is to ensure that each component properly renders and all elements (components, graphql client, etc, work together).
Note that the it/http project is ignored when we perform a release by setting the skip-it Maven profile during the release.
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[x] All unit tests pass on CircleCi.
[x] I ran all tests locally and they pass.

Just noticed, but might be unrelated to this PR. The integration tests correctly report the integration flag for codecov. The flags for unittests, karma and jest are somehow not correctly applied. Maybe we should update the orb or use the bash uploader directly.

@herzog31 I fixed the codecov flags by setting the orb to the 1.1.1 latest version.

@cjelger Yes that helped 👍 . All flags are available now. See for example https://codecov.io/gh/adobe/aem-core-cif-components/src/6f2142699eb5aa56e24dfe6317da4a681b58f9ba/bundles/core/src/main/java/com/adobe/cq/commerce/core/components/client/MagentoGraphqlClient.java.


Rewrite @Unroll tests

This is to fix 'Received a failure event for test with unknown id '2.9'' error
which is cuased by non-unique Spock @Unroll test names.

Fixes #?
See https://builds.gradle.org/viewLog.html?buildId=35948514&buildTypeId=Gradle_Check_VfsRetention_30_bucket48&tab=buildResultsDiv
and https://github.com/gradle/gradle/issues/8787

@bot-gradle test QF

OK, I've already triggered QuickFeedback build for you.

I assume because the other tests don't need to unroll. It seems to be the pattern we usually use (annotate only the methods that actually Unroll).

My experience with this is that the net result is lots of tests with missing @Unroll, which makes the diagnostics worse.

Thanks @jjohannes @ldaley @wolfs !


bug: tapes aren't recorded with human readable body

Hello there 👋
First of all thanks for the effort you have put into this project. We have been using it to mock our API, and its flexibility is key.
I have a bug to report tho, and I believe I found the culprit.
The issue is that it isn't saving the tapes in human-readable form, which makes it a bit harder to debug them.
The issue is related to the case sensitivity of the content-type headers matcher (here) which I believe should be case insensitive.
For instance, our SDK uses the following request to communicate with the API:
  return api
    .request(`checkouts/${checkoutId}`, {
      method: 'PUT',
      headers: {
        'Content-Type': 'application/json',
      },
      body: JSON.stringify(formattedPayload),
    });

And that causes the tapes to be recorded like:
        body: 'ewogICJwYXltZW50X3R5cGUiOiAiYmFu...(content)',

How to reproduce
Make a request using the following content type format:
const talkbackURL = "http://localhost:3030"
fetch(talkbackURL, {
  headers: {
      'Content-Type': 'application/json'
   },
   body: JSON.stringify({"foo": "bar"}),
}).then(console.log)

Check the generated tape
Expected

Human readable body containing {"foo": "bar"}

Outcome

Encoded body

Suggested fix
When checking human-readable content type, ignore the case
// https://github.dev/ijpiantanida/talkback/blob/main/src/utils/headers.ts
static read(headers: any, headerName: string) {
    const caseInsensitiveKey = Object.key(headers)
         .find(h => h.toLowerCase() === headerName.toLowerCase())
    const value = headers[caseInsensitiveKey];
    if (Array.isArray(value)) {
      return value[0];
    } else {
      return value;
    }
  }```

After a bit of debugging my suggestion wasn't correct, I believe there's something else causing this issue

hi @cristianoliveira, glad you like talkback!
I don't seem to be able to reproduce the issue. The http server should already be lower-casing all the headers.
Do you have any decorators that might be modifying them on the tape directly?
Could you also post the full tape that gets generated?

Here is a demo of the issue: https://github.com/cristianoliveira/talkback-issue-demo
If you look at the saved tapes: https://github.com/cristianoliveira/talkback-issue-demo/blob/main/tapes/
The request body from both is encoded
https://github.com/cristianoliveira/talkback-issue-demo/blob/main/tapes/unnamed-1.json5#L11
I was expecting it to be stored in plain JSON. Is this the expected behavior?

Thanks for the demo, that was helpful!
The problem is allowHeaders: [], which strips the content-type header from the request before talkback can process it.
Try allowHeaders: ['content-type']

Aha bingo! It works! I really didn't notice that! Maybe is worth adding a warning in the https://github.com/ijpiantanida/talkback#options about this behavior :)
If you feel it's a good idea I can open a PR. Anyway, thanks again!


USB Mic Audio Routing

I wonder...............how possible it would be if I can route my USB Mic to my Phone? I have zero idea if this is supported by the Android OS.
For reference:

On Linux, audio routing is done using Carla and JACK

I wonder if the same is possible for Android Devices as a way for improved audio quality.

#3880

Closing as not possible


ARROW-5978: [FlightRPC] [Java] Properly release buffers in Flight integration client

Fixes a bug where dictionaries weren't properly released in cleaning up a flight stream.
Travis build: https://travis-ci.com/lihalite/arrow/builds/119663270

@lihalite it looks like this might have broken something in the integration tests?  Your travis and arrow travis both seemed to have failed (i'm retrying the run here)

Ah sorry, I got distracted - this exposed another bug in the Java integration server, which I've been digging into (dictionaries don't get sent back to the client, the client just gets an empty dictionary). I'll close this for now.


Flaky test: delete a rate limit

Unfortunately I realized after merging. The test passed in the PR and locally but failed when merged to the main branch.
https://github.com/3scale-labs/limitador-operator/blob/bfa64bdf8595d085368a1da5593d81d2946589e0/controllers/ratelimit_controller_test.go#L149

RateLimit CRD has been removed


[[as2_gazebo_assets] Tests fails due to pydantic error ] Tests Flake8 fails

ROS2 Version
humble
Version
main
Installation
source
What happened?
Python tests of as2_keyboard_teleoperation are failing.
How to reproduce your issue?
Using colcon test on pkg as2_keyboard_teleoperation after enabling tests of test folder
Relevant log output
No response

fixed in #484


I hope Share can support Android sharing base64 images via url

Currently, this feature is well supported on ios, but android doesn't work at all

Can the Share component share files in android? I am prompted to "Get Resource Failed" when I use it. How do you do it? @jiangyuzhen


filterVideoAndAudio() Stream URL Returns 403 Error

Hello, I've followed Example 1 explicitly and my video does not load. Interestingly, the audio-only and video-only streams do work successfully. After some debugging, I discovered that the returned URL of the combined video and audio stream has a 403 error from googlevideo.com. Why would a 403 Forbidden error occur for the combined stream but not the video-only or audio-only streams?
Here's a refresher on the code snippet I've been referencing:
Example 1: 403 Error for returned URL when .filterVideoAndAudio() is used; but .filterVideoOnly(), for example, plays successfully.
let stream = try await YouTube(videoID: "QdBZY2fkU-0").streams
                          .filterVideoAndAudio()
                          .filter { $0.isNativelyPlayable }
                          .highestResolutionStream()

let player = AVPlayer(url: stream!.url)

I'd be happy to provide more information, thanks in advance.

Did you use the latest version or main branch directly?
I have a fix on main branch that is not published as a new version yet. Will publish it today/tomorrow, if nobody finds bugs.

FYI: I updated the "Cipher" and "Extraction" swift files locally to match the latest GitHub version -- and this resolved my issue.

Great. I also now released the changes as official version 0.2.3.

Great. I also now released the changes as official version 0.2.3.

Thanks so much!


cmd/7g: ginscall for type==2 (defer) is wrong

After BL runtime.newdefer, we want CMP $0, R0.
R0 here must match implementation of runtime.return0.
it's not necessarily REGRT1.
newdefer return 0 in a register (by calling runtime.return0) if everything is alright,
otherwise it means the deferred function has stopped a panic.

I've successfully run a simple defer test program with the replacement runtime!
With defer available, we can run more method tests.


Access denied to  "/buildpack/compilation"

When executing this statement of the Dockerfile
Compile the application source code and remove temp files
WORKDIR /buildpack
RUN  /buildpack && "/buildpack/compilation" /build /cache &&
rm -fr /cache /tmp/javasdk /tmp/opt
The following error appears:
Step 11/20 : RUN "/buildpack/compilation" /build /cache &&   rm -fr /cache /tmp/javasdk /tmp/opt
---> Running in ac04f5b273cd
/bin/sh: 1: /buildpack/compilation: Permission denied
The command '/bin/sh -c "/buildpack/compilation" /build /cache &&   rm -fr /cache /tmp/javasdk /tmp/opt' returned a non-zero code: 126
ERROR: Job failed: exit code 126

RUN chmod +x /buildpack/compilation

The https://github.com/mendix/docker-mendix-buildpack/pull/58 added the chmod so I will close this issue


feature: Add Confiant Ad Quality module docs

🏷 Type of documentation

[x] new module

📋 Checklist

[x] Related pull requests in prebid.js or server are linked -> Ad Quality Module

@muuki88 thanks for a feedback. If I found correct list of lint issues - hope was able to fix them.
I've updated PR with actual rebase on master branch and single commit.
If lint issues still exist - could you please  provide a link where I can see them. Wasn't able to find proper link inside CI section.

@muuki88 now I see results - going to update in a minute.

I really don't understand why I have to approve the runs every time 🥲

I really don't understand why I have to approve the runs every time 🥲

Was it my mistake? To know if I need to change something in my flow.
If so - please sorry.
Btw not 100% clear rules regarding un-ordered lists - it was asking different styles in different places.

No no. I'm just ranting at GitHub an myself 😅😅


updated mappings

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/1.0.0@0943009). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@               Coverage Diff                @@
##             release/1.0.0     #376   +/-   ##
================================================
  Coverage                 ?   98.95%
================================================
  Files                    ?      125
  Lines                    ?     6287
  Branches                 ?        0
================================================
  Hits                     ?     6221
  Misses                   ?       66
  Partials                 ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Custom path for a theme?

Is there any way to do it?
for example:
ZSH_THEME="~/dotfiles/zsh/themes/themenamehere"

Why not to use symlink inside default path?

You can override ZSH_CUSTOM to ~/dotfiles/zsh, then use ZSH_THEME="themenamehere". See Customization. Otherwise, you can just source the theme yourself after ohmyzsh is sourced.


Invalid date is accepted as valid kennitala

The isPerson validation does not account for invalid dates. I see the current code simply checks that the day is <= 31, but 310222 for instance is not a valid date, and hence not a valid kennitala.
Would have to parse the date and validate it.
One way to do it:
const isValid = isNaN(new Date('2022-02-31').getTime())

Related to #16

Interesting. Not a bad idea to validate the months.

This has been implemented on the v2 branch, you can try yarn add<EMAIL_ADDRESS>to test it out.

Available in V2.0.2 now non-beta.


chore(.env.template): document SERVICE_HOST default

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.864%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
547

Relevant Lines:
548

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.864%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
547

Relevant Lines:
548

💛  - Coveralls


Prepare for azure-security-keyvault-jca 2.9.0 release

Description
Prepare for azure-security-keyvault-jca 2.9.0 release
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] 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 more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

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

close for outdates


Feature/rate limit merge proper

Added more descriptive commit messages.

Thanks for adding meaningful commit messages :-)

@frenchbread and/or @elnzv, please review.


Addition of a new port for the RedShell SDK

This PR adds a port for the RedShell SDK
RedShell allows game developers to track Steam attribution for their game.
For more information visit innervate.us
The portfile simply downloads the source from Github, then places the .h, .lib(s), and .dll(s) into their corresponding architecture as well as release / debug.
CLA has been signed by
Adam Lieb
https://github.com/adamlieb
Innervate Inc

@yintothayang,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@yintothayang, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Please disable installation on ARM architecture
if(VCPKG_TARGET_ARCHITECTURE STREQUAL arm)
  message(FATAL_ERROR "Error: redshell does not supports the ARM architecture.")
endif()

Is your library has appcontainer bit? If not then disable the installation for UWP.
if (VCPKG_CMAKE_SYSTEM_NAME STREQUAL WindowsStore)
    message(FATAL_ERROR "Error: redshell does not supports UWP builds.")
endif()

disabled!
thanks for the quick feedback!

Thanks for walking through this with us! I hope that with this experience we can build some really nice documentation for other companies to publish prebuild binaries as well 😄!
As part of merging, I've added a comment to the CONTROL file indicating the current library versions. This way we can form an appropriate strategy for dealing with (for example) openssl CVE's. Please update these whenever publishing new binaries!