fix metadata handler not return

这个 PR 实现什么功能/修复什么问题:
fix metadata handler not return
是否需要 backport 到之前的 release 分支:
release/2.8.0

/cc @Zexi

/cherrypick release/2.8.0

Codecov Report

Merging #480 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #480      +/-   ##
=========================================
+ Coverage    7.39%   7.43%   +0.04%
=========================================
  Files         345     344       -1
  Lines       54176   53991     -185
=========================================
+ Hits         4004    4014      +10
+ Misses      49797   49605     -192
+ Partials      375     372       -3

Flag
Coverage Δ

#aFlag
7.43% <ø> (+0.04%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/hostman/system_service/systemd.go
45.45% <0%> (-9.1%)
:arrow_down:

pkg/cloudcommon/notifyclient/notify.go
1.98% <0%> (-0.62%)
:arrow_down:

pkg/hostman/guestman/guestman.go
3.97% <0%> (-0.23%)
:arrow_down:

pkg/compute/models/cloudsync.go
0% <0%> (ø)
:arrow_up:

pkg/cloudcommon/db/virtualresource.go
0% <0%> (ø)
:arrow_up:

pkg/mcclient/options/servers.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/managedresource.go
0% <0%> (ø)
:arrow_up:

pkg/hostman/guestman/qemu-kvmhelper.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/cloudaccounts.go
1.01% <0%> (ø)
:arrow_up:

pkg/compute/models/guests.go
0.67% <0%> (ø)
:arrow_up:

... and 19 more

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 141da57...c751d1d. Read the comment docs.

/lgtm
/approve


JS exception on Edge iOS related to support for Sets

It seems Edge iOS does not supports Sets as iterable.  This results in the script error
Uncaught TypeError: Set is not a function or its return value is not iterable
    at s.setValue (cmp.js:2:32561)
    at new s (cmp.js:2:32147)
    at new a (cmp.js:2:61891)
    at t.GppModel.decode (cmp.js:2:27797)
    at t.CmpApi.setGppString (cmp.js:2:6667)
    ....

This limitation appears to be undocumented, at least not something you'll find on MDN or caniuse.com.
A potential fix would be to create a utility function to support this
function setToArray<T>(set: Set<T>): T[] {
  const arr: T[] = [];
  set.forEach(item => arr.push(item));
  return arr;
}

Which version of Edge version is this? Based on MDN Set is supported starting Edge 12.

It turns out this report came from someone testing with dev tools on Mac OS when emulating mobile.  The reporter did not see an issue on his actual iOS device.  Closing this issue.


[Bug]: Using binaries from npm not working as expected

What happened?
I'm trying to use the remotion cli as documented with this load statement:  load("<EMAIL_ADDRESS>remotion_bin = "bin") but I get an error:
ERROR: error loading package 'video-production/playground': cannot load '<EMAIL_ADDRESS>no such file
INFO: Elapsed time: 0.531s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (5 packages loaded)
FAILED: Build did NOT complete successfully (5 packages loaded)
currently loading: video-production/playground ... (2 packages)

It also fails when I try to replicate the typescript example. It feels like I'm missing something small but critical.
Version
Development (host) and target OS/architectures:
Output of bazel --version:
bazel 5.4.0
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:
## Nodejs dependencies
http_archive(
    name = "aspect_rules_js",
    sha256 = "686d5b345592c1958b4aea24049d935ada11b83ae5538658d22b84b353cfbb1e",
    strip_prefix = "rules_js-1.13.1",
    url = "https://github.com/aspect-build/rules_js/archive/refs/tags/v1.13.1.tar.gz",
)

load("@aspect_rules_js//js:repositories.bzl", "rules_js_dependencies")
rules_js_dependencies()

load("@rules_nodejs//nodejs:repositories.bzl", "DEFAULT_NODE_VERSION", "nodejs_register_toolchains")
nodejs_register_toolchains(
    name = "nodejs",
    node_version = DEFAULT_NODE_VERSION,
)

load("@aspect_rules_js//npm:npm_import.bzl", "npm_translate_lock")

npm_translate_lock(
    name = "video_production_playground_npm",
    pnpm_lock = "//video-production/playground:pnpm-lock.yaml",
    verify_node_modules_ignored = "//:.bazelignore",
)

load("@video_production_playground_npm//:repositories.bzl", "npm_repositories")
npm_repositories()

## END Nodejs dependencies

Language(s) and/or frameworks involved:
How to reproduce
No response
Any other information?
No response
Fund our work

[ ] Sponsor our open source work by donating a bug bounty

try<EMAIL_ADDRESS>

<EMAIL_ADDRESS>should work in this case
The Bazel package (video-production/playground) where the npm package you're loading the package_json.bzl from is linked to is part of the load path since the same package may be linked to different locations in your WORKSPACE.

@gregmagolan @ztl8702 That worked! I guess I don't fully understand the linking that's going on and how it relates to the external repo structure generated by these rules


How To Disable Main Thread Checker

Im trying to toggle the disableMainThreadChecker flag on my test scheme but when setting the option and generating the project the change is not reflected.
I would like to know if I am either approaching this incorrectly or if XcodeGen even supports this functionality.
Schemes
"schemes" => {
  "Base" => {
    "build" => {
      "targets" => {
        "Base" => "all",
        "BaseTests" => ["test"],
        "BaseUITests" => ["test"]
      },
      "parallelizeBuildables" => true,
      "buildImplicitDependencies" => true
    },
    "run" => {
      "config" => "Debug",
      "environmentVariables" => {
        "OS_ACTIVITY_MODE" => "disable"
      }
    },
    "test" => {
      "config" => "Debug",
      "disableMainThreadChecker" => true,
      "gatherCoverageData" => true,
      "targets" => [
        "BaseTests",
        "BaseUITests"
      ],
      "environmentVariables" => {
        "OS_ACTIVITY_MODE" => "disable"
      },
    },
    "profile" => {
      "config" => "Debug"
    },
    "analyze" => {
      "config" => "Debug"
    },
    "archive" => {
      "config" => "Debug",
      "revealArchiveInOrganizer:" => true
    }
  }
}

Any help would be greatly appreciated!
Cheers

Added in https://github.com/yonaskolb/XcodeGen/pull/601


Upgrade LaunchBar.app to v6.4.1

Thank you for the contribution. It was merged directly as f4a6be3a826b29270c71cc9b7a8e9cfc4f80d769 to keep commit history cleaner. Your contribution is still credited to you.


Fix(secgroups): ICMP Echo Request being blocked

This fixes a thing that never worked before 😄
Currently ICMP echo requests are blocked by the ingress security group rules for ICMP, which looks like this: type=1:code=8 (ICMP type 1 is not used for anything in the ICMP protocol).
After this PR the ingress security group rules will look like this: type=8 (code seems to be omitted by openstack if it is 0) and allows ICMP echo requests to reach the VM.
Egress isn't a problem because the egress security group rule doesn't block anything.

/lgtm

Wasn't that setting intentional that way to mitigate icmp based attacks
like ping floods? (
https://www.cloudflare.com/learning/ddos/ping-icmp-flood-ddos-attack/)
tcptraceroute to the open ports of the load balancer can still be useful.
I would at least make it optional, so icmp floods can be quickly mitigated.
Niclas Schad @.***> schrieb am Fr., 8. Dez. 2023, 16:44:

@.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
https://github.com/stackitcloud/yawol/pull/269#pullrequestreview-1772668855,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANPNTXFHCZDKHRUJHAINJDYIMYXDAVCNFSM6AAAAABAMY35HKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONZSGY3DQOBVGU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Wasn't that setting intentional that way to mitigate icmp based attacks like ping floods? ( https://www.cloudflare.com/learning/ddos/ping-icmp-flood-ddos-attack/) tcptraceroute to the open ports of the load balancer can still be useful. I would at least make it optional, so icmp floods can be quickly mitigated.

I'm very certain that this wasn't intentional, as the old values were entirely wrong. Even the comment a few lines above indicates that: https://github.com/stackitcloud/yawol/blob/f949e16b0ea40a05d6439aae31ec11ea0bb56057/internal/helper/loadbalancer.go#L204
If you need to mitigate this, just add a security group that blocks ICMP.

I would also say it is not intentional but also see the point that it can be problematic.
Just create a secgroup rule is not that easy because they get reconciled. Maybe we should add it with an setting in lb.spec.options to disable it and enable it by default.
wdyt @Kumm-Kai

rotten talked with @Kumm-Kai
/close


Stackdriver Adapter does not batch requests to Context Graph API

As noted in the TODO here:
https://github.com/istio/istio/blob/release-1.1/mixer/adapter/stackdriver/contextgraph/send.go#L122
This API has a  max size of 600K. Sometimes we are calling it with a larger size. Istio needs to batch large requests.
I'd like to get this fix into Istio 1.1 if possible.

Closing, as PR fix is merged.


Fix: nodejs bump to v8 to get build working again

these failures are expected.
we can either merge this, then kick the 1804 build, or close this and merge the 1804 build.
... or, we can modify the peerscout application to not fail on nodejs6 ;)

this PR has been superceded by this one: https://github.com/elifesciences/peerscout-formula/pull/30/files


[Error] BuildIdentity selected for restore does not match APTicket
BuildIdentity selected for restore:
BuildNumber : 18A373
BuildTrain : Azul
DeviceClass : n66map
FDRSupport : YES
MobileDeviceMinVersion : 1253
RestoreBehavior : Erase
Variant : Customer Erase Install (IPSW)
BuildIdentity is valid for the APTicket:
IM4M is not valid for any restore within the Buildmanifest
This APTicket can't be used for restoring this firmware
[WARNING] NOT VALIDATING SHSH BLOBS!
Variant: Customer Erase Install (IPSW)
This restore will erase all device data.
Device found in DFU Mode.
Getting firmware keys for: n66map
Cleaning up...
[exception]:
what=getting keys failed with error: 14745615 (failed to get FirmwareJson from Server). Are keys publicly available?
code=41680964
line=636
file=/Users/runner/work/futurerestore/futurerestore/src/futurerestore.cpp
commit count=308
commit sha  =9554c0068dc50e141872ced5da2bd95baa595805
Done: restoring failed!
if you are on linux please try above command if futurerestore failed
if futurerestore didn't finish succesfully please try to run (with sudo or without) this command: /Users/riber/Desktop/downr1n/binaries/Darwin/futurerestore -t blobs/iPhone8,2-14.0.shsh2 --use-pwndfu --skip-blob --rdsk work/rdsk.im4p --rkrn work/krnl.im4p --latest-sep --latest-baseband ipsw/iPhone_5.5_14.0_18A373_Restore.ipsw
if futurerestore restore sucess, you can boot using  --boot

Try again always show error!
BuildIdentity is valid for the APTicket:
IM4M is not valid for any restore within the Buildmanifest
This APTicket can't be used for restoring this firmware
[WARNING] NOT VALIDATING SHSH BLOBS!
Variant: Customer Erase Install (IPSW)
This restore will erase all device data.
Device found in DFU Mode.
Getting firmware keys for: n66map
Cleaning up...
[exception]:
what=getting keys failed with error: 14745615 (failed to get FirmwareJson from Server).


SQL query option no longer shows up in the + menu

Bug description
With 4.0, it looks like the SQL query option no longer appears in the global + menu. This used to sit below the data section.
How to reproduce the bug

Go to your superset instance with 4.0
Hover over the plus option in the upper right hand corner
Look at all the options

Screenshots/recordings

Superset version
master / latest-dev
Python version
3.9
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[X] I have searched the GitHub issue tracker and didn't find a similar bug report.
[ ] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

@sadpandajoe @dpgaspar
https://github.com/apache/superset/pull/27117 removed the following deprecated endpoint:
@deprecated(new_target="/sqllab")
def sqllab(self) -> FlaskResponse:

Which was registered in Superset's initialization:
appbuilder.add_view_no_menu(SqlLab)

superset init uses the registered views to dynamically generate the permissions, in that case, can_sqllab was generated from that entry and it does not exist anymore in a fresh installation of Superset. Searching for can_sqllab we can still find many references to it in the code base, including the menu item that is missing.
@dpgaspar I think the solution here would be to do a full revision of our permissions given all deprecated endpoints that were removed in 4.0 and update the code to only reference valid permissions. This was hard to spot before because the permissions are dynamically generated.


refactor: changed default Java package

The Java template is evolved by supporting Apache Kafka other than IBM MQ (with JMS).
For this reason, I think that it does make more sense to have as default Java package for the generated code something more general like com.asyncapi (I see it being used in the Java Sprint template as well) instead of a more specific com.ibm.mq.samples.jms (IBM MQ with JMS).
When using the template for Apache Kafka clients, having the generated code under com.ibm.mq.samples.jms by default sounds weird.
Of course, this can be changed providing the package as parameter to the generator but anyway having the current default package name doesn't seems to be right.

/rtm


unzip size always larger then lambda limit

hello this script never worked for me after configuring all the thing now its throwing unzip size large error. any quick lead will be highly appreciated .
just run npm update before deploy command because the steps you have mentioned are throwing unkown async function error.

@tahseen2k8 did you find a solution for this?

@luisfarzati any idea what could be causing the error?


Trying to build on latest beta

Attempting to build install on Apple Watch Series 0 running latest beta OS. Also running latest Xcode beta.
When attempting build, getting consistent build failures.
Errors I'm seeing:
/Users/me/GIOVANNI/gambatte/libgambatte/src/video/ppu.cpp:320:64: Reference to 'uint_least32_t' is ambiguous
/Applications/Xcode-beta.app/Contents/Developer/Platforms/WatchOS.platform/Developer/SDKs/WatchOS3.2.sdk/usr/include/stdint.h:35:24: Candidate found by name lookup is 'uint_least32_t'
/Users/me/GIOVANNI/gambatte/libgambatte/include/gbint.h:50:23: Candidate found by name lookup is 'gambatte::uint_least32_t'
/Users/me/GIOVANNI/gambatte/libgambatte/src/video/ppu.cpp:386:11: Use of undeclared identifier 'dst'

The project hasn't been tested on the latest beta (Xcode 8.3), could you try building on the latest Xcode? (8.2.1)

Appears to be a duplicate of https://github.com/gabrieloc/GIOVANNI/issues/3


How properly remove filter?

Hello!
I have method, which making night vision effect with help of filters, but i dont understand, how properly remove filter?
if(state) {
    rtmpCamera.glInterface.setFilter(SepiaFilterRender())

    val exposureFilterRender = ExposureFilterRender()
    exposureFilterRender.exposure = 3.0f
    rtmpCamera.glInterface.setFilter(exposureFilterRender)
} else {
    // This calls crash of application
    // rtmpCamera.glInterface.setFilter(null)
}

Thanks!

Got it, thank you!


mkpr

Applied mkpr on undefined
sed s/"@kronos-integration\/service": "^.*"/"@kronos-integration\/service": "^9.2.18"/

Codecov Report

Merging #1397 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #1397   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           39        39
  Branches         1         1
=========================================
  Hits            39        39

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 27d196f...b8f66c9. Read the comment docs.


Codeunit::"Reservation Management", 'OnBeforeDeleteDocumentReservation'

Can you allow for HideValidationDialog parameter to be 'var'? I would like to use this event to deliver a BulkDelete Action for a client. And the client does not want to confirm deletion of reservations dozens (or hundreds) of times. The architecture of the Action sets a condition on the Documents to be deleted; then inside of 'OnBeforeDeleteDocumentReservation', we check for that condition, and set HideValidationDialog to True. (but this will only work if HideValidationDialog is 'var'.)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


change strides to ptrdiff_t

#978 has been merged, rebasing should fix the problem on Windows.

Ok i just rebased! let's see.


Updated README for installing on Ubuntu

lxd-stable repo no longer exists (deprecated). Adding alternate steps to install the golang dependency. Tested on Ubuntu 14.04,16.04 and 17.10

Hmm, should I make another commit and modify the go version to 1.10? Or modify the steps to use go 1.8 instead?


Force mouse source when --forward-all-clicks

Right click and middle click require the source device to be a mouse, not a touchscreen). Therefore, the source device was changed only when a button other than the primary button was pressed (see adc547fa6e8e6167cd9633a97d98de6665b8c23a).
However, this led to inconsistencies between the ACTION_DOWN when a secondary button is pressed (with a mouse as source device) and the matching ACTION_UP when the secondary button is released (with a touchscreen as source device, because then there is no button pressed).
To avoid the problem in all cases, force a mouse as source device when --forward-all-clicks is set. As a consequence, pinch-to-zoom using a virtual finger is disabled in this case.
Concretely, in --forward-all-clicks mode:

device source is set to InputDevice.SOURCE_MOUSE;
motion event toolType is set to MotionEvent.TOOL_TYPE_MOUSE;
pinch-to-zoom simulation is disabled.

Otherwise:

left-click mouse event simulates a first virtual finger;
pinch-to-zoom simulation uses a second virtual finger.

For all (virtual or not) finger events:

device source is set to InputDevice.SOURCE_TOUCHSCREEN;
motion event toolType is set to MotionEvent.TOOL_TYPE_FINGER.

Fixes #3568
TODO:

[ ] test that nothing is broken on a computer with a touchscreen with --forward-all-clicks
[ ] test that nothing is broken on a computer with a touchscreen without --forward-all-clicks

Given this remark (we could use 2 mice for pinch-to-zoom), I adapted the implementation to also support pinch-to-zoom if --forward-all-clicks is enabled.
cc @paweljasinski

Here are the test results with --forward-all-clicks done on pixel 6 pro with stock android 13.

play store - unable to scroll up/down with mouse as a touch, but roller works - no worry
aFreeRDP - pinch works
termux - pinch works
RD Client (microsoft) - the app does not implement pinch, in addition pressing ctrl triggers conflicting functionality
aRDP Free - pinch emulation doesn't work, pinch works with real touch

For 2,3,4 and 5 the right click works.

And for all of them, pinch work without --forward-all-clicks on this branch?

And for all of them, pinch work without --forward-all-clicks on this branch?

yes
yes
yes*
same problems
yes

you have to carefully pick the location where you initiate pinch. If the emulated finger gets over the gui keyboard the app interprets it as dragging over the keyboard. This is consistent with real finger touch.

During last testing session discovered problems with middle mouse button. Let me first figure out what is going on.

OK.
FYI, I will probably release a new version (1.25) soon, especially for #3497.
I plan to merge this one too (it improves the behavior, even if it's not perfect), unless there are major problems with it.
It could be changed later if necessary.


Add get_amdgpu_memory_capacity()

Motivation
Current use of get_gpu_memory_capacity() will break build for SGLang on ROCm (since v0.3.5.post2).
Modifications
Added get_amdgpu_memory_capacity(), and rename previous function to get_nvgpu_memory_capacity()

Checklist

[+] Format your code according to the Contributor Guide.
[+] Add unit tests as outlined in the Contributor Guide.
[+] Update documentation as needed, including docstrings or example tutorials.

Worked on v0.3.5.post1 and before.


.Scratch.Add creates two quotation marks

What version of Hugo are you using (hugo version)?

$ hugo version
Hugo Static Site Generator v0.80.0/extended linux/amd64 BuildDate: unknown (from Fedora 33 repo)

Does this issue reproduce with the latest release?
Yes, I just tested it with the binary for 0.81.
The bug
I was testing different ways to add parameters to schema tags and using this {{- .Scratch.Add "social_media_links" 1 -}} seemingly creates two quotation marks which in my case makes the JSON array invalid.
A bit off topic in case someone else needs a workaround:
Using something like this:
   {{ .Scratch.Add "test" (slice .Params.instagram_url .Params.facebook_url) }}
   {{ apply (.Scratch.Get "test") "urlize" "." }}

Creates a perfect array of strings that you can put in an array in JSON.
Edit: I just realized it also creates two extra quotation marks but it solves my issue I had in the schema tag.
Edit2: Using trim like this makes no difference:
{{- trim (.Scratch.Add "test" (slice .Params.instagram_url .Params.facebook_url)) "\"" -}}

Edit3: .Scratch.Set also creates two quotation marks but not .Scratch.Get. I think for now I will just move the the initiation to somewhere outside the JSON object.

Edit 1: This is a SUPPORT question. you should have opened it over at the discourse as was advised when you opened the issue.
When you "print" out a variable, then Hugo shows it as it's defined, which adds the " to the code. You need to format them like the following:
{{ printf "%s" $variable }}

And it's printed as you defined. The %s means string. Other formats - read the docs: https://gohugo.io/functions/printf/#readout

I apologize, I guess my inexperience caused me to believe it was a bug.

No problem, for the future feel free to look by at discourse.gohugo.io - help comes quicker there and if it's a bug we will find that out too :) hope the printf solves your problem.

I did not quite get what you mean with "when you print out a variable" since I am not printing anything intentionally just adding to .Scratch and that adds the ". I will probably open a thread on the discourse soon though, maybe we should discuss it there.

Not so surprisingly I found a post about it but I have no idea about the reasoning for it. To me it looks like unexpected behavior.


bitcode from AArch64 and run it on x86-64

Compiling bitcode back to runnable programs
It’s finally time to make the magic happen — we’re going to take bitcode from an AArch64 program, and make it run on x86-64. We have conveniently ensured that a Clang compiler is installed alongside McSema, and in such a way that it does not clash with any other compilers that you already have installed. Here’s how to use that Clang to compile the lifted bitcode into an executable named /tmp/maze.aarch64.lifted.
remill-clang-3.9 -o /tmp/maze.aarch64.lifted  /tmp/maze.aarch64.bc
I fail to understand these contents. the bitcode from an AArch64program and make it run on x86-64.
there is no special option to remind remill-clang-3.9 to build a binary for running on x86-64.

The bitcode is itself a little "specialized emulator" for the program. As long as the endianness and pointer sizes match, and as long as the bitcode was lifted with --explicit_args and doesn't do any really "interesting" things (e.g. cpuid), then most times, compiling the bitcode to target a different architecture "just works".


Add tip for Comment keyboard Shortcut.

Add the tip for how to create the comments by keyboard shortcut in probably all modern text editors.

[ ] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you so much sir! I am very happy today, as i am capable to contribute to the most starred repository on Github. In future i will definitely look forward for more contributions :)


Can't load Atlas in PIXI 3.0

Hi, I have problem with using PIXI 3.0 and spritesheets. My application on PIXI 2.0 worked well but after updating to 3.0 spritesheets does not load properly. I'm developing application for Tizen device.  I also tried running it locally with Python simple http server and I get different error.
Uncaught TypeError: Cannot read property '_uvs' of undefined

Does something changed regarding spritesheets/atlases?

Lots of things changed with everything. If you are on the latest version (3.0.3) and this is happening, please provide a minimal reproduce case so we can investigate!
Thanks!

Xhr won't work properly with the file:// protocol, you need to go over http:// or https:// (start a web server instead of launching from /opt/usr/apps...

It's not the problem with the file protocol. It's Tizen OS. Loading other files work properly. The only problem are spritesheets. I will prepare reproduction.

I am getting the same issue. Wondering if I am implementing it the right way:
loader = PIXI.loader.add(url)
loader.load(function(loader, result) {
   mc = new PIXI.extras.MovieClip([result[url].texture])
   @stage.addChild(mc)
})

but I guess the MovieClip class expects a json file with the spritesheet details, right?

No it expects an array of texture objects.
If you load a spritesheet in the loader the resulting resource object will have a textures property that are all the frames loaded from that sheet, keyed like they were in the json file. (if you use json hash it is an object, if you use json array it is an array).

Yeah, but in my case I want to load separately each frame (0.png, 1.png, 2.png) and then I was expecting to use it as MovieClip.
Similar to what's happening on the example given at https://pixijs.github.io/docs/PIXI.extras.MovieClip.html.

That's fine, you can load whatever you want. MovieClip doesn't care where it came from or what the textures are, it just takes in an array of textures and cycles through them.

closing for now, let me know if this is still a thing! ta!


Organization is now more general

The organization now has a unified interface
There is only an interface that represents an organization. That
interface exposes the methods expected to be found on an organization.
Tests were updated accordingly.
Admin is now regarded as owner
From the point of view of a consumer of the OrganizationInterface, an
address that is registered as an admin to the organization will be seen
as the organization itself.
Organization is initiated at construction
It is now possible to set an owner, admin, and workers at construction
of an organization.
Smaller changes

Organization

Renamed isOwner to isOrganization
The admin address can now be the same as the owner

Removed OpsManaged and Workers
Removed Owned

Fixes #543

Added @pgev as reviewer so we can be faster when moving the updates to openst-cotracts.


trigger sane pdf reports build

Status
Ready/In Progress/In Hold (Reason for hold)
Related Content Pull Request
Related PR: link to the PR at demisto/content
Related Issues
Related: link to the issue
Description
A few sentences describing the overall goals of the pull request's commits.

Docker Image Ready - Dev
Docker automatic build at CircleCI has deployed your docker image: devdemisto/sane-pdf-reports:<IP_ADDRESS>997
It is available now on docker hub at: https://hub.docker.com/r/devdemisto/sane-pdf-reports/tags
Get started by pulling the image:
docker pull devdemisto/sane-pdf-reports:<IP_ADDRESS>997

Docker Metadata

Image Size: 667.82 MB

Image ID: sha256:30a7cac4ecc4ed652606becd6e4f9acacb096221c90e248677e2bc516ec2ae50

Created: 2022-04-27T06:52:36.390981922Z

Arch: linux/amd64

Command: ["python3"]

Environment:

PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
LANG=C.UTF-8
GPG_KEY=E3FF2839C048B25C084DEBE9B26995E310250568
PYTHON_VERSION=3.9.6
PYTHON_PIP_VERSION=21.1.3
PYTHON_GET_PIP_URL=https://github.com/pypa/get-pip/raw/a1675ab6c2bd898ed82b1f58c486097f763c74a9/public/get-pip.py
PYTHON_GET_PIP_SHA256=6665659241292b2147b58922b9ffe11dda66b39d52d8a6f3aa310bc1d60ea6f7
DOCKER_IMAGE=devdemisto/sane-pdf-reports:<IP_ADDRESS>997

Labels:

org.opencontainers.image.authors:Demisto<EMAIL_ADDRESS>org.opencontainers.image.revision:4eeecc53cc5d7a9dc2e719a265923090a1686977
org.opencontainers.image.version:<IP_ADDRESS>997


DS-R: 3.2.DOC Data & Packages - Generating Data

Description 📜
In this issue, you need to work on the documentation efforts in the "Data & Packages" section under the "Generating Data" subsection. Please ensure that you document in detail.

Domain of Contribution 📊

[x] Datascience with R

Instructions ✔

Kindly make a README going over the subtopic.
Changes should be made under Datascience_With_R branch only and push your PR to the same.
This issue is for GWOC'21 contributors only.
You need to push in a single commit. If there exists more than one commit then you need to squash your commits into one commit.

Hello @micepram ,
I'd like to document this issue. Please /assign me this issue.

Hey @ayushikulshreshta30, I've assigned you to another one where you commented prior. We are trying to do 1 issue / contributor so that everyone can work on something. Could you kindly finish that one and then I'll assign you here.

Hello @micepram ,
I'd like to document this issue.
/assign

/assign

Hi @Meet19aug! How's the progress going along?

Hello Girlscript/Winter-Of-Contributing
I am working on it [girlscript/winter-of-contributing] DS-R: 3.2.DOC
Data & Packages - Generating Data (#1074)], I need 2-3 days more to
complete it.
On Sun, Sep 26, 2021 at 1:34 PM micepram @.***> wrote:

Hi @Meet19aug https://github.com/Meet19aug! How's the progress going
along?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/issues/1074#issuecomment-927254583,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOVVA3N6MIRXMJCY6IQ3UFDUD3HXHANCNFSM5EKUBDEA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hey @Meet19aug, what's the progress?


after install of spleeter: ModuleNotFoundError: No module named '_lzma'

Hello !!
I managed to install spleeter on my debian 10 .
when i launch :

spleeter separate -p spleeter:2stems -o output audio_example.mp3

I have

Traceback (most recent call last):
File "/home/romger/.local/bin/spleeter", line 8, in
sys.exit(entrypoint())
File "/home/romger/.local/lib/python3.9/site-packages/spleeter/main.py", line 256, in entrypoint
spleeter()
File "/home/romger/.local/lib/python3.9/site-packages/typer/main.py", line 214, in call
return get_command(self)(*args, **kwargs)
File "/home/romger/.local/lib/python3.9/site-packages/click/core.py", line 829, in call
return self.main(*args, **kwargs)
File "/home/romger/.local/lib/python3.9/site-packages/click/core.py", line 782, in main
rv = self.invoke(ctx)
File "/home/romger/.local/lib/python3.9/site-packages/click/core.py", line 1259, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/home/romger/.local/lib/python3.9/site-packages/click/core.py", line 1066, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/romger/.local/lib/python3.9/site-packages/click/core.py", line 610, in invoke
return callback(*args, **kwargs)
File "/home/romger/.local/lib/python3.9/site-packages/typer/main.py", line 497, in wrapper
return callback(**use_params)  # type: ignore
File "/home/romger/.local/lib/python3.9/site-packages/spleeter/main.py", line 114, in separate
from .separator import Separator
File "/home/romger/.local/lib/python3.9/site-packages/spleeter/separator.py", line 27, in
from librosa.core import istft, stft
File "/home/romger/.local/lib/python3.9/site-packages/librosa/init.py", line 211, in
from . import core
File "/home/romger/.local/lib/python3.9/site-packages/librosa/core/init.py", line 5, in
from .convert import *  # pylint: disable=wildcard-import
File "/home/romger/.local/lib/python3.9/site-packages/librosa/core/convert.py", line 7, in
from . import notation
File "/home/romger/.local/lib/python3.9/site-packages/librosa/core/notation.py", line 8, in
from ..util.exceptions import ParameterError
File "/home/romger/.local/lib/python3.9/site-packages/librosa/util/init.py", line 84, in
from .files import *  # pylint: disable=wildcard-import
File "/home/romger/.local/lib/python3.9/site-packages/librosa/util/files.py", line 11, in
import pooch
File "/home/romger/.local/lib/python3.9/site-packages/pooch/init.py", line 19, in
from .processors import Unzip, Untar, Decompress
File "/home/romger/.local/lib/python3.9/site-packages/pooch/processors.py", line 14, in
import lzma
File "/usr/local/lib/python3.9/lzma.py", line 27, in
from _lzma import *
ModuleNotFoundError: No module named '_lzma'

I follow this post :
[(https://github.com/pandas-dev/pandas/issues/27532 )]
However for install of spleeter I had install python3.9 with conpilation . Then my lzma install is not accept by python3.9 .
Thank for your help !!
Jérome

| os    | linux librazik based on debian 10 |
| ram  | 6Gb                                              |
| cpu  | Intel inside Core I5 vPro               |

This is not an issue with spleeter, it's a problem with your python installation.. When you compile your own python, you need to add to add in lzma... If you use a prebuilt python distribution the problem should go away..
You can also try installing lzma using apt, yum, etc..


[Flutter 3.13.3][Android][GetX 5 - RC5] Get.currentRoute does not get updated

Describe the bug
Get.currentRoute does not get updated when using BackButtonListener
To Reproduce
Steps to reproduce the behavior:

Use BackButtonListener First Page with a dialog confirmation to exit the app
Add a log in the listener which prints Get.currentRoute
Navigate to second page
Use BackButtonListener in second page with a dialog confirmation. Now, when the user tries to back, it will call both the listener ( not sure if this was the expected behaviour )
On confirmation, the user goes back from the second screen
Now when in first screen, when the user tries to go back, the logger will print the previous route.

Expected behavior
Expected behaviour :

BackButtonListener should not get confused between the two pages.
The Get.currentRoute should be updated

Flutter Version:
3.13.3
Getx Version:
GetX 5 - RC5
Describe on which device you found the bug:
Pixel 6A, Android 14

I have encountered this issue as well.
my implementation of onBackButtonPressed:
if (Get.currentRoute == controller.routePath) {
  //  ask user
  final r = await controller.backDialog();
  if (r) {
    // should back
    Get.back();
  }
  // handled
  return false;
}
// not handled
return true;

Same here, but It doesn't seem to happen every time, i just couldn't figure out exactly what causes it


Fix for: #14 GLIBCXX_3.4.21' not found (required log-plugin.so) / Updated samp-log-core` to 0.5.2

Original PR text:

This PR is partial fix for issue #14.
But issue with GLIBCXX_3.4.21 still occurs with samp-log-core itself, because samp-log uses outdated version of samp-log-core (without statically linked libgcc & libstdc++) - to fix it, we need to update samp-log to use latest version of samp-log-core.

Updated PR description:
This PR fixes error 'GLIBCXX_3.4.21' not found (closes #14)

[x] Actualized CMakeLists.txt
[x] Actualized  appveyor.yml
└ Now, these files almost the same as samp-log-core's files
[x] Bumped to samp-log-core v0.5.2
[x] Pinned C++ & C std's to be -std=gnu++98 &  -std=gnu89, but:

Maybe, we no need to do pinning to specific std's versions, ! but exactly these versions were used at officially supported by SAMP gcc 4.9.2 - libstdc++ 6.0.20.
(it took me a long time to figure it out by googling the gcc documentation, changelogs & migration guides).

Thanks for the PR!
I'm not sure why you added the -std=gnu++98 -pedantic and removed the -Werror GCC/clang compiler flags, could you elaborate on that?

@maddinat0r sorry for late reply.
Removed -Werror because it removed from samp-log-core in this comit - I just synced files between each other via same commit.
What about -std=gnu++98 - that's default value of std option in gcc 4.9.2 which had been used to compile samp itself.
So, just for best fit with samp. If you think that obsolete and no needed, then I will remove it from commit via force push.

@maddinat0r any updates here? Some people facing the same problem and contacted me via email to send them compiled samp-log with this PR included.
I think it's time to merge PR.


added new tables to sqla model

You seem to have a missing import causing the tests to fail. Can you please update and make sure tests pass (you can check locally by calling tox). I think the missing import is PrimaryKeyConstraint.


Add support for defining the MPC dynamic_types through each separate …

…brix11 collection
* brix11/lib/brix11/brix/common/cmds/configure/configurator.rb:
* brix11/lib/brix11/brix/common/cmds/configure/dsl.rb:
* brix11/lib/brix11/brix/common/cmds/configure/mpc_config.rb:
* etc/configurerc:

Temporarily added TAOX11_ROOT/bin/MPC back, this can be removed after TAOX11 https://github.com/RemedyIT/taox11/pull/107 has been merged, that way we always keep a compiling master.


join not functioning correctly

declare function join<const T extends readonly string[], D extends string = ''>(tuple: T, delimiter?: D): Join<T, D>;
There are errors on this type in the source code, not sure how to fix though!

Hi @danharten-sovtech , thanks for reporting! What is your TS version?

Oops, good information to include lol, I am actually using 4.9.5 @gustavoguichard :)

@danharten-sovtech I just checked and removing the const type parameter would make us have to use as const every time we'd use join like so:
import { join } from 'string-ts'

const result = join(['hello', 'world'] as const, ' ')

That would cause a breaking change so we will only support TS 5+ versions.
We'll make it clear in the documentation.

I'd say that's a fair outcome, thank you for the efforts @gustavoguichard. Will trial TS 5+ to see if it works for my proj!

You're welcome @danharten-sovtech !
If you can upgrade your TS version you are only gonna benefit from it in many aspects ;)


chore: Add back and fix fastro

Adds back fastro and fixes what was wrong with it initially

LGTM


Enhancement request

Yup, me again... might as well just use my private fork of your project but in case you would be interested, it could be nice if the first input element could be selected by default when changing page.....
I will submit a PR regarding this "enhancement".

included with pull request #6


Can't find definition for method get_previous_by_created_ts.

Assume I have a Model named Foo.  I've noticed the HistoricalFoo object has a family of methods:

get_previous_by_created_ts
get_previous_by_history_date
get_previous_by_updated_ts
get_next_by_created_ts
get_next_by_history_date
get_next_by_updated_ts

I'd like to look at the definition of these methods but I can't find them in the repo.
The most I've been able to deduce is that the Historical* classes are built using the create_history_model of HistoricalRecords but I can't figure out when the get_previous* and get_next* methods are added.
Can someone enlighten me, please?
>>> from django.apps import apps
>>> HF = apps.get_model('app1', model_name='HistoricalFoo')
>>> help(HF)

This provides:
class HistoricalFoo(django.db.models.base.Model)
 |  HistoricalFoo(id, deleted_ts, name, ad_domain, created_ts, updated_ts, company, history_id, history_date, history_user, history_type)
 |
 |  Method resolution order:
 |      HistoricalFoo
 |      django.db.models.base.Model
 |      builtins.object
 |
 |  Methods defined here:
 |
 |  __str__ lambda self
 |
 |  get_history_type_display = _curried(*moreargs, **morekwargs)
 |
 |  get_next_by_created_ts = _curried(*moreargs, **morekwargs)
 |
 |  get_next_by_history_date = _curried(*moreargs, **morekwargs)
 |
 |  get_next_by_updated_ts = _curried(*moreargs, **morekwargs)
 |
 |  get_previous_by_created_ts = _curried(*moreargs, **morekwargs)
 |
 |  get_previous_by_history_date = _curried(*moreargs, **morekwargs)
 |
 |  get_previous_by_updated_ts = _curried(*moreargs, **morekwargs)
 |
 |  history_object
 |
 |  revert_url(self)
 |      URL for this change in the default admin site.
 |
 |  ----------------------------------------------------------------------
 |  Data descriptors defined here:
....

@sgallen: Actually we can't take credit for those. They are part of the extra instance methods available on all Django models.

@macro1 Thanks!  My google-fu failed me. =(

No problem. I actually wasn't aware that adding a Datetime field would give nice little utility methods like that, so thanks for asking.


[FEAT] pane styles

We can try using inline styles. Then, when initializing a new pane, new "style" tags will not be created. And to apply the theme, we can use css variables. I would do it if it fits into the development plans.

@spb-web Big thanks for actual issue!

Then, when initializing a new pane, new "style" tags will not be created.
This is true, and i looking for best way that we can use styles.

We can try using inline styles
Good to not using inline at all. Basic styles which users could configure must be internal.
But we can use inline for service adjustments such as inversed styles.

And to apply the theme, we can use css variables
We can use some variables.

Feel free to create pull, i will review and adjust if needs.

@spb-web I think we can just apply additional classes for configurations.
E.g.
.cupertino-pane-wrapper .inversed .darkmode {

}

And create styles at beginning.
Feel free to create pull, i will review and adjust if needs.
And please write here or create pull with some css variables you are think good to use.

If we use classes and “style” tags for styling, then we will have to add the removal of unused “style” tags when the panel is destroyed. It's easier for me to use inline styles. Please see an example of this. This is not complete code yet, but it is sufficient.

@roman-rr @spb-web I'm also facing a wired style issue, where panes styles working weirdly ( no animation or wrong opening positions ).
I have three panes on a single page, two opens from the bottom and one opens from the top ( using inverse mode ).
Ex:-
Pane a: - Open from top
Pane b: Open from bottom
Pane c: Open from bottom.
Now when you open Pane a first and then go for pane b, pane b will work weirdly, and same if you do reverse. But when you open Pane b and pane c in sequence all works fine.
After hours of debugging I found that, Cupertino creates  style tags for panes and add them to the head section (which is causing issues here), and when you destroy a pane styles stays there it doesn't get removed.
So for fixing the issues I need to remove them manually. I achieved that by doing the following.
/**
     * close a pane
     * close a pane and removes it from paneMapper holder
     * @param {string} el
     */
    public close(el: string): void {
        // get pane
        const pane = this.getInstance(el);
        if (pane) {
            // delete from pane mapper
            this.paneMapper.delete(el);
            // destroy pane
            pane.destroy({ animate: true });

            // let animation happen, than remove styles
            setTimeout(() => {
                this.removeStyles();
            }, this._defaultSettings.animationDuration);
        }
    }

private removeStyles(): void {
        const styles = document.getElementsByTagName('style');
        for (let i = 0; i < styles.length; i++) {
            const style = styles.item(i);
            const isCupertinoPaneStyle = style.innerHTML.trim().startsWith('.cupertino-pane-wrapper');
            if (isCupertinoPaneStyle) {
                style.parentNode.removeChild(style);
            }
        }
    }

I think @spb-web is right we can use inline-styles, which will eliminate above mentioned issue.
Thanks

@Vishal-Isharani @spb-web i will personally work on it on the next 24 hrs.

#134

@Vishal-Isharani Should be fixed in master branch. But in next few days will be released as 1.2.5 with variables also.

@Vishal-Isharani Should be fixed in master branch. But in next few days will be released as 1.2.5 with variables also.

That's is some quick @roman-rr,
Thanks.


The kiosk deployment should include limits and probes

@moshloop thanks for reporting this issue! Yes you are correct we can add these!

Implemented with #83 & #82


add dbField to formSchema

I don't like this solution but I'm ok to approve if we can talk about it in the near future CC @basememara


Map View / Scroll View App Crash

Hello,
https://github.com/NativeScript/NativeScript/issues/3960#issuecomment-324662048
One of the core team members (I think) added some comments to the above issue on how to resolve the issue with the app crashing with a scroll view / map view on the same page.  The team member suggested some plugin edits.
I am also having this same problem listed in the above issue.
-Randy

Any thoughts on this @dapriett ?

hey @randy-johnson, thanks for following up with the Nativescript team on the issue.  Sorry, haven't had a chance to look at this yet, but seems reasonable.  If you like, you can do a pull requests with the changes, I'll take a look and merge it in.

Hey @randy-johnson ! I tried to fix it, but I couldn't
But luckily I have a work around, I've written the steps to follow it here
https://github.com/NativeScript/NativeScript/issues/4821

I have also the same issue when I come back to the previous screen that has mapView implementation then
the application crashes with the following error. so please help me out to solve the issue @dapriett  and @shiv19
java.lang.ClassCastException: android.view.AbsSavedState$1 cannot be cast to com.telerik.android.primitives.widget.sidedrawer.SideDrawerState

@vivekprajapati12011995 I've written a temporary workaround for it.
https://shiv19.com/how-to-fix-nativescript-google-maps-sdk-map-view-crash-inside-scroll-view/


THRIFT-4306 dlang: public imports for dependencies

If you have an IDL file that imports another IDL file, you can define
a service that accepts, yields, or throws types defined in the
imported file. In this case, the dlang output should ensure that the
types defined in the imported IDL module are visible in the service
interface's module.
The generator failed to do that, resulting in a failure to compile
the generated source in these situations. This was previously
accepted by the compiler due to some long-standing bugs with symbol
visibility, but the compiler bugs have been resolved.
Fixes #4306.

@jfarrell please close this, it was merged in commit 44701e82ad351d7472101ad81aa13e775ffb7384


Extension causes high cpu load

Issue Type: Performance
Extension Name: terraform
Extension Version: 1.3.9
OS Version: Linux x64 5.0.6-200.fc29.x86_64
VSCode version: 1.33.1

:warning: Make sure to attach this file from your home-directory:
:warning:/home/agpenton/mauve.terraform-unresponsive.cpuprofile.txt
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

Missing cpuprofile.txt file, closing bug report.
Thanks for using my plugin and I am sorry that it doesn't work as expected for you without the profile data I cannot debug further.


Setup subdomains for network

https://network.hypersign.id
https://explorer.hypersign.id

For thhe time being we are using the following urls

Network (RPC) : https://stage.hypermine.in/network/node1-rps/
Network (REST): https://stage.hypermine.in/network/node1-rest/
Explorer: https://stage.hypermine.in/explorer/
Wallet: https://wallet.hypermine.in/


Microsoft.AspNetCore.Mvc.Razor.RuntimeCompilation should default add to when using dotnet run

In https://docs.microsoft.com/en-us/aspnet/core/mvc/views/view-compilation?view=aspnetcore-3.0, Runtime compilation is everyone should used features when to development with MVC and Razor Page, It should be default

@TangMonk Thanks for contacting us.
Runtime compilation is mainly a development productivity feature that adds additional dependencies to the app that increase the size in a non-trivial way, which affect the size of the deployed app when this is not a feature that is normally used in production scenarios.
For that reason is not enabled by default and it is unlikely we will do so.

thanks, I think nuget should support load dependency for different environment, just like Bundler, or npm


Container Resources

there is a problem, I opened a container and it gives me too many resources
![Screenshot_20211116-231245_Chrome]
(https://user-images.githubusercontent.com/48773377/142066774-b6d1b3aa-f489-47fd-982a-d8eca0cd38ce.jpg)

container file, and reduce the parameters. there you can arrange everything so that it will be fine.


cinnamon.mint-artwork: 1.7.9 -> 1.8.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/mint-artwork/versions.
meta.description for cinnamon.mint-artwork is: Artwork for the cinnamon desktop
meta.homepage for cinnamon.mint-artwork is: https://github.com/linuxmint/mint-artwork
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.8.0 in filename of file in /nix/store/37gkdqwb8j64gikwas1nnw2gjm4h6ifz-mint-artwork-1.8.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
cinnamon.cinnamon-gsettings-overrides
cinnamon.mint-artwork

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/37gkdqwb8j64gikwas1nnw2gjm4h6ifz-mint-artwork-1.8.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cinnamon.mint-artwork https://github.com/r-ryantm/nixpkgs/archive/2156b3e02040f3ff607b8fe0298de0d645c54681.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/2156b3e02040f3ff607b8fe0298de0d645c54681#cinnamon.mint-artwork

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/37gkdqwb8j64gikwas1nnw2gjm4h6ifz-mint-artwork-1.8.0
ls -la /nix/store/37gkdqwb8j64gikwas1nnw2gjm4h6ifz-mint-artwork-1.8.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

cinnamon.cinnamon-gsettings-overrides
cinnamon.mint-artwork

Maintainer pings
cc @bobby285271 @mkg20001 for testing.

@ofborg build cinnamon.mint-artwork


fix(wandb): handle exceptions

This should handle exceptions when an arg is not available.
I took advantage to fix last step not syncing as well as step number.
@sgugger maybe we can merge it to fix the issue but I believe ignore_exceptions would be better in fastcore.
Also I think it's useful to keep listing those args even if we will have log_args parameters because they serve a different purpose (init args vs parameters used during training). I think the section _log_config will be moved to Learner.gather_parameters.

Thanks for the PR. I will merge and move the ignore_exceptions to fastcore. I just need to think of places we use something similar and see if I need to add behavior to it.


Search directory page text clarification

Hi Seth, all
I would like to suggest couple of changes for the Search Directory Page (the page that you arrive at after you search for a concept). This is the page that lists the hits in the Ontology, Gene/GP and Annotation categories.
Change
Ontology Ontology classes for GO is very jargony
to
Gene Ontology Term or Synonym or Definition [tab] (# of hits)
Also for all three categories the # of hits is on the far right end of the row and my suggestion is to move it closer to the category name (just separated by a tab).
Thanks,
Rama

From the contexts that I've seen, changing to:
description: Gene Ontology Term, Synonym, or Definition.


New package: typeTree v0.1.0

Registering package: typeTree
Repository: https://github.com/cnaak/typeTree.jl
Created by: @cnaak
Version: v0.1.0
Commit: 16f2ec624d4428f188371084b75111990d280b93
Reviewed by: @cnaak
Reference: https://github.com/cnaak/typeTree.jl/commit/16f2ec624d4428f188371084b75111990d280b93#commitcomment-53188989

OK, I'll rename the package.
From above:

After you have fixed the AutoMerge issues, simple retrigger Registrator, which will automatically update this pull request.

Could someone tell me how to simply "retrigger Registrator" so as to have this PR updated (rather than creating a new one)?
Thanks in advance!

Please CANCEL this PR, as I've fixed issues and re-triggered registration (#40470).


Add more schema to exhort openapi.

Change the starts_with() to consider the immediately-preceeding to_lowercase() when matching for CVEs.

+1


Optimised imports - api.py

Fixed imports to only import required items instead of the entire module (faster execution, reduced resource usage)

The impact on memory is 0, the entire module is always imported, regardless if you use from ... import ... or import ....
>>> import sys
>>> from requests import get
>>> print(dir(sys.modules["requests"]))
['ConnectTimeout', 'ConnectionError', 'DependencyWarning', 'FileModeWarning', 'HTTPError', 'JSONDecodeError', 'NullHandler', 'PreparedRequest', 'ReadTimeout', 'Request', 'RequestException', 'RequestsDependencyWarning', 'Response', 'Session', 'Timeout', 'TooManyRedirects', 'URLRequired', '__author__', '__author_email__', '__build__', '__builtins__', '__cached__', '__cake__', '__copyright__', '__description__', '__doc__', '__file__', '__license__', '__loader__', '__name__', '__package__', '__path__', '__spec__', '__title__', '__url__', '__version__', '_check_cryptography', '_internal_utils', 'adapters', 'api', 'auth', 'certs', 'chardet_version', 'charset_normalizer_version', 'check_compatibility', 'codes', 'compat', 'cookies', 'delete', 'exceptions', 'get', 'head', 'hooks', 'logging', 'models', 'options', 'packages', 'patch', 'post', 'put', 'request', 'session', 'sessions', 'ssl', 'status_codes', 'structures', 'urllib3', 'utils', 'warnings']

The impact on speed is ~0, the only difference is an extra name lookup because of the dotted access.

Hey Plutaniano, thanks for your comment; it got me to look into this more. I did some testing with some quick copilot-generated code:
Specific import:
from time import sleep
sleep(0.5)

Module import:
import time
time.sleep(0.5)

Runner code:
import time

def run():
    total_time = 0
    for _ in range(100):
        start_time = time.time()
        exec(open('./speed/1.py').read())
        end_time = time.time()
        execution_time = end_time - start_time
        total_time += execution_time
    average_time = total_time / 100
    print(f"Specific import: {average_time} seconds")

    total_time = 0
    for _ in range(100):
        start_time = time.time()
        exec(open('./speed/2.py').read())
        end_time = time.time()
        execution_time = end_time - start_time
        total_time += execution_time
    average_time = total_time / 100
    print(f"Module import: {average_time} seconds")

for _ in range(5):
    run()

Here are the results (each pair of results runs each script 100 times for a total of 1000 executions):
Specific import: 0.5038385605812072 seconds
Module import: 0.5042011451721191 seconds

Specific import: 0.503937861919403 seconds
Module import: 0.5037545895576477 seconds

Specific import: 0.5042110896110534 seconds
Module import: 0.5042633104324341 seconds

Specific import: 0.5043397688865662 seconds
Module import: 0.5043723654747009 seconds

Specific import: 0.504006495475769 seconds
Module import: 0.504084300994873 seconds

Average Specific import time: 0.50406675529 seconds
Average Module import time : 0.50422445344 seconds
As for the memory usage, it is a fair point as it does still import the entire module; I think this could be changed 🤣.

Your methodology for measuring is not ideal. Time fluctuations in the execution of time.time, open, .read() (because of CPU scheduling and disk access) will overpower the nanosecond increase caused an attribute lookup.
from x import y is faster, you can just count the instructions below, but the difference is so small it doesn't matter. This library will routinely make API calls that take multiple seconds. A nanosecond increase is literally undetectable.
The impact on legibility is not worth it.
In [1]: def just_import():
   ...:     import requests
   ...:     requests.get
   ...:

In [2]: def from_import():
   ...:     from requests import get
   ...:     get
   ...:

In [3]: import dis

In [4]: dis.dis(from_import)
  1           0 RESUME                   0

  2           2 LOAD_CONST               1 (0)
              4 LOAD_CONST               2 (('get',))
              6 IMPORT_NAME              0 (requests)
              8 IMPORT_FROM              1 (get)
             10 STORE_FAST               0 (get)
             12 POP_TOP

  3          14 LOAD_FAST                0 (get)
             16 POP_TOP
             18 LOAD_CONST               0 (None)
             20 RETURN_VALUE

In [5]: dis.dis(just_import)
  1           0 RESUME                   0

  2           2 LOAD_CONST               1 (0)
              4 LOAD_CONST               0 (None)
              6 IMPORT_NAME              0 (requests)
              8 STORE_FAST               0 (requests)

  3          10 LOAD_FAST                0 (requests)
             12 LOAD_ATTR                1 (get)    # this is the extra instruction
             22 POP_TOP
             24 LOAD_CONST               0 (None)
             26 RETURN_VALUE

The idea of optimizing a python program like this, regardless of whether or not it's in a web API wrapper, is inherently very silly.

@kratzky There are also edits to imports and corrected method calls in the code. I ran the tests, it works correctly. Can you merge


Why is my bridge loaded then unloaded (iOS) ?

I'm using djinni and encounter a problem on iOS only.
When starting the app from XCode, my bridge is initialized... then destroyed with the following stack trace:
#0	0x000000010837a79a in djinnius::HelloWorldImpl::~HelloWorldImpl() at /Users/omatrot/Documents/b2bot/sandbox/react-native-djinnius/src/cpp/hello_world_impl.cpp:23
#1	0x000000010837a885 in djinnius::HelloWorldImpl::~HelloWorldImpl() at /Users/omatrot/Documents/b2bot/sandbox/react-native-djinnius/src/cpp/hello_world_impl.cpp:22
#2	0x000000010837be09 in std::__1::__shared_ptr_emplace<djinnius::HelloWorldImpl, std::__1::allocator<djinnius::HelloWorldImpl> >::__on_zero_shared() at /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/memory:3610
#3	0x00000001086020c1 in std::__1::__shared_count::__release_shared() [inlined] at /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/memory:3444
#4	0x00000001086020b8 in std::__1::__shared_weak_count::__release_shared() [inlined] at /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/memory:3486
#5	0x00000001086020b8 in std::__1::shared_ptr<djinnius::HelloWorld>::~shared_ptr() at /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/memory:4412
#6	0x00000001086022a2 in std::__1::shared_ptr<djinnius::HelloWorld>::~shared_ptr() [inlined] at /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1/memory:4410
#7	0x000000010860229a in djinni::ProxyCache<djinni::CppProxyCacheTraits>::Handle<std::__1::shared_ptr<djinnius::HelloWorld>, std::__1::shared_ptr<djinnius::HelloWorld> >::~Handle() at /Users/omatrot/Documents/b2bot/sandbox/react-native-djinnius/deps/djinni/support-lib/objc/../proxy_cache_interface.hpp:137
#8	0x0000000109426920 in object_cxxDestructFromClass(objc_object*, objc_class*) ()
#9	0x0000000109432502 in objc_destructInstance ()
#10	0x0000000109432539 in object_dispose ()
#11	0x000000010943ca6e in objc_object::sidetable_release(bool) ()
#12	0x000000010aa9f64d in -[__NSDictionaryI dealloc] ()
#13	0x000000010943ca6e in objc_object::sidetable_release(bool) ()
#14	0x000000010943d1b2 in (anonymous namespace)::AutoreleasePoolPage::pop(void*) ()
#15	0x000000010aa86136 in _CFAutoreleasePoolPop ()
#16	0x0000000108e68e16 in __NSThreadPerformPerform ()
#17	0x000000010aadebb1 in __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ ()
#18	0x000000010aac34af in __CFRunLoopDoSources0 ()
#19	0x000000010aac2a6f in __CFRunLoopRun ()
#20	0x000000010aac230b in CFRunLoopRunSpecific ()
#21	0x000000010839403b in ::+[RCTCxxBridge runRunLoop]() at /Users/omatrot/Documents/b2bot/sandbox/react-native-djinnius/node_modules/react-native/React/CxxBridge/RCTCxxBridge.mm:246
#22	0x0000000108e313b3 in __NSThread__start__ ()
#23	0x000000010fb8b661 in _pthread_body ()
#24	0x000000010fb8b50d in _pthread_start ()
#25	0x000000010fb8abf9 in thread_start ()

It is then unavailable to the Javascript side.
What could be the case of this problem ?
Thanks in advance.

From context your issue sounds quite specific to React Native, not to the Djinni tool in general.  Did you pull your code from a Djinni fork with React Native support?  Perhaps this one?  If so you should report issues over there, since the Djinni tool has no specific support for React Native.


Feature: Add function to validate that a float conforms to the encoding scheme

(From PR discussion.)
For robust handling of encoded data in a CV stream, it could be useful to have a function in binary-to-float with the signature:
bool isValidByteEncoding(float f);

This doesn't mean that it is a float resulting from encoding -- only that it is valid in the schema. it may be that this isn't that useful in practice because few floats are outside the schema (not sure about that -- NaNs and infinitites and maybe negative zero), but it's something we can at least use internally in asserts and tests.

So the schema uses the sign bits and 23 low bits and sets the exponent to a constant below the sign bit.
That means there's two type of valid:
will be GENERATED is any binary value with the exponent
will be PARSED is any float since we don't check the exponent on input
so which one we choose to mean with this is tricky. I think we mean "couldHaveBeenGeneratedByEncoding" which really means "exponent bits are set to what we think".
Also do we want to make decoding a float with the exponent wrong fail? Or assert bad debug state at least? Probably since we use that range for the sentinels in the protocol

I think there's a middle approach. I don't think decoding should fail if the floats are little unexpected -- just the current "you get what you get" is ok I think. Although we might have a flag in the decoder that tells the user when some anomalous values were seen. Sentinel values are a valid part of the protocol, but not of the encoding, so there is a place for two levels of validation function -- one that checks known sentinels and another for purely data values.  For users that care about accuracy, they can validate as they wish or check the 'anomalous data' flag.
Decoding will just fail if sentinels get messed up. Having odd bytes in a message, will often be meh if you're just sending them to a marquee, but something a module may want to pay attention to if it's interpreting the text (e.g. notes notation). The difference is up to the client -- the protocol should merely provide the affordances for clients to implement the policies that make sense for them. maybe when initializing the decoder you can select a policy for anomaly handling (strict, omit-anomalies, relaxed).
Just thinking out loud...

So one thing I was thinking overnight but can’t do this am is to check the actual range of the encoded data. We have a test that it is in -10,10 but where is it really? The reason I want to check this is because I kinda want to change the sentinels to all be in -10,10 if we have room so the message will survive clipping. Just adding this comment here so I don’t forget it before some am travel even if it is a bit out of place

So based on your reaction to #12 we can add two methods
isVakidDataFloat in binary which basically makes sure you are bound between min and max and not inf or nan
isVakidProtocolFloat in protocol makes sure you are a valid data float or you are exactly one of the sentinels
I think those are the appropriate methods from where we are now.

You've understood me perfectly!  The PR with the const, ranges, and tests achieved everything else but the validity functions (unless I skimmed over them)..

Great I'll add them into that PR also then and push.

And pushed and tested. Woohoo!


Sample wanted for  Kodein-Framework/Kodein-DI

About the library

Description:    Painless Kotlin Dependency Injection
GitHub: https://github.com/Kodein-Framework/Kodein-DI
Official website: https://kodein.org/di/

What the sample could do
Definitely not the meaningless example of the fancy coffee machine with a thermosiphon.
Try to find something simple you could use in the real world.
Wants to contribute the sample?

Say so in a comment here
Follow instructions in CONTRIBUTING.md
Submit your pull request!

Hi @jmfayard, I would like to work on this 💪

Alright @douglascamata, let's go! 👌


Separate grouping logic

We now have quite a few transformers that have a by parameter to apply transformations within groups, for instance:

feature_extraction.Agg
feature_extraction.TargetAgg
feature_extraction.Differ
impute.StatImputer

I'm thinking that it might be a good idea to put the logic of grouping data in a separate transformer. If we overload another operator, say @, then we could do the following:
agg = feature_extraction.Agg(on='x', how=stats.Mean()) @ ('foo', 'bar')

instead of:
agg = feature_extraction.Agg(on='x', how=stats.Mean(), by=['foo', 'bar'])

The advantage is that each estimator doesn't have to handle the grouping logic by itself. This would simplify existing implementations and remove some repetitive code. Another advantage is that we could then do grouping with every estimator, which adds a lot of room for creativity and expressiveness. The disadvantage is users have to be familiar with this operator, but I don't think that's a bad thing, seeing that we're committing to using operator overloading anyway.
@creme-ml/developers what do you think?

Do you mean overloading base.Transformer with something like:
def __matmul__(self, by):
    self.by = by if isinstance(by, (typing.List, typing.Tuple)) else [by]
    return self

Nope. I mean that the operator would create a new transformer that contains the grouping logic. For instance it could be called GroupTransformer. It's responsibility would be to store one transformer for each value of the by attribute. In other words, feature_extraction.Agg would not have to worry about taking care of grouping because that would be handled in GroupTransformer.

Thanks for elaborating Max, that's interesting, would definitely be a helpful abstraction on our side :+1:
On the user side I'm more troubled, I think it makes things more complex by decomposing the abstraction (it requires to create and assemble two objects instead of one). I'm worried it would open the door to usage mistakes and decrease readability

Yeah I agree. Note that users can also use the following syntax:
agg = GroupTransformer(
    estimator=feature_extraction.Agg(on='x', how=stats.Mean()),
    by=['foo', 'bar']
)

I guess we need more input from other people :)

I think it's a great idea.

Yep would be nice to hear more opinions! We could also just separate the grouping logic in the internal api by inheriting from a GroupTransformer class

I've implemented something here.


broken short path links on docs.deis.io

Details

Project: deis

Short path links return a 404 OK, but the full path works. As far as I know we haven't touched anything on our end. Is there something we can do to fix this?
Thank you! :cherry_blossom:

Assuming you meant the second link to be:
http://docs.deis.io/en/latest/installing_deis/quick-start.html
In that case, you're describing the difference between HTML and HTML directory Sphinx builders:
http://www.sphinx-doc.org/en/stable/builders.html#sphinx.builders.html.StandaloneHTMLBuilder
http://www.sphinx-doc.org/en/stable/builders.html#sphinx.builders.html.DirectoryHTMLBuilder
RTD subclasses these, there is an option in your project settings to handle this. Might this have changed?

Thanks! It wasn't immediately apparent to me that "Sphinx HTML" and "Sphinx DirHTML" are two different build types.


[bug] Programs launched with administrator privileges cannot respond to file drag and drop events

Describe the bug
I have a tauri application that can receive file drag and drop messages when launched with standard permissions.
But when I started with administrator privileges, I was unable to receive messages. I followed the implementation method of other applications and proactively modified the Win32 message filter. But still ineffective
mycode
   .setup(| app | {
            let window: Window = app.get_window(MAIN_LABEL).unwrap();
             let hwnd = window.hwnd().expect("Failed to get HWND").0 as *mut _;
            app.listen_global("app-exist",  move |_| app_exist(Some(&window)));
            let success = unsafe { ChangeWindowMessageFilterEx(hwnd, WM_DROPFILES, 1, std::ptr::null_mut()) };
            if success == 0 {
                panic!("Failed to change message filter for WM_DROPFILES");
            }
            let success = unsafe { ChangeWindowMessageFilterEx(hwnd, WM_COPYDATA, 1, std::ptr::null_mut()) };
            if success == 0 {
                panic!("Failed to change message filter for WM_COPYDATA");
            }
            let success = unsafe { ChangeWindowMessageFilterEx(hwnd, 0x0049, 1, std::ptr::null_mut()) };
            if success == 0 {
                panic!("Failed to change message filter for 0x0049");
            }
            Ok(())
        })

What should I do?
Reproduction
No response
Expected behavior
4
Full tauri info output
3

Stack trace
2

Additional context
1

Duplicate of https://github.com/tauri-apps/tauri/issues/8218


Add setting for switching between front and back camera

Camera switching is already hooked up in the backend of the code, now it just needs a setting for it

I will be just giving a list of numbers from 0-10. 0 is back camera in most cases


Don't work in iOS11

I used it in xcode9, when I run it on iOS11, it cannot switch and sliding.

Argument of '#selector' refers to instance method 'handleMenuItemTap' that is not exposed to Objective-C

Solution on #112 fixes problem with scrolling pages on iPhone X.


[TASK] OutStreams as extensible Entity

Issue Description
New custom Print and Plot algorithms could be added as easily as e.g. PostProcessors if we converted them to being the same as other Entities in RAVEN. Right now they are the exception to a lot of algorithms in the Simulation and Steps modules when they probably don't need to be.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or task?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[x] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

Issue closure list approved...
Closure approved via #1329


Node synchronization - Replay

Description
Tasks necessary for synchronizing nodes from peers via replay
### Tasks
- [ ] Subtask: create specification
- [ ] Subtask: support replay
- [ ] https://github.com/gofractally/psibase/issues/234

Status here @swatanabe ? Replay is sufficiently complete, no?


create exported function for resizing disk

libhvee should have a lib function available to resize a vhd file ...

Haven't tested the code, but it can replace what we had and compilation works https://github.com/cfergeau/crc/commit/a02b9122823aec3b470bcad5a7bc168e17493058 (I think I'll develop a love/hate rship with strongunits, looks like a better docker/go-units, but with its own set of constraints to learn to work with :p )


Handle EWOULDBLOCK on Windows

Winsock don't set errno (ref. https://msdn.microsoft.com/en-us/library/windows/desktop/ms737828(v=vs.85).aspx), and defines the value under a different #define variable.
This PR handles both the unix-way and the windows-way to detect an EWOULDBLOCK condition.

This is fixed now as a separate wouldBlock() function. Thanks, I must have removed some kind of #define errno WSAGetLastError() or it has always been broken, don't know which? Thanks for the PR.


Update Breadcrumb component to improve screen reader accessibility

An accessibility issue raised during development of #4950 was that, when a Breadcrumbs component is placed following a Header or Service Header that contains navigation elements, they are both announced (by screen readers) or displayed (in no CSS/reader modes) as a bulleted lists of links, with little affordance to users that they are two distinct navigational elements.
This PR aims to resolve the screen reader issue by adding an aria-label of "Breadcrumb". It does not solve the display issue. Doing so would require the inclusion of some visible or visually-hidden text, judging by @selfthinker's comment on the GOV.UK Publishing Components version: https://github.com/alphagov/govuk_publishing_components/pull/2045#issuecomment-1803596065, though we could choose to modify this PR to do that as well.
As aria-label is only supported in practice by interactive elements and landmarks, the breadcrumb component wrapper has been changed to a nav to make it a landmark element. Doing so also resolves a common accessibility audit complaint that this component does not exist within a landmark (e.g. https://github.com/alphagov/govuk-frontend/issues/1604). This is not strictly required to meet WCAG, but may be considered good practice.
Background
The breadcrumb navigation was previously contained within a nav element, but this was removed in 2013 in an effort to reduce the number of navigation landmarks that appear on the page, as documented in this blog post by @tombye.
However, I think that the Breadcrumbs do not meet the criteria for removing nav as given in the post—the individual links do not provide sufficient labelling on their own and the components' role as a navigational aid can only be inferred through visual context which is not available to users who are blind or with low vision, nor assistive technologies they may use, and for which no fallback is provided.
Compare Breadcrumbs with other nav elements it's common to find on GOV.UK:

Primary navigation has an accessible label and can have context inferred by being within the header (banner) landmark and near the top of the page.
Links in the page footer have visible headings and can have context inferred by being within the footer (contentinfo) landmark.
Most other nav elements that appear on GOV.UK have visible headings (e.g. related content lists), accessible labels (e.g. pagination), or can have some context inferred from their parents (e.g. table of contents has a complementary role and is one of the first things inside of the main landmark). Most provide multiple of these affordances.
Skip links and back links don't typically exist in groups and can have their contexts largely by their labelling.

Breadcrumbs currently have none of these: they lack the nav element, they lack headings (visible or not), lack accessible labelling, their context cannot be inferred from a parent landmark because they don't have one, and it cannot be inferred from the links themselves as (apart from 'Home') they are disembodied section names.
With these changes, our component is much more in line with the example in the WAI ARIA Authoring Guide and the W3C's design system component (which is heavily inspired by ours, their deviation implying they felt we were in the wrong here). The same approach is also used by our friends at ONS, Scottish Government Design System and USWDS, really putting us in the minority with our current approach.
Changes

Changes the wrapping element of the Breadcrumbs component from a div to a nav.
Adds a required aria-label to the Breadcrumb component wrapper.
Adds a new Nunjucks parameter to set the aria-label named labelText. This defaults to "Breadcrumb".
Adds Nunjucks parameter documentation for labelText.
Adds tests to ensure the element is a nav, renders the default aria-label, and that the Nunjucks parameter can override the aria-label.
Updates one existing example and test that set role="navigation", as this was being flagged by the HTML linter as redundant role usage.

I generally agree with Tom's initial assessment back in 2013 that there were way too many navigation landmarks on the page. Although they had worked on removing some, I assume some have moved back in or were added later as www.gov.uk still uses way too many, lots of which should perhaps be removed. (For example, tax codes has 6 navigation landmarks, at least half of which are probably not necessary.)

Breadcrumbs currently have none of these: they lack the nav element, they lack headings (visible or not), lack accessible labelling, their context cannot be inferred from a parent landmark because they don't have one, and it cannot be inferred from the links themselves as (apart from 'Home') they are disembodied section names.

That is a very good point. A point that makes me think maybe this is a good idea.
Something that is a bit problematic is that when breadcrumbs are used on www.gov.uk, they are often not very useful as the website's hierarchy is presented as pretty flat. Therefore the breadcrumbs often only have a "Home" link and then a link to the page's general topic.
I don't know enough about how breadcrumbs are used on services to know if they are more useful there. But I can imagine simple services won't need them at all.
But I guess that is not the problem of the Design System. Or if it is, it's something for guidance, not code.
Overall I'm happy with this change.
The only thing I would like to get input from a content designer for is the label "Breadcrumb".
I would assume that the majority of people won't know what breadcrumbs are, including screen reader users. Or maybe screen reader users know the meaning of the word better than most other users because it is used on lots of websites to label them?
Unless "breadcrumb" explains it better to people (in this case specifically screen reader users), I would try to avoid jargon and explain in plain English what it is or what it does. I like "You are here", but a content designer might have a better idea.

Also, directly related to #4451.

Thanks @selfthinker. I think I've got all the context. I’m not sure I can do better than W3C’s example that just calls it “Breadcrumbs”
Something like "You are here" is tricky because 'here' could be easily misinterpreted to mean any of the breadcrumb links.
If anything, there's the idea of "Page levels" or "Page levels for this page", but that's only slightly less jargon-y but also imprecise as it might be mistaken for something concerning the page itself and overemphasise its importance.

@calvin-lau-sig7, I've tried to google alternative names and also if there has been any research about the name with screen reader users but couldn't find anything for the latter and only very little for the former.
Other than the previously mentioned "You are here" which is used in a couple of places, I also found "Navigation Path". But that's it, nothing else.

I will ask the community, maybe someone knows of any user research in this area.

Thanks for the investigtion @selfthinker!
Continuing our musings, something like "Page level path" could be a bit more descriptive, as long as it's not mistaken for the user's back history.
I might need to better understand the semantics for "You are here"... would the ARIA tag be understood to refer to the whole list, without too much risk of being mistaken for one of the links in the list?

We had similar thoughts when we introduced the pagination component and how we should accessibly label that.
In that instance, we stuck with 'pagination' because, even though it's probably more niche terminology than necessary, it was the labelling that the overwhelming amount of other examples we could find also used for their accessible labels, so it seemed likely that assistive technology users would have encountered it many times before and probably understood what it referred to.
That's just anec-data though, and we didn't specifically test that assumption.

I might need to better understand the semantics for "You are here"... would the ARIA tag be understood to refer to the whole list, without too much risk of being mistaken for one of the links in the list?

Yes, it would refer to the whole list, or rather to the navigation landmark the list is in. That can be discovered by navigating by landmark or arrowing through the page. It would then announce something like "Breadcrumb, navigation" before it gets into the list, so clearly not a link.
Although when directly navigating to a link in the breadcrumbs, it also announces the context, including the landmark. But it would do that in a way the context is clearly announced. Not sure if it might be possible for a less proficient screen reader user who is not used to landmarks and how they present to confuse them with being part of a link in that case. I don't think that would be very common.

It would then announce something like "Breadcrumb, navigation" before it gets into the list

Just clarifying... It would read "[whatever the aria-label says], navigation", so "You are here, navigation" in the other case. (And some screen readers also add "region" or "landmark" at the end.)

The way that Breadcrumbs is currently often used has it only list the ancestors of the current page, but omits the current page (as that is given by the h1). That makes me unsure whether "You are here" would necessarily be an accurate description of the landmark's content.

The way that Breadcrumbs is currently often used has it only list the ancestors of the current page, but omits the current page (as that is given by the h1). That makes me unsure whether "You are here" would necessarily be an accurate description of the landmark's content.

Okay, @querkmachine's point here is a strong one – breadcrumb content often does not actually indicate the current page, instead only showing the earlier or higher-level pages.
Based on this I'd agree we should keep it to "Breadcrumb", as we can't rely on the breadcrumb being a reliable indication of "here".
I'm open to figuring out a more plain English label amongst us that'd work through. E.g. How you got here (if that were accurate).

I'm going to merge this given it's had approval and the consensus seems to be that current default labelling is sensible, even if not ideal. We can still update it later in the cycle without issue.


Follow up to improve gdrive and move auth before service accounts

Followup based on https://github.com/iterative/dvc.org/pull/3473#issuecomment-1112659711

You may disregard these recommendations if you used the Edit on GitHub button from dvc.org to improve a doc in place.

❗ Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.
🐛 Please make sure to mention Fix #issue (if applicable) in the description of the PR. This causes GitHub to close it automatically when the PR is merged.
Please choose to allow us to edit your branch when creating the PR.
Thank you for the contribution - we'll try to review it as soon as possible. 🙏

Looking at it again, I think "Authorization" should even be above "Using a custom Google Cloud Project" (sorry for not catching this earlier). So the structure would be:

Quick start
URL format - provides more detail on the quick start scenario
Authorization - provides more detail on the quick start scenario and introduces auth options for advanced scenarios below
Using a custom Google Cloud Project - explains advanced best practice that is needed to use service accounts
Using service accounts - explains advanced best practice for automation

Thoughts?
Happy to make the changes myself if it's easier once we all agree.

Looking at it again, I think "Authorization" should even be above "Using a custom Google Cloud Project"

I think that was intentional, so that it's not considered as advanced but rather as recommended (in most cases you would want to have an independent app so that your rate limits don't overlap with other DVC users).
Also, it's an optional configuration step, while Auth follows both of them.
So, I don't see changing the order as a fixing something step, but I'm fine with that if you guys decide to this.

Sounds good. I'm not sure the GCP -> Auth -> Service Accounts order is any better than the current GCP -> Service Accounts -> Auth order, so I'm open to closing, merging, or reordering as Auth -> GCP -> Service Accounts. @jorgeorpinel If you have any opinion, happy to go with that.

Sounds like service accounts are "more advanced" so it makes sense to have them last. If it's the recommended method then maybe we should be more explicit about that and move them back up. But it's probably not worth going back to it now... We have to maintain this periodically anyway so I'm sure we'll have another chance in the future.

TBH ideally I'd prefer if we didn't have to be so specific with 3rd party tool docs.


Add Proselint, a linter for English prose

I think it's a great addition to the list, great tool (I am even thinking about maybe trying to contribute to it) but I worry that "English" is not sufficiently descriptive to allow for a clear meaning.
Spelling for example is what worries me the most because I believe it checks for American spelling and not British spelling for example and there may be confusion there.
What do you think if we organize it like "English - American" instead of just "English"?

@caramelomartins We've recently switched over to using the latest version of Bryan Garner's usage guide, Garner's Modern English Usage, as our gold standard. It has an expanded scope that covers both American and British English. To the best of my knowledge, the only rule in Proselint that will flag British spellings is consistency.spelling, which will point out if you've used a word in both it British and American spellings and, importantly, complain about whichever form you used less often. And so I think English is sufficiently broad to cover Proselint. It would be reasonable to create separate listing for English (American) and English (British), but then I'd argue Proselint should go in both categories.
I also know of several other linters for English that I can contribute to this list if you'd like.

@suchow agreed. The fact that we divide might actually be confusing because one wouldn't know where to catalogue a given linter if it supported several dialects...I will merge the PR as it is.
More contributions are very welcome, be it English or any programmatic language! I am going to try and develop a PULL_REQUEST_TEMPLATE.md to help with PR submission.


Add WEMIX3.0 Mainnet & Testnet

@comverser please accept the CLA 😉

recheck

I have read the CLA Document and I hereby sign the CLA

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
255 unchanged lines in 10 files lost coverage.
Overall coverage decreased (-5.9%) to 84.553%

Files with Coverage Reduction
New Missed Lines
%

gnosis/safe/addresses.py
1
0%

gnosis/eth/clients/etherscan_client.py
3
95.59%

gnosis/util/util.py
3
66.67%

gnosis/eth/multicall.py
9
88.35%

gnosis/eth/oracles/superfluid.py
9
55.0%

gnosis/eth/oracles/kyber.py
11
59.18%

gnosis/eth/oracles/cowswap.py
22
35.14%

gnosis/eth/oracles/uniswap_v3.py
28
56.34%

gnosis/eth/ethereum_client.py
38
81.52%

gnosis/eth/oracles/oracles.py
131
50.46%

Totals

Change from base Build<PHONE_NUMBER>:
-5.9%

Covered Lines:
3525

Relevant Lines:
4169

💛  - Coveralls

I found typo mistake. Please let me fix it...

Do the block explorers you add share the same API as Etherscan? It doesn't look so

Do the block explorers you add share the same API as Etherscan? It doesn't look so

@Uxio0, It is not the same API as Etherscan. However, I included it with the thought that it may support the Etherscan format in the future. Should I remove it from the list?

@Uxio0, It is not the same API as Etherscan. However, I included it with the thought that it may support the Etherscan format in the future. Should I remove it from the list?

Yes please, when it supports etherscan API (or blockscout API) you can open a PR again

@Uxio0, I appreciate your feedback and I would like to let you know that I have made the correction.

Could you rebase and resolve conflicts?


Question: where are the starter files ?

where can i find the starter files that are being used with helm create?
i would like to use them as a start point to some customization i require.
Thanks
P.S
for clarification I am looking for something like https://github.com/uptownaravi/helm-starter
but official

@asafbennatan maybe helm create -s?

getting Error: unknown shorthand flag: 's' in -s

@asafbennatan oh. sorry.
-p, --starter string

i got what you mean , i am aware of this flag i plan to use it with a starter i will write , i want to base that starter based on the existing template we get by the default helm create

@asafbennatan I got. you can alias helm create -p test/test?

but how will that help ? , i am looking for similar files that we have in https://github.com/uptownaravi/helm-starter ( those are not that same as what helm create outputs since it contains CHART_NAME )

Are you looking for https://github.com/helm/helm/blob/main/pkg/chartutil/create.go ?

@joejulian exactly , Thanks!


How to refresh QR code?

Suppose calling await driver.qr() times out . How to refresh the qr code ? Is there an api to refresh the qr ?

When the QR expires and need to be clicked, do something like this

Thanks! That solves my problem.


ValueError: cannot reshape array of size 60817344 into shape (1981161,512)

import pandas as pd
import numpy as np
Load the photo IDs
photo_ids = pd.read_csv("unsplash-dataset/photo_ids.csv")
photo_ids = list(photo_ids['photo_id'])
Load the features vectors
photo_features = np.load("unsplash-dataset/features.npy")
Print some statistics
print(f"Photos loaded: {len(photo_ids)}")

After I ran this cell, the error occurred.

The problem seems to be that features.npy is not fully loaded. Can you check if the downloaded file is about 2 GB? If not, please delete it and download again.

The problem seems to be that features.npy is not fully loaded. Can you check if the downloaded file is about 2 GB? If not, please delete it and download again.

Thanks.

Thanks.