Necessary to double click list title to edit

You have to double click on the list title to edit it when there's a WiP limit (except if you happen to click on the badge with the count).
Expected behavior is that single click enables editing.

Fixed


[python] string representation of proto message incorrectly displays UTF8 chinese chararacters

Hello,
Here is what I see when printing a proto object received through grpc having UTF8 string as one of the fields:
field1: "...'"
oerrmsg: "\345\275\223\345\211\215\347\212\266\346\200\201\347\246\201\346\255\242\346\255\244\351\241\271\346\223\215\344\275\234"
field3: "..."

I'd expect to see that
field1: "...'"
oerrmsg: "当前状态禁止此项操作"
field3: "..."

I suppose the __print__ and __repr__ methods should be adjusted...?
Thank you for your help,
note: Ubuntu 16.04, python<EMAIL_ADDRESS>

This smells like a Protocol Buffers problem - are you able to reproduce this problem using only Protocol Buffers and without gRPC at all?

It is indeed a proto related issue: https://github.com/google/protobuf/issues/3125


Sets consistant render scale

One and only change for this pull request. Our team here was running into an issue while using this library with image snapshot tests.
Essentially the render function will render the components at different sizes if you have the scale set differently from device to device, causing image snapshot tests to fail depending on what device you run the tests on.
This change forces the electron environment to keep a consistent scale factor, regardless of what scale the os is set to.
Thank you for your time. Please let me know if you have any questions, concerns, or comments

that looks good! thank you!
how do you do image snapshot tests with it?

published<EMAIL_ADDRESS>

In our jest.electron-runner.setup.js file, we define the function like
global.getComponentSnapshot = component =>
  new Promise((resolve, reject) => {
    const win = remote.getCurrentWindow();
    const { container } = render(component);
    container.setAttribute("style", "width: fit-content");
    const { x, y, height, width } = container.getBoundingClientRect();

    delay(100)
      .then(() => {
        win.webContents.capturePage({ x, y, height, width }, image => {
          resolve(image.toPNG());
        });
      })
      .catch(reject);
  });

so then in our tests we can
describe("Table", () => {
  it("matches the snapshot", async () => {
    const image = await getComponentSnapshot(<Foo />);
    expect(image).toMatchImageSnapshot();
  });
});

👏

@amatthews4851 that is awesome! it'd be nice to have it in open source. or maybe even make it part of this runner


Support native extensions for gems specified as paths

Using local gem directories using :path is very useful for gem development. This is no exception for gems that provide native extensions. However, bundler in its current form doesn't allow compiling C extensions for gems specified as paths, according to the Gemfile manpage:

Unlike :git, bundler does not compile C extensions for gems specified as paths.

This is presumably because we don't want bundler creating files within the local gem's directory. It would be nice if there was a way to tell bundler to force compilation of C extensions in these cases:
gem "mygem", :path => "../mygem", :compile => true
# or
gem "mygem", :path => "../mygem", :extensions => true

If you think this is too dangerous, then it might be useful to have a :build option, so that doing bundle install would automatically gem build and then install those gems. Likewise, bundle update would delete all :build gems and re-build and re-install them.
gem 'mygem', :build => "../mygem"
# or
gem 'mygem', :path => "../path", :build => true

Seven years later I resume that project and I still have the same problem. I need a vendored gem (because the git repo is private) which has extensions and bundler doesn't build them 🤷♂

I've solved it by putting the packaged gem in /vendor/cache


WIP Tests covering base functionality inmplemented.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@4704976). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #24   +/-   ##
=======================================
  Coverage        ?   77.77%
=======================================
  Files           ?       11
  Lines           ?      396
  Branches        ?        0
=======================================
  Hits            ?      308
  Misses          ?       88
  Partials        ?        0

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 4704976...8b1eaf2. Read the comment docs.


POLYGON/OVER makes incorrect title

Reported by ansley b manke on 1 Apr 2016 22:15 UTC
If there's a 3rd argument to the POLYGON command for the colors of the polygons, then the title for the command is given by that variable's title or definition.
If there are just 2 arguments, so the polygon is just drawing a shape, there should not be a title.
This script makes the plot box using the SHADE command, then draws the polygon and the label at the lower left is using the title for the variable ALLMISSING used in the SHADE
yes? set data levitus_climatology
yes? let allmissing = 0/0 + temp
yes? shade/nolabel/y=0 allmissing
yes? polygon/palette=gray/ov {50,180,180,50},{4000,4000,1000,1000}

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2396

Modified by ansley.b.manke on 14 Apr 2016 17:36 UTC


UnityTimeOutException

I am having the exact same issue as issue #44.
Would really appreciate if someone can help me with this.
System: Windows 10
ObstacleTower: v1.1
When I run env = ObstacleTowerEnv('./ObstacleTower/obstacletower', retro=False) after installing the requirements I see the following:
The unity splash-screen is displayed, then a tan background appears with the main character briefly in view. She then falls off the screen leaving only the tan background.
Thanks!

Try the old version of the environment (v1) it worked for me.
Download it at https://storage.googleapis.com/obstacle-tower-build/v1/obstacletower_v1_windows.zip
for windows

I have the same issue on win10 - I was able to get the v1 training. I'm also able to run v1.1 and play manually

Same here. I updated everything but can only play the v1.1 .exe manually, cannot load it as a UnityEnvironment. The old version (v1) works, but in that version the character cannot jump over walls/blocks as opposed to v1.1

Hi all,
Can you try this Windows binary and let me know if it works? This resolved the issue for me locally, but I want to ensure that is the case with everyone before pushing it officially: https://storage.googleapis.com/obstacle-tower-build/v1.1/obstacletower_v1.1_windows.zip

Works for me. It displays a blank window for the game though, is that expected?
Thanks every1 for the reply!

@awjuliani  the new build display a blank window for me as well:

it is working for me - I get the blank window too (but I think I got this on v1) - also it says its a development build so optimization maybe off. I'll leave it running overnight

Yes! I also confirm everything seems to run fine so far, thank you. The blank window is also common for me both in v1 and this new version.

It would be helpful if we can see what the agent is doing @awjuliani .
Can you see the game running in other environments?

i am getting "Average training steps per second: 114.58"

with a 1080

Hi all,
Glad to hear that the new binary resolves the issue for people. The window is black because we disable cameras by default in order to not render frames twice (once to the agent and once to the window). This provides a speed up. We will implement an inference model however so that people can monitor training at realtime with a rendered window.

@awjuliani can you push a release version of v1.1.1 - the one in the link is 'development build'

thanks @awjuliani

@awjuliani can we have the camera issue fixed? I'm getting a black screen right now.

@lezwon The screen is supposed to stay black
@awjuliani Could you provide a fixed Linux build as well?

@MarcoMeter I'd like to see what the agent is doing. Is it possible to enable the cameras?

While using this new development build, calling env.render() does not do anything. I think that its an issue that only happens while running it on windows. Any ideas?

Hi all,
We have an open PR to enable rendering to the environment window and running at realtime to observe training. That is available here: https://github.com/Unity-Technologies/obstacle-tower-env/pull/50. We plan to add it in a new release of the package in the next day or so.

Hi all,
We’ve pushed a v1.2 release of the obstacle-tower-env which allows you to run in “real time mode” https://github.com/Unity-Technologies/obstacle-tower-env/tree/v1.2. We also pushed new binaries for Windows and Linux which should address the issues people have been having: https://github.com/Unity-Technologies/obstacle-tower-env/tree/v1.2#download-the-environment


OS: Feat: Add list of reasons why TOP doesn't support Windows

Because
#547
This PR

Updates !os and /os to link to the list of reasons TOP doesn't support Windows
Updates the test snapshots

Issue
Closes #547
Additional Information

Pull Request Requirements

[x] I have thoroughly read and understand The Odin Project Contributing Guide
[x] The title of this PR follows the location of change: brief description of change format, e.g. Callbacks command: Update verbiage
[x] The Because section summarizes the reason for this PR
[x] The This PR section has a bullet point list describing the changes in this PR
[x] If this PR addresses an open issue, it is linked in the Issue section
[x] If this PR adds new features or functionality, I have added new tests
[x] If applicable, I have ensured all tests related to any command files included in this PR pass, and/or all snapshots are up to date

Abandoning this in favor of just submitting a new one given the changes made in #556


Remove trailing semicolon from bt dft initializer (IDFGH-8058)

Similar to espressif#6554

Thanks for your contribution.


Remove config for mint

https://github.com/liteflow-labs/starter-kit/pull/310
Instead of having an option to disable the minting, if an application does not require minting then they can simply remove the links as they will anyway have to remove the page as well

@EmmanuelDrouin @NicolasMahe I would love your feedback on that, as this will be a breaking change. You can approve if you're good with this change, comment if not


Make maxLeaseId / maxUID an Option

Fixes #215 in the sense that no exception is thrown and data can be read from Dgraph. However, partitioning will be limited to predicate partitioning, which is poor.

@matthewmcneely I have published this fix as a snapshot, please test it in your notebook with
conf.set("spark.jars.packages", "uk.co.gresearch.spark:spark-dgraph-connector_2.12:0.10.0-3.3-SNAPSHOT")
conf.set("spark.jars.repositories", "https://oss.sonatype.org/content/repositories/snapshots")

This should then be able to load the graph, but won't work well for large graphs.


dep: libxml 2.12.0 upgrade (with libxml2 patch for weakref)

What problem is this PR intended to solve?
#3031
This approach uses the patches submitted upstream at:

https://gitlab.gnome.org/GNOME/libxml2/-/merge_requests/229
https://gitlab.gnome.org/GNOME/libxml2/-/merge_requests/230

OK, so even with the patches applied, we're still pulling in ld-linux-x86-64.so.2 because the new thread local storage feature ("TLS") results in a reference to __tls_get_addr that is non-weak. You can read more about this at https://maskray.me/blog/2021-02-14-all-about-thread-local-storage
Turning off TLS by configuring libxml2 with --without-tls avoids this problem. We'll try that next.

CI didn't kick off, so the manual run is https://github.com/sparklemotion/nokogiri/actions/runs/6916120729

OK, so even with the patches applied, we're still pulling in ld-linux-x86-64.so.2 because the new thread local storage feature ("TLS") results in a reference to __tls_get_addr that is non-weak. I am assuming something similar is happening on windows with libwinpthread.
You can read more about TLS at https://maskray.me/blog/2021-02-14-all-about-thread-local-storage
Turning off TLS by configuring libxml2 with --without-tls avoids this problem. We'll try that next.

This seems like the winner.


Failed to pull mussel docker image

it displays
(HTTP code 404) no such image - no such image: microsoft/mssql-server-linux:latest: No such image: microsoft/mssql-server-linux:latest

I think the old images should still be there, but that image is deprecated, and moved from https://hub.docker.com/r/microsoft/mssql-server-linux to mcr.microsoft.com/mssql/server:xxxxxx;l see https://hub.docker.com/_/microsoft-mssql-server

closing this ticket because of the above, but feel free to continue the conversation.


It is not documented how to generate the go bindings for the contracts in the BSC bridge

contract/token.go and mscontract/multisig.go are autogenerated but the docs are missing the command to do so.
Better yet, add a generate.go with the commands in there so we can regenerate with go generate

Added in: https://github.com/threefoldfoundation/tft/commit/96f0f957ba84577752ea05bfcbafbb3be3fda723

@DylanVerstraete can you add it in a generate.go file to be more in line with how these things are handled in go?

added in: 4d0565177852bd8f790e5fa37804522c94bd2e99


Add contributing guidelines

I want to add a CONTRIBUTING.md file to document how to contribute to this project.
File should include how to create a new lerna release.

https://github.com/americanexpress/parrot/pull/50#pullrequestreview-394780429

#56 has added a draft contributing.md


fix: Dependabot vulnerabilities alerts

Describe your changes
fix: Dependabot vulnerabilities alerts. I got 3 mail for this.

Fixes Issue

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

@melloware first one is got fixed, would you like to fix this one-https://github.com/quarkiverse/quarkus-quinoa/security/dependabot/80?


feat: detect typescript based on tsconfig.json

Refactor registerTSNode from @nuxt/typescript into @nuxt-cli. This way users can keep @nuxt-typescript (~55.2 MB) install size into devDependencies and ts-node (1.6 MB) in dependencies. This means much less production install size and also have runtime support.
Move external imports into cli/imports.js for better maintenance.
Only enable typescript for a project if tsconfig.json exists in the rootDir.
Only If tsconfig.json is empty JSON, defaults will be written into it by @nuxt/typescript.

We have to highlight that change in the release notes (as done before with changes on experimental features) :relaxed:

@manniL Sure. Actually it is a complement of #5367. New migration guide would be:

Remove nuxt-ts dependency
Install nuxt and @nuxt/typescript dependencies
Create an empty tsconfig.json file. Nuxt will automatically generate it with defaults at first run, then you can edit it to fit your needs.

Migration guide for 2.5.x projects should be just installing ts-node as dependency if they need runtime support.

@pi0 ts-node shouldn't be removed from package.json then ?

@kevinmarrec Migration guide updated. Does it looks good now? (Feel free editing my comment).

@pi0 ts-node shouldn't be removed from package.json then ?

For lazy people who only install @nuxt/typescript that would be harmless. But can be removed as well.

@pi0 perfect
Yeah I see, but if it's not used anymore within this package, we probably should remove it ?
It would indeed mean we need to warn users in release notes they need to install ts-node back when migrating to 2.6

@kevinmarrec Done. Dependency removed and guide updated. I had to add ts-node to nuxt mono-repo root package.json in order to CI passing

Alright perfect @pi0, worth mentionning that the guide is applicable for 2.4.x -> 2.6.x.
For 2.5.x -> 2.6.x it's only the ts-node step.
Don't know how exactly we're gonna wrap it up in release notes but yeah 🙈

Codecov Report

Merging #5412 into dev will decrease coverage by 44.49%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##              dev    #5412      +/-   ##
==========================================
- Coverage   96.17%   51.67%   -44.5%
==========================================
  Files          74       74
  Lines        2534     2539       +5
  Branches      644      646       +2
==========================================
- Hits         2437     1312    -1125
- Misses         81      946     +865
- Partials       16      281     +265

Impacted Files
Coverage Δ

packages/cli/src/command.js
0% <0%> (-98.98%)
:arrow_down:

packages/typescript/src/index.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/imports.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/utils/typescript.js
0% <0%> (-66.67%)
:arrow_down:

packages/generator/src/generator.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/index.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/options/server.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/run.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/commands/start.js
0% <0%> (-100%)
:arrow_down:

packages/cli/src/options/common.js
0% <0%> (-100%)
:arrow_down:

... and 41 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 91f4eb0...82cdf05. Read the comment docs.


Start Using SAP HANA 2.0, express edition (Virtual Machine Method)

I am confused on step 16 (Step 16: Optional Configuration: Install SAP Enterprise Architecture Designer (Server + Applications Virtual Machine Only)).  I located the eadesigner.tgz on my laptop, but I can't figure out how to get the file to my Virtual Machine directory.  Also, the command seems to be missing a step??  If I can somehow get the file to my Virtual Machine, it seems that I need to extract it.  How and where do I do this?
It is frustrating...  the steps up to step 16 are very detail, and with screen shots.  Step 16 seems to assume the user knows everything.  I need more detail please.

You should login as the hxeadm OS user and then download/sftp the files from your laptop. Extract the files and navigate into the HANA_EXPRESS_20 folder and you will find an installer file with the extension ***.sh. Run it and that will install ea designer.

We've expanded and simplified the EA designer installation instructions. The procedure now shows you how to download the EA Designer installer from inside the VM, using the built-in (command line) download manager utility. Hope this helps.


docs: use correct syntax highlighting tag type

Using analog as a tag type does not highlight the example code on https://analogjs.org/docs/experimental/sfc#host-metadata
Using html is fixing that issue.
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/analogjs/analog/blob/main/CONTRIBUTING.md#-commit-message-guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Other... Please describe:

Which package are you modifying?

[ ] vite-plugin-angular
[ ] vite-plugin-nitro
[ ] astro-angular
[ ] create-analog
[ ] router
[ ] platform
[ ] content
[ ] nx-plugin
[ ] trpc

What is the current behavior?
Using analog as a tag type does not highlight the example code on https://analogjs.org/docs/experimental/sfc#host-metadata
What is the new behavior?
Using html is fixing that issue.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Why is prettier failing? What is wrong?


MM-33748 Support for new mention_count_root and msg_count_root

Summary

Ticket Link

https://mattermost.atlassian.net/browse/MM-33748

@ogi-m please help prioritize this PR by smoke testing with the feature flag for collapsed reply threads off.

Ping @hmhealey on reviewing this asap please, thank you!

@reflog @hmhealey @ogi-m After firing this up again, I believe there's one UX issue remaining for multi-team users—only the current team loads initial thread counts (unreads/mentions), and that can produce inaccurate mention counts in the Team Icons and Title (favicon_title_handler).

@reflog @esethna I'm not able to review this right now since it looks like it needs to be rebased onto master. I assume it's not actually 7000 lines added at this point

@hmhealey sorry about that. A large PR that this was based on got merged yesterday.
This PR is ready for review now. Thanks in advance

Ping @calebroseland @hmhealey @ogi-m please help prioritize reviews here to unblock us starting to test the feature on daily

After a couple passes and some fixes, there're 3 items left on my radar:

Initial Load of multi-team Thread Counts (total_unread_replies and total_unread_mentions in ThreadsState['counts']).
These values pipe into favicon/title indicators and team icon indicators. With only current-team counts loading via GlobalThreadsLink, other-team values are not yet initialized and the indicators will be inaccurate.
WS events will increment other-team values after init load, but we have yet to get them initialized.
--
I tried to do a quick "80%"/beta-quality/hacked version using the existing getThreads api/action, but repeating this on a per-team basis in user actions loadMe and loginComplete is heavy. After a quick try, it seems it would be probably a clunky implementation.
--
I think we should add it to the existing Client4.getMyTeamUnreads and its mattermost-server counterpart, and then watch for RECEIVED_MY_TEAM_UNREADS in the ThreadsState['counts'] reducer to initialize other-team thread counts.

Favicon/Title unread indicator (via getUnreads) needs to factor in total_unread_replies in the same way it already factors in total_unread_mentions. (Currently, I believe the unreads dot behaves as though the feature is off.)

The team icon indicators need to subtract DM/GM-channel thread-reply unread reply/mention counts, otherwise this happens as you click through other teams.

@hmhealey I've address your comments. Please re-review.
@ogi-m please approve


Crash with OG Vive camera

Using the OG Vive (single camera) by manually setting the camera path in config causes a crash in either VR backend.
[2024-03-05T15:55:16Z INFO  index_camera_passthrough] width          : 612
    height         : 460
    fourcc         : YUYV
    field          : progressive
    stride         : 1224
    size           : 563040
    colorspace     : sRGB
    quantization   : default
    transfer       : Rec. 709 transfer function
...
[2024-03-05T15:55:16Z WARN  index_camera_passthrough] No camera parameters found
[2024-03-05T15:55:16Z INFO  index_camera_passthrough::pipeline] Adjusted FOV: [[1.19, 1.19], [1.19, 1.19]]
Error: copy_buffer_to_image_info: `regions[0].buffer_offset` plus the number of bytes being copied is greater than `src_buffer.size()` (Vulkan VUIDs: VUID-VkCopyBufferToImageInfo2-pRegions-00171)

Backtrace (backtrace.log):
...
[2024-03-05T15:25:41Z TRACE index_camera_passthrough] state: Capturing
[2024-03-05T15:25:41Z TRACE index_camera_passthrough] getting camera frame
[2024-03-05T15:25:41Z TRACE index_camera_passthrough] got camera frame Instant { tv_sec: 7818, tv_nsec: 405975511 }
[2024-03-05T15:25:41Z TRACE index_camera_passthrough] state: Capturing
[2024-03-05T15:25:41Z TRACE index_camera_passthrough] getting camera frame
Error: copy_buffer_to_image_info: `regions[0].buffer_offset` plus the number of bytes being copied is greater than `src_buffer.size()`

Stack backtrace:
   0: anyhow::error::<impl core::convert::From<E> for anyhow::Error>::from
             at /home/lea/.cargo/registry/src/index.crates.io-6f17d22bba15001f/anyhow-1.0.75/src/error.rs:551:25
   1: <core::result::Result<T,F> as core::ops::try_trait::FromResidual<core::result::Result<core::convert::Infallible,E>>>::from_residual
             at /rustc/d18480b84fdbf1efc34f62070951334aa833d761/library/core/src/result.rs:1964:27
   2: index_camera_passthrough::pipeline::Pipeline::submit_cpu_image
             at ./src/pipeline.rs:188:9
   3: index_camera_passthrough::pipeline::Pipeline::run
             at ./src/pipeline.rs:323:26
   4: index_camera_passthrough::main
             at ./src/main.rs:415:38
[2024-03-05T15:25:41Z TRACE index_camera_passthrough] got camera frame Instant { tv_sec: 7818, tv_nsec: 422727511 }
[2024-03-05T15:25:41Z TRACE index_camera_passthrough] state: Capturing
[2024-03-05T15:25:41Z TRACE index_camera_passthrough] getting camera frame

Changed config values:
backend = "OpenXR"
camera_device = "/dev/video0"

Do you know what the resolution of the vive camera is?

Technically 640x480, but with this colorspace used it's 612x460 as in the logs.

does it have both eyes' image side-by-side? 612x460 seems to be really small.

No, it's a single camera with a single non-square video output and yes, the quality is not great.

For reference this is what a frame looks like:

oh it's not stereo. OK, that's good.


Autorest Regen Preview to 3.0.0-alpha.20230921.7 by Alexander Sher from refs/pull/3760/head

Triggered from https://github.com/Azure/autorest.csharp/pull/3760

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.GuestConfiguration


Generated commit message is imprecise

Open vscode repo
Open debugModel.ts, line 42
Rename "type" to be called "test". The rename is across files
Generate commit message for these changes
I get "Fix debug view rendering issue" 🐛  I would love if the generated message could figure out this was a rename.

I am not sure what we pass as context now, but could we improve that somehow to work better for cases like this?

Both model and context has improved significantly since the issue was filed. Closing optimistically.


EKS P1Import tests

What does this PR do?

Adds automation for EKS P1 import tests
Refactors common P1 functions
Add tag aws-janitor=marked-for-deletion to cleanup EKS cluster on janitor next run, instead of next+1 run

Checklist:

[x] GitHub Actions:
P1Provisioning - :green_circle: https://github.com/rancher/hosted-providers-e2e/actions/runs/10441389153
P1Import- :green_circle: https://github.com/rancher/hosted-providers-e2e/actions/runs/10454461150

Latest run: https://github.com/rancher/hosted-providers-e2e/actions/runs/10474715134


Sponsorblock: an option to wait for data from their api?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
I tried to search but didn't find a way how to do that
Is there an option to wait for data from sponsorblock api/server and NOT download the video UNTIL the requested data is received?
It makes no sense for example if I say to remove the ads (from a very long video in 4k quality) to proceed with the download only to find out after a few hours that all ads are still here and there was a one-minute downtime on their side or just some random internet issue where yt-dlp failed to fetch the necessary data and now I have to download everything again and manually check if the data was received properly this time; it is so annoying
Once again, everything works fine for the most part but sometimes it may fail due to their API not responding in time or some other error that may be already fixed a second after this first attempt/ping was sent...
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Duplicate of #9953
Could you share some example URLs over in that issue, please?


yeild and takeLatest not working with jest

Env version Details
"react": "18.1.0",
"react-native": "0.70.6",
"react-redux": "^8.0.5",
"redux": "^4.2.1",
"redux-saga": "^1.2.2"
when i run the app normally redux-sagas methods execute properly but when i test same with jest it's not executing the methods
//////saga.js
import {takeLatest,put,delay,call }from 'redux-saga/effects';
import {INCREASE_BURGER,DECREASE_BURGER} from '../Burger/burgerActionTypes';
function* increaseAsync(){
console.log("function call")
console.log("saga-begin")
yield delay(4000);
yield put({
type:"INCREASE_BURGER_UP",
payload:5
})
}
export default function* watchCounterIncrease(){
yield takeLatest(INCREASE_BURGER,increaseAsync);

}

Hi!
Could you please share your jest test? Thanks!

import  watchCounterIncrease from '../MyAssets/Redux/saga/saga';
describe('App  Screen', () => {
const result=watchCounterIncrease();
console.log(result.next().value);
console.log(result.next());
})

Any suggestion?

Please always try to share a repro case in a runnable form - either by providing a git repository to clone or a codesandbox. OSS maintainers usually can't afford the time to set up the repro, even if exact steps are given.

https://codesandbox.io/s/redux-saga-counter-example-forked-32wp08?file=/index-test.js

Thank you for your support, but my concern is if we try to console this value then it is returning as undefined.
iter.next().value as well as delay(1000) is returning as undefined that is the reason the test cases are passing because the both value having undefined as result but in actual it's not giving as expected output.

closing as issue is not related to the this library


Restarting the Server doesn't pick up changes to db.json

When I make changes to db.json I can see the server restart but it doesn't pick my changes.
It seems to cache the first loaded version of db.json.
When I close and re-open VS Code and start the changes are picked up.
I can't figure out how or why it is being cached.

Sorry for the trouble. I will check this and give a fix version as soon as
I can.
On Tue, Feb 21, 2023, 10:23 PM kpmccaffrey @.***> wrote:

When I make changes to db.json I can see the server restart but it doesn't
pick my changes.
It seems to cache the first loaded version of db.json.
When I close and re-open VS Code and start the changes are picked up.
I can't figure out how or why it is being cached.
—
Reply to this email directly, view it on GitHub
https://github.com/R35007/mock-server-vscode-extension/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFREITDLKRJWF677IHJOSALWYTXG3ANCNFSM6AAAAAAVDJETT4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi @kpmccaffrey ,
I have fixed the caching issue. Please install the latest version v19.0.0.

Verified working. Thanks for the quick fix! Great tool!

@kpmccaffrey
I am glad this tool helps you. Please don't forget to give rating and review in vscode marketplace.
Thanks :)


fix: Hold payload queue mutex while accessing specific queues map

What does the PR do?

Checklist

[x] PR title reflects the change and is of format <commit_type>: <Title>
[ ] Changes are described in the pull request.
[ ] Related issues are referenced.
[ ] Populated github labels field
[ ] Added test plan and verified test passes.
[ ] Verified that the PR passes existing CI.
[ ] Verified copyright is correct on all changed files.
[ ] Added succinct git squash message before merging ref.
[ ] All template sections are filled out.
[ ] Optional: Additional screenshots for behavior/output changes with before/after.

Commit Type:
Check the conventional commit type
box here and add the label to the github PR.

[ ] build
[ ] ci
[ ] docs
[ ] feat
[ ] fix
[ ] perf
[ ] refactor
[ ] revert
[ ] style
[ ] test

Related PRs:

Where should the reviewer start?

Test plan:

CI Pipeline ID:

Caveats:

Background

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

closes GitHub issue: #xxx

What's the status of this PR? Was it a bug fix? Is there more to do?

Let's close this PR - DLIS-6877


Prevent NULL tiles from mapping to phy_tile

Signed-off-by: Andrew Butt<EMAIL_ADDRESS>Prevent NULL tile types from mapping to a phy_tile in tile_map.  Fixes #1538 by preventing two logical tiles mapping to the same phy_tile for unsplit tiles.

Built target file_env_Miniconda3-py37_4.8.2-Linux-x86_64.sh
Built target file_env_conda_bin_conda
Built target conda
Built target conda_yapf
yapf needs to reformat /home/travis/build/SymbiFlow/symbiflow-arch-defs/xc/common/utils/prjxray_form_channels.py
CMakeFiles/check_python.dir/build.make:77: recipe for target 'CMakeFiles/check_python' failed

FAILED: cd /tmpfs/src/github/symbiflow-arch-defs-presubmit-install/build/xc/xc7/archs/zynq7/devices/xc7z010-virt && /usr/bin/cmake -E env PYTHONPATH=/tmpfs/src/github/symbiflow-arch-defs-presubmit-install/third_party/prjxray:/tmpfs/src/github/symbiflow-arch-defs-presubmit-install/utils /tmpfs/src/github/symbiflow-arch-defs-presubmit-install/build/env/conda/bin/python3 /tmpfs/src/github/symbiflow-arch-defs-presubmit-install/xc/common/utils/prjxray_arch_import.py --db_root /tmpfs/src/github/symbiflow-arch-defs-presubmit-install/third_party/prjxray-db/zynq7/ --part xc7z010clg400-1 --connection_database /tmpfs/src/github/symbiflow-arch-defs-presubmit-install/build/xc/xc7/archs/zynq7/devices/xc7z010-virt/channels.db --output-arch /tmpfs/src/github/symbiflow-arch-defs-presubmit-install/build/xc/xc7/archs/zynq7/devices/xc7z010-virt/arch.xml --tile-types CLBLL_L,CLBLL_R,CLBLM_L,CLBLM_R,BRAM_L,RIOPAD_M,RIOPAD_S,RIOPAD_SING,CLK_BUFG_TOP_R,CLK_BUFG_BOT_R,CMT_TOP_L_UPPER_T,HCLK_IOI3,PSS2 --pb_types SLICEL,SLICEM,BRAM_L,IOPAD,IOPAD_M,IOPAD_S,BUFGCTRL,PLLE2_ADV,HCLK_IOI3,PSS2 --pin_assignments /tmpfs/src/github/symbiflow-arch-defs-presubmit-install/build/xc/xc7/archs/zynq7/pin_assignments.json --device xc7z010

Loading database from '/tmpfs/src/github/symbiflow-arch-defs-presubmit-install/build/xc/xc7/archs/zynq7/devices/xc7z010-virt/channels.db'
No progressbar disabled because non-interactive terminal.
Traceback (most recent call last):
  File "/tmpfs/src/github/symbiflow-arch-defs-presubmit-install/xc/common/utils/prjxray_arch_import.py", line 1023, in <module>
    main()
  File "/tmpfs/src/github/symbiflow-arch-defs-presubmit-install/xc/common/utils/prjxray_arch_import.py", line 788, in main
    conn, model_xml, complexblocklist_xml, tiles_xml
  File "/tmpfs/src/github/symbiflow-arch-defs-presubmit-install/xc/common/utils/prjxray_arch_import.py", line 620, in insert_constant_tiles
    assert len(results) == 1, results
AssertionError: []

Maybe just filter NULL in the other lookup?

Maybe just filter NULL in the other lookup?

Yeah that seems reasonable.


Ingress controller calls admin api need to support passing token

The current ingress controller calls api interface does not support token, so the deployment needs to apisix default off token authentication function, there is a security risk

Related to https://github.com/apache/apisix-ingress-controller/issues/50

The related PR is #147

The related PR is #147

Supoorted by #147.

Supoorted by #147.


The test is failed. (npm install)

Node v8.9.0
NPM v5.5.1
exports
✓ should export ES6
✓ should export ES5
1) should export rendering service
3 passing (3s)
1 failing
1) exports should export rendering service:
Error: Cannot find module '../render on client'
at require (internal/module.js:11:18)
at Object. (build/react-router/render.js:28:23)
at loader (node_modules/babel-register/lib/node.js:144:5)
at Object.require.extensions.(anonymous function) [as .js] (node_modules/babel-register/lib/node.js:154:7)
at require (internal/module.js:11:18)
at Object. (build/server/render.js:63:15)
at loader (node_modules/babel-register/lib/node.js:144:5)
at Object.require.extensions.(anonymous function) [as .js] (node_modules/babel-register/lib/node.js:154:7)
at require (internal/module.js:11:18)
at Object. (build/server/server.js:29:15)
at loader (node_modules/babel-register/lib/node.js:144:5)
at Object.require.extensions.(anonymous function) [as .js] (node_modules/babel-register/lib/node.js:154:7)
at require (internal/module.js:11:18)
at Object. (server.js:3:18)
at loader (node_modules/babel-register/lib/node.js:144:5)
at Object.require.extensions.(anonymous function) [as .js] (node_modules/babel-register/lib/node.js:154:7)
at require (internal/module.js:11:18)
at Context. (test/exports.js:163:18)

Yes, the push to github repo didn't go through, pushed the fix now.


[ES2016] Only functions with simple parameter list can have strict mode directive

As mentioned in the specification 14.1 Function Definitions:

It is a Syntax Error if ContainsUseStrict of FunctionBody is true and IsSimpleParameterList of FormalParameters is false.

The simplest test case:
esprima.parse('function f(x=1) {"use strict";}');

Expected: An exception indicating the syntax error.
Actual: A valid syntax tree.

Since this is affecting FunctionBody, the test cases shall also cover function expressions and declarations, method definitions, as well as arrow function expressions.

Following V8, the error message for the case should be:

SyntaxError: Illegal 'use strict' directive in function with non-simple parameter list

Is there any kind of IsSimpleParameterList notation in the parser code?


Infinite loading error

Version: 1.5.0
Device: iPhone se
what happends?
GitPoint loads forever an it’s impossible to do anything
what where you expecting?
For it to load
Steps to reproduce:

install and login
change github username
open app

Workaround:
Reinstall the app

Hi @cjavad and thank you for logging this issue. Did you by any chance deauthorize git-point from your github account setting?

Hi @cjavad, does change github username mean changing your logined account name in other places?

I changed my username from javadsm to cjavad via github when this occured

can i take this up?

@cedric05 Sure. Please go ahead.

i tried building and couldn't in my mac. someone else please take up

Same as #477.


mac m1 pro UI - no access to plugin hub sidebar, screenshots etc

Type
Incorrect behavior
Operating System
MacOS
OS Version
os 12.6 Monterey
Bug description
Opened application and UI was different. No access to plugin hub, screenshot tool etc. Also for the first time it shows a fullscreen option (green top left) which has never appeared nor would I use
Client had changed over last few days a few times. Just visual apart from client size changing randomly occasionally and having to reset this in the plugin 'runelite'
Screenshots attached of how it was originally (Tuesday as an example), Friday (yesterday) where visually it changed, then today where it's unusable.
Note that it appears some plugins are functioning, e.g. bank tag layouts and ground items appear, but I cannot access the sidebar
Screenshots or videos

RuneLite version
2.6.11 it seems as i have re-downloaded which didn't solve it.
Logs
No response

Seems like a dupe of #14642 Did you try  Ctrl/Cmd + F11 @downloader55 ?


Cast timestamps to datetime before compare in mssql

Purpose

Cast timestamp to datetime before comparing in mssql to avoid possible infinite loop

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

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/9027456965

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/9027456965
Status: success


Context Menu Position issue in the package details.

There is an office in the context menu position in the package details form.

Open the details for any package.
Right click in the details pan.
The issue should be reproduced.

Version
ChocolateyGUI Version 0.15.0

@fakoua I have just tried to reproduce this problem in the latest pre-release version of Chocolatey GUI and I do not believe that this is an issue anymore.
As such, I am going to close this issue.  Please feel free to re-open if you still believe it to be a problem.
Thanks for reporting the issue.


tree view docs  The code example is missing a comma

Link to the Page
https://www.skeleton.dev/components/tree-views
Describe the Issue (screenshots encouraged!)
The code example is missing a comma
let myTreeViewNodes: TreeViewNode[] = [
	{
		id: 'unique-id'
		content: 'content',
		lead: '(icon)',
		children: [
			//...
		]
	},
	// ...
]

Thanks for closing this @fankaiLiu. In the future, if you tag the issue in the PR comment it'll close automatically. One of those great features from GitHub!

Thanks for closing this @fankaiLiu. In the future, if you tag the issue in the PR comment it'll close automatically. One of those great features from GitHub!
Looks like you did #{XX} but should have been #XX

Thanks for the heads up! I'll make sure to use the correct format next time. #XX  ：）


gk-deploy to resume from Heketi topology load resume

Closes: #105
Signed-off-by: Mohamed Ashiq Liyazudeen<EMAIL_ADDRESS>

This change is

LGTM


Creation of segments out of different vessels systems

Now, the centerline extraction for a given set of points works if all those points have been associated to the same model (e.g., 2 points assigned to portal). We would like to expand the use case to include the possibility to add points in different vessels systems.
As an example, think about  the possibility to add 2 points in portal and 2 points in hepatic for the same point set. At this point, the algorithm for centerline extraction will look to connect the two systems (and will fail because the models should not have any connection).
Idea on how to implement this:

Instead of saving list of points, we could save list of tuples (model, point).
The algorithm that computes the centerlines will iterate over the number of models added in the list of tuples and will runt the centerline extraction as many times with the relevant points.
The id of the centerline (and color) should be the same

I've added the functionality for testing out this functionality in the #137 pull request. Now centerline segments are added to the same model instead of being replaced when pressing the "Add Vessel Centerline Segment" button. As long as you use the same markups point list you will be adding segments to the same model. For this first implementation you will need to use the other Slicer widgets to delete markup points or the combined centerline model.

Pull request #144 is adding segment id to all segment nodes. This can be used to give segments the same id. No GUI is created for this yet.


6.04 核心上线

Pull Request Test Coverage Report for Build 1255

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

Totals

Change from base Build 1215:
0.0%

Covered Lines:
3890

Relevant Lines:
4030

💛  - Coveralls


Zipir Cubs exist but are useless

Mod source
Factorio Mod Portal
Which mod are you having an issue with?

[X] pyalienlife
[ ] pyalternativeenergy
[ ] pycoalprocessing
[ ] pyfusionenergy
[ ] pyhightech
[ ] pyindustry
[ ] pypetroleumhandling
[ ] pypostprocessing
[ ] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[ ] Graphical
[ ] Crash
[X] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
The "Assisted Embryology" research unlocks, among other things, a recipe for Zipir Cubs, but cubs are not used in the Zipir life cycle and no recipes using them exist other than turning them into biomass or ash.
(Additionally, there is a "Zipir Carcass" object that pops up in recipe books and when selecting signals.  No recipes exist for this.)
Steps to reproduce
No response
Additional context
No response
Log file
No response

Thanks for the report. Fixed: https://github.com/pyanodon/pyalienlife/commit/a226697f8fc46cae5d53ef7b33ac182e4710f0ac


Fix/194- log in and register

Describe the bug
After the authentication permission make by the baackend the register and log in are not working as expected and causing that the forntend has to work again with the extension modheader which it no the most effective way
To Reproduce
Expected behavior
The possibility to create an a account and a log in without needing to use the backend should be an option.

Story Points: 8
See the discussion in Frontend S3 v2
Powered by Parabol


Web Parser - Research Dan

Group effort -  Let us know if you need assitance =)

Have been experimenting with scraping sites and removing data from them.  Also put in a pull request to the maker of the scraper that we use


chore(ci): Add gate for failing flaky version compatibility tests

Description
TBD - Draft for now
User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing and quality

[ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

change me!

/test gke-version-compatibility-tests

We have an example (on the first try) 😁
Link: https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/stackrox_stackrox/13024/pull-ci-stackrox-stackrox-master-gke-version-compatibility-tests/1846506677147275264
Log: https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/test-platform-results/pr-logs/pull/stackrox_stackrox/13024/pull-ci-stackrox-stackrox-master-gke-version-compatibility-tests/1846506677147275264/artifacts/gke-version-compatibility-tests/stackrox-stackrox-e2e-test/build-log.txt
===> FLAKY: Test flakiness is within allowed boundaries. Returning success to CI pipeline


db.get() is undefined even though the db loads and the expected files are there

I've added an object using keyvalue like this:
db.put('documents', object);
When I load my db it loads as expected and properly replicates across peers. I know this because if I console.log db I can drill down to db._index._index and see the object I stored. I can also call db.all() and get the same result.
However, when I call db.get('documents'), I get undefined. Any ideas why that might be?
Here's my code that uses db.get():
ipfs.on('ready', async () => {
    const orbitdb = new OrbitDB(ipfs)
    db = await orbitdb.open(address, {
      sync: true,
      create: true,
      overwrite: true,
      localOnly: false,
      type: 'keyvalue',
      write: ['*']
    });

    db.load();
    await db.load(db, 'Loading database...')
    .then(() => {
      console.log(db.get('documents')); //this === undefined
    });
  })
}

uhh db.load only takes a single optional argument: https://github.com/orbitdb/orbit-db/blob/master/API.md#storeloadamount
So your code should look like this:
ipfs.on('ready', async () => {
    const orbitdb = new OrbitDB(ipfs)
    let db = await orbitdb.open(address, {
      sync: true,
      create: true,
      overwrite: true,
      localOnly: false,
      type: 'keyvalue',
      write: ['*']
    });

    db.load()
    .then(() => {
      console.log(db.get('documents')); //this === undefined
    });
  })
}

Also when you await ... don't expect to get a promise

Once again, you're a huge help @lukas2005! Not sure where I even got the idea that the second parameter should be in there. It's working as expected now!


hello_xr is not aware of BSD

Build fails with clang-10:
/usr/ports/misc/openxr/work/OpenXR-SDK-Source-release-1.0.14/src/common/gfxwrapper_opengl.c:38:31: error: unused parameter 'format' [-Werror,-Wunused-parameter]
static void Print(const char *format, ...) {
                              ^

Ah, I assume this isn't the only error you're seeing, since it means that you are missing an OS define for that file. I'm guessing you're on a BSD based on the source path? I suspect the OS_LINUX defines will actually get you what you want here, so if you can confirm that (e.g. adding -DOS_LINUX to CMAKE_C_FLAGS and CMAKE_CXX_FLAGS) we can adjust the build system accordingly.

Yeah, it actually looks like all the utils and such from Facebook/Oculus that are used mainly in hello_xr are unaware of BSD and will need ports: see external/include/utils in addition to gfxwrapper. Hopefully the ports aren't too hard.

FWIW, the platform plugins have been renamed, though there is still an OS_LINUX define in gfxwrapper.
This should not be an impediment to getting the loader to build, however.

I managed to build successfully on FreeBSD 14 with OpenGL+X11. Only required changes were:

Adding a check for defined(__FreeBSD__) in src/gfxwrapper_opengl.h:130 (I imagine there's a more generic define that could work here for other variants)
Adding a check for CMAKE_SYSTEM_NAME STREQUAL "FreeBSD" in src/CMakeLists.txt:245 to determine presentation backend
Adding /usr/local/include as a target include directory in each of the CMakeLists under src/tests

My patch for build https://github.com/KhronosGroup/OpenXR-SDK is:
--- src/CMakeLists.txt.orig     2024-02-17 20:48:29 UTC
+++ src/CMakeLists.txt
@@ -165,7 +165,7 @@ cmake_dependent_option(
 if(WIN32)
     add_definitions(-DXR_OS_WINDOWS)
     add_definitions(-DNOMINMAX)
-elseif(CMAKE_SYSTEM_NAME STREQUAL "Linux")
+elseif(CMAKE_SYSTEM_NAME STREQUAL "Linux" OR CMAKE_SYSTEM_NAME STREQUAL "FreeBSD")
     add_definitions(-DXR_OS_LINUX)
 elseif(ANDROID)
     add_definitions(-DXR_OS_ANDROID)

It's hack.
Correct patch must define -DXR_OS_FREEBSD and add defined(XR_OS_FREEBSD) to all #if defined(XR_OS_LINUX)/#ifdef XR_OS_LINUX lines in sources. Or replace XR_OS_LINUX with XR_OS_POSSIX as suggester above.

ping


test PR with Github API

Please pull this in!

Docs Build status updates of commit 17b02a2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

sdp-test/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 205b3b1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

sdp-test/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit fda9e3a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

sdp-test/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Be more explicit of merging non-.NET assemblies

This is usually the case when using interop libraries
(e.g.: SQLite, Kinect)

some C# 6 usage :D

Updated.


Network.requestIntercepted does not intercept requests when chrome is started with a new user-data-dir for the first time

When attempting to run the code from Intercept-and-block-requests-by-URL the first time chrome is started with a new user-data-dir Network.requestIntercepted doesn't intercept requests. If Chrome is stopped and started again with the same user-data-dir it works as expected. This only happens when not using the --headless flag. When used in conjunction with chrome-launcher it seems as though the code doesn't work as chrome-launcher creates a new user-data-dir every time. Here is a repo demonstrating this behavior: https://github.com/JarrodCTaylor/cri-request-interception

Component
Version

Operating system
macOS Sierra 10.12.6

Node.js
8.2.1

Chrome/Chromium/...
61.0.3163.79

chrome-remote-interface
0.24.5

Is Chrome running in a container? NO

Yep, that's weird, I can confirm it happens.
I suggest you to file an upstream issue or to ask this in the Google Group, feel free to post the link here if you decide to do so.

Related Google Group post: https://groups.google.com/forum/#!topic/chrome-debugging-protocol/YC6LbG8lls0

Closing this; people may want to keep track of the above thread (if there will ever be a reply).

Ops, fixed (https://crbug.com). Yes it might be a good idea IMHO.

crbug link: https://bugs.chromium.org/p/chromium/issues/detail?id=764505


don't dowload whole history of powerlevel9k and powerlevel10k submodules

Cloning prezto's submodules takes considerable amount of time. This is especially noticeable when setting up environment on remote host over SSH with slow connection. Moreover it's really annoying as you realize that time spent for stuff that you never use.
With this PR git will fetch only required shallow commit, as described here, thus saving time over SSH with slow connection.
I selected only powerlevel submodules because they're pretty fat.

Makes sense. Thanks!

IMO we should be shallow-cloning all submodules.

@jeffwidman Unfortunately some submodules have their own submodules which hardly ever use shallow = true. For example powerlevel9k has shunit2 submodule dependency which whole history is about 69 MiB. So git submodule update --init --recursive will still fetch whole history of sub-submodules.

@romkatv ☝

Enabling shallow cloning of powerlevel10k is a good idea. Powerlevel10k documentation lists 12 installation options and only two of them pull full git history. With this PR there will be just one (zimfw doesn't have an option to enable shallow cloning).


e371613

Details
If tz is a kurast bazaar
(Endless Creation/Leave Loop)
or
After entering the Ruined Temple
(Endless Creation/Leave Loop)
Version
e371613

@jang9824  can you upload your config.yaml for that character please?
just remove your login info.

@jang9824 can you upload your config.yaml for that character please?
just remove your login info.
As well can you please upload/show the log from that timeframe and make sure the debug option is shown in settings (if it isnt checked it will only help for new log generation)

koolo-log-2024-05-17-10-40-52.txt
maxGameLength: 500
username:
password:
realm: kr.actual.battle.net
characterName:
commandLineArgs: -ns -MOD minimumgraphics
killD2OnStop: false
enableCubeRecipes: true
health:
healingPotionAt: 65
manaPotionAt: 10
rejuvPotionAtLife: 50
rejuvPotionAtMana: 10
mercHealingPotionAt: 60
mercRejuvPotionAt: 30
chickenAt: 30
mercChickenAt: 10
inventory:
inventoryLock:
- - 1
- 1
- 1
- 1
- 1
- 1
- 1
- 0
- 0
- 0
- - 1
- 1
- 1
- 1
- 1
- 1
- 1
- 0
- 0
- 0
- - 1
- 1
- 1
- 1
- 1
- 1
- 1
- 0
- 0
- 0
- - 1
- 1
- 1
- 1
- 1
- 1
- 1
- 0
- 0
- 0
beltColumns:
- healing
- mana
- rejuvenation
- rejuvenation
character:
class: lightning
useMerc: true
stashToShared: false
useTeleport: true
game:
minGoldPickupThreshold: 500000
clearTPArea: true
difficulty: hell
randomizeRuns: true
runs:
- pit
- terror_zone
- council
- nihlathak
- baal
- diablo
pindleskin:
skipOnImmunities: []
pit:
moveThroughBlackMarsh: false
openChests: true
focusOnElitePacks: false
stony_tomb:
openChests: true
focusOnElitePacks: false
ancient_tunnels:
openChests: true
focusOnElitePacks: false
mephisto:
killCouncilMembers: true
openChests: true
tristram:
clearPortal: true
focusOnElitePacks: false
nihlathak:
clearArea: true
diablo:
killDiablo: true
clearArea: true
onlyElites: true
baal:
killBaal: false
dollQuit: false
soulQuit: true
eldritch:
killShenk: true
terror_zone:
focusOnElitePacks: true
skipOnImmunities: []
skipOtherRuns: true
areas:
- 12
- 29
- 30
- 31
- 33
- 34
- 35
- 36
- 37
- 39
- 66
- 67
- 68
- 69
- 70
- 71
- 72
- 83
- 108
- 121
- 128
- 129
- 130
- 131
leveling:
ensurePointsAllocation: true
ensureKeyBinding: true
companion:
enabled: false
leader: true
leaderName: ""
attack: true
gameNameTemplate: game-
gamePassword: xxx
gambling:
enabled: true
items:
- coronet
- amulet
- ring
backtotown:
noHpPotions: false
noMpPotions: false
mercDied: true

Thank you for posting. it looks like you do not have debug info turned on. Please enable and if you come across that error again lets see if the logs show anything else.
In this settings file you dont have the terror zones: Bazaar (80), Ruined Temple (94), or Desused Fane (95) enabled. Is there another character in which you do? or have some of them enabled for a different character?

Interesting. If the Terror Zone changes to the bazaar while koolo was already running, even if that run is disabled for terror zones, it will still try to do it and go there then exit game.
My last game before this new tz run finished at 13:06:08, so thats why this started so long after the hour.
time=13:06:26 level=DEBUG msg="Fetching map data..."
time=13:06:28 level=DEBUG msg="Fetch completed" ms=2436
time=13:06:28 level=INFO msg="Starting Game #1. Run list: TerrorZone Run: Kurast Bazaar, Ruined Temple, Disused Fane, Tal Rasha's Tomb, Tal Rasha's Tomb, Tal Rasha's Tomb, Tal Rasha's Tomb"
time=13:06:28 level=INFO msg="Running: TerrorZone Run: Kurast Bazaar, Ruined Temple, Disused Fane, Tal Rasha's Tomb, Tal Rasha's Tomb, Tal Rasha's Tomb, Tal Rasha's Tomb"
time=13:06:29 level=DEBUG msg="Loading screen detected, waiting until loading screen is gone"
time=13:06:32 level=DEBUG msg="Load completed, continuing execution"
time=13:06:32 level=DEBUG msg="Checking for character corpse..."
time=13:06:32 level=DEBUG msg="Character corpse not found :D"
time=13:06:32 level=DEBUG msg="Checking for items to identify..."
time=13:06:32 level=DEBUG msg="No items to identify..."
time=13:06:32 level=DEBUG msg="Belt Stats Health: 4/4 healing, 8/8 mana, 4/4 rejuv."
time=13:06:32 level=DEBUG msg="Checking for items to stash..."
time=13:06:32 level=DEBUG msg="No items to stash..."
time=13:06:32 level=DEBUG msg="Checking for items to stash..."
time=13:06:32 level=DEBUG msg="No items to stash..."
time=13:06:32 level=DEBUG msg="TZ area skipped" area="Kurast Bazaar"
time=13:06:44 level=DEBUG msg="Loading screen detected, waiting until loading screen is gone"
time=13:06:47 level=DEBUG msg="Load completed, continuing execution"
time=13:06:47 level=DEBUG msg="CTA found: swapping weapon and casting Battle Command / Battle Orders"
time=13:06:47 level=DEBUG msg=Buffing...
time=13:06:51 level=DEBUG msg="Reached area" area="Kurast Bazaar"
time=13:06:51 level=DEBUG msg="TZ area skipped" area="Ruined Temple"
time=13:06:57 level=DEBUG msg="TZ area skipped" area="Disused Fane"
time=13:06:57 level=DEBUG msg="Destination area not found" area="Disused Fane"
time=13:06:57 level=WARN msg="Game finished with errors, reason: unrecoverable error occurred, game can not continue: attempt limit reached on step: InteractEntranceStep: area Disused Fane [95]  not found. Game total time: 28.77s" supervisor=Doom

특히. 스프링로가 이미 실행 중인 동안 외부의 대상이 시장으로 변경된 대상 실행이 외부 대상에 대해 대상이 되는 경우에도 여전히 머물고 있는 곳으로 이동한 다음 게임을 종료합니다.
이 새로운 tz 달리기 전 마지막 게임이 13:06:08에 끝났기 때문에 한 시간이 지나서 그럴 이유가 있습니다.
time=13:06:26 level=DEBUG msg="Fetching map data..."
time=13:06:28 level=DEBUG msg="Fetch completed" ms=2436
time=13:06:28 level=INFO msg="Starting Game #1. Run list: TerrorZone Run: Kurast Bazaar, Ruined Temple, Disused Fane, Tal Rasha's Tomb, Tal Rasha's Tomb, Tal Rasha's Tomb, Tal Rasha's Tomb"
time=13:06:28 level=INFO msg="Running: TerrorZone Run: Kurast Bazaar, Ruined Temple, Disused Fane, Tal Rasha's Tomb, Tal Rasha's Tomb, Tal Rasha's Tomb, Tal Rasha's Tomb"
time=13:06:29 level=DEBUG msg="Loading screen detected, waiting until loading screen is gone"
time=13:06:32 level=DEBUG msg="Load completed, continuing execution"
time=13:06:32 level=DEBUG msg="Checking for character corpse..."
time=13:06:32 level=DEBUG msg="Character corpse not found :D"
time=13:06:32 level=DEBUG msg="Checking for items to identify..."
time=13:06:32 level=DEBUG msg="No items to identify..."
time=13:06:32 level=DEBUG msg="Belt Stats Health: 4/4 healing, 8/8 mana, 4/4 rejuv."
time=13:06:32 level=DEBUG msg="Checking for items to stash..."
time=13:06:32 level=DEBUG msg="No items to stash..."
time=13:06:32 level=DEBUG msg="Checking for items to stash..."
time=13:06:32 level=DEBUG msg="No items to stash..."
time=13:06:32 level=DEBUG msg="TZ area skipped" area="Kurast Bazaar"
time=13:06:44 level=DEBUG msg="Loading screen detected, waiting until loading screen is gone"
time=13:06:47 level=DEBUG msg="Load completed, continuing execution"
time=13:06:47 level=DEBUG msg="CTA found: swapping weapon and casting Battle Command / Battle Orders"
time=13:06:47 level=DEBUG msg=Buffing...
time=13:06:51 level=DEBUG msg="Reached area" area="Kurast Bazaar"
time=13:06:51 level=DEBUG msg="TZ area skipped" area="Ruined Temple"
time=13:06:57 level=DEBUG msg="TZ area skipped" area="Disused Fane"
time=13:06:57 level=DEBUG msg="Destination area not found" area="Disused Fane"
time=13:06:57 level=WARN msg="Game finished with errors, reason: unrecoverable error occurred, game can not continue: attempt limit reached on step: InteractEntranceStep: area Disused Fane [95]  not found. Game total time: 28.77s" supervisor=Doom

This issue has also occurred in recent commit
I'm running 3 cool-ro
I have the same bug at tz of 3 at the same time
If you shut down Koolo and restart it, it will work normally

Sorry for interrupting in this issue but I noticed you seem to use a mod "-MOD minimumgraphics", would you mind sharing the mod?


Potential typo ("sever" -> "server")

_sever_message_re
# Handle sever messages

match = _sever_message_re.match(message)

All of these are typos, I guess.

Yep. Those are typos. Do you want to create a PR for that?

@PythooonUser
Yes, sure!

Yes, those are typos. Do you want to create a PR for that?

@PythooonUser
I'd also like to ask about using self.nickname atribute here, as well as in TwitchChatEvent docstrings, is this a mistake? I don't see it in __init__.  Actually, I don't see it anywhere in the project.
Should I fix it at the same PR and name it like "Fix typos & unused nickname attr"?
Give me some recommendations please.

No, this is one of the most important variables! :D
Look at _process_server_messages. Here the nickname is set. This variable is send for JOIN, PART and MSG events to identify the user that triggered it.
We had a discussion about the naming once. We stayed with the Twitch way of naming things.

@PythooonUser
Oops!
I searched for self.nickname rather than nickname! Sorry.

No problem. So I think fixing the typos in your PR is sufficient for now ;)


[Python] Consider renaming FixedShapeTensorArray.to_numpy_ndarray to FixedShapeTensorArray.to_numpy

Describe the enhancement requested
FixedShapeTensorArray.to_numpy_ndarray was introduced in https://github.com/apache/arrow/pull/33948#discussion_r1107099766 however we might want to rename it to FixedShapeTensorArray.to_numpy as proposed here https://github.com/apache/arrow/pull/37533#discussion_r1434206512. This would result in overriding current FixedShapeTensorArray.to_numpy.
Component(s)
Python

I am personally in favor of keeping the current name, for the reasons I mentioned in the linked thread.
The to_numpy() method is an existing method on all pyarrow Arrays, and from which you expect to get a 1D numpy array of the same length as the pyarrow array.
I certainly agree that the 2D array is more useful for a FixedSizeTensorArray, but changing to_numpy() to a 2D array just for this class would break consistency with all other types.
(e.g. generally I expect pa.array(arr.to_numpy()) to work (not necessarily preserving the exact type because of limitations of numpy notwithstanding), but that would no longer be the case when we change the return value for FixedSizeTensorArray)

array([array([1, 2, 3, 4, 5, 6], dtype=int8),
       array([ 7,  8,  9, 10, 11, 12], dtype=int8)], dtype=object)

In which circumstances is this a desirable behavior? It even makes a copy, right?

I agree it will typically not be the case, but it can still be the desirable behaviour for any context where you need a 1D array (and expect to_numpy() to give you that). Another example of such context is constructors like pd.Series that expect a 1D array.
FWIW it does not make a copy. Similarly as our ListArray -> numpy convrsion, each subarray is a slice from one converted parent ndarray (of course it still needs to allocate the object-dtype 1D array that hold those zero-copy subarrays).

I agree it will typically not be the case, but it can still be the desirable behaviour for any context where you need a 1D array (and expect to_numpy() to give you that).

It's a 1D array of 1D arrays. What good is that for exactly?

Ah, I expect it to be a 1D array of ndarrays (to preserve the shape of each individual element). But right now it gives you 1D subarrays I assume because it just uses the storage array's to_numpy.
ListArray and FixedSizeListArray do exactly the same, returning a 1D array of arrays.

But a Series of Numpy arrays is quite specific and unusual, isn't it?

It's not that unusual because that's the only way to store that kind of data in pandas (but that's of course more due pandas' limited data type system)


Third party APIs

Quote:
"Your offer is priced at a monthly flat rate. Usage-based pricing and usage-based "true-up" capabilities are not supported at this time."
My SaaS solution calls third-party APIs, that bill on a consumption model. How does one cover this?

Provide link for customer to create their own account with the 3rd party and provide interface for user to plug in generated API key?

Charge an extortionate amount to my customers that covers their "possible" consumption. Obviously, this makes the solution unattractive.

Charge little for the app asuming their consuption is low and risk being out of pocket.

Thanks
3

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6fd16cf1-63d4-4bbd-a13c-bfb8ab051e18
Version Independent ID: f6b27e10-2190-82e2-4999-eb782171c4eb
Content: Azure Marketplace SaaS Applications Technical Publishing Guide
Content Source: articles/marketplace/marketplace-saas-applications-technical-publishing-guide.md
Service: marketplace
GitHub Login: @keithcharlie
Microsoft Alias: keithcharlie

@arcotek-ltd Thanks for the feedback! We are currently investigating and will update you shortly.

@keithcharlie any ideas on this? @arcotek-ltd is looking for Marketplace guidance on Marketplace offerings that call third-party APIs that bill on a consumption model.

Today we have 2 options for marketplace pricing & billing:

Pay-as-you-go virtual machine model where the software license is billed as a per-hour meter and  priced according to the cores/vcpu on the host virtual machine, and the Azure infrastructure costs are billed separately.

*Monthly subscription aaS model where the software license is billed as a single, flat-rate amount presented as one or more plans (ex, CloudEndure's SaaS offerings). This a true, monthly billing commitment that can be canceled by the customer at any time.

These capabilities are only available on Azure Marketplace today, but we are expecting to offer billing capabilities via additional storefronts, offer types, meters, and contract length options in CY 2019. If you have questions, or if you would like a consultation on adapting your pricing to our models, you can contact us at:
https://aka.ms/marketplacepublishersupport

thanks @keithcharlie .
@arcotek-ltd hope Keith's update answers your question. we will now close this issue but don't hesitate to reply to @femsulu if you have any followup questions.


[Security] Temporarily remove test

https://github.com/Azure/sdk-release-request/issues/3155

/check-enforcer override


Private instructions on Events should render as pre

Right now, private instructions on events are not rendered as rich text, although they allow rich text in the text field for creators when making an event. Line endings, especially, should be preserved.
Related to https://github.com/opencollective/opencollective/issues/5461.

We must make sure that new lines are preserved in the frontend and in the email (ticket confirmed), e.g. using the pre tag.

@Betree Can this be a bounty?

@snitin315 Sure, feel free to assign yourself or decline as you requested it first. Also, please provide a screenshot on the PR.

:trophy: This issue has been completed by @snitin315
To claim the bounty, you can either:

Submit an invoice to https://opencollective.com/engineering/expenses/new (please mention the issue number)
Or ask for an Open Collective gift card of the same value (to donate to other collectives)

Thanks, @snitin315!


sanitycheck build_only option can be confusing

  build_only: <True|False> (default False)
    If true, don't try to run the test under QEMU even if the
    selected platform supports it.

A few things:

this should not be a QEMU thing, although we do run most cases in qemu only, there might be other emulation platforms other than Qemu, for example xt-sim, renode, etc.
We do have some testcases which are true 'build_only', meaning that they should never run on anything, i.e not designed to run on hardware or emulation. For example we have footprint benchmarks that remove all kernel functionality with the sole purpose of measuring footprint.
Many 'build_only' tests are marked as such to avoid running them in Qemu, however, they can run on real hardware without problems

the 'build_only' might be confusing  given the 3 cases above, so we should have some other way to mark tests that can run in emulation and hardware, run in emulation only, and finally those tests which are 'build_only'.

Many 'build_only' tests are marked as such to avoid running them in Qemu, however, they can run on real hardware without problems

I don't think this is a good way to look at it.
I think the QEMU targets should be treated like any other HW platform, with YAML files describing capabilities that are filtered on to run tests.
"build_only" should be used only for those tests that really shouldn't be run on any platform, for things  like footprint, or samples which do not report pass/fail status.
If people are using "build only" to just filter out QEMU because QEMU lacks some peripheral emulation or something, that is the wrong way to go about it.
I think the help test should be changed accordingly to remove all reference to any emulator and just say "don't run this just build it"

think the QEMU targets should be treated like any other HW platform, with YAML files describing capabilities that are filtered on to run tests.

Yes, that is the approach I was considering., maybe I was not clear

There's another point to consider:
As discussed in https://github.com/zephyrproject-rtos/zephyr/pull/5349#issuecomment-351051653 , there can be also external test systems, which may be "dumb" (or rather, complex enough already, to not complicate them with additional processing/decision making), which would run any binary built (e.g., on real hardware, which isn't supported by sanitycheck). So, making sanitycheck smart regarding "build_only" isn't enough, "build_only" tests should be still treated as runnable tests (even if sanitycheck optimizes away their running).


Fix values and README for helm chart

Some values were miscategorised in README.md (global.priorityClassName, global.leaderElection.namespace) while others were missing (webhook.affinity, etc.). resources was missing documented default values.
What this PR does / why we need it: Maintaining an accurate README helps users configure cert-manager properly.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

:tada: 👏 thank you for this change!
/lgtm
/approve
/ok-to-test
/release-note-none

/milestone v0.11


code snippets error in official website.

"Table Deletes, Updates, and Merges"
|--Use merge in streaming queries with foreachBatch
val deltaTable = io.delta.tables.DeltaTable.forName("aggregates")
object DeltaTable only has forPath() method.
Can not find this API in below maven dependencies:

io.delta
delta-core_2.11
0.3.0

I noticed some errors here.
Need to change this:
deltaTable.update(                // predicate using Spark SQL functions and implicits
  $"eventType" = "clck"),
  Map("eventType" -> lit("click"));

To this:
deltaTable.update(
  col("eventType") === "clck",
  Map("eventType" -> lit("click"))
)

If there is an open source version of the HTML docs, I'll be happy to make a PR.  Couldn't find them so maybe they're still closed source.

@phoeph  That is a mistake. I will fix it. Thank you.
@MrPowers That is a better example. Thank you.

I am new here.
May I know the next step is to close this issue？


simplejwt token work for all tenants for django-tenants

I just want to mention about my problem. I have already researched many documents but I could not find any solution.
I just try to use django-tenants. Everything seems OK. But If any user gets the token after login, that token works for all tenants.
It's a big leak of security.
I have been thinking about an idea SIGNING_KEY. If I can change SIGNING_KEY for each tenant, it may be fixed. But It did not.
 class TenantJWTAuthenticationMiddleware(MiddlewareMixin):
    def process_request(self, request):
      tenant = Client.objects.get(schema_name=connection.schema_name)
      jwt_secret_key = tenant.jwt_secret_key
      settings.SIMPLE_JWT['SIGNING_KEY'] = jwt_secret_key

This is my Middleware to change SIGNING_KEY for each tenant.
class Client(TenantMixin):

  name = models.CharField(max_length=100)
  paid_until =  models.DateField()
  on_trial = models.BooleanField()
  created_on = models.DateField(auto_now_add=True)
  jwt_secret_key = models.CharField(max_length=100, null=True, blank=True)

  # default true, schema will be automatically created and synced when it is saved
  auto_create_schema = True

 class Domain(DomainMixin):
    pass

This is my model.
So, I added jwt_secret_key into my model and I got this field in Middleware and tried to set SIGNING_KEY for jwt. But still, jwt after user login can be used for all tenants.
Does anyone have any idea about my problem? Any suggestion or amendment to my solution?
Thanks.

What you want to do is add a claim to your JWT. Recall that JWTs are not modifiable. Claims are public, sure, but that's fine.
To create a claim, you want to override a serializer that returns an access token. In there, you can determine how you want to set the claim's value, such as token["tenant"] = f"server-{uuid}"


Cant open cen64 on Windows anymore.

Downloaded cen64 as of today and realized I couldn't open it.
OS: Windows 10
CPU: i7-4770K
I should have AVX support, so I don't know what went wrong here.

Okay, looks like buildbot is broken. Its generating windows EXEs that are 0 bytes in size, so thats why I can't run it. I guess I didn't notice.

@ProjectRevoTPP should be fixed

Confirmed fixed, binary is available on cen64.com website =D


🐛 [firebase_analytics] FlutterFirebaseAnalyticsPlugin has unchecked cast

Bug report
Describe the bug
Build is throwing warnings. compiling with details shows it is an unchecked cast within FlutterFirebaseAnalyticsPlugin.java
/Users/Applications/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_analytics-9.1.7/android/src/main/java/io/flutter/plugins/firebase/analytics/FlutterFirebaseAnalyticsPlugin.java:69: warning: [unchecked] unchecked cast
            list.add(createBundleFromMap((Map<String, Object>) item));
                                                               ^
  required: Map<String,Object>
  found:    Object

/Users/Applications/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_analytics-9.1.7/android/src/main/java/io/flutter/plugins/firebase/analytics/FlutterFirebaseAnalyticsPlugin.java:82: warning: [unchecked] unchecked cast
        bundle.putParcelable(key, createBundleFromMap((Map<String, Object>) value));
                                                                            ^
  required: Map<String,Object>
  found:    Object

Steps to reproduce
Steps to reproduce the behavior:

Compiled with android studio on mac

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.10.5, on macOS 12.2.1 21D62 darwin-x64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.66.2)
[✓] Connected device (2 available)
[✓] HTTP Host Availability

Flutter dependencies
using:
firebase_core: ^1.16.0
firebase_analytics: ^9.1.7
firebase_auth: ^3.3.17
cloud_firestore: ^3.1.14
firebase_crashlytics: ^2.7.2
firebase_messaging: ^11.3.0
firebase_performance: ^0.8.0+11

@bswhite1
I tried using the plugin example with latest version, but didn't get the exception reported. Are there any specific steps to see the log ?

Just pulled the flutterfire firebase_analytics example.
Made the following changes
android/app/build.gradle
compileSdkVersion 31
android/build.gradle
classpath 'com.android.tools.build:gradle:7.1.3'
I also added the following:
allprojects {
gradle.projectsEvaluated {
tasks.withType(JavaCompile) {
options.compilerArgs << "-Xlint:unchecked"
}
}
android/gradle/wrapper/gradle-wrapper.properties
distributionUrl=https://services.gradle.org/distributions/gradle-7.4.2-all.zip
Did a flutter clean, flutter pub get, then build. I see these errors.

I think I'd prefer to do the Map checks in the dart layer before it gets to the android side. We're getting the map from the parameters passed in from  the Dart layerlogEvent()  and calling Firebase Android SDK own logEvent(). My sense is we should update the log event function in the dart layer to stop any unallowed params from being passed in.
The only map allowed, as far as I can tell, is the one that is allowed in the items array here. According to the documentation, every other property has to be a primitive value (numbers and strings).
iOS
android
So I think I'll improve the Map checks in the dart layer and make a note in the android code to explain why it is unchecked.

I am still getting this issue in version 9.3.5 on flutter sdk 3.3.2
Note: C:\src\flutter\.pub-cache\hosted\pub.dartlang.org\firebase_analytics-9.3.5\android\src\main\java\io\flutter\plugins\firebase\analytics\FlutterFirebaseAnalyticsPlugin.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.

Any solutions ?


Payee 1159

[ ] Unit tests passing
[ ] Coverage reached
[ ] Acceptance tests passing
[ ] Reviewed

you may want to add something like in the else statement:
if(rtiToggle.rtiEnabled && taxYear.year >= TaxYear().next.year) {         logger.info(s"RTIAPI - Bad request error received: ${res.body}")       }


Enum tag alias

I've some weird JSON, mostly it's corresponding to an enum with internal tag, but sometimes the tag has different names, e.g.
[
    {
        "type": "Variant1",
        "other_fields": ...
    },
    {
        "type": "Variant2",
        "other_fields": ...
    },
    {
        "kind": "Variant3",
        "other_fields": ...
    },
]

In this case #[serde(tag = "type") doesn't work. The untagged approach also doesn't work because the variants are not distinguishable by their fields only.
Is it possible to add something like #[serde(tag = "type", tag_alias = "kind")?

I have the same issue I think. Here's the repo: https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=14896473b207036832d55e23dc539805

I think I figured it out. Serde internally consumes the internal tag field, so you can't have it present in each struct that maps to each variant. This works: https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=23efadc6a5607b55653409a042e2cab1

I think, that instead of introducing new attribute for specifying an alias multiple #[serde(tag)] (and #[serde(content)]) attributes may be used


Atomics.waitAsync() polyfill does not work in Safari's WebWorkers

Describe the Bug
The wasm-bindgen-futures crate uses Atomics.waitAsync() for muli-threaded/atomic usecases. This is not supported in Safari, therefore it fallsback to a polyfill which uses WebWorkers to fill it.
Unfortunately, spawning WebWorkers from WebWorkers is not supported in Safari. Therefore, the polyfill fails with an error: ReferenceError: Can't find variable: Worker
Steps to Reproduce
An application needs to run a Rust/WebAssembly program in a WebWorker and expose an async function. Call this function now from the WebWorker in JavaScript. The WebWorker in Safari will show the error: ReferenceError: Can't find variable: Worker
Note: I will provide a reproducible demo after I gained some information on this issue.
Expected Behavior
The async function should return a result.
Actual Behavior
A clear and concise description of what actually happened.
If applicable, add screenshots to help explain your problem.
Proposal
Two possible solutions:

Fallback to the single-threaded implementation on Safari or in WebWorkers in general
Find another polyfill which works with Safari (actually the current polyfill has some other issues too: https://github.com/tc39/proposal-atomics-wait-async/issues/32).

I'm not sure why this happens. If I replace the multi-threaded implementation with the single-threaded implementation in wasm-bindgen-futures then everything works nicely. Do I even need the multi-threaded implementation for my usecase?
Additional Context

Downstream issue: https://github.com/maplibre/maplibre-rs/issues/166
Related PR: https://github.com/rustwasm/wasm-bindgen/pull/1514

@alexcrichton sorry for tagging you directly here 😬 Do you still remember what exactly you mean with threadsafe futures here? https://github.com/rustwasm/wasm-bindgen/issues/1379
I'm not exactly sure in which cases the execution of a Future/Promise differs in the sync vs. async case.

This implementation and threadsafe futures is basically a low-level requirement necessary to implement Rust's async functions correctly in wasm. If spawning a worker doesn't work in safari then this probably just won't work on safari for the time being. Either that or you'll need to build an alternative futures runtime which is compatible with safari

This implementation and threadsafe futures is basically a low-level requirement necessary to implement Rust's async functions correctly in wasm.

Does that mean that the implementation in task/singlethreaded.rs is incorrect? If so, we should remove it.
Otherwise, we can add a compiler flag to forcefully use it even if atomics feature is available

Does that mean that the implementation in task/singlethreaded.rs is incorrect? If so, we should remove it.

It's not incorrect, it just assumes that the program is single-threaded and could cause UB if used in a multi-threaded context.
One way to make it work would be to check in Waker::wake whether the current thread is the thread that's being woken, and panic if not. But obviously that breaks any code that needs to wake other threads, which is what waitAsync is needed for.

One way to make it work would be to check in Waker::wake whether the current thread is the thread that's being woken, and panic if not

That would probably be very nice! My application does not use Futures across threads. I use futures only for fetching data vis browser APIs.
I think the singlethreaded impl would be sufficient for that.
I'll try to do a PR which throws a nicer error for safari. Debugging this took quite long, because I had no clue where the error came from.

Atomics.waitAsync has been merged in webkit, safari should be soon
https://bugs.webkit.org/show_bug.cgi?id=241414

Cool, did not expect that to happen! :D

Now in safari tech preview 160

This is now on Safari stable, version 16.4.
https://developer.apple.com/documentation/safari-release-notes/safari-16_4-release-notes#JavaScript

I'm gonna close this as I believe this is non-actionable from wasm-bindgens side and for this to work on Safari it will just require v16.4.


Does upgradeChaincode.js work?

Excuse me. I have question about upgrade chaincode with nodejs sdk.

I have used upgradeChaincode.js but error happened.
import { init, query } from './hlfClient';
^^^^^^

SyntaxError: Unexpected token import
at createScript (vm.js:80:10)
at Object.runInThisContext (vm.js:139:10)
at Module._compile (module.js:616:28)
at Object.Module._extensions..js (module.js:663:10)
at Module.load (module.js:565:32)
at tryModuleLoad (module.js:505:12)
at Function.Module._load (module.js:497:3)
at Function.Module.runMain (module.js:693:10)
at startup (bootstrap_node.js:188:16)
at bootstrap_node.js:609:3
I have search the problem. And try to fix it.
But it seems import, some ES6 and typescript syntax can't work in frontend.

Can other features work with nodejs sdk?
create-channel
join-channel
install-chaincode
instantiate-chaincode

Thanks.

This does not, I didn't finish implementing this, I rewrote this into a .sh script now. I'll push it when I add some other small improvements.
Yes, these features all work using the nodejs sdk.


API Exception For ATC Qualification

Prime candidate for a git blame on my typo!

:tada: This PR is included in version 6.1.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Consuming RESTApi

Hi,
I have used orbit with ember with JSONAPI, very nice implementation. Now I need to work on app which backend is in RESTApi.
Is there any guide on how to use that? or if even this is implemented? if not any plans for that.
thanks

You can refer this page for more details.

@jakecraige well, i can not change backend which is in RESTAPI, i only can use that

@aalasolutions At this point, you'd probably need to build your own serializer to handle it. Extending the JSONAPISerializer & overriding the deserialize method would be great starting point.

I have the same question — if we're working with a bunch of existing APIs and want to "connect" all the data into Orbit, do we always need to extend JSONAPISerializer? I feel like this is a rather common use case. Is there a util that takes any array of json objects and "maps" it according to a given schema and into memory?


Having different colors (back & fore) for levels ?

Hello,
Could be really cool to be able to specify backColor (all colors related) as an array ['#fff', '#0f0'].
The first one for the first level, the second for the second and so on. And in the case you have only defined 3 colors all level upper 3 use the third color.
Regards
Philippe

@ernest33 can't you do this entirely in the CSS?

@skateman Yes this will be possible :-) But as a lazy man i found this could be a good addition ;-)

Well, I would not want to add such a very specific feature if it can be solved much easier. The plan is to reduce the current amount of CSS generated by JS anyway. Sorry...


gtest: 1.12.1 -> 1.14.0

Description of changes
From v1.13.0 onwards, C++14 or later is required.  v1.12.1 supported C++11.
https://github.com/google/googletest/releases/tag/v1.13.0
https://github.com/google/googletest/releases/tag/v1.14.0
As this will require a large rebuild, this change is targeting staging.
Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[x] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

The release note URLs should go into the commit message.

Bumping to the latest upstream commit for rapidjson did not help.

I'll take a peek at rapidjson unless someone beats me to it

#290250 for rapidjson

@vcunat I am proposing a fix for libqtdbustest in #290647.
I have found only two(?) other packages I think are impacted and will also provide fixes for those.

@vcunat You've caught a couple more than me.  The two I have seen fail are net-cpp and (I believe) itpp, but itpp's logs in hydra are missing and just show an error.

I believe that I've now interacted with all x86-64 linux builds where the error message contains the C++14 gtest error message.
All are associated with this PR, above.
Note that in four cases, I've either nudged an existing version bump PR or filed an update request.  These are all cases where upstream have fixed gtest but where there seems to be some complicating factor (e.g. just updating to a new version causes a compile error unrelated to gtest, or one thing can't update until some other thing is ready).  In these cases I feel it's better to defer to maintainers.


Transforming ID from JSON to create object

Hello, I'm having trouble transforming an id key which is returned as a string in JSON and mapping it to an NSInteger type called ID which I use as my primary key. Any suggestions? I crash with a null exception for ID.

Ideally you would use a JSON number if you want it to translate to an integer in your model.
If you can't change the JSON response, you can always use a transformer:
+ (NSValueTransformer *)IDJSONTransfomer {
    return ...
}

You might see how we've implemented several built-in transformers including MCJSONDateTransformer and this might be useful.


Picking out the column names from a select statement

Hi there,
I can't quite figure out if it's possible to determine the column names returned by a select statement. I think I've narrowed it down to:
a) If there's an alias I use
parsed.tree.stmts[0].stmt.select_stmt.target_list[<x>].res_target.name
b) If there no alias I can use
parsed.tree.stmts[0].stmt.select_stmt.target_list[<x>].res_target.val.column_ref.fields.last.string.sval
Is this the best way to achieve this? I think it works, but it seems incredibly clunky... is there an analog for the filter_columns method that would do the same thing?
Thanks!

@skcuslleb There is no helper that gets the target list column names today - but certainly open to adding one, similar to filter columns.
Note that if your goal is to match what Postgres would output this would only work in some cases, as for example, pg_query does not have enough information to resolve "SELECT * FROM table" to a list of the table's column names.


Dragged element index is not going back to previous value.

Line 271 in angular-dragdrop.js file
zIndex = angular.element(jqyouiOptions.helper ? ui.helper : this).css('z-index');
Firstly i thought it is a wrong behaviour but right now I am not so sure.
Example, Dragged Element didn't have css z-index setted at all at beginning and after dragging, stop event added zIndex with value 2780 ? Actually I don't know from where this value came from,to check it i taked out div from all containers and situation was repeatable.
After some time i was wondering if maybe zIndex is taking from element to late, maybe index is added faster then 271 line is executed.
After all i added default z-index on dragged element, and everything works fine.
Question is it necessary to add index on dragged element or it is a wrong behaviour of lib for elements withouth css z-index attr added ?

Please provide working example on JSFiddle or Plnkr.


release time of TREC 2022 Deep Learning Track test set

Hi,
Just curious when the test set of TREC 2022 Deep Learning Track will be released?

I just released the queries, they are available here: https://microsoft.github.io/msmarco/TREC-Deep-Learning.html
thanks,
Nick.


Rule blocked: CORERULES-249

Links to related JIRA Tickets

https://jira.cdisc.org/browse/CORERULES-249
Copied from: https://dev.azure.com/cdisc-org/cdisc-core/_workitems/edit/4757

Rule Information

Standard: SDTM
Rule ID: CG0562

Describe the problem
There needs to be a way to flatten lists of values in the is_unique_set and is_not_unique_set operations.
Proposed rule logic
Check:
  all:
    - name: --REPNUM
      operator: exists
    - name: --REPNUM
      operator: is_not_unique_set
      value:
         - USUBJID
         - --TESTCD
         - $TIMING_VARIABLES
Operations:
  - id: $TIMING_VARIABLES
    name: Role
    value: Timing
    operator: library_variables

In this example, we need value to represent a flat list containing USUBJID, --TESTCD, and all of the values within the $TIMING_VARIABLES reference.

The validation process for the issue is complete. The acceptance criteria is properly implemented. The changes were tested through running unittests. The tests check the edge cases verifying by passing nested lists in the comparator of is_not_unique_set function. The nested lists are flattened successfully and the operation works fine. This fulfills the AC.
See the attached screenshots for reference

Validation Summary:
Validation Methodology:
The validation process incorporated rigorous testing methodologies to ensure the accuracy and reliability of the system. This included:
Unit Testing:
Unit Testing was done to verify the implementation of the Acceptance criteria. Unit tests were designed to test the made changes using edge cases. All those tests were then run using pytest to verify compliance with acceptance criteria.
Test Endpoint:
A rule was written using the logic provided in the PR related to this issue and then tested against dataset files having negative cases regarding the implementation.
Code Review:
The code changes in the code files were reviewed and were found to be in accordance with the Acceptance criteria. Also, the code was reviewed for following:
Code Quality Evaluation
Logic and Functionality Review
Error Handling and Exception Handling Assessment
Security Considerations
Regression Testing:
Regression tests were run to validate that the changes did not affect any functionality of the overall system.
Test Results:
Test Execution:
The tests were executed in the following order:

Code Review: The code for all the changes files was reviewed first to check the proper implementation of the Acceptance criteria
Unit Tests: The unit tests were run using the following command in the terminal from the root of the project
python -m pytests path/to/the/test/files
Test Endpoint: Following rule logic was written to test the implementation using the endpoint
        Check:
          all:
            - name: --REPNUM
              operator: exists
            - name: --REPNUM
              operator: is_not_unique_set
              value:
                 - USUBJID
                 - --TESTCD
                 - $TIMING_VARIABLES
        Operations:
          - id: $TIMING_VARIABLES
            name: Role
            value: Timing
            operator: library_variables.

Regression Tests

Results:
After successful running of unit, Test endpoint rule, regression tests and careful validation, the issue was found to be good to close as the Acceptance criteria was fulfilled and all validations passed.
Deviations and Non-conformances:
No deviations or non-conformances were observed.
Summary of Findings:
Test Results: All tests conducted during the validation process passed successfully, indicating that the system functions as intended and meets the specified requirements. The tests covered various scenarios and edge cases, providing confidence in the system's reliability and correctness.
Unit Regression: The unit regression tests were executed, and no regressions or issues were identified. This indicates that the changes made during the development or modification of the system did not introduce any unintended side effects or break existing functionality.
Code Reviews: The code reviews conducted by both development and QA teams were completed, and no major issues or concerns were identified. The code adheres to coding standards and best practices, demonstrating good code quality, readability, and maintainability.


Add spell check to "value has no .f field" errors

It would help the user if when a field name x.foo is mistyped, the error message gives a hint as to the correct name (did you mean food?). Since we're already producing an error, performance is not really an issue. The challenge is to do it without adding a dependency on a Levenshtein or Soundex package.

Writing a Levenshtein implementation is simple.
I wrote this one in Bazel: https://github.com/bazelbuild/bazel/blob/71b4dcc1e9a86e922b2f9a0a0665627ffd5233c8/src/main/java/com/google/devtools/build/lib/util/SpellChecker.java

Thanks!

I think NoSuchFieldError needs to be a type of error, not a value, so that implementations of SetField can both return an error message and indicate that they would like the message to be augmented by spell checking.  See starlarkstruct.Struct for an example of a type I should have tested this on. :(


replaces: don't error when setting an older version

If we change to warnings instead of errors we could deal with certain projects where older dependencies are required. The current behaviour throws an error when setting in the replaces a dependency that uses an older version than one in the go.mod.
However this error doesn't take into account if we really need this old dep to build the project. Or even if go mod tidy did bumped this version for us.

I changed my opinion about this issue, I'm closing it. We should do this.