 remote path doesn't exist

I occasionally get this error when pushing a directory with subdirectories that have to be created. At least, that seems to be in common. I'm not sure if anything bad actually happens as a result as I'm doing uploads with lots of files and there's no indicator of what remote path doesn't exist. Maybe something is slow at taking note of a newly created directory at times?
Great software, very useful!

What version of drive?

An issue that I could guess that is related to it is here: https://github.com/odeke-em/drive/issues/46

I'm currently using 0.0.7, but I've seen this as well with 0.0.6. I'm currently doing drive push directory, which contains subdirectories, with subdirectories, and then files. You can see from the output below that it sort of happens sporadically. I think it would be useful if instead of just remote path doesn't exist it showed what path that was actually. In the files I'm uploading there are broken symlinks. Could this potentially have something to do with it? I'm a little unclear with drive is trying to do with them, other than the net effect that they don't get uploaded.
546 / 1709 [===============>---------------------------------] 31.95 % 10h54m23s
<nil> remote path doesn't exist
<nil> remote path doesn't exist
<nil> remote path doesn't exist
<nil> remote path doesn't exist
<nil> remote path doesn't exist
<nil> remote path doesn't exist
553 / 1709 [===============>---------------------------------] 32.36 % 10h42m17s
<nil> remote path doesn't exist
<nil> remote path doesn't exist
<nil> remote path doesn't exist
556 / 1709 [===============>---------------------------------] 32.53 % 10h37m10s
<nil> remote path doesn't exist
557 / 1709 [===============>---------------------------------] 32.59 % 10h35m30s
<nil> remote path doesn't exist
558 / 1709 [===============>---------------------------------] 32.65 % 10h33m50s
<nil> remote path doesn't exist
559 / 1709 [================>--------------------------------] 32.71 % 10h32m11s
<nil> remote path doesn't exist
560 / 1709 [================>--------------------------------] 32.77 % 10h30m32s
<nil> remote path doesn't exist
561 / 1709 [================>--------------------------------] 32.83 % 10h28m53s
<nil> remote path doesn't exist
562 / 1709 [================>--------------------------------] 32.88 % 10h27m14s
<nil> remote path doesn't exist
563 / 1709 [================>--------------------------------] 32.94 % 10h25m36s
<nil> remote path doesn't exist
564 / 1709 [================>--------------------------------] 33.00 % 10h23m59s
<nil> remote path doesn't exist
565 / 1709 [================>--------------------------------] 33.06 % 10h22m21s
<nil> remote path doesn't exist
566 / 1709 [================>--------------------------------] 33.12 % 10h20m44s
<nil> remote path doesn't exist
567 / 1709 [================>---------------------------------] 33.18 % 10h19m8s
<nil> remote path doesn't exist
1527 / 1709 [===========================================>-----] 89.35 % 1h29m56s

Thank you for reporting this. While doing a regression test I also noticed it. I am working on it right now. Watch this space :)

@guruthree thanks for the patience, would you mind testing out 482add9c699a5dadd5ef39b202a71b53a13d2b9b to see if this fixes it for you?

I can test it, but I'm not sure how. My knowledge of go extends to go get -u github.com/odeke-em/drive/cmd/drive and I'm not sure how to adapt that to a branch.

No biggie,  please checkout PR #51 I have put up the instructions on how to get to it.

Thanks. It seems to be working so far. When I tried doing drive push -ignore-checksum=true it still tried to checksum files, but with the usual drive push -no-prompt=true -ignore-checksum=true that I've been it did not (as expected).

Hmm, I'll try to investigate that separately? Otherwise, it solves the initial, remote not existant problem? Would you mind providing the feedback on the PR page for the sake of record keeping in relation to the original problem? Thank you.

Looks like the initial problem is solved. I don't know about the checksum thing as it's only happened the one time. I'll start a new issue if it happens again.

Version 0.0.7
Same Problem with  remote path doesn't exist
Applied patch (#51) and no longer getting error message about regarding remote path.
I am now getting googleapi: Error 400:: Parent is not a folder:  ...
Thank you for you time in this!

@dbesancon weirdly it was working until today mid afternoon. I did all things possible even rolled back to ages ago. I kept checking and testing for about one hour and a half, so I suspect an infrastructure problem(I could totally be wrong). I logged a heads up so will wait for them to get back
https://twitter.com/odeke_et/status/559441879733788673

@dbesancon I have found that it was an update by the google golang api https://github.com/google/google-api-go-client/commit/8a098771176331fd2afc6d135240402617dc0f5e that caused the code to break. If you would like to test out the code, go to the last change before it started breaking by:
$ cd $GOPATH/src/google.golang.org/api
$ git reset --hard 3f7dcd0e7a9cbcc8199def5ec2b0be34261ff67c
$
$ cd $GOPATH/src/github.com/google/google-api-go-client/
$ git reset --hard 3f7dcd0e7a9cbcc8199def5ec2b0be34261ff67c

In the meantime while I dig through the Google API and find out more about what tripped things out.

Fixed it by migrating to a curated and stable repo of the Google go client that I know won't be rapidly changing and breaking changes here. You can repeat the patch steps and should be able to test things out.


Fix Maven build

To build plugins for Openfire 4.3.0 pom.xml should include repositories tags pointing to Archiva. If plugin has pages in Admin Console, web.xml should also be added.
For references:
https://github.com/igniterealtime/openfire-clientControl-plugin/pull/11

Fixed in https://github.com/igniterealtime/openfire-externalservicediscovery-plugin/pull/3


77 delete button

I like the approach but this delete component is not re-usable, my suggestion is that you can make a new component under components folder, add the "delete button" there with the modal as well


Upload Images[Feature]

can I upload Images to chat-gpt

this one must use own external storage I think such as sync feature
(in scenario) :
not sure if its good for adding this feature, because of CORS Policy. (eg you can't export the messages as image in exporter page)
because after image is uploaded in external storage then AI Embedding it with functional calling.

No, OpenAI does not have the capability to open this


Version 0.15.2 failes to build on Debian:

Hi!  go install fails for me.  Here is the output where the error occurs.  It's possible I've done something wrong (missing dependency, for example?), or something else.
github.com/zaquestion/lab/cmd
# github.com/zaquestion/lab/cmd
src/github.com/zaquestion/lab/cmd/ci_run.go:134:13: ciCreateCmd.MarkZshCompPositionalArgumentCustom undefined (type *cobra.Command has no field or method MarkZshCompPositionalArgumentCustom)
src/github.com/zaquestion/lab/cmd/ci_run.go:138:14: ciTriggerCmd.MarkZshCompPositionalArgumentCustom undefined (type *cobra.Command has no field or method MarkZshCompPositionalArgumentCustom)
src/github.com/zaquestion/lab/cmd/ci_status.go:87:13: ciStatusCmd.MarkZshCompPositionalArgumentCustom undefined (type *cobra.Command has no field or method MarkZshCompPositionalArgumentCustom)
src/github.com/zaquestion/lab/cmd/ci_trace.go:111:12: ciTraceCmd.MarkZshCompPositionalArgumentCustom undefined (type *cobra.Command has no field or method MarkZshCompPositionalArgumentCustom)
src/github.com/zaquestion/lab/cmd/ci_trace.go:112:12: ciTraceCmd.MarkZshCompPositionalArgumentCustom undefined (type *cobra.Command has no field or method MarkZshCompPositionalArgumentCustom)
src/github.com/zaquestion/lab/cmd/ci_view.go:628:11: ciViewCmd.MarkZshCompPositionalArgumentCustom undefined (type *cobra.Command has no field or method MarkZshCompPositionalArgumentCustom)
src/github.com/zaquestion/lab/cmd/ci_view.go:629:11: ciViewCmd.MarkZshCompPositionalArgumentCustom undefined (type *cobra.Command has no field or method MarkZshCompPositionalArgumentCustom)
src/github.com/zaquestion/lab/cmd/issue_browse.go:48:16: issueBrowseCmd.MarkZshCompPositionalArgumentCustom undefined (type *cobra.Command has no field or method MarkZshCompPositionalArgumentCustom)
src/github.com/zaquestion/lab/cmd/issue_browse.go:49:16: issueBrowseCmd.MarkZshCompPositionalArgumentCustom undefined (type *cobra.Command has no field or method MarkZshCompPositionalArgumentCustom)
src/github.com/zaquestion/lab/cmd/root.go:26:2: unknown field 'ZshCompletionFunction' in struct literal of type cobra.Command
src/github.com/zaquestion/lab/cmd/issue_browse.go:49:16: too many errors

Best wishes,
Julian

@juliangilbey You'll need to make sure to do GO111MODULE=on go install ., you can see travis and the Makefile for reference.


Old versions used in Dockerfile (base image, maven, cmake, sdk/ndk tools, etc..)

Old base Ubuntu 20.10 used which is EOL.
https://github.com/ACINQ/phoenix/blob/67e5a9754d92dc58d3082afa42e4918049244b05/Dockerfile#L33

Ubuntu 20.10 is EOL in: July, 2021
Use 21.04 instead.

Old base adoptopenjdk/openjdk11:jdk-11.0.8_10-alpine

https://github.com/ACINQ/phoenix/blob/67e5a9754d92dc58d3082afa42e4918049244b05/Dockerfile#L2
latest version is: jdk-11.0.11_9-alpine
There also slim version which is better to use if possible: jdk-11.0.11_9-alpine-slim

Old Maven version 3.6.3

https://github.com/ACINQ/phoenix/blob/67e5a9754d92dc58d3082afa42e4918049244b05/Dockerfile#L8
latest version of maven is: 3.8.1

Old SDK tools:

https://github.com/ACINQ/phoenix/blob/67e5a9754d92dc58d3082afa42e4918049244b05/Dockerfile#L38-L39
The code need to be changed as link changed to name commandlinetools-linux:
https://dl.google.com/android/repository/commandlinetools-linux-7302050_latest.zip
from:
https://developer.android.com/studio/index.html#command-tools

Old Cmake

https://github.com/ACINQ/phoenix/blob/67e5a9754d92dc58d3082afa42e4918049244b05/Dockerfile#L43
newest from android sdk tools: 3.18.1

Old BuildTools

https://github.com/ACINQ/phoenix/blob/67e5a9754d92dc58d3082afa42e4918049244b05/Dockerfile#L41
newest version is: 31.0.0

Old NDK:

https://github.com/ACINQ/phoenix/blob/67e5a9754d92dc58d3082afa42e4918049244b05/Dockerfile#L42
https://developer.android.com/ndk/downloads
Latest LTS Version: 21.4.7075529
Latest Stable Version: 22.1.7171670

We've been working on migrating Phoenix to a completely new stack based on Kotlin multi-platform:

https://github.com/acinq/lightning-kmp
https://github.com/acinq/phoenix-kmm

Any work on deterministic builds will be done over there.

Fixed with https://github.com/ACINQ/phoenix/commit/0ff688fc014cc94806daca93799821777c456e7f
Note that this repo is the main repository again, the phoenix-kmm one is deprecated.


Refactor workspace settings

Details

Refactoring workspace settings form to use Form.js and dismiss keyboard after form submission.
Fixed Issues

$ https://github.com/Expensify/App/issues/11195
$ https://github.com/Expensify/App/issues/11195#issuecomment-1263410831
Tests

UI looks as it did before the refactor
Values can be added and edited
Errors are highlighted correctly (input border)
Error messages show up correctly
Form alerts are displayed correctly
Clicking the fix the errors link focuses the first input with error
No duplicate submission of the form occurs (when it's already submitting)

Saving a workspace name should close the keyboard

Go to the workspace section
Click on the workspace name
Form input will come to address the change in the name
Once the desired name is input, press the save button
On the native app and mWeb verified that the keyboard is closed after pressing the save button

[x] Verify that no errors appear in the JS console

QA Steps

UI looks as it did before the refactor
Values can be added and edited
Errors are highlighted correctly (input border)
Error messages show up correctly
Form alerts are displayed correctly
Clicking the fix the errors link focuses the first input with error
No duplicate submission of the form occurs (when it's already submitting)

Saving a workspace name should close the keyboard

Go to the workspace section
Click on the workspace name
Form input will come to address the change in the name
Once the desired name is input, press the save button
On the native app and mWeb verified that the keyboard is closed after pressing the save button

[x] Verify that no errors appear in the JS console

PR Review Checklist

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] iOS / native
[x] Android / native
[x] iOS / Safari
[x] Android / Chrome
[x] MacOS / Chrome
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product was added in all src/languages/* files
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

PR Reviewer Checklist
The reviewer will copy/paste it into a new comment and complete it after the author checklist is completed

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots

Web

https://user-images.githubusercontent.com/25520267/194132341-6104c3c5-3daf-4b9f-bfd8-a0b407048a29.mov
Mobile Web - Chrome

https://user-images.githubusercontent.com/25520267/194132064-0f2b9e74-49ff-427e-bd38-fdec5bfc2c2a.mov
Mobile Web - Safari

https://user-images.githubusercontent.com/25520267/194132325-02ffceb8-b3d8-46fd-9292-321f98224807.mov
Desktop

https://user-images.githubusercontent.com/25520267/194132130-26b35d1a-3760-44b7-bb90-6b2e7133db4f.mov
iOS

https://user-images.githubusercontent.com/25520267/194132155-422f49cf-2691-45d8-b962-64ff050c94ce.mov
Android

https://user-images.githubusercontent.com/25520267/194132170-f68b201c-c128-4610-a29d-fe3d0bd881f0.mov

@rushatgabhane I updated the PR and also reverted the Form.js changes because the issue is fixed here.

@rushatgabhane I just pulled the latest main, it should be fixed the issue by clicking the error text to focus the input.

@rushatgabhane Added the original issue test and update the checklist with the latest template!

Tests passed. Removing Emergency label.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @luacmartins in version: 1.2.19-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @chiragsalian in version: 1.2.19-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


SavePointsToPLY creates faces

When exporting testing point clouds via SampleVisibleMeshSurface, the function SavePointsToPLY generates faces, which I think is not the intended behaviour. Also, there are some mislabellings for the CLI11 options (see -s).
I can send a pull request to fix these minor issues, if you want to.

The -s flag is used for the number of samples is just because -n was already taken.
As for the faces, generating those actually is intended, just because on at least one system we tested we were unable to load .ply files using trimesh which contained only points (no faces). It seems trimesh was deleting unreferenced vertices, which was all of them, so the SavePointsToPLY function adds one face referencing each vert. This is annoying for visualization because the connectivity is random (e.g. in meshlab you'll want to render only the points). There is probably a better fix, but for now this avoids having the points deleted and the evaluation code ignores the faces anyway.

I see, thanks for the clarification!


Add project management bot ability

It seems, as if this approach will not work very well. There are too many intents and defining different interactions with all these intents in stories, especially also considering unhappy paths would be very cumbersome to implement.
The recommended way is to implement dynamic domain data (such as our project list) as a knowledge base. The NLU training data then needs to only add a single intent (query_knowledge_base) for a large variety of different requests. All of which can be handled by the knowledge base deterministically. The actual list or attribute querying will be provided by a set of entities detected via NLU.


CDPS-1039: Reducing prisoner search timeout to 30s

Also taking the opportunity to rename the auth clients to represent the fact that we're sharing the same creds for hitting multiple APIs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.55%. Comparing base (3825e5d) to head (a9fd819).

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #135      +/-   ##
============================================
+ Coverage     85.53%   85.55%   +0.02%
+ Complexity      694      693       -1
============================================
  Files           105      105
  Lines          2205     2209       +4
  Branches        207      207
============================================
+ Hits           1886     1890       +4
  Misses          189      189
  Partials        130      130

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Make Copilot Chat View searchable

Add a "find" function, same as in the text editor, so the user can search for text in the chat window

Agree, currently you can only scroll through the chat window and try to find things, but if you're asking it to generate codes the response would be very long and hard to locate certain texts.


patchelf v0.12

On ppc64le I noticed patchelf threw a warning "patchelf: maximum file size exceeded". It is supposed to be fixed in v0.12 (https://github.com/NixOS/patchelf/issues/222#issuecomment-682489591), hence I am sending this PR. I am not sure if the warning has anything to do with the bug report (corrupted cuDNN shared libraries: https://github.com/conda-forge/cudnn-feedstock/issues/31), but the changelog for v0.12 also states
- Improved the default section alignment choice.

So let's give it a try.
v0.13 just came out a few days ago. I will add it later.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] 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.

Hmmm the CIs are not launched?
@conda-forge-admin, please rerender

Hmmm the CIs are not launched?

It was due to an accident: https://www.githubstatus.com/incidents/rmfrw9dfbtbp

@conda-forge/patchelf @conda-forge/core This is ready.

@conda-forge/patchelf LIEF does not fix cuTENSOR (https://github.com/conda-forge/cutensor-feedstock/pull/17#issuecomment-897073339) so we really need to give the updated patchelf a shot...

@leofang Do you also want to join as a maintainer?
I'd be happy to be listed as well but this won't help immediately as I'm away from a laptop for a week and stuck with the Github mobile app 😉

OK @xhochy I'll add both of us shortly.

@conda-forge-admin, please rerender

@xhochy Ready to go! 🙂

@xhochy Looks like the compilers used to build 0.12 are too new that lead to conflict with some package requirements?
https://app.travis-ci.com/github/conda-forge/cutensor-feedstock/builds/235132096

@xhochy Looks like the compilers used to build 0.12 are too new that lead to conflict with some package requirements?
https://app.travis-ci.com/github/conda-forge/cutensor-feedstock/builds/235132096

I sent a fix #23.


SOSCF with dm21 in pySCF

Hello, I have tried to apply dm21 to transitional metal systems recently. But even with single atom like V, Cr, Mn, I found they are very hard to converge. I tried to apply SOSCF by
mf = scf.UKS(mol).density_fit()
mf._numint = dm21.NeuralNumInt(dm21.Functional.DM21)
mf = mf.newton()
mf.kernel(mo_init,mocc_init)
which is basically the example code for SOSCF in pySCF example.
but it gave me the error message

File "/cluster/home/vuckovics/miniconda3/envs/pyscf-2.4.0/lib/python3.11/site-packages/pyscf/soscf/newton_ah.py", line 853, in kernel
return _CIAH_SOSCF.kernel(self, mo_coeff, mo_occ, dm0)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/cluster/home/vuckovics/miniconda3/envs/pyscf-2.4.0/lib/python3.11/site-packages/pyscf/soscf/newton_ah.py", line 738, in kernel
kernel(self, mo_coeff, mo_occ, dm0, conv_tol=self.conv_tol,
File "/cluster/home/vuckovics/miniconda3/envs/pyscf-2.4.0/lib/python3.11/site-packages/pyscf/soscf/newton_ah.py", line 533, in kernel
rotaiter.send((mo_coeff, mo_occ, dm, vhf, e_tot))
File "/cluster/home/vuckovics/miniconda3/envs/pyscf-2.4.0/lib/python3.11/site-packages/pyscf/soscf/newton_ah.py", line 339, in _rotate_orb_cc
g_kf, h_op, h_diag = mf.gen_g_hop(mo_coeff, mo_occ, fock_ao)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/cluster/home/vuckovics/miniconda3/envs/pyscf-2.4.0/lib/python3.11/site-packages/pyscf/soscf/newton_ah.py", line 205, in gen_g_hop_uhf
vind = mf.gen_response(mo_coeff, mo_occ, hermi=1)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/cluster/home/vuckovics/miniconda3/envs/pyscf-2.4.0/lib/python3.11/site-packages/pyscf/scf/_response_functions.py", line 171, in _gen_uhf_response
rho0, vxc, fxc = ni.cache_xc_kernel(mol, mf.grids, mf.xc,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/cluster/home/vuckovics/miniconda3/envs/pyscf-2.4.0/lib/python3.11/site-packages/pyscf/dft/numint.py", line 2578, in cache_xc_kernel
vxc, fxc = ni.eval_xc_eff(xc_code, rho, deriv=2, xctype=xctype)[1:3]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/cluster/home/vuckovics/miniconda3/envs/pyscf-2.4.0/lib/python3.11/site-packages/pyscf/dft/numint.py", line 2711, in eval_xc_eff
exc, vxc, fxc, kxc = self.eval_xc(xc_code, rhop, spin, 0, deriv, omega,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/cluster/home/vuckovics/miniconda3/envs/pyscf-2.4.0/lib/python3.11/site-packages/density_functional_approximation_dm21/neural_numint.py", line 711, in eval_xc
mol=self._system_state.mol,
^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'mol'

Could you please tell me how to apply newton properly with dm21?

Sorry, DM21 doesn't support second-order derivatives required for Newton optimization (and the way NumInt is patched in the PySCF-DM21 interface would require some careful handling even if it did, as your error shows).
A quasi-Newton method can be used (e.g. one from scipy). You might also find it useful to start from an initial guess produced by a cheaper functional (e.g. B3LYP, as shown in the README)

Only genius can see things in disinterested objective way (nevertheless all knowledge is subjective) and free his ego from his will: https://www.themarginalian.org/2016/06/29/schopenhauer-genius/  Evolution hides truths unless they coincide with survival, ppl constantly make overpositive stories about everything (nietzsche and evolution align): https://scholarcommons.scu.edu/cgi/viewcontent.cgi?article=1052&context=phi Everything is primordial unity we will suffer all painful deaths tortures forever unless we change just google unit 731 WW2 Japan...

maybe I wrote my code wrong.!? I will review my code carefully.
thanks for observing.


Metadata panel won't update after running SQL script

Description
The Metadata panel won't update after running SQL script with the command org.jkiss.dbeaver.ui.editors.sql.run.script.
To force the update, one must navigate away from the Metadata panel, then navigate back to it.
This bug does not affect the command org.jkiss.dbeaver.ui.editors.sql.run.statement.
DBeaver Version
DBeaver Community Edition <IP_ADDRESS>312031432
Operating System
Windows 11 (21H2/10.0.22000.2652)
Database and driver
oracle.jdbc.OracleDriver
Steps to reproduce

SQL Editor → New SQL Script (Ctrl + ])
write SELECT 0 AS "foo" FROM DUAL;
SQL Editor → Execute SQL script (Alt + X)
open Metadata panel
it says foo
replace by SELECT 0 AS "bar" FROM DUAL;
SQL Editor → Execute SQL script (Alt + X)
it's stuck saying foo
replace by SELECT 0 AS "spam" FROM DUAL;
SQL Editor → Execute SQL query (Ctrl + Enter)
it says spam, now

Additional context
No response

Hello @hhmmjjnn
Thanks for the bug report and reasonable explanation. I can reproduce it.


Revert "[flutter_tools] Enable fast start by default"

Reverts flutter/flutter#48746
Several devicelab tests, like concurrent_hot_reload are getting tripped up by the extra stream notifications from the hot restart. This should be relanded with fast-start disabled for most devicelab tests

TBR @zanderso


Update buttons.js

See issue #69.

See https://github.com/ntkme/github-buttons/issues/69#issuecomment-688058636.


[KYUUBI #3045] Support to do admin rest request with kyuubi-adminctl

Why are the changes needed?

Close #3045
Introduce kyuubi-adminctl for admin control.
Usage: kyuubi-adminctl [refresh] [options]

  -v, --version <value>    Using the compiled KYUUBI_VERSION default, change it if the active service is running in another.
  -b, --verbose            Print additional debug output.
  --hostUrl <value>        Host url for rest api.
  --authSchema <value>     Auth schema for rest api, valid values are basic, spnego.
  --username <value>       Username for basic authentication.
  --password <value>       Password for basic authentication.
  --spnegoHost <value>     Spnego host for spnego authentication.
  --conf <value>           Kyuubi config property pair, formatted key=value.

Command: refresh [config] <args>...
        Refresh the resource.
Command: refresh config [<configType>]
        Refresh the config with specified type.
  <configType>             The valid config type can be one of the following: hadoopConf.

  -h, --help               Show help message and exit.

Now support kyuubi-adminctl refresh config hadoopConf to reload hadoop conf.
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

also cc @lightning-L

thanks @ulysses-you
merged to master


remove stolen artwork

please stop using stolen artwork

I assume you are referring to the fact that I used a diffusion model combined with hand-painting to generate parts of this image? I recognize the conflicting nature of how most of these have been trained, but this is not the place to have this discussion. Your contributions can have more impact elsewhere.

please consider the impression that it gives of your project and to yourself. if you cant even be bothered to create a thumbnail yourself then people wont be bothered to use your tool or watch your video. please respect artists

I don't think you read my original reply correctly.

@zyriab I hope you don't mind me deleting your meme comment. I appreciate the support, but I don't want to pile on @mrshmllow with replies.

In fact, let's handle it this way:
@mrshmllow While I don't agree with harassing individual creators that give away their work for free and your point seems slightly disingenuous coming from someone with a profile picture stolen from an Anime, I do agree on the general issue.
So, please shout out your favorite artist collectives or discussion groups fighting for the rights of artists against big corporations in this thread. This allows people to find meaningful ways of contributing to the cause. Afterwards, I will lock this conversation.


Fix sigmoid gradient calculation and move sigmoid into a specialized op

This pull request is inspired by #1722 and may supersede it.
As reported in #1722, calculating the gradient of sigmoid function when input is a large negative f32 may lead to NaNs.
input =
[ -548.3162,  -201.7132,   -74.2032,   -27.2899,   -10.0179,    -3.6269,    -1.1752,     0.0000,     1.1752,     3.6269,    10.0179,    27.2899,    74.2032,   201.7132,   548.3162]
sigmoid =
[  0.0000e0,   0.0000e0, 5.9424e-33, 1.4065e-12,  4.4592e-5,  2.5909e-2,  2.3592e-1,  5.0000e-1,  7.6408e-1,  9.7409e-1,  9.9996e-1,   1.0000e0,   1.0000e0,   1.0000e0,   1.0000e0]
grad =
[       NaN,        NaN,   0.0000e0, 1.4065e-12,  4.4591e-5,  2.5238e-2,  1.8026e-1,  2.5000e-1,  1.8026e-1,  2.5238e-2,  4.4591e-5, 1.4065e-12, 5.9424e-33,   0.0000e0,   0.0000e0]

This pull request solves the NaN problem by exploiting a property of the derivative of sigmoid function, specifically
d/dx sigmoid(x) = (1 - sigmoid(x)) * sigmoid(x)

In this approach, calculating the derivative of sigmoid function relies solely on the already-calculated sigmoid function value and subtractive and multiplicative operations, effectively avoiding NaNs. Furthermore, the number of operations is significantly reduced, leading to ~2.5x performance boost.

Code snippet used to test the proposed approach
use std::{hint::black_box, time::Instant};

use candle_core::{Tensor, Var, Device};
use candle_nn::ops::sigmoid;

fn main() {
    let device = Device::Cpu;
    let x = Var::new(&[-548.3162_f32, -201.7132,  -74.2032,  -27.2899,  -10.0179,   -3.6269,   -1.1752,    0.0000,    1.1752,    3.6269,   10.0179,   27.2899,   74.2032,  201.7132,  548.3162], &device).unwrap();
    let y = sigmoid(&x).unwrap();
    let grads = y.backward().unwrap();
    let x_grad = grads.get(&x).unwrap();
    println!("{}", y);
    println!("{}", x_grad);

    let start_time = Instant::now();
    for _ in 0..100000 {
        let y = black_box(sigmoid(&x).unwrap());
        let grads = black_box(y.backward().unwrap());
        let _ = black_box(grads.get(&x).unwrap());
    }
    let end_time = Instant::now();
    let duration = end_time - start_time;
    println!("avg iter: {} microseconds", duration.as_micros() as f64 / 100000.0);
}

Output using candle-core 0.4.1 and candle-nn 0.4.1
[  0.0000e0,   0.0000e0, 5.9424e-33, 1.4065e-12,  4.4592e-5,  2.5909e-2,
  2.3592e-1,  5.0000e-1,  7.6408e-1,  9.7409e-1,  9.9996e-1,   1.0000e0,
   1.0000e0,   1.0000e0,   1.0000e0]
Tensor[[15], f32]
[       NaN,        NaN,   0.0000e0, 1.4065e-12,  4.4591e-5,  2.5238e-2,
  1.8026e-1,  2.5000e-1,  1.8026e-1,  2.5238e-2,  4.4591e-5, 1.4065e-12,
 5.9424e-33,   0.0000e0,   0.0000e0]
Tensor[[15], f32]
avg iter: 16.92021 microseconds

Output using the proposed approach
[  0.0000e0,   0.0000e0, 5.9424e-33, 1.4065e-12,  4.4592e-5,  2.5909e-2,
  2.3592e-1,  5.0000e-1,  7.6408e-1,  9.7409e-1,  9.9996e-1,   1.0000e0,
   1.0000e0,   1.0000e0,   1.0000e0]
Tensor[[15], f32]
[  0.0000e0,   0.0000e0, 5.9424e-33, 1.4065e-12,  4.4591e-5,  2.5238e-2,
  1.8026e-1,  2.5000e-1,  1.8026e-1,  2.5238e-2,  4.4582e-5,   0.0000e0,
   0.0000e0,   0.0000e0,   0.0000e0]
Tensor[[15], f32]
avg iter: 6.57586 microseconds

Note that although there are numerical inaccuracies in gradients for very large positive numbers, it might be fine because it is still better than NaNs.
I am not sure whether I should write CUDA and Metal kernels. Also, due to the introduction of a customized gradient calculation routine, sigmoid is now a specialized op for Tensor.

Could you use a CustomOp that would be defined in candle_nn only for this? This would avoid adding more complexity to candle-core. Also we will want to have cuda and metal kernels for this as otherwise the models that currently use the sigmoid function won't work when run on these devices.

I have switched to a CustomOp-based implementation, although CPU-only. Curiously, even if I compile the above test script with metal feature flag on my MacBook, the script does not fail for not having a metal implementation.
I am investigating how to integrate cuda and metal code into the codebase, and most importantly, how to call them.

Metal and cuda implementation are optional but it will fail at runtime if one tries to use them on these devices. You can find examples in this ops.rs file that have custom cuda and metal kernels.

@LaurentMazare I added an experimental metal kernel implementation. I admit it is a pain to write kernels and supporting code. However, the code does not run properly on my Mac, as all output seems zero. Maybe my Mac is too old, and if you have a device please test it for me.

Thanks, looks pretty good to me, I've added a test and it worked well on my macbook M2 pro.
I've also exposed a bit more of the cuda internals so as to reduce the duplication, it's certainly not perfect but at least that will do for this one.


Improve documentation

Is your feature request related to a problem? Please describe.
Ok, so I'm going to ask the question, in hopes that this will result in a documentation improvement. I always feel that if someone is asking, and it can be answered by documentation, that is the better way; because then the next person can read the documentation instead of repeating the question (you hope).
I'm having a little trouble understanding the reactive example. I think it's because of the interaction with reactor and webclient, and I'm just not that used to reactor, and my brain is trying to map it to other async libraries and languages I've worked with. I think now I know what it does, and the problem I'm having is verbage (retrieve is not our blocking operation). yay for promises without the async keyword.
I'm not certain if, at this exact second if this is what I should do, it seems like the sensible thing.
Mono<?> otherClientCall() // actual http client

cbFactory.create("otherClientCall").run(otherClientCall()).map(...)

Could you add an example of using a client that already uses reactor? and maybe one that uses CompletableFuture and/or CompletionStage, maybe even executors.
To follow on, how does one write a unit test for this? sample is empty
https://github.com/spring-cloud-samples/spring-cloud-circuitbreaker-demo/blob/main/spring-cloud-circuitbreaker-demo-reactive/src/test/java/org/springframework/cloud/circuitbreaker/demo/reactive/SpringCloudCircuitbreakerDemoReactiveApplicationTests.java
The default constructor on Resilience4JCircuitBreakerFactory is marked as deprecated, and the customizer simply takes that class. making me wonder how that works under the hood and why I'm not just creating that type as a bean instead.

Does this this guide help in terms of an example?
https://spring.io/guides/gs/cloud-circuit-breaker/

No, it has all the same problems I just laid out in my description. It only shows webclient, not a native reactor client, or native java. It doesn't show you how to write tests with it, especially not unit tests to avoid setting up the spring context, even if you did.

to avoid setting up the spring context

So are you not using Spring at all or are you using Spring but just not using WebClient?  What HTTP client are you using?

We are generating a client with open API that uses web client under the hood and exposes reactor, I'm then wrapping it with a repository like interface. The plan is to put the circuit breaker in the interface. The client is developed in a separate repo from the main spring boot app and so I don't really want to involve spring boot a lot in testing. I'm not really trying to spin up a full app here. The reality is that over time I've used any number of HTTP clients... Web client is nice but I don't see why it would be my only choice. Which is why I'm hoping for a couple more examples.

Let me know if this starts to head down the path you are thinking...
https://github.com/ryanjbaxter/cb-183/blob/main/src/main/java/org/example/cbsample/CbsampleApplication.java

yeah, that kind of confirms what I wasn't sure of reading the existing sample.

So does this help explain how to use it with other clients?
Also note that the reactive API requires you return a Mono or Flux, that doesn't mean the thing you are wrapping necessarily needs to return an Mono or Flux but it needs to be able to be wrapped in one.
https://github.com/spring-cloud/spring-cloud-commons/blob/main/spring-cloud-commons/src/main/java/org/springframework/cloud/client/circuitbreaker/ReactiveCircuitBreaker.java

Seems to me spring now supports 3 of it's own clients (side note: why?!!

Why wouldn't we?

and there's cloud openfeign

Spring Cloud OpenFeign uses Spring Cloud Circuit Breaker, not sure what the confusion is there
I think the best way to move forward is to provide me a concrete example of something you are trying to wrap in a circuit breaker that is not working or you or not understanding so I can help fill the gaps.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


WebKit support for CustomElements

I'd suggest this be downgraded to partial support. The spec defines two types of custom elements:

Autonomous custom element
Customized built-in element

https://html.spec.whatwg.org/multipage/scripting.html#custom-elements
WebKit supports the former only, and they are opposed to implementing the latter:
https://github.com/w3c/webcomponents/issues/509#issuecomment-222860736
https://github.com/w3c/webcomponents/issues/509#issuecomment-224212371
https://github.com/w3c/webcomponents/issues/509#issuecomment-230700060
https://github.com/w3c/webcomponents/issues/509#issuecomment-233222085

FWIW, I can PR this but wanted to open this up for discussion first.
We've just had to revisit a bunch of our code after thinking WebKit had full CustomElement support, when it doesn't.

Maybe is= (Customized built-in element) could be list as a separate feature?

I'm writing a series of blog articles on the current state of web components. It's a misconception that Chrome have fully implemented custom elements. If you look at their bug tracker here, you will see two threads that are open :
Implement Customized Builtin Elements: https://bugs.chromium.org/p/chromium/issues/detail?id=652579
Launch customized built-in elements: https://bugs.chromium.org/p/chromium/issues/detail?id=648828
The last post is on January 24.
On January 23 there is a comment which states: "Yes, it is still being worked on"
Without customized built-in elements I would state that the spec has not been fully implemented.
The spec, as it currently is, clearly states the that there are two different types of v1 custom elements, autonomous and customised builtin, and is promoted as such by Google themselves in this article: https://developers.google.com/web/fundamentals/getting-started/primers/customelements

Both browser should be downgraded then in your opinion @TCotton ? I signalled out WebKit here as I know they are actively opposed to implementing the full spec.

@ctolkien you are right in that WebKit have repeatedly stated that they will not implement this feature.
I found it really confusing why it wouldn't work in Chrome until I looked at their tracker

Have updated the PR to also change Chrome to partial support.


Add eligibility tags to home page

Description
Add tags whether an applicant may or may not be eligible for a program on the home page.
This is part of the eligibility work (#3744)
Mocks
Release notes
The title of the pull request will be used as the default release notes description. If more detail is needed to communicate to partners the scope of the PR's changes, use this release notes section.
Checklist
General

[x] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[x] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

User visible changes

[x] Wrote browser tests using the validateAccessibility method
[x] Manually tested at 200% size
[ ] Manually evaluated tab order

New Features

[ ] Add new FeatureFlag env vars to server/conf/application.conf
[ ] Conditioned new functionality on a FeatureFlag
[ ] Wrote browser tests with the feature flag off and on, etc.

Issue(s) this completes
Fixes #4142

Please update unit test coverage for relevantProgramsForApplicant

Updated!


New productsEquals in o2m does not work

The changes introduced in https://github.com/DivanteLtd/vue-storefront-api/pull/480/files to compare products do not work as intended! The check does always seem to return true which messs up orders in all different kind of ways (wrong quantities, order not passing, b/c of wrong product options etc. as products get overwritten with each other and so on).
Reproduce on https://demo.storefrontcloud.io which has API in 1.12.2 running.

Put 2 different items in cart with different quantities and place order.
 I've exported productChecksum as productsEquals in vsf-utilities. Please update vsf-utilities to 1.0.2.


Improve vendors recipe

[x] New features

Sometimes, project may need to install vendors from multiple directories. Although it's quite easy to add them in your deploy.php file, I think it would be easier if it was already included in the vendors recipe.
This PR also includes some code to self update composer from time to time (only when using phar version, the duration can be defined and this feature can be disabled) because once composer is downloaded, it will never upgrade.

[ ] BC breaks?

None that I'm aware of. Default config includes "{{release_or_current_path}}".

[ ] Tests added?

No, could find tests against recipes.

[ ] Docs added?

No, will do if you feel this PR is valuable.

Please split the different features into distinct PRs for better review.

Second thoughts about this:

Should the self update feature be enabled for installed composer (e.g. system wide) via sudo?
It would also be nice to have a feature/flag to force usage of one (composer.phar) over the other (system composer), or vice versa without having to put a composer.phar in {{deploy_path}}/.dep/composer.phar. I've had issues with some hosting providers renaming composer v2 to composer2 (composer being v1) and thus failing to install dependencies.


Add .exitRoundabout ManeuverType

And also the option to include them in a response. By default, this is off.
/cc @1ec5

ref https://github.com/mapbox/mapbox-navigation-ios/pull/561

@1ec5 confirmed exit rotary was added. Added this in https://github.com/mapbox/MapboxDirections.swift/pull/168/commits/1a5e307cfce92d6f0e14b7a401b026d49fc4c0ed

#169 renames the property to RouteOptions.includesExitRoundaboutManeuver.


Docs: Add screenshots of each component in docs

It would be really nice to have screenshots of each component, maybe right below the code snippet. Should be relatively simple to add. I recommend hosting the images in here OR just uploading them to this issue and capturing the URL.

https://github.com/infinitered/ignite/issues/2667#issuecomment-2070773338_

@jamonholmgren / @yulolimum  Do you have any suggestions on good screenshot taking for such a thing? I'd normally just ctrl+cmd+shift+4 but they won't be framed nice or even sized I guess.
Or we could just screenshot the showroom screen part of each one I guess

Hello @frankcalise  , @jamonholmgren
I wonder if someone is already working on this issue?
I would like to contribute to it , this would be my idea to share the screenshot for the different components.
I was thinking to get them from the app template and share  them in a simulator. what do you think or what are your ideas ?

@jmejiamu hey thanks for your interest! you are free to take a go at this one.
How they appear in the simulator is fine. I'm thinking maybe in isolation perhaps so other components on the screen don't confuse users.
Other than that, just keep them consistent throughout the documentation so it has a nice feel to it and users know where to look/what to expect.
You will have to also clone down https://github.com/infinitered/ir-docs if you want to see them work locally as they were hosted at https://docs.infinite.red - let us know if you have any issues.

@jmejiamu are you working on this?

Ah ok, the docs that should be modified belong in this tepo, not ir-docs.
Sorry for the confusion, you just needed to clone that repo and link these docs so you could see your changes as you would view the doc site

@frankcalise I see, sorry I got confused. I will modify work on that again.


DOC: Clarify and Standardize Remote Pull Instructions in Contribution Guide

What's wrong with the existing documentation
The existing documentation for contributing to the Community-Site repository is generally clear and helpful. However, there are a few areas where improvements can be made:
**Remote Pull Instructions: **The instructions for pulling changes from the upstream repository could be clarified. Specifically, it's important to specify the correct branch name to pull changes from, as it may not always be "main". Providing a more generic instruction that accommodates different branch names would be beneficial.
**Consistency: **Ensure consistency in terminology and formatting throughout the documentation. For example, sometimes "Repo" is used, while other times it's "repository". Maintaining a consistent style enhances readability and clarity.
Error Handling: Include instructions or references on how to handle common errors that contributors may encounter during the contribution process. This could help reduce frustration and improve the overall experience for contributors.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Its-Aman-Yadav sir please assign this task to me.

Yeah sure @kvdevlopment  please improve the documentation


throwIf() custom message

It would be great to have custom message using throwIf.
For example
throwIf(count(1)<>0,'to many records')

Nice feature request! By the way, it is very easy to implement.
PS. Many people wonder, what is the practical use case for throwIf. Can you share, please?

We use clickhouse for log storage and monitoring
current query
select concat(r0,r1) as text, if (text='','SUCCESS', text) as result from(
select count(1) as logMessages,
if (logMessages=0,'no logs','') as r0,
countIf(message='connection error') as errors,
if (errors>0,'to many errors','') as r1 from logs)

becomes
select throwIf(logMessages>0,'no logs') as r0,throwIf (errors=0,'to many errors','') as r1 from logs

Also I don't undestand why throwIf needs inverted if clause.

What version of ClickHouse do you use and what is the result of the following commands?
~/ClickHouse$ clickhouse local --query "SELECT throwIf(1)"
Code: 395, e.displayText() = DB::Exception: Value passed to 'throwIf' function is non zero (version <IP_ADDRESS>1 (official build))
~/ClickHouse$ clickhouse local --query "SELECT throwIf(0)"
0

This is the expected behaviour: the function throws an exception if argument is non-zero.

It's ok. I think I was a little confused. Right version is
select throwIf(logMessages=0,'no logs') as r0,throwIf (errors>0,'to many errors','') as r1 from logs


Kubuntu windows lose rounded corners when Kivy app runs

Versions

Python: 2.7.12+
OS: Kubuntu, Kernel 4.8.0-41-generic
Kivy: 1.9.1
Kivy installation method: installed according to Installation on Linux > Ubuntu on the Kivy website

Description
Hi,
First of all, I'd like to say that I just found out about Kivy a few days ago and I'm really excited but also a total noob, so I'm a little afraid of this not being an actual issue.  Nevertheless, I've searched around and found nothing.
So, being a total noob, I installed Kivy according to some instructions, started a python REPL, and typed in the code that I found here.
I was able to type each line in without error, and after typing in the MyApp().run() line sure enough a black window popped up with a white-text label in the center.
I noticed, however, that when the window was created, all of my applications - not just the dummy kivy app - lost their rounded corners.  On Kubuntu, windows have slightly rounded corners, which I think really makes it look a lot more comfortable and user-friendly.  But when the app starts, every single one of my windows lose their rounded corners.  All of the windows are still fully functional, draggable and resizable, but the rounded corners and animation when I minimize the window are gone.  It's not a big deal in terms of functionality, but the user interface is suddenly a lot less familiar without rounded corners and animations.
Firefox before I launch the Kivy app (Firefox picked randomly; all other windows exhibit this behavior):

Firefox after I launch the Kivy app:

It seems like a very subtle detail, but again for me it feels very strange using windows without rounded corners.  I feel like a very picky child as I type this.  It feels like a huge hit on user experience, to have all of your windows suddenly flash and lose their rounded corners right when you start an app.
When the Kivy app window is closed, all other windows regain their previously rounded state.
Code and Logs
import kivy
kivy.require('1.0.6') # replace with your current kivy version !

from kivy.app import App
from kivy.uix.label import Label

class MyApp(App):

    def build(self):
        return Label(text='Hello world')

if __name__ == '__main__':
    MyApp().run()

@brandon-gong I think this would be more of a KDE issue rather than Kivy. But yes it is something that Kivy is doing on Window creation that is causing a KDE bug. But the question of course is what is Kivy doing. But first things first, can you instead install the development branch from github and test if that exhibits the same behavior?

@FeralBytes install from master?  Also, my KDE Plasma version is 5.7.5, KDE Frameworks version is 5.26.0, and Qt version is 5.6.1 (not sure which number you want).

@brandon-gong install from master.

@FeralBytes I cloned it and ran a make install, it does not have this issue anymore! Sorry about that.  I guess its already fixed.  How unsafe to use code from the master branch in production?  Can I safely use it in an app?  I wouldn't want to sacrifice the stability of the app for all platforms just to have some rounded corners on KDE.

@brandon-gong I have only rarely encountered issues with master; so you should be just fine; using it as a stable base.

@FeralBytes Thank you so much for the help!

@brandon-gong No problem. Don't forget to close the issue. Thank you.

I know this is old but I have faced the same Issue with kivy v1.10.1 and plasma 5.13.5 it looks like kivy is somehow blocking the compositor. One easy fix is to disable Allow application to block compositing in kde setting.


Is there any bindings for detectionImages from ARkit?

https://developer.apple.com/documentation/arkit/arworldtrackingconfiguration/2941063-detectionimages
The idea would be to use flash cards as anchor points to spawn 3d model.

not yet, but should be easy to do:

bridge this property detectionImages from JS world to ios (its basically an array of strings with the paths to the image assets)
then we should receive theses nodes in renderer didAddNode and didUpdateNode, they have a new type ARImageAnchor
handle this new type ARImageAnchor there and call onPlaneDetected, or better a new onFeatureDetected or onImageDetected callback

This is actually not that hard and would be a cool Feature to contribute to. PRs would be very welcome!
I don't know when i have time to do it, but it should be doable in 2-3hours.
We could also add support for VerticalPlanes in the same iteration
see also https://github.com/HippoAR/react-native-arkit/issues/124

It‘s coming: https://github.com/HippoAR/react-native-arkit/pull/155

available in https://github.com/HippoAR/react-native-arkit/releases/tag/0.9.0-beta.3


Not working in IE 11

I've created a sample project with latest vue-cli and added feathers libraries. Error on ie11 attached
api/feathers-client.js
import $config from '@/config'
import feathers from '@feathersjs/client'
import io from 'socket.io-client'
const socket = io($config.feathersServer, { transports: ['websocket'], upgrade: false })
const feathersClient = feathers()
feathersClient.configure(feathers.socketio(socket));
feathersClient.configure(feathers.authentication());
export default feathersClient

store/index.js
import Vue from 'vue'
import Vuex from 'vuex'
import feathersVuex from 'feathers-vuex'
import feathersClient from '../api/feathers-client'

const { service, auth, FeathersVuex } = feathersVuex(feathersClient, { idField: '_id' })

Vue.use(Vuex)
Vue.use(FeathersVuex)

export default new Vuex.Store({
    state: {

    },
    mutations: {

    },
    actions: {

    }
})

@marshallswain it's in the @feathersjs/commons/lib/utils.js file

I'm going to close this, since I have no intention to support IE11 in any apps that I build.  If anybody would like to champion this issue, I believe all that is required is proper polyfill support for feathers to work in IE11, in general.  A PR to the docs on how to set it up would be great for those who are interested in IE11 support.  I will continue to focus on the core functionality.  If a situation arises where IE11 support is required for my work, I will gladly implement it.  In the meantime, thank you for opening the issue so I am at least aware of it.

@marshallswain i actually tried to fix this
You would have to remove your dependency on the server-side version of @feathersjs/common, unfortunately, the IE friendly @feathersjs/client does not have the functions you are depending on for this implementation, meaning that you would need to copy the functions you are depending on from @feathersjs/common and add it to this project
I'll do this when I have time and are "forced to" support IE11

Hi there,
I've finally found a way. It's working on the production in couple of days. I'll create a clean project as a sample and share as soon as possible.

Hi there,
I've finally found a way. It's working on the production in couple of days. I'll create a clean project as a sample and share as soon as possible.

Hi!
I have the same exception. Can you help me? How do you resolve this problem?

@stwessl and @cemevren, did you ever published a sample project with IE support?


rust/counter | reimplement motoko/counter in rust

Overview
Counter is the simplest state management canister example, but was previously only implemented in Motoko.
This pull request ports the Motoko example to Rust, to highlight the major syntax differences between languages.
Additions
I have added in rust unit tests that can be run via cargo test
Observations
In Motoko, Nat is considered a first-class primitive, and state can be managed as a class variable:
In Rust Nat32 / Nat64 is a more convenient primitive, as Nat maps to num-bigint::BigUInt which is a heavyweight heap vector that doesn't implement the Copy trait (triggering various borrow checker puzzles).
Additionally, global state requires using RefCell indirection:
thread_local! { static COUNTER: RefCell<Nat> = RefCell::new(Nat::from(0)); }

@sesi200 Thank you for your feedback and code review. I have implemented all your suggestions and rewritten the documentation to describe Rust.
Could you please confirm that what I have written is technically correct, if there is any important information missing, and if the writing style is suitable for the examples repo.
Thank you.

Looking at the Motoko counter example, the correct keyword is indeed stable and not static
actor Counter {
  stable var counter = 0;
}

https://github.com/dfinity/examples/blob/master/motoko/counter/src/Main.mo

Which agrees with the documention

https://github.com/dfinity/motoko/blob/master/design/Stable.md

Thank you for spotting this. I have committed your code review update.


Hacker bots causing 500 error "Unrecognized protocol version"

We have some hacker bots hitting our site with invalid http protocol headers. This causes an exception to throw on the following line and a 500 error. This is annoying because it shows in our error logs as a server error.
I believe the app should 400 in this case and not throw an exception since the client is as fault, not the server. What do you think?
https://github.com/zendframework/zend-diactoros/blob/master/src/ServerRequestFactory.php#L475

Here is the work around I am using for now incase it helps anyone. Another odd thing is that Diactoros throws a generic exception in this case so we must catch all exceptions and check the message with substr()
try {     $server = Server::createServer(         $app,         $_SERVER,         $_GET,         $_POST,         $_COOKIE,         $_FILES     ); } catch (UnexpectedValueException $e) {     if (substr($e->getMessage(), 29) == 'Unrecognized protocol version') {         /**          * Diactoros should return "400 Bad Request" in this case but it          * throws an exception instead. So we must return "400 Bad Request"          * ourselves to prevent 500 errors when hacker bots send us requests          * with invalid HTTP versions.          */         header('HTTP/1.0 400 Bad Request');         echo '400 Bad Request - Unrecognized protocol version';         exit;     } else {         //If the exception is not what we are looking for, let it fly and show in error logs         throw $e;     } } $server->listen();

Note: the work around above prevents some errors but not all.

I honestly think this should be resolved at the application level. Since usage of the factory typically happens during bootstrap of an application, that would be the appropriate location to handle it, and having the factory raise an exception is the best way to notify the consumer that the issue occurred.
I'll likely add something like this to Expressive's Application::run() method; I suggest handling it just the way you've outlined above in your own application.

@weierophinney I guess that solves it for expressive users but we are using diactoros and stagility without expressive. Great job on these packages by the way, we really enjoy how much the middleware pipeline has simplified our app.
If diactoros is providing an "http server", it doesn't seem right for every app that uses diactoros to have to wrap it in a try-catch in order to get it to follow the rules of http. (The rule being to return 400 bad request if the request is bad)
I'm looking at nginx or apache as an example. Neither of those dies and returns a 500 just because a bad request was sent to them. They simply return 400 Bad Request and keep going.

@rodmcnew — Two remarks.
First, Server is not like Apache or nginx; it still requires a web server sitting in front of it (which may be the PHP built-in web server, Apache, nginx, ReactPHP, etc.); it handles a single request at a time as provided by one of these. As such, its a bit of a misnomer; it's more like a front controller than anything. If I had to do it again, I'd likely omit the class entirely, because of the confusion it introduces.
Second, the difference between Expressive and Zend\Diactoros\Server is that the latter doesn't create the request/response when dispatching the application, but, rather, is passed them during instantiation — even if that's when calling Server::createServer().
What this means is that you need to do the exception handling within your bootstrap script. As an example:
try {
    $server = Server::createServer(
        $app,
        $_SERVER,
        $_GET,
        $_POST,
        $_COOKIE,
        $_FILES
    );
} catch (InvalidArgumentException $e) {
    $response = $finalHandler(new ServerRequest, (new Response())->withStatus(400), 400);
    (new Response\SapiEmitter())->emit($response);
    exit(0);
} catch (UnexpectedValueException $e) {
    $response = $finalHandler(new ServerRequest, (new Response())->withStatus(400), 400);
    (new Response\SapiEmitter())->emit($response);
    exit(0);
}

$server->listen($finalHandler);

Yes, agreed, it's kind of a pain. But that's on purpose.
Diactoros and Stratigility are providing low-level tools for HTTP messages and middleware; think of it like the relationship between C and PHP. The idea is to keep the scope narrow, to allow others to build more robust frameworks on top of them. Sure, you can write code using just Diactoros and Stratigility, but it means you have to do more plumbing for yourself; they provide just the basic value objects and infrastructure. If you don't want to manage that plumbing yourself, you use a framework built on top of them.

@weierophinney Thanks for the improved try-catch and explanation. I have some slight doubts about having the code assume that every kind of InvalidArgumentException and UnexpectedValueException comes from bad requests, but we can try this out.

Could you open an issue asking for more specific exception types, please? I
noticed this when doing the solution for Expressive as well, and it would
be a welcome improvement.
On Sep 1, 2016 2:39 PM, "Rod McNew"<EMAIL_ADDRESS>wrote:

@weierophinney https://github.com/weierophinney I guess that solves it
for expressive users but we are using diactoros and stagility without
expressive. Great job on these packages by the way, we really enjoy how
much the middleware pipeline has simplified our app.
If diactoros is providing an "http server", it doesn't seem right for
every app that uses diactoros to have to wrap it in a try-catch in order to
get it to follow the rules of http. (The rule being to return 400 bad
request if the request is bad)
I'm looking at nginx or apache as an example. Neither of those dies and
returns a 500 just because a bad request was sent to them. They simply
return 400 Bad Request and keep going.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zendframework/zend-diactoros/issues/171#issuecomment-244141113,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABlV8kKVcQbrKA8s4Mm53APNbYOW0CRks5qlwJ8gaJpZM4IdQRn
.


Chained calls with object argument indentation

Before the latest Standard update I could do this:
promiseFooBar({
    foo: true,
    bar: false
  })
  .then(fooBar => console.log(fooBar))
  .catch(err => console.log(err))

Now the new indentation rules forces me to do either of the following:
// 1
promiseFooBar({
  foo: true,
  bar: false
})
  .then(fooBar => console.log(fooBar))
  .catch(err => console.log(err))

// 2
promiseFooBar({
  foo: true,
  bar: false })
  .then(fooBar => console.log(fooBar))
  .catch(err => console.log(err))

// 3
promiseFooBar({
  foo: true,
  bar: false
}).then(fooBar => console.log(fooBar))
  .catch(err => console.log(err))

I was very happy using the first example, now not allowed. I understand and respect that the Standard rules are not up to discussion, so I just want to ask:
What way should I indent them?

Forgot one:
let options = { foo: true, bar: false }

promiseFooBar(options)
  .then(fooBar => console.log(fooBar))
  .catch(err => console.log(err))

they all lgtm except:
promiseFooBar({
  foo: true,
  bar: false })
  .then(fooBar => console.log(fooBar))
  .catch(err => console.log(err))

What about that (is more semantic IMO):
promiseFooBar({
  foo: true,
  bar: false
})
.then(fooBar => console.log(fooBar))
.catch(err => console.log(err))

@reg4in It makes it harder to detect a chained call when looking at a lot of code.
But yeah, it also satisfies the Standard rules, so it's a valid option

@albertorestifo I personally don't think so but I won't force anyone to use it :stuck_out_tongue:

Before the latest Standard update I could do this:

Which versions specifically?

I did some digging into this and it looks to have cropped up starting in v5.0.0 when the indent rule got stricter. As far as I can tell, this is an issue relating to how eslint handles the indent rule and isn't something standard can do much about.
There looks to be an open issue for eslint here: https://github.com/eslint/eslint/issues/3614 that discusses adding more customization around indentation and chaining.

@emgeee :+1: Exactly, update to 5.0.0 is what did it for me

Yeah, unfortunately eslint reimplemented the indent rule and it got stricter in some edge cases.
There's nothing we can do about it.


Add a historical geocoder

Using Nominatim?
Using LoC Geocoder?

I would say nominatim is installed & working.
Any improvements / modifications should be in separate tickets.


MFA or firewall issues?

According to this documentation, Azure AD admin sign in doesn't work with accounts that have 2-factor authentication enabled. This is weird because it used to work?
Right now we get the following error "A connection to the server 'ABC.database.windows.net' could not be established. This may indicate an issue with your network connection or firewall configuration. Please check your network connection and try again."
Is this error related to MFA?

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

ID: 9788706b-bc01-9560-4b57-8ad44f2431bf
Version Independent ID: e6ef3341-5520-a10a-bb5f-59166647478c
Content: Azure portal: Query Azure SQL Database using Query Editor
Content Source: articles/sql-database/sql-database-connect-query-portal.md
Service: sql-database
Sub-service: development
GitHub Login: @ninarn
Microsoft Alias: ninarn

@bartvermeersch Are you trying this from the Azure portal? Could you please check if the firewall is configured to allow your client IP address.
I just tried using an account with 2FA and I was able to sign in without issues.

@bartvermeersch Did you get a resolution for this? Am experiencing the same issue.

I am working with my dev team to investigate the issue. I will try to get back to you as soon as possible with info on this issue.

@ninarn Following up to see if you have any updates on this issue?

This is being assigned to the content author to provide more information.

I will be able to follow up early next week with an update.

Thank you for your patience I have an update on this issue. My team is confident that they know the cause of your issue and they have implemented a fix that should go out soon. Please wait until late this week or early next week and try again. Once this fix is in place, if you encounter the issue, you will be able to click 'Try again' and it should work the second time.
I should also add clarification that we have never supported multi-factor authentication within the query editor feature. Meaning that you will not be able to login to the query editor using 2-factor auth. However if you logged in to the Azure portal with AAD MFA, connecting to the query editor with AAD should work. And we expect the fix to be available soon.

I also have MFA enabled account, login using Azure AD authentication on Query Editor takes some time and then fails but if I click Continue it works just fine. So, its not a blocker but not a good experience. Is this expected to be resolved soon?

I'm currently unable to login with the query editor using either the local administrator credentials or with Azure AD. I do have MFA on my account, but for both my AD account and a local administrator, I'm receiving the below message:
A connection to the server 'XXX.database.windows.net' could not be established. This may indicate an issue with your network connection or firewall configuration. Please check your network connection and try again.
To confirm, my local IP address is whitelisted along with Azure Services being allowed. Is anybody else experiencing this issue?

@ryanps1 I confirm that I am facing the same issue. Now if only I can find the default query that Query Editor provides, I would be able to work around this annoyance.
If anyone knows the default Query for searching logged events on SQL, please share!

If anyone else was looking;
SELECT TOP 100 event_time, server_instance_name, database_name, server_principal_name, client_ip, statement, succeeded, action_id, class_type, additional_information FROM sys.fn_get_audit_file('https://xxx.blob.core.windows.net/xxx/xxx/xxx/2019-09-19/00_02_05_012_150.xel', default, default) WHERE (event_time <= '2019-09-19T00:24:49.586Z') /* additional WHERE clause conditions/filters can be added here */ ORDER BY event_time DESC

If anyone else was looking;
SELECT TOP 100 event_time, server_instance_name, database_name, server_principal_name, client_ip, statement, succeeded, action_id, class_type, additional_information FROM sys.fn_get_audit_file('https://xxx.blob.core.windows.net/xxx/xxx/xxx/2019-09-19/xxx.xel', default, default) WHERE (event_time <= '2019-09-19T00:00:49.586Z') /* additional WHERE clause conditions/filters can be added here */ ORDER BY event_time DESC

@ryanps1 Found the issue was related to port 1443 - open that up and you should be fine, as stated by the yellow ribbon after the error is thrown.

Yes, the query editor uses ports 443 and 1443 to communicate. Please ensure you have enabled outbound HTTPS traffic on these ports. You will also need to add your outbound IP address to the server's allowed firewall rules to access your databases and data warehouses.
Please try these and see if you can connect to query editor?

Hi there I am unable to connect to view the Query Editor. I have added the Rules according to the documentation. And I am using the local account to access the Query Editor. It keeps throwing a firewall/proxy error

Same thing. Unable to login to Query Editor. I am following https://docs.microsoft.com/en-us/learn/modules/develop-app-that-queries-azure-sql/3-exercise-create-tables-bulk-import-query-data guide.

Can you please clarify what error messages you are seeing when you try to connect to the query editor?

Hi this has been resolved now. It was our corporation firewall. We needed
to add port 1443 and 1433
On Tue, 3 Mar 2020, 18:04 Ninar Nuemah<EMAIL_ADDRESS>wrote:

Can you please clarify what error messages you are seeing when you try to
connect to the query editor?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/34827?email_source=notifications&email_token=AOMXHVNZKGGZEPAUCAP7XGLRFVBCHA5CNFSM4H7NJSHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENUQZEQ#issuecomment-594087058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOMXHVKN3ZJIV4IIECFFDOLRFVBCHANCNFSM4H7NJSHA
.

#please-close


fix(esp32): delete the secondary judgment  in examples/mesh/internal_communication (IDFGH-13472)

Delete the secondary judgment of whether it is the root node in the sending task.

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

@TroyMitchell911 thanks, will merge it ASAP.


Auto-scaling Ring Buffer - also downscaling automatical?

First of all: many thanks for this library!
What is your question about gnet?
I have a question about the ring buffer: Does the ring buffer also downscaling automatically? I didn't find anything in the code, but maybe I'm blind.
I have 40k open websocket sessions, if I now reduce them to 20k, the memory consumption of the ring buffer remains on the same level, even after 30 min.
Thank you in advance for your answer.

That is odd, it has nothing to do with the ring-buffer size but its number based on your desc, gnet will recycle ring-buffers after connections are closed, so it ought to save some memories.
Could you tell it more detailedly about the I have 40k open websocket sessions, if I now reduce them to 20k? It could some bugs in gnet and I want to ensure it.

Besides, which version of gnet are you using?

Hi panjf2000,
first of all: Thank you for your response. I use version v1.0.0-rc.4. I close the connections and gnet also sees that since it triggers OnClosed.
I benchmark my service by docker container. So I have the service running on the host with gnet and then spawn several containers that connect to the service on the host. Every connection is registered via OnOpened and every disconnect via OnClosed. Then I stop half of the containers to get 20k connections.

Could you use pprof tool to find out the root cause that occupies the most of memory? In that way, we will have an overall view about the memory usage of your gnet server.

At startup:

Opening 20k connections that send every second a text

Close 10k connections so we still have 10k connections

It seems that ring-buffers weren't GCed after connections were closed but I can't tell why that was happening, I will go investigate it further to figure out the root cause.

Or could you add some logs to see if the releasing code have been executed?

Added breakpoint, gets triggered:

Then I have no idea why those ring-buffers from closed connections weren't GCed, I would have to take some time to investigate it.

Hi @panjf2000,
I run the test for longer time and it looks like it's working.
Here's what I did:

started the server
40k connections established
82 MB memory consumption
All connections disconnected
82 MB memory consumption
5 min. later: 166 MB memory consumption with the ring buffer
30 minutes later:

Looks like sometimes the Golang GC takes longer.
Please excuse the unnecessary confusion and unnecessary work.

Please excuse the unnecessary confusion and unnecessary work.

Don't be sorry for that, feel free to open an issue when you encounter any problem when using gnet😄, it is my duty to help gnet users solve problems.
Also glad to hear that GC works, but still I have a doubt that Golang GC takes that long time(30 minutes) to collect garbages cuz GC is triggered by a fixed and short time(2 minutes) if the last GC finishes over 2 minutes ago, so it is still odd. But on the other hand, it also could be a GC mutator assist which might cause a long time pause to GC, maybe that is it.
Anyway, I will keep an eye on the GC in gnet, thanks for your discovery on this.

Hi @patrick-othmer
Have just discovered Gnet myself and am starting some work with websockets. Is the work you've done publicly available (eg github) or is closed source?  Would definitely like to see what you've done (saves me from re-inventing the wheel).
Thanks
Ken

Hi @kpfaulkner,
sorry, it's closed source. But the gnet examples may help you: https://github.com/gnet-io/gnet-examples
Regards,
Patrick


[D&V] Display deprecation and vulnerabilities on search page.

Changes

Retrieve Deprecation and Vulnerabilities from Search Service.
Added Deprecation and Vulnerabilities properties to Package.
Display badges on search page.

Screenshots

Addresses

https://github.com/NuGet/Engineering/issues/4734

Closed due to non-valid branch name since it contains an ampersand "&".


Changing lora_alpha during inference

Hi,
I wanted to know if there's any workaround in peft library, to change the alpha value from that used during training time, for the LoraConfig during inference?
Thanks in advance!

With the goal of changing the LoRA scaling? There is currently no easy way except to loop through the modules and adjust the .scaling argument, but there is a solution in the works in #1951.

Thanks @BenjaminBossan! However, importing set_adapter_scale from peft.utils.scaling_utils leads to an error. Is the function set_adapter_scale to be imported from some other class?

Thanks a lot! This helps!

Hi @SoumiDas!
If you end up using the context manager -- do let us know if you run into any errors :smile:


Message text wrapping

The text in the message panel wraps! Yay!

I signed it!

On May 30, 2017, at 10:57 PM, googlebot<EMAIL_ADDRESS>wrote:
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.
If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
If you signed the CLA as a corporation, please let us know the company's name.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Error opening .png file

VSCode Version: 1.47.1
OS Version:Ubuntu 20.04 LTS Linux x64
Steps to Reproduce:

Does this issue occur when all extensions are disabled?: No

So this error only happens when you have extensions enabled?
Can you share the path to the file? Does it contain special characters such as % or #?

Enabling or diabling the extensions didn't help but the folder name has special charatcter "#" . I will try removing it and the file once again.

Reply by : @Gribesh 27-07-2020/21:08
Removing special character (in my case '#' ) solved this problem. Thanks @mjbvz


Theme website specifically for one parent category

From @alexandermendes on May 18, 2017 12:12
The easiest way to handle the required structure now seems to be creating a separate child site for each core project. There are 200ish volumes of Playbills so this could grow into quite a large site on it's own. Things to be updated through variables or ignored markdown files include:

About page
Homepage tag lines
Navbar links (e.g. "Volumes", "Drawers" or "Languages" instead of the generic "Projects")
Brand ("Convert-a-Card" instead of "LibCrowds")
Project page text
Images (we have a lot of card catalogue photos, we need more for the playbills)

Copied from original issue: LibCrowds/libcrowds-bs4-pybossa-theme#52

This closes #49

See #44 for more background info

Probably makes sense to use different webpack config files to add all of the above

All {{ url_for('project.details', short_name=project.short_name) }} links should redirect straight into the project. Maybe we can find another purpose for the project details page but for now it's an extra and unnecessary level to navigate down.

This is essentially going to be handled by using separate instances of PyBossa and this theme, tailored via config.js for each meta-category.


ci: fix Cilium CLI install in ConformanceKindEnvoyDaemonSet

In cef9595361e970dbee47eed659bac56ea68d73cd we switched to using a cilium-cli GHA for installing the CLI, but this workflow was merged in main at a later point.
We edit the workflow to be consistent with the other workflows when installing the CLI.

/test-1.24-5.4

/test-1.24-5.4

/test-1.24-5.4


Windows 8.1 Defender Problem

Hi...^^
I'm using Windows 8.1 Pro x64.
Turning on Defender when using Dism ++ makes installation or mounting of images very slow.
Turning off the defender is very fast.
Thank you for making a good program.
I wish you good luck.

It's known issue.
Windows Defender will scan per file when opening files, and image operation will create or open lots of files, so it's slow.
So case closed.


Update toml lexer/parser to match the spec

Spec: https://github.com/toml-lang/toml/blob/master/README.md

:thumbsup:
Thank you!


Limit number of free of charge deployments

I was searching for a way to limit the number of free of charge software subscriptions, for example, to limit each customer to 1 active free of charge marketable service subscription. I couldn't find a way to configure this on OSCM. Is there such a feature?

Hi @fabiorauber
Well, for services with access type LOGIN, you can limit the number of registered users per subscription and also the number of concurrent users. I don't know if this is helpful for your case. Otherwise I could think of a solution with a create subscription trigger that checks the number of existing subscriptions and compares this with a given limit parameter.

@fabiorauber, you are right, the customer administrator can remove triggers. I see two options to limit the subscriptions.

The direct and easiest way would be to use the LOGIN access type as described before. See Technology Provider's Guide, chapter 2 for details. This allows the service manager to specify the predefined parameter "Maximum number of registered users for a subscription" (NAMED_USER) individually for the free and for the payed marketable services he provides to his customers.
Another idea which comes in my mind:  You could enhance your technical service implementation in order to track its created instances. I.e. define an integer service parameter MAX_NOF_SUBSCRIPTIONS  with some default value in your technical service XML template. You can implement the check with a few code lines in your provisioning service (implementing asyncCreateInstance).
Count the instances, compare with your service parameter MAX_NOF_SUBSCRIPTIONS given with the InstanceRequest and decide to either complete or abort the provisioning with return of an respective error message.

@fabiorauber
Does it work for you?

@GoebelL, we will try the second approach, I think will work for our use case. Nonetheless, I still believe the best way to solve this problem would be to implement some kind of limit on the marketable service itself, preventing the user from ordering the service beyond certain limit. Don't know the difficulty of such implementation though.


running time is ms or s?

in main_cpu.cpp, printf("Average SVD takes %f microseconds \n ", 1e6*(double(end - start) / 1e6 / CLOCKS_PER_SEC ) ); , this is ms?

double(end - start)/CLOCKS_PER_SEC  is seconds.

the time in main_cpu.cpp is calculte svd for 1e6 times. so the right time = time/1e6, rather than multile 1e6 another.
please give yuor comment.thanks


Kate tears sre

Background

https://datadoghq.atlassian.net/jira/software/projects/DPNCTF/boards/5129?selectedIssue=DPNCTF-160
This adds logs to the adserver service that support TSRE challenge.
Fixes

Change Summary

Additional Notes

Testing Procedure

Related PRs or Issues

Erroneous pull request.


Align epoch ranges with varying step size

If the loaded datasets have varying epoch steps (e.g. ds1 show every 5th epoch an ds2 every 1st epoch) then the line/heatmap visualizations are not correct.
Before we implement this features, we have to check if we need it since we currently just have datasets with step size of 5 epochs.

We agreed on having datasets with equal epoch size and with equal granularity for now.
If this restriction is to strong then we can change the implementation accordingly in the future.


[WIP] publication chapter

closes #9

@fpahlke would be cool if you could chime in, maybe there is still something important missing - thank you :-)

@kkmann this would also be more or less ready for merge. maybe you also want to have a quick look first?


MM-46374 : Move "Themes" type of MM-redux to another location

Summary
Ideally we would like Theme to move to '@mattermost/type' package sometime in future. Currently the new location packages/mattermost-redux/src/selectors/entities/preferences.ts seems to be better alternative. Follow conversation https://community.mattermost.com/core/pl/85auue3yytfutkts84rfefynpe
Ticket Link
https://mattermost.atlassian.net/browse/MM-46374
Related Pull Requests
n/a
Screenshots
n/a
Release Note

NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.


[FLINK-35489][Autoscaler] Ensure METASPACE size is computed before the heap

What is the purpose of the change
If the heap usage with overhead leads to remaining memory to be assigned to the HEAP no memory will be assigned to the METASPACE leading to JVM failure at startup
By setting this before the HEAP we ensure the JVM can start with appropriate amount of METASPACE memory
Brief change log
(for example:)

Compute METASPACE size to set before the HEAP size

Verifying this change

This change added tests and can be verified as follows:

Extended tests to validate that despite a big HEAP usage that would lead to the whole memory being used by it, the METASPACE has been well set with all appropriate memory

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changes to the CustomResourceDescriptors: no
Core observer or reconciler logic that is regularly executed: yes

Documentation

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

Updated the PR to take in account this comment in the jira ticket

Thanks @ashangit for the fix and everyone for the review!
Merging~


Incompatibility

Hi, your component aren't compatible with
com.mcxiaoke.koi:core
com.mcxiaoke.koi:async
please can you check this.

It seems that both our core module are messing with each probably because they have the same name, leading to duplicated files while merging the apk.
I've googled it but can't find how to prevent it. Any ideas?


Under what circumstances does helm upgrade not replace a deployment

I have a helm chart with multiple sub charts that's deployed via a CircleCI workflow. If one of my sub charts has no changes and the image has the same SHA, will helm upgrade skip deploying that?

Hi @Bajix. Have you read the FAQ on how the upgrade strategy works? https://helm.sh/docs/faq/#improved-upgrade-strategy-3-way-strategic-merge-patches
Let us know if you have further questions after reading that document.

@bacongobbler Thanks for getting back to me; that was in fact very helpful! To expand on my previous question, if a manifest has different image tags but those tags have the same SHA, is that considered for there to be no changes? As all of my sub chart images share the same Tag for that specific workflow, it'll always be the case that the manifest points to a different image even when that image has the same SHA as the current live image. Can you confirm the manifest diff comparison uses the image SHA for comparison and not the tag?

the comparison uses the content of the rendered template as-is for the comparison. It does not inspect the manifest and resolves an image's tag to its digest.
Here is the code in question:
https://github.com/helm/helm/blob/7ce29e12fa8ac7195613ffa1a76b2914150ff756/pkg/kube/client.go#L346-L353
Does that help answer your question?

@bacongobbler I think that it would be extremely useful for Helm to start resolving image tag digests before making a comparison as then this could enable mono repos to skip unnecessary partial deployments when individual images haven't changed. When I tried making a workaround for this before, I could not get this to load an ECS image by it's SHA at all and thus there was no viable workaround to prevent unnecessary deployments.


can't see community block on this tweet

i can't see the community block on this tweet → https://twitter.com/danielcberk/status/1647054404581834752
works in incognito.

Thanks for flagging @deadcoder0904, for now to show it toggle on communities button in options.


Build with LUFA and SD/SdFat libraries fail

Hi!,
I'm trying to include SD or SdFat library on a LUFA project but I get build errors.
The project corretly build with "Arduino Leonardo" board, but fail with the "Arduino Leonardo (LUFA)" board:
In file included from C:\Program Files (x86)\Arduino\libraries\SD\src/SD.h:21:0,                  from C:\Users\frolando.DLC0YLZW2\Desktop\SD-1.2.4\examples\Files\Files.ino:3: C:\Program Files (x86)\Arduino\libraries\SD\src/utility/SdFatUtil.h: In function 'void SerialPrint_P(const char*)': C:\Program Files (x86)\Arduino\libraries\SD\src/utility/SdFatUtil.h:63:5: error: 'Serial' was not declared in this scope      Serial.write(c);      ^~~~~~
Here a project created for the tests:

#include <SPI.h>
#include <SD.h>
#include <LUFA.h>
#include <LUFACDCSerial.h>
File myFile;
void setup() {
if (!SD.begin(4)) {
while (1);
}
// open a new file and immediately close it:
myFile = SD.open("example.txt", FILE_WRITE);
myFile.close();
}
void loop() {
// nothing happens after setup finishes.
}

I've attached both logs.
Do you have any suggestions?
Thanks!
build_log_Arduino-Leonardo.txt
build_log_Arduino-Leonardo-(LUFA).txt

After a more deep analysis it seem caused by the "USBAPI.h" where the Serial is disabled by LUFA.
Do you think that I can try to re-enable the Serial object in the "USBAPI.h"?
There is any other workaround?

I've definitely fixed and tested on my Badusb USB (and Arduino Leonardo with SD card reader) replacing the arduino-LUFA files with the original ones from ardunio dir.
In detail I've recover the following 2 files into hardware\arduino\avr\cores\arduino-LUFA:
hardware\arduino\avr\cores\arduino\Arduino.h
hardware\arduino\avr\cores\arduino\USBAPI.h

The fist one has the USBAPI.h include and the second one is API it'self.
The SD and the USB work's well togeder on my setup, but I don't kow it can work for other boards, so you can just close the issue.
Thanks.


Adding some java.util.Queue methods to ArrayRingBuffer and tidy up existing ones

I've improved some existing method and adding coverage for new ones based on java.util.Queue API

Changing some name and still enforcing some contracts

Closing this because already merged via https://github.com/belaban/jgroups-raft/pull/133


Remove sig/att not found error wrapper for keyless

https://issues.redhat.com/browse/HACBS-2161

LGTM with the stated intent, though I think we should do this for all cases, would make it easier to troubleshoot having the whole error message from Cosign

The issues was that the message from cosign is so cryptic that it makes it hard for a regular user to understand what's happening.