Error from Jellyfin causes watchstate to skip

First, awesome application. I was using this to migrate from the linuxserver.io Jellyfin docker container to the official one. Anyway, importing always worked super fast and accurate. Exporting would end up receiving slow responses from Jellyfin about 500-1000 item in causing watchstate to error out but still complete. So, I basically had to force export repeatedly since the watchstates state would  get messed up and report that nothing needed exported. I eventually got it to export everything by repeatedly doing the force export but still have a few watched statuses that never made it. Overall, worked for what I needed but maybe a throttle option for export or ability to wait longer for a response from the server?
Thanks!

Hi, It's nice to see it's working, do you happen to have some logs to help further see where the problem from? using something like
docker exec -ti watchstate console system:logs

the tool is configured to wait 5min per segmented request and then the timeout killer will kick in, it's possible to increase the timeout further using configuration option, and for Jellyfin/Emby you could also use the environment variable WS_LIBRARY_SEGMENT with large number to reduce the amount of requests as well, the segmentation had to be done to combat future plex 400 bad request from larger requests.
if you can give me logs i can give better specific tips

Awesome, I didn’t know about WS_LIBRARY_SEGMENT. I’ll check that out and try to re-run the force export to see if I can get it to do it again. Unfortunately or fortunately, I had gotten everything migrated so I shut the old one down and the new one has the updated watch states. But I’ll check and let you know. Thanks again for an awesome tool.

Logs are still saved in /config/logs/  if you didn't delete the data folder you will find them there, app.{date}.log, anyway glad it's working for you now.
waiting on your results.

Found the log file from yesterday. It's 19MB and has a lot of names of my media. Any particular place you want me to send it to? Could probably email it to you if possible.

Found the log file from yesterday. It's 19MB and has a lot of names of my media. Any particular place you want me to send it to? Could probably email it to you if possible.

Just few lines with ERROR in them, with some logs around them to understand the context.

[2022-09-17T11:58:39.999736-05:00] logger.ERROR: Unhandled exception thrown during request to change play state of [jellyfin2] Episode [removedforprivacy]. {"library":{"id":"a656b907eb3a73532e40e44b968d0225","title":"Shows","type":"tvshows","totalRecords":20699,"url":"http://removedforprivacy:8097/Users/removedforprivacy/items/?sortBy=DateCreated&sortOrder=Ascending&parentId=removedforprivacy&recursive=true&enableUserData=true&enableImages=false&excludeLocationTypes=Virtual&fields=ProviderIds%2CDateCreated%2COriginalTitle%2CSeasonUserData%2CDateLastSaved%2CPremiereDate%2CProductionYear%2CPath&includeItemTypes=Movie%2CEpisode&limit=1000&startIndex=13000"},"segment":{"number":14,"of":21.0,"size":1000},"item":{"id":"2ff84d9c1b86c0dc7ec437a2c0930d65","url":{"App\Libs\Uri":"http://removedforprivacy:8097/Users/removedforprivacy/PlayedItems/removedforprivacy"}},"play_state":"Played","exception":{"file":"/opt/app/vendor/symfony/http-client/Chunk/ErrorChunk.php","line":62,"kind":"Symfony\Component\HttpClient\Exception\TimeoutException","message":"Idle timeout reached for "http://removedforprivacy:8097/Users/removedforprivacy/PlayedItems/removedforprivacy"."}} []
Pretty much all of them look like above. I put removedforprivacy for replacing my user id and server info but hopefully the overall error is there. Let me know if not. All the logs in Jellyfin was just saying it had slow responses for the play state requests. So, I'm guessing watchstate was erroring and just continuing on.

[2022-09-17T11:58:39.999736-05:00] logger.ERROR: Unhandled exception thrown during request to change play state of [jellyfin2] Episode [_removedforprivacy_]. {"library":{"id":"a656b907eb3a73532e40e44b968d0225","title":"Shows","type":"tvshows","totalRecords":20699,"url":"http://_removedforprivacy_:8097/Users/_removedforprivacy_/items/?sortBy=DateCreated&sortOrder=Ascending&parentId=_removedforprivacy_&recursive=true&enableUserData=true&enableImages=false&excludeLocationTypes=Virtual&fields=ProviderIds%2CDateCreated%2COriginalTitle%2CSeasonUserData%2CDateLastSaved%2CPremiereDate%2CProductionYear%2CPath&includeItemTypes=Movie%2CEpisode&limit=1000&startIndex=13000"},"segment":{"number":14,"of":21.0,"size":1000},"item":{"id":"2ff84d9c1b86c0dc7ec437a2c0930d65","url":{"App\\Libs\\Uri":"http://_removedforprivacy_:8097/Users/_removedforprivacy_/PlayedItems/_removedforprivacy_"}},"play_state":"Played","exception":{"file":"/opt/app/vendor/symfony/http-client/Chunk/ErrorChunk.php","line":62,"kind":"Symfony\\Component\\HttpClient\\Exception\\TimeoutException","message":"Idle timeout reached for \"http://_removedforprivacy_:8097/Users/_removedforprivacy_/PlayedItems/_removedforprivacy_\"."}} []

Pretty much all of them look like above. I put removedforprivacy for replacing my user id and server info but hopefully the overall error is there. Let me know if not. All the logs in Jellyfin was just saying it had slow responses for the play state requests. So, I'm guessing watchstate was erroring and just continuing on.

Indeed its as you described, Idle timeout reached
This error means the timeout handler was invoked and killed the request, which means one of this possible scenarios

The connection between the tool and the jellyfin is quite slow network related
jellyfin is not able to reply fast enough 5min to request due to probably fragmented database or probably slow disk.

That’s basically what I assumed had happened. Tool did work when I kept just forcing the export as it would slowly make its way through them a bit more each time lol. Thanks again!

Np glad it works for you.


Add Star Warp to community plugins grid

Description
Adds my Star Warp from Inox Tools to the community plugins list

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
resources/plugins.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

size-limit report 📦

Path
Size

/index.html
5.95 KB (0%)

/_astro/*.js
21.93 KB (0%)

/_astro/*.css
13.81 KB (0%)


Models underneath the inventory menu

Whenever you in size 4 or auto for menu size, the models of the infected are behind your inventory menu so you can't look at them. When trying to look at the infusers crafting recipes. I don't know if others got the same problem, but still wanted to point it out so it could get fixed.

Can you provide a screenshot?

Tried to reproduce the issue, here are the results.
However, the window size is already very small at this time, which is an abnormal situation.

sure, I can give an image. Let me get a image of it first.

sorry if they look small.

That's what I'm talking about. I'm full screened too btw.

I'm not able to replicate this issue, what other mods do you have installed?

Stale; reopen with more information.


wash up implementation

This PR implements the wash start functionality which fixes #264 and fixes #266
Developers will now be able to install wash and run wash up to get the following output (and of course a running host):
wash up

🛁 wash up completed successfully
🕸 NATS is running in the background at http://<IP_ADDRESS>:4222
🌐 The wasmCloud dashboard is running at http://localhost:4000

🛑 To stop the wasmCloud host and the NATS server, run:
/Users/brooks/.wash/downloads/bin/wasmcloud_host stop; kill 98697

or in interactive mode:
wash up -i
🏃 Running in interactive mode, your host is running at http://localhost:4000
🚪 Press `CTRL+c` at any time to exit
17:46:47.251 [info] Wrote "./host_config.json"
17:46:47.251 [info] Wrote "/Users/brooks/.wash/host_config.json"
17:46:47.252 [info] Connecting to control interface NATS without authentication
17:46:47.252 [info] Connecting to lattice rpc NATS without authentication
17:46:47.253 [info] Host NDTX2AZYORL6HARPSXK3CMCA54ZYW2P7XECPMS5SDXD2BQU7SEDA4YC2 (cool-firefly-8971) started.
17:46:47.253 [info] Valid cluster signers: CDV43ERT2JMQSO2AFC3K2AQBLZ6M5353WYYK4LXPICN6QC6GBAJTWRN2
17:46:47.257 [info] Started wasmCloud OTP Host Runtime
17:46:47.259 [info] Running WasmcloudHostWeb.Endpoint with cowboy 2.9.0 at :::4000 (http)
17:46:47.259 [info] Access WasmcloudHostWeb.Endpoint at http://localhost:4000
17:46:47.358 [info] Lattice cache stream created or verified as existing (0 consumers).
17:46:47.358 [info] Attempting to create ephemeral consumer (cache loader)
17:46:47.359 [info] Created ephemeral consumer for lattice cache loader

Some design choices in this PR:

"interactive" mode is an opt-in flag, I felt this was more ergonomic than wash up --daemon
--connect-only is an option to only connect to a NATS server, rather than launching one, so wash can quickly exit instead of failing silently when NATS infrastructure already exists
All defaults mirror https://wasmcloud.dev/reference/host-runtime/host_configure/
If --ctl-host, --ctl-port and their rpc and prov-rpc commands are omitted, then by default I use the nats-host and nats-port arguments instead. This ends up simplifying a command to connect to NATS at <IP_ADDRESS>:4223 from

wash up --nats-host <IP_ADDRESS> --ctl-host <IP_ADDRESS> --rpc-host <IP_ADDRESS> --prov_rpc-host <IP_ADDRESS> --nats-port 4223 --ctl-port 4223  --rpc-port 4223 --prov-rpc-port 4223

to
wash up --nats-host <IP_ADDRESS> --nats-port 4223

In wash-lib we always write a nats conf and use that for configuration to support jetstream domains. I considered accepting a file to write this to, but felt the API got a little less ergonomic if I did so. The downside is that we'd fail if we can't write the file, but wash should be writing to ~/.wash/downloads anyways.
Running wasmCloud in the foreground allows me to stream logs and write them to ~/.wash/downloads/wasmcloud.log when not in interactive mode rather than dealing with var/log/erlang.log.*

Worth noting here too that to run ./bin/wasmcloud_host start on Windows, you first have to install it as a Service and then start it. It's not a big deal to do this, and it still makes capturing logs hard, but the "correct" way to run start is more difficult than running with foreground

This has some unix and windows specific code because I wanted to make sure it worked for all platforms :)
wasmCloud and NATS versions can be specified, but default to a pinned version rather than the latest for compatibility
Instead of a wash up --down or wash up --cleanup or wash drain --hosts command, I decided to output the command to stop the host and NATS as a part of the output to let users stop the resources as necessary. In interactive mode, pressing CTRL+c cleans up running resources.

I'm halfway through a review, but since this is a large PR, it should have a second review from a maintainer

Starting off with design decision stuff, I'll read through the code more closely in a bit.

If --ctl-host, --ctl-port and their rpc and prov-rpc commands are omitted, then by default I use the nats-host and nats-port arguments instead. This ends up simplifying a command to connect to NATS at <IP_ADDRESS>:4223 from
wash up --nats-host <IP_ADDRESS> --ctl-host <IP_ADDRESS> --rpc-host <IP_ADDRESS> --prov_rpc-host <IP_ADDRESS> --nats-port 4223 --ctl-port 4223  --rpc-port 4223 --prov-rpc-port 4223
to
wash up --nats-host <IP_ADDRESS> --nats-port 4223

Very nice! Like this a lot

--connect-only is an option to only connect to a NATS server, rather than launching one, so wash can quickly exit instead of failing silently when NATS infrastructure already exists

Could this be --nats_connect_only (or similar) instead? The current version seems ambiguous to me.

Instead of a wash up --down or wash up --cleanup or wash drain --hosts command, I decided to output the command to stop the host and NATS as a part of the output to let users stop the resources as necessary. In interactive mode, pressing CTRL+c cleans up running resources.

I think that a wash down command would be a better UX, but this is fine for now.


chore(types): export jsx reserved props for custom element types define

export some jsx types for easy to add custom element types.  like taro
https://github.com/NervJS/taro/blob/next/packages/taro-components/types/index.vue3.d.ts#L162

Closing this, because it has conflicts and it would be nice to go through issue first to get a better idea of the needed use case


Update cats-core to 2.0.0-RC3

Updates org.typelevel:cats-core from 2.0.0-RC2 to 2.0.0-RC3.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-core" } ]

labels: semver-pre-release

Codecov Report

Merging #138 into series/0.7.x will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           series/0.7.x     #138   +/-   ##
=============================================
  Coverage         61.18%   61.18%
=============================================
  Files                11       11
  Lines               304      304
  Branches             11       11
=============================================
  Hits                186      186
  Misses              118      118

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 d5124b6...e5ed36f. Read the comment docs.


TODO: verify user before deleting reservation

Currently anyone with the link can delete the reservation. Only the user who made the reservation should be able to delete the reservation

user verification is added before deletion (see commit)


Benchmarking group assignment against ddbst utility - distinguishing "CH2" from "CY-CH2"

Hello all. I am benchmarking the program for complicated molecules and comparing with ddbst group assingment utility. The assignment seems to be on point. There is only one assignment that seems to be different as below:

GCI Identifier:
get_groups_from_smiles("CC(=O)OCC1CCC(C23C14C2CC(C3)C4(C)C)(C)C", UNIFACGroups) produces:

("CC(=O)OCC1CCC(C23C14C2CC(C3)C4(C)C)(C)C", ["CH3" => 4, "CH3COO" => 1, "CY-CH2" => 4, "CY-C" => 4, "CY-CH" => 3, "CH2" => 1])

using ddbst utility I've got:
["CH3" => 4, "CH2" => 5, "CH" => 3, "C" => 4, "CH3COO" => 1 ])

*It seems it is distinguishing "CY-CH2" from "CH2". What is the difference between these two groups? Does it affect the activity coefficient calculation?

at first glance CY seems to be a shortcut for Cycle? if that is the case, then it seems to be correct ?

Oh. Thanks for the clarification. That's seems to be the case. The current algorithm is distinguishing cyclic CH2 from CH2. I don't know if they are separate groups in UNIFAC, but it's nice having an algorithm that distinguish both.


Improvements to README

I defer to @gregvanl for reviewing these wording and grammatical tweaks.

Hi @SiddarthVijay Thank you for submitting a PR. The current wording of the README is fine.


Add calendar icon

Fixes #48

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

vaadin-date-picker.html, line 66 [r1] (raw file):
I would expect it to work like a toggle, and close the calendar if it was opened. Like the triangle in combo-box.
It would make it a bit more consistent together with clear icon. Right now, in the demo which has an initial date value:

click the calendar icon:
shows the calendar, replaces calendar icon with clear icon

click the clear icon (in the same place where the calendar icon was):
hides the calendar, brings calendar icon back

click the calendar icon:
shows the calendar, calendar icon stays in its place

click the calendar icon:
doesn't hide the calendar, unlike the clear icon.

Comments from the review on Reviewable.io

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from the review on Reviewable.io

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

vaadin-date-picker.html, line 66 [r1] (raw file):
Decided to go with a solution that keeps the calendar icon always visible (even when the clear icon is added) and it now also acts as a toggle button.

Comments from the review on Reviewable.io

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from the review on Reviewable.io

The added test

Review status: 1 of 2 files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io

Reviewed 1 of 1 files at r3.

Currently the target size are 24px×24px. Material design guideline suggest at least 48px×48px. But that would feel too big to align with an input field.
As a compromise, in vaadin-upload I used 40px×40px target size for icons (by adding 8px padding from each side), and also assigned them margin-left and margin-right of -4px to have a bit nicer space ratios between icons and between icon and the field edge.
Apart from that, would be nice to have a ripple effect for icons.
I'm fine with resolving this in another ticket. The same problem exists in combo-box.
See: https://www.google.com/design/spec/layout/metrics-keylines.html#metrics-keylines-touch-target-size

Comments from the review on Reviewable.io

Reviewed 2 of 2 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Does the continuous batching technology in the vLLM online service scenario contain the concept of batch size?

The answer is YES, refer to https://github.com/vllm-project/vllm/blob/main/vllm/worker/model_runner.py#L187
In each iteration, the batch_size will change according to the scheduler outputs.

The answer is YES, refer to https://github.com/vllm-project/vllm/blob/main/vllm/worker/model_runner.py#L187
In each iteration, the batch_size will change according to the scheduler outputs.

So, are you saying that I don't need to specify batch size before calling generate, and that I can pass in all inputs at once, and the scheduler will automatically adjust the batches?

The answer is YES, refer to https://github.com/vllm-project/vllm/blob/main/vllm/worker/model_runner.py#L187
In each iteration, the batch_size will change according to the scheduler outputs.

So, are you saying that I don't need to specify batch size before calling generate, and that I can pass in all inputs at once, and the scheduler will automatically adjust the batches?

Yes, given a lot of request seq, scheduler will determine to run the prefill or decode stage by the seq status, and select the seqs to make up a batch to run (according to the max_num_batched_tokens/max_num_seqs/max_paddings defined in https://github.com/vllm-project/vllm/blob/main/vllm/engine/arg_utils.py)

Can I dynamically specify the max_tokens for each sequence in the batch?
  sampling_params = SamplingParams(
            n=1,
            temperature=1.0,
            top_p=1.0,
            max_tokens=output_len
        )

For instance, by passing a list to max_tokens, with each item in the list corresponding to the length of each prompt

No, all promots in one request share the same sampling params. If you want to specify different output len for different prompt, you need to send multiple generate requests.

The answer is YES, refer to https://github.com/vllm-project/vllm/blob/main/vllm/worker/model_runner.py#L187
In each iteration, the batch_size will change according to the scheduler outputs.

So, are you saying that I don't need to specify batch size before calling generate, and that I can pass in all inputs at once, and the scheduler will automatically adjust the batches?

Yes, given a lot of request seq, scheduler will determine to run the prefill or decode stage by the seq status, and select the seqs to make up a batch to run (according to the max_num_batched_tokens/max_num_seqs/max_paddings defined in https://github.com/vllm-project/vllm/blob/main/vllm/engine/arg_utils.py)

Can I dynamically specify the max_tokens for each sequence in the batch?
  sampling_params = SamplingParams(
            n=1,
            temperature=1.0,
            top_p=1.0,
            max_tokens=output_len
        )

For instance, by passing a list to max_tokens, with each item in the list corresponding to the length of each prompt

No, all promots in one request share the same sampling params. If you want to specify different output len for different prompt, you need to send multiple generate requests.

I understand your mean. However, in a scenario where there are numerous requests with varying output lengths, continuous batch processing may lose its effectiveness.
I am curious if it is possible to adapt vLLM to suit my specific scenario with minimal changes, specifically to dynamically specify different output lengths within a single batch.
This inquiry is just for my personal understanding. I would like to know if implementing such a feature would be significantly challenging.
Thank you very much for your assistance.

@Lvjinhong
Forgive me for not explaining clearly, in each iteration, different seqs in one batch can have different sampling params.
So in your scenario, you just need to asynchronously send multiple generate reqeusts with different prompts + sampling params, for example, https://github.com/vllm-project/vllm/blob/main/benchmarks/benchmark_serving.py#L111

However, about the batch_size, as I mentioned in https://github.com/vllm-project/vllm/issues/1985,
in the scheduler the number of pad token will be affected by the longest/shortest prompt, and in a common case, the batch_size will be relatively small（e.g 1/2/3...)
And the gpu computing utilization will not reach very high levels because of the fmha/gemm size is small
So just be careful that the throughput performance is strongly dependent on the prompt length distribution.

@irasin  Thank you for your answer.
The asynchronous request example should also be applicable to offline scenarios, right? My understanding is that once I send the requests in the loop using async, the batching will be handled automatically, correct?
for i in tqdm(range(len(requests))):
  prompt, prompt_len, output_len = requests[i]
  sampling_params = SamplingParams(...)
  llm_outputs = llm.generate(...)

Also, regarding the issue you mentioned in #1985, if I sort the sequences by length before sending the requests, grouping together those of similar lengths, would this improve the utilization rate?

@irasin Thank you for your answer. The asynchronous request example should also be applicable to offline scenarios, right? My understanding is that once I send the requests in the loop using async, the batching will be handled automatically, correct?
for i in tqdm(range(len(requests))):
  prompt, prompt_len, output_len = requests[i]
  sampling_params = SamplingParams(...)
  llm_outputs = llm.generate(...)

No, in your code the generate call in the for loop  is a synchronous call, so the batch_size is always one, resulting in extremely low device utilization.
Only the asynchronous call can handle different prompt with different sampling params in one batch.

Also, regarding the issue you mentioned in #1985, if I sort the sequences by length before sending the requests, grouping together those of similar lengths, would this improve the utilization rate?

Yes, But in the real-world online service scenario, we cannot perform real-time sort processing on user requests (They should be processed as FIFO). So sort ahead-of-time is just meaningless for testing.

Hi, @WoosukKwon, @zhuohan123, @simon-mo,
I'd really like to know your thoughts on https://github.com/vllm-project/vllm/issues/1985

@irasin Thank you for your answer. The asynchronous request example should also be applicable to offline scenarios, right? My understanding is that once I send the requests in the loop using async, the batching will be handled automatically, correct?
for i in tqdm(range(len(requests))):
  prompt, prompt_len, output_len = requests[i]
  sampling_params = SamplingParams(...)
  llm_outputs = llm.generate(...)

No, in your code the generate call in the for loop is a synchronous call, so the batch_size is always one, resulting in extremely low device utilization. Only the asynchronous call can handle different prompt with different sampling params in one batch.

Also, regarding the issue you mentioned in #1985, if I sort the sequences by length before sending the requests, grouping together those of similar lengths, would this improve the utilization rate?

Yes, But in the real-world online service scenario, we cannot perform real-time sort processing on user requests (They should be processed as FIFO). So sort ahead-of-time is just meaningless for testing.

After checking the source code, it seems that args:max_output_len in one batch is chosen as the max one of the batch and then use padding to revise it for shorter examples?
For benchmark code, offline vLLM for llama seems no need to use async operation. So can we regard the reason of the different length of generated tokens between vLLM and hf is the different calculation but not unfitable hyperparam setting?

@irasin Thank you for your answer. The asynchronous request example should also be applicable to offline scenarios, right? My understanding is that once I send the requests in the loop using async, the batching will be handled automatically, correct?
for i in tqdm(range(len(requests))):
  prompt, prompt_len, output_len = requests[i]
  sampling_params = SamplingParams(...)
  llm_outputs = llm.generate(...)

No, in your code the generate call in the for loop is a synchronous call, so the batch_size is always one, resulting in extremely low device utilization. Only the asynchronous call can handle different prompt with different sampling params in one batch.

Also, regarding the issue you mentioned in #1985, if I sort the sequences by length before sending the requests, grouping together those of similar lengths, would this improve the utilization rate?

Yes, But in the real-world online service scenario, we cannot perform real-time sort processing on user requests (They should be processed as FIFO). So sort ahead-of-time is just meaningless for testing.

After checking the source code, it seems that args:max_output_len in one batch is chosen as the max one of the batch and then use padding to revise it for shorter examples? For benchmark code, offline vLLM for llama seems no need to use async operation. So can we regard the reason of the different length of generated tokens between vLLM and hf is the different calculation but not unfitable hyperparam setting?

In my latest tests, I have noticed that the length of tokens generated using vLLM is consistent with those obtained from hf transformers.

@Thewillman   In my latest tests, I have noticed that the length of tokens generated using vLLM is consistent with those obtained from hf transformers.

@Thewillman In my latest tests, I have noticed that the length of tokens generated using vLLM is consistent with those obtained from hf transformers.

It's not a bug of calculation? We just follow the benchmark code and find that it's hard to enforce examples sychronize hf transformer.Do you change the setting or use online version?

@Thewillman In my latest tests, I have noticed that the length of tokens generated using vLLM is consistent with those obtained from hf transformers.

It's not a bug of calculation? We just follow the benchmark code and find that it's hard to enforce examples sychronize hf transformer.Do you change the setting or use online version?

sorry，I will retest later and confirm this again.

@Thewillman In my latest tests, I have noticed that the length of tokens generated using vLLM is consistent with those obtained from hf transformers.

It's not a bug of calculation? We just follow the benchmark code and find that it's hard to enforce examples sychronize hf transformer.Do you change the setting or use online version?

sorry，I will retest later and confirm this again.

Thanks a lot, the different results between accelerate framework and hf transformer seems confuse a large amount of coders

@Thewillman In my latest tests, I have noticed that the length of tokens generated using vLLM is consistent with those obtained from hf transformers.

It's not a bug of calculation? We just follow the benchmark code and find that it's hard to enforce examples sychronize hf transformer.Do you change the setting or use online version?

sorry，I will retest later and confirm this again.

Thanks a lot, the different results between accelerate framework and hf transformer seems confuse a large amount of coders

Hi, @Thewillman, I totally agree with you, but I was wondering if we only use greedy-search and the same prompt, the result of hf transfomers and vllm should be the same? If you have time, can you do some verfiy test?

@Thewillman In my latest tests, I have noticed that the length of tokens generated using vLLM is consistent with those obtained from hf transformers.

It's not a bug of calculation? We just follow the benchmark code and find that it's hard to enforce examples sychronize hf transformer.Do you change the setting or use online version?

sorry，I will retest later and confirm this again.

Thanks a lot, the different results between accelerate framework and hf transformer seems confuse a large amount of coders

Hi, @Thewillman, I totally agree with you, but I was wondering if we only use greedy-search and the same prompt, the result of hf transfomers and vllm should be the same? If you have time, can you do some verfiy test?

The test can be viewed on earlier issue,but nothing helpful advice https://github.com/vllm-project/vllm/issues/2196

@Thewillman In my latest tests, I have noticed that the length of tokens generated using vLLM is consistent with those obtained from hf transformers.

It's not a bug of calculation? We just follow the benchmark code and find that it's hard to enforce examples sychronize hf transformer.Do you change the setting or use online version?

sorry，I will retest later and confirm this again.

Thanks a lot, the different results between accelerate framework and hf transformer seems confuse a large amount of coders

Hi, @Thewillman, I totally agree with you, but I was wondering if we only use greedy-search and the same prompt, the result of hf transfomers and vllm should be the same? If you have time, can you do some verfiy test?

The test can be viewed on earlier issue,but nothing helpful advice #2196

@irasin In this asynchronous scenario, it is necessary to input prompt_token_ids based on pre-tokenization. So, can the tokenizer encode also be performed asynchronously to achieve higher performance?

@Lvjinhong Forgive me for not explaining clearly, in each iteration, different seqs in one batch can have different sampling params. So in your scenario, you just need to asynchronously send multiple generate reqeusts with different prompts + sampling params, for example, https://github.com/vllm-project/vllm/blob/main/benchmarks/benchmark_serving.py#L111

So, By asynchronously sending multiple generate reqeusts, could we make continuous batching with different prompts and sampling params?


move CSS insertion from top of head tag to the bottom

Attends to https://github.com/fuse-box/fuse-box/issues/1444

Something is off with the tests ;(

Are you on a Mac? I've just run the tests on a PC and getting the following error (which appears to be completely unrelated to the failure I see in the link above):

C:\Projects_experiments\fuse-box.fusebox\tests\1549029640142\test.js:29266
throw new Exception_1.Exception('Expected ${this.obj} to have ${target}');
^
Error: Expected C:\Projects_experiments\fuse-box.fusebox\virtual-files\1549029671075\qwerty.js to have /qwerty.js
at ShouldInstance.findString (C:\Projects_experiments\fuse-box.fusebox\tests\1549029640142\test.js:29266:19)
at env.browser.window (C:\Projects_experiments\fuse-box.fusebox\tests\1549029640142\test.js:21779:55)
at Object.done (C:\Projects_experiments\fuse-box.fusebox\tests\1549029640142\test.js:14169:36)
at process.nextTick (C:\Projects_experiments\fuse-box\node_modules\jsdom\lib\jsdom.js:325:18)
at process._tickCallback (internal/process/next_tick.js:61:11)

I need to resolve these tests... They fail from time to time.

Yep, I pulled down master before making my branch.


Uploaded file size limit?

Can you add a file size limit to the options?

That's not what this plugin is for. It should be an easy task to extend it if you need this feature. Maybe have a look at this: https://stackoverflow.com/a/4307882/2783423


Fix typo in readme

still waiting to be merged 2017

I guess I'll take my code elsewhere!

Reopening after an in-depth discussion of why this pull request is necessary.


fping: 4.1 -> 4.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/fping/versions.
meta.description for fping is: '"Send ICMP echo probes to network hosts"'.

Checks done (click to expand)

built on NixOS
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 4.2 with grep in /nix/store/lkv26w01knv15kbfhf32jcp9brpnvzm5-fping-4.2
directory tree listing: https://gist.github.com/1e919df20ee077cb769540a7da076136
du listing: https://gist.github.com/39218ed7b58a938cd761e323d9fec9c1

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

8 total rebuild path(s)
2 package rebuild(s)
2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
fping
smokeping

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lkv26w01knv15kbfhf32jcp9brpnvzm5-fping-4.2 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A fping https://github.com/r-ryantm/nixpkgs/archive/6347dee4b98fe7978d844d39e49d536bb379c2a6.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lkv26w01knv15kbfhf32jcp9brpnvzm5-fping-4.2
ls -la /nix/store/lkv26w01knv15kbfhf32jcp9brpnvzm5-fping-4.2/bin

cc @the-kenny for testing.

https://github.com/schweikert/fping/releases/tag/v4.2
0b2952e1c92084fb081e5df83e31b13071d6dc49 oops O:).


Bugfix: ESL Media (incorrect group definition in shape)

:tada: This PR is included in version 3.14.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Concatenate Error

Above is a snippet of the error I'm getting.

https://github.com/krstphrrr/ingester_v3/commit/e95e08ccad7b515f118d710a3386acd60f9c496d


Make cluster autoscaler work with Kubernetes 1.6

Taints are being moved from annotation to field of Node. See kubernetes/kubernetes#25320 and kubernetes/kubernetes#38957
There's no 1.6 milestone in this repo, but it is needed for 1.6.

Also, write release notes for any special considerations when upgrading to 1.6.

cc/ @fgrzadkowski

/cc @kubernetes/huawei

#38957 has merged, so HEAD now has the new behavior (taint/toleration annotations are ignored by scheduler, and now scheduler pays attention to taint/toleration fields).
Code freeze is getting close...

@mwielgus What's the status of this issue?

ping @mwielgus -- this is getting pretty urgent
cc/ @kubernetes/release-team

CA was done last week. We are polishing couple things/bugfixing before bumping the version in k8s repo.

ref https://github.com/kubernetes/contrib/pull/2437


Social notification contexts

This extracts the notification system out into its own context with 1 entry API at Tilex.Notifications.
It currently implements two public functions: post_created(%Post{}) and post_liked(%Post{}, max_likes_changed).
We no longer pass the conn to the Notifications  as its not needed and it's not blocking post creation like it was before
Tilex.Notifications is a GenServer because we can load the posts associated modules asynchronously. Then both Tilex.Notifications.Notifiers.Slack and Tilex.Notifications.Notifers.Twitter are supervised GenServers because they don't need to be posted synchronously. The supervisor will bring either one of them up again if they go down for whatever reason.
Adding future notifiers (facebook, linkedin, send email, etc..) are fairly straightforward by using the Tilex.Notificiations.Notifier behaviour:
  defmodule Tilex.Notifications.Notifiers.LinkedIn do
    use Tilex.Notifications.Notifier
    def handle_post_created(post, developer, channel, url) do
      # send notification about created post to LinkedIn
    end
    def handle_post_liked(post, developer, url) do
      # send notification about liked post to LinkedIn
    end
  end

Thank you @tmock12! 👍


docs: Updates

More examples, some style tweaks, etc.

Codecov Report

Merging #2241 (652d634) into master (da31e4b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2241   +/-   ##
=======================================
  Coverage   98.54%   98.54%
=======================================
  Files         114      114
  Lines        1860     1860
  Branches      273      273
=======================================
  Hits         1833     1833
  Misses         10       10
  Partials       17       17

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


built-in genome list

Hi, thanks for developing and updating epic2.
I wanted to see what other built-in genomes available in epic2 but can't find the list.
For example, does it have TAIR10 (Arabidopsis) genome? If so, what's the exact word following --gn?

Here are the chromsizes:
https://github.com/biocore-ntnu/epic2/tree/master/epic2/chromsizes
I do not think so. Is TAIR10 on UCSC?
On Thu, May 26, 2022 at 5:39 AM nermah1 @.***> wrote:

Hi, thanks for developing and updating epic2.
I wanted to see what other built-in genomes available in epic2 but can't
find the list.
For example, does it have TAIR10 (Arabidopsis) genome? If so, what's the
exact word following --gn?
—
Reply to this email directly, view it on GitHub
https://github.com/biocore-ntnu/epic2/issues/60, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEHURUVMG7WFH3OXNQ4NSW3VL3W6ZANCNFSM5W7O24EA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Support assetPrefix for Link component ?

Feature request
Support assetPrefix with next/link component.
Describe the solution you'd like
Provide assetPrefix props for next/link component.
Additional context
This feature is needed in case we have multiple NextJS apps on the same domain with different assetPrefix .

Duplicate of #4998


fix add the alias function hist to histogram and fix p0

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.85 seconds
load time: 474 seconds
storage size:<PHONE_NUMBER>7 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230107114655_clickbench_pr_75352.html

same pr which all checks have passed: https://github.com/apache/doris/pull/15708


Realtime value directly from Kafka Stream

Analyse whether it is possible to extract the last computed value directly from Kafka Stream instead of queryingMongoDb[Reference]

Perhaps keep an eye on this Confluence project KSQL.
https://www.confluent.io/product/ksql/
https://github.com/confluentinc/ksql

KSQL is interesting for the backend, but for simplicity, I'd leave it out of the REST API for now.


Implement account domain verification

It would be handy to have a utility function that checks whether the Domain associated with an account object is is verified using an xrp-ledger.toml file. The actual code to do this is pretty straightforward. Here's an example script to do it: verify_domain.py.
I suggest adding this as a new method verify_domain(address:str, client:Client) in xrpl.account or something similar. Maybe it would be more efficient to have it take an AccountRoot object (such as returned by acount_info) to avoid the need for redundant lookups.

There is a bounty that is closely related to this here: https://github.com/XRPLBounties/Proposals/blob/main/open_bounties/0062 Account-based toml Lookup.md
The bounty is to add it to xrpl.org. This issue is slightly different in that it is asking to add the function to xrpl-py


Fix downstream ESM imports, fixes #6573

Uses default import for importing keytar (a CJS module), which is really the only foolproof way of importing CJS from an ESM context.
I confirmed by creating a simple package:
// msal-node-extensions/foo/package.json
{ "type": "module" }

// msal-node-extensions/foo/index.js
import * as msal from "../dist/index.mjs";
msal;

Running node foo/index.mjs on dev gives the following error:
file:///.../microsoft-authentication-library-for-js/extensions/msal-node-extensions/dist/persistence/KeychainPersistence.mjs:3
import { setPassword, getPassword, deletePassword } from 'keytar';
                                   ^^^^^^^^^^^^^^
SyntaxError: Named export 'deletePassword' not found. The requested module 'keytar' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from 'keytar';
const { setPassword, getPassword, deletePassword } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:124:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:190:5)

Node.js v18.17.1

Running the same command on this branch succeeds (no output, exit code 0).
Not sure of a good/reliable way to encode that in an automated test, but I'd be happy to add one if anyone has any suggestions

Codecov Report

Merging #6600 (56b0d0f) into dev (81d34b4) will decrease coverage by 43.24%.
Report is 360 commits behind head on dev.
The diff coverage is 21.62%.

Flag
Coverage Δ

msal-angular
?

msal-browser
?

msal-common
?

msal-core
?

msal-node
?

msal-node-extensions
41.76% <21.62%> (-33.88%)
:arrow_down:

msal-react
?

node-token-validation
?

Files
Coverage Δ

extensions/msal-node-extensions/src/index.ts
100.00% <100.00%> (ø)

...nsions/msal-node-extensions/src/packageMetadata.ts
100.00% <100.00%> (+100.00%)
:arrow_up:

...-extensions/src/persistence/DataProtectionScope.ts
100.00% <100.00%> (ø)

...nsions/msal-node-extensions/src/utils/Constants.ts
100.00% <100.00%> (ø)

...sions/msal-node-extensions/src/utils/TypeGuards.ts
100.00% <100.00%> (ø)

...msal-node-extensions/src/error/PersistenceError.ts
62.50% <66.66%> (-11.58%)
:arrow_down:

...node-extensions/src/persistence/BasePersistence.ts
64.28% <50.00%> (-2.39%)
:arrow_down:

extensions/msal-node-extensions/src/Dpapi.ts
66.66% <66.66%> (-33.34%)
:arrow_down:

...e-extensions/src/persistence/PersistenceCreator.ts
80.00% <85.71%> (-11.18%)
:arrow_down:

...tensions/src/persistence/PersistenceCachePlugin.ts
71.05% <55.55%> (-4.63%)
:arrow_down:

... and 8 more

... and 214 files with indirect coverage changes

@rzyns please rebase onto base branch so we can merge this PR.


Add SSLEngineTest to ensure Signature Algorithms are present during K…

…eyManager calls.
Motivation:
We had a bug which could case ExtendedSSLSession.getPeerSupportedSignatureAlgorithms() return an empty array when using BoringSSL. This testcase verifies we correctly return algorithms after the fix in https://github.com/netty/netty-tcnative/pull/449.
Modifications:
Add testcase to verify behaviour.
Result:
Ensure we correctly return the algorithms.

@netty-bot test this please

@netty-bot test this please

@trustin thanks... I addressed all of the comments buddy :)


Elastic Search 'Amendments' Model Parsing Failure

When parsing 37 CFR 42 and core configured to use elastic search, every PUT to a notice URI fails the same way.  Here are some snippets that don't make it to the console but provide a great deal of context, pulled from local variables in paused client.py post-exception:
Can't merge a non object mapping [amendments.changes] with an object mapping [amendments.changes]`  [{'reason': '[YW_wNku][<IP_ADDRESS>:9300][indices:data/write/index[p]]', 'type': 'remote_transport_exception'}] '[YW_wNku][<IP_ADDRESS>:9300][indices:data/write/index[p]]'
As the request is made, regulations-core/regcore/db/es.py line 115 local variable notice has the following under the amendments key (ie. notice[‘amendments’]):
[

    {'authority': '35 U.S.C. 2(b)(2).', 'instruction': '1. The authority citation for 37 CFR part 1 continues to read as follows:', 'cfr_part': '1'},

    {'changes': [['1-301', [{'action': 'DELETE'}]]], 'instruction': '2. Section 1.301 is removed and reserved.', 'cfr_part': '1'},

    {'changes': [['1-302', [{'action': 'DELETE'}]]], 'instruction': '3. Section 1.302 is removed and reserved.', 'cfr_part': '1'},

    {'changes': [['1-303', [{'action': 'DELETE'}]]], 'instruction': '4. Section 1.303 is removed and reserved.', 'cfr_part': '1'},

    {'changes': [['1-304', [{'action': 'DELETE'}]]], 'instruction': '5. Section 1.304 is removed and reserved.', 'cfr_part': '1'},

    {'instruction': '6. Part 42 is added to read as follows:', 'cfr_part': '1'},
    {'instruction': '7. Part 90 is added to read as follows:', 'cfr_part': '90'}

]

With the debugger paused immediately after this failure, I attempted to pull what we already have there.   Sorry about that -- we don't have any active instances backed by Elastic, so we must have missed this when updating the Notice schema. I'd have to dig in more, but I'm betting the problem is that the amendments (a list) have different schemas (the first has an authority but no changes, for example). It probably makes sense to encode a single schema at parse time rather than tweaking these in the API.


Having Clone feature

Issue type
[X] Feature request
[] Bug report
[ ] Documentation
Environment

Python version: 3.5.2
NetBox version:  v2.3.1

Description
It would be nice if we had clone feature in place to carry over what we have already done on a record. If it was in place, all we would need to do is that tweaking the entries in cloned record and get that submitted with different name.
In fact, that exactly what we do in Zabbix.

I think you can use the import/export functionality to mimic this.

Sounds like a duplicate of #33


Distribution and versioning of models

Long term, it might be worth considering a way to distribute models more cleanly and in a version controlled manner.  When I tried to start a cluster which was using cellpose, all the machines simultaneously tried to download the model files and the http server failed.  This caused the cluster to crash.  In addition, having models outside of version control could lead to reproducibility issues down the line.
As a workaround, we store the model files directly in a branch of our github cellpose fork.  They are small enough this is not a problem, but I don't think it's a great long term solution.  There are likely some good solutions for this, but I don't know the space very well.
FYI, here is the commit that adds the files and updates the code to use them (I have not tested the GUI):
https://github.com/deisseroth-lab/cellpose/commit/cbce0b4339e9d2fb1a0a96eaf74e60970a74a0fc

Sorry, but the commit changed.  It's https://github.com/deisseroth-lab/cellpose/commit/dcbcefadd342cb721cacde48350e3fb04fd36062

Would you all be open to storing the model files are part this github repo / pypi package, perhaps after you stop supporting mxnet?

I will not overwrite the https paths so there will always be the original versions. I agree this is not ideal, but dealing with something like dvc is also annoying and I don't want users to have to do that.
Can you copy the model files to your .cellpose folder on the server so that the download doesn't trigger?

I think if the models do start getting versioned, version control is something people may be forced to get used to.  I've always found git annoying, but here we are having a discussion on github.  :)
But to your point... yes, we could build our docker files with the .cellpose folder pre-installed, so there would be some tracking of versions.   I'd still check in the model files to some repository for doing that build, so we can maintain the versions ourselves somewhere.
For now, we just maintain a fork of cellpose with a branch where we include the models inside the package (see the commit above).  I haven't updated yet to the pytorch update -- it looks like the model loading is more streamlined now, so the required code changes will be fewer.

Hi @carsen-stringer,
I see the issue is closed, but can you please describe current policy on versioning?
For example: is cellpose==1.0.2 expected to download the same models or will it download new models after cellpose==2.* is release?
We'd like to be able to compare results from multiple experiments by using the same segmentation (and experiments are quite separated in time), so it is important to track an actual version used


Update 12_activity_async_completion.md

Fix returned error to match go client https://github.com/uber-go/cadence-client/blob/master/activity/activity.go#L47

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


what is the difference between nacos-sdk-csharp.AspNetCore and nacos-sdk-csharp?

question
what is the difference between nacos-sdk-csharp.AspNetCore and nacos-sdk-csharp? should I install both?
environment

asp.net core webapi (.Net 6)

i found the document: https://nacos-sdk-csharp.readthedocs.io/en/latest/features/servicediscovery.html


Use default popcount for RISC-V instruction

@ZernovNikolay Please check this - we use MIPT-V built-in popcount function to implement pcnt instruction.

I check this version of mipt-mips and all worked. I understood that and by means of what the function "popcount" does. Before that I didn't know about the existence of "bitset" container.

@ZernovNikolay Please approve code review to merge the change


Polyfill requestAnimationFrame for tests (Jest)

This address React warnings when running yarn test.
Before:

After:

Deploy preview for gestalt ready!
Built with commit 76e3c753567ab8a13a79e19c1eb413161ed16c72
https://deploy-preview-161--gestalt.netlify.com

I think this might be fixed by #158 - just checking now.

+1 I don't think this is necessary with jest 22 https://facebook.github.io/jest/blog/2017/12/18/jest-22.html


Remove the back button on secure conversations chat transcript

SORRY THAT THIS TOOK SO LONG :(
The current logic correctly identifies when to show the secure conversations title or the chat title. This logic is encapsulated into its own function so that it can be used to determine back button behavior. The secure chat transcript header style is the same as the regular chat style but with the back button set as nil.
MOB-1888

PR will be closed because it breaks back button on live chat.


Remove opsworks provider that is now EOL

Description of your changes
On May 26, 2024, AWS completed their deprecation cycle for the opsworks service and it is now completely EOL for both new and existing customers. Nobody can use it anymore.
This removes the resources from the provider to shrink our maintenance burden, build time, image sizes, etc.

Fixes #1224
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

can we add a message in Release Notes that people can remove the CRDs for opsworks ?

Maybe we release this in the same version that stops shipping the monolithic provider? It's definitely worth mentioning in the release notes, but given what I've learned about opsworks while reading the deprecation docs, I'd be somewhat surprised if anyone is installing provider-aws-opsworks.


Fixed link to blob patterns

@dchristensen : Thanks for your contribution! The author, @ggailey777, has been notified to review your proposed change.

@dchristensen thanks for taking the time to fix this anchor link! #sign-off


continue running despite errors

should be useful for https://github.com/cockroachdb/cockroach/issues/3087 and in general.

LGTM


Add info about submissions to working group page

You can preview this change here:
Built with commit 9bb427554eb2d8c715519d1d8c5f23be675aabe8
https://deploy-preview-1563--govuk-design-system-preview.netlify.app

Added @EoinShaughnessy as a reviewer for this update.
We probably need a subject matter expert on the rules and governance of the working group (Community Manager?). I'll raise in standup.

Hi @trang-erskine, when you get a chance can you take a look at the text I added in 'Files changed'?
Once you're happy this is ready to merge - however will need some dev help as there's a failed test and the commits need squashing.


The login request is not logged

Summary
In client log file there is no logged request for logging (/j_security_check endpoint).
In case of bad request, there is no way to debug it from log file.
Logs and external information
No response
Reproduction script
session = create_vManageSession(...)

On what platform(s) does it occur?
Not Applicable
Issue Severity
None

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/issues/146


Add a github workflow to validate GEDCOM.io test files

To prevent regressing the validator

Also add a workflow on ubuntu to run the harden-runner workflow?


Autonomous mode does not seem to update Pose X or Y correctly

My Teleoperate mode seems to work fine.  Pose X, Y seem to be updating correctly.   When I run in autonomous the Pose values seem to update much slower/less .  This causes the control loop to increase the velocity much higher than the max level.  I have it set to 1m/s but it is going ~10m/s sec because the Pose update is a fraction of what it should be.  This could be operator error? I am new to this lib and to pathplanner.  I also tried to create a very simple testpath.path  (move robot 1m straight line) . I notice when the wheels are spinning very fast in Autonomous the Pose X,Y are update very little.   Could it be a scaling issue?  I am running MK4 L2, with Falcon 500 , cancoders.  Navx2 gyro.  The robot is on a cart while testing.   Thank you for publishing this lib!

It appears the Pose Y distance calculation is opposite of the motor encoder.  I tried flipping the SwerveModuleConstants but it does not appear to help.   I will need to keep looking at this.   Any suggestions would be appreciated.  I think it is a setup error.

Are you able to share a link to your GitHub? Also, if you can share the wpilog file from the simple test path, that would be very useful.
If the motor encoder was out of phase with the motor, that would impact the pose estimation in both teleop and auto. It is strange that pose estimation works in one mode and not the other since there is little different about those two code paths....

Thank you for your help!    This is the link to my repository.  https://github.com/FRC7224/FRC2023B
With some configuration I think the orientation is correct.  When the robot moves forward, I get a positive x?  I am still having trouble. with  PathPlanner.   I do not think it is creating the trajectory file correctly. I tried to create a very simple path with testpaths1 and it causes the roborio to crash.  testpaths8 (original) is in the code so it runs.   When I run autonomous it goes open loop and way to fast.  Most of these tools are new to me Advantagescope and  Pathplanner.   Any help would be appreciated!!

The Navx may report clockwise as the positive direction when installed in a roboRIO "facing up". If so, that will need to be inverted as the code expects counter clockwise as the positive direction.

Geoff,
Yes, I just noticed that this morning. (Thank  you)  But I don't think this
would cause the path software to go open loop.   I am trying to move in a
straight line 1m.  (keep it simple to start)
Marty
On Mon, Jan 23, 2023 at 1:10 PM Geoff Schmit @.***>
wrote:

The Navx may report clockwise as the positive direction when installed in
a roboRIO "facing up". If so, that will need to be inverted as the code
expects counter clockwise as the positive direction.
—
Reply to this email directly, view it on GitHub
https://github.com/HuskieRobotics/3061-lib/issues/25#issuecomment-1400768063,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSDXB7FOTELCQ4KMC2XTLDWT3CRJANCNFSM6AAAAAAT76UBYU
.
You are receiving this because you authored the thread.Message ID:
@.***>

I agree about the NavX, hopefully, the previous suggestions will help address the autonomous issue.

Thank you for your suggestions! I believe I have the orientation correct now.   I am still having velocity issues in autonomous mode.  I have the AUTO_MAX_SPEED_METERS_PER_SECOND set to 1.0 and I also set it to 1.0 in the path file.   But I am seeing 4.5  m/s.   (note: I have the robot on a cart )

Would you send me the log file where you see this behavior? I can replay it in AdvantageScope against your code in GitHub and have a better sense of what is going on.

I have not been able to get the file saving to work.   I get an error
failed to open media/sda1
Does the flash drive need to be formatted any special way?
I exported from advantage scope program  and the file is attached.
Thank you!
On Sat, Jan 28, 2023 at 12:51 PM Geoff Schmit @.***>
wrote:

Would you send me the log file where you see this behavior? I can replay
it in AdvantageScope against your code in GitHub and have a better sense of
what is going on.
—
Reply to this email directly, view it on GitHub
https://github.com/HuskieRobotics/3061-lib/issues/25#issuecomment-1407450454,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSDXB72IAX5JHIIVJ7DPTLWUVMDJANCNFSM6AAAAAAT76UBYU
.
You are receiving this because you authored the thread.Message ID:
@.***>

I don't think attachments work when responding to GitHub issues. Would you add the file to your GitHub and let me know which file to look at and which branch (if not main) to replay with?

Did you resolve the issue? Assuming so, what was the issue in case others experience similar problems.

7224test2.zip

Note: had to zip it for git to take it
Yes, your directions were helpful.  I swapped some signs and setup values in the swerve constants in order to get the orientation correct.

A potential insight, a tip, and a question.
I think the log file that you sent for testPaths8 drove the wheels at full speed because the robot was on a cart and wasn't able to rotate. The AUTO_MAX_SPEED_METERS_PER_SECOND constant impacts the path that is generated, but it doesn't limit the PID controllers used to try and keep the robot on the auto path. Since the robot is on a cart and cannot rotate, the angle reported by the gyro and the angle specified in the path end up way off. Therefore, the PID controller for the auto path tries to make the robot turn faster by increasing the wheel speeds well beyond 1 m/s. I expect that if you run the testPaths1 copy auto path, the robot will drive at the expected speed on the cart. In addition, now that the geometry is consistent, running the auto paths with the robot on the ground should behave as expected.
I believe the flash drive for logging is just formatted as FAT32. I will have to check at school to verify. If you are using a roboRIO 2, I believe the path is /media/sda1 if the flash drive is in the port closer to the edge of the roboRIO. You can insert the flash drive, ftp into the roboRIO, and list the contents of the media directory to see where it was mounted.
Just to confirm since you closed this issue. Is everything working as expected now?

Thanks this helps!   I will try to create  a path that has the robot driving straight.  This way we will not have any gyro error.  I would like to get the autonomous working more controlled before I take it off the cart.    I can move it to a more open space at the school Monday to try it.   In past year I have had robots drive into the wall.  They can accelerate faster than my space bar reaction. I am running a  roborio 1.0.  I did get it to save a file ones.  A second time it seemed to corrupt the flash drive.  I did not attend to close the issue but it ok.  The initial issue  is resolved.   The Pose values are correct. The information you gave on making sure the configuration  is correct was helpful. You may want to add it to the readme.Marty GubowOn Jan 28, 2023, at 6:36 PM, Geoff Schmit @.***> wrote:
A potential insight, a tip, and a question.
I think the log file that you sent for testPaths8 drove the wheels at full speed because the robot was on a cart and wasn't able to rotate. The AUTO_MAX_SPEED_METERS_PER_SECOND constant impacts the path that is generated, but it doesn't limit the PID controllers used to try and keep the robot on the auto path. Since the robot is on a cart and cannot rotate, the angle reported by the gyro and the angle specified in the path end up way off. Therefore, the PID controller for the auto path tries to make the robot turn faster by increasing the wheel speeds well beyond 1 m/s. I expect that if you run the testPaths1 copy auto path, the robot will drive at the expected speed on the cart. In addition, now that the geometry is consistent, running the auto paths with the robot on the ground should behave as expected.
I believe the flash drive for logging is just formatted as FAT32. I will have to check at school to verify. If you are using a roboRIO 2, I believe the path is /media/sda1 if the flash drive is in the port closer to the edge of the roboRIO. You can insert the flash drive, ftp into the roboRIO, and list the contents of the media directory to see where it was mounted.
Just to confirm since you closed this issue. Is everything working as expected now?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID: @.***>


Add --iree-flow-demote-i64-to-i32 and --iree-flow-demote-f64-to-f32

Add --iree-flow-demote-i64-to-i32 (default false) and --iree-flow-demote-f64-to-f32 to compile MiniLM via Torch-mlir and run on IREE.

See upstream issue https://github.com/google/iree/issues/9087#issuecomment-1135244663

<stdin>:295:11: error: failed to materialize conversion for result #0 of operation 'hal.interface.binding.subspan' that remained live after conversion
    %55 = tensor.expand_shape %54 [[0], [1, 2, 3]] : tensor<1x?xf32> into tensor<1x1x1x?xf32>
          ^
<stdin>:23:3: note: called from
  func.func @forward(%arg0: tensor<1x128xi64>) -> tensor<1x2xf32> {
  ^
<stdin>:315:11: note: see existing live user here: %19 = memref.subview %13[0, 0, 0, %18] [1, 1, 1, %5] [1, 1, 1, 1] : memref<1x1x1x?xf32> to memref<?xf32, affine_map<(d0)[s0] -> (d0 + s0)>> loc(callsite("<stdin>":315:11 at "<stdin>":23:3))
    %75 = tensor.extract_slice %55[0, 0, 0, %64] [1, 1, 1, %74] [1, 1, 1, 1] : tensor<1x1x1x?xf32> to tensor<1x1x1x?xf32>
          ^
<stdin>:325:11: error: failed to run translation of source executable to target executable for backend #hal.executable.target<"vulkan", "vulkan-spirv-fb", {spv.target_env = #spv.target_env<#spv.vce<v1.3, [Shader, GroupNonUniform, GroupNonUniformVote, GroupNonUniformArithmetic, GroupNonUniformBallot, GroupNonUniformShuffle, GroupNonUniformShuffleRelative], [SPV_KHR_storage_buffer_storage_class]>, SwiftShader:CPU, {cooperative_matrix_properties_nv = [], max_compute_shared_memory_size = 16384 : i32, max_compute_workgroup_invocations = 128 : i32, max_compute_workgroup_size = dense<[128, 128, 64]> : vector<3xi32>, subgroup_size = 4 : i32}>}>
    %78 = linalg.generic {indexing_maps = [#map3, #map5, #map4], iterator_types = ["parallel", "parallel", "parallel", "parallel"]} ins(%77, %cst_6 : tensor<1x1x1x?xf32>, tensor<f64>) outs(%76 : tensor<1x1x1x?xf32>) {
          ^
<stdin>:23:3: note: called from
  func.func @forward(%arg0: tensor<1x128xi64>) -> tensor<1x2xf32> {
  ^
<stdin>:325:11: error: failed to serialize executables
    %78 = linalg.generic {indexing_maps = [#map3, #map5, #map4], iterator_types = ["parallel", "parallel", "parallel", "parallel"]} ins(%77, %cst_6 : tensor<1x1x1x?xf32>, tensor<f64>) outs(%76 : tensor<1x1x1x?xf32>) {
          ^
<stdin>:23:3: note: called from
  func.func @forward(%arg0: tensor<1x128xi64>) -> tensor<1x2xf32> {
  ^
compilation from source to vm failed

Invoked with:
 iree-compile /home/prashant/SHARK/shark.venv/lib/python3.9/site-packages/iree/compiler/tools/../_mlir_libs/iree-compile - --iree-input-type=none --iree-vm-bytecode-module-output-format=flatbuffer-binary --iree-hal-target-backends=vulkan --iree-mlir-to-vm-bytecode-module --iree-llvm-embedded-linker-path=/home/prashant/SHARK/shark.venv/lib/python3.9/site-packages/iree/compiler/tools/../_mlir_libs/iree-lld --mlir-print-debuginfo --mlir-print-op-on-diagnostic=false --iree-llvm-target-cpu-features=host --iree-flow-demote-i64-to-i32 --iree-flow-demote-f64-to-f32


BoundedVec MaxEncodedLen microoptimization

@shawntabrizi pointed out that the max encoded length of a bounded struct can be limited by the encoded size of the bound, not the max possible value for the bound's type.

bot merge


Cleanups to dvol

While @lukemarsden, @exarkun and I were working on prototyping push & pull, we made a bunch of cleanups to dvol. This PR collects most of those cleanups.

Nice bunch of changes, a minor comment and a question, then I'll let @robhaswell have the final say

Thanks Jono, always good to see cleanups!
I'm not very happy with the way the JSON output functionality is progressing.  I feel like there is little to be gained by not setting up an output interface right now, as this functionality will have to be installed in every command, really you are just punting it immediately to the very next person who has to do any development. However I don't know what you were trying to achieve while refactoring so it might be acceptable to just punt this to the next person, if you are working on something mostly unrelated.
You can address my comments or just raise issues for them up to you. Otherwise these changes are a good improvement, thanks! Your build failure looks transient so you can merge when ready.

Removed the JSON output stuff, since it's not baked and I don't want it to block the rest of the changes in this PR.

Thanks Jono!


NIFI-5561 - Add component name filtering to S2S Provenance Reporting …

…Task
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[x] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Added VR scope on properties and fixed indentation to have the same everywhere. To see the changes and masking the indentation changes: https://github.com/apache/nifi/pull/2973/files?w=1

Thanks Pierre. Running a final check and will merge.


Extra GET after form POST which receives 422 response

I have a normal CRUD situation where a form in a Rails app POSTs to the server.  If the form is invalid, the server renders the page again along with a 422 status.  Immediately after the client receives the 422 response, it requests the whole page again via GET.  This has the effect of wiping the form state for the user.
I think this is an error because the docs say:

After a stateful request from a form submission, Turbo Drive expects the server to return an HTTP 303 redirect response, which it will then follow and use to navigate and update the page without reloading.
The exception to this rule is when the response is rendered with either a 4xx or 5xx status code. This allows form validation errors to be rendered by having the server respond with 422 Unprocessable Entity....

There are no turbo frames or streams on the page.
Steps to reproduce

Submit a form via POST.
In the controller, render the page again with a 422 status.  I'm using this:  render 'timetables/show', status: :unprocessable_entity

Expected: the page is rendered with the form errors.
Actual: Turbo immediately requests the page again via GET, leading to a fresh form object without any errors or previously entered values.

Logs
Rails:
Started POST "/slots/generate" for <IP_ADDRESS> at 2023-12-01 16:29:37 +0000
Processing by SlotsController#generate as TURBO_STREAM
  Parameters: {"authenticity_token"=>"[FILTERED]", "timetable_form"=>{...}}
  Rendering timetables/show.html.slim
  Rendered timetables/show.html.slim (Duration: 22.0ms | Allocations: 43409)
Completed 422 Unprocessable Entity in 104ms (Views: 23.0ms | ActiveRecord: 0.4ms | Allocations: 65747)

Started GET "/timetables/2023-12-21" for <IP_ADDRESS> at 2023-12-01 16:29:37 +0000
Processing by TimetablesController#show as HTML

Browser console:
POST http://<IP_ADDRESS>:3000/slots/generate 422 (Unprocessable Content)
perform @ turbo.min-ea604a9223…4fb3039c9fe73a.js:1
await in perform (async)
start @ turbo.min-ea604a9223…4fb3039c9fe73a.js:1
submitForm @ turbo.min-ea604a9223…fb3039c9fe73a.js:15
formSubmitted @ turbo.min-ea604a9223…fb3039c9fe73a.js:15
V.submitBubbled @ turbo.min-ea604a9223…4fb3039c9fe73a.js:1

Fetch failed loading: POST "http://<IP_ADDRESS>:3000/slots/generate".
perform @ turbo.min-ea604a9223…4fb3039c9fe73a.js:1
await in perform (async)
start @ turbo.min-ea604a9223…4fb3039c9fe73a.js:1
submitForm @ turbo.min-ea604a9223…fb3039c9fe73a.js:15
formSubmitted @ turbo.min-ea604a9223…fb3039c9fe73a.js:15
V.submitBubbled @ turbo.min-ea604a9223…4fb3039c9fe73a.js:1

Navigated to http://<IP_ADDRESS>:3000/timetables/2023-12-21

Other information
I have found that if I nest the form within a turbo frame, I get the behaviour I expect.
Environment
Rails 7.1.1
Turbo-rails 1.5.0

The problem turned out to be the 422 response was sent with a different layout from the original page.
So the fix looked like:
-  render 'timetables/show', status: :unprocessable_entity
+  render 'timetables/show', layout: 'timetables', status: :unprocessable_entity


Notebook code cell mutli-line highlighting is offset

Highlighting when selecting multiple lines is offset - including some extra characters that aren't actually selected.

Not a regression. Moving to October.


Use /bin/bash as the exec shell on Unixes

Fix #14 by avoiding Dash on Ubuntu systems.
As it happens, this module was working just fine, but the exec shell was exiting in a way that didn't match the exit of its last job.

Once the travis-ci finishes, this should be good to go.  0.10 is failing on Travis because 0.10 does not allow one to set the shell used by cp.exec(), so it's removed in that last commit.  Everything else was passing, so it should be good to go.

awesome \o/


chore(main): release 0.7.12

:robot: I have created a release beep boop
0.7.12 (2022-09-26)
Bug Fixes

Pipelines: Small UX improvements on the run page (a3fbd4e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/BLSQ/openhexa-frontend/releases/tag/0.7.12 :sunflower:


Fix python packaging

Hi,
There have been a few changes in a core Python package that means there is a need to cut a new release of this extension. Until this is done, I will have to remove this package from the main chaostoolkit.org site and documentation as I cannot build it otherwise.
ping @dgzlopes

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


Unable to recognize PCM data in player

I want to use the speech-to-text function in the player, but SPEECH-SDK cannot recognize the sampled PCM data.
The following is the recognition result：
System.out: SpeechUtil startContinuousRecognitionAsync.
System.out:  SpeechUtil Session started event.
System.out: SpeechUtil NOMATCH: Speech could not be recognized.
SpeechUtil: code:SessionId:45da5eead3714f508c128e02dece913a ResultId:36fe080c02d84d05a0a894ab67cc1efe Reason:NoMatch Recognized text:<>.
System.out: SpeechUtil CANCELED: Reason=EndOfStream
System.out: SpeechUtil CANCELED: ErrorCode=NoError
System.out:  SpeechUtil Session stopped event.

Hello，I save the PCM data in the player to a file, then convert the pcm file to a WAV file, and use ffmprobe to view the format。
***deMini Downloads % ffprobe output.wav
ffprobe version 5.0.1 Copyright (c) 2007-2022 the FFmpeg developers
  built with Apple clang version 13.0.0 (clang-13<IP_ADDRESS>)
  configuration: --prefix=/opt/homebrew/Cellar/ffmpeg/5.0.1_2 --enable-shared --enable-pthreads --enable-version3 --cc=clang --host-cflags= --host-ldflags= --enable-ffplay --enable-gnutls --enable-gpl --enable-libaom --enable-libbluray --enable-libdav1d --enable-libmp3lame --enable-libopus --enable-librav1e --enable-librist --enable-librubberband --enable-libsnappy --enable-libsrt --enable-libtesseract --enable-libtheora --enable-libvidstab --enable-libvmaf --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxml2 --enable-libxvid --enable-lzma --enable-libfontconfig --enable-libfreetype --enable-frei0r --enable-libass --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-libspeex --enable-libsoxr --enable-libzmq --enable-libzimg --disable-libjack --disable-indev=jack --enable-videotoolbox --enable-neon
  libavutil      57. 17.100 / 57. 17.100
  libavcodec     59. 18.100 / 59. 18.100
  libavformat    59. 16.100 / 59. 16.100
  libavdevice    59.  4.100 / 59.  4.100
  libavfilter     8. 24.100 /  8. 24.100
  libswscale      6.  4.100 /  6.  4.100
  libswresample   4.  3.100 /  4.  3.100
  libpostproc    56.  3.100 / 56.  3.100
Input #0, wav, from 'output.wav':
  Metadata:
    encoder         : Lavf59.16.100
  Duration: 00:00:23.22, bitrate: 256 kb/s
  Stream #0:0: Audio: pcm_s16le ([1][0][0][0] / 0x0001), 16000 Hz, 1 channels, s16, 256 kb/s

Why can't it be recognized？

@Zhengyi66 Thank you for using Speech SDK. I'm Darren from the SDK team. To help you I will need more information. But before that, please make sure you viewed all the relevant samples in this GitHub repository. First run the closest provided sample to your scenario, and make sure it works.
If the sample(s) run fine on your platform, but your code still does not, please provide the following:

What language is spoken? The default recognition language is American English (en-us). If you are trying to recognize any other language, you need to set the right language code (see quickstart documentations)
Your Operating System and programming language
Share you code (or parts of it) if possible
In particular, how do you feed audio into the SDK? Are you providing WAV file name? Are you feeding audio buffers to the SDK via an input stream? if so, is it a push or pull input stream? (based on your short description above, sounds like you are using an input stream).
Provide a short WAV file representing the audio data you send into the SDK
Turn on SDK logging while you do speech recognition and share the log file. See instructions here.
Make sure you use the latest SDK version (1.22).
16bit/sample (little endian), mono, 16khz sample rate is the default audio input stream format. I think that's what your tool shows, after dumping the audio to a WAV file. So perhaps there is some other issue.
Thanks, Darren.

Thank you for your help。I have found the problem according to your hint。
Have a nice weekend。


ci: make runtime privileged tests not run in parallel

There was a significant flakiness of IPSec-related privileged tests due to the fact that tests in different packages were modifying xfrm states/policies concurrently.
While increasing timeout for test and making it last longer is non-ideal, less flaky tests outweight it.
Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

/test

Rebased to pick up ingress/gateway changes

/test


Relationship Query Support

I've implemented support for Relationship queries. This will allow you to do a query such as
SELECT Id, FirstName, Account.Name FROM Contact

And access the data by fetching i.e.
contacts[0].Account.Name

You can also grab a reference to the Account object, and then pass it around as if you fetched it directly
contacts[0].Account

It works recursively, so you should be able to go as deep as SOQL supports.
I've implemented basic test coverage, and attempted to follow your coding style - but let me know if you'd prefer me to make any changes. I've also rebased off master, so it should be an easy merge.

Accepted and merged


Failed to compile<EMAIL_ADDRESS>

I use adonuxt. After update nuxt from 0.10.7 to 1.0.0-alpha4 try build.
ERROR  Failed to compile with 1 errors                                                                                15:29:11

This dependency was not found:

* !!vue-style-loader?sourceMap!css-loader?sourceMap!../../node_modules/vue-loader/lib/style-compiler/index?{"vue":true,"id":"data-v-54a40582","scoped":false,"hasInlineConfig":true}!sass-loader?sourceMap!../../node_modules/vue-loader/lib/selector?type=styles&index=0!./default.vue in ./resources/layouts/default.vue

To install it, you can run: npm install --save !!vue-style-loader?sourceMap!css-loader?sourceMap!../../node_modules/vue-loader/lib/style-compiler/index?{"vue":true,"id":"data-v-54a40582","scoped":false,"hasInlineConfig":true}!sass-loader?sourceMap!../../node_modules/vue-loader/lib/selector?type=styles&index=0!./default.vue

Please install Node Sass and the corresponding Webpack loader:
npm install --save node-sass sass-loader
or
yarn add node-sass sass-loader

If the nuxt is dependent on the node-sass and sass-loader, why not include them in dependencies so that they are installed automatically?

I don't think Nuxt is dependent on node-sass and sass-loader. This message prompts when you have a lang="scss" in one of your Vue files (component, layout or page). Is it your case?

Yes you are right. In one of the files accidentally left <style lang="scss"></style>. Thanks


feat(store): add additional cards to adorama, asus, bestbuy, b&h, evga, newegg, nvidia, zotac

Description

Adds additional 3080 and 3090 cards to adorama, asus, bestbuy, b&h, evga, newegg, zotac
Adds 3090 FE ID for en-us on Nvidia API
Updates existing 3090 cards from bestbuy with add to cart link
Adds intl=nosplash to existing 3090 links in bestbuy

Testing
Ran locally and verified that new cards were monitored
New dependencies
N/A

@jef Merged another PR into this one. This PR is now ready for review.

Wow... That's all of them 😮

LGTM!

Can you tell I still haven't snagged a 3000 card yet?

You'll get one

Thanks very much for the update!


DSD-170: Fixing and updating the documentation for the Pagination component

Fixes DSD-170
This PR does the following:

Updates the Storybook documentation for the Pagination component to be clearer and to use the MDX format.

Front End Review:

[ ] View the example in Storybook

Yeah, Maybe I'll move that to the main component file or the doc file. Will update.


create function sql format error

create function get_film_count(len_from int, len_to int)
returns int
language plpgsql
as
$$
declare
   film_count integer;
begin
   select count(*)
   into film_count
   from film
   where length between len_from and len_to;

   return film_count;
end;
$$;

After format:
create function get_film_count(len_from int, len_to int)
returns int
language plpgsql
as
$ $ -- there is a space between dollar
declare
   film_count integer;
begin
   select count(*)
   into film_count
   from film
   where length between len_from and len_to;

   return film_count;
end;
$ $; -- there is a space between dollar

Stored procedures are currently completely unsupported.
So sorry, this likely won't be fixed in any near term.

Stored procedures are currently completely unsupported.
So sorry, this likely won't be fixed in any near term.

Would it be possible to avoid formatting those blocks? That way I can still run the sql formatter but it won't break my sql migrations.

Would it be possible to avoid formatting those blocks? That way I can still run the sql formatter but it won't break my sql migrations.

Well... for start, could you tell me which dialect of SQL this is?

Well... for start, could you tell me which dialect of SQL this is?

Postgres, always begins with create (or replace) function ...

Postgres, always begins with create (or replace) function ...

So, for Postgres I've just added support for dollar-quoted strings. So that should take care of the formatter touching anything inside $$...$$ block. Although it'll format the rest of this one one long line, as it knows nothing of functions.
You can try the latest beta here: https://zeroturnaround.github.io/sql-formatter/

So, for Postgres I've just added support for dollar-quoted strings. So that should take care of the formatter touching anything inside $$...$$ block. Although it'll format the rest of this one one long line, as it knows nothing of functions.
You can try the latest beta here: https://zeroturnaround.github.io/sql-formatter/


⚠ DesiTorrents has degraded performance

In 5f7d905, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 1767 ms

Resolved: DesiTorrents performance has improved in 1a16145 after 25 minutes.


Team is not yet configured for notarization.

I am trying to notarize my electron app with this package, however, i keep getting this error:
Error: Unrecognized notarization status: "rejected:
after some more digging, its because apple is returning:
Team is not yet configured for notarization.
anyone encounter a similar issue, it was working fine until just today, very odd.

@gv1122 did you solve the issue?

@gv1122 This normally means you have new things to "sign" in your apple developer account.  Agreements, new terms of use, etc.


Large numbers of css declarations balloon js output file size.

It appears that the use of @inline in scalacss can cause unregulated growth.  I have an (unfortunately proprietary) project that I'm attempting to optimize.  It contains a large number (33) of instances of StyleSheet.Inline, with a total of 109 uses of style.  Compilation size (due to many factors which I'm currently investigating), is at 2.7mb.
A serious culprit visible while eyeballing fastOptJS output is inline expansion of a loop somewhere in the style definitions:
var x$58 = $as_Lscalacss_AVs(jsx$143);
  $m_Lscalacss_StyleS$();
  var c$29 = $m_Lscalacss_Cond$().empty$1;
  var d$27 = new $c_sci_Map$Map1().init___O__O(c$29, x$58);
  var this$431 = $m_s_package$().Vector$1;
  var jsx$146 = this$431.NIL$6;
  var jsx$145 = $m_s_None$();
  var this$432 = $m_s_package$().Vector$1;
  var jsx$144 = this$432.NIL$6;
  var this$433 = $m_s_package$().Vector$1;
  var jsx$142 = new $c_Lscalacss_DslBase$ToStyle().init___Lscalacss_StyleS(new $c_Lscalacss_StyleS().init___sci_Map__sci_Vector__s_Option__sci_Vector__sci_Vector(d$27, jsx$146, jsx$145, jsx$144, this$433.NIL$6));
  this.dsl__Lscalacss_mutable_StyleSheet$Base$dsl$();
  this.dsl__Lscalacss_mutable_StyleSheet$Base$dsl$();
  var this$435 = $m_Lscalacss_Attrs$cursor$();
  var x$59 = new $c_Lscalacss_AV().init___Lscalacss_Attr__T(this$435.attr$2, "pointer");
  var this$437 = $m_Lscalacss_AVs$();
  var array$32 = [];
  var this$438 = this$437.apply__Lscalacss_Attr__T__Lscalacss_AVs(x$59.attr$1, x$59.value$1);
  var start$30 = 0;
  var end$30 = $uI(array$32.length);
  var z$30 = this$438;
  x$30: {
    var jsx$138;
    _foldl$30: while (true) {
      if ((start$30 === end$30)) {
        var jsx$138 = z$30;
        break x$30
      } else {
        var temp$start$30 = ((1 + start$30) | 0);
        var arg1$36 = z$30;
        var index$34 = start$30;
        var arg2$30 = array$32[index$34];
        var x$1$43 = $as_Lscalacss_AVs(arg1$36);
        var x$2$28 = $as_Lscalacss_AV(arg2$30);
        var temp$z$30 = x$1$43.add__Lscalacss_Attr__T__Lscalacss_AVs(x$2$28.attr$1, x$2$28.value$1);
        start$30 = temp$start$30;
        z$30 = temp$z$30;
        continue _foldl$30
      }
    }
  };

The string class__defaults__Lscalacss_Defaults__Lscalacss_IDefaults occurs 669 times in this output.
Disabling all scalajs optimizations with scalaJSOptimizerOptions ~= { _.withDisableOptimizer(true) } reduces the file size by ~700kb and now class__defaults__Lscalacss_Defaults__Lscalacss_IDefaults only appears twice.
This happens to be scalacss 0.3.0, so my next step will be migrating scalacss and scalajs-react to at least get us to the latest release, but I'm fairly confident given the amount of @inlines in the codebase that this is still an issue.

So today I cloned 0.3.0 and removed all @inline declarations, and ran a regular fullOptJS with optimizations on.  Sizes are as follows:
scalacss 0.3.0 (maven) - 2,763kb   (637kb gzip)
scalacss 0.3.0-noinline - 2,328kb (488kb gzip)
So there is definitely something up here.

That's fantastic @mseddon! I think removing all the (non-forwarder) @inlines would be a great idea. This lib only runs for a tiny amount of time at startup; space is much more important than time.

That sounds about right, in fact keeping the forwarders may even shave off a few more kilobytes :)
If I find the time I'll try to send a PR sometime next week, unless someone else beats me to it.

Hey @mseddon. I fixed this in 9815ebfbea35b242f49e0844417173767f1ce9ca.
Shaved 40K off fullOptJS size in a small demo.
Do you wanna try this with your project? Or, are you good to close this issue?

Oh hey- sorry I've been really snowed recently, so didn't have a chance to put that fix together- the changes you made look like they address the real areas where I see the inlines explode.  Sadly to test it with our project, we'd need to upgrade scalacss, and scalajs-react.  Once this is done, I'll re-test and see how well it works, but for now, I'm happy to close- thanks!


Failed to Install PX4 in ROS

I'm new with this, i have tried the instruction in this link https://dev.px4.io/v1.9.0/en/setup/dev_env_linux.html#ros
and sourcing this https://raw.githubusercontent.com/PX4/Devguide/v1.9.0/build_scripts/ubuntu_sim_ros_melodic.sh
after following the installation, i found 4 packages were abandoned and 2 packages failed

how to solve this? help me please

Duplicate of https://github.com/PX4/Firmware/issues/14984. Please have a look at this issue, as the problem is related to the fact that rosdep doesn't get installed on the scripts, requiring an update to these.


Add a new function Replace to Table.enso

As a user, I want a way to match a table against a lookup_table on a key value and then to specify which column will be replaced in the original table. This is differentiated from merge because it applies specifically to one column, rather than to all columns with matching names where the key_value matches.
A scaffold of the API is below:
    ## ALIAS find replace
       GROUP Standard.Base.Calculations
       ICON join
       Finds matching values in a column or set of columns from table based on
       a lookup table, matching to values from `match_column` and replacing
       with `value_column`
       This operation is similar to `Table.update_rows`, but just returns a new
       `Table` instance, instead of updating the table in-place (which is only
       possible for Database tables).

       Arguments:
       - columns: Specifies the columns to match with the lookup table.
       Must identify values uniquely within `lookup_table`.
       - lookup_table: The table to use for looking up values.
       - allow_unmatched_rows: Specifies how to handle missing rows in the lookup.
         If `False` (the default), an `Unmatched_Rows_In_Lookup` error is raised.
         If `True`, the unmatched rows are left unchanged. Any new columns will
         be filled with `Nothing`.
       - on_problems: Specifies how to handle problems if they occur, reporting
         them as warnings by default.

       ? Result Ordering

         When operating in-memory, this operation preserves the order of rows
         from this table (unlike `join`).
         In the Database backend, there are no guarantees related to ordering of
         results.

       ? Error Conditions

         - If this table or the lookup table is lacking any of the columns
           specified in `columns`, a `Missing_Input_Columns` error is raised.
         - If an empty vector is provided for `columns`, a
           `No_Input_Columns_Selected` error is raised.
         - If a single row is matched by multiple entries in the lookup table,
           a `Non_Unique_Key` error is raised.
         - If a column that is being updated from the lookup table has a type
           that is not compatible with the type of the corresponding column in
           this table, a `No_Common_Type` error is raised.
         - If a match_column contains `Nothing` values, either in the table or lookup table,
           a `Null_Values_In_Key_Columns` error is raised.
         - If `allow_unmatched_rows` is `False` and there are rows in this table
           that do not have a matching row in the lookup table, an
           `Unmatched_Rows_In_Lookup` error is raised.
         - The following problems may be reported according to the `on_problems`
           setting:
           - If any of the `columns` is a floating-point type,
             a `Floating_Point_Equality`.
    @columns Widget_Helpers.make_column_name_selector
    replace : (Vector (Integer | Text | Regex) | Text | Integer | Regex) -> Table -> Text | Integer -> Text | Integer -> Problem_Behavior -> Table ! Missing_Input_Columns | Non_Unique_Key | Unmatched_Rows_In_Lookup
    replace self columns:(Text | Integer) lookup_table:Table  match_column:(Text | Integer)=0 value_column:(Text | Integer)=1 on_problems:Problem_Behavior=Problem_Behavior.Report_Warning =
        _ = [lookup_table, field, find_value, replace, on_problems]
        Error.throw (Unsupported_Database_Operation.Error "Table.replace is not implemented yet for the Database backends.")```

Should also have a conversion from a Map to a Table added in.

Is match_column always one column, or do we allow multiple?

Finds matching values in a column or set of columns from table based on
a lookup table, matching to values from match_column and replacing
with value_column

Does this imply that we might compare multiple columns from the main table with a single column in the lookup table? Or is it a straightforward multi-column aggregate key comparison?


Stationary and non-stationary timeseries

Prophet works for any kind of dataset or not? By any kind I mean  that Input dataset to prophet should be Stationary time-series or it works on any kind of dataset?

Prophet works well on datasets that have piecewise trends and weekly/yearly seasonality. The examples in the documentation are pretty typical of the types of time series that Prophet works well for, with strong, consistent seasonalities and trends that can reasonably be modeled with a piecewise function. The details of the model are given in the paper. Does that help?

I have the same doubt

I don't think Prophet will work on non stationary series. It only models trend and seasonality (at different scales). So if working with non stationary series we will have to use methods to make the series stationary such as differentiation.

Non-stationarity in the trend would be find, since it anyway fits a piecewise trend. Non-stationarity in seasonality (i.e., changing seasonality from year to year) would not be fit by the model, unless it was due to a stationary but multiplicative effect.

Suppose the true model is d Y(t) = a + beta*x(t) + e(t), where x(t) is a stationary exogenous regressor and e(t) is random error, and d Y(t) = Y(t) - Y(t-1). If you plug Y(t), and x(t) with add_regressors(), do you think it would work?
My regressors are stationary and dependent variable is not, will this method be able to handle it? If you had an option to specify differencing, then you could do fitting on d Y() then integrate back to Y(t).

@argku The model is
y(t) = trend(t) + seasonality(t) + beta * extra_regressor(t) + noise

so if Y(t) - a + beta*x(t) + e(t), then yes that would be well captured by the model. If extra_regressor(t) is stationary and y(t) is not, will it be able to capture it? It depends on what type of non-stationarity it is. trend(t) is non-stationary, so if a piecewise linear/logistic trend is sufficient to capture the nonstationarity then it will work. Otherwise (including if the non-stationarity is in seasonality(t)), it may not fit the time series well.


Is it possible to compute fun and jacobian at the same time in nonlinear constraint?

According to the documentation, in evaluating objective function f( ), one can evaluate both the function f(x) and grad(f(x)). This can be done by setting boolean value of jac true.
jac{callable, ‘2-point’, ‘3-point’, ‘cs’, bool}, optional

Similarly, I want to compute both constraint function and its jacobian in evaluating nonlinear inequality constraints. However, according to the documentation, no such thing seems to be an option.
jac{callable, ‘2-point’, ‘3-point’, ‘cs’}, optional \\ no boolean!

Is there any way to realize this?

Looking at the source code, I found the Jacobian seems to be computed right after the evaluation of the constraint function. Thus, the workaround would be setting global variable (or making class for that) and writing the code like below:
import numpy as np
from scipy.optimize import *

global jac_cache
jac_cache = None

def ineq_fun(x):
    fx = np.array([1 - x[0] - 2*x[1], 1 - x[0]**2 - x[1], 1 - x[0]**2 + x[1]])
    global jac_cache
    jac_cache = np.array([[-1.0, -2.0], [-2*x[0], -1.0], [-2*x[0], 1.0]])
    return fx

def ineq_jac_fun(x):
    global jac_cache
    if jac_cache is None:
        raise Exception
    return jac_cache

ineq_cons = {'type': 'ineq',
             'fun' : ineq_fun,
             'jac' : ineq_jac_fun}
eq_cons = {'type': 'eq',
           'fun' : lambda x: np.array([2*x[0] + x[1] - 1]),
           'jac' : lambda x: np.array([2.0, 1.0])}

bounds = Bounds([0, -0.5], [1.0, 2.0])
x0 = np.array([0.5, 0])
res = minimize(rosen, x0, method='SLSQP', jac=rosen_der,
               constraints=[eq_cons, ineq_cons], options={'ftol': 1e-9, 'disp': True},
               bounds=bounds)
print(res.x)

Rather than using global variable and class, I finally found using closure like below is more concise and easier to incorporate it into my existing codes:
def generate_ineq_funs():
    member = {'jac_cache': None}
    ineq_jac_fun = lambda x: member['jac_cache']

    def ineq_fun(x):
        fx = np.array([1 - x[0] - 2*x[1], 1 - x[0]**2 - x[1], 1 - x[0]**2 + x[1]])
        member['jac_cache'] = np.array([[-1.0, -2.0], [-2*x[0], -1.0], [-2*x[0], 1.0]])
        return fx

    return ineq_fun, ineq_jac_fun