Can't read my secret from azure key vault

Hi,
when I try to use this plugin, I got some problems
First, i saw this link https://learn.microsoft.com/en-us/azure/load-testing/how-to-parameterize-load-tests
But, its impossible to run teh test without having user.proprieties or jmeter.proprieties file.
Then, when i execute my test, and try to get my secret value, i got this error:
ERROR c.a.s.k.s.SecretClientBuilder: The Azure Key Vault url is malformed.
my key vault format as below :
https://.vault.azure.net/secrets//fddd089393db44c288ea422d8833df64
thx

Hi,
Thx for replay
I use jmeter-plugins-azure-load-testing-stub-0.1.0.jar
Le mar. 5 nov. 2024, 04 h 55, Kuniteru Asami @.***> a
écrit :

Hi,
The following five plug-ins are provided here

jmeter-plugins-azure-eventhubs-0.2.5.jar
jmeter-plugins-azure-load-testing-stub-0.1.0.jar
jmeter-plugins-azure-servicebus-0.3.1.jar
jmeter-plugins-azure-storage-queue-0.1.3.jar
jmeter-plugins-functions-azure-0.4.4.jar

Which one do you use?
—
Reply to this email directly, view it on GitHub
https://github.com/pnopjp/jmeter-plugins/issues/25#issuecomment-2456724798,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACXP3O2Q3AUVRDN5ESCKLLZ7CIZ3AVCNFSM6AAAAABRFAHQTSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJWG4ZDINZZHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi,
This plugin is a Stub that allows custom functions that are only available in Azure Load Testing to work in pure Azure JMeter™.
Enables Apache JMeter™ to use custom functions when creating and testing test scripts to be uploaded to Azure Load Testing.
It is not for use on Azure Load Testing.
If you wish to use this plugin with pure Azure JMeter™, please refer to the following documentation.
https://github.com/pnopjp/jmeter-plugins/blob/master/docs/azure-load-testing-stub.md

This si exactly what i do.
I create my jmeter configuration, I added this plugin to use the __GetSecret() function.
I execute the jmeter jmx file with Azure load testing as showin on https://learn.microsoft.com/en-us/azure/load-testing/how-to-parameterize-load-tests
also, with jmx file, i upload user.proprietes file, this file content the configuration of storetype and auth type
Below the log :
2024-11-04 21:15:05,699 ERROR c.a.s.k.s.SecretClientBuilder: The Azure Key Vault url is malformed.
2024-11-04 21:15:05,700 ERROR j.c.p.j.f.GetSecret: Error calling __GetSecret function with Secret name "azureloadtestingsecret". java.lang.IllegalArgumentException: The Azure Key Vault url is malformed.
java.lang.IllegalArgumentException: The Azure Key Vault url is malformed.
at com.azure.security.keyvault.secrets.SecretClientBuilder.vaultUrl(SecretClientBuilder.java:215) ~[jmeter-plugins-azure-load-testing-stub-0.1.0.jar:?]
at jp.co.pnop.jmeter.functions.GetSecret.getKeyVaultSecret(GetSecret.java:175) ~[jmeter-plugins-azure-load-testing-stub-0.1.0.jar:?]
at jp.co.pnop.jmeter.functions.GetSecret.execute(GetSecret.java:108) [jmeter-plugins-azure-load-testing-stub-0.1.0.jar:?]
at org.apache.jmeter.engine.util.CompoundVariable.execute(CompoundVariable.java:139) [ApacheJMeter_core.jar:5.6.3]
at org.apache.jmeter.engine.util.CompoundVariable.execute(CompoundVariable.java:114) [ApacheJMeter_core.jar:5.6.3]
at org.apache.jmeter.testelement.property.FunctionProperty.getStringValue(FunctionProperty.java:104) [ApacheJMeter_core.jar:5.6.3]
at org.apache.jmeter.testelement.TestElement.get(TestElement.kt:276) [ApacheJMeter_core.jar:5.6.3]
at org.apache.jmeter.config.Argument.getValue(Argument.java:154) [ApacheJMeter_core.jar:5.6.3]
at org.apache.jmeter.config.Arguments.getArgumentsAsMap(Arguments.java:104) [ApacheJMeter_core.jar:5.6.3]
at org.apache.jmeter.engine.PreCompiler.createArgumentsMap(PreCompiler.java:113) [ApacheJMeter_core.jar:5.6.3]
at org.apache.jmeter.engine.PreCompiler.addNode(PreCompiler.java:100) [ApacheJMeter_core.jar:5.6.3]
at org.apache.jorphan.collections.HashTree.traverseInto(HashTree.java:1029) [jorphan.jar:5.6.3]
at org.apache.jorphan.collections.HashTree.traverse(HashTree.java:1012) [jorphan.jar:5.6.3]
at org.apache.jmeter.engine.StandardJMeterEngine.run(StandardJMeterEngine.java:413) [ApacheJMeter_core.jar:5.6.3]
at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539) [?:?]
at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) [?:?]
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) [?:?]
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) [?:?]
at java.base/java.lang.Thread.run(Thread.java:840) [?:?]
Caused by: java.net.MalformedURLException: no protocol:
at java.base/java.net.URL.(URL.java:674) ~[?:?]
at java.base/java.net.URL.(URL.java:569) ~[?:?]
at java.base/java.net.URL.(URL.java:516) ~[?:?]
at com.azure.security.keyvault.secrets.SecretClientBuilder.vaultUrl(SecretClientBuilder.java:213) ~[jmeter-plugins-azure-load-testing-stub-0.1.0.jar:?]

This plugin cannot be used on Azure Load Testing.
This plugin is only for Apache JMeter installed on-premise or on a virtual machine.
azure_load_testing_stub.get_secret.{SECRET_NAME}.store_type and azure_load_testing_stub.get_secret.{SECRET_NAME}.auth_type set in user.profile, etc. are exclusive to this plugin and are not recognized by Azure Load Testing.
For Azure Load Testing, follow this page to set the secret.
https://learn.microsoft.com/ja-jp/azure/load-testing/how-to-parameterize-load-tests
This plugin is intended to use the same jmx files for pure Apache JMeter and Azure Load Testing.
The configuration method for referencing the secret is different for each environment.

Thx for your return


Issue with Zulrah Helper not showing when Jad Phase occurs with the split 'Range / Mage' circle

As per title really. Not sure how long this has been happening as not been to Zulrah for a while but the helper used to show when the Jad phase was going happen and which order the attacks would occur. Now there is no indication at all.

You can  enable prayer icons https://github.com/while-loop/runelite-plugins/issues/67

Literally a UX change that didn't need updating, when learning now I first have to explicity enable prayer icons and it throws me off every time due to how small the icons are on a high resolution screen.

Literally a UX change that didn't need updating, when learning now I first have to explicity enable prayer icons and it throws me off every time due to how small the icons are on a high resolution screen.

Yea. New one is terrible. Old one was so much better


if a tmsp req fails on sync call, client will just hang forever  on .Wait()

more of a development problem than anything, and related to app restart

fixed by https://github.com/tendermint/tmsp/pull/27


Hello, some devices crashed by this: NoClassDefFoundError  EventBusBuilder

I already add proguard this：
-dontwarn org.greenrobot.eventbus.**
-keep class org.greenrobot.eventbus.** { *; }
-keepattributes Annotation
-keepclassmembers class ** {
@org.greenrobot.eventbus.Subscribe ;
}
-keep enum org.greenrobot.eventbus.ThreadMode { *; }
-keepclassmembers class * extends org.greenrobot.eventbus.util.ThrowableFailureEvent {
(java.lang.Throwable);
}

This really hard for me to get what they are trying to say as solution.
Maybe you overwrote a life cycle method with PersistableBundle just by accident. In that case just change to the method without PersistableBundle, for example onCreate (Bundle savedInstanceState).
No i am using
@Override
protected void onStart() {
super.onStart();
EventBus.getDefault().register(this);
}
So this didn't resolve problem.
Use EventBus 3 with an subscriber index. This will avoid reflection and thus the problem altogether. As a positive side effect, registering subscribers and thus app startup time will be much faster.
Is this have to do anything with this crash resolve ?
Remove the offending method from your subscriber class. Either pull out the event handler methods into a new subscriber class, or pull out the offending method into a non-subscriber class.
I didn't include that onCreate (Bundle savedInstanceState,PersistableBundle persistableBundle )
This didn't needed in that case.
If the offending method is public, make it non-public. This works because of some “plan b” logic EventBus applies: EventBus first calls getDeclaredMethods, which will fail. Next, EventBus will try again using the getMethods (“plan b”). The latter will succeed because getMethods only returns public methods. However, keep in mind that is the least efficient way in terms of performance (2 reflection calls instead of 1 with getMethods considering the entire class hierarchy).
I made the method protected still no solution.
@Subscribe
protected void onEvent(JSONObject object) {
Log.e("EventRecieved", object.toString());
updateingGuestList(object);
}
Can you please get back on this. Thanks

Post the stacktrace of your crash.

No response, closing. -ut

this is not a eventbus bug,if your device is pre 5.0 version and use mutidex to avoid 65535 methord problem,Maybe mutidex is the error cause,see this link http://stackoverflow.com/questions/36907916/java-lang-noclassdeffounderror-while-registering-eventbus-in-onstart-method-for

this is not a eventbus bug,if your device is pre 5.0 version and use mutidex to avoid 65535 methord problem,Maybe mutidex is the error cause,see this link http://stackoverflow.com/questions/36907916/java-lang-noclassdeffounderror-while-registering-eventbus-in-onstart-method-for


Make gesture recognizer more robust

1$ gesture is not enough, many people are complaining about that.
Found and implement a core gesture provider, and maybe implement another one in python.

I am playing with a $N/Protractor python port, minor issues left to resolve. Will send a pull request for it soon(ish) and you can have a look if it is suitable for inclusion.

Closing this issue since you're new pull request will made it.


Link misinterpreted in code blocks in markdown file

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

Write any code blocks in any markdown file

It looks like all these properties are interpreted as links in reference style. Indeed, you could command+click them, which of course leads to the wrong address.

codes for copy

```js
const x = {
[double]: string;
}
```
```ts
interface Fruit {
[key]: number;
}
```
```python
dict = {
[a]: "number";
}
```

Hi, I am quite new to open source contributing but as a vs code user I wish to give this issue a try! Could you help with how to go about this issue? Thank you!

The relevant code is:
https://github.com/microsoft/vscode/blob/baa7434480b28d5acf1608715a8e8cf3e47c6847/extensions/markdown-language-features/src/languageFeatures/documentLinkProvider.ts#L206
There's already code that checks if some links occur inside code blocks. This simply needs to be extended to cover all links
The relevant tests are in:
https://github.com/microsoft/vscode/blob/0e65adbda8f3b605d240b341dd8623203e16997d/extensions/markdown-language-features/src/test/documentLinkProvider.test.ts#L17

The relevant code is:
https://github.com/microsoft/vscode/blob/baa7434480b28d5acf1608715a8e8cf3e47c6847/extensions/markdown-language-features/src/languageFeatures/documentLinkProvider.ts#L206
There's already code that checks if some links occur inside code blocks. This simply needs to be extended to cover all links
The relevant tests are in:
https://github.com/microsoft/vscode/blob/0e65adbda8f3b605d240b341dd8623203e16997d/extensions/markdown-language-features/src/test/documentLinkProvider.test.ts#L17

Will get to work, thank you!


How to push streams using their own names

If I do:
push rtmp://<IP_ADDRESS>:1935/live/$name;

on origin, then on edge I get a stream called "$name.flv" with the literal dollar sign. How do I make it so that no matter what stream name comes into origin, the same stream name gets pushed to edge?
Thanks!

Ah found it. Should've read the sample nginx.conf :)
Doing this:
push <IP_ADDRESS>;

will push it to the same app and name on edge as origin.

if with query param how to do?

@trongr It works like a charm!
This should be added to documentation here https://github.com/arut/nginx-rtmp-module/wiki/Directives#push


Added drawing templates

Fixes https://github.com/tetrabiodistributed/papr-print-files/issues/70

Good catch, I have addressed in the new version.
Just for the sake of my own sanity, if anyone for some reason wants to export a drawing template from Fusion, you have to do it through the online Fusion team by downloading the template itself, it doesn't seem to let you do it from within Fusion itself.

A4 fixed in Fusion, so LTGM


Add swab icon

It would be nice to have  swab, tube, and bacterial culture plate icons. Thank you!

Contributed by @XylotrupesGideon with #151

Thank you!


Will I be able to change the records in future?

Suppose now I am using the domain with blogger but in the future I want it to be added to my github.io website. Will I be able to do that? If yes then how?

Yes by editing your file on your fork and then opening a PR with the changes.


Glob pattern dist/**/* doesn't discover top level (within dist/) files

Is your feature request related to a problem? Please describe.
When using buildkite-agent artifact download "dist/**/*" . to download all artifacts in a compiled dist folder that has successfully been uploaded using artifact_paths: "dist/**/*" in the pipeline.yml file, the files downloaded do not contain files such as dist/express.js or dist/worker.js.
This seems unexpected for the given glob pattern.
NOTE: The above pattern does download all files that are contained in subdirectories, for example, dist/apps/express.js and dist/apps/worker.js are successfully downloaded with the above command.
Describe the solution you'd like
A solution that allows me to download files described above with the glob pattern dist/**/*. Otherwise, documentation to explain why this pattern does not work for buildkite-agent
Describe alternatives you've considered
I have found a workaround by using buildkite-agent artifact download "dist/**" ., however, unsure if this was intended behaviour or not.
Additional context
N/A

hey there! this is somewhat of a running issue - there's a bunch of edge cases and gotchas with artifact uploading and downloading. we have some work planned to work on making this better - we'll keep this issue open until we do, but in the meantime, it looks like your workaround is sufficient, if not quite desirable.


[BUG] Parse beers correctly

The way that this tool was parsing beers was broken, as the beer board changed location and also drastically changed in terms of mark-up.
I've moved the web-scraping and parsing logic out into a separate library (to make it more easily testable), see: https://github.com/TomSeldon/tiny-rebel-web-scraper
I've modified this tool to just use that web scraper, and in turn made it a lot simpler.
Note: I've removed the beer ratings (fetched via untapped) as I didn't really understand what it was doing, and frankly I want to spend as little time  working with Coffeescript as possible.
The format of the output has changed, though I'm really not precious about it. The goal here was to make it output something correct again, someone else is welcome to change exactly what is output.
I've also updated the tests to make use of a hubot testing library, making it easier to assert on the messages output. The test(s) really aren't great, but is at least some validation that the tool works. The tests for the web scraping / parsing are a lot more complete.
As the web scraper library uses native promises, this may impact how this Hubot script is run. It will require a version of Node that supports native promises. See: http://node.green/#Promise
It's very much likely that the server running Hubot will need to have Node v4+ installed.

Thanks @studioromeo :)
Re untapped I meant I wasn't sure exactly what was being passed to it in the query, what it was getting back, etc. Could definitely add back in, but tbh I'm being lazy and didn't want to spend much time looking at it. ;)


Move LibraryTest to Methods dir

PSR-4 autloading fix :)

The test failures are unrelated, they look like tests that would've broken a while back with the configurable resource folder change.


Unit testing?

Any documentation for test unit ?

@mhf-ir What do you mean?

if loop run forever how write a sample test unit to create request and mock server dispatch the request and server shutdown after test.

@mhf-ir Testing server shutdown should be as simple as testing your shutdown event class (the one that extends \Igni\Http\Server\OnShutdown)-  you can mockup the \Igni\Http\Server class in your code in order to achieve this.
In order to test your controllers the thing should also similar, you simply declare your test class instantiate your controller (which extends HttpController) and mockup Request class.
Entire application layer with the server flow is already tested in the framework so you don't have to be afraid about that part.

I can provide example code to help you out.

That's will be awesome, try to using non-block swoole base is new and many developers using Chinease platfroms but it's not documentations well like https://github.com/swoft-cloud/swoft.
Documentation and support will be the point.
Thank you for support

@mhf-ir https://github.com/igniphp/framework/tree/master/docs#testing


Don't move the old swapchain when creating a new one

Related to #2464 and #2467
It appears that we can't safely destroy the old swapchain there, so we can't accept it by move.

May be related to #2524 and #2654

In general I agree on this.
Thinking on the implications on this for dx12:
Swapchain destruction or resize in dx12 requires as precondition that no resource attached to the swapchain is in use similar to vulkan's destroy swapchain precondition. For resize we need to destroy or resize the old swapchain in dx12, but trying to enforce the precondition manually appears non trivial to me.

@msiglreith I'm a bit confused on that note. Do you mean that we currently have no solution for resize on DX12?

@kvark There is resize support atm.
If I understand your proposal correctly: we move from create_swapchain with integrated destruction of the old swapchain to create_swapchain and explicit destruction.
DX12 requires for resizing (any of the two options mentioned in my post) that we don't access any of the resources of the swapchain anymore on creation of the new swapchain -> wait_idle/semaphore/fence required.
The new create_swapchain and explicit destruction doesn't require synchronization as mentioned above on the creation but only on the destruction, which makes it more tricky for the dx12 backend.
rant: as always, the easiest way for dx12 WSI support is create render targets and blit on present ..

Thank you for clarification! Should we just insert the wait for idle into dx12 create_swap_chain implementation instead of changing the API then?

On Mar 11, 2019, at 07:11, msiglreith<EMAIL_ADDRESS>wrote:
@kvark There is resize support atm.
If I understand your proposal correctly: we move from create_swapchain with integrated destruction of the old swapchain to create_swapchain and explicit destruction.
DX12 requires for resizing (any of the two options mentioned in my post) that we don't access any of the resources of the swapchain anymore on creation of the new swapchain -> wait_idle/semaphore/fence required.
The new create_swapchain and explicit destruction doesn't require synchronization as mentioned above on the creation but only on the destruction, which makes it more tricky for the dx12 backend.
rant: as always, the easiest way for dx12 WSI support is create render targets and blit on present ..
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I think this is less of an issue now as we have the new swapchain model.


Provide ability to have an internal cluster of Tessera nodes for heightened recoverability

To have a cluster of nodes against each quorum node to resume transaction processing even if one node in the cluster is down. All nodes in cluster will share all private/public key pairs associated with that quorum node. The connection is to listen over HTTP or gRPC with robust authentication with a load balancing layer managing the cluster

Tessera HA Configuration Overview.pptx

Attached document describe how to setup a cluster of Tessera Node for single Quorum node generic for any load balancing solution


Mapping the core file

Changed the way of reading data from the core file. The core file is mapped directly into a virtual memory page.
Issue number of the reported bug or feature request: #
Describe your changes
A clear and concise description of the changes you have made.
Testing performed
Describe the testing you have performed to ensure that the bug has been addressed, or that the new feature works as planned.
Additional context
Add any other context about your contribution here.

Codecov Report
Patch coverage: 70.00% and project coverage change: -0.17% :warning:

Comparison is base (fa0c90a) 90.42% compared to head (f69585b) 90.26%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #126      +/-   ##
==========================================
- Coverage   90.42%   90.26%   -0.17%
==========================================
  Files          48       48
  Lines        5523     5555      +32
  Branches      863      863
==========================================
+ Hits         4994     5014      +20
- Misses        529      541      +12

Flag
Coverage Δ

cpp
75.62% <70.00%> (-0.20%)
:arrow_down:

python_and_cython
99.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

src/pystack/_pystack/mem.h
55.55% <ø> (ø)

src/pystack/_pystack/mem.cpp
86.14% <70.00%> (-3.81%)
:arrow_down:

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

Some local benchmarks:
Before
root@fdfb53906b9d:/src# time pystack core core.630 --locals > /dev/null

real	0m0.374s
user	0m0.098s
sys	0m0.141s

After
root@fdfb53906b9d:/src# time pystack core core.630 --locals > /dev/null

real	0m0.131s
user	0m0.076s
sys	0m0.043s

https://github.com/bloomberg/pystack/commit/1da05550d22a17debeea7fba25bfe0585e68cc34 sets up RAII for this, and I think it's a small improvement over what you've got - @ivonastojanovic would you mind integrating that into your branch? Specifically:

the data type for the mapping member variable becomes std::unique_ptr<char, std::function<void(char*)>>
we provide a lambda as the deleter when assigning it
that deleter calls munmap, and logs if it fails
we check for a map error before assigning it (since we don't want to call munmap in that case)
we flip the order of the two member variables (C++ member variables are destroyed in reverse order, and we need d_corefile_data to be destroyed before d_corefile_size, since the lambda that calls munmap needs to refer to d_corefile_size)


Updating Container CU Version

Just tried to see what would happen if I hopped up the cumulative update version (2016 CU16) of a container and point it as an older version (2016 CU1) database.   Is this a scenario that would/should be supported with a container or would it be expected that an upgrade procedure has already been performed with an installed dev client?
Log:
Change Database Connection
Import Encryption Key
The Demo Database NAV (9-0) CU1 database on the 8579E5237573 server must be converted by Microsoft Dynamics NAV Development Environment before you can use it with this version of Microsoft Dynamics NAV.

Yes, you are right. In this scenario, you have to upgrade/convert the database to be able to connect the docker service.
And yes, it would be nice to be able to upgrade the database in the same process (docker start) :)

@eknraw BTW, I think you could use New-CSideDevContainer from Microsoft/navcontainerhelper repository to simplify the conversion.
I haven`t tried it yet (I have a very small experience with some specific functions only) but I think it should be possible to convert the DB using those scripts instead of having to install/copy C/SIDE files from DVD (and eventually downloading the DVD etc.).

@Koubek Ok, I'll take a look at those functions, thanks!   Obviously what I'm after is the ability to bring up a container and be able to eventually convert with the ClickOnce DEV Client.  Possibly add an option to prevent any type of Client/Database interaction so it doesn't fail?

@eknraw I think I have a solution. I`ll post it tomorrow. You actually won`t need any other toolset and you will be able to convert the database directly when starting the container. Or at least I hope you will ;)

I think you could run the following (of course, you will need to adjust the parameters):
finsql.exe command=upgradedatabase, servername=sqlserver, database=[dbname], ntauthentication=0, username=[sqllogin], password=[sqlpwd], logfile=c:\run\my\upgrade.log

I would place this code into one of the overrides, maybe SetupConfiguration.ps1 could be OK (the last command in this file).

Please remember, that Docker isn't doing magic for you.
Docker just encapsulates the NAV Service Tier in a container and allow you to run it very easily on a machine, sharing resources etc.
Everything else is business as usual.

Of course, but still, with the supported extensibility you anyone can do a lot of specific things.
People need to understand better the extensibility that NAV on Docker provides for them.

Correct, thanks for pointing this out Jakup:-)

@freddydk It might not be magic but it's pretty darn awesome. :)
Yes, it's not a large use case for sure.
I think my answer is probably the following, just confirmed this works fine.  I'm totally ok with this.

Spin up another container that relies on SQLExpress
Point that OneClick DEV environment at my persistent database, upgrade
Remove and replace existing container with proper CU container
ClickOnce client updates automatically
Enjoy not having to install NAV locally. :)


Can't add SSH key

For replay this bug you need:
Test case

Create Environment
Generate SSH key via Tray
Paste SSH to Environment
a. Via button in Tray 'Add key to environmets'
b. Via Hub - Environments - "Environment" - SSH keys - 'Add SSH Key' button

Result:
You observe error:
a. 'Error transferring ... - server replied: Server error
b. 500 Internal server error

Fixed and tested on prod.
@NikolaiB please check.

Manual adding (b) fixes - work correctly
Auto adding  (a) - the same error

Adding a key via Tray works correctly for me.

Please provide details of how you are testing. I will reproduce and fix.

The error with Tray for @NikolaiB is on Windows 10.

Now auto adding also work correctly

Both methods now works normally.


No way to Edit Expense on Mobile

This is with regard to the issue raised at, https://opencollective.slack.com/archives/C03AWL58JRM/p1652743150824589.
It seems that on mobile devices we don't show the expense edit buttons; preventing the user from editing an expense on mobile.
On desktop:

On mobile:

Note that only the link and download buttons are shown.

Ops, I realized this is a duplicate; https://github.com/opencollective/opencollective/issues/5564.


Tests: Limit @backDeploy evolution tests to macOS

Limit the attr_backDeploy_evolution.swift test case to running on macOS and hard code all availability versions it uses. I had hoped to get broader coverage with this test but that goal appears to be unrealistic given the variety of test bot configurations that this test is sensitive to. Hopefully this simplification will prevent further discoveries of incompatible test bot configurations.
Resolves rdar://91181116

@swift-ci please smoke test and merge


How do I get my id in the file uploading Project ?

This id is like a username, you can specify any string as id. There is a default one (1202437) if you do not explicitly specify an id when uploading a file. I did not implement many details (no username-password authentication for now). So, feel free to modify the codes to meet your own needs.


parsing \t in .fnt files

https://www.glyphite.com/ uses \t as delimiter. Your parser can't handle it. Not sure if it libgdx issue or glyphite. https://github.com/libgdx/libgdx/blob/master/gdx/src/com/badlogic/gdx/graphics/g2d/BitmapFont.java#L915-L920

Provide a .fnt generated by glyphite and maybe someone will look into it. :wink:

Here is the default one https://www.dropbox.com/s/g6z5ftqe3pt3nxr/Untitled Scene.fnt?dl=0

I can confirm this issue. I used atom to replace \t with one spacebar and it was enough to load the font in libgdx.

I still wonder if glyphite is simply taking liberties with the format.

See #2960


question: plugin usage

hi there
how can i find and use some plugin such as rmqtt-sys-topic ?

Hello, configure in 'rmqtt. toml', such as:
##--------------------------------------------------------------------
Plugins
##--------------------------------------------------------------------
#Plug in configuration file directory
plugins.dir = "rmqtt-plugins/"
#Plug in started by default, when the mqtt server is started
plugins.default_startups = [
#"rmqtt-retainer",
#"rmqtt-auth-http",
#"rmqtt-cluster-broadcast",
#"rmqtt-cluster-raft",
#"rmqtt-http-api",
#"rmqtt-sys
---- Replied Message ----
| From | @.> |
| Date | 10/24/2023 01:33 |
| To | @.> |
| Cc | @.***> |
| Subject | [rmqtt/rmqtt] question: plugin usage (Issue #47) |
hi there
how can i find and use some plugin such as rmqtt-sys-topic ?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

i add rmqtt-sys-topic in etc/rmqtt.toml
plugins.default_startups = [
"rmqtt-cluster-broadcast"
"rmqtt-cluster-raft"
"rmqtt-auth-http",
"rmqtt-retainer",
"rmqtt-web-hook",
"rmqtt-http-api",
"rmqtt-sys-topic"

]
and add a file etc/plugins/rmqtt-sys-topic.toml with content
$ cat etc/plugins/rmqtt-sys-topic.toml
##--------------------------------------------------------------------
rmqtt-sys-topic
##--------------------------------------------------------------------
#$SYS system message publish QoS
publish_qos = 1
#$SYS system message publish period
publish_interval = "1m"
but my mqtt client can not receive any event from $SYS topics  :(
is there any way to find out why ?

Please review the 'rmqtt-acl.toml' configuration. By default, the 'rmqtt-acl' plugin is enabled, and only connections from the 'dashboard' user or with a client IP of '<IP_ADDRESS>' are allowed to subscribe to the '$SYS/#' topic.
Default 'rmqtt-acl.toml' configuration:
rules = [
    ["allow", { user = "dashboard" }, "subscribe", ["$SYS/#"]],
    ["allow", { ipaddr = "<IP_ADDRESS>" }, "pubsub", ["$SYS/#", "#"]],
    ["deny", "all", "subscribe", ["$SYS/#", { eq = "#" }]],
    ["allow", "all"]
]

well i figure it out, my fault.
i changed rmqtt from 0.2.12 to the latest version 0.2.19 and sys topics works.
thanks a lot for your amazing project !


how to use .env file in production?

In code I have used TraversePath().Load() and it's working good when testing, but when I publish the application as standalone it's not using the .env file. How can I use .env file?

as per the README, I do not recommend doing what you are doing. You should use actual env vars instead of a .env file.
if you are publishing the .env with the application binary, you need to have the .env file in the correct location relative to the Load call you make. If you provide no args, then your .env should be in the same directory as the binary.


Minor: Fix off-by-one in varint size calculation

I was reading through the varint implementation in postcard to sanity-check one of my own implementations, and I spotted a mistake in the code that calculates the maximum number of varint bytes an integer datatype can take.
The old version adds 7 instead of 6, so instead of the behaviour documented in the comment it would overestimate the number of varint bytes required by one for datatypes whole size is a multiple of 7 bits. Since there are thankfully none among the common integer datatypes, the behaviour is correct both before and after the change.

Thanks, great catch!


do not include error message in status condition

error message can contain AWS generated username (when doing
AssumeRole/STS). This can cause a loop of Status() followed by
Reconcile().

/assign @dgoodwin
/cc @suhanime

/lgtm


Replace deprecated set-output command with environment file

Describe the issue
In workflow, set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information, see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Closing as all these got fixed when we moved into jreleaser based new workflows.


Part 5 step 8 error

Part 5 Step 8 seems to be inaccurate as there does not exist a "aurei" folder in any of the AuReiNand versions.

???

Sorry, it would seem my decompression program (AlZip) just can't find the folder.

Odd, it doesn't contain anything though and your program probably skips empty folders.

Either way, sorry for bringing up the false error. Thanks you guys what you guys do.


Add asp-format support in input[type=hidden] tag helper

This is a variant of @Haplois #5109 PR. I'll close that one in a minute. Note this one addresses all of my review comments on #5109.
My second commit primarily adds tests. Also fixes one test that didn't have a Moq setup for the method InputTagHelper now uses when handling input[type=hidden].

/to @ajaybhargavb please review.

:shipit:

4bda1cb and 34a4c8c


Project users list

Updated project resolvers and services to see users for a given project

Codecov Report

Merging #40 (83fa048) into main (cb1b49a) will decrease coverage by 3.23%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
- Coverage   43.83%   40.61%   -3.23%
==========================================
  Files          29       31       +2
  Lines         454      490      +36
  Branches       26       27       +1
==========================================
  Hits          199      199
- Misses        253      289      +36
  Partials        2        2

Impacted Files
Coverage Δ

packages/server/src/project/project.controller.ts
0.00% <0.00%> (ø)

packages/server/src/project/project.resolver.ts
0.00% <0.00%> (ø)

packages/server/src/project/project.service.ts
0.00% <0.00%> (ø)

packages/server/src/user/model/user.model.ts
0.00% <0.00%> (ø)

packages/server/src/user/user.module.ts
0.00% <0.00%> (ø)

packages/server/src/user/user.resolver.ts
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Overmind puts .overmind.sock into root dir instead of .devenv/state

Describe the bug
Overmind puts .overmind.sock into root dir instead of .devenv/state when having process.implementation = "overmind"; configured
To reproduce
{ pkgs, ... }:
{
  processes.entryscript.exec = ''
    ${pkgs.coreutils}/bin/sleep infinity
  '';
  process.implementation = "overmind";
}

Version
 $ devenv version
devenv 1.0.2 (x86_64-linux)

It should really got into DEVENV_RUNTIME. Could you make a PR?


Add separate editor plugin for TileMap and TileSet

Right now TilesEditorPlugin handles both TileMap and TileSet, which results in some errors when both are edited at the same time. Editing multiple objects is not supported; this PR adds separate dummy plugins for TileMap and TileSet, which just forward their methods to the original plugin.
It's a bit awkward, but it functions the same as before and doesn't spit errors.
I also added a note in EditorPlugin docs.

Is this a fix for #74543?

CC @groud

I think this is a good thing to do. I think the implementation is OK but it's maybe too many layers right now, it becomes a bit complex.
I think we should probably remove TilesEditorPlugin and move functions related to TileMaps / TileSets in their respective plugin. For synchronization, we should probably have a TilesEditorSync singleton that would hold and forward the changes. Have you considered such an approach too ?

Hmmm, now that I think about it, TilesEditorPlugin was a hack to allow 2 editors at once, but it's no longer necessary probably 🤔 Removing it might fix some bugs.

Ok done. TilesEditorPlugin now handles some singleton things, but otherwise all editing is done by TileMapEditorPlugin and TileSetEditorPlugin. I had to resort to some tricks to make everything work correctly; I didn't do much testing though (yet).

TilesEditorPlugin now handles some singleton things, but otherwise all editing is done by TileMapEditorPlugin and TileSetEditorPlugin

I think you can renames it then, to avoid any confusion. Something like TilesEditorSynchronizer or something.

But the class does more than just synchronizing, so if anything, it needs a better name.

Maybe TilesEditorsCommons? Or TilesEditorShared? TilesEditorHelper?  TilesEditorHelpDesk?  TilesEditorHotline? TilesEditorReferee? TilesEditorIDK :sweat_smile: ?

Renamed to TilesEditorUtils.

These plugins are not exposed, so they can be renamed at any time.

Thanks!

Is this cherry pickable for 4.1? The issue appears to also occur there


Baggage content is too trace related, .NET specific example is confusing

Couple suggestions:

The contents are too trace related. Baggage can be used for all signals, not just as span attributes. Metrics can have attributes populated from baggage. Same with logs
I'd recommend to use a diff. example as Activity.Current?.SetTag("AccountId", accountId) is .NET specific, which can confuse users as .NET calls its spans "Activity". Users will find it confusing. If we keep the same example, mention in code comment that Activity is .NET's representation of Span.

Agreed overall, although I'd say that Baggage is currently the most useful in tracing since it lets you propagate metadata along the rest of a trace, whereas this kind of capability is less important outside the context of tracing. So while it's true that it's independent from tracing, it's also much more niche to consider it outside the context of tracing, since going through the pains of propagating baggage without getting the benefit of tracing is kinda weird.

@cijothomas Are you up for it? :)

@cijothomas Are you up for it? :)

I'll help review any PRs addressing this. I just don't have the bandwidth to work on an actual PR itself.

Closing as per doc update


Schedulling Migrations

When working with complex release, we might deploy the application several times and once everything is satisfied then run a final clean up, eg. removing a renamed column that was hard to compute, until a full feature validation is done.
If we have a release cadence its easy to know when, but if not at some time in the future should suffice.
Can we somehow update the cli, to generate a scheduled release and the migrator to take the date/version into account?
At the moment we just need to track outside the code but that requires something alerting us, that "enough time passed, so add this clean up/xxx to the next release", which if falls down the cracks we would have unnecessary data lying around.
With this new feature we could just set a date a month from now and forget, we can always add a warn to the user that there is a "scheduled migration to be applied at any point after yyyy-mm-dd", in case it needs to be removed beforehand.
This would be backwards compatible as if removed, nothing would happen since the _sql_migrations table have no knowledge of it as it was not applied. The check and warn could be around this line, maybe?
Please let me know if this is something you find worth it and I can try to shave some time to work on it.
Thanks again for the fantastic work.

I don't think this is a feature we would accept. You can always just schedule a deployment that has the cleanup migrations in it.

Fair call, I didn't think about this possibility.
Thanks for the prompt reply!


Prevent glGetError infinity

if EGL context is not available, the glGetError always return GL_INVALID_OPERATION.
I use my simple EGL to WGL wrapper, and found an infinite loop when destroying interpreter.
https://github.com/metarutaiga/TensorFlowLiteReduce/blob/master/third_party/EGL/egl_windows.c

@googlebot I signed it!

@impjdi Can you please review this PR ? Thanks!

FYI this PR broke internal Google apps and will be rolled back.


Align with OC4IDS

Closes https://github.com/open-contracting/ocds-extensions/issues/135:

Add content from https://standard.open-contracting.org/infrastructure/latest/en/guidance/identifiers/#project-identifiers-in-ocds
Update example to illustrate difference between infrastructure (or PPP) and national budget projects
Rationalise examples since there's no difference in modelling for infrastructure and PPP projects
Demonstrate how to reference a project in OC4IDS
Add OC4IDS ProjectSector codelist to classification scheme codelist

Test failures:
test_csv.py::test_codelist
  codelists/+classificationScheme.csv: 'OC4IDS_ProjectSector' does not match '^([a-z]+([A-Z][a-z]+|EU|GPP)*|[A-Z]+[0-9]?|[a-z]+(.[a-z]+)*|[A-Z]{2}(-[A-Z]+)+|TED_[A-Z]+(_[A-Z]+)*|date-time|LineString|Point|Polygon|MultiLineString|MultiPoint|MultiPolygon|(application|audio|font|example|image|message|model|multipart|text|video)/[\\w.+-]+)$' (items/properties/Code/pattern)

Not sure on how best to format the code or whether the test needs updating, as for the TED_ and EU_ codes.
test_json.py::test_extension_json
  "1.2"
  '1.2' is not one of ['1.1'] (properties/compatibility/items/enum)

With a view to merging the extension into OCDS 1.2, I targetted compatibility with 1.2 (classificationScheme.csv rather than itemClassificationScheme.csv). Not sure what to do about the test.
test_json.py::test_extension_json
  ./extension.json is not valid JSON Schema (1 errors)

Looks like an issue with the test (extension.json isn't expected to by JSON Schema).
test_readme.py::test_example_backticks
test_readme.py::test_example_backticks
  README.md: "planning/project/id" term is not in schema

I think this and subsequent errors are because release-schema.json lacks the planning property. Not sure how best to resolve.

Since this is merging to master (which OCDS 1.1 uses), let's use +itemClassificationScheme.csv.

OC4IDS_ProjectSector

oc4idsProjectSector, per the usual camelCase form.
We only use underscores for TED to exactly match the XML element names, which are all-caps with underscores (with TED_ prefix).

With a view to merging the extension into OCDS 1.2, I targetted compatibility with 1.2 (classificationScheme.csv rather than itemClassificationScheme.csv). Not sure what to do about the test.

We aren't updating any compatibility fields until 1.2 release, so leave it as 1.1.

./extension.json is not valid JSON Schema (1 errors)

This will go away once compatibility is fixed. I've fixed the warning message now.

README.md: "planning/project/id" term is not in schema

The readme wants you to use planning.project.id

@odscjen, I've added some more orientation about the different types of project that a contracting process might be related to.

I made my changes directly, which you can view together here: https://github.com/open-contracting-extensions/ocds_project_extension/compare/8dcdfe3..135-align-with-oc4ids

Your changes look good to me.


Ask user if they want to download the latest model. #321 with test cases passed and resolved

Implementation for check_new_release method and write up code for the user permission if they want to download the latest model . And added test cases of new function in test folder. and clean up the code according to google chromium style. I have created test cases and passed error solved of github issue.

Hey @henrykironde , I have some problem passing the use_bird_release test cases. After running this function the no bird.pt file is created. How can i resolve this?

Describe the problem a little more.

Hey @henrykironde
Expected:- I have expected bird.pt file to be created to pass the test cases of use_bird_release
Actual:- bird.pt file is not created which result in fail of use_bird_release test cases.


Feature Request: Accept custom distance function based on labels

I have a distance matrix precalculated for each pair of labels, and I would like to pass that distance instead of eucliden or some other method.
It will e helpful if the distanceFn accepts a function that takes two labels as arguments and return the distance.

You can already do this. See this example: https://observablehq.com/@fil/umap-simple-topologies

Hi @zdk123,
Thank you for this. But I needed the distanceFn to be invoked using the labels instead of indices for supervised projection. I created a workaround in my local by mapping labels to indices.


Use matrix variable in action step name

Feature Description
I want to use the a matrix value within the name of a step.
Something like this:
jobs:
  build:
    runs-on: ubuntu-latest
    container:
      image: catthehacker/ubuntu:act-latest
    strategy:
      matrix:
        include:
          - docker-id: "example-a"
          - docker-id: "example-b"
    steps:
      - name: Build and push Docker image ${{ matrix.docker-id }}
        ...

Currently the variable is not replaced with the value and is just displayed as is.
As reference: Github supports using a matrix variable name within the step name.
Screenshots

And it may also be needed to support other varibales, so I think it's related to #24604

try ${{ matrix.include.docker-id }}

Unfortunately this does not work either:

Related to #28207


java error

- VSCode Version: Code 1.8.1 (ee428b0eead68bf0fb99ab5fdc4439be227b6281, 2016-12-19T14:42:49.304Z)
- OS Version: Darwin x64 16.1.0
- Extensions:

Extension
Author
Version

swift
Kasik96
0.0.2

javac-linter
faustinoaq
1.2.1

code-runner
formulahendry
0.6.5

ios-snippets
ornithologist
0.0.15

java
redhat
0.0.8

vscode-swift-language
rlovelett
0.1.0

java-imports-snippets
tushortz
0.0.1

java-snippets
tushortz
0.0.1

I keep getting this message every time I launch, even though I already installed java

I think this issue should be triaged in Java repository first.

This issue was moved to redhat-developer/vscode-java#134


Add --cpus options to a container

Changes the class HostConfig to receive the property cpus when creating a  new container.
Add unit test to the specified parameters.
Solves #959

Codecov Report

Merging #1034 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1034   +/-   ##
=========================================
  Coverage     67.39%   67.39%
  Complexity      774      774
=========================================
  Files           176      176
  Lines          3233     3233
  Branches        367      367
=========================================
  Hits           2179     2179
  Misses          897      897
  Partials        157      157

Can someone comment on this?  There is a CpuCount, but is it really Windows only?  It's located under the HostConfig.


Plus sign literals in argument to derive always give an empty charclass

I was processing regexes with various parts of greenery and believe I've found a bug in derive's handling of r"\+".
Not previously documented, so presume to be a bug
I've read the docs and see no mention besides the following:

The following metacharacters and formations have their usual meanings: ., *, +, ?, {m}, {m,}, {m,n}, (), |, [], ^ within [] character ranges only, - within [] character ranges only, and \ to escape any of the preceding characters or itself.

I take this to mean that should be used to escape the +` character (as is conventional, and not surprising).
Expected behaviour
There are tests demonstrating how derive works in the rxelems_test.py module
https://github.com/qntm/greenery/blob/10223e69236f04ee6fd9dfe99262046f0333bda1/greenery/rxelems_test.py#L882-L887
These respectively show that:

a+ (1 or more a), derive'd by a, gives a* (0 or more a)
a+ (1 or more a) or b+ (1 or more b), derive'd by a, gives a* (0 or more a)
abc or ade, derive'd by a, gives bc or de
abc or ade, derive'd by ab, gives c
abc or ade, derive'd by c, gives [] (the empty character class)

From the README:

An empty charclass [] is legal and matches no characters: when used in a regular expression, the regular expression may match no strings.

Unexpected behaviour with r"\+"
The plus sign handling should work like this (checked items are those which do behave like this):

[x] r"a\+".derive("a") should give r"\+"
[x] r"ab\+c".derive("ab") should give r"\+c"

>>> str(parse(r"a\+").derive(r"a"))
'\\+'
>>> str(parse(r"ab\+c").derive(r"ab"))
'\\+c'

[ ] r"a\+b" derived by r"a\+" should give "b"
[ ] r"ab\+c" derived by r"ab\+" should give "c"

>>> str(parse(r"a\+b").derive(r"a\+"))
'[]'
>>> str(parse(r"ab\+c").derive(r"ab\+"))
'[]'

These instead give an empty charclass [], which I think is incorrect.
It appears in cases where the r"\+" sign is in the right hand pattern (i.e. the argument to derive).
I haven't currently located it any further than this but will update this issue if I do.

My guess is that this could be to do with your use of backslashes. Remember the input to the Brzozowski derivative is a plain old string, not a regular expression - as such, it does not need special characters to be escaped in any way. So try parse(r"a\+b").derive("a+").

Oh! Thank you @qntm I hadn't picked up on that :-)


Swap Back to SOL

OK, I got this working how would one go about swapping back to SOL I was able to figure out buying the tokens using the logic just failing to convert back to sol wondering if u can show an example of changing the USDC back to SOL

Hi there,
We use GitHub issues as a place to track bugs and other development-related issues.
Please see the link below to our dedicated support line:
Help Center :  Click Here
Ticket ID: WB240
Note: Click on the live chat icon at the bottom corner of the page to start a conversation.

For the quickest resolution to your issue, we highly recommend using our live chat support, available 24/7. Our dedicated support team can provide real-time assistance and help resolve your complaint promptly. You can access the live chat here: Raydium SUPPORT.
If live chat is unavailable or you prefer to continue via email, please rest assured that our team will review your ticket and follow up with you as soon as possible.


OpenSSL version compatibility fixes and better exception safety

@toregge could you verify that this compiles cleanly with OpenSSL 1.1.0?

On 1.1.0, make TLS version dynamic (but at least v1.2)
On 1.0.1, manually set a P-256 curve for ECDH
Ensure that exception during TLS context construction does not leak SSL_CTX

Also verified to compile and test OK with OpenSSL 1.0.1 and 1.0.2


Investigate topojson "frame" issue

When converting topojson data to geojson in Python there seems to be a global "frame" object that hides everything when filled.

Documentation has been updated to use geopandas instead of topojson. It's much simpler and there is no "frame" issue.


feat: Allow passing autodetect_schema on table update

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

Didn't open an issue because this seems trivial, happy to adapt if necessary.
[x ] Ensure the tests and linter pass (mvn test verify
[x ] Code coverage does not decrease (if any source code was changed)
[x ] Appropriate docs were updated (if necessary)

@alvarowolfx seems like you might be a suggested reviewer here?

@shollyman could you take a look in place of Neenu?  This change is to help a customer resolve an issue they are seeing.

Please also add IT tests to verify the functionality.

Please also add IT tests to verify the functionality.

@Neenu1995 I don't appear to have access to the test buckets to verify a unit test.  I can push what I think it likely a working unit test but would likely need your help to verify.  Is there some other way I can formulate a unit test without access to the buckets?

Hi @emkornfield,
Feel free to add the tests to the PR. We can help with the verification.

@Neenu1995 added a first stab at an integration test.

com.google.cloud.bigquery.BigQueryException: An internal error occurred and the request could not be completed. This is usually caused by a transient issue. Retrying the job with back-off as described in the BigQuery SLA should solve the problem: https://cloud.google.com/bigquery/sla. If the error continues to occur please contact support at https://cloud.google.com/support. Error: 8965006
	at com.google.cloud.bigquery.spi.v2.HttpBigQueryRpc.translate(HttpBigQueryRpc.java:115)
	at com.google.cloud.bigquery.spi.v2.HttpBigQueryRpc.patch(HttpBigQueryRpc.java:285)
	at com.google.cloud.bigquery.BigQueryImpl$15.call(BigQueryImpl.java:696)
	at com.google.cloud.bigquery.BigQueryImpl$15.call(BigQueryImpl.java:693)
	at com.google.api.gax.retrying.DirectRetryingExecutor.submit(DirectRetryingExecutor.java:103)
	at com.google.cloud.RetryHelper.run(RetryHelper.java:76)
	at com.google.cloud.RetryHelper.runWithRetries(RetryHelper.java:50)
	at com.google.cloud.bigquery.BigQueryImpl.update(BigQueryImpl.java:692)
	at com.google.cloud.bigquery.it.ITBigQueryTest.testUpdatePermExternableTableWithAutodetectSchemaUpdatesSchema(ITBigQueryTest.java:1593)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	```

Two test failing :

testUpdatePermExternableTableWithAutodetectSchemaUpdatesSchema - java.lang.AssertionError

testListTablesWithPartitioning - java.lang.ClassCastException: com.google.cloud.bigquery.AutoValue_ExternalTableDefinition cannot be cast to com.google.cloud.bigquery.StandardTableDefinition

@Neenu1995 got integration test work (at least kokoro seems happy).

Thanks @emkornfield. LGTM.


Quickstart docs does not include installation instructions

Describe what should be investigated or refactored
The Quickstart documentation in the README does not describe how to install the maru-runner release and jumps right into using it. A simple pre-req section that quickly describes how to install maru-runner would be helpful for new people to the project.

Thanks @YrrepNoj we'll prioritize this


[BUG][NOWEB] - Cannot read properties of null (reading 'conversation')

Describe the bug
Upgraded to 2024.9.1 from previous version and any message sent to the whatsapp number wont send a request to the webhook and will provide nontype error in the docker logs.
Engine I'm using NOWEB engine
Docker Logs
[12:38:04.230] ERROR (WhatsappSession/49): session:session_b2cu2w - Failed to process incoming message {"session":"session_b2cu2w","sessionRunTimestamp":1724848452249}
[12:38:04.230] ERROR (WhatsappSession/49): session:session_b2cu2w - Cannot read properties of null (reading 'conversation') {"session":"session_b2cu2w","sessionRunTimestamp":1724848452249}
err: {
"type": "TypeError",
"message": "Cannot read properties of null (reading 'conversation')",
"stack":
TypeError: Cannot read properties of null (reading 'conversation')
at WhatsappSessionNoWebPlus.extractBody (/app/dist/core/engines/noweb/session.noweb.core.js:958:28)
at WhatsappSessionNoWebPlus.extractReplyTo (/app/dist/core/engines/noweb/session.noweb.core.js:976:27)
at WhatsappSessionNoWebPlus.toWAMessage (/app/dist/core/engines/noweb/session.noweb.core.js:935:30)
at WhatsappSessionNoWebPlus.processIncomingMessage (/app/dist/core/engines/noweb/session.noweb.core.js:921:31)
}
Trace: TypeError: Cannot read properties of null (reading 'conversation')
at WhatsappSessionNoWebPlus.extractBody (/app/dist/core/engines/noweb/session.noweb.core.js:958:28)
at WhatsappSessionNoWebPlus.extractReplyTo (/app/dist/core/engines/noweb/session.noweb.core.js:976:27)
at WhatsappSessionNoWebPlus.toWAMessage (/app/dist/core/engines/noweb/session.noweb.core.js:935:30)
at WhatsappSessionNoWebPlus.processIncomingMessage (/app/dist/core/engines/noweb/session.noweb.core.js:921:31)
at WhatsappSessionNoWebPlus.processIncomingMessage (/app/dist/core/engines/noweb/session.noweb.core.js:926:21)
[12:54:55.627] ERROR (WhatsappSession/49): session:session_b2cu2w - Failed to process incoming message {"session":"session_b2cu2w","sessionRunTimestamp":1724848452249}
[12:54:55.627] ERROR (WhatsappSession/49): session:session_b2cu2w - Cannot read properties of null (reading 'conversation') {"session":"session_b2cu2w","sessionRunTimestamp":1724848452249}
err: {
"type": "TypeError",
"message": "Cannot read properties of null (reading 'conversation')",
"stack":
TypeError: Cannot read properties of null (reading 'conversation')
at WhatsappSessionNoWebPlus.extractBody (/app/dist/core/engines/noweb/session.noweb.core.js:958:28)
at WhatsappSessionNoWebPlus.extractReplyTo (/app/dist/core/engines/noweb/session.noweb.core.js:976:27)
at WhatsappSessionNoWebPlus.toWAMessage (/app/dist/core/engines/noweb/session.noweb.core.js:935:30)
at WhatsappSessionNoWebPlus.processIncomingMessage (/app/dist/core/engines/noweb/session.noweb.core.js:921:31)
}
Note:
Downgraded back to NOWEB (2024.8.5) and everything is working well.

Fixed in 2024.9.2 :crossed_fingers:
Remember that you can always pin the docker version in case of any errors with new one
https://waha.devlike.pro/docs/how-to/engines/#versions
devlikeapro/waha-plus:latest-2024.8.5

@yeheskel2016 thank you for such detailed report, it helped a lot! :bow:


DependencyDetector

I have found a command that will make is very easy to find out what the server binary files require to run. Im hoping to create a function that will be able to tell me what dependencies are required.
readelf -d srcds_linux

I have created this so if you use ./csgoserver depsdetect it will identify the required dependencies . this is a development tool so I will keep it as a hidden tool.


Modify classList.remove

MODIFIED FILE : fill-colours.js
I have joined two classList.remove in one.
I tried to do an arrow function but it did not work.
Please, have a look. Thank you in advance.

Good job Tania!
Joining the classList.remove is a good idea :)
Another idea could be to use querySelectorAll to select all the inputs and add the eventListener with a "for" bucle.

done the array to add eventListener to all inputs


how to copy/save message data?

Hi,
I am trying to save message data for future parsing/analysis using FileChannel (fc) and temp file
and can see that ByteBuffer is allocated ok, but all data in that temp file is ZEROS:
copyMqtt(mess): mess.len: 20

CopyMqtt(mess): fc.pos1()=0
CopyMqtt(mess): bytesread=20
CopyMqtt(mess): fc.pos2()=20
mcopy.length=20
00000000000000000000
bacopy: =[B@54ba904b  //byte []
copyMqtt(mess): mess.len: 20
CopyMqtt(mess): fc.pos1()=0
CopyMqtt(mess): bytesread=20
CopyMqtt(mess): fc.pos2()=20
mcopy.length=20
00000000000000000000
bacopy: =[B@11b814b8 //byte [] with data

Tried using new String (byte [], charset) with different char sets but result is still the same: all zeros.
Anyone knows how to save message data in String?
All ideas are appreciated!
Br,
George Matveev

Sorry George, but I can't understand your question.You have to copy data
inside Moquette's protocol handling?
Andrea
Il giorno 31/ott/2016 15:06, "George Matveev"<EMAIL_ADDRESS>ha
scritto:

Hi,
I am trying to save message data for future parsing/analysis using
FileChannel (fc) and temp file
and can see that ByteBuffer is allocated ok, but all data in that temp
file is ZEROS:
copyMqtt(mess): mess.len: 20

CopyMqtt(mess): fc.pos1()=0
CopyMqtt(mess): bytesread=20
CopyMqtt(mess): fc.pos2()=20
mcopy.length=20<PHONE_NUMBER>0000000000
bacopy: =[B@54ba904 https://github.com/B/moquette/commit/54ba904b
//byte []
copyMqtt(mess): mess.len: 20
CopyMqtt(mess): fc.pos1()=0
CopyMqtt(mess): bytesread=20
CopyMqtt(mess): fc.pos2()=20
mcopy.length=20<PHONE_NUMBER>0000000000
bacopy: =[B@11b814b https://github.com/B/moquette/commit/11b814b8
//byte [] with data

Tried using new String (byte [], charset) with different char sets but
result is still the same: all zeros.
Anyone knows how to save message data in String?
All ideas are appreciated!
Br,
George Matveev
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/andsel/moquette/issues/230, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA_Y0f0PgZcjaDGzlAdru20CSwcoDwUaks5q5fXygaJpZM4KlDCw
.

Hi Andrea, will try to clarify:
I'd like to spare copy of message for future parsing/analysis (e.g. via SQL server).
And I've tried using ByteBuffer->FileChannel->String(byte []) mechanism.
But result is somewhat strange:

copyMqtt(mess): fc.pos1()=0
copyMqtt(mess): bytesread=20
byte[0]=0
byte[1]=0
byte[2]=0
byte[3]=0
byte[4]=0
byte[5]=0
byte[6]=0
byte[7]=0
byte[8]=0
byte[9]=0
byte[10]=0
byte[11]=0
byte[12]=0
byte[13]=0
byte[14]=0
byte[15]=0
byte[16]=0
byte[17]=0
byte[18]=0
byte[19]=0
copyMqtt(mess): fc.pos2()=20
Arrays.toString(bacopy)=[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]
00000000000000000000
bacopy: =[B@4d6385fe
copyMqt: =

ByteArray (bacopy) of message appears to contain only ZEROs after I allocated Direct ByteBuffer
and got (seemingly) all message data into it via this code:
      bbcopy = ByteBuffer.allocateDirect(mess.getRemainingLength());
	  bbcopy.clear();//clear garbage

	  //1 read from FileChannel into ByteBuffer
	  int bytesread = fc.read(bbcopy);
	  System.out.println("+ copyMqtt(mess): bytesread="+bytesread);//OK

	  //2 write all message bytes into ByteBuffer copy/console
	  for(int i=0; i<bytesread; i++)
	  {
	     fc.write(bbcopy);
	     byte b = bbcopy.get(i);
	     //lets see what's inside???
	     System.out.println("byte["+i+"]="+b);//ZEROs
	     //Nothing! with getChar(i)/getInt(i)
	     //"java.nio.HeapByteBuffer[pos=20 lim=20 cap=20]" with bbcopy
	  }

But as you see from printout above it contains only ZEROs.
So the question remains: how to read/copy message data into String?
Any help would be very much appreciated.
Best regards,
GM

One more thing:
When I send file via broker, contents of the file (e.g. runBroker.sh script)
are correctly displayed on console where I subscribed to the topic (via mosquitto_sub -h localhost),
but in the underlying file used by FileChannel (for read/write)
and in my program/IDE message content appears as ZEROs. VERY Mysterious!
Any clue?
Br,
GM

Actually ALL symbols in that FileChannel underlying file are "^@"!
What could possibly mean?

Hi,
actually the problem seems to be even deeper:
When I try to get message payload length I am not getting anything at all!
    ByteBuffer payload = mess.getPayload();
	byte [] mba = payload.array();//get message data
	int meslen = mba.length;//data length
	System.out.println("*\n copyMqtt: meslen="+meslen);//Nothing

How is this API supposed to work???
Please reply.
/GM

Hi George
you are speaking about JDK's api and Netty's, Moquette use that but I can
only provide support on Moquette API's, that's not part of this project.
Your question seems related to use of copy data from Netty's API, you could
give a look at Netty's docs.
Andrea
On Thu, Nov 3, 2016 at 11:10 AM, George Matveev<EMAIL_ADDRESS>wrote:

Hi,
actually the problem seems to be even deeper:
When I try to get message payload length I am not getting anything at all!
ByteBuffer payload = mess.getPayload();
byte [] mba = payload.array();//get message data
int meslen = mba.length;//data length
System.out.println("*\n copyMqtt: meslen="+meslen);//Nothing

How is this API supposed to work???
Please reply.
/GM
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/andsel/moquette/issues/230#issuecomment-258104138,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA_Y0Vb9dt5M7SY3jUL28uWhRcfvsduYks5q6bMYgaJpZM4KlDCw
.

Here is how it supposed to work:
    //This code WORKS fine!
	byte [] ba = new byte [] {'G','e','o','r','g','e'};
	ByteBuffer bb = ByteBuffer.wrap(ba);
	msg.setPayload(bb);

Later I write it via FileChannel to disk, and here's correct result:
$ cat mqtmessage.dat
George
And the whole program (i.e. my own code) works fine!
But code that comes standard with Moquette:
    //copyMqt = new String(msg.getPayload().array());//NOP
	//copyMqt = new String(msg.getPayload().array());//NOP

does NOT work at all!
The problem is in getPayload() method!
And it seems that was the reason you removed it from original version of Moquette:
https://github.com/andsel/moquette/commit/fedd2965ccf100e8fd8acc3684899593db3fafb8
Who would be the right person to fix this bug? Any ideas/estimations how/what is to be done?
It is MAJOR blocking issue in the whole project, imho.
Br,
GM

Sorry @GeorgeMatveev but I continue to don't understand what you  mean.

It's a little bit difficult to understand which is your test case, in case if you find a discrepancy in Moquette write a unit/integration test to make evident the problem.
Thanks
Andrea

Hi Andrea,
it is actually hard to understand how else I can clarify the problem
I have with msg.getPayload().array() methods...
Why did you remove those lines with msg.getPayload().array() code from HazelcastInterceptHandler.java ???

   LOG.info(String.format("%s publish on %s message : %s\n", msg.getClientID(), msg.getTopicName(), new String(msg.getPayload().array())));

   LOG.info("{} publish on {} message: {}", msg.getClientID(), msg.getTopicName(), new String(msg.getPayload().array()));
   ITopic<HazelcastMsg> topic = hz.getTopic("moquette");

   HazelcastMsg hazelcastMsg = new HazelcastMsg(msg.getClientID(),msg.getTopicName(), msg.getQos().byteValue(), msg.getPayload().array(), msg.getUsername());

   HazelcastMsg hazelcastMsg = new HazelcastMsg(msg.getClientID(), msg.getTopicName(), msg.getQos().byteValue(),

           msg.getPayload().array(), msg.getUsername());

here is the commit YOU made on 22 July:
https://github.com/andsel/moquette/commit/fedd2965ccf100e8fd8acc3684899593db3fafb8

Also if you try to print out results from spi/impl/ProtocolProcessor.java
private static IMessagesStore.StoredMessage asStoredMessage(PublishMessage msg) {
System.out.println("\n* GM ProtProc: msg.QoS="+msg.getQos()+ ", topic="+msg.getTopicName());//
System.out.println("* GM ProtProc: msg.isRetaineFlag="+msg.isRetainFlag());//NOP
System.out.println("* GM ProtProc: msg.payload.array="+msg.getPayload().array());//NOP
You will see nothing on console/IDE.
At least it does NOT work for me.
Can you try it yourself?
This is very simple test/use case!
Cheers,
GM


Make the form mobile friendly

Make the form on defi_form.html and success.html mobile friendly. The labels are the wri g width and the picture is too large. If anyone wants to help, take it, i would be happy.

Closed in https://github.com/chnuessli/defikarte.ch/pull/19
Thanks to @PatriciaHeimfarth


Option to pass Data without copying it

Would like an option to pass data to Angular-Confirm without copying it.  This would allow the ability to have inputs on the confirmation modal that can update the calling parent scope.

I'm a little confused on what you mean here. Can you submit a PR so we can have an implementation to discuss?

Sure thing, will submit a pull request by tomm night.  Thanks!

I"m a terrible person and got caught up in life :/  Closing the issue since I'm too terrible to follow up on it.


Add seasonality parameter for holt-winters functions

This PR adds the possibility to specify the seasonality when asking for a Holt-Winters prediction.
Before, the seasonality was to 1 day by default.

Sorry about that, I'm a python newbie and I must have missed this...
Should be ok now (all tests are ok on my PC)

Codecov Report

Merging #2225 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2225      +/-   ##
==========================================
+ Coverage   79.47%   79.48%   +<.01%
==========================================
  Files          85       85
  Lines        8845     8846       +1
  Branches     1889     1889
==========================================
+ Hits         7030     7031       +1
  Misses       1550     1550
  Partials      265      265

Impacted Files
Coverage Δ

webapp/graphite/render/functions.py
96.04% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Looking much better, the last thing that needs to be done is to update the parameter definitions, you'll want to add an entry for seasonality after the Param('bootstrapInterval', ParamTypes.interval, default='7d', suggestions=['7d', '30d']), line for each function.  This is used to expose the parameter to make the new parameter available in systems like Grafana.

Thanks for your help, it's done as well :+1:

Thanks @LTMXcitrus 🎉


Cannot build angular_constants on new database

When trying to create a new DB:
Traceback (most recent call last):
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/backends/utils.py", line 85, in _execute
    return self.cursor.execute(sql, params)
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/backends/sqlite3/base.py", line 296, in execute
    return Database.Cursor.execute(self, query, params)
sqlite3.OperationalError: no such table: auth_permission

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "manage.py", line 11, in <module>
    exit(main())
  File "/home/finn/OpenSlides/OpenSlides/openslides/__main__.py", line 40, in main
    execute_from_command_line(sys.argv)
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/core/management/__init__.py", line 381, in execute_from_command_line
    utility.execute()
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/core/management/__init__.py", line 357, in execute
    django.setup()
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/__init__.py", line 24, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/apps/registry.py", line 120, in populate
    app_config.ready()
  File "/home/finn/OpenSlides/OpenSlides/openslides/core/apps.py", line 45, in ready
    set_constants(get_constants_from_apps())
  File "/home/finn/OpenSlides/OpenSlides/openslides/utils/constants.py", line 16, in get_constants_from_apps
    out.update(get_angular_constants())
  File "/home/finn/OpenSlides/OpenSlides/openslides/users/apps.py", line 57, in get_angular_constants
    for permission in Permission.objects.all():
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/models/query.py", line 268, in __iter__
    self._fetch_all()
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/models/query.py", line 1183, in _fetch_all
    self._result_cache = list(self._iterable_class(self))
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/models/query.py", line 54, in __iter__
    results = compiler.execute_sql(chunked_fetch=self.chunked_fetch, chunk_size=self.chunk_size)
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/models/sql/compiler.py", line 1061, in execute_sql
    cursor.execute(sql, params)
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/backends/utils.py", line 100, in execute
    return super().execute(sql, params)
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/backends/utils.py", line 68, in execute
    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/backends/utils.py", line 77, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/backends/utils.py", line 85, in _execute
    return self.cursor.execute(sql, params)
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/utils.py", line 89, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/backends/utils.py", line 85, in _execute
    return self.cursor.execute(sql, params)
  File "/home/finn/OpenSlides/.venv37/lib/python3.7/site-packages/django/db/backends/sqlite3/base.py", line 296, in execute
    return Database.Cursor.execute(self, query, params)
django.db.utils.OperationalError: no such table: auth_permission

Way to reproduce:
rm -rf OpenSlides
git clone -b OpenSlides-3 https://github.com/OpenSlides/OpenSlides.git
python3 -m venv .virtualenv
source .virtualenv/bin/activate
pip install -r requirements.txt
python manage.py createsettings

Could you test this in #3843
I think it should be fixed with that pr

Nope. Still broken. The same procedure leads to the same result.

With commit https://github.com/OpenSlides/OpenSlides/commit/8adaa6118a2bb0332680b5ada0a8a92bea99e0a5 the migrations and startup worked fine. It Broke with commit https://github.com/OpenSlides/OpenSlides/commit/732de97ec285fc1cf5e10ab082c1f5b374b4ca47.

PR #3855 fixes the issues.


Add a failing test case for .length() conflict

chai-changes seems to conflict with Chai built-in .length() method
Ref #2

Somehow the addChainableMethod breaks the existing property.
  describe 'addChainableMethod', ->
    it 'does not affect an existing property', ->
       langChain1 = expect([1, 2]).to.to
       langChain2 = expect([1, 2]).has.has
       expect(langChain1).eql langChain2

This fails with the message
AssertionError: expected [Function: assert] to deeply equal { Object (__flags) }
So far I have not found a workaround for it

See https://github.com/chaijs/chai/issues/427
I think using lengthOf instead of length is the way to go

I think using lengthOf instead of length is the way to go

Yeah, I suppose so. That insight should be added to the readme with big emphasis — it's not fun to debug the whole spec suite when adding a new matcher.

This will be fixed in chai, so closing this pull request


import os is missing in rccl_ops.py

System information
Same configuration with https://github.com/ROCmSoftwarePlatform/tensorflow-upstream/issues/369
Code to reproduce the issue
Please use test script described in:  https://github.com/ROCmSoftwarePlatform/tensorflow-upstream/issues/369
Traceback (most recent call last):
  File "./tesu2.py", line 16, in <module>
    y_pre = network(x)
  File "./tesu2.py", line 11, in network
    x0 = rccl.all_sum([x0,x1])
  File "/home/w/miniconda3/envs/py36_test/lib/python3.6/site-packages/tensorflow/contrib/rccl/python/ops/rccl_ops.py", line 49, in all_sum
    return _apply_all_reduce('sum', tensors)
  File "/home/w/miniconda3/envs/py36_test/lib/python3.6/site-packages/tensorflow/contrib/rccl/python/ops/rccl_ops.py", line 217, in _apply_all_reduce
    _validate_and_load_rccl_so()
  File "/home/w/miniconda3/envs/py36_test/lib/python3.6/site-packages/tensorflow/contrib/rccl/python/ops/rccl_ops.py", line 289, in _validate_and_load_rccl_so
    _maybe_load_rccl_ops_so()
  File "/home/w/miniconda3/envs/py36_test/lib/python3.6/site-packages/tensorflow/contrib/rccl/python/ops/rccl_ops.py", line 273, in _maybe_load_rccl_ops_so
    library_filename = os.path.join(resource_loader.get_data_files_path(),
NameError: name 'os' is not defined

We could simply add import os before line 273 to solve the issue.

Thanks!


Google Dataset Search: Invalid string length in field 'description'

Google Dataset Search requires the description field to have a minimum of 50 characters and a maximum of 5000 characters. See https://developers.google.com/search/docs/appearance/structured-data/dataset for the definition as used by Google.
Some datasets have very short descriptions, such as https://datacatalogue.cessda.eu/detail?q=1bf6f5e9e571391922c9dc87cb48742a677156a6962392aa71f4efc4401a3e9f&lang=en. We should not expose these as JSON-LD.

There is also the question of whether such short description lengths should be added to the Data Catalogue at all.


Typeguards for isResource and isReference

I saw a common pattern in our code where we check if resourceType or reference is in an object, and then casts it.
This PR takes advantage of Typescript's user-defined type guards to centralize that check in a utility function and eliminate the cast

Coverage: 93.886% (-0.02%) from 93.905% when pulling 763ca75bc80a6ca33b3f6570b57613cb338f7dea on rahul-typeguards into bc0d9e844dd5a74aaaec65b941003356f3173ef2 on main.


shape Box missing a @property of thickness

as the title
it just miss a define of

    @property
    def thickness(self):
        return self._thickness

    @thickness.setter
    def thickness(self, thickness):
        self._thickness = thickness
        self._update_vertices()

(I'll fix that later, but I'm on a high speed railway train

Thanks for catching it.

PR is on the way

closed by #1120


fix(field): fixed assign default value for nested form.

What kind of change does this PR introduce? Bug fix

Current coverage is 92.73% (diff: 100%)

Merging #363 into master will increase coverage by 0.66%

@@             master       #363   diff @@
==========================================
  Files            16         16
  Lines           769        771     +2
  Methods         150        150
  Messages          0          0
  Branches        197        198     +1
==========================================
+ Hits            708        715     +7
+ Misses           61         56     -5
  Partials          0          0

Powered by Codecov. Last update 2027949...339341a


Release 3.10 download gives version 3.8.1 for --version

Release 3.10 download gives version 3.8.1 for "php ./behat.phar --version"
wget https://github.com/Behat/Behat/releases/download/v3.10.0/behat.phar
--2021-11-02 22:09:49--  https://github.com/Behat/Behat/releases/download/v3.10.0/behat.phar
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://github-releases.githubusercontent.com/1367484/e0873b4e-ba85-4c0b-849b-74e49e5cef7b?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20211102%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20211102T210950Z&X-Amz-Expires=300&X-Amz-Signature=86da702c662a6dec1364b3d711a02303abced3185f7762ca9fff315539a076b6&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=1367484&response-content-disposition=attachment%3B%20filename%3Dbehat.phar&response-content-type=application%2Foctet-stream [following]
--2021-11-02 22:09:50--  https://github-releases.githubusercontent.com/1367484/e0873b4e-ba85-4c0b-849b-74e49e5cef7b?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20211102%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20211102T210950Z&X-Amz-Expires=300&X-Amz-Signature=86da702c662a6dec1364b3d711a02303abced3185f7762ca9fff315539a076b6&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=1367484&response-content-disposition=attachment%3B%20filename%3Dbehat.phar&response-content-type=application%2Foctet-stream
Resolving github-releases.githubusercontent.com (github-releases.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to github-releases.githubusercontent.com (github-releases.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 3163686 (3,0M) [application/octet-stream]
Saving to: ‘behat.phar’

behat.phar                                                                      100%[====================================================================================================================================================================================================>]   3,02M  7,03MB/s    in 0,4s

2021-11-02 22:09:50 (7,03 MB/s) - ‘behat.phar’ saved [3163686/3163686]

php ./behat.phar --version
behat 3.8.1

Note to self: remember about \Behat\Behat\ApplicationFactory::VERSION while releasing versions 🤷🏻♂
I'll release v3.9.2 and v3.10.1 with fixed version data tomorrow.

@pamil maybe that should have a check. We could parse the changelog and compare the versions?

liip/RMT can also help with this.

Hello,
@pamil  any updates ?. Version 3.10 still points to 3.8.1. Not able to execute tests with php8.1.
Deprecated: explode(): Passing null to parameter #2 ($string) of type string is deprecated in 8.1

I'm trying to execute tests from Phpstorm terminal and I receive:
[04-May-2022 15:08:44 Europe/Brussels] PHP Deprecated: explode(): Passing null to parameter #2 ($string) of type string is deprecated in /opt/.phpstorm_helpers/behat.php on line 556
PHP: 8.1.3

@svservices this error is in the phpstorm helper script, not in Behat. So that's where you should report it.

Thanks. I will check with them :).


Local watcher: create context for Register call

I could now understand the reason behind our discussion regarding the context used for Register call.
We should probably use a context with timeout and the timeout should be the maximum time that can be taken for registering. However, this maximum value can be different for different backends. So, it will be tricky to determine what timeout to set here.
After this PR is merged, I will create an issue, where we could discuss this and find a solution.
Originally posted by @manoranjith in https://github.com/hyperledger-labs/go-perun/pull/276#discussion_r763812802

Addressed by #360


Figure out the new naming for hittable

Need to come up with new naming for hittables as it doesn't seem entirely right for me. May have a look at pbrt and see how they call similar entities there.

They call it 'shape', sounds really good to me.

They call it 'shape', sounds really good to me.


scipy.optimize.dual_annealing always rejects non-improving state

scipy.optimize.dual_annealing does not actually accept a state if the improvement is negative, which makes this algorithm not behave in a desired way.
Here is the section to determine whether it accepts a new state with greater function value.
https://github.com/scipy/scipy/blob/master/scipy/optimize/_dual_annealing.py#L260-L271
        pqv_temp = (self.acceptance_param - 1.0) * (
            e - self.energy_state.current_energy) / (
                self.temperature_step + 1.)
        if pqv_temp <= 0.:
            pqv = 0.
        else:
            pqv = np.exp(np.log(pqv_temp) / (
                1. - self.acceptance_param))
        if r <= pqv:
            # We accept the new location and update state
            self.energy_state.update_current(e, x_visit)
            self.xmin = np.copy(self.energy_state.current_location)

Here, self.acceptance_param directly comes from accept parameter with a range (-1e4, -5], e - self.energy_state.current_energy is positive since in this step the function value is greater than current state, and self.temperature_step + 1 is also positive.
In the end, pqv_temp is negative and the state is accepted with zero probability.
According to the tutorial page and the paper it refers, it can be
pqv_temp = (self.acceptance_param - 1.0) * (             e - self.energy_state.current_energy) /                  self.temperature_step + 1., but I am not familiar with the notation and did not open a pull-request.
I inserted a debug message when it moves to a state with greater value, and found that it is never called, regardless of the temperature.
Could somebody check this and fix?
Scipy/Numpy/Python version information:
import sys, scipy, numpy; print(scipy.__version__, numpy.__version__, sys.version_info)
1.3.0 1.15.4 sys.version_info(major=3, minor=6, micro=7, releaselevel='final', serial=0)

@sgubianpm thanks for contributing dual_annealing to SciPy. Do you have a moment to take a look at this bug report? Could be a very simple fix!

This was reported again in gh-10892, and that was fixed by gh-12266.


Right-clicked Key Numbers for the Mini-Keyboard

The problem i am having is not knowing what pitch of note block I'm placing (the amount of times it would take to right-click a note block)
Its a very easy add, what would be added is an option in the settings to label the keys on the keyboard from 0-23 being how many times it would take to right-click the note block. the keys that would be labeled are those in Minecraft's 2-octave range.

^^ This would be an example

@TheCraftedGamer I just implemented this, but I think it looks a bit clogged, especially considering the keys in the 2-octave range are the only ones that have a keyboard shortcut by default.

I'll add it anyway, but if you have any suggestions on how to improve it, please leave a comment!


session_destroy(): Session object destruction failed

What steps will reproduce the problem?

create default yii2-advanced app (development env)
add this component config:

'session' => [
    'class' => 'yii\web\CacheSession',
    'cache' => 'sessionCache',
    'timeout' => 600000
 ],
'sessionCache' => [
    'class' => 'yii\caching\FileCache',
    'cachePath' => '@runtime/cache/session'
]

log in
log out

What is the expected result?
proper logout
What do you get instead?
php warning
Additional info

Q
A

Yii version
2.0.10-dev

PHP version
PHP 7.0.6

Operating system
Linux 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt25-1 (2016-03-06) x86_64 GNU/Linux

in /srv/www/yii2t/vendor/yiisoft/yii2/web/Session.php:198

can you please paste the full stack trace of the error?

@Shin1gami should be OK now. Thanks for reporting.


fix: fox assets names and icon overrides

Description
FOX assets on different networks in the app do not have the same icon and naming scheme. Let's fix this.

Overrides the name, icon and color for the 3 networks FOX is currently supported on in the app (Ethereum, Gnosis and Polygon)
Updates the generatedAssets
Updates the colorMap for assets

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[x] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk
Low, but a lot of Cosmos pools seem to have been removed from Coingecko, might be worth investigating if any of these are in use by our users, but I'm not sure how to proceed here.

Testing
Search for FOX in the Global Search or Assets page and notice all the ShapeShift FOX tokens use the same icons and same naming scheme "Fox on ".

Engineering
☝

Operations
☝

Screenshots (if applicable)
Before:

After:

Product reviewed here.
Remains taking a look at the Cosmos assets that Coingecko seems to have dropped since last update, any idea if it's normal?

As in #4643 I've removed the changes that are not related to FOX assets. The asset-data generation script will have to be updated later in a separate PR, but at least we can get these changes in and they should remain when the script will be fixed for Cosmos/Osmosis assets.


[Enhancement] Optimize ingestion performance for table with materialized index STEP 1

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #7717
Problem Summary(Required) ：

When a table has multiple materialized views, StarRocks will send the chunks to the tablets corresponding to the materialized views, which leads to the enlargement of network transmission bandwidth and unnecessary data serialization and transmission overhead. Since the materialized view and the base table use the same partition column and distribute in the same node when using colocate mv #7451, we can reuse the chunk data of the base table.

materialized index num
main branch
optimize branch
SpeedUp

0
1314MB/s
1310MB/s

1
780MB/s
1274Mb/s
1.6x

3
420MB/s
1241MB/s
2.9x

5
286MB/s
1213MB/s
4.2x

using github_events data, 16 parallel
3 BE with 1.5GB/s network bandwidth
create two columns of materialized view; materialized view computation is relatively small; ensure the bottleneck is network bandwidth and serialization.

NOTE We will enable it after colocate mv index implementation.

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

run starrocks_be_unittest

run starrocks_be_unittest

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

@mergify rebase

run starrocks_fe_unittest

run starrocks_fe_unittest

@mergify rebase

LGTM for the LoadChannel and TabletsChannel parts.


Cannot connect to ASE

Preflight Checklist

[x] I have installed the latest version of Storage Explorer.
[x] I have checked existing resources, including the troubleshooting guide and the release notes.
[x] I have searched for similar issues.

Storage Explorer Version
1.14.2
Regression From
No response
Architecture
i86
Storage Explorer Build Number
No response
Platform
All
OS Version
No response
Bug Description
{
"message": ""{\n  \"name\": \"Error\",\n  \"message\": \"Failed to refresh access token\",\n  \"stack\": \"Error: Failed to refresh access token\\n    at PathClient. (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\se-adls2-extension\\\\dist\\\\src\\\\Clients\\\\StorageClient.js:227:35)\\n    at step (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\tslib\\\\tslib.js:141:27)\\n    at Object.next (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\tslib\\\\tslib.js:122:57)\\n    at fulfilled (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\tslib\\\\tslib.js:112:62)\\n    at processTicksAndRejections (internal/process/task_queues.js:93:5)\"\n}""
}
Steps to Reproduce
Launch Storage Explorer
{
"message": ""{\n  \"name\": \"Error\",\n  \"message\": \"Failed to refresh access token\",\n  \"stack\": \"Error: Failed to refresh access token\\n    at PathClient. (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\se-adls2-extension\\\\dist\\\\src\\\\Clients\\\\StorageClient.js:227:35)\\n    at step (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\tslib\\\\tslib.js:141:27)\\n    at Object.next (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\tslib\\\\tslib.js:122:57)\\n    at fulfilled (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\tslib\\\\tslib.js:112:62)\\n    at processTicksAndRejections (internal/process/task_queues.js:93:5)\"\n}""
}
Actual Experience
{
"message": ""{\n  \"name\": \"Error\",\n  \"message\": \"Failed to refresh access token\",\n  \"stack\": \"Error: Failed to refresh access token\\n    at PathClient. (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\se-adls2-extension\\\\dist\\\\src\\\\Clients\\\\StorageClient.js:227:35)\\n    at step (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\tslib\\\\tslib.js:141:27)\\n    at Object.next (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\tslib\\\\tslib.js:122:57)\\n    at fulfilled (C:\\\\Users\\\\nisjohn\\\\AppData\\\\Local\\\\Programs\\\\Microsoft Azure Storage Explorer\\\\resources\\\\app\\\\node_modules\\\\tslib\\\\tslib.js:112:62)\\n    at processTicksAndRejections (internal/process/task_queues.js:93:5)\"\n}""
}
Expected Experience
No response
Additional Context
No response

@nisjohn898 We no longer support 32-bit (i86) Storage Explorer. Is it possible for you to install the latest 64-bit version and try again?

@@.***>
Can you please let me know how I can download the 64 bit version ?
Thanks
Nisha

From: Richard Gao @.>
Sent: Monday, December 2, 2024 3:29 PM
To: microsoft/AzureStorageExplorer @.>
Cc: Nisha John @.>; Mention @.>
Subject: Re: [microsoft/AzureStorageExplorer] Cannot connect to ASE (Issue #8335)
@nisjohn898https://github.com/nisjohn898 We no longer support 32-bit (i86) Storage Explorer. Is it possible for you to install the latest 64-bit version and try again?

Reply to this email directly, view it on GitHubhttps://github.com/microsoft/AzureStorageExplorer/issues/8335#issuecomment-2513189654, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BA7ZAXAV3XYROBNK7LU5QLD2DTUNJAVCNFSM6AAAAABSTIA6F6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJTGE4DSNRVGQ.
You are receiving this because you were mentioned.Message ID: @.***>

Our releases can be found here: https://azure.microsoft.com/en-us/products/storage/storage-explorer#Download-4 or https://github.com/microsoft/AzureStorageExplorer/releases


Add event helpers

Codecov Report

Merging #14 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #14   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines         390    397    +7
=====================================
+ Hits          390    397    +7

Impacted Files
Coverage Δ

dist/karet.util.cjs.js
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Overriding VectorstoreIndexCreator() embedding

This is the default class of VectorstoreIndexCreator
class VectorstoreIndexCreator(
    *,
    vectorstore_cls: type[VectorStore] = Chroma,
    embedding: Embeddings = OpenAIEmbeddings,
    text_splitter: TextSplitter = _get_default_text_splitter,
    vectorstore_kwargs: dict = dict
)

the default is to use OpenAIEmbeddings as its embedding. What i'm trying to do is to use BedrockEmbeddings which looks like
VectorstoreIndexCreator(embedding: Embeddings = BedrockEmbeddings) but i am getting a "embedding" is not defined" error. Any idea how to do that? Thanks
My code that is getting the error
index = VectorstoreIndexCreator(
    embedding: Embeddings = BedrockEmbeddings,
    ).from_loaders([loaders])

@dosubot i have already configured my AWS credentials profile name, AWS region name through aws configure command, as for my Bedrock model ID check the code below
boto3_bedrock = boto3.client("bedrock-runtime")
llm = Bedrock(
    model_id="anthropic.claude-v2",
    client=boto3_bedrock,
    model_kwargs={"max_tokens_to_sample": 200},
)
bedrock_embeddings = BedrockEmbeddings(
    model_id="amazon.titan-embed-text-v1", client=boto3_bedrock
)

And yes i can get a response from claude succesfully cause im following this workshop it just that when i modify the code like in my earlier post where i modify the VectorstoreIndexCreator class to accept embedding  parameter is when i got the error

okay yeah it works thanks


Make Check Constraints Load Correct Sim Dataset

Tickets addressed: Closes #1168.
Review: By commit
Merge strategy: Merge (no squash)

Description

Resolves the bug by allowing the Simulation Service to look up either a specific SimulationDataset, or the latest one if one isn't specified.
Verification

An e2e Test has been added to validate that constraints loads the specified Simulation Results, even if those results are outdated. It was placed in its own file to avoid any potential caching issues with the existing constraints tests.
A bindings tests has been added to validate the error response if an invalid sim dataset id is passed.
Documentation

No docs need to be updated
Future work

The UncachedSimulationService is not only unused, it's not representative of our system, which now generally expects results to have been cached. As a result, I'd like to flatten this interface.

Record keeping: this was overlooked when I closed https://github.com/NASA-AMMOS/aerie/issues/1006, as I thought @cohansen's work with tracking constraint runs handled this case as well.


Improve Local file system and the fileSystem interface

Please remove SambaFileSystem.java from this PR, it is out of scope and not yet fully functional
Please add LocalFileSystemSender and it's test methods.

LocalFileSystem.java

renameTo() should throw an exception when target file exists and force=false
please remove //TODO Auto-generated method stub comments in open() and close(), replace with comment that no action is required

FileSystemTest.java

please add in all the comments of the methods with underscore
waitForActionToFinish(): please do something with the InterruptedException. Never catch an Exception and ignoring, without even a comment why it can be ignored!
testNotExist() does not need to create the file before deleting it.
avoid double //   //   comments
What is the meaning of testExistsContinue()? It appears to be the same as testExists(). If useful, please document, otherwise remove
existsCheck(): do not need to catch exception when it is only rethrown
testAppendFile(), and also other test methods: please separate the parts for 'setup', 'execute' and 'assert' by whitespace and a comment line.
setWaitMilis(): please add extra 'l': millis in stead of milis