Empty object in ternary operator produces invalid type

Bug Report

🔎 Search Terms
empty object, {}, inferred type

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ

⏯ Playground Link

Playground link with relevant code
💻 Code

const x = true ? 'I am a string' : {} // type of `x` is: {}

🙁 Actual behavior
Type of x is inferred as {}
🙂 Expected behavior
Type of x should be inferred as "I am a string" | {}.
For example, if we try this code:
const x = true ? 'I am a string' : 5

...the type of x is inferred as "I am a string" | 5

Strings are valid {}, so the type gets simplified. This is working as intended.

See https://www.typescriptlang.org/docs/handbook/type-inference.html#best-common-type

Strings are valid {}, so the type gets simplified. This is working as intended.

why isn't this code simplified?
type X = string | {} // `X` is string | {}

Subtype reduction isn't required to happen, but is allowed to happen at any point. There's not a bug here.

but do you think this type reduction is correct / meaningful to developers?

I encountered this issue when composing a dynamic object. For anyone seeing this and trying to find a workaround, couple things to note :

Subtype reduction happens when assigned to a variable, so you might get away with inlining this code instead of storing it in a variable.

(true ? 'I am a string' : {}) // Type is 'I am a string' | {}
const myVar = true ? 'I am a string' : {} // Type is {}

In my case, both return values of the ternary were enumerable, and cloning it with a spread prevented the subtype reduction, even when assigned to a variable.

const computeData = () => ({ foo: "bar"})

const subtypeReduced = true ? computeData() : {}; // Type is {}
const preservedType = {...(true ? computeData() : {}) } // Type is { foo?: string | undefined; }


Add ontology / connections graph visualization

Hi!
I was recently thinking about similar service!
What do you think about ontology/connections graph visualization option?

I think people like looking at graphs but they are ultimately very difficult to navigate as a tool. They are eye candy, not something productive or very useful. Closing for now.


bug: can't assemble 'jmp (r0)'

The assembler can't handle the following instruction:
jmp  (r0)
It issues the error:
JMP Rn is illegal
I believe the error is in assemble.c:
diff --git a/assemble.c b/assemble.c
index 67f0052..47856b1 100644
--- a/assemble.c
+++ b/assemble.c
@@ -1136,7 +1136,7 @@ static int assemble(
                                 return 0;
                             }

-                            if (op->value == 0100 && (mode.type & 07) == 0) {
+                            if (op->value == 0100 && (mode.type & 070) == 0) {
                                 report(stack->top, "JMP Rn is illegal\n");
                                 /* But encode it anyway... */
                             }
which is meant to detect attempts to assemble unexecutable instructions like:
jmp  r0

Hi @bjorend, I don't think this repo is being maintained any more, but
there is my fork: https://gitlab.com/Rhialto/macro11
where this bug has been fixed in commit
https://gitlab.com/Rhialto/macro11/-/commit/a24e3ba3 .

Closing, please follow Rhialto's fork as mentioned above.


Inserting custos do not replace existing ones

In the code, and functionally, when placing a custos on a staff, if one exists already it should be replaced by the new one. Currently this does not happen.

Fixed in e1f77d9


how would i declare formNoValidate with this wrapper

I added noValidate to the Formsy.Form component. and that seems to work.
but now i cannot see the errors i declare on the FormsyText. I tried validationErrors and errorText

I got it to work. instead of validations="isEmail", i was doing validators="isEmail". Problems of working too late 😆


Menu bar item text

What steps will reproduce the problem?

To reproduce just use any drop down menu on the menu bar.
The text overlaps the logo/symbols used in the drop down menus in the menu bar.

Hey @Geo-George, thanks for reporting. This was fixed in our 5.0.1 version, released more than two weeks ago and soon to be part of Anaconda.


Intermittent "Permission denied" errors using sendgrid node client 1.9.2

Issue Summary
Intermittent "Permission denied" errors using sendgrid node 1.9.2
About 50% of requests are failing with this error since Mar 7th.
is there is any API redirection from older version to newer version?
Creating an issue here to see if the client needs to be updated to avoid any anticipated change in behavior.
Steps to Reproduce
Code Snippet
# paste code here

Exception/Log
error - Permission denied

Technical details:

sendgrid-nodejs version: 1.9.2
node version: 10.15.0

Version 1.9.2 is almost 5 years old. Recommend upgrading to a later version to see if problem still exists. That said, I don't know of any recent changes that would cause this sort of behavior. Would need some sample code and more details about the error/exception to be able to debug.


Line spacing for doc.p

This has been an issue. I'm trying to add a double line space and have followed the documentation and Line 720, Line: :double or spacing, doesn't work. Even with adding a custom_style. What am I missing or where can I refer for an example? Thanks.

@jdugan I've tried overriding the default style, and have followed the documentation. It's confusing seeing line 480 or line :double and so on without knowing exactly how to achieve the double spaced line element. Can you provide clarity?
doc.style id: 'Normal', name: 'normal', font: 'Times New Roman', size: 24, line: 720, color: '000000', style: 'Normal'
oc.p('some text'.s, font: 'Times New Roman', size: 24, align: :center, style: 'Normal') do
style 'Normal'
end

@ArisNance This should be done in style block
docx.style do
    id 'CustomStyle'
    name 'Custom Style'
    line 200
end

  docx.p 'some text' do
     style 'CustomStyle'
     align :left
     size 24
     bold true
   end


Moving the computation, not the data

This may be completely off base for your project, but I thought I would throw it out there.
What do you think about trying to move computations closer to the data. For example, if I am maintaining a data-set with ocean temperatures, my cluster could also accept a computational payload to run on the data locally. Moving a result across the wire would be much easier than moving the entire data set.
This of course opens an whole new set of problems, but the problems may be worth the reward.

I think we can close this once we have #388

After a whole year of feedback from the community, we recently published a new version of dat, which focuses on syncing filesystems. We are getting close to testing how to more effectively move code to data. Would you all mind trying out the new dat with npm install -g dat? You can read about the new dat announcement on the website and how it works in the docs.


[BUG] - "InvalidDateTime" when keyboard input enabled

Describe the bug
When keyboard input is enabled, it is possible to easily trigger a "InvalidDateTime" error in the input area
To Reproduce

See https://github.com/cjohnston-nuvalence/ngx-mat-timepicker-keyboard (I couldn't figure out how to get it hooked up to Stackblitz) or simply set [enableKeyboardInput]="true" on a time picker.
Click in the input field to open the picker
In the minutes input, click at the left of the input so the cursor is before the digits
Start entering numbers
Any number >= 1 will immediately trigger the InvalidDateTime error
Any number of 0s may be entered, but the first non-zero number will trigger the InvalidDateTime error

Expected behavior
No error when editing values via keyboard.
Screenshots

Desktop (if execution):

OS: [MacOS, Windows, iOS]
Browser [chrome, safari]

** Versions

ngx-mat-timepicker version 15.1.2
@angular/core version 15.1.0
@angular/material version 15.1.0
typescript version 4.9.4

Sorry pal, but there must be something wrong with your code...
I tried on V14 and V15 and it works just fine.
Cheers

My example is an out of the box install of Angular 15 with Material and your module. There is no "my code" in this case. If you clone the repo and run it you should see the issue. Put the cursor before the digits and start typing...

Ok I was able to reproduce.
Anywayy I agree that it could be handled better, but I don't see why you put the cursor at the beginning.
Just leave the cursor at the end and you will be able to type your numbers without any errors.
Otherwise if you put the cursor on the left, you simply delete before typing.
I'll keep this in mind and try to prevent the "Invalid" to be triggered, but I think it's an edge case...very edge...

Fixed in v 15.1.4 @cjohnston-nuvalence


Sprint 1 - Task: #2.2 Input answer: User can submit an answer

Implemented in commit 2d30ed5df5289f41304280b5a9cf2c45a0dfb10b


Different training efficiency between openpai job and nv-docker run on machine.

Short summary about the issue/question:
Different training efficiency between openpai job and nv-docker run on machine. Bottleneck is about pinned memory usage.
Brief what process you are following:
We deploy openpai on our new cluster. Each node is a physical machine with 8 V100 cards. I test pytorch framework on openpai, which is much slower than I expect. After the investigation, I find out the bottleneck is about the pinned memory usage during training process.
As you may know, the DataLoader class in pytorch has a parameter pin_memory which defines whether using pin memory to accelerate tensor transfer between host memory and device memory. Setting pin_memory to True, the training speed is dramatically 10x slower than False. I also try the same docker image with same code on the same machine, by ssh logging in. I use nv-docker run the image and test the code. The training speed with False option is equivalent to the same False option launching by openpai. The speed with True option is a bit faster than False option as expected.
The only difference between the test, is the docker launching parameters. I checked the log of openpai job. The openpai seems not using nv docker. It launched the docker with docker run --runtime=runc --device=/dev/nvidiactl --device=/dev/nvidia-uvm --device=/dev/nvidia0 --device=/dev/fuse .... While I ran the docker on machine with docker run --runtime=nvidia .... I wonder if the docker run parameter difference results in the bad behavior of pinned memory.
I also ran some test on tensorflow and keras. Both training speed is much slower than running docker on local machine. But I don't find how to disable pin memory use in these two frameworks. So I can't give a comparison test with tensorflow and keras.
However, we also have some virtual OS machine with V100 on cloud deployed openpai. They behave the same with nv docker run on local and openpai launching, as it should be. Which makes me very very confused about our new physical machine cluster...
OpenPAI Environment:

OpenPAI version: 0.11.0
Cloud provider or hardware configuration: V100
OS (e.g. from /etc/os-release): Ubuntu 16.04.4 LTS
Kernel (e.g. uname -a): I'm not sure...
Hardware (e.g. core number, memory size, storage size, GPU type etc.): V100
Others:

Anything else we need to know:
Any advice would be appreciated. Thanks.

could you share your job submission config file?

@fanyangCS
Thanks for the reply. I have assigned very large memory and shm-size for the job to run. As I mentioned in the post, the same job could run correctly on other virtual OS cluster.
Currently I can't reach the cluster. I will post my job config file later.

My test code is based on horovod benchmark code on horovod base image. We have built a private harbor to store docker images. I could push my image and test code to public, if needed.
{
  "jobName": "test_pin_memory",
  "image": "<IP_ADDRESS>/tyj/horovod:0.16.4",
  "virtualCluster": "default",
  "taskRoles": [
    {
      "name": "main",
      "taskNumber": 1,
      "cpuNumber": 6,
      "memoryMB": 10240,
      "gpuNumber": 1,
      "command": "nvidia-smi && mkdir /nfs  && mount -t nfs4 <IP_ADDRESS>:/nfs /nfs  && cd /nfs/tyj/benchmark && python pytorch_synthetic_benchmark.py",
      "shmMB": 10240
    }
  ],
  "jobEnvs": {
    "isDebug": true
  }
}

I pushed my test code to my repo. Hope it could help.
Thanks.

@fanyangCS
Is there any public doc about how to enable nvidia docker runtime? I could share it to our administrator, and make further test. Thanks.

@rtrobin If you used pai to deploy k8s, you'r nodes sure already have nvidia docker runtime installed. If not, execute these commands to install.
This will not configure nvidia-container-runtime as default runtime, and I believe you do not need it to be. With above setup you can run some docker using nvidia runtime just execute docker run --runtime=nvidia xxx. Pai will not start container in nvidia runtime runtime but with almost same environment, you can see task start command here.
Below is just for your information, I do not think you need default container runtime to be nvidia:
To set nvidia runtime as default docker runtime, you can execute ./paictl.py service stop drivers and change the config of set-nvidia-runtime to True and then execute ./paictl.py service start drivers to enable nvidia container runtime as default runtime. This command will execute these commands in each of your worker node.

Sorry for the late reply. We are still trying to make it work. I will post update later.

@xudifsd I also have this problem in pai-v0.13.0.
The nodes have installed nvidia docker runtime, and the following error reports when set the set-nvidia-runtime to true and start service:
Error: failed to start container "job-exporter": Error response from daemon: OCI runtime create failed: container_linux.go:345:starting container process caused"process_linux.go:424:container init caused \"process_linux.go:407:running prestart hook 0 caused \\\"error running hook: exit status 1, stdout: , stderr: exec command: [/usr/bin/nvidia-container-cli --load-kmods configure --device=all --utility -pid=15315 /var/lib/docker/aufs/mnt/90adf647b99ea637b76d0bfa2e49a7f631dbb22c9510a8bf00591e759c5de05]\\\\nnvidia-container-cli: initalization error: cuda error: system has unsupported display driver combination\\\\n\\\"\"": unknown Back-off restarting failed container
@rtrobin did you fix this problem? @xudifsd What should I do to solve this problem? thanks.

@edenbuaa seems your node do not have driver installed correctly. Does this node previous installed some drivers?

@edenbuaa @fanyangCS No, I haven't.
We have done some upgrading work this week. We upgraded our installed docker, openpai, and other stuffs. Now we could use nvidia runtime in openpai. However, the training speed is still the same.
We are doing further investigation now. I will keep the post updated later.

I have found the tricks about my training speed problem. The use of runc or nvidia doesn't affect it.
Our gpu node has two NUMA nodes. The docker parameter --cpus=6 doesn’t allocate 6 cpu cores to the job, but load every cores balanced under cpu rate 6/cpu_core_nums. To run the training job correctly, the docker parameter --cpuset-cpus should be used.

thanks for reporting back. we really appreciate it.


Dropdown list coming with new UI theme has become unreadable

Description:
The dropdown list shown with new UI has come in a grey background with grey text. Can only read properly when we move over the dropdown elements.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Can you please provide a screen shot or elaborate more about this?

This is fixed in EI Tooling 6.3.0


AAD MFA without AAD P1

Does this mean that the only way to have MFA for our AAD Accounts is to have the AAD P1 license?  We would like to add the MFA a la Carte and not all the features of the AAD P1.  thanks for your help!

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

ID: feb03e63-f543-d85b-9daf-16b4d1e13a60
Version Independent ID: da9dcdd2-70d4-e08e-18f2-6e6a1489937a
Content: Azure MFA versions and consumption plans
Content Source: articles/active-directory/authentication/concept-mfa-licensing.md
Service: active-directory
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@gate502
Thanks for your feedback! We will investigate and update as appropriate.

Hi @gate502 , unfortunately that is what it means. MFA is no longer available as a standalone service. Per the licensing agreement, in order to apply MFA to your users the users should have the P1 license.

Hope this helps and sorry that it is no longer standalone! We will now proceed to close this thread.


Pathfind Check does not work.

Console log https://pastes.dev/mSKwTIVN1u
Configuration https://pastes.dev/ksdFhMji5R

Zombie: Pathfind Nerf
Console Log
[17:12:50 INFO]: [NerfFarms] Performing hasPathToPlayer on Zombie
[17:12:50 INFO]: [NerfFarms] Nerfing Zombie because they never could reach the player.
[17:12:50 INFO]: [NerfFarms] configMod Setting clears EXP.
[17:12:50 INFO]: [NerfFarms] configMod Setting clears Drops.

There is no configuration setting to reference.
Issue with pathfinding.

Video shows that issues exist with pathfinding, the player is reachable.
Video demonstrates a standard critical strike.

Creeper: Pathfind Nerf
[17:13:40 INFO]: [NerfFarms] Performing hasPathToPlayer on Creeper
[17:13:40 INFO]: [NerfFarms] Nerfing Creeper because they never could reach the player.
[17:13:40 INFO]: [NerfFarms] configMod Setting clears EXP.
[17:13:40 INFO]: [NerfFarms] configMod Setting clears Drops.

There is no configuration setting to reference.
Issue with pathfinding.

Video shows that issues exist with pathfinding, the player is reachable.
Video demonstrates a standard critical strike.

Originally posted by @Peashooter101 in https://github.com/ADHDMC/NerfFarms/issues/1#issuecomment-1218590342

Received logs from @Wuffeh

Debug: https://pastes.dev/JXoQV8AOpT
Config: https://pastes.dev/Eb2tVHkyfq

[15:56:51] [Server thread/INFO]: [NerfFarms] Performing hasPathToPlayer on Creeper
[15:56:51] [Server thread/INFO]: [NerfFarms] Player Location:<PHONE_NUMBER>1620393 72.0 -71.42617979932977
[15:56:51] [Server thread/INFO]: [NerfFarms] Final Location: 157.0 72.0 -72.0
[15:56:51] [Server thread/INFO]: [NerfFarms] Calculated Distance: 0.8849774720792015
[15:56:51] [Server thread/INFO]: [NerfFarms] Creeper has died and not been nerfed.

[15:57:02] [Server thread/INFO]: [NerfFarms] Performing hasPathToPlayer on Creeper
[15:57:02] [Server thread/INFO]: [NerfFarms] Player Location:<PHONE_NUMBER>1463283 72.0 -73.17392962665788
[15:57:02] [Server thread/INFO]: [NerfFarms] Final Location: 157.0 72.0 -74.0
[15:57:02] [Server thread/INFO]: [NerfFarms] Calculated Distance: 1.2741883512942953
[15:57:02] [Server thread/INFO]: [NerfFarms] Nerfing Creeper because they never could reach the player.

[15:57:12] [Server thread/INFO]: [NerfFarms] Performing hasPathToPlayer on Creeper
[15:57:12] [Server thread/INFO]: [NerfFarms] Player Location:<PHONE_NUMBER>2057512 72.0 -73.44155256831165
[15:57:12] [Server thread/INFO]: [NerfFarms] Final Location: 159.0 72.0 -74.0
[15:57:12] [Server thread/INFO]: [NerfFarms] Calculated Distance: 0.7126858594634751
[15:57:12] [Server thread/INFO]: [NerfFarms] Creeper has died and not been nerfed.

[15:57:22] [Server thread/INFO]: [NerfFarms] Performing hasPathToPlayer on Creeper
[15:57:22] [Server thread/INFO]: [NerfFarms] Player Location:<PHONE_NUMBER>8721937 72.0 -73.87943458839685
[15:57:22] [Server thread/INFO]: [NerfFarms] Final Location: 159.0 72.0 -74.0
[15:57:22] [Server thread/INFO]: [NerfFarms] Calculated Distance: 0.6448619821802303
[15:57:22] [Server thread/INFO]: [NerfFarms] Creeper has died and not been nerfed.

Pathfinder locates the player and so long as the path is within 1 block of the position, it triggers a nerf.
The nerf should not be occurring, player is killing in an open area. Distances are very close to 1, perhaps requiring adjustment / config setting? Might be able to tune to be closer until adding config setting.

Received logs from @RhythmicSys

Debug: https://pastes.dev/M3AKjUniuV

The difference between these logs and the one provided by Wuffeh is that Wuffeh likely performed these on level ground, whereas Rhythmic performed these in cave systems. Noted that increasing the pathfinding distance to 1.5 is not enough.

Determined that Pathfinding will not be a useful determination for mob farms.
Pathfinding's final location may serve as a useful tool for another more specific check, however as a check that happens all the time, is not useful.
Consider the situation where a player is killing a mob right beside them. The mob cannot reach them, but is still close enough that the pathfinding check passes. The mob "cannot fight" so it should be nerfed, but per the given requirements, may still be allowed to pass.
Branch will be closed without merge.


UI: Added missing metrics English locale strings (for 4.18)

Description
This is the same PR as #8560 but targets 4.18.
This PR fixes two missing English locale strings that are missing in the metrics page in 4.18.1.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[x] Trivial

Screenshots (if appropriate):

Please use the original PR instead. Thanks!


Support 0 node agent pool and block manual scale CAS enabled pool

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

AKS


add site-editor and mail-masta to wp-plugins.fuzz.txt

as found in Vulnhub CTF: Symfonos 1

gypsydiver:wp-plugins-update

Thanks for the pull request @gypsydiver


View doesn't populate session context table_names

Describe the bug
Currently, running Q15 fails when attempting to build the second (SELECT) portion of the overall query.
The error is similar to the following:
called `Result::unwrap()` on an `Err` value: Plan("'datafusion.public.revenue0' not found")
thread 'tests::run_q15' panicked at 'called `Result::unwrap()` on an `Err` value: Plan("'datafusion.public.revenue0' not found")', /home/dalto/projects/arrow-datafusion/datafusion/sql/src/planner.rs:161:31

To Reproduce
Run test run_q15 in benchmarks/src/bin/tpch.rs
Expected behavior
I expect the test to fail due to either #3266 or #3267
Additional context
The table names just before failure are:
schema table names: [
    "customer",
    "supplier",
    "region",
    "part",
    "orders",
    "nation",
    "lineitem",
    "partsupp",
]

I'm unsure of where views populate this table, if anyone else has knowledge here that would be great.

@kmitchener @andygrove @alamb

This is happening because of the way tpch is running the test queries. q15 is made up of 3 SQL statements. run_query() attempts to create a plan for each of them before executing them. The second statement in q15 references the revenue0 view created in the first statement (which isn't created yet), so the planning of the second statement fails.
Probably run_query() should be modified to execute the statements in turn, rather than attempting to plan all of them before beginning execution.

That's exactly what I was thinking looking at it last night, and I was just writing a test to confirm by splitting the query. @kmitchener good to know I'm moving in the right direction. Assuming I get a different error by splitting the query, I'll try and rewrite the testing processing in tpch.rs and see if it fixes this problem.

Probably run_query() should be modified to execute the statements in turn, rather than attempting to plan all of them before beginning execution.

This :point_up: . AFAIK, the TPC-H queries are what they are, and we have to run them as such. This one is ensuring we support multiple queries in one command, so to be TPC-H compliant I think it has to "just work".


Invalid js breaks testing

Hello,
The following line results in my test suite failing using this addon, is it possible to resolve?
https://github.com/cogniteev/ember-cli-file-saver/blob/2bd6d5024da9b0e82e78c686f66777465e5737ba/vendor/file-saver.shim.js#L6
not ok 1 PhantomJS 2.1 - Global error: SyntaxError: Unexpected token '}' at http://localhost:7357/assets/vendor.js, line 75518
    ---
        Log: |
            { type: 'error',
              text: 'SyntaxError: Unexpected token \'}\' at http://localhost:7357/assets/vendor.js, line 75518\n' }
    ...```

👏


Add continuous integration with Travis CI

we need tests for REST but also for STREAM API

it shoud help about Python 3 support https://github.com/femtotrader/ig-markets-stream-api-python-library/issues/5

there is some tests for REST API (sometimes failling because of API error.public-api.exceeded-account-allowance exception)
Maybe we should consider using mock
There is NO tests for STREAM API
Feel free to help

Closing this, CI build is fixed and stable


Pull request (GitHub) promotion mechanism

Today, we support a git promotion, which will make a commit directly to a branch, followed by the Argo CD sync.
We need to support a promotion that goes through GitOps PR approvals. So instead of doing a direct git commit, the promotion would:

Open a pull request against the branch
Wait for that PR to be merged or rejected
Fail the promotion if the PR was closed without merging
Otherwise, once merged, perform Argo CD sync

To start, we will support GitHub, with support for other git providers later.

with support for other git providers later.

If we don't highly prioritize those other providers, this might be a really good area to enlist help from the community.

@alexmt has questioned what it means for a promotion to be "long-lived". The PR use case is my rationale for the need for long-lived promotion. IMO, I think it's justification enough because the act of waiting for the PR to be merged, and syncing the app, is part of the overall promotion process. Also, with a fire-and-forget-based approach, I don't think we can effectively achieve the semantics of: only one promotion process should be happening at a time.
I think I am willing to cede that testing and analysis is something that probably should not be part of Promotion job (in favor of a more continuous approach), but I think the PR create + wait for merge is strong justification for a long-lived promotion job.
@alexmt - since you had ideas of something more event-driven (fire and forget). I will put the onus on you to describe how you propose the UX to be for the PR use case.

@jessesuen assigned to you because I think you were already working on this. Correct me if I am wrong.


Update chart reloader to 1.0.103

This PR contains the following updates:

Package
Update
Change

reloader
patch
1.0.101 -> 1.0.103

Release Notes

stakater/Reloader (reloader)
v1.0.103
Compare Source

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


create-react-app fails at yard add

Is this a bug report?
(write your answer here)

Did you try recovering your dependencies?

(Write your answer here.)
Which terms did you search for in User Guide?

(Write your answer here if relevant.)
Environment

(paste the output of the command here)
Steps to Reproduce

(Write your steps here:)

Expected Behavior

(Write what you thought would happen.)
Actual Behavior

(Write what happened. Please add screenshots!)
Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

Please fill out the issue template. There's not enough information here for us to be able to help you.


Indexing tasks stuck when using the OpenAI auto-embedder

Describe the bug
Indexing tasks might get "stuck" in the "processing" state on indexes with auto-embedders like OpenAI.
To Reproduce
Unfortunately, the reproducer is not easy:

Add OpenAI embedder
Add documents to the index.

Expected behavior
The task finishes processing.
Meilisearch version:
Affects at least v1.10 and v1.11

Could be fixed by #5051


Missing emma_ant.properties referenced from build.xml

A tiny issue: build.xml references the missing emma_ant.properties around line 124 which prints the warning:
  [taskdef] Could not load definitions from resource emma_ant.properties. It could not be found.

I eliminated all references to emma—it's no longer developed.


Fix incorrect prop type in Dropzone.jsx

I think children should be of type 'node'?

 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.

Nice catch. This will be released as 0.3.1.


Clarification of gelly-stream features

Hi there.
I read in the presentation on gelly-stream (https://www.slideshare.net/vkalavri/gellystream-singlepass-graph-streaming-analytics-with-apache-flink) that it only supports single-pass graph computations, and moreover with only approximate results. Is that (still) correct?
I'm just getting started with Flink and did not find a way yet to get gelly-streaming running. Do you have step-by-step instructions somewhere that can help me get say ConnectedComponents running?

Hi @sdht0,
The README more or less reflects the current state of the repository. You can find the examples in the example folder and they should all have documentation. The Connected Components example for instance returns exact results.
To run the examples, you can either import the project to your IDE (I use IntelliJ) and run them as normal Java applications or build the project with Maven and use the generated jar.
I hope this helps! Let me know if you have further questions.

Hi again! After a haitus, I was able to try out gelly-streaming again and actually got the connected component examples to work (on vscode but not on IntelliJ). I had a few question I want to clarify:

Is my understanding correct: ConnectedComponentsExample merges the incoming edges into time batches and performs the entire computation for each batch, while IterativeConnectedComponents applies each edge as soon as it comes it to the single maintained output session. Is ConnectedComponentsExample reusing the results of the previous batches?
It looks like a bug in IterativeConnectedComponents, if I input 0 1\n1 2\n1 3 (using nc), it endlessly outputs 4> (3,0) until I input another edge.
How can I scale the execution of the example to multiple threads/workers or run it within a existing flink deployment?

Thanks!

Hi @vasia, pinging again in case you missed the above message.

Hi @sdht0,
Thank you for your patience. I've been busy with a deadline.

Your understanding about the connected components algorithms is correct. Both incrementally compute the connected components of an edge-only stream and "remember" the entire graph history.
That was a bug indeed. Thank you for reporting. I think I have pushed a fix. Let me know if it works for you.
Gelly-streaming does not have its own runtime. It is simply a library that you can run on top of Flink, i.e. you can define parallelism and run programs in parallel as you would normally do in Flink.

Thanks! I had a couple of more questions, but I'll try the new code again and get back to you.


In GCM 2.3 we dropped the symlinks.

In GCM 2.3 we dropped the symlinks.
Let's update our docs on the issue to reflect this.
Rendered: https://github.com/git-ecosystem/git-credential-manager/blob/rename-docs-update/docs/rename.md
Originally posted by @mjcheetham in https://github.com/git-ecosystem/git-credential-manager/pull/1363

test


Question about attention when mask is all zero

My project was inspired by this repo, thanks for your work!
However, there is a question that when mask is all zero, e.g., at the environment reset timestep, the energy is all -inf (-1e20) and the weights in attention are all equal after the softmax. How to deal with this problem? In other words, when there is no history info, how could we represent the history info?
Looking forward to your reply!

If the entire mask is zero then there is no history to attend to. This is only the case for the initial time step of the environment.
The information of the current observation is still propagated via the residual connections.


Unable to Create laravel project

can anyone help me with this?

since the yellow line ist stating an error about ssl
and
I just tested that the shown composer command worked fine
and
version 5.2.29 is definitely outdated
we should close this issue

since the yellow line ist stating an error about ssl
and
I just tested that the shown composer command worked fine
and
version 5.2.29 is definitely outdated
we should close this issue


test.sh: add resurrection, EXTRA_MOUNT, htmlcov

Have this reuse an existing container that's been stopped.
Also make it easy to mount more volumes into the test container.
Generate HTML coverage report.

Not updating release notes draft because this is a change solely to unit tests.


Update Backends in tutorial

Hello, I'd like to update the notebook working_with_backends.ipynb with the latest backends.
What's the first step for me to make this contribution?  Create my own branch?

Hi, thank you for your interest.
We'd be glad to have your contribution.
The first step is to create your own fork (by clicking the fork link on the upper right corner).
Then you can work on the master branch of the fork (or, create a new branch), and do the updates.
After finishing the updates, you can issue pull request to merge your changes.
Please let me know if you need any other information.

Thanks.  That's enough to get me started.
BTW, do you know if they ever plan an bringing ibmqx2 back online?  It's been down for quite a long time now.

OK I created the pull request and it says some conflicts need to be resolved.  They look pretty minor.  Is that something I can do?

Changes merged.


Feature request: Add QtCharts

Hi, I think it would be amazing to add QtCharts to the PyQt package of Conda-Forge. At the moment it is just available with PIP. The Package is called PyQtChart there.
https://doc.qt.io/qt-5/qtcharts-index.html

Thanks for the adding QTCharts and sorry for reopening such an old topic.
But I still can't find and install QtCharts. I'm using Miniconda3 (64 Bit - latest version) together with PyCharm on Windows. I've added conda-forge and I can install pyqt from it. But PyCharm is still complaining that it can't find PyQt5.QtChart?!

Same problem. I can install PyQt5 with conda BUT cannot use QtChart "ImportError: cannot import name 'QtChart' from 'PyQt5' ".

Why isn't QtChart included with conda with works when using pip ?

@stanislasgrare could you please raise a new issue and fill out the info requested in the issue template?


goimports-reviser lint is broken

Description
Arguments are ignored when used with ./...
https://github.com/incu6us/goimports-reviser/issues/93
make lint-imports will always happily exit 0.
Steps to reproduce the issue
No response
Describe the results you received and expected
Will send a fix shortly
What version of nerdctl are you using?
main
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
No response

Fixed by #3334


rsync progress field

Thought of one more thing. How does this code behave when the image doesn't have rsync, i.e. when it is using cp to do the copy and the progress file does not exist?

Thought of one more thing. How does this code behave when the image doesn't have rsync, i.e. when it is using cp to do the copy and the progress file does not exist?
(It looks like probably it will fail gracefully but even so I'd rather that it not add error messages to the KD log in that case.)

Well, I added the checking if rsync is present at the container calling rsync --version. If it is not present, the error will be not raised or logged


Global Interactions count is not working

The Feeds, Articles and all the other counts are not working on my local deployment. Can someone help me on this.

They are not implemented.
https://github.com/vitorfs/bootcamp/blob/master/bootcamp/templates/users/user_detail.html#L64

Can someone help me to that


Fix fix leading with multi level of brace pairs (#34169)

cherry-pick: #34169
fix leading with multi level of brace pairs
example:
leading(t1 {{t2 t3} {t4 t5}} t6) can be reduced to leading(t1 {t2 t3 {t4 t5}} t6) also update cases which remove project node from explain shape plan

run buildall

run buildall


Support to create a poll

Is it possible to create a poll with your SDK?

Hello there,
I have not been able to find documentation of an api allow to create polls.
I also find a conversation on the subject that seems to indicate that it will become available as part of the vNext of Twitter API (https://twittercommunity.com/t/polls/75456/5).
Though I have found that tweets will polls are now returned as part of the Twitter API.
I have created a ticket for this : https://github.com/linvi/tweetinvi/issues/1016.
Note that Tweetinvi v5.0 will NOT support Twitter API vNext. But it will provide a very simple way to generate requests that will allow you to use vNext.


AVRO-3779: [java] any big decimal conversion

What is the purpose of the change
AVRO-3779 propose a new feature for big decimal conversion. Current decimal conversion require to anticipate precision and scale, and so, to know in advance approximative value of it.
This conversions, added to existing, allow to deal with completly random big decimal value. Value is store in byte buffer, completly non human readable, but keep the value.
Verifying this change
This change added tests 'TestBigDecimalConversion' and can be verified as follows:
Run junit new test, or run all unit test of java module
Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? not documented

What I like: the scale is part of the value, giving greater flexibility when reading (schema resolution does not provide a widening conversion for logical types like decimal)
Personally, I would really like this to be documented in this PR.

thanks,
ok, i'll put it doc/content/en/docs/++version++/Specification/_index.md file

Doc added.

this PR does same for rust BigDecimal

So, we can merge this and create a PR for C# that will map this big-decimal type with C# AvroDecimal type ?

Java accept named schema to have logical type, but, specially with second Con, i agree with you.
I'll merge this PR and the rust one as there is no major opposition.

Hey guys! Do you have an ETA for the release of this change by any chance?

I don't know when the 1.12.0 is planned for release, i can put this in 1.11 branch for the future 1.11.4 release

That would be awesome @clesaec !

PR ready to be reviewed @rotilho-bv


Dataset version unnecessarily increasing database size

Currently to produce downloads for major, minor and latest versions - the relevant x.x.x version is copied. For larger datasets, this is quickly adding up in size in the repo.
A system of the pages for major, minor, latest being linked to the relevant version would reduce size.

Have decided removing the json download is the best option for this large dataset - otherwise the size cost isn't huge.


Moving *.cs file in external editor causes the editor to throw error

**Godot version: v3.2.stable.mono.official **
OS/device including version: Mac OS 10.14.6 / Godot Editor
Issue description:
If you move an existing *.cs file to a different directory in an external editor. Godot
throws "CSC(0,0): Source File "xxxx" could not be found.
Steps to reproduce:

Create a C# script in godot editor
Open script in an external editor like vscode
Create a directory in vscode
Move the script to the new dir inside vscode
Switch back to godot
Mono Build Project
You will see the error

Minimal reproduction project:
scratch.zip

That's a problem with the .csproj file. It keeps a reference to the old file
You can delete the old entry yourself in the mean time

Duplicate of https://github.com/godotengine/godot/issues/35252


Update configure-the-operational-data-feed-for-power-bi-with-biztalk-…

…server.md
Add new inputs

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/configure-the-operational-data-feed-for-power-bi-with-biztalk-server.md
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/configure-the-operational-data-feed-for-power-bi-with-biztalk-server.md
:white_check_mark:Succeeded
View

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


Athenz 1.12.x Release with upgrade to Jetty 12

Athenz 1.12.x will include the following changes:

Upgrade to Jetty 12.x / EE9 Release using Jakarta 5.x
Remove all deprecated methods from server side interfaces
Migrate all aws v1 usage from server side code to aws v2 since v1 sdk is EOL
Migrate Apache HttpClient 4.x to 5.x
Server builds are released w/ JDK 17 due to jetty requirement but all client libraries are continued to be built and published with JDK 11 support
Update jjwt to 0.12.x or find a suitable replacement for the same functionality
CI/CD pipeline will be moved from SD to GitHub Actions
Possible package renaming

cert refresher has com.oath which doesn't match with anything else so probably a good candidate for rename

maybe rename server side packages to io.athenz but keep the clients using com.yahoo for another release?

Move AWSPrivateKeyStore implementation from server-common to auth-core where it belongs with the correct package name
Remove single email notification support and only support consolidated email notifications (there is no point of spamming the admin with 20 separate emails where a single email can include all the roles that the admin needs to review)

The following deprecated methods have been removed from server side interfaces:
PrivateKeyStore

getPrivateKey (without algorithm, returning PrivateKey)
getApplicationSecret
getSecret (2 args, without keygroupname)

DynamoDBClientFetcher

getDynamoDBClient (without DynamoDBClientSettings argument)

CertSigner

generateX509Certificate (3 versions - without key id argument)
getCACertificate (2 versions - (without key id argument)

SSHSigner

generateCertificate (without key id argument)
getSignerCertificate (without key id argument)

NotificationServiceFactory

create (without PrivateKeyStore argument)

If you're using the default implementations provided as part of the Athenz release, then no changes are necessary since those are already updated accordingly. Otherwise, for any internal implementations of these interfaces, you must make the necessary changes before deployment.

Old Class
New Class

com.yahoo.athenz.auth.impl.aws.AwsPrivateKeyStoreFactory
com.yahoo.athenz.auth.impl.AwsS3PrivateKeyStoreFactory

com.yahoo.athenz.auth.impl.aws.AwsPrivateKeyStore
com.yahoo.athenz.auth.impl.AwsS3PrivateKeyStore

removed baseUrl argument from HttpDriver class in the athenz_server_common library since it was not used.

@havetisyan
The docker build also fails in our environment for the same reasons as the actions below.
https://github.com/AthenZ/athenz/actions/runs/11227301734/job/31209305932
Has any action been considered for this?

That's already fixed in the following PR: https://github.com/AthenZ/athenz/pull/2763 We just haven't merged it yet since the action only runs when there is an official release.

@havetisyan
I was wondering if these might also need to be addressed. Could you please take a look?
https://github.com/AthenZ/athenz/blob/a3583ee7131fb20ae6b19901b5a243e2ef32225d/docker/util/athenz-mvn-base/Dockerfile#L11
https://github.com/AthenZ/athenz/blob/a3583ee7131fb20ae6b19901b5a243e2ef32225d/docker/util/athenz-builder/Dockerfile#L1

https://github.com/AthenZ/athenz/blob/master/docs/migration-1.11-to-1.12.md


Building a NodeOS derivative

How would I build a NodeOS derivative? Does NodeOS have any such APIs for this? I would like to build an OS with a desktop GUI (most likely using something like Electron), and NodeOS seems perfect for this, but I can't seem to find any APIs for it that I could use.

There's no API like that, but instead several intechangeable layers, so you can modify and reuse them. A GUI using Electron would be a nice thing, problem is about how to make Electron to work without needing to use X11 or Wayland, just only KMS. Do you have any idea?

No, not really. I was hoping using X11 or Wayland (preferably X11) would be possible within NodeOS, but I'll try to look up some solutions for this.

All the results I can find require X11.

There is always OS.js and then there is tightVNC which does in fact leverage x11.

Yes, it could be possible to use them and use NodeOS remotely from a browser.


refactor(dynamo): clean up

Some last-minute cleanup of the source-code @mathisobadia, hope you don't mind. It's mostly simplified to match the style of the other adapters as well.

@balazsorban44 of course no problem, consistency is good! ;) I didn't take the time to check other adapters to see how they were made when I first opened the PR because it was not clear to me which ones were fully updated into the new api and which one were not but good that will all have the same quality standard now!


How to zoom?

Thanks for great work. I am trying to do something like crop part of svg and zoom to the full picture , I think this is one of natural svg operations.
As far as I understand I can implement crop with help of  clip_path with rectangle as clip figure, but how to zoom the picture that I get to the original drawing size?
Here is my play code that generates random points and trying to crop and zoom with no luck:
import drawSvg as draw
import random
import matplotlib.pyplot as plt
import matplotlib.image as mpimg
import math

GRID_SIZE = 32
PTS_TO_GEN = 33

coloursIdsToColNames = {0:'black', 1:'white', 2:'blue', 3:'red', 4:'green', 5:'gray', 6:'olive', 7:'purple', 8:'tan', 9:'tomato'}

def gen_img(imW, imH, cropSize):
    curDrawing1 = draw.Drawing(imW, imH)
    curDrawing2 = draw.Drawing(imW, imH)
    cclip = draw.ClipPath()
    cclip.append(draw.Rectangle(cropSize[0], cropSize[1], cropSize[2], cropSize[3]))

    for i in range(PTS_TO_GEN):
        curCol = random.randint(0, 9)
        curColName = coloursIdsToColNames[curCol]
        cx = random.randint(0, imW)
        cy = random.randint(0, imH)

        curDrawing1.append(draw.Rectangle(cx, cy, 1, 1, fill=curColName))
        curDrawing2.append(draw.Rectangle(cx, cy, 1, 1, fill=curColName, clip_path = cclip))
    return curDrawing1, curDrawing2

def gen_couple():
    random.seed(0)
    imW = 32
    imH =  32

    cropX = 0
    cropY = 0
    cropW = 16
    cropH = 16

    curDrawing1, curDrawing2  = gen_img(imW, imH, [cropX, cropY, cropW, cropH])

    # curDrawing2.setRenderSize(16)  - Not working
    curDrawing2.savePng('scaled.png')
    curDrawing2.saveSvg('scaled.svg')

gen_couple()

I seems like what you want is to set the SVG viewport to the 16x16 unit area of the image.  See this introduction to SVG viewports.
In drawSvg, you can set the viewport size and origin when creating a drawing:
draw.Drawing(width, height, origin=(x, y))

So for your example, use cropSize to define the viewport:
curDrawing1 = draw.Drawing(cropSize[2], cropSize[3],
                           origin=(cropSize[0], cropSize[1]))
curDrawing2 = draw.Drawing(cropSize[2], cropSize[3],
                           origin=(cropSize[0], cropSize[1]))

Also, drawing.setRenderSize(width, height) doesn't crop the drawing.  It just scales it so the output image is the given size in pixels.

Thank you!

Why the curDrawing2.setRenderSize(100, 400).savePng('orig2.png') produces empty  figure (curDrawing2 w = 1, h = 4) and the curDrawing2.setRenderSize(300, 300).savePng('orig2.png') produces not empy
  if this is a zoom

This is becuase setRenderSize does an aspect fit (it doesn't stretch the image if the width and height are not scaled by the same factor).  It scales the viewport to fit while perserving the aspect ratio and possibly showing extra on two sides.  If you only pass a single argument to setRenderSize it will select the other dimension to match.

For anyone like me coming across this while using v2.0.0+, setRenderSize is now set_render_size.


Context menu is rendering off screen if user right clicks near edges of the window

Issue: Like title says, if the user right+clicks near the bottom/top edge of the window some and/or all of the context menu will be rendered off screen. This is the same for is User has they're window adjusted to be narrow and they right+click the right side of the window, only half the context menu is rendered.
Screen recording:
https://user-images.githubusercontent.com/93608357/203457557-faeb916c-1b3d-4b66-b735-d2735c6e8169.mov
https://user-images.githubusercontent.com/93608357/203458056-5dfd677e-d18f-41e1-8cfd-da1608e9ea37.mov

https://github.com/Satellite-im/Uplink/pull/401


Update deprecation notice to point to SqlServerDsc

This change is

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

Comments from Reviewable


Chatterbot Issues with Gitter

I saw that people requested Gitter examples, and some were added. However, I haven't been able to get the bot to connect to Gitter, and when looking at the constructor, it looks so basic. No where in the examples do we pass an OAuth key or secret key. Has anyone been able to connect to Gitter using simply the API key? Here's my sample below attempting to connect to a room called 'meta-game'. (I blocked out my API key for obvious reasons.)
# Create a new ChatBot instance
bot = ChatBot('Ava',
    storage_adapter='chatterbot.storage.MongoDatabaseAdapter',
    logic_adapters=[
        {
            "import_path": "chatterbot.logic.BestMatch"
        }
    ],
    filters=[
        'chatterbot.filters.RepetitiveResponseFilter'
    ],
	# Gitter information
    input_adapter="chatterbot.input.Gitter",
    gitter_api_token="[my_api_key]",
    gitter_room="meta-game",
    gitter_only_respond_to_mentions=True,
    output_adapter="chatterbot.output.Gitter",

    database='ava_gitter',
    database_uri='<IP_ADDRESS>:27017',
    trainer='chatterbot.trainers.ListTrainer'
)

The above code fails to connect to Gitter with the failure code:
chatterbot.input.gitter.HTTPStatusException: u'404 status code recieved'
Has anyone been successful with Gitter? Am I missing something obvious?

I figured this out, and there seems to be a bug when joining a room. I know the room id I wanted to join, so I hardcoded the value in input.gitter.py. I can run this to ground, if anyone else has an issue with it, and they don't know python code well. (The issue seems to be that the call was a post, where only a get is supported to get room information.)

@sal48 Great resolving your issue, if possible could you make a PR to solve this issue?

Sure. After I finish profiling chatterbot to figure out my performance issues, I can make a general fix for the Gitter adapter.
I would like someone else to confirm that they're having issues. Basically, I tried to use the current chatterbot on master, and I was getting 404's when connecting.

Hi @sal48, I can confirm the issue that you encountered.
 31, in __init__
    room_data = self.join_room(self.gitter_room)
  File "/usr/local/lib/python2.7/dist-packages/chatterbot/input/gitter.py", line 58, in join_room
    self._validate_status_code(response)
  File "/usr/local/lib/python2.7/dist-packages/chatterbot/input/gitter.py", line 41, in _validate_status_code
    raise self.HTTPStatusException('{} status code recieved'.format(code))
chatterbot.input.gitter.HTTPStatusException: u'404 status code recieved'

Have you tried adding "/Lobby" to your chat room name?
example:
gitter_room="meta-game/Lobby",
I had the same issue and this resolved it.


Update dependency and Fix warnings

Signed-off-by: koushiro<EMAIL_ADDRESS>

Thanks!

@dpc, can you release a new version with this change ?

@dpc, can you release a new version with this change ?

@dpc  It would be nice of you to release a new version (v2.4.1) so that downstream users can get rid of crossbeam-channel 0.3 in their crate graph.

@koushiro Done.


Verlet.aplo should positions fold before or after step?

     pos←periodic|pos ⍝ Fold positions if periodic boundary conditions set
     pos+←dt×vel+0.5×acc×dt ⍝ Step position

After


[RELENG-4539] ADd webhook management

Codecov Report

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

:exclamation: Current head 7729ca7 differs from pull request most recent head 37d6157. Consider uploading reports for the commit 37d6157 to get more accurate results

@@           Coverage Diff           @@
##             main      #52   +/-   ##
=======================================
  Coverage        ?   50.91%
=======================================
  Files           ?       26
  Lines           ?      766
  Branches        ?        0
=======================================
  Hits            ?      390
  Misses          ?      376
  Partials        ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0c65185...37d6157. Read the comment docs.


Bug with datasources url set to env("DATABASE_URL")

With a prisma/schema.prisma file present with a datasource that uses env() from prisma, the generator fails to generate a prisma.rs:
datasource db {
  provider = "postgresql"
  url           = env("DATABASE_URL")
}

Error:
thread 'main' panicked at 'datasources[0].url.value: invalid type: null, expected a string at line 1 column 1136', $HOME/.cargo/git/checkouts/prisma-client-rust-fa967aa5ad0ec391/97d469e/cli/src/lib.rs:109:25

This is on the latest main branch.
The error occurs when serde_json tries to deserialize the input from the schema.prisma file.

Looks like I didn't have the proper types for env values, should be fixed by cb78ad08b653ed74d427d5e923f82771813a1a58.


refactor svg2js to use txml

Hi,
I refactored the svg2js file. The goal was to use txml instead of the sax parser. The reason for that is, that txml is pure js. There will be no issues about compiling any native modules. It is also much faster than the native module.
Please see: The only files changed are package.json, package-lock.json and the svg2js.js.
What do you think of this? Do you think this can help people who are using SVGO, running it on various platform (local + docker) and it can help people who do not have a c compiler installed.
DISCLAIMER: I am the author of txml

Using txml as XML parser fixes a ton of open svgo issues related to XML parsing.
It also removes the necessity of bindings to the sax parser.

@GreLI: It would be great if this can be reviewed and merged!

Nice to see there is some new work going on here with svgo. I see the svg parser is now a sync function and I think this is good.
I would refactor this PR to also work synchronous. That would actually be the default behavior of txml.
@TrySound  but first I would like to hear your opinion.

Changing parser is a big shift. I will try to fix existing issues first and then come back to this change.
Do you have a plan on maintaining txml? Any popular project use it already?
Through2 look not necessary. Can be replaced with const { PassThrough } = require('through2');

The reason for that is, that txml is pure js. There will be no issues about compiling any native modules. It is also much faster than the native module.

AFAIK rollup will still hoist lazy requires to convert into es modules. You better split stream based extension into separate entry point.

hi, I made an update to txml, to work better with rollup. having two entry points as you suggested, that is pretty good. (now on github only, will publish that update soon to npm.) The parser is now 17kb(uncompressed and still include all comments and formatting.)
I tried to run the refactoring I made in this PR, with the new version. All your new updates and updated tests. Local I have it so far, as that all tests run through, but many produce a somewhat different result (not good).
Are you still interested to consider using txml for svgo? Then I would see if I can pass all tests again within the new version. @TrySound  @strarsis

just a little update, on my current work, I have the current and my refactoring in theory produce the same output just compared via JSON.stringify, but there are still errors. Next I will analyze any other differences.

Wow, it works. all tests passed today.
now I only need to publish the new version of txml, then the regression tests will also pass here.
@strarsis @TrySound how do you like it?

Hey hey! I'll look at this in future, but for now, I've been prioritizing bug fixes and optimizations in existing plugins, similarly to TrySound when he was reviewing this.
I've also been putting off touching dependencies in general:

I don't want to negatively impact the bundle size without more research. txml (249kB) is substantially larger than sax (53.7kB)
This project has numerous maintainers, all of which probably have more experience/context than me regarding SVGO. It'd be wrong for me to take action on my own anyway.
For a project with this much usage, the decision to add/change dependencies is more sensitive in general, imo.

I will come around to this soon though, I wanted to look at our XML library anyway since our "maintained fork" of sax is about as maintained as sax. We may want to consider upstreaming our changes and moving back at least, as the maintainer of sax is alive and even pushed an update lately. ^-^'
Once I've gotten the repository to a more manageable and stable state, it'll be a lot easier to pester all the maintainers to spare some time to discuss this publicly in an RFC (GitHub issue). I'll ping you in it too ofc.


downloadUrlFor and srcSetFor for FileBone with languages are not handled correctly

Afaik we do not handle FileBones with multiple languages not correctly right now. I would prepare a PR draft based on this issue.

Fixed by #1266


Give the opportunity to add a communication message in the home page of onyxia

Maybe we could add the opportunity to add in the region configuration a string message that could be display on the home page of onyxia with the possibility for the user to delete the message.
Like gitlab message feature.
It could be helpful to give some maintenance information, or news.

You had this functionnality before. This was provided by a simple env in onyxia-web. I guess we should do the same.
Is that easy @garronej

Shipped in 2.19.0
See documentation.

[x]: Renders the message as a markdown text, it enables to feature links and to format the text.
[x]: Enable, to specify the severity of the message, default is info.
[x]: i18n: The message is localizable.
[x]: a11y: Screen reader read the message with the correct syntetization voice.
[x]: Persistance: When the message is closed once it dosen't show on next reload, it's reset when the severity or the message is updated by the administrator.

https://github.com/InseeFrLab/onyxia-web/assets/6702424/9067a95a-e5d9-4d35-b4a9-29cf9ce7dc60


pipewire: 0.3.43 -> 0.3.45

Motivation for this change
https://gitlab.freedesktop.org/pipewire/pipewire/-/releases/0.3.44
https://gitlab.freedesktop.org/pipewire/pipewire/-/releases/0.3.45
It is now possible to use pipewire without a session manager (to some extent).
Audible bell from X11 terminals should work with the new X11 module. (seems to be commented out in the default config though)
Lots of bluetooth and pulse-server fixes.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg test installed-tests.pipewire

Pipewire 0.3.45 was just released (in the upstream next branch).

0.3.45 officially out now.

Still complains about the reference cycle, still haven't found where it is.

Probably libcanberra?

Well the error specifically claims the cycle is between the jack and lib outputs but
grep -r $lib $jack
grep -r $jack $lib

only mentions that $jack/lib/libjack.so and $jack/lib/libjackserver.so depend on $lib/lib/libpipewire.so but that's only one direction, no mentions of anything the other way around.
I'm not sure why libcanberra would be related? It is only used in the x11 module and has no explicit dependency on pipewire.

After some staring into nix build -vvvvvvvv(etc), I think I found it. The dependency cycle isn't actually lib -> jack, it's lib -> out -> jack, where lib pulls in out because share/pipewire/minimal.conf references binaries in out, and out pulls in jack because pw-jack adds it to LD_LIBRARY_PATH. I'm guessing the easiest way might be to just move the configs into out instead of lib, because they're only used by the daemon anyway.

This seems to be enough to untangle everything: https://github.com/K900/nixpkgs/commit/4c4aa837eefadffb96ab6133cfe2d3bbdc2b4ba6

Ohhhh it's that thing again. We even have a patch for removing a similar reference from the default config. In both instances the reference doesn't actually do anything since it is commented out.

Oh yeah, I see the patch now. I guess it's a better idea to update that then.

Updated the patch, not sure if it is the most elegant workaround or if something with substituteInPlace could work better.

I think I like the patch more, substituteInPlace can accidentally substitute a useful reference.

Uhh why does ofborg try to run passthru tests on darwin when the derivation has platforms = linux

All peachy on my system now.

Same.


Brownout detector was triggered

Hi I'm using an AI-Thinker board and was able to flash your code.
I activated
https://github.com/arkhipenko/esp32-cam-mjpeg-multiclient/blob/5a8610c7d9f9db71554daf0744c66988f81126ee/esp32_camera_mjpeg_multiclient.ino#L42
But somehow I get a Brownout detector was triggered reboot-loop at OV2640.cpp l184:
https://github.com/arkhipenko/esp32-cam-mjpeg-multiclient/blob/5a8610c7d9f9db71554daf0744c66988f81126ee/src/OV2640.cpp#L184
It seems that the cam can't be initialized with the given config for AI-Thinker.
(I also tried the other configs and they also don't work)
Here the board for reference:
https://www.amazon.de/gp/product/B08MZV1TT9/ref=ppx_yo_dt_b_search_asin_title?ie=UTF8&psc=1

Nevermind, It seems that 3.3V wasn't enough for the board.
After I switched to 5V, the issue solved itself :+1:
Cheers


[11.x] Handle listener after transaction when it implements ShouldHandleEventsAfterCommit

I came across an issue with a queued listener that should be handled after transaction. It just dispatches the listener regardless whether it implements the interface or no.
Look at bug report to understand the issue better: https://github.com/laravel/framework/issues/52440

Upd: I feel like that might not be a bug but rather a mistake in the docs and it should use ShouldQueueAfterCommit instead. If so, ShouldHandleEventsAfterCommit is supposed to be added to listeners that don't implement ShouldQueue since it won't simply work. What's the use case for ShouldHandleEventsAfterCommit then?

On first glance it does look like a documentation error, and that ShouldHandleEventsAfterCommit is only for listeners that are not queued at all.

Updated docs here: https://github.com/laravel/docs/commit/fe1d4567fe5402750c54d3f6277f9b84c8b441a1
Please open another PR if you still see a problem 👍


Inconsistent property SendYsod in EmailOptions

Hi,
I happened to find while browsing, the SendYsod in EmailOptions is assigned to NoYsod internally:
ElmahCore\ElmahCore.Mvc\Notifiers\ErrorMailNotifier.cs:
_noYsod = options.SendYsod;
protected bool NoYsod
{
get { return _noYsod; }
}
I guess the meaning of the property is just reversed.
Best Regards
Wicky

Closed in 1.2.1


neatvnc: 0.5.1 -> 0.5.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/any1/neatvnc/releases.
meta.description for neatvnc is: "A VNC server library"
meta.homepage for neatvnc is: "https://github.com/any1/neatvnc"
meta.changelog for neatvnc is: "https://github.com/any1/neatvnc/releases/tag/v0.5.3"
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.5.3 with grep in /nix/store/2s5m75x2gbicxpnhwcljfb8vkp0c4gx1-neatvnc-0.5.3
found 0.5.3 in filename of file in /nix/store/2s5m75x2gbicxpnhwcljfb8vkp0c4gx1-neatvnc-0.5.3

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
neatvnc
wayvnc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2s5m75x2gbicxpnhwcljfb8vkp0c4gx1-neatvnc-0.5.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

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

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

2 packages built:

neatvnc
wayvnc

Maintainer pings
cc @NickCao for testing.

Result of nixpkgs-review pr 188878 run on x86_64-linux 1

2 packages built:

neatvnc
wayvnc


Add button to open website like wikipedia field

The wikipedia field has a button which opens the page listed in the text box - would be nice if it was added to the website field as well.

See #3075

I don't think it's a good idea to link to untrusted resources. A malicious mapper could put anything in the website= tag, and trick unsuspecting users into visiting that link.

Or create a pop-up to tell that this link may not be safe with "follow the link" or "cancel"

Or create a pop-up to tell that this link may not be safe with "follow the link" or "cancel"

I don't think that a popup is helpful or friendly.  I'd prefer for iD to be a safe place all the time for people who are maybe not very computer savvy.
Users who really want to visit that link can copy-paste it to a new browser tab and decide if it is worth visiting.  (Consider, the website tag value might even contain something like javascript:alert('hi'); or whatever terrible thing you can imagine).

The reason I suggested it was to avoid the need to copy and paste when you want to have a look at a website when editing. I saw the functionality of the wikipedia link and though that would be nice to have.
I was going to say that iD already gives a red outline for an error when the website is no valid, ie there is no http(s)://. However I noticed that this is done by Firefox due to the type=url input field.
Perhaps iD could do a similar thing and only display the button when the website is a valid url? It would also display it in the bottom left of the browser.
I do understand the concerns, though according to taginfo, none of the current website values begin with javascript.


F# Error using Jil library

I'm using the Jil package and getting a weird error.
Repro steps

Add this dependency on project.json

"Jil":"2.14.5"

Push the changes and see the following log message:
commandLineArgs(1,1): error FS215: Multiple references to 'mscorlib.dll' are not permitted

Related information
I've tested the library on F# Scripts locally and works well, no errors:
#I "../packages"
#r "Jil.2.14.5/lib/net45/Jil.dll"
#r "Sigil.4.6.1/lib/net45/Sigil.dll"
open Jil
type Person = {
    Name : string
}
let n = {Name = "Nicolas"}
Jil.JSON.Serialize(n)

Why is this library causing this?

Hipster xD

Package references are automatically added to your function, so you should not have to specify explicit references to them.


PDF Download produces only one page

Describe the bug
I tried to download PDF for Synchapi, but downloaded PDF contains only "Overview" page
To Reproduce
Steps to reproduce the behavior:

Go to https://docs.microsoft.com/en-us/windows/win32/api/synchapi/
Click on Download PDF
PDF contains only "Overview" page

Expected behavior
PDF contains all pages from the list

@mcleanbyron I can't find this Win32 page at the moment. The original issue was PDF build is incomplete.

@mcleanbyron I can't find this Win32 page at the moment. The original issue was PDF build is incomplete.

Unfortunately, the behavior you're seeing is expected due to the unique way that the Win32 API reference docs are built within the docs.microsoft.com platform and the way the generate PDF feature is currently implemented. However, you can generate a PDF that includes the synchapi.h API articles by creating a PDF from the System Services technology page that this header is associated with. Note that the resulting PDF also includes all of the APIs for other headers related to this technology area.

Unfortunately, the behavior you're seeing is expected due to the unique way that the Win32 API reference docs are built within the docs.microsoft.com platform and the way the generate PDF feature is currently implemented. However, you can generate a PDF that includes the synchapi.h API articles by creating a PDF from the System Services technology page that this header is associated with. Note that the resulting PDF also includes all of the APIs for other headers related to this technology area.

#please-close

#please-close


wip Gm 366 app header dropdown

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.14 out of 15 committers have signed the CLA.:white_check_mark: timjenkins:white_check_mark: codecademy-gh-api:white_check_mark: dreamwasp:white_check_mark: codecaaron:white_check_mark: aresnik11:white_check_mark: borisonr:white_check_mark: yaronshahverdi:white_check_mark: aionate0812:white_check_mark: jcenglish:white_check_mark: ahmed-abdallah:white_check_mark: JahazielGuzman:white_check_mark: natalieweesh:white_check_mark: casey-speer:white_check_mark: thedoomshine:x: Josh GoldbergJosh Goldberg seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Task 7 - creating splash screen

You should do a rebase on master branch since there are a lot of changes that you probably didn't do by yourself. After rebase please message me on slack so I can continue doing the review


Ardaas incorrect English translation

Describe the bug
Please see line below. In the English translation it says Guru Arjan when it should be Guru Angad Dev Ji
ਫਿਰ ਅੰਗਦ ਗੁਰ ਤੇ ਅਮਰਦਾਸੁ ਰਾਮਦਾਸੈ ਹੋਈਂ ਸਹਾਇ ॥
phir añgad gur te amaradās rāmadāsē hoīñ sahāi ‖
Then I remember Guru Arjan, Guru Amar Das and Guru Ram Das, may they be helpful to me.
Environment
Web browser https://www.shabados.com/app/ardas/

Hello, thank you for opening an issue to track this. Indeed the translations seem to be off compared to the following sources:

https://old.sgpc.net/CDN/Sikh_Rehat_Maryada_English.pdf
https://sgpc.net/ardas/ or https://old.sgpc.net/ardas/ardas_en1.asp

The closest source to what we currently have matches to this one: https://nitnemsahib.com/ardas-english-translation/, where it correctly says "Then I remember Guru Angad, Guru Amar Das and Guru Ram Das, may they be helpful to me." I cannot find the actual publication for this translation.
Regardless, something will have to be done, since the translation is obviously incorrect.


Bad quality image on french N64 system

Hello, Thank you for this wonderful project. I built such an interface but after connecting to the TV I find that the image is of poor quality. The image is sharp but resolution is really low (the text is almost illegible) and the image shakes sometimes. What could it be caused by? (it's a NUS-001(EUR) N64 system)

thanks for what you do, it's very nice, I wanted to understand why I too have the same problem, the image is a little better than the composite signal, but it is of low quality, on a 22 inch LCD it looks very grainy, if anyone has had the same problem it would be nice if you could share the solution, thanks again for all the projects you make available.

The video output is only 640x480. Some TV's look terrible when viewing such a low resolution, Samsung in particular. Do you get the same problem when viewing on a computer monitor?

The image quality you are seeing is expected because of how this works.
PicoDVI-N64 works with a 320x240 internal framebuffer with rgb555 color depth.
The N64 can output up to 640x480 with rgb777.
The video signal is sampled every second pixel horizontally, and every second row is skipped. It's nearest-neighbor downscaling with other words.
If you see banding, this is because the color depth is rgb555. The N64 has rgb777. This is simply how it is implemented.


WIP

this is kinda somewhat working, I didn't get to clean it up yet, I hope I get to it next Friday. But if anyone wants to go ahead and clean things up so that we can start integrating it with hoodie-server, please go head :)

@HipsterBrown Hey Nick, just in case you are looking into working on something, feel free to pick up where I left in the wip branch. You can pretty much kick my front-end code, instead use the scoped store API that is now available, which should make things much simpler.

I've added tests and updated the client code to use the new scoped Store api. Having trouble fully testing the task.start method because it is tightly integrated with a response from the server. I'm also unsure of how to continue with the other methods, abort & restart.

Looking good so far :+1:

Removed the module globals and window variables from the client code. I also fixed some of the standard and error throwing formatting issues. Anything else we want from this branch, @gr2m @Charlotteis?

Removed the module globals and window variables from the client code. I also fixed some of the standard and error throwing formatting issues. Anything else we want from this branch, @gr2m @Charlotteis?

Just added one nit-picky comment, but everything looks good to me as far as what I was concerned with :) Good work

Fixed! Thanks for being "nit-picky" @Charlotteis. It helps make Hoodie the best it can be. :+1:

@HipsterBrown :heart_eyes:


Too many active player instances causing codec initialization errors.

Current codebase leaks brightcove player views. Not ready to merge, first we need to discuss how comfortable we are with the RN behavior not re-using player views in the first place.


[log-source-app] Compilation errors

Problem
log-source-app does not build.
fix #14923
Change overview

Fix the build issues.

/rebase


label weavedb image

Fixes weaveworks/scope#2610

I think I would have added the owner/source/description stuff at the same time, but this is OK for now.


Add file search ablility for "Explorer" pane

If an opend project have many files I want to search a file in "Explorer" pane directly. Could you add this ablility to be next version release.

Not perfect yet, but it's better than nothing:
https://github.com/microsoft/vscode/issues/10026
...or maybe you can use ctrl + p ?

Use the search panel, Ctrl + p, or install an extension that does it.
Closing as designed.


As/torch

Improvements to torch.py, including parallelization, refactoring, update existing solvers instead of recreating them. Also added devcontainer, set the version of scipy to <1.12, and update build files.

Great to see that the CI is working on this PR!
I have a couple of suggestions, mostly having to do with my taking a shortcut on this before, something that might be handled better with this PR (and going forward):

My reason for not introducing torch as a package dependency was that its a heavy and complex dependency, probably applicable to a small majority of osqp users. Users of the nn submodule may as well see an ImportError on torch and know what they're supposed to do. Now that we have joblib joining that list as well. it may be better something like a [nn] extra so that pip install osqp[nn] installs the dependencies required for the nn functionality.
Alternatively, since joblib is a small and generally useful library, it can be introduced as a direct dependency, and torch left out as before.
In either case, I think moving away from installing stuff in CIBW_TEST_REQUIRES will keep things manageable going forward. (and the CI can simply do a install osqp[dev,nn,..] etc.

I also don't recommend including .devcontainer in this PR, since that might trigger an IDE-specific proliferation of code, but that's just my personal opinion.


Update installation instructions for Cordova version 7

After updating to Cordova version 7 I saw the following trying to install the plugin:
$ cordova plugin add https://github.com/darryncampbell/darryncampbell-cordova-plugin-intent.git
Error: Failed to fetch plugin https://github.com/darryncampbell/darryncampbell-cordova-plugin-intent.git via registry.
Probably this is either a connection problem, or plugin spec is incorrect.
Check your connection and plugin name/version/URL.
Error: npm: Command failed with exit code 1 Error output:
npm ERR! exited with error code: 128

npm ERR!      /Users/.../.npm/_logs/2017-11-13T16_32_33_875Z-debug.log

It seems that for<EMAIL_ADDRESS>you can simply install using the package ID instead of the git URL.
$ cordova plugin add com-darryncampbell-cordova-plugin-intent
So I updated the README accordingly.

Thanks, merged 👍


deprecated call async_setup_platforms

Hi,
The current version of kuma HACS integration (v2.0.2) print an error into the home-assistant console.

WARNING (MainThread) [homeassistant.helpers.frame] Detected integration that called async_setup_platforms instead of awaiting async_forward_entry_setups; this will fail in version 2023.3. Please report issue to the custom integration author for uptime_kuma using this method at custom_components/uptime_kuma/init.py, line 52: hass.config_entries.async_setup_platforms(entry, PLATFORMS)

It's still working well, but to avoid a futur issue, a change will be necessary.
Azlux

Cool, this is, however, not an integration, but an add-on.
I think you are mixing things up. Closing this issue as it is for sure not related to this add-on.
../Frenck

Oups, srry it's indeed not the good project.


XAML Parse Exception

Describe the bug
App is throwing XAML Parse Exception with stack trace:
   at WinRT.ExceptionHelpers.ThrowExceptionForHR(Int32 hr)    at ABI.Microsoft.UI.Xaml.IApplicationStatics.global::Microsoft.UI.Xaml.IApplicationStatics.LoadComponent(Object component, Uri resourceLocator, ComponentResourceLocation componentResourceLocation)    at Microsoft.UI.Xaml.Application.LoadComponent(Object component, Uri resourceLocator, ComponentResourceLocation componentResourceLocation)    at CloudCash.Client.Controls.BusyOverlay.InitializeComponent() in C:\Users\JR\source\repos\JanRajnoha\CloudCash\src\CloudCash.Client\obj\x86\Debug\net5.0-windows10.0.19041.0\Controls\BusyOverlay.g.i.cs:line 37    at CloudCash.Client.Controls.BusyOverlay.DoInitialize(Int32 attempt) in C:\Users\JR\source\repos\JanRajnoha\CloudCash\src\CloudCash.Client\Controls\BusyOverlay.xaml.cs:line 43
and message:
Failed to assign to property 'Microsoft.UI.Xaml.Media.AcrylicBrush.TintColor'. [Line: 18 Position: 27]
Code:

Expected behavior
No exception
Version Info
Windows App SDK - 0.8.2
CommunityToolkit.WinUI.Controls - 7.0.3
WinUI app packaged

Windows version
Saw the problem?

Insider Build (Windows 11 - dev)
Yes

May 2021 Update (19043)

October 2020 Update (19042)

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

similar error to this: https://github.com/microsoft/microsoft-ui-xaml/issues/5814 however the circumstances aren't the same at all.  This is using a winui3 nuget packages that is using the winui2 framework package.. might or might not be related.


Update browserify-with-globs.md

when browserify throw this error
events.js:141
throw er; // Unhandled 'error' event
^
Error: path must be a string
at /.../node_modules/browserify/node_modules/resolve/lib/async.js:16:16
at doNTCallback0 (node.js:428:9)
at process._tickCallback (node.js:357:13)
we need some debugging information : added on('error'...) on the bundle

Recipes are supposed to be simplified to get you started and don't include common error handling logic.


Fixed bug with Profaned Soul Artifact item.

Profaned Soul Artifact has a similar bug to Heart of the Elements. Its buff ID is 592.
Before fix:
https://user-images.githubusercontent.com/7689214/182370329-226c9505-aa9d-43ab-9f53-49db9095538d.mp4
After fix:
https://user-images.githubusercontent.com/7689214/182370465-2ea13957-8e33-46c6-88dd-b332be3560c2.mp4
Also bumped version number.
Tested Mutated Truffle aswell, an accessory that summons a minion similar to Heart of the Elements and Profaned Soul Artifact, however, it was unaffected by this bug.

Thank you for the fix. Merged and published a new version.


In debug logging, print 'fixed' values as floats

I got a bit confused by the following:
2024-04-17T19:51:21.930954Z DEBUG session{session_id=365288}: mmserver::compositor: wheel delta: 0.0 -48.0
[ 768472.301][rs] -> wl_pointer@15.axis_source(2)
[ 768472.333][rs] -> wl_pointer@15.axis(13025, 0, -12288)

This seems to match wayland-backend/client_system, except libwayland prints more than 4 digits.
Truncating to 4 digits makes sense given it doesn't have that much precision anyway.
I guess this should have a CHANGELOG entry.

Merged with a changelog entry as part of https://github.com/Smithay/wayland-rs/pull/732. Thanks!


Vim9: crash when getting key absent from dictionary

Describe the bug
In Vim9 script, trying to get a key absent from a dictionary might make Vim crash.
To Reproduce
Run this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    vim9script
    def Func()
        eval {-> ''}() .. '' .. {}['X']
    enddef
    sil! Func()
EOF
)

Vim crashes.
Expected behavior
Vim doesn't crash.
Environment

Vim version: 8.2 Included patches: 1-2081
OS: Ubuntu 16.04.7 LTS
Terminal: xterm(362)

Additional context

backtrace
valgrind log
asan log

It's a regression caused by some commit between 8.2.1500 and 8.2.2000.  I'll try to bisect and find which one exactly.

The latest commit (8.2.2082) causes another crash, but I'm not sure I can easily reproduce it until this one is fixed.  I'll try though.

It's a regression caused by some commit between 8.2.1500 and 8.2.2000.  I'll try to bisect and find which one exactly.

It's caused by 8.2.1928.

Here is another similar snippet which still makes Vim crash on 8.2.2083:
vim9script

var d: dict<any>
d = {'': 0}

def Func()
    try
        eval '' .. map([''], {_, v -> {}[v]})->join() .. d['']
    catch
    endtry
enddef

sil! Func()

Here is another similar snippet which still makes Vim crash on 8.2.2083:

This one is caused by a regression introduced in 8.2.2082.  Here is a backtrace.

Here is a backtrace.

More logs:

valgrind log
asan log

Actually, there are at least 2 other crashes.  I'll open a new report with snippets to reproduce and logs for each of them.


Role leads to doubled inclusion of the sudoers_dropin_dir

On newer sudo versions the default inclusion statement is @includedir instead of #includedir. After running the role the directory gets included a second time:
@includedir /etc/sudoers.d
#includedir /etc/sudoers.d

This leads to annoying warnings in the terminal when aliases get defined:
/etc/sudoers.d/commands:1:382: Alias "SOMETHING_COMMANDS" already defined

Reference:

It is possible to include other sudoers files from within the sudoers file currently being parsed using the @include and @includedir directives. For compatibility with sudo versions prior to 1.9.1, #include and #includedir are also accepted.

https://www.sudo.ws/docs/man/sudoers.man/#Including_other_files_from_within_sudoers

I incorporated ur suggestion into v1.1.0 that I just published to the galaxy. Can take a peek and lmk if it works for u?

Doesn't seem to work. :(
https://github.com/andrewrothstein/ansible-sudoers/blob/b8524d5c71e5cb36d3db7c6f27f1293987927806/tasks/main.yml#L39-L53
This still adds the #include line if @include is already defined. It now also adds the @include line, even if the #include line is already present. The include should only happen once.
I think a good way to go about that would be:

check if the #include line is present, don't add the line if it's not present
register if the #include line is present
check if the @include line is present, only if the #include line is missing (check via register var). add the @include line if it's missing.

But I'm not sure how to do step 1. lineinfile doesn't allow for checking if a line is present without changing the file?


Update is_categorical_dtype for pandas>=2.1.0

From https://pandas.pydata.org/docs/whatsnew/v2.1.0.html#other-deprecations:

Deprecated is_categorical_dtype(), use isinstance(obj.dtype, pd.CategoricalDtype) instead (GH 52527)

Currently, formulae will surface this warning for pandas>=2.1.0:
.../formulae/terms/call.py:108: FutureWarning: is_categorical_dtype is deprecated and will be removed in a future version. Use isinstance(dtype, CategoricalDtype) instead
  elif is_string_dtype(x) or is_categorical_dtype(x) or isinstance(x, CategoricalBox):

(As of pandas==2.1.4 this is a DeprecationWarning instead of a FutureWarning.)
The change in this PR should be backwards-compatible.
How has this been tested?
In a virtual env w/ pandas==1.3.4, I ran pytest. I also checked that this snippet of code is correct.
import pandas as pd
from formulae.utils import is_categorical_dtype

data = [1, 2, 3, 1, 2, 3]

data_cat = pd.Categorical(data)
assert is_categorical_dtype(data_cat)

data_num = pd.Series(data)
assert not is_categorical_dtype(data_num)

In a virtual env w/ pandas==2.1.4, I ran pytest. I also checked that the snippet of code above is still correct and doesn't raises a warning.

Hi @kddubey! Thanks a lot for the PR. I'm writing to let you know that I see the contribution, but I still didn't have time to check the underlying issue very well. And I'm going to be travelling so I won't be able to check it for the next 2 weeks. Still, the code looks good and tests pass so that's great.

@tomicapretto No worries. And thank you to you and the PyMC team for building bambi! It's a pleasure to explore and use. Happy travels :-)

@kddubey thanks a lot for the contribution :)


Flake8 rule 'E901' reported an invalid point

Flake8 reported an invalid point for the rule E901, with the message:
Original Message: E901 - IndentationError: expected an indented block
Requested Point (NaN: NaN)

# Filename: aws.py
from boto.s3.bucket import Bucket
from boto.s3.connection import S3Connection

class AWSHelper(object):
  def __init__(self, test_case):
    pass

Debug information:
Atom version: 1.15.0
Flake8 version: 2.5.4 (pep8: 1.7.0, naming: 0.3.3, mccabe: 0.4.0, pyflakes: 1.0.0, import-order: 0.8, flake8-docstrings: 0.2.6, pep257: 0.7.0) CPython 2.7.10 on Darwin

Please update your flake8 version (pip install -U flake8). This particular error should be fixed in flake8>=3.2.1.

Closing due to a lack of response, if you are still having issues after updating flake8 please provide code to reproduce this!


Compiler warnings are incorrectly documented, some do not work and some do not exist

Looking at the documentation of compiler flags, specifically those, that are OFF by default, there are many problems. Let's break it down:
Some flags have a wrong default value in the wiki
Some flags are not OFF, so their default value in the wiki should either be WARNING or ERROR (not OFF). These flags are:

visibility
accessControls
const
constantProperty
missingRequire - Maybe this flag should be documented a little more in detail. What module system does it apply to? For example missing import/export statements in ESM modules are type checked by default

Some flags are not recognised by the compiler

undefinedNames

Some flags simply do not work

unusedPrivateMembers

class Car {
  constructor() {
    /**
     * @private
     */
    this.color = 'red'; // should raise error, but doesn't
  }
}

strictPrimitiveOperators

function go() {
  return 555 + 'hello' + {} + new Element() // should raise error, but desn't
}

Sometimes it is very hard to understand what a flag does

missingProperties

Warnings about whether a property will ever be defined on an object. Part of type-checking.

What does that mean?

strictModuleDepCheck

Warnings about all references potentially violating module dependencies.

What does that mean? What is a violation of module dependencies?

typeInvalidation

Warn about properties that cannot be disambiguated when using type based optimizations

What does that mean?
Some flags need more documentation

strictMissingProperties

Warnings for missing properties that forbid accessing subclass props off superclasses

In reality though, this flag does so much more. Only the authors know all the details. From what I have been able to reverse engineer, this flag completely changes the behavior of union types, the all type, Element type, etc. I believe this would be worth mentioning in order to facilitate an easier understanding of all that this flag actually does.
Some flags are wrongly documented

reportUnknownTypes

Warnings for any place in the code where type is inferred to ?. NOT RECOMMENDED!

This documentation is not entirely true, if I declare a variable of unknown (?) type - so it is declared, as opposed to inferred - the compiler also raises error. So the behavior I see is that it does not apply only to inferred types but also to declared types.
/**
 * @param {?} z // raises error
 */
function hello(z) {
  console.log('hello', z);
}

This is either a mistake in documentation or a bug in the compiler - that depends on what was the intended behavior.
There is also an interesting question: Why does the documentation say it is "NOT RECOMMENDED!"?  This would deserve a little more in-depth explanation. It is not immediately obvious why this flag is not recommended. It seems like a useful flag.
Environment
Compiler Version: v20221102
Build command:
java -jar ./scripts/closureCompiler.jar \
  --entry_point=./src/js/index.js \
  --js=./src/**.js \
  --dependency_mode=PRUNE \
  --warning_level=VERBOSE \
  --js_output_file=./dist/bundle.js \
  --module_resolution=WEBPACK \
  --compilation_level=ADVANCED;

  # Plus whatever flag I was testing at that moment, e.g.:
  # --jscomp_error=checkDebuggerStatement
  # --jscomp_error=unusedLocalVariables
  # --jscomp_error=reportUnknownTypes
  # --jscomp_error=strictCheckTypes

Fixed the title as this refers to the warnings page (not flags). The flags get auto updated with each release. The warnings do not.


Error In PHP when using extensions

Proto code bellow:
message BookVersion {
	required uint32 major = 1;
	required uint32 minor = 2;
	required uint32 patch = 3;
}

extend google.protobuf.MessageOptions {
	optional BookVersion version = 50001;
}

message BookData {
	required fixed64 timestamp   = 1;
	required BookInfo version    = 2;
}

message BookAlive {
	option (version).major = 1;
	option (version).minor = 0;
	option (version).patch = 1;

	required BookData data = 1;
}

I am getting php error calling version method not found in class version on BookVersion class

Options currently not supported,
I'll be adding support for it. I'll ping you once i get it merged..

@luisgon1  This https://github.com/protobuf-php/protobuf-plugin/commit/2000dd408ee7dc60c3e3983f928f37062fd11d1f should fix the problem with option (only messages options are supported for now).
<?php

/** @var $version \YourNamespace\BookVersion(); */
$version = \YourNamespace\BookAlive::descriptor()
    ->getOptions()
    ->extensions()
    ->get(\YourNamespace\Extension::version());

Please let me know if it works for you.

It is not working, bellow is the $options set in the class built:
$options = \google\protobuf\MessageOptions::fromArray([
]);
    $options->extensions()->put(\Extension::version(), \BookVersion::__set_state(array(
       'unknownFieldSet' => NULL,
       'extensions' => NULL,
       'major' => NULL,
       'minor' => NULL,
       'patch' => 9,
    )));

It is not defining major or minor
in another code I created it is not even defining the extensions

import "google/protobuf/descriptor.proto";
message BookVersion {
required uint32 major = 1;
required uint32 minor = 2;
required uint32 patch = 3;
}
extend google.protobuf.MessageOptions {
optional BookVersion version = 50001;
}
message BookData {
required fixed64 timestamp   = 1;
required BookVersion version    = 2;
}
message BookAlive {
option (version).major = 9;
option (version).minor = 5;
option (version).patch = 9;
required BookData data = 1;

}

After https://github.com/protobuf-php/protobuf-plugin/commit/b9c91c2baa5b17076c954943f437a82eaa16c9fc it should generate the following :
$options->extensions()->add(\Extension::version(), \BookVersion::__set_state(array(
   'unknownFieldSet' => NULL,
   'extensions' => NULL,
   'major' => 9,
   'minor' => 5,
   'patch' => 9,
)));

please make sure you have the latest version of :

protobuf-php/google-protobuf-proto
protobuf-php/protobuf-plugin
protobuf-php/protobuf

So if I use 0 as an option value it gives NULL.

I have updated everything from the latest versions

Also it only works if you compile everything from 1 proto file.
If I use 2 proto files it does not work.

bookInfo.proto
import "google/protobuf/descriptor.proto";
message BookVersion {
required uint32 major = 1;
required uint32 minor = 2;
required uint32 patch = 3;
}
extend google.protobuf.MessageOptions {
optional BookVersion version = 50001;
}
message BookData {
required fixed64 timestamp   = 1;
required BookVersion version    = 2;
}

books.proto
import "google/protobuf/descriptor.proto";
import "bookInfo.proto";
message BookAlive {
option (version).major = 4;
option (version).minor = 4;
option (version).patch = 5;
required BookData data = 1;

}

@luisgon1  The 0 value issue should be fixed by https://github.com/protobuf-php/protobuf-plugin/commit/5763220478ec2967edf934c054ead9921c614e58,
Please make sure you generate both files otherwise the plugin considers bookInfo.proto a import and wont generate the BookVersion extension.
Something like :
./vendor/bin/protobuf \
    -o ./path-to-php-src\
    -i ./path-to-protos \
    ./path-to-protos/bookInfo.proto \
    ./path-to-protos/books.proto

Closing it for now
Please let me know if the problem continues.

Its working perfect, thanks for the support
On Jun 18, 2016 10:49 AM, "Fabio B. Silva"<EMAIL_ADDRESS>wrote:

Closing it for now
Please let me know if the problem continues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/protobuf-php/protobuf/issues/6#issuecomment-226949345,
or mute the thread
https://github.com/notifications/unsubscribe/AGIoqJaC9hL1bk65DTADRUCTiiqw-Cbvks5qNBN-gaJpZM4HHEu6
.

Thanks @luisgon1