Closes #452 - button hover effect transition

Fixes #452
Figure out how to transition CSS gradients for the button hover effect.
Done so without transitioning gradient specifically but the same hover effect behavior.
...
Checks

[* ] Ran npm run test-build

Changes proposed in this pull request:

Hey @mbao01! Thanks for the PR! I haven't tested this but it looks like it works. Don't know if the negative z-index on the ::before element would mean that it would be behind the button, and if that's the case, doesn't that mean the effect wouldn't show?

Ps: You can ignore hound for now. Still working out the kinks. 😄 The rules it recommends are too strict for anyone to follow, plus I don't think most of them make sense - at least not for this project.

Hey @mbao01! Thanks for the PR! I haven't tested this but it looks like it works. Don't know if the negative z-index on the ::before element would mean that it would be behind the button, and if that's the case, doesn't that mean the effect wouldn't show?

Not exactly @imolorhe, because the z-index is a negative value, the background+content on the element::before selector get's positioned between the background+borders of the element and it's content

I'm don't fully understand why the build on this pr is failing. @imolorhe

@mbao01 I don't fully understand the problem either. It has been happening intermittently on some PRs. Still investigating it.

@mbao01 Can you confirm that you tested the change? If so, I would merge it.

@mbao01 Can you confirm that you tested the change? If so, I would merge it.

I tested the change and it works @imolorhe


Missing: Metronome marks

This will be implemented in 31c-MetronomeMarks.xml.

I'm working on this next.

@rvilarl, in the lilypond example, which marks would you consider the most commonly used?

Those with note equal number

This is how vexml renders the lilypond test, which I think covers most cases.

Ah, I didn't pay close attention to the beat values — they seem wrong. I'm going to take a closer look. Sorry about the noise.


Re-open #3198

Can we re-open #3198? It was closed for inactivity, but was not fixed (AFAIK) and is a popular demand (see https://react-native.canny.io/feature-requests/p/add-overflow-support-to-android)

The bot left some next steps as a comment on #3198 that can help get it re-opened.

@all does RN fixed the overflow prop on android?


OSError: [Errno 22] Invalid argument

I am getting an OSError when running the example. It evidently can see the server, but can't open the socket?
python3 natnet-client-demo.py --server <IP_ADDRESS>
Connecting to <IP_ADDRESS>
Getting server info
Server application: Motive
Server version: Version(major=2, minor=2, build=0, revision=0)
Traceback (most recent call last):
  File "natnet-client-demo.py", line 97, in <module>
    main()
  File "natnet-client-demo.py", line 90, in main
    app = ClientApp.connect('fake' if args.fake else args.server, args.rate, args.quiet)
  File "natnet-client-demo.py", line 35, in connect
    client = natnet.Client.connect(server_name)
  File "/home/swarmtestbed/lvctestbed/ros2_ws/src/python_natnet/src/natnet/comms.py", line 418, in connect
    return cls._simple_connect(server, logger, timeout)
  File "/home/swarmtestbed/lvctestbed/ros2_ws/src/python_natnet/src/natnet/comms.py", line 401, in _simple_connect
    return cls._setup_client(conn, server_info, logger)
  File "/home/swarmtestbed/lvctestbed/ros2_ws/src/python_natnet/src/natnet/comms.py", line 344, in _setup_client
    conn.bind_data_socket(server_info.connection_info.multicast_address,
  File "/home/swarmtestbed/lvctestbed/ros2_ws/src/python_natnet/src/natnet/comms.py", line 60, in bind_data_socket
    self._data_socket.setsockopt(socket.IPPROTO_IP, socket.IP_ADD_MEMBERSHIP, mreq)
OSError: [Errno 22] Invalid argument

That sounds like a problem with your network interface: https://stackoverflow.com/questions/19403635/python-setsockopt-invalid-argument-with-ip-add-membership
Do the official SDK samples work on your machine?


N@tM Feedback

Feedback for night at the museum presentation, please put name before comment

evan pederson- Very good. There is a lot of code and it looks detailed. I would give great grade.

Game is very impressive. I can't believe that you were able to get that many projectiles coded onto the game. Really Good, Man. Keep up the good work!

Game was great, could tell lots of effort was put in. Notebook lesson was very informational. For the game's progression, not much changed, but great work. Aneesh

william windle- I like the cd bar and the bullets

really nice game, very unique and good job on the projectiles, maybe make the cooldown a bit as it's still really rapid. still really good and unique

Shay Mortensen - really nice game, very unique and good job on the projectiles, maybe make the cooldown a bit as it's still really rapid. still really good and unique

Yusuf Khan - Very nice interface! I like the added usage of the projectiles, it clearly demonstrated the outside learning that you did when not in class. This project went above and beyond, very nice!


rewrite in a systems programming language that runs blazingly fast, prevents segfaults, and guarantees thread safety.

🚫


🐛 [Immich] Installed add-on has been removed from repository

Description
Since recently (after installing 1.106.4-2) there is a message the the add-on has been removed from the repository.
Reproduction steps
1. Go to 'settings'
2. Click on 'Installed add-on has been removed from repository'
3.. See error: "Add-on Immich has been removed from the repository it was installed from. This means it will not get updates, and backups may not be restored correctly as the supervisor may not be able to build/download the resources required.

Clicking submit will uninstall this deprecated add-on. Alternatively, you can check Home Assistant help and the community forum for alternatives to migrate to.
"

Addon Logs
[06/17/24 18:50:55] INFO     Setting 'buffalo_l' execution providers to
                             ['CPUExecutionProvider'], in descending order of
                             preference
[06/17/24 18:50:55] INFO     Loading recognition model 'buffalo_l' to memory
[06/17/24 18:55:58] INFO     Shutting down due to inactivity.
[06/17/24 18:55:58] INFO     Shutting down
[06/17/24 18:55:58] INFO     Error while closing socket [Errno 9] Bad file
                             descriptor
[06/17/24 18:55:58] INFO     Waiting for application shutdown.
[06/17/24 18:55:58] INFO     Application shutdown complete.
[06/17/24 18:55:58] INFO     Finished server process [70646]
[06/17/24 18:55:58] ERROR    Worker (pid:70646) was sent SIGINT!
[06/17/24 18:55:58] INFO     Booting worker with pid: 173202
[06/17/24 18:56:11] INFO     Started server process [173202]
[06/17/24 18:56:11] INFO     Waiting for application startup.
[06/17/24 18:56:11] INFO     Created in-memory cache with unloading after 300s
                             of inactivity.
[06/17/24 18:56:11] INFO     Initialized request thread pool with 12 threads.
[06/17/24 18:56:11] INFO     Application startup complete.
[Nest] 1101  - 06/17/2024, 6:58:25 PM     LOG [Microservices:MediaService] Successfully generated JPEG image preview for asset 8d62a3b7-eb38-416e-87d4-72b55ac7478d
[Nest] 1101  - 06/17/2024, 6:58:25 PM     LOG [Microservices:MediaService] Successfully generated WEBP image thumbnail for asset 8d62a3b7-eb38-416e-87d4-72b55ac7478d
[06/17/24 18:58:25] INFO     Setting 'buffalo_l' execution providers to
                             ['CPUExecutionProvider'], in descending order of
                             preference
[06/17/24 18:58:25] INFO     Loading detection model 'buffalo_l' to memory
[06/17/24 18:58:25] INFO     Setting 'ViT-B-32__openai' execution providers to
                             ['CPUExecutionProvider'], in descending order of
                             preference
[Nest] 1101  - 06/17/2024, 6:58:25 PM     LOG [Microservices:MediaService] Successfully generated JPEG image preview for asset db59c2f1-8b1f-4a84-bfaa-7b222a403f83
[06/17/24 18:58:25] INFO     Loading visual model 'ViT-B-32__openai' to memory
[Nest] 1101  - 06/17/2024, 6:58:26 PM     LOG [Microservices:MediaService] Successfully generated WEBP image thumbnail for asset db59c2f1-8b1f-4a84-bfaa-7b222a403f83
[06/17/24 18:58:28] INFO     Setting 'buffalo_l' execution providers to
                             ['CPUExecutionProvider'], in descending order of
                             preference
[06/17/24 18:58:28] INFO     Loading recognition model 'buffalo_l' to memory
[06/17/24 19:03:31] INFO     Shutting down due to inactivity.
[06/17/24 19:03:31] INFO     Shutting down
[06/17/24 19:03:31] INFO     Error while closing socket [Errno 9] Bad file
                             descriptor
[06/17/24 19:03:31] INFO     Waiting for application shutdown.
[06/17/24 19:03:31] INFO     Application shutdown complete.
[06/17/24 19:03:31] INFO     Finished server process [173202]
[06/17/24 19:03:31] ERROR    Worker (pid:173202) was sent SIGINT!
[06/17/24 19:03:31] INFO     Booting worker with pid: 173877
[06/17/24 19:03:41] INFO     Started server process [173877]
[06/17/24 19:03:41] INFO     Waiting for application startup.
[06/17/24 19:03:41] INFO     Created in-memory cache with unloading after 300s
                             of inactivity.
[06/17/24 19:03:41] INFO     Initialized request thread pool with 12 threads.
[06/17/24 19:03:41] INFO     Application startup complete.
[Nest] 1101  - 06/18/2024, 12:00:06 AM    WARN [Microservices:PersonService] Face 30da4651-1944-4f51-94f5-b5ec367e5ccb not found
[Nest] 1101  - 06/18/2024, 12:00:06 AM    WARN [Microservices:PersonService] Face d7cf9ee3-f0cb-4560-86b3-1567a77e3d9d not found
[Nest] 1101  - 06/18/2024, 12:00:10 AM    WARN [Microservices:PersonService] Face 3c6a7af3-107e-4ab3-b649-f97a96b2f71c not found

Architecture
amd64
OS
HAos

This is very strange ; it was confirmed to be re-added there https://github.com/alexbelgium/hassio-addons/issues/1430#event-13168820552

Rebooting the host solved the issue (probably restarting the container would have also sufficed)
Issue can be closed.


PatternFly DataGrid update to reuse common widget

uberfire-extensions
(master) http://github.com/uberfire/uberfire-extensions/commit/e172b27f0

Thanks @cristianonicolai


K64F serial_format with ParityEven causes assertion error

ParityEven is not handled and results in an assertion error.  One-line fix in attached patch.
k64f-even-parity.patch.txt

You might also want to add
  MBED_ASSERT(data_bits == 8);

unless support for e.g. 7bits+parity is forthcoming.


fix: use pool_pre_ping to discard invalid SQL connections when borrow…

…ing from the pool
Describe changes
This fixes https://github.com/zenml-io/zenml/issues/1488
Pre-requisites
Please ensure you have done the following:

[X] I have read the CONTRIBUTING.md document.
[X] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.

Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

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


fix vector compiler errors in Cython versions newer than 0.21.2

__Pyx_FakeReference doesn't play nice with vector::reference
[i] is also faster than .at(i) anyway.

Coverage remained the same at 99.693% when pulling 4a207be038ac5ed3149bffb5e0ce2ac896b2b723 on ndawe:master into 20d6aeb2342ad0f7d6b54e8e96ce4841a2e1745f on rootpy:master.


Documentation should show the SQL triggers

I can maintain this with cog.

https://github.com/simonw/sqlite-history/tree/5bdf91f0b236e9bd634c6bb2086f3ec07a42bf3a#how-this-works


Add corresponding version support instructions

Apisix 3.0 version interface and master version are not compatible,Suggest adding version notes to README.md

2.2.0 has support for apisix 3.x


Failed to minify in create-react-app

Hi there,
When this module is added to a create-react-app in the process of building the application the next message is shown:
"
Failed to minify the code from this file:
./node_modules/data-forge/src/series.js:2235

Read more here: http://bit.ly/2tRViJ9
"
Basically it seems that the series.js contains es6 code when the create-react-app expects all the modules to be in es5 code ..
Would it be possible to get a package build with full es5 support?
Thanks

Hi I'm currently rebuilding DataForge with TypeScript, that means I'll be able to easily target ES5.
Can you wait a few weeks?
The new repo is here, please check up on it and I'll announce in the readme when it's ready.
https://github.com/data-forge/data-forge-ts
I'll also be introducing the new version on my blog in the coming weeks:
http://www.the-data-wrangler.com/


[HOPS-374]

Fixing HOPS-5 ACL-induced bug with incorrect permission check for subtree ops causing TestDFSPermission to fail.
Make sure there is no duplicate PR for this issue

Please check if the PR fulfills these requirements

[x] Tests for the changes have been added and passed (for bug fixes / features)
[x] HOPS JIRA issue has been opened for this PR
[x] All commits have been squashed down to a single commit
[x] The commit message has the following format: [HOPS-XXX] message

Post a link to the associated JIRA issue
https://hopshadoop.atlassian.net/browse/HOPS-374

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

Can one of the admins verify this patch?

Jenkins test this please.

Jenkins test this please.

Jenkins test this please.


Persist is borked

Persist will persist, but sometimes even after running:
iohyve stop bsdguest

Happened after reorganizing the loops in fixing empty variables.
Need to re-tweak stuff in the start function.

I think that'll do for now.
https://github.com/pr1ntf/iohyve/commit/92f0794442c39a1cdf7cb804cdd46254deaba752


Errors thrown by callback swallowed

Hey there,
Errors thrown by the queued callback will get swallowed, there should be a catch on the promise. @domenic corrected this in the polyfill I'd posted on MDN (which was removed for some reason; I just restored it). https://developer.mozilla.org/en-US/docs/Web/API/WindowOrWorkerGlobalScope/queueMicrotask#Polyfill

@zbjornson Thanks for the issue. This is the code I found on MDN:
if (typeof window.queueMicrotask !== "function") {
  window.queueMicrotask = function (callback) {
    Promise.resolve()
      .then(callback)
      .catch(e => setTimeout(() => { throw e; }));
  };
}

Can you explain why the exception is thrown in a setTimeout? Is that so that it can't be caught by another .catch at a higher level?

I think it's this difference, from the OP of https://github.com/whatwg/html/issues/512:

Promise.resolve().then(f) ... creates a promise object and stores any thrown errors there, instead of letting them propagate to "report an exception"

https://html.spec.whatwg.org/multipage/webappapis.html#report-the-error
With setTimeout, the error will invoke window.onerror (like queueMicrotask). Without setTimeout, it won't.
> window.onerror = e => console.log("onerror:", e);

> queueMicrotask(() => { throw new Error("eek"); })
"onerror: Script error."
❌Uncaught Error: eek

> Promise.resolve().then(() => { throw new Error("oops"); }).catch(e => setTimeout(() => { throw e; }))
"onerror: Script error."
❌Uncaught Error: oops

> Promise.resolve().then(() => { throw new Error("oops"); }).catch(e => { throw e; })
// no onerror handler
❌Uncaught (in promise) Error: oops

Thanks for the explanation. I released a fix for this as 1.1.1


[Question] Does floating point precision matter?

Currently, everything is hard coded to use floating point 64 bit. However, some GPUs and CPUs do not support floating point 64 bit.
https://github.com/MouseLand/cellpose/pull/668
I changed the numpy and pytorch code to use floating point 32 bit, and they seem to produce the same predictions for segmentation.
I was thinking of submitting a PR for adding float 32 bit, bfloat16, and float16.
Would this be a good idea?

I'm confused, everything is hard-coded to be FLOAT32, not FLOAT64. Float32 is the default float in pytorch. We do not recommend going below float32 in terms of precision, the speed ups are not huge either -- NVIDIA GPUs are well-optimized for float32

@carsen-stringer
https://github.com/MouseLand/cellpose/blob/a097ecc931ffb8abb44f99579e4f67edde5ff42b/cellpose/dynamics.py#L75
The tensors are initialized as torch.double which is float 64 bit
https://github.com/MouseLand/cellpose/blob/a097ecc931ffb8abb44f99579e4f67edde5ff42b/cellpose/utils.py#L115
Moreover, numpy initializations are also np.float64 which fails on certain CPU and GPUs that only support floating point 32 bit
It would be good to have a selector that allows for the change in precision.


Typed multiple result sets

Plain Npgsql is able to return multiple result sets, which can be read using NpgsqlDataReader:
let con = new Npgsql.NpgsqlConnection (cs)
con.Open ()
let cmd = con.CreateCommand ()
cmd.CommandText <- "select * from actor limit 1; select * from address limit 1;"
let reader = cmd.ExecuteReader ()

while (reader.Read ()) do
    () // actors

reader.NextResult () |> ignore

while (reader.Read ()) do
    () // addresses

On the other hand,
let res = DvdRental.CreateCommand<"select * from actor limit 1; select * from address limit 1;">(cs).Execute ()

only produces a list of actors. I wonder if both actors and addresses could be exposed in a type-safe fashion.

This is doable, but will require some heavy changes to the ISqlCommand and quotations implementations. Current behavior is inline with FSharp.Data.SqlClient type provider - https://github.com/fsprojects/FSharp.Data.SqlClient/issues/331.
Npgsql library does provide row description message per each result set, so this could work. However implementation will go too far away from original provider.
How critical is this for your use case?

It isn't anything I need, frankly. Since one can always fall back to Npgsql (ResultType.DataReader with the TP actually does not work, because only the first result set is populated), not having this is probably no biggie. I just thought it would be a shame not to make use of the type information if it's available upstream :).


backend: autocomplete languages from external endpoint

Currently we use an npm package to get all iso languages for the Location form. We should refactor this when we have a way to retrieve all these values from the backend.

Will be addressed here #543


Make icon square

For now icon has different width and height. It leads to changing proportions in some launchers (KISS and probably some others). It may be a launcher issue, but the same bug was found in Yalp Store. I'd fix it myself but I can't find proper editor for this task

Good point. I will fix this shortly. And test out some strings.

Done.
https://github.com/alaskalinuxuser/app_critical_velocity/commit/2d25afed2ee3b91bda12c9475bbce43050415eff


fix(lavalink): include Sec-WebSocket-Key header

Add the missing header due to a regression in tokio-tungstenite.
Closes #1647.

@james7132 could you try this branch?


Incorrect documentation

[Enter feedback here]
Teams admins can view additional meeting attendance data in the Teams admin center, under Analytics & reports > Usage reports.
This part is incorrect. In the usage reports, there no reference to an attendance report.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9a8b1087-ae40-6813-16bc-a69c90e50d19
Version Independent ID: b808e843-80d8-4a96-c6ed-e8175968eb73
Content: Microsoft Teams meeting attendance report - Microsoft Teams
Content Source: Teams/teams-analytics-and-reports/meeting-attendance-report.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@jesegher Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@jesegher Thank you for reaching out. Please see if this link answers your question - https://support.microsoft.com/office/download-attendance-reports-in-teams-ae7cf170-530c-47d3-84c1-3aedac74d310

I would echo the issue reported by @jesegher. The documentation suggests that an admin can access meeting attendance data (when a user joined and exited the meeting) through the admin "Analytics and Reports" page, which appears to be false. As a tech administrator for a school district facilitating distance learning through Microsoft teams during the COVID epademic, the ability for an admin to pull this data for all meetings, or better, the ability to extract this data through the Graph API (or another automated method) would be a total game-changer for the systems we could provide for our teachers and school administrators.
@Reeced40 The article you referenced shows how a meeting organizer can download the attendance via the Teams Application, but the documentation suggests that an admin can access the same data through the Teams Admin "Usage Report", which appears to be false.

@LolaJacobsen please could you assist with this issue?

@LolaJacobsen Hi Lola, any updates on this?

@jesegher it looks like this article has been update in the interim with more information. Please let me know if this issue can be closed. Thanks again.

@jakefirman  Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

Updated article to remove the reference to meeting attendance reports in usage reports. Thank you.


HaplotypeCaller missing GVCF block for interval with no coverage

using commit 39c988c2dc7f669306d246a2191d16af50496640
(using our test resources)
./gatk-launch HaplotypeCaller -I ~/IdeaProjects/gatk/src/test/resources/large/CEUTrio.HiSeq.WGS.b37.NA12878.20.21.bam -L 20:1-11000000 -R ~/IdeaProjects/gatk/src/test/resources/large/human_g1k_v37.20.21.fasta  -O testa.g.vcf -ERC GVCF --bamOutput testa.outHC.bam

compare to gatk3.4.46
java -jar /Users/akiezun/projects/GATK3.4-46/GenomeAnalysisTK.jar -T HaplotypeCaller -I ~/IdeaProjects/gatk/src/test/resources/large/CEUTrio.HiSeq.WGS.b37.NA12878.20.21.bam -L 20:1-11000000 -R ~/IdeaProjects/gatk/src/test/resources/large/human_g1k_v37.20.21.fasta -ERC GVCF --bamOutput testa.gatk3.outHC.bam --out testa.gatk3.g.vcf

see the first line in the output:
20      1       .       N       <NON_REF>       .       .       END=9999901     GT:DP:GQ:MIN_DP:PL      0/0:0:0:0:0,0,0

it's missing from GATK4 output

This is probably because there is no coverage at all in the interval 1-9999901

why is it different and is it OK?

I'll need to have a look at GenotypeGVCFs to determine whether it's ok, but certainly we are not losing any information by leaving out this block (since it's backed by zero reads).

Issue moved to broadinstitute/gatk #2848 via ZenHub


np.arange does not like int parameter on macos

On MacOs, the following code does not compile. It compiles fine in linux.
Error log:
error.txt
import numpy as np

#pythran export test(int)
def test(M):
    # n = np.arange(0, float(M)) <- Compiles
    n = np.arange(0, M) <- Does not compile.
    return n

Still not fixed at https://github.com/serge-sans-paille/pythran/commit/c3dd1ef72aa7de51d84ab25ea4d936101d8a5802

Does #921 fix the issue?

Confirmed fixed with https://github.com/serge-sans-paille/pythran/pull/921 on macos! Yepee!


CDI-637 rename the builder package to configurator and fix some langu…

…age.

LGTM

You're so fast @tremes !. Looks good now

Looks good.


[BUG]需要0.05版本的~修正這些問題

Describe the bug
A clear and concise description of what the bug is. (If what you are experiencing is NOT a bug but instead a support issue, please open a Discussion instead!)
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Linux]
OS Version/Flavor: [e.g. CentOS 7.2]
CPU: [e.g. Intel Xeon 8175M]

Additional context
Add any other context about the problem here.

Please provide the OS version, silicoin version and the following files:
~/.silicoin/config/config.yaml
~/.silicoin/log/debug.log


Alpine Docker Image

Switch to python:3.8-alpine Docker image in preparation for Kubernetes.

@jslay-excella will this change anything about our (poorly organized, at best) Docker documentation?

Everything looks good and passes. Apache changes are in place. Infra is ready.

Alright. We should communicate with devs who use Docker that the base image is changing before we merge, what that means, and what they should do.
Is there reason to preserve the individual commit history here, or can this branch be squashed entirely, or the history compressed somewhat?
This is fantastic work, @jslay-excella. I love the reduced complexity. It looks good to me, with the above two things!

I've cleaned up the squash commit message to include all the pertinent changes.


Nuke: Alembic model workflow

Description
Create, publish, load Alembic model in Nuke workflow.
Testing Notes

select your model node and create Create 3d Model.
Publish model family.
Load model family.

Task linked: OP-1032 nk geo workflow


Cannot find module 'ncp'

After I install your module, atom throws an error claiming it cannot find the module 'ncp'.  I had to install your module by downloading the git repo and manually placing it in my packages folder as I am behind a firewall at work and cannot install anything directly through atom.
Here is How I produced the error:

Manually place markdown-themable-pdf-master folder in packages folder of atom
Open Atom
Error thrown

Atom Version: 1.0.13
System: Microsoft Windows 7 Professional
Thrown From: markdown-themeable-pdf package, v0.9.0
Stack Trace
Failed to activate the markdown-themeable-pdf package
At Cannot find module 'ncp'

Error: Cannot find module 'ncp'
    at Module._resolveFilename (module.js:334:15)
    at Function.Module._resolveFilename (C:\Users\rr146019\AppData\Local\atom\app-1.0.13\resources\app.asar\src\module-cache.js:383:52)
    at Function.Module._load (module.js:284:25)
    at Module.require (module.js:363:17)
    at require (module.js:382:17)
    at Object.module.exports.markdownThemeablePdf.activate (C:\Users\rr146019\.atom\packages\markdown-themeable-pdf-master\lib\markdown-themeable-pdf.js:184:19)
    at Package.module.exports.Package.activateNow (C:\Users\rr146019\AppData\Local\atom\app-1.0.13\resources\app.asar\src\package.js:245:19)
    at C:\Users\rr146019\AppData\Local\atom\app-1.0.13\resources\app.asar\src\package.js:226:30
    at Package.module.exports.Package.measure (C:\Users\rr146019\AppData\Local\atom\app-1.0.13\resources\app.asar\src\package.js:169:15)
    at Package.module.exports.Package.activate (C:\Users\rr146019\AppData\Local\atom\app-1.0.13\resources\app.asar\src\package.js:218:14)

Commands

Config
{
  "core": {
    "themes": [
      "one-dark-ui",
      "solarized-dark-syntax"
    ],
    "followSymlinks": true
  }
}

Installed Packages
User

language-csharp
language-vb
markdown-pdf
markdown-themable-pdf
node-html-pdf

Dev
No dev packages

Hmm... if you place it manually in to the packages dir, you must load also the dependencies manually. You have two ways:

Open markdown-themeable-pdf folder with atom and run "update package dependencies" (or something similar) or
Download dependencies manually from github. You can find them defined in package.json in the require section

Hope this helps! I could not test it at the moment because I am not at home ;)

Since there is no failure of the package, I close the ticket now. ;) If you need assistance in the use, feel free to write here again and come back to my suggestion.


jit and vamp best practices

Just a suggestion to add a small section to the docs about best practices for quaxifying jitted and vmapped functions. Thanks!

At least with JIT, ths usual best-practice of putting JIT at the top level applies.
For vmap, I don't think it matters: put these in whichever order describes the operations you're trying to perform.
Does that sound reasonable to you?
(I agree it'd be worth emphasising this somewhere.)


Enable barbican barclamp on cloud 9 upgrade jobs (SOC-10477)

Currently there are 3 cloud 9 upgrade jobs:
cloud-mkcloud9-job-upgrade-disruptive-x86_64
cloud-mkcloud9-job-upgrade-nondisruptive-ha-mariadb-x86_64
cloud-mkcloud9-job-upgrade-nondisruptive-ha-without-nodes-upgrade-x86_64

This change update those job templates by adding/parameterizing the
want_barbican_proposal parameter and setting it to 1 on the jobs
definition. This should enable the barbican barclamp for those 3 cloud 9 upgrade jobs.

rebased to master


Update Splice main to version 0.2.5-snapshot.20241003.7153.0.v92e4a881 (automatic PR)

This PR updates branch main of Splice from the latest changes as of version 0.2.5-snapshot.20241003.7153.0.v92e4a881, commit DACH-NY/canton-network-node@92e4a8813f

#59


storybook package.json files don't declare dependency on core-js

Describe the bug
storybook packages are not declaring a dependency on core-js in package.json, unable to run using pnpm as a packager without adding core-js or hacking the core-js dependency in.
To Reproduce
set up react-native project using pnpm rather than yarn/npm.  when bundling, @storybook/react-native and many other @storybook packages are unable to resolve require imports to core-js/
Expected behavior
bundling is successful with defining extra dependencies in project consuming storybook
System:

OS: Windows 10
Device: Desktop
Framework: react-native
Version: 4.1.13

Additional context
pnpm enforces correct dependencies by isolating each package's dependencies in subfolders under it rather than merging everything in the same node_modules common folder.

I am having the same issue when using npm, adding core-js to my devDependencies solved it but I shouldn't have to do that.

This is also happening with<EMAIL_ADDRESS>@andrewmarmion's suggestion solves this, but as he said this shouldn't be needed.

<EMAIL_ADDRESS>is getting close. thanks for your patience everybody!

PS this is not a react-native specific issue

Ok, i figured it out in the end, PR incoming

The issue here is pnpm and it's non-deduping module install. I suspect the only fix would be to depend on core-js from every manager-side package. Is this the right solution?

Another option would be to use a webpack resolve alias

Hmm.. there already is an alias: https://github.com/storybooks/storybook/blob/b19ae6b9e44e8759a4735169059cfa05f104d5e9/lib/ui/paths.js#L15

Without a reproduction I am not totally sure but I am guessing the issue may come from the preview side. I see that app/react has a dependency on core-js but app/react-native does not. @adamcarriere can you post the stack trace of the error you saw?

I'm seeing something similar - I get a ton of errors like:

ModuleNotFoundError: Module not found: Error: Can't resolve 'core-js/modules/es6.string.bold' in '[...]/node_modules/@storybook/ui/node_modules/@storybook/components/dist/tooltip'

In my case<EMAIL_ADDRESS>and<EMAIL_ADDRESS>was working fine. Upgrading react to 16.8.4 causes the error below.

ModuleNotFoundError: Module not found: Error: Can't resolve 'core-js/modules/es7.symbol.async-iterator' in '[...]/node_modules/@storybook/core/dist/client/manager'

@petewarman @ysgk are you using pnpm?
@ysgk this is pretty surprising as @storybook/core does in fact depend directly on core-js. What do you your node_modules look like? Is core-js resolvable from @storybook/core? What version?
I hate to suggest it but you could probably try removing node_modules and re-running your package manager installation.

I’m using yarn.

OK I think the original issue title is the correct way forwards, we should add a dependency on core-js to all our packages

I’m using yarn too.

I forgot to remove yarn.lock. Removing node_modules and yarn.lock and re-running yarn works.
My case seems to be a yarn-related problem.

@ysgk I'm not surprised to hear that, sadly

Whoopee!! I just released https://github.com/storybooks/storybook/releases/tag/v5.1.0-alpha.2 containing PR #5961 that references this issue. Upgrade today to try it out!
Because it's a pre-release you can find it on the @next NPM tag.
Closing this issue. Please re-open if you think there's still more to do.

💥Works for me now - thanks!

@tmeasday is here to save the day

Yo-ho-ho!! I just released https://github.com/storybooks/storybook/releases/tag/v5.0.1 containing PR #5961 that references this issue. Upgrade today to try it out!


How can I switch the kernel of firmware from 5.10.* to 5.15.*?

Halo, I made a fork from your repository and have the same mi router 3. I configured mesh option and if I use firmware with 5.15.* kernel , I hadn't any problems with it. But if I use firmware with 5.10.* kernel, which is the main one today, I had some problems with it and some others.
How can I switch the kernel of firmware from 5.10.* to 5.15.* in my config file and so on? So that I can build my next firmwares including the kernel 5.15.*
TY for your time.

Hello there. The maintainer of the x-wrt repo usually knows better which kernel version to use (referring to this commit https://github.com/x-wrt/x-wrt/commit/cdd527ae3fe6deea4573f8bfb0ccd32807802347)
In theory you could try changing KERNEL_PATCHVER from 5.10 to 5.15 in target/linux/ramips/mt7620/target.mk, but i cannot guarantee you that it will compile and work just fine
Moreover, i haven't owned miwifi r3 for over a year now and i have no idea how openwrt's doing in 2023 :(

I see. What did you choose as a replacement?

At the moment using mi 4a gigabit at one place and mi 3g (old ver.) at the other


TypeScript support

From @KnisterPeter on December 1, 2015 18:19
During export of pattern it would be great to produce definition files (.d.ts) for the patterns.
Copied from original issue: sinnerschrader/patternplate-server#32

From @KnisterPeter on December 2, 2015 1:17
This may help: https://github.com/KnisterPeter/react-to-typescript-definitions


Add support for mapping messages to CloudEvents

Added support for mapping UMessages to/from CloudEvents using Protobuf format.
Addresses #203

@AnotherDaniel would you mind taking another look?

@AnotherDaniel I have updated the NOTICE.md file accordingly.


There should be a way to get all Arc<Mutex<Call>>s

Songbird version: next
Description:
I'm currently implementing a procedure for saving call information (including tracks from the builtin queue) to disk, but it looks like the Songbird type does not provide access to iter over all calls? Should this feature be added or am I approaching it in a wrong way?

This is an API oversight, but I think it might be a little bit rougher to implement than it initially appears. Internally, we use a DashMap to store Calls -- you'd need to look into the best way to expose this without leaking internal implementation details.


Function parameters

Please include the function parameters for each callback in the documentation and examples.
As far as I can tell, these are what I could find:
each: function(element) {
  // what element?
},
noMatch: function(keyword) {
  // keyword - mark
  // regexp - markRegExp
},
done: function(count) {
  // count = totalMatches
},
filter: function(node, term, counter, totalCounter) {
  // node, kw, matches, totalMatches - mark
  // node, match, totalMatches - markRegExp
}

Update: what I mean is in the example files, like this:
https://jsfiddle.net/Mottie/vpav6tL1/145/

Hi @Mottie,
Thanks for submitting.
I'm not sure if I understand your request correctly.
All function parameters are documented in their respective options. For example the each option contains the information Receives the marked DOM element as a parameter (that should clarify what this element is).
In your opinion, should the documentation contain a code overview of all options including parameters?
What I can offer in any case are new checkboxes for function callbacks in the Configurator, with a simple comment inside containing the parameters.

Actually, I was just testing that demo... when those functions are set to null it produces javascript errors... what should the filter function be set as? In this updated demo, I have it return true, but I see a concat error.

Well, mark.js expects functions, not null. If you want to overwrite an option you shouldp pass the corresponding

In the demo I just shared, I replaced all the null with functions, but the filter function isn't working as expected.

Well, mark.js expects functions, not null for callbacks. If you want to overwrite an option you should pass the corresponding Type (see documentation). In case of the callbacks, you would need to pass an empty function. Have a look here to see how it handles default values.
The demo you've shared includes mark.js v6.1.0. There was an API breaking change in v7.0.0, see here. The documentations is always related to the last available release.
Could you please answer my questions?

Sorry, when I read the documentation before today, I thought the function parameters were missing. I see them in the documentation now. What I was expecting was a code example. I think a code overview with all options including parameters would be a nice addition.
I haven't used the Configurator; I've only been looking at the docs and the jsFiddle examples.
Yes, I do need to update to v7.0.0+, thanks.

Thanks for answering.

I think a code overview with all options including parameters would be a nice addition.

Could you please elaborate this a bit? Should it be just an overview like here or something else?
Btw.: If you just have a simple addition you think that would be helpful, you're welcome to add it yourself in the docs inside the website branch.

Yes, just a simple overview, like expand the syntax examples. Something like this would be nice:
Javascript Syntax
var context = document.querySelector(".context");
var instance = new Mark(context);
instance.mark(keyword, {
  accuracy: 'partially',
  className: '',
  debug: false,
  diacritics: true,
  element: 'mark',
  exclude: [],
  iframes: false,
  log: window.console,
  separateWordSearch: true,
  synonyms: {},
  done: function(totalMatches) {},
  each: function(element) {},
  filter: function(node, term, counter, totalCounter) {
    return true;
  },
  noMatch: function(keyword) {}
});

jQuery Syntax:
$(".context").mark(keyword, {
  accuracy: 'partially',
  className: '',
  debug: false,
  diacritics: true,
  element: 'mark',
  exclude: [],
  iframes: false,
  log: window.console,
  separateWordSearch: true,
  synonyms: {},
  done: function(totalMatches) {},
  each: function(element) {},
  filter: function(node, term, counter, totalCounter) {
    return true;
  },
  noMatch: function(keyword) {}
});

Agreed. Todo:

[ ] Add new checkboxes to the Configurator to also add code examples for callback functions
[ ] Add syntax examples with a full list of options
[ ] Update JSFiddle examples to use the latest mark.js versions

I'm going to take this over once I've completed the v8.0.0 milestone (in the next days).

Also, it might be nice if the parameters matched up:
filter: function(node, term, counter, totalCounter) {
  // node, kw, matches, totalMatches - mark
  // node, match, totalMatches - markRegExp
}

What I mean is I can use the same function & parameters if I use the filter for either mark or markRegExp. The extra parameter for markRegExp can be undefined or null, or maybe rearrange them to move the matches to the end.
filter: function(node, term, counter, totalCounter) {
  // node, keyword, matches, totalMatches - mark
  // node, regex,      null, totalMatches - markRegExp
}

@Mottie Passing null as a parameter is a weird idea. But yeah, I could imagine to pass counter after totalCounter. But, most users using the filter callback are working with the counter, not the totalCounter. They would need to define a parameter they don't need.

@Mottie As of v8.0.0 the function parameters of the filter callback of .mark() are now in the same order as markRegExp()

@Mottie I'm going to close this issue now, as I've completed all above named todos.


block-level typography behaves like inline-block when in a row

I've just discovered that placing a block-level element (e.g. h1, p, etc) in a div with class row causes it to behave like an inline-block element!
Example code:
<!doctype html>

    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">

    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/css/bootstrap.min.css" integrity="sha384-Gn5384xqQ1aoWXA+058RXPxPg6fy4IWvTNh0E263XmFcJlSAwiGgFAW/dAiS6JXm" crossorigin="anonymous">

    <title>Hello, world!</title>

Hello, world!

This is a paragraph

What am I doing wrong here?

Operating system and version: Windows 10 x64
Browser and version: Chrome, Firefox, MS Edge (maybe all?)
Reduced test case: https://codepen.io/slinco/pen/XZjbym

You must have .col-* directly within a .row before you content. This is how grid system works

In a grid layout, content must be placed within columns and only columns may be immediate children of rows.
Source: https://getbootstrap.com/docs/4.0/layout/grid/#how-it-works

😊 Well that's subtle, but I can see why that is required now. Thanks.
Is that a change for BS4? I don't remember having to do that in BS3


Return URL in startIfStopped Method

Changes needed for a working vscode remote ssh fix.
Signed-off-by: Will Yang<EMAIL_ADDRESS>

LGTM, there is only a missing space in the commit message Signed-off-by.

Whops!  Fixed.

Closing this as it's not needed anymore.


Option to force writing to archive without actually downloading the video

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[ ] I am the original author of this code and I am willing to release it under Unlicense
[x] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[ ] New extractor
[x] New feature

Description of your pull request and other information
Adds a new option --force-write-archive which will force download archive entries to be written  as far as no errors occur, even if -s or another simulation switch is used.
The original author is @h-h-h-h. This was a pull request that existed before the takedown. I have the author's permission to make this PR. You can confirm it with him if needed

Back when I wrote this, I also edited test/parameters.json in this way:
@@ -7,6 +7,7 @@
     "forcethumbnail": false,
     "forcetitle": false,
     "forceurl": false,
+    "force_write_download_archive": false,
     "format": "best",
     "ignoreerrors": false,
     "listformats": null,

I don't know, what that was for. But, I guess, it couldn't hurt.

@pukkandan and repo owner:
--force-archive might actually also be nice (no synonyms?). Isn't it meaningful enough, especially when it would always be mentioned with --download-archive in one cmdline? It means less typing.

@h-h-h-h I honestly don't care what the actual switch is going to be called.
Also, it not necessary to document all synonyms. There are already switches in youtube-dl that are not documented (Eg: --write-srt). The intend of these is to fix common mistakes, or for backward compatibility. The reason I used --force-download-archive in the PR is because I always kept accidentally typing it instead of --force-write-download-archive.
PS: In case you didn't know, you can partially specify an argument. ie, --force-down works.
I'll just leave it to @blackjack4494 to make the decision on what switches should be used and to make any changes if necessary.

@blackjack4494
Example Usages of the New Switch from My Original youtube-dl PR
Create internet shortcut files for new channel videos [dependant on #184]:
youtube-dl --write-link --skip-download --download-archive .youtube-dl-download-archive --force-write-download-archive --ignore-errors https://www.youtube.com/user/woolybullyshanti/videos

Search for keyword in titles and descriptions of new channel videos; expecting videos of some kind, ignoring those already encountered:
youtube-dl --get-filename --output "youtu.be/%(id)s   +++   %(title)s   +++   %(description)s" --download-archive .youtube-dl-download-archive --force-write-download-archive --ignore-errors https://www.youtube.com/user/woolybullyshanti/videos
| grep "apple"

Write JSON files for new videos instead of downloading, then processing them with some other tool:
youtube-dl --write-info-json --skip-download --download-archive .youtube-dl-download-archive --force-write-download-archive --ignore-errors https://www.youtube.com/user/woolybullyshanti/videos
some-other-tool *.json

Nice use cases. I simply use it to effectively mark some videos in a playlist as "I already watched it, don't download again"

Okay so the flag name is a bit uhm (don't know how to put it) strange/complicated? Also the description could be a slightly different. It may come in handy to have 1-2 alternative flags (synonyms) for this to be honest but that's just my thoughts about this.

@blackjack4494 How about --force-archive like @h-h-h-h suggested.
I think the description is fine. What change do you suggest?

So, we could use:

--force-archive as the main switch, as it's always going to be used along with --download-archive
--force-write-archive as synonym
--force-download-archive as synonym

It may come in handy to have 1-2 alternative flags (synonyms)
[@blackjack4494]

Which synonym should be dropped, if any?

Should the synonyms be documented? Both?


fix(ripple): correct ripple on ios mode component in an md mode app

Short description of what this resolves:
When using an Alert or ActionSheet set to iOS mode, in an MD mode app, the ripple is not contained inside the buttons.
Changes proposed in this pull request:

Use a selector for the ripple containment that works in both iOS and MD mode components, when the app is MD mode.

Ionic Version: 3.x
Fixes: #12382

Hello and thank you for contributing to Ionic! We have been working on porting all of the Ionic components to web components and have recently updated master to reflect this. This significant change has caused this pull request to break. While we really appreciate the time and effort you put into creating this, we are not able to merge it because of the newly introduced conflicts. We are extremely sorry about this. We will not be merging any more features in to v3. If this is a feature and you have the time, please resubmit this PR against the master branch. If this is a critical security issue in v3, we would greatly appreciate it if you would resubmit the PR against the new v3 branch. Thanks so much for your time!


feat: log response body from API to logger

Summary

AMP-77375 FIX: Correctly resolve send() promise on error
AMP-77261 log intermediate response body to logger on retries for Node SDK

Checklist

[x] Does your PR title have the correct title format?
Does your PR have a breaking change?:  No

maybe events should be re-sent on some/all (network) errors - https://github.com/amplitude/Amplitude-TypeScript/blob/main/packages/analytics-core/src/plugins/destination.ts#L157
Thanks, maybe we can fix in another PR. I want to release this to help us debug the 429s in more detail.

some strange-looking flag useRetry prevents event re-sending (it is set to false on explicit flush() call) - I do not understand why the flag is required

Thanks for calling this out. I think the intent is that in Node flush()  is usually called at shutdown when the user wants to kill the application. In this cause we "fire and forget" all events. In general the events will be retried based on the set configuration. And flush() should be reserved for end of app. We should add this to docs.

2. it is set to false on explicit flush() call

@falconandy We assumed that flush would be commonly invoked before a node app exits. Retry was disabled to allow node apps to exist immediately after one last attempt.

I think the intent is that in Node flush() is usually called at shutdown when the user wants to kill the application. In this case we "fire and forget" all events. In general the events will be retried based on the provided configuration. And flush() should be reserved for end of app. We should add this to docs.

Shouldn't it be a method like shutdown()?

Shouldn't it be a method like shutdown()?

We had some strong push back on adding an extra shutdown() method as it is seen as a "special case" and breaks from the shared interface of the Next-Gen SDKs.
Maybe it would be better to update amplitude.flush() to accept a optional useRetry = true in all SDKs. I think it would be better to default to retries to prevent data loss. And in the case that a customer wants a "hard exit" (less likely) they can pass in amplitude.flush(false)


Warnings don't get reported to Webpack

If the Elm compiler spits out some warnings, they should go to Webpack - but I don't think they do. The webpack stats.hasWarnings() is always false, even though there are warnings. I don't know much about the internals of Webpack plugins but can we detect when warnings are produced and pass them onto Webpack so stats.warnings isn't just always an empty array?

@jackfranklin can you try this out? It should be fixed in 4.0.0

closing as no answer, and I think this is fixed

Sorry I never got a chance to try this. I will one day...


Can't see any option to start live server.

If you have a problem launching a live-server, please try following steps:

Make sure you use the right hotkeys. After v2.0.0, the keys became ctrl-alt combination.
Try launching live-server from top menu: Package -> live-server -> Start Server to see if it works. (Can't)

If you press Cmd + Alt + I (capital i) and scroll to the top of the console do you see the below error message?

#57


feat(Argument): attachment type

Please describe the changes this PR makes and why it should be merged:

[ ] Wait for https://github.com/discordjs/discord.js/pull/7200
[x] Old PR: https://github.com/Garlic-Team/GCommands/pull/286
^ Changed branch (more PRs)

Status and versioning classification:
This PR changes the library's interface (methods or parameters added)

Backport for v13 (https://github.com/discordjs/discord.js/pull/7441)

Backport is merged 🎉


MNT: Re-rendered with conda-smithy 3.1.12 and pinning 2018.10.08

R 3.5.1

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Add Key Value Records

Requires adding a Record type to the python side.
Related:

infinyon/fluvio#828.
infinyon/fluvio-client-node#77

Closed in #10.


Edit: update readme format and info

Change Log:

format update
toolchain path info update
task and launch info update
removed paragraph numbering
add table of content

Problem:

Solution:

Issue Tracking: #

looks good


COPT update

Removing this line of code stops COPT calling a floating license from a server in cases where COPT is not used to solve the problem.
Solver still solves as normal on my side after its removal.

The issue is that we do use COPT. We have a floating license, but only a limited number of seats. Each time pypsa is imported it checks out the license even if you not doing solving a problem. Like building a snakemake workflow.

Understood. You are right, we can just remove the check and the import check will be enough. If you resolve the comment, we can merge this. I don't have access to your fork and can't do it myself.


implement type checking on pending and error to throw data null type exception

when checking the pending and error status options, data can still be of type null, although the check completely excludes this option.

Actually, I have no idea how to provide this level of type checks. As I know, it's impossible in TS.
Do you know how to do it?

After some research, I am convinced that it is impossible in the current version of TS.


Are pretrained weights corrupted?

Hi, I tried to load your resnet-50 pretrained weights but displayed error messages like the below.
It seems that the weight is corrupt. Do I need any preparations after downloading the weight? If really corrupted, would you please fix them?
Thanks in advance.
Python 3.8.10 (default, Jun 26 2021, 12:40:55)
Type 'copyright', 'credits' or 'license' for more information
IPython 8.2.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import torch

In [2]: a = torch.load('FractalDB-1000_res50.pth')
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
Input In [2], in <cell line: 1>()
----> 1 a = torch.load('FractalDB-1000_res50.pth')

File ~/.pyenv/versions/3.8.10/lib/python3.8/site-packages/torch/serialization.py:713, in load(f, map_location, pickle_module, **pickle_load_args)
    711             return torch.jit.load(opened_file)
    712         return _load(opened_zipfile, map_location, pickle_module, **pickle_load_args)
--> 713 return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)

File ~/.pyenv/versions/3.8.10/lib/python3.8/site-packages/torch/serialization.py:938, in _legacy_load(f, map_location, pickle_module, **pickle_load_args)
    936 assert key in deserialized_objects
    937 typed_storage = deserialized_objects[key]
--> 938 typed_storage._storage._set_from_file(
    939     f, offset, f_should_read_directly,
    940     torch._utils._element_size(typed_storage.dtype))
    941 if offset is not None:
    942     offset = f.tell()

RuntimeError: unexpected EOF, expected 2813703 more bytes. The file might be corrupted.

In [3]: torch.__version__
Out[3]: '1.11.0'

Dear @hs2022gh ,
So sorry for the delayed reply. I received the same comment recently.
I noticed that the model size was different from our server. Now I replaced the model files on the drive: https://drive.google.com/drive/folders/1tTD-cKKEgBjacCi4ZJ6bRYOv6FsjtGt_?usp=sharing


Add icon on the label in Forms

Duplicates ❌

[X] I have searched the existing issues

Which Figma library is concerned?
Vitamin Web UI Kit
Summary 💡
The forms on vitamin are very great but we need new feature for use it on NEMO (New E-Merch Operator) project.
Need a little icon (16 or 18px) of the right of the label on all Forms (input, text area, select).
It's for add a (i) icon; for add more information of the content to apply on the form (very important on NEMO).
Dependencies 📦
Forms on Vitamin.
Examples 🌈
Vitamix icon 16x16 or 18x18 in the right on label with 8px of padding.
Exemple link :
https://www.figma.com/file/pYPxejpMUjYaczawkDMIvZ/Nemo_Maquettes?node-id=6076%3A148156&t=28tPmeFPTFxsWUUw-1
Motivation 🔦
We can't use Vitamin Forms without this feature.
Request by Louis Bronstein - Product Designer on NEMO
New 🆕

[ ] Validated in a grooming session

Backlog 📋

[ ] Has a good summary
[ ] Has example link(s)

Design in progress 🏗

[ ] Analysis (functional & technical)
[ ] Design refinement
[ ] Dev qualification (properties, breakpoints, tokens etc.)
[ ] Documentation refinement (inside the Figma branch)

Design review 👀

[ ] Figma branch merged
[ ] Documentation review _(and move the documentation into Vitamin Documentation Figma file)

Ready to dev 👍

[ ] Issues are created in all repositories affected (vitamin-<web,android,ios,compose>)

Dev in progress 💻

[ ] All issues are resolved (to replace with issues links)

Thank you @Louis2411 for your request, your ticket will be groom next week :)
We will keep you informed :)

Validaded in grooming ! Need to define the solution (16x16 button is too small :) )

Thx, great !

Hi @Louis2411, we've made a test on Figma libraries and to have a sufficient touch area we need to use an icon alone button.
The behaviour: hover of input and hover of icon button are triggered at the same time because they are not separated in the design construction actually. So it will be breaking change for us.

In conclusion, I'll add your request in the future major release of this component.

Ok, bad news for us.
Thanks anyway

This issue had well been referenced in the different issues for input/textarea & select components for our next major release:

https://github.com/Decathlon/vitamin-design/issues/61#issuecomment-1372366648
https://github.com/Decathlon/vitamin-design/issues/60#issuecomment-1375669071

So we can close it as it's now referenced there.
Thanks again @Louis2411 for your request & understanding, you will be informed in these issues.

OK


Mazumder - Add 2 new columns to SARS-CoV-2 assemblyQC

For SARS-CoV-2 assemblyQC table, please add column 1) number of frameshift and 2) number of stop codons (i.e. Gene | Number of Frameshift | Number of Stop Codon). In addition, values allowed are 0 to 1000 or leave the cell empty.
Please check with Raja for others that are potentially worth considering. Please also wait until Jingyue finishes #182

Complete. Gene was left out per Raja's recommendation.


Document Review - Minor documentation updates

Minor updates to the document:

Including @ijmitch' comments
updated naming conventions for feature branches for epics
"checkout/switch" feature branches

@ijmitch @lauren-li Thanks for the fast feedback. I pushed a new commit :-)


How to use multiple features for y?

Hi,
Can we use other features in the forecasting (i.e. incorporate macroeconomic data)?
Vlad

Not yet. This is on the roadmap for the next version in #101. This is a really natural fit for the method, so I'm looking forward to having it in!


Cow: Watch RFID[\W_]+solutions[\W_]+in[\W_]*+Dubai

Cow requests the watch of the watch_keyword RFID[\W_]*+solutions[\W_]*+in[\W_]*+Dubai. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

RFID[\W_]*+solutions[\W_]*+in[\W_]*+Dubai has been seen in 1 true positive, 0 false positives, and 0 NAAs.


[My Collection] Use product defined ranges to display liquidity values as text

The type of this PR is: Enhancement

This PR resolves CSGN-385
Description
Replaces the static integer values for Liquidity with a string representation (Low, Medium, High, Very High).

After

Before

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[ ] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
#trivial

LGTM 👍
#mergeOnGreen


Change default fan curve and fix bugs

Changed default fan curve to make it less aggressive.
Fixed bugs:

Fixed exception when driver has build number (x.y.z).
Now uses speed from fan curve instead of temp.

I like it, thanks


[BUG] The docs.count is not match for leader and follower

What is the bug?
the docs.count is not match for leader and follower
How can one reproduce the bug?
I am using opensearch 2.9.0 version
Steps to reproduce the behavior:

Start a stress test program, which will create a test index in the leader cluster and write 10,000,000 docs to this index.
such as: nohup ./opensearch-stress write  --opensearch-address "http://{leader_ip:port}"   --index-name "es-bulk-0"  --bulk-batch-size 1000 --bulk-times 10000 &

Before the stress test program ends, start the index replication task in the follower cluster.
such as: curl -XPUT -k -H 'Content-Type: application/json' 'http://{follower_ip:port}/_plugins/_replication/es-bulk-0/_start?pretty' -d '
{
"leader_alias": "leader-cluster-opensearch",
"leader_index": "es-bulk-0"
}'

After the stress test program ends, wait for the index replication task done, the docs.count of the leader index  is 10,000,000, which is as expected, but the docs.count of follower index is always less than 10,000,000.

What is the expected behavior?
When the leader index is writing docs, then start the index replication task. When the leader index stops writing and the index replication task ends, the docs.count of follower index should be equal to the docs.count of the leader index.
What is your host/environment?

OS: Ubuntu 20.04
Version: opensearch 2.9.0
Plugins: cross cluster replication

Do you have any screenshots?
As far as I know, cross cluster replication has two stages, In the first phase, the existing data is synchronized. it will do a snapshot for the segment files of the leader index, then read these files and transfer them to the follower cluster. In the second stage, it reads changes with localCheckPoint from translog to synchronize incremental data
after first stage finished，I found that the docs.deleted of the follower index was 6561, but the stress test program only write docs without specifying _id and did not perform any delete/update/upsert operations.

after the stress test program ends,  the docs.count of the leader index  is 10,000,000, which is as expected

after two stages finished，I found that the docs.count of the follower index is 9993439, which was less than  the leader index. I executed the refresh and flush APIs, but the docs.count was still less than  the leader index, and the difference was exactly the value of docs.deleted in the first stage I found.  (10000000 - 9993439 = 6561)

Do you have any additional context?
This bug can be easily reproduced, only need to start the index replication task when data is being written in batches to the leader index.
I have reproduced this bug many times. I have tried other stress testing programs and this bug always appears. So I hope I can find help here, thx.
btw, If there is no writing to the leader index，then start the replication task，after the task ends，the dos.count of leader index is equal to the follower index.

Can you trigger refresh on follower API and then verify the doc count?


code style and PHPDoc fixes

This commit does not change behavior, it just unifies formatting across the file, fixes some minor bugs, typos and PHPDoc, so it can be better used as a reference
If this is approved, I will appreciate it (it is easier to work with DataService if all code completion actually works correctly), if not, no big deal

Hi becquerel,
Thanks your commit! It is a lot of work to fix the documentation. It will make the SDK documentation much clearer!
Just let you know, I am currently working on the dataService class to remove the PECL OAuth 1.0 extension requirements. Please don't make more changes to the dataService class as I will not able to accept pull request anymore in the future.
Best,
Hao

@hlu2 thank you. I won't make any more changes then. Does this mean no more community-provided pull requests will be accepted ? Or is it a temporary state until you resolve this issue ?

@becquerel It is a temporary state. You can still make changes to the classes located outside of the /core/HttpClients/ folder. The release should go out soon.

Perfect, thank you !
On Thu, May 18, 2017 at 1:19 PM, Hao Lu<EMAIL_ADDRESS>wrote:

@becquerel https://github.com/becquerel It is a temporary state. You
can still make changes to the classes located outside of the
/core/HttpClients/ folder. The release should go out soon.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/intuit/QuickBooks-V3-PHP-SDK/pull/21#issuecomment-302529609,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFcMsvPk40B6pWpMt28-y3BRftwrcMVGks5r7KfFgaJpZM4NfrMs
.


Modifying a built-in script doesn't mark the scene as modified

Godot version:
3.2
Issue description:
When you modify a built-in script, the script nor scene containing it isn't marked as modified. This potentially leads to accidentally unsaved changes on quit.

Steps to reproduce:

Create a scene
Add a built-in script
Save
Type something in the script

perhaps that is also the reason why built-in scripts are not being updated when you edit them during test play? or would that be another issue?

Resolved by #67503


refactor: improve variable namings

Superseded by https://github.com/idos-network/idos-sdk-js/pull/396


Additional property on context: User's Tenant Id

Can we get the Office 365 tenancy Id (the Guid that the Graph would recognise).
This is super useful in the following scenarios.

Telemetry
Extranet scenarios in B2B so you know where the user originates
Using the MS Graph once it comes

In what scenarios are the users tenant id and sites tenant id (#98)  different?

If the user is an 'extranet' user is B2B then they must have come from another tenant. I'm not sure MS Graph makes this available like this though.

So the problem here is that pulling some user info into the context object is expensive.  If we happen to have it already on the server, then it is cheap, but if we have to contact AD to get the information (which this one falls under), then it will slow down the request on the server a fair chunk.


Weekly view

Added a new property 'calendarFormat,' which can either be "monthly" or "weekly".
Simplified the eventMap data structure. Instead of having a map of maps, it's now just a map keyed by date (YYYY-MM-DD) string.
The more interesting change is in getStartMoment().
Please let me know what you think and if you'd like me to clarify and/or clean up anything.
Closes #28

Awesome! Thanks for the PR. Will try and review this weekend.

@noitcudni I am seeing a few issues with how the surrounding logic is works in this mode.
When I attempt to implement your changes on the example app of mater I see two issues that needs addressing before moving further.

Dates are rendered incorrectly.
Month in header changes to the next month when it shouldn't.

Thanks for the feedback. I'll take a look. I didn't know the option of swiping left and right was available.

@christopherdro
Thanks again for you detailed screen shot (how did you make that by the way?)
The following is what I fixed.

Make scrollEnded() aware of the weekly mode.
Preinstantiate prior and next 2 weeks in getMonthStack(), which I renamed to getStack().

@noitcudni Thanks for updating the PR. I will give it another go in the next days.
The app to create gif screenshots is called Licecap.
http://www.cockos.com/licecap/

I noticed that the calendar can actually be "scrolled" vertically, so I changed automaticallyAdjustContentInsets={false} inside the ScrollView. Swiping left and right still work, while scrolling vertically doesn't do anything any longer.

Would love to see this PR get in.

@noitcudni Can this be resolved with master?

I'll do it in the next couple days.
On Sun, Feb 12, 2017 at 8:02 AM, coderdave<EMAIL_ADDRESS>wrote:

@noitcudni https://github.com/noitcudni Can this be resolved with
master?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/christopherdro/react-native-calendar/pull/87#issuecomment-279227957,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE0Z55IM0X-qdIfmKTrecJem-VND-0Uks5rby0AgaJpZM4LXxlI
.

Also, this.state.currentMonthMoment is undefined on line 285 in Calendar.js. Should it be this.state.currentMoment now?

Looks like the PR renames those state variables to exclude the word moment. I think we can leave that out for now as it's likely to continuously create conflicts with other PR's that might get merged before this one.

So, should I stay put and wait?
On Mon, Feb 13, 2017 at 8:35 PM, Christopher Dro<EMAIL_ADDRESS>wrote:

Looks like the PR renames those state variables to exclude the word
moment. I think we can leave that out for now as it's likely to
continuously create conflicts with other PR's that might get merged before
this one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/christopherdro/react-native-calendar/pull/87#issuecomment-279605937,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE0Z5-r0dLAXQrz4F9qcZS5kVjYyV08ks5rcS8sgaJpZM4LXxlI
.

Would love to see this get merged, has there been any progress @noitcudni @christopherdro ?

I have been busy at work. I might be able to start working on this again soon. @christopherdro, would you want me to get started on this soon? Or, do you still want me to hold off and wait for other pull requests to get pulled in first.

@GU5TAF @noitcudni Sorry for the late response. Been quite busy myself the last couple weeks.
I will spend some time this weekend looking over everything.

Would be nice to wait for this to get merged since it has a significant amount of changes.
https://github.com/christopherdro/react-native-calendar/pull/107

What needs to be done to get this merged :)?

I might have some free time now to look into the merge conflicts. Are you guys still trying to prioritize other pull requests?

@coderdave @christopherdro I just did a 3 way merge from your master into my fork's master.  Please review at your earliest convenience.

Hey @noitcudni thanks for the work! But it seems like it's not working properly on ios if I set scrollenabled. I currently on RN 0.44.0.

@Marswang92. What do you mean not working properly? Does it scroll but displays the wrong dates? Or, it doesn't scroll at all?

So if I set scrollEnabled, the height of calendar will be set as the same as monthly and dates will not show up on the first render.

scrollEnabled removed, everything works well.

@noitcudni Any clue?

Hey ,the rendering problem has been solved via setting removeClippedSubviews to be false, my bad.

@Marswang92 sure thing.

should the documentation be updated for these changes?


Different values on the same page?

Hi, I use the library to calculate my header size.
I use 24.h which is equal to 193 pixels.
but when I navigate to a different page and then come back to the old page, 24.h equals 160 pixels, and the header size changes.
Why is that?
I found out that opening and closing the keyboard fixes that issue and the value returns to 193 pixels

Found out that I use Sizer builder on the first page and on the second page there is a library that uses Sizer builder too.
That causes the issue.


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Maximum frequency for stm32l0 increased to 32MHz

Fixes #110

@dbrgn May i ask you label the pr as 'hacktoberfest-accepted'? Looks like they have stricter rules this year.

@akashihi done! As far as I understand the rules, it will count if it's either merged or labelled. Since I don't want to merge myself without someone else looking at it, I added the label.

Good catch! LGTM


object markers render outside of draw distance

reproduction steps:

use shift to mark an object so it is highlighted
get it outside of the draw distance
the highlight will still be drawn even though the object itself won't

example with table in edgeville bank:

I dont see a reason why this should be implemented, and also why is it marked as enhancement

Looks weird depending on gpu draw distance:
top left - 20 draw distance
bottom left - 70 draw distance
also with gpu plugin disabled you'll get bottom left

Yeah when I reported it I thought this was a bug given that it didn't behave the same way as the other markers that wouldn't render outside of draw distance - not an enhancement. It really isn't that important but I thought it should at least be documented?


\ should choose a space if none is inferrable

I wish I could type
D=Derivative()
B=dirichlet()
x=Fun(identity)
u=[B,D^2+x][1.,0.]
and it chooses the space from x.  At the moment, the x is interpreted as Multiplication(x,AnySpace()) so that spaces can be inferred from other operators.  (Since Chebyshev can multiply other spaces like Jacobi).  Some system for choosing a default space should exist when it returns AnySpace.

I’m thinking of adding
choosedomainspace(operator,rangespace)

that will choose a domain space given the range space, but where the range space can be changed in the process.  So we’d have
choosedomainspace(Derivative(),Ultrasperical{0}()) == Ultraspherical{0}()
choosedomainspace(Derivative(),Ultrasperical{1}()) == Ultraspherical{0}()
choosedomainspace(Derivative(),Ultrasperical{2}()) == Ultraspherical{1}()

choosedomainspace(RealOperator(),Laurent()) == ReImSpace(Laurent())

choosedomainspace(::,ds)=ds		# default, and equivalent to current situation

This would navigate down the tree and promote the spaces as neccesary on the way back up.  One use case that's not clear is the original question: should
choosedomainsace(Multiplication(x,UnsetSpace()),AnySpace())
return domainspace(x)?  there could be an issue with the "correct" space coming from another part of the tree... but maybe this is an edge situation where its fine to insist that spaces are specified by hand.

That was easier than expected :) It defaults to
choosedomainspace(A::Operator,::)=domainspace(A)
but operators that support UnsetSpace can override it.  Right now only Multiplication does, but RealOperator is next


Support configurable severity per ruleset/rule in XML and Sarif output

Summary
This PR implements configuring the severity in the XML and Sarif output, which is the first step to implement #3274. The implementation is also different from #3306, because the severity in the configuration can only be a simplified version and does not allow custom strings.
Implementation
Adding the severity as a property of Issue does not work: Each rule initializes the issue first then uses the issue.id to initialize ruleConfig. If we want to use ruleConfig to parse severity, it has to be done after issue is created.
We can break such cycling dependency by refactoring, but I would prefer doing so in the next PR where we can remove the existing Severity and replace it with the new SeverityLevel.
The choice here is to add the severity as a property of Finding, which is appended at the end to mitigate incompatibility. However, since Finding and its children are not data class, I need to create copyWithSeverity to override the given severity.
Followup
I did not deprecate or remove Severity, maxIssues, warningsAsErrors in this PR. If this is a good time to do so, I would be happy to do those in a follow-up PR.
Test
In addition to the unit test, I also performed an integration test in a different repo. Users can configure severity at ruleset level and rule level.
style:
  severity: error
  MagicNumber:
    severity: ignore

I also realized that the severity, for now, is only in XML + Sarif. We are not differentiating severities in HTML or console output. If this is a good idea, please react to this comment so I will open a new issue for supporting them.

For txt, the output should include the severity level.
For console, we should color-code each finding in addition to showing the severity.
For HTML, we should both display the severity and use a different background.

I agree with all except html. I don't think that we should change all the background. We could use a label and only change that label color.

If this is a good idea, please react to this comment so I will open a new issue for supporting them.

For txt, the output should include the severity level.
For console, we should color-code each finding in addition to showing the severity.
For HTML, we should both display the severity and use a different background.

+1 great
I agree with this suggestion. I wouldn't want to build it overly complicated, so simplicity is the key here.

Looks good!
Anyhow, a refactoring would be great in separate PR in order to implement the severity levels properly and keep the architecture tidy. Then there's no need for copying stuff around and no risk to introduce cyclic dependencies with future refactorings.

We can break such cycling dependency by refactoring, but I would prefer doing so in the next PR where we can remove the existing Severity and replace it with the new SeverityLevel.

#3127 prototyped a new Rule API, which is much cleaner, but also not backward compatible. In order to keep the severity within every Issue, it will be a breaking change and I would prefer to wait till Detekt 2.0 for making such changes.
I have seen Detekt 2.0 mentioned in a few issues. Do we know when and how we are planning this? I imagine there will be a separate 2.0 branch where we can merge all breaking changes.

#3127 prototyped a new Rule API, which is much cleaner, but also not backward compatible. In order to keep the severity within every Issue, it will be a breaking change and I would prefer to wait till Detekt 2.0 for making such changes.
I have seen Detekt 2.0 mentioned in a few issues. Do we know when and how we are planning this? I imagine there will be a separate 2.0 branch where we can merge all breaking changes.

At the moment those PRs are tagged in the milestone field with the corresponding detekt version.
Nevertheless, with the number of those PRs growing I suggest to create a new v2.0 branch for simplicity reasons.

We talk a lot about 2.0 but we don't have the list of things that we want to change. So I'm not sure about creating a new branch. #3127 pretends to be retrocompatible. I mean, the idea is to deprecate the old api and keep it until 2.0.

This PR also depends on #3320


Chrome 48 removes  SVGGraphicsElement.getTransformToElement()

Unfortunately the most recent beta version of chrome (48.0.2564.23) removes SVGElement.prototype.getTransformToElement which is used when creating edges.
The motivation behind removing this feature can be found here with the actual Google Chrome issue tracking this here
Therefore using dagre-d3 in this latest beta version of Chrome throws errors like:

TypeError: elem.getTransformToElement is not a function(…)

As [suggested by the JointJS team](http://jointjs.com/blog/get-transform-to-element-polyfill.html) this function can be poly-filled where it is required. We are working around this locally with a polyfil for now, but it clearly would be better for `dagre-d3` to centralise a fix (polyfil or otherwise) so that everyone using Chrome 48 can benefit.

Any thoughts on how best to fix this in `dagre-d3`?

As an example of the impact of this, here is a screenshot of the sentence tokenisation demo in Version 48.0.2564.23 beta (64-bit):

@rbojha, can you post the polyfill you are using? Are you using the same polyfill that JointJS posted?

yes @robhobbes, we are using the same polyfill as JointJs Team posted..
SVGElement.prototype.getTransformToElement = SVGElement.prototype.getTransformToElement || function(elem) {
	return elem.getScreenCTM().inverse().multiply(this.getScreenCTM());
};

This should be fixed in 0.4.13 (verified on Chrome 48). If you see any issues please open a new issue. Thanks.

Wow so the justification for removing this from Chrome was it is too difficult to fix the issues? That sounds like a crappy reason what am I missing?

Some notes for people encountering this issue: I also encountered this issue in v0.4.10, and I tried v0.4.13 and found the dagre-d3/dist/dagre-d3.core.js is missing from that version. It turns out that v0.4.17 works for the current version of Chrome.

After deploying this fix I notice these warnings suggesting tooltips will break shortly:
tipsy.js:30 'SVGElement.offsetWidth' is deprecated and will be removed in M50, around April 2016. See https://www.chromestatus.com/features/5724912467574784 for more details.
tipsy.js:31 'SVGElement.offsetHeight' is deprecated and will be removed in M50, around April 2016. See https://www.chromestatus.com/features/5724912467574784 for more details.

@jrgleason if you're interested, it's been removed from the SVG spec, it's not just Chrome removing a feature.


include the requirement flag on the thrift spec

This amends the parser to include the requirement flag so it can be used for later validation.
I've used this in my own implementation of TProcessor (not included in these changes) to accomplish something like this:
struct Thing {
    1: required string required_field;
}

class ValidatingProcessor(TProcessor):
    def validate(self, result):
        obj = result.success
        for _, s in obj.thrift_spec.items():
            k = s[1]
            r = s[-1] # requirement will be at the end of the tuple
            v = getattr(obj, k)
            if v is None and r:
                raise TApplicationException(
                        message='{} is required'.format(k))

    def process(self, iprot, oprot):
        api, seqid, result, call = self.process_in(iprot)
        if isinstance(result, TApplicationException):
            self.send_exception(oprot, api, result, seqid)
            return
        try:
            result.success = call()
            self.validate(result)
        except TApplicationException as e:
            self.send_exception(oprot, api, e, seqid)
        except Exception as e:
            # raise if api don't have throws
            self.handle_exception(e, result)
        else:
            self.send_result(oprot, api, result, seqid)

Which, when called, produces:
$ python failing_client.py
<backtrace snip>
thriftpy.thrift.TApplicationException: `required_field` is required

Tests are passing but this doesn't work right with complex (map, list, etc.) types. Please don't merge.

Should be fixed now.

Parser has been rebuit, but this feature was included.


docs: :memo: Created a documentation website

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Guidelines: Workflow name

Make the workflow name guidelines a bit more verbose and detailed.
@netlify /docs/guidelines/pipelines/requirements/workflow_name

@jfy133 - updated. Let me know what you think 👍🏻

Ah one last thing: maybe add a tip block saying we know that they're are a couple that don't have that name, but they are there for legacy reasons? To fend off the inevitably 'but sarek and eager'

Good idea - added 👍🏻

I think this is good to go as well


Conversion method to_js does not work on Float?

Discussed in https://github.com/ruby/ruby.wasm/discussions/210

Originally posted by sebnozzi May 12, 2023
According to the documentation I should be able to call to_js in a Float value, right?
https://ruby.github.io/ruby.wasm/Float.html
Then why does this throw an error?
require "js"

(Math::PI).to_js

Output:
(file): eval:3:in `<main>': undefined method `to_js' for 3.141592653589793:Float (NoMethodError)
 (Exception)

Link:
https://try.ruby-lang.org/playground/#code=require+"js"

(Math%3A%3API).to_js&engine=cruby-3.2.0

The TryRuby is a little bit old. Please try with the latest version

It's not TryRuby I tested this on, but on the latest release.
To reproduce:
(open the JS console)
<html>
<head>
 <title>Ruby/WASM - Float Test</title>
 <link rel="icon" href="data:;base64,iVBORw0KGgo=">
</head>
<body>
 <script<EMAIL_ADDRESS> <script type="text/ruby">
   require "js"
   x = 3.2
   x.to_js
 </script>
</body>
</html>


Download attribute on link elements not working

Thanks for the nice project, folks.
I'm using electron to wrap our web application in a webview and distribute this application this way.
Unfortunately we discovered that links with the download attribute don't work.
E.g. http://davidwalsh.name/demo/html5-download.php
Works perfectly fine in Chrome but unfortunately does not inside of the webview in electron.
I digged the issues already and couldn't find any information on this specific case.
Any ideas on this are more than welcome. :bowtie:
Thanks. :)

I am not sure if downloads of any kind are implemented at the moment.

Okay, is there a getting started guide out there?
If it's desired to have this behavior I might spend some time digging into it. :)

Currently you have to hook to the click event of the link and use http module together with fs to save the file to user's disk.

I have the same need (download a file from the renderer), i have currently implemented a download function like your suggestion using IPC in order to permit the renderer to ask to the main process to download a file by passing the URL, but the request need to be authenticated through session, there is a way to share the session? Or using one chromium Download Manager plugin?
Best regards


installer not working properly for ArcGIS Desktop 10.5

Hello,
When I attempt to run the Geoprocessing installer in 10.5 the GP tool fails.  I get the following messages:
===
Messages
Executing: installarcrest true false
Start Time: Mon Apr 10 16:03:30 2017
Executing (installarcrest): installarcrest true false
Start Time: Mon Apr 10 16:03:30 2017
Running script installarcrest...
Failed script installarcrest...
Traceback (most recent call last):
File "C:\Users\mdaffern\Documents\ArcGIS\Packages\installing_arcrest_1B28CE9D-BD96-49FC-9C54-E490E751518B\v105\scripts\install_arcrest.py", line 203, in
main()
File "C:\Users\mdaffern\Documents\ArcGIS\Packages\installing_arcrest_1B28CE9D-BD96-49FC-9C54-E490E751518B\v105\scripts\install_arcrest.py", line 164, in main
arcrest_zip, arcresthelper_zip = download_arcrest()
File "C:\Users\mdaffern\Documents\ArcGIS\Packages\installing_arcrest_1B28CE9D-BD96-49FC-9C54-E490E751518B\v105\scripts\install_arcrest.py", line 41, in download_arcrest
arcrest_zip = os.path.join(scratch_folder, g_ESRI_variable_1)
NameError: global name 'g_ESRI_variable_1' is not defined
Failed to execute (installarcrest).
Failed at Mon Apr 10 16:03:30 2017 (Elapsed Time: 0.03 seconds)
Failed to execute (installarcrest).
Failed at Mon Apr 10 16:03:30 2017 (Elapsed Time: 0.03 seconds)
====
g_ESRI_variable_1 does appear to be undefined.
Kind regards,
Maura

just pushed an update, can you try this one?  Try opening it from the catalog window in ArcMap too

Hi Mike,
I got the same message.  I ensured that I removed the cached GP package from my profile, also.
Kind regards,
Maura

Could you just try the toolbox in the ArcGIS desktop installer folder?  That is what we use to create the installer.  See if that works.
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Esri_ArcREST_tree_master_ArcGIS-2520Desktop-2520Installer_Install-5FArcRest_Tools_Scripts&d=DwIGaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=efuUxjzLmrCIBM7EBaqNA3ZNqv2Q5XO82BlHQxv5rlwEFnYj6qiVqKT1v2yrO47zHlwkmBPF21o3cVzRzs848tbJYWkUY2xyDoeGH_2gFybBn2Ut7FIg7UvqvU7DpCD_nzhrBW8oe4s7cZsvXnLJ4Q&m=o659oVrj5eVBlTTUeGl6idnQlJ-rxnMldjxFfkHtHZ4&s=lrRa7FI7QeQYM-vau1jSd6pRPnPUqkbgkhtS7osdn5Y&e=

Think I found the issue, just pushed an update.  Please try it out and let me know if it works.  The bug was only if you checked the Download, which had and GP replaced variable.  I changed this and GP should not force replace it.

Hi Mike,
This worked!  Thank you for your help regarding this issue.  Sorry for the delayed reply - was on training the past few days.
Kind regards,
Maura

no worries, thanks for reporting it.


Missing https://new-raku.finanalyst.org/syntax/rule

https://docs.raku.org/syntax/rule exists, expect all the URLS in the old site to work in the new. (added .html just in case, fails either way)

This seems to be a corner case I haven't been aware of. It'll need some time to work on.
Are there any others. I had not thought to compare the rendered html files generated by Documentable and those generated by Collection, that should pick up files generated by one and not generated by the other.

@coke Can you point me to where the static html files are located? Is there are tar ball of the html files.

I don't know how to get this without re-doing the previous documentable compilation process. But it occurs to me that before we go live with the new site, we should be sure to take a tarball archive backup of all the current site's assets.

@dontlaugh  - can you get @finanalyst a tarball of the current docs.raku.org for comparison?

@coke
tl;dr
I have looked at the original POD6 source.
a) The source is highly non-standard
b) Its actually very unclear to me how this file is even generated by Documentable, and if I don't understand it, I wonder how many other people today would.
c) No link to syntax/rule seems to exist in the documentation or .htacess
@coke How did you discover this file?
Suggestion: this seems such a corner case, I suggest we move the issue from GoLive to Cleanup.
Detail
Secondary files are created by parsing the text of =headx Pod::Blocks through a grammar and picking out keywords such as prefix method routine. The rest of the source to the next =head is then copied into a new file together with the occurrences of the same keyword in other sources.
The source for the file at issue is in Type/Grammars.pod6 and is
=head2 X<Rules|Syntax,rule>

inconsistent markup
As you can see, there is no ambient text. Instead the whole title is enclosed in a X<> markup.
This is very inconsistent with other titles in the documentation. It indicates that the documentation was written very early on in the documentation process.
Text in Headers is already picked up for inclusion in a Table of Contents, so marking the same text for inclusion in a Glossary is overkill.
puzzling origination
Quite how Syntax,rule is picked out, and not Rules is confusing, as the recurse-to-str sub (the code is common to both Documentable and Colllection) should only pick up Rules.
In addition, none of the regexes in the grammar used to parse the header should match either Rules, or Syntax,rule. So I cant even see how the secondary code is even generated.
no link in documentation
Looking through all the errors in more->anomalous links, there is no entry for syntax/rule. This means that it wasn't registered as a link in the Documentation as it currently exists.
@coke How did you discover the file in the first place?

@coke I think we can eliminate this issue from doc-website. It seemed odd in the Documentable build, its even odder here. I actually don't understand the issue that JJ opened.
@Altai-man also found it strange.
Like I said above, there is no Link I can find in the existing Documentation that references this web page.
I suggest we close this issue.

Well, technically it is rendered: http://<IP_ADDRESS>:10010/syntax/rule
@finanalyst understanding of how grammar creating secondary pages is correct, I understand it as well (related to "how many people issue"). While I will gladly vote for secondary pages being generated for routines, when it comes to language pages, I'd consider them less useful myself, so I don't think it's a big deal. In short: remove it.

@Altai-man I thought you probably were one of the only people :)
You changed my mind about secondaries in general, so I put the work in to generate them.
However, syntax/rule is not generated by Collection, and since the regex rules are the same, I actually don't know how Documentable Doc::Page::Secondary does generate them. Working out why seems more effort than its worth.
So, we can close this issue for the time being


Erroneous conditional in getLoaderOptions for multiRule

When a multiRule is used in the webpack config , in this case:
{
  test: /\.svg$/,
  use: [
    {
      loader: 'svg-sprite-loader',
      options: {
        symbolId: '[name].[hash]',
        extract: true
      }
    },
    'svgo-loader',
  ],
},

an error is raised: TypeError: Cannot read property 'options' of undefined, originating at svg-sprite-loader\lib\utils.js:228.
I've managed to identify the root-cause of the issue being that the conditional used in the map function is r.loader === loaderPath, which will never be true, as r.loader simply is the name of the loader (in this case svg-sprite-loader), while loaderPath is an entire system path. Hence, I'm now submitting a quick-fix, which simply checks for r.loader to be path of the loaderPath. A more reliable solution would be preferable.
Details
OS: Windows 10
Node: 7.2.1
NPM: 3.10.10
svg-sprite-loader: 2.0.4

Coverage remained the same at 95.055% when pulling f03200ea8ca72ea23278ad4ed4b348167ec609fb on wanecek:multirule-options-fix into 3b4045ffcdb80c934ba770086c474ee4c067e66b on kisenka:master.

@wanecek thanks! Published as 2.0.5


Fix  Push notifications subscription expiration/removal

Description:
Related issue: fixes #7758
Example entry for configuration.yaml (if applicable):
notify:
  - name: HTML5PUSH
    platform: html5

Checklist:
If the code communicates with devices, web services, or third-party tools:

[x] Local tests with py.test run successfully.
[ ] Tests handle HTTP response code from the push service (TODO)

It looks like you branched off master. Could you rebase off dev?

I have cherry-picked your commits into a new PR: https://github.com/home-assistant/home-assistant/pull/7874
Thanks for your contribution 👍


Is there a guide on how to start an amm_arb strategy on two dexes? Or a single dex + cex. I'm trying to do so but it won't work.

Hi there? The thing I'm trying to do is that I created a bot with an amm__arb strategy with both connectors on the same network (uniswap mumbai), and in the log pane I'm getting "network not ready: waiting for allowances: use trading interface to do manual approval".

Not yet - note we are in the midst of refactoring how the amm-arb strategy works
https://github.com/hummingbot/hummingbot/pull/6242


[ON_ERROR]: message: Path not found, code: 4005

Can you please look this issue. I am getting [ON_ERROR]: message: Path not found, code: 4005 error when connecting. I am running the example project and changed the configs and channels and events but no avail.
import 'package:laravel_echo_null/laravel_echo_null.dart';
import 'package:pusher_client/pusher_client.dart';

const String appId = "2340143";
const String key = "public-App-Key-001";
const String cluster = "mt1";
const String hostEndPoint = "<IP_ADDRESS>";
const String hostAuthEndPoint = "http://<IP_ADDRESS>:8080/broadcasting/auth";
const String token = "4|6EbdvFEl9TmkPIODZbgBpImqtdMqG2dUavrJLaJBQ";
const int port = 6001;

Echo<PusherClient, PusherChannel> initPusherClient() {
  PusherOptions options = PusherOptions(
    host: hostEndPoint,
    wsPort: port,
    cluster: cluster,
    auth: PusherAuth(
      hostAuthEndPoint,
      headers: {
        'Authorization': 'Bearer $token',
        'Content-Type': 'application/json',
        'Accept': 'application/json',
      },
    ),
  );

  Echo<PusherClient, PusherChannel> pusherEcho = Echo.pusher(
    key,
    options,
    autoConnect: false,
    auth: options.auth?.toJson(),
    authEndpoint: options.auth?.endpoint,
    host: options.host,
    moreOptions: {
      'cluster': options.cluster,
      'encrypted': options.encrypted,
      'activityTimeout': options.activityTimeout,
      'pongTimeout': options.pongTimeout,
      'wsPort': options.wsPort,
      'wssPort': options.wssPort,
      'maxReconnectionAttempts': options.maxReconnectionAttempts,
      'maxReconnectGapInSeconds': options.maxReconnectGapInSeconds,
    },
  );
  pusherEcho.connector.onConnectError(
    (error) => print('[Pusher Connection Error]: ${error?.message}'),
  );
  return pusherEcho;
}

Generally, this error is because the endpoint url is wrong, check it is correct.
Note: If you are using a phone to run the application, make sure that the phone and the computer are connected to the same network, then find the local ip of the computer (in windows, run the command ipconfig in cmd) and replace it with <IP_ADDRESS>.
If still get error tell me to help you.
Thanks for send your issues.


Using an imported model mount the entire folder in the inference server

Bug description
When user is importing a model and starting an inference server for it, the entire user's folder where the model is located is getting mounted in the container which causes performance issues or the inference server not able to start.
Operating system
mac OS
Installation Method
from Podman-Desktop extension page
Version
1.3.x
Steps to reproduce

Download a model in a folder with a lot of files
Import it from ai lab model catalog
Start Inference Server

Relevant log output
No response
Additional context
No response

This is our mountconfig
https://github.com/containers/podman-desktop-extension-ai-lab/blob/21945c5a450db57b95ada11d48f1393e09f03f2a/packages/backend/src/workers/provider/LlamaCppPython.ts#L88-L92
Historically we choose to mount the folder where the model is located because we wanted to support models in non-gguf format, which tends to be multiple files.
This should probably not apply to LlamaCppPython, as it require only the file, we should mount the file as a target, and not the folder