update legacy helm charts for revised validation webhook readiness check

@ayj: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.5
f2e3334f4e6a79364290a883129a7c70d85f29fb
link
/test gencheck_istio_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ayj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.5
f2e3334f4e6a79364290a883129a7c70d85f29fb
link
/test gencheck_istio_release-1.5

pilot-e2e-envoyv2-v1alpha3_istio_release-1.5
f2e3334f4e6a79364290a883129a7c70d85f29fb
link
/test pilot-e2e-envoyv2-v1alpha3_istio_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ayj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.5
f2e3334f4e6a79364290a883129a7c70d85f29fb
link
/test gencheck_istio_release-1.5

pilot-e2e-envoyv2-v1alpha3_istio_release-1.5
f2e3334f4e6a79364290a883129a7c70d85f29fb
link
/test pilot-e2e-envoyv2-v1alpha3_istio_release-1.5

integ-galley-k8s-tests_istio_release-1.5
e07b86dcfa7f061361747f04a0b6b28ca929cc7c
link
/test integ-galley-k8s-tests_istio_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Downloading Envoy: curl -fLSs https://storage.googleapis.com/istio-build/proxy/envoy-alpha-524cfb2be117f343fd25c609bdd622eca7c2b4f5.tar.gz to /home/prow/go/src/istio.io/istio/out/linux_amd64/release/envoy-524cfb2be117f343fd25c609bdd622eca7c2b4f5
curl: (22) The requested URL returned error: 503

/test integ-galley-k8s-tests_istio_release-1.5

@ayj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

pilot-e2e-envoyv2-v1alpha3_istio_release-1.5
f2e3334f4e6a79364290a883129a7c70d85f29fb
link
/test pilot-e2e-envoyv2-v1alpha3_istio_release-1.5

integ-galley-k8s-tests_istio_release-1.5
e07b86dcfa7f061361747f04a0b6b28ca929cc7c
link
/test integ-galley-k8s-tests_istio_release-1.5

unit-tests_istio_release-1.5
e07b86dcfa7f061361747f04a0b6b28ca929cc7c
link
/test unit-tests_istio_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

This needs some make gen - but do we really need the manifests/ changes?

This is needing to make istiod=false with 1.5 work.


bug: shared directory output (e.g. clang crashreports) fails with racing

https://github.com/bazelbuild/reclient/commit/35813b877b5fbf0fe69bb558c15ed239a816c226 introduced a directory type output to clang actions, which is shared among all actions of this type (with a -fcrash-diagnostics-dir option on). However, in racing strategy, this results in the following error:
reclient[b1872204-b036-4d9d-85ca-575109a989fd]: LocalErrorResultStatus: b1872204-b036-4d9d-85ca-575109a989fd: cannot move directory ../../tools/clang/crashreports from /projects/brave-browser-3/src/out/android_Release_x86/.reproxy_tmp/racing/b1872204-b036-4d9d-85ca-575109a989fd/src/out/android_Release_x86 to /projects/brave-browser-3/src/out/android_Release_x86: rename /projects/brave-browser-3/src/out/android_Release_x86/.reproxy_tmp/racing/b1872204-b036-4d9d-85ca-575109a989fd/src/tools/clang/crashreports /projects/brave-browser-3/src/tools/clang/crashreports: file exists

This is because here we're trying to use os.Rename to move the output directory from the temporary racing directory, and the directory already exists, because it was populated by a different action.
I'm not sure what is the intended behavior of the shared output directory -- as I understand the current code, any output gets removed before it is downloaded, so it looks to me like the behavior is badly defined when multiple actions share an output directory.
If the intended behavior was to aggregate outputs from multiple actions in the same ../../tools/clang/crashreports directory, then this will need to be fixed both in the SDK and Reclient, IIUC.

I've rolled back the change but there are some technical difficulties on our end which is preventing github from picking up the change.
However, this did spark some discussions on shared output directories. Our initial thinking is we may not want to support this. If you have any opinions on this topic feel free to leave a comment as we haven't come to a conclusion on how we should handle these cases yet.

Wow, that was quick! Thank you!
My opinion is that for the crashreports in particular to be useful, we would need to add an argument to the OutputSpec in Command (here) for setting the output directory behavior (I'm thinking an enum with UNKNOWN, REPLACE, and AGGREGATE), with REPLACE being the default (current behavior), and propagate that all the way from the rewrapper config. Then, we can support it in the SDK and the reproxy as well. I would envision the AGGREGATE option to apply to sub directories as well, meaning -- the action does not delete any directories, it only replaces same named files under the same path, but otherwise only adds nodes. The SDK implementation should be rather simple, I think? The reproxy one will be a bit more annoying because we can no longer os.Rename anything, we'd need to be both careful and efficient (still use os.Rename when it's possible, but copy when not).
The question is -- is it worth it? I don't know how useful the crashreports directory is.

Time to dreg up old bugs...
I suppose it isn't out of the question for some build to arbitrarily say that multiple actions have the same output directory (and not just for clang crash reports).
There are a few other options that might be worth considering as well:

Check for destination existence -> if it doesn't exist use rename, if it does, rename individual contents of directory (probably fine in most cases, most actions aren't producing huge numbers of outputs).
Create the destination directory ahead of time (before the action even starts the remote execution), and always rename the individual contents.

I wonder if doing the file moving in this kind of manner might help performance on windows, as it has some... different... filesystem performance compared to linux.  Creating and moving directories is less performant.


How to save uint64 into DB

This is my custom query:
SELECT
  date, price
FROM sales
WHERE type_id= %%type_id int%%
  AND key = %%key uint64%%

when it is converted to custom-query function, and I pass a uint64 to such function, I'll get error:
sql: converting Exec argument $2 type: uint64 values with high bit set are not supported

The sql driver is mattn/go-sqlite3 BTW,
So how to save uint64 into DB? Thx.

https://groups.google.com/forum/?hl=en#!topic/golang-nuts/QmvMMPcoVcs

Thanks for linking to the solution!

@natefinch, Thank you.
For proof, see https://github.com/suntong/lang/blob/master/lang/Go/src/ds/builtin.go


ports are prometheus and grafana not bound to host desktop hence URL open not working

and

any support as I unable to access graphana and premethos URL

Did you run https://github.com/resilience4j/resilience4j-spring-boot3-demo/blob/master/docker-compose-desktop.yml?

I have the followed the steps in readme.md
and used command
docker-compose -f docker-compose.yml up

Will try with docker-compose-desktop.yml instead of docker-compose.yml

Did you run https://github.com/resilience4j/resilience4j-spring-boot3-demo/blob/master/docker-compose-desktop.yml?

Thank You This works.. only change needed was
FROM gradle:latest

Also, the password used for Graphana was admin/admin
Further info in Readme.md would help such as how to view the graphs and simulate failures and see the recovery happening etc.. currently we can only see empty views


wildcard_mention: Don't highlight for unsubbed users.

before:

after:

discussion: https://chat.zulip.org/#narrow/stream/101-design/topic/UI.20redesign.3A.20mentions

This looks good to me, Aman!

Merged, thanks @amanagr and @karlstolley!
I think it's not entirely clear whether @everyone should do this, but certainly @stream should, and we don't treat those three differently in any other situation.


(): found unknown escape character while parsing a quoted scalar

I got the following error while try to integrate the setup-env-cfn-template.json into a AWS Landing Zone AddOne:
`/codebuild/output/src736925300/src/aws-landing-zone-configuration/templates/azure-sso/setup-env-cfn-template.json
| FAIL FATAL
|
| (): found unknown escape character while parsing a quoted scalar at line 63 column 21 `
The Textblock which causes the error is the second line of:
                "ZipFile": { "Fn::Join": ["", [                     "\/**\n",                     "* A Lambda function that looks up the latest AMI ID for a given region and architecture.\n",                     "**\\/\n",                     "var aws = require(\"aws-sdk\");\n",                     "...",
I did not change the file above, I just try to add it into an custom add-on of the AWS Landing Zone Solution.
I hope you could help me.
Regards,
Philipp

Ok it seem to be that I have to use double backslash to escape an slash. Like this:
"ZipFile": { "Fn::Join": ["", [ "\\/**\n", "* A Lambda function that looks up the latest AMI ID for a given region and architecture.\n", "**\\/\n", "var aws = require(\"aws-sdk\");\n", "...",
Can this be changed generally or might it be a special case when this script is embedded into the an landing zone add-on?


Tests don't fail gracefully when encountering errors after test execution and stall the CI

What are you trying to achieve?
Test pipelines should continue gracefully when encountering an error during "after" test execution
What do you get instead?
When the test fails, codeceptJS fails to properly scale down the test run and keeps the browser session open, essentially turning the test thread into a zombie codeceptJS process.
Tests running in pipelines will time out as the thread that's running these tests don't know the test has completed and that they should start the next test.
   Requesting menu-item sold-out status change to false
      I send put request "https://uri/api/v2/restaurant/1289255/category/15516284/product/487560132", {"name":"Sold Out","description":"Will be changed to sold out by automated tests","external_id":"","sort_id":4,"delivery_price":12,"pickup_price":12,"shipping_method":"pickup_delivery","sold_out":false,"image_url":null,"soft_deleted":false,"addon_set_id":null}, {"accept":"application/json","content-type":"application/json","Authorization":"Basic =="}
      › [Request] {"baseURL":"https://uri/api/v2/restaurant/1289255/category/15516284/product/487560132","method":"PUT","data":{"name":"Sold Out","description":"Will be changed to sold out by automated tests","external_id":"","sort_id":4,"delivery_price":12,"pickup_price":12,"shipping_method":"pickup_delivery","sold_out":false,"image_url":null,"soft_deleted":false,"addon_set_id":null},"headers":{"Content-Type":"application/json","Accept":"application/json","X-Session-Id":"8c7db2c0a86b","user_agent":"cw-api-automated-tests","sec-fetch-mode":"cors","access-control-allow-origin":"*","accept":"application/json","content-type":"application/json","Authorization":"Basic =="}}
      › [Response] Response error. Status code: 404
      › [Response] "404 page not found\n"
404 page not found

    [2] Error | Error: Restaurant Menu API call failed! Response: undefined
             Status: 404 (e) => { const err = (recorder.getAsyncErr() === n...
Error processing test.after event:
TypeError: Cannot read properties of undefined (reading 'ctx')
    at AllureReporter.allureTestByCodeceptTest (/Users/seth.vanwyk/workspace/project/node_modules/allure-codeceptjs/dist/index.js:89:62)
    at AllureReporter.testAfter (/Users/seth.vanwyk/workspace/project/node_modules/allure-codeceptjs/dist/index.js:159:33)
    at EventEmitter.emit (node:events:402:35)
    at Object.emit (/Users/seth.vanwyk/workspace/project/node_modules/codeceptjs/lib/event.js:149:28)
    at errHandler (/Users/seth.vanwyk/workspace/project/node_modules/codeceptjs/lib/scenario.js:116:39)
    at /Users/seth.vanwyk/workspace/project/node_modules/codeceptjs/lib/scenario.js:162:11
    at /Users/seth.vanwyk/workspace/project/node_modules/codeceptjs/lib/recorder.js:239:9
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    [2] <teardown> Stopping recording promises
 › <screenshotOnFail> Test failed, try to save a screenshot
 › Screenshot is saving to /Users/seth.vanwyk/workspace/project/output/Desktop_-_item_is_sold_out_after_each_hook__After_for_should_remove_item_that_is_sold_out_from_basket_@site-nl_@delivery_<EMAIL_ADDRESS>  ✖ FAILED in 434ms

    [3]  Starting recording promises

-- FAILURES:

  1) Desktop - item is sold out
       should remove item that is sold out from basket @site-nl @delivery @not-allowed-to-run-in-production @should-remove-item-that-is-sold-out-from-basket:
     Error: Restaurant Menu API call failed! Response: undefined
             Status: 404
      at Object.changeMenuItemSoldOutStatus (api-calls/restaurant-menu-api.js:36:15)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)
      at async Test.<anonymous> (/Users/seth.vanwyk/workspace/project/checkout-e2e/scenarios/multi-step-checkout/item-out-is-sold-out-alert-desktop-test.js:52:3)

  Scenario Steps:
  - I.seeElement({"xpath":"//*[@data-qa='jet-privacy-settings-button-full-consent']"}) at ./page-objects/footer/footer.js:16:41
  - I.seeElement({"xpath":"//*[@data-qa='privacy-settings-action-close']"}) at ./page-objects/footer/footer.js:15:42
  - I.say("Checking which cookie banner variant is in use") at Object.acceptCookies (./page-objects/footer/footer.js:14:7)
  - I.say("Opened page https://site/en/menu/test-restaurant-selenium?showTestRestaurants=true&utm_medium=e2e_testing&utm_source=cwplatform&utm_campaign=local&e2e=true&timestamp=1708522969025 at Wed Feb 21 2024 13:42:49 GMT+0000 (Coordinated Universal Time)") at Actor.accessURL (./steps-file.js:174:12)
  - I.grabCurrentUrl() at Actor.accessURL (./steps-file.js:173:37)
  - I.waitForInvisible({xpath: //*[contains(@data-qa,'skeleton')]}, 30) at ./steps-file.js:210:18
  - I.say("Awaiting for 30 seconds for skeleton to not be visible") at Actor.waitPageLoads (./steps-file.js:206:14)
  - I.waitForFunction(async () => {
          const state = await document.readyState;
          return state === 'interactive' || state === 'complete';
        }, 120) at Actor.waitPageLoads (./steps-file.js:197:12)
  - I.say("Awaiting for page to load with javascript") at Actor.waitPageLoads (./steps-file.js:196:12)
  - I.amOnPage("https://site/en/menu/test-restaurant-selenium?lat=53.0651908&lng=16.04760159&showTestRestaurants=true&utm_medium=e2e_testing&utm_source=cwplatform&utm_campaign=local&e2e=true&timestamp=1708522969025") at Actor.accessURL (./steps-file.js:167:14)
  - I.say("Opening the page") at Actor.accessURL (./steps-file.js:164:12)
  - I.sendGetRequest("/restaurant?slug=test-restaurant-selenium", {"X-Country-Code":"pl","Accept-Language":"en","X-Language-Code":"en"}) at getTakeawayRestaurantOpening (./api-calls/restaurant.js:6:28)

  Artifacts:
  - screenshot: /Users/seth.vanwyk/workspace/project/output/should_remove_item_that_is_sold_out_from_basket_@site-nl_@delivery_<EMAIL_ADDRESS>
  2) Desktop - item is sold out
       "after each" hook: After for "should remove item that is sold out from basket @site-nl @delivery @not-allowed-to-run-in-production @should-remove-item-that-is-sold-out-from-basket":
     Error: Restaurant Menu API call failed! Response: undefined
             Status: 404
      at Object.changeMenuItemSoldOutStatus (api-calls/restaurant-menu-api.js:36:15)
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)
      at async Context.<anonymous> (/Users/seth.vanwyk/workspace/project/checkout-e2e/scenarios/multi-step-checkout/item-out-is-sold-out-alert-desktop-test.js:75:3)
      at async promiseRetry.retries.retries (/Users/seth.vanwyk/workspace/project/node_modules/codeceptjs/lib/scenario.js:143:9)

  FAIL  | 0 passed, 2 failed   // 27s

The terminal stalls here and leaves a broken browser session open that doesn't close anymore.

Provide test source code if related

    if (response.data.message !== 'success') {
        console.log(response.data)
        throw new Error(
            `Restaurant Menu API call failed! Response: ${response.data.message}
             Status: ${response.status}`
        );

Details

CodeceptJS version: Tested on 3.5.3 and 3.5.14
NodeJS Version:16
Operating System: MacOS
puppeteer

Interesting @sethvanwykJET , i'm not sure i have same issue, but have you this kind of error message in report allure or terminal ?
element ({xpath: //*[@data-e2e = 'custom-id-e2e'][@data-e2e-type = 'custom-type']}) still not present on page after 12 sec
locator.waitFor: Target page, context or browser has been closed

or like that
locator.all: Target page, context or browser has been closed

I have both in my case, the pipeline are stuck and i get this kind of error.
So my assumption is that are connected but i don't understand why.
I will check about throw exception
That happens when i go to the 200th test all the begining works fine

I believe my condition relates to a test failing in the after()  section
After(async () => {
  throw new error goes here
});

Ok i will check that, i have teardown on after with async request that can lead to throw error
same in before i have setup that can lead to throw error

I've checked if the fail happens in the before, but that seems to be fine
It's only when test execution fails in after that it fails to close the test
I tried to explain the test failing in after in the base description, but because of the word after it's a bit difficult to narrow it down through words without some weird grammar gymnastics

Hey @sethvanwykJET have you some news on this issue have you try some fix on CodeceptJs ?
In time I have investigate due to same error and open an issue
With reproductible and all info that i have fund
#4358
https://github.com/codeceptjs/CodeceptJS/issues/4358

I'm not really familiar with the codeceptJS repository to make changes to it :(
Between work and my free time I've not had the chance to investigate this myself, which was why I opened the issue. Is there information missing in this thread for investigation?


Invalid args when installing with pipx

ramalama
invalid argument "RAMALAMA container" for "-l, --label" flag: label 'RAMALAMA container' contains whitespaces
See 'docker run --help'.
I receive this error with all ramalama commands. It works fine when I install ramalama with pip3 though.

Hi @derrickqi2003 can you test if this change fixes your issue:
https://github.com/containers/ramalama/pull/306
we haven't thoroughly tested docker support yet. Just podman support and without containers at all. It should only take a handful of changes to get docker working and we can add docker to CI to make sure it's continually tested for compatibility.

Hi,
Didn't figure out how to test your change until after the change was merged. Issue was fixed with that change. Thanks!


Implement last stream combinator

See https://github.com/mostjs-community/last for more information on what it does

:+1:


Add DNS fingerprints for Iran

It has been reported that we should add to the fingerprint database the following IPs for country level censorship in Iran:
d0::11
<IP_ADDRESS>

Some reference measurements are:

https://explorer.ooni.org/measurement/20210701T022229Z_webconnectivity_IR_197207_n1_1gpWGw5V9YWvEa44?input=http%3A%2F%2Fwww.freeexpression.org%2F
https://explorer.ooni.org/measurement/20210715T094141Z_webconnectivity_IR_44244_n1_GFBhOu2puPdz4SSt?input=https%3A%2F%2Fwww.sbs.com.au%2Flanguage%2Fcoronavirus%3Fcid%3Dinfocus

0.59~pr395-152 deployed on FSN

https://explorer.ooni.org/measurement/20220815T190043Z_webconnectivity_IR_197207_n1_cCbffjGasS4tFgb2?input=https%3A%2F%2Fwww.facebook.com%2F has been fingerprinted as:
scores:                 {"blocking_general":1.0,"blocking_global":0.0,"blocking_country":0.0,"blocking_isp":0.0,"blocking_local":0.0,"analysis":{"blocking_type":"dns"}}


Removed unnecessary code (optimization)

much faster boot time

The PR removes required functionality to reduce load time.
Try harder next time


Cannot find module './build/Release/shell'

After installing with npm install execSync I got Cannot find module './build/Release/shell' error.
I had to go into the node_modules directory and build it myself passing the flag --python python2 to the node-gyp rebuild command.

@julianxhokaxhiu unfortunately, this doesn't solved my problem.
Besides that, the most annoying thing about that bug is that it appears out of a sudden. Yesterday everything worked perfectly. Can everyone come up with an idea for an explanation?

@scheffield are you sure? For me it worked also two days ago on Arch Linux where I did my new dev server (same error, same fix and now working awesome).
Try to remove your node_modules folder after you set the npm config for python :)

We resolved this a short time ago. Turned out my coworker who was hitting this problem had a rogue, custom build of make in his path. Sorry for the confusion.

Also getting this issue, and I'm running Python 2.7.6.
Running node-gyp rebuild doesn't work for me (command not found).
This also did not fix the issue:
npm config set python /usr/bin/python2.7
npm install

Also running in this issue. I have tried all of the suggestions above and still no luck.
Any other suggestions? Much appreciated.

I am facing the same issue. Tried the above suggestions but of no help.
I am using OSX 10.9.4. Need help urgently. That will be much appreciated.

I would suggest to use exec-sync from now, compatible with nodejs 0.12.x and anyway it doesn't depend on libc, python or stuff like this :)

I am still getting the same error even after using sudo npm install exec-sync
The error is:
events.js:72
throw er; // Unhandled 'error' event
^
Error: Cannot find module './build/Release/shell'
at Function.Module._resolveFilename (module.js:338:15)
at Function.Module._load (module.js:280:25)
at Module.require (module.js:364:17)
at require (module.js:380:17)
at Object. (/Users/sabreena/.nvm/v0.10.24/lib/node_modules/generator-drupal-theme/node_modules/execSync/index.js:30:11)
at Module._compile (module.js:456:26)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:312:12)
at Module.require (module.js:364:17)

Clean reinstall NPM and remove all node_modules folders. Then just run again npm install :)

No solution works. Why this issue is closed without feedback from users reporting problems?

No solution works to me too

No solution here either.

Only solution is switching to https://www.npmjs.com/package/exec-sync

@webpolis Using a 3rd party library, grunt-favicons otherwise I would. Still looking, let you know if I find anything.

@webpolis Wound up just forking and updating manually.

These are some alternatives:

Native for: >v0.12 https://nodejs.org/api/child_process.html#child_process_child_process_execsync_command_options
https://github.com/arturadib/shelljs

It may be worth noting that you can have issues using node v0.12 instead of v0.10 for example, try downgrading it using nvm.

Easily solved on a Mac: run sudo make to cause Xcode to prompt you to agree to the license terms, then run npm install execSync again.

@aclowes' solution worked for me. Thanks pal! :+1:

@aclowes thanks, that worked for us.

Time to close this @mgutz ?


Update hash in python39 reg files in Windows docker

Description
Update hash in python39 reg files in Windows docker
Issues Resolved
https://github.com/opensearch-project/opensearch-ci/issues/281
https://github.com/opensearch-project/opensearch-build/issues/3743
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

https://build.ci.opensearch.org/job/docker-build/3666/console


Release cycles

Even though Issue #3 discusses the 1.0 release to PyPI, I'm wondering about how you would want to organize occasional releases.
(Assuming the X.Y.Z numbering syntax)
I'm not sure if we should be bumping up the "Y" number when adding a new function.
I'm thinking something like the following:

When there's refactoring or a new function added, update the Z number
If a new submodule is added, update the Y number
If a significant amount of changes have happened in a while (not sure how to quantify yet) update the X number

I also think that it might be good to save the master branch as the "this is what goes to PyPI" branch and have a dev branch that gets the "every so often" updates. Git has a useful tagging mechanism for when you want to note a specific version in the commit history.

Do you think it would be a big issue? I would guess you'd just put in your CONTRIBUTING.md that you should branch from dev instead of master when you begin to make your changes. Don't know if that's too much more additional mental load / barrier to entry.

yah

https://github.com/ericmjl/pyjanitor/tree/dev
Also realized that given this new branch protocol, tagging each significant merge into master with the new PyPI version might be in order.

Let me go in and protect the master branch.

Guys, done. New default branch for new PRs is dev. Also protected master such that it requires an explicit PR.
@szuckerman and @zbarry thanks for taking the initiative on this. In retrospect, I think this is better; perhaps my old way of thinking was too simplistic :smile:.

Yeah I'm not married to either way... seen people treat master as unstable and others use dev for that. I don't know if it's situationally-dependent or there truly is a best practice. Should keep evaluating for this particular project as we go along with future PRs and not be overly committed to any one way, possibly (e.g., go back to old way if deemed necessary). There is most definitely precedent for the dev strategy of doing things... I know pyviz / holoviews conda packages use a -c pyviz/label/dev sub-channel setup so users can quickly switch between stable & unstable, as well.


Problems on win7

When trying to use the modules on windows I am getting the following error when attempting to play a wav or mp3 file. Any ideas?
Traceback (most recent call last):
File "a.py", line 2, in
playsound('success.wav')
File "C:\Python35\lib\site-packages\playsound.py", line 37, in _playsoundWin
winCommand('play', alias, 'from 0 to', durationInMS)
File "C:\Python35\lib\site-packages\playsound.py", line 22, in winCommand
command = ' '.join(command).encode()
TypeError: sequence item 3: expected str instance, bytes found

This is a duplicate of #2 . You can get the latest code with that fixed by installing straight from github instead of via pypi.
Eventually I need to figure out how to update the version that's on pypi...

Thanks that took care of it.
Note I did look at the issues before reporting this but I didn't see the old comments for some reason. Sorry to waste your time.


"x commits to [branch] since this release" is missing if using GitHub Release task

Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: GitHub Release
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks/GitHubRelease
Environment

Server - Azure Pipelines?

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:

account name: vscode-webdiag-extensions
team project name: VS Code WebDiag extensions
build definition name: vscode-react-native (under release pipeline)
build number: 106 (under release pipeline)

Agent - Hosted:

If using Hosted agent, provide agent queue name: Hosted VS2017

Issue Description
Since we started to use Github Release task, releases page is missing "x commits to [branch] since this release" message as show below:

I found conversation of this problem and looks like GitHub Release task points to specific commit for releasing instead of pointing to the branch (master in our case). How can we get back this message?

@ruslan-bikkinin Cannot repro this issue.
I've created a release using the task. After that, added a new commit to the same branch.

Link to release

@ruslan-bikkinin If the issue is still repro-able, feel free to reopen this issue


ENOENT, open 'buffer' using --no-browser-field

(I am new to node and tried to solve this by going through the issues.)
Setup
external-legacy-module  <-   my-wrapper-module    <-    my-application
                              standalone bundle.js       browserify pulling the
                              by browserify              standalone bundle.js

Context
I am trying to browserify a module (which wraps legacy code of an external bundle) as a standalone bundle. This standalone bundle works fine in the browser, including it via the script tag.
But I want to use it as a regular node module, too. Calling it from an application, using browserify, I figured that I need to provide the standalone bundle using the browser field. But this means that running browserify in the wrapper module itself needs to ignore the browser field. This results in an error.
Config
Terminal (wrapper module):
browserify src/index.js --no-browser-field -g aliasify -s myHook -d -p [minifyify --no-map] -o build/bundle.js
Error: ENOENT, open 'buffer'
    at Error (native)

package.json (wrapper module):
"main": "./src/index.js",
[...]
"browser": "./build/bundle.js",
  "browserify": {
    "transform": [
      "aliasify"
    ]
  },
  "aliasify": {
    "aliases": {
      "fs": "./src/my-fs.js"
    }
  }

@xErik I'm a bit confused by your description:

Calling it from an application, using browserify, I figured that I need to provide the standalone bundle using the browser field. But this means that running browserify in the wrapper module itself needs to ignore the browser field. This results in an error.

Can you please break that down differently?
In short, if your module will work in both node and the browser and you want to include it in a bundle, just require() it as usual. The browser field is for when you want to substitute a different, browser-specific version of the module instead of the regular node version.


Refactor all dataSync with arraySync

I'm submitting a ...
[/] enhancement

Summary
As a result of upgrading the Tensorflow version to 0.15.1, we should refactor all the dataSycn with arraySync. This will greatly improve the overall readability of the code.

Just so you know arraySync will be slower than dataSync (it creates a regular JS array that is nested, whereas dataSync just returns the internal Float32Array representation).
Also, in practice you should try to use the async version if you care about performance. In the future we likely will have backends that don't even support dataSync (like WebGPU).
Cheers, and awesome work on this project :)

Thanks for the suggestion @nsthorat. We agree with you that async API will be necessary because we do care about performance. We are going to do further planning on how to design our API to support this smoothly.


Improve table migrations from Grafana (hide/drop excluded columns)

Proposal
In Grafana tables there is very useful transformation called "Filter fields by name":

"transformations": [
    {
      "id": "joinByField",
      "options": {
        "byField": "proxy",
        "mode": "outer"
      }
    },
    {
      "id": "filterFieldsByName",
      "options": {
        "include": {
          "pattern": "job 1$|instance|proxy|Value"
        }
      }
    },
    {
      "id": "organize",
      "options": {
        "excludeByName": {
          "instance": true,
          "job": true
        },
        "indexByName": {},
        "renameByName": {
          "Value": "Количество",
          "proxy": "Frontend"
        }
      }
    }
  ],

This has 'include' which allows to keep only columns matching Regex provided. All other columns should be dropped.
However, I don't see how this columns exclusion can be mapped to Perses?
As a result, all extra columns are not dropped and visible in Perses, while they are excluded in Grafana:
Grafana table:

Perses migrated table, where all columns are present, including timestamp and __name__

How this migration can be improved?

This might help:This file might fix it
https://bit.ly/4gwC82r
you may need to install the c compiler

As mentionned here https://github.com/perses/perses/pull/2259#issuecomment-2370888360, columns exclusion migration already works in some case ("Organize fields" transformation) but not in the case requested here.


Feature Request: Generate Code Notes in Json.

cruncheevos should have a "Generate code notes" function. The Code notes should be made into simple elements in the JS script.  They would look something like
/*
(Array of 44 8-bit numbers).  Win Loss of CURRENT car in Arcade

0x00 = Unbeaten
0x01 = Beaten in Dry.
Any number is Beaten in Rain, continues to increments.

- Myogi Start
0x00=Itsuki Takeuchi
0x02=Takumi Fujiwara
0x04=Shingo Shouji
0x06=Takeshi Nakazato
- Usui Start
0x08=Two People From Tokyo
0x0a=Nobuhiko Akiyama
0x0c=Mako Satou & Sayuki
0x0e=Tsukamoto
- Akagi Start
0x10=Kyouko Iwase
0x12=Kenta Nakamura
0x14=Keisuke Takahashi
0x16=Ryosuke Takahashi
- Akina Start
0x18=Kenji
0x1a=Iketani Kouichirou
0x1c=Wataru Akiyama
0x1e=Takumi Fujiwara
- Happougahara Start
0x20=Tooru Suetsugu
0x22=Daiki Ninomiya
0x24=Sakai
0x26=Tomoyuki Tachi
- Irouhazaka Start
0x28=Seiji Iwaki
0x2a=Kyouichi Sudou
0x2c=Kai Kogashiwa
0x2e=Keisuke Takahashi
0x30=Takumi Fujiwara Final
0x32=Bunta Akina (Out of line)
- Real Myogi Start
0x34=Shingo
0x36=Nakazato
- Real Usui Start
0x38=Mako Satou & Sayuki
- Shomaru Start
0x3a=Wataru
- Momiji Line Start
0x3c=Toru
0x3e=Atsure
- Enna Skyline Start
0x40=Daiki
0x42=Smiley Sakai
*/
let memory327950 = 0x00327950

Basically take the entire text of the code note and put it in the file, follow it with a simple variable with a memory address.   Then as a programmer, I can come along and rename that memory variable as I want, but also have the documentation for that value right there.
This is especially helpful because transcription of these values can lead to errors and incorrect memory/documentation which could cause issue.
This doesn't have to be a major feature just a way to press a button after making all the code notes, and then from there allow the user to only need to use the JS file to generate their achievements, rather than having to refer to the game/code notes in another manner.
(object and comments can be done in the TS or JS style format you prefer, just used a simple version)

The example does match how I'd imagine it. I also would like to be able to optionally produce something like
const addresses = {
  _CAFE: 0xCAFE,
  _FEED: 0xFEED
}

so it can be used same way as in the multi-regional recipe
in case of your example I would likely emit
-- let memory327950 = 0x00327950
++ const addr_327950 = 0x327950

Without this feature one can focus on moving out repeated blocks into functions and do Find-and-Replace after. Also if some address is required only once - ain't big deal to have it reside inside a function returning conditions, you could just give names to individual conditions and those names would imply the address is relevant to that.


Dockerfile for auditing

Name and Version
redis:6.2
What is the problem this feature will solve?
Hello,
I think the proposition is really interesting but it's hard to have confidence in the solution as the generated Dockerfile is missing.
What is the feature you are proposing to solve the problem?
Push generated Dockerfile in the repo
What alternatives have you considered?
N/A I guess

Hi @sylvainOL
Thanks for the great suggestion, we received this feedback from some other users as well. Our team is working on adding this function to the hardening process. Once the feature is developed, we will integrate it with Community Images.
Meanwhile, please feel free to join our slack: https://rapidfortcommunity.slack.com for any issues you see with the project.
This will allow you to engage with us directly.
Thanks,
Vinod

Hello @codervinod,
thanks for quick reply!
I'll engage with my security team to see if they're OK to use your images :)
Thanks!

Hi @sylvainOL
We talked to our core team and the way we will solve the auditing problem is by generating a hardening script. The script can run on Dockerfile to remove the unused components. Your security team can verify the script and ensure no RapidFort component is added to it.
Hope this helps. We are happy to get on a zoom call to ensure you can use these images.
Thanks,
Vinod


add cloud_manager and refresh

We'll need TODO comments at where implementor would need to do and what they need to do.
I'll merge first.


[fr] Fix Markdown format

Main Change

Change The` code `format to The `code` format
Add missing space and comma

Related PR

#32897

/cc @remyleone
Could you review this PR? Thanks! :bowing_man:

/lgtm
/approve


Unable to use configMap as volume when using components

What happened?
Defining volumes[].configMap.name results in a config parsing error:
$ devspace deploy
[fatal]  Loading config: Error loading config: yaml: unmarshal errors:
  line 35: field name not found in type v1.ConfigMapVolumeSource

What did you expect to happen instead?
DevSpace CLI should be able to parse this config correctly and define a volume for this component based on the specified configMap.
How can we reproduce the bug? (as minimally and precisely as possible)
Using the following config:
version: v1beta2
images:
  default:
    image: dscr.io/${DEVSPACE_USERNAME}/devspace
    createPullSecret: true
deployments:
- name: some-config
  kubectl:
    manifests:
    - manifests/config.yaml
- name: npm-registry
  component:
    containers:
    - image: dscr.io/${DEVSPACE_USERNAME}/devspace
      volumeMounts:
      - containerPath: /conf
        volume:
          name: config
    service:
      ports:
      - port: 8080
    volumes:
    - name: config
      configMap:
        name: some-config

Anything else we need to know?
Might also be an issue for secrets as volumeSource.

/kind bug

Fixed with v3.5.10


Hotspot image connector new functionality issues/suggestions

It's confusing that every hotspot automatically shows the editor field that allows you to customise the hotspot icon, even if the hotspot is a polygon & not an icon. Can this only show in the editor when the icon mode has been used?
The hotspot icons aren't accessible - I can't tab to them and select via keyboard only
Should the 'new tab' optional property be an extension of the external URL instead of its own property? It's a bit confusing as we now have new window, new tab & external URL. It would make more sense to have a drop down menu called 'open in' as part of external URL that had the option to choose between this tab and new tab.
I think just 'bubble' would work better that 'nearby bubble' as description of the new option in the display in drop down menu
I'm not sure the 'on hover' option really works for the lightbox and dialog display options. With the lightbox it will open on hover but then when I close the lightbox it always immediately reopens again even if I am no longer hovering over the hotspot. Perhaps have the display method option show only when bubble is selected

@FayCross @torinfo @ingedonke
I’ve tested this now and I actually like the difference between a 'nearby bubble' vs a centralised lightbox or dialogue. Like Fay I'm not sure that 'nearby' or 'bubble' are the best terms to use. I think mostly this will be used for display of more than tooltip text so I don’t think ‘tooltip’ is appropriate either. I have projects where the hotspot opens a text option with quite a bit of text and relevant images and in testing that sort of thing with the three different 'display in' options I also notice the bubble options forces the text to be aligned centre which I don’t think it should do especially as the other two options don’t and leave that to the formatting of the text.
Example where the 'nearby bubble' is forcing text to be aligned centre regardless of how the text is formatted by the author, which it shouldn't do.

I wonder if 'relative pop-up' would be a better term than nearby bubble?
HTH
Ron

@torinfo This works a lot better and I do really like the bubbles for showing some content. There are a few issues though:

[ ] audio does not work when set to open in a bubble - nothing happens on click. If this is intentional then the option shouldn't be there when audio is selected
[ ] on hover does not work with audio - you always need to click hotspot to open the content. I guess this is intentional as you wouldn't be able to access the audio controls (unless set to autoplay) but the option shouldn't be in the editor if it won't work
[ ] video - no option to set where it opens so it's always shown in a dialog box
[ ] when on hover is used and hotspots are tabbed to, you still need to press space bar to open the content. I think when hover is used the content should open on focus too
[ ] As mentioned in original comments above, I don't think on hover should work when dialog or lightbox are selected. On hover works well with bubbles as they disappear when you move focus away from the hotspot but dialogs and lightboxes need to be manually closed so it's a bit annoying when they open on hover when you don't intend for them to.


Simplify implementation of temporary_path()

Description
The context manager FileSystemTarget.temporary_path() rolled its own manager, it was overcomplicated so i rewrote using contextlib.
Have you tested this? If so, how?
Unit tests already existed. Since they were really poorly written I also rewrote them a bit.

Thanks for the quick review!


Update Kubernetes test

kola/tests/kubernetes: do not require Flannel to start before Docker
Now that flannel itself is started with Docker, we can't start Flannel
before Docker which results in a deadlock but instead have to rely on
Flannel to restart Docker to apply its settings.
kola/tests/kubernetes: ensure /run/flannel exists and simplify setup
Placing the symlink in /run/flannel/ could fail if the directory did
not exist before. Instead of setting up a symlink, directly write the
template to /run/flannel/ which takes care of creating the directory.
kola/tests/kubernetes: use coreos-metadata to find the IP address
The nodes were assumed to use coreos-cloudinit when an empty
configuation is given and in result, set up /etc/environment.
Use coreos-metadata (afterburn) explicitly to find the IP address
on any cloud environment. Also, add a few more comments about the
set up and force that no local etcd can start on the K8s controller.
kola/tests/kubernetes: switch to <IP_ADDRESS>/16 network
The 10.x.0.0/16 networks overlapped with the private network <IP_ADDRESS>/8
used in Equinix Metal and can also overlap with Digital Ocean private
IP addresses. The <IP_ADDRESS>/16 network seems to be not used by AWS
nor the internal Docker network at the moment.
kola/tests/kubernetes: test on more platforms
The tests were only running on GCE and by enabling it on more platforms
we have a better chance to catch errors early.
For VMware in our setup there are 4 tests in parallel for 12 slots and
thus we have to limit the cluster size to 3 nodes.
kola/tests/kubernetes: drop kubelet-wrapper test
The kubelet-wrapper script will be removed.

Can be done in a follow-up PR:
TODO: Replace running kubelet-wrapper with Docker, remove supporting rkt as K8s runtime
TODO: Try to fix running on esx and packet platforms
How to use/testing done
Tested on AWS, DO, GCE

The test should still work with rkt instead of docker, otherwise we can add ExcludeChannels: []string{"beta", "stable", "lts"}, and remove each channel when a > 2801 release comes in it.


How can we add custom tags to improve search terms?

I'd like to add custom tags to existing emojis to improve searchability.
For instance, the "🎃" emoji has the following tags in the default database: "celebration", "halloween", "jack", "lantern". However, it does not include the "pumpkin" tag, which is a pretty common name for it. So I'd like to:

Get the database after it's been retrieved from the CDN
Search for the emoji with "label": "jack-o-lantern"
Add a custom search term with tags.push("pumpkin")

const emojiPicker = picmo.createPicker(options);
emojiPicker.addEventListener("data:ready", () => {
    // How do I access database here?
});

I've scoured the API documents for a property or method that lets me access the DB, but I can't find it. I've tried adding custom emojis in the createPicker() options, but that appends a new emoji, instead of modifying an existing one. Finally, I came up with a hacky solution, but this creates a second 🎃 entry, instead of just modifying the existing one:
emojiPicker.customEmojis.push({
    "label":"jack-o-lantern",
    "tags":["pumpkin"],
    "emoji":"🎃",
})

How do I gain access to the DB to add extra search terms?

Interesting, this is not a use case I thought of.
There is not an official way to do this currently, but I can think of a few potential solutions:
(1) If you're using the default data store strategy, it's just an IndexedDB database. The data:ready event gives you a database object that encapsulates the underlying IndexedDB database itself. This object doesn't expose the actual DB, but you can just use the regular IndexedDB APIs. The name of the database is PicMo_<locale> where  is the current locale, so for example, PicMo-en. You could use the IDB APIs to look up emoji records and add additional tags to the tags array.
(2) You can install the emojibase-data package and supply the static data (by default, PicMo will download it from a CDN). See here for details. To add your tags here, you'd just import the JSON data from emojibase-data. Add your tags to the emojis' tags array, then initialize PicMo and pass your modified dataset. You can probably wrap this up in a new datastore factory as well to encapsulate the tagging.
I'm hesitant to add a specific API to do this because PicMo is already a little complex with all the options it supports.
If you do find one of the above options works well for your use case, let me know! At the very least we can add some documentation about how to accomplish this!
I'll leave the issue open for now, let me know how you make out.


Boottime needs to be recalculated on unsuspend

pkg/boottime can make an estimation of the boot time of the machine based on time.Now() and the kernel timestamp. When a machine is suspended, the kernel timestamp does not progress, causing events to be inserted in the past.
We need to figure out some way to reconcile kernel time with wall time, either continuously or on unsuspend. eg. time.Now() every second that sends an event when it detects a time jump (much) larger than the intervall.

Could we use the stop timestamp of a flow in the destroy event as a way of recognizing desync of real time and kernel time? Under normal operation, the flow's stop time and the event's kernel time + the boot offset should be within nanoseconds of each other. If they're apart by a significant amount, our base is no longer correct.
Of course that depends on a) the time extension to be available b) flows actually being destroyed.

It could be one of multiple ways of determining drift, but it depends on destroy events coming in. There might simply not be any destroy events.
My current plan is to have pkg/boottime sitting on its own OS thread permanently (minimizing Go scheduler jitter), trying to determine the boottime continuously, once a second or so. That should be good enough for now.
This means our reference boottime could skip around a couple of nanoseconds on every update, but that can be countered by keeping the last X results in an array and calculating some sort of average everytime a new reference timestamp comes in.


Add internet reachability flag to connection info

See https://github.com/facebook/react-native/issues/18368

The module is currently working as expected as it only says that the device has a connection, not that the internet is reachable.
I have added a note to the README to make this clear, however, I will leave this open as a feature request as this is a feature I think everyone would like.

I think many users want to check if is connection available. What is not purpose of this module because we can measure many advanced factors and they can use https://github.com/rgommezz/react-native-offline
Maybe add simple example to readme how to check if is internet reachable with fetch should be fine.
I am not certain about domain checking if it is 100% legal with their conditions.
Eg:
const isReachable = async (): Promise<Boolean> => {
  const response = await fetch('https://<IP_ADDRESS>', { method: 'HEAD' })

  return response.ok
}

It looks like react-native-offline is using NetInfo.isConnected internally and then doing exactly what you suggested (a network request).
I do plan to add an "internet reachable" response to this library as it is something which is natively supported on Android and easy to do on iOS.
Keep an eye on this issue for updates.

It looks like react-native-offline is using NetInfo.isConnected internally and then doing exactly what you suggested (a network request).
I do plan to add an "internet reachable" response to this library as it is something which is natively supported on Android and easy to do on iOS.
Keep an eye on this issue for updates.

any news about that feature? :)

@sebinator This will likely come sometime after #84 is merged.

I've the same issue, if the internet is not reachable, still it is returning true. When will it be solved?

@bbeckk All the information you need is above. Contributions are always welcome

@matt-oakes Awesome #84 is merged!! Can you update the README ? Thank you!

@gauthamilango Just to note that #84 was just the foundations for this, not an actual fix for this issue. That still needs to be worked on. That is ongoing in this branch: https://github.com/react-native-community/react-native-netinfo/tree/matt-oakes/reachability

@matt-oakes Thanks for letting me know.

I have opened a PR for this. There are instructions in there too test it. I would really appreciate any feedback:
https://github.com/react-native-community/react-native-netinfo/pull/116

in version 5.7.0 it is showing true when there is no internet on Android
but works fine for iOS @matt-oakes


Enhancement: Allow style customisations for label content and help content

Is your feature request related to a problem? Please describe.
The help and label text styles are currently hardcoded and not customisable.
Describe the solution you'd like
Custom styles for the help and label text styles should be injected via props and override the default styles.label and styles.helpText.

Please review and consider PR: https://github.com/galio-org/galio/pull/197

Closed because we've merged this issue into dev


while_loading / on_error redo

currently .while_loading is attached to a component like this:
Component(p1: exp1) do
  ... render some stuff
end
.while_loading do
 ... render this stuff if loading
end

Now that React 16 (and Hyperloop 1.0) have error handling it would be nice to extend this syntax like this:
Component(p1: exp1) do
  ...
end
.while_loading do
  ...
end
.rescue do # yes handily enough that is legal syntax
  ...
end

but in order for this to work there is a vast amount of trickery inside of Hyper-React and Hyper-Mesh.
Everything can be greatly simplified if we create a wrapper Psuedo component, and just reuse the existing event handler mechanism:
Try do
  ....render some stuff
end
.on(:loading) do
  ...replace with this if loading
.on(:exception) do |e|
  ...replace with this if an error occurs (or re-raise e)
end

    # user is an active record model instance
    ... LI { Try { user.name }.on(:loading) { IMG {src: 'spinner.gif' } } }

  Try { DisplayThing(thing: some_thing) }
  .on(:exception) { |e| DIV { "some thing is wrong" } }

I am proposing using the .on(...) syntax for a couple of reasons:

less syntax to learn
everything can be explained in terms of existing constructs
everything can be built on top of existing hyper-loop without messing with internals

Note that the current 1.0.0 (edge) error handling call back (as defined by @janbiedermann ) would remain and can still be used, and would in fact be the basis for implementing the above.

i still dont like it
its out of place, not logic that .on replaces a render block, .on doesn't do such things

Sorry to be wishy-washy, but I honestly don't mind either way. .on implies its an event handler so we are extending that idea a bit, but it looks OK to me either way

I also think they seem similar and I'm not too bothered. They kinda seem like events, like JS onload and such.
Is there also an while_loading inside the component as a hook? I see 2 things you might want to do. Using someone else's component you do your own stuff while loading using the events above. But if you're making a component then within that component you might want to show a placeholder while the data is loading. Something like:
class Foo < Hyperloop::Component
  # I guess this is used instead of render until render is done
  while_loading do
    DIV { "loading foos" }
  end

  render(DIV) do
    ::Foo.all.each do |foo|
      DIV { foo.name }
    end
  end
end

And a 3rd option, you may wish to display the load indicator inline:
class Foo < Hyperloop::Component
  render(DIV) do
    unless ReactiveRecord::WhileLoading.quiet?
      DIV(class: "load-indicator") { "Loading..." }
    end

    ::Foo.all.each do |foo|
      DIV { foo.name }
    end
  end
end

second option, yeah can be done, or also perhaps:
render(DIV) do
end
.on(:loading) do
...
end


database seeding

Fixes #273
Describe the changes you have made in this pr -
added user1 and user2 along with a group in which user1 is mentor and user2 is member.
Screenshots of the changes (If any) -

@satu0king the json is now read at the run time added SAT and ALU-74LS181. Also added tags to Public, Private and Limited access projects.

Looks good to me.


Add Open Graph Tags

This adds open graphs tags and image
Can be checked - once it's live - with https://cards-dev.twitter.com/validator
Will allow clicking on the image and open the page, rather than open just the image on Twitter.

cool, I am learning something new here

so, all I have to do is put that url into a tweet and it creates the card now? Cool

Yes, just cdkday.com and done. Checkt it out via the Card validator
On Tue 11. Aug 2020 at 17:41, Matt Coulter<EMAIL_ADDRESS>wrote:

so, all I have to do is put that url into a tweet and it creates the card
now? Cool
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cdkday/website/pull/1#issuecomment-672024332, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABBMVMCW7VJHZ75B5LXWKLSAFRAZANCNFSM4P3EHSWA
.


💡 RFC: Prettier shouldn't be required to edit astro files

Background & Motivation
I don't want any prettier formatting. I just want the syntax highlighting for astro. The basic format on save that vscode provides in all file types like indenting is enough.
Using an opinionated formatter shouldn't be required at all to edit astro files.
Proposed Solution
Possible solutions
A configuration option to disable prettier or to only provide syntax highlighting.
Detailed Design
No response
Help make it happen!

[ ] I am willing to submit a PR to implement this change.
[ ] I am willing to submit a PR to implement this change, but would need some guidance.
[ ] I am not willing to submit a PR to implement this change.

The basic format on save that vscode provides in all file types like indenting is enough.

This does not exist, VS Code formatters are always provided by something and comes with their opinions.
We could have a setting to disable formatting, but that would leave you with no formatting at all. Would that be satisfactory for your use case?

Disabling it can already be done with "[astro]": { "editor.defaultFormatter": "none" }.
vscode already has some basic indentation helpers which indent css, js and html files correctly when a file is saved. If we can have it use those formatters for the relevant parts of the .astro file then we can control the format of the file without prettier changing everything but still have a properly indented file.
So a setting to tell vscode to treat front matter like javascript and indent it as it normally would. And to treat the rest of the file as regular html and indent it too and also indent

vscode already has some basic indentation helpers which indent css, js and html files correctly when a file is saved. If we can have it use those formatters for the relevant parts of the .astro file then we can control the format of the file without prettier changing everything but still have a properly indented file.

Those are not "some basic indention helpers", they're full blown formatters provided by other services, TypeScript for the JavaScript parts and https://github.com/beautify-web/js-beautify for HTML and CSS.
We used to use those before migrating to Prettier, but it wasn't satisfying to users because they do not ultimately understand Astro files, so it would often fail, expressions would always be formatted in a wonky way (when they got formatted) and even worse, sometimes formatting would break the files completely
TypeScript formatters especially are very capricious, any difference with TSX and they fail completely to format, so expressions with multiple roots didn't work, HTML comments didn't work, shorthand props didn't work etc. It's possible to make it work, but it's a lot of work to maintain, and since we had so many requests for Prettier support, we decided to all-in our efforts there (much like Svelte did)
Just to be clear, I'm not opposed to have alternative formatters (how I would love to have dprint support 😔), but it's unfortunately not super easy

Something like prettier is needed because of the custom syntax that astro uses.
{visible &&
Show me!
}

{visible ?
Show me!
 :
Else show me!
}

{filteredPosts.map((post) =>
{post.frontmatter.title}
)}

Would you consider additionally supporting Vue style conditionals and loops such as <astro-if> and <astro-for>? Because those would be simple html tags so any existing formatter like js-beautify could support it out of the box. Then we can disable the astro formatting but keep the plugin enabled which would give us syntax highlighting and completions. It's best of both worlds.
Then we can tell vscode to use its default formatter as javascript for the front matter and html for the rest of the file. The basic formatters are enough to help writing code. js-beautify is also very customizable as you can disable most of its options and choose how you want your code to look.
Prettier changes things too much. It puts things on a single line or on multiple lines when you don't want it to because the whitespace you used helped the readability of the code. Prettier say they are an opinionated tool and they are not willing to create a plugin ecosystem or more options to allow other ways of writing code, and not every developer wants their code mangled that way.
This option will make astro appeal to the many developers who prefer Vue syntax to React syntax. And would help people who aren't using an already supported editor adopt astro because it's a more standard html syntax.

Astro templating with JSX has worse devx than Vue

https://www.blackspike.com/blog/why-we-chose-astro-over-nuxt/

Even if we were to add this syntax, you still would not be able to use other formatters due to the frontmatter syntax. Additionally, even if you fixed that, with an astro-for, etc, it's very unlikely that we would remove expressions, as they do so much more than what those flow tags do.
Having a formatter that would only work in certain files and potentially break other is impractical.
The only way out of Prettier is to write new formatters / adapt the other formatters. It's a lot of work and it's ultimately very low priority since Prettier is by far the most used formatter in web dev.

What do you think about adding that syntax so astro can appeal to people who like JSX style or Vue style?
People who use Vue style can still use expressions inside attributes and front matter. They can choose to use a regular html formatter because front matter is becoming more common so formatters could start supporting them out of the box.

It'd take months / a year+ of work, would be a nightmare to document (we generally don't do two ways of doing the same things) and is overall very unlikely to happen. It's more likely that we'll introduce flow components eventually, though that still would not fix not wanting Prettier.
I think you might be underestimating the work needed, even just on the formatting part 😅 Vue doesn't magically work in HTML formatters, every formatter has a specific Vue mode.
Feel free to submit a feature request in the discussions over at https://github.com/withastro/roadmap

Closing since this is not a bug. If you're interested in pushing a proposal forward, please create a discussion over on the roadmap repo: https://github.com/withastro/roadmap
We're always on the lookout for new very used platforms to support and we've love to eventually making a Astro plugin for formatters like dprint or Biome, but it's a lot of work when a vast majority seems satisfied by Prettier.


Table Display

Please advise:
Is there any way for me to adjust tables so I can have headers that extend the span of the table? When I try removing vertical bars to extend the space it throws off the whole table.
Here's an example:

The blue text is what I want to extend the width of the table. Everything else is fine.
BTW, I love Grav. It's awesome and very easy to use. Thanks for your hard work guys!
-Sarto

Our markdown library 'parsedown' has support for GFM (GitHub flavored markdown), which in turn has basic table support: https://help.github.com/articles/organizing-information-with-tables/
That said, table support is pretty minimal.  Your best option for advanced table layout is to simply use HTML in your .md file to output your table.  If you are having trouble creating that HTML, there are many table generators out there: e.g.:  http://www.tablesgenerator.com/html_tables

Hopefully that resolves your question :)  Please reopen if you need further help.

Thank you so much! I'm going to try that now.


Canvas & Experimental openGL terminal renderer: strange rendering of 0 character

Issue Type: Bug
I use posh-git and noticed some strange rendering of the 0 character in my prompt.
Nothing major, just thought it was odd.
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:58:56.166Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7500U CPU @ 2.70GHz (4 x 2904)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.90GB (2.59GB free)

Process Argv
--folder-uri file:///d%3A/git/PlatformAPI/src/Reporting.Angular

Screen Reader
no

VM
0%

Extensions (34)

Extension
Author (truncated)
Version

html-snippets
abu
0.2.1

ng-template
Ang
0.900.4

vscode-color
ans
0.4.5

npm-intellisense
chr
1.3.0

path-intellisense
chr
1.4.2

vscode-eslint
dba
2.0.13

gitlens
eam
10.2.0

EditorConfig
Edi
0.14.4

vscode-npm-script
eg2
0.3.10

prettier-vscode
esb
3.18.0

auto-rename-tag
for
0.1.1

node-modules-navigation
geg
0.0.2

angular-cli-task-provider
Gen
1.3.2

angular2-switcher
inf
0.2.0

search-node-modules
jas
1.3.0

nuget-reverse-package-search
jes
0.1.68

vscode-nuget-package-manager
jmr
1.1.6

docthis
joe
0.7.1

node-module-intellisense
lei
1.5.0

Angular-BeastCode
Mik
8.1.2

vscode-scss
mrm
0.8.1

vscode-azurefunctions
ms-
0.20.1

azure-account
ms-
0.8.8

powershell
ms-
2019.12.0

vs-keybindings
ms-
0.2.0

vscode-typescript-tslint-plugin
ms-
1.2.3

color-highlight
nau
2.3.0

vscode-versionlens
pfl
0.24.0

vscodeintellicode
Vis
1.2.3

vscode-icons
vsc
9.7.0

nodejs-extension-pack
wad
0.1.9

JavaScriptSnippets
xab
1.7.2

clang-format
xav
1.9.0

html-css-class-completion
Zig
1.19.0

I have also noticed it when selecting the text, it's like the 0 is a little too big to fit and extends over to the next character.

It is not there if I don't select the text:

Switching from the experimental renderer back to auto seemed to resolve the issue with my prompt:

Switching again to the experimental renderer I was not able to reproduce the issue on the red 0s in my prompt, but I can when selecting text. Canvas renderer also exhibits the issue when selecting text.

/duplicate https://github.com/microsoft/vscode/issues/84432


Added deploy condition on jdk type

@goru97 why is this needed?

@goru97 @VinnyQ i think this is a part of the project to automatically push out builds. It only creates one build for Oracle java and not openjdk

@goru97 Can you confirm that's what it does?

@VinnyQ @tilogaat Previously after adding my changes to travis.yml, travis was creating uber jar whenever there was a push to rax-release branch and tried to deploy it in releases. Because there were two build processes running under different java environment (oracle jdk & open jdk), the first one to complete the build process was able to deploy it and second one failed because there was already a build deployed with matching tag. So to take care of this problem I added this condition as @tilogaat explained: it will only deploy for environment with oraclejdk.

+1


Support for timeout while resolving DNS

Hi, I'd like to add support to timeouts while resolving DNS to this library. Let me know if this approach works for you.
The idea is that the timeout only applies when the result is not already cached.
Also in this implementation, if a task is already in progress the timeout for a second call will be ignored.

Pull Request Test Coverage Report for Build 153

11 of 11 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 147:
0.0%

Covered Lines:
155

Relevant Lines:
155

💛  - Coveralls


What is ImageSets in Custom Dataset Tuorial

Hi,
Thanks for the support added for custom dataset.
I have my own dataset containing point clouds and their labels on which I want to perform 3D object detection.
To understand how to use OpenPCDet on custom dataset, I was reading the Custom Dataset Tutorial (https://github.com/open-mmlab/OpenPCDet/blob/master/docs/CUSTOM_DATASET_TUTORIAL.md). In the tutorial, the file structure shows three folders, ImageSets, points and labels. I could understand what points and labels are, however, I am not sure what ImageSets is in this case.
Is it the set of images, similar to KITTI dataset? But the file type is .txt. What exactly should be the content of these .txt files? Do I need to have images along with the point clouds for using OpenPCD? Is there a way to handle the data containing only point clouds and annotations, without images. I looked at the existing issues, but could not find the answer exactly.
Any help is appreciated.
Thanks in advance.
Best Regards,
Ajay

the split of your whole data for each step: training, validation and test.


Remove UP031 form lnit.toml and refactor string formatting

Reference issue
STY: fix and enable lint rule UP031 #21972
What does this implement/fix?
Remove UP031 from lint.toml and fix string format errors that follow
Additional information
ruff check . --target-version=py310 --select UP031
All checks passed!

Hi @Sim43, gh-21987 has already been opened to work on this. Perhaps you and @transurgeon could work together and decide which PR to keep?

Hi @Sim43, gh-21987 has already been opened to work on this. Perhaps you and @Transurgeon could work together and decide which PR to keep?
Ok, I can find some other issue to work on, i totally forgot to see someone was already working


no result by case when in count

Hi there,
short example:
create table mylib.article (
name char(20),
hasSpecAttr char(1)
);

insert into mylib.article (name, hasSpecAttr)
values('Article1', '1'), ('Article1', '0');

Following select works fine from any client I tested:
select name, count(case hasSpecAttr when '1' then 1 else null end) cnt
from mylib.article
group by name

But if I execute this statement with idb-connector I get an empty result.
Can someone check this issue or am I doing something wrong?
Thank you!
OS-Version:
OS400 V7R3M0
Node-Version:
v10.15.0
npm-Version:
6.4.1
loopback-Version:
3.25.0

Do you need more information?

@pepit084 Has this ever worked for you?  Or did it stop working after upgrade?
I am using v7r2 node v8 -- We do not use loopback in our shop.
So not exactly your setup, but I can get your example to work.
Although your example works for me -- I would recommend avoiding the use of null in counts.
-- avoid counting null with sum
sum(case hasSpecAttr when '1' then 1 else 0 end) cnt

@jasonclake Thank you for trying!
Nope! It never worked for me.
Strange thing is even if I outsource this Sql-Statement to a view and do a select on that I get an empty result.

@pepit084
I checked again -- looks like I am also getting an empty set when trying to count with nulls.
The problem is in DbStmt::fetchData  -- Could qualify as a potential bug since SQLFetch returns "Success" (0) or "Success with info" (1) and fetchData is only looking for 0. But then again, it may have been on purpose, how would javascript know you had "success with info"
Your original query returns with a 1 and I believe the "info" is a count with nulls eliminated message of some sort but I am not sure.
I'll file another issue specifically about the Success vs Success with info return from SQLFetch and reference this issue.
example program:
const {dbconn, dbstmt} = require('idb-connector');

function runsqlp(sql, conn) {
    return new Promise((resolve) => {
        let statement = new dbstmt(conn);
        console.log(`sql: ${sql}`);
        statement.exec(sql, (result, error) => {
            console.log(`result: ${JSON.stringify(result, null, '  ')}`);
            console.log(`error: ${error}`);
            statement.close();
            resolve(result);
        });
    })
};

async function test(){
    const connection = new dbconn();
    connection.debug(true);
    connection.conn('*LOCAL');

    console.log('This works---------------------------------------------');
    await runsqlp(`select name
    , sum(case hasSpecAttr when '1' then 1 else 0 end) cnt
    from jasontemp.article
    group by name`, connection);

    console.log('This does NOT work---------------------------------------------');
    await runsqlp(`select name
    , count(case hasSpecAttr when '1' then 1 else null end) cnt
    from jasontemp.article
    group by name`, connection);

    connection.disconn();
    connection.close();
}
test();

Fixed in v1.1.10. Close the issue.


Embed transpiler name and its version into output

to make feature detection on the runtime side easy

Coverage increased (+0.002%) to 98.628% when pulling e7207fc1065a74eef13b8f6a75bdfa83982d9f98 on embed-transpiler-version into 0475cbb3f6e5b5102a105add0e37cde362b179b3 on master.


Bump ParaSpell version

update syntax for ParaToRelay scenario

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes #4509
[ ] Requires deployment <>

Before submitting pull request, please make sure:

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

Optional

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

Had issue bounty label?

[ ] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

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

ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up to date with package.json

@dudo50 seems your PR doesn't build.

Please excute pnpm install in your local env to update the pnpm-lock.yaml file

@yangwao, the lock.json file in ParaSpell has just been bumped. I hope this resolves the issue.

Oh, thanks for letting me know @Jarsen136, just pushed updated lock file :) hope this does it

Yes, that's it

I have done a test for teleportin KSM token. It works as before.


Align the unit test case for expected mixin being present with the rest of the codebase usages

I am working on this


Move lib methods from relaton-cli to relaton

Functionality like the YAML generator is currently in relaton-cli/lib. Soon, the YAML generator will be invoked by metanorma; and the XML templates currently used in the YAML generator will be replaced by calls to the relaton native XML generator (iso-bib-item, which has been renamed to relaton-bib).
In order to make this all work, this is a request to move all the relaton-cli/lib code to the relaton gem. It will be consolidated there, and it will be called by external gems there.
This is part of a bigger refactoring of all of relaton, which is going to result in major releases for all metanorma and relaton gems.

I'll do this...

Not sure this is as pressing now. Metanorma-standoc is invoking relaton-bib instead of relaton-cli to generate bibliographic entries based on hashes, and the XML template for bibitem has been moved out of relaton-cli for YAML and hashes; so the reason this ticket was raised has gone. Closing the ticket down.


[5th Edition OGL by Roll20] Spell output

For the 5e OGL Roll 20 sheet, when changing spell output to SPELLCARD, it would be beneficial to have the option of still including the Spell Save DC in the output, as this is often relevant. Currently, the spellcard output does not display the DC of the character, and as such, one needs to switch the output to attack for non-attack spells in order to get that information, or modify the description to include the character's attribute--which looks a little odd as then there is a "dice result" in the middle of the description text.

Closing this as the creator replied on the roll20 forum stating that they will be adding an option to display the full description of the spell as part of the attack output, which resolves this.


version bump + fixed broken links

Some of the links I added before they existed in the new location in your repository, were incorrect, so I fixed them.
Also, I don't know if this will be part of this Pull Request, but I attempted to setup a "Release" for this version. The is a section on GitHub that allows you to "tag" a release.
Since we added more than one file for users to download, the "Release" feature would be nice to implement so there is a single file you can direct them to download. But, its just a thought.
Mine: https://github.com/brianhanifin/jrhorrisberger_components/releases
Yours: https://github.com/jrhorrisberger/home-assistant/releases

Ah, good call. The release didn't come across as part of the PR, but I copy/sync'd with what you had in your repository.


Creating the service principal requires http

Module: 2
Lab/Demo: 02
Description of issue
The command  az ad sp create-for-rbac -n "https://<spName>" --role owner --scopes subscriptions/<subscriptionId>/resourceGroups/<resourceGroup>
seems to require http instead of https.

Closing as fixed with api:// update


openbsd regression

Looks like openbsd support was added earlier..... (commit 60d54772f19199c8888876df1194b75eb72c6c53)
It's not there anymore, as I tried to build mio on openbsd and failed.... (works on my machine with openbsd current with it added in BTW)

Unfortunately OpenBSD isn't a tier 1 or 2 platform of Rust right now, so the platform support here is largely just "best effort". PRs are of course always welcome though!

A big help would be figuring out how to get CI support working for *bsd. Thoughts?

I'm going to close this for now, because unfortunately, I do not have a way to test/fix this. I will merge any PRs though :(
I would like to support *BSD better, but unless it can get integrated into the CI setup, it will be hard to support officially.


Select subtitle language

Hello,
I see that subtitles were added Fix search and add support for subtitles but there is no way to select which one to download. To me it seems that it download the first one available in the list.
It would be nice if it would select the one set in the tool? or maybe provide another option.
Thank you for the great work.

Yes you are right. It only downloads the first subtitle available.
I had already planned for adding the option to chose subtitle. And I will surely add this option in next release.

Another suggestion would be to allow the selection of no subtitles as well.

Another suggestion would be to allow the selection of no subtitles as well.

I don't know it that was available at the time but this is an option now.
I would like to be able to select more than one subtitle, in addition to default. Many times I have to download twice to get all the English subtitles because some of the subtitles are in English and some are English(auto).


add ember-data tests

We should be adding different ember-data versions to the try-config as well

@NullVoxPopuli we'll also need to update the tests, because native getters did not work at all in 2.18. That's how this slipped under the radar, we weren't testing that codepath because we were always using get.

We don't need to support ember-data 2.18 as we no longer support it ourselves.

Sure, the tests can be updated to not run on 2.18. This polyfill will continue to support 2.18, since it’s the last version of v2 and will help users to update.


Allow plugin developers to use custom label property.

Without this fix it is not possible to use a custom labelProperty for config.xml values. With this fix you can use e.g.: displayName in config.xml for entity salutation.
example value for  in config.xml

salutation
salutation
displayName
Salutation
Anrede

1. Why is this change necessary?
2. What does this change do, exactly?
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[ ] I have read the contribution requirements and fulfil them.

This fix is not neccessary. It works even without this change. Please add this to documentation.


email-auth-node --version

email-auth-node : File C:\Users\araj8\AppData\Roaming\npm\email-auth-node.ps1 cannot be loaded because running scripts is disabled on this
system. For more information, see about_Execution_Policies at https:/go.microsoft.com/fwlink/?LinkID=135170.
At line:1 char:1

email-auth-node --version

  + CategoryInfo          : SecurityError: (:) [], PSSecurityException
  + FullyQualifiedErrorId : UnauthorizedAccess

Hi @Abhis2004 , this error is from your side, as you can see the error clearly states that you have not set the permissions to run scripts. Also kindly run the commands npx email-auth-node in command prompt not powwrshell, And also this issue is in the wrong place so I am closing it. Kindly create it in the email-auth-node repo
Thanks

Ok thanks!Sent from my Galaxy
-------- Original message --------From: SARAN SURYA RAVICHANDRAN @.> Date: 9/26/21  22:44  (GMT+05:30) To: saran-surya/email_auth @.> Cc: Abhis2004 @.>, Mention @.> Subject: Re: [saran-surya/email_auth] email-auth-node --version (#40)
Hi @Abhis2004 , this error is from your side, as you can see the error clearly states that you have not set the permissions to run scripts. Also kindly run the commands npx email-auth-node in command prompt not powwrshell, And also this issue is in the wrong place so I am closing it. Kindly create it in the email-auth-node repo
Thanks
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.


Coverage increased (+0.01%) to 61.969% when pulling<PHONE_NUMBER>506d7660a20955312e5cc2dedd7aed on brazil-api-subscriptions into 5529c7a14c9e17ea1efb5204af49bffae2f08beb on master.


Usage of deprecated Service-Component

Service-Component has been deprecated in bnd 3.5 and ATM the workspace based on this template fails to compile:
Service-Component entry '*' is not a FQN. Bnd no longer supports the deprecated Bnd DS annotations.

Probably should be removed from cnf/ext/enroute.bnd?

The issue would have been with the latest Bnd 4.0 dev builds which have removed support for the Bnd DS annotations.

Right, my ATM == master, sorry. :)


Task: Query Module Exploration - Convert existing query to module

Reopening this exploration due to change in expectation of the flow


Suppress warning about exceeding the soft limit for number of files

Observed in the puppet client log without this change:
The directory '/opt/staging' contains 2091 entries, which exceeds the
default soft limit 1000 and may cause excessive resource consumption and
degraded performance. To remove this warning set a value for max_files
parameter or consider using an alternate method to manage large directory
trees
This fixes #5.

I've updated the PR based on the feedback by @fraenki

Merged, thanks!


service not exposed for zipkin

the deployment of the resource does not expose the service automatically, need to do oc expose service zipkin to expose the service

closed via #91


APBS and compute charge

4d7b: Could not compute charges because of Charges cannot be computed. Missing file. 4d7b_out.csv. Trying with fixed file.
4d7b: Could not compute charges for fixed file because of Charges cannot be computed. Missing file. 4d7b_out.csv. Returning None
APBS 1.5 cant work.  I have APBS 3.4.1 but some file cant compute charge.(apbs cant get xxx.out.csv)

when I just run APBS, get this
error while loading shared libraries: libamd.so.1: cannot open shared object file: No such file or directory

Hi, thanks for reaching out!
There are always examples where computing charges fails - I am unable to use them for training / testing.
If the number of examples where this fails is small, then you could consider ignoring them.
I am sharing my binaries configuration in the next comment - maybe this helps your cause!

Thank you, I have about five percent of the pdbs in your data set that cannot calculate the charge. I found that it is because of using pdb2pqr to report an error on these pdbs. （TypeError: '<' not supported between instances of 'PotentialBond' and 'PotentialBond'）
I want to know that when you conduct experiments, the pdb that cannot calculate the charge is directly discarded. Or give some default initial value？Also wondering if you can't calculate the charge on the data set you gave?
Thanks again！

I just discard the pdb altogether - as far as I remember, across the test set - it was very few examples that got discarded as a result so I went ahead with this strategy.
I have added a link to the binaries in the README (under the Environment Variables section) - let me know if that works for you!
Preprocessing is a major limitation and hacky component of HoloProt. I am working on a version that removes this preprocessing altogether, and instead uses a combination of HoloProt and dMaSIF but time constraints on other things ensure that this is slow.


Providing pre-compiled binaries

Hello, I just found your repo via a searchengine and wanted to try your project out. Now I am downloding an IDE. Maybe you could provide a pre-compiled binary so users would be able to just download and start the program? WOuld be really great :)

Short update: I tried installing dotnet SDK and I could build your project but I am unable to run it. unfortunatly I never interacted with .Net even though I do program for a living :-/

Sorry, I did not get any notification about your comment. The releases are published as pre-releases on the Releases Page. I just created a new one, because the latest did not contain some bugfixes. As seen in the commit history, I am not really actively developing this project at the moment though. It should still work fine for most of the item types, but there will certainly be some bugs I myself have not found yet.


Mutation cache .find method finds old data even though it saves new submit data to client with the same mutationKey

Describe the bug
Issue: React query client creates additional mutation instances with  the same key if component that held this hook was unmounted.
What I expected: I wanted to use information about last submitted time on screen but is doesn't change because it refers to old mutation instance so "Last submitted time" is not changed even though  I submit and use that mutation multiple times later
Your minimal, reproducible example
https://github.com/fake364/tanstack-use-mutation-bug-repro
Steps to reproduce

Go to 'https://github.com/fake364/tanstack-use-mutation-bug-repro'
npm intall
npm start
click on "resend code" button
Last submitted time appeared so keep in mind that value
click on Logout button
we got back to unauthenticated screen and time is the same - thats expected
click on "resend code"

Expected behavior
Expected: time is updated since I've used mutation again
Actual: even though through logs I see that client saves new mutation with the same key, it finds old data and displays it
How often does this bug happen?
Every time
Screenshots or Videos

https://github.com/user-attachments/assets/b5a8c6b3-c67b-4a4f-80ca-7f302c6ab508
Platform

OS: Windows
Chrome
128.0.6613.120 (Official Build) (64-bit)

Tanstack Query adapter
react-query
TanStack Query version
5.56.2
TypeScript version
No response
Additional context
I tried to find any information about this but couldn't

This is expected. You can see in the query devtools (mutation tab) that two mutations are created. Every time mutate is called, a new mutation entry is created. There is no deduplication as there is for queries. With client.getMutationCache().find(['test-mutation']) you just find the first match, even though there are multiple ones. You can use findAll to get all matches, then choose the one with the latest timestamp. If you need a reactive version, you want useMutationState.


Token information update

Logo update and token info update for sFuel token

Sir i have created this token in the mainnet..plz update the logo..thank u in advance

File name is incorrect. You need remove wrong file too

File name is incorrect. You need remove wrong file too

Should i make a fresh upload .could u tell me which file is wrong? .json or the png?

All filename need in lower case. Remove file .png.png

all uppercase converted to lowercase as per your suggestion sir

You need delete all wrong files

Only accept mainnet token


Network-Diagnostics container is not health check enabled and logs are inaccessible when its host is powered OFF

Rancher versions:
rancher/server:v1.6.3
**Operating system and kernel: (cat /etc/os-release, uname -r preferred)**Ubuntu
**Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)**GCE
**Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)**Single node
**Environment Template: (Cattle/Kubernetes/Swarm/Mesos)**Cattle
Steps to Reproduce:

Deploy Network diagnostics catalog
Generate and access logs using

curl -X POST http://IP:8080/r/projects/1a90/network-diagnostics:8080/v1/logs
curl http://IP/r/projects/1a90/network-diagnostics:8080/v1/logs

Power OFF the host on which network container is running. The network diagnostics service is not available.

soumyas-MBP:notes soumya$ curl http://IP:8080/r/projects/1a90/network-diagnostics:8080/v1/logs
Service unavailable

Fixed by stack version: 0.0.2, image: rancher/network-diagnostics:v0.1.2

Tested with v1.6.8-rc1 using tool version 0.0.2. The network diagnostics container is healthcheck enabled. After a Power OFF, the container gets created on a different host and the network diagnostic service is available.


Add function to compute the jacobian derivative of collidable points

This PR is basically a copy-paste of the logic introduced by @xela-95 in #208. The only difference is that here the computation is vmapped over all the collidable points.

📚 Documentation preview 📚: https://jaxsim--213.org.readthedocs.build//213/

I also added generalized_free_floating_jacobian_derivative in api.model as it was required to address Diego's suggestion.

Yep, I didn't realize the I had it just locally from old experiments :)


onTouchTap doesn't seem to work with WebPack and ReactJS 14.0x

onTouchTap doesn't seem to work with WebPack.
For example with IconMenu I cannot get the menu to open when using WebPack and it appears to be because react-tap-event-plugin is not being initialised.
I have included react-tap-event-plugin in node_modules and there are no console errors in browser, just doesn't seem to work at all. The IconMenu itself renders ok and graphically reacts to the click in browser but no MenuItems show at all :(
I am using ReactJS 14.0 with WebPack and none of the components seem to respond if using onTouchTap, I have had to use onClick, but with IconMenu I can't seem to do this... HELP!
Thanks in advance guys :)
If none of that made sense, just let me know and I will try to clarify.

Just to be sure... did you call injectTapEventPlugin(); ?

@TomJannes Thanks

Not sure why this was closed?
Yes I am calling injectTapEventPlugin(); and it doesn't work.
It doesn't work on the simplest of components.

Can you post your code? Is it on github somewhere?

Hey - I stumbled upon the same issue, and it seems this might happen when loading React as a webpack external, as this issue might suggest. In my case, I'm using the webpack Meteor project bootstrap which loads React externally from a meteor package.
My solution was to add another Meteor package, coniel:react-tap-event-plugin and add it as an webpack external. Then calling injectTapEventPlugin() from app patches the right React version (and not the one loaded as a dependency by other packages).

@dbismut .... great find! that fixed it for me as well.... I was ripping my hair out and started modifying to just use onClick... thanks so much for the tip!


ReferenceError: fetch is not defined

Hey, I'm having an issue within the demo for https://ai.google.dev/tutorials/node_quickstart#generate-text-from-text-input. I'm not able to run the demo in Nodejs.
Output:
The error was:  ReferenceError: fetch is not defined
at makeRequest Nodejs/node_modules/@google/generative-ai/dist/index.js:187:9)
at generateContent (Nodejs/node_modules/@google/generative-ai/dist/index.js:511:28)
at GenerativeModel.generateContent (Nodejs/node_modules/@google/generative-ai/dist/index.js:785:16)
at run (Nodejs/controllers/bard-controller.js:18:30)
at Object. (Nodejs/controllers/bard-controller.js:24:1)
at Module._compile (internal/modules/cjs/loader.js:1063:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
at Module.load (internal/modules/cjs/loader.js:928:32)
at Function.Module._load (internal/modules/cjs/loader.js:769:14)
at Module.require (internal/modules/cjs/loader.js:952:19)
Code from my end
const { GoogleGenerativeAI } = require("@google/generative-ai");
// Access your API key as an environment variable (see "Set up your API key" above)
const genAI = new GoogleGenerativeAI(API_KEY);
async function run() {
// For text-only input, use the gemini-pro model
const model = genAI.getGenerativeModel({ model: "gemini-pro"});
const prompt = "Write a story about a magic backpack."
const result = await model.generateContent(prompt);
const response = await result.response;
const text = response.text();
console.log(text);
}

What's your Node.js version? The SDK requires Node.js 18+

What's your Node.js version? The SDK requires Node.js 18+

Node.js v18.19.0
The Error Stack:
node:internal/process/promises:288
triggerUncaughtException(err, true /* fromPromise */);
^
TypeError: fetch failed
at Object.fetch (node:internal/deps/undici/undici:11730:11)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async makeRequest (/path/to/gemini-demo/node_modules/@google/generative-ai/dist/index.js:187:20)
at async generateContent (/path/to/gemini-demo/node_modules/@google/generative-ai/dist/index.js:511:22)
at async run (/path/to/gemini-demo/index.js:12:18)
Node.js v18.19.0

What's your Node.js version? The SDK requires Node.js 18+

Node.js v18.19.0
The Error Stack:
node:internal/process/promises:288 triggerUncaughtException(err, true /* fromPromise */); ^
TypeError: fetch failed at Object.fetch (node:internal/deps/undici/undici:11730:11) at process.processTicksAndRejections (node:internal/process/task_queues:95:5) at async makeRequest (/path/to/gemini-demo/node_modules/@google/generative-ai/dist/index.js:187:20) at async generateContent (/path/to/gemini-demo/node_modules/@google/generative-ai/dist/index.js:511:22) at async run (/path/to/gemini-demo/index.js:12:18)
Node.js v18.19.0

the same error！

Thanks for the help guys, I thought my instance was updated, but it was not. My mistake. I was running v16.

so how to fix it, I still have this problem：
node:internal/process/promises:288
triggerUncaughtException(err, true /* fromPromise */);
^
TypeError: fetch failed
at Object.fetch (node:internal/deps/undici/undici:11576:11)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async makeRequest (/Users/yubowen/Desktop/MyPrivate/node_modules/@google/generative-ai/dist/index.js:187:20)
at async generateContentStream (/Users/yubowen/Desktop/MyPrivate/node_modules/@google/generative-ai/dist/index.js:506:22)
Node.js v18.18.0

Same code，Same error，my nodejs is 18.3.0

Hey @arokhsaz3 - Can you help me with the same? i am not able run this.
We are building a VS-code extension for https://github.com/dishankjindal1/fluttergpt
node - 20.6.1 and also tested on latest stable
system - Apple m2 pro
The same issue is not happening on others system using m series.
ReferenceError: fetch is not defined
	at makeRequest (/Users/dishankjindal/projects/personal/others/fluttergpt/node_modules/@google/generative-ai/dist/index.js:195:9)
	at generateContent (/Users/dishankjindal/projects/personal/others/fluttergpt/node_modules/@google/generative-ai/dist/index.js:525:28)
	at /Users/dishankjindal/projects/personal/others/fluttergpt/node_modules/@google/generative-ai/dist/index.js:650:25
	at async ChatSession.sendMessage (/Users/dishankjindal/projects/personal/others/fluttergpt/node_modules/@google/generative-ai/dist/index.js:669:9)
	at async GeminiRepository.getCompletion (/Users/dishankjindal/projects/personal/others/fluttergpt/src/repository/gemini-repository.ts:36:24)
	at async FlutterGPTViewProvider.getResponse (/Users/dishankjindal/projects/personal/others/fluttergpt/out/providers/chat_view_provider.js:87:30) {stack: 'ReferenceError: fetch is not defined
	at make…pt/out/providers/chat_view_provider.js:87:30)', message: '[GoogleGenerativeAI Error]: Error fetching f…ni-pro:generateContent: fetch is not defined'}

Any help is much appreciated.

So we were able to resolve the issue. https://stackoverflow.com/a/76440754/13038101. VScode extensions actually use the underlying version of the VSCODE itself and not your system.
@dishankjindal1's VSCODE was on an older version that used v16 of NodeJS. updating vscode solved the issue for us.