Fix broken Docker recipe and update dependencies

Jax version dictated by colabfold is too old resulting in a docker build error.
Also updated tensorflow to 2.15 & CUDA to 12.2 along with other packages.

@anuprulez do you want to create a new release?

@anuprulez is the IT on .eu still broken? Because that is why I had started this.

@tuncK yes, its broken I think with the following error message when I run a tensorflow notebook:
2023-12-14 14:09:26.827782: W tensorflow/tsl/framework/cpu_allocator_impl.cc:82] Allocation of 188160000 exceeds 10% of free system memory.
2023-12-14 14:09:28.328554: W tensorflow/compiler/xla/service/gpu/nvptx_helper.cc:56] Can't find libdevice directory ${CUDA_DIR}/nvvm/libdevice. This may result in compilation or runtime failures, if the program we try to run uses routines from libdevice.
Searched for CUDA in the following directories:
  ./cuda_sdk_lib
  /usr/local/cuda-11.2
  /usr/local/cuda
  .
You can choose the search directory by setting xla_gpu_cuda_data_dir in HloModule's DebugOptions.  For most apps, setting the environment variable XLA_FLAGS=--xla_gpu_cuda_data_dir=/path/to/cuda will work.
2023-12-14 14:09:28.339603: W tensorflow/compiler/xla/service/gpu/llvm_gpu_backend/gpu_backend_lib.cc:326] libdevice is required by this HLO module but was not found at ./libdevice.10.bc
2023-12-14 14:09:28.340107: W tensorflow/core/framework/op_kernel.cc:1830] OP_REQUIRES failed at xla_ops.cc:446 : INTERNAL: libdevice not found at ./libdevice.10.bc

I can create a new release but after verifying all the notebooks (except those using Colabfold) by running this container on my VM. Probably next week.

The newly released v0.4 version of this tool throws the same error as above as well as the v0.3:
libdevice not found at ./libdevice.10.bc

Can you please try this again tomorrow: I tried https://github.com/usegalaxy-eu/infrastructure-playbook/pull/1067

I tried it, but unfortunately, it did not work. I tried this solution already directly in the Docker container as well. Still, it does not find libdevice file even though it is present at /opt/conda/nvvm/. I will look into it.

The last time I was dealing with this, it had something to do with:

Paths that it is looking to by default
Versions hardcoded in the filenames (/some/path/xlib.v123.so)


After 'pnpm test' ,Test Suites and Tests  get some fails

Why I get some Test failed when  'pnpm test'?
How to fix them?
please look at the  following output：
 src/core/__test__/directive.spec.ts
  ● Test suite failed to run

    src/core/utils/computeExpression.ts:37:13 - error TS2345: Argument of type 'unknown' is not assignable to parameter of type 'string'.

    37       error(err, expression, el);
                   ~~~

 PASS  src/core/utils/__test__/adjustDeps.spec.ts
 PASS  src/core/__test__/reactive.spec.ts
 FAIL  src/core/utils/__test__/customProp.spec.ts
  ● .customProp › should correctly access custom prop

    The error below may be caused by using the wrong test environment, see https://jestjs.io/docs/configuration#testenvironment-string.
    Consider using the "jsdom" test environment.

    ReferenceError: document is not defined

      3 | describe('.customProp', () => {
      4 |   it('should correctly access custom prop', () => {
    > 5 |     const el = document.createElement('div');
        |                ^
      6 |
      7 |     // @ts-expect-error: .component doesn't exist on HTMLElement
      8 |     el.component = 'foo';

      at Object.<anonymous> (src/core/utils/__test__/customProp.spec.ts:5:16)

Hey sorry for the late response, I didn't get a notification for some reason. Fixed in latest commit

Hey sorry for the late response, I didn't get a notification for some reason. Fixed in latest commit


Classes and methods aren't very safe

Some methods can be accessed from places where it doesn't make sense. For example initializations for audio and other modules should only be accessible from anywhere other than the tahm class, however, some other methods for audio SHOULD be accessible

I think that the problem is in your engine architecture. I quite don't get why are you put definitions Window,Input, and Renderer classes into the Tahm class. Why they can't be stand alone classes defined in the Tahm namespace?
Why they are defined in the private part of the Tahm class, if you are exposing all of them to public API by creating public roll pointers in Tahm class? Why did you choose the role pointers instead of smart one?

The library is inspired by Love2D, Tham class tries to mimic the "love" structure from Love2D, I think that's why it's structured this way.
From what I can understand everything should be managed trough the Tham Singleton, you access Window, Renderer, Graphics, etc "modules" (as they are called in Love2D) trough public members and I think the reason they are nested and private inside the Tham Class is to prevent instantiation from outside the class.
It also ensures a simple syntax, for example draw a rectangle you do:
tahm.graphics->draw->rect()
I don't like the use of the -> operator but this could be easily fixed by just declaring them normally. It should be like this
tahm.graphics.draw.rect()
It's less cumbersome.
First I think we should reorganize the code to be more legible, I think the current code is fine enough, it compiles and does what it supposed to do, it's just weirdly organized before making serious changes.
Then I think all SDL2 subsystems should be initialized right away inside Tahm's constructor with default values (Window, Renderer, Sound, TTF, etc), then call Start() function afterwards not before, the way is implemented right now works because we aren't capable of doing lots of things, like creating a Texture, which wouldn't work because it requires the Video Subsystem and a Renderer to be initialized.
The more I think about how this works the more I like it. Keep it up!

@tamtaasatiani, shouldn't this be closed now? Alongside #7.


Tell node to ignore interval when considering when done

I have a simple sbot that connects to a peer and then disconnects right after. Sadly sbot never closes even after calling sbot.close(). I found that this fixes the problem for me. All the other intervals used in sbot has this pattern.
Code to reproduce:
sbot.gossip.connect(peer, err => {
  if (err) throw(err)
  else console.log("Connected to " + peer.address)

  sbot.gossip.disconnect(peer, err => {
    if (err) throw(err)
    else console.log("Disconnected from " + peer.address)

    sbot.close(true)
  })
})

yup, definitely, good hunting!  I know stuff like this can be really hard to find.

merged into 2.1.3

Thanks for the quick merge! I used the excellent https://github.com/mafintosh/why-is-node-running package to find it.

wow, i didn't know that existed!
On 10/24/18, Anders Rune Jensen<EMAIL_ADDRESS>wrote:

Thanks for the quick merge! I used the excellent
https://github.com/mafintosh/why-is-node-running package to find it.
--
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/dominictarr/pull-inactivity/pull/1#issuecomment-432381819


Updated to use mock-component helper version from ember-test-utils

This project uses semver, please check the scope of this pr:

[X] #none# - documentation fixes and/or test additions
[ ] #patch# - backwards-compatible bug fix
[ ] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Update tests that use mock-component helper to use the version in ember-test-utils
Remove local copy of mock-component helper

👍


Czech translation of Sign in

Hey, one of our translators taking care of the Czech translation noticed that a better translation of "Sign in" would be "Přihlásit se". I think of this line specifically (but this may apply to other places as well): https://github.com/tigrish/devise-i18n/blob/0beb0b6a60a87b21375b80300fed6ff565bf2fc0/rails/locales/cs.yml#L122
Some examples of popular websites using this wording below:

Google

Facebook

Airbnb

Does anyone being responsible for the Czech translation of Devise have an opinion on this? =)

If you think it's better, please update on localeapp or create a pull request. There is no specific person responsible for each language that you need to check with.


[BUG] Sas Uris created using BlobSasBuilder.ToSasQueryParameters() with userdelegation key do not authenticate as expected

Library name and version
Azure.Storage.Blobs 12.19.1
Describe the bug
I am using the BlobSasBuilder.ToSasQueryParameters() and using the uri to download a file to a memory stream. The code for this is pasted below in the "Reproduction Steps" section. This always fails with the exception : "'Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature."
Here is an example URI created which causes the auth failure :
https://suhastestdata.blob.core.windows.net/testpublisher/testregistrypackage.zip?skoid=e6f1eecd-b80f-414c-bf3a-d9c49b28ff74&sktid=72f988bf-86f1-41af-91ab-2d7cd011db47&skt=2023-12-18T19%3A38%3A55Z&ske=2023-12-23T19%3A38%3A55Z&sks=b&skv=2023-11-03&sv=2023-11-03&spr=https&st=2023-12-18T19%3A39%3A18Z&se=2023-12-19T19%3A39%3A18Z&sr=b&sp=r&sig=REDACTED

I have already verified the following :

The UTC time on my machine is set correctly with the actual UTC time
The User delegation key start and and expiry time are before and after the sas uri start and expiry time
When I create the sas uri with the portal specifying the userdelegation , the download works file so the user permissions are not an issue. For your reference this is the URI created when using the portal note that the skoid and the sktid are exactly the same as above . I double checked all the query params and they all are same as the one above.

https://suhastestdata.blob.core.windows.net/testpublisher/testregistrypackage.zip?sp=r&st=2023-12-18T19:30:15Z&se=2023-12-19T03:30:15Z&skoid=e6f1eecd-b80f-414c-bf3a-d9c49b28ff74&sktid=72f988bf-86f1-41af-91ab-2d7cd011db47&skt=2023-12-18T19:30:15Z&ske=2023-12-19T03:30:15Z&sks=b&skv=2022-11-02&spr=https&sv=2022-11-02&sr=b&sig=REDACTED

Expected behavior
Should be able to use user delegation keys to download from blob
Actual behavior
Unable to use user delegation keys to download from blob
Reproduction Steps
The basic code looks like the following . The DownloadToAsync function always fails with authentication fa
var blobServiceClient = new BlobServiceClient(new Uri("https://suhastestdata.blob.core.windows.net/"), cred);
var userDelegationKey = await blobServiceClient.GetUserDelegationKeyAsync(DateTimeOffset.UtcNow, DateTimeOffset.UtcNow.AddDays(5)).ConfigureAwait(false);

var blobContainerClient = blobServiceClient.GetBlobContainerClient("testpublisher");
var blobClient = blobContainerClient.GetBlobClient("testregistrypackage.zip");
var t = blobClient.GetSasUri(TimeSpan.FromDays(1), userDelegationKey);

using (MemoryStream ms = new MemoryStream())
{
    var sasBlobC = new BlobClient(t);
    try
    {
        await sasBlobC.DownloadToAsync(ms).ConfigureAwait(false);
    }
    catch (Exception)
    {
    }
}

The GetSasURI extension function from above is pasted here below
        public static Uri GetSasUri(
            this BlobClient blobClient,
            TimeSpan expiryOffset,
            UserDelegationKey userDelegationKey,
            BlobSasPermissions blobSasPermissions = BlobSasPermissions.Read)
        {
            var sasBuilder = new BlobSasBuilder(blobSasPermissions, DateTimeOffset.UtcNow.Add(expiryOffset));

            sasBuilder = new BlobSasBuilder
            {
                BlobName = blobClient.Name,
                Resource = "b",
                ExpiresOn = DateTimeOffset.UtcNow.Add(expiryOffset),
                StartsOn = DateTimeOffset.UtcNow,
                Protocol = SasProtocol.Https
            };
            sasBuilder.SetPermissions(blobSasPermissions);

              BlobUriBuilder uriBuilder = new BlobUriBuilder(blobClient.Uri)
              {
                  Sas = sasBuilder.ToSasQueryParameters(
                      userDelegationKey,
                      blobClient.GetParentBlobContainerClient().GetParentBlobServiceClient().AccountName)
              };

              return uriBuilder.ToUri();

Environment
.NET core 8
Microsoft Visual Studio Enterprise 2022 (64-bit) - Current
Version 17.8.3

I did some more digging and it seems like the signature is getting generated correctly. The error response I get is as follows :
AuthenticationErrorDetail: Signature did not match. String to sign used was r
2023-12-18T21:50:57Z
2023-12-19T21:50:57Z
/blob/suhastestdata/testpublisher/testregistrypackage.zip
e6f1eecd-b80f-414c-bf3a-d9c49b28ff74
72f988bf-86f1-41af-91ab-2d7cd011db47
2023-12-18T20:42:45Z
2023-12-19T20:42:45Z
b
2023-11-03

https
2023-11-03
b

This means that the server is able to decode the signature correctly , and the URL i used was below and if you compare all the fields with the stringtosign defined here : https://learn.microsoft.com/en-us/rest/api/storageservices/create-user-delegation-sas#version-2020-12-06-and-later , they match exactly. So why does this create the Authentication Error ?
https://suhastestdata.blob.core.windows.net/testpublisher/testregistrypackage.zip?skoid=e6f1eecd-b80f-414c-bf3a-d9c49b28ff74&sktid=72f988bf-86f1-41af-91ab-2d7cd011db47&skt=2023-12-18T20%3A42%3A45Z&ske=2023-12-19T20%3A42%3A45Z&sks=b&skv=2023-11-03&sv=2023-11-03&spr=https&st=2023-12-18T21%3A50%3A57Z&se=2023-12-19T21%3A50%3A57Z&sr=b&sp=r&sig=REDACTED

I figured out the issue. Basically here
https://github.com/Azure/azure-sdk-for-net/blob/aad3be7a5488a95a8c2fbdb2c2610d5f540fcd77/sdk/storage/Azure.Storage.Blobs/src/Sas/BlobSasBuilder.cs#L485
the containername is becoming null causing an incorrect signing for the name.
adding the below snippet fixes it
            sasBuilder = new BlobSasBuilder
            {
                **BlobContainerName= blobClient.BlobContainerName,**

Maybe the bug should be to not create the sas for a blob if the container name is not specified ? Because the error message is pretty confusing and I had to go through the code to actualy figure out the issue.

Yes, so the container name does need to be specified when generating an user delegation sas for Blobs.
We have recently implemented a feature method for generating user delegation sas as shown here. Please leverage this


added GEMPLP

@hushell Could you please help me to check this patch?

@Jiaolong Good good! This one is much better than what I was thinking about.
It's nice to get rid of Region class in the original implementation (OI). I think you have fully understand the MPLP algorithm. I can tell from update_messages(), which is almost different from the OI. You are amazing, man!
The only thing that we should be more careful is the naming rule of variables, functions etc. I'll expand that in the code.

Here I show you the results on yest dataset. The computation is slightly high.
Example 1: tree structure, max-product inference
SGD trained in   47.0507
sgd solver: average training loss = 0.030553
sgd solver: average testing error = 0.113155
Example 2.1: tree structure, graph-cuts inference
SGD trained in   42.9520
sgd solver: average training loss = 0.049546
sgd solver: average testing error = 0.125000
Example 2.2: full-connected graph, graph-cuts inference
SGD trained in   67.9839
sgd solver: average training loss = 0.082026
sgd solver: average testing error = 0.134894
Example 3.1: tree structure, GEMPLP inference
SGD trained in   76.5377
sgd solver: average training loss = 0.031104
sgd solver: average testing error = 0.113852
Example 3.2: full-connected graph, GEMPLP inference
SGD trained in  460.3569
sgd solver: average training loss = 0.030966
sgd solver: average testing error = 0.102425

@Jiaolong Good job! I'll check your code out  and test it on my machine, then I probably give more comments later.

Here are the results from the MPLP V1.0 library.
Example 3.1: tree structure, GEMPLP inference
SGD trained in   80.2897
sgd solver: average training loss = 0.040462
sgd solver: average testing error = 0.120262
Example 3.2: full-connected graph, GEMPLP inference
SGD trained in  213.0953
sgd solver: average training loss = 0.040462
sgd solver: average testing error = 0.120262

@Jiaolong Good to see these comparisons! As we expected, MPLP does a better job than GraphCuts in general graphs and similar to TreeBP in trees. Didn't expect MPLP is so much time consuming, but 460s = <8mins is acceptable? Since GraphCuts was not too bad, maybe better to set it as default.

"the results from the MPLP V1.0 library" is keeping their code as before?
You may want to timing each part of the code to see why the current CGEMPLP is slower. Also check if current CGEMPLP uses more iterations?

MPLP V1.0 library is the one that I kept all original implementation.

I am sorry for the long delay, guys, but this pull request is HUGE :-)

This looks like pretty amazing work, @Jiaolong. Great! Do you think you would be able to showcase in your structure output learning notebook what can we do with it?

@iglesias Thanks for your review! In fact, I have already prepared the notebook. An early version can be found here:
http://nbviewer.ipython.org/gist/Jiaolong/95633b7efd64bbd096f2
I will update it with more detailed explanations and probably with more experiments on other datasets.

I did not see mplp mentioned anywhere in the notebook. Maybe I overlooked?

Sorry, it was an old version. The only clue about MPLP is infer_alg = LP_RELAXATION. :p And now it should be infer_alg = GEMPLP.

All right, got it.
Then, what we need for this pull request is:

Make unit tests simpler (probably just reducing input data) so we don't need to comment any of them, neither to wait long until they finish. If you want to have a larger test, then an integration one (that is, an example in Python modular) is the right place. Anyway, that would go to a future pull request.
Check the function of this FGTestData. I really wonder whether it makes sense to have it in C++. But I need to know more about it first.

Thanks a lot for getting back so quickly, @Jiaolong. Really cool.

The reason I created FGTestData is that the same testing data are used in multiple unit tests, e.g., GrapCuts, GEMPLP, ect, and probably in the future new algroithms. Thus there are too many redudant code. Another reason is that, we haven't implemented functions to read factor graph from files, so we have to run these code to construct factor graph data.

Then, it sounds like unit tests would be a better place for it, right?

Yes, maybe we can move it into unit test folder. However, it can also be used in integral test.

I see your point. To use it from integration tests one could also just generate some data files and push them into the data repository. I admit this is not the most flexible solution though. I leave it up to you, then. If you want to keep the FG data class under structure, it is fine with me.
I will check for other data generators we have in Shogun and see whether it would make more sense there. I will let you know.

There is this DataGenerator class inside the features directory. In my opinion, it is fine to keep this factor graph test data class here under structure, since it is quite specific. @karlnapf?
@Jiaolong, I do suggest to rename it to FactorGraphDataGenerator, to follow the convention with other data classes.

All right, I will change the name.

I just updated the code with your comments. Regarding to the unit tests, valgrind tests are fine:
shogun/build/tests/unit$ valgrind ./shogun-unit-test --gtest_filter="GEMP*"
==4564== Memcheck, a memory error detector
==4564== Copyright (C) 2002-2013, and GNU GPL'd, by Julian Seward et al.
==4564== Using Valgrind-3.10.0.SVN and LibVEX; rerun with -h for copyright info
==4564== Command: ./shogun-unit-test --gtest_filter=GEMP*
==4564==
Note: Google Test filter = GEMP*
[==========] Running 7 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 7 tests from GEMPLP
[ RUN      ] GEMPLP.find_intersections_index
[       OK ] GEMPLP.find_intersections_index (27 ms)
[ RUN      ] GEMPLP.max_in_subdimension
[       OK ] GEMPLP.max_in_subdimension (8 ms)
[ RUN      ] GEMPLP.initialization
[       OK ] GEMPLP.initialization (64 ms)
[ RUN      ] GEMPLP.convert_energy_to_potential
[       OK ] GEMPLP.convert_energy_to_potential (6 ms)
[ RUN      ] GEMPLP.simple_chain
[       OK ] GEMPLP.simple_chain (32 ms)
[ RUN      ] GEMPLP.random_chain
[       OK ] GEMPLP.random_chain (34 ms)
[ RUN      ] GEMPLP.sosvm
[       OK ] GEMPLP.sosvm (30277 ms)
[----------] 7 tests from GEMPLP (30458 ms total)

[----------] Global test environment tear-down
[==========] 7 tests from 1 test case ran. (30498 ms total)
[  PASSED  ] 7 tests.
==4564==
==4564== HEAP SUMMARY:
==4564==     in use at exit: 0 bytes in 0 blocks
==4564==   total heap usage: 7,657,410 allocs, 7,657,410 frees, 164,492,480 bytes allocated
==4564==
==4564== All heap blocks were freed -- no leaks are possible
==4564==
==4564== For counts of detected and suppressed errors, rerun with: -v
==4564== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

Great, thanks a lot for the valgrind log. I see however the GEMPLP.sosvm is still taking too long. Can you reduce that time, please?

The time shown above is measured under valgrind, so it is longer than running at realse mode. It is 452 ms on my PC. But anyway, I will try to reduce more. Currently, the SOSVM data has 4 classes and 16 samples.

Changes Unknown when pulling cfbad41055cb5e463032288dc7578759509f2d91 on Jiaolong:GEMPLP into ** on shogun-toolbox:develop**.

Now the unit test time is reduced to 160 ms.
[       OK ] GEMPLP.sosvm (160 ms)

Oh, sure. Sorry, I did not take into account valgrind when I looked at the running time. 0.4 seconds should be fine, no need to worry more about it.

Perfect, we are ready to go, then. Thanks, @Jiaolong.

Many thanks! :D

Feel free to send an update for the notebook when you have the time. It would be great if it includes as well some short discussion about gemplp.

Sure! I will try to polish the notebook and send it shortly. Thanks!

@Jiaolong @iglesias It's a great news this patch is merged! Sorry I couldn't help for the merge. I spent last fews day on traveling. Thanks for your hardworking! Let's plan our next steps :)


Rust WASM toolchain not installed, even I have installed

Is there an existing issue?

[X] I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

[X] This is not a support question.

Description of bug
When run cargo build --release to substrate-node-template, I met this problem like "Rust WASM toolchain not installed, please install it!", although I have installed it. The details are as follows.
Steps to reproduce
Hello everyone:
When I used commend cargo build --release to substrate-node-template, I get something wrong like this:
error: failed to run custom build command for node-template-runtime v4.0.0-dev (/home/briza/Desktop/substrate/substrate-node-template_ok/runtime)
Caused by:
process didn't exit successfully: /home/briza/Desktop/substrate/substrate-node-template_ok/target/release/build/node-template-runtime-219434d0a12d33e9/build-script-build (exit status: 1)
--- stderr
Rust WASM toolchain not installed, please install it!
Further error information:
error: expected table for configuration key registry, but found string in /home/briza/.cargo/config
I checked my toolchain, and installed: rustup toolchain install nightly-2022-10-20-x86_64-unknown-linux-gnu, but stiil didn't work.
rustup show and rustup nightly show below:
rustup show:
Default host: x86_64-unknown-linux-gnu
rustup home:  /home/briza/.rustup
installed toolchains
stable-x86_64-unknown-linux-gnu (default)
nightly-2021-11-04-x86_64-unknown-linux-gnu
nightly-2021-12-10-x86_64-unknown-linux-gnu
nightly-2022-06-30-x86_64-unknown-linux-gnu
nightly-2022-07-01-x86_64-unknown-linux-gnu
nightly-2022-08-15-x86_64-unknown-linux-gnu
nightly-2022-10-20-x86_64-unknown-linux-gnu
nightly-x86_64-unknown-linux-gnu
active toolchain
stable-x86_64-unknown-linux-gnu (default)
rustc 1.65.0 (897e37553 2022-11-02)
rustup nightly show:
Default host: x86_64-unknown-linux-gnu
rustup home:  /home/briza/.rustup
installed toolchains
stable-x86_64-unknown-linux-gnu (default)
nightly-2021-11-04-x86_64-unknown-linux-gnu
nightly-2021-12-10-x86_64-unknown-linux-gnu
nightly-2022-06-30-x86_64-unknown-linux-gnu
nightly-2022-07-01-x86_64-unknown-linux-gnu
nightly-2022-08-15-x86_64-unknown-linux-gnu
nightly-2022-10-20-x86_64-unknown-linux-gnu
nightly-x86_64-unknown-linux-gnu
installed targets for active toolchain
wasm32-unknown-unknown
x86_64-unknown-linux-gnu
active toolchain
nightly-x86_64-unknown-linux-gnu (overridden by +toolchain on the command line)
rustc 1.67.0-nightly (1eb62b123 2022-11-27)
rust version: rustc 1.65.0 (897e37553 2022-11-02)
Besides, I followed this "https://docs.substrate.io/install/linux/" to reinstall, but still didn't work
what should I do to solve this problem?
thank you all first!

Have you added the WASM target to your toolchain? Something like this rustup target add wasm32-unknown-unknown --toolchain nightly
https://docs.substrate.io/install/linux/ This doc might help you out.

I have followed this, but still can't solve it

I am also facing the same issue

Have the same issue with the following toolchain:
installed toolchains
stable-x86_64-unknown-linux-gnu (default)
nightly-x86_64-unknown-linux-gnu
installed targets for active toolchain
wasm32-unknown-unknown
x86_64-unknown-linux-gnu
active toolchain
stable-x86_64-unknown-linux-gnu (default)
rustc 1.69.0 (84c898d65 2023-04-16)
Btw, I'm on Zorin 16.2 (based on Ubuntu 20.02)

Have the same issue with the following toolchain:
installed toolchains
stable-x86_64-unknown-linux-gnu (default)
nightly-x86_64-unknown-linux-gnu
installed targets for active toolchain
wasm32-unknown-unknown
x86_64-unknown-linux-gnu
active toolchain
stable-x86_64-unknown-linux-gnu (default)
rustc 1.69.0 (84c898d65 2023-04-16)
Btw, I'm on Zorin 16.2 (based on Ubuntu 20.02)

https://docs.substrate.io/install/linux/ This doc might help you out.

@paritytech/docs-audit we need the installation instructions updated. Since 1.69.0 and latest master people need to have the wasm32-unknown-unknown target installed for the stable compiler.


[ACCOUNTING] Outgoing Invoice Callback

[ ] Determine all side effects and add to this issue
[x] Migrate legacy code to new accounting service controller
[ ] Identify all existing code that uses the legacy endpoint and create a follow up ticket to update the call sites to use the new endpoint

Side Effects:
The withdrawal invoice record is updated in the transaction table, it is set to is_pending: false, and marked as completed: true if the status is "completed". The user's balance is decremented if the withdrawal was successful.
This is a ZBD callback endpoint, and so there are no client callsites to update.


Enabling TP-SDS

Implementation of time prioritized SDS from Dreamtime. Had to feed in global_step/max_step when calling self.guidance. Changed this in three popular systems (DreamFusion, ProlificDreamer, Fantasia3D). For other systems changes are similar.

Thank you @yankeesong ! Could you share some results using this strategy?

Sure @bennyguo, will update as soon as I get some results.

Results from DreamFusion:

"a ceramic lion":
Uniform sampling (default):

https://github.com/threestudio-project/threestudio/assets/36912641/14020b88-49c1-4aec-804c-6816d2b5e635
TP-SDS sampling:
https://github.com/threestudio-project/threestudio/assets/36912641/aa052678-3f63-4c14-9b43-95aa98bda886

"a chimpanzee dressed like Henry VIII king of England"
Uniform sampling (default, diverged)

https://github.com/threestudio-project/threestudio/assets/36912641/121e585a-ce84-4bda-a9ea-bab3dd84db2a
TP-SDS sampling:
https://github.com/threestudio-project/threestudio/assets/36912641/7dacb592-677e-498a-b04a-3904e977e5b1

@yankeesong Looks good! I'm working on a general form of implementation of all these types of annealing, and I'll merge your branch when I finish the high-level structure. Leave this PR open for now.


Add eventhub name into load balancer logs

We are consuming messages from multiple event hubs and it is hard to distinguish in logs between messages coming from different event hubs.
This PR adds event hub name into PartitionLoadBalancerEventSource.

/azp run net - eventhub - tests

/azp run net - eventhub - functions - tests

Thanks very much for your contribution, @tovyhnal!   The proposed changes look good.  I've kicked off our live tests to validate that we don't see anything blow up when running against Azure resources or when the log forwarder is active in a Functions context.     Assuming those go well, I'll get this merged in.


How Build New Golang Console App in a Main Go App?

Hello,Go Team.
How Build New Golang Console App in a Main Go App? mean on a system without go compilers with main app.
example :

$go build main.go //this is main app (i dont know how make this!)
./main file.go
and after this , build file.go and save as file.
may compiler without go compiler?
if can not,  available portable version of go?
http://stackoverflow.com/questions/42473834/how-build-new-golang-console-app-in-a-main-go-app

For questions about Go, see https://golang.org/wiki/Questions.


Add support for scenario metadata

Connecting the ETM with the CTM requires that we store the CTM scenario ID with the ETEngine scenario. Integration with other models in the future may require that we add yet more attributes.
I think we have two options:

Add a ctm_scenario_id attribute to scenarios that would allow an API user to set the numerical ID of the CTM scenario.
Add a metadata attribute to scenarios. This would be a JSON object which can be structured however the API user chooses. A freeform "metadata" field might be useful to other users of the API.

I personally lean in favour of the second option, since it means that integration with other models could also use this attribute to store anything of relevance.
However, we'd need to consider what we do when a scenario is cloned. This happens regularly when users create new scenarios through the API and when ETModel creates new saved scenarios., or starts a new scenario based on a saved scenario. Should we also clone the CTM ID / metadata? My gut feeling is that cloned scenarios shouldn't link back to the CTM anymore, but I'm not sure about this. We might need to seek the advice of Kalavasta.
Whichever option is decided, we need to:

Add the field in ETEngine.
Update the API to accept this field.
Validate data provided by the user (if option 1: validate that is a number; if option 2: perhaps validate that they aren't trying to store an excessive amount of data).
Update the API documentation.

@antw @noracato during the client meeting this morning we heard that the CTM scenario database is not created yet; they expect it to be ready by the end of January. Is that a problem for the implementation of this todo?
Notifying @ChaelKruip

Thanks for the clear write up @antw! I went with the metadata idea, because that seemed to me the most future-friendly format.
While working on this, I came about some topics I'd like your opinion on:

How much do we touch the metadata object, and how much do we leave up to the user to decide / do we not care about? As this object is mostly for a users 'own' information, they should have a decent amount of control over it, and we should not care that much. Does that mean:

That we should make the metadata nicely accesible throughout the engine by e.g. converting keys to symbols, and numeric strings to floats and ints? Or just leave it bare?
That we should always merge updated metadata in with the existing metadata, or let users overwrite the whole object?

When to show the metadata. My thought was to only add this information when thedetailed parameter is set to true in API calls.
I put the metadata limit on 2000 characters, is that a fair amount? And would we need more protection than that?


Ability to customize

Currently injected scripts have the type="module" which always implies the defer attribute, this can lead to different execution order of scripts in relation with non-deferred scripts, which is not always desirable, and can lead to some scripts failing, it's needless to say that if all scripts are deferred this is not a problem
defer from MDN

This Boolean attribute is set to indicate to a browser that the script is meant to be executed after the document has been parsed, but before firing DOMContentLoaded.

Scripts with the defer attribute will prevent the DOMContentLoaded event from firing until the script has loaded and finished evaluating.

Scripts with the defer attribute will execute in the order in which they appear in the document.

type="module" from MDN

This value causes the code to be treated as a JavaScript module. The processing of the script contents is deferred.

mapped manually using catalog.collected_js


Getting error when connecting Dynamic 365 business central to Power BI

Hi,
I am trying to connect Dynamic 365 business central online to power BI as instructed here by going through:
Start Power BI Desktop --> Select Get Data --> Online Services --> Dynamics 365 Business Central --> connect,  but it prompt me error below after I signing in:
"
Details: "Microsoft.Mashup.Engine1.Library.Resources.HttpResource: Request failed:
OData Version: 4, Error: The remote server returned an error: (400) Bad Request. (Request data is invalid.)
OData Version: 3, Error: The remote server returned an error: (400) Bad Request. (Request data is invalid.)"
Please advise what is the steps that I am missing.
[Enter feedback here]

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

ID: f562094f-cff2-2c71-c806-e548041718e2
Version Independent ID: 4f1e5021-2ea3-ba4b-8da8-a999589b8962
Content: Building Reports in Power BI Desktop to Display Business Central Data - Business Central
Content Source: business-central/across-how-use-financials-data-source-powerbi.md
Service: dynamics365-business-central
GitHub Login: @jswymer
Microsoft Alias: jswymer

Hello, Thanks for the feedback. Unfortunately, it is hard to determine the cause of this error without knowing more about your environment. One thing you can try, if not already done, is to make sure you have the latest version of Power BI Desktop.
We recommend that you contact Microsoft support through the normal channel. We will then update the docs as a result as needed. Regards.


Add a clang-format style file

clang-format is a command line tool for automatic code formatting, available from homebrew and apt. The style file is just YAML, but clang-format expects a specific file name (with no extension). The style here is based on the LLVM style guide, modified to reflect our current practices.
To see how this applies to our existing code, run
clang-format -style=file [file]

and you'll get the formatted code in standard output. To edit the files in-place, add the -i option.
I don't think we need to run a mass-reformat on our existing code, but we can use this for new files and enforce this guide (within reason) for all future additions.

@hjanetzek Yes, I/we should be rebasing more :D.

@hjanetzek Yeah it's definitely possible to clutter your history if you constantly merge in other branches. I don't find it necessary to merge in other branches more than about once per PR. Personally I kind of appreciate having a verbose history and I feel like it's less error-prone to simply merge. Of course you can rebase if you prefer, maybe you'll change my mind :)


NPSP - Advanced Mapping - Checkbox not displaying as available target field for True/False

We recently restricted the allowed values for mapping Picklist to Boolean to True or False.
Picklists created with these values are not causing our checkbox fields to appear as available target fields
Pre-conditions:
1- Source field is a picklist with values True or False
2- Checkbox created on target object
Steps:
1- Access object group for corresponding target object
2- Click on “New Field Mapping”
3- Select picklist from pre-conditions as source
4- Expand picklist for target field
Expected:
Checkbox is selectable as long as picklist values are True or False
Actual:
Checkbox field not appearing as available for selection

**lurch: attach W-036829

Tracking W-036829


set for all workspaces?

I'd like to set a wallpaper on ALL spaces

I would like that very much, too, but the api that desktoppr is using doesn't provide that option.


Unable to disable Indentation guides

Issue Type: Bug
I am trying to disable indentation guides by setting
    "editor.guides.indentation": false,
    "editor.guides.highlightActiveIndentation": false

and they still appear.
VS Code version: Code 1.66.1 (Universal) (8dfae7a5cd50421d10cd99cb873990460525a898, 2022-04-06T14:49:47.400Z)
OS version: Darwin arm64 21.4.0
Restricted Mode: No

System Info

Item
Value

CPUs
Apple M1 Pro (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 2, 2

Memory (System)
32.00GB (6.68GB free)

Process Argv
--disable-extensions --crash-reporter-id 54ddd194-b78f-478f-ad65-800e1f0af2cc

Screen Reader
no

VM
0%

Extensions disabled

@IllusionMH thanks for the reply.
Yes the problem was with editor.guides.bracketPairs being enabled. I was not understanding that the guides I had enabled were editor.guides.bracketPairs and not normal editor.guides.indentation.


encounter 'undefined is not a function' error while running example app

Running the example app on Android simulator and got following error:

try to add it manually, not with react-native link

I had the same error and I had to manually link it

From the looks of the error message this has nothing to do with the library itself. And a lot of updates of RN have passed in the meantime, so I'm going to close this issue.
If you still encounter problems, feel free to open a new issue.


Added a cop to enforce assert_empty over assert(actual.empty?)

PR adds cops to enforce the use of assert_empty over assert(actual.empty?)
Ref: https://github.com/rubocop-hq/minitest-style-guide#assert-empty

Can you add the changelog entry and resolve conflicting files?

Yep. I'll do it in today.

@koic : Done. Please have a look.

Is the changelog entry forgotten to be committed?

I missed to add it. I'll add it now. Sorry about that.
On Wed, Sep 11, 2019 at 5:44 AM Koichi ITO<EMAIL_ADDRESS>wrote:

Is the changelog entry forgotten to be committed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rubocop-hq/rubocop-minitest/pull/10?email_source=notifications&email_token=AB3AXKFNXDMZLZYQ7P2XVKDQJAZ6HA5CNFSM4ITUXID2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6M3MTI#issuecomment-530167373,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB3AXKDYWH5CXVTOA7SBOJ3QJAZ6HANCNFSM4ITUXIDQ
.


New postgres example

This example employs latest features in !Persistent.
Changes:

Sets fixed user id 200 for postgres, because volume usually outlives a container
Now uses persistent volume instead of tmpfs
Cleaner initialization command that uses user-id instead of su (also works on all host systems)
Adds example of how to prefill the database with data
Removes useless environment variables

/cc @anti-social, @popravich, @Orhideous, @brabadu

Looks much better afterall :+1:


Gracefully handle "task --list" when there are no tasks

Description
This PR adds graceful handling of the task --list command under two circumstances:
One: There is no "tasks" section in pyproject.toml:

Two: The "tasks" section is empty:

This also fixes issue #51
Changes

Created two new test cases under ListTasksTestCase for each edge case
Created a new exception class: EmptyTasksSectionError
Updated HelpFormatter to raise the aforementioned exception in case the tasks list is empty
Renamed HelpFormatter into TasksListFormatter to better emphasize its role in the system (also renamed file from help.py to list.py)

Closes #51


Broadcast event to the rootscope

Emit the event "popup.shown" to the root scope. It can be tapped into.

Would you be able to rebase and clean up the conflicts?

Greetings @peterchibunna!
Thank you very much from all of us on the Ionic team for submitting this pull request. We appreciate your commitment to improving Ionic. Unfortunately, we are unable to merge this particular PR, as it may cause regressions or is already out of date with regard to recent fixes. We apologize, as we know how much time you may have put into the work.
We hope you will continue to contribute to Ionic in the future. Please note that we are more likely to merge a smaller PR that has a passing test and changes only one meaningful feature in the project.
For more information, please review our Pull Request Guidelines.
Best regards,
Ionitron

Why won't it be out of date when it takes you more than 6 months to review a pull request.


How does it perform against nginx?

Is this question because of curiosity or do you really want to replace nginx with this minimal server in high traffic scenarios?

I've been using nginx for serving of static and php content for many many years without thinking about it. When I came across your repo it struck me that since I also put everything in a container now adays it might be time to see if there is something better suited than nginx... Hence the question. Thanks for the link, I'll google a bit and probably continue to use nginx for the time being.


Graphics Rework

This PR significantly overhauls 32Blit's graphics underpinnings, notably:

Change the hi-res buffer from RGB565 to RGB888
Swap RGBA type for the generic Pen which creates a Pen compatible with RGB888 and Mask surface types

Things to note:

The FRAMEBUFFER now occupies 225k in RAM_D2. This is large enough to fit the 320x240 RGB888 and 160x120 RGB888 display back buffers.

This PR also drags a few quality-of-life changes kicking and screaming along with it. Chiefly some build settings and minor code tweaks to silence or remove some of the warning spam that builds are prone to.

Has duplicate commits and is a nightmare to rebase. RIP!


[Bug]:

Bug description
Steps to reproduce the problem:

I am using puppeteer to render a pdf  file from html and send as an attachment to email. The text in the pdf is grey and too small. I have used  html {
-webkit-print-color-adjust: exact !important;
} as an inline style but didn't worked

Puppeteer version
7.0.0
Node.js version
16.15.0
npm version
4.0.1
What operating system are you seeing the problem on?
Windows
Relevant log output
No response

Please try using the latest version of Puppeteer (and the latest Chromium) as version 7.0.0 is deprecated. I am not sure if it will work in the newer version but there is a chance. Please also include a complete reproduction (e.g., HTML/CSS + Puppeteer script).

Please try using the latest version of Puppeteer (and the latest Chromium) as version 7.0.0 is deprecated. I am not sure if it will work in the newer version but there is a chance. Please also include a complete reproduction (e.g., HTML/CSS + Puppeteer script).

Thank you for your quick response. I have updated to the latest version and it worked.


Preserve yaml formatting

Opening this PR to discuss the implementation of https://github.com/Dashlane/dbt-invoke/issues/10
I've started to implement a --preserve-yaml-formatting flag, very similar to ideas formulated in  https://github.com/Dashlane/dbt-invoke/issues/10.
The basic functionality is there but the PR is not complete yet as I'd like to discuss the general approach first. The default behavior remains untouched, ruamel.yaml is used to preserve yaml formatting if desired. This also holds for comments.
Open questions:

Should we implement the usage of both yaml parsers (PyYAML for the default mode and ruamel.yaml to preserve formatting? I feel like one should try to use the most common parser as often as possible.
How to properly implement tests? I haven't found a nice way to implement a test for that without really just testing ruamel.yaml. Any ideas?
To me, this feels like a "never or always" decision. Is there a way to add this as a configuration so that one would not have to pass the flag every time one runs dbt-invoke. I could also imagine consequences could be quite annoying if users simply forget to put the flag. What are the options here?

environment variables?
Config file (is this possible out of the box through pyinvoke?)
Changing default behavior? (not keeping the format on unformatted yaml files would be less cumbersome that loosing formatting on formatted yaml files)

ToDos

[x] Update README
[ ] bump _version.py
[x] Implement additional tests?

Thank you for taking on this topic @ciklista. I plan on looking into your list of open questions over the next few days and will report back with my thoughts.
I do have one initial thought regarding your third point: It seems like a good idea to change the default behavior and preserve yaml formatting by default.  I agree that it is worse to lose formatting in a yaml file that is intended to be formatted than it is to keep formatting in a yaml file that is intended to be unformatted.  I imagine it would be be more difficult for a dbt-invoke user to undo the former than it would be for them to fix the latter.

I'm leaning towards using only ruamel.yaml (and removing PyYAML completely).  I haven't yet encountered a compelling reason to support both (hopefully I'm not overlooking something).

As for implementing tests, we can add a pre-existing property file with comments and custom formatting for one of the test models (tests/test_dbt_project/models/marts/core/customers.yml for example)

The property file can be missing columns to simulate a case in which a dbt-invoke user has added columns to a model and wants to update its corresponding property file.
Since dbt-invoke will attempt to add information to the property file for the model's newly created columns, we can test to ensure that in the process of adding information dbt-invoke doesn't have any unintended effects on the comments or formatting (by checking that the result is identical to another yaml file we create with the expected output).
We can also do a similar test in which we simulate removing information from a property file as a result of columns being removed from a model.

Lastly, I still feel the same as my initial reaction to your third point; preserving formatting should be the default.

@ciklista thanks again and I'm curious to hear your thoughts!  I apologize for the delay in my response.

yes, I think you are right. Just using ruamel should be a lot cleaner and simpler.
I agree on the default behavior.
concerning tests: yes, I agree that testing two files instead of the parsed dicts would be the way to go to test the formatting.

I won’t be available for the rest of the week but I’ll try to take a look at it beginning of next week.

I've looked a little further into ruamel and I think we don't even need to have multiple cases. From what I can see so far, we might as well just use the rt mode of ruamel all the time. I've played around with the other modes (e.g. 'safe'), but they format yaml files pretty roughly. I guess these make most sense when one really wants to control the formatting.
For more info on the ruamel modes, see: https://stackoverflow.com/a/51318354
@robastel you mentioned that you are using the formatting functionality from standard pyyaml to ensure that property files follow a basic format. What does that include? Does this only concern line breaks? If so, that should work with ruamel too, see test test_long_string()

I've added a couple of tests to check the behavior of different yaml multiline formats. Turns out yaml is a nightmare in that regard. For more, see also here: https://stackoverflow.com/a/21699210/15202709
I've only covered multiline tests and comment tests so far. Is there more we should test for?

That makes sense to me about using typ="rt" all the time.
The formatting functionality I was using from pyyaml was to break long lines and to have consistency in the amount of indentation. I didn't originally realize that ruamel rt behaves the same way in those regards.  I'm happy to learn that is the case.
In terms of other tests, I think it would be a good idea to try to cover the cases originally reported in https://github.com/Dashlane/dbt-invoke/issues/10#issue-991196817

Maintaining empty lines:- name: X

- name: Y

Maintaining bracketed lists, rather than converting to the hyphen form:values: ["A", "B"]

The above two cases appear to work smoothly out of the box with ruamel rt.
One other case that was mentioned may be more difficult:

Maintaining custom indentation level:    - name: X
    - name: Y

It seems we could allow the user to specify a custom indentation scheme like mentioned here: https://yaml.readthedocs.io/en/latest/detail.html#indentation-of-block-sequences

But I'm okay with not including that as part of this PR if it's too much. I haven't looked into what would be the best way to go about it yet.

Added requested changes as well as tests for the two cases mentioned in https://github.com/Dashlane/dbt-invoke/issues/10
I'll open issues for the following so that they can be looked at in future PRs:

Remove empty lines if desired. This thread might help: https://stackoverflow.com/questions/42708668/removing-all-blank-lines-but-not-comments-in-ruamel-yaml
Set indentation if desired. See https://stackoverflow.com/a/44389139


3d trace types - feature parity

General

Menu order: Traces, Layout, Axes, Legend, Color Bars, Notes, Shapes, Images, Sliders, Menus
Place Violin next to Box

3D scatter
StyleTrace

can’t select data source for text
on hover - ‘text’ missing
Line type broken when multiple traces - can’t get color

Layout

aspect ratio missing - manual mode broken in cloud sometimes - setting xyz brings back to auto

Axes

Empty sections: Layout

Color Bars

Tick color -> Color
Borders - Background Color broken - also broken in cloud - plotly returns rgba instead of rgb, when you change it to rgb in json, ColorPicker works  "rgba(245, 117, 117, 0)"

Images

Relative position to grid missing

3D Line
StyleTrace

rename Line Color -> Color

Axes

Empty sections: Layout

3D Surface
StyleTrace

Colorscale Range missing - auto/custom, min, max

Layout

aspect ratio missing

Color Bars

rename Vertical padding -> Padding, same for Horizontal

3D Mesh
StyleTrace

Show Contours - should hide color and width when “hide”

Layout

aspect ratio missing
Empty sections: Layout

Color Bars

Horizontal Positioning - rename horizontal position -> position

Ternary

hover actions - replace flaglist with dorpdown, since you can't deselect both and it behaves funny

Closes #516
Addresses #606
Addresses #524

Notes:

gl3d_bunny-hull.json mock crashes Style/Traces panel
(actually many of the gl3d mocks crash that panel)
The aspectratio stuff only applies to the first scene in this case (i.e. it will be weird if there are two scenes), so it'll have to move to the new Subplots pane... I'm tempted to remove it in this PR and add it in when we do subplots. IIRC there was some stuff like this when we last looked at polar as well, so when we do the subplots panel we'll have to sweep through and find all the places where we mapped an attr to polar.* or geo.* and move those to the applicable subplot fold I guess :)

💃 ! I think this also closes #520 and #606


Any easy way to verify secp256r1 (NIST P256) ECDSA signature?

The custom curve params for the NIST curve does not seem to work. Anyway/any work around as to how a P256 ECDSA signature?
Using curve params as:
const secp256r1Params: CurveParams = {
   name: 'secp256r1',
   modulus: 0xFFFFFFFF00000001000000000000000000000000FFFFFFFFFFFFFFFFFFFFFFFFn,
   order: 0xFFFFFFFF00000000FFFFFFFFFFFFFFFFBCE6FAADA7179E84F3B9CAC2FC632551n,
   a: -3n,
   b: 0x5AC635D8AA3A93E7B3EBBD55769886BC651D06B0CC53B0F63BCE3C3E27D2604Bn,
   generator: {
     x: 0x6b17d1f2e12c4247f8bce6e563a440f277037d812deb33a0f4a13945d898c296n,
     y: 0x4fe342e2fe1a7f9b8ee7eb4a7c0f9e162bce33576b315ececbb6406837bf51f5n,
   },
};```

Hey! Unfortunately, we currently only support foreign curves with curve parameter a !== 0, and here a = -3
https://github.com/o1-labs/o1js-bindings/blob/3fca906694f102dc60bf64745324de234cd6812d/crypto/elliptic-curve.ts#L531-L532

Thank you for the reply, I would see I'd see for myself if I could do something about this.
Also another completely unrelated issue/doubt, is there an easy way to pass a composite struct having string data inside to a zkProgram?
Example:
'''class P256_Signature_Field extends Struct({
sig: String,
msg: String
}){
constructor(value:{sig:string, msg:string}){
super(value)
}
}'''
This inherently doesn't give any errors, but the values of sig and msg are not accessible inside prove-able code (inside the methods of zkProgram) i.e I get no value from these fields of sig and msg.

Also another completely unrelated issue/doubt, is there an easy way to pass a composite struct having string data inside to a zkProgram?

You can't work with strings inside provable code, you would have to convert your strings into Field elements using an encoding

I think this was a recent feature request and should remain open


Playground: make Navbar header and footer sticky on scroll

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


Settings Subsystem: Settings have to be manually added to the project settings

The problem
Not sure how this happened, but right now it's not possible to simply use a setting like text_speed in a fresh project. It won't save (or more correctly it won't load). The only fix is to manually add the text_speed setting to the project settings. This is not documented and not the intuitive approach that we originally intended. So this should be fixed, probably by having dialogic automatically  create these settings when enabled.

I just encountered this bug too.
The setting is correctly saved to global_info.txt but when loading, the settings subsystem iterates through all project settings and then overwrites the value with one loaded from global_info.txt. This is also very inefficient because the save subsystem loads and parses the file once per setting.
Additionally the settings subsystem reloads all settings on every clear() which won't have any effect after this bug is fixed as long as the save subsystem exists. I thought the settings subsytem is meant for global configuration? If so, then it should never be cleared/reloaded.


Stop staking bug

Got this bug when calling the api method StopStaking

I wonder if its related to a double stake output

PR 1392 does not fix this as I was unable to figure out how to reproduce this. In examining the code the dispose on the staking should have waited on the staking task to complete before setting the cancellation token to null so not sure how it got into the above state where it was executing a staking task with a null cancellation token.

Mine is also reproducible: https://github.com/stratisproject/StratisBitcoinFullNode/issues/1430


Grafana graphs brake when selecting "All" servers

@Rurouni any ideas on this, re "All servers"? https://al-hardy.blog/2017/04/28/asp-net-core-monitoring-with-influxdb-grafana/#comment-15

Or if it's not confidential if you could share the configuration of your dashboard, that would also be awesome.
Thanks,
Mark

Btw. I managed to set up the aggregation nicely for all the graphs, but none of the tables. I experimented quite a lot, but couldn't get aggregation working there at all, I always end up seeing all the single metric values in a long list.
Is it possible that the Table Panel works differently with Prometheus than with InfluxDB?

You guys are ahead of me on this one. We have very simple proof of concept dashboards atm and I am doing avg/sum across all tags with filter only by service name so I don't need All/multi-select.
So I am quite interested on what works best as well...
From reading docs: http://docs.grafana.org/features/datasources/prometheus/

When the Multi-value or Include all value options are enabled, Grafana converts the labels from plain text to a regex compatible string. Which means you have to use =~ instead of =.

so ~ is the way to go. and I will probably would just have  a single dashboard with avg/sum with multiselect/all because seeing pods metrics side by side is rarely needed and it doesn't scale to xxx number of pods....

@Rurouni
I see, then I guess we are on the same page.
One minor change: I also had to change the metric of the response time from millisecond to second.
One thing bothering me is that occasionally I get a huge spike in the throughput graph that I cannot explain, for example my endpoints have a throughput of ~10-20 rpm, and occasionally the graph shows a spike of ~500 rpm like this:

And if I check the logs of my api, I cannot see a spike in the number of my requests anywhere, only in Grafana.
Have you encountered anything like this?

@markvincze, I didn't realise you were referring to having a per server metric on the endpoint monitoring :) Could you PR your Grafana dashboard and I'll upload the changes to Grafana. Here's the dashboard in the repo.
I've checked a few of my production api's for random spikes in per endpoint throughput, I don't see anything like that. However I am using the InfluxDB reporter which is just plotting the 1 min rate calculated by App Metrics which is slightly different to how the Prometheus reporter is working.

Do you see the same thing on the overall throughput of your app?

@alhardy Sure, I'll issue a PR. I removed all the Table Panels from my dashboard because they weren't working at all, do you want me to include that in the PR, or leave those untouched?
Yep, the spikes are consistently there on the per endpoint breakdown, and on the overall throughput.

@markvincze Cool, yeah leave the table panels untouched and I'll have a look.
Interesting re spikes, I'll run some tests on my local setup too see if I can replicate.

@markvincze No, I haven't seen such spikes, but then I am not running it in any realistic environment yet.
From the look of it it may be a GC spike that compacted many requests to be deleayed and processed at once, but who knows... I would first try to isolate if it's problem in source data or AppMetrics or Prometheus itself. If you have logs of all requests in ElasticSearch you can at least check if it the first one. Then write simple polling C# program that reads /metrics endpoint into a file so see if it's aggregated data from AppMetrics that's wrong, if all of this is correct than it leaves you with ether Pormetheus QL, or Grafana :)

@markvincze Did you end up doing anymore investigation here?

Hi @alhardy, no, unfortunately I didn't have any time to look into this yet.

OK will leave the issue open for now

Can't replicate


added some more CP programs

Description
I have added some of the latest competitive programming problems and solution from contests which I have taken part and found interesting.
Fixes # (issue)
Type of change

Please delete options that are not relevant.

[X ] New feature (non-breaking change which adds functionality)

Checks

[X] Solution file

[X] README.md

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works

Sorry, I am trying but it's not getting removed.

Sorry, I am trying but it's not getting removed.

Okay no worries I will try it myself, for now merging

Thank you soo much!


Trying to learn options greeks from your code.

https://github.com/wateryan/greeks/blob/fada39485b2dbd21b8de2edf38ef794dda4a0ab0/src/common.rs#L3
What exactly is d1/d2 trying to do here?

I think it is explained in this paper - it seems to match up. http://www.ltnielsen.com/wp-content/uploads/Understanding.pdf


Add company address to footer

Link to Ticket
Description of Changes Made
The footer now includes the company address, including the registered company number and VAT number, as hardcoded values.
How to Test
Test by loading the any page of the website and verifying the information can be seen in the footer element below the logos.
Screenshots

Desktop

Mobile

MR Checklist

[X] Add a description of your pull request and instructions for the reviewer to verify your work.
[X] If your pull request is for a specific ticket, link to it in the description.
[X] Stay on point and keep it small so the merge request can be easily reviewed.
[X] Tests and linting passes.

Unit tests

[ ] Added
[X] Not required

Documentation

[ ] Updated build docs
[ ] Updated editor guidelines (See https://intranet.torchbox.com/torchbox-com-project-docs for the private link, for Torchbox employees only)
[ ] Updated field spec (See https://intranet.torchbox.com/torchbox-com-project-docs for the private link, for Torchbox employees only)
[X] Not required

Browser testing

[X] I have tested in the following browsers and environments (edit the list as required)

Latest version of Chrome on Ubuntu
Latest version of Firefox on Ubuntu

[ ] Not required

Data protection

[X] Not relevant
[ ] This adds new sources of PII and documents it and modifies Birdbath processors accordingly

Light and dark mode

[X] I have tested the changes in both light and dark mode
[ ] The change is not relevant to dark and light mode

Accessibility

[ ] Automated WCAG 2.1 tests pass
[ ] HTML validation passes
[ ] Manual WCAG 2.1 tests completed
[ ] I have tested in a screen reader
[ ] I have tested in high-contrast mode
[ ] Any animations removed for prefers-reduced-motion
[X] Not required

Sustainability

[ ] Images are optimised and lazy-loading used where appropriate
[ ] SVGs have been optimised
[ ] Perfomance and transfer of data considered
[ ] If JavaScript is needed alternatives have been considered
[X] Not required

Pattern library

[ ] The pattern library component for this template displays correctly, and does not break parent templates
[ ] The styleguide is updated if relevant
[X] Changes are not relevant the pattern library

Hi @ianmeigh, I tested this locally and the layout of the footer now looks quite broken, with the carbon message right over to the right. I can also see there is a pipeline failure (although that looks like it is a temporary blip).
Could you look at the layout again, and then I'll do a full review?

:point_up:  Fixed in 804ec6379ee00d1a73f3dfa11f0fc05e14ec31ea.


ASTNodeTraverser.h contains spurious inlay hints

Steps to reproduce

In the LLVM codebase, open clang/include/clang/AST/ASTNodeTraverser.h
Observe that the file contains a number of spurious parameter and type hints near the top

I suspect the spurious hints are related to this include of AttrNodeTraverse.inc near the bottom of the file. The hints look like they belong to the code in that .inc file, but incorrectly surfaced in ASTNodeTraverser.h.

Here's a minimal testcase.
foo.hpp:
struct S {
  void foo(int param);
  #include "foo.inc"
};

foo.inc:
  void bar() {
    foo(42);
  }

The include being at non-global scope is important for some reason; if I move it to global scope (and adjust the included file contents accordingly), i do not get a spurious hint.


ClosedXML can't read and show The specified package is invalid. The main part is missing.

OpenXmlSDK create xlsx ClosedXml can't read, but NPOI,Epplus can read, and show message The specified package is invalid. The main part is missing.

ClosedXml Code :
void Main()
{
	var path = @"C:\Users\Wei\AppData\Local\Temp\099e17f7-a98b-42d5-8b75-4e6775914f9c.xlsx";
	using (var workbook = new XLWorkbook(path))
	{
		var worksheet = workbook.Worksheets.First();
		var cells = worksheet.Cells();
	}
}

Message : The specified package is invalid. The main part is missing.
Source : DocumentFormat.OpenXml
StackTrace :
   at DocumentFormat.OpenXml.Packaging.OpenXmlPackage.Load()
   at DocumentFormat.OpenXml.Packaging.OpenXmlPackage.OpenCore(String path, Boolean readWriteMode)
   at DocumentFormat.OpenXml.Packaging.SpreadsheetDocument.Open(String path, Boolean isEditable, OpenSettings openSettings)
   at DocumentFormat.OpenXml.Packaging.SpreadsheetDocument.Open(String path, Boolean isEditable)
   at ClosedXML.Excel.XLWorkbook.LoadSheets(String fileName) in C:\projects\closedxml\ClosedXML\Excel\XLWorkbook_Load.cs:line 42
   at ClosedXML.Excel.XLWorkbook.Load(String file) in C:\projects\closedxml\ClosedXML\Excel\XLWorkbook_Load.cs:line 33
   at ClosedXML.Excel.XLWorkbook..ctor(String file, XLEventTracking eventTracking) in C:\projects\closedxml\ClosedXML\Excel\XLWorkbook.cs:line 718
   at ClosedXML.Excel.XLWorkbook..ctor(String file) in C:\projects\closedxml\ClosedXML\Excel\XLWorkbook.cs:line 709
   at UserQuery.Main() in C:\Users\Wei\AppData\Local\Temp\LINQPad6\_zeygkceo\zmngrx\LINQPadQuery:line 5
   at LINQPad.ExecutionModel.ClrQueryRunner.Run()
   at LINQPad.ExecutionModel.Server.RunQuery(QueryRunner runner)

I've tried unzip xlsx file like below:
Folder Tree and link :  https://github.com/shps951023/MyFiles/tree/master/2021-03-02/099e17f7-a98b-42d5-8b75-4e6775914f9c
│  [Content_Types].xml
│
├─xl
│  │  sharedStrings.xml
│  │  styles.xml
│  │  workbook.xml
│  │
│  ├─worksheets
│  │      sheet1.xml
│  │
│  └─_rels
│          workbook.xml.rels
│
└─_rels
        .rels

Xlsx link :  https://github.com/shps951023/MyFiles/blob/master/2021-03-02/099e17f7-a98b-42d5-8b75-4e6775914f9c.xlsx

The key point is it have to have CompressionOption and ContentType and Uri and Package information.


Identity Server 4 is generating long values for"nonce" + "state" parameter

I work for the Azure Identity Support Team and I am currently working with a customer where we are encountering the following issue:
Customer registered a custom application under his Azure AD.  When users access their application, using the app URL, it opens the Microsoft log-in screen, as expected. If one user tries to authenticate with wrong password it gives the option to reset the password, which is expected as well. When the user hits 'reset it now' button, he gets a Page not found error message. This password reset action generates a long URL that exceeds the minimum length of characters, reason why the page cannot be found.
On the other hand, if a user does not use the application URL to reset his password, but for example azure.portal.com, he hits the 'reset it now' button and it works as expected. He is redirected to the SSPR page with no errors being prompt.
For the non-working scenario, what we understood so far was that this is due to the maximum URL length limit that is being reached. The CTX parameter from https://login.microsoftonline.com/ is longer than expected. Also, this happens since the affected App is sending a "nonce" parameter + "state" parameter with too long values. This can be seen when capturing a Fiddler Trace.
That being said, our suggestion was for the customer to reduce the size of the values (nonce and state) that are being emitted by the app: https://docs.microsoft.com/en-us/azure/active-directory/develop/v1-protocols-openid-connect-code
Nevertheless, customer informed me that they are using the open source Identity Server 4, which is generating those 2 parameters.
Our question now is: I there any way to reduce the size of ‘nonce’ and ‘state’ parameter?

Our question now is: I there any way to reduce the size of ‘nonce’ and ‘state’ parameter?

This is not an IdentityServer issue. It's a problem with state management in how ASP.NET Core's OIDC plumbing works combined with a client application that needs to use upstream token servers with URL limits (most commonly with multiple federation hops). We do have a solution, though. From the docs: http://docs.identityserver.io/en/latest/topics/signin_external_providers.html#state-url-length-and-isecuredataformat
My question now is: AAD increased its URL limit. Why can't live?

Any update?

Any update on the issue? closing for now - feel free to re-open if it needs further discussion.


Dependency Overrides in startup event gives TypeError: 'Depends' object is not callable

How can dependencies in the startup event be overwritten for testing?
Below a minimal example. When executing the test, Model is a Depends object and not a MagicMock. When putting the depends in normal endpoints it works just fine.
api.py
from my_package.model import Model
from fastapi import Depends, FastAPI
app = FastAPI()

def model_provider():
    return Model

model = None

@app.on_event("startup")
def startup(Model = Depends(model_provider)):
    global model
    model = Model()

@app.get("/prediction")
def predict(x):
    return model.predict(x)

tests/test_api.py
from my_package import api
from unittest.mock import MagicMock
from fastapi.testclient import TestClient

def test_predict():
    model_mock = MagicMock()
    model_mock.predict = MagicMock(return_value=lambda x: x)
    api.app.dependency_overrides[api.model_provider] = lambda: model_mock
    with TestClient(api.app) as client:
        prediction = client.get("/prediction?x=1")
        assert prediction.text == "1"

```

The documents seems to hint that you can only use Depends for request functions. Perhaps this could be extended to app events as well?

+1 this would be great to use dependencies in event handlers. Now it is Starlette events which know nothing about FastAPI DI

+1, the DI is a very important design feature of FastAPI that should be usable in event handler IMO.
Unless already planned for future release, I would like to contribute to this, but this doesn't seems that easy to implement it in a clean way. By reading DI implementation currently used, I realize that it is deeply connected to request / response game, with no separation of concern allowing to just refactor fastapi.dependencies.utils.solve_dependencies function to handle an event only use case.
A quick intuition is that it should have a more granular Dependent / Depends hierarchy like having a base Dependent definition without any Body, Form, Request, Response, etc ... support and adding those to a child RequestDependent before to finally break solver function to support either both use case or smartly trigger errors when unsupported request deps are required in an "non request" context.
But maybe there is a simpler way ?

@Faylixe I took a look, and I agree with your assessment. It looks doable without breaking changes, but yeah a lot of refactoring would be needed.


Feature Request: Reduce providers required for bootstrap

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Is your feature request related to a problem?

The target bootstrap subscription will have all providers installed by the azurerm provider if it is the very first Terraform run. This can cause a delay if there is a regional outage in Azure.
Describe the solution you'd like

Explicitly define the the required providers for the bootstrap subscription and only install those. Turn off the default behviour of the provider.

Additional context

v4 of the azurerm provider has new capabilities that could be leveraged for this.


docs: update support page to include known limitations

https://keptn.sh/docs/1.0.x/install/k8s-support/
should be updated up to v1 and should mention the fact that we use bitnami MongoDB so this won't work on arm based nodes (e.g., on a Mac M1) due to this https://github.com/bitnami/charts/issues/7305

Hey, @RealAnna, I would like to work on this issue. Can you assign this to me?

Hey @bhavyastar thanks for volunteering! I am assigning the issue to you :)

Hey @thisthat, Thanks for assigning this to me. I am working on this and I also understand it but I just want to confirm whether what I am thinking is the right one or not!!

There are separate tables for versions 0.17, 0.18 and so on. So, I have to make a table with respect to version 1.0 and mention all the support from 0.17,01.18 and so on.

Hey @bhavyastar, the goal of this ticket should be changing this page https://keptn.sh/docs/1.0.x/install/k8s-support/ with the following:

replace 0.17.x with 1.x.y
add a disclaimer that we only support amd64 architecture. Arm-based nodes, e.g., a kind cluster on a mac book with the M1 processor, are not supported due to our dependency on the Bitnami Mongo Charts.

I hope this clarifies your questions :)

I can work on this if it is still open?

Hey @shayyxi, It has already been done and merged.

@RealAnna, @thisthat or @bhavyastar please close this issue if it is done. Thanks.

@bhavyastar If merged, please close this issue
Thanks 👍🏻 )

Since Keptn V1 has reached EOL on December 22nd, 2023, this issue has been closed.


Out of date server

I can't get it to work because the FTB Presents Direwolf20 1.12 is saying my server is out of dat.
I tried editing the server version but it can't find the version: MV 1.12.2 and MPV 2.3.0.
Can you help me?

@theboy30 This is probably due to some mismatching versions in the mod-pack. Make sure you use the same Direwolf20 client version as the server version. Sorrily I have not been able to update this for a year. But the latest version of direwolf server is 1.10.2_1.13.0. You can see the versions here: https://hub.docker.com/r/feedthebeast/direwolf20/tags/


renderFailure on subroute

I renderFailure never gets called on a subroute. Say i have this:
<Route path="/" queries={rootqueries}>
  <Route path="/test" queries={testqueries} renderFailure={err =>
Error
} />
</Route>

if testqueries fails with this response:
{
  data: { testdata: null },
  errors: [{ message: 'Test error', locations: [{ line: 1, column: 14}] }]
}

it thinks data fetched ok and tries to render normally.

Failures aren't localized to routes – so it's actually failing to render the parent in your case, if this is on a fresh page load.

@taion  will failures to be localized in future releases ?

Maybe? Who knows.

In case anyone else is unclear about what this means, @taion mentioned in Reactiflux#relay

global 404ing in react router based on missing data is more or less impossible

So the way I'm dealing with this problem is to delegate missing data handling to the container. You could use a lifecycle method to check if data is missing and return a different component or re-route to a different component, such as a 404 page. If your container is a stateless component, you can do something like:
const ProjectContainer = (props, context) => (!props.project
  ? <NotFoundPage message={'Project not found.'} />
  : <TransitionWrapper>

      ...


Exception message handling in ZCL_ABAPGIT_OBJECTS_PROGRAM=>DESERIALIZE_DYNPROS

While importing a dynpro with abapGit I realized the problem that i do not get the real error message, but "Error from RPY_DYNPRO_INSERT: 6".
Debugging the issue got me to the error e166 in SAPLSIFP FORM I_DDIC_IN.
The SAP function 'RPY_DYNPRO_INSERT' that is called in the given method throws the correct error, but the error detail gets lost in:
        zcx_abapgit_exception=>raise( |Error from RPY_DYNPRO_INSERT: { sy-subrc }| ).
because
    CALL METHOD ('ZCL_ABAPGIT_MESSAGE_HELPER')=>set_msg_vars_for_clike
      EXPORTING
        iv_text = lv_text.

sets a new sy even this is not neccessary if we have a sy-msgid and sy-msgno from the called sap coding.
I would suggest to change ZCX_ABAPGIT_EXCEPTION in a way that error messages from sap coding (with a sy-msgid and sy-msgno) will not overwritten or to use
zcx_abapgit_exception=>raise_t100( ).
instead of
zcx_abapgit_exception=>raise( |Error from RPY_DYNPRO_INSERT: { sy-subrc }| ).
in
ZCL_ABAPGIT_OBJECTS_PROGRAM=>DESERIALIZE_DYNPROS
after
CALL FUNCTION 'RPY_DYNPRO_INSERT'

yea, use raise_t100

fixed in https://github.com/larshp/abapGit/pull/2941, closing


feat: kubernetes chaos monkey chart

https://github.com/asobti/kube-monkey

add this item for tracking in documentation as an add-on.

i don't see this having a place in this repository. i'm going to close this and add the issue to openstack-helm-addons.


It would be nice to support multiple Prometheus registries

At the moment because the default registry is used this is not possible. For example if you have an application with two Gin servers the second gin-prometheus handler will fail with an error like:
time="2021-03-09T17:43:51Z" level=error msg="requests_total could not be registered in Prometheus" error="duplicate metrics collector registration attempted"
time="2021-03-09T17:43:51Z" level=error msg="request_duration_seconds could not be registered in Prometheus" error="duplicate metrics collector registration attempted"
time="2021-03-09T17:43:51Z" level=error msg="response_size_bytes could not be registered in Prometheus" error="duplicate metrics collector registration attempted"
time="2021-03-09T17:43:51Z" level=error msg="request_size_bytes could not be registered in Prometheus" error="duplicate metrics collector registration attempted"

Cheers!

#50
I have same issue and made a pr for it.
Maybe it can help you :)


enable local volumes

In preparation for 4.1. Lets re-renable local storage
cc @wongma7 @deads2k

/retest

/lgtm


missing dependency of nebula_sensor_driver

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
nebula_sensor_driver is a dependency, but rosdep can't find it.
Expected behavior
When running rosdep install -y --from-paths src --ignore-src --rosdistro $ROS_DISTRO, rosdep can install all dependencies
Actual behavior

ERROR: the following packages/stacks could not have their rosdep keys resolved
to system dependencies:
common_sensor_launch: Cannot locate rosdep definition for [nebula_sensor_driver]

Steps to reproduce
rosdep install -y --from-paths src --ignore-src --rosdistro $ROS_DISTRO
Versions
ubuntu 22
humble
autoware git main branch
Possible causes
No response
Additional context
No response

@felixf4xu
I guess you already solved this issue long time ago but just a comment for clarification:
Nebula is not available on ROS build farm yet, thus it is necessary to build from sources. You can pull repository with VCS tool.


Virtual screen reader interactivity?

Hi there,
Great initiative and great library! I am trying to use virtual-screen-reader with my Vue components and testing-library. Is it possible to have events from JS or interactive elements captured? I've noticed fireEvent from '@testing-library/vue' doesn't change anything and neither does virtual.press('Enter'). Not sure if it's possible but I don't see any examples.
Thanks!

In theory both should work 😅 sounds like have missed a trick somewhere!
Can you provide a minimal repro? (Though admittedly sounds like a hello world would suffice… just not so familiar with Vue these days)


[fix](regression test) fix unstable case auto bucket test

bucket num = min(partition_size/1G,   avaible disk /50 GB + 1)
sometimes, the pipeline test backend's available disk  < 50GB,  then the test will failed.
Considering this case is only to test autobucket_max_buckets,  we set autobucket_max_buckets=1 to torrent with any available disk size.
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Show a helpful message if trying to dispatch when Covalent is not started

Description
If you attempt to dispatch when Covalent is not started, you get a trace with a ConnectionRefusedError: [Errno 61] Connection refused. For an experienced user, it's obvious this is caused by the dispatcher server not running, but for others it might not be so obvious. We should implement more graceful exception handling, so that the user gets a helpful message like "The Covalent dispatcher server is not running. You must start Covalent before dispatching your workflow."
Acceptance Criteria

[x] The LocalDispatcher.dispatch method displays a pretty message describing the error.
[x] The error message appears when a ConnectionRefusedError is raised.
[x] The error message is printed using Covalent's logger, logger.app_log, and is raised at the ERROR log level.
[x] A unit test is created in the corresponding test file, tests/covalent_tests/dispatcher_plugins/local_test.py (this file does not currently exist). Note that unit tests in the Covalent CI are run without the server running, so any attempt to dispatch a workflow within the unit test should return this error message.

Hello, I would like to work on this issue. Could you please assign it to me?

Thanks for participating in Hacktoberfest @rutvikk05!  Looking forward to your contribution :+1:

Thanks @wjcunningham7!
I've started working on this, but not sure how to approach the problem. In which file should I make the changes to change the error message?

@rutvikk05 the file you'll want to take a look at is covalent/_dispatcher_plugins/local.py.  That will include the classes and methods referenced in the acceptance criteria.

Windows is not supported. Here is the list of supported platforms.

hey @Aravind-psiog, I no longer have responsibility for issue assignment, @wjcunningham7 can assign the issue

PR created @scottwn @wjcunningham7 . https://github.com/AgnostiqHQ/covalent/pull/1364. Please let me know if anything has to be done.


Release v2 (frictionless@5 compatible)

Overview
To follow - https://github.com/frictionlessdata/frictionless-py/issues/1120

DONE https://github.com/marketplace/actions/frictionless-repository


Revive gencord

Revive time?

Yep I plan on working on it more later.

:bowtie:

Gencord is dead. It's official!
:bowtie: :neckbeard: :shipit: :octocat: :atom: :electron:

it was dead and now its being revived kind of

no i like my message

pls contribute :D

no u

could restart if we wanted, but think we are too far in :(

may as well rewrite it

no

wait rewrite to deno xd

No

thats a good idea tbh

maybe but we're already far in

idk

I meannn true but

We might as well restart, the whole project is a mess.

idk tho

We might as well restart, the whole project is a mess.

hahahhaaha @eternalmoon1234 someone agress with me HAHAHAHAH

so uh nuke the repo? or

i mean what are we going to do differently thats already here

Let me know. I think its best if we restart now, and make a better lay out of what should be done.

Idk, but I feel like we honestly don't know what we are doing lol, we should just start over and have a better plan but its up to you lol

i think its cuz your new to the codebase, but yeah maybe we should restart

https://imgur.com/a/AlcvBEY

oof

yeah lets just restart, probably better to do  it anyways. I don't think we need to delete the repo just delete all the files but again up to you

honestly lets just leave it for now, we might as well continue ig

yeah thatll be the better choice


Accessibility and styling fixes

aria-pressed attribute
margin changes

LeanRooster

textbox role
placeholder pointer styling

(going to reuse the last package version)


fix: header not show

General

✏ Mark the necessary items without changing the structure of the PR template.

closes #86

[x] Pull request template structure not broken

Type

ℹ What types of changes does your code introduce?

👉 Put an x in the boxes that apply

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

ℹ Check all checkboxes - this will indicate that you have done everything in accordance with the rules in CONTRIBUTING.

👉 Put an x in the boxes that apply.

[x] My code follows the style guidelines of this project
[x] Is the code format correct
[x] Is the git submission information standard?
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@jinmao88 Please review this Pr.


Diagnostics, changes & fixes to node shutdown etc.

Fix a couple of signal handling bugs
Upgrade node activity tracking to track information on multiple stack frames
Track node activity for life of exchange rather than within each action handler
Add additional synchronization logic to shutdown/factory reset logic
Remove additional stack frames in critical path
Add additional structural detail in SIGUSR2 diagnostic dumps
Add active request information to SIGUSR2 diagnostic dumps
Make conformance errors clearer for missing fields
Add missing references to sibling fields for constraint validation

Nice adjustments, lets see if they already help with @JimBuzbee reported decommission issue


network-transport-zeromq-0.2 test suite failure

The test suite fails with the following error in our CI environment: http://hydra.cryp.to/build/521388/nixlog/1/raw. Do you have any idea how to remedy that issue?

Hi, Peter.
Thanks for the report. In current implementation stopping the thread is very expensive operation.
I'll so the easiest solution would be to reduce number of calls. But I'll definitely look at possibilities
to improve current situation. In latest commits situation is slightly better but still not ideal.

I've pushed a workaround for this problem, but I'll keep bug open unless there will be a better fix.


chore: fix linter

fix govgen import path in gci tool configuration in  the .golangci.yml file
fix import order in respect of that configuration (make format can do that automatically)
fix usage of //nolint: unparam

Not sure if we should target feat/custom-gov-module or main branch, give me your though!

@tbruyelle let's target main, I think it makes sense especially as we are approaching merging also the gov modifications.

Thinking about it, even if we target main', we still need a linter PR for the feature branch, for the gov' module we've added. So let's merge it there.

Thinking about it, even if we target main', we still need a linter PR for the feature branch, for the gov' module we've added. So let's merge it there.

makes a lot of sense!
LGTM


Maybe revert back to github.com/babolivier/go-doh-client?

Overview
We should probably revert to the github.com/babolivier/go-doh-client impementation of DoH used in older versions rather than the current github.com/likexian/doh of DoH
Description
The change in v0.10 to likexian's DoH implementation causes the loss of the ability to switch to different DoH providers. It is now basically hard-coded to use CloudFlare and Google's DNS (which are made from the client itself and not proxied). As such, if those addresses (cloudflare-dns.com and dns.google) is unresolvable, it would cause DoH not to work, exposing the client to classic DNS poisoning.
Additional context
The issue here is that likexian/doh isn't really a DNS-over-HTTPS client: What it does is actually use the JSON API endpoints of some resolvers (eg. dns.google/resolve) to fetch addresses. As such, it is not spec (RFC8484) compliant and some DNS servers (which users might prefer because provided ones are unreachable, which does happen) won't work.

Another option we can do is make our own lightweight resolver. Since we only need to deal with A (and maybe AAAA) addresses we might be able to fit one in

Thank you for your research. greatq job. the reason why i switched to another doh client was the previous one doesn't seem to be maintained anymore. let me think of what other alternatives we can have.

I mean in one way yeah maybe we could bring it up-to-date
On the other hand, the spec (RFC8484) really didn't change (otherwise it will break a lot of things). So it's probably fine for it to be that way


Fix blockcharts by using keys for output_element_series

@michieldenhaan pointed out the blockcharts were broken. I removed all references to ids and exchanged them for keys.

@noracato I'm not sure what this does. It seems to conflict with with master and I don't understand how this mr relates to @antw's last commit. Maybe its best if he reviews this?

Ah I see @antw already fixed this in his last commit, just after I put this PR up :) I'll close this one!


Analysis stuck on "generating application stack report"

For the repository: https://github.com/codecentric/spring-boot-admin
Reported by @ravsa @sunilk747

The challenge here with trigger stack analysis at workspace level as above repo has both pom.xml and package.json which we don't support multiple ecosystems as of now.
To start with an ideal approach would be to check for root level pom, if any consider it as maven project and proceed.
NOTE: User can still trigger stack analysis on particular manifest file be it pom.xml, package.json.
cc @samuzzal-choudhury @miteshvp any inputs/suggestions.

Sure @miteshvp I agree we need to support package.json as well in this scenario we'll discuss it and take it as the enhancement. For time  I feel we can have my approach.

This issue was moved to openshiftio/openshift.io#4251


[BUG] Python version/environment for notebooks

Describe the bug
Please define exact Python requirements for running notebooks.
Both notebooks fails with Azure ML compute instances.
For 'Python 3.10 - SDK v2' environment I get

ModuleNotFoundError                       Traceback (most recent call last)
Cell In[2], line 9
6 from pathlib import Path
7 from zipfile import ZipFile
----> 9 import magic
10 import pandas as pd
11 import requests
ModuleNotFoundError: No module named 'magic'
For 'Python 3.8 - AzureML' env I get error:
TypeError                                 Traceback (most recent call last)
Cell In[8], line 7
1 def upload_files(
2     file_directory: str,
3     storage_name: str,
4     batch_size: int = 100,
5     overwrite: bool = True,
6     max_retries: int = 5,
----> 7 ) -> requests.Response | list[Path]:
8     """
9     Upload files to a blob storage container.
10
(...)
19     (i.e. a few seconds before. The solution "in practice" is to sleep a few seconds and try again.
20     """
21     url = endpoint + "/data"
TypeError: unsupported operand type(s) for |: 'type' and 'types.GenericAlias'

Thanks @mkoivi. For the Python 3.10 environment in AML, did you receive the ModuleNotFound error after running the pip install cell? magic is an interesting package because on some OS's, it requires the installation of a system package (i.e. apt-get install) in order for the python package to work. We use the package primarily as an automated safeguard to identify file encoding issues before users attempt to upload data to the accelerator.
In my limited research, magic seemed like the best package to classify file encoding. If you're aware of a better approach, I'd be happy to swap out magic for something else.

Thanks @mkoivi. For the Python 3.10 environment in AML, did you receive the ModuleNotFound error after running the pip install cell? magic is an interesting package because on some OS's, it requires the installation of a system package (i.e. apt-get install) in order for the python package to work. We use the package primarily as an automated safeguard to identify file encoding issues before users attempt to upload data to the accelerator.
In my limited research, magic seemed like the best package to classify file encoding. If you're aware of a better approach, I'd be happy to swap out magic for something else.

ModuleNotFound error occured when running the second cell, containing the import clauses. I'm not very familiar with Python libraries unfortunately, so I'd like to have as straightforward deployment as possible.


KOGITO-8316: SWF Viewer - Parallel state requires more details

Hello @romartin, @LuboTerifaj,
there is a PR we discussed yesterday. For more info see KOGITO-8316.
Thank you!

Hi @LuboTerifaj,
great catches! Fixed.


[WFCORE-598] [WFLY-4716] Implement the new ordered resource methods i…

…n custom resource implementations
introducing some temporary code to preserve compatibility with core
until released
A core PR is to follow.
This PR overrides new methods introduced by the core PR, while commenting out the @Override annotations so it will work once the core one has been merged, and also compile beforehand.
https://issues.jboss.org/browse/WFLY-4716 is for removing this temp code following a core release.

Windows Build 1656 is now running using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db

Linux with security manager Build 1484 is now running using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db

Linux Build 6580 is now running using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db

Linux with security manager Build 1484 outcome was SUCCESS using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db
Summary: Tests passed: 825, ignored: 319 Build time: 0:06:13

Linux Build 6580 outcome was FAILURE using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db
Summary: Tests failed: 1 (1 new), passed: 2873, ignored: 374, muted: 1 Build time: 0:49:31
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.client.descriptor.EJBClientDescriptorTestCase.testEJBClientContextWithNoReceiversConfiguration: <no details avaliable>

Windows Build 1656 outcome was SUCCESS using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db
Summary: Tests passed: 2875, ignored: 374 Build time: 0:50:27

Retest this please

Linux Build 6582 is now running using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db

Windows Build 1658 is now running using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db

Linux with security manager Build 1486 is now running using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db

Linux with security manager Build 1486 outcome was SUCCESS using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db
Summary: Tests passed: 825, ignored: 319 Build time: 0:06:34

Linux Build 6582 outcome was FAILURE using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db
Summary: Tests failed: 1 (1 new), passed: 2874, ignored: 374 Build time: 0:47:47
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.timerservice.mgmt.PersistentCalendarTimerManagementTestCase.testResourceExistence: <no details avaliable>

Windows Build 1658 outcome was SUCCESS using a merge of d9cdf07ba1efda125c26d0d72b3d17435dd5f1db
Summary: Tests passed: 2875, ignored: 374 Build time: 0:49:45

Linux with security manager Build 1497 is now running using a merge of f366049b445d400ea30a50530f2ce85d60b16422

Windows Build 1669 is now running using a merge of f366049b445d400ea30a50530f2ce85d60b16422

Linux Build 6593 is now running using a merge of f366049b445d400ea30a50530f2ce85d60b16422

Linux with security manager Build 1497 outcome was SUCCESS using a merge of f366049b445d400ea30a50530f2ce85d60b16422
Summary: Tests passed: 828, ignored: 320 Build time: 0:04:54

Linux Build 6593 outcome was SUCCESS using a merge of f366049b445d400ea30a50530f2ce85d60b16422
Summary: Tests passed: 2878, ignored: 375 Build time: 0:47:28

Windows Build 1669 outcome was SUCCESS using a merge of f366049b445d400ea30a50530f2ce85d60b16422
Summary: Tests passed: 2878, ignored: 375 Build time: 0:47:33

Retest this please!

Windows Build 1692 is now running using a merge of b292e8f2a0350e96e6d9a9afde38367057692aec

Linux with security manager Build 1520 is now running using a merge of b292e8f2a0350e96e6d9a9afde38367057692aec

Linux Build 6616 is now running using a merge of b292e8f2a0350e96e6d9a9afde38367057692aec

Linux with security manager Build 1520 outcome was SUCCESS using a merge of b292e8f2a0350e96e6d9a9afde38367057692aec
Summary: Tests passed: 828, ignored: 320 Build time: 0:05:48

Linux Build 6616 outcome was SUCCESS using a merge of b292e8f2a0350e96e6d9a9afde38367057692aec
Summary: Tests passed: 2872, ignored: 375 Build time: 0:48:18

Windows Build 1692 outcome was SUCCESS using a merge of b292e8f2a0350e96e6d9a9afde38367057692aec
Summary: Tests passed: 2872, ignored: 375 Build time: 0:48:50

Linux Build 6626 is now running using a merge of e4c2ce2c226b284eab996f7b5bad85d4a79ea1ff

Windows Build 1703 is now running using a merge of e4c2ce2c226b284eab996f7b5bad85d4a79ea1ff

Linux with security manager Build 1530 is now running using a merge of e4c2ce2c226b284eab996f7b5bad85d4a79ea1ff

Linux with security manager Build 1530 outcome was SUCCESS using a merge of e4c2ce2c226b284eab996f7b5bad85d4a79ea1ff
Summary: Tests passed: 828, ignored: 320 Build time: 0:06:36

Windows Build 1703 outcome was SUCCESS using a merge of e4c2ce2c226b284eab996f7b5bad85d4a79ea1ff
Summary: Tests passed: 2872, ignored: 375 Build time: 0:46:53

Linux Build 6626 outcome was SUCCESS using a merge of e4c2ce2c226b284eab996f7b5bad85d4a79ea1ff
Summary: Tests passed: 2872, ignored: 375 Build time: 0:46:58


Add json encoded values example

Added et rewrote example related to #361.
As I wrote the examples, I noticed that there was already an example as the last line of the Containers section but unfortunately I missed it during my troubleshooting. I tried to make it more explicit for newcomers by creating a new section related to JSON encoded values.

 Sorry, I'll remake this pull request.