scroll restoration overhaul

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Should fix double recording navigation issue(still checking on that) and removes the need for scroll event listener.
Related to #3835

Afraid I'm seeing some wildly inconsistent behaviour here:
https://user-images.githubusercontent.com/1162160/153728097-9f09ecb7-686e-4b61-9dc0-2e5061163d5b.mov

@Rich-Harris @benmccann we have a decision to make, we are currently decision-less about this thing:
so, the code that's currently being used is not firing the events/hooks, (beforeNavigate and that stuff) if the user is navigating to the hash on the same page, we're re-rendering the same page, and then telling the browser to deal with the navigation event itself(that's happening because we never call event.preventDefault() in that if check)
so the choices are:
a) remove that code (if) completely and things should work correctly, the beforeNavigate would fire, but I tried this and some tests fail
b) do this:
if (hash !== undefined && base === location.href.split('#')[0]) {
    event.preventDefault();
    this.save_scroll_state();
    const info = this.parse(url);
    if (info) {
        await this.renderer.update(info, [], false);
        location.hash = hash; // this will automatically pushState and set the state to null and then trigger hashchange event
        this.save_scroll_state(); // by calling this method, it will also replaceState with object instead of null, we could also use history.replaceState({}, '', url.href);
    }
    return;
}

or even simple(but without re-rendering):
if (hash !== undefined && base === location.href.split('#')[0]) {
    event.preventDefault();
    this.save_scroll_state();
    const info = this.parse(url);
    if (info) {
        location.hash = hash; // this will automatically pushState and set the state to null and then trigger hashchange event
        this.save_scroll_state(); // by calling this method, it will also replaceState with object instead of null, we could also use history.replaceState({}, '', url.href);
    }
    return;
}

1@Rich-Harris also scrolling test are wrong, at least about hash handling, they go success but they should fail, also I'll implement the fix for scrolling now(option B) and remove the listener tomorrow, because it's more complicated than I originally though, and It's 10pm where I live (London time + 1 :) ) so I'm tired now. I'll implement it tomorrow.

Can you update the PR title and description to reflect what this change does? It's hard to understand the context for it right now since I don't think it removes the need for the scroll event listener and the description is unclear as to whether it fixes the duplicate history entry

I sent a PR with a failing test that's commented out: https://github.com/sveltejs/kit/pull/3880. After it's merged you can rebase this PR and uncomment it

It makes no sense that these test are failling, because I'm calling store.set maybe there is some racing.

I'll now check why it failed, I have an idea

@benmccann I think the test is wrong because you are assigning a number to const, and then comparing it twice to different values.

@benmccann the tests are again being flaky after that latest in_view updates, but I just tested it locally and it passes.

@Rich-Harris @benmccann I tried without that setTimeout hack and setting the stores manually, but it gets either too complicated, or unreliable, so this the best option that definitely works, and passes the tests.
So I think you should merge it, and if there is a better option we'll do another PR.

I'm having an extremely difficult time keeping up with the state of this PR. 46 commits is entirely too many. Can you mark this as draft until it's in its final state? Is it ready for review now or is there more that you're changing still?

It's done now, those were mostly test retries.

I'm having an extremely difficult time keeping up with the state of this PR. 46 commits is entirely too many. Can you mark this as draft until it's in its final state? Is it ready for review now or is there more that you're changing still?

It's done now, those were mostly test retries.

Or I can close this PR and open a new with only the changes, it would be much clearer.

Ok. It's not a blocker to make all the tests pass. We know they're flaky and still commit PRs even if one test run fails. We should get them passing regularly, but I'd prefer to do one thing per PR and address the flakiness separate from fixing bugs

I did just notice one issue - though it's also present in master, so isn't a blocker. If you visit http://localhost:3000/docs/layouts#nested-layouts then press Ctrl+W followed by Ctrl+Shift+T it will reopen at the top of the page rather than the anchor

I did just notice one issue - though it's also present in master, so isn't a blocker. If you visit http://localhost:3000/docs/layouts#nested-layouts then press Ctrl+W followed by Ctrl+Shift+T it will reopen at the top of the page rather than the anchor

fixed that and ready to merge

We can't merge this until there's a test that fails without this PR — are you able to add one?

When I enable the test included in this PR on master, the app behaves as expected. The test fails because playwright reports the URL as incorrect — probably a race condition — but other than that things are fine.
By contrast, when I check out this PR the behaviour is totally broken.
I'm sorry, but I really don't understand what this PR is doing!

When I enable the test included in this PR on master, the app behaves as expected. The test fails because playwright reports the URL as incorrect — probably a race condition — but other than that things are fine.
By contrast, when I check out this PR the behaviour is totally broken.
I'm sorry, but I really don't understand what this PR is doing!

No worries, but how do you mean, the behaviour is totally broken
https://kit-svelte-dev-z8e747sj9-svelte.vercel.app/ latest commit, looks good to me, even the Ctrl+W and Ctrl+Shift+T (reopen closed tab issue @benmccann mentioned that was present isn't there anymore)

Ah. Turns out master behaves as expected on Firefox but not Chrome. This PR behaves as expected on Chrome but not Firefox...

@Rich-Harris can you tell what's wrong on Firefox and I'm sure I can fix it

https://user-images.githubusercontent.com/1162160/154121599-5cd3171d-cfc4-4929-bb62-f4630c747fd4.mov

@Rich-Harris okay, I understand the issue, I just found that hashchange acts differently on FF, and that's the reason, but I think I know how to fix it.

@Rich-Harris fixed, please test now, it should work in Chrome/Edge and FF

Thanks. It turns out I can get the test to pass with just the most recent change, so I've opened a separate PR with just that: https://github.com/sveltejs/kit/pull/3931
Is there a separate issue that this PR is also solving? Given how volatile scroll handling has turned out to be, I'd favour a cautious (and test-driven) approach to making changes

Thanks. It turns out I can get the test to pass with just the most recent change, so I've opened a separate PR with just that: #3931
Is there a separate issue that this PR is also solving? Given how volatile scroll handling has turned out to be, I'd favour a cautious (and test-driven) approach to making changes

Yeah, to be honest I thought of that as well, but this solution is much safer, than relaying on the scroll event, both Remix(https://github.com/remix-run/remix/blob/1fd70960e4d88740df5bf407a6ba2cd2b9549459/packages/remix-react/scroll-restoration.tsx) and Next.js(https://github.com/vercel/next.js/blob/81b537c04e694930a2fb9e6870ba9c5805269683/packages/next/shared/lib/router/router.ts) are using a solution similar to this. I based this solution on theirs. because scroll event is pretty bad to rely on, and on low powered devices(and Svelte is great for those) scroll event can cause throttling(lagging) of scrolling. I've seen it happen many many times. So consider this, but that PR is okay too, if that's what you want.

In this PR we are fully using our own scroll handling, so in the future if we decide to add some scroll behavior that's more complicated, we can do it with PR, but not with that with scroll event listener.

and there is also that Ctrl+shift+t bug that's not present in this PR, I know it's not that big of deal, but still, one of many upsides of this PR comparing to that one.

For the sake of the git history I think what I'll do is merge #3931 first, so that this PR becomes purely about eliminating the scroll handler and the ctrl+shift+t bug (which we'd ideally have a test for — not yet sure if that's possible with playwright)

Can we add a comment explaining why the hash_navigating variable is needed and listening to hashchange is not sufficient?

Can we add a comment explaining why the hash_navigating variable is needed and listening to hashchange is not sufficient?

https://github.com/sveltejs/kit/pull/3937

#3936 addresses the Cmd-Shift-T bug separately from everything else

Ignore #3936, I've opened #3938 which supersedes it.
I think that PR covers everything here, but more simply — as far as I can see we don't need to track sveltekit:index in both history.state and sessionStorage, we don't need to track sveltekit:exited, and we only need to update sessionStorage in beforeunload, unless I'm missing something?


decouple hit tests from the bindable concept

I think it's just a temporary coincidence that non-linear elements are bindable at the same time (in the future we may wanna make filled linear shapes bindable too).
Either way, I'd rename distanceToBindableElement since it couples unrelated code to the binding concept, but want to run it through @xixixao first.
Also, distanceToRectangle() name isn't correct either as it tests against text elements, too (→ not a rectangle).

On distanceToBindableElement, sure. I couldn't find a good name for these ("polygonal?", "shape?", "regular?")
distanceToRectangle -> distanceToRectangular (also would be good to have a reusable name)
Alternatively we can use explicit verbose names (distanceToRectangleOrText etc)


Broken link for Microchip on Vendor Support page

On the Vendor Support page, the link for Microchip gives a 404.

Issue should not have been opened here.


Problem with seq2seq models in prediction

@mrry Hi, I am confused with a prediction problem implemented in TensorFlow seq2seq. I found that in seq2seq inference code, there must be a embedding matrix to step forward. But now, my problem is a continuous prediction problem, such as price prediction. The output must be a sequence of floating numbers. I don't know how to embed these numbers, so what should I do with TensorFlow seq2seq? Thanks a lot!!

Thanks for reaching out. We try to keep this issue tracker limited to bugs and feature requests. We recommend posting on StackOverflow for community-driven support. We have this policy so that guys like @mrry will have as much time as possible to make TensorFlow better for everyone. Thank you for your understanding.


General README.md changes

@lorencerri If you want to open that pull request and merge it directly into master, you can. Dev is only there to get a lot of features in, test them and make sure everything is fine before releasing a version. Stuff like bug fixes and changes who doesn't have much an impact in the next version can be directly merged into master since dev may take some time before being merged into master

Sounds good, just finishing up changing some things real quick then I'll push it

Committed directly to master


 Open Source

Hi there! I was scrolling the 'O' section and thought it would be a good idea to add an Open Source section. It would be a great way for programmers of all levels to learn how to start their Open Source journey.

It's nice bro start

We have an "Open Source Ecosystem" section in books. Is there another list with resource we could group into a new section?

Hmm..maybe something about Blogging?


Extend the policy to handle DefaultAllowPrivilegeEscalation

The original Kubernetes PSP featured also a DefaultAllowPrivilegeEscalation option.
Quoting the official docs:

DefaultAllowPrivilegeEscalation - Sets the default for the allowPrivilegeEscalation option. The default behavior without this is to allow privilege escalation so as to not break setuid binaries. If that behavior is not desired, this field can be used to default to disallow, while still permitting pods to request allowPrivilegeEscalation explicitly.

We should extend this policy to handle this scenario as well.
Acceptance criteria

The policy gains mutation abilities
If DefaultAllowPrivilegeEscalation is set to false and a securityContext doesn't provide an explicit value for allowPrivilegeEscalation, the policy will mutate the incoming object to ensure allowPrivilegeEscalation = false

This is required to close https://github.com/kubewarden/policy-hub/issues/32

This is addressed starting from version 0.1.9


Currently unable to fold up in vertical direction.

Hi! I'm currently able to fold in the horizontal direction, but I wanted to fold upwards. What is weird is that I've set the hPanels: 3 like in the demo6, but it still does horizontal. I'll post if I find solution, maybe something is overriding my setting..
awesome code btw, ty.

nevermind! :joy_cat: I was being silly and didn't specify an anchor 'top'.


File not found

I have equal configurations for standard artifact archive plugin, and minio upload plugin (see screenshot).
Artifacts get archived without problems.
Minio fails with "No such file or directory"
ERROR: Failed to upload files java.io.FileNotFoundException: /Volumes/Data/Repo/workspace/openssl_android/deploy/libcrypto.so (No such file or directory) 	at java.io.FileInputStream.open0(Native Method) 	at java.io.FileInputStream.open(FileInputStream.java:195) 	at java.io.FileInputStream.<init>(FileInputStream.java:138) 	at java.io.FileInputStream.<init>(FileInputStream.java:93) 	at org.jenkinsci.plugins.minio.MinioUploader.perform(MinioUploader.java:146) 	at hudson.tasks.BuildStepCompatibilityLayer.perform(BuildStepCompatibilityLayer.java:78) 	at hudson.tasks.BuildStepMonitor$2.perform(BuildStepMonitor.java:32) 	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:779) 	at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:720) 	at hudson.model.Build$BuildExecution.post2(Build.java:185) 	at hudson.model.AbstractBuild$AbstractBuildExecution.post(AbstractBuild.java:665) 	at hudson.model.Run.execute(Run.java:1753) 	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43) 	at hudson.model.ResourceController.execute(ResourceController.java:98) 	at hudson.model.Executor.run(Executor.java:404)

Tested with Windows and MacOS slaves.

ping @NitishT

Thanks @harshavardhana
@varnav let me try to reproduce it myself. Will keep you posted.

@varnav the plugin will upload files from the workspace after they are created once the project is built. Looks like the artifact archive plugin deletes the files after upload. Can you confirm this?

Artifact archive is not a plugin, it is a core feature of Jenkins. It does not delete files.
I've ran into another problem when testing. I've restarted docker container and it has different credentials now. I've updated credentials in "Configure System", but as I see in the logs, minio plugin still uses old credentials, so I'm getting 'access denied' errors.

I've ran into another problem when testing. I've restarted docker container and it has different credentials now. I've updated credentials in "Configure System", but

You should persist the credentials using environment variables - default docker env is ephemeral
docker run -p 9000:9000 --name minio1 \
  -e "MINIO_ACCESS_KEY=AKIAIOSFODNN7EXAMPLE" \
  -e "MINIO_SECRET_KEY=wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY" \
  -v /mnt/export/minio1:/export \
  -v /mnt/config/minio1:/root/.minio \
  minio/minio server /export
```

No, the problem is that minio plugin is not using the credentials that are currently set in "Configure System", it is using the old credentials that were set first time.

Yes, the plugin doesn't know about the Minio server credential every time a new server comes up. To use the same credentials in the plugin, you should persist the credentials when you create the Minio container using environment variables.
docker run -p 9000:9000 --name minio1 \
  -e "MINIO_ACCESS_KEY=AKIAIOSFODNN7EXAMPLE" \
  -e "MINIO_SECRET_KEY=wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY" \
  -v /mnt/export/minio1:/export \
  -v /mnt/config/minio1:/root/.minio \
  minio/minio server /export
```

So, this means there is no way to change credentials in the plugin? What this setting means then?

okay, to clarify

To change the credentials in the plugin, Go to Manage Jenkins >> Configure System. Then scroll to the Minio upload configuration section & provide the new credentials.

Does this not work for you?

Yes, this does not work for me, as I say above: I've updated credentials in "Configure System", but as I see in the logs, minio plugin still uses old credentials

Can you paste the log here?

Credential is not equal to one in the settings (see screenshot above)
/Volumes/Data/Repo/workspace/openssl_android/deploy/openssl-1.0.2j
ERROR: Minio error
error occured
ErrorResponse(code=AccessDenied, message=Access denied, bucketName=openssl.android, objectName=null, resouce=/openssl.android, requestId=149D924BCF88F5C1, hostId=null)
request={method=HEAD, url=http://<IP_ADDRESS>:9000/openssl.android, headers=Host: <IP_ADDRESS>:9000
User-Agent: Minio (amd64; amd64) minio-java/dev
x-amz-content-sha256: e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
x-amz-date: 20170127T080632Z
Authorization: AWS4-HMAC-SHA256 Credential=NDCVPVO3QT8LZDSHY3L0/20170127/us-east-1/s3/aws4_request, SignedHeaders=host;x-amz-content-sha256;x-amz-date, Signature=91d4e2413d502a2d1cbd72cf4ed084900bef052a2e5d10d48a6d688e6b400fc7
}
response={code=403, headers=Accept-Ranges: bytes
Server: Minio/RELEASE.2017-01-25T03-14-52Z (linux; amd64)
Vary: Origin
X-Amz-Request-Id: 149D924BCF88F5C1
Date: Fri, 27 Jan 2017 08:05:06 GMT
Content-Type: text/plain; charset=utf-8
OkHttp-Sent-Millis:<PHONE_NUMBER>681
OkHttp-Received-Millis:<PHONE_NUMBER>684
}

	at io.minio.MinioClient.execute(MinioClient.java:576)
	at io.minio.MinioClient.executeHead(MinioClient.java:646)
	at io.minio.MinioClient.bucketExists(MinioClient.java:1303)
	at org.jenkinsci.plugins.minio.MinioUploader.perform(MinioUploader.java:127)
	at hudson.tasks.BuildStepCompatibilityLayer.perform(BuildStepCompatibilityLayer.java:78)
	at hudson.tasks.BuildStepMonitor$2.perform(BuildStepMonitor.java:32)
	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:779)
	at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:720)
	at hudson.model.Build$BuildExecution.post2(Build.java:185)
	at hudson.model.AbstractBuild$AbstractBuildExecution.post(AbstractBuild.java:665)
	at hudson.model.Run.execute(Run.java:1753)
	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
	at hudson.model.ResourceController.execute(ResourceController.java:98)
	at hudson.model.Executor.run(Executor.java:404)
Build step 'Upload build artifacts to Minio server' changed build result to UNSTABLE
Finished: UNSTABLE

Thanks! Let me go through this and try on my end.
As for your earlier issue, I tested and was able to upload files after the build process. Can you tell me the environment you are using? Jenkins running in a container?

No, Jenkins is not inside a container, it is pure Debian 8 + Jenkins 2.32 from WAR.

Okay, lets clarify the credentials issue and then move to the file not found. I will get back to you after I try this.

Hi @varnav the credentials issue was indeed a bug, its fixed now. Can you please try with the attached .hpi file again.
You'll need to uninstall the current Minio storage plugin. Then unzip the attached file and install the plugin file (.hpi) via Manage Jenkins >> Manage Plugins >> Advanced >> Upload Plugin.
I tested again, the file upload is working fine. Please try and let us know how it goes.
minio-storage.hpi.zip

Settings are fine now, but "file not found" is still there. It sees the file (if I use wildcard, it says exact file name in error message, with correct full path), but can't got further.
I've tested this with different job, with different node (Windows based now), error is the same. Archive artifact option is now turned off, no effect.
Anything I can do to debug this?

@varnav If its okay for you, can we discuss on call and find out the issue? We hang out on slack: slack.minio.io Please join and ping me, so we can decide on a time and see what exactly is causing the issue.

As discussed with @varnav on call today, the plugin works when the build happens on the master. But if the build is done on the slave, files are not uploaded. Will add that feature and upload the .hpi for validation.
Once validated, will create a new release.

Hey @varnav I have finally been able to add the feature - slave build files should now be uploaded to the Minio server too.
It would be great if you could try this and let us know if it worked fine for you. Once this is done, we'll make a release.
minio-storage.hpi.zip

Closing as it is now possible to upload Jenkins distributed build files Minio via the plugin. Please reopen if there is an issue.
We released version 0.0.2 with this fix: https://repo.jenkins-ci.org/releases/org/jenkins-ci/plugins/minio-storage/0.0.2/


Date fields behavior - enter date manually

In audit list, date fields behavior, enter date manually needs to be added. @yawitz
I've added a good portion of code to support this, but there appears to be a data binding issue that is causing any entry in the input field to be automatically converted to a Date object.

Let's look at this today.

@laurelnaiad https://github.com/angular-ui/bootstrap/issues/2886 tracks the same issue I'm seeing where a Date object is getting created, even when the value for the input is invalid.  There is a comment requesting a change to the code.  I've verified that the suggested change works.
Should I go ahead and make a pull request with this change?  It's that or we attempt to work around it (which would most likely be quite difficult).

So the change is to remove  || new Date(viewValue)?
Either way, can you put the change into a fork on your account so we can play with it?

https://github.com/clockworked247/angular-ui-bootstrap/commit/95bef7f5d89934d74b5e65669b2570846cc3b586

Unable to complete the interaction details as spec'd, due to ng-blur, ng-click and ng-keydown.
Missing:

On ENTER keypress, attempt to apply value in input
On BLUR, attempt to apply value in input

I have isolated a simplified version of my directive in a plnk, but am not able to replicate the issue there.  See:
http://plnkr.co/edit/u645bPv1dHT3ThKgIHrl?p=preview
So at the very least, this means that the Angular Bootstrap datepicker directive is NOT gettting in the way.  The question is.... what is?

I am able to verify it on Chrome / Firedox both on  Mac / Windows  .
On IE9  browser , as soon as you manually enter the date to the to/from fields they fade away .
Todo for QA : once this bug is fixed Verify this https://github.com/marklogic/marklogic-samplestack/issues/414  on IE9 browser also .

PR https://github.com/laurelnaiad/marklogic-samplestack/pull/57 is in that fixes the IE9 issue.  Once that's merged, this can go to test.  Thanks!

Working on IE9 now.


Job stucked

I try to use BigBench for benchmarking on my CDH installation. It is stucked in Data generation state.
State:  ACCEPTED
FinalStatus:    UNDEFINED
Started:    Fri Mar 11 09:12:04 +0100 2016
Elapsed:    1hrs, 3mins, 45sec
Tracking URL:   UNASSIGNED
It is maybe because of overusage of resources. I have already checked and modified yarn.scheduler.maximum-allocation-mb, mapreduce.map.memory.mb, mapreduce.map.java.opts, mapreduce.reduce.memory.mb, mapreduce.reduce.java.opts values.
I have found the rule that:
mapreduce.map.memory.mb < yarn.nodemanager.resource.memory-mb < yarn.scheduler.maximum-allocation-mb
Cluster is running on 2 KVM virtualized VM. Each have 4 CPU cores, first VM have 36 GB RAM and 80 GB disc space, second have 16 GB RAM and 40 GB disc space.
How should values above configured to run jobs on this infrastructure? All the jobs are waiting for execution in ACCEPTED state. I have calculated "optimal" values with tuning spreadsheet, but there are also all the jobs in ACCEPTED state. If one is RUNNING in a short time, it falls back to ACCEPTED again.

Closed - not a BigBench specific problem but general cluster setup issue.
Please understand that we cannot provide support for cluster setup issues.
You may try the google group:
https://groups.google.com/forum/#!forum/big-bench
Regarding cluster setup, the following section of our readme might be helpful:
https://github.com/intel-hadoop/Big-Data-Benchmark-for-Big-Bench#cluster-setup


zsock_socket() should support proto==0

Is your enhancement proposal related to a problem? Please describe.
Most implementations of BSD-style sockets, both those aiming at comprehensive POSIX compliance (e.g. Linux, macOS, *BSD) and those intentionally deviating from it (e.g. WinSock, lwIP), allow calling socket() with the proto argument set to 0, which causes the protocol to be defaulted to a typical choice.
E.g. socket(AF_INET, SOCK_STREAM, 0) is equivalent to socket(AF_INET, SOCK_STREAM, IPPROTO_TCP) and socket(AF_INET, SOCK_DGRAM, 0) is equivalent to socket(AF_INET, SOCK_DGRAM, IPPROTO_UDP).
This is actually mandated by POSIX (see here and here), but as I mentioned above, even implementations that do not aim for full POSIX compliance typically support this usage.
This makes it harder to port some libraries and applications that would otherwise be trivially portable to Zephyr, but use socket() in this way.
Describe the solution you'd like
I'd like zsock_socket() to behave as POSIX-compliant socket() would for this case. A minimal viable solution would be to make (AF_INET|AF_INET6, SOCK_STREAM) default the protocol to IPPROTO_TCP and (AF_INET|AF_INET6, SOCK_DGRAM) default the protocol to IPPROTO_UDP, as these are the two most common cases.

Sure, this makes sense. I will prepare a PR for this.

This is actually a bug, marking as such.


Add field Rename to GobblinTrackingEvents converter

@htran1 can you take a look?

@htran1 Fixed. Thanks!


doesn't work 😕

I follow the instructions to the letter (for Traefik without SSL), but it doesn't work: safari does't find http://localhost/whoami
I'm on macOS Big Sur, ssh-ing into an rpi4 in the home wifi network.

If you are ssh-ing into your rpi and you want to access it from another computer on your network you need to use the IP address of your rpi. So if your rpi has an IP address of a a <IP_ADDRESS> then you should be able to access whoami from http://<IP_ADDRESS>/whoami.
Localhost will always reference the computer you are on, I should probably make that clearer in the docs.

Oh, right, of course. I see whoami now. But I cannot access other services. Am I not configuring them well? Here's an example:
 homeassistant:
    container_name: homeassistant
    image: homeassistant/raspberrypi4-homeassistant:stable
    volumes:
      - /home/pi/homeassistant/config
      - /etc/localtime:/etc/localtime:ro
    restart: unless-stopped
    labels:
      - 'traefik.enable=true'
      - 'traefik.http.routers.homeassistant.rule=PathPrefix(`/homeassistant{regex:$$|/.*}`)'
      - 'traefik.http.services.homeassistant.loadbalancer.server.port=80'
    networks:
      - pi

the traefik dashboard shows it's up, but it has a weird IP (and I can't access it)

So looking at the home assistant docs there are a few things wrong with your setup.
Home Assistant runs on port 8123 instead of port 80. You also need to specify where to mount the config directory in the container.
Your setup should probably look something like this:
 homeassistant:
    container_name: homeassistant
    image: homeassistant/raspberrypi4-homeassistant:stable
    volumes:
      - /home/pi/homeassistant/config:/config
      - /etc/localtime:/etc/localtime:ro
    restart: unless-stopped
    labels:
      - 'traefik.enable=true'
      - 'traefik.http.routers.homeassistant.rule=PathPrefix(`/homeassistant{regex:$$|/.*}`)'
      - 'traefik.http.services.homeassistant.loadbalancer.server.port=8123'
    networks:
      - pi

As there isn't an issue with this repository I am going to close this issue. If you need any help getting home assistant setup they have a few channels you can get help on https://www.home-assistant.io/help/

thanks for the help (it still doesn't work, but I'll find the way)


Slow refresh The Old Reader Account

NNW 6.0d5 refresh time for a The Old Reader account is slow, refresh takes several minutes.

I just tried it and it completed in seconds for me.
How many feeds do you have?  Is it always slow or does it vary?

I just tried it and it completed in seconds for me.
How many feeds do you have?  Is it always slow or does it vary?

I have 151 feeds. It's always slow for me.
On Sun, 31 Jan 2021 at 21:22, Maurice Parker<EMAIL_ADDRESS>wrote:

I just tried it and it completed in seconds for me.
How many feeds do you have? Is it always slow or does it vary?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Ranchero-Software/NetNewsWire/issues/2772#issuecomment-770444418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALYLB2GVN6AUVQ6XHCA3KLTS4W3X7ANCNFSM4W3DNASA
.

I have 151 feeds. It's always slow for me.
On Sun, 31 Jan 2021 at 21:22, Maurice Parker<EMAIL_ADDRESS>wrote:

I just tried it and it completed in seconds for me.
How many feeds do you have? Is it always slow or does it vary?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Ranchero-Software/NetNewsWire/issues/2772#issuecomment-770444418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALYLB2GVN6AUVQ6XHCA3KLTS4W3X7ANCNFSM4W3DNASA
.

Have you synced using another RSS reader before?  If so how does NNW compare?
If you feel comfortable sharing you OPML file with me, I can take a look and see how much data is being transferred. You can send it to<EMAIL_ADDRESS>

Have you synced using another RSS reader before?  If so how does NNW compare?
If you feel comfortable sharing you OPML file with me, I can take a look and see how much data is being transferred. You can send it to<EMAIL_ADDRESS>

I've been using FeeddlerPro on iOS for the past 8-9 years. It synches
almost instantly.
Last time I used a desktop RSS reader was NNW back when Google Reader was
available.
I will send my OPML file under separate cover.
On Sun, 31 Jan 2021 at 22:53, Maurice Parker<EMAIL_ADDRESS>wrote:

Have you synced using another RSS reader before? If so how does NNW
compare?
If you feel comfortable sharing you OPML file with me, I can take a look
and see how much data is being transferred. You can send it to
<EMAIL_ADDRESS>—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Ranchero-Software/NetNewsWire/issues/2772#issuecomment-770457430,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALYLB2BVSCT4M23OLOOJVJDS4XGNJANCNFSM4W3DNASA
.

Thanks. Is it possible that you have a really high unread count number?  I'm just trying to figure out where the bottleneck is in the syncing process.

My unread count is around 101,500 now.
Op ma 1 feb. 2021 om 12:17 schreef Maurice Parker<EMAIL_ADDRESS>
Thanks. Is it possible that you have a really high unread count number?
I'm just trying to figure out where the bottleneck is in the syncing
process.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Ranchero-Software/NetNewsWire/issues/2772#issuecomment-770780518,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALYLB2GPA2YZTRDX4DQFGWTS42EUTANCNFSM4W3DNASA
.

Does your FeeddlerPro unread count match The Old Reader count?

Yes, it does.
Op ma 1 feb. 2021 om 13:27 schreef Maurice Parker<EMAIL_ADDRESS>
Does your FeeddlerPro unread count match The Old Reader count?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Ranchero-Software/NetNewsWire/issues/2772#issuecomment-770819703,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALYLB2E2GCJQPVGGDKGWMALS42MZJANCNFSM4W3DNASA
.

I did some research on this today and FeeddlerPro isn't actually doing any syncing.  It downloads you list of feeds and unread counts to display.  Only when you select a feed does it download the article for you to view.
NetNewsWire syncs with The Old Reader and always has your articles available on your computer. That is how we provide things like fast searching of articles.
Unfortunately the web services that The Old Reader provides doesn't allow us to only get the newest unread articles for your account.  We have to ask for all the unread articles for a user.  In this case, this over 100k in articles.  That takes network bandwidth and time to download.
My recommendation is to use The Old Reader's web interface to mark very old articles as read.  If you mark everything older than 1 month as read, you should see good syncing performance from NetNewsWire.


hideBelow is responsive to incorrect breakpoints

Description
When I use hideBelow: 'md' or hideBelow="md", I expect the element to hide responsively on the md breakpoint (not inclusively, defaults defined here), but instead it responds to the breakpoint that is one level above (i.e. hideBelow="md" is responsive on the lg breakpoint).
Link to Reproduction
https://play.panda-css.com/ZkLH0TJfET
Steps to reproduce

Go to reproduction link
Select Preview view
Adjust Preview to the lg breakpoint value (1024px), and tinker back and forth around that breakpoint.
See error. It is responsive to the lg breakpoint even though md breakpoint (768px) is passed.

JS Framework
No response
Panda CSS Version
0.13.1
Browser
No response
Operating System

[X] macOS
[ ] Windows
[ ] Linux

Additional Information
No response

I too am experiencing this issue, but  I have noticed even though I have it set to md =  768px , for displayBelow, the display will disappear at lg sometimes even before  randomly between lg and xl and it happens differently for laptop monitor vs desktop monitor.

Thanks for the update


Performance of large word lists

I ended up creating a similar tool for one of my projects that uses the larger and more comprehensive word lists found here: https://github.com/berzerk0/Probable-Wordlists.
Dealing with such large lists is problematic because of their sheer size as this will require loading a large number of strings into memory or doing an operating system level comparison on the file.  I found that I could get reasonable performance by recognizing that a large number of the words in the large lists were invalid due to other restrictions (i.e., not long enough, no number, no symbol, etc...).

Thanks for the tip, the project you suggested seems to have a more comprehensive word list! I will definitely take a look at replacing my current wordlist with one from over there.
Good idea pruning the wordlists based on your password restrictions (length and character type requirements). Unfortunately for this project, I can't assume there are any restrictions 😬 (perhaps I can assume there is a length restriction). So at the cost of additional time/space, I have decided to make this library as general as possible without any assumption of password restrictions.
In additional to any password restrictions, it also might be nice to let the user know they are using a common known password. This choice to left to the implementer.

Another suggestion for you... instead of trying to use a regex to search the list as a giant string, you might do the following...

load in the list as a sorted array
use bsearch to look for the potentially bad password.

common_passwords = File.readlines(filename).map(&:chomp).map(&:downcase).uniq.sort
# the sort, downcase and unique could be done on the file ahead of time for efficiency

password_found = !common_passwords.besearch { |common_password| password.downcase <=> common_password }.nil?

This will do a binary search through the entire list, which should be much faster than a regex match. (my very basic experiments suggest it's about 5 times faster).
I would also do the downcase so that you are doing case insensitive matching (PaSsWoRd isn't really better than password).

Thanks for the suggestion, I had issues using Array#bsearch but I found using using Ruby Set to be extremely quick. There is probably a trade-off with additional space (memory) but worth investigating.
PRs are welcome :) There is a benchmark file to compare the various ways to detect a bad password if you want to compare your solution https://github.com/gchan/password_blacklist/blob/master/bin/benchmark

Yeah, sets are even more efficient.  Use that.

Some benchmark results:
Comparison:
        Set#include?:  4847468.9 i/s
       Array#bsearch:   497979.7 i/s - 9.73x  slower
        String#match:     5164.2 i/s - 938.67x  slower

The downside is that a Set requires a significant amount of memory (I believe it was more than 10x) so I decided to go for the more space efficient solution for now. However there is no reason why we couldn't write a fast version that uses Set (e.g. PasswordBlacklist::FastChecker).
I've taken your suggestion of performing case insensitive matching. This will be released in version 0.2.0.


Documentation: Resource and Remote Observer Processor links are inverted

When browsing the processors on the main branch, I noticed the links to the resource processor points to the remoteobserver processor in the root README.md.
The same is true vice versa.
These links just need to get flipped back around and I would be happy to do it myself.

Indeed! Thanks for the heads-up @git-johnson. We'd be honored to accept your contribution fixing this :smile:

#1253

Resolved, thanks @astencel-sumo & @swiatekm-sumo!


errorCount throwing an error

What version of Sass Lint are you using?
1.10.2
Please include any relevant parts of your configuration
What did you do? Please include the actual source code causing the issue.
just ran the linter on my main.scss file
What did you expect to happen?
it should have outputted the errors
What actually happened? Please include any error messages given to you by Sass Lint.
throw new exceptions.SassLintFailureError(errorCount.count + ' errors were detected in \n- ' + errorCount.files.join('\n- '));
SassLintFailureError: 51 errors were detected in

src/style/main.scss
at Function.sassLint.failOnError (/Users/andrewjones/Documents/codePen/did/darkdiner/node_modules/sass-lint/index.js:313:11)
at detectPattern (/Users/andrewjones/Documents/codePen/did/darkdiner/node_modules/sass-lint/bin/sass-lint.js:31:10)
at Object. (/Users/andrewjones/Documents/codePen/did/darkdiner/node_modules/sass-lint/bin/sass-lint.js:79:3)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.runMain (module.js:604:10)
at run (bootstrap_node.js:390:7)

If you're using a IDE plugin have you tried the CLI too?

I also get this on first try. Have you solved this or just moved on?

no and this project doesn't seem to be active eiher so just uninstalled it I believe.

Have you tried to use this command to validate your .scss files
node ./node_modules/.bin/sass-lint -v -q
There you should focus on -v -q flags.
Also in common you can stuck with another problem of max errors limitation settings.
If one of the published configs I saw limitation of 50 errors as a maximum.

@aljones15 and @DanielSundberg This looks like @AndyTyurin has hit the nail on the head. Check your configs and how you're running Sass-lint. If you post those and the commands you're using and report back we can look into it a little bit more.

It's clearly described in the readme and the docs though so I'm not sure what else can be done to help if you haven't read them..

In my opinion running "sass-lint" with default sample config should result in an error message I can understand without reading documentation.
If you don't agree with that I that's fine with me, just wanted to contribute here.

I'm sorry you feel that way but i can't understand why you refer to it as an issue when it's clearly described in the docs and the readme.. By design it may not be the friendliest first experience agreed, but until we move into a new major version I'm afraid that's what we're stuck with, if you have feedback or a solution then please open an issue and describe what you think the default should be and what flags should be used to get our current default behaviour or at least start the discussion around it in the relevant places so we can address these issues for new users going forward..
We give the RTFM as you call it because it's the reason we wrote them and invested a lot of time into them, to help new users such as yourself and it takes more time to answer issues frequently where people haven't read through the readme properly which is the reason why so many projects like this end up abandoned because our inboxes and notifications get jammed up. We're happy you contributed and glad you've solved your issue and if you feel there's a better solution than what we have now then please raise an issue or suggestion to start a discussion about where to go next.

@DanPurdy, thanks for taking the time to write your reply, of course I can understand your point since also been on your side on a number of projects.

sass-lint -v -q ;)

@DanPurdy Have you moved to a new major version yet? Just run into the same issue.


codes at Java need to be revisited

ALL CODES added without class decration
and this is invalid

Thanks for working on this. :)


Add Scheduling Configuration reference doc

kubernetes/enhancements#1451
kubernetes/enhancements#785
Updates include:

Changes from v1alpha2
New VolumeBinding extension points
Redirect from old path

/sig scheduling

Thanks all for the feedback. This PR is still WIP, but section header and id is now in its intended state.

Hi @alculquicondor 👋 , Please rebase your PR to avoid major merge conflicts :) Thanks!

/assign @Huang-Wei
cc @pancernik

cc @cofyc for VolumeBinding changes

Latest revision looks good to me.

I'm not sure who is reviewing for SIG Scheduling. I think that this should have technical review (and I also hope that this is easy to review).
@kubernetes/sig-scheduling-pr-reviews this is reach for tech review. Please /lgtm when happy.

Preview: Scheduler Configuration

@ahg-g  mind giving lgtm?

/lgtm
Tin, I already reviewed the PR

/assign

cc @cofyc for VolumeBinding changes

VolumeBinding changes LGTM

LGTM

@savitharaghunathan or @sftim anything else for approval?

Changes since https://github.com/kubernetes/website/pull/21437#issuecomment-654498552 look good to me
/lgtm


Issue History comments

On the ‘Issue History’ section of the Issue Details page, are all comments viewable to all users?
For example where dev/design add comments to issues, are these then viewable to the client? If so, would it be worth considering something along the lines of a private or internal comment feature?

Additionally, I think it would be quite important to be able to open private, or non client facing issues where internal QA or feedback required some additional updates. I think this would help integrate the system fully into the working process if it was used to log all issues on a project, internal and external.

Internal comments are a good idea, something I'll probably implement and assign to this thread. As for the latter comment, you can currently set issues (and even whole projects) to "hidden from client" by checking a box on creation / update. Is this the type of feature you were looking for? (This also wouldn't be hard to implement for comments, which would address the former).

Ah, ok, yes, being able to create 'hidden' or non client facing issues on a project solves that, probably would have been helpful if I clicked the 'Hidden from Client?' tick box to see what happens!


Add some features to code builder

What I did:

Rewrote found classes into ES6 standard
Added some new handy methods to CodeBuilder class, so that you can control indentation easier (an example will in another pull request with Reqwest target for Rustm which I'll upload later)
Added %q (escape quotes and other special characters) and %v (stringify json) options in format. Before each call used %s, so in many cases it led to unescaped quotes.
Tried to replace %s to %q where seems appropriate. Used VSCode replace functionality and Regex, tried to do as accurately, as I can. Still passing tests, so I hope I did it completely. Won't be worse in any case.

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

I will upload Reqwest target for Rust when it will be 100% clear whether this pull request is going to be merged or not

I see. Should I close this pull request, rework these features and make a new PR?
I'd like to discuss each feature separately:

This update fixes unescaped quotes (although by changing some %s-es to %q-s in the target files). Should I undo that as well?
The update with adding %q and %v format rules - do I understand correctly, that you suggest to still implement it (it's a few lines of code and a new file), but without moving the whole CodeBuilder class to ES6?
The same question, but about indent, unindent and reindent methods. Can I still add them to the "native" CodeBuilder, or do I better move them inside the Rust target, where I actively use them?

The update with adding %q and %v format rules - do I understand correctly, that you suggest to still implement it (it's a few lines of code and a new file), but without moving the whole CodeBuilder class to ES6?

The same question, but about indent, unindent and reindent methods. Can I still add them to the "native" CodeBuilder, or do I better move them inside the Rust target, where I actively use them?

In both cases, yes you can still implement them in the CodeBuilder, that's fine! Just avoid converting the whole thing or making unnecessary changes if possible. Adding methods isn't likely to cause conflicts (or those conflicts will be easy to fix). Modifying existing methods etc is much more likely to cause conflicts, so we should minimize that, where possible.

This update fixes unescaped quotes (although by changing some %s-es to %q-s in the target files). Should I undo that as well?

Good point! I think for places where the quotes and useful and currently missing, yes, let's add the fix to change them to %q, that's worth fixing. If the quotes are already there, so we wouldn't actually change anything about the output, don't bother.

Should I close this pull request, rework these features and make a new PR?

Up to you - I'm fine with editing this one, or a new PR. It would be nice to end up with a clean git history in the end either way though. If you're happy rebasing and squashing you can do that within this PR and force-update it, or you can make a totally new branch and build a new set of changes there if that's easier, totally up to you.
Does that all make sense? Let me know if you have any other questions.

%q does not add quotes, it only escapes some characters (like \n and ") in the string. So it is only useful inside quotes inside generated code lines (exactly where I only added them), so that, for example:
// this
format('Hello("%q")', '{ "world": true }')
// would produce:
// Hello("{ \"world\": true }")

//while this:
format('Hello("%s")', '{ "world": true }')
// would produce:
// Hello("{ "world": true }")
// you see this breaks the string

For example, check out https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Sec-CH-UA header; it actually contains quotes and is present in many (if not all) requests from my chrome browser.
So I can completely remain this commit, right?

So I can keep this commit, right?

Sure - we can include this in the CodeBuilder and format helpers, that's fine, as long as it doesn't require rewriting the whole class completely. Skimming through the code, it looks like we should not keep most of the snippet changes to use %q though, because they're not really necessary (e.g. because they're string interpolations of HTTP methods, protocols or hostnames, none of which can ever contain quotes or newlines). Happy to keep any where there are valid values of the string that genuinely might need escaping, e.g. header values and bodies, or if it's just not clear either way.
The overall idea is that we should make changes here that make real valuable improvements to the snippet output. I just want to remove/shrink all changes that are optional (i.e. which don't ever change the output or support future changes that do) so we can minimize merge conflicts in future.
Converting to ES6 classes is optional, and doesn't change output, so we should remove that. It looks most of the changes of existing snippets to %q are optional and don't change output (e.g. all the tests still pass testing against the previous output) so we should skip most of those too. However, adding the CodeBuilder logic to allow using %q and the new indent methods isn't optional, because that's required to implement your Reqwest PR. Does that make more sense?

Should I change them back to %s, add a new test case with quotes everywhere they might be met in real requests, and then change to %q-s where the test fails? :\

Up to you! If you have time and you're interested in doing that, feel free. If it's difficult to do that though, you're welcome to just revert those changes instead, and we can add them in later incrementally as we spot specific cases where this matters.

Ok


Always installResources() on a fresh install

@chrisdembia Please let me know if you need help working on this one or I can take it over. Thank you

@chrisdembia I think we will need a build-id and have it accessible from the GUI code so that we can check for a new build and use that to call installResources. This will be very helpful anyway for frequent build/distributions long term maintenance.

Should we use a combination of the opensim-gui, opensim-models, and opensim-core commit hashes?

But opensim-models is a submodule, it needs new commit to make it to opensim-gui
HHashes are good but don't provide order so date followed by gui-commit may be good enough ID

Ah good point. However, opensim-core is not being used as a submodule (even though it currently exists as a submodule).
I'm fine with adding the date as well.

I wasn't prompted to install resources on subsequent builds so that part seem to have worked as expected.

If anybody has cycles to verify this that would be great. @carmichaelong @jimmyDunne @nickbianco Specifically:

install build from PR
Verify you get prompted to installResources on first launch and never after
Report what you see in About dialog for "Product Version"
I got expected results on my windows machine but would like to see what others see as well.

I can fix the CI scripts to avoid the version appearing twice.

Any volunteers to verify on either Mac or Windows @jenhicks @carmichaelong @nickbianco @jimmyDunne

I tried out on Mac. I think everything worked as expected. See images below.

Perfect, thank you @jenhicks

Verified on Windows 4.0-2018-08-23-480caf33.


Weird behaviour with tmux

So, tmux crashes - or at least involuntarily exists on behalf of me - whenever I source zgen/zgen.zsh. I have removed everything else from my .zshrc, so it's definitely related to calling source zgen/zgen.zsh.
More symptoms: tmux will open fine at first, but when I add a split, the split just dies right after. Sometimes, it will succeed creating the split, but creating another one may tear the whole thing down again. Tmux would then exit and print [exited]. The behaviour that surprises me most is that, say it managed to create 3 splits, closing one of them would bring down the entire tmux run.
My zgen install is up-to-date, I am on OSX (iTerm2) and zsh 5.0.2 (from brew).

Alright, it seems related to my .tmux.conf as well. It all works fine on a completely empty .tmux.conf. Adding set-option -g default-command "reattach-to-user-namespace -l zsh" to it, causes the behaviour. So something about the combination of reattach-to-user-namespace and zgen?

 Do you still have this issue?

Yes, the problem remains to exist with the most basic of .zshrcs:
source "${HOME}/.zgen/zgen.zsh"
if ! zgen saved; then
  # load oh my zsh
  zgen oh-my-zsh
  zgen save
fi

(copy and paste from project README)
I can see that without specifiying the syntax-highlighting plugin, zgen just enables it, and it appears to be the main culprit.

zsh-syntax-highlighting should not be enabled by default with that .zshrc. It may come from an old init script. Make sure you have refreshed the .zgen/init.zsh file by either removing it or running zgen reset and then restart the shell.
I've understood that reattach-to-user-namespace is a OSX namespace wrapper that does some questionable stuff on the shell. Unfortunately I don't have the resources to reproduce this issue.

Alright, removing ~/.zgen/init.zsh does work with the above .zshrc. But this one seems to fail:
source "${HOME}/.zgen/zgen.zsh"
if ! zgen saved; then
  # load oh my zsh
  zgen oh-my-zsh
  zgen oh-my-zsh plugins/vi-mode # <--- also load vi-mode
  zgen load zsh-users/zsh-syntax-highlighting
  zgen save
fi

The same thing happens for zplug. Hm, maybe it's vi-mode then... but vi-mode works fine on it's own, also. It's something about the combination.

Yeah, it happens even with this .zshrc:
source ~/.zplug/repos/robbyrussell/oh-my-zsh/plugins/vi-mode/vi-mode.plugin.zsh
source ~/.zplug/repos/zsh-users/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh

I am happy to close this, I guess it's not relevant to zgen in particular. Your call.

Good luck with the issue hunting.


ClassDocumentHydrator::hydrateSingleResource() enforces use of stdClass

See: https://github.com/woohoolabs/yang/blob/acdd0a11383bd23f405a4d42f4cea22f8f41eb57/src/JsonApi/Hydrator/ClassDocumentHydrator.php#L29
Which prevents the use of a custom hydrator which returns other objects.
PR here: https://github.com/woohoolabs/yang/pull/24

@kocsismate do you need meer information on this issue and PR? Or are the intents clear?

Hey @holtkamp Sorry for the delay!
Yes, the intents are clear (and I agree with it completely), my only problem is (and it's the reason I haven't answered yet) that technically, it's a BC break. :/ I am not sure though that the change breaks any compatibility in the practice. What do you think about the BC aspect of the change?

technically, it's a BC break.

mmm, is it?
An object is "looser" than specifically an instance of stdClass...
All instances of stdClass are an object, so AFAIK this is not a BC break but a bug fix :neckbeard:
So a patch version would suffice to release this.

I said that it's a BC break mainly because if someone overrides the hydrateSingleResource() then they will face an issue. At least below PHP 7.4 😎 https://3v4l.org/YaolE
All in all, let's merge your PR, it brings quite some benefits while its BC break aspect doesn't seem much in practice. Thank you for your work!

I said that it's a BC break mainly because if someone overrides the hydrateSingleResource() then they will face an issue. At least below PHP 7.4 😎 https://3v4l.org/YaolE

aaah, did you mean it like that. Mmm, yeah that is true.. Fingers crossed nobody did that yet 😉
Thank you for your effort!


Can I compute 2nd derivative of the loss computing from a fine-tuned model-w.r.t pre-trained model's input with higher

Hi, I don't know much about meta-learning. But I'm faced with a problem similar to implement meta-update with Pytorch which is illustrated in this Pytorch issue.
I have a pre-trained model W1 and two data point (x1, y1), (x2, y2). Firstly, I fine-tune W1 with (x1, y1) in one epoch, thus the model update  can be obtained by
And then, I get the test loss for (x2, y2) w.r.t fine-tuned model   by .
What I want to do is to use the test_loss to update x1 by doing

Howerver, as in meta-learning, the Module parameters updates  will perform in a non-differentiable way. So can I solve this problem with higher? Thank you in advance.

Yes this is what the library is meant to help you do: in near-native pytorch (i.e. you can take existing code and make hopefully minimal changes) you can unroll the training loop of a model, and take gradient with regard to just about anything. The use-case you describe is just the basic use-case, so good news :)
Closing this issue, but please re-open if you need any clarification. Otherwise, I recommend looking at example code and experimenting yourself.


Hide Category-links (along with hiding categories) on xxxDialog.ts

In my xxxDialog.ts i dynamically hide some category based on some user settings.
The point is that the Category-Links displayed on top/right are still showing the hidden category
(see below picture as a sample).

I cannot see any identifier to hide the link matching the category i want to hide.

Cross posted on StartSharp repositories.


Update to 1.2 release of Wicked Good XPath.

Thanks for your hard work. I've noticed the upstream patches you have submitted in the last few days.
Please test this PR using the example mentioned in the README.

Tested against word_of_the_day.js with an updated XPath reflecting the change of http://www.merriam-webster.com/word-of-the-day.


Synchronize master and v8

Moving features and fixes from master branch to `v8

I'm going to merge this PR and create a second one to add commits from v8 to master so that they're fully in sync. But I think we need to clarify our branching strategy moving forward so this doesn't keep happening.


Grid Customizer not working

I have copied the exact sample for the Power Apps Grid custom control and it executes the custom code but no visible UI changes occur on text or currency. Please advise next steps?

@WhiteyC
Can you clarify which sample you are referring to?
Is it the one mentioned here Customized editable grid?
With the sample code from here: https://github.com/microsoft/PowerApps-Samples/tree/master/component-framework/PowerAppsGridCustomizerControl
I think next steps are to use the steps here to debug and understand the issue better Debug code components
If you identify a specific issue, please let us know.

@WhiteyC Can you clarify which sample you are referring to?
Is it the one mentioned here Customized editable grid? With the sample code from here: https://github.com/microsoft/PowerApps-Samples/tree/master/component-framework/PowerAppsGridCustomizerControl
I think next steps are to use the steps here to debug and understand the issue better Debug code components
If you identify a specific issue, please let us know.

Yes, I am using the Grid Customizer Control. I have added a debugger; in the "int" and that triggers fine. However the cell render does nothing.
export const cellRendererOverrides: CellRendererOverrides = {
["Text"]: (props, col) => {
// Render all text cells in green font
return <Label style={{ color: 'green' }}>{props.formattedValue}
}
}
public init(
    context: ComponentFramework.Context<IInputs>,
    notifyOutputChanged: () => void,
    state: ComponentFramework.Dictionary
): void {
    debugger;
    const eventName = context.parameters.EventName.raw;
    if (eventName) {
        const paOneGridCustomizer: PAOneGridCustomizer = { cellRendererOverrides, cellEditorOverrides };
        (context as any).factory.fireEvent(eventName, paOneGridCustomizer);
    }
}

The control builds fine and deploys fine. Added to Power apps grid control as a customizer.

I've seen the same. It started suddenly, probably because of a Platform update last Sunday night.
It happended that the control I was developing worked, and a few minutes later, without any change, it stopped working.
I have also older controls which worked before: all of them stopped working since then.

I've seen the same. It started suddenly, probably because of a Platform update last Sunday night. It happended that the control I was developing worked, and a few minutes later, without any change, it stopped working. I have also older controls which worked before: all of them stopped working since then.

Good to see I'm not the only one going through this. It's the latest update on the Power Apps Grid control.

@WhiteyC  & @brasov2de
Adding @HemantGaur
It sounds like you are describing an issue that is larger than this specific sample. This sounds like a platform level issue rather than something we can address by changing this sample code. Am I understanding you correctly?
In this case, it is important that you contact technical support and report the issue. We don't have the resources to troubleshoot this with you.
If there is an issue with a specific sample, then we do want to get that sample fixed. Otherwise, this isn't the appropriate place to look for technical support.

I notified @HemantGaur and there seems to be a corresponding known issue in production.
Not yet sure if the fix is deployed to all regions at this point.
Since there is nothing actionable related to this sample code, I'm going to close this issue.

@JimDaly is there any info about when fix will be released?

@morgutrin / @WhiteyC  / @brasov2de This fix will be available by late March for most geos.

@morgutrin / @WhiteyC / @brasov2de This fix will be available by late March for most geos.

Can I ask where you found this information in regards to a fix?

@WhiteyC
@jasongre is the program manager for this feature.

@WhiteyC
@jasongre is the program manager for this feature.

Thanks Jim. That's not giving me any confidence in MS support as I have a case raised on this and they have not given me any information of this kind. Not helpful when you are trying to earn a living with this product and have deadlines.

Hello everyone, I am having a problem with the controlcustomizer and the column filters. I explain... When I filter some of the columns, sometimes, many times, the funnel icon and the option to clean filters, disappear from the column header. And the times that they don't disappear, when I clean filters, all the results are not shown again. In other words, it is as if I only get the filter the first time and the grid remains with that filter until I refresh the page.
Any help on this would be very welcome. This problem is driving me crazy and is going to force me to remove a control that has taken me many hours of development.
Thanks.


Failed to read temperature, humidity, co2, pressure

pyaranet4 --history works as expected, but pyaranet4 produces the error:
--------------------------------------
Connected: Aranet4 1BAE8 | v0.4.14
Updated 20 s ago. Intervals: 60 s
249 total readings
--------------------------------------
Traceback (most recent call last):
  File "/home/ighor/.local/lib/python3.10/site-packages/pyaranet4/pyaranet4.py", line 477, in _read_value
    value = await self._client.read_gatt_char(uuid)
  File "/home/ighor/.local/lib/python3.10/site-packages/bleak/__init__.py", line 571, in read_gatt_char
    return await self._backend.read_gatt_char(char_specifier, **kwargs)
  File "/home/ighor/.local/lib/python3.10/site-packages/bleak/backends/bluezdbus/client.py", line 679, in read_gatt_char
    assert_reply(reply)
  File "/home/ighor/.local/lib/python3.10/site-packages/bleak/backends/bluezdbus/utils.py", line 20, in assert_reply
    raise BleakDBusError(reply.error_name, reply.body)
bleak.exc.BleakDBusError: [org.bluez.Error.NotPermitted] Read not permitted

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/ighor/.local/bin/pyaranet4", line 8, in <module>
    sys.exit(main())
  File "/home/ighor/.local/lib/python3.10/site-packages/pyaranet4/__main__.py", line 61, in main
    basic_overview(a4)
  File "/home/ighor/.local/lib/python3.10/site-packages/pyaranet4/__main__.py", line 76, in basic_overview
    print("CO2:         {:d} ppm".format(a4.current_readings.co2))
  File "/home/ighor/.local/lib/python3.10/site-packages/pyaranet4/pyaranet4.py", line 155, in current_readings
    return self._get_readings(simple=False)
  File "/home/ighor/.local/lib/python3.10/site-packages/pyaranet4/pyaranet4.py", line 284, in _get_readings
    data = self.read_from_uuid(uuid)
  File "/home/ighor/.local/lib/python3.10/site-packages/pyaranet4/pyaranet4.py", line 230, in read_from_uuid
    value = self.loop.run_until_complete(self._read_value(uuid))
  File "/usr/lib/python3.10/asyncio/base_events.py", line 646, in run_until_complete
    return future.result()
  File "/home/ighor/.local/lib/python3.10/site-packages/pyaranet4/pyaranet4.py", line 479, in _read_value
    raise Aranet4UnpairedException("Error reading from device. Check if it is properly paired.")
pyaranet4.exceptions.Aranet4UnpairedException: Error reading from device. Check if it is properly paired.

The problem appears because if UUID_CURRENT_READING_FULL requested it produces "Failed to read: org.bluez.Error.NotPermitted", when UUID_CURRENT_READING_SIMPLE forced everything works fine. Possible solution is fallback to UUID_CURRENT_READING_SIMPLE when UUID_CURRENT_READING_FULL failed.


Add Buttondown

Hey Shawn,
I've added Buttondown to the Email Tools List. It's not for hacktoberfest ;)

lol nice comment


⚠ ClamAV has degraded performance

In f567388, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 91 ms

Resolved: ClamAV performance has improved in 4bf7b41.


vdk-jupyter: add run result to the deployment result dialog

What: Added run result message to the deployment result dialog ( if run before deployment is selected)
Current dialog:

Why: It used to not contain the successful run message which left the suers with confusion around whether really the run was successful
Signed-off-by: Duygu Hasan<EMAIL_ADDRESS>

The text here looks badly formated.
It would have been better to have spaces or new line between "sent suffully!New .."


CAN"T RUN DOTNET TRY

Describe the bug
Hi everyone, I made all of the steps like it says on the video and on the site. When i open MAC terminal and try to write dot net try and press enter .... Nothing happens. (First screenshot)

Then i try to open Vusial Studio on MAC and type dotnet  try  then always come up this:

I try to go and install everything and i still can't opened dotnet try.
WHAT TO DO.....?
Did this error occur while using dotnet try or online?

[x] dotnet-try

Please complete the following:

OS  - [ ] macOS - Ventura

same problem


Option to render all tracks in a file

It would be nice if there was an option to render all of the NSF tracks into a single video, or to individual videos, at once. This would be useful for soundtrack/album videos where visuals (like the background) are typically shared. I'm aware of the command-line interface allowing batch processing but having an equivalent in the GUI would be handy.

I think there are a lot of benefits to making the renderer framework capable of batch rendering. The current system works to automate batch rendering, but it's not the most usable. For example, there's no way for a script to tell how many tracks are in an NSF without relying on some other utility/some freak hack to read the header, or prompting the user for input. It's also a bit silly to render out a bunch of individual files and then have to stitch them together later (which can also be annoying if using FFmpeg since you also need to generate an index file for the concat demuxer).
Here's a random list of things that might be nice to have when doing batch renders:

CLI facilities for getting metadata from NSF files: track count, metadata, NSFe track titles/durations, etc.
CLI and GUI options for batch rendering NSF files to a single video (what would go into the metadata?)
CLI and GUI options for batch rendering NSF files to multiple videos (how would the files be named?)

The first feature is a pretty easy addition considering the value it adds for scripting/utility capabilities, so I think it'd make sense for the next release. The others will require a handful of GUI changes and refactoring on the render pipeline, so those will probably have to wait a bit.

Do note that switching tracks in the current version shouldn't reset any customization settings (channel colors, background, etc.) so after customizing the render, you can just select every track individually and render them out without needing to go back and fix the settings every time. NSF2 modules are better since NSFPresenter can use the time extfield for the render duration, which means you won't need to update the duration in between tracks.


Read from a remote file

This long-awaited PR supports the ability to read .mmdb and .mmdb.gz files from a remote address rather on your local file system.

Coverage increased (+0.6%) to 97.207% when pulling 3de477a6d3512385dcf1c93a62c9f0a6033c7c11 on acconrad:master into 7554c6b90958dba7e91de55ecdef507ab282e92d on mneudert:master.

You even made a test for it :+1:
I took your branch and made some additions to it: acconrad-master
What did I do?
Basically just one real change. And that was using :httpd to load one of already available fixture files for testing instead of an official download.
Everything else is just cleanup stuff. Like ensuring :inets is started so the http request does not fail because of that. Might not be necessary but also should not really hurt. On top of that I moved all file logic into the loader and did some cleanup to respond with a better error message when the http request fails.
If that looks good to you I would probably improve the docs a bit more and maybe squash some commits together. And then just merge what you did in the first place :shipit:

@mneudert sounds good! Do you want to just merge your branch into this PR so we can get this out the door?

All cleaned up and merged :heart: :yellow_heart: :blue_heart: :green_heart: :purple_heart:

Will release a new version tomorrow if I don't find any arguments against that plan.


[BUG]BF16_Optimizer.load_state_dict() got an unexpected keyword argument 'load_serial'

deepseed >=0.10.1
I train my model with bfloat16
when I load checkpoint , errors occured!
-> 2869 self.optimizer.load_state_dict(state_dict_list=zero_sd_list,
   2870                                load_optimizer_states=load_optimizer_states,
   2871                                load_from_fp32_weights=self.zero_load_from_fp32_weights(),
   2872                                checkpoint_folder=checkpoint_folder,
   2873                                load_serial=load_serial)
   2875 if self.load_universal_checkpoint():
   2876     logger.info(f'loaded universal zero checkpoints from {checkpoint_folder} for rank {self.global_rank}')

TypeError: BF16_Optimizer.load_state_dict() got an unexpected keyword argument 'load_serial'

deepspeed==0.10.3 still has the bug

Hello @janelu9, @passaglia, do you have a small example to reproduce this issue?
It will be very helpful to know the configs used to train the bfloat model.

Hello @janelu9, @passaglia, do you have a small example to reproduce this issue? It will be helpful to know the configs used to train the bfloat model.

my ds_config:
"bf16": {"enabled": True}"

Hello @janelu9, could you please try this PR to see if it fixes this issue.

@ShijieZZZZ Yes, https://github.com/microsoft/DeepSpeed/pull/4434 fixed the issue for me! 🥳

Thanks for confirming. I will close the issue then.


Fix options_ui in OSX Chrome

This PR aims to fix UI bug reported by @alanshaw:

Notes on options_ui in Chrome

fix is to either specify min-width and overflow controls, or explicitly open options_ui page in a new tab

open_in_tab is being deprecated, so we would have to do it manually

@lidel did this work for you? Mine still doesn't scroll.
I've had to add width and height to the body element to get scrolling working again. However that messes with how it's displayed on Firefox. So, I added a detection which adds an is-chrome class to the <html> element so that we can apply the styles only on chrome.

Note: the last commit that triggered a jenkins build was cancelled as I'm upgrading jenkins + workers. You'll see the build failed for now but as soon as everything is back, I'll retry this build. Sorry for the inconvenience

@alanshaw I failed to mention that my Chrome looked fine all the time  :) problem seems to be specific to OSX build of Chrome. Thanks for fixing it!
I noticed and fixed double scrollbar in Firefox, should not impact your fix for OSX, but please double check.
Is it ready to merge?

Yep, all good :thumbsup:


Import Error: No module named xxx

Tried on RHEL 7, macOS, and Windows 7 running Anaconda env's 3.4.4 and 3.5 all result in same error.
from etlalchemy import ETLAlchemySource, ETLAlchemyTarget
import pymysql
import cx_Oracle

o_db = ETLAlchemySource("db connection info")
m_db = ETLAlchemyTarget("db connection info")
target.addSource(source)
target.migrate()

results in
ImportError:  No module named 'ETLAlchemySource'
Import Error: No module named 'ETLAlchemyTarget'

I think there is an issue with the __init__.py.  I deleted the original and created a new one using nano and I am now able to get past the __init__.py.  I now am failing on
    from StringIO import StringIO
ImportError: No module named 'StringIO'

I can import it now will commit changes and request pull

Closing - this has been addressed on the Python 3.5 branch


WinCLIP improvements

📝 Description
Two small improvements to the WinCLIP torch model:

Set device dynamically when collecting text embeddings before inference. This is needed so that setup can be called after the model has been moved to cuda.
Encode the images without gradient in the forward pass. This is to preserve gpu memory when the model has not been set in eval mode by the user.

✨ Changes
Select what type of change your PR is:

[x] 🐞 Bug fix (non-breaking change which fixes an issue)
[ ] 🔨 Refactor (non-breaking change which refactors the code base)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] 📚 Documentation update
[ ] 🔒 Security update

✅ Checklist
Before you submit your pull request, please make sure you have completed the following steps:

[ ] 📋 I have summarized my changes in the CHANGELOG and followed the guidelines for my type of change (skip for minor changes, documentation updates, and test enhancements).
[ ] 📚 I have made the necessary updates to the documentation (if applicable).
[ ] 🧪 I have written tests that support my changes and prove that my fix is effective or my feature works (if applicable).

For more information about code review checklists, see the Code Review Checklist.

Could we also fix #1641 in this PR?


Merge pull request #7 from MicrosoftDocs/master

merge branch

Docs Build status updates of commit c1cf648:
:warning: Validation status: warnings
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit aa8875a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/gaze.md
:warning:Warning
View
Details

mixed-reality-docs/gaze-and-commit.md
:white_check_mark:Succeeded
View

mixed-reality-docs/gaze-in-directx.md
:white_check_mark:Succeeded
View

mixed-reality-docs/gaze.md

Line 63: [Warning] Illegal link: `<a href="index.md#news-and-notes">coming soon</a>` -- missing bookmark. The file index.md doesn't contain a bookmark named news-and-notes.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Upgrading cluster 1.20.0 <-> 1.19.0 Loop

I deployed a basic cluster trough and the cluster in constantly upgrading itself between cloudnative-pg:1.19.0 and cloudnative-pg:1.20.0
cloudnative-pg helm
---
apiVersion: helm.toolkit.fluxcd.io/v2beta1
kind: HelmRelease
metadata:
  name: cloudnative-pg
  namespace: default
spec:
  interval: 15m
  chart:
    spec:
      chart: cloudnative-pg
      version: 0.18.0
      sourceRef:
        kind: HelmRepository
        name: cloudnative-pg
        namespace: flux-system
  maxHistory: 2
  install:
    createNamespace: true
    remediation:
      retries: 3
  upgrade:
    cleanupOnFail: true
    remediation:
      retries: 3
  uninstall:
    keepHistory: false
  values:
    crds:
      create: true
    config:
      data:
        INHERITED_ANNOTATIONS: kyverno.io/ignore

Cluster Yaml
---
apiVersion: postgresql.cnpg.io/v1
kind: Cluster
metadata:
  name: postgres
  namespace: default
  annotations:
    kyverno.io/ignore: "true"
spec:
  instances: 3
  imageName: ghcr.io/cloudnative-pg/postgresql:15.2
  inheritedMetadata:
    labels:
      kube-image-keeper.enix.io/image-caching-policy: ignore
  primaryUpdateStrategy: unsupervised
  storage:
    size: 20Gi
    storageClass: freenas-iscsi-csi
  superuserSecret:
    name: cloudnative-pg-secret
  # bootstrap:
  #   recovery:
  #     source: &previous-cluster postgres-v1
  postgresql:
    parameters:
      max_connections: "600"
      shared_buffers: 512MB
  monitoring:
    enablePodMonitor: true
  backup:
    retentionPolicy: 30d
    barmanObjectStore:
      wal:
        compression: bzip2
        maxParallel: 8
      destinationPath: s3://mdegouw-testing03/
      endpointURL: https://s3.us-west-004.backblazeb2.com
      serverName: postgres-v1
      s3Credentials:
        region:
          name: cloudnative-pg-secret
          key: aws-region
        accessKeyId:
          name: cloudnative-pg-secret
          key: aws-access-key-id
        secretAccessKey:
          name: cloudnative-pg-secret
          key: aws-secret-access-key
  # Note: externalClusters is needed when recovering from an existing cnpg cluster
  # externalClusters:
  #   - name: *previous-cluster
  #     barmanObjectStore:
  #       wal:
  #         compression: bzip2
  #         maxParallel: 8
  #       destinationPath: s3://mdegouw-testing03/
  #       endpointURL: https://s3.us-west-004.backblazeb2.com
  #       s3Credentials:
  #         region:
  #           name: cloudnative-pg-secret
  #           key: aws-region
  #         accessKeyId:
  #           name: cloudnative-pg-secret
  #           key: aws-access-key-id
  #         secretAccessKey:
  #           name: cloudnative-pg-secret
  #           key: aws-secret-access-key

Here you can see thats its upgrading node postgres-3 from 1.19.0 to 1.20.0
Cluster Summary
Name:               postgres
Namespace:          default
System ID:<PHONE_NUMBER>007984160
PostgreSQL Image:   ghcr.io/cloudnative-pg/postgresql:15.2
Primary instance:   postgres-1
Status:             Upgrading cluster Restarting instance postgres-3, because: the instance is using an old init container image: ghcr.io/cloudnative-pg/cloudnative-pg:1.19.0 -> ghcr.io/cloudnative-pg/cloudnative-pg:1.20.0
Instances:          3
Ready instances:    2
Current Write LSN:  0/E0630B0 (Timeline: 1 - WAL File:<PHONE_NUMBER>0000000000000E)

And a couple of minutes later it's "upgrading" the postgres-3 from 1.20.0 to 1.19.0
Cluster Summary
Name:               postgres
Namespace:          default
System ID:<PHONE_NUMBER>007984160
PostgreSQL Image:   ghcr.io/cloudnative-pg/postgresql:15.2
Primary instance:   postgres-1
Status:             Upgrading cluster Restarting instance postgres-3, because: the instance is using an old init container image: ghcr.io/cloudnative-pg/cloudnative-pg:1.20.0 -> ghcr.io/cloudnative-pg/cloudnative-pg:1.19.0
Instances:          3
Ready instances:    2
Current Write LSN:  0/F01A020 (Timeline: 1 - WAL File:<PHONE_NUMBER>0000000000000F)

Did i miss something?

Hi @mdegouw
Can you confirm if this still happening ?
Regards,

Hi @sxd
Yes it is still happening.
It updates/downgrades the instances one by one constantly.

We have made several improvements and changes in the meantime. Can you please confirm it is still happening @mdegouw? thanks.

Probably this has already been fixed with the latest release 1.22, please open a new issue if this happens again


🐛 Bug ➾ after set up of scaffold there is an error in the logs which may be from taq being started with no parameters

🚥 Status (Internal Taqueria Team Use Only)

[x] 🔬 Investigated and Verified
[ ] ⚗ Solution Identified and Designed
[ ] 🧫 Dev Implementation of Fix
[ ] 🧪 Fix Tested and Validated
[ ] 🏆 PR Merged

🆘 What happened?
I looked in the log created by the scaffold project and found a taq error message consistent with taq being called with no parameters.
🆘 Steps to Reproduce?

install taq
taq scaffold https://github.com/ecadlabs/taqueria-scaffold-taco-shop.git testProject
look in directory testProject
look at the file scaffold.log
find the string 'Not enough non-option arguments: got 0, need at least 1' in the scaffold log

🪵 Relevant log output
,0,Project taq'ified!
    ,[1,"\n><EMAIL_ADDRESS>setup\n> npm run setup:taqueria && npm run setup:app\n\n\n><EMAIL_ADDRESS>setup:taqueria\n> npm install && taq init\n\n\nup to date, audited 554 packages in 2s\n\n53 packages are looking for funding\n  run `npm fund` for details\n\nfound 0 vulnerabilities\nProject taq'ified!\n\n><EMAIL_ADDRESS>setup:app\n> cd ./app && npm install\n\n\nadded 1937 packages, and audited 1938 packages in 15s\n\n181 packages are looking for funding\n  run `npm fund` for details\n\n31 vulnerabilities (3 moderate, 19 high, 9 critical)\n\nTo address issues that do not require attention, run:\n  npm audit fix\n\nTo address all issues (including breaking changes), run:\n  npm audit fix --force\n\nRun `npm audit` for details.\ntaq <command>\n\nCommands:\n  taq init [projectDir]            Initialize a new project\n  taq opt-in                       Opt-in to sharing anonymous usage analytics\n  taq opt-out                      Opt-out of sharing anonymous usage analytics\n  taq install <pluginName>         Install a plugin\n  taq uninstall <pluginName>       Uninstall a plugin\n  taq add-contract <sourceFile>    Add a contract to the contract registry\n  taq rm-contract <contractName>   Remove a contract from the contract registry\n  taq list-contracts               List registered contracts\n  taq ligo                         This task allows you to run arbitrary LIGO na\n                                   tive commands. Note that they might not benef\n                                   it from the abstractions provided by Taqueria\n  taq compile <sourceFile>         Compile a smart contract written in a LIGO sy\n                                   ntax to Michelson code, along with its associ\n                                   ated storage/parameter list files if they are\n                                    found             [aliases: c, compile-ligo]\n  taq test                         Provided by more than one plugin. The option\n                                   --plugin is required.\n  taq start sandbox [sandboxName]  Starts a flextesa sandbox    [aliases: start]\n  taq stop sandbox [sandboxName]   Stops a flextesa sandbox      [aliases: stop]\n  taq list accounts <sandboxName>  List the balances of all sandbox accounts\n  taq deploy <contract>            Deploy a smart contract to a particular envir\n                                   onment                   [aliases: originate]\n  taq transfer <contract>          Transfer/call an implicit account or a smart\n                                   contract (specified via its alias or address)\n                                    deployed to a particular environment\n                                                                 [aliases: call]\n  taq fund                         Fund all the instantiated accounts up to the\n                                   desired/declared amount in a target environme\n                                   nt\n  taq instantiate-account          Instantiate all accounts declared in the \"acc\n                                   ounts\" field at the root level of the config\n                                   file to a target environment\n  taq clean                        Clean all the Taqueria-related docker images\n  taq create <template>            Create files from pre-existing templates\n\nOptions:\n      --version     Show version number                                [boolean]\n      --build       Display build information about the current version[boolean]\n  -p, --projectDir  Path to your project directory               [default: \"./\"]\n  -e, --env         Specify an environment configuration\n  -y, --yes         Select \"yes\" to any prompt        [boolean] [default: false]\n      --help        Show help                                          [boolean]\n\nTaqueria is currently in BETA. You've been warned. :)\n\nNot enough non-option arguments: got 0, need at least 1\n","npm WARN deprecated<EMAIL_ADDRESS>This functionality has been moved to @npmcli/fs\nnpm WARN deprecated<EMAIL_ADDRESS>Modern JS already guarantees Array#sort() is a stable sort, so this library is deprecated. See the compatibility table on MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#browser_compatibility\nnpm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-url#deprecated\nnpm WARN deprecated<EMAIL_ADDRESS>flatten is deprecated in favor of utility frameworks such as lodash.\nnpm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained\nnpm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained\nnpm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-terser\nnpm WARN deprecated<EMAIL_ADDRESS>Please see https://github.com/lydell/urix#deprecated\nnpm WARN deprecated<EMAIL_ADDRESS>https://github.com/lydell/resolve-url#deprecated\nnpm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated\nnpm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.\nnpm WARN deprecated<EMAIL_ADDRESS>Use your platform's native performance.now() and performance.timeOrigin.\nnpm WARN deprecated<EMAIL_ADDRESS>Please use @jridgewell/sourcemap-codec instead\nnpm WARN deprecated<EMAIL_ADDRESS>some dependency vulnerabilities fixed, support for node < 10 dropped, and newer ECMAScript syntax/features added\nnpm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 does not receive security updates since 2019. Upgrade to chokidar 3 with 15x fewer dependencies\nnpm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 does not receive security updates since 2019. Upgrade to chokidar 3 with 15x fewer dependencies\nnpm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.\nnpm WARN deprecated<EMAIL_ADDRESS>babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.\nnpm WARN deprecated<EMAIL_ADDRESS>Moved to 'npm install @sideway/address'\nnpm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-babel.\nnpm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.\nnpm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained\nnpm WARN deprecated<EMAIL_ADDRESS>Switch to 'npm install joi'\nnpm WARN deprecated<EMAIL_ADDRESS>This SVGO version is no longer supported. Upgrade to v2.x.x.\nnpm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Some versions have web compatibility issues. Please, upgrade your dependencies to the actual version of core-js.\n"]

🐘 How impactful is this bug?
🤔 Notable but managable
⏱ Prevalance
when you do a scaffold
💻 Operating System
Windows -> Linux VM
🕸 System Architecture
x64
🌮 Taqueria Version
v0.24.2
🌿 Node.js Version
v16.13.1
☎ Contact Details
<EMAIL_ADDRESS>Code of Conduct

[X] I agree to follow this project's Code of Conduct

no longer can be replicated, and not important enough to keep trying


Even if a skip/include directive field with an argument in them are not processed properly

Describe the bug
I created a fragment that has an optional field. That field requires a filter argument.
If I decided to use the @include directive to skip that field when not required, I still need to pass in the argument.
To Reproduce
Steps to reproduce the behavior:
1.) Create a fragment with a field that requires an argument.
fragment UserFragment on User {
    id
    relatedPosts(itemList: $itemList) @include(if: $withRelatedPosts)
}

2.) Use that fragment in query with withRelatedPosts set to false
query getUserWithoutPosts($withRelatedPosts: Boolean! = false) {
     users {
          ...UserFragment
     }
}

3.) Try to run codegen results in
      GraphQLDocumentError: Variable "$itemList" is not defined by operation "getUserWithoutPosts".

My codegen.yml config file:
overwrite: true
schema: "./schema.gql"
generates:
  src/generated/schema.ts:
    plugins:
      - "typescript"
  src/:
    preset: near-operation-file
    presetConfig:
      extension: .generated.ts
      baseTypesPath: /generated/schema.ts
    documents: 'src/**/*.gql'
    plugins:
      - "typescript-operations"
      - "typescript-graphql-request"
    config:
      exportFragmentSpreadSubTypes: true

Expected behavior
Expected behavior is that it should not require the argument since the field is omitted in this case.

Tracking in: https://github.com/dotansimha/graphql-code-generator/issues/3836


Persistent answer after 6 wrong guesses

Thanks for the unlimited Wordle games.
Suggestion: Suppose I have exhausted all the 6 guesses and then the answer (probably it is a new word for me) appears on the new screen. I had to ESC to see my guesses when investigating what I could have done better. Many times, I forget the spelling of that new word. It will be nice if the answer stays there along with our guesses like in Wordle.
Thanks,
Rajesh.

(not saying youre wrong, but as a workaround, you can now just refresh the browser and get the popup back)

yeah, for now I would just go with the workaround mentioned above


[Editors] hdbcalculationvew form based editor to be contributed to the Web IDE

hdbcalculationvew form based editor to be contributed to the Web IDE. This is needed due to the non-human readable format used (XML). The editor should have all the needed features to inspect and modify the hdbcalculationview artefact.

This issue is closed as the project is no longer maintained


Storybook: Replace deprecated knobs addon

Task Description

@storybook/addon-knobs is deprecated in favor of @storybook/addon-controls (part of @storybook/addon-essentials), which actually can also replace @storybook/addon-actions.
I've played around with this locally already to see how feasible it would be.
Switching to @storybook/addon-essentials would mean we have to update all our storybook components, which will take some time.

@swissspidy  - pea pod could def tackle this in the next few months. Would you be ok with us taking it?

@BrittanyIRL Yes absolutely! That would be fantastic.


Mount Volume Fails for EC certs while Key is selected to be eported.

Describe the bug
Mount volume always fails if I choose keys in flex volume for EC certificates.  Logs:
Events:
  Type     Reason       Age                   From                               Message
  ----     ------       ----                  ----                               -------
  Normal   Scheduled    5m15s                 default-scheduler                  Successfully assigned default/mysql-keyvault-sp to aks-agentpool-50967501-3
  Warning  FailedMount  56s (x2 over 3m12s)   kubelet, aks-agentpool-50967501-3  Unable to mount volumes for pod "mysql-keyvault-sp_default(b5d4efb8-8604-11ea-bc3e-624e96f1e750)": timeout expired waiting for volumes to attach or mount for pod "default"/"mysql-keyvault-sp". list of unmounted volumes=[private-key-volume]. list of unattached volumes=[mysql-persistent-storage private-public-key-volume private-key-volume public-key-volume cert-volume default-token-xh4x6]
  Warning  FailedMount  55s (x10 over 5m13s)  kubelet, aks-agentpool-50967501-3  MountVolume.SetUp failed for volume "private-key-volume" : mount command failed, status: Failure, reason: /etc/kubernetes/volumeplugins/azure~kv/azurekeyvault-flexvolume failed, /Users/anishramasekar/go/src/github.com/Azure/kubernetes-keyvault-flexvol/azurekeyvault-flexvolume/main.go:80 +0x129

Steps to generate Cert:

Created a CSR (where keyproperties.exportable is true, keyType": "EC")
Got this CSR signed by a third party CA
Merged the generated public key/cert in Key Vault.

Authentication used: SP
NOTE: Same flexVolume settings and steps to generate works perfectly for RSA certificate. Able to fetch Key for RSA certs, but for EC certs it fails.
Steps To Reproduce
Create an EC, mark keys as exportable.
Use SP to authenticate and try to fetch Key with Flex Volume
Expected behavior
Shoule be able to fetch Keys for EC certificates in plain text
Access mode: service principal
Kubernetes version
1.15.x

Hi @bhardwahnitish19 , a quick question. When you say "I choose keys in flex volume for EC certificates" do you mean you just need private key in the pod?
What I understand is following:

You are creating an EC "certificate" object in key vault first. You get the CSR signed and enable the certificate by uploading the signed CSR. Right?
Now the certificate object in key vault is a combination of both, the public and the private part. You need this EC key alone into your application-specific pod (preferably in PEM format). Right? Also please correct me if you need it "some other format"!

If this so, then @ritazh would this require some different enhancement to the csi-driver than the one mentioned here?

Hi @chintanr97
Please find my comments inline:
Do you mean you just need private key in the pod? I need both public & private key in pod. But, need them in different locations like /var/privatekey & /var/publickey. These must be in pem format so that the application can utilize easily without any type conversions.
To achieve this, I am trying to export key and mount it at /var/privatekey and trying to export cert to /var/publickey. (Using 2 flex volume respectively)
1. You are creating an EC "certificate" object in key vault first. You get the CSR signed and enable the certificate by uploading the signed CSR. Right? Correct
2. Now the certificate object in key vault is a combination of both, the public and the private part. You need this EC key alone into your application-specific pod (preferably in PEM format). Right? Also please correct me if you need it in "some other format"!  PEM format would be perfect for now.

Great! I understood! Hope the updated comments here help the project owners to create the required solutions.


[TwigComponent] add priority to PreMount hook

Q
A

Bug fix?
no

New feature?
yes

Tickets
n/a

License
MIT

Adds the ability to control the order in which PreMount methods are called via a priority parameter on the attribute.

Rebased & fixed conflicts.

Thanks Kevin!


Pw/fix status code validation

Tests are broken right now, will push fixed tests

Tests are broken right now, will push fixed tests


Queue + Function - Batch Size Config

Hi, thanks for this awesome package!
I can't seem to find any docs on setting the message batch size for an SQS queue's lambda handler. Is this possible?
I remember it was there in the previous version of SST as a Queue param.
Thanks!

Never mind! Found it at the second prop in queue.subscribe() as transform.eventSourceMapping.batchSize


Crashed while connecting to a locally-hosted server.

Leafish panicked while trying to load a locally-hosted server.
Steps to recreate:

Start a Minecraft server (server.jar) on 1.16.5 (3GB RAM)
Enter IP and join on Leafish

It panics, displaying: ```
"spawned 102 Zombie
thread 'spawned 102 Zombie
' panicked at 'called Result::unwrap() on an Err value: Error { kind: UnexpectedEof, message: "failed to fill whole buffer" }', src\world\mod.rs:1298:62"

I'm not sure if it's the same issue, but I got a similar error that seems to be caused by an issue in reading light data. It should be fixed in #246.

@hellomynameisaaaaaaaaaaaaa It should be working now on the latest version. Can you test it to see if this is still an issue?

@nathanruiz

@hellomynameisaaaaaaaaaaaaa It should be working now on the latest version. Can you test it to see if this is still an issue?

Not the original poster but I was subscribed to this issue as I had difficulties with crashing upon connecting as well. I can confirm that when building and running from the repo I can now successfully login to a locally-hosted server

@terrarier2111 This issue should be resolved now