DIGITALTWIN - Define first version of the pricing and offer

Define what exactly are we bringing to the user for their lifetime payment. What exactly will be "Lifetime Free"?
Define fee structure and business model, what are the limitation of the usage (capacity limitation, amount of users on a call, etc).
Define costs for users, - what is the regular use from people and how much it will cost for us, we should understand value proposition ourselves and explain it to people
Split lifetime package 50/50 between service fee and capacity fee and then explain how long will it last in normal usage conditions
Explain in simple terms how much capacity is needed for certain use-cases and how much it costs and present bundled packages for capacity plans
Define invite only model

We offer only 100 USD lifetime package for now, with no capacity included. We have to describe it in the wiki

Moved to done because the definition of the 100 USD package has been approved and is ready for implementation.


Split package into small reusable parts

Hi @rmcaixeta , nice to see the effort around models of local anisotropy. I see that there is a lot of code duplication here where you copied solvers from GeoStats.jl and modified them to achieve what you wanted quickly. As we discussed over Zulip, the best way to maximize the impact is to split the package into small components that can be moved to GeoStatsBase.jl or Meshes.jl for greater reuse across the framework. After all, local anisotropy should be an option in the existing Kriging and simulation solvers, not completely separate solvers.
I am concerned that if we don't tackle this issue now we will just lose the opportunity to have a cohesive treatment of local anisotropy. The effort will be disconnected from the rest of the framework, will get outdated, and that is not a great outcome.

Thanks for registering that, @juliohm
This package was a lab for me to learn julia and more about local anisotropies implementation. The framework will get evolved with time, I'll keep it updated with GeoStats framework until that. I believe that after refactoring kriging for block estimation and cokriging, things will get simplified here too.
I understand that there are two parts to deal: kriging and rotation conventions.
For the first, I think one day it might be enough to pass just a LocalGeoData in the EstimationProblem; or a custom variogram object for kriging (with LocalAnisotropy embedded) and the rest will be natural. I'm not totally sure right now because it has been some time since I coded kriging part, but if variogram function for kriging matrices are built with a variogram function like γ(GeoData1, GeoData2, ind1, ind2) instead γ(point1, point2), it'd be possible to remove a lot a kriging code here.
For the second, I belive that ideal solution would be an extra package that deals with rotation conventions. Where anyone could add its own rotation convention rules and output rotation matrix, quaternions, reverse rotation matrix etc. But that might be a little bit challenging too, there are multiple ways of naming and structure conventions, as it is here or in Meshes might not be enough yet.


feat: update to latest "ai" package and openai

Description

We want to use Open AI's new Structured Outputs feature
To do that, we need to update to the latest OpenAI package
But we were blocked from doing that because we use both it and Vercel's "ai" package and there is a difference in the streaming protocol between the two versions
This makes a relatively minimal change to how we send text back from the /chat endpoint by formatting each chunk with 0:${JSON.stringify(chunk)}\n
This means we are now on latest "ai" and "openai"
From here we could then use the Sructured Outputs generation, which in turn should remove this issue where we are seeing JSON and incorrect lesson output on the left hand side of the UI
This leaves us with Langchain at this very old version, so we should look at updating it separately

The changes in this PR are incorporated into https://github.com/oaknational/oak-ai-lesson-assistant/pull/91 so we may not need to merge this if we approve that.

This is superceded by https://github.com/oaknational/oak-ai-lesson-assistant/pull/91 which includes everything in this PR


Squash v12 depcrecation warnings

Foundry v12 has deprecated several ways to access helpful methods, in addition to some handlebars helpers.  Make those warnings go away.
https://github.com/mkscho63/sta/issues/106

[ ] Review complete list actor and item character sheets for warnings
[ ] Squash the warnings in Handlebars
[ ] Squash the warnings in JavaScript
[ ] Validate against "legacy" STA installs
[ ] Verify backwards compatability of new API- does this require minimum version being set to v12?

This was addressed by this PR.  https://github.com/mkscho63/sta/pull/109


Deep Learning

Hi @jungrob please provide further details or a description of your issue, thanks.
Feel free to reopen.


feat(server): add express tests

Description
Please include a summary of the change and which issue is fixed. Please also include relevant
motivation and context. List any dependencies that are required for this change.
Related https://github.com/dotansimha/graphql-yoga/issues/2655
Type of change

[x] Tests only

Checklist:

[x] I have followed the
CONTRIBUTING doc and the
style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[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 and linter rules pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Further comments
Went down this rabbit hole trying to identify root cause of odd behaviour of status texts in graphql-yoga. The issue isn't here, but given that I've written these tests I thought they may be helpful to prevent future regressions so figured would at least submit the PR :)

Thanks @domdomegg <3


Cannot Clone repo

Issue description
This is probably my fault.  But why can't I clone or download the code from this repo.  No option is given on the web page and when i use the web URL in GitHub desktop it fails?  What am I doing wrong?  Screenshot attached.
Screenshot 2022-11-26 134502

Yeah, what's up with this?

Thanks

Have you tried cloning the whole repo and not just a single directory?

Yes I did that yesterday.  and it worked fine.  Was this working before and I just hadn't found it?


Need to have external session storage for stateless services in kubernetes

Right now there is only a SQLite support that can't be used in stateless clustered services in kubernetes, as there is no mounted persistant volume and also there several instances of same pods are running, so we need to share DB across several pods.
So we need to have cloud native storage for data (PostgreSQL or anything else) and S3 for file storages.
I tried to find any solution and found this old request
https://github.com/tdlib/td/issues/225
Provided solution is applicable only for monoliths or statefull services without cluster support.

This is not possible. Telegram apps can't be stateless, because this would be extremely ineffective.


JsonTextReader auto supports decimal precision (28 significant figure…

JsonTextReader auto supports decimal precision (28 significant figures) &
double range (10e324).

Thanks for the PR but this isn't a feature that I want to add.


npm module name is already in use

I just wasted an impressive amount of time trying to figure out why oy didn’t have a renderTemplate method. Turns out I had installed the wrong oy. Looks like @cistov beat you all to that URL by a few years.
me right now:

oyvey and oy-vey are available. Installing from Github works pretty well, too.
Aaaaaaaaaanyways. if you need me, I’ll be over here, yelling into a pillow until my yeller can’t yell no more.

Augh! What a detail to overlook. I think oyvey or oy-vey works for now and will publish tomorrow. In the meantime, the PR works—accepting. Thanks @meyer! Excited to get this off the ground.

Updated. Published under oy-vey.


User Interface

We need to allow the user to decide between the frameworks of appium and espresso, and then feed us an arbitrary JSON file. Preferably, this would be through command line.

Still need to let them put in an arbitrary JSON file. Otherwise, the switches are there now.


publish for Scala 2.13.0-M4?

M4 JARs are on Maven Central now
in the 2.13 community build we are using https://github.com/SethTisue/scalacheck/tree/newCollections (forked from @lrytz who did most of the work)

#411 changes the API, due to the type changes (Stream -> LazyList, Traversable -> Iterable). I'm not sure how we should handle the releases and versioning from here on. It feels wrong to just release #411 as another 1.14.0 build. @SethTisue @lrytz Any thoughts on that?

I think the we should probably revert the Stream -> LazyList change, but instaed add built-in generators for LazyList in the 2.13 version.
For the Traversable -> Iterable change, we can basically introduce a new type alias which, in 2.10/2.11/2.12 would be Traversable, and 2.13 would be Iterable.
Both of the above can be done with version-dependent source directories. This is a bit annoying to have, but I think it's better than having a separate branch for 2.13.
Let me know what you think, I'll put it in the PR.

Once released, re-enable MiMa for 2.13.0-M4 in build.sbt (mimaPreviousArtifacts).

I think the we should "revert" the Stream → LazyList change, and instead add built-in generators for LazyList in the 2.13 version.

Changing those as described would would be necessary to publish 1.14.0 for 2.13.0-M4.  Otherwise,  it doesn't seem like the 1.14.0 release would be source compatible.

That was done in the PR (https://github.com/rickynils/scalacheck/pull/411) (which is still ready for review :relaxed:).

Ah, ok. Sorry, it wasn't last I checked. I'll look again.

The signature for Shrink was still using LazyList instead of Stream in the 2.13 build.

Since Mima can't really check compatability with new versions of Scala, one other way to verify compatability is to try to export Scalacheck's test suite for 1.14.0 and try to compile it with this new 1.14.0 build (alongside the existing binaries).  This assumes Scalacheck has sufficient test coverage of itself, but that may be a flawed assumption.
Regardless, I'd say it's worth publishing a 1.14.0 binary for 2.13.0-M4.  If the 1.14.0 for 2.13.0-M4 doesn't work, then either fix what's broken by releasing a 1.14.1 for all versions of Scala,  or if appropriate move everything to 1.15.0.
This is the procedure I did locally, and it checks out now that Shrink was just reverted in the PR:

Rebase the 2.13.0-M4 branch onto the old 1.14.0 tag so a 2.13.0-M4 binary can be built:

$ git fetch upstream master
$ git fetch -t upstream 1.14.0
$ git fetch lrytz m4
$ git checkout m4
$ git rebase --onto 1.14.0 upstream/master lrytz/m4

Publish just the Java M4 binary locally:

$ sbt ++2.13.0-M4 jvm/publishLocal

Then clone the 1.14.0 tag, and delete all the code except the test suite.

$ cd /tmp
$ git clone --branch 1.14.0 --depth 1 git@github.com:rickynils/scalacheck.git scalacheck-1.14.0
$ cd scalacheck-1.14.0
$ git rm -fr ./src/main
$ git rm -fr ./jvm/src/main

Overwrite the build as Java only by dropping the plugins, and add just a dependency for 1.14.0:

$ git rm -fr ./project
$ git reset -- project/build.properties
$ git checkout -- project/build.properties
$ git rm build.sbt
$ cat << EOF > build.sbt
scalaVersion := "2.12.6"
crossScalaVersions := Seq("2.10.7", "2.11.12", "2.12.6", "2.13.0-M4")
libraryDependencies +=
  "org.scalacheck" %% "scalacheck" % "1.14.0"
EOF

Compile the test suite against 1.14.0 for all Scala versions:

$ sbt +update +test:compile

Since Mima can't really check compatability with new versions of Scala

I'm not sure I understand what you mean here; When building with a new Scala version, there is nothing to compare against, and there doesn't need to be.
Binary compatibility needs to be checked only for new releases of scalacheck with an existing Scala binary version, e.g., 1.14.1 for 2.12 needs to be binary compatbitle with 1.14.0 for 2.12.
But 1.14.0 for 2.13 doesn't need to (and probably cannot) be binary compatible with 1.14.0 for 2.12.

Indeed, there is no way for 2.13.0-M4 to be binary compatible.  I was trying to verify that at least what ScalaCheck publishes as 1.14.0 for 2.13.0-M4 (everything in org.scalacheck) should at least be source compatible with 1.14.0.   Since the PR for M4 has version-specific directories and collections library changes, that's more difficult to verify.  Also, MiMa doesn't have a 2.13.0-M4 jar to compare with, so it can't help here, either.  Locally publishing the PR for M4 and running the original 1.14.0 test suite is an alternative, albeit crude, way to verify compatability.  It would have found the change with Shrink, for example.

I've now rebased and merged #411 to the 1.14.0_sonatype branch, and published JVM and JS builds of ScalaCheck 1.14.0 for Scala 2.13.0-M4.
Thanks for all your efforts @lrytz and @ashawley!

Yay, thanks @rickynils!


Character types docs

Codecov Report

Merging #22 into master will increase coverage by 0.04%.
The diff coverage is 95%.

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
+ Coverage    84.5%   84.55%   +0.04%
==========================================
  Files          23       23
  Lines        3292     3309      +17
==========================================
+ Hits         2782     2798      +16
- Misses        510      511       +1

Impacted Files
Coverage Δ

pyasn1/type/univ.py
81.63% <ø> (ø)
:white_check_mark:

pyasn1/type/char.py
87.06% <95%> (+1.21%)
:white_check_mark:

Continue to review full report at Codecov.

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


Main branch documentation sometimes is updated to an older version

For example, right now master-api.flutter.dev is the documentation for 6cf50ef942, which is from 14h old at this point, while a few hours ago it was for 47b188e, which is 9h old at this point. I'm guessing we run the tests out of order but the documentation shard just unconditionally uploads to the server and overwrites whatever is there, even if it's newer?
cc @gspencergoog

This should be fixed with https://github.com/flutter/flutter/pull/132510


Move big demo data files out of core deck.gl repo

About the big data files in the demo directory, we should move them out of this repository.
We mustn't let deck.gl become one of those 1GB monster repos that takes an hour to clone (if GitHub even supports such large repos).
A first suggestion is that we set up a new repo with just the demo data files, and we update our demo load its data from there. When then use git filter-branch to purge them completely from this repo.

Just as a reference, as of deck.gl v4.0.0-rc.5, the data transferred for a clean clone from github is 68.16 MiB.

We have removed the dependencies on master, now we need to remove them from the repo by rewriting history.


feature/216 Endpoint for slack bot to interact with backend

Endpoint for slack bot

Currently working on trying to get the slack bot to interact with the front end this is currently not working but trying to raise discussion and get help.
Things that need to be done:

[x] api documentation
[ ] making the connection secured so not anybody can get data
[x] registering the https://operationcode.org/api/v1/slack/users_info endpoint
[ ] discussion on how to implement better

Issue Resolved attempt at endpoint addition

Fixes #216
The reason for this:

Get a list of all users to see who doesn't have a registered slack account, then add slack account using the slack api
Continually send slack user information to the database when we get new users.
Get single individual information (interests, slack_name, etc) using email as the key.

I don't have a way to get the user_id, and the slack bot doesn't login, so I'm working on creating a unique token that both the database have and my slack bot has to verify before doing any of these requests.
Right now my biggest issue is getting around not being able to implement the route or test locally.
web_1                 | F, [2018-01-15T21:03:45.767623 #13] FATAL -- :
web_1                 | F, [2018-01-15T21:03:45.767663 #13] FATAL -- : ActionController::RoutingError (No route matches [GET] "/api/v1/slack/user_info"):
web_1                 | F, [2018-01-15T21:03:45.767677 #13] FATAL -- :
web_1                 | F, [2018-01-15T21:03:45.767694 #13] FATAL -- : actionpack (5.0.2) lib/action_dispatch/middleware/debug_exceptions.rb:53:in `call'```

Closing this out in favor of the Slack integration issues 1 - 6


Vertex declaration fixes

Fixes #2969
(The last commit fixes it.)
I need to verify that this is what actually happens on Windows.
cc @kdobrowo

Verified that this fixes the issue mentioned

I checked on Windows and it does indeed return a 0 FVF if a vertex declaration contains any element that can't be mapped.


Not working!!

When I put the Beelogger file and clicked on it in windows 7 64bit and typed some words but, nothing!
I did not receive anything to my mail
Indeed, I checked when I inter my mail, pass, smtp and the port!
please help me!

unfortunally the error was in custom smtp method, so use the standard with GMAIL, it are fixed now. Reclone and happy pentest.


Callback not rendering anything

I'm following the tutorial here exactly as mentioned: https://github.com/ratiw/vuetable-2-tutorial/wiki/lesson-06
For some reason, I'm not getting anything rendered in the nicknames column?
`
data() {
    return {
        name: 'Restaurants',
        fields: [
            {
                name: 'name',
                title: 'User'
            },
            'email',
            {
                name: 'birthdate',
                titleClass: 'center aligned',
                dataClass: 'center aligned'
            },
            {
                name: 'nickname',
                callback: 'allcap'
            },
            {
                name: 'gender',
                titleClass: 'center aligned',
                dataClass: 'center aligned'
            },
            {
                name: 'salary',
                titleClass: 'center aligned',
                dataClass: 'right aligned'
            }
        ]
    };
},
methods: {
    allcap(value) {
        return value;
    }
}`

Your allcap function returns the given value without doing anything with it.
In the tutorial, it is supposed to return the value with all uppercase letters using toUpperCase() function.
If this is not what you mean, please explain more or put your code in jsfiddle or codepen.

Found the issue, looks like there was something wrong with my node_modules so I cleaned it up

i found this issue too, how to solve this problem? i'm tired with it


Automated cherry pick of #2227: - feature: refact snapshot related code - feature: rbd snapshot - bugfix: ceph cache iso image - delete disk create snapshot and guest create snapshot - fix auto snapshot - fix disk reset

Cherry pick of #2227 on release/2.11.
#2227: - feature: refact snapshot related code - feature: rbd snapshot - bugfix: ceph cache iso image - delete disk create snapshot and guest create snapshot - fix auto snapshot - fix disk reset

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@7ecf021). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #2239   +/-   ##
==============================================
  Coverage                ?   7.11%
==============================================
  Files                   ?     452
  Lines                   ?   71148
  Branches                ?       0
==============================================
  Hits                    ?    5060
  Misses                  ?   65672
  Partials                ?     416

Flag
Coverage Δ

#aFlag
7.11% <0%> (?)

Impacted Files
Coverage Δ

pkg/cloudcommon/db/opslog.go
2.66% <ø> (ø)

pkg/compute/models/storagedrivers.go
22.22% <ø> (ø)

pkg/compute/models/regiondrivers.go
20% <ø> (ø)

pkg/compute/guestdrivers/managedvirtual.go
0% <ø> (ø)

pkg/compute/guestdrivers/virtualization.go
0% <ø> (ø)

pkg/compute/models/guest_actions.go
0% <ø> (ø)

pkg/compute/models/guestdrivers.go
18.18% <ø> (ø)

pkg/compute/guestdrivers/base.go
0% <ø> (ø)

pkg/compute/models/snapshots.go
1.87% <0%> (ø)

pkg/compute/models/disks.go
0.74% <0%> (ø)

... and 2 more

Continue to review full report at Codecov.

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


Azure Storage Blob - Custom Metadata extraction function is incorrectly case-sensitive

Package Name:
azure.storage.blob

Package Version:
12.23.1

Operating System:
OS X

Python Version:
Python 3.11.9

Describe the bug
Function deserialize_metadata is intended to extract metadata header names from a case insensitive dict, but it does so in a comprehension that calls .items() on the dict which causes the case insensitvity to be lost.
def deserialize_metadata(response, obj, headers):  # pylint: disable=unused-argument
    try:
        raw_metadata = {k: v for k, v in response.http_response.headers.items() if k.startswith("x-ms-meta-")}

https://github.com/Azure/azure-sdk-for-python/blob/9079caf686e2a29a3fcda96517fa15c547035667/sdk/storage/azure-storage-blob/azure/storage/blob/_shared/response_handlers.py#L63
To Reproduce
It's not trivial to reproduce, in our case we are working on an API compatible virtual storage. But the Azure SDK must receive a request with headers of the mixed case format 'X-Ms-Meta-Example', which it will then fail to parse into the returned Metadata.
Expected behavior
The above field should be recognised as a custom metadata field named 'example'.

Hi @docuinn-mw Daithi, thanks for the question! We were able to locally reproduce the issues, and saw that X-Ms-Meta-Example is not processed correctly when sending the request. We're currently working on a fix. I linked the PR here that will resolve the issue once we finalize the design, implementation, and testing.

Hi @docuinn-mw Daithi, I just merged the PR that contains the fix for this, and it will be available in our next release (scheduled for early November).
Just a note for future reference as well as others that may end up here: Using the SDK against anything other than the official Azure Storage service is not recommended or supported. The Storage SDKs are designed to work with only the official Azure Storage service and we do not generally take fixes/requests for any other API-compatible services as we just don't have the resourcing to support other random services. Azurite is the one exception here since it is an internally maintained product.
In this particular case, I'm fairly sure the service would not return any case but x-ms-meta so this was not technically an issue. We took the fix in this case because it was a simple fix that ultimately hardens the SDK but as I mentioned, this is not something we would normally take.


fixes & tests for updating orders and items.

Resolves #8  and #18  .

added code for in routes.py for updating an order.
added code for in routes.py for updating an item inside an order.
added test cases for updating an order.
added test cases for for updating items inside an order.
added test cases for bad requests/ illegal method calls etc.
Test Coverage is currently at 94.82%

Looks good to me, no conflict with the master branch.


Feature/use del to remove

Using the orbit-db command db.del to implement remove.
Also increased timeouts in the jest tests to 20s since the default 5s timeouts got triggered.

Failed in linting. Will update.

Thanks for fixing the timeout thing also! I think it was causing issues on circleci sometimes.

@oed Thanks for the comments! Updated.

@oed Updated encryption/decryption with JSON.serialize() and JSON.parse().


Generate changelog from commit messages

Writing up the changelog for each release takes a non-trivial amount of time. It would be useful if we autogenerated it based off of commit messages and commit authors. Commit messages would have to follow our standard changelog format, but this is easy given the "squash and merge" GH feature. Commits that started with [trivial] or [docs] or other tags could be automatically ignored.

closing as duplicate of the more up-to-date #2032


Manager Bussines

Originally posted by @ABDULMAJEED901 in https://github.com/violet-org/boysenberry-repo/issues/75

Originally posted by @ABDULMAJEED901 in https://github.com/violet-org/boysenberry-repo/issues/75

Hi


Fixed error when adding tags to user model with name of Name

The related_query_name function was using the model name instead of the related_name attribute which was causing a clash between the model name and the tag.name.  By returning the field's related name instead of the model name  when it exists, fixed this problem.

Thanks! Could you add a test to avoid future regressions?

I believe this was fixed in 0c47c93a73a174ab28542bae8db7d10ceb6a535b. Let me know if you continue to experience issues.


Remove Jenkinsfile and docker-compose setup

This supports us switching from Jenkins to Github Actions as our primary CI tool for the design system. Removes the Jenkinsfile along with any CI related Docker code.
We may want to reintroduce a docker-compose set up in future more geared towards local development. But for now our current docker set up is largely intended for CI and is not required for our actions workflows.
This might fail unless we tell Jenkins not to build anymore, let's find out.

This does indeed stall, so will wait for @simonwgill to do his bit and then we can rerun this. Good to know just how much will go though 🗑

Is it worth us doing some / all of the fixes in the breakout JIRA ticket here too?

Sorry just saw Jenkinsfile was completely gone here too. Yeh gotta block on keeping existing behaviour first.

What existing behaviour are you thinking of, specifically? Plan is to remove the jenkins status check and add some/all of the github actions to the branch protection rules. Anything else missing?

Guard here: https://github.com/citizensadvice/design-system/pull/1245/files#diff-e6ffa5dc854b843b3ee3c3c28f8eae2f436c2df2b1ca299cca1fa5982e390cf8L20
browser/driver vers
There's also some other stuff in the breakout JIRA ticket, but that's not a hard blocker. I can fix up those issues tomorrow or something when I have chance

Guard here: https://github.com/citizensadvice/design-system/pull/1245/files#diff-e6ffa5dc854b843b3ee3c3c28f8eae2f436c2df2b1ca299cca1fa5982e390cf8L20

browser/driver vers
Mind being more specific?

So ideally we need the guard permissibleFail still present, as this is something which would be reduced functionality (and not desirable).
We also need to alter the browser versions (and therefore inherent driver versions), on the new yml config file, ideally mirroring the ones set in Jenkinsfile

So the reason for the failing one for the iPhone is it's using safaridriver 12
The short answer is this has a lot of inherent bugs in it, and can be considered flaky.
The long answer is the way in which safaridriver works can be considered JWP compatible, not W3C compatible. Back before selenium 3.2-3.5 (ruby), the drivers were in a mixed state and some supported JWP compatible endpoints and some supported W3C endpoints, safari however was not one of these. As such safari can be considered non-W3C compliant in actuality vs it being W3C-compliant technically (Like a lot of the drivers, it's closed source proprietary software). You can see evidence of this in both the hacky configs we use AND the patches further to these hacky configs.
Safari specific config: https://github.com/citizensadvice/design-system/blob/master/testing/features/support/driver.rb#L30 (This is very bad code and unsupported by capybara, and they get angry at it even being in existence)
Safari hacky overrides: https://github.com/citizensadvice/ca_testing/blob/main/lib/ca_testing/patches/capybara.rb

Interesting. At least using the pinned versions as is everything seems to pass OK

So it looks like we can omit any extra boilerplate right now.

The short answer is this has a lot of inherent bugs in it, and can be considered flaky.

I think I misread. Is the idea that it can fail, not that it always will? In which case is it better to omit it from the matrix if we can't rely on it? Smaller stable list feels nice. If not, I think I get what you mean 👍

I'll just fix up the incorrect stuff later so I'm not holding stuff up.

Sounds perfect 👍


Allow cancel a query while it is loading

I tried to implement it. But after the query result has started to rendering by react is not possible to cancel the rendering. So I will close it while I don't have a better idea an time to implement it.


Adds ProxyGet to public API

Introduces ProxyGet to public API. This precedes an update to Proxy.from_name("my-proxy") to replace ProxyGetOrCreate.
I intended to have that RPC support both the client get method and internal proxy create methods but came to the conclusion that it is better to separate those methods because it is not desirable to create a Proxy lazily. (That's the case because it takes several seconds to create.)

@prbot approve


fix(das): Only debug log sampled header if it was actually successful

Self explanatory

Codecov Report

Merging #1295 (4e11388) into main (6adfd8a) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1295      +/-   ##
==========================================
- Coverage   55.75%   55.74%   -0.01%
==========================================
  Files         170      170
  Lines       10254    10259       +5
==========================================
+ Hits         5717     5719       +2
- Misses       3966     3968       +2
- Partials      571      572       +1

Impacted Files
Coverage Δ

das/worker.go
93.54% <100.00%> (+0.56%)
:arrow_up:

header/core/listener.go
52.83% <0.00%> (-5.67%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Revert "DOC-2793: Add SSO note for Cloud Console login"

Reverts cockroachdb/docs#13163

This change is


Make a guide for M-of-N Multisignature wallets

We need a guide to be written for M-of-N for example 3 of 5 keys required to send Loki,
this can also probably been up streamed to Monero docs since they are also missing a guide like this

I'll wait till we implement mms functionality into the loki-wallet-cli before writing this document. Transfer.sh isnt a secure means of sending key files.

M-of-N
make_multisig <threshold> <MS_data_1> [<MS_data_2> ... <MS_data_N>]
Where <threshold> = M and <MS_data_N> = the N'th person of the multisig wallet.

Guide finished as of https://github.com/loki-project/loki-docs/commit/5636befe539bd4860da6e46386ad93e4733ac53e


Nexmo SMS integration

Hi,
does anybody have an idea of integrating nexmo with this?

@kandiki These type of questions should go on StackOverflow http://stackoverflow.com/questions/tagged/angular-meteor. Also provide us with more details, things that you've tried, expected and actual result, errors that you had etc.


Make control + arrow keys (move to beginning/end of word shortcut) not move to end of periods or commas

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
So, I couldn't find a package which did this.
Essentially, the control + right/left arrow (or alt + right/left arrow if you're on a Mac) allows you to go to the end or beginning of the word. I think that's well-known. However, for some reason, Atom works a bit differently.
This is what happens in Atom as you press the alt + right arrow shortcut in a line (the | is the cursor)
|if 20,000

if| 20,000

if 20|,000

if 20,|000

if 20,000|

So, as you can see, the cursor stops at the comma. However, this is much different to how it works in other apps. I tested it in Pages, TextEdit, Safari, as well as a bunch of other third-party apps, and they all work like this:
|if 20,000

if| 20,000

if 20,000|

Simple and easy to use.
So, why do I want Atom to change its behavior to the second way instead of using its current method? Because it makes text editing and going to other places within a line extremely annoying. You always have to keep pressing alt + right to go somewhere, and sometimes this problem makes navigating as slow as having to press the right arrow key on its own, which is a real bummer.
Is the comma the only character that causes this to happen? No, there is also . and '. One this about the single quote '. Additionally, there's another problem: alt + backspace (or control + backspace) also stops at these characters. That means that if you want to backspace the word don't in Atom, you need to press alt + backspace three times, whereas virtually in app other than Atom this shortcut only needs to be pressed once. You have the same problem with text selection via alt + shift + arrow key as well, making text editing in Atom a burden in general.
This is why I honestly think that this behavior should be changed.
Versions
macOS 10.12.3
Atom    : 1.13.1
Electron: 1.3.13
Chrome  : 52.0.2743.82
Node    : 6.5.0
apm  1.14.1
npm  3.10.5
node 4.4.5
python 2.7.10
git 2.10.1
Additional Information
None

Thanks for taking the time to contribute!
We noticed that this looks like a duplicate of https://github.com/atom/atom/issues/13651?  If I'm mis-understanding, let me know.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.

@rsese Thank you. Yes, it seems like it's related to that issue in a way, and the way you handle duplicates I think this issue could be considered a duplicate of that one. I didn't find that other issue probably because I searched up control + arrow instead of alt + arrows.

No problem at all, we have a large number of open issues so it's definitely not always straightforward to find the things you're looking for :v:


IMX290 crashes or hangs after first exposure

Not all the time, but most. This here seems to fix it:
diff --git a/src/indi_pylibcamera/CameraControl.py b/src/indi_pylibcamera/CameraControl.py
index 50497ca..7bc8813 100755
--- a/src/indi_pylibcamera/CameraControl.py
+++ b/src/indi_pylibcamera/CameraControl.py
@@ -150,7 +150,7 @@ class CameraSettings:
             self.is_ReconfigurationNeeded(NewCameraSettings)
             or (self.camera_controls != NewCameraSettings.camera_controls)
          )
-        return is_RestartNeeded
+        return is_RestartNeeded or True

     def is_ReconfigurationNeeded(self, NewCameraSettings):
         """would using NewCameraSettings need a camera reconfiguration?
@@ -161,7 +161,7 @@ class CameraSettings:
             or (self.ProcSize != NewCameraSettings.ProcSize)
             or (self.RawMode != NewCameraSettings.RawMode)
         )
-        return is_ReconfigurationNeeded
+        return is_ReconfigurationNeeded or True

     def __str__(self):
         return f'CameraSettings: FastExposure={self.DoFastExposure}, DoRaw={self.DoRaw}, ProcSize={self.ProcSize}, ` +\
~

Note that this only works with stellarmate when you do the song and dance from here: https://indilib.org/forum/ccds-dslrs/12177-indi-libcamera-driver.html?start=216#97003
Otherwise. you can't even install PiCamera2

In case it's important, here's my /boot/config.txt
dtoverlay=imx290, clock-frequency=37125000

Hi Anjok,
thank you for reporting and providing a workaround.
The development of libcamera and its Python binding picamera2 is very dynamic. During the last months I saw that features got not only added but also changed in an incompatible way. That makes it very difficult for me to make the indi_pylibcamera driver compatible to a wide range of libcamera/picamera2 versions. I try to make it work with the latest versions but often that is at the expense of compatibility to older versions. I am sorry that Stellarmate and Astroberry have old versions of libcamera and picamera2 and indi_pylibcamera is not compatible. Hopefully these distributions will get updates soon. In the Wiki is a description how I setup my Raspberry Pi based on the standard Bullseye OS.
Back to the issue you reported. According to the picamera2 documentation it is allowed to capture multiple frames between starting and stopping the camera. That works with HQ and V1 camera but not with the IMX290. With your workaround you force a camera stop and re-start before every frame capture. It is save doing this. The only concern I have is that it may add delay. I remember that the stop start functions block execution for some seconds (depending on exposure time?). That would make the "Fast Exposure" feature inefficient. I need to do some tests.
I would propose to add a configuration switch in the INI file which allows to force the stop/restart (as in your workaround). That gives the highest flexibility. If you agree it will become part of the next release.
Regards,
Ronald

Hi, thanks for your reply and the great work you did with this driver!
I did some work on the indi libcamera driver and it really wasn't fun... given this mix of astroberry not being on bullseye, stellarmate having this really outdated versions etc it's a train wreck waiting to happen :)
You can do this any way you think is best, but this is a simpler version of the one above. If self.CamProps['Model'] was present at the time of the call it would be easiest, but it's apparently loaded later and I didn't know if I can move it. If you can do that, you can remove the or True.
I guess that this is a global problem and not just my setup, so I'd just add this to the code and not fiddle with settings.
This seems to work fine for both the imx290 and imx519 I have (and at least for me it's needed for both of them).
index 50497ca..b4434cd 100755
--- a/src/indi_pylibcamera/CameraControl.py
+++ b/src/indi_pylibcamera/CameraControl.py
@@ -160,8 +160,10 @@ class CameraSettings:
             or (self.DoRaw != NewCameraSettings.DoRaw)
             or (self.ProcSize != NewCameraSettings.ProcSize)
             or (self.RawMode != NewCameraSettings.RawMode)
+            # this apparently isn't loaded here...
+            # or (self.CamProps["Model"] in ['imx290','imx519'])
         )
-        return is_ReconfigurationNeeded
+        return is_ReconfigurationNeeded or True

     def __str__(self):
         return f'CameraSettings: FastExposure={self.DoFastExposure}, DoRaw={self.DoRaw}, ProcSize={self.ProcSize}, ` +\

OK, so at least for the imx519 you can move the or True to is_RestartNeeded. Dunno what difference this makes. I just checked a few pics with differing exposure times.
index 50497ca..1de45db 100755
--- a/src/indi_pylibcamera/CameraControl.py
+++ b/src/indi_pylibcamera/CameraControl.py
@@ -150,7 +150,7 @@ class CameraSettings:
             self.is_ReconfigurationNeeded(NewCameraSettings)
             or (self.camera_controls != NewCameraSettings.camera_controls)
          )
-        return is_RestartNeeded
+        return is_RestartNeeded or True

     def is_ReconfigurationNeeded(self, NewCameraSettings):
         """would using NewCameraSettings need a camera reconfiguration?

I would propose to add a configuration switch in the INI file which allows to force the stop/restart (as in your workaround). That gives the highest flexibility. If you agree it will become part of the next release.

The problem with that is that I have multiple cameras. I don't know how this would work with the INI file? I apparently need to change /boot/config.txt and reboot when I use a new one, but I'd prefer if I didn't need to change this in multiple places. I would think checking the names of known non working cams would be more robust... we just need to get the cam props up front (or have it return True on the first run)

I understand. What about an INI switch "ForceRestart" with 3 states:

"yes" means always restart
"no" means behavior as it is in the present release and
"auto" means force restart for a known list of cameras (IMX290, IMX519, maybe more) and old behavior for other cameras.

I can make the "auto" the default when the switch is not in the INI file. In that way you do not need to change something on your INI file and the driver detects your cameras automatically. When someone comes with a different non-working camera I have a quick workaround with "yes". Once the kernel driver for your cameras gets fixed you can test it with "no".
I already do some special handling for different camera chips. It would be only a relatively small code change.

Implemented but not tested in branch 53-imx290-crashes-or-hangs-after-first-exposure. It is even not tested for syntax errors. If family allows I will test with my cameras tomorrow.

Tried with the imx290, seems to work fine. No hangs.  what do you think about changing the logging to actually end up in kstars? It’s a bit inconvenient to look in /tmp…

Thank you for testing. I also tested with HQ camera and did not found any issues (neither with the auto nor with the yes setting). Unfortunately I do not have a setup with V1 camera anymore. But I believe no one will use V1 for astrophotography - it is too bad.
Forwarding the logging to kstars is a good idea. But before I do this I will need to reduce the amount of messages. I will work on it.


Fixed bug where =spawnlamp random XP was wrong range.

Description:
With the spawnlamp change, it would end up choosing random XP that actually spanned 3 levels:

The entire level,
the entire level before it,
and 1 xp into the next level.

Changes:
Fixes the XP range so that it's only the entirety of the chosen level.
Other checks:

[x] I have tested all my changes thoroughly.

Thanks


Stuck on payemt page

Hi everyone,
It seems the initialized payment and webView runs, but no response is not gotten from the  "snapshot".  I tried printing in onNavigationRequest and _checkTransactionStatusSuccessful but it seemed  "_checkTransactionStatusSuccessful"  is never called because it doesn't print anything placed there.  I however noticed this in the debug console window.
"sendCancelIfRunning: isInProgress=falsecallback=ImeCallback=ImeOnBackInvokedCallback@236984737 Callback=android.window.IOnBackInvokedCallback$Stub$Proxy@f1e1952"

Hmmm.. Any findings @greencodder

Still nothing....I had to try inappwebview and I could read when success message is rendered and navigate to a verification page, then call the verify transaction API to verify transaction using the reference code before proceeding but I am having a hard time listening to "Cancel Payment" event.  do I need any system permission enabled? @popekabu

@greencodder Sorry for the late response. I would like to know if you were able to use it or what your final resolutions are?

Hello,
Anyone manage to pass these?
its stuck-on payment page doesnt execute any function after payment. not cancellation

Hello,
Anyone manage to pass these?
its stuck on payment page doesnt execute any function after payment. not cancellation

Same issue. transactionCompleted is never called.

@popekabu try the inappwebview onroutechange  callback you can try matching the url since successful payment url is different from failed one

To be honest.  If the results are good i will upload a new
version and mention you as part of the contributors in my readme.
Thank you
On Tue, 16 Apr 2024 at 11:32 PM, Collins Mark @.***>
wrote:

@popekabu https://github.com/popekabu try the inappwebview
onroutechange callback you can try matching the url since successful
payment url is different from failed one
—
Reply to this email directly, view it on GitHub
https://github.com/popekabu/pay_with_paystack/issues/17#issuecomment-2060069865,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGMACNMUCLLQ5E5RKJP2L3DY5WYHFAVCNFSM6AAAAABAYXR7KSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRQGA3DSOBWGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Alright then

From: Daniel Kabu Asare @.>
Sent: Wednesday, April 17, 2024 7:05 AM
To: popekabu/pay_with_paystack @.>
Cc: Collins Mark @.>; Comment @.>
Subject: Re: [popekabu/pay_with_paystack] Stuck on payemt page (Issue #17)
To be honest.  If the results are good i will upload a new
version and mention you as part of the contributors in my readme.
Thank you
On Tue, 16 Apr 2024 at 11:32 PM, Collins Mark @.***>
wrote:

@popekabu https://github.com/popekabu try the inappwebview
onroutechange callback you can try matching the url since successful
payment url is different from failed one
—
Reply to this email directly, view it on GitHub
https://github.com/popekabu/pay_with_paystack/issues/17#issuecomment-2060069865,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGMACNMUCLLQ5E5RKJP2L3DY5WYHFAVCNFSM6AAAAABAYXR7KSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRQGA3DSOBWGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHubhttps://github.com/popekabu/pay_with_paystack/issues/17#issuecomment-2060444630, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUY6AOOL4WEK3JKXAFLMEWDY5YGJRAVCNFSM6AAAAABAYXR7KSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRQGQ2DINRTGA.
You are receiving this because you commented.Message ID: @.***>

Thank you Sir
On Thu, Apr 18, 2024 at 11:25 AM Collins Mark @.***>
wrote:

Alright then

From: Daniel Kabu Asare @.>
Sent: Wednesday, April 17, 2024 7:05 AM
To: popekabu/pay_with_paystack @.>
Cc: Collins Mark @.>; Comment @.>
Subject: Re: [popekabu/pay_with_paystack] Stuck on payemt page (Issue #17)
To be honest.  If the results are good i will upload a new
version and mention you as part of the contributors in my readme.
Thank you
On Tue, 16 Apr 2024 at 11:32 PM, Collins Mark @.***>
wrote:

@popekabu https://github.com/popekabu try the inappwebview
onroutechange callback you can try matching the url since successful
payment url is different from failed one
—
Reply to this email directly, view it on GitHub
<
https://github.com/popekabu/pay_with_paystack/issues/17#issuecomment-2060069865>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AGMACNMUCLLQ5E5RKJP2L3DY5WYHFAVCNFSM6AAAAABAYXR7KSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRQGA3DSOBWGU>

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub<
https://github.com/popekabu/pay_with_paystack/issues/17#issuecomment-2060444630>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/AUY6AOOL4WEK3JKXAFLMEWDY5YGJRAVCNFSM6AAAAABAYXR7KSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRQGQ2DINRTGA>.
You are receiving this because you commented.Message ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/popekabu/pay_with_paystack/issues/17#issuecomment-2063636332,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGMACNMHS5QJOIW6PFZVGSDY56UQ3AVCNFSM6AAAAABAYXR7KSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRTGYZTMMZTGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


MAX7219 change type/modules/pins from webpage

I see that the MAX7219 settings is hard coded into the bin file. And I think that it could be nice to be able to change HW type, number of modules and pins used via the web interface.
Hardware type is dynamic and can be changed at run time. You need to use the MD_MAX72xx method setModuleType().
Look at the MD_MAX72xx example MD_MAX72xx_Dynamic_HW.
Is there anyway that I could assist you in getting this defined by software via the webpage?
Ref.: https://arduinoplusplus.wordpress.com/2017/04/14/parola-a-to-z-adapting-for-different-hardware/

This can be closed, same as #14


Rename single-character variables

Issue #55

Looks good, thanks.
+1.

+1

Ok, nobody seems to care about cls vs class_, so lets go with the current convention for now. Thanks for this @dcolligan.


 with react-instantsearch-hooks-router-nextjs triggers double onStateChange

🐛 Current behavior
When there's a  component being used, making updates to any of the algolia states causes onStateChange to be called twice. Once with the configure object, second time without.
First console log of uiState:
{
instant_search: Object
      configure: Object
           hitsPerPage: 30
      query: "12"
}

Second console log of uiState:
{
instant_search: Object
      query: "12"
}

🔍 Steps to reproduce

Add  as a children under . Add any configuration to , in my case I used hitsPerPage
Add custom onStateChange handler. Add a console log to observe uiState.
Trigger onStateChange by making updates to any of the algolia components (Hierarchical Menu, etc)
Observe the two console logs being printed.

Live reproduction
https://codesandbox.io/s/epic-euler-k14dg9?file=/pages/index.tsx:2035-2048
💭 Expected behavior
onStateChange should only be called once, and always with configure object. The extra state change is causing unexpected re-renders throughout the application. Thank you.
Package version
react-instantsearch-hooks-router-nextjs: 6.40.0, react-instantsearch-hooks-web: 6.38.3, react-instantsearch-hooks-server: 6.39.2
Operating system
No response
Browser
No response
Code of Conduct

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

Hi @LefanTan, thanks for raising this issue and providing a CodeSandbox for it !
Indeed this is something we have overlooked, Configure does not persist in the URL. However calling setUiState without it doesn't seem to override the SearchParameters so it doesn't trigger any new network request nor a re-render.
What's causing the re-render is actually the push action with the Next.js router, here's a screenshot of React DevTools profiler on the CodeSandbox you sent :

There are 5 renders when clicking on a refinement list item :

Upon clicking on the refinement list item, it is re-rendered to show that it's checked
After receiving results from Algolia, we update the UI accordingly
Seems to be re-rendering Next.js links before the route push ?
Route has changed, hydrateRoot  is called and re-renders the whole page
RouteAnnouncer gets re-rerendered

You can notice that even if setUiState is called twice, it actually doesn't trigger a render the second time, it's the Next router that does.
This is more of a drawback from having to use Next.js router in order to have functioning popState actions. I think there are ways to push Next.js compatible state without triggering a re-render of the whole page, but I wouldn't recommend it as it's not future-proof.
I think the second render is fine as it's done in a debounced manner after the user has totally typed their query, and it's done when they would normally look at the results.
Now I'm aware this creates unnecessary noise when using a custom onStateChange, so I'll still look for a fix for that (I think if we call setUiState only when the push wasn't from InstantSearch should do the trick), but be aware that this won't change the fact that a whole re-render is done when the URL changes.

I have the same - it performs 3 queries with each click, which causes jumps and shows different query execution times many times.

@aymeric-giraudet Thanks a lot!


Finished fetching IPMI resource status data in 10.002 seconds (success: False)

I've tryied setting it up but it won't connect can connect via the command line using  ipmitool -I lanplus -H <IP_ADDRESS> -U root -P XXXXXXXX sdr elist all does the addon support lanplus as a connection method

lanplus is supported. In fact it's the first connection type that the server tries. Maybe I'll improve this to be able to spit the error in the log.

following up this what is being shown on the webserver {"success":false,"message":"Wrong connection data provided!"}

Please try to open the following url in your browser:
http://YOUR_HASS_IP:9595/?host=YOUR_SERVER_IP&port=623&user=YOUR_USER&password=YOUR_PASSWORD and let's see what it returns.
YOUR_HASS_IP is the IP address of your home assistant server.
YOUR_SERVER_IP is the IP address of the IPMI server.

now that's returning json data

That's exactly what the integration does when using the addon.

well looks like the connect information isn't getting passed for some reason

Might be related to https://github.com/ateodorescu/home-assistant-ipmi/issues/10

that as fixed the problem

fixed in https://github.com/ateodorescu/home-assistant-ipmi/releases/tag/v1.1.1


[DS-2879] Make it configurable if reviewers should be notified about return to pool: XmlWorkflow implementation

Support for the "notify.returned.tasks" config in the configurable (xml) workflow with the difference that the default for the configurable workflow is false.

+1 by inspection

+1 as well. Small and obvious update to keep the XML workflow "synced" with the default one

See #1262


Corrected typo - missing period - in Waypoint: Continue working with Nodejs Servers

Hope I'm doing this correctly. This is my first pull request.

Thanks for the contribution! In the future please name your branch in accordance with the conventions described in the Pull Request Contribute wiki article. Happy coding!

Will do. @ltegman - The example given was "fix typo in basic-javascript challenge" - should my future PRs be all lower case and present tense?

@emersonp The way to named your PR was fine, the problem was that you didn't create a branch in your local repository. For example, in the case of a fix like this you'd do something like the following before making your code change: git checkout -b fix/waypoint-continue-working-with-nodejs-typo. This will create a new branch on your local repository called fix/waypoint-continue-working-with-nodejs-typo and switch to it. Then, after you had completed your change and committed them to that branch you'd do git push -u origin fix/waypoint-continue-working-with-nodejs-typo. This will push the new branch you created and worked in to your remote on Github.
This blog post has a pretty good run down of the full process for FCC specifically that might help simplify it.

Oh, okay - awesome, thanks!


Update CODEOWNERS for support / DevProd

Add monitoring of .github for support automation
Group DevProd / Support items together

Merging as the change was previously approved and the new version is only moving lines around to be more consistent

@bot-gradle merge

The merge queue build has started. Click here to see all failures if any.

The merge queue build has started. Click here to see all failures if any.


Plays over Connect, but not locally

I can't figure out if I'm being a complete newb or misunderstanding the purpose of this Skill?
I can use it to play/control Spotify on another device, but it won't stream music directly through the Mycroft.
If I ask it to list all Spotify devices, it will list my PC, or None (depending if the PC has Spotify open or not). I'd like to stream Spotify directly through my Picroft, am I missing a trick here?
Thanks

You can install Raspotify to turn Picroft into a client as well.

The skill is basically a spotify connect skill and will play locally if there is a Spotify connect device there. as @ConorIA suggest you can install Raspotify on the picroft. This comes pre-installed on the Mark-1 but I'm not 100% sure it's bundeled on the picroft image. you might need to install it separately. If you configure the client to have the same name as the mycroft device the skill will default to this device.

@forslund I am running this on Mark 1 and I am seeing something similar to above. When I newly install the skill or reboot, it recognises Mark 1 as a Spotify device and works perfect but once I leave the Mark 1 alone for a few hours, it doesn't see itself as a device anymore and will only play to other devices.
Is this something you know about?

@JamesPoole I've seen this myself from time to time and asked the librespot team. Apparently the client doesn't gracefully handle disconnects from from the spotify servers. And can be left hanging in a running but not connected state.
I'll see if I can create a workaround restarting the client if it seem to be lost.

Super, thanks for all the work!

@forslund am running this in a raspbian instance with mycroft installed, and it seems to be failing to recognize the spotify device. Raspotify is installed, and the device name for raspotify is correctly configured. The device is properly paired with my spotify account. When I attempt inputting "pause spotify", I get the following error:
17:02:08.595 - mycroft-spotify_forslund:__pause:779 - INFO - Pausing Spotify...
17:02:08.764 - mycroft-spotify_forslund:pause:184 - ERROR - http status: 404, code:-1 - https://api.spotify.com/v1/me/player/pause?device_id=None:Device not found
Is there a config somewhere I missed?

@DeIonizedPlasma That looks like a bug, thanks. I'm not sure it'll do much harm though.
Currently mycroft can only pause spotify playback it has started in the first place. In the case above it looks like there is no current device id so the pause command fails. this would be expected if mycroft did not start playback.

Thanks for the quick reply. I know that mycroft was properly connecting to spotify, because when I stopped all mycroft processes it paused spotify playing on a different device.
I think I may have fixed it now, although I'm not sure how. I had been running mycroft in debug mode, and upon switching to just cli it seems to be partially working. It won't pause/play if I attempt to do so immediately after starting, regardless of whether I am playing music on another device. If I tell it to play a song, it will do so on whatever device is set to play through spotify, and will then play/pause properly on all devices. The only thing that seems strange about this operation is the need to queue a song first on mycroft even if all the controls are for audio on a different device.

I'm a bit confused about the suggestion to install and configure raspotify. From what I can see, the skill tries to manage "librespot" itself if the device is in MANAGED_PLATFORMS. If I disable this MANAGED_PLATFORMS check, simply having librespot installed seems to be enough to make it all work. So I'm wondering, why is this MANAGED_PLATFORMS check there? All it seems to do is break the ability to play on the speaker on Picroft without manually installing and configuring a complete Raspotify installation as well (as opposed to only having librespot installed).

Raspotify is the name of the package for the pi. It installs librespot and sets up aservice running it

The problem with picroft is that the sysyem isn't set up with the application preinstalled. The same goes for desktop installations where librespot need to be built from source or the spotify desktop app can be used.

Yeah but won't it just fall back to trying to play on another device when it can't run librespot locally?
Even if you install raspotify, that isn't enough on the Picroft, you have to manually configure that service to run it. Just installing it seems to be enough if you remove the MANAGED_PLATFORMS check though

You should also shutdown the service so two instance aren't running.

similar to #135 ?


[bug] Client.get_users_tweets() exclude=replies but returned reference_tweets.type=replied_to

well from today and now on I expect a lot of changes in Twitter API v2, but is there any explanation for this?
my code is as follows:
import tweepy as tw
import configparser.ConfigParser(interpolation=None)
config.read(CONFIG)
bearer_token = config['twitter']['bearer_token']
client = tw.client(bearer_token=bearer_token)

problem start here
test = client.get_users_tweets(735358344106549248,
                        exclude='replies',
                        max_results=5,
                        tweet_fields=['public_metrics','created_at'],
                        expansions='referenced_tweets.id.author_id')
for t in test.data:
    if getattr(t, 'referenced_tweets'):
        print(t.referenced_tweets[0].type)

it returns
replied_to
quoted
quoted

I expected replied_to would never be returned since it was exclude by the func parameter

This is a bug with Twitter's API where using exclude=replies with the GET /2/users/:id/tweets endpoint doesn't exclude replies to the user's own Tweets.
I was made aware of the bug through a support post in the Tweepy Discord server and reported the bug to Twitter back in November. The developer advocate I reported it to said they confirmed the bug and reported it, but they no longer work there now, so I don't really have a means to follow up on it.

thank you for clarifying the issue 👍


Create migration-registry before backup/restore

Create the migration registry deploymentconfig and related
resources (secret, imagestream, service) in source and destination
clusters before starting backup or restore.

LGTM.
Nothing to add beyond what @djwhatle has noted.  Nice Job!

Updated with changes. Haven't tested the updated PR yet since I'm hitting rebase-related problems right now.

OK, looks like the updated PR works, but now I have to merge/rebase and test again with the annotation PR merged.


Fixed #32908 -- Allowed select_for_update(skip_locked) on MariaDB 10.6+.

ticket-32908
See https://mariadb.com/kb/en/select/#skip-locked

@hramezani Thanks for checking :+1:

@pope1ni Thanks :+1:


keep_val is ignored for the last instruction

When compiled without stop_time, keep_val=False has no effect for the last instruction - the final value is kept instead of setting to 0. The same unexpected behavior for both AO and DO channels. However, it works when using stop_time with some buffer.
Minimal example to reproduce (link to the notebook):
from nistreamer.streamer import NIStreamer

ni_streamer = NIStreamer()
ao_card = ni_streamer.add_ao_card(max_name='PXI1Slot3', samp_rate=1e6)
my_ao = ao_card.add_chan(chan_idx=0)

ni_streamer.clear_edit_cache()

# Instr 1
my_ao.constant(t=1, dur=1, val=0.5)

# Instr 2
my_ao.sine(
    t=3, dur=1,
    amp=1.0, freq=10.12345678, phase=1.23,
    keep_val=False
)

# Instr 3
my_ao.constant(t=5, dur=1, val=1.5, keep_val=False)

# Instr 4
my_ao.constant(t=6.5, dur=0.5, val=2.0, keep_val=False)

# comment out subsequent instructions from the end
# to see that neither resets to 0 when it is at the end of the sequence

This is designed behavior since keep_val has no effect on the last instruction (beyond which the streamer cannot control the device). Use add_reset_tick() to explicitly invoke the desired behavior. See nistreamer/debug_scripts/final_keep_val.ipynb.


Block non-admin users from certain requests

Together with the gateway, limit access to backend routes so the "USER" role tokens can only access the:

GET routes
PUT routes for their own user model

Opened issue in wisp-gateway. Only change in wisp-users is to send the user role in the introspection response.

/auth/introspect already returns the user's role


Bug: Error "Can't resolve './DIR' in 'PATH'. Rule: eslint: n/no-missing-import"

Environment
Node version: v20.14.0
npm version: v10.7.0
ESLint version: v9.5.0
eslint-plugin-n version: v17.9.0
Operating System: macOS 14.5
What rule do you want to report?
n/no-missing-import
Link to Minimal Reproducible Example
https://github.com/nanoexpress/ultimate-oss/tree/refactor
What did you expect to happen?
Auto-choose /index without warning but results are below

Participation

[ ] I am willing to submit a pull request for this issue.

Additional comments
No response

This is because, as it's an ESM import, we dont allow index.* main files.
Imports require a full file path.
I assume you're putting your code through a transpiler that is incorrectly resolving this :thinking:

If I’m add /index suffix, it works but without /index suffix it does not working

Yeah, so in this case, the import is technically missing. This is because you should have the full file url in the import (this includes the file extension).
This I believe is to match node to browsers more closely, as exact file paths are required in a browser environment.

The import style you're attempting to use in the legacy require pattern with the esm import.
You can see us patching require here (where we allow it to lookup files via the main field in the package json, and searching for index when a directory is provided):
https://github.com/eslint-community/eslint-plugin-n/blob/1bcb006e6d7ab4949a653d663801ee4dd9b0dc35/lib/util/import-target.js#L279-L286

We do attempt to compensate for typescript extensions here:
https://github.com/eslint-community/eslint-plugin-n/blob/1bcb006e6d7ab4949a653d663801ee4dd9b0dc35/lib/util/import-target.js#L291-L294
Taking the mapping from here:
https://github.com/eslint-community/eslint-plugin-n/blob/1bcb006e6d7ab4949a653d663801ee4dd9b0dc35/lib/util/get-typescript-extension-map.js#L21-L27

https://dev.to/ayc0/typescript-50-new-mode-bundler-esm-1jic
I was wondering you can safely disable the rule if using ts + bundler - ts will report an error if you are importing a non-existent module. 🤔

If it's ok then i close this issue. Code with /index.ts works so no issue just looks a much different


Add STM32F746 to list of supported USB devices

I have tested with the USB serial example and can confirm all works as expected

Great, thank you @andybest!

Great, thank you @andybest!


Symfony/Doctrine fail to find an entity based off of its binary UUID

I have created a new Symfony app with version 7.0.7 and doctrine/orm ^3.1. I followed the instructions to use binary UUIDs and then created a CRUD for my entities. Entity A is standalone and works well. Entity B has a ManyToOne relation to A and while the form displays fine, I can't validate it and thus create a new B.
I was able to pin the issue here, in vendor/symfony/doctrine-bridge/Form/ChoiceList/ORMQueryBuilderLoader, line 70:
            // Filter out non-integer values (e.g. ""). If we don't, some
            // databases such as PostgreSQL fail.
            $values = array_values(array_filter($values, fn ($v) => (string) $v === (string) (int) $v || ctype_digit($v)));
        } elseif (\in_array($type, ['ulid', 'uuid', 'guid'])) { // <-----
            $parameterType = ArrayParameterType::STRING;

now if I modify this to:
            // Filter out non-integer values (e.g. ""). If we don't, some
            // databases such as PostgreSQL fail.
            $values = array_values(array_filter($values, fn ($v) => (string) $v === (string) (int) $v || ctype_digit($v)));
        } elseif (\in_array($type, ['ulid', 'uuid', 'guid', 'uuid_binary'])) {
            $parameterType = ArrayParameterType::STRING;

then it works like a charm.
I feel like I have missed something in the setup of this lib but I can't pinpoint what/where.
config/packages/doctrine.yaml
doctrine:
    dbal:
        url: '%env(resolve:DATABASE_URL)%'
        profiling_collect_backtrace: '%kernel.debug%'
        use_savepoints: true
        types:
            uuid_binary:  Ramsey\Uuid\Doctrine\UuidBinaryType

I also tried this in my Kernel but that does not work either:
public function boot(): void {
     parent::boot();
    \Doctrine\DBAL\Types\Type::addType('uuid_binary', 'Ramsey\Uuid\Doctrine\UuidBinaryType');
    $entityManager = $this->getContainer()->get('doctrine.orm.default_entity_manager');
    $entityManager->getConnection()->getDatabasePlatform()->registerDoctrineTypeMapping('uuid_binary', 'binary');
}

In your case, binding a string parameter won't work, because the column is not a CHAR type but a BINARY type.
A BINARY type though does not seem to be supported in doctrine.
If I bind the binary representation of the ID in a param I get an error from the ORM.
This is the parameter:
->setParameter(':column', Uuid::fromString($client->getId()->toString())->getBytes())

and the error is Unhandled match case 16  which is coming from this method in doctrine/orm and is simply missing a match for ParameterType::BINARY.
Adding the type explicitly results in the same, binding it as string has the same effect as @BenoitDuffez, where no results are returned.

Your comment is valid for querying the database. Indeed we need to set a parameter to the bytes of the UUID. One note about your code though, I think it would be easier with
->setParameter(':column', $client->getId()->getBytes())

indeed, the getId returns a UUID that you can extract the bytes from.
However, my ticket was about forms. If you declare relationships between two entities, they are linked with their UUID and it works fine in the database. The problem is that when saving the form, the form data type is uuid_binary and Symfony has a static check that does not include this (but does include uuid).
Changing Symfony code is wrong, perhaps an upstream patch would be accepted but it also feels wrong to me. I don't know if there are events or configuration items that can be leveraged by this library so that Symfony knows how to handle uuid_binary.
PS: by "Symfony" I mean symfony/doctrine-bridge

@BenoitDuffez You are right, they are not related, I will make a separate issue for my problem.


New to the list

ADD:
from : https://default.tmail.thehp.in/mailbox
a.zeemail.xyz
And add:
from https://demo.lobage.com/trashmails/en
trashmails.xyz

Thanks @DanielAbsi - the lobage.com seems not to work and it seems to be the demo of this software which can be purchased
thehp.in - we will add to our system, thanks

Thanks @DanielAbsi - the lobage.com seems not to work and it seems to be the demo of this software which can be purchased
thehp.in - we will add to our system, thanks

it's still a disposable. some people use it becuase they know that it's for demo and not for disposable. people are looking for not detected solutions.

@DanielAbsi - this domain trashmails.xyz seems not to be hosted there... We do not want to add domains separate from providers. Have you tried it? This service looks like a demo and the income does not work. One theory is that this domain trashmails.xyz is used from different provider we do not know the url

default.tmail.thehp.in has been added to the system, we automatically detect new domains (if any)

Now from the with the same interface: minutemailbox.com
add freemailus.com


LoadFromStream Fails

Hello,
The SaveFromStream works, but LoadFromStream raise an exception.

What Delphi version?

XE7up1 - the error: unexpected end of file were "{" was expected but found "["

Do you have an example JSON string? My JSON strings all work.

Just use JsonDataObjectsTest and check only LoadFromStream test.

No problem with XE7Up1 and XE8Up1 and the TestLoadFromStream test case on my system. And I don't have local changes in the working copy.

Here is a sample project: http://we.tl/P0azSt6nOm

In your sample project the memory stream is at the end of the stream. So there is nothing to read for LoadFromStream and that's why the EOF exception is thrown.
A "vStringStream.Position := 0;" helps.
But this only gives me "Unexpected end of file where "{" was expected.". You got unexpected end of file were "{" was expected but found "[".. So either the stream that you use in your actual application is at a different position than "end of stream"/"begin of stream" or something else is wrong.

Ok that was what i forgot to put on the sample  ...
The actual problem is that i parse the record into JsonObject (as you can see
in my sample) and the i'm trying to save the Json into CLOB field.
(CLOB Field: http://www.elevatesoft.com/manual?action=viewtopic&id=edb2sql&topic=String_Types)
with this line:
FieldByName('Data').AsString := vJsonObject.ToJSON;
This save fine the JSON to the Clob field. (IOW, I can see the JSON text from the DB)
Now the problem is when i try to load it ...
var
vStream: TMemoryStream;
FJsonObject: TJsonObject;
begin
vStream := TMemoryStream.Create;
FJsonObject := TJsonObject.Create;
//btw, this doesn't work -> FSuperObject.Parse(FExaminationQuery.FieldByName('Data').AsString);
TBlobField(FExaminationQuery.FieldByName('Data')).SaveToStream(vStream);
vStream.Position := 0;
FJsonObject.LoadFromStream(vStream);
throughs the below error:
Expected "," but found value
procedure TJsonReader.AcceptFailed(TokenKind: TJsonTokenKind);
now, if i save the clob data into a file, from notepad++ i can read it but it has not character encoding checked on the notepad++.
If i set it to UTF8, save the file and load it from JsonObject it parse it correctly !!!
So i the problem was the encoding !!!
Load/Parse it as Unicode encoding solved the problem:
FJsonObject.LoadFromStream(vStream, TEncoding.Unicode);
IOW, it parse it from the Database correctly.
So, the LoadFromStream detects a Unicode text incorrectly as UTF8 or i must set it my self ?
Thank you for your help !!

procedure LoadFromStream(Stream: TStream; Encoding: TEncoding = nil; Utf8WithoutBOM: Boolean = True);
If no encoding is specified it assumes UTF-8 (Utf8WithoutBOM=True) or ANSI (Utf8WithoutBOM=False). There is no way to identify UTF-16 without a BOM. So you have to tell JsonDataObject that your stream is TEncoding.Unicode.

Got it :)
Thank you a lot !!


"signature not found" - but payment goes through anyway

We're getting an intermittent issue on iOS whereby we are receiving a client error ("grpc_message":"unhandled error from SubmitTransaction: signature not found","grpc_status":13), but the transaction actually does go through anyway.
Further to this (and really important to us) is that the Event Webhook does not get called in this case.
tx: 28DYsuJ3zyds4MUVAnguUneKKmjZgRAr9pFS5S2QrWoCYPo1S7Xh1L353AHdbryxADhD7aebXN64GTzYm8u8A5e2
:white_check_mark: Client success
:white_check_mark: Payment goes through on blockchain.
:white_check_mark: Agora event webhook is fired
tx: 4KQYYBTCGyoSspwrJess7KA9duMf3d8ZVXFuNFhSugjGSBwHu7kESRfwt4M5esq3qV5dm2iCMikWRf7uHnhGCeXe
:x:  Client error (signature not found)
:white_check_mark:  Payment goes through on blockchain
:x:  Agora event webhook is NOT fired
Full iOS SDK error:
transientFailure(error: Error Domain=io.grpc Code=13 "{"created":"@1619040502.719064000","description":"Error received from peer ipv4:<IP_ADDRESS>:443","file":"ios/Pods/gRPC-Core/src/core/lib/surface/call.cc","file_line":1056,"grpc_message":"unhandled error from SubmitTransaction: signature not found","grpc_status":13}" UserInfo={io.grpc.HeadersKey={
}, io.grpc.TrailersKey={
}, NSDebugDescription={"created":"@1619040502.719064000","description":"Error received from peer ipv4:<IP_ADDRESS>:443","file":"ios/Pods/gRPC-Core/src/core/lib/surface/call.cc","file_line":1056,"grpc_message":"unhandled error from SubmitTransaction: signature not found","grpc_status":13}, NSLocalizedDescription=unhandled error from SubmitTransaction: signature not found})

Thanks @showered this is an issue that is being tracked - the problem arises when Agora doesn't receive a definite response from the blockchain.
A patch is in the works where Agora will return the tx id when the response is ambiguous (not a success or fail) - that will enable the client to confirm the status of the transaction

Thanks @hitwill that's good to hear. Do you think the webhook issue will also be resolved as part of the same fix? Or will we need to tweak our flow so that the tx id needs to be manually looked up?

@showered - The webhook unfortunately can't be guaranteed in this case. It's actually Solana that gives an ambiguous response so Agora is also uncertain. However, once it returns the tx id to the client, the client (or server) would need to use it to confirm the definite response.

https://github.com/kinecosystem/kin-ios/blob/7df69dc329339f2d088be36ab2aee57b5054fdb0/KinBase/KinBase/Src/KinAccountContext.swift#L532-L536
Wondering if these lines have our hint. At the very least it's a redundant else if, could be more. But it does look like Android has some errors defined that iOS does not.

That's definitely duped code. I wonder if the dev meant to put a different condition there.

This issue is now resolved in version 2.0.0.
Thank you to everyone involved in providing the fix!


Update Button.js

Added some comment, so the code looks more beautiful.

Issue:

Please create an issue before making a contribution. This contribution messes up the code and does not solve any issue or provide any enhancements.


Appstudio update test-component-pac-xzev

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-xzev-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
12 seconds
clone-repository

✅ Succeeded
19 seconds
build-container

✅ Succeeded
18 seconds
deprecated-base-image-check

✅ Succeeded
31 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
7 seconds
clair-scan

✅ Succeeded
17 seconds
inspect-image

✅ Succeeded
7 seconds
label-check

✅ Succeeded
6 seconds
optional-label-check

✅ Succeeded
5 seconds
show-summary

✅ Succeeded
5 seconds
show-sbom


Uable to get SYNC request after successfull account linking : stackdriver error SYNC: Request ID<PHONE_NUMBER>849770471 failed with code: INTERNAL_ERROR

the console stackdriver  log is this :+1: {  insertId:  "1lzw0tcg2xlyxvv"    logName:  "projects/testsmarthome-648a6/logs/actions.googleapis.com%2Factions"    receiveTimestamp:  "2019-04-24T13:36:50.274761403Z"    resource: {   labels: {    action_id:  "SMART_HOME_SYNC"        project_id:  "testsmarthome-648a6"        version_id:  ""       }   type:  "assistant_action"     }  severity:  "ERROR"    textPayload:  "SYNC: Request ID<PHONE_NUMBER>849770471 failed with code: INTERNAL_ERROR"    timestamp:  "2019-04-24T13:36:50.265392966Z"   }
2.my server can't receive any SYNC request ,after successfull account linking
3. is my android system error? make the INTTERNAL_ERROR flag?
4. i try many times! but the stackdirver log alway the same result;

the console stackdriver log is this
{ insertId: "1lzw0tcg2xlyxvv"  logName: "projects/testsmarthome-648a6/logs/actions.googleapis.com%2Factions" receiveTimestamp: "2019-04-24T13:36:50.274761403Z"  resource:        {             labels: { action_id: "SMART_HOME_SYNC" project_id: "testsmarthome-648a6" version_id: ""        }  type: "assistant_action" }  severity: "ERROR"  textPayload: "SYNC: Request ID<PHONE_NUMBER>849770471 failed with code: INTERNAL_ERROR" timestamp: "2019-04-24T13:36:50.265392966Z"  }
my server can't receive any SYNC request ,after successfull account linking
is my android system error? make the INTTERNAL_ERROR flag?
i try many times! but the stackdirver log alway the same result;

can  anybody help me,?

Is there an issue with your fulfillment URL that may mean that it is not being called, or maybe an issue getting an authorization token?

yeah ！ thanks。。the reason is my repsonse of authorization token is wrong，should be json，but i return  header 302 redirect

i got SYNC request now

but request body is null，should be this
{
    "requestId": "ff36a3cc-ec34-11e6-b1a0-64510650abcf",
    "inputs": [{
      "intent": "action.devices.SYNC"
    }]
}

but i got nothing

I'm also facing the same issue while trying to link account, below is the stackdriver log
{   insertId: "1ubusoyg3tiik82"     logName: "projects/mansaa-smart-home/logs/actions.googleapis.com%2Factions"     receiveTimestamp: "2019-04-26T08:38:31.552561849Z"       resource: {      labels: {     action_id: "SMART_HOME_SYNC"         project_id: "mansaa-smart-home"         version_id: ""        }    type: "assistant_action"      }   severity: "ERROR"     textPayload: "SYNC: Request ID<PHONE_NUMBER>0653367970 failed with code: INTERNAL_ERROR"     timestamp: "2019-04-26T08:38:31.520518688Z"    }
Also on the google home app, after account linking the assistant handoff shows

account linking failed
@Fleker can you please look into this as well and suggest what is going wrong?

@sky9971  i fix that question！i know the reason，but now i have another question

@Lxido can you please guide me out of this?
here's my email<EMAIL_ADDRESS>

@sky9971
the reason is response is wrong！，the repsones of token request is json，json format is like that
{
"access_token":"'.$token.'",
"refresh_token":"'.$token.'",
"id_token":"dmcxd329ujdmkemkd349r",
"token_type":"Bearer",
"expires_in":3600
};

you should check your responses @sky9971

now i got another question.
when i get the Sync request or  devices.Query. i found the request post body is null, i can't get post data,because it just nothing

now  you can seee the POST is null.
it's my talk to google assistent 'turn off the light'
i get the request from google home.but POST data is null,i can't how to do next,just get the request

@Lxido I did checked my token endpoint and verified the response. the response is in json format.
Are you sure it is because of token response?

@sky9971 now i get sync request after chage the json response .

so i think question is the json response;

@Lxido  @Fleker
I tried to link the action from two different phones first from nexus 5 and second from iPhone 6.
iPhone app shows "Mansaa Smart Home Linked" in the toast at the bottom of the screen but does not show any devices whereas the android app shows "account linking failed" in the assistant handoff screen.
A little brief intro to what I'm doing
My authroization url is something like : www.xyz.com/auth
As the actions calls this url for oauth setup, it is redirecting to my login page on www.abc.com/login with the help of location header 302 status code.
I the user is able to login and generate auth grant code. I'm passing that to Google's actual redirect url when the user clicks on authorize button.

If you're having account linking questions, then you should investigate account linking issues as that issue is different from the original issue here.
@Lxido how do you know the request is coming from the Google Assistant if the request is empty?

Hi @Fleker  I got your point but I am also getting the sync request failed with internal_error code.
I'll also look into investigating account linking, but for the mean time can you please help me with this internal_error code in sync request? In understanding what is the causing this?

Account linking occurs before any other requests like SYNC are made. If account linking does not work, then that's the first point to start.

@Fleker  the device list is already add the google home。
i try to trigger the turn on the device command，then my backend  received the request and print the original data，and found  that the request content is empty。
as shown int the screenshot above

i sync the devices list or turn on the device was all trigger this uri。
then my backend receive the request is empty，not just like official example：

{
    "requestId": "ff36a3cc-ec34-11e6-b1a0-64510650abcf",
    "inputs": [{
        "intent": "action.devices.SYNC"
    }]
 }

i can receive all trigger request,for example,sync devices turn on device ,turn off device,but backend always got those request is empty

original header

  ["header"]=>
  array(9) {
    ["https"]=>
    string(2) "on"
    ["host"]=>
    string(21) "api-de.ifanscloud.com"
    ["connection"]=>
    string(5) "close"
    ["content-length"]=>
    string(3) "386"
    ["content-type"]=>
    string(30) "application/json;charset=UTF-8"
    ["google-assistant-api-version"]=>
    string(2) "v1"
    ["authorization"]=>
    string(39) "Bearer 9ef2dbb95aed2a6a0346f04b6a104fc2"
    ["user-agent"]=>
    string(85) "Mozilla/5.0 (compatible; Google-Cloud-Functions/2.1; +http://www.google.com/bot.html)"
    ["accept-encoding"]=>
    string(15) "gzip,deflate,br"
  }

request content is

  ["request"]=>
  NULL
  ["cookie"]=>
  NULL
  ["get"]=>
  NULL
  ["files"]=>
  NULL
  ["post"]=>
  NULL
  ["tmpfiles"]=>
  NULL

@Fleker can you help me ,thanks

It's not clear what's happening here, as you definitely should be getting a request body. I think there should be some other parameter you should be accessing. What language/framework are you using?


chore: change sort order for retrieving err items

This is being done to address the system test flakiness by having
the most recently created error item listed first in the list
of error items retrieved from the error reporting service.

This isn't a user visible change.  The errors-api-transport file, used to retrieve error groups from the API, is only used in the system test.


TryStatement should use handler instead of handlers

The TryStatement node has changed according to https://github.com/estree/estree/issues/1 from
interface TryStatement <: Statement {
    type: "TryStatement";
    block: BlockStatement;
    handlers: [ CatchClause ];
    guardedHandlers: [];
    finalizer: BlockStatement | null;
}

to
interface TryStatement <: Statement {
    type: "TryStatement";
    block: BlockStatement;
    handler: CatchClause | null;
    finalizer: BlockStatement | null;
}

The differences are:

handlers (an array of CatchClause) is changed to handler (a single CatchClause, if any)
guardedHandlers is gone

See also https://bugzilla.mozilla.org/show_bug.cgi?id=742612.
(Migrated from https://code.google.com/p/esprima/issues/detail?id=433)

Since this a breaking change for Esprima (requiring a major version bump), I will also file an issue for a transition step.

I've updated it so that the empty guardedHandlers should be also removed (ref https://github.com/estree/estree/issues/24).

From our downstream tests, this is going to break (at least) js2cofee and webpack. I plan to propose a change to those projects before changing this.


PhoneField not working when switching language

Please list the package(s) involved in the issue, and include the version you are using
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>
Describe the bug
The PhoneField component works in the language the site is loaded in. When switching language, it is not possible to click the phone_country_select label on the right of the field, which shows countries and their country codes. This is true for when adding a custom field yourself, or when using the OOTB field which is part of the contact input.
Steps to reproduce the behavior:

Log out
Add product to cart and go to checkout in language A
Go back and change site language to language B
Go to checkout and input 4 digits as phone number
Click the flag on the right of the input field
Nothing happens (empty dropdown menu)

Expected behavior
A dropdown menu of all countries to choose from
Screenshots
Current behavior:

Expected behavior:

Do you see the issue consistently across different devices and browsers?

Tried it on desktop/mobile on chrome and safari, encountering the same issue

I was able to repro on my test shop. are you by any chance using the Geolocation app? For me what I noticed was if there was a mismatch of currency with language I'd see this error in my console and the PhoneField dropdown would be empty

If I go back to the store and select the correct currency + language combination (Euro + Fr) then it would work as expected
https://github.com/Shopify/ui-extensions/assets/82546995/c1282636-ae20-4e07-8da7-7a5e5341e086
I'll file a ticket internally


Alt-tabbing sometimes creates duplicate "ghost" windows

I've got an interesting problem and I am 95% sure it's related to the library.
When I use the library, using Alt+Tab on Win 11 sometimes causes a duplicate ghost window to appear:

There are three proper windows for Trans Neuronica here, even though it really is only a single one. Closing it will cause blank pseudo-apps to remain in there, same as you can see in the second row. The issue is very finicky to replicate though so it's possible that I was just lucky and it has nothing to do with it. Nevertheless, here are results from some of my attempts:
electronEnableSteamOverlay(false)
const x = require('./steamworks.js/index.js');
x.electronEnableSteamOverlay(false);
console.log(x);

Successfully duplicated a ghost after a few dozen of alt tabs
electronEnableSteamOverlay(true)
const x = require('./steamworks.js/index.js');
x.electronEnableSteamOverlay(true);
console.log(x);

Successfully duplicated a ghost after a few alt tabs
Comment out both appendSwitch()
const x = require('./steamworks.js/index.js');
x.electronEnableSteamOverlay(true);
console.log(x);

And I commented out the following two lines in index.js:
//electron.app.commandLine.appendSwitch('in-process-gpu')
//electron.app.commandLine.appendSwitch('disable-direct-composition')

Failed to duplicate a ghost after dozens of tries.
But also it's impossible to show the steam overlay.
Comment out in-process-gpu
const x = require('./steamworks.js/index.js');
x.electronEnableSteamOverlay(true);
console.log(x);

And I commented out one of the lines in index.js:
//electron.app.commandLine.appendSwitch('in-process-gpu')
electron.app.commandLine.appendSwitch('disable-direct-composition')

Duplicated a ghost after a few tries.
But also it's impossible to show the steam overlay.
Comment out disable-direct-composition
const x = require('./steamworks.js/index.js');
x.electronEnableSteamOverlay(true);
console.log(x);

And I commented out one of the lines in index.js:
electron.app.commandLine.appendSwitch('in-process-gpu')
//electron.app.commandLine.appendSwitch('disable-direct-composition')

Failed to duplicate a ghost after dozens of alt tabs.
And steam overlay seems to work correctly.
Summary
It seems that disable-direct-composition causes the issue and doesn't seem to be required at a first glance. I am sure that command wasn't added for no reason, so it would be great if:

Comments were added to explain why both switches are added (as well as what exactly is the purpose of frame invalidation)
And have a way to prevent disable-direct-composition from being appended.

I'm happy to add a PR for the latter but the former will require someone who knows the whys.

Hey, which version of electron are you using to reproduce this?

@ceifa The version I have in package.json is 23.1.3 with electron-packager on version 17.1.1 (though I don't suppose the latter matters).

The disable-direct-composition is used to disable Microsoft DirectComposition. It being enable can cause weird render behaviors on the overlay over windows.
Maybe this is not a issue anymore on the newest versions... I will try remove it later.


Blast physics miscollision

After creating blast asset, family, actor, classic rigid body doesn't collide with it

Fixed


Error in "Specify receptor information"

File "/home/yyx/DiffPepBuilder/experiments/process_receptor.py", line 84, in read_peptide_info
raise ValueError(f"Receptor '{receptor_name}' not found in JSON file.")
ValueError: Receptor 'alk1' not found in JSON file.
I remember seeing this error in an issue before, and it was suggested to delete a certain cache folder, but today I found that the issue is no longer present

Please ensure that the JSON file specified by the --peptide_info_path flag includes the receptor entry "alk1", as demonstrated in this example file. Regarding the similar issue you mentioned, you can find more details in this issue. Please note that this problem has been addressed in the latest version of our code.

刚刚看源码找到问题了...原来判断是判断文件夹内的所有pdb是否在json里面

感谢指出！我们刚刚对代码进行了修改，现在受体在JSON中没有条目也可以正常处理了。
Yes, and apologies for the inconvenience! The script is designed to facilitate the batch processing of multiple receptors. We have just made slight modifications to the code to support scenarios where some receptors might be processed without their prior details being recorded in the JSON file.


Replacing all BigInt reference to JSBI.BigInt - thought?

Summary
To use JSBI internally to provide wider browser support.
A proof of concept is created at #101
Motivation
Using JSBI allow the library to be used in browser that still doesn't support BigInt natively.
JSBI syntax can be transform with babel-plugin-transform-jsbi-to-bigint. However the reverse may not be easy.
Browsers that we are still supporting includes iOS Safari 12 and 13 that doesn't support BigInt yet.
Note
Pros

Provide wider compatibility, JSBI can be transformed back to BigInt with a babel plugins. This can be done either as part of the library's build pipeline or by consumer

Cons

Codebase is much more verbose and potentially harder to maintain

Question
Is this acceptable at this stage?

This is now done (in #101 )


Autorest Regen Preview to 3.0.0-alpha.20230803.5 by Dapeng Zhang from refs/pull/3483/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3483

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.ChangeAnalysis


Issues building after recent we-lang split

toki!
I'm working on a translation of wttr.in into Toki Pona (WIP, I'm hoping to work more on the translation with some other speakers before submitting it), but unfortunately I can't seem to get it to build via Docker.
(additionally, I actually had to change the alpine:3 to alpine:3.15 as installing pyjq failed with Python 3.10, which is the default in the latest Alpine 3.x release; 3.15 contains Python 3.9.)
somasis ~/src/wttr.in tok ∴ docker build ./
Sending build context to Docker daemon  10.77MB
Step 1/23 : FROM golang:1-alpine as builder
 ---> 155ead2e66ca
Step 2/23 : WORKDIR /app
 ---> Using cache
 ---> fb374e4d8e12
Step 3/23 : COPY ./share/we-lang/we-lang.go /app
COPY failed: file not found in build context or excluded by .dockerignore: stat share/we-lang/we-lang.go: file does not exist

After discussing it with a friend I think it's commit 8fc5c84e95d11883aadd5a3ee4736bde94b4a16c causing things not to build. Reverting it lets it build. But, I figured I ought not submit the pull request before discussing it further since this translation work touches the files split apart by that commit.
Thanks.

Toki, Kylie! Kama pona.
Oops. I had to check the docker build before. I will fix it today, it is a trivial thing. Please use the new (splitted) version for your translation; it should be easier too, because the messages there are extracted intro a separate file

The problem should be fixed now. Please try again.


How to disable cpu caching (IDFGH-10471)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
I want to disable/enable cpu caching in the interrupt function so that I can calculate the read and write cycles of the peripheral registers correctly

calculate the read and write cycles of the peripheral registers correctly

You don't have to disable the cache if you just want to measure the time of read/write cycles. There're other approaches. Please check out this component

According to the ESP32 (and ESP32-S3) TRM, CPU access to peripheral registers is not cached.
Only access to code/rodata in Flash is cached. If you place your interrupt handler function into IRAM, then execution should be predictable (up to the variable interrupt latency due to other factors).

According to the ESP32 (and ESP32-S3) TRM, CPU access to peripheral registers is not cached.
Only access to code/rodata in Flash is cached. If you place your interrupt handler function into IRAM, then execution should be predictable (up to the variable interrupt latency due to other factors).

I need the error between interrupts to be the lowest, for example: 50us delay +/-0.01us max to trigger DMA transmission on time, if using cache the above error will change incorrectly due to  read and write changes.


How to preserve xml entity during save

Input:
Iván
Output:
Iván
I want to preserve all the xml entity during save. How to do that?

I’m not sure what the request is but if I had to guess, you can disable parsing entities by removing parse_escapes flag from the default parsing flags (i.e. parse_default & ~parse_escapes) and disable quoting when saving the document by using format_no_escapes (i.e. format_default | format_no_escapes). You will have to take care of escaping data that is added to the document in C++ though.


disable kube/openshift api server operators before running tests

/assign @dmage
I feel good about this one.

/retest

/retest

/retest

flakes:
https://github.com/openshift/cluster-image-registry-operator/issues/221
https://github.com/openshift/cluster-image-registry-operator/issues/223
both seem new/unrelated to the the GC/+apiserver issues that i think this PR fixes.

/retest

/lgtm

/retest

Added a commit to disable the two flaky tests.

/retest

/retest


OCPBUGS-3881: Revert CatalogSource security changes for PSA

Signed-off-by: perdasilva<EMAIL_ADDRESS>

Note: The verify test will never change as this revert change is being made downstream only.

Carrying over comment from OCPBUGS-3881:
Here's another, small change option to ensure IBM clusters are not impacted, instead of the wholesale revert all PSA changes + re-enable them at a future release :
Introduce logic downstream to add spec.GrpcPodConfig.SecurityContextConfig:legacy for any CatalogSource created in the openshift-marketplace namespace
Mark openshift-marketplace namespace as PSA privileged
That way in 4.13(or whichever release we decide is ready), we just have to revert 1 in downstream (few lines of code) + determine if it's safe to mark openshift-marketplace as restricted again (it not required, but is recommended however to run workload in restricted mode)
Reverting PSA changes wholesale has cascading effects on the work that's already in motion (eg labelSyncer controller, doc changes etc) that all needs to be audited again

There was a discussion here about the suggested changes we could make in 4.12: https://coreos.slack.com/archives/C04C5P2PMR6/p1669069382681739
This PR doesn't align with that discussion. Is that because we've moved on or arrived at a different consensus? Can we get consensus on the plan?

closing this PR as it doesn't meet what original plan T_T