[1.0] [Android] Crash when deviceController subscribeToPath

Reproduction steps
1. Connect the device in Android ChipTool app.
2. Then Click WILDCARD.
3. Input Endpoint:1, Cluster ID: 6, Attribute Id: 0.
4. Click SUBSCRIBE. App will crash.

Bug prevalence
Every time.
GitHub hash of the SDK that was being used
4f7669b052b16bd054227376e1bbadac85419793
Platform
android
Platform Version(s)
No response
Type
Platform Issue
Anything else?
Code:
 deviceController.subscribeToPath(subscriptionEstablishedCallback,
                                       resubscriptionAttemptCallback,
                                       reportCallback,
                                       ChipClient.getConnectedDevicePointer(requireContext(),
                                       addressUpdateFragment.deviceId),
                                       listOf(attributePath),
                                       null,
                                       minInterval,
                                       maxInterval,
                                       keepSubscriptions,
                                       isFabricFiltered)

Logs:
matterlog.txt

I found the reason. The param eventPaths can't be null. Fix by:
deviceController.subscribeToPath(
                subscriptionEstablishedCallback,
                resubscriptionAttemptCallback,
                object : ReportCallback {},
                devicePointer,
                listOf(chipAttributePath),
                listOf(),
                minInterval,
                maxInterval,
                true,
                false
            )


Fix the Stepper in the config wizard

As a user, I want the Config Wizard Stepper to reflect the steps which are required for the flow that I select, so that I have an accurate preview of the steps required to add new systems.
AC

Given that I select the manual inputs, when the first step loads, then the stepper updates to show 3 scanning steps
Given that I select the AWS Scanner, when the first step loads, then the stepper updates to show steps specific to the AWS Scanning
Given that I select the Okta Scanner, when the first step loads, then the stepper updates to show steps specific to the Okta scanner.

Figma

Updated stepper: https://www.figma.com/file/NeFvOOb1RT2fcdqZT0vBF2/Fidesctl-Management-UI?node-id=110%3A6861
Okta Scanning: https://www.figma.com/file/NeFvOOb1RT2fcdqZT0vBF2/Fidesctl-Management-UI?node-id=108%3A5125
AWS Scanning: https://www.figma.com/file/NeFvOOb1RT2fcdqZT0vBF2/Fidesctl-Management-UI?node-id=29%3A943
Manual Input: https://www.figma.com/file/NeFvOOb1RT2fcdqZT0vBF2/Fidesctl-Management-UI?node-id=110%3A6861

TBD: Just need to determine the final number of steps.

Note to myself to add these details to the ticket https://ethyca.slack.com/archives/C0114BAK160/p1666633875802189

My "designs" are in the pink box. We should review with Design Team and have a mini planning meeting to finalize

@mfbrown - I am still extremely confused by the Figma designs I am meant to follow, and when I click your Figma links, for some reason they don't lead to a specific node or design. There are also multiple flows on the page.
Could you please show me (maybe in screenshots added here for simplicity):

The starting state of the stepper, before a scanner or the manual entry is chosen
What the stepper looks like with manual entry
What the stepper looks like with AWS
What the stepper looks like with Okta

Moving to blocked until the above is clearer to me ^ Just want to make sure we're all on the same page for the stepper. Thank you.

The starting state of the stepper, before a scanner or the manual entry is chosen
There will be no stepper until a scanner/manual entry is chosen
What the stepper looks like with manual entry
There is no stepper for manual entry now
What the stepper looks like with AWS
see the second row with the pink background
What the stepper looks like with Okta
see the second row with the pink background

Stepper is not coming back, for now. Can close @mfbrown , @Kelsey-Ethyca , I think


Add test - after commit is executed on final commit

Checklist

[x] This test fails on the 10.x branch.
[x] This test passes on the latest version of the fix/gh-48658 branch.

Thank you!!


Remove workaround for sorting glyph names

Afaik fontmake currently sorts glyph names before passing them to fontmake/feaLib; this was done to work around https://github.com/behdad/fonttools/issues/504. Since @moyogo has kindly fixed this upstream bug, this workaround can now be removed.

I don't think anything was done to fontmake itself, just a one-time operation on the Roboto sources (I've opened an issue for that). Thanks for the heads up!


chore: Move back to creates.io version of criterion

Follow up to this commit: https://github.com/lycheeverse/lychee/commit/91c8bc3ef5ed5b7650c73e9ca373d119f027f1ae
https://github.com/bheisler/criterion.rs/pull/602 has been merged and is included in 0.5.1 of criterion, so we can now start using the crates.io version again.

That's great news. Thank you!


feat(kit): TuiCalendarRange add item property for correctly switch value outside

Closes #8564
Closes #8527

I'ts okay?

Please tell me where to place playwright snapshots correctly?

@zhd-dm we do not store screenshots in the repository

@zhd-dm we do not store screenshots in the repository

How should I do screenshot tests correctly?

@splincode ready to review


Support for custom backend URL

Warden should support custom Admin domain, for example:
backend.enterprise.test

Closing issue for now due to inactivity. Also worth mentioning, since Warden 0.2.0 a wildcard rule has been configured on Traefik to route *.example.com to the project environment with example.com also routing (apex domain) to the project environment as of Warden 0.4.4.


Installed but won't show up in game

Describe the issue
Totalrp3 is installed but won't show up in game in my addon list. It's definitely installed.
Expected behaviour
Give us a clear and concise description of what you expected to happen.
Issue checklist
Please select any applicable options from the below list.

You can replace the "[ ]" with "[x]" for each option below to tick it.

[x] The issue occurs with the latest version of the addon.
[ ] The issue occurs with Total RP 3 as the only enabled addon.
[x] The issue occurs only with other addon(s) enabled (please provide a list). (astral keys, auctionator, bagnon, dbm, details, handynotes, leatrix, omnicd. plater. postal. prat 3.0, sexymap. tomtom, vuhdo, WA, wq tracker

Steps to reproduce
Give us a list of steps to reproduce the issue with.

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

Additional files
Please attach any additional applicable files such as screenshots, error logs, or profile data files. Files can be dragged and dropped into this report to attach them for upload.

The addon not showing up in the list means the addon is not properly installed.
If you're using an addon client, make sure the right WoW version (and path) is selected.
If you're installing manually, make sure both folders (totalRP3 and totalRP3_Data) are placed in Interface/AddOns directly.


algorithm underlying table extract and parsing

Hi,
I tried to search on your website, also here within the code; basically, how can I figure out what algorithm are you using for detecting, extracting tables and turning them into html tags. Or what algorithm are you using to parse the layout of a page.
I am asking, because I am benchmarking multiple methods, and I wonder if I should cite/use your pipeline or rather the original model.
Thank you

Hi @naarkhoo - the open source library uses table transformer for table extraction. Check out the model card below for details.

https://github.com/Unstructured-IO/model-cards/blob/main/TATR/model_card_tatr_opensource.md

Thanks for the answer and sorry to bring up this again;
But I also meant the content and of the table which is turned into HTML;
seems TATR is only doing 1)table detection 2)table structure recognition (rows, columns). so I wonder how text/numbers from tables are extracted and eventually compiled into HTML


Revert "Override MediaQuery for WidgetsApp"

Reverts flutter/flutter#81295
This breaks internal tests. test/OCL:380874320:BASE:381245548:1624543113950:7fda35d8

FYI to author @atoka93

@renyou @Piinks is there any way to re-land it later on? Can I fix the test/s?

@renyou @Piinks is there any way to re-land it later on? Can I fix the test/s?

Hey @atoka93 if you can re-open this change as a new PR, we can look at the failing tests and discuss changes to make it not breaking. Feel free to ping me on the new PR. :)

Why so important commit had been reverted back ?
And still not in stable for so long time ?


Error multiple queries

I try run and works fine
$el = $xmlReader->value('infNFe.det.2')->sole();
Result

But a try multiple queries and give error
$el = $xmlReader->value('infNFe.det.2.prod')->sole();

Hey @fabianosfbr thanks for reporting this - just so I can get a bit more info...

What version of XML Wrangler are you using?
What reader are you using? E.g Reader::fromString() or Reader::fromStream() etc

When you say "try multiple queries and give error" do you mean making multiple searches like this?
$reader->value('infNFe.det.2')->sole(); // Works

// And then

$reader->value('infNFe.det.2.prod')->sole(); // Falls over?

Hey @fabianosfbr I'm going to close this issue for now - please feel free to re-open if you are still having this issue.


Release 0.8.0

/test-all

/test-whole-conformance

The whole conformance test suite passed.
=== Print all results ===
+ sonobuoy results<PHONE_NUMBER>25_sonobuoy_c82f524e-ba52-485f-a51a-de830801dc7c.tar.gz
Plugin: e2e
Status: passed
Total: 4992
Passed: 261
Failed: 0
Skipped: 4731

Plugin: systemd-logs
Status: passed
Total: 3
Passed: 3
Failed: 0
Skipped: 0

The EKS / GKE tests are currently running as well.

The whole conformance test suite passed.
=== Print all results ===
+ sonobuoy results<PHONE_NUMBER>25_sonobuoy_c82f524e-ba52-485f-a51a-de830801dc7c.tar.gz
Plugin: e2e
Status: passed
Total: 4992
Passed: 261
Failed: 0
Skipped: 4731

Plugin: systemd-logs
Status: passed
Total: 3
Passed: 3
Failed: 0
Skipped: 0

The EKS / GKE tests are currently running as well.

It's faster than I imagined, I saw it enabled e2e-parallel, which skipped a few "Serial" tests. Let me run "Serial" ones manually.

When switched to private CI, I missed that enable-parallel is removed in whole-conformance test. Now I removed it.

/test-whole-conformance

@antoninbas all "Serial" and "Conformance" tests passed in my local test. I used --e2e-focus "Serial.*Conformance" as Conformance is always the last word.


Clarify the key object definition for the key management API

Fixes https://github.com/matrix-org/matrix-doc/issues/1907

It's very annoying that Swagger doesn't support the thing we want to do (anyOf on a response schema). It supports it just fine for request parameters, but not in the response schema for some reason.
Error is the cryptic:
JSON_OBJECT_VALIDATION_FAILED",
  "stack": "SyntaxError: Swagger schema validation failed.
  Data does not match any schemas from 'oneOf' at #/paths//keys/claim/post/responses/200
    Data does not match any schemas from 'oneOf' at #/paths//keys/claim/post/responses/200/schema
      Data does not match any schemas from 'anyOf' at #/schema/properties/one_time_keys/additionalProperties
        Data does not match any schemas from 'anyOf' at #/properties/one_time_keys/additionalProperties/additionalProperties
          Data does not match any schemas from 'anyOf' at #/additionalProperties/type
            No enum match for: string,[object Object] at #/type
            No enum match for: {\"type\":\"object\",\"title\":\"KeyObject\",\"properties\":{\"key\":{\"type\":\"string\",\"description\":\"The key, encoded using unpadded base64.\"},\"signatures\":{\"type\":\"object\",\"description\":\"Signature for the device. Mapped from user ID to signature object.\",\"additionalProperties\":{\"type\":\"string\"}}},\"required\":[\"key\",\"signatures\"]} at #/type/1
          Expected type boolean but found type object at #/additionalProperties
        Expected type boolean but found type object at #/properties/one_time_keys/additionalProperties
      Additional properties not allowed: properties at #/schema
    Missing required property: $ref at #/paths//keys/claim/post/responses/200

errrm... looks like swagger. maybe? possibly? </dog_goggles>

It's definitely swagger, although the diff is (and swagger, annoyingly) is very confusing. It's supposed to be fix the nesting levels while also making documentation a thing.

Something isn't happy in CS /keys/upload:


All schema files are returning 404

What happened?
The schema file URLs that used to work are now returning 404. E.g. https://opentelemetry.io/schemas/1.7.1 or https://opentelemetry.io/schemas/1.14.0
What did you expect would happen?
It should be a 200 and return the content of relevant file from https://github.com/open-telemetry/opentelemetry-specification/tree/main/schemas
What is the name + path of the page where you encountered the issue?
https://opentelemetry.io/schemas/*
What browser, OS, and platform were you using when you encountered the problem?
Chrome Version 105.0.5195.125 (Official Build) (x86_64), Mac OS 12.5

@open-telemetry/docs-approvers can someone help with this? I am not sure when exactly did we break this.

The publishing of schema files was previously automated in https://github.com/open-telemetry/opentelemetry.io/pull/715

@chalin would have the most context here.

1.4.0 and 1.5.0 seem to work, so it looks like something broke with 1.6.0

ok. the following versions work:
1.10.0   1.11.0   1.12.0  1.4.0   1.5.0   1.6.1   1.7.0  1.8.0   1.9.0

looks like the spec version has not been upgraded for 1.13 and 1.14. git submodule version is at ae87c81640d18763d4df51b898c55c49e2370328, which is 1.12

@svrnm will you fix it or want me to take a look at this?

@svrnm will you fix it or want me to take a look at this?

I tried it but tbh my git skills are not advanced enough: I tried to pull the 1.14 commit in the Submodule and not HEAD but couldn’t figure it out
So if you know how that would be appreciated

Let me try.

Hi all. I'll be back at my desk in about 15 min. I can give it a try then if you'd like. Let me know.

Ok, back. Btw, there are some extra issues to be dealt with -- see #1495. I can address all of those in one go.

Ok, back. Btw, there are some extra issues to be dealt with -- see #1495. I can address all of those in one go.

Please do and ping me when done so that I can re-run the verification job on the spec repo: https://github.com/open-telemetry/opentelemetry-specification/pull/2868

@tigrannajaryan - I'm on it. The spec has changed, resulting in broken links, so I'm fixing that now.

@tigrannajaryan et al.: the PR is ready for review

#1851

The schema file URLs that used to work are now returning 404. E.g. https://opentelemetry.io/schemas/1.7.1 or https://opentelemetry.io/schemas/1.14.0

@tigrannajaryan - I don't see any 1.7.1 schema under https://github.com/open-telemetry/opentelemetry-specification/tree/main/schemas. Did you mean 1.6.1?

The new spec is live. If you would like any further changes, let me know and I'll include them in a followup PR.

The publishing of schema files was previously automated in #715

To clarify: the schema files get published once the spec content-module is updated. The update of the spec isn't automated since it usually requires manual intervention, such as fixing broken links into the spec pages.

@tigrannajaryan - I don't see any 1.7.1 schema under https://github.com/open-telemetry/opentelemetry-specification/tree/main/schemas. Did you mean 1.6.1?

My mistake. I was looking for a schema that never existed.


Feature/time units

This PR adds a timeUnitFactorSeconds to allow users to control the time units. I'm using a float timeUnitFactorSeconds rather than a string timeUnit like we had discussed because it matches the current implementation for spatial units.
We also need to discuss how to handle this new version on the front end, is it time to increment the version of trajectoryInfo or can we just check for the existence of this field for now and wait to increment after other imminent changes?
Pull request recommendations:

[ ] Name your pull request your-development-type/short-description. Ex: feature/read-tiff-files
[ ] Link to any relevant issue in the PR description. Ex: Resolves [gh-12], adds tiff file format support
[ ] Provide relevant tests for your feature or bug fix.
[ ] Provide or update documentation for any feature added by your pull request.

Thanks for contributing!

Are these values such as 1e-9 ever going to be actually multiplied against time values?  Precision can become an issue.  I'm not sure I understand why the float representation is preferred against the string representation for units.

Codecov Report

Merging #18 (52ce80d) into master (8534bb9) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
+ Coverage   83.57%   83.67%   +0.09%
==========================================
  Files          34       34
  Lines        1017     1023       +6
==========================================
+ Hits          850      856       +6
  Misses        167      167

Impacted Files
Coverage Δ

simulariumio/custom_converter.py
100.00% <ø> (ø)

simulariumio/cytosim/cytosim_converter.py
96.31% <ø> (ø)

simulariumio/data_objects/agent_data.py
100.00% <ø> (ø)

simulariumio/physicell/physicell_converter.py
96.15% <ø> (ø)

simulariumio/readdy/readdy_converter.py
76.69% <ø> (ø)

simulariumio/tests/conftest.py
100.00% <ø> (ø)

...lariumio/tests/converters/test_custom_converter.py
100.00% <ø> (ø)

...ariumio/tests/converters/test_cytosim_converter.py
100.00% <ø> (ø)

...iumio/tests/converters/test_physicell_converter.py
100.00% <ø> (ø)

...lariumio/tests/converters/test_readdy_converter.py
100.00% <ø> (ø)

... and 3 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 8534bb9...52ce80d. Read the comment docs.

Are these values such as 1e-9 ever going to be actually multiplied against time values? Precision can become an issue. I'm not sure I understand why the float representation is preferred against the string representation for units.

Yeah, if I'm understanding your question correctly they will be multiplied against the time values. I'm starting to wonder if it would be better for me to get a new test data file to develop with first, and when I'm sure that's the format I want, we update simulariumio? (ie convert this PR to draft for now?)

Are these values such as 1e-9 ever going to be actually multiplied against time values? Precision can become an issue. I'm not sure I understand why the float representation is preferred against the string representation for units.

Would this make a difference given the output here is JSON? (are precision errors being observed in the output files or further down the line?)

Are these values such as 1e-9 ever going to be actually multiplied against time values? Precision can become an issue. I'm not sure I understand why the float representation is preferred against the string representation for units.

Would string-vs-float make a difference given the output here is JSON? (are precision errors being observed in the output files or further down the line?)

@EricJIsaac by "string units" we mean unit: "ns" as opposed to timeUnitFactorSeconds: 1e-9, sorry that was confusing

We already had this discussion about how to represent units. For the spatial units, we might multiply the factor by the distances and for at least one of my test trajectories last year (I think listeria?) I needed something other than an order of magnitude (i.e. 5e6). I think it would be really confusing if the time and spatial unit factors were handled differently. Can the front end have a function that gets a string unit from the factor? In python you could use pint, it just seems like fairly simple functionality?

We already had this discussion about how to represent units. For the spatial units, we might multiply the factor by the distances and for at least one of my test trajectories last year (I think listeria?) I needed something other than an order of magnitude (i.e. 5e6). I think it would be really confusing if the time and spatial unit factors were handled differently. Can the front end have a function that gets a string unit from the factor? In python you could use pint, it just seems like fairly simple functionality?

Getting a string unit from the factor would be straightforward if the numbers were guaranteed to be nice, for example the time values range from 0 to 15 and the factor supplied is 1e9. Then converting this to 0 to 15 ns would be trivial. But if we had an input like this: time values range from 300 to 600 and the factor is 5e6, then I think I'd want to do some math which might include multiplying the time values by the factor. I think doing this type of math might have more consequence for time units compared to spatial units, because the calculation with the spatial unit is done once when the bounding box is drawn, but with time values we might have to convert the timestamp for every frame that comes in if the factor is not straightforward (e.g. 5e6)??? Maybe/hopefully there is a way to avoid this, I just can't think of one off the top of my head.
I just found a feature of the math.js library (which we don't have installed currently) that seems like it might do what Python's pint does, but again I'm not sure it would be super helpful for a weird case like the above with a factor of 5e6.
I can roll with this time unit factor and just try developing with it. But I'd like to hear @toloudis 's thoughts first since he's the one who first brought this up 😄

We already had this discussion about how to represent units. For the spatial units, we might multiply the factor by the distances and for at least one of my test trajectories last year (I think listeria?) I needed something other than an order of magnitude (i.e. 5e6). I think it would be really confusing if the time and spatial unit factors were handled differently. Can the front end have a function that gets a string unit from the factor? In python you could use pint, it just seems like fairly simple functionality?

Getting a string unit from the factor would be straightforward if the numbers were guaranteed to be nice, for example the time values range from 0 to 15 and the factor supplied is 1e9. Then converting this to 0 to 15 ns would be trivial. But if we had an input like this: time values range from 300 to 600 and the factor is 5e6, then I think I'd want to do some math which might include multiplying the time values by the factor. I think doing this type of math might have more consequence for time units compared to spatial units, because the calculation with the spatial unit is done once when the bounding box is drawn, but with time values we might have to convert the timestamp for every frame that comes in if the factor is not straightforward (e.g. 5e6)??? Maybe/hopefully there is a way to avoid this, I just can't think of one off the top of my head.
I just found a feature of the math.js library (which we don't have installed currently) that seems like it might do what Python's pint does, but again I'm not sure it would be super helpful for a weird case like the above with a factor of 5e6.
I can roll with this time unit factor and just try developing with it. But I'd like to hear @toloudis 's thoughts first since he's the one who first brought this up 😄

I didn't mean to reopen a huge can of worms.  My opinion is that it's safer and clearer to specify a unit from a well known enumerated list of units, and a number multiplier along with it.  So 5e3 becomes [5.0, "km"].
Maybe this is all too nitpicky and the single float value is fine.  Here's the weird case, when the exponent is "in-between":
If a user specifies 4e5, that's equivalent to 400km, right? Or is it 0.4 Mm? (We would have to convert the 5 into the nearest power that gives a recognizable string unit, and multiply the remaining factor against the 4?)

We already had this discussion about how to represent units. For the spatial units, we might multiply the factor by the distances and for at least one of my test trajectories last year (I think listeria?) I needed something other than an order of magnitude (i.e. 5e6). I think it would be really confusing if the time and spatial unit factors were handled differently. Can the front end have a function that gets a string unit from the factor? In python you could use pint, it just seems like fairly simple functionality?

Getting a string unit from the factor would be straightforward if the numbers were guaranteed to be nice, for example the time values range from 0 to 15 and the factor supplied is 1e9. Then converting this to 0 to 15 ns would be trivial. But if we had an input like this: time values range from 300 to 600 and the factor is 5e6, then I think I'd want to do some math which might include multiplying the time values by the factor. I think doing this type of math might have more consequence for time units compared to spatial units, because the calculation with the spatial unit is done once when the bounding box is drawn, but with time values we might have to convert the timestamp for every frame that comes in if the factor is not straightforward (e.g. 5e6)??? Maybe/hopefully there is a way to avoid this, I just can't think of one off the top of my head.
I just found a feature of the math.js library (which we don't have installed currently) that seems like it might do what Python's pint does, but again I'm not sure it would be super helpful for a weird case like the above with a factor of 5e6.
I can roll with this time unit factor and just try developing with it. But I'd like to hear @toloudis 's thoughts first since he's the one who first brought this up 😄

I didn't mean to reopen a huge can of worms. My opinion is that it's safer and clearer to specify a unit from a well known enumerated list of units, and a number multiplier along with it. So 5e3 becomes [5.0, "km"].
Maybe this is all too nitpicky and the single float value is fine. Here's the weird case, when the exponent is "in-between":
If a user specifies 4e5, that's equivalent to 400km, right? Or is it 0.4 Mm? (We would have to convert the 5 into the nearest power that gives a recognizable string unit, and multiply the remaining factor against the 4?)

I like the [5.0, "km"] idea. I think it prevents working with really small or really large numbers, gives the user flexibility (with the 5.0), gives the user/developer clarity ("km" is the unit meant to be displayed), and in the vast majority of cases the first number is going to be 1 which means we won't do any multiplying.
If we applied it to spatial units as well, then we won't have the issue of the user having to follow 2 different protocols for space units and time units.


Not able to connect with SSL

The image tomcat:8.5.57-jdk11-openjdk-buster seems to have missing some Root CA certificates. I'm not able to connect to various SSL sites, e.g. curl https://api.push.apple.com:443 results in:
curl: (60) SSL certificate problem: unable to get local issuer certificate
More details here: https://curl.haxx.se/docs/sslcerts.html

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.

My tomcat webapp throws this error constantly:
sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target

Any idea what is causing this issue?

Looks like https://github.com/docker-library/openjdk/issues/414#issuecomment-644263515
This upstream ca-certificates change was the beginning of June https://tracker.debian.org/news/1150838/accepted-ca-certificates-20200601deb10u1-source-into-proposed-updates-stable-new-proposed-updates/
Taking an image that was before this change: 8.5.5 works as expected
$ docker run -it --rm tomcat:8.5.55 curl https://api.push.apple.com:443
{"reason":"MethodNotAllowed"}

But an image built with the updated ca-certificates will have that error, so the version right after:
$ docker run -it --rm tomcat:8.5.56 curl https://api.push.apple.com:443
curl: (60) SSL certificate problem: unable to get local issuer certificate
More details here: https://curl.haxx.se/docs/sslcerts.html

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.

 Done
Building dependency tree
Reading state information... Done
7 packages can be upgraded. Run 'apt list --upgradable' to see them.
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following packages will be upgraded:
  ca-certificates
1 upgraded, 0 newly installed, 0 to remove and 6 not upgraded.
Need to get 158 kB of archives.
After this operation, 3072 B disk space will be freed.
Get:1 http://deb.debian.org/debian buster-updates/main amd64 ca-certificates all 20200601~deb10u1 [158 kB]
Fetched 158 kB in 0s (8163 kB/s)
debconf: delaying package configuration, since apt-utils is not installed
(Reading database ... 12563 files and directories currently installed.)
Preparing to unpack .../ca-certificates_20200601~deb10u1_all.deb ...
Unpacking ca-certificates (20200601~deb10u1) over (20190110) ...
Setting up ca-certificates (20200601~deb10u1) ...
debconf: unable to initialize frontend: Dialog
debconf: (No usable dialog-like program is installed, so the dialog based frontend cannot be used. at /usr/share/perl5/Debconf/FrontEnd/Dialog.pm line 76.)
debconf: falling back to frontend: Readline
Updating certificates in /etc/ssl/certs...
13 added, 15 removed; done.
Processing triggers for ca-certificates (20200601~deb10u1) ...
Updating certificates in /etc/ssl/certs...
0 added, 0 removed; done.
Running hooks in /etc/ca-certificates/update.d...
done.

root@f11e3c56c9f6:/usr/local/tomcat# curl https://api.push.apple.com:443
curl: (60) SSL certificate problem: unable to get local issuer certificate
More details here: https://curl.haxx.se/docs/sslcerts.html

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.

But I don't understand why there is an error, the site api.push.apple.com does not have any expired certificate: https://www.ssllabs.com/ssltest/analyze.html?d=api.push.apple.com

Sounds like https://bugs.debian.org/962596 (upstream "bug" in debian package; workaround discussed in the bug; still waiting on fix to get pushed out)

Thanks @yosifkit
This temporary workaround fixed my problem:
RUN wget --no-check-certificate -c https://www.geotrust.com/resources/root_certificates/certificates/GeoTrust_Global_CA.pem \
	&& mkdir /usr/local/share/ca-certificates/extra \
	&& mv GeoTrust_Global_CA.pem /usr/local/share/ca-certificates/extra/GeoTrust_Global_CA.crt \
	&& update-ca-certificates \
	&& rm -rf /usr/local/share/ca-certificates/extra

You don't need the --no-check-certificate since geotrust.com is signed by one of the other certs.

The package was updated, seems to work for me. Somebody else able to confirm? So I can close the issue?


fix: depend on a real, deployed RPC version

What
Roll back RPC dependency
Why
main and the latest released version of the CLI are totally broken
because the 3.3.5 release of the RPC was yanked. While we are waiting
for that yank to be rolled back (it was related to a bug in simulation,
which does not introduce security vulnerabilities and which did not need
to be yanked), rolling back the RPC version may work well enough. I
say "may" because I don't really know if there are other dependencies on
the latest RPC here. Let's see if CI passes, i guess?
Known limitations
Unknown!

The RPC dep has been unyanked so we shouldn't need to do this now. Please reopen if I'm misunderstanding.


XSLTProcessor somehow available on macOS X

I've been scratching my head over an unexpected behavior.
I'm trying to read a webpage that requires XSLTProcessor using phantomjs-node, and although I'm well aware that QtWebkit does not support XSLT, for some reason my crawling code works perfectly on a macOS X machine. Here's a snippet that proves the existence of XSLTProcessor:
var phantom = require("phantom");

var phInstance = null;
phantom.create()
    .then(instance => {
        phInstance = instance;
        return instance.createPage();
    })
    .then(page => {
        page.evaluate(function() {
            // this gives [object XSLTProcessorConstructor] on macOS X
            console.log(XSLTProcessor);
        });
    })
    .catch(error => {
        console.log(error);
        phInstance.exit();
    });

On Windows/Linux, this would give ReferenceError: Can't find variable: XSLTProcessor, as expected.
Does anyone know why this happens, and how to make XSLTProcessor available on Windows/Linux as well?

This is more of a phantomjs question and not this module.

On second thought, I actually suspect it has something to do with the phantomjs-prebuilt package, as the provided phantomjs binaries for macOS X might have been built differently (with XSLT on?)
But still, I'd love to receive insights from others and see if this is replicable.

I don't think so. I tried it with my brew install of phantom js on mac and it does work. I didn't have a windows machine to confirm though.
Sent via my iPhone

On Nov 16, 2016, at 9:11 AM, Kent<EMAIL_ADDRESS>wrote:
On second thought, I actually suspect it has something to do with the phantomjs-prebuilt package, as the provided phantomjs binaries for macOS X might have been built differently (with XSLT on?)
But still, I'd love to receive insights from others and see if this is replicable.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Closing since it doesn't seem related to this repo.


compile_commands.json doesn't update when config changed

Xmake Version
2.8.6
Operating System Version and Architecture
Windows 11 Pro 23H2 22631.3155
Describe Bug
xmake f -m release
xmake

See that compile_commands.json has "-DNDEBUG" and remember its last changed timestamp.
xmake clean
xmake f -m debug
xmake

See that compile_commands.json still contain "-DNDEBUG" and its last changed timestamp still the same.
Expected Behavior
compile_commands.json should be updated on the next build when xmake configuration was changed to reflect current built config.
Project Configuration
add_rules("mode.debug", "mode.release")
set_languages("c++20")

add_rules("plugin.compile_commands.autoupdate")

add_requires("gtest")
add_packages("gtest")

target("googletest-example")
    set_kind("binary")
    add_files("main.cpp")

Additional Information and Error Logs

rebuild log
build cache stats:
cache directory: build\.build_cache
cache hit rate: 0%
cache hit: 0
cache hit total time: 0.000s
cache miss: 0
cache miss total time: 0.000s
new cached files: 0
remote cache hit: 0
remote new cached files: 0
preprocess failed: 0
compile fallback count: 0
compile total time: 0.000s

[100%]: build ok, spent 0.016s

try it again. xmake update -s dev

Great! Just wonderful!


Bug fixes for GCENodeDriver.ex_create_firewall()

Remove incorrect firewall direction check for allow/deny rules.
Only default source_ranges to <IP_ADDRESS>/0 if direction is INGRESS.
Bug fixes for GCENodeDriver.ex_create_firewall()
Description
Fixed two small bugs in GCENodeDriver.ex_create_firewall().
The first issue was that a direction check was being applied to determine whether allowed or denied would be passed to the GCE API call. Both ingress and egress rules can have allowed or denied blocks so this shouldn't be conditional on the direction.
The second issue was that source_ranges was automatically set to [<IP_ADDRESS>/0] if no other source filters were applied. Since source filters are invalid for egress rules, I added a check for the INGRESS direction prior to setting a default source range.
Status
done, ready for review
Checklist (tick everything that applies)

[ ] Code linting (required, can be done after the PR checks)
[ ] Documentation
[ ] Tests
[ ] ICLA (required for bigger changes)

Codecov Report

Merging #1245 into trunk will increase coverage by <.01%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##            trunk    #1245      +/-   ##
==========================================
+ Coverage   86.27%   86.27%   +<.01%
==========================================
  Files         351      351
  Lines       67973    67973
  Branches     5987     5987
==========================================
+ Hits        58641    58644       +3
+ Misses       6888     6886       -2
+ Partials     2444     2443       -1

Impacted Files
Coverage Δ

libcloud/compute/drivers/gce.py
76.63% <66.66%> (-0.04%)
:arrow_down:

libcloud/test/compute/test_ec2.py
97.88% <0%> (+0.16%)
:arrow_up:

libcloud/test/compute/test_upcloud.py
91.39% <0%> (+1.32%)
:arrow_up:

Continue to review full report at Codecov.

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

Is this still relevant?
If it is, can you please add corresponding test cases?

Thanks for your contribution.
Since this PR hasn't been updated for a long time, I will close it.
If the change still relevant please feel free to re-open this PR (and in that case, please also sync it with the latest trunk).


Proposal: Dagger 2.0

For a while now, we have floated ideas about full-graph generation and how we might evolve Dagger from half static, half runtime to a fully static, fully generated, fully awesome DI solution.  In working with Dagger in a large application on the server side we ran into some particular pain points that shaped how we might want a static solution to look.  So, I present a proposal for Dagger 2.0 that hopefully bridges the gaps between what's suitable on Android and what makes sense for very large applications, with ridiculous numbers of bindings.
https://docs.google.com/document/d/1fwg-NsMKYtYxeEWe82rISIHjNrtdqonfiHgp8-PQ7m8/edit?usp=sharing
Everybody should be able to comment.  Feedback on this bug and in the doc are both welcome.
I have also developed a prototype for this solution.  It works pretty well, but the code is still slightly embarrassing.  I will try to get that cleaned up and into a github repository shortly.  I'll update the bug when that happens.

This happened.


Support for VK_EXT_graphics_pipeline_library

Please support the graphics pipeline library EXT. This extension is seeing significant use in layering efforts, mainly in DXVK.
There's a bunch of assertion failure, due to (non-exhaustive list):

empty pSetLayouts elements in VkPipelineLayoutCreateInfo
ShaderModule replaced by VkShaderModuleCreateInfo in pNext of VkPipelineShaderStageCreateInfo
empty pipelineLayout in VkGraphicsPipelineCreateInfo

This can be easily reproduced by running dEQP-VK.pipeline.pipeline_library.* in VKCTS with layer attached, on a supported driver (Nvidia is the easiest one but you can also try lavapipe).

labeling P1 because of DXVK

@ishitatsuyuki Can you still replicate any issues you were having with graphics pipeline library?  Could you test current dev to see if it resolves your capture issues?

Tested Overwatch 2 and it doesn't launch with the layer attached, #1120 may be related. CTS looks fine however.

Actually dEQP-VK.pipeline.pipeline_library.graphics_library.misc.non_graphics.shader_module_info_comp seems to fail, might be a good repro.


Possible browser issue?

This is the code that I have put in.  When I test the link myself, it does everything the challenge mentions.  I am just unable to complete this challenge.  Am I missing something?
Challenge adjust-the-hover-state-of-an-anchor-tag has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

CatPhotoApp

@jecmstudios That challenge is explicitly looking for the hover color to be blue. Try without using a Hexcode.
This doesn't seem to be a browser issue so it will most likely get closed out. When you get stuck you can always click the Help button on each challenge and/or join the full chat app at Gitter for help questions

I think we should add the test case to check even hex code or even rgba code

This test is looking for color:blue; and nothing else. If you open it up to hex code and/or rgba code you will have to define all the possibilities of blue. That wouldn't work so well because you would have to figure out where to draw the line between what's accepted as blue and what's not. I think this is a pretty simple test and you are guided to just using the word blue.

If we write just blue as the color code test is not completed,  I found out that for the black color the test checks for RGB value for the black

Hmm then this may be a browser bug, because I'm not seeing the same results:

The test case check for RGB value $('a').css('color') == 'rgb(0, 0, 0)' may be this would be causing the issue.

And if we want to anyone to write color in value then we should change color #000 to black which would make more sense then.

@jecmstudios thanks for reporting.
@donofriov @BhaveshSGupta this is a duplicate of https://github.com/freeCodeCamp/freeCodeCamp/issues/13468
Closing, please feel free to continue on the other thread.

@raisedadead This issue is because of #10749, You can see the test case it uses jQuery's CSS method


UnderlineNav throws error when screen resizes to a point where no items can render

Description
https://user-images.githubusercontent.com/3901764/211390398-e7cd542e-0e68-4b8e-8359-e8b0fd93c2bf.mov
Video showing the browser being resized to a smaller size. As it gets smaller, items from the underline nav disappear from the list of items. When the space is small enough that no items can be rendered, an error is thrown
Steps to reproduce

Visit the storybook for UnderlineNav: https://primer.style/react/storybook/iframe.html?args=&id=drafts-components-underlinenav-features--overflow-template&viewMode=story
Make the window smaller until the error shows up

Version
v35.16.0
Browser
No response

FYI @broccolinisoup!

Oh thanks for reporting this @joshblack! I'll see what is going on 👀


Sqlite "RETURNING" clause not supported by most Android "native" SQLite versions

Hi,
Electric (react-native ?) client's code makes use of the "RETURNING" clause in the SQLite queries.
This "clause" is supported in SQLite 3.35.0 and above (https://www.sqlite.org/lang_returning.html) the issue is that these versions of SQLite are not supported before API 34 of Android (https://developer.android.com/reference/android/database/sqlite/package-summary) which means less that 5% of the Android market share (https://gs.statcounter.com/os-version-market-share/android).
Issue Reproduction
As an example, testing the provided react-native application with a Pixel 5 (API 33) fails at startup with a
Object {
  "code": 0,
  "message": "near \"RETURNING\": syntax error (code 1 SQLITE_ERROR): , while compiling: UPDATE main._electric_oplog SET timestamp = ?
      WHERE rowid in (
        SELECT rowid FROM main._electric_oplog
            WHERE timestamp is NULL
        ORDER BY rowid ASC
        )
      RETURNING *",
}

this issue is not encountered when :

running this query without the RETURNING clause
running on a Pixel 6 (API 34).

Possible fixes
I guess there are 2 ways of fixing this problem :

remove the RETURNING clauses
use a SQLite "driver" lib that ships with its own SQLite (in a supported version), like

https://github.com/margelo/react-native-quick-sqlite
https://github.com/OP-Engineering/op-sqlite

Do I miss something or are there something planned about this ? (I'd happily make a new adapter for react-native-quick-sqlite)
Best,
Matthieu

Hi @err0r500,
We are aware of this problem in React Native.
The same problem has been reported here: https://github.com/electric-sql/electric/issues/473
A workaround was introduced in expo, cf. https://github.com/expo/expo/issues/24283
Unfortunately, there's not a similar workaround for React Native because the underlying driver we use is no longer being maintained. So we will need to add support for newer drivers that work with React Native.


Add legal actions with one player to python binding

Current legal_actions(self) is pass through the python binding but not legal_actions(self, player). Therefore when trying to convert a simultaneous python game to turn based python game, the state has no legal actions. Convert_to_turn_based function does not seems to have been tested in python, which may explain this issue. For 'more' explanations, please refer to https://github.com/TheoCabannes/open_spiel/issues/6

Yes, this hasn't been tested with Python games - sorry about that!
I'll do it now and report back.

This has been done by https://github.com/deepmind/open_spiel/commit/7b659bbdf54e63da4eea98feebee3ec64f7c449e. Thanks @elkhrt


Possible ways of aligning depth and RGB streams after being sent through WiFi

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
I have a D435i connected to an on board computer(intel NUC) and I am able to send RGB and depth streams over WiFi but they have different latencies when arriving the host computer. I know the rs-align example demonstrates aligning depth and rgb streams but the two streams are no longer realsense objects at the host computer.  Is there maybe a constructor that converts the realtime streams  to some realsense objects so i can utilize the rs-align example to align both streams?

There was a case in the past week of a RealSense user whose project needed real-time communication between an Android tablet and a desktop computer.  I suggested a remote computer control application to take full control of the target computer wirelessly from the tablet screen.
https://github.com/IntelRealSense/librealsense/issues/5971#issuecomment-596127497
The Chrome Remote Desktop application I suggested can also use a computer to take control of another computer.  So I wonder if the host computer could run the Remote Desktop application and take real-time control of the NUC by an operator sitting at the desktop.  That way, you would be accessing the original unconverted camera data instead of sending the streams over wi fi and breaking the alignment.

Thank you @MartyG-RealSense for your response. I think my intended system setup is a bit different from that user. My project is to send depth and rgb streams wirelessly from the client side (D435i + intel NUC) to the host computer and display it in Unity 3D (installed on host computer). Thus, I am not sure how taking control over the target computer would help me in this case.
The reason I asked this question is because I think there should be a way of doing it. In the end, all these data streams and frames are just bytes. I am guessing that the reason rs-align works is because there are timestamps encoded in each frame of both rgb and depth streams and under the hood, rs-align just matching those timestamps. If that's the case, aligning both streams on the host computer should be straightforward.

Intel did a multicam demonstration in January 2018 where they captured a raw point cloud with a camera on one computer and sent the data to another computer along a network for final processing with the help of Unity.
https://www.intelrealsense.com/intel-realsense-volumetric-capture/
So it should certainly be possible to capture raw data and send it to another computer and do the alignment in Unity at the destination instead of before transmission.  Is it a project requirement please that alignment is done at the host end and not at the destination?
RealSense transmits large volumes of data at higher resolutions, which is why cabled transmission is usually more practical than Wi-Fi.

Alignment can happen anywhere and for my purpose,  it makes more sense that the alignment happens after transmission. Is this project you mentioned about open source? If so, do you mind including the link of it? Thank you very much for your help

Intel didn't make that particular volumetric capture project open source.  They did later present a long seminar on similar techniques though and publish it on YouTube.
https://www.youtube.com/watch?v=VSHDyUXSNqY
It is probably going to be easier to go with your planned approach of one camera, one wi-fi connection.   Though wired connections are best, I have previously looked at RealSense wi-fi setups that use flight drone tech to accomplish the streaming.
https://github.com/IntelRealSense/librealsense/issues/4819#issuecomment-529093164

This case will be closed after 7 days from the date of writing this if there are no further responses.   Thanks!


Can't save all files

When trying to save all files :wa I get:
E141: No file name for buffer 37

From session to session the buffer number changes, and if I check what's in this buffer, it is the message generated for floating windows such as errors or warnings.
It doesn't happen consistently, only time to time, not sure how could I provide more information in order to tackle down the issue.

You have unnamed buffer with changes, it's your issue.

It should be nofile filetype which doesn't affect file save, can't reproduce your issue.

Well... it is not. Any idea what could be causing it?

No, you should try minimal vimrc


Deleting symbols first

A note is written telling: "If you experience problems with generating symbols, try to delete existing symbols first."
My question is how to do this?

Documentdetails
⚠ Dit gedeelte niet bewerken. Het is vereist om problemen te koppelen tussen docs.microsoft.com en GitHub.

ID: 156ead7e-1ab8-d9fb-44ca-e9029435e5a6
Version Independent ID: 5a6899ff-c91d-d2ab-b3d6-0153808a106f
Content: Running C/SIDE and AL Side-by-Side - Business Central
Content Source: dev-itpro/developer/devenv-running-cside-and-al-side-by-side.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Hello, thanks for the feedback!
The note either needs to be expanded with some more information or removed to avoid confusion.
How to delete the symbols would depend on where you are experiencing the problem.
If you experience issues with generating symbols, the log file from the finsql.exe provides some insight into what went wrong. Symbol references are stored in the Object Metadata table of the database. If you need to delete the symbol references, then you use a query on the table.
If the issue involves downloading and publishing symbols in Visual Studio code, then symbols can be deleted from the alpackages folder in the project.
I hope this help, otherwise let us know.
Thanks


Whack-the-mole certificate title incorrect?

For Whack-the-mole, there's a discrepancy between the name of the activity and the name on the certificate. The certificate says "Whack-em-all".

Additionally, I found an instance of "Whack-a-Mole", here:

Certificate is fixed…I should probably add a trademark symbol on the brand name. Good catch!


Update Readme.md with examples in .tsx and .mdx files

Could you update Readme.md with code examples showing how to use translations in .tsx and .mdx files?
Thanks.

This addon empowers you to use your already existing react-i18next setup in your project inside of storybook.
You’ll need to read the react-i18next documentation for instructions on how to use it.
https://react.i18next.com/


Move nonLinear recall button when controls are active so nonLinears can be recalled

Branching off of #22.
Currently when a nonLinear is closed, the recall button renders under the video controls. You can click it if you're quick, but I'd like to be able to bump it up on the y plane when controls are activated so you can click on it more easily.

I confirm. I think the problem is in ima3.js.
same issue if I set nonLinearWidth and nonLinearHeight

We should expand this to move the entire ad up when controls are shown, if possible.

maybe just a vertical offset given as option during ad init can do it.

any update on this issue?

I'm currently working on this - unfortunately it's not trivial. There are so many elements layered on top of one another that the mouse events (mouseover, mouseout) are not acting as they should. I'd really like to avoid having the ad 30 pixels above the bottom of the video when the controls aren't showing because it looks awful. If you'd like that option then it's easy to add on your own - just change the "top" style on the adDisplayContainer div from 0px to -30px (video.js controls in the current version are 30px high, but you may need to change this for future releases).

I've opened https://github.com/videojs/video.js/issues/1879 in an attempt to remedy the above issue.


need parameter

instead of date table with m in powerquery he miss parameter (from date To date)

You should modify the parameters at the beginning of the script - you can write dynamic code reading from other tables in the data model. A request for parameters in an interactive way (similar to Power Query) is not possible in DAX.


Race in [UTMQemuManager connect] and QAPI calls

Before the connection is successful, it is possible for a QAPI action to be called (and fail). Currently, this will trigger the error handler, but ideally we make it more user friendly by disabling UI options until QMP is connected.

Reimplemented connection code should be more robust against this.


Request: Sync rev of sync_with_poetry and poetry

First of all, great hook!
The idea behind the hook is to have a single source of truth for all our packages, right? The rev of all my pre-commit hooks are managed in the pyproject.toml except for sync_with_poetry and poetry itself.  I have to adjust my .pre-commit-config.yaml manually, for these two.
Perhaps, sync of these two special cases can be covered by sync_with_poetry as well?

I have tried using a custom db.json which includes:
    "poetry": {
        "repo": "https://github.com/python-poetry/poetry",
        "rev": "${rev}"
    },

It did not work.

Relevant part of .pre-commit-config.yaml:
- repo: https://github.com/floatingpurr/sync_with_poetry
    rev: 0.4.0
    hooks:
      - id: sync_with_poetry
        args: [--db, .sync-with-poetry-db.json]
- repo: https://github.com/python-poetry/poetry
    rev: 1.2.0  # sync_with_poetry fails
    hooks:
      - id: poetry-check
      - id: poetry-lock
      - id: poetry-export
        args: ["-f", "requirements.txt", "-o", "requirements.txt"]

First of all, great hook!
The idea behind the hook is to have a single source of truth for all our packages, right?

Thank you! Yes, you are right. That's exactly the reason why this hook exists.
Regarding your question, I can't replicate what you are describing. Here's my test:
starting .pre-commit-config.yaml
minimum_pre_commit_version: 2.15.0
repos:
  - repo: https://github.com/floatingpurr/sync_with_poetry
    rev: 0.4.0
    hooks:
      - id: sync_with_poetry
        args: [--db, .sync-with-poetry-db.json]
  - repo: https://github.com/python-poetry/poetry
    rev: b6d4b0adafb45204aadace143f73a789c1c0e33e # sync_with_poetry works!
    hooks:
      - id: poetry-check
      - id: poetry-lock
      - id: poetry-export
        args: ["-f", "requirements.txt", "-o", "requirements.txt"]

snippet of pyproject.toml
[tool.poetry.dependencies]
python = "^3.10"

[tool.poetry.group.dev.dependencies]
poetry = "^1.2.0"
pre-commit = "^2.20.0"

[build-system]
requires = ["poetry-core"]
build-backend = "poetry.core.masonry.api"

Resulting .pre-commit-config.yaml after sync_with_poetry
minimum_pre_commit_version: 2.15.0
repos:
  - repo: https://github.com/floatingpurr/sync_with_poetry
    rev: 0.4.0
    hooks:
      - id: sync_with_poetry
        args: [--db, .sync-with-poetry-db.json]
  - repo: https://github.com/python-poetry/poetry
    rev: 1.2.0 # sync_with_poetry worked!
    hooks:
      - id: poetry-check
      - id: poetry-lock
      - id: poetry-export
        args: ["-f", "requirements.txt", "-o", "requirements.txt"]

Did you put Poetry in your dev dependencies?

I am using sync_with_poetry as a hook. All hooks are by default CLI tools and except sync_with_poetry they are all installable. Thus I can put them in my pyproject.toml and control their version there. Currently, I have to maintain only the version sync_with_poetry manually or with perhaps with pre-commit update; rest of the hook versions are handled with sync_with_poetry.
I think it would be nice to have sync_with_poetry in pyproject.toml as well.

Oh, I see. Currently, sync_with_poetry is not on PyPI. You just need to install from this repo. E.g.,:
$ poetry add git+https://github.com/floatingpurr/sync_with_poetry

and you'll get in pyproject.toml the following entry:
sync-with-poetry = {git = "https://github.com/floatingpurr/sync_with_poetry"}

and in poetry.lock:
[[package]]
name = "sync-with-poetry"
version = "0.4.0"
description = ""
category = "main"
optional = false
python-versions = "^3.6.1"
develop = true

then you just need to track it in the db file:
{
    "sync-with-poetry": {
        "repo": "https://github.com/floatingpurr/sync_with_poetry",
        "rev": "${rev}"
    }
}

and you are all set.
But pay attention! This is not the recommended way to interact with this piece of code. Having a local version of sync_with_poetry is not necessary and it can mess up your Python dependencies without a valid reason.
Hope this helps.

Thanks, that sounds legit.

But pay attention! This is not the recommended way to interact with this piece of code. Having a local version of sync_with_poetry is not necessary and it can mess up your Python dependencies without a valid reason.

I believe since pre-commit maintains its own versions that should be fine.


Fix slow running tests when using Sqlite3.

closes #6355
Updated testing config to set a few settings on sqlite.
If you already have a config.js file you will have to add the pool config option to that file as well to test out the faster tests.

If you already have a config.js file you will have to add the pool config option to that file as well to test out the faster tests.

Did you test that and find it to be true, or is it an assumption based on how Ghost works? I'm just checking because afaik the tests always use config.example.js as it can be relied on, whereas config.js cannot. Sooo this change should fix it regardless :wink:

@ErisDS I had not fully looked at it. I did notice it reads both and notice the Knex intense was cached after the first read. Maybe we should make this easier to understand and make a different test only config which tests only read from as I thought maybe this was a bug but good to know it was just a test thing.

Ok cool.
With regard to the changes, do you have a link to some documentation on the pool & sqlite3 settings? I couldn't find them with a quick google and it might be good to have them in the commit message in case we bump into this in future. Particularly as the version of knex we are on currently is quite old, I'd like to get it upgraded soon and I know that the newer versions use a completely different dependency for handling pooling.

There was a pull request about setting default connections stuff with Postgres that was how I found out about the option. It is a pat of Knex not the pooling library so I think the newer versions of Knex support it as well. Here are the links:
https://github.com/tgriesser/knex/issues/14
The thing is you are getting the raw sqlite object not a knex object so that is why I use run instead of raw. I could not find this setting in any of the docs but I might pull it down and submit a PR adding it as it is a great feature.

Thanks for the extra info :+1:


Added new available Amazon regions

Added the Amazon regions created until January 2023 that are not included in the AWSRegion type.
Motivation:
Although there is a way to work with non existing regions at AWSRegion class, It could be helpful add the new available Amazon regions to the mentioned class.
The regions added in this PR are listed below.

Africa

Cape City (af-south-1)

Asia Pacific

Melbourne (ap-southeast-4)
Hyderabad (ap-south-2)
Yakarta (ap-southeast-3)

Europe

Milano, Italy (eu-south-1)
Madrid🤘😜, Spain  (eu-south-2)
Zurich, Switzerland (eu-central-2)

Middle East

UAE(me-central-1)

Modifications:
This PR presents only modification in the AWSRegion class, specifically in the all static variable in order to add the new static variables created for each of the new regions
Those new variables have been added following the current ordering criteria.
Result:
The all static var looks like that
static var all: [AWSRegion] = [
    Self.af_south_1,
    Self.ap_northeast_1,
    Self.ap_northeast_2,
    Self.ap_east_1,
    Self.ap_southeast_1,
    Self.ap_southeast_2,
    Self.ap_southeast_3,
    Self.ap_southeast_4,
    Self.ap_south_1,
    Self.ap_south_2,
    Self.cn_north_1,
    Self.cn_northwest_1,
    Self.eu_north_1,
    Self.eu_south_1,
    Self.eu_south_2,
    Self.eu_west_1,
    Self.eu_west_2,
    Self.eu_west_3,
    Self.eu_central_1,
    Self.eu_central_2,
    Self.us_east_1,
    Self.us_east_2,
    Self.us_west_1,
    Self.us_west_2,
    Self.us_gov_east_1,
    Self.us_gov_west_1,
    Self.ca_central_1,
    Self.sa_east_1,
    Self.me_central_1,
    Self.me_south_1,
]

And the static variables for those new regions...
public static var af_south_1: Self { AWSRegion(rawValue: "af-south-1")! }

public static var ap_northeast_1: Self { AWSRegion(rawValue: "ap-northeast-1")! }
public static var ap_northeast_2: Self { AWSRegion(rawValue: "ap-northeast-2")! }
public static var ap_east_1: Self { AWSRegion(rawValue: "ap-east-1")! }
public static var ap_southeast_1: Self { AWSRegion(rawValue: "ap-southeast-1")! }
public static var ap_southeast_2: Self { AWSRegion(rawValue: "ap-southeast-2")! }
public static var ap_southeast_3: Self { AWSRegion(rawValue: "ap-southeast-3")! }
public static var ap_southeast_4: Self { AWSRegion(rawValue: "ap-southeast-4")! }
public static var ap_south_1: Self { AWSRegion(rawValue: "ap-south-1")! }
public static var ap_south_2: Self { AWSRegion(rawValue: "ap-south-2")! }

public static var cn_north_1: Self { AWSRegion(rawValue: "cn-north-1")! }
public static var cn_northwest_1: Self { AWSRegion(rawValue: "cn-northwest-1")! }

public static var eu_north_1: Self { AWSRegion(rawValue: "eu-north-1")! }
public static var eu_south_1: Self { AWSRegion(rawValue: "eu-south-1")! }
public static var eu_south_2: Self { AWSRegion(rawValue: "eu-south-2")! }
public static var eu_west_1: Self { AWSRegion(rawValue: "eu-west-1")! }
public static var eu_west_2: Self { AWSRegion(rawValue: "eu-west-2")! }
public static var eu_west_3: Self { AWSRegion(rawValue: "eu-west-3")! }
public static var eu_central_1: Self { AWSRegion(rawValue: "eu-central-1")! }
public static var eu_central_2: Self { AWSRegion(rawValue: "eu-central-2")! }

public static var us_east_1: Self { AWSRegion(rawValue: "us-east-1")! }
public static var us_east_2: Self { AWSRegion(rawValue: "us-east-2")! }
public static var us_west_1: Self { AWSRegion(rawValue: "us-west-1")! }
public static var us_west_2: Self { AWSRegion(rawValue: "us-west-2")! }
public static var us_gov_east_1: Self { AWSRegion(rawValue: "us-gov-east-1")! }
public static var us_gov_west_1: Self { AWSRegion(rawValue: "us-gov-west-1")! }

public static var ca_central_1: Self { AWSRegion(rawValue: "ca-central-1")! }
public static var sa_east_1: Self { AWSRegion(rawValue: "sa-east-1")! }
public static var me_central_1: Self { AWSRegion(rawValue: "me-central-1")! }
public static var me_south_1: Self { AWSRegion(rawValue: "me-south-1")! }

The availability of AWS Lambda services in those new regions have been checked with the AWS Regional Services webpage.

@swift-server-bot test this please


Expand the bar to fullscreen / Increase height and width

I tried to do it , but I am dumb as a brick.

There's no such option.

Ok


More Themes 🎨

Currently Bruno only supports light and dark themes. These colors were inspired by the default vscode light and dark themes.
I propose to support these additional 6 themes to be supported by default

Community Material Light
Community Material Dark
Atom Light
Atom Dark
Community Palenight
Community Nord

If someone wan't to pick this up, updating this in bruno should be straightforward

You need to add a file here
You need to update the options here

After we reach v1 (auth support, enhanced collection runner, load testing, hooks, global env and so on) milestone, then we can think of a implementing a theme store architecture which allows to browse and install themes from github repo's

Hello @helloanoop. Can I take up this issue?

@soumilroy Go ahead. Pick one theme first.

@helloanoop The ipcRenderer is returning undefined. I'm new to electron so I'm not sure if this issue specific to the platform. I followed all instructions, dug online for solutions but didn't help. Bug?
 Bruno won't load inside a browser.
# run electron app (terminal 2)
npm run dev:electron

This will open an bruno in a separate window.
If you are still facing issues, please post in our discord dev channel

i can tackle this issue. please assign it to me!

Hey @sanjai0py This is being planned as a part of golden edition [paid plan](https://www.usebruno.com/pricing
Hence we are not able to accept contributions on this feature.
Thanks for your understanding.


there is no security process for receiving security critical bug reports

When someone finds a vulnerability, confidential info must be made available to a small selection of core developers who can handle the information responsibly.
There is usually a need for back-and-forth discussion between the reporter and the developers.
Oftentimes, there is a "responsible disclosure" deadline set by the reporter, at which point information about the vulnerability would be publically disclosed.
Finally, when a bugfix is available, the public should be informed by the rakudo devs with exact information about versions that are vulnerable, what patches exist, what mitigations are available, etc.
This ticket shall be used for discussion to figure out what kind of technology we would set up to get this to work, and what the finer details should be.
Please feel free to look at other projects to see how they are handling stuff, and what other teams have done wrong in the past.

OK, while we're figuring it out, here's a temporary solution.
If you found a security issue in Rakudo/NQP/MoarVM or any other important project (e.g. Cro), contact jnthn by email:

@jnthn –<EMAIL_ADDRESS>
If you want to make sure that your report is end-to-end encrypted, you can mail me:

@AlexDaniel –<EMAIL_ADDRESS>(you can find my pgp key here, as well as in many other places)

Historically, we had a mail alias setup in a fashion similar to Perl 5's, but I am pretty sure it was on RT, and only received a few test mails.

There was a short discussion about this on IRC. Maybe we should try https://hackerone.com/


Support custom GridTools source folders in the gt_src_manager

Move the repo configuration to entries in the config module instead of harcoded values. Since the code generation is customized to a specific GridTools version, the source manager should still check that the provided repo has a compatible version (check with @havogt which strategy should be used for this check: same major/minor? same major and equal or higher minor? ...)
Also add basic tests to test this component.

This is supported, currently user responsibility to point to appropriate versions.


test(swc/wasm): add test cases for minify

Description:

relates to https://github.com/swc-project/swc/issues/5275

PR adds simple test cases for the wasm, as we are moving towards to make @swc/wasm as first class citizen. This PR actually fails since newly added test case not able to pass.

BREAKING CHANGE:

Related issue (if exists):

Interesting thing is it shows different failure than original issue.
   RangeError: Maximum call stack size exceeded

      at swc_ecma_parser::lexer::state::<impl core::iter::traits::iterator::Iterator for swc_ecma_parser::lexer::Lexer<I>>::next::{{closure}}::h33d6f7aea7ae9423 (wasm:/wasm/17487d42:1:9174010)
      at swc_ecma_parser::lexer::state::<impl core::iter::traits::iterator::Iterator for swc_ecma_parser::lexer::Lexer<I>>::next::h07de51b652b79274 (wasm:/wasm/17487d42:1:18208348)
      at swc_ecma_parser::parser::input::Buffer<I>::bump_inner::{{closure}}::h947975deb7378d5c (wasm:/wasm/17487d42:1:67380653)
      at core::option::Option<T>::or_else::hb735b09add21ca84 (wasm:/wasm/17487d42:1:30757873)
      at swc_ecma_parser::parser::input::Buffer<I>::bump_inner::h4372a3cb4430fd0b (wasm:/wasm/17487d42:1:28925442)
      at swc_ecma_parser::parser::input::Buffer<I>::cur::h39d003bfad042fa1 (wasm:/wasm/17487d42:1:52676026)
      at swc_ecma_parser::parser::expr::<impl swc_ecma_parser::parser::Parser<I>>::parse_subscript::h6df51f4270d7254d (wasm:/wasm/17487d42:1:1043749)
      at swc_ecma_parser::parser::expr::<impl swc_ecma_parser::parser::Parser<I>>::parse_subscripts::h79f99ead3e8e0bd6 (wasm:/wasm/17487d42:1:35258862)

Oh?
Interesting..

I don't have good explanations. Fyi, @swc/core works fine.

@kdy1 one more: weirdly, this seems only happening on the debug build. 🤔

Closing as stale


Update telemetry data format

Simplify event names.

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Command Guidelines

[ ] Each command and parameter has a meaningful description.
[ ] Each new command has a test.

(see Authoring Command Modules)

View a preview at https://prompt.ws/r/Azure/azure-cli/5605
This is an experimental preview for @microsoft users.


Enhancement: More flexible bootstrap

This will allow apart of 'converter injection' to initialize plugin this way:
new CssInlinerPlugin([
	'cleanup' => true,
	'useInlineStylesBlock' => true,
	'stripOriginalStyleTags' => true,
]);

which is equivalent of this:
$converter = new CssToInlineStyles();
$converter->setCleanup(true);
$converter->setUseInlineStylesBlock(true);
$converter->setStripOriginalStyleTags(true);

new CssInlinerPlugin($converter);

Actually I suggest it for compatibility with Yii2 framework. With this change it's possible to enable CssInlinerPlugin in this easy way in Yii2 config file:
return [
    'components' => [

        [...]

        /**
         * Mailer
         *
         * @link https://github.com/yiisoft/yii2-swiftmailer
         * @link http://swiftmailer.org/docs/messages.html
         */
        'mail' => [
            'class' => 'yii\swiftmailer\Mailer',
            'messageConfig' => [
                'charset' => 'utf-8',
            ],
            'transport' => [
                'class' => 'Swift_SmtpTransport',
                'host' => 'mail.domain.com',
                'username' =><EMAIL_ADDRESS>                'password' => '...',
                'plugins' => [
                    [
                        // connect CssInlinerPlugin
                        'class' => 'Openbuildings\Swiftmailer\CssInlinerPlugin',
                        // pass arguments to CssInlinerPlugin's constructor
                        'constructArgs' => [[
                            'cleanup' => true,
                            'useInlineStylesBlock' => true,
                            'stripOriginalStyleTags' => true,
                        ]],
                    ],
                ],
            ],
        ],
    ],
];

Existing implementation when CssToInlineStyles object is injected is already perfect.
I believe this change will be good addition that allow to use plugin in more flexible way (maybe other frameworks).
Thanks!

@armab Hello again! Do you think you would be able to rebase this to fix the conflicts and possibly squash the commits into one? A section in the readme for Yii configuration would be really nice as well. Thank you!

@hkdobrev, nope.
Not working with PHP for 1+ year already and don't have needed environment like Yii2 to test this PR/make sure it's still actual.

@armab Thanks and sorry for the late response! I can prepare a new PR which squashes all this and it's still attributed to you.
@ivank Do you think we still need that?


"Required module not found" when main is pointing to a folder and not explicitly to index.js

If package.json is pointing to a folder: "main": "./lib" flow is not able to open the module.     If I replace it with "./lib/index.js" it works fine.
Passport is one example of popular node module where this is an issue

I think this is the same issue as #278

Thx, i couldn't find it.  Closing this one


Add additional files to neutron-sanity-check

neutron-sanity-check expects to be passed all the --config-file/--config-dir
options the Neutron services receive.
Unfortunately, when we pass in a file/directory that doesn't exist, the script
crashes. So we're building the list of existing files/dirs in a shell script
and pass only the ones they're present.
We could do the same in Ansible, but that would generate a lot of duplicate
output and be more confusing to the users.

So, I've looked at how neutron-sanity-check works, and it appears it enables checks based on what it finds in the config files. The thing is, we cannot blindly pass all the configuration files and directories we find on the machine, but instead we should only pass the ones that will appear when calling the neutron binaries. One more thing to consider: the order of --config-file and --config-dir options is important.
I'm not exactly sure what is the best way to get the list of options passed to neutron binaries.

Hm, right. So the authoritative place for these is the systemd unit files. Although in HA setup, we're managing some services via pacemaker and I'm not sure if Neutron is one of them.
We could try to read this list from the unit files (that's how I built this list manually), but that would be platform-specific (TripleO only supports only RHEL and CentOS so we should be fine for now). A lot of these config files and directories passed to neutron don't actually exist which is why I had to add the shell script.
What I tried to do here is to have a default list that can be overridden, but I get your point.
So, do you have any ideas on how to proceed with this?


Issue/5919 JUnit_Coverage

JUnit Coverage
Results:
[INFO]
[INFO] Tests run: 25, Failures: 0, Errors: 0, Skipped: 0
[INFO]
[INFO]
[INFO] --- jacoco-maven-plugin:0.8.4:report (post-unit-test) @ viptela-sdwan-driver ---
[INFO] Loading execution data file C:\project\viptela-sdwan-driver\target\test-results\coverage\jacoco\jacoco.exec
[INFO] Analyzed bundle 'viptela-sdwan-driver' with 17 classes
[INFO]
[INFO] --- maven-jar-plugin:3.2.2:jar (default-jar) @ viptela-sdwan-driver ---
[INFO] Building jar: C:\project\viptela-sdwan-driver\target\viptela-sdwan-driver-1.0.1-SNAPSHOT.jar
[INFO]
[INFO] --- spring-boot-maven-plugin:2.6.8:repackage (repackage) @ viptela-sdwan-driver ---
[INFO] Replacing main artifact with repackaged archive
[INFO]
[INFO] --- sortpom-maven-plugin:2.8.0:sort (default) @ viptela-sdwan-driver ---
[INFO] Sorting file C:\project\viptela-sdwan-driver\pom.xml
[INFO] Pom file is already sorted, exiting
[INFO]
[INFO] --- maven-install-plugin:2.5.2:install (default-install) @ viptela-sdwan-driver ---
[INFO] Installing C:\project\viptela-sdwan-driver\target\viptela-sdwan-driver-1.0.1-SNAPSHOT.jar to C:\Users\gsiddala.m2\repository\com\ibm\viptela-sdwan-driver\1.0.1-SNAPSHOT\viptela-sdwan-driver-1.0.1-SNAPSHOT.jar
[INFO] Installing C:\project\viptela-sdwan-driver\pom.xml to C:\Users\gsiddala.m2\repository\com\ibm\viptela-sdwan-driver\1.0.1-SNAPSHOT\viptela-sdwan-driver-1.0.1-SNAPSHOT.pom
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  39.788 s
[INFO] Finished at: 2022-09-22T15:55:50+05:30
[INFO] ------------------------------------------------------------------------

Tests run: 35, Failures: 0, Errors: 0, Skipped: 0
[INFO]
[INFO]
[INFO] --- jacoco-maven-plugin:0.8.4:report (post-unit-test) @ viptela-sdwan-driver ---
[INFO] Loading execution data file C:\project\viptela-sdwan-driver\target\test-results\coverage\jacoco\jacoco.exec
[INFO] Analyzed bundle 'viptela-sdwan-driver' with 13 classes
[INFO]
[INFO] --- maven-jar-plugin:3.2.2:jar (default-jar) @ viptela-sdwan-driver ---
[INFO] Building jar: C:\project\viptela-sdwan-driver\target\viptela-sdwan-driver-1.0.1-SNAPSHOT.jar
[INFO]
[INFO] --- spring-boot-maven-plugin:2.6.8:repackage (repackage) @ viptela-sdwan-driver ---
[INFO] Replacing main artifact with repackaged archive
[INFO]
[INFO] --- sortpom-maven-plugin:2.8.0:sort (default) @ viptela-sdwan-driver ---
[INFO] Sorting file C:\project\viptela-sdwan-driver\pom.xml
[INFO] Pom file is already sorted, exiting
[INFO]
[INFO] --- maven-install-plugin:2.5.2:install (default-install) @ viptela-sdwan-driver ---
[INFO] Installing C:\project\viptela-sdwan-driver\target\viptela-sdwan-driver-1.0.1-SNAPSHOT.jar to C:\Users\gsiddala.m2\repository\com\ibm\viptela-sdwan-driver\1.0.1-SNAPSHOT\viptela-sdwan-driver-1.0.1-SNAPSHOT.jar
[INFO] Installing C:\project\viptela-sdwan-driver\pom.xml to C:\Users\gsiddala.m2\repository\com\ibm\viptela-sdwan-driver\1.0.1-SNAPSHOT\viptela-sdwan-driver-1.0.1-SNAPSHOT.pom
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  33.538 s
[INFO] Finished at: 2022-09-23T14:12:42+05:30
[INFO] ------------------------------------------------------------------------


[BUG] ListEdgeNodes on CdnProfiles throws an Exception

Describe the bug
I am trying to use the SDK to get the list of IP addresses for CDN edge nodes. I created a service principal using the following Azure CLI command:
az ad sp create-for-rbac --sdk-auth

My code looks like this:
var creds = SdkContext
    .AzureCredentialsFactory
    .FromServicePrincipal(
        "[client-id]",
        "[client-secret]",
        "[tenant-id]",
        AzureEnvironment.AzureGlobalCloud);

var azure = Azure
    .Configure()
    .Authenticate(creds)
    .WithDefaultSubscription();

var edgeNodes = azure.CdnProfiles.ListEdgeNodes();

// ... do other things

When I call this method, I get an ErrorResponseException with a message of Operation returned an invalid status code 'Forbidden'. I can call all the other methods on the CdnProfiles object without any issue. Also, my service principal is a Contributor on the subscription, so should have access.
I noticed that using the following code works:
async Task Main()
{
    var clientId = "[client-id]";
    var clientSecret = "[client-secret]";
    var tenantId = "[tenant-id]";
    var tenantDomainName = "[tenant-domain-name]";
    var subscriptionId = "[subscription-id]";
    var authority = $"https://login.microsoftonline.com/{tenantId}/{tenantDomainName}";

    var authResult = await GetAccessTokenAsync(clientId, clientSecret, authority);

    var cdn = new CdnManagementClient(new TokenCredentials(authResult.AccessToken))
        { SubscriptionId = subscriptionId };

    var edgeNodes = await cdn.EdgeNodes.ListAsync();
}

private static Task<AuthenticationResult> GetAccessTokenAsync(string clientId, string clientSecret, string authority)
{
    var authContext = new AuthenticationContext(authority);
    var credential = new ClientCredential(clientId, clientSecret);
    return authContext.AcquireTokenAsync("https://management.core.windows.net/", credential);
}

I asked a question about this on stackoverflow.
Expected behavior
I should receive a list of edge nodes from the API.
Actual behavior (include Exception or Stack Trace)
What is the actual behavior?
I get an ErrorResponseException with the following message:

{"error":{"code":"AuthorizationFailed","message":"The client '7400aa2d-3213-41d7-a52d-586dd7d46360' with object id '7400aa2d-3213-41d7-a52d-586dd7d46360' does not have authorization to perform action 'Microsoft.Cdn/EdgeNodeInners/read' over scope '/providers/Microsoft.Cdn' or the scope is invalid. If access was recently granted, please refresh your credentials."}} | {"error":{"code":"AuthorizationFailed","message":"The client '7400aa2d-3213-41d7-a52d-586dd7d46360' with object id '7400aa2d-3213-41d7-a52d-586dd7d46360' does not have authorization to perform action 'Microsoft.Cdn/EdgeNodeInners/read' over scope '/providers/Microsoft.Cdn' or the scope is invalid. If access was recently granted, please refresh your credentials."}}
-- | --
{"error":{"code":"AuthorizationFailed","message":"The client '7400aa2d-3213-41d7-a52d-586dd7d46360' with object id '7400aa2d-3213-41d7-a52d-586dd7d46360' does not have authorization to perform action 'Microsoft.Cdn/EdgeNodeInners/read' over scope '/providers/Microsoft.Cdn' or the scope is invalid. If access was recently granted, please refresh your credentials."}}

To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

Run the code above.

Environment:

Name and version of the Library package used: Microsoft.Azure.Management.CDN 4.40 and Microsoft.Azure.Management.Fluent 1.34.0
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): Windows 10 ASP.NET Core 3.1
IDE and version : LINQPad

Tagging and routing to the team best positioned to assist.

@weidongxu-microsoft for help, thanks

@rjregenold Thanks for reporting. This seems a pretty crazy bug.


Do not show height graph again if closed

(for desktop)
reported in #1654 from @easbar

Just a minor issue, but if I close the height graph (say because I want to see more of the map) and then drag one of the route markers it pops up again.

Good catch! We could save the state of the height graph as url parameter or cookie. WDYT?

Better just temporary in some JavaScript object to avoid complexity. Or we just use elevation=false :) ?

Better just temporary in some JavaScript object to avoid complexity.

Sure we can certainly do that as well :+1:

Or we just use elevation=false :) ?

But then we can't easily enable it anymore, unless we change the URL to elevation=true.

BTW: this requires a change in the Leaflet.Heightgraph, I created an issue there: https://github.com/GIScience/Leaflet.Heightgraph/issues/57

I released a new version of Leaflet.Heightraph that offers the option to pass a callback function to remember the expand state.

Thx, will update the version!

But then we can't easily enable it anymore, unless we change the URL to elevation=true

What do you mean here? If I currently load it on a phone I cannot force it to display the widget.

What do you mean here? If I currently load it on a phone I cannot force it to display the widget

If we close the widget and set elevation=false, then no widget will be shown anymore at all. Unless someone understands the url parameters they can't enable the widget anymore.
On a phone the widget is not shown at all in general.

When rethinking about this. The best solution would be to hide the widget per default ("click the close button") on smaller display sizes, so that the user has a chance to see it even on mobile.

Would be nice, yes and somehow storing the state (in the elevation parameter?) so that it stays closed.


Form is submitted when we select an option from dropdown using enter key

I have a ng2-completer component inside my form. When we select an option using mouse click, model gets the selected value.
But if we use enter key to select the value, model does get the correct value but along with it even the form gets submitted.

@supriya2992 can you provide some sample code?

Yes sure. Here is the simple example:

I have not even used a submit button here. Still when I select a country from dropdown, submit(... , ...) function is called.

I tried to write the code but somehow the form tag  is not getting included while posting the comment.
I am attaching a file containing a small code snippet for your reference.
If you see the code, i don;t have a submit button there. I have a submit function on form which is getting called when I select the option from dropdown using enter key
ng2-completer_issue.txt

hmm and for me, the mouce click doesnot seem to have any effect! ( i do have overrideSuggested="true")
here is the code bit
 <ng2-completer class="form-group" [ngModelOptions]="{standalone: true}" [(ngModel)]="restaurantName"
                             [dataService]="dataService" [minSearchLength]="0" inputName="Restaurant name"
                             placeholder="Restaurant name" overrideSuggested="true"></ng2-completer>

@oferh so I did some research into my problem and I realize why it was happening, I personally feel its a bug, but your opinion could differ.
As guessed, it is caused because of the overrideSuggested="true". What I feel should happen even in this case is, if the user selects an item from the dropdown i.e.

dropdown is present AND
item is highlighted AND
overrideSuggested is TRUE

still select the highlighted field. makes no sense if I click on a highlighted row, and the result in the test field is a few characters that I wrote..
But that is my opinion, personally I love what you have done here, and am super thankful for it.

I have this error two :'(

@oferh  any progress on this issue?

@supriya2992 not yet - had no time to work on it in the last couple of weeks

Just in case somebody else has the same problem, this is a workaround for the moment
handle keydowns events like
 <form (keydown)="keyDownFunction($event)">
and then prevent default for enter events:
  keyDownFunction(event) {
    if(event.keyCode == 13) {
      event.preventDefault()
    }
  }

As a part of the fix first press on enter will select the value the 2nd will pass the keypress to the form


Add support simple variant extensions

Related to https://github.com/mirage/repr/issues/4
This allows to write things like:
type 'a s = [ `S of int | `R ] [@@deriving repr]
type my_polyvar = [ s | `T of string ] [@@deriving repr]

This can for instance be useful to help https://github.com/mirage/irmin/pull/1935#discussion_r907488037

Would need a few more tests to see of this works with multiple extensions. There's some List.rev and @ mixes that I am not sure 100% sure.


Implement ResourceAccessorService to allow extension ResourceAccessor implementations DAT-9361

Environment
Liquibase Version:
Liquibase Integration & Version: <Pick one: CLI, maven, gradle, spring boot, servlet, etc.>
Liquibase Extension(s) & Version:
Database Vendor & Version:
Operating System Type & Version:
Pull Request Type

[ ] Bug fix (non-breaking change which fixes an issue.)
[ ] Enhancement/New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
A clear and concise description of the issue being addressed.  Additional guidance here.

Describe the actual problematic behavior.
Ensure private information is redacted.

Steps To Reproduce
List the steps to reproduce the behavior.

Please be precise and ensure private information is redacted
Include things like

Files used - sql scripts, changelog file(s), property file(s), config files, POM Files
Exact commands used - CLI, maven, gradle, spring boot, servlet, etc.

Actual Behavior
A clear and concise description of what happens in the software before this pull request.

Include console output if relevant
Include log files if available.

Expected/Desired Behavior
A clear and concise description of what happens in the software after this pull request.
Screenshots (if appropriate)
If applicable, add screenshots to help explain your problem.
Additional Context
Add any other context about the problem here.
Fast Track PR Acceptance Checklist:

[ ] Build is successful and all new and existing tests pass
[ ] Added Unit Test(s)
[ ] Added Integration Test(s)
[ ] Added Test Harness Test(s)
[ ] Documentation Updated

Need Help?
Come chat with us on our discord channel

Hey @wwillard7800 - are you planning to still use this PR one day or can we close it?

Hey @wwillard7800 - are you planning to still use this PR one day or can we close it?

We can close it.


IPPM: Option for more than one input node

Currently only one is an option, but in future we will have (e.g.) left/right inputs

This will be fixed by #225, but its appearance will be improved by #363


Question marks instead of in-text citations

Reported by an user:
I face some problems, as I would need the APA option, which leads to question marks
instead of in-text citations (even if the AMA option seems to get the
in-text citations right).
Attachment with MWE: test2.zip.

Note-to-self: try setting latex-min-runs: 3 in test2.qmd. If that works integrate then in _extension.yml.


Add support for SSL_ca_path

A few things, a) You submitted this pull request without a description, so the motivation remains a mystery and even start a discussion, b) The patch has no tests, c) The attribute name ca_path is inconsistent with ca, cert and key.

c) The attribute name ca_path is inconsistent with ca, cert and key.

Do you have a better name for it? I tried to be consistent with the SSL_ca_path option.

@cziebuhr No, perhaps all related attributes should be renamed if there's going to be multiple variants of some. Now that you mention it, there's another inconsistency, Mojo::Server::Daemon would also need something like ca_path to be consistent with Mojo::UserAgent.


SubBuffer in registerStore not always defined

This line throws exceptions.
If you rapidly change a PgSubscription via the change-method, you end up with some exceptions because subBuffer may sometimes be undefined.
I think a simple check for undefined should solve the issue.

Please make a pull request with a test for this feature and I will release
a new version.
On Jun 15, 2015 7:29 AM, "waeltken"<EMAIL_ADDRESS>wrote:

This line
https://github.com/numtel/meteor-pg/blob/master/lib/PgSubscription.js#L67
throws exceptions.
If you rapidly change a PgSubscription via the change-method, you end up
with some exceptions because subBuffer may sometimes be undefined.
I think a simple check for undefined should solve the issue.
—
Reply to this email directly or view it on GitHub
https://github.com/numtel/meteor-pg/issues/6.

Thanks for your the reply.
I am not quite sure on how to write the test for that. I've not used tinytest yet. Found your blog post though.
What i would want to do is:

Subscribe to a publication
That publication should be artificially slow (in my real world use-case it is a little slow).
While we are still waiting for the initial response for the publication we change the subscription via the change method.
When the initial response arrives and we already changed that subscription it is no longer found hence the subBuffer is undefined.

I am currently using your meteor-pg library in combination with the postgis extension to postgres to display geoinformation on a map and do geospacial queries. It's imho the best of the available postgres packages in the meteor world. Will probably release my code for that in the near future, so would be really good to learn how to write those tests for packages. Currently i use the xolvio:cucumber package for end-to-end testing with velocity, not tinytest though.

I'll see if i can update the pull request soon. Thx for the feedback.

Just released version 1.0.1 incorporates your test and uses your same technique of exiting early from the registerStore update method. Thank you for the help!

Thanks, Mr. Green. ;-)

Am 14.07.2015 um 21:41 schrieb Ben Green<EMAIL_ADDRESS>Just released version 1.0.1 incorporates your test and uses your same technique of exiting early from the registerStore update method. Thank you for the help!
—
Reply to this email directly or view it on GitHub https://github.com/numtel/meteor-pg/issues/6#issuecomment-121353175.


Add test for FourierChecking circuit

Summary
Fixes #4348
Details and comments
Added a test for FourierChecking circuit by computing the expected matrix multiplying Hadamard matrices with matrices created from input truth tables for functions f and g. The mentioned matrices are multiplied in the order given in section 3.2 of this paper: H -> f -> H -> g -> H. Additionally, three invalid input tests are added to check that a CircuitError is raised for the following cases of f and g truth tables: different lengths; both truth tables containing just a single element; truth table lengths not corresponding to 2**n.

This looks good to me from the testing/coverage perspective. Maybe makes sense than @ajavadia or @jaygambetta (original authors of the code) have a look too.

Hi again @Cryoris and @1ucian0,
I was hoping you wouldn’t mind if I follow-up about this. I was wondering if there is anything I can do to improve this so that we could move forward with merging this? Thanks for your time 🙂

Thanks, @1ucian0 🙂


Maximum Covariance Analysis is slower than one in the nicrie/xmca

Hello, I recently migrated the dependency library from nicrie/xmca to nicrie/xeofs, it seems that the speed of SVD decomposition has slowed down significantly. I found that the calculation function was transferred from np.linalg.svd to sklearn.utils.extmath.randomized_svd. I personally don’t know the reason for this replacement. Maybe there are some tricks to speed up the calculation?
minimal reproducible code (xmca==1.4.2, xeofs==1.0.5)
import pooch
pooch.retrieve(url="https://downloads.psl.noaa.gov/Datasets/noaa.oisst.v2/sst.mnmean.nc", known_hash=None, path ='.', fname='sst.mnmean.nc')
pooch.retrieve(url="https://downloads.psl.noaa.gov/Datasets/cmap/enh/precip.mon.mean.nc", known_hash=None, path ='.', fname='precip.mon.mean.nc')

data_input1 = xr.open_dataset('sst.mnmean.nc', chunks = 'auto').sst.sel(time = slice('1982-01-01', '2022-12-31'))
data_input2 = xr.open_dataset('precip.mon.mean.nc', chunks = 'auto').precip.sel(time = slice('1982-01-01', '2022-12-31'))

Calculates very fast on my computer, about 5 seconds (It has nothing to do with the use of dask after the data chunk)
from xmca.xarray import xMCA

mca = xMCA(data_input1, data_input2)
mca.normalize()
mca.apply_coslat()
mca.solve(complexify=False)

But the calculation using xeof is very slow, and the result cannot be obtained in a few minutes
from xeofs.models import MCA

model = MCA(n_modes = 5, standardize = False, use_coslat = True)
model.fit(data_input1, data_input2, dim = 'time')

Great catch! You're correct, the issue you mentioned isn't related to dask. The cross-covariance from your data has dimensions 64800 x 10368, which is near the maximum size solvable on a typical laptop using np.linalg.svd. For such large matrices, the randomized SVD from sklearn is recommended as it's faster. That's why it is the default option in xeofs.
However, xmca is still able to handle the datasets with np.linalg.svd, because it first conducts standard PCA on each data set prior to computing the cross-covariance. Given that your data sets have 492 time steps, they can be fully represented by 492 PCs. Thus, when performing MCA in this reduced PCA space, the SVD solution should be swift.
Currently, xeofs doesn't offer this PCA preprocessing. But, good news – it's present in the development branch. If you need it urgently, install the development version:
pip install git+https://github.com/nicrie/xeofs.git@develop

The development version then allows you to specify the number of PCA modes. Since in your case the rank of the input matrices are 492, you can easily provide all PC modes so you don't loose any information by using the PCA solutions as input for MCA. In the development version you would specify it like this:
model = MCA(n_modes=5, standardize=False, use_coslat=True, n_pca_modes=492),
model.fit(data_input1, data_input2, dim='time')

Otherwise, if you don't mind waiting a couple of days more, I'll be releasing a new version with this feature soon.

Yeah, just what I wanted, thank you. I found this trick, looking forward to the new release

this feature is now available in the new release v.1.1.0  (see #80 )