Fixed docstring that was causing latex error

The docstring for pysal.esda.mapclassify.Map_Classifier has a small error, see http://pysal.readthedocs.io/en/latest/library/esda/mapclassify.html
This is a small fix. Please let me know if I should open an issue or target dev with this pull request instead.
Thanks!

Coverage remained the same at 83.074% when pulling 3ea7d01e7e4e5630f4d482dd0e5b570b508cd6dc on calebrob6:master into 2223c823eccc378131095bba3b6f282504f0cb0b on pysal:master.

Coverage remained the same at 83.074% when pulling 3ea7d01e7e4e5630f4d482dd0e5b570b508cd6dc on calebrob6:master into 2223c823eccc378131095bba3b6f282504f0cb0b on pysal:master.

Coverage remained the same at 83.074% when pulling 3ea7d01e7e4e5630f4d482dd0e5b570b508cd6dc on calebrob6:master into 2223c823eccc378131095bba3b6f282504f0cb0b on pysal:master.

Coverage remained the same at 83.074% when pulling 3ea7d01e7e4e5630f4d482dd0e5b570b508cd6dc on calebrob6:master into 2223c823eccc378131095bba3b6f282504f0cb0b on pysal:master.

Coverage remained the same at 83.074% when pulling 3ea7d01e7e4e5630f4d482dd0e5b570b508cd6dc on calebrob6:master into 2223c823eccc378131095bba3b6f282504f0cb0b on pysal:master.

Looks good.
Thanks!


I am facing 'Error: Unable to resolve module os' in this file<EMAIL_ADDRESS>

Hello Everyone,
I am facing this below error, I am using this library in my RN project i think there are some code which is trying to use the Node.js os module, which is not available in the React Native environment. React Native doesn't support Node.js core modules like os, fs, etc. Is there newer or older version of OpenTelemetry that offers better React Native support.
error: Error: Unable to resolve module os from<EMAIL_ADDRESS>os could not be found within the project or in these directories:
node_modules
dependencies:
"@opentelemetry/api": "^1.9.0",
"@opentelemetry/exporter-metrics-otlp-http": "^0.55.0",
"@opentelemetry/exporter-trace-otlp-http": "^0.55.0",
"@opentelemetry/resources": "^1.28.0",
"@opentelemetry/sdk-metrics-base": "^0.31.0",
"@opentelemetry/sdk-trace-base": "^1.28.0",
"@opentelemetry/sdk-trace-web": "^1.28.0",
"react": "17.0.1",
"react-native": "0.64.2",
"react-native-get-random-values": "^1.11.0"

Hi @Moonubanna - could you provide some info on how you are you importing this?
From what I can tell, the metro.config.js should default to resolving whatever is specified in the "browser" field of a package, so I'd assume that this should work fine for resource detectors and you should get a no-op detector instead of the Node.js one.
Are you able to share your metro.config.js?
Also - please note that React Native is not officially supported by this project.

@Moonubanna friendly ping


Add UI tests for automated release testing

We now have unit and integration tests over at TravisCI and some basic UI-Tests to manually run on devices.


Adopt k8s affinity fix from NWX

So my understanding of this is "prefer to schedule the analytics endpoint pod on a node that is not running an analytics service deployment". Is that correct?

Yeah, the topologyKey is the "group by" basically.
When trying to find a node, group by the hostname, and try to find a node that is not currently running the node.
When using pre-emptible nodes, all of our pods would migrate to the same node, and then when that node would finally be killed we would have a bit of downtime.

Yeah, the topologyKey is the "group by" basically.
When trying to find a node, group by the hostname, and try to find a node that is not currently running another version of the service.
When using pre-emptible nodes, all of our pods would migrate to the same node, and then when that node would finally be killed we would have a bit of downtime.

I found this article helpful.
In this case the topology key is kubernetes.io/hostname which is unique per node. So what this configuration does is saying "do not schedule this pod on a node that already has a pod running that is labeled app: service-analytics". This helps avoid the issue @ScottBrooks just outlined.
Now you can go fancier and use some other topology key that is actually shared across several nodes, such as region. This way only one node in a particular regional node will ever run this pod, and it'll schedule the next one in a different region.


WSL crashes shortly after starting up

Windows Version
Microsoft Windows [Version 10.0.19045.5131]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[x] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-1
Distro Version
Arch Linux (https://apps.microsoft.com/detail/9mznmnksm73x)
Other Software
No response
Repro Steps
Scenario 1

Start WSL from the Terminal
Wait for some time

Expected Behavior
Scenario 1
The WSL will continue to run until the specified idle time.
Scenario 2
Kitty will open via RDP
Actual Behavior
Scenario 1
WSL crashes shortly after starting up. Only the message [exited] is shown.
Scenario 2
Kitty does not open
Diagnostic Logs
Log to start WSL in the terminal with it crashing shortly after: WslLogs-2024-11-21_12-34-56.zip
Log trying to run wslg to open kitty: WslLogs-2024-11-21_12-31-07.zip

After a long time trying out different things, I have "fixed" part of the problem using the following steps:

Export the WSL Instance
Unregister the WSL Instance
Import the WSL Instance
Run into WSL as root
su to my user

If the default user is set to the personal user, it will still crash
This inhibits the crash. However, GUI applications still do not work.

@J4yTr1n1ty: What kind of command line do you see "crashing" ? I do see that a process is exiting with error code = 1, but from what I see it's unclear whether this issue is coming from WSL itself, or from that process. The Virtual machine is still running at the end of the first trace you sent.

Hey @OneBlue, what happened for me is that I opened up the WSL, and it showed [exited] after a short while. Since I have not changed anything on my I first assumed it to be a WSL issue. However, it could be very well possible that it's the distribution or a process starting up in the distribution. Is there an easy way to trace this?

An interesting thing I found is that if I remove the default user from the WSL configuration, I can access the virtual machine as root and change my user after the shell is initialized. This completely removes the "crashing" state.


Add to homebrew

Add to homebrew for macs :)

Shouldn't this be closed now?

brew install jadx works.


add docker ftp server to destroy_and_rebuild script

link in readme. needs user added during provisioning and output ip (or use port?)

now using static IPs with docker --net parameter, should make it easier.


CI: Reuse check schema

What does this PR do?
Part of https://github.com/Lightning-AI/lightning/issues/12545.
Same as https://github.com/Lightning-AI/lightning/pull/14469.
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[n/a] Did you make sure to update the documentation with your changes?
[n/a] Did you write any new necessary tests? [not needed for typos/docs]
[n/a] Did you verify new and existing tests pass locally with your changes?
[n/a] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[x] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@akihironitta - Just curious, what does this do? Sorry, I don't have much context. Approving as I trust you on this ;)

Haha, thanks for asking 😆
The actual workflow file sits in another repository: https://github.com/Lightning-AI/utilities/blob/215a98333e5fe8906156a290ad5f577e176572d1/.github/workflows/check-schema.yml#L25-L37
, and basically, it just checks if yml config files (such as azure and GHA workflow files) are in the correct structure, e.g. indentation, typo, etc.
For example, here's the azure schema file that defines what the correct structure is for azure pipelines' config files.
cc @Borda just in case you have something to add here.

Thanks, @akihironitta for the context. This helps! So does this mean that we'll have to rename .azure-pipelines folder to .azure_pipelines? Check shchema job is failing in this PR:
File "/home/runner/.local/lib/python3.8/site-packages/identify/identify.py", line [44](https://github.com/Lightning-AI/lightning-flash/runs/8242093713?check_suite_focus=true#step:7:45), in tags_from_path
    raise ValueError(f'{path} does not exist.')
ValueError: .azure_pipelines//*.yml does not exist.

We have .azure-pipelines folder, so maybe that's not expected?

Thank you @krshrimali for pointing it out! I have it fixed now :)

Awesome, thank you so much! 🔥 Let's merge once all relevant tests are green.

It seems the job added here is failing. Looking into it

@akihironitta could be that some inputs shall be a string, but they are passed as numbers...

It was indeed due to the incorrect schema provided by MSFT. true and false are valid values in the Azure Pipelines service but are considered invalid with the schema file as reported in the linked threads:

schema definition: https://github.com/microsoft/azure-pipelines-vscode/blob/2a607113c4c0fb844d28ca0582a50920448c0e3e/service-schema.json#L4008
https://github.com/microsoft/azure-pipelines-vscode/issues/446
https://developercommunity.visualstudio.com/t/Azure-Pipelines-schema-inconsistently-li/1668070

In short, what we can do now is to quote all true and false occurrences so that we have "true" and "false" 😄

I see another issue around PublishCodeCoverageResults@1, and it turned out that this is something @Borda reported to their repos quite a while ago 😄

https://github.com/microsoft/azure-pipelines-vscode/issues/412
https://github.com/python-jsonschema/check-jsonschema/issues/10

@Borda Would you say it's worth removing PublishCodeCoverageResults@1 to have the schema check pass?


Social media image preview

When posting links to news articles on social media, the platforms often show a post preview image. But for us this is always the same image. Would be cool if we could have a different image for each article. In the past I sometimes deleted the default image but linkedin basically does not show posts without an image... So for now I past the image manually but would be cool if we could set dynamically what goes into the header. The relevant section is probably this one:
https://github.com/maplibre/maplibre.github.io/blob/ece62083ff7cea8ebef10b0b8108c9d0fde683be/layouts/partials/head.html#L10-L24
@louwers @maxammann do you know how we could do this?

og:image is the source of the image basically. It should change according to the page content.
We can/should use a platform that allows writing content instead of rolling out our own such as WordPress or Medium, IMHO.
Note that there are strict copyright law in regards to images, so we should be careful when adding those...

We can pull the image from the YAML frontmatter.
I would stick with a static site generator. The only reason to use a CMS for a blog or website in my eyes if non-technical people need to contribute to it.

I am happy to explicitly set the image path in every news article.

The only reason to use a CMS for a blog or website in my eyes if non-developers need to contribute to it.

I can't express how deeply I disagree with this statement 😃.
The reason we are maintaining a site here with all the issues we find and take time to solve is a complete waist of a lot of time...
When you write a blog post in medium for example images are suggested, the style is well defined and easy to have pages look the same, SEO it's out of the box, it will suggest content to others users and so on and so on. It's a lot less work than what we do here with a lot more value.
I'm very technical, and I would still suggest to write blog posts in meduim instead of here in general...


[Summer2021] Translate the alert_spi.md

According to this document, then translated it.

PTAL @ptyp

you forget to add some links.

sorry, please ignore me

@ptyp PTAL Thx


⚠ Bofrost.de has degraded performance

In 7b64f85, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1382 ms

Resolved: Bofrost.de performance has improved in 21b12ca after 312 days, 4 hours, 50 minutes.


5m token gone just with errors

Describe the bug
info Using wasm build of next-swc

warn Attempted to load @next/swc-linux-x64-gnu, but an error occurred: Cannot load native addon because loading addons is disabled<EMAIL_ADDRESS>warn Attempted to load @next/swc-linux-x64-gnux32, but it was not installed
warn Attempted to load @next/swc-linux-x64-musl, but an error occurred: Cannot load native addon because loading addons is disabled<EMAIL_ADDRESS>info Creating an optimized production build ... [webpack.cache.PackFileCacheStrategy] Skipped not serializable cache item 'Compilation/modules|/home/project/node_modules/next/dist/build/webpack/loaders/css-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[2]!/home/project/node_modules/next/dist/build/webpack/loaders/postcss-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[3]!/home/project/src/app/globals.css': No serializer registered for PostCSSSyntaxError
 while serializing webpack/lib/cache/PackFileCacheStrategy.PackContentItems -> webpack/lib/NormalModule -> webpack/lib/ModuleBuildError -> PostCSSSyntaxError
Failed to compile.

./src/app/globals.css:1:1
Syntax error: /home/project/src/app/globals.css The border-border class does not exist. If border-border is a custom class, make sure it is defined within a @layer directive.

1 | @tailwind base;
| ^
2 | @tailwind components;
3 | @tailwind utilities;

./src/app/globals.css
Syntax error: /home/project/src/app/globals.css The border-border class does not exist. If border-border is a custom class, make sure it is defined within a @layer directive. (1:1)

1 | @tailwind base;
| ^
2 | @tailwind components;
3 | @tailwind utilities;

at new WebpackError (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:488214)
at new HookWebpackError (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:307902)
at tryRunOrWebpackError (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:308539)
at __webpack_require_module__ (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:130357)
at __nested_webpack_require_153265__ (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:129784)
at eval (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:130649)
at symbolIterator (/home/project/node_modules/next/dist/compiled/neo-async/async.js:1:14519)
at done (/home/project/node_modules/next/dist/compiled/neo-async/async.js:1:14899)
at Hook.eval (https://zp1v56uxy8rdx5ypatb0ockcb9tr6a-oci3.w-credentialless-staticblitz.com/blitz.134daa3c.js:40:1441309)
at Hook.CALL_ASYNC_DELEGATE (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:13:25925)

-- inner error --
Syntax error: /home/project/src/app/globals.css The border-border class does not exist. If border-border is a custom class, make sure it is defined within a @layer directive. (1:1)

1 | @tailwind base;
| ^
2 | @tailwind components;
3 | @tailwind utilities;

at Object.eval (/home/project/node_modules/next/dist/build/webpack/loaders/css-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[2]!/home/project/node_modules/next/dist/build/webpack/loaders/postcss-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[3]!/home/project/src/app/globals.css:2:7)
at eval (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:901184)
at Hook.eval (https://zp1v56uxy8rdx5ypatb0ockcb9tr6a-oci3.w-credentialless-staticblitz.com/blitz.134daa3c.js:40:1441309)
at Hook.CALL_DELEGATE (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:13:25819)
at eval (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:130390)
at tryRunOrWebpackError (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:308493)
at __webpack_require_module__ (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:130357)
at __nested_webpack_require_153265__ (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:129784)
at eval (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:130649)
at symbolIterator (/home/project/node_modules/next/dist/compiled/neo-async/async.js:1:14519)

Generated code for /home/project/node_modules/next/dist/build/webpack/loaders/css-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[2]!/home/project/node_modules/next/dist/build/webpack/loaders/postcss-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[3]!/home/project/src/app/globals.css
Import trace for requested module:
./src/app/globals.css

Build failed because of webpack errors

info Creating an optimized production build .
~/project 16s
❯ npm start

<EMAIL_ADDRESS>start
next start

ready started server on <IP_ADDRESS>:3000, url: http://localhost:3000
info Loaded env from /home/project/.env
Error: Could not find a production build in the '/home/project/.next' directory. Try building your app with 'next build' before starting the production server. https://nextjs.org/docs/messages/production-start-no-build-id
at NextNodeServer.getBuildId (/home/project/node_modules/next/dist/server/next-server.js:345:23)
at new Server (/home/project/node_modules/next/dist/server/base-server.js:151:29)
at new NextNodeServer (/home/project/node_modules/next/dist/server/next-server.js:172:9)
at NextServer.createServer (/home/project/node_modules/next/dist/server/next.js:179:24)
at async eval (/home/project/node_modules/next/dist/server/next.js:200:31)
at async NextServer.prepare (/home/project/node_modules/next/dist/server/next.js:160:24)
at async Server.eval (/home/project/node_modules/next/dist/server/lib/render-server.js:123:17) {
type: 'Error'
}

Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-gyvg3q
Steps to reproduce
info Using wasm build of next-swc

warn Attempted to load @next/swc-linux-x64-gnu, but an error occurred: Cannot load native addon because loading addons is disabled<EMAIL_ADDRESS>warn Attempted to load @next/swc-linux-x64-gnux32, but it was not installed
warn Attempted to load @next/swc-linux-x64-musl, but an error occurred: Cannot load native addon because loading addons is disabled<EMAIL_ADDRESS>info Creating an optimized production build ... [webpack.cache.PackFileCacheStrategy] Skipped not serializable cache item 'Compilation/modules|/home/project/node_modules/next/dist/build/webpack/loaders/css-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[2]!/home/project/node_modules/next/dist/build/webpack/loaders/postcss-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[3]!/home/project/src/app/globals.css': No serializer registered for PostCSSSyntaxError
 while serializing webpack/lib/cache/PackFileCacheStrategy.PackContentItems -> webpack/lib/NormalModule -> webpack/lib/ModuleBuildError -> PostCSSSyntaxError
Failed to compile.

./src/app/globals.css:1:1
Syntax error: /home/project/src/app/globals.css The border-border class does not exist. If border-border is a custom class, make sure it is defined within a @layer directive.

1 | @tailwind base;
| ^
2 | @tailwind components;
3 | @tailwind utilities;

./src/app/globals.css
Syntax error: /home/project/src/app/globals.css The border-border class does not exist. If border-border is a custom class, make sure it is defined within a @layer directive. (1:1)

1 | @tailwind base;
| ^
2 | @tailwind components;
3 | @tailwind utilities;

at new WebpackError (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:488214)
at new HookWebpackError (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:307902)
at tryRunOrWebpackError (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:308539)
at __webpack_require_module__ (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:130357)
at __nested_webpack_require_153265__ (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:129784)
at eval (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:130649)
at symbolIterator (/home/project/node_modules/next/dist/compiled/neo-async/async.js:1:14519)
at done (/home/project/node_modules/next/dist/compiled/neo-async/async.js:1:14899)
at Hook.eval (https://zp1v56uxy8rdx5ypatb0ockcb9tr6a-oci3.w-credentialless-staticblitz.com/blitz.134daa3c.js:40:1441309)
at Hook.CALL_ASYNC_DELEGATE (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:13:25925)

-- inner error --
Syntax error: /home/project/src/app/globals.css The border-border class does not exist. If border-border is a custom class, make sure it is defined within a @layer directive. (1:1)

1 | @tailwind base;
| ^
2 | @tailwind components;
3 | @tailwind utilities;

at Object.eval (/home/project/node_modules/next/dist/build/webpack/loaders/css-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[2]!/home/project/node_modules/next/dist/build/webpack/loaders/postcss-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[3]!/home/project/src/app/globals.css:2:7)
at eval (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:901184)
at Hook.eval (https://zp1v56uxy8rdx5ypatb0ockcb9tr6a-oci3.w-credentialless-staticblitz.com/blitz.134daa3c.js:40:1441309)
at Hook.CALL_DELEGATE (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:13:25819)
at eval (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:130390)
at tryRunOrWebpackError (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:308493)
at __webpack_require_module__ (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:130357)
at __nested_webpack_require_153265__ (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:129784)
at eval (/home/project/node_modules/next/dist/compiled/webpack/bundle5.js:28:130649)
at symbolIterator (/home/project/node_modules/next/dist/compiled/neo-async/async.js:1:14519)

Generated code for /home/project/node_modules/next/dist/build/webpack/loaders/css-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[2]!/home/project/node_modules/next/dist/build/webpack/loaders/postcss-loader/src/index.js??ruleSet[1].rules[2].oneOf[12].use[3]!/home/project/src/app/globals.css
Import trace for requested module:
./src/app/globals.css

Build failed because of webpack errors

info Creating an optimized production build .
~/project 16s
❯ npm start

<EMAIL_ADDRESS>start
next start

ready started server on <IP_ADDRESS>:3000, url: http://localhost:3000
info Loaded env from /home/project/.env
Error: Could not find a production build in the '/home/project/.next' directory. Try building your app with 'next build' before starting the production server. https://nextjs.org/docs/messages/production-start-no-build-id
at NextNodeServer.getBuildId (/home/project/node_modules/next/dist/server/next-server.js:345:23)
at new Server (/home/project/node_modules/next/dist/server/base-server.js:151:29)
at new NextNodeServer (/home/project/node_modules/next/dist/server/next-server.js:172:9)
at NextServer.createServer (/home/project/node_modules/next/dist/server/next.js:179:24)
at async eval (/home/project/node_modules/next/dist/server/next.js:200:31)
at async NextServer.prepare (/home/project/node_modules/next/dist/server/next.js:160:24)
at async Server.eval (/home/project/node_modules/next/dist/server/lib/render-server.js:123:17) {
type: 'Error'
}

Expected behavior
working without any issue
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 129
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 172359072,
  "usedJSHeapSize": 149802404,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = YDX64
Chat ID = f5a93876583d

Additional context
No response

@YDX64 please review issue #678 for tips on being more efficient with token usage


Add a list of supported document types for source-files

Double check, but I believe it's:

Markdown (with various extensions)
MDX
JSON
YAML

Might be worthwhile to note what it is that tells Contentlayer how to process content. Is it the file extension?
cc @schickling

Might be worthwhile to note what it is that tells Contentlayer how to process content. Is it the file extension?

It's not the file extension but the contentType.

@schickling And if contentType is not set, does it fall back to the file extension, or does it have a static default (e.g. markdown?)


Package has been ignored because it contains invalid configuration.

After updating to latest versions I've been getting the following warning on app startup:
warn Package @graphql-codegen/cli has been ignored because it contains invalid configuration. Reason: Cannot find module<EMAIL_ADDRESS>warn Package @graphql-codegen/fragment-matcher has been ignored because it contains invalid configuration. Reason: Cannot find module<EMAIL_ADDRESS>warn Package @graphql-codegen/introspection has been ignored because it contains invalid configuration. Reason: Cannot find module<EMAIL_ADDRESS>warn Package @graphql-codegen/typescript has been ignored because it contains invalid configuration. Reason: Cannot find module<EMAIL_ADDRESS>warn Package @graphql-codegen/typescript-document-nodes has been ignored because it contains invalid configuration. Reason: Cannot find module<EMAIL_ADDRESS>warn Package @graphql-codegen/typescript-graphql-files-modules has been ignored because it contains invalid configuration. Reason: Cannot find module<EMAIL_ADDRESS>warn Package @graphql-codegen/typescript-graphql-request has been ignored because it contains invalid configuration. Reason: Cannot find module<EMAIL_ADDRESS>warn Package @graphql-codegen/typescript-operations has been ignored because it contains invalid configuration. Reason: Cannot find module<EMAIL_ADDRESS>
I haven't changed anything related to GraphQL Codegen. I'm on TypeScript 4.4.2, and here is the relevant part of my DEV dependencies:
"devDependencies": {
	...
    "@graphql-codegen/cli": "2.1.1",
    "@graphql-codegen/fragment-matcher": "3.1.0",
    "@graphql-codegen/introspection": "2.1.0",
    "@graphql-codegen/typescript": "2.1.2",
    "@graphql-codegen/typescript-document-nodes": "2.1.2",
    "@graphql-codegen/typescript-graphql-files-modules": "2.1.0",
    "@graphql-codegen/typescript-graphql-request": "^4.1.2",
    "@graphql-codegen/typescript-operations": "2.1.2",
	...
  }

To Reproduce
Have the latest versions of GraphQL codegen for TypeScript in your dev dependencies, run your app, check console.
Expected behavior
The warning shouldn't display.
Environment:
OS: macOS 11.3.1
@graphql-codegen/...: See above
NodeJS: 14.17.2
Additional context
This wasn't the case before updating to latest versions. Note that this is a minor annoyance, not a breaking bug or anything. Everything still work as expected otherwise.

@dotansimha, I have just installed the latest version of the @graphql-eslint package and I don't see "./package.json": "./package.json" in<EMAIL_ADDRESS>

Thanks @stelmakhivan
@kamilkisiela @ardatan i think it's an issue with bob maybe?

Reason: Cannot find module<EMAIL_ADDRESS>
.json.js, probably require. I don't think it's related to bob

When I add "./package.json": "./package.json" manually (see this comment https://github.com/dotansimha/graphql-code-generator/issues/6599#issuecomment-974742983) then warnings are fixed
I am using react native in my project.
But after I install the latest version of @graphql-eslint, "./package.json": "./package.json" is missed in exports even if I have this line already in code (see https://github.com/dotansimha/graphql-eslint/blob/master/packages/plugin/package.json#L11)
It works perfectly with other libraries, like these:
react-hook-form
zora
effector
react-final-table

lol, didn't know you need to put package.json in package.json to import/require package.json :D

so yeah, it gets lost at some point

lol, didn't know you need to put package.json in package.json to import/require package.json :D

react-native-community/cli#1168

Fixed in 3.3.0 version


Subliminal doesn't work when the code is converted to a compiled executable

subliminal module in python which is used to get subtitles , doesn't work when compiled to an executable using pyinstaller. here is my complete code
import subliminal
import sys,os,requests
import argparse;
import logging;
from babelfish import Language
import dbm.dumb

# configure the cache
subliminal.region.configure('dogpile.cache.dbm', arguments={'filename': 'cachefile.dbm'})

parser = argparse.ArgumentParser()
parser.add_argument('--Path' , type=str , default=None,help="Path for the video or a directory of videos.")
args=parser.parse_args()

if not args.Path:
	args.Path = input("Enter the path for the video file or a directory containing videos : ") ;

if os.path.isfile(args.Path):
	videos = [subliminal.scan_video(args.Path)];
else:
	videos = subliminal.scan_videos(args.Path) ;

subs = subliminal.download_best_subtitles(videos,{Language('eng')} ) ;
print(list(subs.values()))
for v in videos:
	subliminal.save_subtitles(v, subs[v]) ;

The subtitle downloads perfectly when run from IDE or console directly. But it fails to get any subtitle , after compiling .
EDIT 1 :

I found that when i call the pool.list_subtitles( )   , the self.providers list in the argument is empty and so , none of the subtitles are donwnloaded.

Have you tried to run pyinstaller on the CLI?

Did you find a solution?
I have the same problem.
Looking also at #2747 and #802 I couldn't see a way to fix this.
First the error was that it missed the alpha3b module from babelfish
I imported all the modules myself but now I'm stuck with this:
Traceback (most recent call last):
  File "secondary.py", line 152, in process
  File "lib\site-packages\subliminal\core.py", line 581, in list_subtitles
  File "lib\site-packages\subliminal\core.py", line 144, in list_subtitles
  File "lib\site-packages\subliminal\core.py", line 105, in list_subtitles_provider
  File "lib\site-packages\stevedore\extension.py", line 326, in __getitem__
KeyError: u'podnapisi'

I had success in making it work . THE ISSUE IS WITH how Pyinstaller Handles Entry Points in different modules .
When you go to the python packages folder of subliminal it has a entry points.txt file . There is a configuration file available from pyinstaller which address this issue with entrypoints

Thank you for the answer!!
I found the entry points.txt file but I can't find the pyinstaller's configuration file that uses the entrypoints (or maybe I do not understand what the solution is and what to do with the text at the first place :o)
Any more info or instructions?
TIA

I suppose that the configuration file that you are referring to is this recipe.
I copied the code to my ".spec" script and added an
a = Entrypoint('main.py', 'console_scripts', 'myEntryPoint',
               pathex=['D:\\Apps\\DEV\\PROJECTS\\4Build\\Subber'],
               excludes=['_tkinter'])

entry, instead of the
a = Analysis(['main.py'],
             pathex=['D:\\Apps\\DEV\\PROJECTS\\4Build\\Subber'],
             hiddenimports=[],
             hookspath=None,
             runtime_hooks=None,
             excludes=['_tkinter'])

but I still cant understand what values I should use for the 'console_scripts', 'myEntryPoint' arguments or how to use the entry_points.txt variables...
Any help will be appreciated!

Here is my spec file contents :
block_cipher = None
options = [('v',None,'OPTION')]

a = Analysis(scripts=['main.py',r'subtitle_entry_points.py'],
      pathex=[r'C:\Users\Natesh\Documents\Myprojectfolder','C:\Program Files\Python35\Lib\site-packages'],
             binaries=[],
             datas=['guessit/tlds', 'guessit/tlds'), ('babelfish/data', 'babelfish/data') , '.')],
             hiddenimports=['dogpile.cache.backends.file','subliminal','dogpile.cache','stevedore','appdirs'],
             hookspath=['.'],
             runtime_hooks=[],
             excludes=[],
             win_no_prefer_redirects=False,
             win_private_assemblies=False,
             cipher=block_cipher)

pyz = PYZ(a.pure,
  a.zipped_data,
  cipher=block_cipher)

exe = EXE(pyz,
          a.scripts,
        exclude_binaries=True,
          name='Subtitler',
          debug=True,
          strip=False,
          upx=True,
          console=True )
coll = COLLECT(exe,
              a.binaries,
              a.zipfiles,
              a.datas,
              strip=False,
              upx=True,
              name='Subtitler')

As you can see in the Analysis function arguments , I have used the "subtltle_entry_points.py" file . Make this same file in your project folder and paste the below code inside the "subtitle_entry_points.py" file . Then you are good to go with using the above spec code to compile . Make sure that you have guessit and babelfish folders also in your project folder (as mentioned in many other sites addressing this issue)
import subliminal.providers.shooter
subliminal.providers.shooter.ShooterProvider()

import subliminal.providers.addic7ed
subliminal.providers.addic7ed.Addic7edProvider()

import subliminal.providers.legendastv
subliminal.providers.legendastv.LegendasTVProvider()

import subliminal.providers.opensubtitles
subliminal.providers.opensubtitles.OpenSubtitlesProvider()

import subliminal.providers.podnapisi
subliminal.providers.podnapisi.PodnapisiProvider()

import subliminal.providers.subscenter
subliminal.providers.subscenter.SubsCenterProvider()

import subliminal.providers.thesubdb
subliminal.providers.thesubdb.TheSubDBProvider()

import subliminal.providers.tvsubtitles
subliminal.providers.tvsubtitles.TVsubtitlesProvider()

import subliminal.converters.legendastv
subliminal.converters.legendastv.LegendasTVConverter()

import subliminal.converters.addic7ed
subliminal.converters.addic7ed.Addic7edConverter()

import subliminal.converters.thesubdb
subliminal.converters.thesubdb.TheSubDBConverter()

import subliminal.converters.tvsubtitles
subliminal.converters.tvsubtitles.TVsubtitlesConverter()

import subliminal.converters.shooter
subliminal.converters.shooter.ShooterConverter()

Thank you for your help, but it didn't work :o(
I changed my spec file like this:
# -*- mode: python -*-

a = Analysis(['main.py', 'entry_points.py'],
             pathex=[r'D:/Apps/DEV/PROJECTS/4Build/Subber',
                     r'D:/Apps/DEV/PYTHON/Python27/Lib/site-packages'],
             datas=[(r'D:/Apps/DEV/PYTHON/Python27/Lib/site-packages/langdetect/profiles', r'langdetect/profiles'),
                    (r'D:/Apps/DEV/PYTHON/Python27/Lib/site-packages/langdetect/utils', r'langdetect/utils'),
                    (r'D:/Apps/DEV/PYTHON/Python27/Lib/site-packages/babelfish/data', r'babelfish/data')],
             hiddenimports=['dogpile.cache.backends.file','subliminal','dogpile.cache','stevedore','appdirs'],
             hookspath=None,
             runtime_hooks=None,
             excludes=['_tkinter', 'Crypto'])

pyz = PYZ(a.pure)

exe = EXE(pyz, a.scripts,
          exclude_binaries=True,
          name='Subber.exe',
          icon='Subber.ico',
          debug=False,
          strip=None,
          version='Subber.version.py',
          upx=False,
          console=False)

coll = COLLECT(exe, Tree('D:\\Apps\\DEV\\PROJECTS\\4Build\\Subber\\z'),
               a.binaries,
               a.zipfiles, a.datas,
               strip=None,
               upx=False,
               name='Subber')

and although i builds OK with all the needed files (I add some more dependencies)  when I run it and try to search I get this error:
Application started at 2017-12-24 19:35:36
Traceback (most recent call last):
  File "secondary.py", line 153, in process
  File "Lib\site-packages\subliminal\core.py", line 581, in list_subtitles
  File "Lib\site-packages\subliminal\core.py", line 144, in list_subtitles
  File "Lib\site-packages\subliminal\core.py", line 105, in list_subtitles_provider
  File "Lib\site-packages\stevedore\extension.py", line 326, in __getitem__
KeyError: u'podnapisi'

I only use podnapisi from subliminal, nothing else..
Can you spot any errors in the spec?
I also tried to import your entry_points.py in the main.py with no results.
Is it possible that it will not work with python 2.7.13?

You have not included any data which I had included in my Wi-Fi and if you are still not getting with this method try to pinpoint the location where error is occurring in compiled binary by using logging module and see the objects which are not eating properly initialised because of entrypoint issues. You can save Python objects inside file using python's pickle module

I think that the problem is not with the data (I got all the dependencies) but with the stevedore library.
It uses the extension.py to dynamically create the providers when asked but it doesn't, so there is no "podnapisi" provider and I get a KeyError.
I'll try to fake the provider obj and see what happens.
Thank you for your help anyway...

Yes ur right, what I'm saying is while ur debugging and ur flow reaches that extention.py, save the entire object or data as a file using pickle module and add this code directly in that extention.py . U can then take that file directly and read it and get the object in ur code. U need to do this carefully in the extention.py itself or any other place which needs that object...

Yes, that is what I did and it seems to be working (for now ;o)

can u please tell me the line number in extention.py and which object i should save using pickle ? i have forgotten that where i did previously.

Well, its been some time and I have forgotten everything too..
Looking at the code I use, I can see a couple of changes to the stevedore.extension.py
The first is where I load the object in class ExtensionManager before the __init__ as a class variable. (It could be anywhere actually):
    with open("podnapisi_obj.txt") as pod_file:  # This is a hack for PyInstaller
        PODNAPISI = cPickle.loads(pod_file.read())

The other is in the __get_item__:
        if name == "podnapisi":  # This is a hack for PyInstaller
            return self.PODNAPISI

The object is this:
podnapisi_obj.txt
I don't remember how I got it, but it should be what the __get_item__ returned normally...

I tried using it , still no luck. is it possible to send your extention.py file and hopefully all the related files so that i can just copy what u have edited ?

I saved all the object files not just for podnapisi . and even after i returned the object that is returned from saved pickle object , i still get empty dict_values[] (no subtitles )

Can you check just with the Podnapisi at first, to make sure there is not something else wrong?

I copied ur code . Changed the required code to work for python3 .( just some prints and cpickle to pickle) . Its working in python directly . can u share your spec file config so that i can compile it like u did and check it ?

Here you go Subber.zip

Remember to add the pickled object to the app's path..


fix invalid HTML in examples

For some reason the recent version of Trunk refused to serve some of the examples due to them missing the  tags. Adding the tags calms it down and allows it to work again. Trivial change, but would save someone the pain of having to fix examples before running them with trunk serve.

Without this, you get errors like:
2024-05-12T05:58:09.811900Z ERROR ❌ error
error from build pipeline

Caused by:
    0: HTML build pipeline failed (1 errors), showing first
    1: failed to finalize asset pipeline
    2: Document has neither a <link data-trunk rel="rust"/> nor a <body>. Either one must be present.

@DJMcNab Should we merge this directly or ... ?

I believe it is indeed a bug in trunk, but having valid html here is nice too:) I will make a bug report for trunk as well (assuming they do not have one already)
12.5.2024 9.03.39 Daniel McNab @.***>:

@.**** approved this pull request.
Thanks for the PR.
Is this due to a recent change in trunk? That error message doesn't look like it's correctly describing the issue, as there is a body tag.
It would be good to open an issue there, or link to it from this if there's an existing one.
But this is an improvement for us, so thanks!
—
Reply to this email directly, view it on GitHub[https://github.com/linebender/xilem/pull/304#pullrequestreview-2051296099], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ABNIL3R4NGTAKEUTK67RH53ZB4A3XAVCNFSM6AAAAABHSQTNWSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDANJRGI4TMMBZHE].
You are receiving this because you modified the open/close state.
[Seurantakuva][https://github.com/notifications/beacon/ABNIL3XMZTXHHZCERQQOC2DZB4A3XA5CNFSM6AAAAABHSQTNWSWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTT2IRFWG.gif]

Perhaps see https://github.com/trunk-rs/trunk/issues/791


high priority

I am getting the following error on my local machine as well as the ec2 master node:
terraform-kubeadm % kubectl --kubeconfig admin.conf get nodes
The connection to the server localhost:8080 was refused - did you specify the right host or port?

This should most probably be resolved with the bug fix in 0.2.6.


Allow object literals to bind attributes

Currently if I do something like this
<my-tag myvar={test: 1}></my-tab>

it will bind the evaluated string expression to opts.myvar is there some way to bind the literal directly without using
 riot.mount('my-tag', {myvar: {test: 1}) ?

Hello. How about: http://jsfiddle.net/a0oq3zdv/ ?

This produces a string expression, I have updated your jsfiddle with an example:
http://jsfiddle.net/a0oq3zdv/2/

<my-tag hello="{ \{ world: 'hello' \} }"></my-tag>

http://jsfiddle.net/a0oq3zdv/3/

Awesome thanks


Change default temp folder

We are using Jenkins CI and are having an issue where our (very small) /tmp folder fills up.  This seems to be largely from temporary files generated by karma.  Is there a way to change the folder to another location?

export TMPDIR=~/tmp
worked for me


Improved LB algorithm (at the moment, hashing the mac address, it's good but not perfectly balanced).

The current modulo N algorithm is good but not perfectly balanced. Take a look at ways to improve it.
Maybe try https://godoc.org/stathat.com/c/consistent ?

i think rr/modulo algos are good enough, people can still implement new if they need.


Why choose 'atom' as the name of editor

Hi @Lovelcp
it was originally called Atomicity and then simplified to just Atom.
There's a great blog post on the Atom blog with the history here: http://blog.atom.io/2015/06/25/atom-1-0.html
Hope that answers your question! :smile:


Add KeyExists

Add KeyExists Function to check if Key is missing but default Value is vaild setting.

Thanks for your contribution. I appreciate it. Note that this function is necessary as you can already test for this with GetValue(). Just test for NULL return as it is never a valid value as all values must be strings.
Not Exists? = ini.GetValue(section, key) == NULL
Exists? = ini.GetValue(section, key) != NULL
I may include some convenience functions for SectionExists/KeyExists.


Can SyncParam support List or array?

I tested it. It seems that I can't...
 public class SyncParamExample : MonoBehaviour
    {
        [Serializable]
        public class MyClass
        {
            public int intVal;
            public float floatVal;
        }
        public List<int> arrayVal;
        public int[] array2Val;
        public List<MyClass> arraysVal;
        public MyClass classVal;

Supported in 1.3.0🎉
Thanks for the report!


Fix to schema generation for IPvAny{Address,Interface,Network} (#498)

Change Summary
Minor fixes to schema generation for ipaddress related types
Related issue number
#498

Checklist

[x] Unit tests for the changes exist
[x] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[x] HISTORY.rst has been updated

if this is the first change since a release, please add a new section
include the issue number or this pull request number #<number>
include your github username @<whomever>

great thanks.


WIP: AVRO-2335 Remove Joda Time library

Thanks, @nandorKollar I was still working on it :) Ready for review now! 👍

This likely needs to be slit into two parts...  One part is changing the various test classes to using and expecting the jsr310 types.   That would definitely help make sure the jsr310 stuff is fully working and is something I would support getting in for 1.9.
The other part is the actual removal of joda-time altogether.   As mentioned on the JIRA, I'm completely against the complete removal of joda-time for 1.9.    1.9 needs to continue to allow joda-time to allow a quicker migration.

Thanks for the feedback @dkulp
I've set this to 1.10, and won't merge this until 1.9 has been released. I fully agree with you that we can't move this into 1.9. Having jsr310 as a default 1.9 is already a breaking change. I will test in the upcoming days to get Divolte working with Avro 1.9 snapshot.

Big +1 on moving this to 1.10, otherwise adoption will suffer and we really need other projects to catch on Avro in particular for security issues in the dependencies that are not needed anymore in 1.9.

Gentle ping @Fokko, it seems this PR need a rebase to master. Are you still working on this?

@nandorKollar I'm happy to pick this up, but right now I'm extremely limited on time, feel free to take it over if you feel like it.

I need to revisit this later.


Does this support DLL's

Hey Dean,
Thanks for this sample and guide, I was just wondering if this supports dynamic loading off DLL's as well. Thank you!

Closing because it does support DLL's, managed to get it working


#4867 remove footer from explore

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

[ ] Bugfix
[ ] Feature
[x] Refactoring

Context

[x] Closes #4867
[ ] 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
[ ] 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?

[x] 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.

pay 20 usd
perfect

pay 20 usd


feat(plot): toggle y axis type

Adds the ability to show integers as y axis labels for plots instead of floats.
Commit is signed using git commit -S -m "..."

Hi
Ive created #66  which includes your code + some minor changes.
Thanks for you PR.


Refactored code for new cameras

Please ensure the above guidelines for contributing are met.
Fixes Issue: #
Changes proposed in this pull request:
Added the functionality to check for camera type while connecting
Refactored the init function
Modified the data structure for enabling streams
Removed looping for stream related functions
Added a common function for setting streams
Refactored functions to enable inheritance
Renamed functions to reflect ROS terminologies
Reordered the function definitions in their calling order
Refactored the function to set static camera options
Fixed coding standards
Added pointcloud flag in test
Automated test for camera options
Updated comments
Refactor of README file is pending

Closing this PR as these changes are included in PR-76


Can't delete #MeasurementToDo box in typedefs in Measurement Logic.vi

Bug Report
v<IP_ADDRESS> has bookmarks (#MeasurementToDo) for the measurement control and result typedefs in Measurement Logic.vi that cannot be deleted and are not being detected in bookmark manager.
Repro or Code Sample
Can't delete the green text box

MeasurementToDo is not being recognized (not bolded) in bookmark manager

Expected Behavior
Per past template versions, the green text box MeasurementToDo is supposed to just be a floating text box on the front panel that can be deleted when no longer needed.
Current Behavior
The v<IP_ADDRESS> template for developing measurement plugins has the #MeasurementToDo for the measurement ctl and results typedefs embedded in the typedef itself although it is not visibly shown when you open the type def from the control or indicator in the measurement logic.vi. This means you can't delete the bookmark because it's not being recognized in bookmark manager.

If I right click > open TypeDef  --> the opened typedef does not have the green text box.
If you disconnect the control from the typedef, then you can delete the green text box.

Possible Solution

Context

Your Environment

OS & Device: Windows
ni-measurement-service version <IP_ADDRESS>
IS Pro Pre-release d555 build
LV 2024 Q1

AB#2797761

✅ Successfully linked to Azure Boards work item(s):

Bug 2797761


Cleanuptask

Add in cleanup tasks that will ALWAYS run for now....with a comment on how to alter it for production where we only want to clean up IF the build is cancelled or fails.

Wrong branch, sorry.


Karanvirsb/issue#127

Made a simple open browser feature following the create-react-app library. #127

Hey @karanvirsb! Thanks for jumping in, but this isn't what I intended from the issue. I was thinking of adding a line in the mirrorful cli script (code found here: https://github.com/Mirrorful/mirrorful/tree/main/packages/mirrorful/cli) or in the package json, as you have it, that prompts the opening of localhost:5050 even if it's not already running.
in other words:
npx mirrorful # should open up localhost:5050 in the browser

I believe you code right now will only work if localhost:5050 is already running?

Hey, @alexdanilowicz I followed the create-react-app react-scripts here: https://github.com/facebook/create-react-app/blob/d960b9e38c062584ff6cfb1a70e1512509a966e7/packages/react-dev-utils/openBrowser.js#L140. I found that they used the open library to help them open the browser. If I move the node ./openBrowser.mjs from the script and add it to a different line in the package.json like this:
"scripts": {
    ...
    "openBrowser": "node ./openBrowser.mjs",
    ...
}

It will still open the browser and show this:

Hey, @alexdanilowicz I have moved openBrowser.ts to the helpers in the cli and added it to the Mirrorful cli init script and it seems to work well. If we moved around the openBrowser function within the init script it determines how early the browser will open.

Hey @karanvirsb awesome! incredible, thanks so much for your time and help with this! Left a few comments.

By the way, @karanvirsb, are you in our Slack community?
Gave you a shoutout here:

Hmmm, I think the lag of the site not being available thought is not a good user experience.
Demostration here:
https://www.loom.com/share/9734c09251f347deba19540086ca3e33
Really appreciate the hard work on this, but I think we either

Need to ensure it doesn't even pop up when not available.. perhaps put it in some sort of timeout? That might be too hacky though. But I think it would be good to see the console output too.
If we can that to work, not a big deal as we are probably going to move to a cloud editor soon.

I think the delay is mostly due to the next dev and the spawn child sync. I tried fixing it by making the spawn async. setTimeout might be too hacky since you need to know how long the next dev takes to run.

Nice, I like it and think it's a fantastic improvement. I made a few changes to the language that is outputted. Let me know what you think.
@Teddarific — do you have thoughts here? I would prefer to release 4.0.1 first, and then we merge this change to be released with 4.0.2.

Nice, I like it and think it's a fantastic improvement. I made a few changes to the language that is outputted. Let me know what you think.
@Teddarific — do you have thoughts here? I would prefer to release 4.0.1 first, and then we merge this change to be released with 4.0.2.

Nice, I think it looks good, I like the changes!


[DOC-3592] Update PL templates topics

Update PL template topics
The AIDA help bot quotes parts of documentation from HDH. This is fine when you are in a module-specific topic. However, when AIDA quotes Platform docs, this can cause seemingly incorrect instructions, such as “go to Deployments” when you are in CI.
Looking at the entire page, it becomes clear that Deployments is used as an example, but AIDA doesn’t always provide that clarifying context.
Due to the number of modules we have now, it would be better for the Platform docs to be module agnostic, or provide some clarification that “XYZ module is used as an example, but you can do this in any/ABC modules too”.
For reviewers, previews are available:
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65948fabe4b44f234aecfda4--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6595b9c65c8bca0b9cb1dd44--harness-developer.netlify.app


Requesting to be passed the original Fastify request object in callback

Feature Request:
Pass back the original Fastify request when invoking the callback.
Details:
I'd like to use a single default onRequestClosed callback to handle all aborted requests. When this callback is invoked, I'd like to have access to the underlying Fastify request object if possible.
Alternatively, maybe there is a way to use this plugin in combination with Fastify's built-in request lifecycle hooks to achieve a similar result. (If this works, maybe a quick example in the docs would be helpful to others heading down a similar path)
Background
Fastify provides separate request/response logging out-of-the-box. In our environment, we found by combining the request / response logs - troubleshooting issues based on log data was much easier. Overall this approach works well, but I recently discovered that our current approach will not log request information during a client abort. (This is how I eventually discovered fastify-racing plugin....thanks!)
Our current logging setup:
fastifyInstance.addHook("onRequest", (req, reply, done) => {
    // track overall response time
    reply.startTime = Date.now();
    done();
});

// use hook based logging instead of internal logger to provide a single
// combined log message per-request, instead of the default request/reply
// logging
fastifyInstance.addHook('onResponse', (req: FastifyRequest, res: FastifyReply) => {
    if (req.url === healthEndpoint) {
        logger.info({
            req,
            res
        });
    } else {
        logger.info({
            req,
            res
        });
    }
});

Hey! Thanks for the issue, and the feedback! 🙂
I don't see an issue with having the context of the request injected in the callback. But rather than injecting it through the parameters of the callback, set the request object as the callback this context, similar to:
app.get('/', (req, reply) => {
    const signal = req.race(function (evt) {
        const result = result.type === 'aborted' ? '' : `${result}-world`

		this.log.info({ result }, 'Request result');

        reply.send(result)
    })
})

Subsequently, you can also take advantage of the onSend hook, which will always be called, regardless of whether the request has been aborted or not.
You can differentiate when aborted or when not by checking calling req.race().aborted.
e.g.
app.addHook('onSend', (req, reply, payload, done) => {
  const aborted = req.race().aborted;

  if (aborted) req.log.info('request aborted');
  else req.log.info('request finished');

  // Remember to call done to allow fastify continue its lifecycle
  done(null, payload)
})

PRs are always welcomed 😄

Hey @mjpowersjr, fastify is about to get support to the hook you're looking for soon, take a look at fastify#4582.
This plugin will still useful if you want to take certain action just at the moment the request is closed, but for the purpose you're looking for, the new hook makes a better fit 🙂


v1.3.0-rc1 Release Proposal

Description
This branch will not be merged, but will be used to create a prerelease named v1.3.0-rc1 that contains the new groups module.  After groups are released in Provenance 1.13.0, this branch can be repurposed, after removing the TODOS, to have the groups protos ready to use.
If I receive approvals, I will create said prerelease branch for use in early-access development.  I appreciate your time and attention.

The real v1.13.0 client changes are now PR'ed


docs: add a new ssh key using GitHub cli

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/authentication/connecting-to-github-with-ssh/adding-a-new-ssh-key-to-your-github-account?tool=cli
What part(s) of the article would you like to see updated?
https://docs.github.com/en/authentication/connecting-to-github-with-ssh/adding-a-new-ssh-key-to-your-github-account?tool=cli#adding-a-new-ssh-key-to-your-account
The steps mentions to first authenticate with the GitHub CLI. I did this in the current terminal session so, I moved onto the next step. Specifically, gh ssh-key add ~/.ssh/id_ed25519.pub --title "memorable title". After pressing enter, I got the following:
HTTP 404: Not Found (https://api.github.com/user/keys?per_page=100)
This API operation needs the "admin:public_key" scope. To request it, run:  gh auth refresh -h github.com -s admin:public_key

I assume when I first authenticated (I just ran gh auth login. Not in any specific context) I did not request these additional rights. I ran the auth refresh command which took me through the original flow (I authenticated through a web browser). Once I completed the authentication flow and reran ssh-key add, all is good and the key is added to my profile on GitHub.
May be a small thing, but it might catch some folks off guard.
Additional information
No response

Hi @schalkneethling, thanks for creating the issue! gh auth login doesn't request all scopes right away so it's normal that you need to request additional scopes as you run different commands. This isn't always something we reflect in our docs
I think we could add a line like If you're prompted to request additional scopes, follow the instructions in the command line. before the command. Let us know if you'd be up for opening a PR to do that! I'll add the help wanted label so you or anyone else can do that 🙂

Hi @isaacmbrown ,
I have raised a PR: https://github.com/github/docs/pull/27571 with your suggestion above.
Could you pleased take a look whenever you get a chance?
Thanks!


Navigation from Test Explorer to code

Double-clicking an element in the Test Explorer view will navigate the user to the file/line number in the editor that holds the definition of that Test Explorer element.

File - open the file, place the cursor at the top.
Suite - open the file containing the suite, place the cursor at the beginning of the line where the class (suite) is declared.
Function - open the file containing the function, place the cursor at the beginning of the line where the function is declared.

This may require extending the model we use for our tests a bit, or extending the API we make available to the TreeViewHandler. Today, we can run a specific test can we navigate to it from anyplace? If there is a trivial or well known call that can do this today, it may lower this task's complexity a bit!
Test for completion:

Double-clicking on a file in the test explorer view puts my cursor at line 1, col 1 of that file.
Double-clicking on a suite in the test explorer view puts my cursor at the line the suite/class is declared, col 1.
Double-clicking on a function in the test explorer view puts my cursor at the line the function is declared, col 1.

Code Lense provider has this today, we will need to make use of it in a new API that we will share with the Test Explorer.

This is no longer done by Double clicking , but by clicking Open icon. Also the cursor does not move, but the focus is changed to the correct place. If this is the expected behavior @DonJayamanne , go ahead and close it.

Let's get clarification from PMs


Consider making non-ISO USB images using 'zpool checkpoint'

Addressing https://github.com/helloSystem/ISO/issues/4, in BSD Now Episode 383: "Scale the tail" it is explained that sing just ZFS it is not possible to generate a writeable pool from read-only media plus read-write media (such as RAM).
https://www.freebsdnews.com/2021/01/01/bsd-now-episode-383-scale-the-tail/
But it is suggested that since few people are using actual DVD media anymore, we could construct non-ISO disk images with ZFS on them (similar to what NomadBSD is doing), but then using zpool checkpoint.
During the boot process, one could go back to the last checkpoint.
Advantages:

~100 seconds boot time saved
RAM requirement goes from 4GB to ~1GB

Disadvantages:

Does not work from truly read-only media like DVDs
Truly read-only media are preferred for auditing purposes ("I know for sure that no one tampered with the software")
May not work for systems like Ventoy

Maybe we can at least find a way to boot into Xorg from truly read-only media by mounting a few tmpfs in the right places like /etc, /var, /usr/local/etc, /usr/local/var? This way one could at least install the system from a DVD. Is it worth the extra hassle? I don't know.

Wonder whether this approach might work around https://github.com/helloSystem/ISO/issues/75#issuecomment-753476482 uname and other commands not working in single user mode …

Wonder whether this approach might work around https://github.com/helloSystem/ISO/issues/75#issuecomment-753476482 uname and other commands not working in single user mode …

Don't think the topics are related.

Don't think the topics are related.

How would one do this to the system one has booted from?
zpool export zroot
zpool import --rewind-to-checkpoint zroot
Cannot unmount '/': Invalid argument

So one would probably only be able to do this from a ramdisk.

How would one do this to the system one has booted from?
zpool export zroot
zpool import --rewind-to-checkpoint zroot
Cannot unmount '/': Invalid argument

So one would probably only be able to do this from a ramdisk.


Lazy Load Allow User Specified Ratio

Hi,
Amazing framework by the way. Is it possible to make a very simple change to allow users to specify the initial src attribute on the image tags. I know there is imagesLazyLoadPlaceholder parameter but my page has varying image ratios which cause the page to jump when they load if they are not all a certain ratio. Some of my photos are 16x9 or 4x3 on one page etc. So I can't just set one placeholder image in imagesLazyLoadPlaceholder parameter.
In Framework7 1.4.2
If we change line 4295
if ($(this).attr('data-src')) $(this).attr('src', placeholderSrc);
to
if ($(this).attr('data-src') && !$(this).attr('src')) $(this).attr('src', placeholderSrc);
This allows users to simply specify the image tag as follows and scales the placeholder image correctly. Where lazy-16x9.jpg is 16x9 pixels in size and lazy-4x3.jpg is 4x3 pixels in size.

It would be great if we could get this added?

You could make this change on a fork and provide a pull request

@vestaxpdx as @tobya suggested could you open a PR?


'xmpp/jid' is not in the npm registry

Hey,
I get the following error in our Jenkins job during npm install:
10:58:49 npm WARN deprecated<EMAIL_ADDRESS>use @xmpp/xml package <https://www.npmjs.com/package/@xmpp/xml>
10:58:50 npm ERR! 404 Not Found
10:58:50 npm ERR! 404
10:58:50 npm ERR! 404 'xmpp/jid' is not in the npm registry.
10:58:50 npm ERR! 404 You should bug the author to publish it
10:58:50 npm ERR! 404 It was specified as a dependency of 'node-xmpp-core'
10:58:50 npm ERR! 404
10:58:50 npm ERR! 404 Note that you can also install from a
10:58:50 npm ERR! 404 tarball, folder, or http url, or git url.

Its available at NPM via npmjs.com/package/@xmpp/jid, but it is not available in the NPM registry as registry.npmjs.org/@xmpp/jid but that is the way npm wants to fetch and resolve it.

what's your npm version? you need a version greater than 2.7.0
see https://docs.npmjs.com/getting-started/scoped-packages

Thanks @sonnyp for the info!
Did not knew that and we used npm v1.4.21 because that is the only version which is in the official Debian repo.

this work for me:
npm config set @xmpp:registry https://registry.npmjs.org/


websocket: return error in Subscribe() instead of log.Fatal.

I am writing a service, which receives ticks from bitfinex. As it must run 24/7, and able to reconnect on a network error, log.Fatal is not a correct way of handling connection errors for me.
I think, that it is better to return an error if we can't send subscribe messages.

it makes perfect sense


Update List Item documentations lacks information for complex columns

Issue: Update List item doesn't give an example on how to update a complex column in a list. For example, I am trying to add users to a "Person or Group" column, but I don't know how the JSON should be formatted. I've tried to match the formatting that was used to do this with the SharePoint REST API, but that only returns errors. Any guidance in the documentation for columns other than primitive types would be very helpful.
Article: https://github.com/microsoftgraph/microsoft-graph-docs/blob/master/api-reference/v1.0/api/listitem-update.md

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


Update fold docs

Continues on from #519
cc @StuartAxelOwen

Merging soon if no comments.

I would really love to use an example reducing function that's not just (a, a) -> a, just to demonstrate that it's possible.  What about something like:
>>> bag.from_sequence(range(3)*5).fold(lambda agg, item: agg | {item}, lambda agg1, agg2: agg1 | agg2, initial=set()).compute()
set([0, 1, 2])

added

Looks good!


[Brew] Wrong version of brew detected

Extension
https://www.raycast.com/nhojb/brew
Raycast Version
1.77
macOS Version
sonoma 14.5
Description
Hi, i don't think this is strictly a bug but i'm having issues with the application that i would like to resolve since i find it very useful. I have both versions of brew installed (x86 - arm ) since i need both but the x86 version is somehow used as default in the terminal command given by the application. This made a mess since i just recently found out that 3/4 of the stuff i downloaded were running trough rosetta.  Yes the brew path is correct and the default in the terminal is arm. I uninstalled and reinstalled while also wiping the x86 brew but i still face the same issue. Is there a way to change the command issued by the application? What may have i done incorrectly? ( maybe having briefly x86 as default when first installing?)
Steps To Reproduce
not sure what to put here
Current Behaviour
No response
Expected Behaviour
No response

Yeah, we check for brew in the PATH first, then fallback to checking the current architecture and choosing /opt/homebrew or /usr/local as appropriate. I'm not sure if we should change it now since some users may be relying on their PATH environment 🤔
Two things you can do:

Set a "Custom Brew Executable Path" in Settings -> Extensions -> Brew (see screenshot).
Or you can add /opt/homebrew/bin as the first item in /etc/paths. Raycast will load /etc/paths at startup and add each item to the app's PATH environment.

Thank you for the explanation but i actually tried both methods and i still got the same issue. I'm no technician but i'm pretty sure i followed the PATH fix correctly and it did not change. Regarding the custom path i simply changed it to the default brew arm location correct? Definetly not sure of my raycast skills.
Sorry for the delayed answer

I'm no technician but i'm pretty sure i followed the PATH fix correctly and it did not change.

You need to quit and re-open Raycast to ensure that any change to /etc/paths is picked up by Raycast.

Regarding the custom path i simply changed it to the default brew arm location correct?

It should be the full path to the brew executable e.g. /opt/homebrew/bin/brew

Yea i put the wrong path in the homebrew custom executable but now it works! I'm still not sure why but the paths fix seems to not work for me even trying multiple reinstalls and resets. Anyway than you very much for the help.

Yea i put the wrong path in the homebrew custom executable but now it works! I'm still not sure why but the paths fix seems to not work for me even trying multiple reinstalls and resets. Anyway than you very much for the help.


Allow Requires 1.0

Requires 1.0 was a non breaking change, so it is fine to allow Requires 1.0.

Thanks!


flags added

Cna also add a Makefile for everything if you want. I use it to run everything.
I also use minio and nats. I also store binaries in Minio. Its a nice way to quickly build out fleets.

Cheers for this PR. I think we might want a bit more coverage in the flag robustness. This adds a couple flags, but not everything related to nats, like the server endpoint and extra various consumer options. I might take a pass on this branch myself to fill it out the way we want.
Also, no Makefile needed. The build process does not seem complex enough to warranty it.

I'm happy to merge this as-is just to expose helpful flags to get things started. I can revisit in another issue and possibly break them before we look at a first tag release.


export declare interface RouteGenerator {   segments: Segment[];   handlers: HandlerEntry[]; }   Cannot find name 'Segment'.  I think Segment class was missing in the recent change

I'm submitting a bug report
I'm submitting a feature request

Library Version:
major.minor.patch-pre

Please tell us about your environment:

Operating System:
OSX 10.x|Linux (distro)|Windows [7|8|8.1|10]

Node Version:
6.2.0

NPM Version:
3.8.9

JSPM OR Webpack AND Version
JSPM 0.16.32 | webpack 2.1.0-beta.17

Browser:
all | Chrome XX | Firefox XX | Edge XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView

Language:
all | TypeScript X.X | ESNext

Current behavior:
Expected/desired behavior:

What is the expected behavior?

What is the motivation / use case for changing the behavior?

@EisenbergEffect I hope this would be resolved soon, appreciate your help.

Both Segment and RecognizeResults seems to be failing in typings as well.


Docs(TileLayer): workaround leafdoc issue with @ symbol

when used as a comment in docstring instead of as a directive start.
Simply replaced by "@" as a workaround for now.
Fix #5668

:+1:


Removed not-context-manager and ordered lint rules

Summary
This should close out #9614
Details and comments
Removes not-context-manager and orders lint rules alphabetically

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
12 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.01%) to 89.746%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/expr.rs
1
94.02%

crates/qasm2/src/lex.rs
5
92.62%

crates/qasm2/src/parse.rs
6
96.69%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
63560

Relevant Lines:
70822

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 89.74%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
6
92.37%

crates/qasm2/src/parse.rs
6
97.61%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
63573

Relevant Lines:
70841

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
12 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.006%) to 89.746%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
6
92.11%

crates/qasm2/src/parse.rs
6
97.15%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
63802

Relevant Lines:
71092

💛  - Coveralls


Display “Contents Preview” files on the project contents page

Display “Contents Preview” files on the project contents page
Hector says “the first 100 files are easy”

Question regarding the asset nodes: Top 100 directories or top 5 directories with top 100 files? Whichever is easiest and whatever looks the most like a traditional file-system (top directories with children). End-users need to be able to readily identify and confirm that this is the project belonging to the end-user. Try and use the default number of files returned within the Mediaflux server response.

aterm asset.query should be used by the assignee, and there are established patterns for this within the code base.

How are we going to connect a Project to Mediaflux? Querying Mediaflux using the directory name? Querying using the DOI?

The directory name may not match the namespace. Mediaflux also offers different mount points for different protocols (hence, directory path will be inconsistent). Collection asset ID or Project ID (which is the DOI) should be used. Performance might not be the same for both.

For asset.query commands, a collection asset ID can be used to refine the query for this feature.

https://github.com/pulibrary/tiger-data-app/blob/main/app/models/mediaflux/http/query_request.rb

Use the Collection Asset ID for querying the Project. There will need to be documented the namespace for each Project, and then under that one will find the Collection Asset. [PROJECT_NAME] + "-ns", however, names are metadata which can vary; IDs are persistent. Hence, an ADR might need to be created in order to formalize the guidelines for this.

The model will have the Collection ID

A hard-coded Collection ID will be used for demonstrating this feature

Long-term, want to attempt a metadata query for the DOI, but this is a part of another ticket

This is favored for the long-term solution by everyone, as the Collection Asset ID is arbitrary

Acceptance Criteria:

[ ] Extend the "Contents Preview" View
[ ] Display whatever Mediaflux gives the client by default when you query by the Collection ID which may be input by hand
[ ] Document explicitly what Mediaflux gives the client by default (comment on this ticket would be fine)

Hey team! Please add your planning poker estimate with Zenhub @bess @carolyncole @hectorcorrea @JaymeeH @jrgriffiniii @kelynch @leefaisonr


Redis cluster + Pub/Sub

Predis blocks Consumer creating on Redis cluster connections with next message:
Cannot initialize a PUB/SUB consumer over aggregate connections.
But it supported by Redis cluster, next string is quote from cluster specification:
The cluster bus is also used in order to propagate Pub/Sub messages across the cluster
http://redis.io/topics/cluster-spec#clients-and-servers-roles-in-the-redis-cluster-protocol

This is unfortunately a limitation of the current design of Predis. It's not possible to use commands such as SUBSCRIBE that do not carry a key (which is used internally to calculate the corresponding slot in the cluster against which the command is executed). Furthermore some higher-level abstractions, such as Predis\PubSub\Consumer, do not work with aggregate connections. I have plans to solve this in the long run, but I don't have any roadmap as of now.

:+1:
My understanding is that we can SUBSCRIBE and PUBLISH to any node of a redis cluster and the messages should flow correctly, so there's no need to compute a slot, no ?
Really needing this :-(

@ahfeel I use code below as temporary workaround:
$connections = iterator_to_array($predis->getConnection()->getIterator());
$connection = $connections[array_rand($connections)];

It returns single connection to random node and it's possible to use SUBSCRIBE/PUBLISH via this connection.

@m157y Thanks for the tip ! Is it actually correct to make any request to any node (out of the pub/sub scope) ?

@ahfeel Yes, PUB/SUB messages will be propogated to all nodes via Cluster Bus. It's described here: http://redis.io/topics/cluster-spec#clients-and-servers-roles-in-the-redis-cluster-protocol

Thanks for the answer @m157y ;) Actually my question was wider, because I knew that already for the pub/sub, my question is can I send a get or put request to any node and it will forward it properly ? Or this is only valid for pub/sub ?

@ahfeel, if you're talking about operations with keys (e.g. set/get), so not, it'll return something like (error) MOVED 12182 <IP_ADDRESS>:6382. And some actions, like flushdb will work only for specified node. So if you want to flush db (e.g. flushdb 0), you'll need to send it to all nodes of the cluster.

Allright, sounds clear ;) Thanks !
On Thu, Feb 4, 2016 at 2:46 PM, Andrey<EMAIL_ADDRESS>wrote:

@ahfeel https://github.com/ahfeel, if you're talking about operations
with keys (e.g. set/get), so not, it'll return something like (error)
MOVED 12182 <IP_ADDRESS>:6382. And some actions, like flushdb will work only
for specified node. So if you want to flush db (e.g. flushdb 0), you'll
need to send it to all nodes of the cluster.
—
Reply to this email directly or view it on GitHub
https://github.com/nrk/predis/issues/293#issuecomment-179847877.

--
Jérémie BORDIER

Starting with Predis v2.0 (currently under heavy development), when using redis-cluster it is possible to initialize a PUB/SUB context doing the following:
// Get a client for a random node of the cluster by slot.
$nodeClient = $client->getClientBy('slot', mt_rand(0, 16383));
// Initialize the PUB/SUB context using the client for the retrieved node.
$pubsub = $nodeClient->pubSubLoop(['subscribe' => 'channel:foo']);

This creates a client out of a random node in the cluster, but it is possible to target specific nodes using the other variants supported by Predis\Client::getClientBy() (e.g. getting a client targeting a specific key).
I'm closing this issue as this solution is generalized enough but I can't exclude that I won't make this mechanism automatic in the client when dealing with redis-cluster, I'll decide later on the development of the next major version as this has a lower priority.

@nrk
How is your progress with Predis v2.0?

Given the maturity of redis cluster, any plans of having this supported?

@nrk, You said this in 2016.

Starting with Predis v2.0 (currently under heavy development)

Is it still under development or is the package being abandoned?

@DJPretorius see https://github.com/nrk/predis/issues/587 (pinned issue)


The cache of main.dart.js for the Web version after you release a new version

Use case
The problem is when you make a new version of the Flutter application with flutter build web command and upload it to a web-server then all the users who used the application have a previous cached main.dart.js  version on their devices, and usually, it won't be downloaded again after you click "Refresh" button if the browser (especially with Safari on iPhone).
Proposal
The proposal is to have a hash of the build inside the name of the js bundle.
For example, instead of main.dart.js, we would have main.dart.28eb5183cd91b813b08c80ab5d6f33e0.js and it will be cached only until new release.
We already have a release hash number in buildDir variable while we build an application (we can observe it in .dart_tool/flutter_build folder) so we can use it.

@jonahwilliams I recall this being fixed. This is related to how service workers work or used to work.

This should be fixed by https://github.com/flutter/flutter/pull/56786, but we haven't done a dev release in nearly 3 weeks so it is unsurprising that users are still hitting this case.

Hi,
it seems like it is still not working on the master branch. The file is still named main.dart.js and the browser caches it until you delete the cache. Is there any workaround besides renaming the file on every release?

I don't see this working. Maybe I'm misunderstanding. I change my flutter web project, rebuild it and deploy it, but refreshing the browser doesn't get the new build. I have to clear the application data. How do I get this working automatically? Do I have to add a version in index.html for the js file?
I'm on the latest beta
[✓] Flutter (Channel beta, 1.20.0-7.2.pre, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.0)
[✓] VS Code (version 1.47.2)
[✓] Connected device (2 available)

Hi @sgehrman , could you file a new issue with more details - such as the contents of the network activity tab/application tab after you've deployed the new version - but before clearing the application data?
Its possible we might need more aggressive cache-busting params like: https://github.com/flutter/flutter/pull/62076

@sgehrman Your problem might be with the service worker not activating after update. Check your Application tab in Chrome. A simple page refresh is likely not enough to switch to the new service worker.
@jonahwilliams Is there currently any way to force the service worker to activate without closing all related tabs?

@jonahwilliams I'm having the same problem. Should I open a new issue?

Same issue, please fix it

the same problem

Same issue

Same problem.
Nexus 6P phone.
Chrome browser.
Caches for infinity. Have to manually clear cache. No user will do that, so they don't receive updates. What is the solution?

I had the same problem,
I found this answer https://stackoverflow.com/a/61720981
and it worked for me, as a temporary fix.
edit your index.html, before building and change the version number, in this line

I have a similar problem, writing the version in an index file is hardcode, we are waiting for a fix.

I have a similar problem, writing the version in an index file is hardcode, we are waiting for a fix.

@nikpozdnyakov
So a temporary solution is to change version in the index HTML file?

Changing the version doesn't seem to solve the problem. A force refresh is still needed.

Changing the version doesn't seem to solve the problem. A force refresh is still needed.

locally this solution helps, perhaps your webserver stores the page cache

Same issue

I found this answer https://stackoverflow.com/a/61720981
and it worked for me, as a temporary fix.

edit your index.html, before building and change the version number, in this line

<script src="main.dart.js?version=1" type="application/javascript"></script>

I tried this but it doesn't work and it loads old page again.

I found this answer https://stackoverflow.com/a/61720981
and it worked for me, as a temporary fix.

edit your index.html, before building and change the version number, in this line

<script src="main.dart.js?version=1" type="application/javascript"></script>

I tried this but it doesn't work and it loads old page again.

My environment is built up on Azure Stack. For me this workaround only works for a blob container with web-access. When deploying the build to an app service / web server it just doesn't recognize the change and the chache still stays the same.
Any ideas how to solve this?

Is this fixed yet? Still seems to be happening

I had the same problem,
I found this answer https://stackoverflow.com/a/61720981
and it worked for me, as a temporary fix.
edit your index.html, before building and change the version number, in this line

Yes, I have also solved this issue by adding this in index.html "main.dart.js?version=1"

I had the same problem,
I found this answer https://stackoverflow.com/a/61720981
and it worked for me, as a temporary fix.
edit your index.html, before building and change the version number, in this line

Yes, I have also solved this issue by adding this in index.html "main.dart.js?version=1"

It didn't work for me .


⚠ Bofrost.be has degraded performance

In 4de608e, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 917 ms

Resolved: Bofrost.be performance has improved in 2ba606d after 13 minutes.


update README for Go1.16

WHAT
Update README to use Go1.16 recommended installation. ref: https://golang.org/doc/go1.16#go-command
WHY
It's officially recommended.
Related issue (if exists)
N/A

Codecov Report

Merging #50 (df58718) into main (36c18fe) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #50   +/-   ##
=======================================
  Coverage   84.67%   84.67%
=======================================
  Files          17       17
  Lines         535      535
=======================================
  Hits          453      453
  Misses         74       74
  Partials        8        8

Continue to review full report at Codecov.

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


Run LOGOUT instead of SAAVEVM when sasving during idle

This PR works as advertised.
Once (small) issue remaining: when a user returns from the forced logout (using the "Resume previous session" option under Run Medley), they are in IDLE (with bouncing logo, etc.).  If they never exit this IDLE, then the session will never terminate.
I will open a separate issue for this - its a low probability event so not clear how much effort it is worth to fix.


Writing USB key broken after April Update

After April update I cannot copy files on USB key preserving their timestamps.
Before the update, I had created the mount point:
sudo mkdir -p /mnt/e/
and could copy files preserving timestamps:
sudo mount -t drvfs e: /mnt/e
/bin/cp -p foo /mnt/e/
rsync -av bar /mnt/e/

I did and used this since Fall Creators Update.
Now, I get:
/bin/cp -p foo /mnt/e/
/bin/cp: preservati gli orari di '/mnt/e/foo': Operazione non permessa

/usr/bin/rsync -av foo /mnt/e/
sending incremental file list
foo
rsync: mkstemp "/mnt/e/.foo.lMsStT" failed: Operation not permitted (1)

sent 103 bytes  received 111 bytes  428.00 bytes/sec
total size is 0  speedup is 0.00
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1183) [sender=3.1.1]

If you do not have a quick solution about this, what I described is a big regression of WSL April Update which has brought WSL back before Fall Update was released...

Maybe #2770

@SvenGroot - would you mind taking a look at this? Seems like this should work.

Hi @benhillis, please explain what does it mean Seems like this should work. Here also this does not work anymore:
rsync -av foo.tmp/ /mnt/e/foo.tmp/
sending incremental file list
created directory /mnt/e/foo.tmp
rsync: chgrp "/mnt/e/foo.tmp/." failed: Operation not permitted (1)
rsync: chgrp "/mnt/e/foo.tmp/bar" failed: Operation not permitted (1)
./
bar -> foo.txt
foo.txt
rsync: mkstemp "/mnt/e/foo.tmp/.foo.txt.ZUiZwf" failed: Operation not permitted (1)

sent 169 bytes  received 312 bytes  962.00 bytes/sec
total size is 14  speedup is 0.03
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1183) [sender=3.1.1]

and this does not work:
tar -xvof foo.tar.gz -C /mnt/e/
foo.tmp/
foo.tmp/bar
tar: foo.tmp/bar: funzione "utime" non riuscita: Operazione non permessa
foo.tmp/foo.txt
tar: foo.tmp/foo.txt: funzione "utime" non riuscita: Operazione non permessa
tar: foo.tmp: funzione "utime" non riuscita: Operazione non permessa
tar: foo.tmp: impossibile cambiare il modo a rwxr-xr-x: Operazione non permessa
tar: Uscita con stato di fallimento in base agli errori precedenti

That means all my scripts to save my WSL home are broken and I have to resume the old strategy I adopted when WSL knew only C fix drive, i.e. creating a tarball, moving it in Windows and then unpacking it on the USB key...
It is a very good result, congratulations to the developers...

I'm saying it sounds like a bug that we should investigate.

@benhillis wrote:

Please keep your discourse professional and civil.

Now you should prove that you are civil restoring what I wrote... if you developers have created a mess, you should be happy that someone stress this...
In any case, for the sake of completeness, it seems that adding the option metadata,
sudo mount -t drvfs e: /mnt/e -o metadata
allows cp - p, tar and rsync to work as expected.
In the example you have put on the WEB, often mount is used without sudo but if I do not use sudo I get an error:
mount -t drvfs e: /mnt/e -o metadata
mount: only root can use "--options" option

mount -t drvfs e: /mnt/e
mount: only root can use "--types" option

Usually I use the same USB key both in Windows and WSL. Now, when I mount it adding metadata option, the Windows files are root root while those copied with WSL are utente utente. Maybe this has its logic but intuitively an user would expect utente utente...
Hi @benhillis, I hope you do not yet censor what I wrote, you would make you a bad figure...

This is indeed a manifestation of #2770. When you mount your USB stick with default options, all the files will be owned by root, and your user doesn't have the permission to change the timestamps in that case. DrvFs used to bypass the owner check, but this is no longer the case since we added support for different owners with the "metadata" option.
Mounting with "metadata" works in this case because it allows the files you create to be owned by you, rather than root, however it's not the only option (and doesn't solve the problem for files already created by Windows, which don't have metadata and would therefore still be owned by root. For this reason, we added mount options to DrvFs that specify the owner uid and gid to use for files that don't have metadata.
The solution is to mount your USB drive using mount options setting the default uid/gid for files without metadata:
sudo mount -t drvfs e: /mnt/e -o uid=1000,gid=1000
You should use your actual uid/gid instead of 1000 (run id to determine your default account's uid/gid). Using the metadata option shouldn't be necessary unless you want to use non-default Linux permissions. If you do want to use this option, I recommend you use it together with the uid/gid options.
DrvFs mount options are described at the bottom of this blog post.

I'm having a similar issue, but -o metadata doesn't solve it: if I try to run rsync on a USB stick I get errors like this:

This is indeed a manifestation of #2770. When you mount your USB stick with default options, all the files will be owned by root, and your user doesn't have the permission to change the timestamps in that case. DrvFs used to bypass the owner check, but this is no longer the case since we added support for different owners with the "metadata" option.
Mounting with "metadata" works in this case because it allows the files you create to be owned by you, rather than root, however it's not the only option (and doesn't solve the problem for files already created by Windows, which don't have metadata and would therefore still be owned by root). For this reason, we added mount options to DrvFs that specify the owner uid and gid to use for files that don't have metadata.
The solution is to mount your USB drive using mount options setting the default uid/gid for files without metadata:
sudo mount -t drvfs e: /mnt/e -o uid=1000,gid=1000
You should use your actual uid/gid instead of 1000 (run id to determine your default account's uid/gid). Using the metadata option shouldn't be necessary unless you want to use non-default Linux permissions. If you do want to use this option, I recommend you use it together with the uid/gid options.
DrvFs mount options are described at the bottom of this blog post.

I think some of the issues are coming from how and when WSL will automatically mount  a drive. Maybe you can provide some insight on my question below?
Issue: I have noticed that WSL will sometimes automatically mount a drive. If it automatically mounts its the user/owner is set to the process that launched the bash.exe. If it does NOT automatically mount the drive, and I have to manually mount it using sudo mount -t drvfs e: /mnt/e, then the user/owner is set at root.
The general trend I have noticed is as follows:
If a USB drive is connected, then WSL will NOT automatically mount the drive
If an HDD is connected using SATA to USB cable, then WSL will automatically mount the drive
Why is this happening? What command can one use to determine if it was automatically mounted?

@Zythyr - When you run sudo, you effectively become root.  If you want to mount as a specific user, you will have to specify the uid and gid mount options:
For example:
sudo mount -t drvfs e: /mnt/e -o uid=1000,gid=1000

@Zythyr - We only automatically mount fixed drives.
When you run sudo, you effectively become root. If you want to mount as a specific user, you will have to specify the uid and gid mount options:
For example:
sudo mount -t drvfs e: /mnt/e -o uid=1000,gid=1000

Yes, I understand that, however my question was: Why is it that WSL sometimes automatically mounts a drive and sometimes it doesn't? My use case is for using rsync. Prior to me knowing that I have to use sudo mount -t drvfs e: /mnt/e -o uid=1000,gid=1000  instead of sudo mount -t drvfs e: /mnt/e I randomly had permission related issues. Now I realize it was because of how drive was originally mounted (automatically or manually). Why is it that WSL automatically mounts external HDD but does NOT automatically mount USB flash drive?

@Zythyr - USB flash drives are not fixed.

@Zythyr - USB flash drives are not fixed drives, they are removable.

Aren't external drives removable? I have a HDD connected to PC using SATA to USB cable. It act just like a USB flash drive, so why does WSL discriminate between the two?
Independent of a drive is "removable" or not, why does WSL not automatically mount it? Isn't this a logical approach? For example, if I were using a Ubuntu OS, if I connect a USB flash drive, it would automatically mount the drive, it don't have to manually go in the Terminal to mount it... Shouldn't WSL mimic this same behavior?

Why is it that WSL sometimes automatically mounts a drive and sometimes it doesn't?

Part of confusion derives from the choice of nomenclature.... (1) Drives aren't auto-mounted in WSL in the sense of the word commonly used in Linux. They're static mounted, at startup. Your SATA drive on a wire is treated as a fixed drive because that's what it looks like at startup. Your USB stick does not. (2) WSL doesn't actually have "drives" in the Linux sense either. There are no block devices in WSL that come and go, like in Real Linux.

For example, if I were using a Ubuntu OS, and I connect a USB flash drive, it would automatically mount the drive, I don't have to manually go in the Terminal to mount it... Shouldn't WSL mimic this same behavior?

The landing zone for that is #560 (message). Rhetoricals usually don't get much of a reply, so you can probably consider that part answered by yourself (affirmative or negative; whichever you prefer).

With all due respect, from my POV this behavior of WSL respect to USB devices is of no usefulness and I renounced it. I prefer to use a Windows folder as a sort of "cloud" to exchange data between WSL and the USB device. Usually, in the time it would take to mount (and then to unmount) the device as suggested in this discussion I have already transferred the data (with the help of MSYS2 in wich USB device are automounted).
Merry Christmas and Happy New Year.


[FLINK-5670] Properly clean up local RocksDB directories

We have to change the instance path to not include too many nested
directories, otherwise the Keyed backend cannot properly clean up the
whole directory hierachy.

@uce, @StefanRRichter  Could you please have a look, I think it would be good to have that in Flink 1.2.

+1 LGTM

@uce are you checking this with the Job where you initially discovered it?

Just verified it with the job. Works fine now! All directories were cleaned up properly.

Thanks 👍 😃


Not able to use qainsights/jmeter-docker-extension in Windows Laptop

Thank you for developing this Extension.
Actual Steps I did as a user

Installed the Latest Desktop Docker 4.25.0 (126437)
Pulled the docker pull qainsights/jmeter image and installed the qainsights/jmeter-docker-extension.
Developed the TestScript under TestPlan in Jmeter and Tested it.
Added the details in the qainsights/jmeter-docker-extension and clicked on Run Jmeter Test and got below error in the notification

Error creating container WARNING: The requested image's platform (linux/arm64/v8) does not match the detected host platform (linux/amd64/v4) and no specific platform was requested please check Docker daemon logs.

Later got to know post debugging, the image is for Linux platform.

Then again I tried to pull the windows image using below one, but unfortunately it couldn't pull it.
docker pull qainsights/windows-docker-jmeter

Here are my System Details:
OS: Microsoft Windows 10 Enterprise
Version: <IP_ADDRESS>
I wanted to check and explore qainsights/jmeter-docker-extension in Windows Laptop. If you have a moment, could you kindly share any insights or suggestions on resolving this matter? Your expertise would be immensely valuable.

@vishnuprakash9845 The Apache JMeter Docker Extension works only on Linux/Mac OS. It will not work in Windows yet. I have no plans to make it compatible with Windows for now; as 99% of containers will run on *nix OSes.
For qainsights/windows-docker-jmeter - it will work for AMD arch. If your arch is different, you can build it by referring to my repo.