Add test for pinning that fetches remote chunks

relates to: ethersphere/bee#571
Adds pin-remote check that performs pinning of bytes upload and then checks all those chunks are present locally using debug API.
One partly unfortunate thing is that because of how the Bee operates, if there is too few nodes in the topology, it is possible that all of them will replicate all chunks between themselves. As an attempt to "combat" this, few changes were made:

number of chunks that are being generated for upload was increased to 8000
timeout to wait for chunks replication is set to 15 seconds
there is "verification" step that queries each node for expected chunks, and only on the nodes that do not have all the chunks does it perform pinning

initial idea was to return an error if every node has all the chunks, however since that may make this test "flaky" single log line is written to note the fact

In relation to suggested change for the test:

about the test itself, the fact that we are relying on nodes that might have or not have certain chunks due to syncing means that we could potentially simplify the test scenario. in general, i think that beekeeper scenarios should be as much as possible under our control from beekeeper, thus, i would consider the following:

generate random data

get all data chunk references locally before uploading them (reuse pkg/check/localpinning/localpinning.go#addresses())

upload the data to one node

wait for syncing

delete all known data chunks on a random node (debugapi DELETE /chunks/{addr})

pin with the traversal api on the node from previous step

check same node has all deleted chunks

the only thing that is currently not being done is deleting chunks from the node, all the other steps are the same.
I have already explained my reasoning for not doing delete through debug API, but it probably should be fine, and I will update test to this.


How to install the PlistBuddy from this repo

I can't install a binary command on path as follow:
/usr/libexec/PlistBuddy

You can not install the PlistBuddy from this repo. I guess you need to install Xcode which probably installs the PlistBuddy in its installation phase.


doc: fix link to EPIC-HP design doc

Hi there,
just a small fix cause I am getting annoyed over having to fix the link manually all the time.
Current Link goes to a PR for a COLIBRI design document instead of EPIC.
Best Regards

This change is

Hi @HawkCorrigan
Thank you for the contribution 👍


revert: use v3.10-canary as work-in-progress version again

v3.10.1 has been released, we should not update the built
container-images with the same version. Use v3.10-canary for everything
that is merged in the release-v3.10 branch before we release a next
version.
Signed-off-by: Niels de Vos<EMAIL_ADDRESS>

This is actually a duplicate of #4337


Add authentication mechanism example

The service APIs require authentication and we should also add an example for that.
A first approach could be based on external authentication in the k8s ingress controller together with an identity provider, e.g. nginx together with keycloack (https://kubernetes.github.io/ingress-nginx/examples/auth/external-auth/)
However, this might not be possible on all clusters, in particular on managed k8s instances.
An alternative solution could be based on additional reverse proxys or api gateways deployed alongside the services in order to handle  authentication and authorization.

Shovon will be looking into a minimal setup including Grafana as an example service and keycloak.


[🐛 Bug]: generateStaticParams forces to be on the edge on nested routes.

next-on-pages environment related information
System:
        Platform: darwin
        Arch: arm64
        Version: Darwin Kernel Version 24.1.0: Thu Oct 10 22:08:48 PDT 2024; root:xnu-11215.41.3~5/RELEASE_ARM64_T6000
        CPU: (10) arm64 Apple M1 Pro
        Memory: 32 GB
        Shell: /bin/zsh
Package Manager Used: npm (10.8.1)

Relevant Packages:
        @cloudflare/next-on-pages: 1.13.5
        vercel: 37.12.1
        next: 15.0.1

Description
When deploying a Next.js application on Cloudflare Pages with @cloudflare/next-on-pages, dynamic routes using static parameters (e.g., [locale] for language-based URLs) experience unexpected build errors if they contain nested routes. Specifically, routes like /[locale]/page.tsx (e.g., /en, /nl) build and render correctly. However, nested pages such as /[locale]/test/page.tsx (e.g., /en/test, /nl/test) fail to build unless configured to use the Edge Runtime by exporting export const runtime = 'edge';.
This requirement appears to be a misconfiguration in @cloudflare/next-on-pages, as the Edge Runtime should not be mandatory for these routes. This issue does not occur when deploying the same Next.js app structure to Vercel, where dynamic and nested routes function without needing the Edge Runtime, as expected.
Expected Behavior:
Both /[locale]/page.tsx and /[locale]/test/page.tsx routes should build and render without requiring the Edge Runtime. These routes work as expected in a Vercel deployment, with dynamic and nested routes rendering without Edge Runtime configuration. The requirement for the Edge Runtime on Cloudflare Pages is unexpected and blocks deployment for dynamic, multi-language route structures.
Observed Behavior:
The build fails for nested dynamic routes unless the Edge Runtime is explicitly set, which should not be necessary and deviates from typical Next.js behavior on other platforms. This leads to inconsistencies between platforms and prevents successful deployment on Cloudflare Pages for standard Next.js dynamic route setups.
Reproduction

Clone the repository: https://github.com/rickmartensnl/cf-pages-dynamic-edge-issue.
Run the following command to build the project:npm run pages:build

Observe the build output. Cloudflare Pages generates a series of warnings and errors indicating that non-static routes must be configured with the Edge Runtime to build correctly.

Pages Deployment Method
None
Pages Deployment ID
No response
Additional Information
><EMAIL_ADDRESS>pages:build
> npx @cloudflare/next-on-pages

⚡ @cloudflare/next-on-pages CLI v.1.13.5
⚡ Detected Package Manager: npm (10.8.1)
⚡ Preparing project...
⚡ Project is ready
⚡ Building project...
▲  Vercel CLI 37.12.1
▲  WARNING: You should not upload the `.next` directory.
▲  Installing dependencies...
▲  up to date in 276ms
▲  33 packages are looking for funding
▲  run `npm fund` for details
▲  Detected Next.js version: 15.0.1
▲  Running "npm run build"
▲  ><EMAIL_ADDRESS>build
▲  > next build
▲  ▲ Next.js 15.0.1
▲
▲  Creating an optimized production build ...
▲  ⚠ Found lockfile missing swc dependencies, run next locally to automatically patch
▲  ⚠ Found lockfile missing swc dependencies, run next locally to automatically patch
▲  ⚠ Found lockfile missing swc dependencies, run next locally to automatically patch
▲  ✓ Compiled successfully
▲  Linting and checking validity of types ...
▲  Collecting page data ...
▲  ⚠ Found lockfile missing swc dependencies, run next locally to automatically patch
▲  ⚠ Using edge runtime on a page currently disables static generation for that page
▲  Generating static pages (0/8) ...
▲  Generating static pages (2/8)
▲  Generating static pages (4/8)
▲    Generating static pages (6/8)
▲  ✓ Generating static pages (8/8)
▲  Finalizing page optimization ...
▲  Collecting build traces ...
▲
▲  Route (app)                              Size     First Load JS
▲  ┌ ○ /                                    5.64 kB         105 kB
▲  ├ ƒ /_not-found                          149 B          99.3 kB
▲  ├ ● /[locale]                            149 B          99.3 kB
▲  ├   ├ /nl
▲  ├   └ /en
▲  ├ ● /[locale]/test                       149 B          99.3 kB
▲  ├   ├ /nl/test
▲  ├   └ /en/test
▲  └ ƒ /api/hello                           149 B          99.3 kB
▲  + First Load JS shared by all            99.1 kB
▲  ├ chunks/215-a5930070353d5622.js       44.6 kB
▲  ├ chunks/4bd1b696-21eeec6ae99fe726.js  52.6 kB
▲  └ other shared chunks (total)          1.91 kB
▲  ○  (Static)   prerendered as static content
▲  ●  (SSG)      prerendered as static HTML (uses generateStaticParams)
▲  ƒ  (Dynamic)  server-rendered on demand
▲  Traced Next.js server files in: 122.955ms
▲  Created all serverless functions in: 29.208ms
▲  Collected static files (public/, static/, .next/static): 2.345ms
▲  Build Completed in .vercel/output [9s]
⚡ Completed `npx vercel build`.
⚡ Invalid prerender config for /[locale]/test
⚡ Invalid prerender config for /[locale]/test.rsc
⚡ Invalid prerender config for /[locale]
⚡ Invalid prerender config for /[locale].rsc

⚡ ERROR: Failed to produce a Cloudflare Pages build from the project.
⚡
⚡      The following routes were not configured to run with the Edge Runtime:
⚡        - /[locale]/test
⚡
⚡      Please make sure that all your non-static routes export the following edge runtime route segment config:
⚡        export const runtime = 'edge';
⚡
⚡      You can read more about the Edge Runtime on the Next.js documentation:
⚡        https://nextjs.org/docs/app/building-your-application/rendering/edge-and-nodejs-runtimes

Would you like to help?

[X] Would you like to help fixing this bug?

After some more testing I found out that when using<EMAIL_ADDRESS>it also failed for the /[locale] endpoint, which means between 1.12.1 and 1.13.1 there was a fix for this issue, but did not fully fix the issue.

I have same issue

same here

same

Same :/

The above fix didn't work for me.
Issue seems to occur starting from<EMAIL_ADDRESS>where I'm getting an issue when building static routes without runtime = 'edge' despite the route being fully static (dynamic = "error", dynamicParams = false).
Error occurs in a path myPath/[[...args]]/ -- might be related?
Could be related to #833 -- for now I stick to<EMAIL_ADDRESS>

The above fix didn't work for me.
Issue seems to occur starting from<EMAIL_ADDRESS>where I'm getting an issue when building static routes without runtime = 'edge' despite the route being fully static (dynamic = "error", dynamicParams = false).
Error occurs in a path myPath/[[...args]]/ -- might be related?
Could be related to #833 -- for now I stick to<EMAIL_ADDRESS>

The above fix didn't work for me.
Issue seems to occur starting from<EMAIL_ADDRESS>where I'm getting an issue when building static routes without runtime = 'edge' despite the route being fully static (dynamic = "error", dynamicParams = false).
Error occurs in a path myPath/[[...args]]/ -- might be related?
Could be related to #833 -- for now I stick to<EMAIL_ADDRESS>
yes, the same worked for me.

The above fix didn't work for me.
Issue seems to occur starting from vercel/vercel<EMAIL_ADDRESS>(release), where I'm getting an issue when building static routes without runtime = 'edge' despite the route being fully static (dynamic = "error", dynamicParams = false).
Error occurs in a path myPath/[[...args]]/ -- might be related?
Could be related to #833 -- for now I stick to vercel/vercel<EMAIL_ADDRESS>(release).

yes, the same worked for me.

What nexths and next-on-pages version are you using? I've tried downgrading vercel to 34.4.0, but with latest 1.13.7 next-on-pages version I still get an error about edge runtime.

What nexths and next-on-pages version are you using? I've tried downgrading vercel to 34.4.0, but with latest 1.13.7 next-on-pages version I still get an error about edge runtime.

Ah yes -- I forgot to mention, I'm using<EMAIL_ADDRESS>


Fix: increase max number of domains to 100 (match with Let's Encrypt)

CC @jc21
Let's make the limit of domains count equal to the Let's Encrypt limit, as suggested here https://github.com/NginxProxyManager/nginx-proxy-manager/pull/3382#issuecomment-1854005778, when the limit was raised from 15 to 30 in last December.
In my use case, I have a big SAN cert - lots of subdomains and "www." is needed to be supported too, so everything twice; I am not able to configure this in nginx-proxy-manager at the moment.
ref: https://github.com/NginxProxyManager/nginx-proxy-manager/issues/3218, https://github.com/NginxProxyManager/nginx-proxy-manager/pull/3382
I would be glad if you could approve & merge this change :)
Thx and have a nice day,
D.

Thanks @jc21! 🙂 Could you say when you will make a new release to the :latest Docker tag?


change line protocol to reflect changes in newest influx (now with tests!)

Building off of #232, this fixes the tests and ensures that everything works. @xbs13's code was checking if the value was an int after it had been str'd, so I moved it earlier in the call stack.
On a side note, it appears there was a change in 0.9.3-rc3 (possibly a previous rc) that causes the tags to be returned when calling query. I didn't look into it much, just changed the tests to handle the results coming back.

(the tests are failing b/c it's running against influxdb 0.9.2)

ah! I just saw this after I committed the fix :~

@xbs13 no problem! I got the tests fixed for ya too :)

@djeebus thank you!

@xbs13 @djeebus
Thank you both! I have merged the changes :)
I'll tag the release today.


Single benchmark issue with GCloud CLI

Hello there,
I've set and init my gcloud current project as usual. I run the single benchmark with right configuration and project ID.
Then I get this error message with an depreciate command on gcloud CLI apparently.
Error:
2018-07-27 11:03:15,962 8a514f14 Thread-2 iperf(1/1) vm_util.py:321 INFO     Ran gcloud compute networks create pkb-network-8a514f14 --format json --quiet --project manawa-20009664 --mode auto. Got return code (2).
STDOUT:
STDERR: ERROR: (gcloud.compute.networks.create) `mode` has been removed. Please use `subnet-mode` instead.
Usage: gcloud compute networks create NAME [optional flags]
  optional flags may be  --bgp-routing-mode | --description | --help | --mode |
                         --range | --subnet-mode

For detailed information on this command and its flags, run:
  gcloud compute networks create --help

After that PerfKit try to run that in loop but nothing change...
I use the latest release 1.13.0, is it normal? Maybe I could create that manually and force with a flag?
Thanks in advance 👍

Hi w3st3ry,
It looks like that has been fixed in head (https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/commit/2db3b2125d884ef0153be0a9c04e55ea5335bc09), but we haven't cut a new release since then.  You can either run PKB from head (recommended) or roll gcloud back to a version which supports mode.


LinkedIn is not defined

Getting the following error when trying to sign in with LinkedIn:

Uncaught ReferenceError: LinkedIn is not defined @linkedin.js:16

packages:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
js:
Meteor.linkWithLinkedIn();

Have I missed anything? Would bozhao:link-accounts need an api.use() to get access to the exported vars such as LinkedIn from jonperl:linkedin or make use of Package[]?
To evade this error I did this on the client:
Meteor.linkWithLinkedIn = function (options, callback) {
  ...

  var credentialRequestCompleteCallback = Accounts.oauth.linkCredentialRequestCompleteHandler(callback);
  Package['jonperl:linkedin'].LinkedIn.requestCredential(options, credentialRequestCompleteCallback);
};

i could confirm i had the same issue and solution from @JohanTh works for me as well...
thank You..

Add only jonperl:linkedin package directly, not jonperl:accounts-linkedin.

@JohanTh @delgermurun If you could identify where the issue is in the code a PR would be much appreciated. If there is not an issue, maybe a note in the README to the appropriate package should suffice.


the property of cortical area in the leftbar menu collapsed everytime I click new.

branches:
FEAGI: structured-errors
BV: standardized_web_error_handling
https://github.com/feagi/brain-visualizer/assets/65916520/4b6680ed-36d5-4156-b711-45eedf17c83b

resolved


Spectrum side changes for "Spectrum addition #1952"-PR in Specter Desktop

This is a PR with Spectrum side changes to make https://github.com/cryptoadvance/specter-desktop/pull/1952 work. It also includes some front end and chore (version bumps etc.) clean up.

Latest status is this stacktrace:
Traceback (most recent call last):
File "/Users/heinzi/coding/specter-desktop/src/cryptoadvance/specter/util/checker.py", line 53, in _execute
self.callback()
File "/Users/heinzi/coding/specter-desktop/src/cryptoadvance/specter/specter.py", line 153, in
self.checker = Checker(lambda: self.check(check_all=True), desc="health")
File "/Users/heinzi/coding/specter-desktop/src/cryptoadvance/specter/specter.py", line 208, in check
if self.rpc is None or self.node.info.get("initialblockdownload", True):
File "/Users/heinzi/coding/specter-desktop/src/cryptoadvance/specter/node.py", line 54, in info
for r in self.rpc.multi(
File "/Users/heinzi/coding/spectrum/src/cryptoadvance/specterext/spectrum/bridge_rpc.py", line 77, in multi
with self._app.app_context():
File "/Users/heinzi/.pyenv/versions/.specter-desktop-env-3.9.0/lib/python3.9/site-packages/werkzeug/local.py", line 422, in get
obj = instance._get_current_object()
File "/Users/heinzi/.pyenv/versions/.specter-desktop-env-3.9.0/lib/python3.9/site-packages/werkzeug/local.py", line 544, in _get_current_object
return self.__local()  # type: ignore
File "/Users/heinzi/.pyenv/versions/.specter-desktop-env-3.9.0/lib/python3.9/site-packages/flask/globals.py", line 47, in _find_app
raise RuntimeError(_app_ctx_err_msg)
RuntimeError: Working outside of application context.
This typically means that you attempted to use functionality that needed
to interface with the current application object in some way. To solve
this, set up an application context with app.app_context().


Two-column state layout

Make the long list of states more condensed. This is part of the #4004 but it's taking a while to get Iowa onboarded, so I'm breaking it out.
Added definition of ELR by using the full term the first time it's mentioned. "Electronic Laboratory Reporting (ELR) connection"

Desktop

Mobile

Pull reviewers stats
Stats for the last 30 days:

User
Total reviews
Median time to review
Total comments

MauriceReeves-usds
56
1h 25m
27

carlosfelix2
27
7h 44m
66

jimduff-usds
20
13h 41m
19

cwinters-usds
20
3h 51m
8

TomNUSDS
16
1h 25m
42

ahay-agile6
14
3h 2m
7

whytheplatypus
13
1h 59m
29

oslynn
11
1h 4m
5

jbiskie
11
31m
0

kevinhaube
7
1h 25m
0

jorg3lopez
6
1h 24m
5

brick-green-agile6
6
16h 35m
2

sethdarragile6
6
2h 19m
0

JosiahSiegel
5
1h 19m
2

meckila
5
2h 38m
9

rhood23699
5
47m
0

bgantick
4
2h 23m
3

RickHawesUSDS
3
4h 6m
0

TurkeyFried
2
21h 48m
1

jeremy-page
2
8m
0

clediggins-usds
2
8h 59m
1

acoushawk
1
16h 10m
0

snesm
1
16m
0

Closing, @TomNUSDS merged these in with #4004


Add an asmdef for Unity

This enables unsafe code in every supported Unity version and optimizes compilation to not recompile LiteNetLib every time.

From Unity doc:
"This is also the reason why the assembly definition files do not support setting of preprocessor directives (defines), as those are static at all times."
LiteNetLib uses some defines for platforms (IOS/switch) so this will be a problem.

Unitys defines are defined within asmdefs normally, I think they mean that you can't set custom defines for an asmdef separately from the rest of code.

I think i need to test this before upload...

I've realised, this introduces a small breaking change for people who already had it placed in an asmdef, then they'll have to reference the new one from the old one. I don't think this really matters, since it's just 3 clicks to add a reference.


Fix incorrect proper name translation and phrases with wrong meaning

Some occurrences of proper names, like "React" and "propTypes" were wrongly translated to Portuguese.
Also fixed mixed line endings (CF by LF).

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

Thank you for your contribution to the FreeCodeCamp Portuguese Translation!
Contributions like yours help millions of users to learn programming in their native language.
Hope you keep helping us improve our Portuguese Translation with more contributions.


improvement(marginfi-v2-ui): add Coinbase and SquadsX wallets to extended options

Issue
Both the Coinbase and SquadsX team raised issues with me this past week that users are unable to use their respective wallets to onboard to marginfi.
Solution
I add SquadsX and Coinbase wallet adapters inside the src/hooks/useAvailableWallet.ts file and conditionally added these adapters to the addAbsentWallets function.

Can you rebase this branch on main @masterzorgon

rebased!

I made some changes to the onboarding process. Previously, we filtered the wallet list to show only relevant options for social, Sol, and Eth flows. However, this caused an issue where users could not find their wallet during onboarding. To address this, I removed the forced filter for the Sol and Eth flows so that all installed wallets are now visible. Additionally, I have reordered the wallets to prioritize certain ones.
let me know your thoughts @masterzorgon @chambaz

I made some changes to the onboarding process. Previously, we filtered the wallet list to show only relevant options for social, Sol, and Eth flows. However, this caused an issue where users could not find their wallet during onboarding. To address this, I removed the forced filter for the Sol and Eth flows so that all installed wallets are now visible. Additionally, I have reordered the wallets to prioritize certain ones.
let me know your thoughts @masterzorgon @chambaz

I tend to agree here. I think the way we had attempted to do it makes sense, reducing choice paralysis, and simplifying the flow. however from feedback users and wallet providers expect to see their option available and I think your latest solution is the easiest option. one day I pray for a true Wallet Standard and this implementation nightmare goes away.

I made some changes to the onboarding process. Previously, we filtered the wallet list to show only relevant options for social, Sol, and Eth flows. However, this caused an issue where users could not find their wallet during onboarding. To address this, I removed the forced filter for the Sol and Eth flows so that all installed wallets are now visible. Additionally, I have reordered the wallets to prioritize certain ones.
let me know your thoughts @masterzorgon @chambaz

I made some changes to the onboarding process. Previously, we filtered the wallet list to show only relevant options for social, Sol, and Eth flows. However, this caused an issue where users could not find their wallet during onboarding. To address this, I removed the forced filter for the Sol and Eth flows so that all installed wallets are now visible. Additionally, I have reordered the wallets to prioritize certain ones.
let me know your thoughts @masterzorgon @chambaz

I made some changes to the onboarding process. Previously, we filtered the wallet list to show only relevant options for social, Sol, and Eth flows. However, this caused an issue where users could not find their wallet during onboarding. To address this, I removed the forced filter for the Sol and Eth flows so that all installed wallets are now visible. Additionally, I have reordered the wallets to prioritize certain ones.
let me know your thoughts @masterzorgon @chambaz

I see that the SquadsX Wallet option is not available during the onboarding flow.

Rebasing on main rn, but should be good to go when @chambaz approves it


Add Dependabot configuration for GitHub Actions updates

This PR adds a Dependabot configuration that checks once a week if the GitHub Actions are still using the latest version. If not, it opens a PR to update them.
It will actually open few PRs, since only major versions are specified (like v3), so only on a major release (like v4) it will update and open a PR. But it helps actively keep GitHub Actions workflows up to date and secure.
See Keeping your actions up to date with Dependabot.

Awesome, thank you!


Consider clarifying/correcting our docs on mapping dates

In our public-facing Slack, a user has asked for more clarity (or possibly a correction) about mapping dates in our docs. He didn't say which doc/s specifically he was looking at.
User's thread

Our Internal Slack discussion
Someone needs to investigate whether our docs are misleading or incomplete on this and fix.
Lucy notes that our new type system guide might be useful. It uses DateTime and @db.Date as an example . It might be sufficient to cover this subject, or it might be a good resource when you update any other pages for this ticket.

Closing this because there's nothing actionable. The user did not clarify what part of the documentation confused him, and the existing documentation, to the best of my knowledge, is accurate.


Add more disabled swiftlint rules

There are a number of rules that we tend to disable whenever we encounter them and it seems to make sense to me that we just add these to the list of disabled rules as they're not really ones that we care about.
If we're in agreement then I will also go ahead and remove all of the places that we have done //swiftlint:disable <redundant_rule_name>.
Thoughts on keeping any of these or others we'd like to add? 🤔

totally down with these 4!


[SPARK-14158][SQL] implement buildReader for json data source

What changes were proposed in this pull request?
This PR implements buildReader for json data source and enable it in the new data source code path.
How was this patch tested?
existing tests

cc @marmbrus  @liancheng @yhuai

Test build #54180 has started for PR 11960 at commit 3efd4a1.

Test build #54180 has finished for PR 11960 at commit 3efd4a1.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class HadoopFileLinesReader(file: PartitionedFile, conf: Configuration) extends Iterator[Text]

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54180/
Test FAILed.

Overall looks good. Let's fix the serialization issue.

Test build #54232 has started for PR 11960 at commit dd6cef3.

Test build #54232 has finished for PR 11960 at commit dd6cef3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54232/
Test FAILed.

Test build #54323 has started for PR 11960 at commit 59ae3b5.

Test build #54323 has finished for PR 11960 at commit 59ae3b5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54323/
Test PASSed.

Test build #54394 has started for PR 11960 at commit e69551b.

Test build #54394 has finished for PR 11960 at commit e69551b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54394/
Test PASSed.

Merged build finished. Test PASSed.

merging to master!


Parser should provide hook to return the AST AND file path which the AST lives in

Feature request

What is the expected behavior?
A parser hook should return the file path where the AST under investigation lives.
What is motivation or use case for adding/changing the behavior?
It will benefit developers who need to traverse AST and know exactly where the AST lives.
How should this be implemented in your opinion?
Either the chunk instance provides tappable AST or the parser instance provides file location property.
Are you willing to work on this yourself?
yes with guidance.

See parser.state.module

Given the following code, parser.state is undefined:
        compiler.hooks.compilation.tap(
            'Plugin',
            (compilation, { normalModuleFactory }) => {
                const handler = (parser, parserOptions) => {
                    console.log(parser.state.module);
                    parser.hooks.evaluate
                        .for('ArrowFunctionExpression')
                        .tap('Plugin', expression => {
                          ...
                };
                normalModuleFactory.hooks.parser
                    .for('javascript/auto')
                    .tap('DibsAsyncCheckPlugin', handler);
            }
        );

Directly tapping into normalModuleFactory with compiler.hooks.normalModuleFactory.tap('DibsAsyncCheckPlugin', compilation => { also returns an undefined parser.state. Given that the src indicates parser.state = OldState, what will be the appropriate hook that I should use?

I also tried the code below but the callbacks seem to be asynchronous so the file paths don't correspond to the target file.
    compiler.hooks.normalModuleFactory.tap('Plugin', factory => {

        factory.hooks.module.tap('Plugin', (data, param) => {

            console.log(data.request);

            data.parser.hooks.evaluate
                .for('ArrowFunctionExpression')
                .tap('Plugin', expression => {

                     console.log(data.request);

                });
        });


poster plugin img role presentation for accessibility

because the poster img (image) is for presentation purposes and this change solves an accessibility issue where accessibility auditors require all img tags to have either an "alt" attribute or role="presentation".

Add test to make sure the role attribute is there and is set to presentation

alt attribute for good measure


Regression bug: unexpected keyword argument 'token_info'

Connexion 1.1.7 breaks my app https://github.com/zalando-incubator/aws-credentials-service (it worked with Connexion 1.1.4 and earlier):
[2017-04-05 11:03:07,029] ERROR in app: Exception on /aws-account-roles/jdoe [GET]
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/flask/app.py", line 1982, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.5/dist-packages/flask/app.py", line 1614, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.5/dist-packages/flask/app.py", line 1517, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python3.5/dist-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/usr/local/lib/python3.5/dist-packages/flask/app.py", line 1612, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.5/dist-packages/flask/app.py", line 1598, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/local/lib/python3.5/dist-packages/connexion/decorators/decorator.py", line 66, in wrapper
    response = function(request)
  File "/usr/local/lib/python3.5/dist-packages/connexion/decorators/security.py", line 90, in wrapper
    return function(request)
  File "/usr/local/lib/python3.5/dist-packages/connexion/decorators/validation.py", line 299, in wrapper
    return function(request)
  File "/usr/local/lib/python3.5/dist-packages/connexion/decorators/decorator.py", line 42, in wrapper
    response = function(request)
  File "/usr/local/lib/python3.5/dist-packages/connexion/decorators/parameter.py", line 188, in wrapper
    return function(**kwargs)
TypeError: get_account_roles() got an unexpected keyword argument 'token_info'

There is no parameter token_info defined, but Connexion tries to pass it to my handler function now :disappointed:

This is pretty bad as I suspect other apps using Token Info to break too...

Some bug can be reproduced with Connexion's OAuth example: https://github.com/zalando/connexion/tree/master/examples/oauth2

@hjacobs It should be ok after #432 . Users should not even notice after that gets merged. 👍

My app now works again with https://github.com/zalando/connexion/releases/tag/1.1.8 :tada:


Make deprecation warning clearer

Context
Currently, if a build issues deprecation warnings, the output will show this:
Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.1-branch-lptr_vfs_darwin_file_watcher-20191203160126+0000/userguide/command_line_interface.html#sec:command_line_warnings

I've read several comments from folks not really understanding what this means. I think we could make the message a bit clearer:
Deprecated Gradle features were used in this build. This means that your build is likely to break when you'll upgrade to the next major release (Gradle 7.0).

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

See https://docs.gradle.org/6.1-branch-lptr_vfs_darwin_file_watcher-20191203160126+0000/userguide/command_line_interface.html#sec:command_line_warnings

Note the presence of additional blank lines to make it clearer too.
Ideally we should even detect if the deprecation warning comes from the user scripts or external plugins.

Hi @melix , I'm completely new to all of this. I have gotten this error message when trying to gradle generateSource for JabRef. When I got this the Build failed, is this because of possible incompability to future gradle versions? To me this doesn't sound like it could be responsible for failing the Built. Anyway, I am very desperate for a solution, if you have any idea how I can fix this I would really appreciate it.

The message above is a warning. It doesn't fail the build unless you explicitly have set warning mode to fail. The message tells you the build is going to fail if you upgrade Gradle to the next major.

Yes, that makes more sense! Thank you for the quick response.


Recommendation vs joohoi/acme-dns-certbot-joohoi

Question: Do you now recommend this software versus joohoi/acme-dns-certbot-joohoi?
They appear to be direct alternatives, or is that incorrect?
Thanks!

They are direct alternatives, you are right. And yes, this was written to
replace the python wrapper.
This client addresses many shortcomigs of acme-dns-certbot-joohoi
On Wed 23. Jun 2021 at 22.54, Cris Perdue @.***> wrote:

Question: Do you now recommend this software versus
joohoi/acme-dns-certbot-joohoi?
They appear to be direct alternatives, or is that incorrect?
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/acme-dns/acme-dns-client/issues/9, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABH6DJOOIJBIFEV2SBVHLQ3TUI3XFANCNFSM47GPDJ2Q
.

Thanks much, that is very helpful info.
For what it's worth, there is a tutorial on using the old Python version,
at https://www.digitalocean.com/community/tutorials/how-to-acquire-a-let-s-encrypt-certificate-using-dns-validation-with-acme-dns-certbot-on-ubuntu-18-04.
Perhaps the author would be interested in creating an updated tutorial when you feel your new software is ready for more widespread use.

By looking at old Python code version, it seems to avoid issue #13:

https://github.com/joohoi/acme-dns-certbot-joohoi/blob/908aefb0ea3925aaeaf78a7ca32e6a526b1bb563/acme-dns-auth.py#L122-L123

It's a problematic issue with the new go client which may affect usage with most registrars when attempting to obtain a wildcard certificate.

That's a good observation @oblitum , I'll fix that in Go client!

...and it's fixed

Does this new client have dockerised deployment or must we manage scripts etc. manually? (Sorry, I'm new to this, but long-time user of certbot.)


SOURCE CONTROL: GIT - ISSUES

Issue Type: Bug
I just can't demap, remap, do nothing with Source Control: GIT... how can i reconfigure this??? There's nothing in options.
Azure Data Studio version: azuredatastudio 1.7.0 (e1280022d69b651cfff04b30e830904575c8acda, 2019-05-08T00:55:40.928Z)
OS version: Windows_NT x64 6.3.9600

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E5-2680 v4 @ 2.40GHz (4 x 2400)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_offsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
6.00GB (0.70GB free)

Process Argv

Screen Reader
no

VM
100%

Extensions (10)

Extension
Author (truncated)
Version

managed-instance-dashboard
joc
0.3.0

admin-pack
Mic
0.0.2

agent
Mic
0.38.0

dacpac
Mic
0.2.0

import
Mic
0.7.0

profiler
Mic
0.7.0

server-report
Mic
0.1.5

sql-dw
Mic
0.0.1

whoisactive
Mic
0.1.1

sql-search
Red
0.3.3

@MAL- I'm not clear what's the issue describing.  What do you mean by "demap" and "remap"?
Are you experiencing the same problem in VS Code?  We haven't customized the experience at all from what we pick-up from upstream, AFAIK.


How is the code licensed?

Hello Kurapan,
thanks for the implementation inKeras, which was very instructive for me. Unfortunately, I can't find any information about the license and its possible uses. Can you tell me under which license the code is published here? Many thanks in advance.

Hello Christian!
Sorry for the inconvenience. I've added a license file to the project. It's MIT.

Thanks a lot! Now I can use the code. This helps a lot :)


Add ttir.scatter op to the TTIR dialect

As part of the effort to run stableHLO models on TT silicon, adding an implementation of the scatter op in the TTIR dialect.
This op currently has limited functionality, but enough to pass the current models, primarily due to the current limitations in tt-metal, for more information please see https://github.com/tenstorrent/tt-mlir/pull/1279.

Update: The op with limited functionality, as is currently implemented in metal (see issue https://github.com/tenstorrent/tt-metal/issues/4294) is now implemented in the TTIR dialect (https://github.com/tenstorrent/tt-mlir/pull/1279).
However, as described above, we currently do not support custom indexes, nor custom function for merging the two tensor operands of the scatter, which is supported in StableHLO (https://github.com/tenstorrent/tt-mlir/issues/1278), so I'm leaving this issue open for now until we get full support.


Format with AdRoll/rebar3_format

WIP do not merge.
Tried https://github.com/AdRoll/rebar3_format
(it has a lot of config options though - could be useful to match MIM style).

Could close, it is PR for demo only.


[BUG] wrong results in using-feature-columns.ipynb nb after Preprocessing with NVT

Describe the bug
The using-feature-columns.ipynb nb is giving wrong results after doing the first preprocessing step which is as below:
cat_names = ColumnGroup(
    [
        "Type",
        "Breed1",
        "Color1",
        "Color2",
        "MaturitySize",
        "Gender",
        "FurLength",
        "Vaccinated",
        "Sterilized",
        "Health",
    ]
)
cont_names = ColumnGroup(["Age", "Fee", "PhotoAmt"])
label = ColumnGroup(["AdoptionSpeed"])

label_feature = label >> (lambda col: cp.where(col == 4, 0, 1))
workflow = nvt.Workflow(cat_names + cont_names + label_feature)

for split in ["train", "valid", "test"]:
    output_path = os.path.join(data_dir, "processed", split)
    if not os.path.exists(output_path):
        os.makedirs(output_path)

    dataset = nvt.Dataset(os.path.join(data_dir, split + ".parquet"))
    if split == "train":
        shuffle = nvt.io.Shuffle.PER_WORKER
        workflow.fit(dataset)
        workflow.transform(dataset).to_parquet(
            output_path=output_path, shuffle=shuffle, out_files_per_proc=1
        )
    else:
        shuffle = False
        workflow.transform(dataset).to_parquet(
            output_path=output_path, shuffle=shuffle, out_files_per_proc=1
        )

The issues:

the original train, valid and test sets do not have null values for the feautes that we use in training, but after doing preprocessing with NVT, we get some <NA> for each column.
Although we set  out_files_per_proc=1 both for train and valid, we get 2 output parquet files in the output path.
The final gdfs total num_rows is grater than the original file's num_rows.

Steps/Code to reproduce bug
To reproduce the issues above, run the using-feature-columns.ipynb nb and check the preprocessed
Environment details (please complete the following information):

Environment location: [Bare-metal, Docker, Cloud(specify cloud provider)]: Docker container
Method of NVTabular install: [conda, Docker, or from source] Docker

If method of install is [Docker], provide docker pull & docker run commands used

docker run --gpus=3 -it -p 9999:8888 -p 9797:8787 -p 9796:8786 --ipc=host --cap-add SYS_PTRACE gitlab-master.nvidia.com:5005/rapidsdl/docker/rapidsdl_joc:latest

I also ran the notebook in a conda env using rapids 0.16 and rapids 0.17. I have had the same issues.

@rjzamora would you mind looking into the issues I reported above? Did we change anything regarding to writing to parquet the output files?  Do you have any idea about the problem? Thanks for your time.

@rnyak - I wasn't able to reproduce the file-count problem (I always get 1 file with out_files_per_proc=1), but I did notice that index misalignment was causing the other issues during column concatenation.  I added a fix for this to #566 -  Could you check if the problems are resolved in that branch?

@benfred closing. It is now working after the fix in https://github.com/NVIDIA/NVTabular/pull/566.


Pinning a version of click to fix travis

Click recently came out with a new version where "Subcommands that are named by the function now automatically have the underscore replaced with a dash. If you register a function named my_command it becomes my-command in the command line interface." This breaks tests as it affects some cli commands superset load_examples is now superset load-examples.
Pinning to the previous version of click before this change to fix travis.
@williaster @john-bodley @kristw

Codecov Report

Merging #5975 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5975      +/-   ##
==========================================
+ Coverage   63.52%   63.56%   +0.04%
==========================================
  Files         393      393
  Lines       23667    23667
  Branches     2638     2638
==========================================
+ Hits        15034    15044      +10
+ Misses       8620     8610      -10
  Partials       13       13

Impacted Files
Coverage Δ

superset/connectors/sqla/models.py
81.13% <0%> (+0.75%)
:arrow_up:

superset/db_engine_specs.py
55.7% <0%> (+0.87%)
:arrow_up:

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 569f221...b80ad44. Read the comment docs.

Sounds like that means that installing using pip install superset won't work. We should fix setup.py as well.


Only restore changes

I have an remote DB that I need to take a backup of every hour then restore a the local DB with the remote data, this may be outside of scope, but would is it possible to only restore the changes rather than the whole database, currently if I do a restore all the data is dropped and I don't have access to the data for 15 minutes or so every hour.
I think I'm potentially looking at this wrong and want to do a differential database backup.
Cheers.

below is what I came out in the first thought:
Save the date time of every data export action taken from remote DB >> "LastBackupTime"
Then every rows in all table will have "last update time" > "LastUpdateTime".
Whenever you perform the next data export from remote DB, only get new data by comparing the "LastUpdateTime" to "LastBackupTime". All rows with "LastUpdateTime" older than "LastBackupTime" will be exported.
At MySqlBackup.NET part, it will be something like this:
DateTime LastBackuptime = GetLastBackupTime();

var dic = new Dictionary<string, string>();

dic["tableA"] = string.Format("SELECT * FROM `tableA` where last_update_time>'{0:yyyy-MM-dd HH:mm:ss}';", LastBackuptime);
dic["tableB"] = string.Format("SELECT * FROM `tableB` where last_update_time>'{0:yyyy-MM-dd HH:mm:ss}';", LastBackuptime);
dic["tableC"] = string.Format("SELECT * FROM `tableC` where last_update_time>'{0:yyyy-MM-dd HH:mm:ss}';", LastBackuptime);
dic["tableD"] = string.Format("SELECT * FROM `tableD` where last_update_time>'{0:yyyy-MM-dd HH:mm:ss}';", LastBackuptime);

using (MySqlConnection conn = new MySqlConnection(connstr))
{
    using (MySqlCommand cmd = new MySqlCommand())
    {
        using (MySqlBackup mb = new MySqlBackup(cmd))
        {
            cmd.Connection = conn;
            conn.Open();

            mb.ExportInfo.WrapWithinTransaction = true;
            mb.ExportInfo.RowsExportMode = RowsDataExportMode.Replace;
            mb.ExportInfo.TablesToBeExportedDic = dic;
            mb.ExportToFile("C:\\backup.sql");

            conn.Close();
        }
    }
}

DateTime newBackupTime = DateTime.UtcNow;

SaveNewBackupTime(newBackupTime);

Always use "DateTime.UtcNow" for recording the execution time, especially if both of your Databases are in different timezone.

mb.ExportInfo.RowsExportMode = RowsDataExportMode.Replace;

In stead of traditional "INSERT INTO", the data will be using "REPLACE INTO".
This works same as the "INSERT", instead, if the primary existed, the row will be deleted then "INSERT" with new data. This will not work on tables with no primary key.

Hey thanks for the reply, I have forked and been messing around with trying something similar to this making changes to reflect this https://stackoverflow.com/questions/16974120/mysql-import-only-missing-rows I have been looking at making changes to MySqlBackup.cs
Line: 391 Export_WriteLine(string.Format("DROP TABLE IF EXISTS {0};", tableName));
&
Line: 690  if (rowsExportMode == RowsDataExportMode.Insert)                 sb.Append("INSERT INTO");
else if (rowsExportMode == RowsDataExportMode.InsertIgnore)
sb.Append("INSERT IGNORE INTO ");             else if (rowsExportMode == RowsDataExportMode.Replace)                 sb.Append("REPLACE INTO ");`
Basically I think I'm on the right track, I'd like to get your thoughts, thanks!

What you mean is you don't want the "DROP TABLE" and "CREATE TABLE" statement in the dump file, and you wish to use the statement "INSERT IGNORE". Is it?

That’s correct, I think that will give me the indebted behaviour right?
Get Outlook for iOShttps://aka.ms/o0ukef

From: Adrian Voo<EMAIL_ADDRESS>Sent: Thursday, May 2, 2019 12:16 pm
To: MySqlBackupNET/MySqlBackup.Net
Cc: Madison Courto; Author
Subject: Re: [MySqlBackupNET/MySqlBackup.Net] Only restore changes (#39)
What you mean is you don't want the "DROP TABLE" and "CREATE TABLE" statement in the dump file, and you wish to use the statement "INSERT IGNORE". Is it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MySqlBackupNET/MySqlBackup.Net/issues/39#issuecomment-488534822, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALTRXYNF5VLVTYDCZ42BSETPTJFF7ANCNFSM4HGQHF7A.

Then, you can apply 2 options to perform that:
mb.ExportInfo.ExportTableStructure = false;
mb.ExportInfo.RowsExportMode = RowsDataExportMode.InsertIgnore;

This line:
mb.ExportInfo.ExportTableStructure = false;
will skip and ignore "DROP TABLE" and "CREATE TABLE".
This line:
mb.ExportInfo.RowsExportMode = RowsDataExportMode.InsertIgnore;
In stead of the ordinary "INSERT INTO", now the dump will use "INSERT IGNORE"

Awesome thanks a lot mate!
Get Outlook for iOShttps://aka.ms/o0ukef

From: Adrian Voo<EMAIL_ADDRESS>Sent: Thursday, May 2, 2019 12:31 pm
To: MySqlBackupNET/MySqlBackup.Net
Cc: Madison Courto; Author
Subject: Re: [MySqlBackupNET/MySqlBackup.Net] Only restore changes (#39)
Then, you can apply 2 options to perform that:
mb.ExportInfo.ExportTableStructure = false;
mb.ExportInfo.RowsExportMode = RowsDataExportMode.InsertIgnore;
This line:
mb.ExportInfo.ExportTableStructure = false;
will skip and ignore "DROP TABLE" and "CREATE TABLE".
This line:
mb.ExportInfo.RowsExportMode = RowsDataExportMode.InsertIgnore;
In stead of the ordinary "INSERT INTO", now the dump will use "INSERT IGNORE"
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MySqlBackupNET/MySqlBackup.Net/issues/39#issuecomment-488536453, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALTRXYI3V33BLN5AWRBXU4TPTJG7TANCNFSM4HGQHF7A.

You're welcome.


What does 'primary wallet' mean?

...i know what it means, but our users will have no idea. It needs a (i) helper text, or description in addition to the field name.

Can it show ENS rather than the full address?


How do you handle mounts coming and going?

Hey, I authored a similiar project https://github.com/keegancsmith/kubernetes-disk-exporter
I have one major issue, as mounts come and go the rootfs mount doesn't see the new mounts. Does your implementation have the same issue? I see you set privilege: true, which my k8s daemonset is missing.

While I have not tested the specific case of

Exporter starts on
Pod with new PVC/PD scheduled on $node
Ensure new PVC/PD present in exporter metrics

I have not found any such issue.

Regarding privileged: true, I can't remember exactly the issue. I think it was that I could not view, or could not access the mounts of other users if I did not have the equivalent of root privileges. I tried with capabilities, but could not find any that would allow me to access the mounts. I scribbled something down in https://github.com/joar/k8s-pv-disk-usage-exporter/blob/master/example/daemonset.yaml#L18-L21

@joar thanks for the response. As reference this is the issue I have for this https://github.com/keegancsmith/kubernetes-disk-exporter/issues/1
To see if you are affected try getting a pod with a pv disk to move nodes (ie delete a pod attached to a replicaset).
I will try setting privileged true on a cluster at work to see if it resolves it.

@keegancsmith For each metric scrape, I list all available mountpoints:
https://github.com/joar/k8s-pv-disk-usage-exporter/blob/134c9eb4e9b1e348b1cd90b6795963ab7ddb08be/disk_usage_exporter/collect/partitions.py#L24-L33
using psutil.disk_partitions() (linux version)
I then filter
https://github.com/joar/k8s-pv-disk-usage-exporter/blob/134c9eb4e9b1e348b1cd90b6795963ab7ddb08be/disk_usage_exporter/collect/__init__.py#L187-L188
the mountpoints
https://github.com/joar/k8s-pv-disk-usage-exporter/blob/134c9eb4e9b1e348b1cd90b6795963ab7ddb08be/disk_usage_exporter/collect/partitions.py#L36-L41
to only include paths that look like
https://github.com/joar/k8s-pv-disk-usage-exporter/blob/134c9eb4e9b1e348b1cd90b6795963ab7ddb08be/disk_usage_exporter/collect/partitions.py#L44-L63
I also extract the PV name from the mountpoint
https://github.com/joar/k8s-pv-disk-usage-exporter/blob/134c9eb4e9b1e348b1cd90b6795963ab7ddb08be/disk_usage_exporter/collect/partitions.py#L36-L41
Finally to get the FS usage for each mountpoint, I run psutil.disk_usage()
https://github.com/joar/k8s-pv-disk-usage-exporter/blob/e6ec2eeca9bf944d9b6e517d863440a971ce4c2c/disk_usage_exporter/collect/__init__.py#L27-L32
I also query Kubernetes for the PV and PVC's name and labels in order to attach them to the mountpoint FS metrics.


TPM 1.2 vs 2.0

This page has missing detail around TPM versions. Autopilot requires TPM 2.0 and UEFI firmware for white glove and self-deploying scenarios. TPM 2.0 is also needed to automatically encrypt the local drive when joining a device to Azure Active Directory. If those aren't required then TPM 1.2 can be used (which is the only option on a lot of existing hardware still under support).

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

ID: 1aafc065-6ee6-e3cf-31ba-ab3dd277fb23
Version Independent ID: 35d3102e-4ff7-9408-8dbd-513b7dfff5c0
Content: Windows Autopilot requirements - Windows Deployment
Content Source: windows/deployment/windows-autopilot/windows-autopilot-requirements.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

Hello, @thommck
The issue is under investigation and you will be notified with any changes on the way.
Thank you for providing feedback.

The missing link has been included via the attached PR. Thank you for reaching out for feedback.


test: create dashboard test cases

Description
Create test cases to coverage Dashboard feature
How Has This Been Tested?
It was tested locally using SparWebApp on test environment
Checklist

[ x ] I have read the CONTRIBUTING doc
[ x ] I have performed a self-review of my own code
[ x ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have already been accepted and merged

outdated


Wrapper for 3d touch plugin

Hello,
This is a wrapper for https://github.com/EddyVerbruggen/cordova-plugin-3dtouch
I'm using it in https://itunes.apple.com/app/id1059812566 for 3d touch quick actions.
I wrote about how to use it here: https://github.com/nickplesha/ng-cordova-3dtouch
Let me know if it fits and should I change anything.

Can you include some docs for this for the gh-pages branch?

Sure! I created the PR for the docs here: https://github.com/driftyco/ng-cordova/pull/1225


Update this content

Is this page still accurate or is 18.04 actually not supported even though there are multiple resources that claim otherwise?

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

ID: 52db1d76-fb68-c44f-1162-3e30e8c2b793
Version Independent ID: 696aa3ee-bd95-2acf-9021-ef4b67663f1a
Content: Supported Ubuntu virtual machines on Hyper-V
Content Source: WindowsServerDocs/virtualization/hyper-v/Supported-Ubuntu-virtual-machines-on-Hyper-V.md
Service: na
Product: windows-server-threshold
GitHub Login: @shirgall
Microsoft Alias: kathydav

@shirgall should be able to help here

Hi @larsiwer ,
It seems that @shirgall has not commited since July 18, 2018. Also in a duplicate issue there was an attempt to ping him on May 4th but no answer since. Is there anyone else who can solve the problem?

I haven't had the time to update the Windows Server pages in a while, but I am working with the docs team to queue up some help. 18.04 is supported. So is 18.10. The recommended kernel to use is "linux-azure".

Hi @shirgall,
Thank you very much for the update!
Updating this docs page is very important as it's Microsoft Support policy to decline support cases if a Microsoft Docs page lists your configuration as unsupported. Therefore, Ubuntu 18.04 cannot be used in production until this page is updated.

We now have 19.04 (disco dingo), which seems to be as compatible as earlier versions...
Is the azure kernel still the recommended one on hyper-v server 2016?
What are the requirements to RDP into an Ubuntu running on hyper-v server 2016?

I need info as well, I'm using server 18.04 in production with the Linux-azure kernel. Only problem is that it is eating all of my dynamic memory. If I allocate 10GB of max memory. the disk cache chews it all up, even if the required ram is only 2GB. Need some help here. There is no way the confine the disk cache to 80% by my knowledge.

#please-close


helm: Add service account annotations for all charts

In order to use AWS EKS IAM roles for service account, we must annotate
service account with IAM role. More details can be found here
https://docs.aws.amazon.com/eks/latest/userguide/iam-roles-for-service-accounts-technical-overview.html
Fixes: #11303
Support service account annotations for helm charts

Coverage increased (+0.004%) to 44.432% when pulling e96e0d04536e2259c26e51679e973e9e49e6be59 on sayboras:feature/sa-annotations into 6fcf0cdd667945e59c764242a00e923c8cf3bc77 on cilium:master.

test-me-please

test-me-please

test-me-please


More package upgrades

Through my package upgrade work, I found that the K8s API package was actually blocking upgrade paths. Replacing it let more upgrades through.
Followup to:

#106

Re-running CI, but I created an issue for that flaky templates test: https://issues.redhat.com/browse/ACM-5349


[Relax][Pass] Skip data type node for CSE pass

The problem is seen when an arg of relax op is dtype

Thanks, this is probably another case that doesn't make sense to fold. Could you please also update the comment in that function?

Thanks @slyubomirsky! I have updated the comment.


CMake+Ninja detects GCC as the default compiler on Windows

Description
CMake+Ninja detects GCC as the default compiler on Windows, causing builds that are meant to use MSVC break. See the result of this example workflow run, where I try to build a project with CMake+Ninja+vcpkg+pwsh:
https://github.com/FranciscoPombal/qBittorrent/runs/1136169917?check_suite_focus=true
This does not happen if I force CMake to use a VS generator, but I'd rather use Ninja.
Area for Triage:  C/C++
Question, Bug, or Feature?:  Bug
Virtual environments affected

[ ] macOS 10.15
[ ] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[ ] Ubuntu 20.04 LTS
[x] Windows Server 2016 R2 - maybe? I assume it also happens here.
[x] Windows Server 2019

Expected behavior
Build tools such as Ninja are able to detect the latest cl.exe as the default compiler on Windows.
Actual behavior
GCC is detected and used instead by default.
Repro steps
Here is a minimal example of a step definition for reproducing the problem.
- name: build project
  shell: pwsh
  run: |
    cmake -B build -G "Ninja" -DCMAKE_BUILD_TYPE=RelWithDebInfo -DCMAKE_TOOLCHAIN_FILE=${{ env.VCPKG_DEST_WIN }}/scripts/buildsystems/vcpkg.cmake -DVCPKG_TARGET_TRIPLET=x64-windows-static
    cmake --build build

See https://github.com/FranciscoPombal/qBittorrent/runs/1136169917?check_suite_focus=true as a real-world example.
Sample output excerpt:
-- The CXX compiler identification is GNU 8.1.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/ProgramData/chocolatey/bin/c++.exe - skipped
-- Detecting CXX compile features
...

Extra notes
I am aware of the many possible workarounds for this, including dedicated actions for setting up MSVC environment, vswhere, etc. But this is a simple case where I'm only really interested in the latest cl.exe, and all VS components that come installed are sufficient for my use case. Why can't the latest cl be detected and used by default? Maybe this is a bug with the vcpkg toolchain file or something though.

Hello @FranciscoPombal,
Thank you for contacting us.
Did you try simply specify CMAKE_C_COMPILER and CMAKE_CXX_COMPILER variables like in an example below?
cmake -B build -G "Ninja" -DCMAKE_BUILD_TYPE=RelWithDebInfo -DCMAKE_TOOLCHAIN_FILE=${{ env.VCPKG_DEST_WIN }}/scripts/buildsystems/vcpkg.cmake -DVCPKG_TARGET_TRIPLET=x64-windows-static -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++

@Darleev
Hi, I did try passing -DCMAKE_CXX_COMPILER=cl, which did not work (I want cl in this case and would expect it to be the default, not clang).

@FranciscoPombal Currently, cl compiler is not in the path by default on Windows agents. In order to add it to agent I have done the following steps:
    - name: Setup MSBuild.exe
      uses: warrenbuckley/Setup-MSBuild@v1

    - name: build qBittorrent
      shell: cmd
      run: |
        cmake -B build -G "Ninja" ^
          -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DCMAKE_BUILD_TYPE=RelWithDebInfo ^
          -DCMAKE_TOOLCHAIN_FILE=${{ env.VCPKG_DEST_WIN }}\scripts\buildsystems\vcpkg.cmake ^
          -DVCPKG_TARGET_TRIPLET=x64-windows-static-release ^
          -DVERBOSE_CONFIGURE=ON ^
          -DCMAKE_CXX_COMPILER=cl ^
          -DCMAKE_C_COMPILER=cl ^
          --graphviz=build\target_graph.dot
        cmake --build build

You can see my example in the forked repo.. As far as I see in Github Action log here, cl compiler was successfully set in runtime:
-- The C compiler identification is MSVC 19.27.29111.0
-- The CXX compiler identification is MSVC 19.27.29111.0

Could you please verify on your side and provide us a result?
We are looking forward to your reply.

@Darleev
I ended up going with a similar solution:
    # NOTE: this is necessary to correctly find and use cl.exe
    - name: setup devcmd
      uses<EMAIL_ADDRESS>
    - name: build qBittorrent
      shell: cmd
      run: |
        cmake -B build -G "Ninja" ^
          -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DCMAKE_BUILD_TYPE=RelWithDebInfo ^
          -DCMAKE_TOOLCHAIN_FILE=${{ env.VCPKG_DEST_WIN }}\scripts\buildsystems\vcpkg.cmake ^
          -DVCPKG_TARGET_TRIPLET=x64-windows-static-release ^
          -DVERBOSE_CONFIGURE=ON ^
          --graphviz=build\target_graph.dot
        cmake --build build

https://github.com/FranciscoPombal/qBittorrent/runs/1143742011
-- The CXX compiler identification is MSVC 19.27.29111.0
...

Currently, cl compiler is not in the path by default on Windows agents.

Yes, but is this intentional? Is using an action like ilammy/msvc-dev-cmd or microsoft/setup-msbuild the "standard" way of getting things to build with MSVC in GitHub actions? I noticed the former can only be used with cmd, which is not that flexible. Can the former at least be used with pwsh?

I think it is expected behavior.
There is an additional devCmd (Visual Studio Command Prompt) on Windows machines. It is installed with Visual Studio.
If you run cmake from devCmd shell, it will initialize Visual Studio environments: use MSVC compiler and other Visual Studio tools.
If you run cmake from default shell, VS tools won't be available (if you don't have clang, it will fail to find any compiler).
By default, default shell is used in GitHub Actions.
Tasks like microsoft/setup-msbuild and ilammy/msvc-dev-cmd just enable Visual Studio Command Prompt.
You can find more details in microsoft/setup-msbuild task description

Yes, but wouldn't it make sense to have the environment initialized by default (with the possibility of overriding, of course), with the latest version of the tools? It would be more convenient for the simpler use cases. That being said, I guess including a single action and 2 additional lines is not that much more trouble.

I am not sure that it makes sense. Visual Studio Command Prompt is additional shell that is installed by Visual Studio and intended for specific goals - building applications via VS. it is just one of possible scenarios of using image. We should not set it by default. There is an official Microsoft task microsoft/setup-msbuild to enable this environment that takes about 1 second to setup this environment.
Visual Studio Command Prompt sets additional environment variables, modifies PATH. If we apply it for default bash, a lot of customers' builds will stop to work and will cause a lot of issues with building in any other system outside MSVC like gcc, g++, clang and etc. Customers will have to unset all these variables to build outside MSVC.
We prefer to keep default environment clean and consistent with default Windows shell (outside of Visual Studio Command Prompt).

Well, makes sense, and I can definitely live with that.

I just want to state that I agree with @FranciscoPombal, MSVC should be detected by default as C/CXX compiler on Windows. In a complex build matrix it adds yet another variable that needs to be initialized or another conditional step that needs to be executed.

I just want to state that I agree with @FranciscoPombal, MSVC should be detected by default as C/CXX compiler on Windows. In a complex build matrix it adds yet another variable that needs to be initialized or another conditional step that needs to be executed.

Had the same problem just now.The solution proposed by @FranciscoPombal no longer works. It results in an error:
Error: The set-envcommand is disabled. Please upgrade to using Environment Files or opt into unsecure command execution by setting theACTIONS_ALLOW_UNSECURE_COMMANDSenvironment variable totrue. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/
g++ compiler it is then...

I want to use cmake+gcc to build a python binary module on Windows runner, but no matter what I did, cmake keeps using msvc cl.exe compiler.
here are the things I've tried and failed:

setting CMAKE_GENERATOR="MinGW Makefiles", cmake complains "CMake Error: CMAKE_GENERATOR was set but the specified generator doesn't exist. Using CMake default.", https://github.com/fangq/mcxcl/actions/runs/5694484870/job/15435761075
setting CMAKE_ARGS=-GNinja, the variable CMAKE_ARGS is processed in setup.py and pass to cmake command line. however, I got an error saying Ninja and platform x86 can not be simultaneously specified, https://github.com/fangq/mcxcl/actions/runs/5696240373/job/15440973441
setting CMAKE_ARGS=-GNinja and disable the -A x86 flag inside setup.py, then cmake complains that "CMAKE_MAKE_PROGRAM is not set.", https://github.com/fangq/mcxcl/actions/runs/5696475315/job/15441616218
setting CMAKE_ARGS=-G"MinGW Makefiles", then cmake complains "MinGW Makefiles" (note the double-quotes) is not supported, despite that it is listed right under, https://github.com/fangq/mcxcl/actions/runs/5696530979/job/15441772404
setting CMAKE_ARGS=-G "MinGW Makefiles" , cmake fails silently, without giving a reason, https://github.com/fangq/mcxcl/actions/runs/5696803957/job/15442521719
forcing setup.py to use gcc by using python setup.py bdist_wheel --compiler=mingw32, then setup.py complains that --compiler flag is not supported, except that it should, https://github.com/fangq/mcxcl/actions/runs/5696450596/job/15441539524

I am running out of ideals.
does anyone know how can I ask cmake to use mingw64 gcc to compile on Windows runners?

never mind, figured it out. either ninja or mingw generators have been pre-installed on github windows runner, despite they are being printed in the cmake -G output list. it was really confusing ...
once I installed mingw-w64-x86_64-ninja in msys2 and add -GNinja to cmake, it finds gcc and everything works fine.
see my report here: https://github.com/actions/runner-images/discussions/7998#discussioncomment-6584504

There is an official Microsoft task microsoft/setup-msbuild to enable this environment that takes about 1 second to setup this environment.

An Important note at the beginning of microsoft/setup-msbuild's readme states that:

Please note this tool does NOT add other Visual Studio tools (like cl) to PATH

So what should we use instead? As @EwoutH mentionned, it is notably annoying when setting up a build config matrix.

Ano ito
On Mon, Jun 3, 2024, 1:52 AM Elie Michel @.***> wrote:

There is an official Microsoft task microsoft/setup-msbuild to enable
this environment that takes about 1 second to setup this environment.
An Important note at the beginning of microsoft/setup-msbuild's readme
states that:
Please note this tool does NOT add other Visual Studio tools (like cl)
to PATH
So what should we use instead? As @EwoutH https://github.com/EwoutH
mentionned, it is notably annoying when setting up a build config matrix.
—
Reply to this email directly, view it on GitHub
https://github.com/actions/runner-images/issues/1621#issuecomment-2143966744,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BIXWL6UPIT7NCWU4F5FPULDZFNLUBAVCNFSM4RTDLVP2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMJUGM4TMNRXGQ2A
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


[Dragon 2] Remove levy references

Remove levy references from code (e.g. MosaicProperties) since it will be dropped from the catapult-server code https://github.com/nemtech/catapult-server/issues/37

Although it is planned for dragon II, it should be fine to drop levies support right now since it's not working right now.


Emit metrics after segmenting failure

Currently if Mist fails early enough (i.e we catch the failure in one of the triggers), we don't write any metric to Prometheus.

I looked at an example of this in prod and saw that after the "segmenting failed" error we then get a log line "Finished job and deleted from job cache" from here where we also emit the vod pipeline metrics so doesn't that mean we're covered? Or are we saying we want an extra metric to say specifically "segmenting failure", that could be a tag on the existing metrics i guess.
Vod example from loki

@mjh1 Great! Sounds like this was maybe fixed during the pipeline work then - I'll close


Attempt to build Caddy with Coraza V2 fails due to dead URL

I attempted to build Caddy 2.8.4 with Coraza in a Dockerfile using the command from the docs:
xcaddy build --with github.com/corazawaf/coraza-caddy/v2
The process fails due to a 404 received for that URL. I assume this is due to the V3 upgrade that appears to be currently happening.

I gave a quick go to the command outside of docker and looks okay:
▶ xcaddy build --with github.com/corazawaf/coraza-caddy/v2
2024/06/21 18:01:43 [INFO] Temporary folder: /tmp/buildenv_2024-06-21-1801.1563029557
2024/06/21 18:01:43 [INFO] Writing main module: /tmp/buildenv_2024-06-21-1801.1563029557/main.go
package main

import (
	caddycmd "github.com/caddyserver/caddy/v2/cmd"

	// plug in Caddy modules here
	_ "github.com/caddyserver/caddy/v2/modules/standard"
	_ "github.com/corazawaf/coraza-caddy/v2"
)

func main() {
	caddycmd.Main()
}
2024/06/21 18:01:43 [INFO] Initializing Go module
2024/06/21 18:01:43 [INFO] exec (timeout=10s): /opt/homebrew/bin/go mod init caddy
go: creating new go.mod: module caddy
go: to add module requirements and sums:
	go mod tidy
2024/06/21 18:01:43 [INFO] Pinning versions
2024/06/21 18:01:43 [INFO] exec (timeout=0s): /opt/homebrew/bin/go get -d -v github.com/caddyserver/caddy/v2
go: added github.com/beorn7/perks v1.0.1
go: added github.com/caddyserver/caddy/v2 v2.8.4
go: added github.com/caddyserver/certmagic v0.21.3
go: added github.com/caddyserver/zerossl v0.1.3
go: added github.com/cespare/xxhash/v2 v2.2.0
go: added github.com/go-task/slim-sprig v0.0.0-20230315185526-52ccab3ef572
go: added github.com/google/pprof v0.0.0-20231212022811-ec68065c825e
go: added github.com/google/uuid v1.6.0
go: added github.com/klauspost/cpuid/v2 v2.2.7
go: added github.com/libdns/libdns v0.2.2
go: added github.com/mholt/acmez/v2 v2.0.1
go: added github.com/miekg/dns v1.1.59
go: added github.com/onsi/ginkgo/v2 v2.13.2
go: added github.com/prometheus/client_golang v1.19.1
go: added github.com/prometheus/client_model v0.5.0
go: added github.com/prometheus/common v0.48.0
go: added github.com/prometheus/procfs v0.12.0
go: added github.com/quic-go/qpack v0.4.0
go: added github.com/quic-go/quic-go v0.44.0
go: added github.com/zeebo/blake3 v0.2.3
go: added go.uber.org/mock v0.4.0
go: added go.uber.org/multierr v1.11.0
go: added go.uber.org/zap v1.27.0
go: added go.uber.org/zap/exp v0.2.0
go: added golang.org/x/crypto v0.23.0
go: added golang.org/x/exp v0.0.0-20240506185415-9bf2ced13842
go: added golang.org/x/mod v0.17.0
go: added golang.org/x/net v0.25.0
go: added golang.org/x/sync v0.7.0
go: added golang.org/x/sys v0.20.0
go: added golang.org/x/term v0.20.0
go: added golang.org/x/text v0.15.0
go: added golang.org/x/time v0.5.0
go: added golang.org/x/tools v0.21.0
go: added google.golang.org/protobuf v1.34.1
2024/06/21 18:01:44 [INFO] exec (timeout=0s): /opt/homebrew/bin/go get -d -v github.com/corazawaf/coraza-caddy/v2 github.com/caddyserver/caddy/v2
go: downloading github.com/corazawaf/coraza-caddy/v2 v2.0.0-rc.3
go: downloading github.com/corazawaf/coraza/v3 v3.0.3
go: downloading github.com/jcchavezs/mergefs v0.0.0-20230405222254-20429875efdd
go: downloading google.golang.org/genproto v0.0.0-20230706204954-ccb25ca9f130
go: downloading github.com/petar-dambovaliev/aho-corasick v0.0.0-20211021192214-5ab2d9280aa9
go: added github.com/corazawaf/coraza-caddy/v2 v2.0.0-rc.3
go: added github.com/corazawaf/coraza-coreruleset v0.0.0-20230723190514-7bdcbcff3d5a
go: added github.com/corazawaf/coraza/v3 v3.0.3
go: added github.com/corazawaf/libinjection-go v0.1.2
go: added github.com/jcchavezs/mergefs v0.0.0-20230405222254-20429875efdd
go: added github.com/magefile/mage v1.15.0
go: added github.com/petar-dambovaliev/aho-corasick v0.0.0-20211021192214-5ab2d9280aa9
go: added github.com/tidwall/gjson v1.14.4
go: added github.com/tidwall/match v1.1.1
go: added github.com/tidwall/pretty v1.2.1
go: added rsc.io/binaryregexp v0.2.0
2024/06/21 18:01:47 [INFO] exec (timeout=0s): /opt/homebrew/bin/go get -d -v
go: downloading github.com/go-chi/chi v4.1.2+incompatible
2024/06/21 18:01:48 [INFO] Build environment ready
2024/06/21 18:01:48 [INFO] Building Caddy
2024/06/21 18:01:48 [INFO] exec (timeout=0s): /opt/homebrew/bin/go mod tidy
2024/06/21 18:01:48 [INFO] exec (timeout=0s): /opt/homebrew/bin/go build -o /tmp/caddy -ldflags -w -s -trimpath
2024/06/21 18:01:53 [INFO] Build complete: ./caddy
2024/06/21 18:01:53 [INFO] Cleaning up temporary folder: /tmp/buildenv_2024-06-21-1801.1563029557

I assume this is due to the V3 upgrade that appears to be currently happening.
It might be a bit misleading but v2 is the version of coraza-caddy, that is the connector. It has as a dependency Coraza v3. We have to update the dependency to latest Coraza v3.2 version and perform checks with the latest Caddy version but as far as I can tell, the xcaddy build command is expected to stick to the one that you tried

Derp. It was a DNS issue from my side. I should have blamed DNS first like always. Closing this and thanks for the quick response.


APEX-43: assigning classToStringCodec to TUPLE_CLASS attr in PortContext

This is a bug fix.

test this please

test this please


Use exportToCookie but opt-out of serialization

Hi there! I'm just getting started with pocketbase in a svelte-kit project. As you might know, svelte-kit is pre-1.0.0 and is subject to breaking changes, with the cookies interface being one of them. The pocketbase js-sdk readme shows how to integrate with svelte-kit's SSR: create the client, initialize the auth store with the request cookie, and at the end of the request update the cookie with the state of the auth store.
If you were to implement the example from the readme...
// src/hooks.js
import PocketBase from 'pocketbase';

export async function handle({ event, resolve }) {
    event.locals.pocketbase = new PocketBase("http://<IP_ADDRESS>:8090");

    // load the store data from the request cookie string
    event.locals.pocketbase.authStore.loadFromCookie(event.request.headers.get('cookie') || '');

    const response = await resolve(event);

    // send back the default 'pb_auth' cookie to the client with the latest store state
    response.headers.set('set-cookie', event.locals.pocketbase.authStore.exportToCookie());

    return response;
}

You would encounter the error TypeError: Failed to execute 'set' on 'Headers': Headers are immutable. The workaround is to set the cookie using the cookies interface on the Handle event. The problem is that the cookies.set function has the signature (name: string, value: string, opts: CookieSerializeOptions) => void, while the auth store's exportToCookie function serializes the cookie and returns a string. This leads to a workaround where we need to deserialize the cookie from exportToCookie before using cookies.set:
export async function handle({ event, resolve }) {
  // ...truncated

  const response = await resolve(event);

  // wasteful: exportToCookie serializes the cookie and then we must deserialize the cookie
  const authCookie = cookie.parse(client.authStore.exportToCookie());
  cookies.set('pb_auth', authCookie.pb_auth, {
    path: authCookie.Path,
    expires: new Date(authCookie.Expires),
    sameSite: 'strict',
  });

With that established, I guess there are two things here: the code example in the svelte-kit section of this repo's readme is broken and there's a workaround required to integrate with Svelte-Kit. For the second problem, where pocketbase serializes the cookie and in svelte-kit it's necessary to deserialize the cookie before setting it, I'm not sure where the fix is best suited. Should svelte-kit accept serialized cookie headers and handle them internally? Or should pocketbase expose an escape hatch to return the cookie before serializing it? In svelte-kit, the fix would add complexity to the internals for more work, while the pocketbase fix would add complexity to the API for less work. I personally am leaning towards the pocketbase escape hatch. Maybe a pocketbase.authStore.exportToCookieRaw()?

@sea-grass I'm not able to reproduce the "TypeError: Failed to execute 'set' on 'Headers': Headers are immutable" error.
In what version this was introduced? I've tested with the most recent one - sveltejs/kit 1.0.0-next.543.
The only thing I had to change in the readme was hooks.js to hooks.server.js and adding async to the +server.js action but that is already updated in the rc branch - https://github.com/pocketbase/js-sdk/tree/rc

Thanks @ganigeorgiev. Let me see if I can reproduce in the latest kit. In my repo, I'm also using trpc-sveltekit so it's possible that it's the one marking response headers as immutable (in which case I'll have to determine if I'll ever need to update the auth store after trpc handles a request or if I should only set the cookie after kit handles a request). I'll dig a bit more and may link to a reproduction repo in a bit.

I'm closing the issue since I wasn't able to reproduce it.
In any case, I don't think it will be very useful to export the cookie without the key because it will be confusing to have only the cookie value + its attributes as a string. I'm not actually sure how in your example this even works, because the exported string is with cookie attributes while at the same time you are setting a custom ones (aka. path, sameSite, expires, etc.).
If you need only the cookie data value, you can just use:
JSON.stringify({ token: client.authStore.token, model: client.authStore.model })


No example

There is no example how to use this library by importing plugin.
Example from main readme and from kpy-sample (which uses local library) not working

For now I got error from setup.py:

usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
or: setup.py --help [cmd1 cmd2 ...]
or: setup.py --help-commands
or: setup.py cmd --help
error: no commands supplied

I will add PR for how-to when it will work)

After assembly there is no generated folder
Here is my build.gradle.kts:

build.gradle.kts
import com.martmists.kpy.plugin.PythonVersion
import org.jetbrains.kotlin.gradle.ExperimentalKotlinGradlePluginApi
import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinNativeTarget
import java.util.*

plugins {
  alias(libs.plugins.kotlin.multiplatform)
  alias(libs.plugins.kpy)
}

val targetName = "native"

kotlin {
  jvm()
  val hostOs = System.getProperty("os.name")
  val isMingwX64 = hostOs.startsWith("Windows")
  val isArm64 = System.getProperty("os.arch") == "aarch64"
  val nativeTarget = when {
      hostOs == "Mac OS X" && !isArm64 -> macosX64(targetName)
      hostOs == "Linux" && !isArm64 -> linuxX64(targetName)
      hostOs == "Mac OS X" && isArm64 -> macosArm64(targetName)
      hostOs == "Linux" && isArm64 -> linuxArm64(targetName)
      isMingwX64 -> mingwX64(targetName)
      else -> throw GradleException("Host OS is not supported in Kotlin/Native.")
  }

  sourceSets {
      val commonMain by getting {
          dependencies {
          }
      }
      val commonTest by getting {
          dependencies {
          }
      }
  }

  nativeTarget.apply {
      val main by compilations.getting {

      }

      binaries {
          staticLib {
              @OptIn(ExperimentalKotlinGradlePluginApi::class)
              compilerOptions {
                  optIn = listOf(
                      "kotlinx.cinterop.ExperimentalForeignApi",
                  )
              }
          }
      }
  }
}

kpy {
  target = targetName
  generateStubs = true
  pyVersion = PythonVersion.Py310
  moduleName = rootProject.name
  version = rootProject.version

}

For now I got error from setup.py:

Are you installing it with python setup.py install or pip install .? The latter might not work because of the somewhat recently introduced build isolation.

When I run pip install . I got error:

Error
{'packages': [], 'package_dir': {'': 'D:\\WorkSL\x07ngrydata\\library\x08uild/generated/ksp/native/nativeMain/resources'}}
    running bdist_wheel
    error: error in 'egg_base' option: 'D:\WorkSLngrydata\libraryuild/generated/ksp/native/nativeMain/resources' does not exist or is not a directory
    [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel

And there no build/generated/ksp... folder. What I should run to generate this?

run pip install . --no-build-isolation should do the trick I believe

run pip install . --no-build-isolation should do the trick I believe
Same error. There no generated folder.
Perhaps I have the wrong build script, what do you think?


enormous amount of FastThreadLocal instances

Expected behavior
FastThreadLocal instances aren't growing
Actual behavior
FastThreadLocal instances are growing - about 550K instances
Minimal yet complete reproducer code (or URL to code)
Heap dump https://cmoviepic.bj.bcebos.com/production%2Foom%2Fwalle_api_bj-B-01_heap.hprof.tar.gz
Netty version
4.1.39.Final
Can anybody help to realize why it's happening?

Actual behavior is "FastThreadLocal instances aren't growing"? In my app, the actual behavior is that,  FastThreadLocal is growing bigger and bigger.

@songjiesdnu
I fixed the description.

@mrniko
I read some source codes of redisson, and debug it for a while. Here is what I find about how  it's happening.

In class org.redisson.client.handler.ConnectionWatchdog，when method channelInactive(ChannelHandlerContext ctx) is invoked, bootstrap.connect(SocketAddress remoteAddress) is invoked.

Why not make the field state  static?

Fixed in Redisson lib

@mrniko can you link the commit ?

@normanmaurer
Sure. Here it is https://github.com/redisson/redisson/commit/3a47ed40c0e088937c839111ee062861bc0d21ec


Feature/interactive regression test plots

THIS PULL REQUEST IS READY TO MERGE
Feature or improvement description
This pull requests modifies the regression testing suite to replace the visualizations with interactive visualizations, better accentuate which cases have failed in the export summary, and to clean up a few spots in the code to make it more pythonic. This work was done under the guidance of @rafmudaf.
Impacted areas of the software
This only impacts the regression testing suite.

@RHammond2 The Bokeh JavaScript files don't load for me. This is what I see in the JS console:
[Error] Failed to load resource: the server responded with a status of 403 () (bokeh.min.js.map, line 0)
[Error] Failed to load resource: the server responded with a status of 403 () (bokeh-widgets.min.js.map, line 0)

Any idea why?

@rafmudaf I can't seem to find anything related to this issue.

I thought we discovered that the error above was only a problem when generating plots for cases that hadn't actually failed, but I ran this again after compiling in single precision so that most test cases fail and still same issue. So this is still pending.
On another note, since each plot is opened by clicking on a link (the id selector #), it can take a lot of clicks on the browser back button to get back to the main page. Adding a back to home button may be worthwhile.

So this issue isn't actually blocking any of the functionality. There was a bit of confusion as to what was expected for output because we were looking at a case where none of the tests failed causing there to be no plots that are output. Even when plots are output the "error" comes up.
Where would the back to home button lead to for this if we are only looking at a single case? Would the idea be to access any of the cases as opposed to going through each directory separately?


Missing schemaSetting in case of googlepubsub protocol

Describe the bug
When using googlepubsub as protocol using version 1.0.0 generated asynapi(3.0.0) docs does not add schemaSetting under bindings->googlepubsub, which is required for passing asynapi(3.0.0) validation.
Dependencies and versions used
spring-wolf v1.0.0 (CloudStream)
Code example
After some initial investigation I found out that class CloudStreamFunctionChannelsScanner in method buildMessageBinding always map EmptyMessageBinding which is wrong.
Stack trace and error logs
googlepubsub property must have required property schemaSettings

Thank you @SheheryarAamir for the feature request and contribution, it has been published now.
We highly appreciate if you spare some minutes to update the documentation as you have the knowledge of the google pubsub bindings, for example here: https://github.com/springwolf/springwolf.github.io/blob/master/docs/configuration/documenting-bindings.md


Force LF line endings for bin script.

npm install got-swag --global pulls down got-swag as a DOS formatted file. These Windows-style line endings trigger env: ‘node\r’: No such file or directory when Node tries to run the file in a Linux/POSIX shell, see https://github.com/npm/npm/issues/4607
I added an EditorConfig and .gitattributes to enforce LF line endings in the repo's working directory, no matter the platform.
The trivial change to got-swag forces an update when you pull.

I thought I had fixed this issue by commiting LFs in the first place. Many thanks 🍻 !

Now contained in v1.1.1 - thanks again!


Add offsetCount to fflib_QueryFactory

Getter / setter for offsetCount (replaces offset)
Deep clone set offset
Add offset clause to generated SOQL string

Related to issue #159

Thanks @JonnyPower for the submission! 👍


Evaluate dotnet watch targets with inprocess msbuild

https://github.com/dotnet/tye/pull/526/files#diff-e13301364f733f4592426aba2352d01cR57 currently calls dotnet msbuild to call a target. This can be improved by calling msbuild inprocess.

We aren't going to do this due to probably needing to move away from msbuild inprocess.


Fix variables in variable are not load correctly

see details in commits.

Can we disable squash here too, please? What's the use case?

Can we disable squash here too, please? What's the use case?

My commits are well organized in this PR. I hope so.
The Github cannot track changes when we perform force push. It's hard to look at changes closely. If someone is not to use force push, it's also good to see delta for each push, and then it can be squashed into one commit. It's hard to find a workable solution in Github to track changes like Azure Git repos. So, leaving the two options to users to choose. If people are not part of our team, we will merge code for them.


Process compass runtime ID

Description
Changes proposed in this pull request:

process compass runtime Id
KEB knows, when Provisioner registered the runtime in the Compass or not
the Provisioner response processing is not yet implemented because Provisioner part is not done.

Related issue(s)

The annotation is applied only in the provisioning phase!

/retest


List command is not working

kiex list command throw out an error
kiex elixirs

   elixir-1.3.3/Users/wiwattamongkhonchit/.kiex/bin/kiex: line 946: elixir-1.3.3: syntax error: invalid arithmetic operator (error token is ".3.3")

I'm not bash script expert, so I'm not sure what's going on here but looking from the error and the code it seems like
target_alias_list="${ealiases[$version]}"

array dereference is trying to evaluate $version expression which currently contain elixir-1.x.x and throw an error.

The is caused by #61 that introduces associative arrays declare -A ealiases=() which is a Bash 4 feature. If you use Bash 3.2 (that comes with macOS Sierra for example) the problem manifests.
So kiex master is unusable on vanilla mac because Apple is reluctant to upgrade Bash due to the licensing change in Bash 4. The workaround is to upgrade Bash manually http://apple.stackexchange.com/questions/193411/update-bash-to-version-4-0-on-osx
Sidenote: Perl5 is the most portable scripting platform.

Reverted changes until this can be written to support Bash 3.2.

@taylor gahhhhh, my bad. I already have the fix for Bash 3 here: https://github.com/greymouser/kiex/tree/unalias+delete
I'm not sure I know the correct git steps in this case; my branch doesn't have the rollback. Should I create a new branch from upstream master on my side, revert the reversion, and cherry-pick my change in? Advice appreciated.

@taylor A new pull request against master has been created with Bash 3 support.


Update text-startswith-callingcodes.json adding Mexico

Added Mexico

Q
A

Bug fix?
no - yes?

New sample?
no - yes?

Related issues?
fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
Please describe the changes in this PR. Sample description or details around bugs which are being fixed.
Guidance
You can delete this section when you are submitting the pull request.

Please update this PR information accordingly. We'll use this as part of our release notes in monthly communications.
Please target your PR to 'master' branch.

Thanks for your contribution! Sharing is caring.

Thank you!


Fix registration flow for Async Pretty Printer

Right now, pretty printers are instantiated and registered at the start up path. This PR moves these steps to onFirstPeerRegistered() for NetworkPeerManager's listener.

Can you split this out into 2 PRs one that solves #255 the other that fixes the registration flow.

This looks good to me, @rickbrew anything else you want to add?

Yeah I think we're good here. Squash 'n merge

yayayay!


InfiniteScroll Usage for one of my Projects

This is some awesome piece of code and I would like to use it in one of my projects, but however I face some issues when integrating it. What I want is the following:

User has a form where he can enter his search criteria. When he clicks submit, I make a REST API call to fetch the results. The results are transformed into my model class.

By default I only fetch 10 items on every REST API call. So for the first time the user loads these 10 items by clicking the Search button

From next time onwards, I should be able to just scroll down which will in turn load 10 more items from the REST API

I tried to use your scroll directive and I'm stuck with one issue which is in my Typescript which is as below:
import { Component, OnInit } from '@angular/core';

import { PowerPlantService, UserService } from '../shared';
import { User } from '../shared/models/user.model';
import { PowerPlant } from '../shared/models/powerplant.model';
import {PowerPlantSearchParams} from '../shared/models/powerplantsearchparams.model';

@Component({
  selector: 'app-home-page',
  templateUrl: './home.component.html'
})
export class HomeComponent implements OnInit {
  // Represents the PowerPlantTypes
  powerPlantTypes = ['RampUpType', 'OnOffType'];
  // Represents the status of a PowerPlant
  powerPlantStatuses = ['Active & Disabled', 'Active', 'Disabled'];
  // Represents the search form
  model: any = {};
  // represents the list of PowerPlant data
  powerPlants: PowerPlant[];
  scrollCallback;

  currentPage = 1;

  constructor(private powerPlantService: PowerPlantService) {
    // this.currentUser = JSON.parse(localStorage.getItem('currentUser'));
    // Set the initial values for the drop down fields in the UI
    this.resetForm();
  }

  ngOnInit() {}

  resetForm() {
    this.model.powerPlantOrg = '';
    this.model.powerPlantName = '';
    this.model.powerPlantType = '';
    this.model.powerPlantStatus = '';
  }

  searchPowerPlants() {
    const powerPlantSearchParams = new PowerPlantSearchParams(
      this.model.powerPlantType,
      this.model.powerPlantOrg,
      this.model.powerPlantName,
      this.model.powerPlantStatus);
/*
    this.powerPlantService.searchPowerPlants(powerPlantSearchParams).subscribe(result => {
      this.powerPlants = <PowerPlant[]> result;
    }); */
    this.scrollCallback = this.searchPowerPlants().bind(this);

    return this.powerPlantService.searchPowerPlants(powerPlantSearchParams, this.currentPage)
      .do(this.processData);
  }

  private processData = (newPowerPlants) => {
    this.currentPage++;
    this.powerPlants = this.powerPlants.concat(newPowerPlants.json());
  }

  allPowerPlants(onlyActive: boolean = false, page: number = 1): void {
    this.powerPlantService.allPowerPlants(onlyActive, page).subscribe(result => {
      this.powerPlants = <PowerPlant[]> result;
    });
  }
}

In the code above line number 50
this.scrollCallback = this.searchPowerPlants().bind(this);
does not compile!  It complains that property bind does not exist on type Observable
Any thoughts on how to get this going? Or Do you have any alternative approach that I could try?

Hi @joesan, Thanks for getting in touch and appreciation.
You are calling the function while binding, that's the issue.
it should be
this.searchPowerPlants.bind(this)
Let me know..

I figured how by working through the errors that I got and I exactly did what you have posted. You can see my example here:
https://github.com/joesan/plant-simulator-ui/blob/master/src/app/home/home.component.ts
I have one other question though. In my home.html screen, I have a Search button. When I load this page, I call the searchPowerPlants(), bind the method for scrolling. But later on when the user enters a search criteria, I would like to clear the existing results, present the new results and any subsequent scrolling should satisfy this new search criteria.
Have you got any ideas as to how I could do this?

@joesan  Sorry couldn't get back to you sooner, I was away.
Do you have a plunker/stackblitz available for this scenario, so I can test it out ?

Do you know if it is easy to import the whole GitHub project into plunker or stackblitz? If not, it is going to be harder for me to set up the whole project! Would it be possible for you to clone it and try it locally? You just need this project and it does not have any other dependency!


Make parse_declaration panic if given leftover tokens.

Export consume_declaration.
Add unit test.

Would it be possible to get a release with this?

Sure. Ping me in a week if I haven't made one.