fix(app): Add command text for reloadLabware

Overview
Closes EXEC-681.
Test Plan and Hands on Testing

Changelog
Render reloadLabware commands, which happen when you call the PAPI set_offset() method, as "Reloading [labware name]".
Review requests
I'm mostly just blind-following the existing patterns. Are there any nuances I missed?
Risk assessment
Low.

Failing test appears is unrelated to this PR.


False positive on NAC/Ensure the AWS EKS Cluster is not public

Finally got this working on a cluster, but now getting false positives?
Run `aws eks describe-cluster` and review the contents of the `resourcesVpcConfig` block. `endpointPrivateAccess` should be `true`. Or, if `endpointPublicAccess` is `true`, `publicAccessCidrs` should be set to something other than `<IP_ADDRESS>/0`.

🚽 aws eks --no-verify-ssl describe-cluster --name int1
{
    "cluster": {
        "name": "int1",
        "arn": "arn:aws:eks:us-east-1:xxxx:cluster/int1",
        "createdAt": "2019-06-09T22:51:27.668000-04:00",
        "version": "1.13",
        "endpoint": "https://xxxx.us-east-1.eks.amazonaws.com",
        "roleArn": "arn:aws:iam::xxx:role/xxxx",
        "resourcesVpcConfig": {
            "subnetIds": [
                "subnet-xxx",
                "subnet-xxx"
            ],
            "securityGroupIds": [
                "sg-xxx"
            ],
            "vpcId": "vpc-xxxx",
            "endpointPublicAccess": false,
            "endpointPrivateAccess": true,
            "publicAccessCidrs": []
        },
        "logging": {
            "clusterLogging": [
                {
                    "types": [
                        "api",
                        "audit",
                        "authenticator",
                        "controllerManager",
                        "scheduler"
                    ],
                    "enabled": true
                }
            ]
        },
...

Thanks for calling that to our attention, @chiefy.  It should be addressed via https://github.com/darkbitio/inspec-profile-eks/commit/bd4486858f0aed4838d1845ad26300df116df3ae which was tagged 0.1.3.  I've bumped that version in the Dockerfile here in the mkit repo and pushed an updated image to our Dockerhub side.

Closing for now.  Thanks!


Allow peek content to scroll

First, great lib. Thanks.
I think it would be great to allow peeked content to scroll. Similar to the behavior when expanded, when you hit the end of the scrollable, then it pulls open to an expanded view. Definately as a state switch, because it's more of an edge case.
Also, to add to this, I think it would be super cool to define a second sheet that hangs below the first (or just wherever you want) that animates in as you move from peeked to expanded. Could be cool.

Can you give some use cases for either of these?

Sure. In my case I'm extending bottom sheet to make a shopping cart.
When an item is added, the cart slides up into "peek". This basically is a list view. I've extended the peek size to either be the height of this product list or the maximum size that would be found if the view was expanded. Conversely, the product scrollview also sizes itself to the maximum size of the parent view too.
The end result is a a scroll list of products. When drag the view into an "expanded" state the payment methods from your account are revealed below and you've moved into checkout mode. If you drag back down you move back into cart mode. If you dismiss the view you're back into buy mode.
So in my case, I need to have a similar action to expanded. Where while in peeked if you drag up or down, it passes that event to the sheet unless the sheet isn't scrollable, upon which it then drags the bottomSheetView into expanded or hidden states.
This is a very cool library and I think it would also be very handy to further add a dynamic element to attach it to any side. I could see uses with top-down peeks as well as left or right peeks.
Again, thanks for the open source.

This wouldn't really work with how the library works. When the bottom sheet is created, its height is actually the same as the parent. This creates an issue where if you wanted to scroll the list in its peeked state, the bottom ___ items of the list would conceivably never be visible in the peeked portion, since they're considered visible in the context of the whole sheet itself. The sheet isn't really expanding when you pull it up, rather you're just translating it up from off-screen.
A potential work around for this would maybe be to allow for scrolling until it hits the bottom, and then translating the sheet up. I think this behavior would be weird and not exactly obvious, so I don't think it's something we want to figure out just for a niche use case. You'd probably be better off extending BottomSheetLayout and implementing this behavior yourself.

I think it would also be very handy to further add a dynamic element to attach it to any side

There's something for that in the material spec and support library already. It's called the Navigation Drawer ;)

Going to close this for now, we can revisit if more people express an interest.


spark.ui.enabled configuration ignored

Disabling the spark UI with a configuration option using spark.ui.enabled (as documented here) doesn't appear to be respected. e.g.
from pyspark import SparkConf, SparkContext

conf = SparkConf()
conf.set("spark.executor.memory", "2g")
conf.set("spark.ui.enabled", "false") # or conf.set("spark.ui.enabled", False)
SparkContext()

import databricks.koalas as ks

I have tried both a string value and a boolean value. I still get this warning when I run koalas with or without that option:
21/06/29 00:05:58 WARN Utils: Service 'SparkUI' could not bind on port 4040. Attempting port 4041.

(or indeed if I try setting it to "true"/True)

In which env did you run the codes above? It's likely that SparkContext is already initialized before you running these codes.

I think I forgot the conf=conf in SparkContext... my bad


Consider scrubbing troubleshooting sections of help to be organized by symptom

Consider organizing troubleshooting sections by symptom/problem so that it's easier for people to find the solution to the problem they are experiencing in that moment.

@brendandburns you've taken on the troubleshooting section -- does this fit with how you've thought of the doc?

We definitely need some kind of organization of our troubleshooting docs:

docs/user-guide/application-troubleshooting.md
docs/user-guide/debugging-services.md
docs/user-guide/compute-resources.md#troubleshooting
https://github.com/GoogleCloudPlatform/kubernetes/wiki/Debugging-FAQ
https://github.com/GoogleCloudPlatform/kubernetes/wiki/Services-FAQ
https://github.com/GoogleCloudPlatform/kubernetes/wiki/User-FAQ

And also

docs/cluster-troubleshooting.md

I'll use docs/troubleshooting.md as a central place for organizing all other troubleshooting docs.

SGTM


WebGL LiveQuery Callbacks are not fired.

In WebGL the sub-scribal client is not re-acting to events or firing callback methods.
Move LiveQuery for WeGL listener to JS with jslib interface to Unity,

Created js code in Templates index.js for connecting and communicating with a websocket.
Created MoralisLiveQueriesGL.cs and moralisLiveQueryGl.jslib as a bridge between Unity and javascript.

Added a bunch of logs in the js side to figure out why W.S. connection is connecting but response is false.
Current response received and passed through final method as {"isTrusted": true} but response sent to C# layer is undefined.

Receiving the 'true' response, back to original error '190' nothing in Google or Unity docs about what this means ...

Fix checked into the metaverse branch.


Replacing sys.exit in python tools

In GitLab by @hlux on Apr 12, 2018, 11:34
Matti has raised an issue with the sys.exit statements in our systems code. He suggested replacing them on the following grounds (email to J. Shimwell on same topic):

sys.exit() considered harmful:
Ok to be honest, I don't really know what people think of this, but when running in an IDE at least, this throws less clear error messages:
print('material_card_name must be provided when making a serpent_material_card from a material')
sys.exit()
gives:
Material()
A list of elements present within the material must be specified
An exception has occurred, use %tb to see the full traceback.
SystemExit
/home/mc2056/anaconda3/lib/python3.6/site-packages/IPython/core/interactiveshell.py:2870: UserWarning: To exit: use 'exit', 'quit', or Ctrl-D.
warn("To exit: use 'exit', 'quit', or Ctrl-D.", stacklevel=1)
Basically the first thing you see is SystemExit in bright red :D
whereas,
raise ValueError('A list of elements present within the material '
                           'must be specified')

Material()
Traceback (most recent call last):
File "", line 1, in
Material()
File "/home/mc2056/Code/neutronics_material_maker/neutronics_material_maker/nmm.py", line 1478, in init
raise ValueError('A list of elements present within the material '
ValueError: A list of elements present within the material must be specified
The latter gives me an Error which relates to the problem, and a code line_number (without having to type %tb in the console). The former ok I guess if you are working from the command line, but in some IDEs / consoles, sys.exit() will actually close the console (at least in my experience).

Hence, we could consider a different approach to sys.exit in python.  @jmorris-uk, @mkovari and @mkumar any comments?
I always thought appropriate error treatment is to catch all errors and if necessary cleanly exit with sys.exit. However, following Matti's argument this seems to be extremely unhelpful when running with his IDE.

This isn't a problem any more


Build failure after adding project reference

After adding the web app project reference to resource group project my build stopped working. Below is the error;
error MSB4057: The target "Package" does not exist in the project.
Done building project "FunctionApp1.csproj" -- FAILED.
Done building project "ResourceGroup1.deployproj" -- FAILED.
It seems like it's not generating the package.zip.
I'm currently using using VS2019 (<IP_ADDRESS>). RG project is using .NET Framework 4.8 & app project is .NET Core 3.

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

ID: 640c3387-4a7e-71d6-5c53-43ad28b65234
Version Independent ID: 62ac189f-ef81-48d2-b726-d51fa3c3262b
Content: Create & deploy Visual Studio resource group projects - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/create-visual-studio-deployment-project.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@minh-dang Thank you for your query, we will further look into it and get back to you at the earliest

@BharathNimmala-MSFT - Thanks! Just want to add some extra info which might be useful. I was actually trying to do it with Azure Function V3 (NET Core 3). When testing with a ASP.NET Core Web App, that approach work. What different between the two are the project SDK.
Azure Function use this - Project Sdk="Microsoft.NET.Sdk"
ASP.NET App use this - Project Sdk="Microsoft.NET.Sdk.Web"
Seems like the web version generate the package.zip but the other one does. I actually wanna try get Azure Function one to work with it so any helps on this matter is greatly appreciated!

@minh-dang Thanks again for reaching out to us. If you are referencing to the step 7 with in the add code to project section in the document  and trying to add extension to the project and in your case adding Azure Function as extension and you are not able to see the package.zip please let us know. A quick repro with VS 2017 helped to create the .zip package.

Please note that this channel (the Azure Docs repository) is only for driving improvements related to the Azure documentation. At this point it appears that there's no further edits or updates needed to the documentation. We will go ahead and close this issue , however we will continue to investigate the specific issue you are seeing with you directly and update this thread with any findings that may help others. Requesting you to please send an email to<EMAIL_ADDRESS>with link to this thread and additional details on which step in the document you are trying to run the build and seeing errors so that we can take it forward.
Looking forward to hear from you. Thank you.


Fix blackened the code

Nothing much, just applied black as below
black -l 80 .
https://black.readthedocs.io/en/stable/

dope


Fix typo group -> project in readme

On reading your readme file in the projects section I've spotted it looks like you meant project here, rather than group.

Glad you pay more attention than I do!

Ugh, sorry @stevencrake-nscale

The base branch requires all commits to be signed.

Get secure!

Tableflip

What was the magic sauce?


[Bug]: error: Class "App\Filament\Resources\RadioButtonImage" not found

What happened?
i get this error:
Class "App\Filament\Resources\RadioButtonImage" not found
How to reproduce the bug
RadioButtonImage::make('layout')
->label('Templates')
->options(
Template::all()->pluck('image', 'id')->toArray()
),
This is in my resource
Package Version
latest
PHP Version
8.1
Laravel Version
10
Which operating systems does with happen with?
Linux
Notes
RadioButtonImage::make('layout')
->label('Templates')
->options(
Template::all()->pluck('image', 'id')->toArray()
),
i get this error:
Class "App\Filament\Resources\RadioButtonImage" not found

You need the Facade:
use IbrahimBougaoua\RadioButtonImage\Facades\RadioButtonImage;


Release 0.2.26

Release of

#494
#419
#503
#411
#506
#509
#511

To do

[x] Bump version number in package.json
[x] Generate changelog
[ ] Greate GitHub release

@SayyareHamit
This is ready to be tested in the QA environment 🧪
-----------------------------
| Platform     | Web       |
| Environment  | QA        |
| Version      | 0.2.26    |
| Commit       | 99c8d6d   |
-----------------------------


Receiving and displaying

I can't seem to read in the items from the database and display them properly. I can't figure out why, I'm using an ajax get method when on click but nothing appears.

I created a function that had the ajax call inside of it and that did the trick


feat(Markdown): render MarkdownCheckboxes

Overview

Adds feature to add checkboxes within Markdown

PR Checklist

[ ] Related to designs:
[x] Related to JIRA ticket: web-2007
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change
[x] This PR includes testing instructions tests for the code change
[x] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Navigate to the Markdown Gamut page
See the Markdown checkboxes under List section
Ensure you can check and uncheck the checkboxes
Profit!

PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.165-alpha.acd9a4.0@codecademy/gamut-labs@33.0.1-alpha.acd9a4.0@codecademy/gamut@48.3.1-alpha.acd9a4.0@codecademy/monaco@0.6.60-alpha.acd9a4.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62d95d37c5466010a5016d6f--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.165-alpha.2508f4.0@codecademy/gamut-labs@33.0.1-alpha.2508f4.0@codecademy/gamut@48.3.1-alpha.2508f4.0@codecademy/monaco@0.6.60-alpha.2508f4.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62d99a47df1fcc1fc3835c7a--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.165-alpha.a0c528.0@codecademy/gamut-labs@33.0.1-alpha.a0c528.0@codecademy/gamut@48.3.1-alpha.a0c528.0@codecademy/monaco@0.6.60-alpha.a0c528.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.165-alpha.31078e.0@codecademy/gamut-labs@33.0.1-alpha.31078e.0@codecademy/gamut@48.3.1-alpha.31078e.0@codecademy/monaco@0.6.60-alpha.31078e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62df067b680ca91f0beb8647--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.169-alpha.2cd356.0@codecademy/gamut-labs@33.0.5-alpha.2cd356.0@codecademy/gamut@49.1.1-alpha.2cd356.0@codecademy/monaco@0.6.64-alpha.2cd356.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62e4324d72176d28a862c148--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.169-alpha.00ef59.0@codecademy/gamut-labs@33.0.5-alpha.00ef59.0@codecademy/gamut@49.1.1-alpha.00ef59.0@codecademy/monaco@0.6.64-alpha.00ef59.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62e43f15e5c2793f70a6ce08--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.169-alpha.39fba2.0@codecademy/gamut-labs@33.0.5-alpha.39fba2.0@codecademy/gamut@49.1.1-alpha.39fba2.0@codecademy/monaco@0.6.64-alpha.39fba2.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62e7e924fb289d661c7416fd--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://62eaa0e229789a0b96c062b6--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.171-alpha.7124cf.0@codecademy/gamut-labs@33.0.7-alpha.7124cf.0@codecademy/gamut@50.0.1-alpha.7124cf.0@codecademy/monaco@0.6.66-alpha.7124cf.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62eaaa625abaff006510debb--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.171-alpha.9afe98.0@codecademy/gamut-labs@33.0.7-alpha.9afe98.0@codecademy/gamut@50.0.1-alpha.9afe98.0@codecademy/monaco@0.6.66-alpha.9afe98.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62ebcd8f34651b147941bd40--gamut-preview.netlify.appDeploy Logs


to-do list

[x] FLOW CONTROL https://httpwg.org/specs/rfc9113.html#n-the-flow-control-window
[ ] A server that is attempting to gracefully shut down a connection SHOULD send an initial GOAWAY frame with the last stream identifier set to 231-1 and a NO_ERROR code. This signals to the client that a shutdown is imminent and that initiating further requests is prohibited. After allowing time for any in-flight stream creation (at least one round-trip time), the server MAY send another GOAWAY frame with an updated last stream identifier. This ensures that a connection can be cleanly shut down without losing requests. https://httpwg.org/specs/rfc9113.html#GOAWAY
[ ] a server MAY request that the client abort transmission of a request without error by sending a RST_STREAM with an error code of NO_ERROR after sending a complete response (i.e., a frame with the END_STREAM flag set).
[ ] A server SHOULD treat a request as malformed if it contains a Host header field that identifies an entity that differs from the entity in the ":authority" pseudo-header field. The values of fields need to be normalized to compare them (see Section 6.2 of [RFC3986]). An origin server can apply any normalization method, whereas other servers MUST perform scheme-based normalization (see Section 6.2.3 of [RFC3986]) of the two fields.
[ ] A server MUST NOT indicate that a stream has not been processed unless it can guarantee that fact. If frames that are on a stream are passed to the application layer for any stream, then REFUSED_STREAM MUST NOT be used for that stream, and a GOAWAY frame MUST include a stream identifier that is greater than or equal to the given stream identifier.

Remove dispatcher-stream queue
It leads to code like this. It's limited to 1 single element anyway. Increasing the limit has never show any improvement.
It should be replaced by something like ValueAsync (but using just 2 future vars). This solves the issue because there can only ever be a single value waiting, and when closing from a stream, the dispatcher can deal with the frame being put. It can latter be changed to a synced value that needs to be consumed before resuming if needed.

fix: flow-control must take into account data frame padding.


kernel as kwarg or part of struct

There appear to be two ways to pass a kernel to inducingpoints, either as a field of alg or as a kwarg to inducingpoints. Should we choose a single canonical way to do this?

Ups you're right. I would be of the opinion that it should always be a kwarg but StdDPP and kDPP don't respect that...


🚧Refactor unit test assertions

Complete The Item Below

[X] I have updated the title without removing the 🚧 emoji.

Description
Refactor all of the unit test assertions to use the FluentAssertions library.
This will greatly cleanup the unit testing side of things and make the creation of unit tests easier.
Acceptance Criteria

[ ] Latest version of FluentAssertions Nuget package added to the project
[ ] Refactor the files below:

[ ] LoadEmbeddedResourceExceptionTests.cs - 50 lines
[ ] SystemDisplayExceptionTests.cs - 50 lines
[ ] AtlasSubTextureDataTests.cs - 114 lines
[ ] CornerRadiusTests.cs - 159 lines
[ ] FontAtlasMetricsTests.cs - 43 lines

ToDo Items

[x] Change type labels added to this issue.  Refer to the Change Type Labels section below.
[x] Priority label added to this issue.  Refer to the Priority Type Labels section below.
[x] Issue linked to the correct project (if required).
[ ] Issue linked to the correct milestone (if required).

Issue Dependencies
No response
Related Work
No response
Additional Information:
Change Type Labels

Change Type
Label

Bug Fixes
🐛bug

Breaking Changes
🧨breaking changes

New Feature
✨new feature

Workflow Changes
workflow

Code Doc Changes
🗒documentation/code

Product Doc Changes
📝documentation/product

Priority Type Labels

Priority Type
Label

Low Priority
low priority

Medium Priority
medium priority

High Priority
high priority

Code of Conduct

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

Hi, can I take this issue? 😃

Hi, can I take this issue? 😃

Indeed!!


Error when using npm-check-updates js api after build by unbuild

Related issue: https://github.com/raineorshine/npm-check-updates/issues/1204
Reproduction: https://github.com/qmhc/ncu-test
NCU author said it is an issue of unbuild, NCU works correctly as a module both in js and ts.
But I am not sure about this, currently I have used some packages and only found NCU will take errors.

~> #468


error parsing multiline array of tuples

I have discovered a very peculiar parsing error with the following examples:
actor Main
  new create(env: Env) =>
    let a: Array[(String, String)] = [
      ("0", "1"),
      ("1", "2"),
      ("2", "3")
    ]

    for (x, y) in a.values() do
      env.out.print(x+" "+y)
    end

produces the output:
0 1
1 2
2 3

But removing a comma, as so:
let a: Array[(String, String)] = [
      ("0", "1"),
      ("1", "2") // no comma
      ("2", "3")
]

produces the output:
0 1
2 3

But if the tuples contain U8s rather than strings, as so:
let a: Array[(U8, U8)] = [
      (0, 1),
      (1, 2) // no comma
      (2, 3)
]

throws an error:
main.pony:6:9: Cannot infer type of unused literal
      (1, 2) // no comma
        ^

We covered this in the sync call. While the parser is doing the right thing, the error message is totally unhelpful. We could catch a TK_SEQ with more than one element as an array element and ast_error something useful that tells the programmer "did you meant to put a sequence here? if so, put it in parens".

I tried @Theodus's examples replacing the , with ; and got much better error messages
The only one I have questions about is:
actor Main
  new create(env: Env) =>
let a: Array[(String, String)] = [
      ("0", "1"),
      ("1", "2")
      ("2", "3")
]

    for (x, y) in a.values() do
      env.out.print(x.string() +" " + y.string())
    end

which produces...
Error:
/Users/sean/code/pony-scratch/issue-1017/a.pony:3:34: syntax error: unterminated array literal
let a: Array[(String, String)] = [
                                 ^
    Info:
    /Users/sean/code/pony-scratch/issue-1017/a.pony:4:17: expected terminating ] before here
          ("0", "1"),

which feels like it could be better.

I'm marking this as "needs discussion during sync" to address the change in this related to recent Arrays-as-sequence change

At sync we decided new error message is good. No need to address anymore as there is no longer an issue.


Update dependency zod to v3.22.4

This PR contains the following updates:

Package
Type
Update
Change

zod (source)
dependencies
patch
3.22.3 -> 3.22.4

Release Notes

colinhacks/zod (zod)
v3.22.4
Compare Source
Commits:

d931ea3 Lint
8e634bd Fix prettier
4018d88 docs: add @sanity-typed/zod to ecosystem (#2731)
15ba5a4 docs: add zod-sandbox to README ecosystem links (#2707)
699ccae Export jsdoc with @deprecated when building (#2717)
dfe3719 Fix sanity-typed links (#2840)
cd7991e fix ulid regex (#2225)
7cb4ba2 Remove stalebot
9340fd5 Lazy emojiRegex
e7a9b9b 3.22.4

Configuration
📅 Schedule: Branch creation - "before 4am on the first day of the month" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Self-Hosted Renovate Bot. Find all logs here.

 You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Audit tests

As a maintainer, I should keep a good test coverage with relevant test cases.
Audit tests to check i
test cases are relevant
all tests pass
coverage is above 85%
Acceptance criteria
1 - updated tests with high coverage

tests still under update due to #20

Testing the SDK integration with Flutter under development is unproductive, due to the amount of mocking necessary.
We are using #8 for testing the SDK manually. After 1.0 we can create automated tests that will do e2e testing.

Tests for packages doesn't work without host application. Need to implement host application to run tests

Automated tests doesn't work even with host application. The build for tests fails because TikiSdk does not have arm64 architecture. Needs further investigation

Added a host app in the project and tests worked with manually built binaries.
https://github.com/tiki/tiki-sdk-flutter/issues/89 will automate the creation of the binaries.
https://github.com/tiki/tiki-sdk-flutter/issues/85 will solve the failed tests (errors produced by the method channel)


Merge ClusterInstall contract to master

/assign @dgoodwin @joelddiaz
brings in implementation of clusterinstall contract from cluster-installs branch

/lgtm


NME for Gaussian Splatting

WIP

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15677/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15677/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15677/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15677/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15677/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15677/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15677/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15677/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15677/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15677/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15677/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15677/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15677/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15677/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15677/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15677/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

LGTM!
Have you been able to test in WebGPU?

PG is fine with standard GS and nodematerial but in NME, the preview never finishes loading the GS; even though I can see it  being loaded. The message "Please wait" doesn't disappear. I'm taking a look.


DataLoader seems to infer more Data objects than exist?

🐛 Bug
I am trying to train a model using the PyTorch Geometric Dataloader. However, I noticed that sometimes it gives me this error:
100%|██████████| 2301/2301 [01:39<00:00, 23.11it/s]
100%|██████████| 203/203 [00:09<00:00, 22.01it/s]
100%|█████████▉| 203/204 [00:09<00:00, 21.07it/s]
Traceback (most recent call last):
  File "/path/to/run2.py", line 459, in <module>
    results = test_model()
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/autograd/grad_mode.py", line 26, in decorate_context
    return func(*args, **kwargs)
  File "/path/to/run2.py", line 130, in test_model
    for data in tqdm(test_loader):
  File "/home/vym1/nn2/lib/python3.7/site-packages/tqdm/std.py", line 1167, in __iter__
    for obj in iterable:
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 435, in __next__
    data = self._next_data()
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 1065, in _next_data
    return self._process_data(data)
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 1111, in _process_data
    data.reraise()
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/_utils.py", line 428, in reraise
    raise self.exc_type(msg)
FileNotFoundError: Caught FileNotFoundError in DataLoader worker process 11.
Original Traceback (most recent call last):
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/utils/data/_utils/worker.py", line 198, in _worker_loop
    data = fetcher.fetch(index)
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/utils/data/_utils/fetch.py", line 44, in <listcomp>
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch_geometric/data/dataset.py", line 187, in __getitem__
    data = self.get(self.indices()[idx])
  File "/path/to/DataSet.py", line 171, in get
    data = torch.load(osp.join(self.processed_dir, 'data_{}_{}.pt'.format(idx, self.split)))
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/serialization.py", line 581, in load
    with _open_file_like(f, 'rb') as opened_file:
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/serialization.py", line 230, in _open_file_like
    return _open_file(name_or_buffer, mode)
  File "/home/vym1/nn2/lib/python3.7/site-packages/torch/serialization.py", line 211, in __init__
    super(_open_file, self).__init__(open(name, mode))
FileNotFoundError: [Errno 2] No such file or directory: 'dataset/social_network.line_graph/processed/data_3045_test.pt'

You can see from the status bars above the error that this only occurs with the test dataset (the last bar), and not the train (top) or validation (middle). The code for Data generation is the same; just a name change.
There are 3045 test Data files that I wrote out (0 - 3044). I'm not sure why it's asking for the 3045th one. I also noticed that sometimes if I restart the run this error vanishes and doesn't appear. Is there a reason why this might be happening?

This is weird. I guess self.len is set to the correct value? Are you using num_workers > 0?

Yes, I am using 16

Can you test if the same error occurs in case if less workers/zero workers?

I will try it. Will zero workers still allow it to run? Is num_workers the number of additional workers?

num_workers=0 means that data loading will happen in the main thread.

Seems to work better now, thanks.


python312Packages.weaviate-client: 4.5.1 -> 4.5.6

Diff: https://github.com/weaviate/weaviate-python-client/compare/refs/tags/v4.5.1...v4.5.6
Changelog: https://github.com/weaviate/weaviate-python-client/releases/tag/v4.5.6
Description of changes
Fix build (https://hydra.nixos.org/build/257141426)

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 307728 run on x86_64-linux 1

2 packages marked as broken and skipped:

chatgpt-retrieval-plugin
chatgpt-retrieval-plugin.dist

4 packages built:

python311Packages.weaviate-client
python311Packages.weaviate-client.dist
python312Packages.weaviate-client
python312Packages.weaviate-client.dist


refactor: more consistent with other listener managers

Renamed EventAnalyzerListenerFactoryManager to IEventAnalyzerListenerManager
Consistent with:

ISegmentParserListenerManager
ErrorLogParserListenerManager
PerfDataParserListenerManager

Renamed ILogAnalysisListenerFactoryManager to ILogAnalysisListenerManager
Consistent with:

ISegmentParserListenerManager
ErrorLogParserListenerManager
PerfDataParserListenerManager

Removed extends Service from IEventAnalyzerListenerManager & ILogAnalysisListenerManager
Consistent with ISegmentParserListenerManager plus they were not being registered as service implementations.
Renamed EventAnalyzerListenerFactoryManager.factories() to IEventAnalyzerListenerManager.getEventAnalyzerListenerFactories()
Consistent with:

ISegmentParserListenerManager.getSpanListenerFactories()
ILogAnalysisListenerManager.getLogAnalysisListenerFactories()

I can't rerun the fail task, GHA seems not responding.

Assuming moving forward I'm allowed to merge once tests pass and reviewers have approved changes. Let me know if not.

It is OK to merge, but you should remove all commit details, as they are not very useful.
We are using squash merge.
You could check the latest commit log, it includes two useless sentences.

Ah, I see. Will do going forward.


tour: pic stage do not work

Context: https://tour.golang.org/moretypes/18

Ha should read the exercise first ;)


Harden NAT outbound NACL rules for internet traffic

User Story
We need to harden to private subnet NACL rules and restrict outbound connections to TCP port 80 and port 443. We also require an inbound rule for TCP traffic on ephemeral ports only.
Inbound:
140 | <IP_ADDRESS>/0 | TCP | 1024-65535 | ALLOW | Allows inbound return traffic from the NAT device in the public subnet for requests originating in the private subnet.
Outbound:
100 | <IP_ADDRESS>/0 | TCP | 80 | ALLOW | Allows outbound HTTP traffic from the subnet to the internet.
110 | <IP_ADDRESS>/0 | TCP | 443 | ALLOW | Allows outbound HTTPS traffic from the subnet to the internet.
These rules need to be updated in the the Terraform NACL config.

Reviewed current configuration
In addition to the listed changes I also found that public and private subnets are controlled with the same NACL rules in the automation, this requires to be changed in this ticket. The requirements for private and public subnets with regards to NACL rules are not the same.

Separated Public and Private NACL rule configuration
Configured private NACLs to allow ingress high ports only for
centralised NAT
Configured public NACLs to all ingress for 443 only as default


ISPN-6402 Default GMS.join_timeout is too long

https://issues.jboss.org/browse/ISPN-6402

Change the production default to 5000ms, and the tests to 2000ms.
Set the jgroups.join_timeout property for Arquillian/JCache tests

Triggered CI manually

I ran the server testsuite locally and it passed. @danberindei would it be possible to have this for 8.2.x as well ?

Pushed to master. Thanks.

@tristantarrant https://github.com/infinispan/infinispan/pull/4199 includes both this and the ISPN-6399 workaround.


Error in build_pc_matrices Function: ValueError Due to Mismatched Index Arrays

hi,
I am encountering an issue with the PhaBOX software while running the analysis. Specifically, I receive a ValueError when the software attempts to build the sparse matrix in the build_pc_matrices function. The error message is:
ValueError: mismatching number of index arrays for shape; got 0, expected 2
Building the cluster and profiles (this may take some time...)
Using MCL to generate PCs.
Saving files
Traceback (most recent call last):
File "/data/ckzhu/ckzhu1/PhaBOX/main.py", line 460, in
matrix, singletons = build_pc_matrices(profiles, contigs_csv_df, pcs_csv_df)
File "/data/ckzhu/ckzhu1/PhaBOX/scripts/ulity.py", line 309, in build_pc_matrices
matrix = sparse.coo_matrix(([1]*len(profiles), (zip(*profiles.values))), shape=(len(contigs), len(pcs)),
File "/data/ckzhu/ckzhu1/miniconda3/envs/phabox/lib/python3.9/site-packages/scipy/sparse/_coo.py", line 99, in init
self._check()
File "/data/ckzhu/ckzhu1/miniconda3/envs/phabox/lib/python3.9/site-packages/scipy/sparse/_coo.py", line 188, in _check
raise ValueError('mismatching number of index arrays for shape; '
ValueError: mismatching number of index arrays for shape; got 0, expected 2
Could you please assist in resolving this issue?
Thank you for your support!
Fang

Hi there,
Please check whether your scipy=1.7.1 using conda list
If not, please run conda install scipy=1.7.1
Best,
Jiayu

Hi there,
The PhaBOX is upgraded to a 2.0 version with more powerful usage. Besides, it can handle all kinds of viruses now. Feel free to check!
Best,
Jiayu


Feature : Adding Page Titles for All HTML Files

GitHub Issue: Add Page Titles to HTML Files:

Adding a appropriate page title for all html files in the website.

@ANSHIKA-26
Pls assign me this issue, give me labels, and level.

hey , this can never be merged , a lot of people are simultaneously working on this project , we would need to change all the links , which might cause more issues than solve hence i do not wish to implement this even though its needed , thanks


Fixes a bug in Sample.Any

(I introduced the bug while merging a different PR...)
Also adds a test for the combine fn and exposes the fn.
Documents the difference between any() and fixedSize().
Fixes a bug in CombineFnTester for empty input.
R: @wtanaka

Alternatively R: @tgroh


I don't seem to be getting generative tests with invalid values for path variables

I don't seem to be getting generative tests with invalid values for path variables - it is working for my request body, but not for query params; are there any simple gotchas?

Hi @jturnergresham
Thanks for reaching out.
At present, the generative tests include only the request body and headers.

Thanks Jay.


Fixing page width.

Fixing page width to 95% to make the page take up the full width. This is due to the body having being changed to absolute in a previous change. This fix makes the page go back to full width

Sorry I didn't realise you had set body to be position:absolute, what are you actually trying to do?


SOV.approveAndCall

SOV is ERC20 with an additional function: approveAndCall
ApprovalReceiver: base implementation of a contract which need approval
inherited contracts: RSOV, Staking, Vesting, DevelopmentVesting
FeeSharingProxy is an exception here since this contract needs approval for different tokens and uses SOV only for getting voting power of the user

I requested a review from evgeny as well

I requested a review from evgeny as well


train  cls always nan...

I modified your program ,train according to my own data set, and in the process of training, CLS loss is always nan,
2019-04-27 22:02:27: step90    image_name:b'70844.jpg' |	 loc_loss:11.512925148010254 |
cls_loss:nan |	 cnt_loss:0.6479541063308716 |	total_loss:nan |
per_cost_time:8.343699216842651s
I downloaded the pre-training model and put it in the designated place
cfgs.py:CLASS_NUM = 11
labe_dict:
elif  cfgs.DATASET_NAME == 'xxxx':
NAME_LABEL_MAP = {
'0': 0,
'1': 1,
'2': 2,
'3': 3,
'4': 4,
'5': 5,
'6': 6,
'7': 7,
'8': 8,
'9': 9,
'10': 10,
'11': 11
}
'0'  is  background
Is there anything else I need to change?

Sorry, I am still debugging this code, and I have only experimented on the coco and voc2007 datasets. Maybe you can try to adjust the learning rate and warm up （use multi_gpu_train.py）. @AiIsBetter

Sorry, I am still debugging this code, and I have only experimented on the coco and voc2007 datasets. Maybe you can try to adjust the learning rate and warm up （use multi_gpu_train.py）. @AiIsBetter

Yes, it's normal to change the learning rate. I forgot about it. But as you say, the results are worse. Hope to use the debugging code as soon as possible. Thank you。 But your code is enough for me to study the paper, haha.

Sorry, I am still debugging this code, and I have only experimented on the coco and voc2007 datasets. Maybe you can try to adjust the learning rate and warm up （use multi_gpu_train.py）. @AiIsBetter

Yes, it's normal to change the learning rate. I forgot about it. But as you say, the results are worse. Hope to use the debugging code as soon as possible. Thank you。 But your code is enough for me to study the paper, haha.

hello ,have you solved this problem yet?，I have the same problem.


add ocamlformat github action

expectation for this PR is to fail (since the code isn't ocamlformat-formatted).
//cc @PizieDust

So, this is my proposal.  What do you think?

So, this is my proposal.  What do you think?

This will be amazing. Is the pipeline costly to run? If so, we can always just try to remember to run ocamlformat before pushing, else this is a really awesome idea to have the CI format and add a commit.

It took 3 minutes. I think if you've your editor with ocamlformat, that's fine. But if not, don't worry, and the CI will take care of it.

It took 3 minutes. I think if you've your editor with ocamlformat, that's fine. But if not, don't worry, and the CI will take care of it.

Okay great. I do have ocamlformat. I guess I just have to run ocamlformat --inplace before pushing


Fixing incorrect cardano-cli version number

1.14 isn't out yet, in progress.

@disassembler I think you made mistake closing it, look what is happening:
PR from:
input-output-hk:dcoutts/release-1.14.0 from matiwinnetou:dcoutts/release-1.14.0

@matiwinnetou: Would you like to reopen this against master?


Add fingerprint and exists method to StaticFile. Fixes #52

Should we make fingerprint a getter or a callable? If a callable, we may be able to extend it (i.e. you could request the md5 instead of the default sha256). That's how Hugo works FWIW.
Also, should we make exists a callable (ie. exists()) because it has to make the fs operation?

Sounds good; I agree with your reasoning. I think the ability to change the hash function used for the fingerprint doesn't solve the "build as little as necessary, as much as needed" test, so we can avoid adding the ability to customize the hash function for the fingerprint for now, until someone needs it.
Regarding async within Nunjucks, it's actually possible – which is great! – but somewhat annoying. Using async calls in Nunjucks limits the tags you can use, and you need to change the syntax around things such as for loops:
https://mozilla.github.io/nunjucks/templating.html#asynceach
If we need to support async file operations in the future we can add something like an existsAsync for example (which is a bit nasty of a name, but at least obvious in how it works.
I'll go ahead and change the fingerprint to a property.

Updated to a property
Updated to use md5 as it's a bit faster than sha256


Sample code for the latest Sticker app in Teams?

The Sticker app/messaging extension in Teams (March 2020) is different from the app described here.  Is it possible to have this code updated to match that app?   In particular, the current sticker app has a nav bar to the left of the search interface.  How does one enabled that in a messaging extension?

Hi @GauravES ,
Sadly, the nav bar on the left is not currently available to third party developers, so the feature cannot be added to the template.
Thank you for your question! We can take it into consideration when prioritizing new features for the future.


CH-#123 - RSYSLOG COOKBOOK :: Extending Attributes in lieu of hardcoded values

Description
Removes hardcoded values of $Umask, $FileCreateMode and $DirCreateMode. They can now be set using the attribute.
Added attributes for $DirOwner, $DirGroup and $Templates. These can now be set using attributes.
Issues Resolved
https://github.com/chef-cookbooks/rsyslog/issues/123
Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

/cc @tas50

lgtm :+1:

I don't think this is correct, these properties (Owner, Group, CreateMode) are sticky and apply to all entries that are below them (rsyslog conf files are parsed top-down) - so they should be set just above each entry that they are needed, this PR would change the FileOwner etc.. for all log files instead of only for the ones that are needed

@Argoday This change is correcting previously hardcoded and missing attributes which would atleast let the user to set them.

Sry, to be more specific - this change improves the cookbook to allow the top-of-file defaults to be changed, however in regards to implementing {DRQS 93976638} and/or any other request that would want to use these settings - this comes up short.
As-is this enables all log entries to have a single owner however we should we able to set the owner individually for each log entry

@rmcleod8 - retracting my previous comment .. didn't pay enough attention to which repo this is

Thanks for the fixes


Provide more background information on the Taylor Diagram

Replace italics below with details for this issue.
Describe the Task
The Taylor Diagram has minimal background information, requiring users to refer to the original paper by Taylor.  To improve the user experience, provide additional information so users do not need to read Taylor's paper to use the plotting feature.
Time Estimate
Estimate the amount of work required here.
Issues should represent approximately 1 to 3 days of work.
Sub-Issues
Consider breaking the task down into sub-issues.

[ ] Add a checkbox for each sub-issue here.

Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee

[ ] Select engineer(s) or no engineer required
[ ] Select scientist(s) or no scientist required

Labels

[ ] Select component(s)
[ ] Select priority
[ ] Select requestor(s)

Projects and Milestone

[ ] Select Repository and/or Organization level Project(s) or add alert: NEED CYCLE ASSIGNMENT label
[ ] Select Milestone as the next official version or Future Versions

Define Related Issue(s)
Consider the impact to the other METplus components.

[ ] METplus, MET, METdataio, METviewer, METexpress, METcalcpy, METplotpy

Task Checklist
See the METplus Workflow for details.

[ ] Complete the issue definition above, including the Time Estimate and Funding Source.
[ ] Fork this repository or create a branch of develop.
Branch name: feature_<Issue Number>_<Description>
[ ] Complete the development and test your changes.
[ ] Add/update log messages for easier debugging.
[ ] Add/update unit tests.
[ ] Add/update documentation.
[ ] Add any new Python packages to the METplus Components Python Requirements table.
[ ] Push local changes to GitHub.
[ ] Submit a pull request to merge into develop.
Pull request: feature <Issue Number> <Description>
[ ] Define the pull request metadata, as permissions allow.
Select: Reviewer(s) and Development issues
Select: Repository level development cycle Project for the next official release
Select: Milestone as the next official version
[ ] Iterate until the reviewer(s) accept and merge your changes.
[ ] Delete your fork or branch.
[ ] Close this issue.

Thanks for creating the issue and the opportunity to provide input to the future documentation, Minna. I think some clarification is needed to avoid misinterpretation, but the User's Guide does discuss the fact that the RMSE is bias-corrected and that the standard deviation is normalized, but most of my misinterpretation came from the data which was present in the METviewer R data tab (originally from the MET Point-Stat CNT output) that didn't agree with what I was seeing in the plot. Perhaps some linkage between the R data values and the values used to generate the diagram in an example might be useful along with the formulas you provided in the discussion would help.

Would you be amenable to reviewing the updated documentation (not anytime
soon but before the coordinated release sometime in the summer)?
On Tue, Apr 16, 2024 at 4:43 PM jwraby @.***> wrote:

Thanks for creating the issue and the opportunity to provide input to the
future documentation, Minna. I think some clarification is needed to avoid
misinterpretation, but the User's Guide does discuss the fact that the RMSE
is bias-corrected and that the standard deviation is normalized, but most
of my misinterpretation came from the data which was present in the
METviewer R data tab (originally from the MET Point-Stat CNT output) that
didn't agree with what I was seeing in the plot. Perhaps some linkage
between the R data values and the values used to generate the diagram in an
example might be useful along with the formulas you provided in the
discussion would help.
—
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/issues/435#issuecomment-2060029456,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4UJHTBXM6773SWBBW6R33Y5WSPZAVCNFSM6AAAAABGKFZTPWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRQGAZDSNBVGY
.
You are receiving this because you were assigned.Message ID:
@.***>

Yes. I would appreciate the opportunity to help on that!


adds constructors to docs

I put it in the order I prefer:

constructors
isThing()
all other functions

Oops, Stream.of doesn't appear to be a function...

Thanks a lot for the PR :+1:
Overall this looks good but I've left a few comments regarding some smaller details :smile:

Looks great! Thanks a lot.


Milestone 0.9.1

Changes Unknown when pulling 7a2bf450559a009571eaf401efe9f3aa2860f747 on milestone-0.9.1 into ** on master**.

Changes Unknown when pulling cb27027adb6de01102e253915f9a75243b215dd4 on milestone-0.9.1 into ** on master**.


Request: gdb 14.1 integration

Thanks for the heads up. I didn't know there was a new release this past
weekend. I have a seemingly-working update (b0ce23d) which required a fix
to one of my patches and a workaround for a new GDB build bug. I'll need
to try it out for awhile before putting it into a release, though. GDB is
probably the least reliable program included in w64devkit, so I need to
make sure it's in a usable state. If you want to help, build the gdb-14.1
branch yourself and give GDB a thorough test drive.
The master branch hasn't been touched in nearly three months (46e4afd),
and I've been dogfooding that commit daily this whole period. That means
it's in a great state for a release, so I'm going to make a new w64devkit
release within the next day or so with GDB 13.1. If 14.1 behaves at least
as well 13.1, it will make it into the release after this.

GDB 14.2 has been released now. Maybe this is worth revisiting.

Thanks for pointing this out, @Peter0x44. I made a gdb-14.2 branch with
the upgrade, so you can try it for yourself. Unfortunately GDB 14.2 still
busted on i686-w64-mingw32. I didn't report it upstream, so the GDB devs
are probably unaware that it's been broken for two releases now.

@skeeto have you reported this upstream yet, or should I do it? It seems like it won't be addressed unless that happens, there is no one regularly testing i686-w64-mingw32.

I have not reported this upstream, in part because I haven't figured out
how to present it in a useful way. Inability to natively build GDB on the
target in question makes it a lot more complicated. So if you'd like to
tackle this report, go for it!

@skeeto if it helps to investigate any further, I had some upstream discussions:
i found the reason, gnulib/config.h does this:
#define __MINGW_USE_VC2005_COMPAT 1
which in turn enables this in _mingw.h:
#define _USE_32BIT_TIME_T
...
so bfd and gdb have different opinions what time_t really is

bug report: https://sourceware.org/bugzilla/show_bug.cgi?id=31636

Nice find.. seems either gdb or mingw have to agree.. in the meanwhile possibly a custom gdb patch (to conform to mingw time struct) would be fastest option to fix?

@oscarbg you're free to try! I think commenting out the mentioned define in gnulib may work, would be good to get confirmation.

Or alternatively, adding the define into CFLAGS manually. That sounds easier than the patch.

Thanks, @Peter0x44! I've updated the gdb-14.2 branch with the CFLAGS and
CXXFLAGS defines, and it seems to do the trick. It's still unclear to me
why this suddenly became a problem in 14: gnulib is unchanged between 13
and 14, and 13 was already using stat.
I'll test drive GDB 14.2 for awhile, especially 32-bit, and merge if it
works at least as well as 13.1.

it's still unclear to me why this suddenly became a problem in 14
Further details are in the bug report.

Thanks again, @Peter0x44. Amazing how effective it is simply to reach out
and ask someone more familiar with the problem! (I need to get better at
that.)
With this fix, GDB 14.2 has been working well for me today. No more
hiccups. So I'll likely merge that branch soon.


Add SampleCount statistics

Add SampleCount statistics, fix https://github.com/ivx/yet-another-cloudwatch-exporter/issues/124

Awesome thanks :)


Could not find com.hendraanggrian.appcompat:socialview:0.3. Required by:     project :app Search in build.gradle files

i cant add a social view and image croper dependencies in my projest

You are probably missing the deprecated jcenter repository.

please read https://github.com/ArthurHub/Android-Image-Cropper/issues/875
And update to the new repository
Please close this issue.
So the Warning can be more visible


Test Failiing due to MissingPluginException flutter_keyboard_visibility on macOS

Bug report
Describe the bug
══╡ EXCEPTION CAUGHT BY SERVICES LIBRARY ╞══════════════════════════════════════════════════════════
The following MissingPluginException was thrown while activating platform stream on channel
flutter_keyboard_visibility:
MissingPluginException(No implementation found for method listen on channel
flutter_keyboard_visibility)
Steps to reproduce
With the latest version update 0.3.0 we are getting this exception on running UI tests.
Expected behavior
Test should pass without error

Additional context
Test which is failing context:-

open model_sheet on TAP of button
Golden Test compare the view with the saved screenshot

Thanks for reporting. I will check this but meanwhile can you check this one: https://github.com/MisterJimson/flutter_keyboard_visibility/issues/115

@dhirajhimani Do you think we can share your learning in the ReadMe file? Do you think you can provide us how you fixed it so that we can share in ReadMe?

@ulusoyca I made the keyboard visibility to false, since this was the cause for failure
KeyboardVisibilityTesting.setVisibilityForTesting(false);


coralogix_tco_policy_logs value conversion error

Affected Resource(s)

coralogix_tco_policy_logs

Terraform Configuration Files

Please include all Terraform configurations required to reproduce the bug. Bug reports without a functional reproduction may be closed without investigation.
terraform {
  required_providers {
    coralogix = {
      source  = "coralogix/coralogix"
      version = "~> 1.8.17"
    }
  }
  required_version = "1.5.4"
}

provider "coralogix" {
  api_key = xxxxxxxxx
  env     = "EUROPE2"
}

locals {
  test = [{
    rule_type : "includes"
    names : ["-dev"]
  }]
}

resource "coralogix_tco_policy_logs" "tco_policy" {
  count = 1

  name        = "test"
  priority     = "medium"
  order        = count.index + 1
  severities   = ["debug", "verbose", "info", "error", "warning", "critical"]
  applications = local.test[count.index] # gives error
#  applications = local.test[0]                 # works
}

Expected Behavior
I would expect the resource to apply properly, without throwing any error.
Actual Behavior
A plan or apply fails before refreshing state, with the following message:
│ Error: Value Conversion Error
│
│   with coralogix_tco_policy_logs.tco_policy,
│   on coralogix_tco.tf line 118, in resource "coralogix_tco_policy_logs" "tco_policy":
│  118:   applications = local.test[count.index]
│
│ An unexpected error was encountered trying to build a value. This is always an error in the provider. Please report the following to the provider developer:
│
│ Received unknown value, however the target type cannot handle unknown values. Use the corresponding types package type or a custom type that handles unknown values.
│
│ Path: applications
│ Target Type: *coralogix.TCORuleModel
│ Suggested Type: basetypes.ObjectValue
If the "count.index" is instead hardcoded to 0, it works. The issue prevents me from creating multiple tco policy rules with configuration from a yaml file.
Steps to Reproduce

terraform plan

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Hi @leoffj.
Please let me know if v1.8.20 fixed it.

Yes, it is working as expected now! Thanks a lot for the quick fix.


Performance degradation when DB is unavailable

Hello,
We are using JDBC river plugin to collect some custom metrics from the database and it mostly works like a charm.
However, sometimes in test environments there are network issues or smth which prevent JDBC river plugin from getting data from the DB.
First, JDBC river just logs errors with appropriate time intervals:
[ERROR][river.jdbc.SimpleRiverSource] while opening read connection: jdbc:sqlserver://... The TCP/IP connection to the host ... has failed. Error: "Connection refused: connect. Verify the connection properties. Make sure that an instance of SQL Server is running on the host and accepting TCP/IP connections at the port. Make sure that TCP connections to the port are not blocked by a firewall..."

Then it takes 4 to 8 hours to notice that something goes wrong: ES logs are continuously growing on that node where JDBC river plugin is hosted: it is spawning lots of messages (once we saw 38000 entries\sec!) like
[INFO ][river.jdbc.RiverMetrics  ] pipeline org.xbib.elasticsearch.plugin.jdbc.RiverPipeline@18ab0a26 is running: river jdbc/jdbc_river_sqldatabase metrics: 0 rows, 0.0 mean, (0.0 0.0 0.0), ingest metrics: elapsed 58 seconds, 0.0 bytes bytes, 0.0 bytes avg, 0 MB/s

and leads to lack of space and translog corruption on that particular node. Additionally, so large amount of threads causes general performance degradation (at least there are lots of thread switches).
It seems that at some point in the code you're creating but not releasing the thread (please, see the attached image).

We are hosting elasticsearch nodes on Windows machines with server Java 1.8.25; we are using elasticsearch 1.4.0 and JDBC river version <IP_ADDRESS>.
Best regards,
Artyom

@jprante,
Sorry for a long delay.
We checked the issue in <IP_ADDRESS> and it seems to be not reproducible any more.
I will check <IP_ADDRESS> and give you another update.


Add a short description to setup.py

The short description will show up when doing a pip3 search openapi-spec-validator and the short description is also used by external tools.
For example the gitlab license tools takes, in order, description then long description. In this case, the long description is the contents of README.md: eg: https://jlecomte.gitlab.io/-/projects/rd-webhooks/-/jobs/325696880/artifacts/gl-license-management-report.json

@julien-lecomte thank you for the contribution. LGTM


Add more template tags

Is your feature request related to a problem? Please describe.
I would like to have a template tag to the read.amazon.com url like https://read.amazon.com/?asin=B003ODIWEG&language=en-US
Describe the solution you'd like
Add a new template tag. I.E readLink (like appLink for the kindle app)
Describe alternatives you've considered
One could build these link with the book metadata, maybe?

This url tag would be great!
Also a tag to know the date when we read (and finished) the book would be awesome (ex. taking the first or latest date of the highlight).
In the My Clippings.txt, there is the date for each highlight. Ex: "Added on Saturday, January 31, 2015 12:32:03 AM"

Another vote for being able to pull in the date each highlight was made, even better if it would be possible to specify the date format so that it would link to daily notes of the same date, etc.

@yfr this already exists (for any book that is bought via Amazon, since it will have an associated ASIN)

Thanks. This works for me.


oneDNN: 2.2.1 -> 2.3.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/oneapi-src/oneDNN/releases.
meta.description for oneDNN is: "oneAPI Deep Neural Network Library (oneDNN)"
meta.homepage for oneDNN is: "https://01.org/oneDNN"
meta.changelog for oneDNN is: "https://github.com/oneapi-src/oneDNN/releases/tag/v2.3.2"
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.3.2 in filename of file in /nix/store/3f1f7ib0vn411r1gg2s9v8ykbjhpyj6k-oneDNN-2.3.2

directory tree listing: https://gist.github.com/1d30a6fa87e145c29370f3a6e2e71f05

du listing: https://gist.github.com/5fff94b38c654135319a4a931926340d

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
oneDNN

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

dnnl (oneDNN)

Maintainer pings
cc @alexarice @bhipple for testing.

Result of nixpkgs-review pr 134217 run on x86_64-linux 1

1 package built:

oneDNN


Add Move language

Description

Adds Move programming language with the .move extension
Checklist:

[x] I am adding a new language.

[x] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3Amove+KEYWORDS+NOT+nothack

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

Main.move    wrote it myself  on this pr
[Vector.move)]

Sample license(s): mit

[x] I have included a syntax highlighting grammar: [vscode-move-syntax]
[ ] I have updated the heuristics to distinguish my language from others using the same extension.

hi, this ci not successful , but  It's   seem  not my code ,I need help ,or tell me how to do

hi，i'm change my code ，help re-run  ci

Please update the OP to point to where you got the new sample from and its licence.

The test failures tell you what you need to do to fix things. I recommend running the tests locally (bundle exec rake test) until they pass and then push your changes.

The test failures tell you what you need to do to fix things. I recommend running the tests locally (bundle exec rake test) until they pass and then push your changes.

hi   i change some code order  ,  I'm try local    all test pass , can try  run CI again  ,3q
`Finished in 20.045657s, 79.7180 runs/s, 1623.5936 assertions/s.
1598 runs, 32546 assertions, 0 failures, 0 errors, 0 skips`

o my god ~  It's too strict , I formatted it a little bit for licence again ，I'm sorry ，Commit the code for the first time for this ，I‘m changed it  ,try it again 😭😭

3q (thanks) you very much

I would like to ask when it will be released to Github

Probably some time during the week of 11 July

Hey @lildude, thanks for merging it in! How does the syntax get updated? Is it auto-fetched from the repo on tags/pushes? More specifically - what do I need to do to fix current errors in syntax?

How does the syntax get updated?

I manually update all the submodules whenever I make an update (step 3) so go ahead and make your fixes and they'll be pulled in the next time I make a release. I made a new one yesterday so another won't be for a few months unless something critical comes up that requires a release sooner.

Thank you for quick response @lildude. Yes, if that it possible, I would ask you to do it sooner. Only half our ecosystem migrated to the newer syntax, and their code got wrong highlighting. And they were the ones who started the process, so it feels completely unfair.
I will make sure that the we don't allow mistakes like this one in the future. Honestly didn't even know how it works until @jolestar and @uvd made it happen.

I have a question, why can't  find project be searched by language tag   ，Are there any rules that can be indexed      https://github.com/search?q=language%3AMove&type=Repositories&ref=advsearch&l=Move&l=

I have a question, why can't find project be searched by language tag ，Are there any rules that can be indexed https://github.com/search?q=language%3AMove&type=Repositories&ref=advsearch&l=Move&l=

See my previous response at https://github.com/github/linguist/issues/5952#issuecomment-1168505070

3q


capitalization update

@barlanmsft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@jdeckerms hi, updating the capitalization for Intune in this PR.


Odd number of elements in hash assignment at /usr/local/share/perl/5.30.0/Binance/API.pm line 671

Hi Lari,
Happy new year.
I take some time to finish my code with your Binance::API.
So now when I ask $api->ticker_price like this
my $paire=$mycoin->{"asset"}."USDT";
@coursbinance= $api->ticker_price($paire);
I receive this message in the syslog : "Odd number of elements in hash assignment at /usr/local/share/perl/5.30.0/Binance/API.pm line 671"
Do you know why, and where is my error ?
Regard,
Marc

The method takes a hash as a parameter, not a scalar. You need to use:
$api->ticker_price( symbol => $paire );
Happy new year!
On 2 January 2021 1.25.30 EET, baloo98<EMAIL_ADDRESS>wrote:

Hi Lari,
Happy new year.
I take some time to finish my code with your Binance::API.
So now when I ask $api->ticker_price like this
my $paire=$mycoin->{"asset"}."USDT";
@coursbinance= $api->ticker_price($paire);
I receive this message in the syslog : "Odd number of elements in hash
assignment at /usr/local/share/perl/5.30.0/Binance/API.pm line 671"
Do you know why, and where is my error ?
Regard,
Marc
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/taskula/binance-perl-api/issues/12

The method takes a hash as a parameter, not a scalar. You need to use:
$api->ticker_price( symbol => $paire );
Happy new year!
On 2 January 2021 1.25.30 EET, baloo98<EMAIL_ADDRESS>wrote:

Hi Lari,
Happy new year.
I take some time to finish my code with your Binance::API.
So now when I ask $api->ticker_price like this
my $paire=$mycoin->{"asset"}."USDT";
@coursbinance= $api->ticker_price($paire);
I receive this message in the syslog : "Odd number of elements in hash
assignment at /usr/local/share/perl/5.30.0/Binance/API.pm line 671"
Do you know why, and where is my error ?
Regard,
Marc
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/taskula/binance-perl-api/issues/12

Hi Lary,
I need to go to quicly to test sell market order with your API, but it
still in my to do.
As all is now in USDT, why the $api->account() [that give me my spot
position correctly] dont give me the USDT quantity ?
On binance I can see them at the top of the list.
Regards,
Marc
Le sam. 2 janv. 2021 à 13:38, Lari Taskula<EMAIL_ADDRESS>a
écrit :

The method takes a hash as a parameter, not a scalar. You need to use:
$api->ticker_price( symbol => $paire );
Happy new year!
On 2 January 2021 1.25.30 EET, baloo98<EMAIL_ADDRESS>wrote:

Hi Lari,
Happy new year.
I take some time to finish my code with your Binance::API.
So now when I ask $api->ticker_price like this
my $paire=$mycoin->{"asset"}."USDT";
@coursbinance= $api->ticker_price($paire);
I receive this message in the syslog : "Odd number of elements in hash
assignment at /usr/local/share/perl/5.30.0/Binance/API.pm line 671"
Do you know why, and where is my error ?
Regard,
Marc
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/taskula/binance-perl-api/issues/12

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/taskula/binance-perl-api/issues/12#issuecomment-753420216,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARZZUSTDXZR2RNF3FBFS6BLSX2BKPANCNFSM4VQTP2HQ
.

Hi Lary,
I need to go to quicly to test sell market order with your API, but it
still in my to do.
As all is now in USDT, why the $api->account() [that give me my spot
position correctly] dont give me the USDT quantity ?
On binance I can see them at the top of the list.
Regards,
Marc
Le sam. 2 janv. 2021 à 13:38, Lari Taskula<EMAIL_ADDRESS>a
écrit :

The method takes a hash as a parameter, not a scalar. You need to use:
$api->ticker_price( symbol => $paire );
Happy new year!
On 2 January 2021 1.25.30 EET, baloo98<EMAIL_ADDRESS>wrote:

Hi Lari,
Happy new year.
I take some time to finish my code with your Binance::API.
So now when I ask $api->ticker_price like this
my $paire=$mycoin->{"asset"}."USDT";
@coursbinance= $api->ticker_price($paire);
I receive this message in the syslog : "Odd number of elements in hash
assignment at /usr/local/share/perl/5.30.0/Binance/API.pm line 671"
Do you know why, and where is my error ?
Regard,
Marc
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/taskula/binance-perl-api/issues/12

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/taskula/binance-perl-api/issues/12#issuecomment-753420216,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARZZUSTDXZR2RNF3FBFS6BLSX2BKPANCNFSM4VQTP2HQ
.

Sorry it's a mistake from me.
Le mar. 12 janv. 2021 à 15:27, Marc Travaillé<EMAIL_ADDRESS>a
écrit :

Hi Lary,
I need to go to quicly to test sell market order with your API, but it
still in my to do.
As all is now in USDT, why the $api->account() [that give me my spot
position correctly] dont give me the USDT quantity ?
On binance I can see them at the top of the list.
Regards,
Marc
Le sam. 2 janv. 2021 à 13:38, Lari Taskula<EMAIL_ADDRESS>a
écrit :

The method takes a hash as a parameter, not a scalar. You need to use:
$api->ticker_price( symbol => $paire );
Happy new year!
On 2 January 2021 1.25.30 EET, baloo98<EMAIL_ADDRESS>wrote:

Hi Lari,
Happy new year.
I take some time to finish my code with your Binance::API.
So now when I ask $api->ticker_price like this
my $paire=$mycoin->{"asset"}."USDT";
@coursbinance= $api->ticker_price($paire);
I receive this message in the syslog : "Odd number of elements in hash
assignment at /usr/local/share/perl/5.30.0/Binance/API.pm line 671"
Do you know why, and where is my error ?
Regard,
Marc
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/taskula/binance-perl-api/issues/12

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/taskula/binance-perl-api/issues/12#issuecomment-753420216,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARZZUSTDXZR2RNF3FBFS6BLSX2BKPANCNFSM4VQTP2HQ
.

Sorry it's a mistake from me.
Le mar. 12 janv. 2021 à 15:27, Marc Travaillé<EMAIL_ADDRESS>a
écrit :

Hi Lary,
I need to go to quicly to test sell market order with your API, but it
still in my to do.
As all is now in USDT, why the $api->account() [that give me my spot
position correctly] dont give me the USDT quantity ?
On binance I can see them at the top of the list.
Regards,
Marc
Le sam. 2 janv. 2021 à 13:38, Lari Taskula<EMAIL_ADDRESS>a
écrit :

The method takes a hash as a parameter, not a scalar. You need to use:
$api->ticker_price( symbol => $paire );
Happy new year!
On 2 January 2021 1.25.30 EET, baloo98<EMAIL_ADDRESS>wrote:

Hi Lari,
Happy new year.
I take some time to finish my code with your Binance::API.
So now when I ask $api->ticker_price like this
my $paire=$mycoin->{"asset"}."USDT";
@coursbinance= $api->ticker_price($paire);
I receive this message in the syslog : "Odd number of elements in hash
assignment at /usr/local/share/perl/5.30.0/Binance/API.pm line 671"
Do you know why, and where is my error ?
Regard,
Marc
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/taskula/binance-perl-api/issues/12

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/taskula/binance-perl-api/issues/12#issuecomment-753420216,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARZZUSTDXZR2RNF3FBFS6BLSX2BKPANCNFSM4VQTP2HQ
.

Hi Lary,
I'm back after Market order test, and I have an error.
when I call the API with:
$retAPI=$api->order(
symbol => 'BTCUSDT',
side   => 'BUY',
type   => 'MARKET',
timeInForce => 'GTC',
price =>5000
);
[Binance::API::order] Parameter "quantity" required at
/usr/local/share/perl/5.30.0/Binance/API.pm line 834.
Where am I wrong please ?
Regards,
Marc
Le mar. 12 janv. 2021 à 15:32, Marc Travaillé<EMAIL_ADDRESS>a
écrit :

Sorry it's a mistake from me.
Le mar. 12 janv. 2021 à 15:27, Marc Travaillé<EMAIL_ADDRESS>a écrit :

Hi Lary,
I need to go to quicly to test sell market order with your API, but it
still in my to do.
As all is now in USDT, why the $api->account() [that give me my spot
position correctly] dont give me the USDT quantity ?
On binance I can see them at the top of the list.
Regards,
Marc
Le sam. 2 janv. 2021 à 13:38, Lari Taskula<EMAIL_ADDRESS>a
écrit :

The method takes a hash as a parameter, not a scalar. You need to use:
$api->ticker_price( symbol => $paire );
Happy new year!
On 2 January 2021 1.25.30 EET, baloo98<EMAIL_ADDRESS>wrote:

Hi Lari,
Happy new year.
I take some time to finish my code with your Binance::API.
So now when I ask $api->ticker_price like this
my $paire=$mycoin->{"asset"}."USDT";
@coursbinance= $api->ticker_price($paire);
I receive this message in the syslog : "Odd number of elements in hash
assignment at /usr/local/share/perl/5.30.0/Binance/API.pm line 671"
Do you know why, and where is my error ?
Regard,
Marc
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/taskula/binance-perl-api/issues/12

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/taskula/binance-perl-api/issues/12#issuecomment-753420216,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARZZUSTDXZR2RNF3FBFS6BLSX2BKPANCNFSM4VQTP2HQ
.

Hi Lary,
I'm back after Market order test, and I have an error.
when I call the API with:
$retAPI=$api->order(
symbol => 'BTCUSDT',
side   => 'BUY',
type   => 'MARKET',
timeInForce => 'GTC',
price =>5000
);
[Binance::API::order] Parameter "quantity" required at
/usr/local/share/perl/5.30.0/Binance/API.pm line 834.
Where am I wrong please ?
Regards,
Marc
Le mar. 12 janv. 2021 à 15:32, Marc Travaillé<EMAIL_ADDRESS>a
écrit :

Sorry it's a mistake from me.
Le mar. 12 janv. 2021 à 15:27, Marc Travaillé<EMAIL_ADDRESS>a écrit :

Hi Lary,
I need to go to quicly to test sell market order with your API, but it
still in my to do.
As all is now in USDT, why the $api->account() [that give me my spot
position correctly] dont give me the USDT quantity ?
On binance I can see them at the top of the list.
Regards,
Marc
Le sam. 2 janv. 2021 à 13:38, Lari Taskula<EMAIL_ADDRESS>a
écrit :

The method takes a hash as a parameter, not a scalar. You need to use:
$api->ticker_price( symbol => $paire );
Happy new year!
On 2 January 2021 1.25.30 EET, baloo98<EMAIL_ADDRESS>wrote:

Hi Lari,
Happy new year.
I take some time to finish my code with your Binance::API.
So now when I ask $api->ticker_price like this
my $paire=$mycoin->{"asset"}."USDT";
@coursbinance= $api->ticker_price($paire);
I receive this message in the syslog : "Odd number of elements in hash
assignment at /usr/local/share/perl/5.30.0/Binance/API.pm line 671"
Do you know why, and where is my error ?
Regard,
Marc
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/taskula/binance-perl-api/issues/12

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/taskula/binance-perl-api/issues/12#issuecomment-753420216,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARZZUSTDXZR2RNF3FBFS6BLSX2BKPANCNFSM4VQTP2HQ
.

It work fine, thank's

It work fine, thank's


No longer see the mail log

Hi Huan,
I can no longer see the incoming/outgoing mail log in the docker logs since using the latest version 1.1.
Is this so by design?

No, it's not by design.
The 1.1 had fixed some minor issues only, with a few commits.
So please feel free to help yourself fix it, and send a PR back to here will be welcome.

Was experience this too, and got it somewhat fixed in my previously submitted pull request

When will this be released to the registry? Currently I'm trying to troubleshoot a problem and have not found a way to view any logs /var/log/mail.log also seems to be empty.
Which PR was it? Is there a workaround?

I am not using this image anymore
but previously it was fixed in this pull request
https://github.com/huan/docker-simple-mail-forwarder/pull/68


CamelSplitIndex is property?

Issue description
Hi!
The documentation says we can get the split index dynamically, but it refers to it as a header variable, but I think this is a property variable:

Feedback Type (Optional)
incorrect statement
Page Title on SAP Help Portal (prefilled)
Define Iterating Splitter
Page URL on SAP Help Portal (prefilled)
https://help.sap.com/docs/PRODUCT_ID/368c481cd6954bdfa5d0435479fd4eaf/d61d6ecc8c8d453baa95c59e85f9cab1.html?state=PRODUCTION&version=Cloud&locale=en-GB

Hello @gaborperak,
Yes, you are right!
I have corrected this issue. The change is visible after the next documentation patch, likely in the next few days.
I the following topic, it is correctly stated that it’s a property.
Headers and Exchange Properties Provided by the Integration Framework | SAP Help Portal

Thank you for your input!


CTECH-1556: Removes repository causing test failures.

Signed-off-by: Paul Saunders<EMAIL_ADDRESS>Pull Request Checklist

[x] Read the contributing guidelines
[x] Tests pass

Description of the PR
removes reference to repo that is unreachable.

/LGTM


Created Arweave class

#6
Changes

Created FileStorage Abstract class
Created Arweave class implementing FileStorage class
Added tests for Arweave class
Added example script for Upload functionality

@SadashayKanungo Please resolve mr conflicts


Part of #111 ("Newsletter 9: Coordination/Tracking")

Thank you very much :)

Thanks for thinking to include me! :)


Unable to push changes for creating a PR

Hello,
Firstly great repo! I'm trying to make a contribution.
Have cloned the repo locally, and added a question, and when trying to push after creating a branch, it is saying
remote: Permission to lydiahallie/javascript-questions.git denied to k-sai-kiranmayee.
fatal: unable to access 'https://github.com/lydiahallie/javascript-questions.git/': The requested URL returned error: 403

I don't have the SSH key. Looking forward to contribute and learn, any help is much appreciated 🤞
Thanks,
K. Sai Kiranmayee

You have to fork it, not just clone it.
You fork it, clone your fork and then open the PR.

Thank you @alexandercerutti for your quick response! 😄
My current Git username is linked to my current organization. So, there is no other way to raise PR without forking it? If there is any, please let me know, I will be glad!!
Thanks!

Nope, you should be the owner or maintainer of the repo.
Is there any issue for it to be linked to your organization?
Mine is linked too but I use it for both open-source and personal things (I do have three emails registered in the same profile, so one is for the job and one for personal things + recovery).

Thank you @alexandercerutti, I have added my personal mail to this itself. Closing this!


Support maskable icons

Android and other OSes now support auto-masking / cropping icons when a parameter in the icons is set. Would be nice with support for it!
Should just be passing purpose: "maskable" if the developer has set it.
https://css-tricks.com/maskable-icons-android-adaptive-icons-for-your-pwa/

Not sure it actually works that way already, but it would be nice if all "unknown" properties (like purpose) are printed to the final JSON output regardless.

@nixolas1 Can you please test the PR #115. It adds support for the purpose property.
Can be installed from here npm install --save git://github.com/Onset/webpack-pwa-manifest.git#44fd8ff17c 630b1581f0216af8856e1953445f9b.

That PR fixes this issue for me.

Can we get this merged sometime?

Yes please.

I think this repo is dead, and I don't want to be beholden to this repo adding the random properties for PWA manifests going forward.  I spent some time figuring out how to do this with extract-loader and remove this plugin as a dependency, and I'm pretty happy with the results:
manifest.js
const Icon192 = require("./images/192.png")
const Icon512 = require("./images/512.png")

const manifest = {
	icons: [
		{
			src: Icon512,
			sizes: "512x512",
			type: "image/png",
		},
		{
			src: Icon192,
			sizes: "192x192",
			type: "image/png",
		},
	],
	name: "App",
	short_name: "App",
	orientation: "portrait",
	display: "standalone",
	start_url: "/",
	background_color: "#ffffff",
	description: "description",
}

module.exports = JSON.stringify(manifest, null, 2)

webpack.config.js
		{
				test: /manifest.js$/,
				use: [
					{
						loader: "file-loader",
						options: {
							name: "ui/[name].[hash].json",
						},
					},
					"extract-loader",
				],
			},

index.html (using html-webpack-plugin)
		<link rel="manifest" href="${require("./manifest.js")}">

@thequailman Neat. Thank you for the snippets.


Add z-index negative defaults

This should add support for negative z-indexes when using Tailwindcss from a CDN as discussed in #329 with @hacknug.
Should add:

.-z and it's other breakpoints utilities.
.-z-10 it's other breakpoints utilities.

Thanks.

I have the feeling Adam will say no to .-z, everything else looks good. Let's wait to hear from him 👍

I feel like I've never needed this, mind sharing a concrete example where it's significantly nicer than trying to solve it with positive z-index values?

I guess for pseudo elements and something might already be z-index'd you would want a negative to counter that?, I've seen this used before. Think it could be a good addition.

@adamwathan it helps bringing something behind everything else without touching more than that background element. Something like this I had on a project of mine:

    <slot name="head" />

    <slot name="main" />

I had the same use case as @hacknug, @adamwathan.

In the process of rebuilding my site in Tailwind and I ran into a need for this. Think it could be handy if there was a single negative z-index.
@adamwathan my use-case was I wanted to preset an image with an inset box shadow over the top of it (prior art). There is potentially another way to do this in Tailwind with positive indexes (?), but this was the quick solution I implemented which utilises a single negative z-index.
Not sure if this is a good enough use case to bring this into the core of Tailwind, but I personally would have found a single negative value out of the box handy.
Thanks for Tailwind ❤


Add host function calls for arkworks to Polkadot

Same as @tomaka said.
Also not sure we already need this. AFAIK this is still in experimental mode.

I removed all references to Substrate. I can also close the PR for now and re-open later?

Same as @tomaka said.
Also not sure we already need this. AFAIK this is still in experimental mode.

I removed all references to Substrate. I can also close the PR for now and re-open later?

Are we 100% sure that we will need all these host functions and also that they will enable the functionality we want to have?

In theory, we could live without all of them. But every single host functions brings a performance improvement for an arithmetic operation on elliptic curve points, between 1.34x and up to 10.57x, so I would consider their existence as justified! They speed up the usage of the elliptic curves BLS12_381, BLS12_377, BW6_761, ED_ON_BLS12_377, ED_ON_BLS12_381, AFAIK they are all required somewhere for either Sassafrass or ZKP proof verification @burdges ?

We do want the multi_miller_loop, final_exponentiation, and msm host calls for BLS12-381 in polkadot this year.
We usually reference benchmarks to the repo that generates them, so that others could rerun them.  It's the benchmarks which justify adding those host calls, but not adding other host calls, so they should be referenced from the PPP document.
As for other curves, but We'll discuss the benchmarks this week or next to make decisions, but..
Right now the ed_on_* msms do not benefit so much, not sure the problems.
We'll likely do BW6-767 for BLS12-381 but arkworks just adding this https://github.com/arkworks-rs/curves/pull/154  We only need BW6-767 so that polkadot and kusama can track one another's finality, not sure if this is really a priority.  We'll discuss BLS12-377 and BW6-761 too.

@bkchr how do you guys future proof error types in host calls?  At present multi_miller_loop cannot fail, but Alistair noticed cofactor checking could happen inside the Miller loop, which requires an error type.  We do not envision ever returning more than just this error type, but who knows.

We want to be able to do with host functions all these and more. We'll want more curves than these. A key reason that Ethereum right now is so bad for ZK is that the only elliptic curves it has precompiles for are  the BN254 curve for pairings, which isn't so secure these days, or secp256k1, which is bad for proving systems because it isn't FFT friendly. Support for more interesting curves would make Polkadot more attractive for ZK developers. In particular SNARK friendly 2-chains of pairing curves like BW6-681/BLS12-377 here or for pairing cycles like the Pasta curves that aren't in this PR. That means that the things Aleo or Mina are doing aren't possible on Ethereum and they aren't really efficient enough on Substrate right now but they could be, with more host functions.
Now I did question whether we need seperate host functions for every curve, but @burdges seems to think that will make maintanability easier.

We want to be able to do with host functions all these and more. We'll want more curves than these. A key reason that Ethereum right now is so bad for ZK is that the only elliptic curves it has precompiles for are the BN254 curve for pairings, which isn't so secure these days, or secp256k1, which is bad for proving systems because it isn't FFT friendly. Support for more interesting curves would make Polkadot more attractive for ZK developers. In particular SNARK friendly 2-chains of pairing curves like BW6-681/BLS12-377 here or for pairing cycles like the Pasta curves that aren't in this PR. That means that the things Aleo or Mina are doing aren't possible on Ethereum and they aren't really efficient enough on Substrate right now but they could be, with more host functions.
Now I did question whether we need seperate host functions for every curve, but @burdges seems to think that will make maintanability easier.

I  remember that Ashutov's original implementation used a call function handler and exposed only one new host function.
But I also remember, that @pepyakin mentioned to me, the only acceptable way to implement it, would be to expose one host call for every arithmetic operation.

We already address versioning and deprecation from the substrate side, doing it again wastes effort, unless the substrate flavor winds up inadequate somehow, which seems unlikely.
@achimcc Can you fix the scalar multiplication benchmarks?  like https://github.com/achimcc/substrate-arkworks-examples/blob/4dcd81ffe1be1fe64e10252c703d61a8a80d366d/pallets/template/src/bls12_381.rs#L153
At present, they come out insanely fast with variable time arithmetic, which prevents us understanding the other benchmarks.  You'll need random-ish scalars and points generated before the benchmark starts, like with the others.

@bkchr How do you guys future proof error types in host calls? At present multi_miller_loop cannot fail, but Alistair noticed cofactor checking could happen inside the Miller loop, which requires an error type. We do not envision ever returning more than just this error type, but who knows.

If we just need to report that there was an error, we could just use () as error type? Or do you expect some kind of special handling of the error types?

Yes, we should just use () for the error type in both cases I guess, thanks for the sanity check.  ;)
Reasoning:
A final exponentiation could fail if you feed it bad data, only doable with parachain code that's either intentionally flawed, or messes up G_T computations, ala MIPP ala snarkpack or marlin, etc.  It's still just an invalid transaction/batch, but not even reachable from polnk, groth16, etc. without batching.  In theory, there are type II pairings where even the Miller loop between a point and itself outputs a zero, but likely nobody ever uses type II, much less these shitty ones.
In future, we might've miller loops return an extra error that represents points not in the subgroup, which currently we'd check in deserialization.    And parachain could still do distinguish if necessary since these are separate host calls.

Yes, we should just use () for the error type in both cases I guess, thanks for the sanity check. ;)
Reasoning:
A final exponentiation could fail if you feed it bad data, only doable with parachain code that's either intentionally flawed, or messes up G_T computations, ala MIPP ala snarkpack or marlin, etc. It's still just an invalid transaction/batch, but not even reachable from polnk, groth16, etc. without batching. In theory, there are type II pairings where even the Miller loop between a point and itself outputs a zero, but likely nobody ever uses type II, much less these shitty ones.
In future, we might've miller loops return an extra error that represents points not in the subgroup, which currently we'd check in deserialization.  And parachain could still do distinguish if necessary since these are separate host calls.

I replaced PairingError by () everywhere in the code.

Why not support bn254 and ed_on_bn254 as well?

Please lets add group element additions as well

Please lets add group element additions as well, i see we already have multiplications

@burdges @bkchr  should I do it? Would add even more host functions. We could also try to only add additions, AFAIK the mul's use the additions under the hood via the double and add algorithm for computations?

However, I remember that in my original benches, we were benchmarking mul's with a multiplication by two instead of using a random field element, this was effectively a curve point addition and the benchmarks revealed a slow down when using host functions due to host call and (de-)serialization overhead. So I'd be dubious that we achieve a performance improvement by host calls for additions!

So I'd be dubious that we achieve a performance improvement by host calls for additions! @seunlanlege

I think it's worth investigating, I'm happy to run the needed benchmarks myself. Lots of use cases for computing the sum of a lot of group elements.

It's always nice if someone else tries their hand at benchmarks @seunlanlege  We'd various difficulties with our own benchmarks.
Indeed, if you read the single scalar multiplications in https://github.com/achimcc/substrate-arkworks-examples/blob/main/benchmarks-comparison.md then you'll see they report way too fast.  This is because those still use the scalar 2 and the scalar mults are variable time.  See https://github.com/achimcc/native-bench-arkworks/blob/47d02ddc4fef3ac1bc61f149e9f75723456458fc/src/ed_on_bls12_377.rs#L20 for example.  I'd seen what I wanted from the benchmarks so I felt no need to ask for improvements.
In fact, addition should be some fixed multiple (1-0.33) of those single 2*point multiplications in the current benchmarks, so yes you'd have a 5x speed up, as opposed to the 6-8x speed up seen elsewhere.
We've various small concerns around addition though like projective vs affine coordinates, ala mixed addition, and doublings, and special cofactor clearing optimizations.  In many cases, we should choose if we give them separate hostcalls or if we suck them up into an existing hostcall at the cost of an extra if check.  We might also choose where subgroup checks occur.  It all risks a maintenance burden though.
We should really do partial aggregation for protocols which see significant usage, which invariably absorbs more of the additions.  Schnorr signatures like sr25519 have half-aggregation.  SnarkPack is way more powerful.
We need some substrate work before partial aggregation:  substrate needs non-Merklised transient storage https://github.com/w3f/PPPs/pull/11 which your PVF creates in on_initialize, appends to in transactions, and sends to the batch verifier in on_finalize.  We'll mostly need people to actually implement and deploy partial aggregation though.   It's massively easier to do partial aggregation on substrate/polkadot than on smart contract platforms, so doing all this gives polkadot advantages unavailable to ETH, et al.
tl;dr We should spend resources on partial aggregation instead of smart contract flavored optimizations.

We will archive this repository in favor of the new RFCs repository by the Fellowship. So, I'm closing the issue here. I recommend opening a PR at the new repository.