Set config file for systemd nrpe.service

Current version of cookbook does not support change of user/group in OSs that use systemd as default init system, for example CentOS 7.
This patch adds needed functionality.

any updates?

Hi @scopenco
One additional step is required for systemd which is to reload daemon with "systemctl daemon-reload" command if a new systemd service is added / changed / etc. Can you please include this as well.
Can you also think of Ubuntu and Debian latest versions as they use systemd as well?
Also for happy Travis-CI, can you take into account the inline comment I made in your PR?
Thanks
-Suleyman

@scopenco If you are happy with PR #49 can you close this one?
TIA
-SNK

ok


Change Calendar & Program Year References to Dynamic Variables

User Story
As a system architect, I want various forms to automatically update themselves whenever they reference calendar years or Program Years, so I don't have to manually change these in every spot every year.
Current Behaviour:

Supporting Users uses a dynamic field when referencing Calendar Years or Program Years.
Everything else uses static tecxt strings when mentioninig Calendar Years or Program Years. This includes:
Applicaiton Form, questions about income
Request a Change Forms, questions about income

Acceptance Criteria

[ ]  Make all references to Calendar Year dynamic
[ ]  Make all references to Program Year dynamic

I am proposing this be an enhancement; I think it is totally manageable for us to update these 5-8 areas manually via content reviews intead of initiating conversations about how this can be automated. Suitable for enhancement in my opinion.

@HRAGANBC  enhancement for sure. This covers us on those pesky details that need to be updated each year like the "tax year" one.

@CarlyCotton @Joshua-Lakusta  Can we close this one considering we already implemented dynamic update?


Updates content on home page, menus, and scanner landing page

Changes include:

Updates menus to combine "Try" and "Install sections" into one called "Get webhint"
Updates copy on scanner landing page as part of scanner retirement
Updates copy on home page and changes absolute links to relative links
Adds some needed styles for new content

Note: I'll add a follow-up PR in the hint repo to update some of the docs in the User Guide.


[numcpp] Add new port

Describe the pull request

What does your PR fix?
Add new port: https://github.com/dpilger26/NumCpp

Which triplets are supported/not supported? Have you updated the CI baseline?
N/A. No.

Does your PR follow the maintainer guide?
Yes.

Hi @myd7349
Thanks for this PR.
The failure on x64-linux is like his:
-- Installing: /mnt/vcpkg-ci/packages/numcpp_x64-linux/share/numcpp/copyright
-- Performing post-build validation
There should be no empty directories in /mnt/vcpkg-ci/packages/numcpp_x64-linux
The following empty directories were found:

    /mnt/vcpkg-ci/packages/numcpp_x64-linux/share/NumCpp

If a directory should be populated but is not, this might indicate an error in the portfile.
If the directories are not needed and their creation cannot be disabled, use something like this in the portfile to remove them:

    file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/a/dir" "${CURRENT_PACKAGES_DIR}/some/other/dir")

Found 1 error(s). Please correct the portfile:
    /agent/_work/1/s/ports/numcpp/portfile.cmake
-- Performing post-build validation done
Error: Building package numcpp:x64-linux failed with: POST_BUILD_CHECKS_FAILED

Could you please fix it?

Hi @myd7349
Thanks for this PR.
The failure on x64-linux is like his:
-- Installing: /mnt/vcpkg-ci/packages/numcpp_x64-linux/share/numcpp/copyright
-- Performing post-build validation
There should be no empty directories in /mnt/vcpkg-ci/packages/numcpp_x64-linux
The following empty directories were found:

    /mnt/vcpkg-ci/packages/numcpp_x64-linux/share/NumCpp

If a directory should be populated but is not, this might indicate an error in the portfile.
If the directories are not needed and their creation cannot be disabled, use something like this in the portfile to remove them:

    file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/a/dir" "${CURRENT_PACKAGES_DIR}/some/other/dir")

Found 1 error(s). Please correct the portfile:
    /agent/_work/1/s/ports/numcpp/portfile.cmake
-- Performing post-build validation done
Error: Building package numcpp:x64-linux failed with: POST_BUILD_CHECKS_FAILED

Could you please fix it?

Thanks. I will look into it this weekend.

/azp run

Thanks for your contribution!


feat: 🎸 Add chain types for 5.2.x and 5.3.x

Description
Add types spec for 5.2.x and 5.3.x chain
Breaking Changes
NA
JIRA Link
NA
Checklist

[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 9.2.0-alpha.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 9.4.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 9.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Bordesf add colorjitter

Closing and directly merging color jitter code to randaug branch. Not worth doing the greyscale edits just yet


fix 1.20.4 server compat

Merge base version

MC Version: 1.20.4
SpeedRunIGT Version: 13.6

Changes
fix illegal client packet registering on server and sending c2s packets too early

LGTM, I appreciate it :)


remove sklearn dependence

Description
remove the sklearn package and use torch.solve instead.
Motivation and Context
Make the module more robust to multicomponent and large dataset.
How Has This Been Tested?
Test set is updated and use stricter criteria
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds or improves functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation improvement (updates to user guides, docstrings, or developer docs)

Checklist:

[x] My code follows the code style of this project and has been formatted using black.
[x] All new and existing tests passed, including on GPU (if relevant).
[x] I have added tests that cover my changes (if relevant).
[ ] The option documentation (docs/options) has been updated with new or changed options.
[ ] I have updated CHANGELOG.md.
[x] I have updated the documentation (if relevant).

@nw13slx looks like only some of the random generations use generator here? did some commits not push?


[local_auth] Flutter package

Hi all the newest package of local_auth Flutter package only include BiometricType.strong or BiometricType.weak.
Is there anyway to detect BiometricType.Face and BiometricType.Fingerpint ?
Both BiometricType.Face and BiometricType.Fingerpint are not accessible in the new package

Hi @desmondlee95700,

BREAKING CHANGE: Enum BiometricType has been expanded with options for strong and weak, and applications should be updated to handle these accordingly.

There is a Breaking Change since 2.0.0 and it seems it is missing the relevant documents for that change, see https://github.com/flutter/flutter/issues/102164#issuecomment-1119308677

and it seems it is missing the relevant documents for that change

That comment is outdated; see https://pub.dev/packages/local_auth#enrolled-biometrics (and the section just above that).

the newest package of local_auth Flutter package only include BiometricType.strong or BiometricType.weak.

This is not accurate; the other types still exist, and are returned on iOS.

Is there anyway to detect BiometricType.Face and BiometricType.Fingerpint

The package no longer exposes deprecated APIs on Android. You could certainly use platform channels to query the deprecated APIs in your own application if you want to continue using an approach that the platform explicitly discourages, and anyone in the community could release a plugin to do so.

Thanks for all the replies. Let's say android have fingerprint id and face id enabled. How do the biometric.strong detect fingerprint id and face id?

I'm not sure what the question is exactly. If you're asking how Android classifies authentication mechanisms into classes you'd need to consult Android documentation.


Remove unclear libp2p section

Telegram user @bonetrader reported:
https://docs.gny.io/run-bare-metal-node/configure.html#configure-public-ip

Closed with PR: #77

Closed with PR: #77


Support opening Forge-based modded instances

Howdy! I have no idea how this would work, but it would be awesome to support opening Forge instances with native Apple silicon support!

I have same question.

Hey, sorry I haven't worked on this yet. Been very busy. It is still something I'd like to address but in the meantime I'd recommend other programs for this type of thing.
Especially now that no changes are needed other than LWJGL and Java version, it's a lot easier to do it manually as well.

See #10


Telegram bridge: empty entries for posts with text formatted as a quote

Describe the bug
Telegram has a "quote" type of text formatting  for some time now. Posts that contain text with this formatting appears in the feed as an empty entry with "[no title] [no content]"
To Reproduce
Steps to reproduce the behavior:

Subscribe to any public Telegram channel
Wait for a post with the text formatted as a quote

give example pls

This post, for example, contains a couple of such quotes
https://t.me/bbcrussian/68899

and the feed entry looks like .

for unknown resons, the content is omitted from tg preview page: https://t.me/s/bbcrussian/68899


Feature/ars 430 submit answers

This PR:

Implements the application submission
Tweaks the application complete page in line with prototype

Currently, a UUID with the hyphens removed is used as the application number but the suitability of  this will be reviewed and amended in the next PR.


Fix exported typed dictionaries in .NET having an incorrect hint

Cherry-picked and amended based on https://github.com/godotengine/godot/pull/97948#discussion_r1797545874
The other fixes in #97948 need more work, but this commit should be ready to be evaluated independently.

Thanks!


Reduce size of assets for better performance

Is your feature request related to a problem? Please describe.
Images low too slow, even on my fiber connection.
Here's Netlify complaining:

11:06:10 PM: asset size limit: The following asset(s) exceed the recommended size limit (244 KiB).
11:06:10 PM: This can impact web performance.
11:06:10 PM: Assets:
11:06:10 PM:   img/background.b785fbde.png (794 KiB)
11:06:10 PM:   fonts/materialdesignicons-webfont.ff13d121.woff (312 KiB)
11:06:10 PM:   fonts/materialdesignicons-webfont.541e65fb.eot (648 KiB)
11:06:10 PM:   fonts/materialdesignicons-webfont.fc03f7f1.ttf (647 KiB)
11:06:10 PM:   css/chunk-vendors.60695ae6.css (558 KiB)
11:06:10 PM:   js/chunk-vendors.c7500ba0.js (490 KiB)
11:06:10 PM:   banner.png (319 KiB)
11:06:10 PM:   banner_ja.png (339 KiB)

Describe the solution you'd like

Reduce the size of the images, but not too much. Keep the quality high.
Can't do much about reducing materialdesignicons, that's a bundle.

Message me for the link to design assets if you work on this.


Cards neon

Description 📜

Added neon themed various text cards.
CSS can be copy-pasted and used directly in any future projects

This would be great if you provide any of this   📝

[x] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[x] Animation design
[x] Others

Checklist ✅
]

[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm Hacktoberfest contributor

Screenshots / Gif (Compulsory) 📸

good work @sanskaromar
this is something which stands out from other contributions :)


Bugfix: Bump addon-framework and mute health prober

Besides, this PR might need a rebase after #43 is merged.

@Somefive reverted go mod to 1.17 by sticking w the last commit of ocm libraries at 1.16


fix: remove categorize page from navigation stack

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/45410
PROPOSAL: https://github.com/Expensify/App/issues/45410#issuecomment-2228825703
Tests

Prerequisite: There is at least one workspace.

Go to self dm.
Create any track expense if there is no.
Choose "Categorize it" in actionable whisper.
Select any workspace. Then the category page opens.
Select any category option. Then the confirmation page opens.
Click "back" button.
Verify The participant page opens.

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/d960f4c4-24af-4cd8-9ec3-907b79e43fb0

MacOS: Desktop

https://github.com/user-attachments/assets/d960f4c4-24af-4cd8-9ec3-907b79e43fb0

If the user doesn't have a workspace this flow gets weird, because the workspace is gonna be created in the last step and there is nothing to select.
https://github.com/user-attachments/assets/d33f30c5-059e-456d-a47a-88e736cde33a

The best thing would be to close the entire RHP since there is nothing to be done there.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/rlinoz in version: 9.0.11-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/francoisl in version: 9.0.11-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/francoisl in version: 9.0.11-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 9.0.12-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Lazy load not working properly when 'items' is set to less than 2

Hi there, I am trying to do an almost full page slider with lazy images.
I am setting my 'items' to 1.2 for instance, and this is where the trouble arises.
The lazyload loads the images for the current slide and for the previous slide but not for the next one.
Whenever i set items to more than 2, it works perfectly. But I need it to work with less than 2.
Here is a jsfiddle: http://jsfiddle.net/k3mvLs0k/
Would be great with some input on this issue, thanks!

I believe the value of the items option was intended to be of type integer. You might have fewer issues if you use a whole number ... however, the real problem here is that you have the option: center:true ... change that false and you should work.
The logic behind the lazy loader is that it should defer loading until the image is in view. With center:true and items < 2 it thinks the slide is only halfway into view ...so, it shouldn't load it yet.
// line 1855
 n = (settings.center && Math.ceil(settings.items / 2) || settings.items),

Aha, but the problem is that if I don't set center:true then i can't see the edges of the previous and the next images, which is the effect I want. With center:false I can only see the edges of the next, and not the previous as well.
With items set to 2.01 or anything else > 2 (not only integers) and with center:true, it will preload images that are only partly visible – it would be nice to have it work the same way with items set to < 2.

@witrin's prefretch option should help you. Copy the code from here and paste it over the lazyload section of your owl.carousel.js file.
You'll have to set the lazyPrefetch option to 2 or more.

Perfect! That works. Thank you very much.

Glad I could help .... that prefetch mod resolved an issue I was having as well. great stuff!

Thanks again, @aleroy


[fix] use domain as message-id instead of internal hostname

coverage: 82.85%. first build when pulling 0f9cdde437ae124b35bc8dc1df6eb513f963561e on fix-message-id into 1985d5e81bdda76a524121b127cd83b1007d8097 on master.


Email should be required for PowResetPassword extension

Hi @danschultzer,
When trying to reset password, the first step form that asks for the user's email shouldn't accept empty email input submit. I tried to require the email field in my app User schema changeset but the extension just ignores it.
Currently when submiting empty email there is no error message, just a redirection with a flash message confirming that an email was sent with password reset instructions.
Also a fake email submit will have the same behaviour, though I can understand in this case that it is maybe for security concern.
But at least shouldn't we prevent user from submiting empty email input.
Is there a way to fix that I'm unaware of?

Also a fake email submit will have the same behaviour, though I can understand in this case that it is maybe for security concern.

Yeah, it shouldn't leak information whether an account exists or not based on the e-mail. So regardless if the reset password succeeds or not the same success message returns. This means that also empty input will return success response.
If you got custom templates, then you can just add a HTML5 require attribute to the email field, or maybe even add email type for email validation.
I don't see any easy way of handling this in Pow. The controller just passes on the params and the context logic only returns an nil if the user can't be found. The PowResetPassword.Plug module would probably be the place to handle this, but I think it might make the logic more complex than it needs to be.

Thanks I think the custom templates with HTML5 require attribute to the email field should be fine. The email validation already works. thanks


PaginationFilter dropdown not placed properly and must press hold to show dropdown

Description
PaginationFilter dropdown location not located properly and not toggled, must pressed hold.

Expected result
Dropdown located properly based on aside props and toggle on open dropdown

@IzzanNurdin
can you make sample code to reproduce this bug ?

make sure dont use margin top at body level or outer layout. use padding-top instead


Add support for configuring S3 addressing style

The environment variable S3GW_S3_ADDRESSING_STYLE supports the following options:

auto
path
virtual
It defaults to auto.

Additional enhancements:

Use a more detailed error message in case of an endpoint connection error.
Improve several unit tests.

Fixes: https://github.com/aquarist-labs/s3gw/issues/770

The new EndpointConnectionError response looks like

which helps to identify the real problem much easier that a simple Endpoint not found.


support postgresql create index syntax

The postgresql CREATE INDEX statement uses the USING clause to indicate the indexing method, e.g.
CREATE INDEX myindex1 ON mytab USING GIST (mycol)
This PR adds support for this syntax.
The PR passes all tests, and a test has been added for the USING syntax.
testCreateIndexIssue633 in CreateIndexTest.java will likely now pass.
Most other SQL dialects indicate the indexing method in the parameters between CREATE and INDEX
CREATE (parameter)* INDEX
and in some cases, there can be two parameters, e.g. in TransactSQL one can say
CREATE Unique Clustered INDEX
The current code will only pick up "Unique" for Index.type and will drop "Clustered".
In general, Index.type should be List

Coverage increased (+0.02%) to 83.832% when pulling 9d74c6da03976eae8ce34e8403fcf4520031d109 on ted-johnson:index5 into 83dbac2d9841d21edfa0ad1e37763a84d381932a on JSQLParser:master.

Coverage increased (+0.02%) to 83.832% when pulling 9d74c6da03976eae8ce34e8403fcf4520031d109 on ted-johnson:index5 into 83dbac2d9841d21edfa0ad1e37763a84d381932a on JSQLParser:master.


common: fix a precision issue in appendArc api

For sweep angles very close to a multiple of pi/2, an extra cubic curve segment was added, which was
populated with NaN values due to finite precision limits.

sample that didn't work on my machine:
    auto shape1 = tvg::Shape::gen();
    shape1->appendArc(150, 250, 180, 350, 90.000012, true);
    shape1->strokeFill(255, 0, 0);
    shape1->strokeWidth(13);
    if (canvas->push(std::move(shape1)) != tvg::Result::Success) return;

    auto shape2 = tvg::Shape::gen();
    shape2->appendArc(150, 350, 180, 360, 90.000012, true);
    shape2->strokeFill(255, 255, 255);
    shape2->strokeWidth(2);
    if (canvas->push(std::move(shape2)) != tvg::Result::Success) return;

before:
the first arc (he second one contained NaN values and caused the program to hang):

after:

another example:
    auto shape3 = tvg::Shape::gen();
    shape3->appendArc(600, 450, 180, 10, 0.000002, true);
    shape3->strokeFill(255, 255, 255);
    shape3->strokeWidth(2);
    if (canvas->push(std::move(shape3)) != tvg::Result::Success) return;

before:

after:


How to disable this plugin by Javascript

Currently we have some customers reporting that this plugin can't open a Security Encrypted PDF so that I would like to use Javascript to check if this Plugin is enabled to guide them to disable this.

Duplicate of #8834


Registration

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/74820
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.0.1 -m "<description of version>" 35c8e85a384e7baee025016afa4df4d6eb23b2da
git push origin v0.0.1


DLL not found lbffmpeg.dll

Describe the bug
When I run the player it says DLL not found when I try to play a stream
To Reproduce
Steps to reproduce the behavior:

Open player
Enter valid stream
Press start

Expected behavior
The stream plays
Desktop (please complete the following information):

OS: Wndows 10

Additional context
I've tried adding libffmpeg to the path so I don't think that's the issue

I did have to retarget the libffmpeg helper project as well

There is another issue already open for this


Faster NaN checks

Thanks to @mikmoore  (https://discourse.julialang.org/t/fastest-way-to-check-for-inf-or-nan-in-an-array/76954/33?u=milescranmer)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
984

Relevant Lines:
1109

💛  - Coveralls

Hm, probably worth having a larger n evaluation benchmark. Doesn’t seem to be noticeable at the current size

Maybe I should create a case-switch over array size, and get the optimal unroll length accordingly.

Seem to get LoopVectorization.jl segfaults on Windows. Should probably disable it there.

Closing as LoopVectorization is deprecated.


Server returned HTTP response code: 403 for URL

Hi Team,
I have setup the sample code by following the instructions from the document link. When I run the producer command, I see the below error.
INFO Creating the topic kafka-java-console-sample-topic (com.eventstreams.samples.EventStreamsConsoleSample)
ERROR REST POST request failed with exception: java.io.IOException: Server returned HTTP response code: 403 for URL: https://xxxx.containers.appdomain.cloud/admin/topics (com.eventstreams.samples.rest.RESTRequest)
java.io.IOException: Server returned HTTP response code: 403 for URL: https://xxxx.containers.appdomain.cloud/admin/topics
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(Unknown Source)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(Unknown Source)
at java.lang.reflect.Constructor.newInstance(Unknown Source)
at sun.net.www.protocol.http.HttpURLConnection$10.run(Unknown Source)
at sun.net.www.protocol.http.HttpURLConnection$10.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at sun.net.www.protocol.http.HttpURLConnection.getChainedException(Unknown Source)
at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(Unknown Source)
at sun.net.www.protocol.http.HttpURLConnection.getInputStream(Unknown Source)
at sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(Unknown Source)
at com.eventstreams.samples.rest.RESTRequest.post(RESTRequest.java:162)
at com.eventstreams.samples.rest.RESTAdmin.createTopic(RESTAdmin.java:46)
at com.eventstreams.samples.EventStreamsConsoleSample.main(EventStreamsConsoleSample.java:176)
After doing some research over the internet, I also tried adding the code in
connection.setRequestProperty("User-Agent", "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.67 Safari/537.36");
in the RestRequest, but still no luck.
I get response from the browser, but not from the java code. Please help and let me know what am I missing here.

Closing this issue as there's been no reply for a few weeks.
Please re-open with the requested data if this is still an issue.


Add more image operations

Adds 2 more image operations (Sharpen image and Convert image format).
Sadly Jimp doesn't support outputting as GIF so we're limited to JPEG, PNG, BMP and TIFF.

Ready to merge this. We just need to fix the conflict and I'm also getting a bug in the 'Add text to image' op:
jimp.js:34740 Uncaught (in promise) Error: HTTP Status 404 for url http://localhost:8080/assets/images/Roboto72White.png
    at XMLHttpRequest.eval (jimp.js:34740)


Fix account history contract in the simple bank account application

Description
There is a bug in the account-history contract in the simple bank account application, and it does not display the asset data as follows:
❯ ../scalardl-java-client-sdk/client/bin/scalardl execute-contract --properties ../conf/client.properties --contract-id account-history --contract-argument '{"id": "a111"}'
Contract result:
{
  "status" : "succeeded",
  "history" : [ {
    "id" : "a111",
    "data" : "",
    "age" : 1
  }, {
    "id" : "a111",
    "data" : "",
    "age" : 0
  } ]
}

When I updated the application in #64, I overlooked this bug because the document doesn't give any instructions on how to use the account-history. So I've also added the instruction for running the account-history to the document in addition to the bug fix.
Related issues and/or PRs

https://github.com/scalar-labs/scalardl/pull/61

Changes made

Fix the bug in the account-history contract
Update the document to describe how to use the account-history contract

Checklist

[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have updated the documentation to reflect the changes.
[x] Any remaining open issues linked to this PR are documented and up-to-date (Jira, GitHub, etc.).
[x] Tests (unit, integration, etc.) have been added for the changes.
[x] My changes generate no new warnings.
[x] Any dependent changes in other PRs have been merged and published.

Additional notes (optional)
N/A
Release notes
N/A

I've confirmed that all steps related to contract executions have been executed successfully.
applications/simple-bank-account/contract on  fix-account-history-contract-in-sample-app [!?] via 🅶 via ☕ v21.0.4
❯ ${SCALAR_SDK_HOME}/client/bin/scalardl execute-contract --properties ../conf/client.properties --contract-id create-account --contract-argument '{"id": "a111"}'
Contract result:
{
  "status" : "succeeded",
  "message" : "account a111 created"
}

applications/simple-bank-account/contract on  fix-account-history-contract-in-sample-app [!?] via 🅶 via ☕ v21.0.4 took 6s
❯ ${SCALAR_SDK_HOME}/client/bin/scalardl execute-contract --properties ../conf/client.properties --contract-id create-account --contract-argument '{"id": "b222"}'
Contract result:
{
  "status" : "succeeded",
  "message" : "account b222 created"
}

applications/simple-bank-account/contract on  fix-account-history-contract-in-sample-app [!?] via 🅶 via ☕ v21.0.4 took 6s
❯ ${SCALAR_SDK_HOME}/client/bin/scalardl execute-contract --properties ../conf/client.properties --contract-id deposit --contract-argument '{"id": "a111", "amount": 100}'
Contract result:
{
  "status" : "succeeded",
  "old_balance" : 0,
  "new_balance" : 100
}

applications/simple-bank-account/contract on  fix-account-history-contract-in-sample-app [!?] via 🅶 via ☕ v21.0.4 took 6s
❯ ${SCALAR_SDK_HOME}/client/bin/scalardl execute-contract --properties ../conf/client.properties --contract-id transfer --contract-argument '{"from": "a111", "to": "b222", "amount": 100}'
Contract result:
{
  "status" : "succeeded",
  "from_old_balance" : 100,
  "from_new_balance" : 0,
  "to_old_balance" : 0,
  "to_new_balance" : 100
}

applications/simple-bank-account/contract on  fix-account-history-contract-in-sample-app [!?] via 🅶 via ☕ v21.0.4 took 6s
❯ ${SCALAR_SDK_HOME}/client/bin/scalardl execute-contract --properties ../conf/client.properties --contract-id account-history --contract-argument '{"id": "a111"}'
Contract result:
{
  "status" : "succeeded",
  "history" : [ {
    "id" : "a111",
    "age" : 2,
    "data" : {
      "balance" : 0
    }
  }, {
    "id" : "a111",
    "age" : 1,
    "data" : {
      "balance" : 100
    }
  }, {
    "id" : "a111",
    "age" : 0,
    "data" : {
      "balance" : 0
    }
  } ]
}

@jnmt

Thank you for the update! I left one request. Can you handle it when you have time?

BTW, I cannot see your request. Could you please check if your comment applied properly on GitHub?

@KodaiD Sorry, I probably made a mistake. My request was related to the document part. Since we have moved the documents to the docs-internal-scalardl, can you update it in the docs-internal-scalardl instead of this repository to reflect the changes properly?

@jnmt Thanks for reviewing! I've removed the update in README.md in this repo. I've updated the documentation in doc-internal (in https://github.com/scalar-labs/docs-internal-scalardl/pull/226).


Fix: Not compatible with --nolegacy_external_runfiles

This PR aims for fixing issue #74.
Tested on M1 Mac 13.4.1 and x86_64 Ubuntu 20.04.6 LTS with buildifier-prebuilt <IP_ADDRESS> under Bazel 6.3.2, works either with --nolegacy_external_runfiles or not.

https://github.com/keith/buildifier-prebuilt/pull/69/files#r1260398463
i think it will fail ci on windows given this comment

Thanks for the comment :)
Now the CI checks are complete and all three tests are passed. Also, from this line in the same commit you mentioned, we can know that the out_ext would be .bash for non-windows platform, so that this PR will not affect Windows users.

thanks!


Skewed Arrows, 0.7.X support and more!

This is a big work inside modcharting tools (adding more modifiers, a tutorial for lua and a skew sprites [simulates 3d somehow])
Huge thanks to Glowsoony for the help with 0.7.x and Skewed Sprite fix!

Yo gonna use this! But what about specifying the version? Since there are 2 version inside the source folder, I'm using psych 6.3

Yo gonna use this! But what about specifying the version? Since there are 2 version inside the source folder, I'm using psych 6.3

use the 0.6.3 folder then

Please note that this PR is a test! Skewed notes are still being tested on to see the limits it may have! so far its already looking complete from the code me and Edwhak have seen. Please know 0.7.x is still experimental and I am testing that Version while Edwhak is testing 0.6.3 and so far skewed notes are a major success.

hey sustain note is invisible help

Haxelibs can have their own import.hx files iirc

e

I have a problem. Error with compiling.
C:/HaxeToolkit/haxe/lib/fnf-modcharting-tools/git/source/modcharting/Modifier.hx:13: characters 8-17 : Type not found : PlayState C:/HaxeToolkit/haxe/lib/fnf-modcharting-tools/git/source/modcharting/Modifier.hx:14: characters 8-12 : Type not found : Note C:/HaxeToolkit/haxe/lib/fnf-modcharting-tools/git/source/modcharting/PlayfieldRenderer.hx:27: characters 8-17 : Type not found : PlayState C:/HaxeToolkit/haxe/lib/fnf-modcharting-tools/git/source/modcharting/PlayfieldRenderer.hx:28: characters 8-12 : Type not found : Note C:/HaxeToolkit/haxe/lib/fnf-modcharting-tools/git/source/modcharting/PlayfieldRenderer.hx:42: characters 24-33 : Type not found : StrumNote C:/HaxeToolkit/haxe/lib/fnf-modcharting-tools/git/source/modcharting/NoteMovement.hx:13: characters 8-17 : Type not found : PlayState C:/HaxeToolkit/haxe/lib/fnf-modcharting-tools/git/source/modcharting/NoteMovement.hx:14: characters 8-12 : Type not found : Note C:/HaxeToolkit/haxe/lib/fnf-modcharting-tools/git/source/modcharting/ModchartUtil.hx:14: characters 8-17 : Type not found : PlayState C:/HaxeToolkit/haxe/lib/fnf-modcharting-tools/git/source/modcharting/ModchartUtil.hx:15: characters 8-12 : Type not found : Note source/psychlua/FunkinLua.hx:56: characters 17-30 : Expected }

Uhh my sustains are invisible

### Some things to know. (For Psych ONLY)

If your on 0.7.x (0.7 or higher) turn off GPU Caching, this is not a bug, what happens is that GPU Caching destroys the image and re builds it, storing the memory on GPU instead of CPU, turn it off.
2.Check what version you put in you project.xml! define the version as  the value is the version your on so it would be "0.6.3" or whatever version your on! also be sure to define the variable "PSYCH" as

wait, variable backupGpu doesn't work at all?

In the editor if it's not visible Im check that out since it should be, but if neither in playstate nor modcharteditor then it's a bug (in the case that GPU rendering is on) but if it's not working in the editor try with it off and if it works then I'll check that out

firstly i had them invisible in playstate, but when i entered editor they became visible even in playstate

If they became visible after entering the editor then I see, it is the fact GPU Rendering is turned on, the Editor has support for turning it off but by default PlayState won't clearly since the support is only for the editor.

weird

also question i'm kinda dumb lol, how skew modifier works lol, i think i don't understand smth

SkewModifier really doesn't work sometimes or Idk if it works cause I find that sometimes its depending on a few things, but Skew works by applying a skew to the receptors such as looking like its bending forward

also question i'm kinda dumb lol, how skew modifier works lol, i think i don't understand smth

Hey, im Mt creator and i'll tell, Skew modifier its a modifier that uses subValues (X.Y,XD,YD) so in order to properly make it work you should first enable the base modifier
(have in mind it should already been created)
lua should be:
set(beat, [[
1,skewMod
]])
in editor you go to events, pick the skew modifier and add value 1 (add time to 0 to instantly do it since "SET" in editor doesn't work properly yet)
then you are able to modify sub values


toLower breaks parameters with allowed value validation

Bicep version
Bicep CLI version 0.4.613 (d826ce8411)
Describe the bug
Passing a string through toLower() before passing it as a parameter with allowed values results in an error
To Reproduce
Define the parameter as follows
@allowed([
  'linux'
  'windows'
])
param operatingSystem string

Then pass a value to it as follows.
  params: {
    operatingSystem: toLower(appServicePlan.operatingSystem)
  }

This results in the error Error BCP036: The property "operatingSystem" expected a value of type "'linux' | 'windows'" but the provided value is of type "string".

+1

Closing as dup of #4134


Improved fusion regression

Improved fusion regression by:

Modified fusion yaml files to include regression yamls and run-name
Adjusted fuse-regression script accordingly
Added error handling for missing run-files

If well, should address https://github.com/castorini/anserini/issues/2619

@Stefan824 can you please fix the conflict?


fix(alerts): check if the policy file is available

Fixes NR-108091

I'm thinking it may suffice to check permissions on the /tmp directory only a few times...once for removing files, and again for file creation.  Since this is all in the same task, we probably only need to do it once per operation - so checking on the removal/write for /tmp/policy.json may be all we need
https://github.com/newrelic/open-install-library/blob/36677e6362a2d41cbcda58e21a4ff26c653f075f/recipes/newrelic/infrastructure/alerts/golden.yml#L73-L86
or to be certain on /tmp/condition.json add an exit if we cannot delete, keep the exit 2 you already have , but then remove the remaining exit 2 checks from subsequent condition.json file interactions
https://github.com/newrelic/open-install-library/blob/36677e6362a2d41cbcda58e21a4ff26c653f075f/recipes/newrelic/infrastructure/alerts/golden.yml#L114-L127


Remove old references

remove pre 0.7 tekton tasks
remove from readme references to old version of tekton and ocp3

Close by #74


cmd/go: tests failing on linux-amd64-longtest

Several tests are failing:

TestScript/mod_get_pseudo_other_branch
TestScript/mod_get_pseudo_prefix
TestScript/cover_pkgall_multiple_mains

https://build.golang.org/log/a96b4779e564bb3bae7d9495a6088b0d9018ad9c

@cherrymui Could you take a look at the failure with go test cmd/go -run=TestScript/cover_pkgall_multiple_mains? A bisect indicates it was broken by CL 174657. For reference, the test case was added for CL 164877 and CL 168200. Those CLs fixed a link error that showed up when multiple main packages are linked into the same executable, which is possible with -coverpkg=all.

@jayconrod thanks for catching this.
I'm not very familiar with -coverpkg=all, so I'm not sure I understand how it supposes to work. Running go test -coverpkg=all -vet=off -x -work ./mainonly ./xtest (disabled vet for simplicity) in the tree of cover_pkgall_multiple_mains test, I got
...
cat >$WORK/b009/importcfg.link << 'EOF' # internal
packagefile example.com/cov/xtest.test=$WORK/b009/_pkg_.a
packagefile os=$WORK/b011/_pkg_.a
packagefile testing=$WORK/b025/_pkg_.a
packagefile testing/internal/testdeps=$WORK/b041/_pkg_.a
packagefile runtime=$WORK/b002/_pkg_.a
packagefile internal/cpu=$WORK/b004/_pkg_.a
packagefile internal/bytealg=$WORK/b003/_pkg_.a
packagefile runtime/internal/atomic=$WORK/b006/_pkg_.a
packagefile runtime/internal/sys=$WORK/b008/_pkg_.a
packagefile runtime/internal/math=$WORK/b007/_pkg_.a
packagefile example.com/cov/mainonly=$WORK/b054/_pkg_.a
packagefile internal/errinternal=$WORK/b013/_pkg_.a
packagefile internal/reflectlite=$WORK/b014/_pkg_.a
packagefile errors=$WORK/b012/_pkg_.a
packagefile internal/race=$WORK/b019/_pkg_.a
packagefile sync/atomic=$WORK/b020/_pkg_.a
packagefile sync=$WORK/b018/_pkg_.a
packagefile io=$WORK/b017/_pkg_.a
packagefile unicode=$WORK/b027/_pkg_.a
packagefile unicode/utf8=$WORK/b028/_pkg_.a
packagefile bytes=$WORK/b026/_pkg_.a
packagefile math/bits=$WORK/b034/_pkg_.a
packagefile math=$WORK/b033/_pkg_.a
packagefile strconv=$WORK/b035/_pkg_.a
packagefile reflect=$WORK/b032/_pkg_.a
packagefile sort=$WORK/b036/_pkg_.a
packagefile internal/fmtsort=$WORK/b031/_pkg_.a
packagefile internal/oserror=$WORK/b015/_pkg_.a
packagefile syscall=$WORK/b021/_pkg_.a
packagefile time=$WORK/b022/_pkg_.a
packagefile internal/poll=$WORK/b016/_pkg_.a
packagefile internal/syscall/unix=$WORK/b023/_pkg_.a
packagefile internal/testlog=$WORK/b024/_pkg_.a
packagefile strings=$WORK/b037/_pkg_.a
packagefile fmt=$WORK/b030/_pkg_.a
packagefile flag=$WORK/b029/_pkg_.a
packagefile runtime/debug=$WORK/b038/_pkg_.a
packagefile context=$WORK/b040/_pkg_.a
packagefile runtime/trace=$WORK/b039/_pkg_.a
packagefile example.com/cov/xtest=$WORK/b055/_pkg_.a
packagefile example.com/cov/xtest_test=$WORK/b056/_pkg_.a
packagefile bufio=$WORK/b042/_pkg_.a
packagefile regexp=$WORK/b043/_pkg_.a
packagefile runtime/pprof=$WORK/b045/_pkg_.a
packagefile regexp/syntax=$WORK/b044/_pkg_.a
packagefile compress/gzip=$WORK/b046/_pkg_.a
packagefile encoding/binary=$WORK/b048/_pkg_.a
packagefile io/ioutil=$WORK/b051/_pkg_.a
packagefile text/tabwriter=$WORK/b053/_pkg_.a
packagefile compress/flate=$WORK/b047/_pkg_.a
packagefile hash/crc32=$WORK/b049/_pkg_.a
packagefile path/filepath=$WORK/b052/_pkg_.a
packagefile hash=$WORK/b050/_pkg_.a
EOF
cd .
/Users/cherryyz/src/go/pkg/tool/darwin_amd64/link -o $WORK/b009/xtest.test -importcfg $WORK/b009/importcfg.link -s -w -buildmode=exe -buildid=cFSniVrc4KAyuUkCTFpY/TaDY2tf6UZUCfoMX5CGy/WvpKoGw2_XEdpH0o4Z9v/cFSniVrc4KAyuUkCTFpY -extld=clang $WORK/b009/_pkg_.a
# example.com/cov/xtest.test
2019/05/09 17:23:12 duplicate symbol main.main (types 1 and 1) in main and $WORK/b054/_pkg_.a(_go_.o)

This is linking xtest.test, but I don't understand why it links example.com/cov/mainonly into it (the packagefile example.com/cov/mainonly=$WORK/b054/_pkg_.a line above).
If linking this in is actually necessary, could the go command pass -p example.com/cov/mainonly when compiling  mainonly.cover.go instead of -p main?

Experimented a little more, with a module m, which has three main packages, m1, m2, m3, each with test,

Doing go test -coverpkg=all ./... would build three binaries m1.test, m2.test, m3.test. Each of them is trying to link against other main packages, i.e. it is building m1.test with m2 and m3 (which compiled with -p main) linked in, which fails.
Without the -coverpkg=all, it builds three binaries, each of which is built without linking against other main packages, i.e. m2 and m3 are not linked into m1.test in non-coverpkg=all mode. This works fine.
Doing go test -coverpkg=m/m1,m/m2 ./..., it builds m1.test with m2 linked in but not m3. So apparently packages mentioned in the -coverpkg flag gets linked in.

I don't think we should link in other main packages, because they cannot be imported, even with -coverpkg=all, they are surely not covered.

I don't think we should link in other main packages, because they cannot be imported, even with -coverpkg=all, they are surely not covered.

Each synthetic test main package (the actual main) imports all packages mentioned by the -coverpkg pattern. testing.RegisterCover gets called with the cover blocks from all these packages. When we compute the coverage percentage, these packages count, even though their code may be unreachable. Their inits may have side effects, too.

If linking this in is actually necessary, could the go command pass -p example.com/cov/mainonly when compiling mainonly.cover.go instead of -p main?

Maybe. I'll check on this tomorrow.

Each synthetic test main package (the actual main) imports all packages mentioned by the -coverpkg pattern. testing.RegisterCover gets called with the cover blocks from all these packages. When we compute the coverage percentage, these packages count, even though their code may be unreachable. Their inits may have side effects, too.

Thanks for the explanation. This seems weird to me, as -coverpkg could actually change the behavior of the test (e.g. from pass to fail). But I guess the decision has already been made...

@cherrymui Agreed, it's definitely weird. It looks like -p with the full import path fixes it though. I don't think any change needs to be made in pluginPath.

@jayconrod, thanks for the fix!

I don't think any change needs to be made in pluginPath.

I'm not sure why pluginPath is relevant here, though.

I initially thought pluginPath might be relevant, since it may be called to set the -p argument, and it was changed in CL 174657. I don't think it would ever be called for tests or coverage though.

Ok, thanks for the explanation. pluginPath is only used in buildmode=plugin, and is not used here.

@gopherbot Please open backport issue to 1.12


How to Handle ABC+Start Key Press?

Hi Johannes,
I want to catch the ABC+Start key press such that it exits back out to the title screen. Currently when player 1 hits ABC+Start the game freezes. I tried adding an input callback but it looks like the game freezes before it gets there.
How can I handle ABC+Start key press? Thanks in advance.

Hi, I will add something in the engine

Fixed. I added jo_core_set_restart_game_callback() where you can pass your function

Thanks, I verified it works.


Fix typos

There's a genuine misspelling of Qiskit at the start of the list of "Qiskit is...", as well as redundancy in the sentence structure for the second-to-last bullet point in that list.

What are the issues this pull addresses (issue numbers / links)?
Did you add tests to cover your changes (yes/no)?
Did you update the documentation accordingly (yes/no)?
Did you read the CONTRIBUTING document (yes/no)?
Summary
Details and comments

 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.


Risk for large Image Layer due to chown

If the base directory for you assembly contains files and you use  in  then it will chown -R the whole directory. So if you have a server in that directory and the assembly just copies in config then the whole server-directory will be chown:ed. This means that all the files have been changed in that Image Layer and is added to the Image size.
Don't know if this will be optimized or not, but it's probably good for users to know.
The only solution I can think of is copying the files to /tmp/folder, chown -R that folder and then move it to the target destination

Completely agreed that there is a gotcha when not using an empty directory as basedir. I implemented your suggestion for 0.13.7
Just to be sure: That happens for you when you are using a <user> in the assembly configuration, right ?


cutscene:battlerText() Results in an error

I'm trying to make an act cutscene with the enemy dialogue boxes but everytime I try to call the cutscene:battlerText() method this error occors
src/engine/game/battle/battler.lua:119 -
attempt to index field 'sprite' (a nil value)

okay nevermind it fixed itself????


Preview bug fixes

Argh, nice catch. :/
https://github.com/zathras-crypto/omnicore/blob/0.0.10-Z-PreviewBugFixes/src/omnicore/tx.cpp#L1456
https://github.com/zathras-crypto/omnicore/blob/0.0.10-Z-PreviewBugFixes/src/omnicore/tx.cpp#L904
... too. Feel free to merge by the way, if you want to start to build - the height check looks good.

By the way, how did you notice it? I assume this didn't cause a crash, but messy log output.

By the way, how did you notice it? I assume this didn't cause a crash, but messy log output.

I was debugging something and saw some unexpected output in the log, so just fixed it up :)


global: make alpha release

Last release to make, so that we can bump all package versions once they have been released.


Display issue in the admin panel

Hello guys,
Thanks for this amazing library by the way. However I have a small display issue.
I'm using:

django-material==0.13.1
django==1.10.6
Python 3.6

Here is a screen shot :

All of the following should be on the same level and appears as followed when loading the admin panel:

Thanks for the help

Thats Chrome bug. Just set page zoom to default.


please help setup guide

wget http://rocketeer.autopergamene.eu/versions/rocketeer.phar
$ chmod +x rocketeer.phar
$ mv rocketeer.phar /usr/local/bin/rocketeer
rocketeer ignite
require(phar:///usr/local/bin/rocketeer/vendor/phpunit/phpunit/src/Framework/Assert/Functions.php): failed to open stream: phar error: "vendor/phpunit/phpunit/src/Framew
ork/Assert/Functions.php" is not a file in phar "/usr/local/bin/rocketeer" in phar:///usr/local/bin/rocketeer/vendor/composer/autoload_real.php on line 58
PHP Fatal error:  require(): Failed opening required 'phar:///usr/local/bin/rocketeer/vendor/phpunit/phpunit/src/Framework/Assert/Functions.php' (include_path='phar:///usr/local/bin/r
ocketeer/vendor/phpseclib/phpseclib/phpseclib:phar:///usr/local/bin/rocketeer/vendor/phpunit/php-text-template:phar:///usr/local/bin/rocketeer/vendor/phpunit/php-timer:.:/usr/share/ph
p:/usr/share/pear') in phar:///usr/local/bin/rocketeer/vendor/composer/autoload_real.php on line 58

Yes the current Phar got corrupted, trying to fix it


Playground crash due to AKAudioFile+Processing

Hello,
The playground Playback / Editing Audio Files crashes.
This is due to the fact that when not provided with names, AKAudioFile+Processing methods return an empty string for the name and not a random unique name as stated in the playground comment.
Changing the processing methods signatures with
name: String = UUID().uuidString

instead of
name: String = ""

seems to fix the issue.  I have PR ready if you want, but I do not know enough of AudioKit's internals to be sure about the fix.  Let me know.

Sure, please PR.

Thanks!


Update azure-monitor-agent-mma-removal-tool.md

@guywi-ms
Can you review the proposed question (in the diff)?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thank you for reaching out to us! We appreciate your interest in improving our documentation.
Product questions and comments are beyond the scope of documentation pull requests on GitHub. Documentation pull requests are used to update documentation with known information.
We can help direct you to the right channel for technical support questions and general documentation feedback.
Technical Support

Create a support ticket
Microsoft Azure Monitor Q&A question page

Documentation Feedback
To provide documentation feedback, see Provide feedback for Microsoft Learn content.
#please-close


libsamplerate: Add single precision compilation option

This adds cocci rule that changes all double type occurences to fp_t type that is defined in config.h file.
Depending on LIBSAMPLERATE_USE_SINGLE_PRECISION_FP option, one can enable single precision floating point arithmetic.

Related newt PR: https://github.com/apache/mynewt-newt/pull/577


Small style tweak to header and new story

Tiny style change and supporting storybook to allow splitting the header into two parts

Description

See JENKINS-39763.

Submitter checklist

[X] Link to JIRA ticket in description, if appropriate.
[ ] Change is code complete and matches issue description
[ ] Appropriate unit or acceptance tests or explanation to why this change has no tests

Reviewer checklist

[ ] Run the changes and verified the change matches the issue description
[ ] Reviewed the code
[ ] Verified that the appropriate tests have been written or valid explanation given

@reviewbybees

LGTM :bee: (if I am qualified).
Tried it locally


flash: Fix end address for firmware.

Fixes #425.

Awesome, thanks a lot! I'm super-glad to have helped a tiny bit -- I'll need to find some free time to catch up to master, and then I've got a few more little things I could contribute (though they might have to go upstream to litex)

Please do reach out if you need help. Improving support for the nexys_video is something I definitely support!


[parametric/dotnet] enable auto-instrumentation and minor code clean-up

Description
This PR makes some updates to the .NET app used for parametrics tests:
https://github.com/DataDog/system-tests/tree/main/parametric/apps/dotnet

update nuget package references to latest versions
enable automatic instrumentation (not used yet, but required for OTel test endpoints coming soon)

change Datadog.Trace nuget package to Datadog.Trace.Bundle
add required environment variables

allow running app without env var APM_TEST_CLIENT_SERVER_PORT (easier to run as a stand-alone app)
misc code clean-up
added .gitignore and .dockerignore files in parametric/apps/dotnet

Motivation
Automatic instrumentation in required for the .NET tracer to support the OpenTelemetry API. The rest is bonus :)
Additional notes

none

Moving this to #1231 due to huge conflicts with main.


Fix bugs and make the widget independent on scroll controller.

Hi!
It fixes the bug with initState and makes the widget working properly without specifying the ScrollController and scrollDirection. Now it can determine scroll metrics (including scroll axis) using Notifiers. There is no need also to access the PrimaryScrollController since it may not exist on the current context.

I am not 100% sure if what you explain is exactly my error, but I encountered this:
I/flutter ( 1494): The following exception was thrown building Expanded(flex: 1)
I/flutter ( 1494): :
I/flutter ( 1494): dependOnInheritedWidgetOfExactType<PrimaryScrollController>() or dependOnInheritedElement() was called before _ScrollShadowState.initState() completed.
I/flutter ( 1494): When an inherited widget changes, for example if the value of Theme.of() changes, its dependent widgets are rebuilt. If the dependent widget's reference to the inherited widget is in a constructor or an initState() method, then the rebuilt dependent widget will not reflect the changes in the inherited widget.
I/flutter ( 1494): Typically references to inherited widgets should occur in widget build() methods. Alternatively, initialization based on inherited widgets can be placed in the didChangeDependencies method, which is called after initState and whenever the dependencies change thereafter.
I/flutter ( 1494): debugCreator: Expanded ← Column ← _ListViewOverlay ← Builder ← Semantics ← _FocusInheritedScope ← Focus ← _InheritedProviderScope<SearchCubit?> ← InheritedProvider<SearchCubit> ← BlocProvider<SearchCubit> ← MapOverlay ← Positioned ← ⋯
I/flutter ( 1494): #0      StatefulElement.dependOnInheritedElement.<anonymous closure> (package:flutter/src/widgets/framework.dart:5321:9)
I/flutter ( 1494): #1      StatefulElement.dependOnInheritedElement (package:flutter/src/widgets/framework.dart:5364:6)
I/flutter ( 1494): #2      Element.dependOnInheritedWidgetOfExactType (package:flutter/src/widgets/framework.dart:4383:14)
I/flutter ( 1494): #3      PrimaryScrollController.maybeOf (package:flutter/src/widgets/primary_scroll_controller.dart:142:53)
I/flutter ( 1494): #4      PrimaryScrollController.of (package:flutter/src/widgets/primary_scroll_controller.dart:160:42)
I/flutter ( 1494): #5      _ScrollShadowState.initState (package:flutter_scroll_shadow/src/scroll_shadow.dart:119:55)
I/flutter ( 1494): #6      StatefulElement._firstBuild (package:flutter/src/widgets/framework.dart:5219:55)
I/flutter ( 1494): #7      ComponentElement.mount (package:flutter/src/widgets/framework.dart:5062:5)
I/flutter ( 1494): #8      Element.inflateWidget (package:flutter/src/widgets/framework.dart:3971:16)
I/flutter ( 1494): #9      Element.updateChild (package:flutter/src/widgets/framework.dart:3702:20)
I/flutter ( 1494): #10     ComponentElement.performRebuild (package:f

I got this error with a similar setup to:
Column(
      crossAxisAlignment: CrossAxisAlignment.stretch,
      children: [
        Expanded(
          child: ScrollShadow(
            color: context.materialTheme.backgroundColor,
            child: Text('test'),
          ),
        ),
      ],
    )

This does not happen with version 1.0.2

@memoriasIT does this happen on my PR revision? Could you provide me with full example of code to reproduce the error, please?
As far as I can see, the mistake is in the _ScrollShadowState.initState (package:flutter_scroll_shadow/src/scroll_shadow.dart:119:55). There is no initState now in my version of code, so it seems that you tried to create ScrollShadow without ScrollController on the current version of package, which contains the bug.

Okay nice, I checked on your revision and it doesn't seem to happen indeed :)
For me this would trigger the error with the package from pub dev:
import 'package:flutter/material.dart';
import 'package:flutter_scroll_shadow/flutter_scroll_shadow.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(widget.title),
      ),
      body: Column(
        crossAxisAlignment: CrossAxisAlignment.stretch,
        children: [
          Expanded(
            child: ScrollShadow(
              color: Colors.red,
              child: CustomScrollView(
                physics: const ClampingScrollPhysics(),
                slivers: [],
              ),
            ),
          ),
        ],
      ),
    );
  }
}

For me this would trigger the error with the package from pub dev

Yeah, for me too ) That's why I suggested the maintainer to merge this PR.

Hi @rickypid, sure, no problem. Did it and updated the commit.


Add CAP_NET_BIND_SERVICE to hyperkube.

Add CAP_NET_BIND_SERVICE to hyperkube so it can bind privileged ports as
non-root.

Thanks, can you please merge when you have a chance?

LGTM


Angular2-Toaster installs own version of Angular 2

Hi there!
Can you tell me why Angular 2 and RxJS is in "dependencies" instead of "peerDependencies" as it should be? Angular2-Toaster goes as a "plugin", not standalone service, right? Check https://github.com/valor-software/ng2-bootstrap/blob/development/package.json for example.

There are plans to move these dependencies to peerDependencies per the conversation in #48. However, currently peerDependencies breaks the Karma test shim configuration. Once that is patched, the dependencies will be updated to peerDependencies.
Thanks for raising this issue. This is an important priority and something that needs to change in the very near future.


[Bug] Error running ./prepare_libs.sh on a M1 Pro system

🐛 Bug
Getting this error while running ./prepare_libs.sh. I'm using Dolly-v2 as the model, and from what I could understand from the documentation and the code, there was no need of editing the ./prepare_libs.sh function for this in specific as there are no editable attributes/variables in it
`

cmake ../.. -DCMAKE_BUILD_TYPE=Release -DCMAKE_SYSTEM_NAME=iOS -DCMAKE_SYSTEM_VERSION=14.0 -DCMAKE_OSX_SYSROOT=iphoneos -DCMAKE_OSX_ARCHITECTURES=arm64 -DCMAKE_OSX_DEPLOYMENT_TARGET=14.0 -DCMAKE_BUILD_WITH_INSTALL_NAME_DIR=ON -DCMAKE_SKIP_INSTALL_ALL_DEPENDENCY=ON -DCMAKE_INSTALL_PREFIX=. -DCMAKE_CXX_FLAGS=-O3 -DMLC_LLM_INSTALL_STATIC_LIB=ON -DUSE_METAL=ON
^R
CMake Error at CMakeLists.txt:50 (add_subdirectory):
add_subdirectory given source "tvm" which is not an existing directory.

-- system-nameiOS
-- VERSION: 0.2.00
CMake Error at CMakeLists.txt:57 (tvm_file_glob):
Unknown CMake command "tvm_file_glob".
-- Configuring incomplete, errors occurred!
`
Environment

Operating system: MacOS
How you installed MLC-LLM : condo
How you installed TVM-Unity : pip
Python version: 3.11.3

I had this issue before and got similar errors to you such as add_subdirectory given source "tvm" which is not an existing directory. for example.
The problem I faced was that I did not build tvm from source and was using the prebuilt package.
Once I built tvm from source following the documentation here: https://mlc.ai/mlc-llm/docs/install/index.html, making sure to export TVM_HOME as the docs specify so your local shell knows which tvm to use (in case you installed a prebuilt package), I did not run into this issue and was able to build it successfully.

Did you set TVM_HOME after installation? https://mlc.ai/mlc-llm/docs/install/index.html#option-1-prebuilt-package

Ok, turned out my rustup and rustc were not compatible; adding the rustc version from rustup to PATH solves the issue: export PATH=~/.cargo/bin:$PATH

Glad you figured it out!


Add a [Bool] data annotation

Using dotnet core 2.2, parsing JSON on a REST interface, using [Required] to ensure fields aren't missed.
Along comes a boolean field. If it's missing from the JSON it silently gets transformed to false, when I want it to throw a 400.
Current workaround is [Range(typeof(bool), "false", "true")]. What would be neater is [Bool].

Thanks for contacting us, @frank-borkin.
@dougbu, do we have an existing mechanism to achieve this?

When using JSON, use Json.NET attributes e.g. [JsonProperty(Required = Required.Always)]. If you were not using [FromBody], our [BindRequired] attribute would help in this scenario.

@dougbu Am I right that [JsonProperty(Required = Required.Always)] would simply ensure that it was set, but not validate that it's true || false ? Having something which casts any other or no type to true or false is not desired.

@frank-borkin your question was about missing Boolean properties. But, I'm pretty sure Json.NET fails de-serialization when a value cannot be converted. @JamesNK can you confirm?

Look at the RequiredAttribute source code. For bools it's basically just checking if the field is null.
The validators run after the JSON is deserialized to C# object and at that point the boolean field is already its default type value - false.
You should either change your field to bool? or use the JSON-specific validation attributes.

Newtonsoft.Json attempts to convert/coerce values. For example with boolean true works and "true" works. An error will be thrown if the JSON value can't be used.

Thank you for your feedback. We're closing this issue as the questions asked here have been answered.


[SOLVED] Assets missing after packaging using file-loader

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Please describe your issue:
[SOLVED] Assets missing after packaging using file-loader.
I wanted to leave this here for anyone searching the same thing as me since it took way too long to figure out what was going on in this plugin. It seems electron forge webpack plugin puts the entry html and js in a directory that is specified as the renderer name. (In the example it is called "main_window") but doesn't tell file-loader about or something 🤷🏻♂
Given a simple structure like this:
src
  |- assets
    |- someImage.jpg
  |- main.js
  |- renderer.js
index.html
webpack.main.config.js

And the @electron-forge/plugin-webpack config:
{
"plugins": [
        [
          "@electron-forge/plugin-webpack",
          {
            "mainConfig": "./webpack.main.config.js",
            "renderer": {
              "config": "./webpack.renderer.config.js",
              "entryPoints": [
                {
                  "html": "./index.html",
                  "js": "./src/renderer.js",
                  "name": "main_window"
                }
              ]
            }
          }
        ]
      ]
}

You should have this webpack config when using file-loader:
const CopyWebpackPlugin = require("copy-webpack-plugin")
const path = require("path")

module.exports = {
  // ...rest of config
  module: {
    test: /\.(jpg|png|svg|ico|icns)$/,
    loader: "file-loader",
    options: {
      name: "[path][name].[ext]",
      publicPath: "..", // move up from 'main_window'
      context: "src", // set relative working folder to src
    },
  },
  plugins: [
    new CopyWebpackPlugin(
      [{ from: path.join("src", "assets"), to: "assets" }],
      {
        ignore: [".gitkeep"],
      }
    ),
  ],
},

@hennessyevan which file would you put the code at the bottom? In webpack.renderer.config.js?

I have followed the advice above, and can see the static files now are copied to .webpack/renderer/assets correctly, however I still get an error for instance on the following line:
import logo from './assets/logo.svg';
The error is:
TS2307: Cannot find module './assets/logo.svg' or its corresponding type declarations.
It seems that webpack is still not able to locate the file correctly. What am I missing here? Any help appreciated, thanks!


Issue rendering with Fragment  <></>  ( out of order rendering / parse error )

Describe the bug
WORLDHello .
Incorrect rendering order
https://playground.solidjs.com/?hash=-1334550019&version=1.4.1

       Hello <>{'WORLD'}</>.

https://playground.solidjs.com/?hash=-1738191781&version=1.4.1

     <>{"WORLD"}</>

/main.tsx: Property arguments[0] of CallExpression expected node to be of a type ["Expression","SpreadElement","JSXNamespacedName","ArgumentPlaceholder"] but instead got undefined

Your Example Website or App
https://playground.solidjs.com/?hash=-1334550019&version=1.4.1
Steps to Reproduce the Bug or Issue
see repro in playground.
Expected behavior
content of fragment should render in order.
Hello World.
and should not cause errors.

I can fix this but curious why you are using fragments like this. Is there something you are trying to achieve?

it was discovered incidentally, when I (mis)used <></> to silence a type error
https://playground.solidjs.com/?hash=-1259118341&version=1.4.1

Ok yeah. I'm pretty sure the solution is to compile it away as if it doesn't exist. Of course I need to detect that case first but yeah sure.

Yeah this is taking too much time to figure out right now. I'm just going to throw with a more human readable error. Atleast for now.

I only tested it with a simple modified test under dom.spec and fragments , so I don't really know the wider implications.
    path.get("children").forEach(child=>{
      if (child.isJSXFragment()) {
        const fragmentChildren = child.node.children
        child.replaceWithMultiple(fragmentChildren )
      }
    })

this will come before or after filterChildren(path.get("children"))
https://github.com/ryansolid/dom-expressions/commit/74e757e881a13f44ab884f2ea75f50e959d4cf4d#diff-a9df2c51ba8fab92870f64f43f93a10e151c352b72ee4a1b6dfaa5eb97677750R618

https://astexplorer.net/#/gist/735c0d4397fe5b51f8dd8a45f6e6804e/e8bae2a3fff9e23d38da28bb5185e1e7dff6062e

Yeah I was playing with stuff like this but I was hitting edge cases. I think just saying no support makes the most sense right now.


Motion / Springs tutorial has A11y errors

On Part 2 / Motion / Springs the svg element has an accessibility warning:

A11y: <svg> with mousemove, mousedown, mouseup handlers must have an ARIA role

I'm no expert in ARIA yet, but I tried role="application" just to see what would happen; unfortunately, all that did was to change the error into

A11y: Non-interactive element <svg> should not be assigned mouse or keyboard event listeners.

It feels like this is a hard coded dead end, because if application cannot greenlight mouse or keyboard events, then what can?
Not sure if the example is actually bad style or if the second warning is nonsense and should accept role="application".

Note: Part 2 / Actions / The use directive has a similar problem with its div:

A11y: <div> with click, keydown handlers must have an ARIA role


Support installation of scoped modules

This PR does the following:

Installation of scoped modules. All scoped modules will be installed into node_modules/.store/@{scope}!{name}@{version}
Pass pkgSpec to more methods.
More debugging:

add tarball to finish debug message to match shasums to tarball fetches.
Debug symlinkSelf.

saw this already, good show :)

v0.4.0 now has this, thanks a ton.


[BUG] EntityRelationInfo "from" is deserialized as a protected attribute

Thingsboard version: 3.4.4PE
tb-rest-client version: commit eba3d2c
sample code:
user_id = UserId(id="my user id", entity_type="USER")
query = EntityRelationsQuery(
    filters=[],
    parameters=RelationsSearchParameters(
        root_id=user_id.id,
        root_type=user_id.entity_type,
        direction="FROM",
    )
)
raw_data = tb.entity_relation_controller.find_info_by_query_using_post(body=query, _preload_content=False)
data = tb.find_info_by_query(query)

Here is EntityRelationInfo according to the latest swagger doc

Here is a response deserialized manually:

Here is a response deserialized automatically:

"from" gets automatically deserialized as a protected attribute (leading underscore).
Is this intentional ?

Hi @tacensTECSOL, thanks for your interest in TB Python REST Client!
Yes, it is intentional, because it is a reserved keyword in Python (from ... import ...).

Hi,
Thanks for the clarification !


when connecting to https by IP address, client hello message has a server_name SNI block.

When connecting to https by IP address requests.get("https://<IP_ADDRESS>/"), the client hello message includes a server_name extension
Expected Result
No server_name extension in the client hello. The server I am talking to immediately closes the connection when it sees a server_name block with a literal ipv4 address. The SNI Spec says that Literal IPv4 and IPv6 addresses are not permitted in "HostName".
Actual Result
There was a server_name extension in the client hello, and the server I am talking to immediately closes the connection.
Reproduction Steps

Set up wireshark monitoring packets.

 import requests
 requests.get("https://<IP_ADDRESS>")

Observe the client hello message in wireshark:

Frame 570: 571 bytes on wire (4568 bits), 571 bytes captured (4568 bits) on interface 0
Ethernet II, Src: BizlinkK_XX:XX:XX (9c:eb:e8:XX:XX:XX), Dst: JuniperN_XX:XX:XX (08:81:f4:XX:XX:XX)
Internet Protocol Version 4, Src: <IP_ADDRESS>, Dst: <IP_ADDRESS>
Transmission Control Protocol, Src Port: 4811, Dst Port: 443, Seq: 1, Ack: 1, Len: 517
Secure Sockets Layer
    TLSv1.2 Record Layer: Handshake Protocol: Client Hello
        Content Type: Handshake (22)
        Version: TLS 1.0 (0x0301)
        Length: 512
        Handshake Protocol: Client Hello
            Handshake Type: Client Hello (1)
            Length: 508
            Version: TLS 1.2 (0x0303)
            Random
            Session ID Length: 0
            Cipher Suites Length: 110
            Cipher Suites (55 suites)
            Compression Methods Length: 1
            Compression Methods (1 method)
            Extensions Length: 357
            Extension: server_name
                Type: server_name (0x0000)
                Length: 12
                Server Name Indication extension
                    Server Name list length: 10
                    Server Name Type: host_name (0)
                    Server Name length: 7
                    Server Name: <IP_ADDRESS>
            Extension: ec_point_formats
            Extension: elliptic_curves
            Extension: SessionTicket TLS
            Extension: signature_algorithms
            Extension: Heartbeat
            Extension: Padding

System Information
Windows 10 creators update, windows subsystem for linux.
$ uname -a
Linux ZeroGravitas 4.4.0-43-Microsoft #1-Microsoft Wed Dec 31 14:42:53 PST 2014 x86_64 x86_64 x86_64 GNU/Linux
$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.2 LTS
Release:        16.04
Codename:       xenial
$ apt policy python-requests
python-requests:
  Installed: 2.9.1-3
  Candidate: 2.9.1-3
  Version table:
 *** 2.9.1-3 500
        500 http://archive.ubuntu.com/ubuntu xenial/main amd64 Packages
        100 /var/lib/dpkg/status

Is is easy enough for me to work around this (refer to the machine by name), but it took me ages to figure out what was going wrong, so here is something hopefully googleable.
The server I was connecting to is Windows Server 2016 Datacenter (azure VM).

Yeah, this is a known limitation of Requests that boils down to the absence of a reliable tool for distinguishing IP addresses and hostnames. I think a sensible enhancement might be to use the ipaddress module if it's present in your environment, but this enhancement would have to be applied in urllib3. Do you mind opening a report over there?


Add suport for unhandled field trackInventoryByDefault

The Shop type defines the field trackInventoryByDefault, which is also available in the ShopSettingsInput input, but it's not used in the code anywhere. Most likely the field was added a long time ago in one of the first versions of API and removed before we started properly using field deprecation. It sounds useful, however, to have a field that would be used as a default value for ProductVariant.trackInventory.
TODO:

Add logic that would use the Shop.trackInventoryByDefault value (if it is set) to populate trackInventory value (if not provided) in the following mutations:

productVariantCreate
productVariantUpdate
productVariantBulkCreate
productVariantBulkUpdate

Add a description to Shop.trackInventoryByDefault explaining that this field is used as a default value for ProductVariant.trackInventory
Extend the description of all trackInventory fields to mention that if the field is not provided, Shop.trackInventoryByDefault would be used.

@maarcingebala do we need to use  Shop.trackInventoryByDefault in update operations?
let's say we have scenario were Product variant created using productVariantCreate and passed trackInventory as false but Shop.trackInventoryByDefault is true and while making doing productVariantUpdate  user has not passed any arguments like trackInventory so code will automatically pick the Shop.trackInventoryByDefault and mark that field as true that does not make any sense because user don't want to update that field.

@Smit-Parmar Yes, it should work as you described, so only when the user is creating a variant and has not provided the trackInventory in the input, we should use the value from Shop.trackInventoryByDefault. We can skip populating that field in update operations as it might be unexpected for the user. I've updated the description; thanks!


Add support for combined ws streams

There's two types of combined streams to support, each with a diff use case. The first one is easier than the second:

static

topics are known before the stream is opened
these topics wont ever change for this stream after it is opened
if disconnected, this can just respawn using the URL of the stream

dynamic

exact topics are unknown before the stream opens
we need the ability to subscribe to new topics after the stream opens

a partial implementation is here.
I'm not sure how to determine the wskey. We could just dynamically generate one, and increment if the max limit is hit and a new one is made (see below). This is essentially just a primary key used to store state for a ws connection.

we need the ability to unsubscribe from some topics after the stream opens.

a partial implementation is here.
Same problem as before, not sure yet how to determine the wskey used to store a list of topics for that connection (or how to find the correct wskey using the topic passed into unsubscribe()). Could just be a query in wsStore to find any/all wsKeys that have that topic.

we will need to persist which topics are currently subscribed for this stream

if the stream disconnects, we can reconnect and we know which topics to re-subscribe to.
already partially implemented in this.wsStore, see the above partial implementation.
needs to be refactored similar to FTX connector, because topics are objects (not just strings).
if easy to do, would be nice to keep it consistent with how the ftx connector does it (as it also tracks topics using objects used to sub to them).

Stretch goals for dynamic combined streams

we may need to track how many topics are currently active on a stream (because there is a max limit of 200 per stream).

if that cap is reached,

we may need to see if we have other combined streams
maybe using a wskey prefix (get all wskeys that start with `combined)
or some dictionary to track this state

if another combined stream is available AND max limit is not reached, request subscribe via that combined stream
if not, spawn new combined stream (with new wskey) and send topic sub into the newly spawned stream.

Cheers!!
I'm not sure how to create a combined stream to this API.
I think I just made a workaround that makes it, but maybe you have it previously considered and can throw me a bone here.
I'm trying to reach all 5m klines available in one stream. I've done it before directly with Binance API but I like the way you parse the response, so I made this.
This is what I've done:
.
Is there a better way to do this?

This seems like a way to do it. For the wskey, I would recommend to include all dynamic props...maybe something like symbols.sort().join().replace('USDT') isn't too long of a string? Reasoning is if you want to have multiple calls to this with diff symbols, you don't want any risk for wskeys to get mixed up. They should be unique, or when a reconnect happens you'll get the wrong websocket url being reconnected.
maybe streamName (in getWsKeyWithContext) would also be good as something suggestive like kline_multiple. Might need some testing to see how the reconnect logic behaves if your internet connection (and thus the stream) drops.

Cheers!
Done that and so far so good. I'll keep an eye on it and tell you if any errors comes up.
Thanks again for your work!!

Glad to hear! Would you mind sharing the snippet you're currently using? Are reconnects working OK? Would be great to add it to the lib.

Yeah, already tried the reconnects and it went all fine.
We were just talking in Telegram. I can send you anything else you need.
The snippet to connect all the cryptos at the same time:
`/**

Get exchange info to retrieve assets
*/
function getAllAssets() {
futuresClient
.getExchangeInfo()
.then((x) => {
openWebsockets(x);
})
.catch((e) => console.log(e));
}
function openWebsockets(exchangeInfoData) {
exchangeInfoData.symbols.forEach((c, index) => {
if (
c.contractType === config.exchangeAssetInfo.contractType && // Looking for just perpetuals
c.quoteAsset === config.exchangeAssetInfo.quoteAsset             // Looking for just 'USDT' base asset
) {
allWs[index] = wsClient.subscribeKlines(c.symbol, "3m", "usdm"); //I like to have it this way if I want to disconnect from all.
}
});
}`


Centralize the code for opening/creating files for serialization to a single function

Centralize the code for opening/creating the streams used  for serialization in a single place to avoid to repeat always the same code.

Done.


Not receiving SD-WAN metrics

I have latest build with SD-WAN metrics added, however I'm not seeing sd-wan metrics them after scraping the target.
Here is mine fortigate profile configuration:
edit "monitor"
        set secfabgrp read
        set netgrp custom
        set fwgrp custom
        set vpngrp read
        set system-diagnostics disable
        config netgrp-permission
            set cfg read
        end
        config fwgrp-permission
            set policy read
        end

I wonder if something has to be changed in profile configuration?

From the SD-WAN PR it can be seen that the permissions required should be "access_group": "netgrp.cfg", which you have listed.
What version of FortiOS are you running?
If you go to https://your-fortigate-here.local/api/v2/monitor/virtual-wan/health-check?vdom=* when logged in as a user, does it show the health checks you are looking for?

I will list this as support until we have narrowed it down a bit.

Yes, I do receive result
http_method | "GET"
-- | --
results | {…}
vdom | "root"
path | "virtual-wan"
name | "health-check"
status | "success"
serial | "*********"
version | "v6.4.2"
build | 1723

Would you be OK with sharing the .json results in full (censoring IPs and serials are of course OK)?
I am having a bit of trouble reading that output - and I am interested mainly of the structure of the results object.

Sure. Here you go.
[   {     "http_method":"GET",     "results":{       "AWS_US_SLA":{         "port9":{           "status":"up",           "latency":112.75773620605469,           "jitter":0.03916727751493454,           "packet_loss":0,           "packet_sent":24741283,           "packet_received":24665286,           "session":309,           "tx_bandwidth":11419643,           "rx_bandwidth":697136,           "state_changed":1614233414,           "sla_targets_met":[             1           ]         },         "port10":{           "status":"up",           "latency":100.31079864501953,           "jitter":0.089132942259311676,           "packet_loss":0,           "packet_sent":24741283,           "packet_received":24724974,           "session":10007,           "tx_bandwidth":32588134,           "rx_bandwidth":159458598,           "state_changed":1607412315,           "sla_targets_met":[             1           ]         }       },       "AWS_EU_SLA":{         "port9":{           "status":"up",           "latency":30.875497817993164,           "jitter":0.14193330705165863,           "packet_loss":0,           "packet_sent":24741283,           "packet_received":24734234,           "session":309,           "tx_bandwidth":11419643,           "rx_bandwidth":697136,           "state_changed":1614233414,           "sla_targets_met":[             1           ]         },         "port10":{           "status":"up",           "latency":29.78489875793457,           "jitter":0.57146674394607544,           "packet_loss":1,           "packet_sent":24741283,           "packet_received":24713582,           "session":10013,           "tx_bandwidth":32588134,           "rx_bandwidth":159458598,           "state_changed":1607412315,           "sla_targets_met":[             1           ]         }       },       "gitlab":{         "port9":{           "status":"up",           "latency":35.57550048828125,           "jitter":0.04386698454618454,           "packet_loss":0,           "packet_sent":7421632,           "packet_received":3416877,           "session":309,           "tx_bandwidth":11419643,           "rx_bandwidth":697136,           "state_changed":1614233414,           "sla_targets_met":[           ]         },         "port10":{           "status":"error"         }       },       "GOOGLE_DNS":{         "port9":{           "status":"up",           "latency":22.144998550415039,           "jitter":0.01653289794921875,           "packet_loss":0,           "packet_sent":24741283,           "packet_received":24728256,           "session":309,           "tx_bandwidth":11419643,           "rx_bandwidth":697136,           "state_changed":1614233414,           "sla_targets_met":[             1           ]         },         "port10":{           "status":"up",           "latency":7.2254014015197754,           "jitter":0.031633313745260239,           "packet_loss":0,           "packet_sent":24741283,           "packet_received":24733843,           "session":10017,           "tx_bandwidth":32588134,           "rx_bandwidth":159458598,           "state_changed":1607412315,           "sla_targets_met":[             1           ]         }       }     },     "vdom":"root",     "path":"virtual-wan",     "name":"health-check",     "status":"success",     "serial":"*****",     "version":"v6.4.2",     "build":1723   } ]

Thanks. Is there anything being mentioned in the output of the exporter? If there is a permissions issue, hopefully there would be a warning.

Yeah, there is.
2021/03/02 13:52:24 Probe of "https://fortigate.local:4434" failed, took 1.332 seconds
2021/03/02 13:52:38 Error: json: cannot unmarshal number into Go struct field SLAMember.results.sla_targets_met of type string


Reverse order of columns in the benefits table

Fixes #39041

@alexcnichols : Thanks for your contribution! The author(s) have been notified to review your proposed change. @jm-aditi-ms

@qianw211, can you review this PR? Thanks.

Or @LauraBrenner, can you review this PR? Thanks.

#sign-off

@qianw211: I'm sorry - only the author of this article, @jm-aditi-ms, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Hi, could you please help sign-off on this PR?  The author is no longer with our team.
Thanks!
Qian
From: PRMerger9<EMAIL_ADDRESS>Sent: Tuesday, September 17, 2019 9:16 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Qian Wen (Aditi Staffing LLC<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Reverse order of columns in the benefits table (#39042)
@qianw211https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fqianw211&data=02|01|v-qiwe%40microsoft.com|a65619daf0fd4a2d9a0208d73b8a5250|72f988bf86f141af91ab2d7cd011db47|1|0|637043337604548279&sdata=1ccYDEJ5M5qvwLjkwG81tHU1Y5loZza3wgcbchX3fBs%3D&reserved=0: I'm sorry - only the author of this article, @jm-aditi-mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjm-aditi-ms&data=02|01|v-qiwe%40microsoft.com|a65619daf0fd4a2d9a0208d73b8a5250|72f988bf86f141af91ab2d7cd011db47|1|0|637043337604548279&sdata=JIQra8Y%2Fu%2BQY4m59mlyyORX4uOUE9eKZMvzUbKheAeU%3D&reserved=0, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F39042%3Femail_source%3Dnotifications%26email_token%3DAJZZVIQ2MIYIOCVXVR5GMVDQKD7DZA5CNFSM4IXQ5WHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD65CGLQ%23issuecomment-532292398&data=02|01|v-qiwe%40microsoft.com|a65619daf0fd4a2d9a0208d73b8a5250|72f988bf86f141af91ab2d7cd011db47|1|0|637043337604558274&sdata=miQk81VNmeD9DEFyxuNaniQAb0cDdSQSQ9EaFvK%2FHiQ%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJZZVISVJMTSOHVTO6Y4AULQKD7DZANCNFSM4IXQ5WHA&data=02|01|v-qiwe%40microsoft.com|a65619daf0fd4a2d9a0208d73b8a5250|72f988bf86f141af91ab2d7cd011db47|1|0|637043337604558274&sdata=4D%2FJ8BKuP%2Ff3C6er0QOce56G1eKU0Wii0D1l2Grzugc%3D&reserved=0.

Thanks, Qian. The PR has been merged.
Kristine Toliver, Technical Editor
AQUENT at Microsoft | Content + Learning
Cloud + AI Developer Relations
Editorial service for technical contenthttps://review.docs.microsoft.com/en-us/help/contribute/contribute-apex-editorial-service?branch=master
From: Qian Wen (Aditi Staffing LLC<EMAIL_ADDRESS>Sent: Tuesday, September 17, 2019 9:19 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Mention<EMAIL_ADDRESS>C&L Pull Request Review Team<EMAIL_ADDRESS>Subject: RE: [MicrosoftDocs/azure-docs] Reverse order of columns in the benefits table (#39042)
Hi, could you please help sign-off on this PR?  The author is no longer with our team.
Thanks!
Qian
From: PRMerger9 <notifications<EMAIL_ADDRESS>Sent: Tuesday, September 17, 2019 9:16 AM
To: MicrosoftDocs/azure-docs <azure-docs<EMAIL_ADDRESS>Cc: Qian Wen (Aditi Staffing LLC) <v-qiwe<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Reverse order of columns in the benefits table (#39042)
@qianw211https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fqianw211&data=02|01|v-krmaxw%40microsoft.com|9b63d25eb3ab481bd82608d73b8ac8f6|72f988bf86f141af91ab2d7cd011db47|1|0|637043339569465254&sdata=qSIeYAlO2Gu%2Bg6r%2BZQWw3SUj1uJ1BYUEcIrDhXftrHs%3D&reserved=0: I'm sorry - only the author of this article, @jm-aditi-mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjm-aditi-ms&data=02|01|v-krmaxw%40microsoft.com|9b63d25eb3ab481bd82608d73b8ac8f6|72f988bf86f141af91ab2d7cd011db47|1|0|637043339569475209&sdata=lmd6bpjlyRi7iBIby8eEY394u9swuZg%2FBb49%2FDp50Ts%3D&reserved=0, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F39042%3Femail_source%3Dnotifications%26email_token%3DAJZZVIQ2MIYIOCVXVR5GMVDQKD7DZA5CNFSM4IXQ5WHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD65CGLQ%23issuecomment-532292398&data=02|01|v-krmaxw%40microsoft.com|9b63d25eb3ab481bd82608d73b8ac8f6|72f988bf86f141af91ab2d7cd011db47|1|0|637043339569475209&sdata=LdS4SQPY4MK9AvdH2pY2MdjglsJuqvBlfYC0nXI0cKw%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJZZVISVJMTSOHVTO6Y4AULQKD7DZANCNFSM4IXQ5WHA&data=02|01|v-krmaxw%40microsoft.com|9b63d25eb3ab481bd82608d73b8ac8f6|72f988bf86f141af91ab2d7cd011db47|1|0|637043339569485165&sdata=fHqsIFZVR%2Bb6ZmxGOkK0ePzgj%2BhBP1fXeveS7334o0M%3D&reserved=0.

Thank you!
From: Kristine Toliver (Aquent LLC<EMAIL_ADDRESS>Sent: Tuesday, September 17, 2019 10:21 AM
To: Qian Wen (Aditi Staffing LLC<EMAIL_ADDRESS>MicrosoftDocs/azure-docs<EMAIL_ADDRESS>MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Mention<EMAIL_ADDRESS>C&L Pull Request Review Team<EMAIL_ADDRESS>Subject: RE: [MicrosoftDocs/azure-docs] Reverse order of columns in the benefits table (#39042)
Thanks, Qian. The PR has been merged.
Kristine Toliver, Technical Editor
AQUENT at Microsoft | Content + Learning
Cloud + AI Developer Relations
Editorial service for technical contenthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Freview.docs.microsoft.com%2Fen-us%2Fhelp%2Fcontribute%2Fcontribute-apex-editorial-service%3Fbranch%3Dmaster&data=02|01|v-qiwe%40microsoft.com|78930d953a704baa583b08d73b9357ed|72f988bf86f141af91ab2d7cd011db47|1|0|637043376330515165&sdata=YqUkzSSjRhqzPChhUJs0OIfWHgtsDjc6vx197gUZ3V4%3D&reserved=0
From: Qian Wen (Aditi Staffing LLC) <v-qiwe<EMAIL_ADDRESS>Sent: Tuesday, September 17, 2019 9:19 AM
To: MicrosoftDocs/azure-docs <reply<EMAIL_ADDRESS>MicrosoftDocs/azure-docs <azure-docs<EMAIL_ADDRESS>Cc: Mention <mention<EMAIL_ADDRESS>C&L Pull Request Review Team <techdocprs<EMAIL_ADDRESS>Subject: RE: [MicrosoftDocs/azure-docs] Reverse order of columns in the benefits table (#39042)
Hi, could you please help sign-off on this PR?  The author is no longer with our team.
Thanks!
Qian
From: PRMerger9 <notifications<EMAIL_ADDRESS>Sent: Tuesday, September 17, 2019 9:16 AM
To: MicrosoftDocs/azure-docs <azure-docs<EMAIL_ADDRESS>Cc: Qian Wen (Aditi Staffing LLC) <v-qiwe<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Reverse order of columns in the benefits table (#39042)
@qianw211https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fqianw211&data=02|01|v-qiwe%40microsoft.com|78930d953a704baa583b08d73b9357ed|72f988bf86f141af91ab2d7cd011db47|1|0|637043376330515165&sdata=b2cdioPner%2FbeuNcHqZ1e1QunNc8SeZp449MnqfFIMs%3D&reserved=0: I'm sorry - only the author of this article, @jm-aditi-mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjm-aditi-ms&data=02|01|v-qiwe%40microsoft.com|78930d953a704baa583b08d73b9357ed|72f988bf86f141af91ab2d7cd011db47|1|0|637043376330525158&sdata=ZiZXCu55omUradpIROSvrXFNZHHyw7DJ1zGzV%2FYpkRg%3D&reserved=0, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F39042%3Femail_source%3Dnotifications%26email_token%3DAJZZVIQ2MIYIOCVXVR5GMVDQKD7DZA5CNFSM4IXQ5WHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD65CGLQ%23issuecomment-532292398&data=02|01|v-qiwe%40microsoft.com|78930d953a704baa583b08d73b9357ed|72f988bf86f141af91ab2d7cd011db47|1|0|637043376330525158&sdata=ylRVQWKZ5yY%2Bwlh3YBr5HVRBaelCQtpzi1Wn1eduPn0%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJZZVISVJMTSOHVTO6Y4AULQKD7DZANCNFSM4IXQ5WHA&data=02|01|v-qiwe%40microsoft.com|78930d953a704baa583b08d73b9357ed|72f988bf86f141af91ab2d7cd011db47|1|0|637043376330535153&sdata=5W5fw4amKbZ%2FWQqA58IpQoyY%2FSE6nB9HNgPa5nG9Hvc%3D&reserved=0.


Provide predicted aligned error (PAE) for locally folded proteins

Would it be possible to output PAE for locally folded proteins?
I know, that it is possible to get PAE from the ATLAS, but how can I get pae for proteins folded on my computer?

You can get the PAE using this code snippet (adapted from the ESMFold ColabFold notebook):
with torch.no_grad():
        output = model.infer(sequence)

#Retrieve PAE values from the model outputs
pae = (output["aligned_confidence_probs"][0].cpu().numpy() * np.arange(64)).mean(-1) * 31
mask = output["atom37_atom_exists"][0,:,1] == 1
mask = mask.cpu()
pae = pae[mask,:][:,mask]
np.savetxt("filename.txt", pae, "%.3f")