The pasted entities don't display automatically when successfully canceling the copy activity

Storage Explorer Version: 1.32.0-dev (95)
Build Number: 20231025.1
Branch: rel/1.32.0
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Sonoma 14.0(Apple M1 Pro)
Architecture: x64/x64/arm64
How Found: Ad-hoc testing
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Tables.
Create a table -> Import a large csv file.
Right click the table -> Click 'Copy Table'.
Paste the table to another storage account.
Click 'Cancel' next to the activity log.
Check whether the pasted entities display automatically.

Expected Experience
The pasted entities display automatically.
Actual Experience
The pasted entities don't display automatically.
Additional Context
The pasted entities display after manually refreshing.

Assuming entities were copied, we should show how many were, and refresh data explorers as needed.


bump CI version to v0.0.1

bump up CI version.

I suppose we need update script to upload images to mircoshift and the change can be ref to kepler's CI. :-)  I will take a look and update later.

Hi @rootfs , I suppose for operator test, we need do further updates for microshift. please review this PR.

@husky-parul can you take a look?

Hi @rootfs , I suppose for operator test, we need do further updates for microshift. please review this PR.

Yes, absolutely I agree. I was planning to integrate microshift for Operator but I am happy to add further updates once this PR is merged.

@SamYuan1990 I was thinking can we also add creating an instance of Kepler once the Operator is deployed since it will also allow us to validate things e2e for instance there can be one scenario where there is any pre-req that is supposed to be handled by the Operator installation but didn't happen and when Kepler instance is launched it fails?

@SamYuan1990 I was thinking can we also add creating an instance of Kepler once the Operator is deployed since it will also allow us to validate things e2e for instance there can be one scenario where there is any pre-req that is supposed to be handled by the Operator installation but didn't happen and when Kepler instance is launched it fails?

For kepler repo, it's run e2e test case and it's cover it's own test case for kepler pod.
For kepler Operator, it's running as an operator as a pod.
For this PR, it's about CI as kepler-action bump up.
For kelper Operator,  I suppose it 's better to as @rootfs  and @piparul for this question as it's out of this PR's scope.

which the question is: for any operator release with kepler release, will we have/add a test case which 1st deploy kepler pod with specific version, and then run e2e test case download from kepler(with specific version) repo.


Authentication failed

I have noticed an issue with the  parsing of passwords when they are provided via ENV or Config
Given the following password
GNMIC_PASSWORD='|Bo|Z%TYe*&$P33~'
I see this in the debug logs when i start up a subscriber
[config] 2021/11/18 17:25:24.532736 /home/runner/work/gnmic/gnmic/config/targets.go:105: read target config: {"name":"testing","address":"<IP_ADDRESS>:50051","username":"automation","password":"|Bo|Z%TYe*\u0026$P33~","timeout":2000000000,"insecure":true,"skip-verify":true,"subscriptions":["interface_stats"],"outputs":["STDOUT"],"retry-timer":10000000000,"gzip":false,"token":""}

[config] 2021/11/18 17:25:24.532759 /home/runner/work/gnmic/gnmic/config/targets.go:119: targets: map[testing:{"name":"testing","address":"<IP_ADDRESS>::50051","username":"automation","password":"|Bo|Z%TYe*\u0026~","timeout":2000000000,"insecure":true,"skip-verify":true,"subscriptions":["interface_stats"],"outputs":["STDOUT"],"retry-timer":10000000000,"gzip":false,"token":""}]

Not only is it doing some UTF encoding but it actually seems to be removing characters after the $
If i try this via CLI it works as expected and this is in the debug output
[gnmic] password='|Bo|Z%TYe*&$P33~'(string)

Hi,
Thanks for catching and reporting this.
The UTF encoding comes from JSON Marshaling the target configuration and is only related to log printing, the password is still stored as it is in ENV or config file.
The real issue is the "disappearing" characters after $, that comes from trying to expand ENV variables in the config.
I did a few tests and it seems that a reasonable fix is: do not expand password fields if its value does not start with $
I will push a PR with the fix today.

Hi, the fix is part of release v0.21.0.
If you can please give it a try.


Introduce a verbose/list option to the build command

In the output of a docker build command it would be nice to enable a verbose mode to list the files added to the bundle. It would help with debugging.

Yes, running pkger list before pkger doubles the build time. Please add pkger -v


Rename all images and add alt tags

The images are not properly named.
To do:  Rename the images properly in snake case for example web_development_icon

can you please assign this to me?

Can I please work on this issue?

hey cool, im also working on this

Hey @UshasriMavuri1999 , assigning the task to you! 🚀

hlo sir, i would like to do this :>
it will be really kind of u if u can let me do this too :)

@harshgoel05 checkout my PR and tell me if you still need any changes

Hey! @ShobhitRathi it looks great but it was already assigned to @UshasriMavuri1999, will check her PR and if that's not great. Will merge yours.

@harshgoel05 cool you can merge it anytime you want

Closing this #14 #19


When answering a mail the name of the receipient is shortened

As soon as I answer an email from somebody who has a dot in his mail adress, Rainloop shortens the displayed name to the first name.
Example:
<EMAIL_ADDRESS>becomes To: "Sandra"<EMAIL_ADDRESS>This looks like I have her mail address in my contacts with her first name. This is pretty inappropriate.

RainLoop does not do anything special with the mail address.
I checked your issue but cannot confirm it.
Сould you give me the example email with such problem ?

It's not about the mailadrress itseld, it's an issue with the the name that is automatically displayed for the contact. Please see my live example above.
It looks like I'm sending an email to an old friend, but it's a business contact here in this case.


Add docs for PipeReader/PipeWriter.Create(stream)

In the stream section we currently show AsStream() for going from PipeReader and PipeWriter to a Stream but we don't show the other way using PipeReader/PipeWriter.Create(stream).

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

ID: 2f5384a7-5e96-d977-7748-317a2c758f63
Version Independent ID: 06e72212-a052-a351-0e0b-1b002ce214de
Content: I/O pipelines - .NET
Content Source: docs/standard/io/pipelines.md
Product: dotnet
Technology: dotnet-standard
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Assign @davidfowl as well.

Done.


SHACL minInclusive not working for any numeric type other than xsd:integer

Hi,
I've tried to validate a double value by checking that it is between a minInclusive and maxInclusive value as shown below.
Prefix mappings:
prefix :       <http://example.org/>
prefix sh:     <http://www.w3.org/ns/shacl#>
prefix xsd:    <http://www.w3.org/2001/XMLSchema#>
prefix schema: <http://schema.org/>
prefix rdfs:   <http://www.w3.org/2000/01/rdf-schema#>

data
:echotime  a schema:QuantitativeValue ;
        schema:unitCode  "second" ;
        schema:value     "0.002996"^^xsd:double .

shape:
schema:QuantitativeValue a sh:NodeShape, rdfs:Class ;
   sh:nodeKind sh:IRI ;
   sh:property [
    sh:path schema:value ;
    sh:minCount 1;
    sh:maxCount 1;
    sh:datatype xsd:double ;
    sh:minInclusive 0.001 ;
    sh:maxInclusive 0.01 ;
  ] .

But I got the following error:
"valid": false,
  "type": "Result",
  "message": "Not valid",
  "shapeMap": "No solutions",
  "errors": [
    {
      "type": "ErrorInfo",
      "error": "sh:notNumericError FocusNode: \"0.002996\"^^<http://www.w3.org/2001/XMLSchema#double> NotNumeric violation. Expected \"0.002996\"^^<http://www.w3.org/2001/XMLSchema#double> to be a number Node: <http://example.org/echotime>, Constraint: _:f92ea4e9dfea4c6b207b56bc9fb9c96c, path: PredicatePath(<http://schema.org/value>)"
    }
  ]

While a xsd:decimal as schema:value type pass the validation:
data
:echotime  a schema:QuantitativeValue ;
        schema:unitCode  "second" ;
        schema:value     "0.002996"^^xsd:decimal .

An untyped schema:value also pass the validation:
data
:echotime  a schema:QuantitativeValue ;
        schema:unitCode  "second" ;
        schema:value     0.002996 .

I've looked at the code and performed some debug. Here are what I've seen as potential sources of the above issues (please correct me if I'm wrong):

Looking at  the code at JenaMapper.jenaNode2RDFNode(r: JenaRDFNode): RDFNode, it looks like only IntegerLiteral and DecimalLiteral are handled for numeric values. DoubleLiteral is not there and double values are converted to DatatypeLiteral probably.

minInclusive constraint is checked by calling compareIntLiterals(n, _ <= _, minInclusiveError, "minInclusive") which relies on checkNumeric. In the checkNumeric function IntegerLiteral, DecimalLiteral and DoubleLiteral are checked and cast to int. If the value to be validated is not successfully converted to IntegerLiteral, DecimalLiteral or DoubleLiteral and its floor number inferior or equal to (<= ) to the provided value in the schema then the validation will fail. It is easy then to see that for values between [0,1] the cast to an int value will always give zero.

This is valid for minInclusive, maxInclusive, minExclusive, maxExclusive.
Any idea why all numeric values are cast to int when they are checked for minInclusive, maxInclusive, minExclusive, maxExclusive constraints ?

Thanks for your report. You were right in both issues. I have repaired them, and it seems to work now as expected.
As an example, you can check the following link


Align Type aliases between netfx and netcore

This PR solely aligns type aliases between netfx and netcore versions so they can be merged easier later.
Part of #2953

Looks good, but going forward, I think we can combine a couple of PRs like this together. I'm starting to get fatigue from reviewing these all independently 😅

Apologies, I really thought it makes it easier for review to have smaller, more targeted PR's... Guess I was wrong :)
Will combine them in the future

There's definitely a balance :) I think we'll still take stuff when it's bite-sized like this, but it just might be slow with all the overhead. On the other hand, gigantic PRs are a lot more likely to just be outright rejected

/azp run

/azp run

AZP needs a smack on the head

@cheenamalhotra Anything I can do to move this along? Thx


Treat bindata.go as autogenerated

Bindata.go is auto-generated so add to .gitignore and have Dockerfile regen on container build
Fixes #58
Signed-off-by: Steve Sloka<EMAIL_ADDRESS>

Looks really good to me @stevesloka! This will definitely be handy!


[Java] When user disable class registration, turn off frequent and annoying log warnings

What do these changes do?
When user decides to disable class register, we have enough log to show risk, this PR turn off frequent and annoying log warnings in high concurrency environment.
Related issue number
none
Check code requirements

[ ] tests added / passed (if needed)
[ ] Ensure all linting tests pass, see here for how to run them

@wtis-cj Thanks for your contribution, the class registration log can be used for security audit. Is is possible to add a new switch in FuryBuilder for disabling log instead of just skipping it?

@chaokunyang add config: classRegistrationWarningIgnored, default false.
Only takes effect when using FuryBuilder#requireClassRegistration(false)

These suggestions make no sense

@wtis-cj Thanks for your suggestion and contribution. The method naming is a history issue. We have two kinds of naming in FuryBuilder:

withXXX: a builder style naming
verbXXX: ignoreTimeRef/requireClassRegistration

I try to keep naming less messy, but I must admit that we should keep this from new contributor to reduce their work.


init fails on buildkite

The init environment fails for the latest main branch on buildkite: https://buildkite.com/clima/surfacefluxes-ci/builds/1106#_ It looks like some package issues. @akshaysridhar or @charleskawczynski Could you take a look? Thanks!

Hm, looks like we may just need to update climacommon? ClimaCore is now on modules: climacommon/2024_05_27. It looks like https://github.com/CliMA/SurfaceFluxes.jl/pull/172 was approved, but never merged due to some failure. I can try to rebase it.

Closed by #172

Ah, ok, thank you!


[alpha][fix-5538] : sync Observer 'current' properties when optimistic reading occurs

This is a follow-up to https://github.com/TanStack/query/pull/5573 for the v5-alpha version

When I run tests locally, I get a weird behavior of failures, but this one seems to be consistent: https://cloud.nx.app/runs/W8l5UK3y5A

The test in vueQueryPlugin.test.ts seems to regress, not sure why .. I tried seeing the observer part and it seems to work okay.
useQuery tests seem weird, some tests when ran independently pass, and do not pass under pnpm test locally.

/cc @TkDodo do you know why tests did not get executed for this PR ? I think we should give it extra care.
Also, when running tests locally from my IDE, I needed to rebuild the query-core manually to take changes.

do you know why tests did not get executed for this PR ? I think we should give it extra care.

because GitHub sees you as a first time contributor and thus doesn't run the pipeline until someone approves :/ Have a look at the nx cloud report now: https://github.com/TanStack/query/pull/5611#issuecomment-1597122428

Also, when running tests locally from my IDE, I needed to rebuild the query-core manually to take changes.

yeah this is a recent change by @lachlancollins in the alpha branch, I also stumbled upon it.
You'd need to use the pnpm run watch command in the root to automatically pick up changes.

@incepter ignore the vue-query tests, seems to fail on alpha everywhere. but the react-query test is probably an issue:

https://cloud.nx.app/runs/cpF8yZtCAu

Hey @TkDodo , sorry for the late reply.
You are right, in v5, placeHolderData acts like keepPreviousData too and has been enhanced.
The fix I found is to completely remove the need of testing on placeholder data, just assign the current properties whenever we have a new result.
The Tests fail in my windows machine which makes me always question whether the failure is real or not. Im not sure if we can improve this behavior

thanks. The missing errors are just flakiness I guess. I'll likely merge both solutions (main + alpha) today

@incepter the vue test fails locally for me as well on this PR.
@DamianOsipiuk could you maybe have a look at this here?

the test in question works fine locally on the alpha branch, so maybe we are regressing here after all

the test in question works fine locally on the alpha branch, so maybe we are regressing here after all

Yeah it is surely a regression, since leaving the properties alone and not assigning anything in the observer would make the test pass.
I fact, this test was about to drive me crazy today since I do not know vue. But managed to finally find the culprit:
When we assign the current properties of the observer, say via combineResult, then calling updateResult won't actually result in notifying the component, because updateResult would call again createResult and get an object that shallow equal the previous result, then it won't get notified because the updateResult function just quits when objects are shallowly equal.
I think I've hit the end of the road this way. Any solution would I think of using the globals to make and would not be predictable and easy.
Do you guys think of a solution ? I think the v5 would follow new mechanics, I will be thinking of a new solution that would work and not break anything.

Adding this seems to fix the failing vue test. Im not sure of the used approach but it seems that it s broken in that function and we could solve it this way.

can I somehow run the CI before the approval ? It kind of annoys me not to be able to have the same behavior locally with the tests.

sorry, it's a github thing for first time contributors :/ I also think its annoying

Codecov Report
Patch coverage has no change and project coverage change: +0.65 :tada:

Comparison is base (f57cebe) 89.03% compared to head (2f99fca) 89.69%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##            alpha    #5611      +/-   ##
==========================================
+ Coverage   89.03%   89.69%   +0.65%
==========================================
  Files          72       63       -9
  Lines        2390     2134     -256
  Branches      622      530      -92
==========================================
- Hits         2128     1914     -214
+ Misses        216      188      -28
+ Partials       46       32      -14

Impacted Files
Coverage Δ

packages/query-core/src/queryObserver.ts
83.50% <0.00%> (-2.67%)
:arrow_down:

... and 9 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Drop messages from batch on disconnect

When we're disconnected from the Azure we lose lock on the messages that we're processing and they are redelivered. This causes both, premature batch flush and duplicated claims. For example:

assume  we have batch configured to max 4 elements
we receive messages: a,b,c
azure disconnects so we lose lock on messages a,b,c
we reconnect
azure redeliver messages a,b,c
first batch gets full (messages: a,b,c,a) so it's flushes
we detect duplicates but the batch is already flushed so we register claims a, b, and c (3 despite batch size is set to 4)
messages b and c starts a new batch
we receive new claims d, e
the second batch is triggered (with messages b,c,d,e) there are no duplicates in this batch so all of them get registered

After this we end up with 2 transactions:

with claims a,b,c
with claims b,c,d,e
So the first batch isn't full and the second contains duplicates.

To solve this issue this PR changes that on disconnect all messages are invalidated before the batch is triggered so it's not flushed due to redelivered messages.

@emeka sure, updated description with some example


Backport empty memories fix and make 110 point release

In https://github.com/WebAssembly/binaryen/pull/5145,
Binaryen has made a fix for https://github.com/paritytech/substrate/pull/12280#issuecomment-1281564349.
We can probably backport this to Binaryen 110, which we are already forking, and make a point release that wasm-builder can use.

Done.


[operator] Manage nginx-ingress-{controller,k8s-backend}

How to categorize this PR?

/area dev-productivity
/kind enhancement
What this PR does / why we need it:
With this PR gardener-operator starts to manage nginx-ingress-controller & nginx-ingress-k8s-backend deployments sharing the same code as seed nginx-ingress-controller.
Co-authored by @vlvasilev
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/7016
Special notes for your reviewer:
Release note:

`gardener-operator` is now managing `nginx-ingress-controller` & `nginx-ingress-k8s-backend`

/assign

/retest

The Integration and E2E seem to passed locally:
(⎈|N/A:N/A)➜  gardener git:(add_nginx-ingress_to_gardener_operator) ./hack/test-integration.sh ./test/integration/operator/garden
> Installing envtest<EMAIL_ADDRESS>with setup-envtest if necessary
using envtest tools installed at '/Users/i353953/Library/Application Support/io.kubebuilder.envtest/k8s/1.26.1-darwin-arm64'
> Integration Tests
ok  	github.com/gardener/gardener/test/integration/operator/garden	16.849s

#####

(⎈|N/A:N/A)➜  gardener git:(add_nginx-ingress_to_gardener_operator) make test-e2e-local-operator
go build -o hack/tools/bin/ginkgo github.com/onsi/ginkgo/v2/ginkgo
./hack/test-e2e-local.sh operator --procs=1 --label-filter="default" ./test/e2e/operator/...
> E2E Tests
Running Suite: Test E2E Operator Suite - /Users/i353953/go/src/github.com/gardener/gardener/test/e2e/operator
=============================================================================================================
Random Seed:<PHONE_NUMBER>

Will run 2 of 2 specs
------------------------------
Garden Tests Create Garden, Rotate Credentials and Delete Garden [Garden, default, credentials-rotation]
/Users/i353953/go/src/github.com/gardener/gardener/test/e2e/operator/garden/create_rotate_delete.go:41
  > Enter [BeforeEach] TOP-LEVEL @ 06/15/23 08:22:41.16
  < Exit [BeforeEach] TOP-LEVEL @ 06/15/23 08:22:41.188 (28ms)
  > Enter [It] Create Garden, Rotate Credentials and Delete Garden @ 06/15/23 08:22:41.188
  STEP: Create Garden @ 06/15/23 08:22:41.188
  STEP: Verify CA secrets of gardener-operator before rotation @ 06/15/23 08:27:12.209
  STEP: Verify old etcd encryption key secret @ 06/15/23 08:27:12.225
  STEP: Verify old etcd encryption config secret @ 06/15/23 08:27:12.236
  STEP: Verify old service account key secret @ 06/15/23 08:27:12.242
  STEP: Verify secret encryption before credentials rotation @ 06/15/23 08:27:12.252
  STEP: Request new client certificate and using it to access virtual garden @ 06/15/23 08:27:13.688
  STEP: Request new dynamic token for a ServiceAccount and using it to access target cluster @ 06/15/23 08:27:13.802
  STEP: Start credentials rotation @ 06/15/23 08:27:13.855
  STEP: Verify CA secrets of gardener-operator after preparation @ 06/15/23 08:35:15.734
  STEP: Verify etcd encryption key secrets @ 06/15/23 08:35:15.748
  STEP: Verify combined etcd encryption config secret @ 06/15/23 08:35:15.769
  STEP: Verify service account key bundle secret @ 06/15/23 08:35:15.775
  STEP: Verify secret encryption after preparing credentials rotation @ 06/15/23 08:35:15.799
  STEP: Use client certificate from before rotation to access target cluster @ 06/15/23 08:35:15.86
  STEP: Use dynamic ServiceAccount token from before rotation to access target cluster @ 06/15/23 08:35:15.878
  STEP: Request new client certificate and using it to access target cluster @ 06/15/23 08:35:15.908
  STEP: Request new dynamic token for a ServiceAccount and using it to access target cluster @ 06/15/23 08:35:26.057
  STEP: Complete credentials rotation @ 06/15/23 08:35:26.12
  STEP: Verify CA secrets of gardener-operator after completion @ 06/15/23 08:39:37.61
  STEP: Verify new etcd encryption key secret @ 06/15/23 08:39:37.62
  STEP: Verify new etcd encryption config secret @ 06/15/23 08:39:37.63
  STEP: Verify new service account key secret @ 06/15/23 08:39:37.636
  STEP: Verify secret encryption after credentials rotation @ 06/15/23 08:39:37.65
  STEP: Use client certificate from before rotation to access target cluster @ 06/15/23 08:39:37.705
  STEP: Use dynamic ServiceAccount token from before rotation to access target cluster @ 06/15/23 08:39:47.705
  STEP: Use client certificate from after preparation to access target cluster @ 06/15/23 08:39:57.706
  STEP: Use dynamic ServiceAccount token from after preparation to access target cluster @ 06/15/23 08:39:57.749
  STEP: Request new client certificate and using it to access target cluster @ 06/15/23 08:39:57.778
  STEP: Request new dynamic token for a ServiceAccount and using it to access target cluster @ 06/15/23 08:39:57.915
  < Exit [It] Create Garden, Rotate Credentials and Delete Garden @ 06/15/23 08:39:57.978 (17m16.791s)
  > Enter [DeferCleanup (Each)] Garden Tests @ 06/15/23 08:39:57.978
  STEP: Clean up objects in virtual garden from client certificate access @ 06/15/23 08:39:57.984
  STEP: Clean up objects in shoot from dynamic ServiceAccount token access @ 06/15/23 08:39:58.034
  < Exit [DeferCleanup (Each)] Garden Tests @ 06/15/23 08:39:58.071 (93ms)
  > Enter [DeferCleanup (Each)] Garden Tests @ 06/15/23 08:39:58.071
  STEP: Delete Garden @ 06/15/23 08:39:58.071
  < Exit [DeferCleanup (Each)] Garden Tests @ 06/15/23 08:40:34.276 (36.205s)
• [1073.117 seconds]
------------------------------
Garden Tests Create, Delete [Garden, default, simple]
/Users/i353953/go/src/github.com/gardener/gardener/test/e2e/operator/garden/create_delete.go:47
  > Enter [BeforeEach] TOP-LEVEL @ 06/15/23 08:40:34.276
  < Exit [BeforeEach] TOP-LEVEL @ 06/15/23 08:40:34.289 (13ms)
  > Enter [It] Create, Delete @ 06/15/23 08:40:34.289
  STEP: Create Garden @ 06/15/23 08:40:34.289
  STEP: Verify creation @ 06/15/23 08:44:31.374
  STEP: Verify virtual cluster access using token-request kubeconfig @ 06/15/23 08:44:31.426
  < Exit [It] Create, Delete @ 06/15/23 08:44:31.464 (3m57.175s)
  > Enter [DeferCleanup (Each)] Garden Tests @ 06/15/23 08:44:31.464
  STEP: Delete Garden @ 06/15/23 08:44:31.464
  STEP: Verify deletion @ 06/15/23 08:45:07.679
  < Exit [DeferCleanup (Each)] Garden Tests @ 06/15/23 08:45:07.708 (36.244s)
• [273.432 seconds]
------------------------------

Ran 2 of 2 Specs in 1346.550 seconds
SUCCESS! -- 2 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS

Ginkgo ran 1 suite in 22m36.402229916s
Test Suite Passed

The Integration and E2E tests seem to pass locally:

github.com/gardener/gardener/test/integration/gardenlet/seed/seed integration test is failing, you tested github.com/gardener/gardener/test/integration/operator/garden locally.
Also please check the failing unit test.
Operator test is flaking as of now ref- https://github.com/gardener/gardener/issues/8036, rerun should pass.

/remove-lifecycle stale


Android app does not work properly

Hi,
migration is over (from HockeyApp) and ios is now working normally but Android only dumps the ThirdParty side, and you can see the normal call stack, and UE4 internal native code is not properly analyzed and is stacked in Unsymbolicated in Diagnostics.
Step1.
Curl -X POST "https://api.appcenter.ms/v0.1/apps/{0}/{1}/release_uploads\" -H "Accept: application/json" -H "X-API-Token: {2}" -H "Content-Type: application/json""
Step2.
if (IsAndroid)
step2Args = string.Format("-F "ipa=@{0}" -F "libs=@{1}" {2}", ApkFilename, Symbols.zip, Step1 upload_url);
else
step2Args = string.Format("-F "ipa=@{0}" -F "dsym=@{1}" {2}", ipaFilename, X.dSYM.zip, Step1 upload_url);
Step3.
Curl -X PATCH "https://api.appcenter.ms/v0.1/apps/{0}/{1}/release_uploads/{2}\
Final
Curl -X PATCH "https://api.appcenter.ms/v0.1/apps/{0}/{1}/releases/{2}\
(When creating an APK, I created a Proguard setup, obfuscated, and compressed the code using the Distribution option.)
Have you missed anything while migrating to AppCenter?

@haneal can you please reach out to us via support so we can better investigate the problem? You can mention this issue.


pipeline does't want to pause

i use a camera DMK37BUX273 and I have ubuntu OS. I can run tcam-caputure and gst-launch-1.0 tcambin ! videoconver! ximagesink, but I can't able to save that stream . When I use the lauch code from save_stream_file (like gst-lauch-1.0 tcambin ! video.... ! tee ......queue......ximagesink t.!......avimux!..... filesink name=fsink): this one is provided in the tis docu in example section, it says that error: pipeline does't want to pause, error: from element.....No file name specified for writing. ....Freeing pipeline.......
How can I save that live stream video with a small size in a avi file format? Please help me out? I mean I can watch the video and can't save it using the provided example code by tis docu. What may be the issues? Any leads?
Thank you very much!

You do not have audio, so following should be fine:
save to *.mp4 file
gst-launch-1.0 -v -e tcambin
! videoconvert
! x264enc
! mp4mux name=mux
! filesink location="bla.mp4"

Yes, thank you very much Stefan, you are a lifesaver. Sorry, I did not check my email.
It works now. Thanks a lot!

:-)


Albums are displayed as artists in 'Browse'

Hi!
Albums are displayed as artists in the browse section. And the songs in the albums are displayed as albums. The songs ('albums') then dont contain anything. For artists at the top level of browse, everything is fine, meaning that I get to the albums after clicking the artist and the albums contain playable songs. Can you change the detection so that albums are not displayed as artists anymore? Or might this be an issue with my music library? I checked this in my emby webinterface and there it was fine.
Best, vrs

thats really strange cause i cant reproduce this. could you run mopidy in debug mode? what frontend do you test this with?

I'm tested it with ncmcpp and Musicbox-Webclient.


gccgo: single-clause select improperly lowered (missing interface conversion?)

What version of Go are you using (go version)?
tip gccgo

$ go version
go version go1.12.2 gccgo (GCC) 10.0.0 20190719 (experimental) linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
linux/amd64
What did you do?
Run this program:
https://play.golang.org/p/URnDqSGuHmp
Code:

package main

import "fmt"

type SC interface {
	C()
}

type testSC struct {
	id string
}

func (tsc *testSC) C() {
}

func (tsc *testSC) String() string {
	return tsc.id
}

func main() {
	ch := make(chan SC, 10)
	tsc := testSC{id: "blah"}
	select {
	case ch <- &tsc:
	default:
	}
	sc := <-ch
	fmt.Printf("%p %v\n", sc, sc)
}

What did you expect to see?
Successful run with output
pointer-value "blah"
What did you see instead?

From what I can tell in the debugger, the value 'tsc' on line 18 is being passed directly to the channel send as a struct, as opposed to being converted to an interface value.
Also of interest is that if you change line 13 to rename the testSC "C" method to something like "D", the compile continues to go through, as opposed to what we want which is a compiler error. This seems likely to be related.

CC @cherrymui

I poked around a little on this.  Looking at how things worked prior to CL 184998, in clauses::Select_clause::lower_send there is this code:
// Note that copying the value to a temporary here means that we
// evaluate the send values in the required order.
Temporary_statement* val = Statement::make_temporary(valtype, this->val_,
loc);
which introduces a type conversion from the RHS type (testSC) to the channel element type (a.SC). This conversion isn't getting created after 184998.   I'll see if I can figure out the right place to restore the type conversion.

Thanks for looking into this. You're right about the temporary statement. In Select_statement::lower_two_case we didn't use the right type to create the temporary.


Accordion cleanup.

Description
Additional Cleanup around the accordion code and out of the box styling. The styles should be revisited.
Type of change
Please delete options that are not relevant.

[X] Breaking change (Accordion label moved to a slot)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] Visually
[X] yarn test

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] My changes generate no new warnings
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules

Waiting on heading changes from https://github.com/phase2/outline/pull/237 before this is ready to merge.


Python: Bug: [PYTHON] Chat history serializing error

Describe the bug
I have a user message and an azure open ai assistant message in the chat history. I'm trying to serialize the chat history object to a json string so I can save it. When I try to use the serialize function in the chat history class, I get the error below.
"Error: Unable to serialize ChatHistory to JSON: Error serializing to JSON: TypeError: 'MockValSer' object cannot be converted to 'SchemaSerializer'"
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Get a json string
Screenshots
If applicable, add screenshots to help explain your problem.
Platform

OS: Windows
IDE: VS Code
Language: Python
Source: pip package version 1.1.2

Additional context
Add any other context about the problem here.

Hi @nguyena2 are you able to provide us with more information (possibly the entire chat history if it doesn't contain sensitive info) about what is contained in the chat history and why the serialize/model dump would be failing?

@moonbox3 @markwallace-microsoft I can confirm that we are seeing this issue (or something similar) as well.
pf.flow.test failed with UserErrorException: PydanticSerializationError: Execution failure in 'agent': (PydanticSerializationError) Error serializing to JSON: PydanticSerializationError: Unable to serialize unknown type: <class 'type'>
I am able to replicate this in the function calling stepwise planner starting in Semantic Kernel 1.2.0 simply by adding the SK Core Time Plugin to the kernel and asking a question that invokes any of those functions.
At the end of the process we use ChatHistory.model_dump_json() (same as is being called in ChatHistory.Serialize()) to output the steps taken by the planner and this is what is triggering the error. I am able to successfully use the ChatHistory.model_dump() to explore the Chat History object and I have not been able to identify anything about the payload itself that would be causing the serialization error.
I have replicated the behavior in Python 3.11 and 3.12, and can confirm that the issue can be resolved by moving the project back to Semantic Kernel 1.1.2.
If y'all feel that this is a separate issue, I am happy to document it in another Github Issue and I am available to demonstrate my example if so desired.

Thanks for the extra context, @denny-nolan-kochcc. Tagging @eavanvalkenburg for visibility.

@nguyena2  and @denny-nolan-kochcc the issue is caused by a change we made so that the result in a FunctionResultContent can now be any type, instead of before where we cast everything to a string, which was limiting other use cases, so could you check the types used internally that are being put in there, you might also do [msg.to_dict() for msg in chat_history.messages] for inspection purposes, while I work on a fix.

@nguyena2 and @denny-nolan-kochcc the issue is caused by a change we made so that the result in a FunctionResultContent can now be any type, instead of before where we cast everything to a string, which was limiting other use cases, so could you check the types used internally that are being put in there, you might also do [msg.to_dict() for msg in chat_history.messages] for inspection purposes, while I work on a fix.

@nguyena2 and @denny-nolan-kochcc the issue is caused by a change we made so that the result in a FunctionResultContent can now be any type, instead of before where we cast everything to a string, which was limiting other use cases, so could you check the types used internally that are being put in there, you might also do [msg.to_dict() for msg in chat_history.messages] for inspection purposes, while I work on a fix.

what are we looking at in this screen @nguyena2 ?

Hello, could you tell us when you are going to fix that?
Thanks,


Landing page errors for GCP000366

We’ve had an email from this user (submission https://www.ebi.ac.uk/gwas/deposition/submission/62a87ed1bc08100001ccd5e0) who noticed their GCSTs appear in the Catalog but once you click on them, all that appears is this message:
‘The study accession GCST90104542 cannot be found in the GWAS Catalog database’. eg. https://www.ebi.ac.uk/gwas/studies/GCST90104542
The sumstats are available on the FTP fine: http://ftp.ebi.ac.uk/pub/databases/gwas/summary_statistics/GCST90104001-GCST90105000/
This is a ticket that was open for this submission to edit the reported traits https://app.zenhub.com/workspaces/gwas-59df823c4a6feb3786810391/issues/ebispot/goci/783
I think there’s been a data release since the edit was made. Not sure if the trait edit is the cause of the issue. Please investigate.

@earlEBI  could you please confirm this has been fixed ? Thanks

@sprintell Seems fine now, yes.

The user has contacted us again because the message is appearing for all studies again.
I added PMID last week and published. The GCSTs look good on dev, but not in prod: https://www.ebi.ac.uk/gwas/studies/GCST90104542

Is the issue caused by every data release?

@earlEBI this seems to be ok currently, please confirm after next DR that it is still visible

@earlEBI could you please confirm this for the current DR in staging?

@ljwh2 looks fine


flutter: Move package overrides to build-support

Description of changes
This moves the Flutter package overrides to the build-support directory, as decided in https://github.com/NixOS/nixpkgs/pull/228282#discussion_r1177455022.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Not sure how relevant it is to this PR, but how would the package-overrides work when build support for dart is added? Would there be 2 package-overrides directories (one in build-support/flutter and another in build-support/dart)? Or will it be better to keep a single package-overrides directory in build-support/dart?

That's a good point. I was thinking about build-time C dependencies for Flutter apps when I first made the override system, but it is useful for supplying runtime dependencies for all FFI applications as well.
After #229770 is merged, I think it would make sense to move the override files to dart/package-overrides, implement the override system in buildDartPackage, and then rebase buildFlutterApplication on top of buildDartPackage with a custom buildScript.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2171

Most other eco systems I remember (ruby, rust, python, ...) have this split, to keep the buildSupport directory generic

I don't think package overrides align with builders and fetchers in build-support...


[1.18.2] Infinity Backpack's item auto-pickup doesn't work

Industrial Foregoing Version: industrial-foregoing-1.18.2-<IP_ADDRESS>-8
Titanium Version: titanium-1.18.2-3.5.6-38
Infinity Backpack doesn't pick up items as expected, even when fueled
The problem is - Backpack has already have stone stored inside and mined stone gets to player's inventory instead of backpack's inventory

Translation: Auto-pickup is set to "Items only" mode
P.S. All slots have "Void excess" and "Restock" modes turned off

Can confirm the issue, tested every setting and the item just lands inside the inventory.

I have also been having this issue, have tried everything but the item still goes to the inventory instead of the backpack.


Python wrapping: Segment tube using minimum path

Work In Progress

Hi - please check line lengths (80 max) and then it can be merged!


Enable EFK as an installation choice in the on-boarding package

split efk installation config in a seperate file: docker-compose-efk.yml

split efk installation config in a separate file docker-compose-efk.yml


grunt-cli not found when running embark run

Running "concat:css" (concat) task
File generated/dapp/css/app.min.css created.

Done, without errors.
Completed in 0.938s at Fri Jun 26 2015 18:56:00 GMT+0900 (JST) - Waiting...
>> File "app/js/index.js" changed.
>> Local Npm module "grunt-cli" not found. Is it installed?

Running "concat:app" (concat) task
File generated/dapp/js/app.min.js created.

Running "concat:css" (concat) task
File generated/dapp/css/app.min.css created.

just a warning. but should be removed in 0.5.0 or 0.6.0

fixed in 0.9.0


Comments unreadable on VCS diffs

The background color for added lines in a version control diff makes it hard to read comments.

Hi @bendardenne :wave:, thanks for your contribution :+1:
It looks like you're using a outdated version of the Nord plugin. The colors from your screenshot are related to PR #25 which reduced the alpha layer value, but was not working well with comments, so in #159 the colors were changed to match the ones also used in the Nord Visual Studio Code port project. The change was shipped with the Nord plugin version 0.12.0.
❗ Please also note that modifying editor schemes of plugins will create a copy of the scheme that is placed in the colors folder within your default IDE configuration directory directory and that this copy won't be updated when the plugin gets updated.
Unfortunately JetBrains hasn't solved this "problem" which often caused false-positive issues in theme repositories regarding "non-working theme plugins" or "outdated styles".
In order to update your copy you either need to manually edit the theme files and add changes made to the plugin or delete the copy and re-create it with you custom style based on the latest plugin version. You can export your customized theme as XML file or dedicated theme plugin to backup your changes later on and when updating to later Nord plugin versions later on.
Can you please make sure that you're running the currently latest Nord plugin version and you're not using a customized editor scheme by checking the content of your colors folder?

Can this be closed as stale?


fix warnings: syncShutdownGracefully not available in async contexts

Motivation:
Warnings are annoying. Companion of #2994
Modifications:
Fix all the syncShutdownGracefully not being available in async contexts warnings.
Result:
Everybody happier.

/__w/swift-nio/swift-nio/Tests/NIOPosixTests/SerialExecutorTests.swift:59:13: error: initialization of immutable value 'eventLoopGroup' was never used; consider replacing with assignment to '_' or removing it
        let eventLoopGroup = self.group
        ~~~~^~~~~~~~~~~~~~


Killer coe 3- lost all my work, black page only HELP!

I HAVE KILLER CODE 3 and can not access any of my Visual Code files, nothing will reopen as on the message, just a black screen, I have no files

Can you share a screenshot?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Removed literals.go from flyteplugins and reusing it from flyteidl

TL;DR
Please replace this text with a description of what this PR accomplishes.
Type

[ ] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
https://github.com/flyteorg/flyte/issues/757
Follow-up issue
NA

Why did the coverage drop? otherwise lgtm

@kumare3 if you look at the coverage details

59.93% 100% -1.56%
So the changes in coverage are 100% covered but the total coverage reduced by 1.56 % since removal of the files which were contributing to the coverage
The change show -325 lines, but it doesn't out of what total. But it must equal to 1.56 % which got deleted.


Update database compatibility

Purpose

Update Database compatibility to include oracle 19c

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


Fix - docker apt package cache

When we run apt-get update in a Dockerfile (or any similar command), we need to make sure that it is not using the cache. This PR adds a fix to our docker files so that apt cache is busted when it is updated
See approach here: http://lenguyenthedat.com/docker-cache/

 Bust cache when running apt-get update


Add theme related req to requirements.txt

darkdetect and qdarkstyle are 2 optional dependencies for theme management.
I think they are used by mne-python before invoking the browser, but since theme selection is a key feature of the browser, how about adding them as a requirement here as well? This way, a base install of mne will include them by default.
If they are used by other GUI through mne-python, then maybe it would be worth adding them to one of the requirements_extra.txt of mne-python as well.

I also think it should be hard requirements and therefore go to setup.py or .cfg as well

Yep, I only now realize that the setup.py does not read the requirements from the .txt file. Is this on purpose?

Is this on purpose?

It's possible that requirements.txt includes/included all optional dependencies, like the requirements.txt in MNE-Python. After adding qdarkstyle and darkdetect, though, there might not be any more optional dependencies (other than PyOpenGL, which is handled via an [opengl] option already)

Actually, it was missing some dependencies and it was missing version limitations. So I guess there is no point in not loading the dependencies from the .txt file. I added the parser from MNE.

Thanks @mscheltienne !

We don't we drop requirements.txt instead?
IMHO we're not making proper use of it anyway: it's supposed to specify exact versions to be used for a specific deployment, not just a list of deps with loose version constraints (or none at all). This is what should go to setup.py
As it is now, one has to manually "parse" the code that imports requirement.txt in setup.py to figure which are the actual dependencies of this package...

IMHO we're not making proper use of it anyway: it's supposed to specify exact versions to be used for a specific deployment, not just a list of deps with loose version constraints (or none at all).

You mean static pins? Or the whole dependency tree (including dependencies of dependencies of ...)? This actually hasn't been how I've seen it used most places
https://github.com/pandas-dev/pandas/blob/main/requirements-dev.txt
https://github.com/scipy/scipy/blob/main/doc_requirements.txt
And it's not how we've used it in MNE. I'd rather stay consistent with how we've used it in MNE as much as possible.

We don't we drop requirements.txt instead?

As a dev I've found it useful multiple times to have a requirements.txt that I could pip -r with first. It's also a nice standard format that's easy to read (and parse, etc.).


Fix the NPM listing

Still fix for MMENG-2719, it should not append package.json to browser requests, that should be just for api/content.

Okay, I will merge this first for tests. thanks.


Use PackageName name as the name of subscription

What this PR does / why we need it:
This fix will ensure that no duplicate subscriptions are created based on the same operator package name and different channels. Instead, if there are two OperandRequests with the same operator but different channels, ODLM will create only one subscription with the package name, as only one operator/subscription with a single channel is allowed to be installed in one tenant.
Which issue(s) this PR fixes
Fixes # https://github.ibm.com/IBMPrivateCloud/roadmap/issues/64818
Before/After fix in
Request two same operator with diff channel
apiVersion: operator.ibm.com/v1alpha1
kind: OperandRequest
metadata:
  name: example-service
spec:
  requests:
    - operands:
        - name: cloud-native-postgresql-v1.22
        - name: cloud-native-postgresql
      registry: common-service

Before fix will see fail to create two subs:

After fix, successfully created one sub:

/retest


MSL: Add support for texture arrays

We add 0.5 to layer index to match round-to-nearest semantics expected by HLSL code.
Without doing that, on NVidia GPUs rasterizer can have a slight rounding error if you pass the texture array layer via interpolator, and the implicit cast inside sample call would truncate something like 3.999999 to 3.

Looks good, thanks!


removed '?' from output 'Is CA?: true/false'

the output states 'Is CA?:'  where I find the question mark not needed/of value but more confusing as of why is there no question mark after 'Name Valid'?.

Good find! I just merged it with some new changes and released it via v1.2.1  (see releases)
Thanks!


Windows on Arm is not supported ?

Hello,
I'm getting the following error when importing the AzureAD module. I'm using Windows on Arm, is there a workaround ?
Import-Module : The current processor architecture is: X86.  The module 'C:\Program Files
(x86)\WindowsPowerShell\Modules\AzureAD\<IP_ADDRESS>\AzureAD.psd1' requires the following architecture: Amd64.

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

ID: 156a552a-fe64-e5dc-b86a-b7d8ccbeee1d
Version Independent ID: d83f235e-3ae2-ad71-30c6-0bd7f95fb894
Content: Windows Virtual Desktop service principal role assignment - Azure
Content Source: articles/virtual-desktop/create-service-principal-role-powershell.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@irvinhg Thanks for the feedback. We are investigating the issue and will update you shortly.

@ChristianMontoya Can you please add your comments on this.

@irvinhg This is a compatibility issue as per the error you have shared.
Requesting you to open a support ticket for troubleshooting the issue.
If you don't have the ability to open a technical support ticket, you can email me with Subject as "ATTN: Vikas" at<EMAIL_ADDRESS>and provide me with your SubscriptionID and link to this thread. I can then enable your subscription for one time free support.

@irvinhg Any update on the issue, please open a support request or as mentioned earlier if you don't have the capability to open support case email me the requested details.
Also, once you get the issue fixed, request you to reply back here on the thread with the resolution steps for the benefit of the community.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


Modify configs of 01-ai/Yi-34B-Chat to make model verified

Modify the configs.yaml file to a script without requiring a local GPU, in order to verify our model.

Thanks @HyperdriveHustle I'm trying to run that now but this will require a GPU as huggingface_local_completions uses local inference.
Do you have it on a hugging face inference endpoint instead? or anywhere else I could run decoding using an API key?
You can message me at @yann on Discord if it's easier there


Develop

Added update APIs for Triggers and Macros, and CRUD For Automations.

@stephenc Please review and Merge.


vencord: 1.10.2 -> 1.10.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for vencord is: Vencord web extension
meta.homepage for vencord is: https://github.com/Vendicated/Vencord
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.10.3 with grep in /nix/store/c5663prbl8p8qbgrrzc9x7h1k50xmz9n-vencord-1.10.3
found 1.10.3 in filename of file in /nix/store/c5663prbl8p8qbgrrzc9x7h1k50xmz9n-vencord-1.10.3

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

vencord
vencord-web-extension
webcord-vencord

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/c5663prbl8p8qbgrrzc9x7h1k50xmz9n-vencord-1.10.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A vencord https://github.com/r-ryantm/nixpkgs/archive/b87c518e5c52d890fd404ba75ce66594088a680b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b87c518e5c52d890fd404ba75ce66594088a680b#vencord

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/c5663prbl8p8qbgrrzc9x7h1k50xmz9n-vencord-1.10.3
ls -la /nix/store/c5663prbl8p8qbgrrzc9x7h1k50xmz9n-vencord-1.10.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

vencord
vencord-web-extension
webcord-vencord

Maintainer pings
cc @FlafyDev @NotAShelf @Scrumplex for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


Fix extension search

fixes #59750
Added search queue.

I agree that there is no need to wait previous searches. So I replaced search queue with cancelable promise.

You are right. I passed CancellationToken to ExtensionsListView#show as optional parameter and turned off view update if promise was canceled. Also I found out that onDidAddViews callback makes search query and wrapped search code in a cancelable promise.

Conflicts resolved

Fixed and hence closing this PR


angular-katex auto-render show error ParseError: KaTeX parse error: Unexpected

how to make angular-katex auto-render,it show error ParseError: KaTeX parse error: Unexpected character: '$' at position 0: ̲$When a \ne 0$

The ''$' characters are not necessary. Everything you type in the input will be in math mode.

I suppose you want "When" to appear as text. You can use the \text command:

Let me know if this fixes your problem.

aha,you are very kind @tfoxy .
when i print "When a \ne 0, there are two solutions to (ax^2 + bx + c = 0) and they are x = {-b \pm \sqrt{b^2-4ac} \over 2a}." in live demo,it show

show word without blank.
i add $ delimiter expect angular-katex just compile the math expression between $ show like this

i also want this work well in chinese.
i print  数学公式：$\left(\sum_{i=1}^{k}i\right) +(k+1)$

and want it show like this

could give me a live demo or some help please.Thinks.
i have try your suggest and it didn't work

I think I understand what you want.
You want to use the auto-render extension.
In the following demo, I added the katex-auto-render directive:
Live Demo
Does this behaves as you expected? If it does, I will push the changes and release a new version.

Aha…very very perfect, it is just what i want.
Waiting for you pull it soon.

I just uploaded version 0.8.0
To use the auto-render function, you need to include the auto-render.min.js file from katex. If you use bower, it is inside the dist/contrib folder. If not, you can download it from the katex releases.
By default, this extension does not use $ as a delimiter. The default options are in the README.
But you can add the $ delimiter in the config phase of angular:
angular.module('app', [])
  .config(function(katexConfigProvider) {
    katexConfigProvider.defaultOptions.delimiters = [{left: "$", right: "$", display: false}]
  });

Finally, in the html, you need to add the katex-auto-render attribute.

I hope these instructions are clear.

yes, it is work smoothly after your help,thanks again.

Great! I'm glad it worked. I'm closing this issue.


Support document type "Supplement"

NIST has the "Supplement" document type and also the Addendum (https://github.com/metanorma/metanorma-nist/issues/259).
"Supplement" is a numbered attribute like Addendum:

"Supplement 1" is just called "Supplement"
"Supplement 2" is just called "Supplement 2"

Examples:

https://nistdigitalarchives.contentdm.oclc.org/digital/collection/p15421coll1/id/8075
https://nvlpubs.nist.gov/nistpubs/Legacy/SP/nistspecialpublication305supp22.pdf
https://books.google.com.hk/books?id=cWcEAQAAIAAJ&dq=nist+special+publication+supplement+-172

Representation: e.g.

In PubID, the "Supplement":

short: "Suppl. X"
long: "Supplement X"

Originally posted by @ronaldtse in https://github.com/metanorma/nist-pubid/issues/1#issuecomment-884927792

From NIST PubID 2021-08.

This is already implemented.


Stone Paper Scissor Via Recusrion

📥 Pull Request
issue  #1547
Description
The Stone-Paper-Scissors Game is a classic hand game that involves two players making simultaneous choices between three options: Stone (Rock), Paper, and Scissors. Each option has specific interactions that determine the outcome: Stone crushes Scissors, Scissors cut Paper, and Paper covers Stone. This simple yet strategic game is not only fun but also a quick way to settle disputes or make decisions.
In the digital version of the game, players can compete against a computer opponent or each other. The game features an intuitive interface for selecting moves, real-time feedback on the outcomes of each round, and score tracking to determine the overall winner after multiple rounds. The combination of chance and psychology makes it an engaging experience, as players attempt to predict their opponent's moves while strategically choosing their own.
With its straightforward rules and fast-paced nature, the Stone-Paper-Scissors Game appeals to players of all ages. It's an excellent way to develop quick thinking and decision-making skills while enjoying friendly competition. Whether played casually or as part of a larger game night, the game remains a timeless favorite that fosters interaction and fun.
Fixes  #1547
Type of change

[ ✅] New feature (non-breaking change which adds functionality)
[✅ ] Documentation update

Checklist:

[ ✅] My code follows the style guidelines of this project
[✅ ] I have performed a self-review of my code
[✅ ] I have commented my code, particularly in hard-to-understand areas
[✅ ] I have added tests that prove my fix is effective or that my feature works
[✅ ] New and existing unit tests pass locally with my changes
[ ✅] Any dependent changes have been merged and published in downstream modules

@ajay-dhangar , Please check it  now ; if is all okay or not

@ajay-dhangar , any Update please ?


Insert text for tab completion when there is only one completion.

When the only choice is "foo", and the user hits , the
input buffer should fill with "foo", instead of showing that
"foo" is the only selection.

:+1:

This is not a good idea since user may find the only choice is actually not what he wants, so maybe hit tag again (or hit escape) to go back to the original edit buffer.


Add optional ECR auth to nodeadm

Issue #, if available:
Description of changes:
allow ecr authentication when hitting non-open registries
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Testing Done

See this guide for recommended testing for PRs. Some tests may not apply. Completing tests and providing additional validation steps are not required, but it is recommended and may reduce review time and time to merge.

/ci build


object persistency in AR

I want to create an application that store the position,rotation and scale of multiple object and when i close application and reopen than all the stored object must be at there respective position.
I am able to place object with the stored position but issue is that i can't place than at correct position as earlier.
the object are overlapping with each other.
Do you have any solution around that.I am looking for android app side.

We currently don't have a solution for this. This is a new area of AR that people are trying to solve, some are calling it AR Cloud -> https://medium.com/super-ventures-blog/arkit-and-arcore-will-not-usher-massive-adoption-of-mobile-ar-da3d87f7e5ad
Here are some companies that are working on solutions for persistent AR/AR Cloud. We have not tried them ourselves but they might be what you are looking for:
Sturfee -> https://sturfee.com/
6D.ai -> https://twitter.com/6d_ai


directory structure for architecture-specific code and headers (2)

This brings the last bits from #12797:

remove remaining headers from src/platforms, and move to platforms/common/include (with transitional include support for px4_{defines,log,time}.h used in submodules).
remove src/drivers/boards (which contained arch-specific subdirectories for stm32 and kinetis) and move under platforms/nuttx/src/px4.

I do not like dropping the px4 prefix from px4_config.h

The prefix is now part of the include directory: <px4_platform_common/config.h> (For headers that only apply to the current platform/OS, it would be <px4_platform/xy.h>). If you prefer I can readd it for config.h.

This is too massive to review.

Yeah, sorry for that, it's not possible to break it down further while keeping commits atomic.

I do not like dropping the px4 prefix from px4_config.h

The prefix is now part of the include directory: <px4_platform_common/config.h> (For headers that only apply to the current platform/OS, it would be <px4_platform/xy.h>). If you prefer I can readd it for config.h.

Please do for just config.h

This is too massive to review.

Yeah, sorry for that, it's not possible to break it down further while keeping commits atomic.

No worries

Please do for just config.h

Done.


test connection but no respose

Hi,
I am new for zabbix and grafana. Need to try to use this nice application.
Having test connection during adding data source with below

No response from testing result.
Need your help.
Br,
hatta

Hi! It mean that grafana can't connects to Zabbix server due some reasons.

Hi alex,
How to solve it n debug it. No error  in zabbix server and grafana server.
Thx and Br,
Hatta

Hi Alex,
i got 421 response with below
<IP_ADDRESS> - - [17/Jan/2016:07:09:01 +0700] "GET /zabbix/api_jsonrpc.php HTTP/1.1" 412 - "-" "Mozilla/5.0 (Windows NT 6.1; WOW64; rv:18.0) Gecko/20100101 Firefox/18.0"
<IP_ADDRESS> - - [17/Jan/2016:07:10:23 +0700] "GET /zabbix/api_jsonrpc.php HTTP/1.1" 412 - "-" "Mozilla/5.0 (Windows NT 6.1; WOW64; rv:18.0) Gecko/20100101 Firefox/18.0"
[root@simulator /usr/share/zabbix]# zabbix_server -V
Zabbix server v2.4.7 (revision 56694) (12 November 2015)
Compilation time: Nov 13 2015 05:31:52
[root@simulator /usr/share/zabbix]# grafana-server -v
Version 2.6.0 (commit: v2.6.0)
api_jsonrpc.php already checked and same with installation procedure.
Br,
hatta

@hattaagung see https://github.com/alexanderzobnin/grafana-zabbix/wiki/Installation#note-for-zabbix-22-or-less

Hi alex,
Yes. I already checked it and it is same. I installed zabbix with versin 2.4.7.
Do you have another ideas ?
Br,
Hatta

@hattaagung it's a CORS issue. Check your web server configuration.

Hi Alex,
now come with new error like below

web server log is ok now
<IP_ADDRESS> - - [18/Jan/2016:10:51:42 +0700] "OPTIONS /zabbix/api_jsonrpc.php HTTP/1.1" 200 - "-" "Mozilla/5.0 (Windows NT 6.1; WOW64; rv:18.0) Gecko/20100101 Firefox/18.0"
what is the cause error.config is undefined.
Br,
hatta

@hattaagung Can you check server response in browser dev tools?

Hi alex,
Could you please inform me whts the issue ? Why no request to zabbix ?
Br,
Hatta

@hattaagung Can you please show Response tab instead Headers in dev tools?

Hi alex,
Response Tab is blank, but i found log in console
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://<IP_ADDRESS>/zabbix/api_jsonrpc.php. (Reason: CORS header 'Access-Control-Allow-Origin' does not match '*, *').
what is means :( ? i already adding enable-cors in httpd config.

thnx god...it solved now :(.
you can close this issue.

OK, good!


Use Clang for Windows (m68)

Description of Change
Switch to using Clang 9 instead of MSVC for Win32 builds.
Bugs Fixed

Resolves #758
Related to #686

API Changes
None.
Behavioral Changes
There shouldn't be any visible changes except for performance improvements. Google has been optimizing skia for Clang, and as a result the MSVC binaries are significantly slower. With this PR, we are using Clang on Windows desktop/Win32 so we get all those improvements now.
UWP is still using MSVC, but that will be changed at a later stage.
PR Checklist

[x] Has tests (if omitted, state reason in description)
[x] Rebased on top of master at time of PR
[x] Changes adhere to coding standard
[x] Updated documentation

Hello mattleibow,
Thank you for all these, I still curious to know who can I compile C# application using clang?

@TabishRafiq This is just for the native C++. We are moving from MSVC to Clang. For C#, nothing has changed and you will still use Roslyn/MSBuild. Also, this is just for our code, not any apps or libraries using SkiaSharp.

So the problem of slow drawing with rotated canvas has been solved on C#?

Yes. The actual drawing takes place in the C++ code, the C# was just a glue layer.

Cool, thank you! keep it up man your work is tremendous


RGB <-> HSV conversions should be done in numpy

If it were straight-forward, I'd just write it and not bother with filing an issue.  But I made a little progress and wanted to record it.
The trouble is that the algorithm as written has a decision in it.
There exists good numpy code to do this - https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/colors.py
You can see it's non-trivial.  I'd use that code directly, but it's all [0, 1] based (whereas we are [0, 255] based).  (I could extract out just that file, it only has a single tiny dependency on the whole system.)

I don't get the question... are you trying to avoid a numpy dependency?
My main hangup on that btw is that not all.... wait... nevermind, all
platforms DO have a pre-compiled version of numpy in PyPi! It used to be
that it would download from PyPi but then have to be compiled and it took
FOREVER.
Add numpy as a dependency. With one condition... it WILL NOT go live until
3.5. I don't want to be adding dependencies to sub-minor bug-fix releases.
So if you add it now, fine, but if we have to push a 3.4.x bugfix we'll
have to patch master directly and then bring the fix into dev so master
doesn't gain that dependency.
On Sun, Feb 4, 2018 at 8:17 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

If it were straight-forward, I'd just write it and not bother with filing
an issue. But I made a little progress and wanted to record it.
The trouble is that the algorithm as written has a decision in it.
There exists good numpy code to do this - https://github.com/matplotlib/
matplotlib/blob/master/lib/matplotlib/colors.py
You can see it's non-trivial. I'd use that code directly, but it's all [0,
1] based (whereas we are [0, 255] based). (I could extract out just that
file, it only has a single tiny dependency on the whole system.)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/610, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AA6a6sC5AZY8FBwKINfK1x51_pStOuVjks5tRa3ggaJpZM4R4kJJ
.

These are all 4.0 release issues.
The "dependency" is the link I posted - it is a file in a huge library
matplotlib that has a C compilation step, etc.  I do not want to have that
as a dependency!
I think 3.5 is great timing for including numpy as a dependency, though.
On Sun, Feb 4, 2018 at 2:33 PM, Adam Haile<EMAIL_ADDRESS>wrote:

I don't get the question... are you trying to avoid a numpy dependency?
My main hangup on that btw is that not all.... wait... nevermind, all
platforms DO have a pre-compiled version of numpy in PyPi! It used to be
that it would download from PyPi but then have to be compiled and it took
FOREVER.
Add numpy as a dependency. With one condition... it WILL NOT go live until
3.5. I don't want to be adding dependencies to sub-minor bug-fix releases.
So if you add it now, fine, but if we have to push a 3.4.x bugfix we'll
have to patch master directly and then bring the fix into dev so master
doesn't gain that dependency.
On Sun, Feb 4, 2018 at 8:17 AM, Tom Swirly<EMAIL_ADDRESS>wrote:

If it were straight-forward, I'd just write it and not bother with filing
an issue. But I made a little progress and wanted to record it.
The trouble is that the algorithm as written has a decision in it.
There exists good numpy code to do this - https://github.com/matplotlib/
matplotlib/blob/master/lib/matplotlib/colors.py
You can see it's non-trivial. I'd use that code directly, but it's all
[0,
1] based (whereas we are [0, 255] based). (I could extract out just that
file, it only has a single tiny dependency on the whole system.)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/610, or mute the
thread
<https://github.com/notifications/unsubscribe-
auth/AA6a6sC5AZY8FBwKINfK1x51_pStOuVjks5tRa3ggaJpZM4R4kJJ>
.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/610#issuecomment-362907127,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsngCmVQrgQRpC6Kl2q-jlwgUtZ4Tks5tRbGqgaJpZM4R4kJJ
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com


appWithTranslation is causing the React state to not consist when doing client-side page navigation.

🐛 Bug Report

Wrapper _.app.js with the appWithTranslation HOC  is causing the React state to not consist when doing client-side page navigation.
To Reproduce
I created this repo with a simple example that you can clone and try:
https://github.com/pgburrack/next-i18next-issue

In the example repo I created a DummyProvider with the initial state.
const [state, setState] = useState({
    a: null
  });

Steps to reproduce:

start repo by running npm run dev + open http://localhost:3000/
Open console developer tools. I would expect to see:

{a: null}
{a: null}
{a: 'foo'}
{a: 'foo'}

👆 this is the state that is in the DummyProvider.
3. Click on "Go to about page" (while keeping the console open)

You will see that the state in DummyProiver is initiated again.
{
  a: null
}

Note: if you comment out appWithTranslation and just export the App component.
you will see that the state is being kept on client-side navigation.
Expected behavior
When navigating to the /about page i would expect the DummayProvider to be
{a: 'foo'}

Your Environment

runtime version: Node, v18.15.0, testing on chrome
i18next version: "23.7.16"
"next-i18next": "15.0.0"
"react-i18next": "13.4.1"
"next": "12.2.4",
os: Mac
any other relevant information

P.S i found that if I pass options to appWithTranslation it will work as expected.
Example:
export default appWithTranslation(MyApp, {
  i18n: {
    defaultLocale: DefaultLocale,
    locales: SupportedLocales,
  })

The docs are not saying to do that as well, as we have that information in next-i18next.config.js so i am not sure why i would need to do that.
I suspect that the following line of code is causing the issue
https://github.com/i18next/next-i18next/blob/master/src/appWithTranslation.tsx#L120
    return i18n !== null ? (
      <I18nextProvider i18n={i18n}>
        <WrappedComponent {...props} />
      </I18nextProvider>
    ) : (
      <WrappedComponent key={locale} {...props} />
    )

I saw that when doing the navigation  will disappear.
Could use help with this 🙏

I also noticed that the issue is only happening when one page is translated and the other is not.

I also noticed that the issue is only happening when one page is translated and the other is not.

That's exactly the problem...
if a page is not using next-i18next, there is nothing that will correctly initialize next-i18next...
So you can either use next-i18next on every page or pass the nextI18nextConfig config via appWithTranslation... (btw: you don't need to create a new "empty" config, you can just pass in the original config)
import nextI18nextConfig from '../../next-i18next.config';
//...
export default appWithTranslation(MyApp, nextI18nextConfig);

Hi @adrai
Thanks for the quick reply and the suggestion!
To give you some context on my original issue which made create this example repo and this Github issue.
I started to add next-i18next to a few page in an existing NextJs website that has a lot of pages.
I started seeing bugs which made me start investigate and find the issue with next-i18next.
I am wondering if there is a way next-i18next can always make sure to wrap an app with the <I18nextProvider i18n={i18n}> provider and either default i18n to an empty object to avoid react state from getting cleared? not sure what are the implications of that?
OR
update the README/docs to mention the issue here and how to solve it?
basically to make sure other people are not facing the same issue 😃
Maybe something like:
Partially translated website
OR
Start using next-i18next Incrementally
If you want to only use next-i18next on some pages make sure to do the following
import nextI18nextConfig from '../../next-i18next.config';
//...
export default appWithTranslation(MyApp, nextI18nextConfig);

Feel free to provide a PR to update the readme

@adrai sure 😄
just confirm: you don't think it is feasible or good idea to always wrap the app with a Provider? and if no, why? 🙏

no, we tried this in the past, and it resulted in other errors

Opened a PR
https://github.com/i18next/next-i18next/pull/2260


fix: guaranteed material loading order, processing cant get component…

Guaranteed material loading order, processing can't get components correctly

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.liujuping seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

fix https://github.com/alibaba/lowcode-engine/issues/329


UnorderedKey: Show check name in message

Right now the message looks like this:
.env:2 The FOO-BAR key should go before the FOO-BAR key

Need to add the check name to the message:
.env:2 UnorderedKey: The FOO-BAR key should go before the FOO-BAR key

New to rust myself! Mind if I take this up?

@mgrachev Thank you!


Manually run miniFE, nasFFT, nekbone, and gadget with existing script, but using edit distance filter

estimate 1

Modify existing scripts used to generate test profile trace data to actually run with the edit distance filter.  Remove all geopm modifications from benchmarks.  Run with the monitor.
Done criteria:

The runtime doesn't fail, or an issue is filed for each failure.
Examine profile trace and EPOCH_COUNT events

Parent story: #1180

Few questions/comments:

Is this referring to adding all these apps to GEOPM as an integration test or is it just a step towards it?
Note that GADGET is not fully detected by APD unless we do some manual trace filtering which is not implemented in GEOPM.
Ali is working on adding HACC to this.

Nekbone, minife, and FFT can run successfully.  Issue opened for gadget: #1188


How to support Bootstrap 5 form-floating class and floatingInput label?

Good day,
Not an issue, so let me know if I should move this somewhere else?
Was just hoping someone could point out how / if it is possible to handle the following with simple form?
This is a default example of what a Devise template would look like using simple_form:
<%= simple_form_for(resource, as: resource_name, url: session_path(resource_name)) do |f| %>

    <%= f.input :email,
                required: false,
                autofocus: true,
                input_html: { autocomplete: "email" } %>

    <%= f.button :submit, "Log in" , class: 'btn btn-primary mt-2' %>

<% end %>

My question is how to make use of the form-floating class and floatingInput label similar to the standard from_for example below?:
This is a default example of what a Devise template would look like using with form_for:
   <%= form_for(resource, as: resource_name, url: session_path(resource_name)) do |f| %>

      <%= f.email_field :email, autocomplete: "email", type:"email", class: "form-control", id:"floatingInput", placeholder<EMAIL_ADDRESS>required: false, autofocus: false %>

      Email address

    <%= f.submit "Log in" , class: "w-100 btn btn-lg btn-primary" %>

<% end %>

Any guidance / assistance would be appreciated!

Hey! Please use StackOverflow for asking question, we don't use issues for questions.
TL;DR You can check how it's done in https://github.com/heartcombo/simple_form-bootstrap. It's a repo with example usages of Bootstrap with SimpleForm. Here's floating label example here https://simple-form-bootstrap.herokuapp.com/examples/floating_label


condition +CPT

synonym +camptothecin
I'll pop this in myself if A's on Healx

done - PECO:0000314


Side-menu for a Book shows links from other books as well

Describe the bug
The side-menu for a book (type = book) shows links from other books as well when there is more than one book created.
Consider folder structure as -
   Courses
          - _index.md (type =docs/book)
          - Book 1
                - _index.md (type =book)
                - Chapter 1
                        - _index.md (type =book)
                        - topic 1.md
          - Book 2
                - _index.md (type =book)
                - Chapter 2
                        - _index.md (type =book)
                        - topic 2.md

The path /courses correctly show two books listed viz - Book 1 & Book 2.
Now, if I go to Book 1, I expect to see links to /courses (back page), link to Book 1 (Home page of the book) and link to /book 1/ chapter 1 only, however, I see links to Book 2 & Chapter 2 as well.
Example - https://relaxed-kare-ee5b14.netlify.app/courses/book1/
Source   - https://github.com/Vaidic/academic-kickstart/tree/master/content/courses

Academic Version: latest master build commit 6d04dbe81a7bec7f1b757c2017162684bf02f1b7
Hugo Version: 0.74.3
Browser/OS: macOS Catalina

If courses/_index.md is of type book, then it'll be included in the book.
If you would like a list of books, a normal page can be created with links to the books that you'd like to link to.

@gcushen thanks for the quick response.
I think I figured it out now. So, each book should be in its own folder directly under the content folder & to create an index page listing all books, we can use Markdown.

so now it's impossible to create book list page automatically?
if you have multi books, you should:

create different book folders under content.
create a book list page by writing book's link into it  manually.

book-chapter-page is greate idea, but can't auto organize books is inconvenient :(


Carthage issue framework schemes dependency problem.

When using the command ---> carthage update --platform iOS
*** Building scheme "OAuthSwift" in OAuthSwift.xcworkspace
*** Skipped building SwiftForms due to the error:
Dependency "SwiftForms" has no shared framework schemes for any of the platforms: iOS

+1
Yep, just ran into the same issue.
If you believe this to be an error, please file an issue with the maintainers at https://github.com/ortuman/SwiftForms/issues/new


client.json obsolete?

When experimenting with fedimint-cli and modules, it seems that the client.json file is no longer used since the config is sourced from the client.db/ directory.

fedimint-client/src/lib.rs
Line1515
        Some((_, config)) => {
            // TODO: Enable after <https://github.com/fedimint/fedimint/pull/2855>
            // assert!(
            //     config_source.is_none(),
            //     "Alternative config source provided but config was found in DB"
            // );
            Ok(config)
        }
        ```

In particular, this file is saved to disk during the devimint setup process. But now that configs are stored in the DB when joining a federation, fedimint-cli we no longer needs or uses this file so I think it would be OK to just stop saving it to disk.

i found a reference
https://github.com/fedimint/fedimint/blob/023c8f11e146b120712856888c11b14b2a16d153/devimint/src/federation.rs#L193


🐛 BUG: .run() throws D1 Error in cloud

What version of Wrangler are you using?
2.4.2
What operating system are you using?
Mac
Describe the Bug
await env.DB.prepare('insert into users (name, email) values (?, ?) returning *').bind(name, email).run();
throws Exception in cloud
"exceptions": [
{
"name": "Error",
"message": "D1_ERROR",
"timestamp":<PHONE_NUMBER>085
}
]
These three below work as expected in cloud
await env.DB.prepare('insert into users (name, email) values (?, ?) returning *').bind(name, email).all();
await env.DB.prepare('insert into users (name, email) values (?, ?) returning *').bind(name, email).first();
await env.DB.prepare('insert into users (name, email) values (?, ?) returning *').bind(name, email).raw();
All four requests do work as expected locally

According to the docs, .run() should be used for queries that return no results. While it probably shouldn't be a generic error - it should be returning an error.

It may or may not fail, it's completely up to the API design, at least should be consistent locally and in the cloud

Agreed

Hi—thanks for reporting this! That's definitely a local/remote behaviour mismatch that we should fix. In the meantime, you can often get more helpful error message by checking error.cause (in this case the error message is ERROR 9007: SQL execute error: Execute returned results - did you mean to call query?:
export default {
  async fetch(
    request: Request,
    env: Env,
    ctx: ExecutionContext
  ): Promise<Response> {
    await env.DB.exec(
      "create table if not exists users (name text, email text);"
    );
    try {
      await env.DB.prepare(
        "insert into users (name, email) values (?, ?) returning *"
      )
        .bind("a", "b")
        .run();
    } catch (e) {
      return new Response(e.cause);
    }
    return new Response("Hello World!");
  },
};


create a link to vitables

From<EMAIL_ADDRESS>on 2010-03-09T18:34:40Z
If the data of the workspace is saved to HDF5 vitables could be launched
externaly to inspect and query the arrays in the workspace.
Both are generated in QT.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=171

From<EMAIL_ADDRESS>on 2010-03-09T15:34:54Z
http://vitables.berlios.de/screenshots/index.html

From pierre.raybaut on 2010-03-16T14:43:09Z
Status: Accepted

From pierre.raybaut on 2010-03-31T09:22:05Z
Status: Fixed


checkenv should be written in osh

checkenv should be coded in openshell, not in C.

started

fixed


Validation fails to detect empty string for "id" Parameter in "m365 spo list get" command.

Priority
(Low) Something is a little off
Description
The validation for the id parameter in the m365 spo list get command does not properly handle an empty string. This results in the validation passing incorrectly and returning same result as m365 spo web get
Similar issues exists in a few other commands within the CLI.
Steps to reproduce

Run the m365 spo list get --id "" --webUrl https://contoso.sharepoint.com command.
Notice that the validation does not fail as expected and the command is executed despite the empty id.

Expected results
The validation for the id parameter should properly handle an empty string.
Actual results
The command executes without proper validation and returns spo web get result.
Diagnostics
No response
CLI for Microsoft 365 version
v7.9.0
nodejs version
v20.9.0
Operating system (environment)
Windows
Shell
PowerShell
cli doctor
No response
Additional Info

The issue is with unhandled empty string in command validation.

Yes, unfortunately, this is a "known" issue. Currently, we mostly check on if (args.options.id) but in fact, we should if (args.options.id !== undefined) because an empty string validates to false. Unfortunately, the majority of all commands suffer from this.
@waldekmastykarz I'm wondering if this is solved within ZOD? If that's the case, I don't think it's worth fixing it.

Yes, unfortunately, this is a "known" issue. Currently, we mostly check on if (args.options.id) but in fact, we should if (args.options.id !== undefined) because an empty string validates to false. Unfortunately, the majority of all commands suffer from this. @waldekmastykarz I'm wondering if this is solved within ZOD? If that's the case, I don't think it's worth fixing it.

Yes, this should be fixed in zod, because it has a separate required/optional check. I suggest, that we prioritize fixing this issue by migrating this command to zod as soon as possible so that we can confirm the fix.


Test Results show all tests passing when using an ordered test when tests are actually failing

Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
TFS on-premises-2017
Private Agent- Server2012R2
Version 2.122.1
We use the run functional test task to execute an ordered test.
The run summary and test results show 1/1 tests passing but the trx file shows a failed test .
We are migrating from labdefaulttemplate11.xaml to a release definition.
The xaml builds would correctly report a failed test

tfsservice_G1TTEST1 2018-01-26 08_17_00.txt

@johnwhoffman Ordered test in Run functional test task is not supported out of the box. Check for workarounds https://blogs.msdn.microsoft.com/allendm/2016/08/09/executing-ordered-tests-on-remote-machines-using-run-functional-tests-task/

hope the above blog helps. Closing the issue


Use timeseriesPanel instead of panel when creating panels

What this PR does:
Update panel to timeseries panel
Which issue(s) this PR fixes:
Fixes: https://github.com/cortexproject/cortex-jsonnet/issues/44
Depends on: https://github.com/cortexproject/cortex-jsonnet/pull/57
Checklist

[x] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@friedrichg I think I fixed it, could you take another look? 🙏🏼