Protect login from brute force

Failed logins are cached for 24 hours (hashed IP and the amount of attemps). If there are more than five failed attempts the IP is blocked for 24 hours.

 [Detector] Captcha after too many failed logins


Fix completion and help fail when missing config

This fixes #78
It bypasses config loading when koyeb completion or koyeb help is called

Hello! I'm very sorry it took me forever to review your PR.
I rebased your modifications and merged: https://github.com/koyeb/koyeb-cli/pull/106
Thanks a lot for your contribution!
I'm currently actively working on the CLI so if you decide to contribute again, I will definitely review your changes must faster.
Thanks again!


Angular 19

@vitaliibendyk whenever we update to a new Angular version we create a commit with BREAKING CHANGE (see this https://github.com/shiftcode/sc-ng-commons-public/pull/33/commits/46102035f219c1c54cd5799879216cfd3e15f32e for example).
With this comment a new major version will be created, so make sure to update the peerDependencies definition to reflect that (sample: libs/components/package.json, not sure the ^7.0.0-pr23.0 version is correct):
"peerDependencies": {
    "...": "...",
    "@shiftcode/ngx-core": "^7.0.0 || ^7.0.0-pr23.0",

You now have two options: create an additional commit with BREAKING CHANGE comment or create a new branch and make it correct already in first commit (preferred, since we then have a clean CHANGELOG)


Indexing documentation - theses

closes #1399

@hackartisan ^

one second I added an extra # let me remove it

ok @hackartisan can you please re-review?


feat(namespace,api,cli): adding default value to announcement on namespace

I am adding a default value to the announcement if no value is set on the announcement.

adding a new commit, to ensure that the default message will happen on the creation of namespace via cli and api

I think you can squash these two commits into one. Since they depend on each other, combining them would make the history more concise and descriptive, in my opinion.

Thanks for the advice @heiytor, I'll maintain this code with two commits since they are related but the scope with each other touches is different and I prefer to keep it this way as separate commits.


fix spinner_three_bounce plugin so that dots animate in sequence

Previously the first 2 dots animated at the same time, not sure if this was intentional?

I think this was not intentional, but this way the animation seems better. Thanks @tjenkinson!


Provide facade functionality to perform common checks/tests for analysis

When performing an analysis there are important procedures that usually have to be performed for an analysis. Hence tensorwaves could standardize these to some degree. The issue serves more as a discussion platfrom for now. @wgradl @redeboer @Leongrim @sebastianJaeger please feel free to extend and modify this issue. The procedures are:

input output checks
statistical validation. This means performing N independent full toy MC data analysis. One full toy MC data analysis consists of generating a toy MC data sample, and fitting the sample. All in all this returns N fit results, which can be combined into residual distributions for the fit parameters.

Bootstrapping could be used here, to minimize the number of data samples that need to be created

Moving this to compwa-org as this is best implemented in an overarching, PWA-specific compwa library that provides façade functions for QRules, AmpForm, and TensorWaves.


alua: reduce the debug logs

When the loglevel is in debug mode, for each IO/non-generic SCSI cmd
there will be one debug message output, and there will be aweful
amount of logs full of the log file.
Signed-off-by: Xiubo Li<EMAIL_ADDRESS>

@mikechristie
I couldn't reopen that old PR(https://github.com/open-iscsi/tcmu-runner/pull/393), so just create a new one.


chore(deps-dev): update eslint dependency

I think CLIEngine should be replaced with ESLint class.

@ota-meshi Fixed 👍


fix(scroll-list): issue with incorrect shown items calculation

Because sometimes we get 0 / 0 which is NaN, and in fight Math.max(Infinity, NaN) NaN wins.

Approved by @PeterShershov 🤓


Things to be documented in minor 2.3

@root - is available to a component's view model but should probably not be used.
#ref - reference bindings
[parent]=child - child -> parent reference bindings.

Progressive loading with <can-import>

All documentation should be up-to date now.


Moved the ModelFactory code to its own project

Referenced in #26
Doing this helps us remove the constraint from being tied to Umbraco v7.1.4 - we can add support for v6.x (maybe even v4.10.x?)

Beat me to the punch! This was gonna be part of v6 support. Will help muchly!

Thought you'd said "fire ahead"? (in #26) ;-)

Hehehe... Ace. :+1:


Fix timeout in MTRSwiftPairingTests.

3 seconds is not enough time to do commissioning, sometimes.

Fast-tracking platform-specific fix with platform owner review.


Add natspec for limit order

nit: What about capitalizing first letter of each description to serve as a start of sentence? I think this would especially helpful to separate param and its description.


Remove @akveo/ng2-completer and ng2-completer

Please read and mark the following check list before creating a pull request (check one with "x"):

[x] I read and followed the CONTRIBUTING.md guide.
[x] I read and followed the New Feature Checklist guide.

Short description of what this resolves:
Fixes #5814. smart-table dependency did change in version 1.7.0, see https://github.com/akveo/ng2-smart-table/blob/master/CHANGELOG.md
(I recreated the MR because i pushed the package-lock.json inadvertently)

It must be merged asap,
ref: https://github.com/akveo/ngx-admin/issues/5893

🤟


formatter: Multi-line declaration formatting.

This issue was originally filed by<EMAIL_ADDRESS>
A problem with the formatter, it converts:
var sum = 0,
    volume = 0;
to:
var sum = 0, volume = 0;
I think the original is much better (and standard JavaScript style).
However, it should convert:
var sum,
    volume;
to
var sum, volume;

This issue has been moved to dart-lang/dart_style#317.


depth_from_video_in_the_wild: scale factor of the predicted translation vector

@gariel-google
Dear author, Thanks for sharing the source code of the paper.
I have a question about the scale factor of the predicted translation vector.
Basically, I tried to use the predicted depth and  the predicted egomotion to warp the image T to image T+1.
I first project image T using the predicted depth get a 3D pointcloud then use the predicted translation vector, rotation matrix and camera intrinsic to project the 3D pointcloud back to an image space (T+1). However, I can’t get a proper warped image T+1 WITHOUT scaling the translation vector.  After some trial and error, I found that the scale factor for the translation vector is about 0.001.
Therefore, my question is: do you need such a scaling factor for the translation vector when you try to project image T to image T+1 (or inversely) to compute the reconstruction loss? I am quite confused as I understand the predicted depth’s scale and the predicted egomotion’s scale should be matched and thus we should be able to do reconstruction without scaling those variables.
Any help would be appreciated. Thank you in advance.

Hi @liyingliu,
Indeed the translation vector is scaled by a learned factor. The code is here. If you use motion_field_net directly, and load form the checkpoints, all this should be taken care of for you.
Hope that helps!


Error when I compile the app

I'm using Stencil on my App but when I try to compile it I have this error:
Command /bin/sh failed with exit code 1
A shell task (/usr/bin/xcrun codesign --force --sign - --preserve-metadata=identifier,entitlements /Users/rubenfernandez/Library/Developer/Xcode/DerivedData/APP_NAME/Build/Products/Debug-iphonesimulator/APP_NAME.app/Frameworks/Stencil.framework) failed with exit code 1:
/Users/rubenfernandez/Library/Developer/Xcode/DerivedData/APP_NAME/Build/Products/Debug-iphonesimulator/APP_NAME.app/Frameworks/Stencil.framework: replacing existing signature
/Users/rubenfernandez/Library/Developer/Xcode/DerivedData/APP_NAME/Build/Products/Debug-iphonesimulator/APP_NAME.app/Frameworks/Stencil.framework: resource fork, Finder information, or similar detritus not allowed
How can I solve it?
Thanks in advance

@Rubenfer Does not seem to be a Stencil issue: https://developer.apple.com/library/content/qa/qa1940/_index.html


Working with Stencil.js components requires extra setup

Command
serve
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
As stated in issue #26345 setting up Angular 17 with Stencil.js components needs some extra configuration, which is anything but obvious. Also, the mentioned ticked was locked due to inactivity, but aside from a workaround no work seems to have been done to solve the issue.
Minimal Reproduction
The reproduction repo is taken from the other repo: https://github.com/zerock54/Angular-17-esbuild-Stencil-ng-serve-issue
Exception or Error
No response
Your Environment
Angular CLI: 17.0.10
Node: 18.16.1
Package Manager: npm 9.5.1
OS: win32 x64

Angular: 17.0.9
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1700.10
@angular-devkit/build-angular   17.0.10
@angular-devkit/core            17.0.10
@angular-devkit/schematics      17.0.10
@angular/cli                    17.0.10
@schematics/angular             17.0.10
rxjs                            7.8.1
typescript                      5.2.2
zone.js                         0.14.3

Anything else relevant?
The workaround provided in the other issue does work, but as mentioned by comment https://github.com/angular/angular-cli/issues/26345#issuecomment-1842761606 I had to disable the Angular cache.

Hi @riker09,
Adding the below config as mentioned in https://stenciljs.com/docs/config-extras#enableimportinjection does work even with cache enabled.
  extras: {
    enableImportInjection: true,
  },

If the problem is that the changes to the components is not reflected in the browser. This is expected and since you are using npm link, you need to specify the linked library in the tsconfig path mappings. This will cause the library to be bundled as part of the application and not handled by vite optimizer.
/* To learn more about this file see: https://angular.io/config/tsconfig. */
{
  "compileOnSave": false,
  "compilerOptions": {
     ...
    "paths": {
      "test-compo/loader": ["./node_modules/test-compo/loader"]
    }
}

Clarification: I actually don't use Stencil myself. It's a closed-source third-party component library that uses Stencil.js itself.

I will take it up with the authors of the third-party library. Thanks for your quick reply!


FUN3D Aero Coordinate Derivatives don't work with swap_yz on

In flat plate test case, got dGdxa0^T psi_G in y and z directions were flipped with a negative sign. Is this because the swap_yz was on? Do we need to make a warning for not using swap_yz when extracting aero coordinate derivatives. Why was there also a negative sign?
TODO: try this with the diamond wedge case in node 1, y and z directions (two separate runs)
Subsonic flat plate derivative tests, caveat swap_yz = .true. was on:
test-fun3daero-coords.txt

Ran the same test with swap_yz = .false. now and the FUN3D aero coordinate derivatives now work...
fun3daero-coords.txt

After changing some of the FUN3D source code, I am now able to get machine precision aerodynamic coordinate derivatives when swap_yz_axes from the FUN3D namelist is on/off. Closing this issue now


Enable tests for ol.geom.flat.reverse

Good catch.
Do we have a CLA from you?

I just submitted it.

@elemoine were you able to verify that I submitted a CLA?

I'm not the one receiving CLA submissions. @tschaub?

@elemoine it looks to me like you do have edit rights to the ICLA form responses.  I found the responses by going to the CLA readme, clicking on the ICLA form, finding the "edit this form" button, and then clicking "view responses."  Yes, there are outdated docs around and we should migrate to CLAHub.

I'm not seeing the "edit this form" button for some reason.


flexible DT for different grid comps

flexible DT for different grid comps
Description

Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Trivial change (affects only documentation or cleanup)

Checklist:

[ ] I have tested this change with a run of GEOSgcm (if non-trivial)
[ ] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[ ] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

@atrayano , In Moist grid comp, the implementation for flexible DT is already in place.  The problem is the ring time. If we create the generic alarm in different way, we don't need to change Moist grid comp ( may be other grid comps too). In Config rc file, if we add "MOIST_DT:" and "MOIST_ALARM_RING_AT_0: YES", it should work.

I'm going to re-open this until @weiyuan-jiang and @atrayano say whether to close or not.

@bena-nasa @weiyuan-jiang This isn't needed anymore, right? I mean, if anything Ben has new code needed by Bill for his "chemistry on different DT"?


Subclasses of pieces that add more default filters for the frontend are frustrating to edit because you can't find future-dated blog posts and the like

I'm addressing this with a new findForEditing method which is used consistently when we want to see stuff we can edit without regard to when it was published, etc. Basically inclusive of everything except trash as long as we have the permissions for it.

Resolved in #575


Visibility on Spatial overrides/cascades over children (no opt-out)

Godot version:
3.2.2.stable.mono.official
Issue description:
Toggling the 'Visible' property on a spatial also hides children of that node even though they have 'Visible' still toggled on. And as there's no toggle to disable nodes in general as in other engines (which implicitly hides children), there's no apparent accessible way to make a child visible but its parent invisible.
One cumbersome workaround would be to move parents to a different layer and adjust the camera's culling mask. This workaround is only feasible for games with little usage of layers and will fail silently if that invisible layer is used for different reasons later on.
It seems the editor even shares this toggle in the scenetree (eye-icons). Toggling visibility with the cascading effect there (in the editor) makes sense from a users standpoint. But ideally (as it's also used in builds) it should not cascade and the editor toggle be a separate state with overridable behaviour (see unity's hierarchy). Or alternatively the visibility just be overridable for children. Or alternatively a plain 'Active' toggle for nodes be added.
Steps to reproduce:
Nest 2 Spatials with something visible (geometry), toggle Visible on the parent.

This isn't a bug. Please suggest features on the Godot proposals repository after searching for possible duplicates.


Analyzer doesn't properly ignore generic method type parameters in spec mode.

I'm adding generic type parameters to the core libraries (https://codereview.chromium.org/2529393002).
In practice, they should act the same as the existing comment-based type parameters (that is, ignored in spec mode, only meaningful in strong mode).
When running tools/tests.py -cdart2analyzer -rnone, I get warnings like:
FAILED: dart2analyzer-none release_x64 benchmark_smoke/benchmark_smoke_test
Expected: Pass
Actual: StaticWarning
CommandOutput[dart2analyzer]:

stderr:
WARNING|STATIC_WARNING|ARGUMENT_TYPE_NOT_ASSIGNABLE|/usr/local/google/home/lrn/dart/co/sdk/third_party/pkg/stack_trace/lib/src/chain.dart|204|16|8|The argument type '<T extends num>(T, T) → T' can't be assigned to the parameter type '(dynamic, dynamic) → dynamic'.
WARNING|STATIC_WARNING|ARGUMENT_TYPE_NOT_ASSIGNABLE|/usr/local/google/home/lrn/dart/co/sdk/third_party/pkg/stack_trace/lib/src/trace.dart|289|18|8|The argument type '<T extends num>(T, T) → T' can't be assigned to the parameter type '(dynamic, dynamic) → dynamic'.

The line of the error is using max from dart:math as a function parameter, and max has been made generic. The analyzer seems to leak the strong-mode generic function type into spec mode checking. In spec mode, there is no such thing as a generic function type.
Similarly, it gives warnings like:
WARNING|STATIC_TYPE_WARNING|INCONSISTENT_METHOD_INHERITANCE|/usr/local/google/home/lrn/dart/co/sdk/third_party/pkg/collection/lib/src/wrappers.dart|290|7|15|Inconsistent declarations of 'fold' are inherited from (dynamic, (dynamic, E) → dynamic) → dynamic, <T>(T, (T, E) → T) → T.

where the generic type leaks and the override with a non-generic method causes a warning.

taking a look.
ARGUMENT_TYPE_NOT_ASSIGNABLE -- this one is pretty easy, we need to make sure generic tear offs are instantiated (to their bounds)
INCONSISTENT_METHOD_INHERITANCE -- I'll have to take a look where this one originates from, but we definitely want to remove the generic function types when ERROR_VERIFIER is taking a look at things.
One fix that would probably hit both of these at once would be to change the subtyping rule to instantiate to bounds before doing function subtype checks.

https://codereview.chromium.org/2621433003/


Error "AADSTS90023 Request is malformed or invalid." is missing

And it doesn't provide information about what exactly is wrong.

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

ID: a826ee6f-f457-7109-9f77-67c57977d4e7
Version Independent ID: 11766f27-492e-a579-dced-8a42528cbb47
Content: Azure Active Directory authentication and authorization error codes
Content Source: articles/active-directory/develop/reference-aadsts-error-codes.md
Service: active-directory
GitHub Login: @CelesteDG
Microsoft Alias: celested

@slaFFik  Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

Thank you for your feedback, @slaFFik - At this time, the only additional info I was able to find is that it maps out to an invalid authentication service request. I'll add that info to the description column, but no other additional info for now.

But what exactly is wrong is missing, and that is not good.

Hi @slaFFik - Can you please provide some more info about your scenario and what you're trying to do so we can understand where the issue might be stemming from?

Hi @CelesteDG
Sure.
I use https://github.com/thephpleague/oauth2-client library to make requests.
Using its getAuthorizationUrl( $url_options ) method with these options:
$url_options = array(
	'state' => $state,
	'scope' => [ 'https://graph.microsoft.com/mail.send' ],
);

I get the correct URL, which is the same as per your Docs. Proceeding to that url and providing consent - I'm redirected back to my web-server to returnURL with proper 2 GET params (state and code). State is correct and I save the code to use it later to retrieve token like this:
try{
	$creds = $this->client->getAccessToken(
		'authorization_code',
		array(
			'code' => $this->options['auth_code'],
		)
    );
} catch ( IdentityProviderException $e ) {
    // I'm here receiving that exception, all the time.
}

And that's where I get an exception (when oauth library does the response parsing):

IdentityProviderException
League\OAuth2\Client\Provider\Exception\IdentityProviderException Object
(
    [response:protected] => Array
        (
            [error] => invalid_grant
            [error_description] => AADSTS9002313: Invalid request. Request is malformed or invalid.
Trace ID: 1aab5404-bbce-466b-9370-0436849cde00
Correlation ID: 08ceb46e-730c-439f-ab11-1da0158a1e7b
Timestamp: 2019-02-13 18:30:13Z
            [error_codes] => Array
                (
                    [0] => 9002313
                )

            [timestamp] => 2019-02-13 18:30:13Z
            [trace_id] => 1aab5404-bbce-466b-9370-0436849cde00
            [correlation_id] => 08ceb46e-730c-439f-ab11-1da0158a1e7b
        )

    [message:protected] => invalid_grant
    [string:Exception:private] =>
    [code:protected] => 0
    [file:protected] => C:\OpenServer\domains\am\smtp\wp-content\plugins\wp-mail-smtp\vendor\league\oauth2-client\src\Provider\GenericProvider.php
    [line:protected] => 222
    [trace:Exception:private] => Array(...)
    [previous:Exception:private] =>
)

Invalid_grant means that the thing you're using for your grant (the authorization code) was malformed.  I'll update the error message to make some suggestions about what could be wrong.  In this scenario, we saw this exception on the backend: System.FormatException: Invalid Input. Length of original grant (X): 37, Length after conversion to Base64 (without padding) (Y): 37. Y%4 can never be 1.
I've seen this occur before when the code has either not been url decoded, or been url encoded too many times.

Thanks for answers, you gave me some ideas.
That's definitely NOT the encode/decode issue, as the auth_code error (invalid_grant) is returned when something wrong with auth_code itself, but it can't have decoding issues due to its simplicity.
What is TTL for auth_codes? Right now I assume that sending old/expired auth_code generates that error on your side. Also, how are already used auth_codes are processed? Is the error still invalid_grant? Any additional description for those cases?

Auth_codes are good for about 5 minutes.  We do have separate errors for expired or reused codes - both are very explicit in the issue.  The code here is clear that this is the result of something having too many or too few characters in it that is breaking the base64 decoding - it could possibly be the client secret as well.
What scope was this requested for? I see an odd branch in the logic that might give a clue.  openid+profile+email only?
Feel free to send me a fiddler trace or full URL along with initial redirect response in email to hirsin @ Microsoft.com

@slaFFik - Please see the info from hpsin for any additional follow up questions.
#please-close

@slaFFik Closing this issue since the conversation was taking to email.

For future reference:
I was getting this error AADSTS90023 Request is malformed or invalid. when I sent a sanitized version (which is also lowercased) of the code back to MS API to retrieve tokens.
So the issue was indeed the incorrect code (lowercased in my case).

Thanks! That's good to know. Thank you. I'll try to think of a way we can provide a better hint here about the code not being what we expected.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Slava Abakumov<EMAIL_ADDRESS>Sent: Friday, February 15, 2019 1:15:11 PM
To: MicrosoftDocs/azure-docs
Cc: Hirsch Singhal; Comment
Subject: Re: [MicrosoftDocs/azure-docs] Error "AADSTS90023 Request is malformed or invalid." is missing (#24804)
For future reference:
I was getting this error AADSTS90023 Request is malformed or invalid. when I sent a sanitized version (which is also lowercased) of the code back to MS API to retrieve tokens.
So the issue was indeed the incorrect code (lowercased in my case).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F24804%23issuecomment-464201979&data=02|01|hirsin%40microsoft.com|2e4fdb31ce614c12b28b08d6938aabd9|72f988bf86f141af91ab2d7cd011db47|1|0|636858621128771751&sdata=ZVjf2nNo3vkh7Dd55StDv4jJSEnsppBynmCFY07w%2FnM%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABltOy6vJ6MMJ6O09y4hiaJSc4v5vRa1ks5vNyNfgaJpZM4a3gGU&data=02|01|hirsin%40microsoft.com|2e4fdb31ce614c12b28b08d6938aabd9|72f988bf86f141af91ab2d7cd011db47|1|0|636858621128771751&sdata=vlPz%2BtwhpQn1%2Fgrh0flx1x0x%2Bvetwwru53ZOOyWa3mE%3D&reserved=0.

Hello @hpsin ...I wondered if you might be able to confirm if I am hitting the same error message (System.FormatException: Invalid Input. Length of original grant) that was seen above? I'm getting exactly the same error message: AADSTS90023 Request is malformed or invalid..
If you can check this for me what info would you need from myself?
Thanks,

You've probably touched the code or secret somehow in flight by either URL encoding it or decoding it once too many times.  To help I'd need to see the actual request being made. Note that the server needs to match the string against something in its database - it's helping you along by indicating that you've done something to the string that changed the length of one of the input parameters.  Take a look at the Postman traces to see how this should work. https://www.getpostman.com/collections/f77994d794bab767596d

Thanks for getting back to me @hpsin :)
That Postman collection is really handy, thanks for sending that through. I'd effectively replicated that setup inside Postman whilst dissecting requests/responses in the OAuth Sandbox...just to confirm I was sending the correct params. I can't see anything unusual when comparing to what I'm sending...although...
Something that stands out is that I'm getting invalid_grant with "AADSTS90023 Request is malformed or invalid.." but I'm making a request with grant_type: refresh_token. That doesn't sound right to me - I'd expect invalid_grant be a response for the initial OAuth steps (when I request access from the customer via the OAuth login screen)...in other words not when I'm trying to refresh an expired access token (which I believe to be the case in my situation).
To your point about malformed authorization code or secret I managed to reproduce this with Postman by stripping one character from the authorization code param I was sending...exactly the same error message. I'm trying to work out how to confirm the authorization code is malformed without logging it out (although don't they expire after they have been used to get the first access token - and from then on my application uses only the refresh token for any subsequent access tokens?). I'm pretty sure I don't store the authorization code in my database...so if the authorization code is getting corrupted it must be happening somewhere between receiving the authorization code and sending it back in the following request to get the first access token (and refresh token)...does that theory sound correct to you?
Many thanks again 🏆

For future reference:
I was getting this error AADSTS90023 Request is malformed or invalid. when I sent a sanitized version (which is also lowercased) of the code back to MS API to retrieve tokens.
So the issue was indeed the incorrect code (lowercased in my case).

Hello @slaFFik, thanks for your details on this issue. I'm in the same situation and have one question for you: were you receiving the error message intermittently? If the issue was in your source code (which it sounds like because you mention lowercasing) then I'd expect you to be receiving the error message every single time you made the requests.

I have the same problem, someone help me
AADSTS9002313: Invalid request. Request is malformed or invalid.\r\nTrace ID: 99dcd85a-84fa-47ab-b1c2-03f75e470600\r\nCorrelation ID: 01e0c950-74df-44c3-8649-7bb43b4baf4e\r\nTimestamp: 2021-11-04 23:06:30Z",


[general] Outputting Foundation credits/charset rule

I'm working on a WordPress Foundation starter theme which I hope to release soon!
I have 2 issues that I want to try and solve, purely for aesthetic reasons really.
Foundation credits
When importing the foundation file through Sass, we get the version and credits as a comment. I understand the reason for this, but I credit Zurb elsewhere in the project, and its being output in the stylesheet when I'd prefer not to (I have the WordPress theme details, description at the top already). I have a patch ready to submit for a PR for this, pending a response from point 2...
charset rule
The other issue I can't work out is the @charset "UTF-8"; declaration that appears at the top of the stylesheet (above the WordPress required theme details). Is there any way we could choose where to include this, or include it in global styles? I'm sure there's a good reason not to do this but I'd love to know either way!

The version is necessary for our CSS/custom downloads, as it helps us figure out what version someone is using when debugging issues. Would your patch just remove the comment? Unfortunately it would be difficult for us to justify that for one use-case.
The @charset declaration has to be the very first thing in a stylesheet. For that reason, we don't put it in Foundation itself; however, in our template projects, we put it at the very top of the app.scss entry file. (See an example here.)

Thanks @gakimball - I fully understand about the comment and why you would need it. My idea was this commit. Basically a variable that if true (on by default) would output the comment. I thought this would cover the css/custom downloads as it would be there by default. What do you think?
Thats strange about the @charset - I don't define it anywhere in a stylesheet, but its appearing at the top of the compiled stylesheet. Not to worry I'll investigate further!
I have another question about the Motion UI includes (@include motion-ui-transitions; @include motion-ui-animations;) but I guess thats better to raise over there? They seem to be outputting the css at the top of my stylesheet, even though they are at the bottom.
I look forward to giving back by releasing my theme soon!

Looking forward to seeing your theme for sure :) We're building up a new list of F6-specific resources and we can add it there.
As for the poster comment thing, it's just too specific of a feature to justify having in the codebase. If you need to remove the comment, there are tools that can strip CSS comments, either standalone or as part of a code compressor.
The Motion UI thing is an unfortunate side-effect of how @extends works. There might be a way to fix it, though. If you want to open an issue at zurb/motion-ui we can discuss it.


Support viewing JLD2 files

Is your feature request related to a problem?

JLD2 is a file type natively created by Julia programming language, and it is heavily used in high performance scientific programs, JLD2
JLD2 is designed as comprising a subset of HDF5, though I'm non-expert in this repository, I think it might be reachable to support JLD2 with likewise interface.

Alternatives you've considered
The authors of JLD2.jl are eagering for a vscode-extension to visualize JLD2 files, see discussions below
https://discourse.julialang.org/t/jld2-preview-in-vscode/80050/6
https://github.com/julia-vscode/julia-vscode/issues/2863

Additional context
Currently opening a JLD2 using H5web gives error message like below, any hint to solve it?
HDF5-DIAG: Error detected in HDF5 (1.14.2) thread 0: #000: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5D.c line 403 in H5Dopen2(): unable to synchronously open dataset major: Dataset minor: Can't open object #001: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5D.c line 364 in H5D__open_api_common(): unable to open dataset major: Dataset minor: Can't open object #002: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5VLcallback.c line 1980 in H5VL_dataset_open(): dataset open failed major: Virtual Object Layer minor: Can't open object #003: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5VLcallback.c line 1947 in H5VL__dataset_open(): dataset open failed major: Virtual Object Layer minor: Can't open object #004: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5VLnative_dataset.c line 321 in H5VL__native_dataset_open(): unable to open dataset major: Dataset minor: Can't open object #005: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Dint.c line 1429 in H5D__open_name(): can't open dataset major: Dataset minor: Unable to initialize object #006: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Dint.c line 1494 in H5D_open(): not found major: Dataset minor: Object not found #007: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Dint.c line 1689 in H5D__open_oid(): unable to load type info from dataset header major: Dataset minor: Unable to initialize object #008: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Omessage.c line 432 in H5O_msg_read(): unable to read object header message major: Object header minor: Read failed #009: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Omessage.c line 487 in H5O_msg_read_oh(): unable to decode message major: Object header minor: Unable to decode value #010: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Oshared.h line 61 in H5O__dtype_shared_decode(): unable to decode shared message major: Object header minor: Unable to decode value #011: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Oshared.c line 358 in H5O__shared_decode(): unable to retrieve native message major: Object header minor: Read failed #012: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Oshared.c line 172 in H5O__shared_read(): unable to read message major: Object header minor: Read failed #013: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Omessage.c line 432 in H5O_msg_read(): unable to read object header message major: Object header minor: Read failed #014: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Omessage.c line 487 in H5O_msg_read_oh(): unable to decode message major: Object header minor: Unable to decode value #015: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Oshared.h line 74 in H5O__dtype_shared_decode(): unable to decode native message major: Object header minor: Unable to decode value #016: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Odtype.c line 1338 in H5O__dtype_decode(): can't decode type major: Datatype minor: Unable to decode value #017: /__w/libhdf5-wasm/libhdf5-wasm/build/1.14.2/_deps/hdf5-src/src/H5Odtype.c line 154 in H5O__dtype_decode_helper(): bad version number for datatype message major: Datatype minor: Unable to load metadata into cache

Do you have a sample file available for testing?

We will definitely strive to support JLD2 files. Thanks for opening an issue in the H5Web repo. As explained, the problem comes from h5wasm not currently supporting committed datatypes. I've opened an issue on the h5wasm repo too: https://github.com/usnistgov/h5wasm/issues/80

Thanks to @bmaranville and @axelboc work, we made good progress towards this. I could read this simple JLD2 file without issue using the main branch of H5Web:  example.zip

Generation script

# Inspired by https://github.com/JuliaIO/JLD2.jl?tab=readme-ov-file#jld2
using JLD2
jldsave("example.jld2"; x, y, z)
jldopen("example.jld2", "r+"; compress = true) do f
f["large_array"] = zeros(10000)
end
jldopen("example.jld2", "r+") do file
mygroup = JLD2.Group(file, "mygroup")
mygroup["mystuff"] = 42
end

I can confirm!
Though, your example file here does not actually rely on the fix given that it does not use compound datatypes.
The fact that the following file now works is quite impressive:
julia> using JLD2

julia> struct InnerStruct
           x::String
           y::Int
       end

julia> struct OuterStruct
           a::Int
           b::InnerStruct
           c::NTuple{3,Int}
       end

julia> jldsave("nested_compound.jld2"; data=OuterStruct(1, InnerStruct("two",3),(4,5,6)))

nested_compound.zip

Here are two more files which are conceptually interesting:
JLD2 uses h5 references to refer to mutable fields inside structs and tracks object identities while saving and loading.
This allows storing and loading recursive structures as well as obj identity preservation.
In the second example, the field array is encoded and loaded only once. Both struct fields refer to the same memory after loading.
H5Web currently gives you no way to view / de-reference linked datasets in this way.
That avoids all the potential pitfalls of circular references but also prevents viewing some of the data encoded in JLD2.
Importantly though, it does not error even with files like this.
In my eyes, this is good enough for a release at this stage. Future feature ideas might be enhanced pretty printing for compound types and enabling the manual loading of referenced datasets.
julia> using JLD2

julia> mutable struct RecursiveStruct
           x::Float64
           y::RecursiveStruct
           RecursiveStruct(x)=new(x)
           RecursiveStruct(x,y)=new(x,y)
       end

julia> jldsave("recursive.jld2"; r=^C

julia> r = RecursiveStruct(1)
RecursiveStruct(1.0, #undef)

julia> r2 = RecursiveStruct(2, r)
RecursiveStruct(2.0, RecursiveStruct(1.0, #undef))

julia> r.y = r2
RecursiveStruct(2.0, RecursiveStruct(1.0, RecursiveStruct(#= circular reference @-2 =#)))

julia> jldsave("recursive.jld2"; r)

julia> load("recursive.jld2")
Dict{String, Any} with 1 entry:
  "r" => RecursiveStruct(1.0, RecursiveStruct(2.0, RecursiveStruct(#= circular …

julia> struct ObjIDPreservation
           arr1::Vector{Int}
           arr2::Vector{Int}
       end

julia> arr = [1,2,3,4,5,6]
6-element Vector{Int64}:
 1
 2
 3
 4
 5
 6

julia> obj = ObjIDPreservation(arr, arr)
ObjIDPreservation([1, 2, 3, 4, 5, 6], [1, 2, 3, 4, 5, 6])

julia> obj.arr1 === obj.arr2 # references the same memory
true

julia> jldsave("objidpreservation.jld2"; obj)

julia> data = load("objidpreservation.jld2", "obj")
ObjIDPreservation([1, 2, 3, 4, 5, 6], [1, 2, 3, 4, 5, 6])

julia> data.arr1 === data.arr2
true

objidpreservation.zip
recursive.zip

Sorry I don't really know how to use the main branch version of h5web, but I think tests above is good for most jld2 file case I will use

#47 improves support for committed datatypes and #48 allows JLD2 files to open directly in H5Web. This should be sufficient to bring basic support for most JLD2 files and resolve this issue.
I've started a discussion thread in the H5Web repo with improvement ideas mentioned in this issue so as to not lose track of them. Feel free to continue the discussion and/or create proper feature requests for these ideas over there.


simplify casting of the (q)tls.ClientSessionCache

This should be equivalent to a reinterpret_cast in C++, i.e. we don't need to copy bytes around any more.

Codecov Report

Merging #2389 into master will decrease coverage by 0.09%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2389      +/-   ##
==========================================
- Coverage   85.98%   85.89%   -0.09%
==========================================
  Files         119      119
  Lines        8906     8894      -12
==========================================
- Hits         7657     7639      -18
- Misses        928      931       +3
- Partials      321      324       +3

Impacted Files
Coverage Δ

internal/handshake/client_session_cache.go
80.39% <100%> (-16.43%)
:arrow_down:

server.go
83.74% <0%> (+0.61%)
: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 d31dcda...7026324. Read the comment docs.


use sdk's getWithdrawalsByAddress error

use getWithdrawalsByAddress to get withdrawals，but retrun erros
1、the address had Withdrawal return error:
const withdrawals = await tr.CROSS_CHAIN_MESSAGER.getWithdrawalsByAddress("0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266")
error: Error: input is not a hex string: undefined
2、the eoa address unwithdrawal return []
const withdrawals = await tr.CROSS_CHAIN_MESSAGER.getWithdrawalsByAddress("0xFca9E706e5b0AE97B8F98F747F57Fa64f75EC48D")
3、the l2 bittoken  address return []
const withdrawals = await tr.CROSS_CHAIN_MESSAGER.getWithdrawalsByAddress("0xdEAddEaDdeadDEadDEADDEAddEADDEAddead1111")

command line: yarn hardhat test test/estimate_gas.ts


Added Datez

@Mazyod Is this project Apple Watch only? (Asking because it is marked with :watch:)

@victorgama Oops, I figured the :watch: would mean it supports Apple Watch. I updated the change.

Thank you!


Fix fee estimation issue when sending through a deep link

Description
If a user opens a payments deeplink they get directed straight to the SendConfirmation screen and therefore they didn't have a fee estimation, so the Send button wasn't clickable.
This PR makes it so that the fee estimation is made if it's not already in the Redux store.
Other changes
N/A
Tested
Manually
How others should test

On a fresh install, open a deep link like celo://wallet/pay?address=0xB2539a7dcc69a0B28c7Ca73f1D2FF06b9d59E5b9&displayName=Test&currencyCode=KES&amount=0.1
See that you can send correctly.

Related issues

Fixes #1911

Backwards compatibility
N/A

I've been unable to reproduce this in e2e test. I've attempted to clear the redux store by uninstalling and reinstalling, but I am still able to send to the deep link provided in the example after completing the restore flow.

Thanks! Would be great to cover this with a test.

Thanks for keeping me accountable :)

Hey @MuckT We have verified the above issue in latest android internal release build and latest iOS testflight release build V:1.25.2 and observed that : User is redirected to homepage after selecting valora application via deep link
refer attachment : kotanipay.mp4

Hey @MuckT we have verified the above issue in latest Android internal release build V:1.25.2 and latest iOS testflight build V:1.25.2 and updated the status in test rails
Android 11:
https://valoraapp.testrail.io/index.php?/tests/view/7426&group_by=cases:section_id&group_order=asc&group_id=33
iOS 14 :
https://valoraapp.testrail.io/index.php?/tests/view/7411&group_by=cases:section_id&group_order=asc&group_id=33
Observation : User is redirected to homepage instead of send amount page after performing deep link flow bug
Devices : Google Pixel 2XL(11.), iPhone 12(14.7.1)
CC: @gnardini

Hey @MuckT we have verified the above task in latest Android internal release build V:1.25.2 and updated the status in test rails
Android 8:
https://valoraapp.testrail.io/index.php?/tests/view/7504&group_by=cases:section_id&group_order=asc&group_id=33
Observation : User is redirected to homepage instead of send amount page after performing deep link flow and here is the link for bug
Devices : Samsung Galaxy J5 Prime(8.0.0)
CC: @gnardini

Hey @MuckT we have verified the above task in latest Android internal release build and latest iOS Testflight build V:1.25.2 & updated the status in test rails
Android 10:
https://valoraapp.testrail.io/index.php?/tests/view/7519&group_by=cases:section_id&group_order=asc&group_id=33
iOS 13:
https://valoraapp.testrail.io/index.php?/tests/view/7534&group_by=cases:section_id&group_order=asc&group_id=33
Observation : User is redirected to homepage instead of send amount page after performing deep link flow and here is the link for bug for it.
Devices : Samsung Galaxy M40(10.0), iPhone 7(13.3)
CC: @gnardini
Thanks..!!

Hey @MuckT, we have verified the above task in latest Android internal release build V:1.25.2 and updated the status in test rails. User is redirected to homepage instead of send amount page after performing deep link flow, we have raised a bug.
Test rail link:
Android 9: https://valoraapp.testrail.io/index.php?/tests/view/7549
Bug: Link
Devices : Redmi Note 5 pro (9.0)
CC: @gnardini
Thanks..!!

Hey @MuckT, we have verified the above task in latest Test flight release build V:1.25.2 and updated the status in test rails. User is redirected to homepage instead of send amount page after performing deep link flow, we have raised a bug.
Test rail link:
iOS 15: https://valoraapp.testrail.io/index.php?/tests/view/7564
Bug: Link
Devices : iPhone 13 mini (15.1.1)
CC: @gnardini
Thanks..!!


limit SearchIndexer MatchCount

As a user of SearchIndexer I am concerned about memory usage if MatchCount is allowed to grow without bounds.  Provide a parameter to the user that enables them to select an upper limit on MatchCount for their application.

https://github.com/imagibee/Gigantor/pull/24


Added ability to specify which path goes with which method

As you know most of the frameworks have ability to specify which path goes with which method . it will reduce the codes of views and codes will be more readable because every view work for a specify method .

You can use built-in decorators.


Remove custom media type for Pull Request Reviews API

The custom media type application/vnd.github.black-cat-preview+json can be removed from go-github as the "Pull Request Reviews API" is no longer in preview but has become official.
See GitHub Developer announcement:
https://developer.github.com/changes/2017-05-09-end-black-cat-preview/

@gmlewis please assign this to me

@gmlewis @shurcooL Pull Requests have three categories - Reviews, ReviewComments and Review Requests. Is this change required for all of them. The documentation points are reviews only.

I think it's all endpoints that currently set the custom application/vnd.github.black-cat-preview+json Accept header. They no longer need to.

This change breaks Enterprise users -- we don't have reviews out of preview yet :(.

Thanks for reporting @swsnider.
Re-opening for further consideration.
How should we go about dealing with this, @gmlewis?
A short term solution is to revert this removal of the preview API.
However, a long term solution is deciding on our policy. Should latest master target GitHub.com
API only, or GitHub.com and GitHub Enterprise APIs? The challenge with doing the latter from what I can see is that their API chances/announcements are more difficult to track down for Enterprise API. But maybe I haven't looked hard enough.

The problem is that even after the Enterprise version with this fixed is released, you're typically still at the mercy of your IT org to upgrade your instance in a timely manner, so this may not be soluble by this project :(.

Right. So it might be the case that a better/more viable solution is to, unfortunately, use an older version of go-github via vendoring.

Yeah, that's what I'm doing for now

I think the "good-for-beginners" label could be removed from this,

I had a talk with @willnorris about this, and I believe our resolution was that semantic versioning (#376) would be the best way to support GitHub Enterprise API users (and that they would need to vendor the version that best supported their use-case), and that the latest master target the latest GitHub API.
As such, I think the "good-for-beginners" label still holds for this issue.

We can't help IT departments that are slow to upgrade, and vendoring (together with semver) is the right solution there.
But I kinda like the policy of not removing preview media types until they have been released in GitHub Enterprise (assuming we can track that without too much trouble).  We could go ahead and create issues like this when APIs come out of preview for github.com, and apply a special issue label, but don't actually remove the code.  Then, when the next GitHub Enterprise version is released, we go through the issues to see which ones we can remove.  It's a little bit of manual work but it's only done periodically.

As such, I think the "good-for-beginners" label still holds for this issue.

Note, the original task of removing the media type has already been done in 11516601ad905b8307923d093a487b6a5446f2c4, @gmlewis. I've reopened the issue only to discuss the outcome regarding Enterprise not having support for this yet. Aside from discussion, this issue isn't actionable, so I'll remove that label.

It also sounds like we've come to a conclusion about the long term solution. Thanks @gmlewis and @willnorris. Further discussion about semver and tagging can be resumed in #376. I'll close this issue, but feel to continue discussion there, or open a new issue if there's something specific to address.


logging: Use G_MESSAGES_DEBUG to enable debug & info logging for sysl…

…og handler.
This isn't "the right way" to do this but it's better than the current
state of affairs. This patch disables info and debug messages for the
syslog handler by default. If the G_MESSAGES_DEBUG environment variable
is set to "all" then info and log messages are enabled.
This resolves #30
Signed-off-by: Philip Tricca<EMAIL_ADDRESS>

<3 thanks


Converting a timeSeriesRDD to Spark MLlib's distributed matrices

Added 2 small methods to TimeSeriesRDD that allow users to convert an object of that class to one of Spark's MLlib distributed matrices (specifically IndexedRowMatrix and RowMatrix). This can be useful for spark timeseries users to take advantage of distributed computations, like covariance calculation, and column statistics.

I rebased and pushed the commits separately


Notifications not working

Hello,
I'm new with brunch and task runner :). My config is Win7 64, brunch 1.8.0 and notifications never appear.
I try to :
test on simple project
test with build and watch command
install manually growl for windows
install growl npm https://www.npmjs.com/package/growl
install loggy https://github.com/paulmillr/loggy
I never seen one notifications... But everything else works with brunch!

https://github.com/brunch/brunch/blob/master/docs/config.md#notifications
http://www.growlforwindows.com/gfw/help/growlnotify.aspx

Also another tip from https://github.com/tj/node-growl#windows - make sure you've unpacked the growlnotify files into a folder that's in your PATH environment variable

Little mistake, growlnotify was unpacked in wrong folder... Thank you!


do not install user-identity-mapper

KUBESAW-169
there is no need to install the user-identity-mapper binary locally (together with the ksctl binary), so the PR is moving the user-identity-mapper directory out of cmd folder

having both ksctl and user-identity-mapper under cmd make it clear that we have 2 CLIs

user-identity-mapper is not really a cli that you would run locally on the daily basis. it's more to be used as a container.
Anyway, how would you change the make install command so it installs only ksctl binary? When you run make install now, then it installs ksctl as well as user-identity-mapper, and there is no need to install user-identity-mapper  locally.

having both ksctl and user-identity-mapper under cmd make it clear that we have 2 CLIs

user-identity-mapper is not really a cli that you would run locally on the daily basis. it's more to be used as a container.

well, it's a CLI nonetheless, so it makes sense to keep it under the cmd folder IMO

Anyway, how would you change the make install command so it installs only ksctl binary? When you run make install now, then it installs ksctl as well as user-identity-mapper, and there is no need to install user-identity-mapper locally.

Sure, then it's just a matter of updating the install makefile goal so we only have the ksctl binary when running make install, and have a separate makefile goal used when building the Docker image?

We don't need to build the user-identity-mapper via a makefile target at all - this is done in the Docker image when it's being built:
https://github.com/kubesaw/ksctl/blob/3b26733db38131551bc1fe08ff9dd6853ca50e62/cmd/user-identity-mapper/Dockerfile#L16-L19
so again, how would you change the go install command so it installs ksctl only (using the go native functionality, which means also in the location of either GOBIN or  GOPATH)? I tried that, but I'm doing some mistake there as I cannot achieve the expected outcome, or I'm missing some important flag. Any help is more than welcome :-)

with ${GO_PACKAGE_PATH}/cmd/ksctl/... instead of ${GO_PACKAGE_PATH}/cmd/... seems to work on my machine: I did not have a user-identity-mapper binary after running make install

works for me too! I was about to suggest the same thing.

ok, there was some error between my chair and my keyboard. I was almost 100% sure that I tried that and it didn't built the ksctl binary properly. I maybe omitted the last three dots /... :man_shrugging:  :man_facepalming:
Anyway, thanks for your proposal
Closing this PR...


Validate capabilities in the host execution context

As of #992, we enforce 3DOM capabilities only in the scene graph execution context and not in the host context. We should add validation to the host context to ensure that even if an enterprising script author manages to e.g., manipulate material-properties even when the capability has not been allowed, the host context does not apply those manipulations to the scene graph.

This no longer exists


Process=>protocol field change to metadata

Metadata v6: fields in type/process schemas will be moving to type/protocol schemas on Monday 4 June.
https://humancellatlas.slack.com/files/U6CB89VLJ/FAX897Y8P/HCA_Metadata_Standard_Proposed_Major_Update____process_schema_fields_to___protocol_schemas

@theathorn to ask Metadata group for example bundles, and metadata schema versions that will be used.
Reassign once we have test bundle and metadata schema versions


vswhere.exe problem

ERROR: Could not parse vswhere.exe output
I already installed vswhere.exe from wiki, but there's problem here and I can't install matplotlib

Couldn't parse it in what? What shell are you using? What parameters did you pass? More information is needed.

You should file an issue with contourpy since they are calling vswhere. There is not enough to go on here, and they appear to be calling it incorrectly - likely not passing -utf8, but that's only a guess based on what little a screenshot - no logs or command lines - can tell me. If they think they found an issue they should open an issue with specifics so we can work out the details.


Modelling a 360 degree view lidar sensor to detect obstacles in Mujoco environments

I want to test and compare various state-of-the-art robot navigation algorithms in Mujoco environments. To do so I want to use a 360 degree view laser range finder  as the master sensor for the robot to navigate in obstacle free space. Can anyone please let me know, if such a sensor is already modeled and available off the shelf for Mujoco. It would be of great help.
I agree that Gazebo simulator already provides such a feature, but the environment and robot model that I'm working on is built in Mujoco simulator. So, could anyone please help me out.
I also came across this safetyGym  where they've a mention about providing such sensor off the shelf. Not sure whether that would work. If anyone has already worked on this, it would of great help if you can share your experience.

I also want to make such a design. Have you solved it

Thank you. That's very helpful to me. I like it, but I do not kown how to fix it


Add Free Universal Sound Separation (FUSS) Dataset

🚀 Feature
Add the FUSS dataset to pytorch.datasets.
Motivation
While Pytorch Audio contains a variety of clean speech datasets, it does not contain any noise samples or room impulse response samples, making the setup of problems in speech or audio separation (or simply training recognition in the presence of noise/reverb) harder.
This dataset will potentially be well supported and used, as it is part of DCASE 2020
Pitch
Implement this under torchaudio.datasets, exposing a very similar API to existing datasets. Both ssdata (dry) and ssdata_reverb (reverberated) should be implemented, potentially as different functions.
Alternatives
An alternative would be adding separate datasets for noise samples (e.g. NoiseX-92 if licensing permits) and room impulse responses, in a format that allows for easy mixing by the user into train/validation/test sets, combined with the existing speech datasets already available in the API.

Thanks for suggesting this! Let's take a little bit of time to discuss choice of datasets that would be good to add: #550. Thoughts?

Agree this should be decided in general, will close this.

Since we do not yet have too many datasets, I'll say that, if you are willing to open a pull request implementing this dataset following our current templates, I'll be happy to review it. :)
I'll re-open this issue in case you would like to work on this. Would you be interested in doing so?


Crashes entire OS

I started up Hypermine for the first time and looked around a bit. After just a couple minutes both my screens stopped receiving output from my video card. The computer didn't appear to shut down; I had to do a hard shutdown and restart. I don't see any log files so I'm not sure how to give more information.
I'm going to try leaving some audio playing in the background so that I at least know whether that gets stopped by the crash.

Sadly I haven't reproduced the behavior yet.

System-wide failures are the responsibility of your drivers and/or hardware.

OK, fair point.


Provide option to see queries when sending requests to BigQuery

it'd be pretty cool if we can see the query being sent to bigquery in the tags!

Ideally it would work similar to the PG wrapper: https://github.com/epsagon/epsagon-node/blob/master/src/events/pg.js

Added in https://github.com/epsagon/epsagon-node/issues/178


UI Node's transform is sometimes incorrectly calculated

Context

Bevy 0.13.2
Ubuntu 22.04.4
Resolution 3840x1600

2024-05-26T11:06:44.685619Z  INFO log: Guessed window scale factor: 1
2024-05-26T11:06:44.720176Z  WARN log: InstanceFlags::VALIDATION requested, but unable to find layer: VK_LAYER_KHRONOS_validation
2024-05-26T11:06:44.728709Z  INFO log: Debug utils not enabled: debug_utils_user_data not passed to Instance::from_raw
2024-05-26T11:06:44.740176Z  INFO log: Using X11 platform
2024-05-26T11:06:44.811299Z  WARN log: Detected skylake derivative running on mesa i915. Clears to srgb textures will use manual shader clears.
2024-05-26T11:06:44.811326Z  INFO log: Adapter Vulkan AdapterInfo { name: "NVIDIA GeForce RTX 3080 Laptop GPU", vendor: 4318, device: 9372, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "545.29.06", backend: Vulkan }
2024-05-26T11:06:44.811348Z  INFO bevy_render::renderer: AdapterInfo { name: "NVIDIA GeForce RTX 3080 Laptop GPU", vendor: 4318, device: 9372, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "545.29.06", backend: Vulkan }

What you did
I spawned a UI node that acts as a root for its child nodes. It's attached to the bottom of the screen and full screen width.
cmd
            .spawn((
                Name::new("Portrait dialog root"),
                DialogUiRoot,
                TargetCamera(camera),
                NodeBundle {
                    // centers the content
                    style: Style {
                        width: Val::Vw(100.0),
                        bottom: Val::Px(0.0),
                        position_type: PositionType::Absolute,
                        flex_direction: FlexDirection::RowReverse,

                        ..default()
                    },
                    ..default()
                },
            ))

What went wrong
When I spawn this node for the first time it has about one in five chance that it would have an incorrect transform x calculated (making it invisible.)

Also, I can fix an incorrect spawn by changing the position from absolute to relative and back to absolute again in the egui, after which the value is corrected:

Sounds like a system ordering issue if it's occurring irregularly.

I thought perhaps the fact that I am not setting a height and relying on the content to expand that node might be an issue. Setting min height in the styles did not help though.
What's a good way for me to provide more information? I tried RUST_LOG=bevy_ui=trace but this yields not logs.

I'd really like a minimal reproducible example. Failing that, a reproducible example at all so we can test fixes would be super useful.
Normally I'd tell you to test this upstream in taffy, but this definitely doesn't seem like the issue. Testing it against main (after https://github.com/bevyengine/bevy/pull/10690) might be useful though.

I seperated out the relevant code to this repol. When I run this repo a few times, I encounter the problem.

Run the code
Press spacebar
Close the project
Repeat until one time the UI is spawned but not visible

The culprit seems to be the WorldInspectorPlugin. If I don't add it, I am unable to reproduce the issue after many runs. With it, it's that 1/5 chance.
Now, it's unclear whether merely adding some extra systems increases the odds of this happening due to, as you pointed out, some specific ordering condition, or whether the plugin directly breaks something that bevy_ui calculations rely on.

As in #13155 there is a problem in bevy_ui (or rather in taffy api), when there is only one dimension (width or height) is set with PositionType::Absolute. As workaround you can add height: Val::Px(384.0) in root Style instance, then behavour of your UI becomes much stable.

This workaround is not perfect. The issue is much more rare, but still exists even with it.
I think we should compare more info from root's data
Correct values

Uncorrect values

I think that the most valuable is Node::calculated_size.

I think
pub fn toggle_spawn(
    mut cmd: Commands,
    asset_server: Res<AssetServer>,

    camera: Query<Entity, With<DialogCamera>>,
    root: Query<Entity, With<DialogUiRoot>>,
    windows: Query<&Window, With<PrimaryWindow>>,
) {
    ...
    let width = if let Ok(window) = windows.get_single() {
        Val::Px(window.width())
    } else {
        Val::Vw(100.0)
    };

    let root = cmd
        .spawn((
            Name::new("Portrait dialog root"),
            DialogUiRoot,
            TargetCamera(camera),
            RenderLayers::layer(25),
            NodeBundle {
                // centers the content
                style: Style {
                    width,
                    bottom: Val::Px(0.0),
                    position_type: PositionType::Absolute,
                    flex_direction: FlexDirection::RowReverse,

                    ..default()
                },
                ..default()
            },
        ))
        .id();

would be better temporary workaround

Agree that this is another instance of https://github.com/bevyengine/bevy/issues/13155

I thought perhaps the fact that I was not setting a height and relying on the content to expand that node might have been be the issue. Setting min height in the styles did not help though.

Hmm... this was also my understanding of the issue.

Thanks everyone for their support. Given that the linked issue is very active and the suggested temporary workaround, I will close this and pay attention to #13555. Once 0.14 is released, assuming the fix will be included, I will test this again and reopen in case I observe the behavior.

@porkbrain I think you should reopen this issue, because taffy's ratio blindness is just a part of problem, the issue which we mentioned contains description of bug, that is stable! Every run we get same result, but bug in this issue is floating. Second part (more important) of this problem is that some times node lives in zero-sized environment!
I tried figureout why calculated size of node contains zero and here is results of investigation:

calculated_size gets zero from rounded_size, which gets zero from layout_size, which gets zero from layout, which gets zero from ui_surface, which takes this info from taffy. Why?
Because earlier, before starting this recursion ui_surface feeds taffy layout with available_space, which is set by render_target_resolution, which was taken from camera.size. All of them some times are (0, 0). Why?
Because camera.size was set by (0, 0) earlier from camera.physical_viewport_size(), which gets zeros from physical_target_size(), which gets zeros from camera.computed.target_info.physical_size. Why?
Because camera.computed.target_info is set from new_computed_target_info, which is set from normalized_target under certain conditions. Why this mechanism sometimes does not work correctly?
I didn't figured out yet, but I could try later. I assume that the sequence of launching the systems for spawning camera, attaching it to window and determining it's properties is important here. Maybe @Aceeri or @Weibye can correct me here?

Some experimenting I did a while back is that the resolution sometimes gets set to 0, 0 when minimized and under other certain conditions depending on the platform. But I don't understand why that would stick rather than get updated here.

why that would stick rather than get updated here

Looks like camera fixes it's resolution, but taffy still continues to store zeros in layout despite being given the correct data at step 2

Update:

why that would stick rather than get updated here

The reason is that compute_camera_layout() updates resolution in root_nodes.implicit_viewport_node which containes node for viewport (this is correct, because available_space relates to viewport), but then update_uinode_geometry_recursive() updates geometry starting from user ui root node which is just a child of viewport node. So user ui root node doesn't get updated camera.size. @bardt, shouldn't camera.root_nodes contain node of viewport instead of ui root node?

@alice-i-cecile I did migrate example to main (with #13555 fix) and example became much more stable (I ran ~30 times example to reproduce bug), but bug still persist

@bugsweeper Maybe open a new issue tracking the viewport updating issue


Refactor http method string to constant in go code

Modifications:

Change to use a constant that exists in the official library instead of string value

Thanks for your first-time pull request! Could you sign the ICLA first? https://feedback.line.me/enquete/public/799-Z6ygVz3U

I agreed. 😄

@kkamdooong You have to enter your 'mailing address', not your e-mail address. Could you sign again?

@trustin Oh..I'm sorry. I fixed that.

ICLA confirmed.

Thank you for your first PR!


Suppression

This prototype add support for suppressing rule. The syntax supported in this prototype is
[System.Diagnostics.CodeAnalysis.SuppressMessageAttribute("RuleName", "RuleSuppressionID")]. Any arguments supplied beyond that will be ignored.
This attribute can be used with classes, functions in classes or normal functions. Normal functions need to have param block to use this feature (the attribute will be placed on top of the param block).
To use this attribute at the script level, the script needs to have a param block (again the attribute will be placed on top of the param block).
In this prototype, RuleSuppressionID is only implemented for AvoidUninitializedVariable (of the form PSAvoidUninitializedVariable). If this argument is supplied, then we will only suppress diagnostic record that matches the RuleSuppressionID.

LGTM.

pls merge the pull request. looks good

Thanks. Merged!


Update dependabot.yml

Codecov Report
Patch and project coverage have no change.

Comparison is base (756470a) 100.00% compared to head (e3965e5) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #17   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            7         7
=========================================
  Hits             7         7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Cannot navigate the file using keyboard arrows

VSCode Version: Code - Insiders 1.11.0-insider (27d1dbd4adb0ca1ff1d905430b25580c8934329d, 2017-03-02T11:11:16.467Z)
OS Version: Windows_NT ia32 10.0.10586
Extensions:

Extension
Author
Version

one-monokai
azemoh
0.2.4

path-intellisense
christian-kohler
1.2.0

Theme-monokai-best
gerane
0.0.2

tsimporter
pmneo
1.2.13

vscode-icons
robertohuertasm
7.3.0

todotasks
sandy081
0.4.0

Steps to Reproduce:

Open a file
Indicator is not in the correct position and is not possible to navigate the file using the keyboard's arrows keys

Same issue, no extensions in common.
$ code-insiders --version
1.11.0-insider
c995ecf2e94a00aa7eda81ed842e787b999e83ae
$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=16.10
DISTRIB_CODENAME=yakkety
DISTRIB_DESCRIPTION="Ubuntu 16.10"
$ uname -a
Linux ubuntudev 4.8.0-44-generic #47-Ubuntu SMP Wed Mar 22 14:27:57 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

When I try to set a keymap for "cursorDown", this is what I see when I press the down arrow:

I pressed the down arrow, not enter.
When I press left arrow, right arrow, or up arrow, I see "unknown".

I bisected the versions available:
1.11.0-1490594535 BAD
1.11.0-1490347334 BAD
1.11.0-1490252542 GOOD
1.11.0-1490166151 GOOD
1.11.0-1489993365
1.11.0-1489737822
1.11.0-1489649811 GOOD
1.11.0-1489389218
1.11.0-1489129388
1.11.0-1489044044
1.11.0-1488956710
1.11.0-1488871392
1.11.0-1488784809 GOOD
1.11.0-1488524610
1.11.0-1488439219
1.11.0-1488265408
1.10.0-1488204485
1.10.0-1487968147
1.10.0-1487921388
1.10.0-1487833419
1.10.0-1487747028
1.10.0-1487661178
1.10.0-1487338510
1.10.0-1487314590
1.10.0-1487229188
1.10.0-1487199085 GOOD

@AaronFriel
What keyboard layout do you use? Can you please run F1 > Developer: Inspect Key Mappings, save the output to a file and attach it here.
Is there any output when running code-insiders from the command line in the terminal you launch it from?
In today's insiders build, can you please try:

Open the define keybindings widget (the one you are already using)
Press down arrow
Hover over the input field (this shows the raw values we get from the keydown event)

The first three values for code, keyCode and key are the values coming in from the browser keydown event
The other values are our interpretation of the keypress in the UI, user settings, kb dispatch tables, etc.

I installed 1.11.0-1490775389 from the Ubuntu repository.

What keyboard layout do you use?

Layout info:
{
	"model": "pc105",
	"layout": "us",
	"variant": "",
	"options": "",
	"rules": "base"
}

Can you please run F1 > Developer: Inspect Key Mappings, save the output to a file and attach it here.

Keymappings.txt

Is there any output when running code-insiders from the command line in the terminal you launch it from?

None, should I run it with a command line argument to see some output?

Open the define keybindings widget (the one you are already using)
Press down arrow
Hover over the input field (this shows the raw values we get from the keydown event)
The first three values for code, keyCode and key are the values coming in from the browser keydown event
The other values are our interpretation of the keypress in the UI, user settings, kb dispatch tables, etc.

Down arrow:

Left arrow:

Up arrow:

Right arrow:

The initial enter appears to be stuck as the chord if I press it first, but I often get just unknown, same values though on hover.

In case it's relevant:
LANG=en_US.UTF-8
LANGUAGE=
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_COLLATE="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_PAPER="en_US.UTF-8"
LC_NAME="en_US.UTF-8"
LC_ADDRESS="en_US.UTF-8"
LC_TELEPHONE="en_US.UTF-8"
LC_MEASUREMENT="en_US.UTF-8"
LC_IDENTIFICATION="en_US.UTF-8"
LC_ALL=

@AaronFriel
Thank you very much for following through with this information.
From the Keymappings.txt file, it appears that our C++ node module reads the OS keyboard layout correctly (i.e. what scan code + modifier combo produces what character).
From us printing unknown, it also appears that our code interpretation logic is correct, i.e. we have no clue what to do with the keys Covert, Lang3 and NonConvert.
The problem is that, under your system, Electron/Chromium completely messes up keydown.code at runtime, which I thought is the only trustable value (you can see my analysis in #17521).
Moreover, it looks like keydown.keyCode and keydown.key are correct which is a somewhat good thing. It means we can look into a workaround.
Action for me:
I will add a setting (off by default) which will force keyboard dispatching to use keydown.keyCode. That should serve as a workaround for you or anyone else running into this insane issue.
Action for you:
Since the root cause of this issue appears to be in Electron/Chromium, could you please use Chromium and try the arrow keys at https://jsfiddle.net/v1gz5c5m/1/ . You could also compare the output with Firefox (the output should be the same, i.e. scan codes should match what you press ).
If Chromium prints good scan codes in the jsfiddle, it means that Chromium has already fixed this issue and once we adopt newer versions (we are now on version 53), we should at some point get the fix.
If Chromium prints crazy scan codes, then please consider creating an issue against them at https://bugs.chromium.org/p/chromium/issues/list

"keyboard.dispatch": "keyCode" can be used to forcefully go to keyCode based dispatching.

This is probably not good, right? My Chromium on Linux is also showing weird keycodes on arrow presses.

This is Google Chrome on Windows:

One last quirk of my environment: I am using X11 forwarding. Is that possibly what the issue is? I would guess not, because it's reading the appropriate "key" value, it's the "code" that's very strange. And my locale, keyboard configuration, and so on are identical between the Windows OS and the Ubuntu 16.10 VM.

This appears to be a similar issue with Atom:
[1] https://discuss.atom.io/t/ctrl-arrow-key-sends-unrelated-combination-to-atom/36405/3
[2] https://github.com/electron/electron/issues/8116
But it also looks to be slightly different, their issue seems to be that the code is incorrect when using modifiers. My arrow keys don't work when not using modifiers.

@AaronFriel Having the issue reproducing under Chromium is somewhat of a good thing. It means you are fully equipped to file a bug against https://bugs.chromium.org/p/chromium/issues/list
All that would be needed for you to prove it is a Chromium issue is to run the same test under Firefox and show that Firefox does the right thing. If you create an issue, please link to it here in the comments. Thanks!
P.S. starting with today's insiders you should be able to use "keyboard.dispatch": "keyCode" in user settings to workaround this quirk.

Should "keyboard.dispatch": "keyCode" cause it to work the same as in 1.10? With that setting, I can't repro the issues that I see in 1.10, like https://github.com/Microsoft/vscode/issues/1492

@alexandrudima Could you help me out by describing what a good bug report would look like? It would be very helpful!

I am very confused! Firefox has the same issue?

But if I use the xev utility, I see the right codes?
KeyPress event, serial 29, synthetic NO, window 0x800001,
    root 0x2df, subw 0x0, time 179649750, (429,410), root:(2511,555),
    state 0x0, keycode 104 (keysym 0xff54, Down), same_screen YES,
    XLookupString gives 0 bytes:
    XmbLookupString gives 0 bytes:
    XFilterEvent returns: False

KeyRelease event, serial 32, synthetic NO, window 0x800001,
    root 0x2df, subw 0x0, time 179649843, (429,410), root:(2511,555),
    state 0x0, keycode 104 (keysym 0xff54, Down), same_screen YES,
    XLookupString gives 0 bytes:
    XFilterEvent returns: False

KeyPress event, serial 32, synthetic NO, window 0x800001,
    root 0x2df, subw 0x0, time 179650015, (429,410), root:(2511,555),
    state 0x0, keycode 100 (keysym 0xff51, Left), same_screen YES,
    XLookupString gives 0 bytes:
    XmbLookupString gives 0 bytes:
    XFilterEvent returns: False

KeyRelease event, serial 32, synthetic NO, window 0x800001,
    root 0x2df, subw 0x0, time 179650125, (429,410), root:(2511,555),
    state 0x0, keycode 100 (keysym 0xff51, Left), same_screen YES,
    XLookupString gives 0 bytes:
    XFilterEvent returns: False

KeyPress event, serial 32, synthetic NO, window 0x800001,
    root 0x2df, subw 0x0, time 179650359, (429,410), root:(2511,555),
    state 0x0, keycode 98 (keysym 0xff52, Up), same_screen YES,
    XLookupString gives 0 bytes:
    XmbLookupString gives 0 bytes:
    XFilterEvent returns: False

KeyRelease event, serial 32, synthetic NO, window 0x800001,
    root 0x2df, subw 0x0, time 179650484, (429,410), root:(2511,555),
    state 0x0, keycode 98 (keysym 0xff52, Up), same_screen YES,
    XLookupString gives 0 bytes:
    XFilterEvent returns: False

KeyPress event, serial 32, synthetic NO, window 0x800001,
    root 0x2df, subw 0x0, time 179650718, (429,410), root:(2511,555),
    state 0x0, keycode 102 (keysym 0xff53, Right), same_screen YES,
    XLookupString gives 0 bytes:
    XmbLookupString gives 0 bytes:
    XFilterEvent returns: False

KeyRelease event, serial 32, synthetic NO, window 0x800001,
    root 0x2df, subw 0x0, time 179650812, (429,410), root:(2511,555),
    state 0x0, keycode 102 (keysym 0xff53, Right), same_screen YES,
    XLookupString gives 0 bytes:
    XFilterEvent returns: False

I can even rebind arrows with xmodmap and those show up correctly in the "key" field, but not in the "code". For example, here's every arrow rebound to up:

Note that Chromium and Firefox agree:

keyCode: 38 (up arrow)
key: ArrowUp

But what on earth is code coming from?!

@AaronFriel Even if both browsers show the same weird values for code, I still think this is an issue with their extraction of code, possibly a bug both browsers share...
I am no expert in this topic, but a quick search in Chromium's sources shows how they extract these values from an XEvent:

e.code:
https://cs.chromium.org/chromium/src/ui/events/keycodes/keyboard_code_conversion_x.cc?dr=C&l=882
DomCode CodeFromXEvent(const XEvent* xev) {
  int keycode = (xev->type == GenericEvent)
                    ? static_cast<XIDeviceEvent*>(xev->xcookie.data)->detail
                    : xev->xkey.keycode;
  return ui::KeycodeConverter::NativeKeycodeToDomCode(keycode);
}

e.key
https://cs.chromium.org/chromium/src/ui/events/keycodes/keyboard_code_conversion_x.cc?dr=C&l=905
DomKey GetDomKeyFromXEvent(const XEvent* xev) {
  XEvent xkeyevent = {0};
  XKeyEvent xkey;
  if (xev->type == GenericEvent) {
    // Convert the XI2 key event into a core key event so that we can
    // continue to use XLookupString() until crbug.com/367732 is complete.
    InitXKeyEventFromXIDeviceEvent(*xev, &xkeyevent);
    xkey = xkeyevent.xkey;
  } else {
    xkey = xev->xkey;
  }
  // There is no good way to check whether a key combination will print a
  // character on screen.
  // e.g. On Linux US keyboard with French layout, |XLookupString()|
  //        * Returns '?' for ctrl-shift-/
  //        * Returns '§' for shift-/
  //      According to spec the DomKey for ctrl-shift-/ should also be '§'.
  // The solution is to take out ctrl modifier directly, as according to XKB map
  // no keyboard combinations with ctrl key are mapped to printable character.
  // https://crbug.com/633838
  xkey.state &= ~ControlMask;
  KeySym keysym = XK_VoidSymbol;
  XLookupString(&xkey, NULL, 0, &keysym, NULL);
  base::char16 ch = GetUnicodeCharacterFromXKeySym(keysym);
  return XKeySymToDomKey(keysym, ch);
}

e.keyCode
https://cs.chromium.org/chromium/src/ui/events/keycodes/keyboard_code_conversion_x.cc?dr=C&l=494
// Get an ui::KeyboardCode from an X keyevent
KeyboardCode KeyboardCodeFromXKeyEvent(const XEvent* xev) {
  // Gets correct VKEY code from XEvent is performed as the following steps:
  // 1. Gets the keysym without modifier states.
  // 2. For [a-z] & [0-9] cases, returns the VKEY code accordingly.
  // 3. Find keysym in map0.
  // 4. If not found, fallback to find keysym + hardware_code in map1.
  // 5. If not found, fallback to find keysym + keysym_shift + hardware_code
  //    in map2.
  // 6. If not found, fallback to find keysym + keysym_shift + keysym_altgr +
  //    hardware_code in map3.
  // 7. If not found, fallback to find in KeyboardCodeFromXKeysym(), which
  //    mainly for non-letter keys.
  // 8. If not found, fallback to find with the hardware code in US layout.

  KeySym keysym = NoSymbol;
  XEvent xkeyevent = {0};
  if (xev->type == GenericEvent) {
    // Convert the XI2 key event into a core key event so that we can
    // continue to use XLookupString() until crbug.com/367732 is complete.
    InitXKeyEventFromXIDeviceEvent(*xev, &xkeyevent);
  } else {
    xkeyevent.xkey = xev->xkey;
  }
  KeyboardCode keycode = VKEY_UNKNOWN;
  XKeyEvent* xkey = &xkeyevent.xkey;

  // ....

}

I really have no idea what is really going on, but I do see a discrepancy in handling the xev->type == GenericEvent case. Then again, it might not mean anything. I simply suggest for you to open an issue and follow up with any additional information they might ask from you.

@roblourens Thanks for trying to verify. On OSX, the 1.10 behaviour was more complicated than using keyCode. On OSX, in 1.10, there was some piece of code that would try to "normalize" e.keyCode. However, there was no such code on Linux, so I think one could reproduce a former linux issue by switching "keyboard.dispatch": "keyCode"

@roblourens "keyboard.dispatch" patch works for me, as well.
I am still perplexed by what causes the weird keycode values.


Corrcov typo fix

Fixing typo in recent PR #4089

[x] Closes https://github.com/pydata/xarray/pull/4089#issuecomment-634157768
[x] Passes isort -rc . && black . && mypy . && flake8

Hello @AndrewWilliams3142! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file xarray/tests/test_dask.py:

Line 997:1: W293 blank line contains whitespace

Also, could I ask a git question? Is there a way of getting a "clean" version of xarray to do PR branches off of without just re-forking? I've tried a few different suggestions on StackOverflow but just keen to know what other people's workflow is :) thanks again

thanks, @AndrewWilliams3142.
About git / github: I think the most common workflow is that you have a local branch (commonly named master, but could be anything) that tracks origin/master and a lot of features branches you use to open PRs. If you think there are changes to master you need in your branch (or because Github complains about merge conflicts), you can use
git checkout master
git pull origin master  # could also be `git pull` or `git pull origin`
git checkout <feature-branch>
git merge master

Cheers @AndrewWilliams3142 !


Raise "mapping.total_fields.limit" from the api

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v2 (for Elasticsearch 1.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ x] elastic.v5 (for Elasticsearch 5.x)
Please describe the expected behavior
Raise "mapping.total_fields.limit" from the api.
Please describe the actual behavior
I can't find nothing about in the package.
Any steps to reproduce the behavior?
Randomly create high volume indices.
A reference:
https://discuss.elastic.co/t/total-fields-limit-setting/53004/6
Thank you!

There is no DSL for creating a mapping or a template. You have to pass the mapping either as a serializable JSON struct or a string via BodyJson or BodyString (see here).


TypeError: this.metersReadingData is not a function

Great work on this plug-in, @grzegorz914.
After updating to 5.4.3, I'm seeing this error and my Envoy appears as "unresponsive":
[8/23/2021, 1:31:51 PM] [enphaseEnvoy] Device: <IP_ADDRESS> Envoy, productionCtData error: TypeError: this.metersReadingData is not a function
    at envoyDevice.updateProductionCtData (/home/ubuntu/.nvm/versions/node/v14.17.5/lib/node_modules/homebridge-enphase-envoy/index.js:2936:209)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)

Fixed

Thanks! It's working now.


Deployment Manager Repair API

For status checks on the develop branch, please use TransportPackage-Foundation
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=57248)
and run the build against your PR branch with the default parameters.
For status checks on the main branch, please use microsoft.ProjectReunion
(https://dev.azure.com/ms/ProjectReunion/_build?definitionId=391&_a=summary)
and run the build against your PR branch with the default paramters.
NOTE:
GetStatus doesn't currently check for PackageStatus of the package before returning the package status. It's work for another PR.

/azp run

/azp run

/azp run

/azp run


AutoFocus?

The lack of auto focus keeps us from using this. Does the ZXing control have an AutoFocus method? If so, can you expose it in this control so I can call it periodically? If this is not possible, is there another way to force a focus adjustment?

Hi Roland,
AutoFocus is set by default for IOS and Android Handlers. I'm not sure if anyone wants disable it for use it as a property. Maybe if you want force a focus, change the zoom factor trigger a focus in the camera.
Android example:
previewBuilder.Set(CaptureRequest.ControlMode, Java.Lang.Integer.ValueOf((int)ControlMode.Auto));

Hi Roland,
I have tested the application on IOS/Android and the camera focuses itself every few seconds if you move an object closer or further away. Does it work the same way for you?
I have been looking at the repository that you indicate and the Android AutoFocus function is empty, so it does nothing...

Thanks for looking into this.
Our current app is web based and uses Angular and a ZXing plugin to scan barcodes. We scan high volumes of barcodes in a greenhouse environment. Users complain about how slow the auto focus is. It makes it impossible to use for the high scan volume we have. So we are trying to find another solution.
Your MAUI plugin works as you describe it. It does auto focus for me too, eventually, every few seconds when changing the distance between the barcode and the phone. But it is very cumbersome and too slow. The auto focus of the regular built-in camera is much faster. Not sure why ZXing can't be improved. Being able to tap into the view to focus on that distance (like regular camera apps) would also be helpful or at least expose an auto focus function that actually works.
Thanks for your great work making this work in MAUI but our search for a barcode scanner camera plugin, regardless of dev environment (web, MAUI, Flutter ...) continues.

Hi Roland,
Right now I understand the problem. In the new release (1.3.5) you can find ForceAutoFocus method, it forces the auto focus trigger in the camera. I have not could test it in IOS (at least for the next week). I hope it works fine for you.

Thank you for adding this! I added a button to the view under the camera view. Pushing it calls the ForceAutoFocus  method. It looks like it doesn't do anything. The focus still takes a very long time. Nothing really happens when I call that new method.

Hi Roland,
Could you record a video with the behaviour? I my devices if I have a unfocus preview image and call the function, the camera focus the image.

Is there any news regarding this? We need to put the phone quite far away to get focus but other libs focus very clear on close distance.


OC-451 Using SoapUI as described in User Guild does not work

The User Guid suggest a few soap request that were not up-to-date anymore.

 You should sign our Contributor License Agreement in order to get your pull request merged.

All is well.


Tutorial to production

bring tutorial to production

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

@karstenmelato Are you planning a group with these? How do they relate with existing groups? Also, can we schedule a time to create a set of "progression" missions (beginner, intermediate, advanced) to promote your content as a "lesson plan"? I will review the tutorials in the next few days.

@karstenmelato please update the PR as described above and in separate email -- feel free to reach out to me

I'll close as this is old


feat!: change default of attached_to_untracked to false

FYI, this changed a default, but didn't update the default for attached_to_untracked in README.md (still says true).

Look at the contents of the PR

I'm curious, what's the reasoning behind this change of default? (if there is one)

I got feedback from multiple people that they hated it and I understood their reasoning. Fwiw I'll still continue to use it.


nuScenes validation set detection result

Hello, I am a student working on 3D Multi-Object Tracking. I would like to use your method as the detector for our tracking algorithm, but we need to debug it offline. May I ask if you could open-source the detection results on the nuScenes validation set?

Hello, I am a student working on 3D Multi-Object Tracking. I would like to use your method as the detector for our tracking algorithm, but we need to debug it offline. May I ask if you could open-source the detection results on the nuScenes validation set?

Hello, have you trained this model? I'm using A800 here but still not enough to train. If you've trained, can you give me an answer to the reason for my lack of memory here, thank you

Hello, I am a student working on 3D Multi-Object Tracking. I would like to use your method as the detector for our tracking algorithm, but we need to debug it offline. May I ask if you could open-source the detection results on the nuScenes validation set?

Thanks for your interest in our work. You can generate the result file of detection results on val set by loading the provided checkpoint and inferring on the val set.

Hello, I am a student working on 3D Multi-Object Tracking. I would like to use your method as the detector for our tracking algorithm, but we need to debug it offline. May I ask if you could open-source the detection results on the nuScenes validation set?

Hello, have you trained this model? I'm using A800 here but still not enough to train. If you've trained, can you give me an answer to the reason for my lack of memory here, thank you

Can you provide the specific error information? It is not normal that A800 cannot accommodate MSMDFusion training.


Proposal: Modulo Operation in Austral.Pervasive

Proposal
The following is a proposal for the interface for modulo in austral
typeclass NatModuloOp(T: Type) is
  -- Only implemented for only unsigned integer types as to be entirely unambiguous
  -- remainder function with no footguns
  method umod(lhs: T, rhs: T): T;
end;

typeclass IntModuloOp(T: Type) is
  -- Implemented only for signed types.
  -- Not named mod in order to force the programmer to choose between umod and smod
  -- Implemented as python-style modulo
  method smod(lhs: T, rhs: T): T;
end;

typeclass FloatModuloOp(T: Type) is
  -- Explicit floating point only mod operator performs exactly as specified by IEEE 754.
  method fmod(lhs: T, rhs: T): T;
end;

Strengths

Floating point modulo are separated and cannot be confused with other modulo implementations
umod removes footguns involving a negative lhs and/or a negative rhs
smod makes the programmer think about the implications about using signed modulo and its behavior

Flaws

Aborts on modulo by zero
smod implementation was chosen somewhat arbitrarily by me

Alternatives
Division By Zero
One solution is to require NonZero for the denominator
record NonZero[T: Free]: Free is
  value: T;
end;

typeclass MakeNonZero(T: Free) is
  -- aborts on failure
  method makeNonZeroOrFail(n: T): NonZero[T];
  method makeNonZeroSafe(n: T): Option[NonZero[T]];
end;

Using this would make the type signatures of  umod and smod  be
method smod(lhs: T, rhs: NonZero[T]): T;
method umod(lhs: T, rhs: NonZero[T]): T;
This
Signed Modulo implementation
I am admittedly inexperienced in this area.
The only signed integer modulo handling I am familiar with come from Python and C.
Why I didn't choose the C-style
As far as I understand C's modulo for negative numbers is equivalent to
int smod(int lhs, int rhs) {
  int lhs_sign = (lhs < 0) ? -1 : 1;
  return umod(abs(lhs), abs(rhs));
}

This being so simple to implement yet so rarely used suggests that this is a footgun used unintentionally.
Why I chose the Python-style
This convincing argument.
Default to hardware
We could just default to the architecture's modulo.
I think this is a bad idea as it will create inconsistencies. Additionally, some architectures may not implement it which forces us to decide how it works anyway.
No signed rhs for smod
There are some arguments that a negative number on the rhs is not useful enough to be allowed as it will be used unintentionally.
Austral only allows for typeclasses to have one generic type (for now).
This means that smod would likely need to be implemented as:
method smod(lhs: T, rhs: Nat64): T

Resources from other langs
Golang dev justifies it's modulo
Guido van Rossum mentions it when talking about flooring division
Feedback is welcome :)

I will probably have to study this in a bit more detail, I have probably used modulo like six times in my entire career.

P.S. you could change the spellings of t_div and t_mod. (Or any of the other spellings. Not here to bike-shed.) I know Haskell uses quot and rem respectively.
We could add t_div_mod for regularity.
I didn't discuss error handling much because I'm sure that @eudoxia0 will have an opinion. Personally I've always been happy to write my own wrappers as needed.

Copying one more comment from the Discord: method smod(lhs: T, rhs: Nat64): T is a foot-gun with floor division semantics since smod(-1, big number) is liable to overflow.

After Ada, I think it makes sense to implement rem as truncated remainder. Ada's rem is C's %.


[EN] Feedback for: /web/fundamentals/getting-started/your-first-progressive-web-app/step-02?hl=en

Seems the link : https://weather-pwa-sample.firebaseapp.com/step-04/ does not work.

@petele Quite a few reports in the tracker about this one. Last time I looked into it:
  // Uncomment the line below to test with the provided fake data
  // app.updateForecastCard(fakeForecast);

At the bottom of the app.js file needs updating. Uncomment the app.updateFOrcastCard for it to load properly. Looks like the demo on firebase needs updating, not sure how that is handled. Can you see about getting it done please?

I think this was fixed. In the mean time, try https://codelabs.developers.google.com/codelabs/your-first-pwapp/


config: Fix 'optional' -> 'OPTIONAL' for 'windows'

The shift happened in #574 and the windows entry landed in parallel with #573.  See also #617.

:+1:

LGTM

LGTM


Sending a large amount of USDC results in an L2_BRG: bonderFee must meet minimum requirements error

When sending 100,000 or more USDC from Polygon to L1, an RPC error will occur and the console will read: message: "execution reverted: L2_BRG: bonderFee must meet minimum requirements".
I would expect the bonder fee that is calculated on the frontend to be dynamic based on the input amount.

Closing because this is now out of date.


iot2050-firmware-update: Refactor the update logic

Make the firmware update go with a more reasonable logic and a better prompt

Can you please summarize the new logic as well?

@BaochengSu  All is done.


CASMMON-236 - Grafana Dashboard for Kyverno

Summary and Scope
Monitor Kyverno policy metrics with Prometheus
As part of shasta monitoring, it may benefit you to have monitoring capabilities over both the state and execution of cluster-applied Kyverno policies. This includes monitoring over any applied changes to policies, any activity associated with incoming requests, and any results produced as an outcome. If enabled, monitoring will allow you to visualize and alert on applied policies, and is critical to overall cluster observability and compliance.
Kyverno exporter to get metrics from service monitor. Installation of the dashboard for the kyverno metrics through the .json file.
Issues and Related PRs
https://jira-pro.its.hpecorp.net:8443/browse/CASMMON-236
Tested on:

Mug successfully

Please increase service monitor scrape interval to 30m.