Test of using BTRFS

Take a look at this @cimryan please.
Changes:

backingfiles is created as btrfs (and tools are installed)
vfat disks are mounted at the start of the loop instead of in the loop. My thinking is that the normal pattern of the Pi being shut down and restarted with the car is basically a given. So fix them once each "boot". I don't think this should be needed during the archive mount and copy
snapshot is created/mounted/deleted during archiving, files are copied from there, and drives are never removed from the host. To see how stability is improved/isn't.

Problems:

Since it's a snapshot, files are copied not moved. If we go down this path, we can figure out how to deal with that. Some people want them copied not moved anyway, so a purge algorithm might be better anyway.
I reverted to using your partitioning logic; parted doesn't seem to be loving the way I was doing it, or I ended up creating a bug. I'll figure this out separately as I like the simpler logic
I'm missing something basic but campercent less than 100 ends up causing the music disk to be created too big. Again probably some basic thing I missed

It's a proof of concept, but I like it, if it tests stably (and better than using a simultaneous ro mount which is another option to explore). btrfs is used in production pretty heavily so I don't feel too worried about it...yet.

Moving saved clips off the drive will require disconnecting the virtual drive from the Tesla. It seems reasonable to disconnect the drive just before archiving.

Copying the ephemeral cam files seems like it would be another set of steps in archiveloop.

I have a few thoughts here; for example, we could have pre-mount actions that move off existing files, and/or a configurable prune routine (by space, by date, etc.).
All things being equal, I don't love disconnecting the drive, but I can't empirically prove that it's not the best way to do things. :) It just seems cleaner to say "once the drive is online, it has one goal: stay online and stable", since its purpose is to be as available as possible in case the footage is needed.


🔖 release version 0.8.12

Generated summary (powered by Graphite)

TL;DR
This pull request updates the version of the Python Rebalancer (pyrb) package from 0.8.11 to 0.8.12 in the pyproject.toml file.
What changed
The only change in this pull request is the version number of the pyrb package. The version has been incremented from 0.8.11 to 0.8.12.
-version = "0.8.11"
+version = "0.8.12"

How to test
To test this change, you can follow these steps:

Pull the changes from this branch into your local environment.
Run poetry install to install the updated package.
Verify the installed package version by running poetry show pyrb.

Why make this change
This change is necessary to release the new version of the pyrb package. The new version includes all the changes and improvements made since the last release (0.8.11). By incrementing the version number, we can distribute these updates to the users.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #73

PR #74

PR #75

PR #76

PR #77

PR #78   👈

This stack of pull requests is managed by Graphite.

Merge activity

Jan 2, 9:24 AM: @mingi3314 started a stack merge that includes this pull request via Graphite.


Ensure synchronous command execution is lazy

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix.
What is the current behavior? (You can also link to an open issue here)
Synchronous commands execute their workload immediately upon a call to Execute instead of requiring a subscription.
What is the new behavior (if this is a feature change)?
Synchronous commands now require a subscription in order for their workload to execute.
The documentation clearly states that subscription is required before commands execute, and that is indeed how asynchronous commands work. So this change brings the code into line with the documentation.
Does this PR introduce a breaking change?
Technically, I suppose I'd have to say yes. However, the circumstances under which this would break someone are:

They are using a synchronous command (which are less common than asynchronous commands, in my experience).
They are executing the command and not subscribing (which is less common than binding the command or using InvokeCommand).

So anyone who does break isn't using the framework correctly and would be bitten if, for example, they tried the same thing with an asynchronous command. Also, if they're subscribed to IsExecuted or to the command itself, those subscriptions will never fire if they are not subscribing.
Please check if the PR fulfills these requirements

[X] The commit follows our guidelines: https://github.com/reactiveui/reactiveui#contribute
[X] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Coverage increased (+0.2%) to 66.133% when pulling 111ad1f4d91d88d66e79752a79ffd57441bd4672 on kentcb:lazy-sync-commands into 8e9963f05ab38cf13d158240d2f1a6bc5d6ecbb2 on reactiveui:develop.

It's quite inconsistent alright, but to be fair, I always thought it was a design decision. Since the code in the command isn't asynchronous, there's not much reason to "subscribe" onto it. But that's an assumption on my part!
The reason I'm telling is because we currently do have (some) code that functions without the Subscribe part because we thought it was by design. And hastily upgrading RxUI would render those functions un-executed, maybe even without noticing. Since unit-testing platform-code is still a hassle it might very well go undetected. (Since the caller is often the View in the platform-code.)
I'm not saying this is a bad change, I'm all for consistency, but we need to make sure to inform users well of this. 👍

Coverage increased (+0.2%) to 66.688% when pulling eaac01f3cd72fa78b60b7a790e79abd5669aad24 on kentcb:lazy-sync-commands into 427885ea13fac68f60871784e84c81fa0aef2386 on reactiveui:develop.


version 5.0.1 can not work on react-native version 0.74.5 and 0.72.1

after set RootSiblingParent as the root element and call
let sibling = new RootSiblings(<View
style={{top: 0,right: 0,bottom: 0,left: 0,backgroundColor: 'red'}}
/>);
show nothing. How to use this version.  But lib react-native-root-toast dependent on this lib works. Any on can help me for this issue?

missing position:absolute?

it works after add position:'absolute'， thx @sunnylqm


fix(dom): wait for snabbdom's post hooks to clean vnodes (#667)

Initially, vnodes were cleaned (EventListener destroyed) during the snabbdom's `remove` or `destroy`
hook. Thus, if the vnode is, in fact, used latter on in the DOM, those listeners are now gone.
This patch only removes those nodes at the end of snabbdom's process. Thus if the nodes are needed
latter on, they won't be destroyed
ISSUES CLOSED: #667

[x] I added new tests for the issue I fixed/built
[x] I ran make test FOO for the package FOO I'm modifying
[x] I used make commit instead of git commit

side note: created a pack and tested it against my actual project, I can confirm it fixes #667 :)

@atomrc sorry for the lack of update. I'll try to find some time to look at this on the weekend. Been moving house recently so hard to find time to focus.

Good PR! Thanks a lot @atomrc !

My pleasure. Thanks for the review!
Any idea when could be the next release of @cycle/dom?


Update JasmineGraph link prediction

Update JasmineGraph link prediction related to federated implementation.

These changes will include in the PR https://github.com/miyurud/jasminegraph/pull/168. Hence closing this PR.


feat(pointcloud): Clear stash after pageChange

:tada: This PR is included in version 1.12.0-alpha.5 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.8.0-alpha.5 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.16.0-alpha.5 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


"01" returns "เอ็ดบาทถ้วน

Should be "หนึ่งบาทถ้วน"

@narze I added new PR to handle these cases.

Sorry I didn't have time to check issues, will review shortly

This bug is still present in version 0.7.0. The current test covers "-01" but "01", which might be why you overlooked this. I'll add more tests to this now that I've found more bugs.

This bug is still present in version 0.7.0. The current test covers "-01" but "01", which might be why you overlooked this. I'll add more tests to this now that I've found more bugs.

I totally missed it! 😱 Thank you for the PR, @rootEnginear .

😱 😱 😱


chore: project addition by whiteknight16

Add a new project to the list
Space invader game
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue
[X] My project has contribution guidelines
[X] My project has a Code of conduct
[X] My project has README
[X] My project has a License

you don't need to create multiple issues for projects addition at a time


spacy intallation - numpy 1.16.2 which is incompatible

During spacy installation, I got this message, could it has implications for the RNN Lab exercise?
(mxnet_dev) @local:~/repos/oreilly-mxnet-workshop$ pip install spacy -U --quiet
mxnet 1.5.0b20190304 has requirement numpy<1.15.0,>=1.8.2, but you'll have numpy 1.16.2 which is incompatible.
mxnet-cu92 1.4.0 has requirement numpy<1.15.0,>=1.8.2, but you'll have numpy 1.16.2 which is incompatible.

If spacy is installed, version of numpy should not be a problem. Thanks for trying it out! Closing for now. Reopen if you face any issues.


Peer Review - Reflection

Artifact Under Review
Reflection Section
Team Number for Team Doing the Review
Team 27
Description of Issue
Team should go more in depth on roles and call out names, for example, mentioning that Ahmed will be in charge of the ML side of things while the other teammates are in charge of data science in the project. This would help to aid the team by giving it more structure
and every team member would feel as though they have a dedicated role. Overall good work!

@Tan630


PowerShell script to add all default NSG rules for Azure AD Domain Services

With the need to use PowerShell to add one of the default NSG rules for Azure AD Domain Services I think it would be beneficial if the Docs had a script to add all of the rules at once. This would allow someone who set up their own NSG for AADDS without the rules to be able to add the rules all at once instead of using a mixture of the portal and PowerShell. The below script is what I have come up with to add all of the NSG rules.
$nsgName = "NSG name"
$rgName = "Resource group Name"
$ruleName = "AllowSyncWithAzureAD", "AllowRD", "AllowPSRemoting"
$serviceTagName ="AzureActiveDirectoryDomainServices", "CorpNetSaw", "AzureActiveDirectoryDomainServices"
$priority = 101, 201, 301
$destinationPortRange = "443", "3389", "5986"

for ($i=0; $i -lt $priority.length; $i++ ){
Get-AzNetworkSecurityGroup `
 -Name $nsgName `
-ResourceGroupName $rgName |
 Add-AzNetworkSecurityRuleConfig `
-Name $ruleName[$i] `
-Access Allow `
-Protocol Tcp `
-Direction Inbound `
-Priority $priority[$i] `
-SourceAddressPrefix $serviceTagName[$i] `
-SourcePortRange * `
-DestinationAddressPrefix * `
-DestinationPortRange $destinationPortRange[$i] |
    Set-AzNetworkSecurityGroup
}

Thank you

@atwerae -
Thank you for your question.
Can you please share with us the URL of the document that you are looking at ?
Thank you for your understanding.

@atwerae - Can you please share with us the URL of the document that you are looking at ?
Thank you for your understanding.

@atwerae -
Since we have not heard back from you we will now proceed to close this thread.
However if you can share additional information regarding the documentation you are referring and circle back on this thread , we will gladly reopen the issue and continue the discussion. Thank you.


TypeError: Cannot read properties of null (reading 'FactoryTimestamp')

Hi, I have the following issue, right after starting the docker-container. It seems to be connected to nightscout and the librelink cloud correctly.
The error might occure because I have no recent blood glucose values in the libre cloud. (But I plan using it in the future, since I have to switch from libre2 to libre3).
nightscout-libre-link_1  |
nightscout-libre-link_1  | ><EMAIL_ADDRESS>start
nightscout-libre-link_1  | > node index.js
nightscout-libre-link_1  |
nightscout-libre-link_1  | 2022-03-18T17:41:04.531Z [info]: Started
nightscout-libre-link_1  | 2022-03-18T17:42:07.692Z [info]: Logged in to LibreLink Up
nightscout-libre-link_1  | 2022-03-18T17:42:15.168Z [info]: Found 1 LibreLink Up connection.
nightscout-libre-link_1  | 2022-03-18T17:42:15.169Z [info]: -> The following connection will be used: XXX (Patient-ID: XXXXXX-XXXX-XXXXX-XXXX-XXXXXXXXXXXX)
nightscout-libre-link_1  | 2022-03-18T17:36:15.329Z [info]: Received blood glucose measurement
nightscout-libre-link_1  | /usr/src/app/index.js:186
nightscout-libre-link_1  |     const measurementDate = getUtcDateFromString(glucoseMeasurement.FactoryTimestamp);
nightscout-libre-link_1  |                                                                     ^
nightscout-libre-link_1  |
nightscout-libre-link_1  | TypeError: Cannot read properties of null (reading 'FactoryTimestamp')
nightscout-libre-link_1  |     at uploadToNightscout (/usr/src/app/index.js:186:69)
nightscout-libre-link_1  |     at getGlucoseMeasurement (/usr/src/app/index.js:116:9)
nightscout-libre-link_1  |     at processTicksAndRejections (node:internal/process/task_queues:96:5)
nightscout-libre-link_1  |
nightscout-libre-link_1  | Node.js v17.7.1

hi @dassystem, can you make sure you have current measurements in the LibreLink Up app? i have not come across this issue myself yet. :(

Whether the application has fresh data from a client or not, the server should not crash. Not having data could have several reasons: A fresh setup, missing internet connection of the uploader, issues on the side of abbots site. In these cases, certainly not the whole application should break.
Unfortunately I don't know javascript very well, but I assume that we need to check for null values when loading it to the variable in index.js line 186.
const measurementDate = getUtcDateFromString(glucoseMeasurement.FactoryTimestamp);
An additional try catch might solve it.

Hey @timoschlueter, thanks for the mention!
I'm now getting a error message in the log (which is correct!) and the application is not crashing anymore (which is even better!)
nightscout-libre-link    | 2022-06-14T13:11:48.104457492Z ><EMAIL_ADDRESS>start
nightscout-libre-link    | 2022-06-14T13:11:48.104477066Z > node index.js
nightscout-libre-link    | 2022-06-14T13:20:00.425171582Z [info]: no authTicket.expires
nightscout-libre-link    | 2022-06-14T13:20:00.425530543Z [info]: renew token
nightscout-libre-link    | 2022-06-14T13:20:00.760973318Z [info]: Logged in to LibreLink Up
nightscout-libre-link    | 2022-06-14T13:20:00.954207820Z [info]: Found 1 LibreLink Up connection.
nightscout-libre-link    | 2022-06-14T13:20:00.955213701Z [info]: -> The following connection will be used: <MY-NAME> (Patient-ID: <MY-UUID>)
nightscout-libre-link    | 2022-06-14T13:20:01.130118050Z [info]: Received blood glucose measurement items
nightscout-libre-link    | 2022-06-14T13:20:01.132070664Z [error]: Error getting glucose measurements Cannot read properties of null (reading 'FactoryTimestamp')

So at least this unhappy-path is working as expected and this bug issue can be closed. Thank you!

awesome! :)


Add wasmer web assembly submodule and a POC engine

The wasmer-d submodule supports running web assembly outside of the web browser. The class node/WasmEngine.d provides a POC engine to run web assembly programs within agora.
This is still very much a work in progress. There are some unit tests that can be used to experiment.

FYI things should run on Alpine since v2: https://github.com/wasmerio/wasmer/commit/ea4ea47e174f33734ffd3533e93e469281c076e0

This is what scares me a bit: https://github.com/wasmerio/wasmer/blob/master/lib/vm/src/trap/traphandlers.rs


Fix a bug with secrets masking

This pull request fixes a bug which would cause the masking could to modify the original dictionary in place instead of modifying a copy of it and using a copy to interpolate the log message.

Nice.
+1


Pyflyte docs formatting

TL;DR
Update docs formatting for pyflyte. RestructuredText is a little bit different to Markdown... 😢
Type

[X] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[x] Unit tests added
[X] Code documentation added
[x] Any pending items have an associated Issue

There is a linting error, you could fix it by merging master.

@pingsutw Thanks for the review! I merged master.


[Do Not Merge] Updating for @derivative(of:) usage in Raw TensorFlow Operators tutorial

The  current Raw TensorFlow Operators tutorial uses the deprecated @differentiable(vjp:) syntax, which will be removed in the 0.9 release toolchain (and is already removed in the nightlies). This replaces that implementation and description with the modern equivalents.

I think we need this in the 0.9 release branch first and this outstanding.  Please don't merge until 0.9 Colab goes live.

Actually, does this already work in 0.8?

This patch should be valid in 0.8 and 0.9.

Yes, I can confirm this works in 0.8 as well, wasn't sure if the best approach was to start here and cherry-pick back to tensorflow-0.9.


BUG - undo/redo disfunctional

Does the issue occur in the electron-app or the web-app?

[ ] midi-bricks-electron
[ ] midi-bricks-web-app
[x] both

Can you determine, which version was used?
Version: 1.2.8-alpha.14
Expected behavior
An undo/redo feature should work in all concerns: Layout Mode, Settings Mode, MIDI learn.
Additional context
If the feature is just designed for Layout mode, it should only appear in Layout Mode.
If it doesn't work properly yet, we shouldn't have it in the App for now.

Just want to mention, that i know the feature is not perfect. Anyhow, please keep asking yourself the question, if you need any possibility to get any layout-changes undoable or not. If yes, let us create another ticket to improve undo redo behaviour. Otherwise, we could put that into an extra alpha version release. This is just a MVP!

It's an important feature, because in Layout mode the Automatic Gravity can completely mess up your entire layout in a second.
So we should keep it for the future, but exclude it from the releaes, until it works fine.

at a first shot layout undo is only shown in layout mode, see new page layout release


Making About us page responsive

I would like to work on this.

I would also like to work on this.

@Yasir761 are you still working on this issue?

@Yasir761 are you still working on it ?

@hparashar27 Can I take this issue?

@Saikat-Mandal  would you like to work on this issue ?

@hparashar27 yeh sure !

@AbhishekBhamare you can this issue

@hparashar27 should i use taiwind css or just normal css only ?

you have done it greatly, would you like to work on the issue #48


strategy for organizing graphics functions in devel...

@doutriaux1  Making progress with a series of branches that include new metrics in, for example, devel/monsoon/scripts and devel/monsoon/lib.    We need to decide how best to partition analysis (metrics) codes from graphics fcns.   Question: when developing a plotting routines, should they go in devel/monsoon/scripts and devel/monsoon/lib or devel/monsoon/scripts and devel/monsoon/graphics ?

@gleckler1 at the moment you have /src/python/graphics which contains all graphic (the portrait.py) functions, and /src/python/pcmdi that contains all the metrics calculation functions.. To me I think having these separated (graphics from calcs) is a good idea.. A rename of the pcmdi to something more descriptive could be useful..
@doutriaux1 thoughts?

@doutriaux1 @durack1  Its a 2 part question... the first is where to put graphics in "devel" branches ... we want to do this consistently to ease how its dealt with in setup.py... 2nd question is where do we them to go in master when merged.   I think PD's suggestion is part 2.

@gleckler1 just to comment (you're actually doing this, I'm just commenting) would it be smart to have a complete directory tree under devel, so src/python/devel/monsoon_wang and then under this directory duplicate the "functions" and "graphics" directories, that would then be folded back into the master /src/python/graphics and /src/python/pcmdi or "function" trees?
At least in my head it would be relatively straight forward to then extract the contents of the subdirs out of the branch/devel subdir and place them in their appropriate "master" subdirs as part of the install..?

@durack1 @doutriaux1 Thats what I'm thinking.

scripts go in scripts, that's easy
now all code should go under lib at that point you have 3 choices
if it's tiny put in graphics.py under monsson/lib
if it's so big you need to break it then put it inside directory: monsoon/lib/graphics/files.py
if it's going to be used by many many branches then put it under src/python/graphics

@durack1 @doutriaux1   How do you define "tiny"  or "so big" ?
Can't we decide where they should go based on logic?
I don't have strong feeling how/where we do this, except I would like it to be sufficiently self describing by have all graphics under one tree rather than scattered about.

@gleckler1 I see two options, use directories to separate stuff.. Or use a naming convention that allows for this information to be gleaned from filenames.. Probably better to chat about this with a whiteboard..

ok stick it in monsoon/lib/graphics/ directory


Change to Docker Api

This changes the interaction with docker from the cli to the api.
The API is better suited for this task, because it provides more and easier accessible information.
It also allows easier testing, because it can be mocked which wasn't the case for the cli calls.
This changes the behavior, so that all docker compose projects/services on the current host are found and not just ones that are in the root folder.
I think that I will add some way to limit the containers which are inspected by this program in a later PR

This closes #31


Update babel config js to avoid warnings

Dependent on #486 so merge that one first.

@runephilosof-abtion I've merged #486 can I rebase and merge this?

Squashing is so annoying. If it had been merged normally, the rebase would not have given a conflict.

Squashing is so annoying. If it had been merged normally, the rebase would not have given a conflict.

I'm with you on that.
However; squashing is more forgiving and does not require as well-structured commits as rebasing. It lowers the git skill requirements slightly.

Squashing is so annoying. If it had been merged normally, the rebase would not have given a conflict.

I'm with you on that.
However; squashing is more forgiving and does not require as well-structured commits as rebasing.

I guess what I want is for Github to fast forward if there is already only one commit and it is already at the tip of the base branch.


[FEAT] new integrate @tanstack/router

Is your feature request related to a problem? Please describe.
I am doing integrate @tanstack/router to my refine project, @tanstack/router has nice DX. I want to add a new package in to refine packages.
Describe alternatives you've considered
No response
Additional context
No response
Describe the thing to improve

[ ] add a new package @refine/tanstack-router
[ ] docs update

I would be happy to help with this one

Hey @okxiaoliang4, thanks for the issue! Right now, we are not considering adding support for @tanstack/router.
However, you can create an npm package yourself and publish it. We can add your package to our documentation's community packages section.


Add automated basic test

I'd propose to start adding tests to catch bugs like #130 before they hit larger groups of users. I've added 2 very basic tests:
Hassfest:
Runs on every push / pull requests and daily to keep up with HA updates
Validates the HA plugin
CI:
Runs on every push / pull requests and weekly to keep up with the latest python version (to be prepared for future HA updates), used python versions are 3.10, 3.11 and latest 3.x
The current test is very basic and only checks if the sensor entity can be loaded correctly. Checks for OWM, calculations, etc should be added in the future .

hey @mathijsmeulendijks I am no expert on testing and the way you set it up. Any chance you would be willing to help set up testing for the 2.0 branch?

I'm not that familiar with it either, but this is a good opportunity to learn more about it. In about 2 weeks I can take a closer look, in the meantime I'll read up on testing a bit more.


Firrebase install problem

What version of gRPC and what language are you using?
1.11.0 Javascript
What operating system (Linux, Windows, …) and version?
System Windows_NT 10.0.17134 64x
What runtime / compiler are you using (e.g. python version or version of gcc)
python 2.7
What did you do?
created new project
npm init
tried to install firebase, via
npm install firebase --save
What did you expect to see?
firebase installed ^^
What did you see instead?
C:\Projects\reactnative\test>npm install

><EMAIL_ADDRESS>install C:\Projects\reactnative\test\node_modules\grpc
> node-pre-gyp install --fallback-to-build --library=static_library

node-pre-gyp ERR! Completion callback never invoked!
node-pre-gyp ERR! System Windows_NT 10.0.17134
node-pre-gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Projects\\reactnative\\test\\node_modules\\grpc\\node_modules\\node-pre-gyp\\bin\\node-pre-gyp" "install" "--fallback-to-build" "--library=static_library"
node-pre-gyp ERR! cwd C:\Projects\reactnative\test\node_modules\grpc
node-pre-gyp ERR! node -v v8.11.2
node-pre-gyp ERR! node-pre-gyp -v v0.10.0
node-pre-gyp ERR! This is a bug in `node-pre-gyp`.
node-pre-gyp ERR! Try to update node-pre-gyp and file an issue if it does not help:
node-pre-gyp ERR!     <https://github.com/mapbox/node-pre-gyp/issues>
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.

npm ERR! code ELIFECYCLE
npm ERR! errno 6
npm ERR<EMAIL_ADDRESS>install: `node-pre-gyp install --fallback-to-build --library=static_library`
npm ERR! Exit status 6
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\unoeg\AppData\Roaming\npm-cache\_logs\2018-05-22T09_08_40_439Z-debug.log

2018-05-22T09_08_40_439Z-debug.log
Anything else we should know about your project / environment?
then i tried to install grpc separatly
C:\Projects\reactnative\test>npm install grpc --save

><EMAIL_ADDRESS>install C:\Projects\reactnative\test\node_modules\grpc
> node-pre-gyp install --fallback-to-build --library=static_library

node-pre-gyp ERR! Completion callback never invoked!
node-pre-gyp ERR! System Windows_NT 10.0.17134
node-pre-gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Projects\\reactnative\\test\\node_modules\\grpc\\node_modules\\node-pre-gyp\\bin\\node-pre-gyp" "install" "--fallback-to-build" "--library=static_library"
node-pre-gyp ERR! cwd C:\Projects\reactnative\test\node_modules\grpc
node-pre-gyp ERR! node -v v8.11.2
node-pre-gyp ERR! node-pre-gyp -v v0.10.0
node-pre-gyp ERR! This is a bug in `node-pre-gyp`.
node-pre-gyp ERR! Try to update node-pre-gyp and file an issue if it does not help:
node-pre-gyp ERR!     <https://github.com/mapbox/node-pre-gyp/issues>
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.

npm ERR! code ELIFECYCLE
npm ERR! errno 6
npm ERR<EMAIL_ADDRESS>install: `node-pre-gyp install --fallback-to-build --library=static_library`
npm ERR! Exit status 6
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\unoeg\AppData\Roaming\npm-cache\_logs\2018-05-22T09_13_16_433Z-debug.log

2018-05-22T09_13_16_433Z-debug.log
tried to install via yarn and building grpc from source
C:\Projects\reactnative\test>node -v
v8.11.2

C:\Projects\reactnative\test>python -V
Python 2.7.15

C:\Projects\reactnative\test>npm -v
6.1.0-next.0

I've the same issue:
npm -v
6.0.0
node -v
v8.11.1
No python.
Windows 7

C:\dev\lunchtime>npm i firebase

><EMAIL_ADDRESS>install C:\dev\lunchtime\node_modules\grpc
> node-pre-gyp install --fallback-to-build --library=static_library

node-pre-gyp ERR! Completion callback never invoked!
node-pre-gyp ERR! System Windows_NT 6.1.7601
node-pre-gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\dev\\lunchtime\\node_modules\\grpc\\node_modules\\node-pre-gyp\\bin\\node-pre-gyp" "install" "--fallback-to-build" "--
library=static_library"
node-pre-gyp ERR! cwd C:\dev\lunchtime\node_modules\grpc
node-pre-gyp ERR! node -v v8.11.1
node-pre-gyp ERR! node-pre-gyp -v v0.10.0
node-pre-gyp ERR! This is a bug in `node-pre-gyp`.
node-pre-gyp ERR! Try to update node-pre-gyp and file an issue if it does not help:
node-pre-gyp ERR!     <https://github.com/mapbox/node-pre-gyp/issues>
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

npm ERR! code ELIFECYCLE
npm ERR! errno 6
npm ERR<EMAIL_ADDRESS>install: `node-pre-gyp install --fallback-to-build --library=static_library`
npm ERR! Exit status 6
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\xxx\AppData\Roaming\npm-cache\_logs\2018-05-22T11_43_10_774Z-debug.log

This is a known bug in node-pre-gyp: mapbox/node-pre-gyp#367.

just tried that and now this is the output
C:\Projects\reactnative\auth\node_modules>npm install firebase --save --force
npm WARN using --force I sure hope you know what you are doing.

><EMAIL_ADDRESS>install C:\Projects\reactnative\auth\node_modules\grpc
> node-pre-gyp install --fallback-to-build --library=static_library

node-pre-gyp ERR! Completion callback never invoked!
node-pre-gyp ERR! System Windows_NT 10.0.17134
node-pre-gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Projects\\reactnative\\auth\\node_modules\\grpc\\node_modules\\node-pre-gyp\\bin\\node-pre-gyp" "install" "--fallback-to-build" "--library=static_library"
node-pre-gyp ERR! cwd C:\Projects\reactnative\auth\node_modules\grpc
node-pre-gyp ERR! node -v v8.11.2
node-pre-gyp ERR! node-pre-gyp -v v0.10.0
node-pre-gyp ERR! This is a bug in `node-pre-gyp`.
node-pre-gyp ERR! Try to update node-pre-gyp and file an issue if it does not help:
node-pre-gyp ERR!     <https://github.com/mapbox/node-pre-gyp/issues>
+<EMAIL_ADDRESS>added 83 packages from 58 contributors, updated 1 package and audited 32813 packages in 27.218s
found 2 vulnerabilities (1 low, 1 high)
  run `npm audit fix` to fix them, or `npm audit` for details

C:\Projects\reactnative\auth\node_modules>

firebase folder created but i cannot use firebase lib( package not found

@unoegohh can't help you there I was able to access firebase after the force install so I don't know...Best of luck.

As @murgatroid99 said, this isn't a problem with gRPC itself, with a link to the actual problem, so this is kind of the wrong location to discuss it.

Hello, I had the same problem, I disabled my antivirus (kaspersky) and it ran like a charm. Hope it can help.

Disabling Kaspersky worked for me,
Thanks eraldinho

npm install firebase --force worked for me ^^. Thanks guys!

I had the same issue too, disabling Kaspersky antivirus worked like magic. Thanks.

This was supposably solved in mapbox/node-pre-gyp v0.10.1, yet gPRC (or at least the version used by Firebase) uses v0.10.0.

First, we are using a caret dependency, so the package being picked up is going to be 0.10.1. Second, the node-pre-gyp dependency is bundled with the grpc package. So you get whichever version we are bundling. 1.13.1 is now bundling node-pre-gyp 0.10.1.

Also, firebase had supposedly updated to grpc 1.13.1.

@nicolasnoble I'm installing the latest version of firebase, and within the error is the following message:
node-pre-gyp ERR! node -v v10.1.0
node-pre-gyp ERR! node-pre-gyp -v v0.10.0

I don't think firebase has released their latest version yet. Follow https://github.com/firebase/firebase-js-sdk/pull/1042 instead.

i have the same problem,
i try     npm install firebase --force
and the problem solved

The node-pre-gyp issue was fixed a while ago, and everything has been published with the fix in. Closing this one.

Still get the error when trying to install

Please stop bumping an old, resolved bug, and open new ones instead, with the full description of your issues so we can try helping your specific case.


cannot sign in on iOS 16+

Hey there,
thanks for the effort. I am not able to sign in on iOS 16. I try to follow your guide, but when I uninstall the signed in app store app and open your application afterwards it is signed out and I am not able to sign in again.
Is there a way to make it work in iOS 16 as well?
thx

Thanks for your feedback.
Yes, unfortunately this is a problem only on iOS 16. It also has nothing to do with SCMusicPlus, but rather SoundCloud itself. The app can detect when it is sideloaded (which is why iOS 15 Trollstore users have no issues). When the app has detected that it is not the App Store application, it will reject all sign in credentials even when they are valid.
I have not been able to work around this issue yet from within the tweak, but thanks to a couple of users, we were able to confirm that Google Accounts can still sign in. Personally I don't like linking Google or Facebook to my other accounts so I do not recommend this workaround to those who are willing to wait for an actual bypass (no telling when it is coming, I will admit it might take a while since I'm new to this). You could also create a dummy account and use it only for this application.
To sign in on iOS 16, you have a couple choices (personal preference):

Pair your Google Account with your existing SoundCloud account
Use your Google Account with a new SoundCloud account
Pair your Facebook Account with your existing SoundCloud account
Use your Facebook Account with a new SoundCloud account

From here, sign into the app store app and continue the sideloading steps like normal.
**If you skip Google Account sign in to the app store app and try to login directly on the SCMusicPlus ipa, you will have to use the same mitigation as the sideloaded YouTube users. The instructions to this can be found here: https://github.com/qnblackcat/uYouPlus/discussions/447. I will look into solving this issue very soon so that you all can at least sign in on the modded ipa directly.
If for some reason anything explained above does not work, let me know.

I just came across from this by a friend on discord and he sent me this because I was trying to find a SC++ IPA that works for IOS 16 for my friends and while reading this it looks like you have done amazing work! : ) I just wanted to ask you if you can just confirm that this IPA that you have developed also has full premium advantages as other SC++


Export poetry dependencies without python_version interval

[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] I have searched the FAQ and general documentation and believe that my question is not already covered.

Feature Request
I have been looking for a way to export my project dependencies. I know there is the command poetry export with its flags, which performs what I am looking for. However, I added additionally the flags  --without-hashes and  --without-urls to turn it less verbose than before.
On the conventional exporting way with pip freeze, the format corresponds to package_name==package_version. I can perform this same pattern on Linux with the command below,  but it would be great to have an additional flag --without-pyversion
poetry export --without-hashes --without-urls | awk '{ print $1 }' FS=';'

I disagree with trying to force the programmer to accept one way of doing requirements.txt. The package should enable the programmer to reach the result they want, not dictate the format of a script and try to unnecessarily force the programmer to accept all of its commands and idiosyncrasies.
People who need a requirements.txt file without the python_version string are going to figure it out on their own regardless, but wouldn't it be much better for the package itself to have that as an inbuilt option.

@annie-maria this: https://www.youtube.com/watch?v=dQGrvCAmt-A

@annie-maria Thank you for sharing the command. I also had to remove the strict python_full_version marker since I work in GCP App Engine standard environment and there are few options for a python environment.
I think a --without-markers flag can be a nice addition.

I come here after wasting some hours trying to figure out why my Cloud Function deployment wasn't working. It turns out that Google have a taste for plain requirements.txt .
Thank you @brunolnetto

I had the exact same problem with windmill. Their pip installer is very picky, too https://github.com/windmill-labs/windmill/issues/4376

entertainingly the most recent comment in support of this arrived near enough simultaneously with a demonstration of why it is the wrong thing to do https://github.com/python-poetry/poetry-plugin-export/issues/28#issuecomment-2371007822

Mixed opinions. To be in control is as desired as to rely on automatic actions. Faith and Science eternal battle.

What about specifying the target Python version in the export? Then the export is correct and no conditionals for the python version are needed. I just want to use sane dependency pinning where the build-in dependency installer does silly things.


Is this replacement for log4net?

or ES appender only?
I use log4net in my WCF service.
Should I replace it with the logstash or to add the logstash to the references?

Hi, this is just an appender, you need log4net to be installed so it can load this appender.
Pay attention you add the appender correctly to the log4net configuration file.


Distributed command line cannot run

Describe the bug

I use the command to run but cannot start a swarming. Please like the picture
locust -f load.py --master[2020-07-01 16:03:38,423] Marcs-MacBook-Pro.local/ERROR/stderr: Traceback (most recent call last):
[2020-07-01 16:03:38,423] Marcs-MacBook-Pro.local/ERROR/stderr: File "/Users/user/PycharmProjects/APILoadTest/venv/bin/locust", line 10, in
[2020-07-01 16:03:38,423] Marcs-MacBook-Pro.local/ERROR/stderr:
[2020-07-01 16:03:38,423] Marcs-MacBook-Pro.local/ERROR/stderr: sys.exit(main())
[2020-07-01 16:03:38,423] Marcs-MacBook-Pro.local/ERROR/stderr:
[2020-07-01 16:03:38,423] Marcs-MacBook-Pro.local/ERROR/stderr: File "/Users/user/PycharmProjects/APILoadTest/venv/lib/python3.8/site-packages/locust/main.py", line 495, in main
[2020-07-01 16:03:38,424] Marcs-MacBook-Pro.local/ERROR/stderr:
[2020-07-01 16:03:38,424] Marcs-MacBook-Pro.local/ERROR/stderr: runners.locust_runner = MasterLocustRunner(locust_classes, options)
[2020-07-01 16:03:38,424] Marcs-MacBook-Pro.local/ERROR/stderr:
[2020-07-01 16:03:38,424] Marcs-MacBook-Pro.local/ERROR/stderr: File "/Users/user/PycharmProjects/APILoadTest/venv/lib/python3.8/site-packages/locust/runners.py", line 346, in init
[2020-07-01 16:03:38,425] Marcs-MacBook-Pro.local/ERROR/stderr:
[2020-07-01 16:03:38,425] Marcs-MacBook-Pro.local/ERROR/stderr: self.server = rpc.Server(self.master_bind_host, self.master_bind_port)
[2020-07-01 16:03:38,425] Marcs-MacBook-Pro.local/ERROR/stderr:
[2020-07-01 16:03:38,425] Marcs-MacBook-Pro.local/ERROR/stderr: File "/Users/user/PycharmProjects/APILoadTest/venv/lib/python3.8/site-packages/locust/rpc/zmqrpc.py", line 41, in init
[2020-07-01 16:03:38,426] Marcs-MacBook-Pro.local/ERROR/stderr:
[2020-07-01 16:03:38,426] Marcs-MacBook-Pro.local/ERROR/stderr: self.socket.bind("tcp://%s:%i" % (host, port))
[2020-07-01 16:03:38,426] Marcs-MacBook-Pro.local/ERROR/stderr:
[2020-07-01 16:03:38,426] Marcs-MacBook-Pro.local/ERROR/stderr: File "zmq/backend/cython/socket.pyx", line 550, in zmq.backend.cython.socket.Socket.bind
[2020-07-01 16:03:38,426] Marcs-MacBook-Pro.local/ERROR/stderr: File "zmq/backend/cython/checkrc.pxd", line 26, in zmq.backend.cython.checkrc._check_rc
[2020-07-01 16:03:38,426] Marcs-MacBook-Pro.local/ERROR/stderr: zmq.error
[2020-07-01 16:03:38,426] Marcs-MacBook-Pro.local/ERROR/stderr: .
[2020-07-01 16:03:38,426] Marcs-MacBook-Pro.local/ERROR/stderr: ZMQError
[2020-07-01 16:03:38,426] Marcs-MacBook-Pro.local/ERROR/stderr: :
[2020-07-01 16:03:38,426] Marcs-MacBook-Pro.local/ERROR/stderr: Address already in use
[2020-07-01 16:03:38,426] Marcs-MacBook-Pro.local/ERROR/stderr:
`
Expected behavior

Hope it can run well
Actual behavior

Cannot start swarming even though click the button of starting a swarming, it cannot go

Steps to reproduce
Environment

OS: Mac OS
Python version:3.8
Locust version:2.4
Locust command line that you ran: locust -f YouthBombAPI.py --slave
Locust file contents (anonymized if necessary):

Locust version:2.4
This version does not exist. What is the output of locust -V

locust1.1
@cyberw

Do you have another instance of locust running? Address already in use would suggest the port the master uses to communicate with slaves is busy.

Nope， I closed all the instance

Check for any left over python processes. Something must be using that port. Restart your machine if nothing else helps :)

Yeah, right, restart it and then it disappeared.

I tried to improve the logging in https://github.com/locustio/locust/pull/1471


Add record count to OneData File with Delta version upgrade.

Currently with delta version 2.0.2 we had remove recount count as the current version does not provide it. Add it back during upgrade.

We are now on Delta 2.4.X so this can be easily done. Labeling as a "good first issue" in case anyone wants to pick it up.

We are now on Delta 2.4.X so this can be easily done. Labeling as a "good first issue" in case anyone wants to pick it up.
as part of this change do we want to remove the record count from deltaActionsConverter as it was workaround

Also the change is not present in the github code in main branch the code is changed in "core/src/main/java/io/onetable/delta/DeltaActionsConverter.java"
https://github.com/apache/incubator-xtable/blob/main/core/src/main/java/org/apache/xtable/delta/DeltaActionsConverter.java

@gzagarwal  The history may be hard to find but there is some method we can now call to get the row count if I am remembering correctly instead of computing it ourselves.

@gzagarwal The history may be hard to find but there is some method we can now call to get the row count if I am remembering correctly instead of computing it ourselves.

Sure @the-other-tim-brown let me look into the xtable code and test case. if i find some thing then will ask to assign the jira.

@gzagarwal awesome! We're excited to have you start contributing!

Hi @the-other-tim-brown,
I really like this project! I wanted to take a look at something to pick up get used to things.
I came across this issue and I think it hasn't been closed yet. I found these two commits, so the previous method to get the count is the second(I think?)
https://github.com/apache/incubator-xtable/commit/cdc0422cbe86107b6e752562f1b0395e9a5cc728
https://github.com/apache/incubator-xtable/commit/e48374c998a39b24deb68e44a4a765e9c4da5fb4

Hi @the-other-tim-brown,
I really like this project! I wanted to take a look at something to pick up get used to things.
I came across this issue and I think it hasn't been closed yet. I found these two commits, so the previous method to get the count is the second(I think?) cdc0422
e48374c

@prithvijitguha there should be a way to get the record count directly from the delta objects after upgrading. Take a look and raise a PR if you would like!

Hi, Is someone working on it? I am new to this project and would like to get started.

@Reactor11 I don't think anyone is so I have assigned you the issue. Let me know if you need any help getting started!

Hi @the-other-tim-brown - Thanks for assigning me this issues, I would need help in getting started!
Let me know how can we connect for some time perhaps!


Add support for -fullpath to maelstrom-go-test.

This seems to be mostly relevant to the compilation stage (and not the test running stage), but I really don't know.
The creation of this issue is part of #319.

It looks like I was wrong. This seems to be a flag we would pass to the test binary, not go test -c.

We should validate that it gives the path based on the host machine not the container.


Invalid Bundle. The bundle Frameworks/GoogleMapsUtils.framework does not support the minimum OS Version specified in the Info.plist

Hi
do you know why I got this error?

Xcode 15.3
iOS min 16.4
SPM google maps utils 4.2.2

Thanks

Have you solved this problem?

Have you solved this problem?

Removed SPM and CocoaPods and imported as XCFramework
it's insane how a giant like Google has these ancient SDKs that aren't updated, don't support SPM, and create constant problems.

I changed the xcode version to 15.2 and it fixed

I changed the xcode version to 15.2 and it fixed

I'm glad you solved it but it's not a fix, it's a workaround, due to Google's unwillingness to properly update these SDKs

Goole, WAKE UP, please!!!!

Both the Maps SDK for iOS v9 and this utility library v6 support a minimum of iOS 15. Please re-open if you still see this error after updating to google-maps-ios-utils v6.0.0. See the README for installation instructions.


How to change request url from /_nuxt/xx to /nuxt/xx

I want to use Aliyun SLB for app load balance. Nuxt http request always starts with /_nuxt, but Aliyun SLB cannot use '_'.
Is there any ways to change url prefix ?
Thanks

This question is available on Nuxt.js community (#c7059)

maybe nginx url rewrite is work

someone has other options?

@caszhou https://nuxtjs.org/api/configuration-build#publicpath


Turn supporters update script into a Rake task

[x] Add rake as a dependency
[x] Create a Rakefile with a task that runs the supporters update code

Fixed by @nickybourque


[Testing]: Initial testing for our rollout

Is there an existing issue for this?

[X] I have searched the existing issues

Description of the bug
This is a placeholder of all the things I want to try to test before we release

[x] Transactions.csv is still properly logging in case we need to do an import after registration is complete
[x] Hit API endpoint with PHP
[x] Hit API endpoint with Postman / Insomnia
[x] Send lots of test data
[ ] Clean up database
[ ] Tell Rick & Russ we are ready to go

High level of things to be tested

Hit the API
Ensure users are in database
Ensure barcodes are generated
Trace down any errors needed

@stanbery - previous list of play to win from Russ
Play to Win Numbers.csv
Here is my test play to win games to upload and see the list populated, change or modify as you see fit
test-ptw.csv
I haven't tried uploading the same document or even multiple csv's. I'm not sure that is a requirement but can test if needed


mpv: added optional wayland support

This PR adds an optional wayland support to the mpv video player.
Tested on a NixOS installation, builds and runs without problem
Things done:

[X] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)

[X] Built on platform(s): NixOS

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip" :
The command gets me an error about the dwarfFortress package wich seems totally unrelated, but a grep on the nixpkgs repo shows that the only package depending on mpv is pythonPackages.mpv which builds fine.

[x] Tested execution of all binary files (usually in ./result/bin/)

[X] Fits CONTRIBUTING.md.

Do you have tested it on a Wayland environment?

Yes, I did. The OSD doesn't appear on wayland, but I assume it just hasn't been ported yet. Otherwise, it works.


Update user guide

Previously, the user guide appeared cluttered and less focused on user friendliness. The updated user guide:

Segregates the parts in terms of importance.
Provides a summary table with links to specific parts.
Fix incorrect details.

Further improvements:

Add more visuals to aid explanation.

Closes #161
Current interface (ran locally):

The add sample command is wrong currently since we are not taking in address anymore.


bootstra RTL.LESS BUG

When i install Package Twitter.Bootstrap.RTL.Less from Nuget and try compile a less file and say Errorr
In grid.less in 231 row has word left who i think is error
After i delete and save everything was good
Please watch this problem because next time when i go to update may cause new problem

I'd suggest contacting whoever managers that particular package (https://www.nuget.org/packages/Twitter.Bootstrap.RTL.Less/ ?), as it's nothing to do with bootstrap core.


Documentation fixes

fixes #79

[x] fixed "Programming Constructs" image link to "Overview5"
[x] added "Reducers" section in "Programming Constructs"
[x] clarified "field=expr" in reduce article
[x] fixed formatting of join parameter table, and while I was in there, simplified explanation of what different options do (new sentences above param table). The doc update partially addresses https://github.com/juttle/juttle/issues/76

@dmehra, reducer changes look good.

@rlgomes says "looked good the first time I skimmed through", merging.

I noticed all of the blocks where we do
```juttle
... | ...

and was wondering what it would take to have real juttle highlighting in github but it seems the library that takes care of this highlighting on github won't take support for a language until its used by hundreds of repos: https://github.com/github/linguist/blob/master/CONTRIBUTING.md#adding-a-language


[FEA] Profiling tool to highlight the most critical errors such as cudaErrorIllegalAddress

I wish Profiling tool to highlight the most critical errors such as cudaErrorIllegalAddress.
This is because this type of error normally means bad GPU and it might cause data corruption in the output data.
Currently Profiling tool can detect it in the failed_tasks.csv but we would want to highlight it in the log saying there might be a bad GPU on which node.

Does cudaIllegalErrorAddress show up in the failed_tasks.csv today?  And if so, then what type of highlighting are you looking for to show up in the profiler output?

It could be there. My thought is : Since cudaErrorIllegalAddress is a more critical error than other errors such as OOM, we should make it special in our profiling tool output.
For example, we can say :
On node with IP address/FQDN xx.yy.zz.uu, there is a cudaErrorIllegalAddress errors. There might be bad GPU there. Please contact CSP technical support or vendor to identify this problematic GPU.


Build script fails...

Amazing tutorial, thanks so much.
When I try to build the application though, I keep getting this error...
Here's the output:
>npm run-script package

><EMAIL_ADDRESS>package /Users/michele.pasin/Dropbox/code/electron/sound-machine-electron-guide
> electron-packager ./ SoundMachine --all --out ~/Desktop/SoundMachine --version 0.30.2 --overwrite --icon=./app/img/app-icon.icns

The strict-ssl parameter is deprecated, use download.strictSSL instead
The strict-ssl parameter is deprecated, use download.strictSSL instead
The strict-ssl parameter is deprecated, use download.strictSSL instead
The strict-ssl parameter is deprecated, use download.strictSSL instead
The strict-ssl parameter is deprecated, use download.strictSSL instead
The strict-ssl parameter is deprecated, use download.strictSSL instead
Packaging app for platform linux ia32 using electron v0.30.2
Packaging app for platform win32 ia32 using electron v0.30.2
spawn wine ENOENT

npm ERR! Darwin 15.3.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run-script" "package"
npm ERR! node v4.4.3
npm ERR! npm  v2.15.1
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>package: `electron-packager ./ SoundMachine --all --out ~/Desktop/SoundMachine --version 0.30.2 --overwrite --icon=./app/img/app-icon.icns`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>package script 'electron-packager ./ SoundMachine --all --out ~/Desktop/SoundMachine --version 0.30.2 --overwrite --icon=./app/img/app-icon.icns'.
npm ERR! This is most likely a problem with the sound_machine package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     electron-packager ./ SoundMachine --all --out ~/Desktop/SoundMachine --version 0.30.2 --overwrite --icon=./app/img/app-icon.icns
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs sound_machine
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!
npm ERR!     npm owner ls sound_machine
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/michele.pasin/Dropbox/code/electron/sound-machine-electron-guide/npm-debug.log

I'm not sure if this will do it for you, but I had a similar issue and this worked for me.
Electron made updates since the tutorial was written. In your main.js file,  electron needs to be added the new way. I was having the same issue.
const electron = require('electron'); // Module to control application life. const {app} = electron; // Module to create native browser window. const {BrowserWindow} = electron;


nz-input-number does not show trailing zero

Reproduction link
https://stackblitz.com/edit/angular-j3gtkx?file=src%2Fapp%2Fapp.component.ts
Steps to reproduce

Add a 0 to the end of the input in the component HTML (3.5 -> 3.50).
Click out of the input box.
Notice the number changes from 3.50 back to 3.5.

What is expected?
Expect trailing zeros to conform to the specified nzPrecision value.
What is actually happening?
Trailing zeros are not displaying to meet nzPrecision requirement.

Environment
Info

ng-zorro-antd
15.0.2

Browser
Version 108.0.5359.124 (Official Build) (x86_64)

The use case for this is when dealing with a number that typically uses a trailing zero, such as a currency. $3.50 is normally displayed (not $3.5).

@skwny
[nzFormatter]="formatterDollar"

formatterDollar = (value: number): string => {
    if (String(value).split('.').pop().length !== 2) {
      return `${value}0`;
    } else {
      return `${value}`;
    }
};

@OriginRing has the right solution. Thank you 🙏


SCALE_BASE for LineItem and PartSupplier

The LineItemGenerator uses the OrderGenerator.SCALE_BASE constant (which is 1,500,000) and the PartSupplierGenerator uses the PartGenerator.SCALE_BASE (which is 200,000).  According to http://www.tpc.org/tpc_documents_current_versions/pdf/tpc-h_v2.17.3.pdf, these should be 6,000,000 (for LineItem) and 800,000 (for PartSupplier). any reason why the SCALE_BASE values differ from the spec?

Seems PartSupplier uses a magic "four suppliers per part" and the LineItem count is derived from the Orders. See #22 for my actual problem.


test: failed test - css parsing cause infinity loop

https://github.com/windicss/vite-plugin-windicss/issues/63

Reference https://github.com/windicss/windicss/pull/138


fix(fstream): fixes security vulnerability

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update meta.yaml

Adding the run_exports ensures that other packages depending on XTB, will propagate the build dependency further to the running environment. That is, one only needs to specify xtb in the host section of the meta.yaml.
For further details: https://docs.conda.io/projects/conda-build/en/latest/resources/define-metadata.html#export-runtime-requirements

Checklist

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

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

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

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

I am not sure why Travis fails after merging. I was under the impression that no builds would be run there after re-rendering...

I am not sure why Travis fails after merging. I was under the impression that no builds would be run there after re-rendering...

Looks like the connection was reset by Travis CI for the ppc64le build
("Connection broken: ConnectionResetError(104, 'Connection reset by peer')", ConnectionResetError(104, 'Connection reset by peer'))

Seems like we might have to retry this later...

Looks like the connection was reset by Travis CI for the ppc64le build
("Connection broken: ConnectionResetError(104, 'Connection reset by peer')", ConnectionResetError(104, 'Connection reset by peer'))

Seems like we might have to retry this later...


[Fix #3511] Style/TernaryParentheses false positive

Fix false positive for a ternary with ranges
Fixes #3511

[x] Wrote [good commit messages][1].
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.

The fix looks good, but it also needs a changelog entry.

done

:green_heart:

I'm new to this whole AST thing but empirically I realized that a braced expression has a :begin root node, while non-braced has anything but it... I will change to begin_type?
And all the previous tests pass, so it seems that I didn't break anything >.<

There's one small caveat. Something like x; y will also introduce a begin node, so ideally you should still check if this begin node starts with a ( and perhaps add a test for the x; y scenario. (begin nodes simply indicate there are more than 1 expression in a scope).

I've just checked x; y generates a :send

I've added a couple of tests. What else should I check for?

I've just checked x; y generates a :send root node

[2] pry(main)> Parser::CurrentRuby.parse("x; y")
=> s(:begin,
  s(:send, nil, :x),
  s(:send, nil, :y))

Doesn't look like a send root node to me.

Anyways, I recalled that in the context of a ? this won't matter as ? binds stronger than ;.

💚

👍

Great to see this fixed, just ran into this with this nice line:
        "cellar" => (cellar = bottle_spec.cellar).is_a?(Symbol) ? cellar.inspect : cellar,


fix: cli commands

Such CLI looks like the after the do process, it cleans up some left resources, similar examples https://github.com/helm/helm/issues/1920 and https://github.com/transferwise/pipelinewise/issues/573


remove missing folders; run as root so the...

remove missing folders; run as root so the go build can complete w/o write errors
Signed-off-by: Nick Boldt<EMAIL_ADDRESS>

apply this PR, then
podman build . to verify it builds

replaced by more complete #2


Allow narrow sidebar showing only icons without text - similar to pinned horizontal tabs.

Do you mean for pinned tabs or all tabs?

For all tabs. Aa square buttons. And show text on mouse-over either as tooltip or expanding the button. It would be selectable by another option next to the "Compact view" e.g. "Button view" - see image:

Unfortunately Chrome's side panel has a minimum width and can't be changed which prevents having this feature.


Post Issuance Check: Owner Ref

fixes #4788
/kind feature
/milestone v1.8
This PR creates a post issuance check for the owner reference on issued Certificate Secrets. If --enable-certificate-owner-ref is disabled, the issuing controller secrets manager will proactively ensure that the controller has not set an owner reference to all the issued Certificate's Secrets. Similarly, if  --enable-certificate-owner-ref is enabled, the secrets manager will proactively ensure all Certificate Secrets have the correct owner reference set.
Feature: Post Issuance Check: proactively ensure that owner reference on issued Secret's reflects the `--enable-certificate-owner-ref` controller flag.

Hi! As part of the effort of migrating from Bazel to Make, we have added two new Prow jobs that require this PR to be rebased on top of master in order to pass. The two Prow jobs that were added are pull-cert-manager-make-test and pull-cert-manager-make-e2e-v1-23.
Please let me know if you need help rebasing. Thanks! 🙏

@JoshVanL this is still marked WIP but is in the 1.8 milestone; any chance of this being completed or should I remove from the milestone?

@SgtCoDFish Hm, I think the WIP for this PR is no longer true. Taking a look through the code it looks as though it does all the objectives of the changes and it is all tested. I'll remove the WIP.

@SgtCoDFish I'll do quick rebase and get bazel hack tests to pass

The error in the make-e2e-v1-23 job is due to the 1.8.0-alpha.1 GitHub release that is still in draft. We will be able to re-run the job as soon as the version is published.
fyi @SgtCoDFish

/test pull-cert-manager-make-test


[13.3.x]: docs(material/tabs): Update public documentation with new mat-tab-nav-bar/panel guidance

Patch port of https://github.com/angular/components/commit/3a98da840de059e6f7ed5c65b81e28b6c95f26a8

LGTM


Chaining the same combine method causes a crash while mapping

Describe the bug
Running a query built by calling the same combine method called twice causes an error from within tranproc's combine (while trying to map the output).
To Reproduce
Fire up Postgres, createdb rom_bug_repro, and then run this script.
To see it work properly, comment out one of the .combine(:credit_card_purchase_charge) lines at the end, and the query completes.
It's not normal to have 2 identical combines chained like this, but in a large app, where they may be spread around different relation methods, it's indeed possible (this issue came from a real app of mine).
require "bundler/setup"
require "rom"
require "rom/sql"

# Configure database URL

config = ROM::Configuration.new(:sql, "postgres://localhost/rom_bug_repro")
config.plugin(:sql, relations: :auto_restrictions)

rom = ROM.container(config)

# Migrate database

rom.gateways[:default].tap do |gateway|
  migration = gateway.migration do
    change do
      drop_table? :credit_card_purchase_charges
      drop_table? :charges

      create_table :charges do
        primary_key :id
      end

      create_table :credit_card_purchase_charges do
        column :charge_id, :integer
        primary_key [:charge_id]
        index :charge_id
        column :card_name, :text
      end
    end
  end

  migration.apply gateway.connection, :up

  # Populate data

  gateway.connection.tap do |connection|
    connection.execute("INSERT INTO charges (id) VALUES (1)")
    connection.execute("INSERT INTO credit_card_purchase_charges (charge_id, card_name) VALUES (1, 'Jane Card')")
  end
end

# Define relations

module Relations
  class Charges < ROM::Relation[:sql]
    schema :charges, infer: true do
      associations do
        has_one :credit_card_purchase_charge
      end
    end
  end

  class CreditCardPurchaseCharges < ROM::Relation[:sql]
    schema :credit_card_purchase_charges, infer: true do
      associations do
        belongs_to :charge
      end
    end
  end
end

# Register relations and finalize rom container

config.register_relation Relations::Charges
config.register_relation Relations::CreditCardPurchaseCharges

rom = ROM.container(config)

class ChargeRepo < ROM::Repository::Root[:charges]
  def for_order
    charges
      .combine(:credit_card_purchase_charge)
      .combine(:credit_card_purchase_charge)
      .to_a
  end
end

# Instantiate repository and inspect output

repo = ChargeRepo.new(rom)
p repo.for_order

Output according to script above:
Traceback (most recent call last):
        19: from rom_bug.rb:82:in `<main>'
        18: from rom_bug.rb:75:in `for_order'
        17: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/bundler/gems/rom-7245b8cb6dad/core/lib/rom/relation/materializable.rb:15:in `to_a'
        16: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/bundler/gems/rom-7245b8cb6dad/core/lib/rom/relation/combined.rb:76:in `call'
        15: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/bundler/gems/rom-7245b8cb6dad/core/lib/rom/mapper.rb:97:in `call'
        14: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/bundler/gems/rom-7245b8cb6dad/core/lib/rom/mapper.rb:97:in `reduce'
        13: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/bundler/gems/rom-7245b8cb6dad/core/lib/rom/mapper.rb:97:in `each'
        12: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/bundler/gems/rom-7245b8cb6dad/core/lib/rom/mapper.rb:97:in `block in call'
        11: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/composite.rb:30:in `call'
        10: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/function.rb:47:in `call'
         9: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/function.rb:47:in `call'
         8: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/array.rb:123:in `combine'
         7: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/array/combine.rb:11:in `combine'
         6: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/array/combine.rb:33:in `group_nodes'
         5: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/array/combine.rb:33:in `map'
         4: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/array/combine.rb:33:in `each'
         3: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/array/combine.rb:33:in `each_with_index'
         2: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/array/combine.rb:33:in `each'
         1: from /Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/array/combine.rb:35:in `block in group_nodes'
/Users/tim/.asdf/installs/ruby/2.5.1/lib/ruby/gems/2.5.0/gems/transproc-1.0.2/lib/transproc/array/combine.rb:40:in `group_candidates': undefined method `[]' for nil:NilClass (NoMethodError)

Output when commenting out one of the combines:
[#<ROM::Struct::Charge id=1 credit_card_purchase_charge=#<ROM::Struct::CreditCardPurchaseCharge charge_id=1 card_name="Jane Card">>]

Expected behavior
The second combine is ignored or gracefully handled in some way, and the actual results are returned.
Your environment
n/a

This seems to work in #548
[ruby-2.6.3][node-v8.12.0] git(547-fix-dupped-combine)
➜  rom be ruby tmp/repros/547.rb
[#<ROM::Struct::Charge id=1 credit_card_purchase_charge=#<ROM::Struct::CreditCardPurchaseCharge charge_id=1 card_name="Jane Card">>]


Datatype 44

File: palm_cloth_01.vphys_c
It's in NTRO, but no instance of it.

Found more files:
models/heroes/undying/undying_flesh_golem.vphys_c
models/heroes/undying/undying.vphys_c
models/items/witchdoctor/bonkers_the_mad/bonkers_the_mad.vphys_c
models/courier/seekling/seekling.vphys_c


RemoteSSH to Remote Server No Longer Works

Type: Bug
Previously, I was able to connect to my HPC server at my school and connect with remote-ssh to use the remote file explorer, terminal, file editng etc. Recently (last night), I stopped being able to log in. Same credentials, same host target. I get an issue that says glibc and libstdc++ are not installed on the server.
Seems like it's associated with an update on VS Studio Code.
VS Code version: Code 1.86.0 (05047486b6df5eb8d44b2ecd70ea3bdf775fd937, 2024-01-31T10:28:19.990Z)
OS version: Windows_NT x64 10.0.22621
Modes:
1-Remote - SSH.log

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i5-1130G7 @ 1.10GHz (8 x 1805)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.73GB (5.77GB free)

Process Argv
--crash-reporter-id 9952cbc3-66eb-449e-991e-39dffd84a4cc

Screen Reader
no

VM
0%

Extensions (20)

Extension
Author (truncated)
Version

svg-viewer
Dhe
1.1.1

copilot
Git
1.156.0

copilot-chat
Git
0.12.0

remotehub
Git
0.62.0

vscode-gutter-preview
kis
0.30.0

debugpy
ms-
2024.0.0

python
ms-
2024.0.0

vscode-pylance
ms-
2023.12.1

jupyter
ms-
2024.1.0

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.17

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

remote-ssh
ms-
0.108.0

remote-ssh-edit
ms-
0.86.0

azure-repos
ms-
0.38.0

remote-explorer
ms-
0.4.1

remote-repositories
ms-
0.40.0

remote-server
ms-
1.5.0

wolfram
Wol
1.7.0

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscorecescf:30445987
vscod805:30301674
binariesv615:30325510
vsaa593:30376534
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
vscrpc:30673769
a9j8j154:30646983
962ge761:30951796
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
accentitlementsc:30887149
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
dsvsc019b:30953937
3ef8e399:30949928

/duplicate and expected to be fixed in upcoming recovery release

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Ddc 1385

I added a new feature for cache grouping, so please can you merge to the master branch

Why are you including @beberlei's commits in your branch ? They are part of a WIP branch (see #151) so I don't see why you should request a merge into master for them

Other point: your commit does not follow the Doctrine coding standards

btw, I don't see what is the relation between your change and DDC-1385

mmmm. I am sorry, This is my first time contributing with doctrine, but I am trying :)

I only change one file "lib/Doctrine/ORM/AbstractQuery.php"
so I think I did something wrong :(

It seems like you created your branch based on the WIP branch instead of the master branch.
Regarding coding standards, you need to use 4 spaces per level, not tabs.

You can try to start over by doing the following in your terminal:
git checkout master
git checkout -bQueryCacheTags
git cherry-pick a316c63
git cherry-pick b2921c6
git push origin QueryCacheTags

Thanks Beberlei, I did that :)


[3.0] Cache

This issue has been moved from the zendframework repository as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html

Original Issue: https://api.github.com/repos/zendframework/zendframework/issues/7211
User: @marc-mabe
Created On: 2015-02-12T19:03:23Z
Updated At: 2015-11-06T20:22:37Z
Body
I would like to "fix" some weird  logic from cache component.

Reduce arguments by reference
Move / Remove resource manager logic

the the cache storage adapter directly receives the basic resource (instances of Memcache, Memcached, MongoClient, Redis, MongoCollection, ...)
Where to move this logic to or should that be removed and leaving it up to the user?

Thoughts ?
ping @Ocramius @weierophinney

There are already issues open for this:

Move / Remove resource manager logic = #24
Reduce arguments by reference = #8


[DDC-3087] Add readonly attribute to Field ORM Mapping to avoid generating a setter

The change allows one to enter a field attribute called "readonly". What this does is when using doctrine:generate:entities, only a getter is generated, setter is omitted.
Example use on a created field that uses Timestampable and should not have a setter:
    /**
     * @var /DateTime $createdAt
     *
     * @Gedmo\Timestampable(on="create")
     * @ORM\Column(type="datetime", readonly=true)
     */
    private $createdAt;

due to the lack of multiple @GeneratedValue annotations, setting other columns to readonly would be an easy solution to take advantage of database features (without tinkering with lifecycle events).


Key Missing from DLT When Both Key and Value Deserialization Fails

When the key or value fails deserialization, both the value and key are populated in the DLT with the failing byte[].
When they both fail, only the value byte[] is populated; both should be populated.

Moved to 2.7 - can't change the behavior - the key exception is left in the header when both fail.


Fix: use default tls cert suite.

Description
We used to have this PR https://github.com/stolostron/backplane-operator/pull/471 to specify the cert suites because at that time ANP has a bug https://github.com/kubernetes-sigs/apiserver-network-proxy/issues/503.
Now the bug is fixed, we need to remove this line to use the default cert suite of tls.  Otherwise, in the latest environment, it will fail because it contains some unsupportted cert method.
Related Issue
https://issues.redhat.com/browse/ACM-11441
Changes Made
Removed a arg in cluster-proxy-addon configuration.
Screenshots (if applicable)
Add screenshots or GIFs that demonstrate the changes visually, if relevant.
Checklist

[x] I have tested the changes locally and they are functioning as expected.
[x] I have updated the documentation (if necessary) to reflect the changes.
[x] I have added/updated relevant unit tests (if applicable).
[x] I have ensured that my code follows the project's coding standards.
[x] I have checked for any potential security issues and addressed them.
[x] I have added necessary comments to the code, especially in complex or unclear sections.
[x] I have rebased my branch on top of the latest main/master branch.

Additional Notes
Add any additional notes, context, or information that might be helpful for reviewers.
Reviewers
Tag the appropriate reviewers who should review this pull request. To add reviewers, please add the following line: /cc @reviewer1 @reviewer2
Definition of Done

[x] Code is reviewed.
[x] Code is tested.
[ ] Documentation is updated.
[ ] All checks and tests pass.
[ ] Approved by at least one reviewer.
[ ] Merged into the main/master branch.

/assign @dislbenn
Hi, @dislbenn , please take a look at this when you have time, thanks!

/lgtm

/cherry-pick backplane-2.5

/cherry-pick backplane-2.4

/cherry-pick backplane-2.3


Event Request code unit 398 Sales Tax Calculate OnAfterCallExternalTaxEngineForDoc

Please provide more details

As no details provided, we cannot execute this request.


Multi-line array definitions are not re-indented

Using version 1.10, this code
        $this->assertEquals(
            [
            'lowUpperBound'        => '50',
            'highLowerBound'       => '90',
            'coverage-html'        => '/tmp/report',
            'coverage-clover'      => '/tmp/clover.xml',
            'json'                 => '/tmp/logfile.json',
            'plain'                => '/tmp/logfile.txt',
            'tap'                  => '/tmp/logfile.tap',
            'logIncompleteSkipped' => false,
            'junit'                => '/tmp/logfile.xml',
            'testdox-html'         => '/tmp/testdox.html',
            'testdox-text'         => '/tmp/testdox.txt',
            ],
            $this->configuration->getLoggingConfiguration()
        );

in PHPUnit's own test suite is not reformatted as
        $this->assertEquals(
            [
                'lowUpperBound'        => '50',
                'highLowerBound'       => '90',
                'coverage-html'        => '/tmp/report',
                'coverage-clover'      => '/tmp/clover.xml',
                'json'                 => '/tmp/logfile.json',
                'plain'                => '/tmp/logfile.txt',
                'tap'                  => '/tmp/logfile.tap',
                'logIncompleteSkipped' => false,
                'junit'                => '/tmp/logfile.xml',
                'testdox-html'         => '/tmp/testdox.html',
                'testdox-text'         => '/tmp/testdox.txt',
            ],
            $this->configuration->getLoggingConfiguration()
        );

as I would expect.
Am I missing a rule (https://github.com/sebastianbergmann/phpunit/blob/5.0/.php_cs is the configuration I use) or is there an issue with a rule that I use?

This should be done by the align_double_arrow which is in the .php_cs file of the PHPUnit repo, so it is not a configuration issue.
The fixer was update about 12 days ago, did you have the issue with the fixer before that as well?

I only noticed this today and have not updated my php-cs-fixer since July (I am using the latest PHAR release, v1.10; sorry for not mentioning that in the original report).

Having the same problem with latest 1.11-DEV.

Thanks for the details :+1:
I'll have look soon.

Yes, indentation is missing but that is about spaces vs. tabs anyways, right?

align_double_arrow should just add spaces before the =>s in an array to align them. indentation is responsible for ensuring blocks have the correct indentation level.

There is no fixer yet to align array's content. indentation won't do that.
It's not a bug, but simply lack of feature.

Any update on this? Would be nice if php-cs-fixer could fix this. Thanks!

I'm :+1: for that feature, it would be really nice.
Sadly, I don't have time to create it soon, maybe you want to help with it?

A nice feature

A nice feature

No one working on this feature?

@tonivdv AFAIK there is no work on this ATM, maybe you can help out?

@SpacePossum I'll see what I can do.

Has there been follow-up on this? The deprecated PSR-2 indentation rule used to auto-fix array spaces but replacement rule indentation_type for @PSR2 or @Symfony mode doesn't affect array spaces.
Specifically I want to auto-fix Laravel PHP array-based files translation files.
Package "barryvdh/laravel-translation-manager" outputs 2 spaces so I've used php-cs-fixer to enforce consistent 4 spaces indentation. See pull request proposed for that project: https://github.com/barryvdh/laravel-translation-manager/pull/173
<?php

return array(
  'first-name' => 'First Name',
  'last-name' => 'Last Name',
  'language' => array(
    'en' => 'English',
    'es' => 'Spanish',
  ),
);

to
<?php

return [
    'first-name' => 'First Name',
    'last-name' => 'Last Name',
    'language' => [
        'en' => 'English',
        'es' => 'Spanish',
    ],
];

indentation fixer was enforcing one to use 4 spaces over tabs.
it was indeed replaced by indentation_type, which is configurable and is enforcing to use 4 spaces of tab.
yet, it is not supposed to reindent arrays. we never have such a fixer. this file is a feature request, yet it didn't met much support from community.
If you also want to have this feature, please send PR proposal :+1:

Closing due to amazing work of #3135 by @julienfalque
that was a long life of this feature request indeed.... glad to finally close it!


[dependencies] update to React ^15.5.0 and use prop-types

This PR bumps React to ^15.5.0 across all packages and updates to use prop-types in the @data-ui/data-table package. Still seeing some warnings in dev mode.

Coverage remained the same at 78.306% when pulling b2468e03f235a7fa01f24908ada39a1bc205aa46 on chris--react-v into 9aa7edf1993a954bec8c77e1f8b18038bb600a94 on master.

Coverage remained the same at 78.306% when pulling 25352a05a090809df5c6442a6717dbd7e35f81ce on chris--react-v into 9aa7edf1993a954bec8c77e1f8b18038bb600a94 on master.


Handle user song information questions

When a user asks something like What is the title of the current song?, the bot should be able to respond by saying something like This song is Viva La Vida.
Needed:

LUIS model with intents to recognise the type of question (what information is being requested?)
Functions to retrieve information concerning the current song
...

Functions implemented that read status information from the cmus music player, e.g. current song title, artist, album ...

LUIS model implemented to recognise information request for song title, album, artist, year.

Works primitively.


Save raw output data from execution events

Signed-off-by: Katrina Rogan<EMAIL_ADDRESS>TL;DR
This change handles receiving raw output data for successful execution events and saving that inline. As part of this change, we also update the observations for output data size and can use this to inform whether to offload output data in the future if the size ends up being particularly large in practice and affects db performance.
Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[ ] Smoke tested
[x] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
https://github.com/flyteorg/flyte/issues/1327
Follow-up issue
NA

tested with a flyte deployment with propeller set to send event data both inline and by reference and verified that calling Get*ExecutionData return the full inputs and outputs for both


Gpt 3.5 SWE-Agent results

Describe the issue
Hi, is there a results.json for GPT 3.5 on SWE-bench Lite that I could see?
Mostly interested in the # resolved / 300.
Thanks!
Optional: Relevant documentation page
No response

nope it probably wont work well with 3.5


Segmentation fault (core dumped) when running hello world example for desktop on WSL

I've followed the installation steps on WSL. But when I ran the hello world example for desktop with the command I found in MediaPipe docs : bazel run --define MEDIAPIPE_DISABLE_GPU=1
mediapipe/examples/desktop/hello_world:hello_world. The build completed successfully and there is a segmentation fault (core dumped) without any error :

Configuration :
WSL : Ubuntu 20.04 LTS
GPU : Intel UHD Graphics 600
Bazel : 2.0.0
gcc : 9.3.0
g++ : 9.3.0
opencv : 4
ffmpeg : 4.2.2

Thanks for reporting this issue. While we are still working on a proper way to fix it, here is a temporarily workaround:
diff --git a/mediapipe/framework/legacy_calculator_support.h b/mediapipe/framework/legacy_calculator_support.h
index 6a76101..07475e0 100644
--- a/mediapipe/framework/legacy_calculator_support.h
+++ b/mediapipe/framework/legacy_calculator_support.h
@@ -41,13 +41,11 @@ class LegacyCalculatorSupport {
     // The constructor saves the current value of current_ in an instance
     // member, which is then restored by the destructor.
     explicit Scoped(C* cc) {
-      saved_ = current_;
-      current_ = cc;
     }
-    ~Scoped() { current_ = saved_; }
+    ~Scoped() {}

     // The current C* for this thread.
-    static C* current() { return current_; }
+    static C* current() { return nullptr; }

    private:
     // The value to restore after exiting this scope.

The solution is adding the following code to mediapipe/framework/legacy_calculator_support.h at line 68.
template <>
thread_local CalculatorContext*
    LegacyCalculatorSupport::Scoped<CalculatorContext>::current_;
template <>
thread_local CalculatorContract*
    LegacyCalculatorSupport::Scoped<CalculatorContract>::current_;

The fix is released in https://github.com/google/mediapipe/commit/e9fbe868e55fa23aaabc31f9f847c22287062850.


[wip stretch goal] Live stream discovery

Hey everyone, Justin and I wanted to get one more thing done before my time in cape town is over (sniff), and so we thought about adding a lil bit of discovery to the live streams. Currently you can only find live streams if you go straight to a project, no deep-linking or push notifications yet :/
So, we just added a lil "Kickstarter Live" entry to the discovery filters:

Tapping that takes you to a feed of live, upcoming and recent live streams:

Tapping those cards either goes to the countdown, live stream or reply.
NB
This PR has a few things that I wanna call out:

We are now hitting two api endpoints on the live api: one to get a list of events and one to get a single event. these two endpoints return subtly different json unfortunately and so i had to do a bit of gymnastics to accommodate for this. i'll call out the lines in my self-review.

Due to the above, i had to change the live stream models quite a bit, which then wreaked havoc on the lenses. So, I wanted to try something new to ease the creation of lenses for us (at least until we get code gen). If we set all of the fields of a struct to public private(set) var field, we still get to act like we are immutable to the outside world, but inside this file only we can make lenses like:

extension Model {
  public enum lens {
    public static let field = Lens<Model, Int>(
      view: { $0.field },
      set: { var new = $1; new.field= $0; return field; }
    )
  }
}

not only is it less code, but it doesnt break when we add new fields. whatdya'll think?

hey @awakinglim mind weighing on proper phrasing for Live in the discovery overlay access point? Curious about balance of brand / UX / translation considerations. This will show cards for live, upcoming and recent streams.

@ktman I see the value in both options — leaving as Kickstarter Live and changing to a more direct Live streams label.
The problem with the latter is that we haven't translated the KSLive marketing page or web interface yet, so for parity throughout I'd like to go with the former and leave as is. It'll allow us to really own the feature, and once the user taps it'll be very clear what they're looking at.

not only is it less code, but it doesnt break when we add new fields. whatdya'll think?

makes sense to me re: those lenses!


Does this task work on non-windows agents?

Not 100% sure if non-windows OS is supported?
Running through azure devops yaml piplines, with 'ubuntu-latest'.
Gives the below.

It does not. The task is built with Powershell and the Linux and Mac agents
don't support tasks written in powershell as long as Microsoft doesn't add
a powershell core executionhandler.
You could add a 2nd job to your pipeline, have it run Windows and do the
promotion from there as a workaround.
On Wed, Jun 3, 2020, 07:36 btrepp<EMAIL_ADDRESS>wrote:

Not 100% sure if non-windows OS is supported?
Running through azure devops yaml piplines, with 'ubuntu-latest'.
Gives the below.
[image: image]
https://user-images.githubusercontent.com/262933/83599549-0cb82800-a59f-11ea-8b88-835662cfd290.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/renevanosnabrugge/vsts-promotepackage-task/issues/57,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA724S7FC53JKQXJOGPHLILRUXOONANCNFSM4NRKXGJQ
.

@jessehouwing  Maybe we can add support for PowerShell Core? You can run the 'normal' PowerShell@2 task with the input pwsh: true.

@jessehouwing  Maybe we can add support for PowerShell Core? You can run the 'normal' PowerShell@2 task with the input pwsh: true.

No, it's not easy to convert this extension to pwsh-core. The agent initializes the context for the task and for the SDK to work. The agent has 3 modes:

powershell runtime hosted. The agent will actually host the powershell code. This mode is deprecated since October 2019.
powershell3 is an out of process host which spins up a powershell instance and has it execute the task.
Node/Node10 is similar to powershell but fires up a Node 6 or Node 10 runtime to execute the task.

Only the Node/Node10 runtime is supported on non-windows-agents. So to enable this task on Linux, the task needs to be ported to a task host that is cross-platform. Maybe a thin wrapper in Node could in turn invoke pwsh, but that introduces a bunch of new headaches around return codes, error stream handling and such.


demo photos in README are broken.

The first few photos in the README won't load.

In my browser all looks fine. Check can you see this gif https://github.com/Sk0rpion/MagicPie/blob/master/MagicPie.gif?raw=true. If no, please check this gif url https://raw.githubusercontent.com/AlexandrGraschenkov/MagicPie/master/MagicPie.gif.

I think it was github's problem. After I posted the issue, none of them were working for me. Now they are all working again.


Migrated NeoAutocomplete to tailwindcss

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[x] Refactoring

Context

[x] Closes #8335
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[x] I've tested it at </ksm/collection>
[x] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Ecosystem Telegram?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖[deprecated] Generated by Copilot at 47da901
Refactor NeoAutocomplete.scss to use TailwindCSS instead of custom CSS properties and remove unnecessary import.

🤖[deprecated] Generated by Copilot at 47da901

variable.scss
Gone with the autumn wind
TailwindCSS shines

thanks!
pay 20 usd

😍 Perfect, I’ve sent the payout
💵 $20 @ 5.55 USD/DOT ~ 3.604 $DOT
🧗 16SpvdDgKNiQ3c53DxX7refnQcKUD3uRNim3Z1HBJLNGrtSP
🔗 0x78c5e7682b3bbe69a30d770cc1756e85e0e136767b02a753c7b3e8851f0b1c2b
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


GH action - auto-update PRs

What does this PR do?
This shall update PR which is ready-to-go but need to be tested in action as not sure how it will act if there are some collisions, and if it works we can also drop a comment message
see: https://github.com/marketplace/actions/auto-update
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified
[x] Check that target branch and milestone match!

Did you have fun?
Make sure you had fun coding 🙃

"update" branch means it will trigger CI too? that would be very costly considering drone takes almost 20 mins per run

"update" branch means it will trigger CI too? that would be very costly considering drone takes almost 20 mins per run

"update" branch means it will trigger CI too? that would be very costly considering drone takes almost 20 mins per run

it will work only with ready-to-go label, as @SeanNaren asked for :]

"update" branch means it will trigger CI too? that would be very costly considering drone takes almost 20 mins per run

it will work only with ready-to-go label, as @SeanNaren asked for :]


make NodeJS REPL get correct randomBytes()

NodeJS REPL imports crypto by default, this breaks the original logic.

Thanks for the fix!


yt-dlp using --netrc option to download from Twitter is yielding .mp4 files with no audio, video is playing

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
US
Provide a description that is worded well enough to be understood
Use yt-dlp to download the specified mp4 video from Twitter.
Observe that while video plays, no audio is heard.
https://twitter.com/vision4theblind/status/1733837664673988751
Others were noted, including: https://twitter.com/Thekeksociety/status/1733670116473319587
(All instructions to update to nightly or master failed, so this is reported against 2023.11.16)
Provide verbose output that clearly demonstrates the problem

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

Complete Verbose Output
% yt-dlp --netrc "https://twitter.com/vision4theblind/status/1733837664673988751" -o "Comedy central illuminati commercial.mp4" -vU
[debug] Command-line config: ['--netrc', 'https://twitter.com/vision4theblind/status/1733837664673988751', '-o', 'Comedy central illuminati commercial.mp4', '-vU']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [24f827875] (zip)
[debug] Python 3.11.6 (CPython arm64 64bit) - macOS-14.1.1-arm64-arm-64bit (OpenSSL 3.2.0 23 Nov 2023)
[debug] exe versions: ffmpeg 6.0 (setts), ffprobe 6.0
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2023.05.07, mutagen-1.46.0, requests-2.31.0, sqlite3-3.44.2, urllib3-2.0.3, websockets-11.0.3
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests
[debug] Loaded 1901 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[twitter] Downloading login page
[twitter] Downloading flow token
[twitter] Submitting JS instrumentation response
[twitter] Submitting username
[twitter] Submitting password
[twitter] Submitting account duplication check
[twitter] Logging in
[twitter] Extracting URL: https://twitter.com/vision4theblind/status/1733837664673988751
[twitter]<PHONE_NUMBER>673988751: Downloading GraphQL JSON
[debug] [twitter] Extracting from video info:<PHONE_NUMBER>581247488
[twitter]<PHONE_NUMBER>673988751: Downloading m3u8 information
[debug] Sort order given by extractor: res, br, size, proto
[debug] Formats sorted by: hasvid, ie_pref, res, br, size, proto, lang, quality, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, asr, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info]<PHONE_NUMBER>581247488: Downloading 1 format(s): http-2176
[debug] Invoking http downloader on "https://video.twimg.com/ext_tw_video/1733837595581247488/pu/vid/avc1/576x1024/nYXy56Qvd1JPgVI_.mp4?tag=12"
[download] Destination: Comedy central illuminati commercial.mp4
[download] 100% of    5.70MiB in 00:00:00 at 89.29MiB/s

Update: followed up on the URL emitted in 3rd to last line of debug output.
https://video.twimg.com/ext_tw_video/1733837595581247488/pu/vid/avc1/576x1024/nYXy56Qvd1JPgVI_.mp4
no audio plays on this in the browser
I claim there must be other .mp4 files which could be found to return here …

Confirm I have the same issue with Twitter.
Although free online ones like ssstwitter(dot)com are doing the same thing, no audio.
Also trying to send a video saved this way via WhatsApp errors out saying "This type of video can't be sent" when its literally .mp4
@bashonly

I have this problem too. The downloaded videos work in Firefox but VLC can't play the audio.
As a temporary workaround, the files can be fixed with FFmpeg by running ffmpeg -i [input] -c copy [output].

So does specifying FFmpeg as (external-)downloader result in good media?

So does specifying FFmpeg as (external-)downloader result in good media?

Yeah it does. Though I don't think we can force ffmpeg as external FD for http formats, nor can we force a fixup for them (without modifying core code).
I think the best solution is to prioritize HLS formats.
This is a dupe of #8117


Make CUDA integration work again on the new API

Describe the bug
OpenGL backend no longer plays nicely with OpenCL and CUDA
Background
https://github.com/Devsh-Graphics-Programming/Nabla/pull/28#discussion_r561240809
Environment
OpenGL with Windows and X11
ES and Vulkan not necessary.

CUDA only now.

#517 in progress