@Api decorator not working properly with nest versioning

Hey guys!
I'm having some issues when creating an endpoint on nestjs + ts-rest that receives a param and has versioning configured by nest have you guys ever faced these issues?
Example:
Contract route
 getIndustryTag: {
    method: 'GET',
    path: `/industry-tags/:uid`,
    responses: {
      200: ResponseIndustryTagModel.nullable(),
    },
    summary: 'Get an industry tag',
  },

Controller
@Controller({
  version: '1',
})
export class IndustryTagsController {
  constructor(private readonly industryTagsService: IndustryTagsService) {}

  @Api(server.route.getIndustryTag)
  async findOne(
    @ApiDecorator() { params: { uid } }: RouteShape['getIndustryTag']
  ) {
        ....
    }

The end route for this would be http://localhost:3000/v1/industry-tags/uid-123 but since the contract route doesn't know about the versioning that was applied, I receive the string industry-tags has the param instead of the uid-123
Even if I apply the v1 to the contract path it will become v1/v1/industry-tags so it wouldn't help either
The only solution I can see working is managing the versioning inside the contract only and letting go of the nestjs versioning system, but I wanted to know if there are other options for this and if this is a known issue
Thanks!

This is a super interesting point, we can definitely work on this, the root problem here is that ts-rest is trying to instantiate the path for you, to avoid typos and improve dev x.
Out of interest, would this also work by adding "/v1" to your baseUrl when you call this api?
Or is having the ability to add /v1/.. in the contract important?

I believe that adding the version to the contract (whole contract and endpoint specific) would be the best approach.
This way, if you are trying to achieve a full type-safety project with API and frontend both of them will know which version of the endpoint they are using just by accessing the contracts information.
So maybe something like this would work:
contract.router({
  version: '2':
  getIndustryTag: {
      method: 'GET',
      path: `/industry-tags/:uid`,
      responses: {
        200: ResponseIndustryTagModel.nullable(),
      },
      summary: 'Get an industry tag',
      version: '1'
  },
)}

What do you think?

I've been playing with this locally for a while, and seeing how it's played out in reality.
One issue we might face is if we add "version" to the contract, not all versioning across frameworks, or even the same framework will be consistent
Some APIs may have one of the following

A path like /vX/...
A path like /version-X/...
A versioning header

I'm not entirely sure if we want to work around this, in the worst case, for your use case I imagine in the front end you could postfix the baseUrl with "/v1"? Honestly, I'm not sure about this one, so I'm happy to implement something if we can agree on a solution

Related to #79, happy to hear you both out for suggestions on implementation here.
@Gabrola have you considered this whilst looking at the Nest implementation at all?

@PedroFonseca17, since you submitted this issue has any other workaround helped, or is this still something you'd want from ts-rest?

For now, closing as "won't fix", but let's loop back around to this, especially for the 4.0 release.

For now, closing as "won't fix", but let's loop back around to this, especially for the 4.0 release.

nest js support is too bad actually

nest js support is bad

so when is v4.0 planned ?
would expect some thing better for nest js integration it should have been easy specially for nest js as it embrases abstraction


Option to ignore hidden dirs

Just as the title suggests, can an option be provided to ignore hidden dirs? I tried
$ gdu -i /home/name/.[a-zA-Z]*
Error: accepts at most 1 arg(s), received 138

It did not work.

Nice idea! I will add it to the next release.

Thanks. Look forward to it.

Thanks for including a new option -H. I just noticed a typo there. It is beginning rather than beggining.

Thanks for including a new option -H. I just noticed a typo there. It is beginning rather than beggining. Look forward to the release with this option/

Just released. Thanks for notice, I will fix it in the next patch version.


fix: ManyToMany Scrolling

Problem:

ManyToMany field in changeview is broken

For issue #376
Solution:

Create a new class (MULTIPLE_SELECT_CLASSES)

remove the truncate class
add the overflow-y-auto

Hey, thanks for PR which pointed me at the right direction. I created new PR #421 fixing the issue.


[FAQ] How to join list into string

Feature Request
Very common user case: When we want to join our list into a string with some separator (like a comma ",").
L = ['a', 'b', 'c']
S = L.join(',')

How to get list of all functions for string and list?

Hello @borgius You can see the string spec https://www.kcl-lang.io/docs/reference/lang/spec/datatypes#string to find more information.

Hey @Peefy , made a Pr #373

Closed by #373


Add selection answer type and its options

What problem does the pull request solve?
Adds an answer_settings JSON column to the pages object. When returned from the API it will look something like this:
"answer_settings": {
  "allow_multiple_answers": true,
  "selection_options": [
    {"name": "Option 1"},
    {"name": "Option 2"},
  ]
}

The intent is for this field to be flexible for different answer types - for example, this field will likely end up containing flags for autofill related settings for other fields.
Trello card: https://trello.com/c/bSqKTDu8/224-add-selection-question-type-to-api
Checklist

[x] I've used the pull request template
[x] I've linked this PR to the relevant issue (if mission work)
[x] I've written unit tests for these changes (if code change)
[n/a] I've updated the documentation in (If any documentation requires updating)

[n/a] README.md
[n/a] Elsewhere (please link)

@thomasiles It's reasonably easy I think, ActiveResource does a good job reading it and mapping it to a class, setting up the fields in a form is slightly fiddlier.


Make the "Join Me" CTA linkable

Issue
The Join Me section is designed nicely, but is not clickable. We've received feedback that users are expecting to be directed to the sign up portion.
Request
Make the "Join Me" call to action clickable. Clicking on it should direct the user to the sign up portion.

I'll take this. :)

Even i also want to work on this issue.

@kexline4710 I want to work on this

@ if pull request solves your issue please accept it :)

Thanks everyone. I accepted the first one submitted!


CrowdHuman_val_fbox.json and CrowdHuman_val_vbox.json

Could you please release the CrowdHuman_val_fbox.json and CrowdHuman_val_vbox.json?

Hi. I am not sure whether this is still open or not, but I implemented a CrowdHuman (CH) ingestion script myself. It is, I believe, fairly straightforward to use. It is a command-line tool that takes the CH annotation file and produces a COCO-like annotation JSON file out of it. Subsequently, it can be easily loaded into the SiamMOT pipeline for training. It will behave as an image-based dataset, thus, all the necessary augmentations to fake the tracking process will seamlessly work. If there are any questions or suggestions, feel free to contact me.
ingest_crowdhuman.txt

Hi @mondrasovic
Could you please give me the similar code that could takes the MOT20 file as input and produces a COCO-like annotation JSON file for training?
Best regards.


Chorus effect inspector update voices automatically

I update property "voice_count" in audio_effect_chorus.cpp to close #20003 issue.

#31627

PR title on the website and the commit message should stand on their own, without people having to check what 31627 is about.

Force pushed an amend to make the commit message more explicit.

Thanks!

Cherry-picked for 3.2.4.


Remote sort reset/clean broken by changes made for #1752.

The changes made for Issue #1752 breaks the ability to remove/reset a sort when remote sorting is used.
Prior to this change, clearing a sort used to work by:

Using Table.cleanSort(), and
Setting SortOrder = undefined, SortName = '' in the table options.

Doing this with 4.2.0 (I would assume in 4.1.5 - I just tried upgrading from 4.1.4) results in BootstrapTable throwing the error: "The type of sort field and order should be both with String or Array".
The changes made for issue #1752 do not take into account/provide any way to reset the current sort without triggering the above error. i.e. TableDataStore.setSortInfo(sortOrder, options.sortName) enforces that sortOrder must never be undefined.

@dawnmist when you got the error?  call cleanSort? or set sortName and sortOrder? and are you using the remote sort right?
Regards,
Allen

Yes, using remote sorting.
Yes, occurs when Option.SortOrder = undefined and Options.SortName = ''.
This commit introduced the problem, because the function it calls validates that SortOrder must be the same type as SortName - but when there is no sort SortOrder (previously) was undefined. So the test fails and throws the Error "The type of sort field and order should be both with String or Array". https://github.com/AllenFang/react-bootstrap-table/commit/06b356021e3364d0048f8aba8e934e07bb7a02fc

@dawnmist I see, I think the sortName is supposed to be undefined right? i know this patch will break your program and make you feel this might be a bug. Yes, that's a problem that I didn't handle it well at before. Anyway, I'll suggest to clean the sort status by undefined in this circumstances.
A example I used it before: https://github.com/AllenFang/react-bootstrap-table/blob/master/examples/js/sort/manage-sort-external-table.js#L28-L29
Please let me know if you have any question or idea, thanks a lot !!!


Remove: 5AAB TV

Channel ID (required)
5AABT.ca
Reason (required)
Not a TV channel
Notes
No response

Why do you think so?


Issue 108 Split-off prepare- from publishGhPages

Step 1 on #108
I've also added a .editorconfig file which defines a coding style which can be used by one of the many EditorConfig plugins, see http://editorconfig.org/ for all details.
I've now configured it to use tab indents for java/groovy files, as I noticed that you used tab-indents.

Looks good.


HHH-14456 Add DialectResolutionInfo#unwrap to be able to detect Cockroach

https://hibernate.atlassian.net/browse/HHH-14456

Looks fine to me.


Morning Session Peer to Peer code

Peer to peer code review

Issue on GithubPages Deployment ( Suggestion to include index.html rather than home.html )

Thank you very much for the feedback, your suggestion solved the problem.
The issue has been resolved in the 'Review' branch.


Question: why not run etcd in docker ?

Did you consider this option ?
What are your thoughts about it ?
Thanks ! Great guide btw ;)

I'm not sure if this is even possible, e.g. during startup this might turns into a chicken or the egg problem. Wouldn't recommend this, but I get the idea. Please let me know your findings 👍


Cut a new Spark-Kubernetes 2.2 release

Prerequisites:

[x] Finish the rename (https://github.com/apache-spark-on-k8s/spark/pull/497)
[x] Remove deprecated configuration now (https://github.com/apache-spark-on-k8s/spark/pull/501)
[x] Update RBAC documentation (https://github.com/apache-spark-on-k8s/spark/issues/500)
[x] Update poms for 2.2 release 0.4.0 (https://github.com/apache-spark-on-k8s/spark/pull/508)

Post release:

[ ] Update documentation to point to 2.2 docker images. (currently points at 2.1) (https://github.com/apache-spark-on-k8s/userdocs/issues/16)

cc @mccheah @ash211 @erikerlandson @apache-spark-on-k8s/contributors

SparkR support into this new release, as we push upstream?

I think it would be the next one.  These are pretty separate and a release
is long overdue. The plan was to cut the release today - was doing some
manual testing. Should be able to tag the branch and start merging other
stuff in right after.
On Sep 24, 2017 8:55 AM, "Ilan Filonenko"<EMAIL_ADDRESS>wrote:

SparkR support into this new release, as we push upstream?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache-spark-on-k8s/spark/issues/499#issuecomment-331719064,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA3U50tpNIWXbvPnJzMnYBWret1I78qCks5slnuEgaJpZM4PdkRl
.

Built and pushed all v2.2.0-0.4.0 images. Pending tests. It would be great if folks can run some tests against them.

Any potential concerns that @liyinan926 images built differently than @foxish build script?

I just tried using the script to build and push. The images should be good.

New release published at https://github.com/apache-spark-on-k8s/spark/releases/tag/v2.2.0-kubernetes-0.4.0.

@liyinan926 for posterity can you post the build commands you used here.  The ones I used to cut the previous release are here

Sure, the commands I used are:
./dev/make-distribution.sh --pip --tgz -Pmesos -Pyarn -Pkinesis-asl -Phive -Phive-thriftserver -Pkubernetes -Phadoop-2.7 -Dhadoop.version=2.7.3
./dev/make-distribution.sh --pip --tgz -Pmesos -Pyarn -Pkinesis-asl -Phive -Phive-thriftserver -Pkubernetes -Phadoop-2.7 -Dhadoop.version=2.7.3 -Phadoop-provided

Published the release announcement to spark dev and user lists


GA splunk-otel-android

Note: This could be understood as a continuation of #133.
I have done front-loaded a lot of evidence toward this GA effort in https://signalfuse.atlassian.net/browse/APMI-3264.
Which GDI repository do you wish to GA?
splunk-otel-android
Does the repository follow the latest tagged minor release in GDI specification?

[x] Has an appropriate maintainers team.
[x] Permissions set correctly.
[x] Branch protection in place.
[ ] Dependencies appropriately locked down.
[x] GitHub Applications set up per spec.
[x] Follows the configuration requirements, if appropriate.
[x] Follows the semantic convention requirements, if appropriate.
[x] Required Files in place.

[x] CHANGELOG.md
[x] CODE_OF_CONDUCT.md
[x] CONTRIBUTING.md
[x] .github/CODEOWNERS
[x] LICENSE
[x] README.md

[x] Build status badge
[x] Getting started
[x] Troubleshooting
[x] Link to official Splunk docs
[x] License information

[x] SECURITY.md

[ ] Releases done to spec.
[x] Type specific requirements (remove what doesn't apply)

[x] Real User Monitoring Library

[x] Documents all supported configuration parameters.
[x] Documents how to configure manual instrumentation.
[x] Documents supported instrumentation.
[x] Documents supported platforms.

How long has the GDI repository been public?
At least 411 days.
Is the repository known to be used today?
YES.
Is there a date by which this approval is needed?**
No.
Additional context
The Splunk Android SDK supports the RUM product and has not undergone significant destabilizing changes in many many months. It has been used in production by customers for months, and we believe that it is compliant with the latest GDI specification. We are intending to donate the code in this SDK to the OpenTelemetry community.

I believe that somebody other than the Java team folks should take a look at it - ping @signalfx/gdi-specification-maintainers

I will take it. I will probably start working on it next week,

These are the only issues I have found

https://github.com/signalfx/splunk-otel-android/issues/343
https://github.com/signalfx/splunk-otel-android/issues/345

This is really great 👏

I certify that meets v1.4.0 of the GDI Spec.
I guess it is time to remove the "BETA" notice and publish a v1.0.0 release of https://github.com/signalfx/splunk-otel-android

I certify that splunk-otel-android meets v1.4.0 of the GDI Spec.
I guess it is time to remove the "BETA" notice and publish a v1.0.0 release of https://github.com/signalfx/splunk-otel-android


Add back process method for CJS support

Closes https://github.com/mihar-22/svelte-jester/issues/66 https://github.com/mihar-22/svelte-jester/issues/65 https://github.com/mihar-22/svelte-jester/issues/63
It turns out that processAsync is not supported in CJS mode, so we still need the old process method

Can you find some reference in the docs from Jest that this is really the case.

I think, ESM support in Jest is experiment (according to docs and because of all the flags involved), which means it is still written in CJS. Now asynch transformers were introduced in v27. To me this means Async support in CJS should be superior.
I think we have to look for the root of the problem, once we are sure the transpilation worked.

@benmccann pls import import { execSync } from 'child_process' in transformer.js file.
Another thing: For some reason I need to add the jsdom environment in package.json to run my tests, like:
"scripts": {   "test": "jest --env=jsdom" }
In svelte-jester 1.8.2 this was not necessary, do you know why is that?

Can you find some reference in the docs from Jest that this is really the case?

Unfortunately the docs for processAsync are quite lacking (there were none until I added some and I didn't know about that restriction, so I didn't mention it), but here's a reference for it: https://github.com/facebook/jest/issues/11458#issuecomment-850454026

Another thing: For some reason I need to add the jsdom environment in package.json to run my tests, like:
"scripts": { "test": "jest --env=jsdom" }
In svelte-jester 1.8.2 this was not necessary, do you know why is that?

That's not related to svelte-jester. It's because the default environment changed to Node in Jest 27

pls add import { execSync } from 'child_process' in transformer.js file.

done. thanks for catching that! I'd tested this, but I guess not with the preprocess option, so I missed that

Can you find some reference in the docs from Jest that this is really the case?

Unfortunately the docs for processAsync are quite lacking (there were none until I added some and I didn't know about that restriction, so I didn't mention it), but here's a reference for it: facebook/jest#11458 (comment)

Another thing: For some reason I need to add the jsdom environment in package.json to run my tests, like:
"scripts": { "test": "jest --env=jsdom" }
In svelte-jester 1.8.2 this was not necessary, do you know why is that?

That's not related to svelte-jester. It's because the default environment changed to Node in Jest 27

Hm, this refers only to the require being synchronously. But this is just the module resolution, not the execution of the function. If this is true, then no one using a CJS library would be able to use an async function. Which I don't think is true.
I will have a look at the code from jest calling the transformer now.

This is the code loading the transformer: https://github.com/facebook/jest/blob/master/packages/jest-transform/src/ScriptTransformer.ts#L268
It is a check, if the export is createTransformer or process*
Another example is this e2e-test: https://github.com/facebook/jest/blob/master/e2e/transform/async-transformer/my-transform.cjs

Hm, still unsure. I am preparing a test case to validate this.
Jest references async transformers a lot with ESM:

https://github.com/facebook/jest/issues/9504

https://github.com/facebook/jest/pull/11191

pls add import { execSync } from 'child_process' in transformer.js file.

done. thanks for catching that! I'd tested this, but I guess not with the preprocess option, so I missed that

Thanks, tested and new changes on the branch are working

@sebastianrothe I don't know the Jest internals well enough to have any idea of whether there might be a way to get processAsync to run in CJS mode.
The way the code is being called without this PR is:
    at ScriptTransformer.transformSource (node_modules/@jest/transform/build/ScriptTransformer.js:584:7)
    at ScriptTransformer._transformAndBuildScript (node_modules/@jest/transform/build/ScriptTransformer.js:759:40)
    at ScriptTransformer.transform (node_modules/@jest/transform/build/ScriptTransformer.js:816:19)
    at Runtime.transformFile (node_modules/jest-runtime/build/index.js:1463:47)
    at Runtime._execModule (node_modules/jest-runtime/build/index.js:1380:34)
    at Runtime._loadModule (node_modules/jest-runtime/build/index.js:1034:12)
    at Runtime.requireModule (node_modules/jest-runtime/build/index.js:866:12)
    at jestAdapter (node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:79:13)
    at runTestInternal (node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (node_modules/jest-runner/build/runTest.js:481:34)

(line number might be off by a line or two as I'm running a locally-modified version)
There are async versions of many of these methods. I.e. transformSourceAsync, _transformAndBuildScriptAsync, transformAsync, transformFileAsync. But for some reason those aren't being called.
Maybe you can file an issue to ask in the Jest issue tracker or something. I'm afraid I don't have any insight other than seeing the comment where he said "async transform is only supported when using ESM"

@sebastianrothe I don't know the Jest internals well enough to have any idea of whether there might be a way to get processAsync to run in CJS mode.
The way the code is being called without this PR is:
    at ScriptTransformer.transformSource (node_modules/@jest/transform/build/ScriptTransformer.js:584:7)
    at ScriptTransformer._transformAndBuildScript (node_modules/@jest/transform/build/ScriptTransformer.js:759:40)
    at ScriptTransformer.transform (node_modules/@jest/transform/build/ScriptTransformer.js:816:19)
    at Runtime.transformFile (node_modules/jest-runtime/build/index.js:1463:47)
    at Runtime._execModule (node_modules/jest-runtime/build/index.js:1380:34)
    at Runtime._loadModule (node_modules/jest-runtime/build/index.js:1034:12)
    at Runtime.requireModule (node_modules/jest-runtime/build/index.js:866:12)
    at jestAdapter (node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:79:13)
    at runTestInternal (node_modules/jest-runner/build/runTest.js:389:16)
    at runTest (node_modules/jest-runner/build/runTest.js:481:34)

(line number might be off by a line or two as I'm running a locally-modified version)
There are async versions of many of these methods. I.e. transformSourceAsync, _transformAndBuildScriptAsync, transformAsync, transformFileAsync. But for some reason those aren't being called.
Maybe you can file an issue to ask in the Jest issue tracker or something. I'm afraid I don't have any insight other than seeing the comment where he said "async transform is only supported when using ESM"

I did ask him in the Jest issue.
Nontheless, I do have a test which consumes the CJS build and it is passing. Let my try to create a new Svelte project and try it there as well.

https://github.com/facebook/jest/issues/11458#issuecomment-915001052 means, we need to bring the old code back for the CJS version.
I would like to bring back the tests for the syncTransformer as well. I will try to add those commits tomorrow.

I'm having some problems with the typescript test. We should really use the Github Actions running our tests on every PR.

We could possibly merge this PR as is (all tests are passing with it) to get people unbroken and then handle adding the tests in a new PR. I agree it'd be nice to run the tests on the CI

I'm having some problems with the typescript test. We should really use the Github Actions running our tests on every PR.

Found it.

We could possibly merge this PR as is (all tests are passing with it) to get people unbroken and then handle adding the tests in a new PR. I agree it'd be nice to run the tests on the CI

Well, the tests run only for the async build. So I think it is important to have them pass for the sync build as well.
I will have some more time for this in the evening (in 3 hours).

We could possibly merge this PR as is (all tests are passing with it) to get people unbroken and then handle adding the tests in a new PR. I agree it'd be nice to run the tests on the CI

I added my commits to yours. Can you please have a look: https://github.com/mihar-22/svelte-jester/pull/71/files/4c72f25507b1fc5c457ef2da80bd62524990cfa5..75c83c690de2156f37fc577912c81fc05d2937df

removed esbuild, because with Jest supporting ESM it is not needed anymore for the tests
svelte-preprocess > 4.7.4 gave me errors on the test with Typescript -> needs more investigation, because they changed the typescript support
for cjs, we can't export const process because this variable is already taken by node and we need it to pass env-vars to the child process


Implement simple reactive interface for reads.

@dpasek-senacor, could you also add a test verifying if the changes helped and it's possible to enumerate through the long stream (e.g. a few thousand events)?

Done here: https://github.com/EventStore/EventStoreDB-Client-Java/pull/106


FIX Don't update LastEdited on skipped writes

fixes #3642

In my testing of this I've noticed that the FixtureBlueprintTest has a test that tests a feature that implies the setting of Created in fixture files will be persisted in the DB (but they won't).
The FixtureBlueprintTest::testCreateWithClassAncestry checks that a fixture written with data which specifies a Created field will report that field back - however this checks the current object in memory and not the data written to the DB or that is pulled out when a DataObject::get() is performed...
This leads to a false positive implying the Created date can be set in a fixture when it cannot.


User reports being unable to drag sprites - iPad

TL;DR: A user has reported being unable to use a finger to change sprite positions on iPads at school.
We have only 1 report of this so far but since it would be a serious issue if repro-able, I figured I'd create an issue for it.
From freshdesk no. 80926:

I have been using Scratch Jr for several years on the iPads in my school. Suddenly, on some of the iPads, you cannot reposition a sprite on the stage using finger touch. All the command blocks work as normal and you can add, delete sprites etc.

I asked:

Thanks for reporting this issue. Can you send us the version number for the browser and iOS version you're using, as well as whatever information you can find about the model/ version of the tablets?
Is this happening consistently on all the iPads, or only on some of them?
About when did the issue start, and when was the last time you updated Scratch Jr or the iPad software?

Sounds like it may have resolved itself?

Thanks for your very prompt response.  whilst reading your email, I noticed that you asked about which version of Scratch was installed, so I updated to the latest version for one of the iPads that wasn't working.  Bingo! it works fine now.  I think our device management program (Lightspeed) did not update all the iPads to the latest version of Scratch).  Thanks for prompting me in the right direction.

I'll close this until we get further reports or an update from this user (that it isn't actually working now). 👍


error: RPC failed; HTTP 408 curl 18 HTTP/2 stream 7 was reset

Thanks for this action!
My workflow has been working for ~2 years. Last week, this error appeared when I try to move a 2.1MB file to another repo. The issue and work-around are discussed here.
tldr; This action needs the following added:
git config --global http.version HTTP/1.1

I'm currently using this fork that has the fix work-around in place.
PR #78 looks like it is attempting to fix the same issue.

@nseidle
My fork has the fix in place + git-lfs support

Thanks for this action!
My workflow has been working for ~2 years. Last week, this error appeared when I try to move a 2.1MB file to another repo. The issue and work-around are discussed here.
tldr; This action needs the following added:
git config --global http.version HTTP/1.1

I'm currently using this fork that has the ~fix~ work-around in place.
PR #78 looks like it is attempting to fix the same issue.

git config --global http.version HTTP/1.1


[BUG] Locale files are not uploading to Shopify

Describe the bug
When using either packer start or packer watch, modifying existing locale files locally does not upload changes to the Shopify theme. New files added to the locales folder do not upload either.
To Reproduce
Steps to reproduce the behavior:

packer start
make changes in en.default.json and save
refresh the browser

Expected behavior
"Uploading to Shopify..." should appear in the console and changes should be seen in the browser.
System (please complete the following information):

OS: Mac OSX 11.3
Node Version v12.22.1
NPM Version 6.14.12
Browser chrome 90.0.4430.93

Packer (please complete the following information):

Version v2.1.5

Theme (please complete the following information):

Shopify Packer theme True
Version 1.0.13

I have the same issue. Is there any update on this?

Looks like this is also true of settings_schema.json – looks like the command only looks for Liquid files being updated. Would likely be worth observing JSON files too, as these are changed frequently during theme development.


Save connection request only when getting 200 code

Currently SDX controller saves all connection requests, but failed connection request doesn't have breakdown, and cannot be deleted. This PR only saves connection request when getting 200 code.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 52.57%

Totals

Change from base Build<PHONE_NUMBER>1:
0.07%

Covered Lines:
769

Relevant Lines:
1486

💛  - Coveralls


Avernic defender is not properly outlined

Title really sums this one up, the inventory tag does not properly outline the avernic defender
https://gyazo.com/f7817dabd582d901bb775c01dc675d71

fixed in 197c667edfb9ffc1500bb6e721eb0d0e4194dc3e


Improvements

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 15 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.3%) to 98.387%

Totals

Change from base Build<PHONE_NUMBER>:
2.3%

Covered Lines:
61

Relevant Lines:
62

💛  - Coveralls


versions

In the master branch, the project version in the pom.xml is 2.1.0-SNAPSHOT, which is behind already released 2.1.1. What does it mean?
PS:
Is bridgedb2.x the stable release branch? The version there is still 2.1.0-SNAPSHOT as well.
Was it merged into the master?..
Do you want to eventually deploy/release to e.g., OSSRH public maven repository (and then to Maven Central)?

Hi Igor, more comments to follow...

At this moment Ant is still the official build system (which may cause the pom.xml version to be outdated)
master branch is under development (but will fix the version shortly)
once Maven is the main build system, then I do like to push to a central Maven repos (help welcome!)
master should be uptodate with respect to bridgedb2.x (just checked that the latest patches are included)
master shows a few regressions on Jenkins: https://jenkins.bigcat.unimaas.nl/job/bridgedb (with Ant)/lastCompletedBuild/testReport/ and https://jenkins.bigcat.unimaas.nl/job/bridgedb/lastCompletedBuild/testReport/

For now, I fixed the version numbers. Sorry for the inconvenience.


[keymgr/dv] Update testplan and covergroup plan

We can review this on V2 review meeting
Signed-off-by: Weicai Yang<EMAIL_ADDRESS>

Hi all, thanks for attending the keymgr V2 review. I have updated the testplan as discussed. Please take a look


Cocoapods latest version

Hi. Do you still support Cocoapods for installation? The latest version that shows up as available is 7.6.6 (https://cocoapods.org/pods/Bond) but that is quite an old release, from December 2019.
Thanks!

Do we have any update on Cocopods version issue? Is this library still being maintained by the cummunity?


Updated old PSoC 6 implementation to use hal_infineon/mtb-pdl-cat1

Updated old PSoC 6 implementation to use hal_infineon/mtb-pdl-cat1
This PR is reflected changes according to Task#1 for RFC#42883
This PR depend on: https://github.com/zephyrproject-rtos/hal_infineon/pull/2
NOTE: after merge hal_infineon, the west.yaml should be finally updated.

It seems that PDL API not changed too much since version 3.1.0. Is that correct?

yes, the interface was not changes, but 3.1.0 is a little bit old (from PSoC Creator).
gpio  1.20.1 ->  1.50.1  gpio changelog
uart/spi   2.20 -> 2.80   scb changelog
you can find change log for each driver in online documentation https://infineon.github.io/mtb-pdl-cat1/pdl_api_reference_manual


Error: spawn useradd ENOENT

Reference http://ldapjs.org/guide.html to add user, but command report error:
/etc/passwd LDAP server up at: ldap://<IP_ADDRESS>:1389
node:events:368
      throw er; // Unhandled 'error' event
      ^

Error: spawn useradd ENOENT
    at Process.ChildProcess._handle.onexit (node:internal/child_process:282:19)
    at onErrorNT (node:internal/child_process:477:16)
    at processTicksAndRejections (node:internal/process/task_queues:83:21)
Emitted 'error' event on ChildProcess instance at:
    at Process.ChildProcess._handle.onexit (node:internal/child_process:288:12)
    at onErrorNT (node:internal/child_process:477:16)
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawn useradd',
  path: 'useradd',
  spawnargs: [ '-m', '-c', 'Created via ldapadd', '-s', '/bin/bash', 'ldapjs' ]
}

node version: v16.13.1
npm version: 8.1.2

Please provide a minimal reproduction.

I'm sorry. It isn't clear what I should be doing with that example. It starts a server and nothing else. But it looks like you are using child_process to spawn an external command, and that is what is generating the error. The error you posted originally is stating that the useradd binary does not exist. Your spawn call should include an environment where the useradd binary is in the PATH.

👋
On February 22, 2023, we released version 3 of this library. As a result, we are closing this issue/pull request.
Please see issue #839 for more information, including how to proceed if you feel this closure is in error.


sidecar: trim service ports based on egress listeners and merge services [DEBUG/DONOT MERGE/REVIEW]

@ramaraochavali: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test lint_istio

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

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

Test name
Commit
Details
Rerun command

lint_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test lint_istio

pilot-e2e-envoyv2-v1alpha3_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test pilot-e2e-envoyv2-v1alpha3_istio

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

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

Test name
Commit
Details
Rerun command

lint_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test lint_istio

pilot-e2e-envoyv2-v1alpha3_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test pilot-e2e-envoyv2-v1alpha3_istio

integ-new-install-k8s-tests_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test integ-new-install-k8s-tests_istio

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

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

Test name
Commit
Details
Rerun command

lint_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test lint_istio

pilot-e2e-envoyv2-v1alpha3_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test pilot-e2e-envoyv2-v1alpha3_istio

integ-new-install-k8s-tests_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test integ-new-install-k8s-tests_istio

pilot-multicluster-e2e_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test pilot-multicluster-e2e_istio

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

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

Test name
Commit
Details
Rerun command

pilot-e2e-envoyv2-v1alpha3_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test pilot-e2e-envoyv2-v1alpha3_istio

integ-new-install-k8s-tests_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test integ-new-install-k8s-tests_istio

pilot-multicluster-e2e_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test pilot-multicluster-e2e_istio

lint_istio
4e4a82cb00579642ffc06ee78187bf75a28a1956
link
/test lint_istio

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

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

Test name
Commit
Details
Rerun command

pilot-multicluster-e2e_istio
506fc46c2d899b36fd10ec1b1284deccb34700c2
link
/test pilot-multicluster-e2e_istio

lint_istio
4e4a82cb00579642ffc06ee78187bf75a28a1956
link
/test lint_istio

integ-new-install-k8s-tests_istio
4e4a82cb00579642ffc06ee78187bf75a28a1956
link
/test integ-new-install-k8s-tests_istio

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

@ramaraochavali: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
fea2e95453b7876e076069318e12040a678c6799
link
/test integ-new-install-k8s-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
fea2e95453b7876e076069318e12040a678c6799
link
/test integ-new-install-k8s-tests_istio

integ-istioio-k8s-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test integ-istioio-k8s-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-istioio-k8s-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test integ-istioio-k8s-tests_istio

integ-new-install-k8s-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test integ-new-install-k8s-tests_istio

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

/test pilot-multicluster-e2e_istio

/retest

@rshriram Found a minor issue with the code with the config used in e2e tests. I pushed with out my fix and it failed. With fix it passed consistently four times. I have added additional test to cover that.
@howardjohn PTAL, FYI. Sorry for the noise

@ramaraochavali: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test integ-new-install-k8s-tests_istio

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

/test pilot-multicluster-e2e_istio

/cherrypick release-1.4

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

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test integ-new-install-k8s-tests_istio

unit-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test unit-tests_istio

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

/test pilot-multicluster-e2e_istio

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

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test integ-new-install-k8s-tests_istio

unit-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test unit-tests_istio

pilot-multicluster-e2e_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test pilot-multicluster-e2e_istio

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

/test pilot-multicluster-e2e_istio

/test pilot-multicluster-e2e_istio

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

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test integ-new-install-k8s-tests_istio

unit-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test unit-tests_istio

pilot-multicluster-e2e_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test pilot-multicluster-e2e_istio

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

/test pilot-multicluster-e2e_istio

/test pilot-multicluster-e2e_istio

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

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test integ-new-install-k8s-tests_istio

unit-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test unit-tests_istio

pilot-multicluster-e2e_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test pilot-multicluster-e2e_istio

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

/test pilot-multicluster-e2e_istio

/test pilot-multicluster-e2e_istio

I ran /test pilot-multicluster-e2e_istio test multiple times today and it passed all the time except in two cases for unrelated reasons. So @rshriram if you are good with the code changes, I will rerun the unit test (did not do because - it might merge).

/test pilot-multicluster-e2e_istio

/test pilot-multicluster-e2e_istio

/test pilot-multicluster-e2e_istio

/test pilot-multicluster-e2e_istio

/test pilot-multicluster-e2e_istio

/test pilot-multicluster-e2e_istio

/test pilot-multicluster-e2e_istio

/retest

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

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test integ-new-install-k8s-tests_istio

unit-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test unit-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-new-install-k8s-tests_istio
e12c032ee37daa5978a7f063c5bace17ff2ccce9
link
/test integ-new-install-k8s-tests_istio

integ-istioio-k8s-tests_istio
841ea7f119c1d31fef04f817c90a28caae77bcb0
link
/test integ-istioio-k8s-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-istioio-k8s-tests_istio
841ea7f119c1d31fef04f817c90a28caae77bcb0
link
/test integ-istioio-k8s-tests_istio

integ-new-install-k8s-tests_istio
841ea7f119c1d31fef04f817c90a28caae77bcb0
link
/test integ-new-install-k8s-tests_istio

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

@rshriram: #18323 failed to apply on top of branch "release-1.4":
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	pilot/pkg/model/sidecar.go
M	pilot/pkg/model/sidecar_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/model/sidecar_test.go
CONFLICT (content): Merge conflict in pilot/pkg/model/sidecar_test.go
Auto-merging pilot/pkg/model/sidecar.go
CONFLICT (content): Merge conflict in pilot/pkg/model/sidecar.go
Patch failed at 0001 trim ports from service

In response to this:

/cherrypick release-1.4

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


[CSUB-98] Rework register transfer

The "0" transaction ID weirdness is still here, I'd like to remove it once we make it so exempt doesn't require a dummy transfer (CSUB-112).

Codecov Report

Merging #128 (b1ab103) into dev (1f7b23e) will decrease coverage by 0.16%.
The diff coverage is 81.45%.

@@            Coverage Diff             @@
##              dev     #128      +/-   ##
==========================================
- Coverage   56.75%   56.58%   -0.17%
==========================================
  Files          27       27
  Lines        4946     4971      +25
==========================================
+ Hits         2807     2813       +6
- Misses       2139     2158      +19

Impacted Files
Coverage Δ

pallets/creditcoin/src/lib.rs
78.06% <40.00%> (-4.26%)
:arrow_down:

pallets/creditcoin/src/helpers.rs
91.48% <95.00%> (+4.21%)
:arrow_up:

pallets/creditcoin/src/tests.rs
99.35% <100.00%> (+<0.01%)
:arrow_up:

pallets/creditcoin/src/types.rs
66.52% <0.00%> (-0.86%)
:arrow_down:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Resume marc_liberation:recap_dump

https://github.com/pulibrary/marc_liberation/blob/cd7226c1e3b97a630c5e334dc1004916e985d0ef/config/schedule.rb#L75

There is a PR out for review to fix the problem with the SCSB exports. Once the voyager_helpers PR is merged, updates need to be gotten from today's date (February 17, 2021).

Following on @mzelesky comment ^ when we update the voyager helpers in marc_liberation, ran the rake task with 02-17-2021 timestamp.
Then uncomment the rake task and let it run daily. https://github.com/pulibrary/marc_liberation/blob/cd7226c1e3b97a630c5e334dc1004916e985d0ef/config/schedule.rb#L75

This was tested and resumed. Now it's paused again because of alma migration.


Failed creating descriptor heap in Unreal Engine 5 D3D12

Description

D3D12_DESCRIPTOR_HEAP_DESC.NumDescriptors 1000016, but the limit for shader visible view heaps is 100000.
Steps to reproduce

Start UE5 Editor
Start Capture anywhere
Open Capture in Renderdoc

Environment
Github Source Unreal Engine main branch

RenderDoc version: v1.29
Operating System: Windows 11
Graphics API: D3D12

I'm currently utilizing the Unreal Engine ue5-main branch, with the latest commit dating back to August 26, 2023.

I am using an RTX 4070 Ti. I apologize for not being able to switch the GPU for testing purposes.

OK, can you please test on the stable version 5.3.1 using a test/demo project to see if it reproduces there. If it does, can you please go to Help -> Send Error Report as well as selecting to include the capture with the report.

Please don't test with a locally built version of UE5, please use one of the stable binary releases since that's the only way I can reproduce the same thing that you're testing

Oh. I get it, I happen to have a binary 5.3.1 version in local.

Please don't test with a locally built version of UE5, please use one of the stable binary releases since that's the only way I can reproduce the same thing that you're testing

😓 The test results are the same in the binary 5.3.1 UE version.

I don't see any report from you. Did you send it like I asked in the earlier comment?

I apologize for responding late. I have just sent an error report. Link: https://renderdoc.org/bugreporter/report/WslsuFqDC8dYl

Can you go to a recent windows SDK (or install one as needed), and run dxcapsviewer.exe? The path should be something like C:\Program Files (x86)\Windows Kits\10\bin\10.0.22000.0\x64\dxcapsviewer.exe though the version number after the bin may be different. Can you then expand DXGI Devices, your 4070 Ti, and share the value for 'Resource Binding'? It should be Tier 3.
Something is strange because on my 4070 I can open the capture fine and as a tier 3 device it should be able to create a heap with more than 1000000 descriptors fine - the exact limit is indeterminate but it should be large enough for 1000016.

Sure.

It seems that the resource binding is at Tier 3.

Can you run this program and let me know what the output is? I've included the source as well as an exe if you'd prefer to compile it yourself, it also includes the D3D12 dlls from the 1.711.3 preview SDK to test a query for max heap size.
For me on my 4070 I get:
Resource tier is: 3
Max heap size is: 1000000
Created heap with 1000000
Created heap with 1000001
dbg message 0: ID3D12Device::CreateDescriptorHeap: D3D12_DESCRIPTOR_HEAP_DESC.NumDescriptors 1000001, device supports D3D12_RESOURCE_BINDING_TIER_3 which supports shader visible descriptor heap size larger than 1000000, but this may fail to create on some TIER_3 hardware.
Created heap with 1000008
dbg message 0: ID3D12Device::CreateDescriptorHeap: D3D12_DESCRIPTOR_HEAP_DESC.NumDescriptors 1000008, device supports D3D12_RESOURCE_BINDING_TIER_3 which supports shader visible descriptor heap size larger than 1000000, but this may fail to create on some TIER_3 hardware.
Created heap with 1000016
dbg message 0: ID3D12Device::CreateDescriptorHeap: D3D12_DESCRIPTOR_HEAP_DESC.NumDescriptors 1000016, device supports D3D12_RESOURCE_BINDING_TIER_3 which supports shader visible descriptor heap size larger than 1000000, but this may fail to create on some TIER_3 hardware.
Created heap with 1015000
dbg message 0: ID3D12Device::CreateDescriptorHeap: D3D12_DESCRIPTOR_HEAP_DESC.NumDescriptors 1015000, device supports D3D12_RESOURCE_BINDING_TIER_3 which supports shader visible descriptor heap size larger than 1000000, but this may fail to create on some TIER_3 hardware.
Created heap with 1015540
dbg message 0: ID3D12Device::CreateDescriptorHeap: D3D12_DESCRIPTOR_HEAP_DESC.NumDescriptors 1015540, device supports D3D12_RESOURCE_BINDING_TIER_3 which supports shader visible descriptor heap size larger than 1000000, but this may fail to create on some TIER_3 hardware.
Failed creating heap with 1015550
dbg message 0: ID3D12Device::CreateDescriptorHeap: D3D12_DESCRIPTOR_HEAP_DESC.NumDescriptors 1015550, device supports D3D12_RESOURCE_BINDING_TIER_3 which supports shader visible descriptor heap size larger than 1000000, but this may fail to create on some TIER_3 hardware.
Failed creating heap with 1016000
dbg message 0: ID3D12Device::CreateDescriptorHeap: D3D12_DESCRIPTOR_HEAP_DESC.NumDescriptors 1016000, device supports D3D12_RESOURCE_BINDING_TIER_3 which supports shader visible descriptor heap size larger than 1000000, but this may fail to create on some TIER_3 hardware.

It looks like officially speaking nvidia doesn't support more than 1 million descriptors, though I'm surprised if this has only broken for you just now as it's extremely common for applications to create the maximum size descriptor heap.

It seems right.  Nvidia doesn't support more than 1 million descriptors.

It's not the case in general or this would have been discovered much sooner, but it definitely seems to be the case for you.
Out of interest can you close RenderDoc and all other 3D programs that might be using the GPU including UE5, and then run the test program? Is the output any different?

The above testing environment already in merely have 3D programs. It only has a Visual Studio and Chrome Program.

Fair enough, that's pretty conclusive then.
I guess maybe what I can do is try to create the slightly larger descriptor heap on tier 3 devices but have a fallback to create it at the smaller size if that fails. The main problem is that because of D3D12's design this means there are no descriptors available for RenderDoc to use which sometimes it needs...

BTW, I asked someone to run the program with Nvidia 3080. The result is the same as me

It doesn't really matter, but are the people running this test with that result all on windows 11? can you test on windows 10 - that's what I'm using.

Yep, the people are running Windows 11. I could run the test on Windows 10, but I will send the report in tomorrow.

I tested this program on another machine that has 3060ti and Windows 10. Test reports indicate the same as before. It cannot create more than 1 million descriptors.

fyi i've got the exact same issue

win11 22621.1992, 4090 driver 545.84, store 5.3.1 ue

just tried driver 536.99 and 537.58 and renderdoc just crashes after loading with no popup

Hi, I encountered the same problem on Renderdoc v1.29.  Win10, 3070ti, 545.84, UE5.3.1.
When I use Nightly version 2023-10-26, Capture frame seems all right, but it look will crash when open the capture result without any message.
So, I pull the latest vesion 3746f83 and built it locally. Open the capture result will cause crash on this:

You can download my capture result from: CrashCapture
Hope I provided enough information.

Please open a new issue for a different crash, this issue is only for the original report and combining one issue to cover more than one bug will only cause confusion.

I test 2023-10-26 Nightly version. Renderdoc will crash when opening the capture file. Same as

So, I pull the latest vesion 3746f83 and built it locally. Open the capture result will cause crash on this:

Does that mean the original issue is fixed for you or not? I wasn't able to test it so I needed confirmation from someone else who could reproduce it.
As in my previous comment if the original issue is fixed please open a new issue with a reproducible test case for any new problems. It's not a good idea to keep one issue open tracking different crashes as that can confuse the issue.

No the issue is not fixed, I still got the issue after the 10-26 update. Also after the update there are circumstances where renderdoc simply crashes without even producing the error message

OK, that's strange since it sounded like it was fixed for some people maybe? Can you please use help -> send error report after seeing this error, and include the capture as well as your github username so that I know it's you from this issue.
If it's crashing that is not this bug so please do not use this issue to discuss it. Please send a crash report if possible or open a new issue to track it. Trying to track multiple different bugs in the same issue will only cause confusion and make it harder to fix them.

Hi, I encountered the same problem on Renderdoc v1.29. Win10, 3070ti, 545.84, UE5.3.1. When I use Nightly version 2023-10-26, Capture frame seems all right, but it look will crash when open the capture result without any message.
So, I pull the latest vesion 3746f83 and built it locally. Open the capture result will cause crash on this:
You can download my capture result from: CrashCapture
Hope I provided enough information.

I have the same problem,how do you solve this?

Yes I Solved. Close API Validation follow issue #3104.
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2023年12月22日(星期五) 下午3:10
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [baldurk/renderdoc] Failed creating descriptor heap in Unreal Engine 5 D3D12 (Issue #3092)
Hi, I encountered the same problem on Renderdoc v1.29. Win10, 3070ti, 545.84, UE5.3.1. When I use Nightly version 2023-10-26, Capture frame seems all right, but it look will crash when open the capture result without any message.
So, I pull the latest vesion 3746f83 and built it locally. Open the capture result will cause crash on this:
You can download my capture result from: CrashCapture
Hope I provided enough information.
I have the same problem,how do you solve this?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


Documentation errors

[ ] In documentation on boost.org links to APIs of core classes are not there, however, everything is fine here on GitHub.
[ ] snippets do not work (for example see this and this).

Yes and I wonder if it is not related to https://github.com/boostorg/compute/issues/621

Is this considered as solved ?

Is this issue still open?

Is there anyone with Boost build infrastructure set up to try, please?


ui: Require records community

:heart: Thank you for your contribution!
Description
depends on: inveniosoftware/invenio-rdm-records#1283
closes: inveniosoftware/product-rdm#183
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[x] I'm aware of the code of conduct.
[x] I've created logical separate commits and followed the commit message format.
[x] I've added relevant test cases.
[x] I've added relevant documentation.
[x] I've marked translation strings (for relevant code).
[x] I've followed the CSS/JS and React guidelines (for relevant code).
[x] I've followed the web accessibility guidelines (for relevant code).
[x] I've followed the user interface guidelines (for relevant code).
[x] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[x] I've NOT included third-party code (copy/pasted source code or new dependencies).

Third-party code
If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

Closing this PR as the changes were mistakenly made on the master branch. I've created a new branch with these changes and opened a new PR here: 2642. Thanks for your understanding.


Yeelight errors in log, recently

Home Assistant release with the issue:

0.90.0
Last working Home Assistant release (if known):
0.87? Not sure, it was several main versions ago
Operating environment (Hass.io/Docker/Windows/etc.):

Hassio system 150
Component/platform:

Yeelight
Description of problem:
I see several types of error in the log related to my yeelight bulbs:
019-03-21 19:28:42 ERROR (SyncWorker_4) [homeassistant.components.light.yeelight] Unable to update bulb status: Bulb closed the connection.
2019-03-21 19:49:14 ERROR (SyncWorker_6) [homeassistant.components.light.yeelight] Error when calling <function YeelightLight.set_colortemp at 0x7f0d05db2f28>: {'code': -1, 'message': 'client quota exceeded'}
2019-03-21 19:49:14 ERROR (SyncWorker_3) [homeassistant.components.light.yeelight] Unable to turn the bulb on: {'code': -1, 'message': 'client quota exceeded'}
2019-03-21 19:49:14 ERROR (SyncWorker_7) [homeassistant.components.light.yeelight] Unable to update bulb status: {'code': -1, 'message': 'client quota exceeded'}
2019-03-21 19:50:21 ERROR (SyncWorker_6) [homeassistant.components.light.yeelight] Error when calling <function YeelightLight.set_colortemp at 0x7f0d05db2f28>: Bulb closed the connection.
2019-03-21 19:52:20 ERROR (SyncWorker_6) [custom_components.aurum] Unable to fetch data from AURUM. no element found: line 1, column 0
2019-03-21 20:20:09 ERROR (SyncWorker_6) [homeassistant.components.light.yeelight] Unable to update bulb status: Bulb closed the connection.
2019-03-21 20:20:10 ERROR (SyncWorker_0) [homeassistant.components.light.yeelight] Error when calling <function YeelightLight.set_colortemp at 0x7f0d05db2f28>: Bulb closed the connection.
2019-03-21 20:40:47 ERROR (SyncWorker_1) [homeassistant.components.light.yeelight] Unable to update bulb status: Bulb closed the connection.
2019-03-21 20:41:50 ERROR (SyncWorker_9) [homeassistant.components.light.yeelight] Unable to turn the bulb on: Bulb closed the connection.
2019-03-21 20:51:05 ERROR (SyncWorker_4) [homeassistant.components.light.yeelight] Unable to update bulb status: Bulb closed the connection.
2019-03-21 21:11:43 ERROR (SyncWorker_2) [homeassistant.components.light.yeelight] Unable to update bulb status: Bulb closed the connection.
2019-03-21 21:42:33 ERROR (SyncWorker_9) [homeassistant.components.light.yeelight] Error when calling <function YeelightLight.set_brightness at 0x7f0d05db2d08>: Bulb closed the connection.
2019-03-21 22:13:26 ERROR (SyncWorker_7) [homeassistant.components.light.yeelight] Unable to update bulb status: Bulb closed the connection.
2019-03-21 22:23:42 ERROR (SyncWorker_0) [homeassistant.components.light.yeelight] Unable to update bulb status: Bulb closed the connection.

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
light:
  - platform: yeelight
    devices:
      192.168.x.yy1:
        name: TV links
        use_music_mode: False
        model: color2
      192.168.x.yy2:
        name: TV rechts
        use_music_mode: False
        model: color2

Traceback (if applicable):

Additional information:
I'm running the Circadian custom component so there is lots of comms to the light to change the brightness and the color when the bulbs are on.

This is the bug/issue tracker for Home Assistant. Sorry, we are not able to help with custom component.
Please use the forum for questions about installation, setup, configuration and usage of custom components.

I'm sorry, but I do not agree that this issue is being closed: the errors are originating from the yeelight component. Which is not a custom component.

Hello. I have 6 yeelight bulbs. For a few months everything worked correctly, but for the last month I noticed that the color setting does not work through homekit, while through the HA interface everything works correctly. When I want to change the color and I choose one of them back, the default setting is set. Color change works properly on xiaomi gateways. I noticed 2 messages in the log:
Unable to update bulb status: {'code': -1, 'message': 'client quota exceeded'}
and
Unable to update bulb status: Bulb closed the connection.


Significant delay when dragging

First, your calendar is absolutely great! There's a little problem I faced when dragging to create a long event: the event marks only show up after some time, giving the impression that it's slow.
I tested the sample page on Chrome 68 and Firefox 61, both on Win 10.

@ClickerMonkey I also have another issue, which is probably not related to your code, but I'd appreciate if you could help: when I use vuetify a la carte, importing
Vue.use(Vuetify, {
  components: {
    VApp,
    VNavigationDrawer,
    VFooter,
    VList,
    VBtn,
    VIcon,
    VGrid,
    VToolbar,
    transition

a click in any day in ds-calendar component causes it to go all blue until I move the mouse. Am I missing some import?
I'd also like to ask if it is possible to replace moment.js, since it is a large library, and it is required by the dayspan dependency.

@ClickerMonkey thanks, importing VMenu did it


Enable and disable plugins via the API

This PR does the following:

Plugin activation is now handled directly from the settings page
Plugin activation uses the API
Plugins can now also be deactivated via the API
Renders table of plugins via API access, rather than templated on the server side (because who wants templates?)

For reference, here's a screenshot of the new interface:


(1.12.2) ProjectRed/ForgeMultipart Microblocks Have Incorrect Footsteps

ProjectRed was updated to 1.12 the other day, so I had a look through it's features, including the old microblocks feature. The issue with the microblocks is that since they are all based off of the same block id with varying nbt data, they all have the same footsteps in DS. To solve this, I think you'd have to read the nbt tag containing the block id the microblock is mimicking.
On a semi-related note, I also noticed this problem with the blocks from ArchitectureCraft, and while I haven't used BWM in a while (so I can't be sure) I imagine that mod's semi-new dynamic miniblocks update may also create a problem with footsteps.
ProjectRed: <IP_ADDRESS>
DS: <IP_ADDRESS>
Forge: <IP_ADDRESS>80

I haven't looked at PR to see if there is an API where the underlying block type can be determined.  I do this with facade/cover support for EnderIO and Thermal Dynamics.  I really don't want to do NBT searching.

Ah, that seems wiser.

Just started looking at this for 3.5.  Forge Multipart is written in Scala.  I don't want to learn scala.  Sheesh.

Pushed a BETA of v3.5 to CurseForge.  Note that the new version has a library dependency on OreLib (another library mod I made).  Make sure you read the release notes prior to upgrade.


Mesages and users are not showing

Hey nic project but when I imported on my android studio and make a apk of that users and mesaages are not showing after sending messages why?
Incomplete project on github

Hey I am unable to see messages and users in app plz tell my why this is happening  but messages are storing in firebase database but not fetching in app

It's solved now

No till not fix

Saving data on firebase but unable to see in app

try now, the messages should be appearing


Particles Missing 2.

Hi,
You have closed my first topic and it wasn't an invalid Label.
I said that Particles are missing TILL 2.6.0.
I have update, that is why i said "TILL", may be "SINCE" would be better.
Sorry i'm not english.
I'm have installed each update since April and none of them have particle.
I'm actually using 2.6.7.
Version 2.6.2 was supposed to bring back particles, but it did not.
You are fast making invalid Label, before reading and trying to understand what i'm talking about.
Spigot version: CraftBukkit version git-Spigot-3996d7d-6b843d9 (MC: 1.10.2) (Implementing API version 1.10.2-R0.1-SNAPSHOT)
Was not working on 1.10 either.

Referencing #366
"Til" or "till" means "until", so you were saying particles were missing in versions before 2.6.0. Thank you for clarifying, but don't blame me for your mistake.
Particles work just fine for me. Do you have "show-npc-effects" set to "true" in config.yml? And does your Minecraft client have particles set to "All" in the settings?

Sorry, my mistake.
I had disable them because they were spamming on 1.10.
Everything is working.
Thanks.


How to test the speed of LSTR?

Hi,
Many factors can affect the inference speed of the model. Such as the evaluation method(batch mode, stream mode) of the LSTR. Could you please provide more details, such as the batch_size? It would be better if you could provide the speed testing code.

We used the stream mode and processed one video (batchsize as 1) from the beginning to the end.


Nonfunctional Sort Function on Individual Bill Page

Describe the bug
the sort function isn't working when i toggle between oldest first and recent first
To Reproduce

Go to Bills Page
Go to Individual Bill Page (ie, H.100)
Click on sort button

Expected behavior
When I click "Oldest First", for the testimonies to reorder
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser Chrome

Removing for beta, we'll add it back in when we get it connected to the new backend


chore: ignore WebStorm IDE files

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[x] Other, please describe: ignore WebStorm IDE files

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
Why can have .vscode ?
Can't have .idea !

Thanks but see https://github.com/vuejs/vue/pull/6074 and https://github.com/vuejs/vue/pull/4383. In general this kind of nitpicking is not a good idea for PR especially when it's tied to personal workflow and can be solved in one's personal workstation (in this scenario with a global .gitignore).
.vscode was probably added because most people contributing (including the core team) uses it

Thanks but see #6074 and #4383. In general this kind of nitpicking is not a good idea for PR especially when it's tied to personal workflow and can be solved in one's personal workstation (in this scenario with a global .gitignore).
.vscode was probably added because most people contributing (including the core team) uses it

what's puzzle me is why project vue-router can and vue can't ?
https://github.com/vuejs/vue-router/commit/8ed5f6a52ba7d6af53cb71756307be8003ac0026#diff-a084b794bc0759e7a6b77810e01874f2


[Web Monitor] - Hotfix + Network view

Motivation
To add the new Network view which displays VALs and VFNs IPs and ports reachable.
To fix web-monitor count of proofs in the current epoch.
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
Start local swarm, tower and web-monitor, and check count.
Related PRs
(If this PR adds or changes functionality, please take some time to update or suggest changes to the docs at https://developers.diem.com, and link to your PR here.)
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

@soaresa  bump. This is ready to merge if you can make the small changes above.


Make trailing semis discard expr

Eg
fn foo() { 5 }

Should return 5
But
fn bar() { 5; }

should not

Closed by https://github.com/aDotInTheVoid/skate/commit/d838f29a50691cac68f9dfc264db66b63338f3a4


add thousand sparator to chart label

Hi,
i tried to add thousand separator to my chart but still got error. bellow is my code using callback function but it doesn't work.  need your help for this issue.
$options = array(
'responsive' => true,
'callbacks'=>array(
'label'=>function (tooltipItems, data) (
return data.datasets[tooltipItems.datasetIndex].label + ' : ' + tooltipItems.yLabel.toLocaleString();
)
)
'scales' => array(
'yAxes' => array(
array(
'scaleLabel' => array(
'display' => $showYaxes,
'labelString' =>$yaxeslabel,
'fontSize' =>10,
'fontStyle'=>'bold',
'borderWidth'=>1,
),
'ticks'=>array(
'beginAtZero'=> true,
)
            )
        ),
        'xAxes'=> array(
            array(
              'gridLines'=>array(/*
                'zeroLineColor'=> "black",
                'zeroLineWidth'=> 2*/
                'display'=> true
              ),
              'ticks'=>array(
                'autoSkip'=>false,
                'maxRotation'=> 30,
                'minRotation'=> 30,
              ),
              'scaleLabel'=>array(
              'display'=> false,
              'fontSize'=>16,
              'labelString'=> $chartTitle
              )
            )
        )
    ),
    'legend' => array (
        'display' => $displayLegend,
        'position' => $legendPos,
        'labels' => array (
        'fontSize' => 12,
        'usePointStyle'=> true,
        )
    ),
    'title'=>array (
        'display'=>true,
        'text'=> $chartTitle,
        'fontSize' => 12,
        'position'=>$titlePos

    )
);

Thanks,
Jajas

Hi! Sorry for the late response. This issue was noticed in #3 and my solution was to send in the callback function as a string. So you need to enclose it in ' or ".

i tried your sample using this syntax
    'responsive' => false,
    'tooltips' => array(
        'callbacks' => array(
            'label' => 'function(tooltipItem, data){return "Hello world";}'
        )
    )
);

and i got error on browser console
Chart.js:8132 Uncaught TypeError: callbacks.label.call is not a function
    at Chart.js:8132
    at Object.helpers.each (Chart.js:4811)
    at ChartElement.getBody (Chart.js:8125)
    at ChartElement.update (Chart.js:8205)
    at Chart.Controller.eventHandler (Chart.js:4493)
    at Chart.Controller.<anonymous> (Chart.js:4415)
    at HTMLCanvasElement.events.(anonymous function)```

Which version of Chart.js-PHP did you end up using? Passing callbacks functions is one of the new features in 2.0.

Chartjs.zip
im using this attached version

and call this script to generate chart

Yeah but that is Chart.js, I was asking about the PHP wrapper Chart.js-PHP. It's easy to mix up the names when they are so similar.

ChartJS.zip
another file in my project i got this one.

That looks like the 1.0 version. So if you want to use callbacks you have to upgrade to 2.0 and install it via composer as I told you in #6.

for version 2.0, is there any sample package like version 1.0
I am newbe for using composer.

Yes at https://github.com/Ejdamm/Chart.js-PHP/releases but you will still
need to use composer. There are easy to follow-guides at Composer's
website. It is probably possible to remove some lines of code to use it
without composer but i am not able to help with that for a couple of weeks.
Den fre 20 juli 2018 10:19jajasuparja<EMAIL_ADDRESS>skrev:

for version 2.0, is there any sample package like version 1.0
I am newbe for using composer.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Ejdamm/Chart.js-PHP/issues/8#issuecomment-406525471,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACcrr_st1tr00Mhq_mdO7t8qlKDIcEj1ks5uIZKQgaJpZM4VGs70
.

Run composer got this warning. if i have to upgrade my php version. lot of thing to re-configure. meanwhile i will use native chart.js script

One way to use 2.0 without composer would be to just replace the line
require '../vendor/autoload.php';
with
require '../src/ChartJS.php';


Integrate Nitrokey FIDO U2F commands

Integrate Nitrokey FIDO U2F commands. See the complete tool at:

https://github.com/Nitrokey/nitrokey-fido-u2f-firmware/blob/master/tools/u2f_zero_client/client.py

Nitrokey FIDO U2F is deprecated.


CEPH_2024-05-05

Thanks for the submission! The current output reflected your submission is:
https://hopkinsidd.github.io/rsv-forecast-hub_website/
(We are currently investigating the broken lines in the graphs)


feat: add KusionStack

FYI. I am the creator and master of KusionStack, an open source tech stack to build IDP.

@adohe 👍


Update metadata.js

change name and description

👷 Deploy Preview for timely-brigadeiros-dd9fc4 processing.

Name
Link

🔨 Latest commit
f9b97a5f6c211759448ca937f88bd7e6f9925616

🔍 Latest deploy log
https://app.netlify.com/sites/timely-brigadeiros-dd9fc4/deploys/64bfada98fbfa30008252cbf


Unable to install with version 7.7

Seems as if the clsx version is out of sync.
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/clsx
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@mantine/core
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!     peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/mantine-flagpack
npm ERR!       mantine-flagpack@"*" from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/mantine-flagpack
npm ERR!   mantine-flagpack@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Fixed in 4.0.1


loss=nan

Hello, reppy4620.
When I used LJspeech corpus, I modified "config/text_encoder/num_vocab:160". The program can run.
But when epoch=170, I have a problem：”b=3.985, disc=nan, gen=nan, fm=nan,“.
I don't have this problem with "jsut".
Have you ever encountered this problem? I don't know if it's the data set or if I changed ”num_vocab“ from 100 to 160
Thank you for your reply。

Hi Jimvc, sorry for late reply.
This is an issue I have experience with as well.
It's likely that luck also plays a part in this issue.
Stability is a challenging aspect, and it may be due to training with fp16.

Thank you,reppy4620.
I changed the learning rate from 2e-4 to 1e-4. Let's see how it goes.


Updates for Adversarial Patch attack in TensorFlow

Description
This pull request adds support for models predicting logits in addition to existing support for models predicting probability.
Type of change
Please check all relevant options.

[x] Improvement (non-breaking)
[ ] Bug fix (non-breaking)
[ ] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Codecov Report

Merging #878 (97bd910) into dev_1.5.2 (1deea2b) will decrease coverage by 43.92%.
The diff coverage is 16.66%.

@@              Coverage Diff               @@
##           dev_1.5.2     #878       +/-   ##
==============================================
- Coverage      64.66%   20.73%   -43.93%
==============================================
  Files            176      176
  Lines          17123    17132        +9
  Branches        3167     3168        +1
==============================================
- Hits           11073     3553     -7520
- Misses          4805    13365     +8560
+ Partials        1245      214     -1031

Impacted Files
Coverage Δ

.../adversarial_patch/adversarial_patch_tensorflow.py
12.43% <16.66%> (-53.71%)
:arrow_down:

...rt/defences/detector/poison/clustering_analyzer.py
8.22% <0.00%> (-86.71%)
:arrow_down:

...ces/detector/evasion/subsetscanning/scanningops.py
11.59% <0.00%> (-85.51%)
:arrow_down:

art/attacks/evasion/carlini.py
8.40% <0.00%> (-80.87%)
:arrow_down:

...defences/detector/poison/ground_truth_evaluator.py
17.80% <0.00%> (-79.46%)
:arrow_down:

...ient_descent/projected_gradient_descent_pytorch.py
16.93% <0.00%> (-78.23%)
:arrow_down:

...escent/projected_gradient_descent_tensorflow_v2.py
18.33% <0.00%> (-76.67%)
:arrow_down:

art/attacks/evasion/hop_skip_jump.py
13.17% <0.00%> (-75.61%)
:arrow_down:

...ttacks/inference/membership_inference/black_box.py
12.26% <0.00%> (-74.24%)
:arrow_down:

art/metrics/verification_decisions_trees.py
13.75% <0.00%> (-73.02%)
:arrow_down:

... and 118 more

Codecov Report

Merging #878 (97bd910) into dev_1.5.2 (1deea2b) will decrease coverage by 43.92%.
The diff coverage is 16.66%.

@@              Coverage Diff               @@
##           dev_1.5.2     #878       +/-   ##
==============================================
- Coverage      64.66%   20.73%   -43.93%
==============================================
  Files            176      176
  Lines          17123    17132        +9
  Branches        3167     3168        +1
==============================================
- Hits           11073     3553     -7520
- Misses          4805    13365     +8560
+ Partials        1245      214     -1031

Impacted Files
Coverage Δ

.../adversarial_patch/adversarial_patch_tensorflow.py
12.43% <16.66%> (-53.71%)
:arrow_down:

...rt/defences/detector/poison/clustering_analyzer.py
8.22% <0.00%> (-86.71%)
:arrow_down:

...ces/detector/evasion/subsetscanning/scanningops.py
11.59% <0.00%> (-85.51%)
:arrow_down:

art/attacks/evasion/carlini.py
8.40% <0.00%> (-80.87%)
:arrow_down:

...defences/detector/poison/ground_truth_evaluator.py
17.80% <0.00%> (-79.46%)
:arrow_down:

...ient_descent/projected_gradient_descent_pytorch.py
16.93% <0.00%> (-78.23%)
:arrow_down:

...escent/projected_gradient_descent_tensorflow_v2.py
18.33% <0.00%> (-76.67%)
:arrow_down:

art/attacks/evasion/hop_skip_jump.py
13.17% <0.00%> (-75.61%)
:arrow_down:

...ttacks/inference/membership_inference/black_box.py
12.26% <0.00%> (-74.24%)
:arrow_down:

art/metrics/verification_decisions_trees.py
13.75% <0.00%> (-73.02%)
:arrow_down:

... and 118 more

Hi @minhitbk Thank you. Actually, I haven't thought if it. Let me try to move the check to __init__. I think it will require extra code to handle wild cards in the input shape.


Prepare to produce a nightly build of EventHubs extension

We need a package ref to Microsoft.Extensions.Azure

/azp run net - eventhub - ci


Remove warning about @frozen without library evolution

This warning is produced if you annotate an enum with @frozen but
build without library evolution enabled. The problem is that if you
only build with library evolution enabled for release builds, this
leaves you with a warning for all debug builds. The downside of this
change is that if you never build with library evolution, you may have
this unnecessary annotation without noticing.

We haven't tested that thoroughly. At some point it was mentioned that it would negatively affect build performance I believe?
But I think the core issue is that for distributable libraries the build processes could vary where users support a combination of build systems and they may not want (or be able) to always support this configuration for all of them.

I wouldn't expect library evolution to negatively impact build performance, but there are performance implications across module boundaries.
But I suppose there is a valid case for a library that's both available as a source package and a binary framework.

(Sorry, I just made an edit to the title. It makes perfect sense after reading everything, but the original title could be read as though it was trying to enable the removal of @frozen from types without triggering ABI incompatibility issues, rather than the removal of the warning. Just wanted to remove that ambiguity so that other people don't jump out of their seat like I did.)

@swift-ci please smoke test and merge

Awesome thanks!!

@swift-ci please smoke test

@harlanhaskins thanks a ton! Is this something that could / should be cherry picked into the next release? (Or maybe that branch hasn't even been cut?)

The branch for Swift 5.2 is gonna be cut on December 9th, so this’ll make it into Swift 5.2.

Great thanks!


Size mismatch for overlapping seq

https://github.com/PyTorchRL/pytorchrl/blob/19147505b16d0748e826815e12cf6d15af927f80/pytorchrl/agent/storages/off_policy/per_buffer.py#L207
Hello,
I have noticed there is size mismatch in above codes:
idxs size = sequence_length + overlap_length
errors size = non_overlap_length + overlap_length
so later when we try to broadcast here:
https://github.com/PyTorchRL/pytorchrl/blob/19147505b16d0748e826815e12cf6d15af927f80/pytorchrl/agent/storages/off_policy/per_buffer.py#L213
we got mismatch in sizes

Thank you very much for pointing it out. The issue is now solved. We corrected the sizes and now both errors and idxs match:
idxs size = non_overlap_length + overlap_length
errors size = non_overlap_length + overlap_length
version 1.3.61 included the fix


Malformed account verification link

Hi,
I registered an account for the 4minitz demo server.
The activation link I got was

https://demo.4minitz.com/#/verify-email/CANi403EVUpcy0VEG...

and the account was not verified when clicking on it.
Removing the "/#" solved it for me, so there might be an issue in the creation of the verification link.
Best regards

Thanks for letting us know.
I think it may be more of a routing issue. The link is generated by meteor itself or a core meteor package so that link should be ok. But maybe our client side routing is getting in the way here.
@derwok @felixble
Could this be the root cause? Do we need to add a /#/ there? https://github.com/4minitz/4minitz/blob/develop/client/router.js#L34
Any way to check this on a local install without a mail server attached?

It seems we have code in place to strip the odd "#/" from the URL like suggested here
We have this code here:
https://github.com/4minitz/4minitz/blob/df6c6af4f8603f6eca78c74157d3edc697e23d6b/imports/config/EMailTemplates.js#L14
I'm pretty sure this code was called in the past.
Currently is seems this code is not called anymore, as the EMail sent has this text:
Hello Wolfram Wok2,
To verify your account email, simply click the link below.
http://localhost:3100/#/verify-email/ocqrKCr8l1Rk3a3QUdVzl1Jbjhc7BlUexuP9q44W07W
Thanks.

which is the default libary text.

So, we have to find out why our accounts config code is not called anymore..

Ah! Before configuring the AccountsPackage we check if email is enabled. We do this via GlobalSettings.isEMailDeliveryEnabled() - but GlobalSettings are not initialized at this point in time.
As our non-reachable code is serverside, we may happily use Meteor.settings.email.enableMailDelivery instead


Fix Module Import Issue in preprocess.py for Enhanced Compatibility

Changes Made

Modified sys.path within preprocess.py to dynamically include the path to the crystallm module. This change resolves the ModuleNotFoundError by ensuring Python can locate and import crystallm regardless of the current working directory.

Rationale
The preprocess.py script previously assumed a specific working directory for successful module importation. This assumption could lead to execution errors when the script was run from a different directory, limiting its flexibility and usability. By dynamically adjusting the sys.path, we cater to a broader range of use cases, making the script more robust and user-friendly.
I welcome feedback on this change, particularly regarding:

Any potential impacts on existing workflows I might have overlooked.
Suggestions for further testing or scenarios to consider ensuring the robustness of this fix.

Thank you for considering this contribution.

Thank you for making this contribution. I agree, this approach is more flexible, and allows the script to be run from any directory. I don't see any issues with this change. However, there are a number of other scripts in the bin folder which assume a working directory; for example: deduplicate.py, generate_cifs.py. I think it would be good to change those scripts as well to use the proposed approach.
Since there are a number of scripts that need to be updated, perhaps we can create a path_setup.py file in bin with the contents:
import os
import sys

def setup_path():
    script_dir = os.path.dirname(os.path.abspath(__file__))
    parent_dir = os.path.abspath(os.path.join(script_dir, '..'))
    if parent_dir not in sys.path:
        sys.path.insert(0, parent_dir)

and then call setup_path() from the relevant scripts:
from path_setup import setup_path
setup_path()

I agree with @kianpu34593 that installing crystallm as a module is a way to solve this problem. I have created PR #6 which adds a setup.py file and an extra installation step. @hoon-ock Please let me know if this alternate approach addresses your issue.


fix(es/parser): parse decorators after export

Description:

BREAKING CHANGE:

Related issue (if exists):

fix #6984

I should put some parser-related test cases, but I'm not sure where to put them.

You can add one to https://github.com/swc-project/swc/tree/e7911ee594ec1c85b4f0a4fe19833f737d4576dd/crates/swc_ecma_parser/tests/js or https://github.com/swc-project/swc/tree/e7911ee594ec1c85b4f0a4fe19833f737d4576dd/crates/swc_ecma_parser/tests/typescript

thread 'errors_tests__typescript_errors__esDecorators__classDeclaration__esDecorators_classDeclaration_exportModifier__file6_ts' panicked at 'assertion failed: `(left == right)`
  left: `
  x Decorators may not appear after `export` or `export default` if they also appear before `export`.
   ,-[$DIR/tests/typescript-errors/esDecorators/classDeclaration/esDecorators-classDeclaration-exportModifier/file6.ts:3:1]
 3 | // error
 4 | @dec export @dec class C6 {}
   :             ^^^^
 4 |
   `----
`,
 right: `
  x Decorators may not appear after `export` or `export default` if they also appear before `export`.
   ,-[$DIR/tests/typescript-errors/esDecorators/classDeclaration/esDecorators-classDeclaration-exportModifier/file6.ts:3:1]
 3 | // error
 4 | @dec export @dec class C6 {}
   :             ^^^^
   `----
`:

It appears that there is a formatting error present, with an extra 4 added to the line number.


feat(CI): add main.yml github actions workflow

Tests won't pass until we add env variables UPCLOUD_API_TEST_USER & UPCLOUD_API_TEST_PASSWORD need to create a test account and add those to repo secrets


NuGet for newly added wechat_qrcode?

I see https://github.com/shimat/opencvsharp/pull/1375 is merged, this is super cool, will it create a nuget package soon?

After adding wechat_qrcode, one GitHub Action is not working properly, and I cannot publish the new NuGet package without fixing it.
https://github.com/shimat/opencvsharp/runs/4988054483?check_suite_focus=true
Frankly speaking, I think that some pure C# libraries (e.g. ZXing.NET) are sufficient for QR code detection/recognition functions without relying on OpenCV. I've been struggling a lot lately to keep OpenCvSharp working on many platforms, and I'm reluctant to make changes that will make it harder to build. If OpenCvSharp runs on Windows and Ubuntu, that's good enough for my personal use.
If it is difficult to solve this problem, I am sorry to say that I may remove the wechat_qrcode feature.

Actually ZXing.NET is not as good as wechat_qrcode in performance and accuracy.

I think we need to bump the version of cmake to make the build succeed. Maybe use 3.1-focal (20.04) instead of 3.1-bionic (18.04)?


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-extensions/kn-plugin-operator-approvers
/assign knative-extensions/kn-plugin-operator-approvers
Produced by: knative-extensions/knobots/actions/update-deps

/lgtm
/approve


How can I add schema directives?

I like to create Enforcing access permissions as described here, so I can do Authorization and ACL
https://www.apollographql.com/docs/graphql-tools/schema-directives/#enforcing-access-permissions
I cant find anything in the documentation

You should be able to use directives the same way in SDL (see this example from tests - at the very bottom). Those directives should be available on type instance via $type->astNode or $type->extensionASTNodes (if type was extended).

@vladar How can you use schema directives from php when extending ObjectType. Do I need to pass an extension/ASTNode in the $config?

@Tyler-Gauch directives are not intended to be used this way, see: https://github.com/graphql/graphql-js/pull/746#issuecomment-301554231 . You can inject your own metadata into the config object for tracking runtime information.

@mwijngaard You are right. We will also add extensions feature in one of the next versions. But for now, just use a custom config. It still makes sense to keep custom metadata under extensions config key.
Closing this but feel free to re-open if you want to continue the discussion.


Trying to run the example

Hello, I'm trying to run this example but the console gives me this:
[HMR] Waiting for update signal from WDS...
preloadResources.js:15 Uncaught TypeError: Cannot read property 'loader' of undefinedpreloadResources @ preloadResources.js:15(anonymous function) @ index.js:62
client?cd17:37 [WDS] Hot Module Replacement enabled.
Any ideas?
Thanks in advance!

I also ran into the same problem when running "pixi.js": "^4.1.0"
I am guessing its has something with webpack configuration.
Maybe related to this
https://github.com/pixijs/pixi.js/issues/1854


fix: externalized error message and toggle developer tools (#734)

Fix https://github.com/sassoftware/vscode-sas-extension/issues/734
Suggestion from jbodart-argenx: https://github.com/sassoftware/vscode-sas-extension/issues/685#issuecomment-1849425397

Externalized error message 'There was an error executing the SAS Program. See [console log] for more details.'

The 'console log' in the show error message dialog should be a link to open the developer tools.

Testing

Change the language to zh-cn to check if the error message is change to Chinese.

Click the hyper link [console log] to see if the developer tools is opened.

https://github.com/sassoftware/vscode-sas-extension/pull/747#issuecomment-1888687870


[BUG] (Regression) SkiaSharp.Views.Forms nuget package doesn't provide anymore netstandard binaries

Description
I tried to upgrade from SkiaSharp.Views.Forms 1.68.0 to 1.68.1 but I couldn't because the former provided a lib/netstandard1.3 folder, while the new version doesn't provide any netstandard libs (I only see folders MonoAndroid, Xamarin.Mac20, Xamarin.iOS, tizen40 and uap10.0).
(FYI I need this to be able to run a Xamarin.Forms app under GTK platform, i.e. Linux)

I have almost similar problem.
Since moving from 1.68.01 to 1.68.1, I  cannot override SKCanvasViewRendererBase at the UWP project. Decompiling points me always to the Mono version.

@knocte The netstandard library is actually moved to the ref folder so that the compiler never copies it to the device. It was done to solve https://github.com/mono/SkiaSharp/issues/1011. You typically should not be using the netstandard library directly as it contains stub methods / missing methods as it is just for reference only.
But... If it is working for you, then maybe we can have a look at splitting the assembly into two - one that is a "base" assembly in the lib folder for all to use and one that is in the ref folder so that it does not conflict with the platform.
We can't move it back because of #1011 in which there are two assemblies with the same name and the first (which is the netstandard) is always picked. Moving it to the ref folder means that the platform assembly is picked. It might have been working for you because you have a different assembly name for GTK?
Another options is that you can send in a PR with the code you are using for GTK and we can have a look at merging that in - then you don't need to worry about that anymore.

@lothargrieb We just spotted that in #1042 and a fix is in #1044. I have pushed out a fix to the preview feed:
https://aka.ms/skiasharp-eap/index.json
Try version <IP_ADDRESS>-preview.2 as that should have the fix. (there is a 1.68.2 series, but that is actually older because I decided to get .1.1 out first with this fix)

Another options is that you can send in a PR with the code you are using for GTK

Not sure what you mean with this? I just use SkiaSharp.Views.Forms in my project because one single static method I make use of: SkiaSharp.Views.Forms.Extensions.ToSKColor(). I'm sure this can be made available in a .netStandard binary which works across all platforms, or am I missing something?

Is this related to https://github.com/mono/SkiaSharp/issues/379 ? I don't understand how this has worked before.

This used to work because I had the nuget install the netstandard dll into platforms that were not supported. This was incorrect behavior because if you were to install the WPF nuget or the (hopefully soon) GTK nuget you would get duplicate dlls - the one from the core forms and the one from the WPF/GTK nuget. In addition, several paths would also throw exceptions
See https://github.com/mono/SkiaSharp/pull/1012
It used to work because there was no WPF nuget to cause this issue.
If you are using just one single static method, I would really suggest just adding this extension method directly into your project instead of referenceing the NuGet with is not supported in this scenario:
https://github.com/mono/SkiaSharp/blob/master/source/SkiaSharp.Views.Forms/SkiaSharp.Views.Forms.Shared/Extensions.cs#L63-L66
The SkiaSharp.Views.Forms NuGet is only supported in Android, iOS, macOS, UWP and Tizen platforms. There is a separate package for WPF. All other uses are not supported and will cause issues if used.
I have created an issue to actually track this implementation.

Closing this for now as this is "technically" a regression as a result of fixing a bug. It was also not a supported pattern. For GTK, we will be getting a new NuGet that has much more features #1089
As a result, this particular use case will be addressed at some point with #1065, but not right now. The progress will be tracked on that issue.