use yaml instead of toml for config.toml

What is the problem you're trying to solve
Being able to automate/script changes is very important in todays modern environment, its also nice to have a human readable format that clearly shows any implied hierarchy. There is not a toml package on apt/snap that will script changes to a field in a toml file, there are various tools but they must be installed through golang or pip. The config.toml file is also quite large and hard to read. yaml is much easier to read and work with thanks to mature tooling like yq. Its also likely that consumers of containerd are working with it in k8s which is primarily yaml.
Describe the solution you'd like
Use a .yaml file instead.
Additional context
No response

Maybe I'm misunderstanding something, but mikefarah's yq package (which is installable via snap) supports TOML

not fully.

Also it would be nice to be consistent with k8s.

YAML is not the only one human readable format in the world. The TOML is also a good one. For instance, the rust community is using TOML for cargo manifest. Kubernetes chooses YAML, which doesn't mean that we should follow this rule. We should find a suitable one for us. The containerd has been using TOML for years. I would like to see the real pain point to config containerd with TOML. Thanks

FWIW: While Kubernetes doesn't support TOML, it does generally support other formats (json, proto) in APIs if not config files. They all have pros and cons and I personally don't think "everything will use yaml" was an intention versus "let's promote declarative config", which the containerd TOML already is.
Seeing as contanainerd isn't only used for Kubernetes, supporting multiple serialization formats is complicated, and tooling already configures containerd TOML for years .... I don't think this is worth it.
If you're bootstrapping a Kubernetes distribution from scratch you will also probably need to configure things like systemd drop-ins that are in a custom syntax that doesn't even have standard parsers/serializers AFAIK.
Maybe someone should write a yq/jq for toml :-)

Should mention: https://github.com/containerd/containerd/issues/3289 doesn't work super well at the moment because it clobbers entire fields IIRC (see e.g. https://github.com/containerd/containerd/issues/5837#issuecomment-1422262431), so patching config is a bit of a pain-point, but probably config.d support could be improved, or a yq like tool recommended.
Things like the config_path for registries have helped in the meantime (though again that has been a painful transition because it disable the old inline config fields instead of merging with them ...).

For those running into this problem specifically to avoid a Docker dependency in your Kubernetes (kubeadm) installs something like the following will work on vanilla Ubuntu:
_install_containerd_set_systemdcgroup() {
  sudo mkdir /etc/containerd 2>/dev/null || true
  sudo bash -c "containerd config default > /etc/containerd/config.toml"
   python3 <<'EOM'
import toml
import os

with open("/etc/containerd/config.toml","r") as t:
  conf = toml.load(t)

conf['plugins']['io.containerd.grpc.v1.cri']['containerd']['runtimes']['runc']['options']['SystemdCgroup'] = True

with open(f'/tmp/containerd-config.toml', 'w') as f:
  toml.dump(conf, f)
EOM
  sudo mv /tmp/containerd-config.toml /etc/containerd/config.toml
  sudo systemctl restart containerd
}

I've always hated using sed hacks for this sort of thing as appears in Ansible playbooks a lot. But this isn't pretty, just accurate.

Even though I love TOML (made the logo), I also would like to use YAML for this since updating values inline would become a trivial yq one liner. Would a PR be accepted to autodetect config files with yaml or yml suffixes if they exist?


How can I login from the login page?

I  have IdentityServer 4 setup with SignInManager and UserManager and a custom store in mongo.
I my AccountController.cs I have
   if (ModelState.IsValid)
            {
                var result = await _signInManager.PasswordSignInAsync(model.Username, model.Password, model.RememberLogin, lockoutOnFailure: true);
                if (result.Succeeded)
                {
                    var user = await _userManager.FindByNameAsync(model.Username);

I want to be able to simply go to the loginpage (serverhost/account/login) and type the username and password and login. I am getting the error "sub claims is missing".
I added the ProfileService but this does not get called (from what I see it's being called by the DefaultTokenService.CreateIdentityTokenAsync ).

 I missed adding AddAspNetIdentity<>().
That was missing. Thanks!


Add support for .co.uk and uk.com domains

These are both tLDs and should not be tested for SPF, DMARC, etc.

Exclude the onmicrosoft.com too

This is an interesting topic for a couple of reasons:

What is a technical implementation that allows for filtering well-known managed second-level domains? I am not aware of an organization restricting this, just multiple that maintain well-known domains within their own standards.
Technically this is still accepted risk. Microsoft should have parked records for onmicrosoft.com, the UK should have parked records for those top-level and second-level domains. Otherwise your organization has some implicit, al be it very little, trust and thus risk that those could be spoofed.

Open to suggestions on how to identify the first elegantly and then potentially flag as a warning for the second.

Since the admin can't do much here to fix TLDs it might be best to avoid showing this as failed.
We could still list them in the table here and flag as ignored.
I think we can hard-code the domains in the PowerShell cmdlet and folks can submit PRs to add them.
We should also add support to pass in domains to be ignore for this test. When we add support for configuring parameters (hopefully this quarter), admins will be able to exclude other domains.
Thoughts @soulemike?

Since the admin can't do much here to fix TLDs it might be best to avoid showing this as failed.
We could still list them in the table here and flag as ignored.

This sounds like a reasonable approach. Only use the domains explicitly registered with Entra for the pass/fail, then show the results for the hierarchy but as ignored for the overall result.

We should also add support to pass in domains to be ignore for this test. When we add support for configuring parameters (hopefully this quarter), admins will be able to exclude other domains.

Yep, allowing an ignored domains array as a property makes sense as well.
I can work on wireframing these two changes up.


Fix basis gates

Description
fixes #75
Currently there the basis_gates argument of the compile function is ignored when compiling circuits to JSON. This fix simply passes the value of basis gates to the dag2json compiler so that the JSON can be compiled in a user specified set of basis_gates as intended.

@ewinston done!


Reach a number using binary search in java

Information about Algorithm
 using binary search we will do this -> Given the integer target, return the minimum number of moves required (i.e., the minimum numMoves) to reach the destination.
(Type here)
Have you read the Contributing.md and Code of conduct

[x] Yes
[ ] No

Other context
 yes it's Leetcode medium problem
(Type here)

hey please assign this issue to me under Hacktoberfest tag


[dog-bug][redirect-filenames]

Understand app features; Entry points for Teams apps

Docs Build status updates of commit b45ca7c:
:warning: Validation status: warnings

File
Status
Preview URL
Details

:warning:Warning

Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

Line 0, Column 0: [Warning: redirected-file-not-removed - See documentation] Redirected file 'concepts/capabilities-overview.md' are still in the repo. After adding a file to the redirection JSON file, you must delete the original file from the repo.
Line 0, Column 0: [Warning: redirected-file-not-removed - See documentation] Redirected file 'concepts/extensibility-points.md' are still in the repo. After adding a file to the redirection JSON file, you must delete the original file from the repo.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit ed7e913:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

msteams-platform/concepts/capabilities-overview.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/extensibility-points.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[nodejs] errors with mount-plain webpack bundled js file

Description
wskdebug for mount-plain.js (used in not zipped actions) is failing for webpack bundled actions while the same bundled js file works with mount-require.js (used in zipped action)
Error
> wskdebug my-bundled-action path/to/myaction.js
#do the invocation
Activation: 15632c7e0da648a0a32c7e0da6c8a022
2019-11-07T09:18:34.487541760Z RangeError: Maximum call stack size exceeded
2019-11-07T09:18:34.487609654Z     at Object.eval (/code/index.js:97:16)
2019-11-07T09:18:34.487619477Z     at __webpack_require__ (/code/index.js:21:30)
2019-11-07T09:18:34.487623701Z     at Object.module.exports.exports.main (/code/index.js:92:18)
2019-11-07T09:18:34.487627441Z     at __webpack_require__ (/code/index.js:21:30)
2019-11-07T09:18:34.487631131Z     at eval (/code/index.js:85:18)
2019-11-07T09:18:34.487634802Z     at eval (/code/index.js:88:10)
2019-11-07T09:18:34.487638372Z     at eval (/code/index.js:140:16)
2019-11-07T09:18:34.487642048Z     at load (eval at NodeActionRunner.init (/nodejsAction/runner.js:80:45), <anonymous>:30:16)
2019-11-07T09:18:34.487646464Z     at main (eval at NodeActionRunner.init (/nodejsAction/runner.js:80:45), <anonymous>:45:24)
2019-11-07T09:18:34.487650648Z     at main (eval at NodeActionRunner.init (/nodejsAction/runner.js:80:45), <anonymous>:48:12)
2019-11-07T09:18:34.487856218Z XXX_THE_END_OF_A_WHISK_ACTIVATION_XXX
2019-11-07T09:18:34.488231343Z XXX_THE_END_OF_A_WHISK_ACTIVATION_XXX
Completed activation 15632c7e0da648a0a32c7e0da6c8a022 in 18.025 sec

The problem seems to come from nested evals
Webpack config
entry: [
            actionPath
          ],
          output: {
            path: buildDir,
            filename: buildFilename,
            libraryTarget: 'commonjs2'
          },
          // see https://webpack.js.org/configuration/mode/
          mode: 'production',
          target: 'node',
          optimization: {
            // error on minification for some libraries
            minimize: false
          },
          // the following lines are used to require es6 module, e.g.node-fetch which is used by azure sdk
          resolve: {
            extensions: ['.js'],
            mainFields: ['main']
          },

Workaround

rename path/to/myaction.js to index.js
zip it
update your action with the new zip action
=> wskdebug works fine

This might be fixed by #27.

👍 seems to fix it


Update logging so that it includes date and time

Change logging formatting so that it includes date and time. It'll help us debug performance-related issues

is anyone working on this issue ?

Not yet, would you like to pick it up?

Thanks for the update.  I can pick it up . Also my holiday has started. Do you have any projects that I could take on during this time?

Cool! What kind of stuff are you interested in? Anything else that you'd like us to do and which isn't on the list?

Thank you for considering me. I am proficient in C, C#, Python, IoT, React.js, and .NET, and I easily adapt to other programming languages. I am actively exploring Microsoft products like Azure and am eager to delve deeper into DevProxy. With ample time available, I can dedicate myself fully to any project assigned to me, ensuring not to waste my holiday. Please let me know if there are any projects available, both within and outside the repository. Thank you once again

We've got several improvements which are documented in the different issues in the repo. Other than that, we've got an exploration opportunity for how we could replace the Titanium Proxy part of Dev Proxy with Yarp. Would you be interested in looking into it?

Yes sure . I will try my best to contribute.

We've got several improvements which are documented in the different issues in the repo. Other than that, we've got an exploration opportunity for how we could replace the Titanium Proxy part of Dev Proxy with Yarp. Would you be interested in looking into it?

where I should provide updates on my progress and where discussions regarding the exploration of replacing Titanium Proxy with Yarp should take place?

We've got several improvements which are documented in the different issues in the repo. Other than that, we've got an exploration opportunity for how we could replace the Titanium Proxy part of Dev Proxy with Yarp. Would you be interested in looking into it?

where I should provide updates on my progress and where discussions regarding the exploration of replacing Titanium Proxy with Yarp should take place?

...

We've got several improvements which are documented in the different issues in the repo. Other than that, we've got an exploration opportunity for how we could replace the Titanium Proxy part of Dev Proxy with Yarp. Would you be interested in looking into it?

where I should provide updates on my progress and where discussions regarding the exploration of replacing Titanium Proxy with Yarp should take place?

We've got several improvements which are documented in the different issues in the repo. Other than that, we've got an exploration opportunity for how we could replace the Titanium Proxy part of Dev Proxy with Yarp. Would you be interested in looking into it?

where I should provide updates on my progress and where discussions regarding the exploration of replacing Titanium Proxy with Yarp should take place?

If you're building it on top of the existing Dev Proxy code base, you could open a draft PR so that we can discuss the different changes in context. If you're experimenting on a separate project, we can continue the discussion here: https://github.com/microsoft/dev-proxy/issues/84


docs: document how to handle deprecated byName references

Add section in the FAQ that explains the error and how to fix it.
Update the error message to reference the new FAQ section.

@brentleyjones @luispadron Will either of you have time to review this PR this week?


Configure ceph dashboard backend and dashboard_frontend_vip

This change introduces a new set of tasks to configure the
ceph dashboard backend and listen just on the mgr related
subnet (and not on '*'). For the same reason the proper
server address is added in both prometheus and alertmanger
systemd units.
This patch also adds the "dashboard_frontend_vip" parameter
to make sure we're able to support the HA model when multiple
grafana instances are deployed.
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1792230
Signed-off-by: Francesco Pantano<EMAIL_ADDRESS>

Thanks for the review, I'll try to address all the listed items.

Overall the workflow looks good but some details need to be reworked.
* You need to add the IPv6 support to the mgr addr fact

Done in my last update

* `groups[mgr_group_name]` isn't enough because you can deploy without specifying any mgr group so the mgr will be automatically colocated with the mon nodes [1] (maybe something like `groups[mgr_group_name] | default(groups[mon_group_name])`)

Done in my last update

* I think we should set the mgr addr fact like we do for mons/rgws [2] (@guits WDYT?). We only need to set a fact with a list (like `[{'name': 'mgr0', 'addr': 'x.x.x.1'},{'name': 'mgr1', 'addr': 'x.x.x.2'}, ...]` so we won't need to have a dedicated task file (configure_dashboard_backends.yml) with `include_tasks + with_items`.

I agree w/ you, we should consider moving these tasks in ceph-facts role to make them available in ceph-dashboard and avoid including another yaml file just to set the mgr backends.
I'll try to address this item in the next update.

  Also be careful about the `inventory_hostname` vs `ansible_hostname` because `groups[mgr_group_name]` uses the first one while `config set mgr mgr/dashboard/xxxx` is using the second one (to be verified)

[1] https://github.com/ceph/ceph-ansible/blob/master/site.yml.sample#L84-L88
[2] https://github.com/ceph/ceph-ansible/blob/master/roles/ceph-facts/tasks/set_monitor_address.yml

jenkins test centos-container-all_in_one

jenkins test centos-container-podman

@fmount I pushed an update to fix the new line issue in the ceph-defaults role and to fix the ipv6 mgr fact (it was using ansible_all_ipv4_addresses)

jenkins test centos-non_container-all_daemons

jenkins test centos-non_container-collocation

@fmount I pushed an update to fix the new line issue in the ceph-defaults role and to fix the ipv6 mgr fact (it was using ansible_all_ipv4_addresses)

Oh great thanks a lot for your help @dsavineau!

jenkins test centos-non_container-all_daemons

jenkins test centos-non_container-collocation

jenkins test centos-non_container-update

jenkins test centos-non_container-all_daemons

jenkins test centos-non_container-update

jenkins test centos-non_container-collocation

jenkins test centos-non_container-all_daemons

jenkins test centos-non_container-collocation

jenkins test centos-non_container-update


boards: add stm32mp15 ev1 flashing procedure

Signed-off-by: Vanessa Maegima<EMAIL_ADDRESS>Readiness

[ ] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Why merge this PR? What does it solve?
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."

[ ] Run spelling and grammar check, preferably with linter.
[ ] Avoid changing any header associated with a link/reference.
[ ] Step through instructions (or ask someone to do so).
[ ] Review for wordiness
[ ] Match tone and style of page/section.
[ ] Run make linkcheck.
[ ] View HTML in a browser to check rendering.
[ ] Use semantic newlines.
[ ] follow best practices for commits.

[ ] Descriptive title written in the imperative.
[ ] Include brief overview of QA steps taken.
[ ] Mention any related issues numbers.
[ ] End message with sign off/DCO line (-s, --signoff).
[ ] Sign commit with your gpg key (-S, --gpg-sign).
[ ] Squash commits if needed.

[ ] Request PR review by a technical writer and at least one peer.

Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.

Docs for 4bbf1a39775a2572f2de8c20eb92eb9a1a7b49e3 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1650/docs/artifacts/html/index.html

Docs for 26eafe27084cb30315ca69293b9b70a0149f4fde are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1651/docs/artifacts/html/index.html

Docs for e8c0f49fba9e819391311d9504199b65a2a05bb0 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1652/docs/artifacts/html/index.html

Docs for 2c90ec0763ac93c09d5e624cecddb9979b4694e5 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1653/docs/artifacts/html/index.html


Card payment

PayPal Android SDK Version
0.0.10
Environment
Sandbox
Android Version & Device
No response
PayPal dependencies
paypal-card-payments-sdk = { module = "com.paypal.android:card-payments", version.ref = "paypal" }
Describe the bug
I follow the documentation about how to make the card payment.
Whenever I start payment approval, I immediately get an error:
Error: 0 - Description: No value for message

BTW, no message field is mentioned in the code snippet in the documentation.
To reproduce

Follow documentation
Start the card payment approval

Expected behavior
No error or at least understandable error
Screenshots
No response

I passed the access token instead of the client id to the config. Maybe it is an issue. Whenever I started passing client id to the config I started to see other errors.

Actually, no, this also doesn't work.
I follow the documentation for the card approval here https://github.com/paypal/Android-SDK/tree/main/docs/CardPayments
As for now, I can not make the card payments. Here is the info I got during the last testing.
Whenever I create the config with code that works for the native payment:
val coreConfig = CoreConfig(
    clientId = providerTokens!!.paymentAccessToken,
    environment = Environment.SANDBOX
)

I’m getting an error in the callback:
Error: 0 - Description: No value for message.

Whenever I create the config with code that API requires:
val coreConfig = CoreConfig(
    clientId = providerTokens!!.clientId,
    environment = Environment.SANDBOX
)

I’m getting an error in the callback:
Error: 404 - Description: The specified resource does not exist. -> [Issue: INVALID_RESOURCE_ID.
Error description: Specified resource ID does not exist. Please check the resource ID and try again.]

The correlation ID is 8dc297e4d9f8c.
I don’t see anything in the log related to PayPal.

@emartynov circling back to this one. Does this still occur?
A 404 may be common when an order cannot be found that's associated with the orderId property passed in through CardRequest. It could also 404 if the order was created on the server side with a client id that's different from the one used when calling CardClient#approveOrder().

I think I can do the card payment just fine now.

Ah, I remember the casue/fix - the card order can not be authorised and should always be collected.


explain refresh parameter of mqtt activity message with examples

Examples for PR #13

Clicked the wrong button.

Sorry, I based it on the actual main state, that had not included the use_node_id yet. Should have waited a minute for you to pull :-)


Electron 4.0.x: build failures in integration test on macOS

There are still some sporadic native crashes on our integration tests on macOS:

https://dev.azure.com/vscode/VSCode/_build/results?buildId=21367
https://dev.azure.com/vscode/VSCode/_build/results?buildId=21364

After emmet tests:
Electron(2642,0x7fffa44a6380) malloc: *** error for object 0x7fbe60759b90: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug
Received signal 6
 [0x000102c2ed2c]
 [0x000102c2ec11]
 [0x7fff6c092f5a]
 [0x000000000000]
 [0x7fff6be301ae]
 [0x7fff6bf2e8a6]
 [0x0001021b0f0c]
 [0x000105c21604]
 [0x000105c208b3]
 [0x000105ca880b]
 [0x000105cb860d]
 [0x000105ca8c63]
 [0x000105b8aca6]
 [0x000105b8b019]
 [0x000105ba57ae]
 [0x000102aca568]
 [0x000102aca63e]
 [0x0001017dd929]
 [0x0001017dda3e]
 [0x0001017e3741]
 [0x0001017dd454]
 [0x000102a35bf8]
 [0x000103e2ad5d]
 [0x000102a34ca4]
 [0x000100dd3654]
 [0x000100dccf10]
 [0x7fff6bd84015]
[end of stack trace]
./scripts/test-integration.sh: line 25:  2642 Abort trap: 6           ./scripts/code.sh $ROOT/extensions/emmet/test-fixtures --extensionDevelopmentPath=$ROOT/extensions/emmet --extensionTestsPath=$ROOT/extensions/emmet/out/test --disable-extensions --skip-getting-started --disable-inspect --user-data-dir=$VSCODEUSERDATADIR
##[error]Bash exited with code '134'.
##[section]Finishing: Run Integration Tests

@deepak1556 it looks like the first build with 4.2.9 was showing this same issue still:https://dev.azure.com/vscode/VSCode/_build/results?buildId=24732
Are we sure the fix made it into that Electron build from Prague?
Electron(27865,0x7fffa6554380) malloc: *** error for object 0x7fb36ff70750: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug
Received signal 6
 [0x00010a9bbb0c]
 [0x00010a9bb9f1]
 [0x7fff6e140f5a]
 [0x000000000000]
 [0x7fff6dede1ae]
 [0x7fff6dfdc8a6]
 [0x000109f3d3cc]
 [0x00010d9ae434]
 [0x00010d9ad6e3]
 [0x00010da3563b]
 [0x00010da4543d]
 [0x00010da35a93]
 [0x00010d917ad6]
 [0x00010d917e49]
 [0x00010d9325de]
 [0x00010a856e58]
 [0x00010a856f2e]
 [0x000109569da9]
 [0x000109569ebe]
 [0x00010956fbc1]
 [0x0001095698d4]
 [0x00010a7c20b8]
 [0x00010bbb7b3d]
 [0x00010a7c1164]
 [0x000108b5fad4]
 [0x000108b58f10]
 [0x7fff6de32015]
[end of stack trace]


u-checkbox and v-model with an array

Description
Is it possible at all to use u-checkbox to pull/push an array? ex:

            <template #label>

                {{ channel.label }}

            </template>

found the answer: i was missing a :value="channel.id"


Hi~I have a question. Foggy Cityscapes dataset has multiple indexes (0.01, 0.02, 0.005).Which index is used in this paper

0.02


Fix integer overflow and underflow in DeferralKey methods and add corresponding tests

Description
This PR addresses two critical issues in the DeferralKey implementation that could lead to integer overflow and underflow, potentially causing panics or incorrect behavior. It also includes new test cases to verify the correctness of the fixes.
Issues Fixed:

Prevent Integer Overflow in range_for_up_to_consensus_round

Issue: When consensus_round equals u64::MAX, using checked_add(1).unwrap() in DeferralKey::range_for_up_to_consensus_round causes a panic due to overflow.
Solution: Replace checked_add(1).unwrap() with saturating_add(1) to safely handle the maximum value without causing a panic.

Prevent Integer Underflow in transaction_deferral_within_limit

Issue: In transaction_deferral_within_limit, when future_round is less than deferred_from_round, subtracting them causes an underflow, resulting in incorrect behavior.
Solution: Use checked_sub to safely perform the subtraction, and handle the None case by returning false.

Test plan

Test Name: test_range_for_up_to_consensus_round_no_overflow
Description: Verifies that DeferralKey::range_for_up_to_consensus_round does not overflow when consensus_round is u64::MAX, and that it correctly returns future_round equal to u64::MAX.

Test Name: test_transaction_deferral_within_limit_underflow
Description: Ensures that transaction_deferral_within_limit correctly handles cases where future_round is less than deferred_from_round, preventing integer underflow and returning false.

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[X] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:
[ ] REST API:

@mystenmark would be a better reviewer for this change

@xiaodi007 Thanks for the contribution - see the comments above. I believe it is impossible for either issue to occur, but in the second case we could be more defensive.

Thank you for your valuable feedback. I change my code to be more defensive.
Add a debug_fatal! to enforce the invariant during object creation; and adjust future_round to be at least deferred_from_round in release builds to maintain the invariant without panicking in new_for_consensus_round.
Add debug_assert!(future_round >= deferred_from_round); and use saturating_sub in transaction_deferral_within_limit.
Thanks again for your insightful feedback! I've made the updates to the PR as you suggested. I'll keep contributing actively. The updates are ready now—please take a look!

@aschran Done!

@aschran Hi, could you please take a look to see if anything else needs to be revised?

@mystenmark Thanks so much for your help! Following your guidance, I was able to fix the issue, and I learned a lot in the process. Really appreciate you taking the time to support me and share your expertise!


Cast to types explicitly in region properties

In scikit-image 0.12.0 some region properties values have changed types. For instance, area is now an integer. To remain backwards compatible, we cast everything to the type expected so as to match. In other words, area will still be a double precision float when using scikit-image 0.12.0.

Passes all the tests. Will release as v0.1.0a50.


when writting html file i do not get any suggesions

Type: Feature Request
when i type ! mark on the html file i do not get any suggessions on it and i also tried going on setting in emmet turn onning the setting but at last i requesting you..
VS Code version: Code 1.84.2 (1a5daa3a0231a0fbba4f14db7ec463cf99d7768e, 2023-11-09T10:51:52.184Z)
OS version: Windows_NT x64 10.0.19045
Modes:

\gifPlease

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


enable gpt_bigcode with bucket_internal bucket_size

What does this PR do?
This PR fixes  the indentation issue causes the apply_FusedSDPA not executed properly, It also enable the gpt_bigcode with bucket_internal bucket_size.

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

hi @libinta this was done by me in https://github.com/huggingface/optimum-habana/pull/1218 why did you upload your own version?


Abstract classes with external methods fail if they are not declared in the main web/index.dart file

EDIT: as @a14n pointed out, the problem is not related to dart_google_maps but always happens when a file is imported, that contains a abstract class like this:
abstract class _LatLngBounds {
  external factory _LatLngBounds();
}

@kulshekhar noted that adding the @JS() annotation fixed the problem, but adding this annotation shouldn't be required.

Original issue:
This is a bit of a weird issue, and it took me a while to find it. I have first reported it at a14n/dart-google-maps#37 and have created a very simplistic demo repo to showcase the problem:
If dart_google_maps and angular2 are both included in a lib/ file, and the web/index.dart file includes that, there are lots of errors like these:
[Error from Dart2JS on angular_maps|web/index.dart]:
packages/google_maps/src/core/mvc/mvcobject.dart:19:3:
External method without an implementation.
  external factory _MVCObject();
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

However, if all code is simply in web/index.dart there are no errors.
I'm assuming that this is an issue with the angular2 transformer, but am not sure.
EDIT: to reproduce both results, simply clone the demo repo, and run pub get && pub build and it will fail with the errors. Then copy the complete code from lib/app.dart into web/index.dart and it will work.

Just had a quick look and it's not specific to google_maps package.
The following code works well with pub build :
--- web/index.dart
import 'package:angular2/core.dart';
import 'package:angular2/platform/browser.dart';

abstract class _LatLngBounds {
  external factory _LatLngBounds();
}
@Component(
  selector: 'main-app',
  template: 'foo'
)
class MainApp {}

main() async {
  bootstrap(MainApp);
}

but when you split it into 2 files, pub build fails with
[Error from Dart2JS]:
web/index.dart:4:3:
External method without an implementation.
  external factory _LatLngBounds();
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

--- web/index.dart
import 'package:angular_maps/app.dart' as app;
main() => app.main();

--- lib/app.dart
import 'package:angular2/core.dart';
import 'package:angular2/platform/browser.dart';

abstract class _LatLngBounds {
  external factory _LatLngBounds();
}
@Component(
  selector: 'main-app',
  template: 'foo'
)
class MainApp {}

main() async {
  bootstrap(MainApp);
}

@a14n, I'm not sure if this is relevant but your example compiles if _LatLngBounds is annotated with @JS()

@kulshekhar thanks for the info.
However, even if the @JS() is not there, pub build should fail.

@matanlurey I just try to run the above code again with angular-2.2.0 and the issue is still there. Did you close this issue because you did not reproduce it ?

This isn't related to Angular Dart unfortunately. We don't do anything specific via dart2js.

Sorry I thought it was caused by the angular2 transformer. But by removing it I get the same issue. I will try to reduce this sample and file a bug against sdk.
Thanks.


sumstat_merger done

Based on the original sumstat merger, following changes are to be made:
0. still keeping the yml generator and output list generator module before it.

Output each individual, qced sumstat as vcf file (item 1 in proposal of #153 )
Once 1 is done, based on the output_list in 0, merged the output vcf into a multi sample(studies) vcf via bcftools merged.
An example of merging are shown as followed, the format will automatically be the union of both vcf (ES:SE:CS:PIP from ES:SE and ES:CS:PIP). The header will be updated as well.

hs3163@csglogin:/mnt/mfs/statgen/xqtl_workflow_testing/ROSMAP/fine_mapping/mvsusie_rss$ bcftools merge /mnt/mfs/statgen/xqtl_workflow_testing/ROSMAP/multivariate/METAL/AC-DLPFC.8.METAL.vcf.bgz  test.vcf.gz | grep -a5  27146595
8       27131936        8:27131936      A       C       .       PASS    .       ES:SE   0.1359:0.0369   .:.     .:.    .:.
8       27146595        8:27146595      C       T       .       PASS    .       ES:SE:CS:PIP    0.0247:0.1088:.:.      -1.50674e-09:.:0:6.34046e-10                                                                                -1.80867e-09:.:0:6.34046e-10     -1.50684e-09:.:0:6.34046e-10
8       27151096        8:27151096      G       A       .       PASS    .       ES:SE   0.1891:0.0585   .:.     .:.    .:.
8       27153998        8:27153998      G       C       .       PASS    .       ES:SE   0.0751:0.0334   .:.     .:.    .:.

hs3163@csglogin:/mnt/mfs/statgen/xqtl_workflow_testing/ROSMAP/fine_mapping/mvsusie_rss$

Now the sumstat merger will generate a collection of temp.vcf for individual tissue, produce a merged vcf from it, then renamed the vcf with the correct tissue names. Still figuring out a way to add correct tissue name to each individual vcf while retaining the yml design.

With the overhaul of yml generator, the  sumstat merger can now correctly generate sumstat for with correctly named vcf.
Merging of duplicated snps by bcftools merge is yet to be tested.

the updated version of read_sumstat and snp_match_dup was written in the module, those code will be removed once added to cugg packages


[ISSUE-24] Fix overwrite of spark config in connection objects

Clone and create spark configuration for source and destination connection objects

@abinaya21 There was another fix that went in today, so you will need to upgrade the pom version to 2.5 & then we should be good to merge. Thanks!

Updated pom version.


Support for VUE 3

Hello everyone. I was wondering if there was a way around for Vue 3 support?

It is on the works. Today I was able to get it working. You can take a look in the next branch.
For the next version I'm keeping all the fundamentals the same but I'm changing all of the internals and some of the APIs to take advantage of the Composition API.
So there are a couple of big items left before I would consider using it on a production app:

[ ] Rework the API to take advantage of the Composition API (breaking changes)
[ ] Duplicate all the API for people still using the Object API
[ ] Write the new documentation

Thank you for bearing with me, it makes me very happy to see people actually using the library :)

Duplicates #33


No sound from the Freeverb

Doesn't seem to be working now since the _dampeningScale has been removed.

I'm not getting that same behavior.
http://jsfiddle.net/yotammann/LwLny67u/

Oh yeah I must be doing something wrong, apologies.

no problem. glad it works.


Cant connect to hue bridge due to mandatory update

 is a comment.
You will see these throughout the issue template. Make sure you do not type your comments
between these characters or we will not be able to see what you wrote!

4. Links
To create a hyperlink, type the [text you want to link](followed by the url in parenthesis)
-->
Describe the bug

Cant connect to hue bridge due to mandatory update
Steps to Reproduce

I tried the stable release and once the bridge is connected in the Android App it gives a message that there is a mandatory update of the bridge that needs to be installed, but this update cannot be installed
I also tried the (dev) version, however this also is not working, once you click on the include message in Home Assistant and next in the hue app it just disconnects in the app without any message (probably related to the mandatory upgrade of the (virtual)hub).
Expected behavior

App succesfully able to connect to the hub.


Add basic multi-agent functionality to ULTRA

For a future version of ULTRA:
Add the ability to have multiple (and potentially different) ULTRA agents undergo training in the same scenarios of a task.

[x] Give ultra/utils/episode.py the ability to record data from multiple agents.
[x] Modify ultra/train.py to support the creation and organization of multiple agents.
[x] Add functionality to ULTRA's scenario generation to support the use of multiple agents.

Add other tasks later...
This is not an issue that needs to be addressed immediately, but can act as just a discussion for future planning if needed.

Has been implemented in ULTRA 0.2


Update footer styles

Problem
Everything is looking really good, but the footer could be updated to match the styles of the rest of the project:

This is related to another issue with text color being too close to the background color in dark mode:

Proposed Solution

The header and footer should match, so the background color of the footer and the text color should be updated to use the same blue and white color scheme.
When dark mode is toggled, ideally the text color in the footer shouldn't change. This should be a small adjustment to the CSS for the toggle.

Hi @marcnjaramillo i would love to work on this

I will like to work on this issue


hackerrank_maximum_passengers_problem.cpp

Hi guys here is the sollution for the maximum passenger hackerkrank problem
Hope this will help you guys Thank you ....!

Bro plz merge it quickly bro as i am late for the fest , 🙏Plz add the hacktoberfest-accepted lable so as to not get excluded


#102-- update software release

Build failed due to
bundle exec rake lint
./software/releases.md:30: MD013 Line length


How to bring a folder with images or 3D objects, instead of text

That way of calling multiplayer files was a good idea to implement, but how can I bring images or 3D objects instead of text?
I mean the script

I also have the same question


Sand>Tainted Soil>???

Dirt

how much flux is in the area?
if you've been doing too much careless alchemy in the area this could easily happen

Duplicate of #607
Sadly it's not a priority for him.

Ahh I misunderstood the issue i thought it was about taint spreading not about turning into sand into dirt


adding changes to bmi_snow17 units and other updates that were missed…

… in the last commit (small)

Hi Andy,
I was trying to build snow17 in the standalone mode using gfortran on UCS
linux servers yesterday in the build/ directory using the Makefile there. I
got an error caused by "-fcheckall". Change to "-fcheck=all" make it work
as you have it elsewhere in the same file. I have the link attached below:
https://github.com/NOAA-OWP/snow17/blob/master/build/Makefile#L48-L50
Shengting
On Wed, Nov 2, 2022 at 10:09 AM Andy Wood @.***> wrote:

Merged #24 https://github.com/NOAA-OWP/snow17/pull/24 into master.
—
Reply to this email directly, view it on GitHub
https://github.com/NOAA-OWP/snow17/pull/24#event-7724371607, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACA4SRLC2PLDKYNCZHA6253WGJ73VANCNFSM6AAAAAARVFXBQU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Bug in engine.pm

[andrew@ad-f22 sqtest]$ sqitch engine add pg sqtest
Can't locate object method "config" via package "App::Sqitch::Command::engine" at /home/andrew/perl5/lib/perl5/App/Sqitch/Command/engine.pm line 88.
Looks like this line should refer to $self->sqitch->config rather than to $self->config
At any rate that seemed to fix it for me.

Dupes #259, was fixed in f4d75dd. Thanks!

Er, also #255. Guess I should do a release soon.

+1 on releasing this fix soon, great library btw

Done, v0.9994 has been released!


Fix factory girl

move factory girl path adjustment to spec_helper as it should not be in the Engine as its only for test environment
specs should always be run from the root of the plugin not from the root of manageiq core, because rspec loads .rspec and you'll want to load your .rspec from this repo and not the one from core
use either:
~/src/manageiq-consumption fix_factory_girl ⇣⇡
❯ pwd
/Users/hild/src/manageiq-consumption

~/src/manageiq-consumption fix_factory_girl ⇣⇡
❯ bundle exec rake
** master - DB vmdb_test on localhost
** master - DB vmdb_test on localhost

Randomized with seed 14812
..**.

Pending: (Failures listed here are expected and do not affect your suite's status)

  1) ManageIQ::Consumption::ShowbackCharge return costs
     # Not yet implemented
     # ./spec/models/showback_charge_spec.rb:18

  2) ManageIQ::Consumption::ShowbackCharge calculates cost
     # Not yet implemented
     # ./spec/models/showback_charge_spec.rb:17

Finished in 0.40283 seconds (files took 11.61 seconds to load)
5 examples, 0 failures, 2 pending

Randomized with seed 14812

or
❯ bundle exec rspec spec/models/showback_charge_spec.rb
** master - DB vmdb_test on localhost

Randomized with seed 48563
.*.*

Pending: (Failures listed here are expected and do not affect your suite's status)

  1) ManageIQ::Consumption::ShowbackCharge calculates cost
     # Not yet implemented
     # ./spec/models/showback_charge_spec.rb:17

  2) ManageIQ::Consumption::ShowbackCharge return costs
     # Not yet implemented
     # ./spec/models/showback_charge_spec.rb:18

Finished in 0.37748 seconds (files took 10.77 seconds to load)
4 examples, 0 failures, 2 pending

Randomized with seed 48563

Coverage decreased (-0.3%) to 77.099% when pulling 8c35beb064e9a960a8a2f3f27021187c0fb70278 on durandom:fix_factory_girl into a53773a041e526a5901a618bed8771000a36a17d on miq-consumption:master.

Coverage decreased (-0.3%) to 77.099% when pulling 8c35beb064e9a960a8a2f3f27021187c0fb70278 on durandom:fix_factory_girl into a53773a041e526a5901a618bed8771000a36a17d on miq-consumption:master.

Coverage decreased (-0.3%) to 77.099% when pulling 8c35beb064e9a960a8a2f3f27021187c0fb70278 on durandom:fix_factory_girl into a53773a041e526a5901a618bed8771000a36a17d on miq-consumption:master.

Coverage decreased (-0.3%) to 77.099% when pulling bd4f95dbbc003626f919290519f29a8d1a1518eb on durandom:fix_factory_girl into a53773a041e526a5901a618bed8771000a36a17d on miq-consumption:master.


Update 'mouse' and 'clicked' language to 'pointer' and 'tapped'

Resolves
Resolves GH-1472
Proposed Changes
Move

go to (pointer)
glide (1) secs to (pointer)
point towards (pointer)

Events

when this sprite tapped
when stage tapped

Sensing

<pointer down?>
(pointer x)
(pointer y)
(distance to (pointer))

Reason for Changes
Scratch 3.0 will work on a wide variety of devices including Chromebooks, Windows Surface, iPads, and Android tablets. Many of these devices either exclusively support touch or support a mixture of touch and mouse interaction. Because of this, we would like to retire use of the terms "mouse-pointer" and "clicked" and use the more generic terms "pointer" and "tapped". This will impact a large number of blocks, but feels important in order to properly support touch platforms moving forward.
Notes

Some of these changes were made in https://github.com/LLK/scratch-blocks/pull/1469
After this lands I'll create a small follow-up PR in the GUI to update the order of the (distance to (pointer)) to fully close out this issue
I've kept the opcode and message IDs the same for now as I'd like to make sure this "sticks" (testing and feedback from the community)
I still have some hesitation around the term "tapped" (particularly as we aren't updating the when "flag" clicked block, but I'm happy to try this and test it out

Reference

/cc @ntlrsk @SpeakkVisually @carljbowman

Between negative feedback from resource partners that came in today (major changes to blocks coming in too late) and some of the questions / concerns around this I suggest we close this PR and backlog this issue till post release.
/cc @carljbowman @ntlrsk @ericrosenbaum


added OBS videos, edited readme file and changes autographa folder names to scribe

Had multiple issues with this PR and the changes are been added in other PR. #closing


Feedback to user

feedback regarding what is going on

loggin / logout
connected / disconnected
files received
files uploaded
...

Activity window allows notifying user about events.
Todo:

"balloons" in tray


Remove usage of tox

tox requires that development workstation is equipped with two python installations (2.6 and 2.7). However it could be somehow difficult to achieve as some linux distribution no longer support usage of 2.6.
Notice that travis is able to test multiple version of python, see: https://docs.travis-ci.com/user/languages/python

If Travis supports it, we can definitely use that; it's rare that there are issues with 2.6/2.7 support. But we can't stop testing Python 2.6 because that's the ancient version of Python that the Hadoop appliance has.

I've been bit by 2.6/2.7 issues that tox has caught. I agree that we need to keep testing with 2.6, and tox is convenient for doing that locally.


warning: UIManager['YoutubeView'] is no longer supported. Use UIManager.getViewManagerConfig('YoutubeView') instead

React Native version: 0.60.3
React : 16.8.6

It is just warnings, when time is up, we can upgrade or If you have time, push a commit

im facing same issue on react-native: 59.10


A setting to highlight focused line on Minimap

On Minimap, we can see the lines with errors are highlighted in red. I think we should have a setting to highlight the current line, that will be very convenient to keep track between the editor and minimap.

Looking for the same thing. As far as I can recall, this used to be like this in the past.. but somehow regressed (or is it just my setup?)

I'll add my vote to this as well. There is actually a current line indicator in the scrollbar next to the minimap, which is definitely better than nothing, but it's just a bit too mall and subtle. It would be great if this could be extended across the minimap as well.
(In case it's helpful I tend to work with vertical size set to fill, in which case drawing the line at the same position in the minimap as it appears in the scrollbar would work fine. In the other modes the indicator would need to be at a different position.)

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Edit conversations tags

User story
As a collective admin or as the creator of the conversation, I can edit the tags for it after it's created.
Best solution for this problem
Add a edit button on the top right of the tags, on the conversation page. Best would be to use the InlineEditField so that we keep the styles consistent. When clicked, replace the tags list by a StyledInputTags with Save / Cancel buttons.

Expected tests

Add an E2E test in https://github.com/opencollective/opencollective-frontend/blob/1557dc8329d3e7ab606ed433593de4fed7cda5a8/test/cypress/integration/17-conversations.test.js#L41 that changes the tags and ensure new tags are properly added.

Testing the feature
Docs about testing conversations: https://docs.opencollective.com/help/contributing/development/testing-features/conversations

Require https://github.com/opencollective/opencollective-api/pull/3201

:gift: :money_with_wings: A $200 bounty was attached to this issue. Anyone submitting a Pull Request will be rewarded with $200 when the Pull Request is reviewed, accepted and merged.
:arrow_right: More info

@Betree working on this.

@joshikeno I'm deploying https://github.com/opencollective/opencollective-api/pull/3201 to staging now so that you'll have the API ready

@Betree It would be nice if testuser -> testCollective has most features enabled by default.
What do you think?

@joshikeno which features?

I was wondering why the conversation section wasn't enabled by default and thought there was a special reason until I referenced the documentation.

Yep, conversations are enabled per default for new collectives


update subscribe for meetings link on landing page

here https://www.asyncapi.com/ we have section about meetings

subscribe button points to https://www.asyncapi.com/newsletter and it is wrong. It should point to https://www.asyncapi.com/community/meetings

/gfi js


Sidekiq 6.0 does not initialize due to a NameError

Environment
Using Ruby 2.6, Rails 5.2, I upgraded Sidekiq to the new 6.0 version this morning, which changes how Sidekiq logging is configured.  Upon startup, I received the following error:
/Users/rfine/.asdf/installs/ruby/2.6.3/lib/ruby/gems/2.6.0/gems/rails_semantic_logger-4.4.2/lib/rails_semantic_logger/engine.rb:111:in `block in class:Engine': uninitialized constant Sidekiq::Logging (NameError)
The line of code in rails_semantic_logger that is causing this is:
      # Replace the Sidekiq logger
      Sidekiq::Logging.logger = SemanticLogger[Sidekiq] if defined?(Sidekiq)

The Sidekiq upgrade notes identify the change -- how Sidekiq logging is newly configured in 6.0:
Sidekiq.configure_server do |config|
  config.log_formatter = AcmeCorp::PlainLogFormatter.new
  # config.log_formatter = Sidekiq::Logger::Format::JSON.new
end

@MisterFine1 I made a PR that attempts to solve this issue. Could you give it a look?


Support for features / optional dependencies

Hey, great plugin!
For example this https://github.com/aws/sagemaker-python-sdk/blob/49c664893339b42815b2fe8a91d912194199e8ba/setup.py#L64-L72 might become:
[tool.hatch.metadata.hooks.requirements_txt.features]
local = ["requirements/extras/local_requirements.txt"]
scipy = ["requirements/extras/scipy_requirements.txt"]
all = [
  "requirements/extras/local_requirements.txt",
  "requirements/extras/scipy_requirements.txt",
]
test = [
  "requirements/extras/local_requirements.txt",
  "requirements/extras/scipy_requirements.txt",
  "requirements/extras/test_requirements.txt",
]

& maybe the top-level option should become an array too?
[tool.hatch.metadata.hooks.requirements_txt]
files = ["requirements.txt"]

@ofek, I attempted an implementation in #13. However I don't understand why you suggest above calling the replacement optional-dependencies section features. According to my tastes, it introduces redundant and confusing terminology, but on the other hand, you tend to have good reasons for your design decisions. :smile: Which name shall we use?

oh awesome! it's up to you, Hatch refers to them as features b/c it's easier to type/remember and it matches https://doc.rust-lang.org/cargo/reference/features.html

Great, thanks for the context! I'd prefer to keep optional-dependencies, but I added that groups of optional dependencies are "also known as "features"" in the README.


Feature: Get video object by plex "key" identifier

Its a bit more direct than having to identify a video by a query string (especially when looking for an episode of a tv series)
re #2

Is this a duplicate of issue #2?

Oh, no its not. Just misread it.


Manually create zip bundle containing fix for #43 and upload to Partner Center twas-cluster offer.

Please assign this to @edburns .

Preview link

Hello @majguo , I observe this is still failing with un-entitled.
The input IBMid account is Unentitled.

I am still investigating.

Hello @majguo here is the imageReference from the version with the preview link above:
"imageReference": {
    "publisher": "ibm-usa-ny-armonk-hq-6275750-ibmcloud-aiops",
    "offer": "2021-04-27-twas-cluster-base-image",
    "sku": "2021-04-27-twas-cluster-base-image",
    "version": "9.0.20210624"
},

This is the latest available version of the base image.

Here is the osProfile:
                "osProfile": {
                    "computerName": "[if(equals(copyIndex(), 0), variables('name_dmgrVM'), concat(variables('const_managedVMPrefix'), copyIndex()))]",
                    "adminUsername": "[parameters('adminUsername')]",
                    "adminPassword": "[parameters('adminPasswordOrKey')]",
                    "linuxConfiguration": "[if(equals(parameters('authenticationType'), 'password'), json('null'), variables('const_linuxConfiguration'))]",
                    "customData": "[base64(concat(parameters('ibmUserId'), ' ', parameters('ibmUserPwd')))]"
                },

@edburns I just tried the preview link with your IBMid credential, and it successfully deployed a tWAS cluster without un-entitled error:

Looks like it's available now, I used the twas-offer from Azure Portal and successfully created a twas cluster using Ed's IBMid credential which contains a comma, and observed that issue #43 is not reproduced.


Add option to sort objects by name or type in object browser

Changes
This PR will add an option for sorting the objects in object browser by name or type:

When this option is enabled, objects are sorted by name and type. If disabled, objects are sorted by type and name - the type being the MI object type, so the list will be sorted like in PDM.
The option is enabled by default to preserve the current behavior of this extension.
Checklist

[x] have tested my change
[ ] updated relevant documentation
[ ] Remove any/all console.logs I added
[x] eslint is not complaining
[ ] have added myself to the contributors' list in CONTRIBUTING.md
[x] for feature PRs: PR only includes one feature enhancement.

I will look into how where we should put this global object and then we should be good to get going!

Will leave update today. Sorry it's been a minute.

@chrjorgensen Here is what I think:
See the schemas directory. We keep constants and initial values in there. Perhaps the getObjectTypeMap method could be extracting into a new ts file and put inside this directory and then brought into IBMiContent.js.
What do you think of that?

@worksofliam I'm sorry, but my TypeScript knowledge is not enough to make a ts import file - so I made a JavaScript import file instead. 😞

@chrjorgensen no problem! I will let this in, and then I'm going to do another PR where I convert it to TS. I will let you review that!


update rdma read/write api

Add local offset lofs, remote offset rofs, and number of bytes size to the parameters of rdma_read, rdma_write, rdma_flush, post_send, and post_receive.
Moreover, since the destination buffers are usually registered and directly operated on in RDMA applications, we skip memcpy which copies between registered buffer and external buffer if buffer is set to nullptr.

rdma_write and rdma_flush have overlapping functionality. Consider removing rdma_flush.

Read/Write are heavily overloaded.
I argue not to pass external buffer into read/write() and make read/write() only operable on registered memory region. If caller wants to write to external buffer, it is trivial for it to perform memcpy() from registered buffer to external buffer.
I am considering to change read() to update(), write() to flush(). update() will perform a RDMA read to refresh the buffer, while flush() writes local buffer to remote. Clients can directly read/write on the local buffer. When they need to sync, they will call update/flush()


Follow OCI container image annotation standards when generating containers

As a part of generating a docker container when running something like: spin registry push ttl.sh/something:1h, we get very basic information about the container.
➜ docker manifest inspect ttl.sh/something:1h
{
    "config": {
        "digest": "sha256:668d372ccc29ffe6f0000bbddbff67d97c2812dbd48b5338ae8532ab6b7c4e59",
        "mediaType": "application/vnd.oci.image.config.v1+json",
        "size": 209
    },
    "layers": [
        {
            "digest": "sha256:2bcd094bfd433033c641264d94d0afbd0adabc666d69433d2f766e0efbe5e598",
            "mediaType": "application/vnd.wasm.content.layer.v1+wasm",
            "size": 36276905
        },
        {
            "digest": "sha256:3f05015b248403ca99a9bf1861202d107885ae300f5ff165708415603a39c248",
            "mediaType": "application/vnd.fermyon.spin.application.v1+config",
            "size": 596
        }
    ],
    "schemaVersion": 2
}

But there is an OCI standard for pre-defined annotations.
It would be great if spin could generate most of these annotations out of the box, if possible, guess them from context. But make it possible to set all values through  thespin.toml file.

Thanks for creating this issue @NissesSenap.  I'm sensing there might be a couple enhancements from this ticket.
Auto-populate relevant annotations
Of the pre-defined annotations, these seem to be natural choices for auto-generating, either from relevant image data or from pre-existing spin.toml metadata:

org.opencontainers.image.created - time of spin registry push ?
org.opencontainers.image.authors - application.authors from spin.toml
org.opencontainers.image.version - application.version from spin.toml
org.opencontainers.image.title - application.name from spin.toml
org.opencontainers.image.description - application.description from spin.toml

Any others you're seeing?
Support annotation values via manifest?
Other annotations that can't currently be automatically derived today can be supplied at time of publishing, eg via spin registry push --annotation org.opencontainers.image.url=https://example.com.
However, as you've mentioned above, it may be handy to set annotation values in the manifest itself.  This would require a bit more thought around design/ergonomics... my first instinct is maybe a new top-level section in the manifest eg [registry] and annotations could be defined under [registry.annotations] - perhaps this section would allow for any entry and any number of them as long as each entry is a key/value pair, eg annotation = value.  Any annotation also provided at time of publishing via CLI (eg via --annotation annotation=value) would take precedence over the key/value set in the manifest.  (In the future, I could envision other metadata under [registry] such as a default registry name, reference, etc.)
Is this along the lines of what you were thinking of?

Hi @vdice , I agree with the pre-defined annotations that you listed.
I also like the idea of creating a new section in the manifest called [registry].
As you point out, future things like default registry name would definitely be useful.
For example, I can see how me as a platform engineer would provide my developers with a spin template that would contain a few of these defaults, to make it easy to build things in CI smoothly.


Server as peer to process decoded video frames

Hey,
This looks like an awesome module. I'm about to dig in deep, but before I do, just want to make sure it can do what I'm hoping:
I'd like my Node server to act as a WebRTC peer and allow me to process decoded video frames in real time on the server.
Is it doable? Any pointers in the right direction? I've started reading through the code in mediasoup/test, but am still not quite sure where I might be able to get access to the video stream.
Thanks!

mediasoup won't decode video frames but just SRTP packets. Those RTP packets can be handled by the JavaScript application:

http://mediasoup.org/api/#rtpReceiver-on-rtpraw
http://mediasoup.org/api/#rtpReceiver-on-rtpobject

But, as stated above, mediasoup won't decode a opus or vp8 stream.


Research addition of customer provided Prometheus/Grafana instance to OSM

As of today OSM brings up a Prometheus and Grafana instance as a part of its control plane, we need to extend this out to
give customers the ability to bring their own Prometheus and Grafana instance.
Research steps and process for adding these to the mesh and enabling metrics collection.

Resolved in #1018


Object doesn't support property or method 'resetUI'

I am using the ajaxFileUpload control in a web app running under IIS7 on Windows 2012 server. After choosing the file to upload, I click 'Upload' and the following error occurs:    Object doesn't support property or method 'resetUI' and the debugger stops in the function below.
raise_uploadError: function(e) {
var eh = this.get_events().getHandler("uploadError");
if(eh) {
eh(this, e);
}
    this._processor.resetUI();

    this._canceled = false;
    this._isUploading = false;
    this._isUploaded = true;
    this.enableControls(true);
},

Would you please specify browser version that you use when this error occurs?

I'm using IE11. The server has .Net 4.5.1 installed.

This happens when a browser does not support HTML 5 and an error occurs during uploading.
IE 11 does support HTML 5, so please try the following:

Check your document mode in Developer Tools (F12) - it seems the page is reset to an older document mode that does not support HTML 5.
You can change page document mode to Edge (Default) from the dropdown list to make sure a page works properly in HTML 5 mode.
This can happen when you set a site to be rendered in Compatibility Mode.

Make sure this tag is added to the <head> section of a page:
<meta http-equiv="X-UA-Compatible" content="IE=Edge" />

If you still encounter the error every time you try to upload a file, this can mean your site is running in Medium trust environment. In this case, please apply this workaround: https://github.com/DevExpress/AjaxControlToolkit/issues/43#issuecomment-203437189

If you run a site on a web farm, please use the codebase starting with this commit: https://github.com/DevExpress/AjaxControlToolkit/commit/051d675233b61601cbcba899284f72ac3d0cde5c

Changing the doctype to HTML5 and adding the meta tag worked. Thanks for your help!


docs: Fix up unparsed SCM_WEB literals

These weren't being parsed, and so were rendered as SCM_WEB rather than
having the github link. Fix them up.

This change is

test-docs-please


Reduce queries with prefetching

Requests can easily blow out to hundreds of queries making it very slow. I'm not sure what the "best way" would be to solve this but I needed to fix it quickly and was able to go from 200 requests in a list of 50 objects to just 1.
Hopefully, there's some better way to have this built-in that would support fragments etc.
@strawberry.django.type(models.InventoryItem, pagination=True)
class InventoryItem:
    id: auto
    name: str
    category: 'InventoryCategory'
    supplier: 'InventorySupplier'

    def get_queryset(self, queryset, info, **kwargs):
        selection_set_node = info.field_nodes[0].selection_set
        fields = [selection.name.value for selection in selection_set_node.selections]
        related = [field for field in fields if field in ['category', 'supplier']]

        return queryset.select_related(*related)

This would be great improvement and I agree 100% with you. We really should do that. We probably would like to optimize the field fetching and query only fields which are actually requested by the end user.

The way I'm implementing it is the following:
class RootModel(models.Model):
    ...

class Model(models.Model):

    root_model = models.ForeignKey(RootModel)
    ...

class ReverseRelationModel(models.Model):

    root_model = models.ForeignKey(RootModel, related_name="reverse_relation_set")
    ...

@strawberry.django.type(ReverseRelationModel)
class ReverseRelationModelType:
    """ReverseRelationModel type."""

    root_model: "RootModelType"

@strawberry.django.type(RootModel)
class RootModelType:
    """RootModel type."""

    reverse_relation_set: List["ReverseRelationModelType"]

@strawberry.django.type(Model)
class ModelType:
    """Model type."""

    id: strawberry.django.auto
    root_model: RootModelType

    def get_queryset(  # pylint: disable=no-self-use
        self,
        queryset: QuerySet[Model],
        info: strawberry.types.Info,
        **_kwargs: T.Any,
    ) -> QuerySet[Model]:
        fields = get_selected_fields_from_info(info)

        if "rootModel" in fields:
            queryset = queryset.select_related("fkey")
            if "reverseRelationSet" in fields["fkey"]:
                queryset = queryset.prefetch_related("reverse_relation_set")
        return queryset

from strawberry.types import Info
from strawberry.types.nodes import FragmentSpread, SelectedField, Selection

def get_selected_fields_from_info(info: Info) -> dict[str, dict]:
    """Fetch selected fields from fragments and ."""
    assert len(info.selected_fields) == 1
    return get_selected_fields_from_selections(info.selected_fields[0].selections)

def get_selected_fields_from_selections(selections: list[Selection]) -> dict[str, dict]:
    """Convert SelectedField's selection to dict of dicts."""
    res: dict[str, dict] = {}
    for sel in selections:
        if isinstance(sel, SelectedField):
            res.update({sel.name: get_selected_fields_from_selections(sel.selections)})
        elif isinstance(sel, FragmentSpread):
            res.update(get_selected_fields_from_selections(sel.selections))
    return res

But this feels hard to make generic. Especially if you need special prefetching cases using django's Prefetch.

Hey guys. Just to point out that I created an optimizer extension in this lib here that does that and more: https://github.com/strawberry-graphql/strawberry-graphql-django

@bellini666 does that library supersede this one, or that is an add-on to this library?

@aareman it is basically an extension to this one, with some fixes, improvements and other new features (some that are still to come in the next days).
To use only the optimizer extension you would only need to add the extension itself to the schema and replace your strawberry.django.type and strawberry.django.field by my implementation. That implementation is a subclass of the one from here, but it makes sure that resolver querysets are optimized, and there's also an important optimization that avoids forcing the resolver to use sync_to_async when the model's values are already prefetched (and thus, won't hit the db).

Good job @bellini666 , It would be great to see all these improvements in core package one day.

Hey @la4de , thanks! :). Yes, I'm planning on porting it to the core package, together with other improvements that make sense.

@bellini666 Is there anything I can do to help porting that stuff over to the core package. All of those features are much needed

Hey @hiporox . Help is always welcome, but this is something that I'm planning on incorporating to this project pretty soon (in the following weeks). I'm already very familiar with the code there and know some places I need to adjust here, so it should be pretty straight forward for me.
Also, I saw that you are sending some PRs and I'll review them all by the weekend. Thank you very much for helping the project :)
Having said all of that, if you want to use the optimizer right now while it is still not officially ported here, you can install my lib, import and use just the DjangoOptimizer extension and it should work out of the box for all places that you return a queryset.


Packaging fails with Serverless Version 1.61.x

After upgrading the Serverless Framework Version to 1.61.x the packaging fails with the following error:
Serverless: Generated requirements from /path/to/requirements.txt in /path/to/.serverless/requirements.txt...
Serverless: Using static cache of requirements found at /home/ubuntu/.cache/serverless-python-requirements/98230c43d557f2de15b2896aab825590e262c81f5024fe42d48dbe9c4f848587_slspyc ...
Serverless: Packaging service...
Serverless: Excluding development dependencies...
Serverless: Injecting required Python packages to package...
Type Error ---------------------------------------------
TypeError: Cannot read property 'length' of undefined
at DataWorker.emit (/path/to/node_modules/jszip/lib/stream/GenericWorker.js:121:54)
at DataWorker.end (/path/to/node_modules/jszip/lib/stream/GenericWorker.js:65:18)
at DataWorker._tick (/path/to/node_modules/jszip/lib/stream/DataWorker.js:92:21)
at DataWorker._tickAndRepeat (/path/to/node_modules/jszip/lib/stream/DataWorker.js:72:10)
at Immediate. (/path/to/node_modules/jszip/lib/utils.js:381:18)
at processImmediate (internal/timers.js:439:21)
at process.topLevelDomainCallback (domain.js:130:23)
 For debugging logs, run again after setting the "SLS_DEBUG=*" environment variable.

Get Support --------------------------------------------
Docs:          docs.serverless.com
Bugs:          github.com/serverless/serverless/issues
Issues:        forum.serverless.com
Your Environment Information ---------------------------
Operating System:          linux
Node Version:              12.13.1
Framework Version:         1.61.2
Plugin Version:            3.2.7
SDK Version:               2.3.0
Components Core Version:   1.1.2
Components CLI Version:    1.4.0

I ran into the same issue.
npm i -g<EMAIL_ADDRESS>

Same problem with serverless  version 1.63.0. Downgrade to version 1.60.5 fixes the problem temporarily.

It's still happening in 1.67.0.

this seems to be resolved with serverless 1.69.3
Can anyone else confirm?

Right now I can just find version 1.69.0 and this one doesn't still work in my case

i was incorrect.  it is version 1.69.0 that is working for me now
version 5.1.0 of serverless-python-requirements

{
  "serverless": "^1.69.0",
  "serverless-python-requirements": "^5.1.0"
}

Works for me after I delete the yarn.lock file and the node_modules folder.

Even after deleting the lock file and the node modules folder I get the following output:
Serverless: Generated requirements from ./requirements.txt in .serverless/requirements.txt...
Serverless: Using static cache of requirements found at /Users/xxx/Library/Caches/serverless-python-requirements/98230c43d557f2de15b2896aab825590e262c81f5024fe42d48dbe9c4f848587_slspyc ...
Serverless: Packaging service...
Serverless: Excluding development dependencies...
Serverless: Injecting required Python packages to package...

  Type Error ---------------------------------------------

  TypeError: Cannot read property 'length' of undefined
      at DataWorker.emit (/Users/xxx/dev/repos/auth/node_modules/jszip/lib/stream/GenericWorker.js:121:54)
      at DataWorker.end (/Users/xxx/dev/repos/auth/node_modules/jszip/lib/stream/GenericWorker.js:65:18)
      at DataWorker._tick (/Users/xxx/dev/repos/auth/node_modules/jszip/lib/stream/DataWorker.js:92:21)
      at DataWorker._tickAndRepeat (/Users/xxx/dev/repos/auth/node_modules/jszip/lib/stream/DataWorker.js:72:10)
      at Immediate.<anonymous> (/Users/xxx/dev/repos/auth/node_modules/jszip/lib/utils.js:381:18)
      at processImmediate (internal/timers.js:456:21)
      at process.topLevelDomainCallback (domain.js:137:15)

     For debugging logs, run again after setting the "SLS_DEBUG=*" environment variable.

     Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Issues:        forum.serverless.com

     Your Environment Information ---------------------------
     Operating System:          darwin
     Node Version:              14.1.0
     Framework Version:         1.69.0
     Plugin Version:            3.6.9
     SDK Version:               2.3.0
     Components Version:        2.30.6

Should I may use a different node version? Any suggestion would be appreciated.

~/src$ node --version
v13.12.0

~/src$ serverless --version
Framework Core: 1.69.0
Plugin: 3.6.9
SDK: 2.3.0
Components: 2.30.5

Also still seeing this with serverless 1.7.0 and serverless-python-requirements 5.1.0:
Serverless: Injecting required Python packages to package...

  Type Error ---------------------------------------------

  TypeError: Cannot read property 'length' of undefined
      at DataWorker.emit (/xxx/node_modules/jszip/lib/stream/GenericWorker.js:121:54)
      at DataWorker.end (/xxx/node_modules/jszip/lib/stream/GenericWorker.js:65:18)
      at DataWorker._tick (/xxx/node_modules/jszip/lib/stream/DataWorker.js:92:21)
      at DataWorker._tickAndRepeat (/xxx/node_modules/jszip/lib/stream/DataWorker.js:72:10)
      at Immediate.<anonymous> (/xxx/node_modules/jszip/lib/utils.js:381:18)
      at processImmediate (internal/timers.js:456:21)
      at process.topLevelDomainCallback (domain.js:137:15)

  Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Issues:        forum.serverless.com

  Your Environment Information ---------------------------
     Operating System:          darwin
     Node Version:              12.16.3
     Framework Version:         1.70.0
     Plugin Version:            3.6.11
     SDK Version:               2.3.0
     Components Version:        2.30.10

Can also confirm that 1.60.5 works without any issues

The combination of serverless 1.71.1 / Node 12.16.3 works for me; whereas serverless 1.64.0 did not work (which matches the reports above that the problem started in 1.61.0 probably). All using serverless-python-requirements 5.1.0.

I am still plagued with this issue. I have been locked on version 1.35.1 of serverless and each time i got to upgrade i have this issue.  Any traction on resolving this issue?

@dschep Are you aware of any work around for me? Or any insight on what the root cause might be? Seems like i have been locked on a old version of serverless long enough that its causing a problem for me. I'm assuming this issue doesn't plague everyone or there would be much more traction. The stack doesn't mean much to me but i can make some educated guesses that this is during the zip process and that maybe the list of files to zip is undefined?

Today i noticed that if i remove the serverless-s3-sync that this issue is resolved.

I think in my case the reason for that error was that I added manually some plugins in plugin section of serverless.yaml
plugins:
  - serverless-plugin-scripts
  - serverless-tag-cloud-watch-logs
  - serverless-python-requirements <= added manually

After deleting it and adding with sls plugin install --name plugin-name everything went fine. Hope it will help sb.


UI glitch with inline chat

Testing #208411
See the video for detailed repro steps, but there are glitches, especially while typing multi-line for inline chat, the focus goes back and forth between what I am typing and the beginning of the inline chat prompt.
https://github.com/microsoft/vscode/assets/62267334/c3ef18e3-6f9c-49eb-9c5d-fbde39af4e14

This issue lacks

the full prompt you are typing

@anthonykim1 please add details

Exact/full prompt I am typing:
import python ast and import python basic math library that I can use to insert in the current file

Github copilot version: v1.176.0
Github copilot chat version: v0.14.2024032601
Insider version: Version: 1.88.0-insider
command + i (control +i on windows) and typing multi-line prompts have focuses move back and forth between multi-line prompt and beginning of the prompt.

Thanks for clarifying. This should be better since the widget now resizes when multiple lines are received. Tho, once it reaches its max height it still fails to reveal the end/final line. Leaving that for April

Folding this into https://github.com/microsoft/vscode-copilot/issues/6649#issuecomment-2225007522 for which we finally found out the root cause


Update decoder.ts

fix: Underrun error on COMPLETED_ORDER: End of message reached. Please report to https://github.com/stoqey/ib - code: 505 - reqId: -1

Hello,
Thanks for your PR.
Basically I don't understand the reason behind most of your changes; and your changes on decoder.ts hide the problem instead of fixing it. Having an exception raised when we identify inconsistent data is what we want.
May I suggest you to update your PR?
Thanks


fix: chain id for goerli

What I did
Use correct chain ID for goerli
How I did it
changed it
How to verify it
I used this list to verify: https://besu.hyperledger.org/en/stable/Concepts/NetworkID-And-ChainID/
when typing goerli in chain list, it shows Optimism for some reason, probably how this happened.
Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

Wonder why it was 420 :thinking:


Inso installed via NPM doesn't work on WSL2

Expected Behavior
Inso CLI tests run in WSL bash should work the same as they do in Windows command prompt
If I run a test from the WSL bash prompt, the npm-installed cli an exported design collection, it should function the same as it does in a Windows command prompt.
Actual Behavior
So using the inso cli on Windows command line (installed from npm) works great.
If, however, I switch to one of my WSL Linux distros, install the same inso cli via NPM, I get no love.
The test is ridiculously simple, to POST a JSON payload to a locally-running API endpoint.  On Windows, the test passes successfully. On WSL, sad face.
The actual test logic is blindingly simple:
const response1 = await insomnia.send();
expect(response1.status).to.equal(200);

Here's me running the command in a windows command prompt succesfully, then immediately switching to WSL (bash) and running the very same command, but with a weird error showing the response1.status is 'undefined'
Windows Command Line
D:\dev\na\na
λ  inso run test Test-To-Run --src api-automation\insomnia-file.json --env dev --verbose --disableCertValidation
» Data store configured from file at D:\dev\na\na\api-automation\insomnia-file.json
» Load api specification with identifier Test-To-Run from data store
» Found 0.
» Load workspace with identifier Test-To-Run from data store
» Found 0.
» Load unit test suite with identifier Test-To-Run from data store
» Found 1.
» Load base environment for the workspace wrk_07456ff1e3f84bc9a356155084502515 from data store
» Found 1.
» Load sub environment with identifier dev from data store
» Found 1

  Test-To-Run
    √ Request-Being-Run (1699ms)

  1 passing (2s)

WSL / BASH
D:\dev\na\na
λ bash
 /mnt/d/dev/na/na$ inso run test Test-To-Run --src api-automation/insomnia-file.json --env dev --verbose --disableCertValidation
› Data store configured from file at /mnt/d/dev/na/na/api-automation/insomnia-file.json
› Load api specification with identifier Test-To-Run from data store
› Found 0.
› Load workspace with identifier Test-To-Run from data store
› Found 0.
› Load unit test suite with identifier Test-To-Run from data store
› Found 1.
› Load base environment for the workspace wrk_07456ff1e3f84bc9a356155084502515 from data store
› Found 1.
› Load sub environment with identifier dev from data store
› Found 1

  Test-To-Run
    1) Request-Being-Run

  0 passing (974ms)
  1 failing

  1) Test-To-Run
       Request-Being-Run:
     AssertionError: expected undefined to equal 200
      at Context.<anonymous> (/tmp/insomnia-testing/0.5711905741178316-test.ts:11:33)

As you can see in the error message at the bottom, it appears the response is not populated / is 'undefined'
Reproduction Steps
Create an API endpoint which is available to both windows command line and WSL bash.  A wget or tracert should confirm that both command lines can access the URL in question.
Create a simple design collection with a test and request which does a POST or GET to this endpoint
Call this test from the Insomnia desktop client.  Works great.
Call this test from the inso cli from a Windows command prompt.  Works great.
Call this test from the inso cli  from a WSL bash prompt.  Fails
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
Inso version: 2.4.0
What operating system are you using?
Windows
Operating System Version
Windows Version	10.0.19043 Build 19043,    WSL version: 2
Installation method
npm install --global
Last Known Working Insomnia version
?

Thanks @chrissmithptml , just to confirm, what are your node/npm versions?

Couldn't reproduce this one so far.
filipe@win10:~/personal/insomnia-sync-experiment$ inso run test "Example tests" --env "OpenAPI env" --verbose --disableCertValidation
› Data store configured from git repository at /mnt/c/Users/filipe/personal/insomnia-sync-experiment
› Load api specification with identifier Example tests from data store
› Found 0.
› Load workspace with identifier Example tests from data store
› Found 0.
› Load unit test suite with identifier Example tests from data store
› Found 1.
› Load base environment for the workspace wrk_840ae195af1a4094a403b75e097ca988 from data store
› Found 1.
› Load sub environment with identifier OpenAPI env from data store
› Found 1

  Example tests
    ✓ Returns 200 (1322ms)
    ✓ Returns 404 (230ms)

  2 passing (2s)

This was running on a WSL 2, distro is Ubuntu 20.04.3 LTS
Windows info: Windows 10 Pro 21H2 19044.1387
node version v12.18.3
npm version 6.14.6

Running on node in windows directly (same node and npm versions as mentioned above):
C:\Users\filipe\personal\insomnia-sync-experiment>inso run test "Example tests" --env "OpenAPI env" --verbose --disableCertValidation
» Data store configured from git repository at C:\Users\filipe\personal\insomnia-sync-experiment
» Load api specification with identifier Example tests from data store
» Found 0.
» Load workspace with identifier Example tests from data store
» Found 0.
» Load unit test suite with identifier Example tests from data store
» Found 1.
» Load base environment for the workspace wrk_840ae195af1a4094a403b75e097ca988 from data store
» Found 1.
» Load sub environment with identifier OpenAPI env from data store
» Found 1

  Example tests
    √ Returns 404 (1388ms)
    √ Returns 200 (262ms)

  2 passing (2s)

C:\Users\filipe\personal\insomnia-sync-experiment>

ha!  I was just looking at that.  I'm on v14.18.2.  I noted when trying to build the inso cli locally, there's a nasty error in there relating to one of the package versions you use not being supported on 14+

Just to be 100% sure that it wasn't some jiggery pokery with my Ubuntu 20.04 LTS distro, I nuked my WSL distro completely, reinstalled a fresh one, installed the versions of nodejs and npm you outlined above
$ node --version
v12.18.3
$ npm --version
6.14.6
$ inso --version
2.4.0

After much back and forth and then debugging through the inso code locally, it came down to WSL networking.  It relates back to calling inso (or any command for that matter) attempting to hit a url on the host Windows workstation, which is NOT supported on WSL2 at this point without all kinds of hacks.
I am instead creating a docker image with inso installed which can more easily call the host workstation using docker-compose 'extra_hosts' values.


get_historical_klines / get_klines data has gaps

Describe the bug
get_historical_klines and get_klines data has gaps in it.
To Reproduce
client.get_historical_klines('ADABTC', '5m', '2 hours ago UTC')
and plot the data
I've also tried get_klines and it gives the same results
Expected behavior
expecting the data to be at regular intervals of 5m, but instead I have gaps here and there (see log)
Environment (please complete the following information):

Python version: [e.g. 3.6]
Virtual Env: [conda]
OS: Windows
python-binance version [ 0.7.9]

Logs or Additional context
2021-03-02 08:50:00
2021-03-02 08:55:00
2021-03-02 09:00:00
2021-03-02 09:05:00
2021-03-02 09:10:00
2021-03-02 09:15:00
2021-03-02 09:20:00
2021-03-02 09:25:00
2021-03-02 09:30:00
2021-03-02 10:10:00
2021-03-02 10:15:00
2021-03-02 10:20:00
2021-03-02 10:25:00
2021-03-02 10:30:00
2021-03-02 10:35:00
2021-03-02 10:40:00
2021-03-02 10:45:00
2021-03-02 10:50:00
2021-03-02 10:55:00
2021-03-02 11:00:00
2021-03-02 11:05:00
2021-03-02 11:10:00
2021-03-02 11:15:00

Ya even I noticed the gaps while downloading the data.
is there any fix? I am also using the same version of Binance API, but with Python 3.8

Was Binance down or undergoing maintenance during that period of time?

When I'm calling the client.get_historical_klines('ADABTC', '5m', '2 hours ago UTC') method I receive an array the first entry looks like the timestamp in tics. How do you guys plot ur data? I dont know how to split this data into "high" "low" "open" "close" to plot it... Dose anyone have an example for me please?...


Intermittent very long latency in XRT operations

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): mater
Python version: N/A
Bazel version (if compiling from source): 0.16.1
GCC/Compiler version (if compiling from source): 7.2.0
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce: N/A

Describe the problem
On occasion, I've seen XRT operations take significantly longer than I'd expect and then they usually do (into the 10s of seconds). Attaching GDB while this is happening reveals that it is spending most of its time in TF Graph level optimization passes, particularly EncapsulateXlaComputations (which don't really make any sense in conjunction with XRT ops). I don't have an exact reproducer for when this happens, but it appears that it's more likely to happen upon the first XRTAllocate after reconnecting after a client crashed. I believe, I managed to capture a log  at VLOG level 2 while this was happening (see gist at https://gist.github.com/Keno/3a5e0dc86d3829f5712c5e1a5f65161b). Hopefully that should aid in figuring out what's going on. At the end of the gist, it tried to dump the serialized representation of the ~500MB sized model into the log, so I interrupted it there. I can try to reproduce it again, letting it finish dumping the serialized model if that would be helpful.
cc @michaelisard

I have also seen this behavior. Profiling shows that (at least one major) bottleneck is Encapsulator::GetFunctionNameAttr inside EncapsulateXlaComputationsPass, which itself is mostly AttrSlice::Find—this call seems to take >10s cumulatively.

The profile was from before https://github.com/tensorflow/tensorflow/commit/24333d8e55bdd995089e93122750340bf8d1ddba, which fixes the specific issue I was seeing. I'm looking into whether there are other bottlenecks.

It looks like some Grappler passes still take a lot more time than I might expect (the graph is only a small handful of XRT ops) and it looks like this is due to large constant tensors that are repeatedly copied or checked for equality. In particular TensorProto::merge and FastAttrValueHash are each responsible for several seconds of CPU time in a single gRPC call for a graph composed of a handful of XRT ops, if those ops include large tensors.

Is there a need to pass large constants in XRT? Does the problem go away if
you use placeholders instead?
On Tue, Oct 16, 2018 at 1:34 PM James Bradbury<EMAIL_ADDRESS>wrote:

It looks like some Grappler passes still take a lot more time than I might
expect (the graph is only a small handful of XRT ops) and it looks like
this is due to large constant tensors that are repeatedly copied or checked
for equality. In particular TensorProto::merge and FastAttrValueHash are
each responsible for several seconds of CPU time in a single gRPC call for
a graph composed of a handful of XRT ops, if those ops include large
tensors.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/22975#issuecomment-430389461,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFIK9QOxtOykMMIGGNn1U4tz2fEhszjTks5ulkLNgaJpZM4XbaNO
.

Let me try that change. I may have taken the API examples a bit too seriously ;)

@azaks2  Hi, could you please look into this issue ?

Also, slightly related in that it's caused by a grappler optimizer, the logs get spewed with a bunch of xrt related messages from pin_to_host_optimizer:
2018-10-25 15:36:35.278265: I tensorflow/core/grappler/optimizers/pin_to_host_optimizer.cc:194] Could not find KernelDef for: XRTExecute
2018-10-25 15:36:35.279518: I tensorflow/core/grappler/optimizers/pin_to_host_optimizer.cc:194] Could not find KernelDef for: XRTExecute
2018-10-25 15:36:35.280739: I tensorflow/core/grappler/optimizers/pin_to_host_optimizer.cc:194] Could not find KernelDef for: XRTExecute
2018-10-25 15:36:35.284193: I tensorflow/core/grappler/optimizers/pin_to_host_optimizer.cc:194] Could not find KernelDef for: XRTExecute
2018-10-25 15:36:35.285282: I tensorflow/core/grappler/optimizers/pin_to_host_optimizer.cc:194] Could not find KernelDef for: XRTExecute
2018-10-25 15:36:35.286265: I tensorflow/core/grappler/optimizers/pin_to_host_optimizer.cc:194] Could not find KernelDef for: XRTExecute
2018-10-25 15:36:35.315119: I tensorflow/core/grappler/optimizers/pin_to_host_optimizer.cc:194] Could not find KernelDef for: XRTReadLiteral

Doesn't seem to be causing a problem, but is a bit annoying. Let me know if I should open a separate issue.

Nagging Assignee @harshini-gadige: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @harshini-gadige: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


Publish application symbols

📝 Provide a description of the new feature
I wanted to investigate the memory leak from #8378, but couldn't find the symbols (.pdb files).
Please publish them with the app or on a symbol server.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

 But excuse me I don't understand what you want

@giggio
thank you for offering to help but the leak has been fixed already.
For future reference, the symbols are available here https://github.com/microsoft/PowerToys/releases/tag/v0.27.0

That's all I needed. Thanks!


Case-insensitive DNS names

We notice that DNS validation in Pilot's model validation is using a regex which does not support uppercase dns entries. There is a newer RFC 4343 which states that the DNS names are case insensitive. Also in kubernete's DNS validation this change has already taken place.
Is this a potential change that we could PR?

@rshriram

Shouldn’t make a difference in current code. Go for it

Fixed by #5528


add some bundle to import into database, and add some details on README file

I added correct modifications to ontology.py


Task view not displaying correctly when taskbar on top.

Due to Microsoft rewriting the taskbar for Windows 11 the taskbar on top is not officially supported. At the moment application views are not displayed when hovering over a taskbar icon. An alternative for this is to use alt-tab or the new Windows 11 Task View functionality (which is on the taskbar by default).
Currently there is no real solution for this issue. If this issue bothers you, it is suggested to leave the taskbar on the bottom.

Closed now, since a more extensive issue report is currently open.


New minor version v0.2.0

New model L5.4 based
Added Helper file with functions for app support

New model L5.4 based
Added Helper file with functions for app support

Travis CI integration failed, but no problem with testing right now.


Add property to control dependency poms.

The behavior of the shade plugin creates a dependency reduced POM
and places it in the basedir rather than the artifacts directory. This
behavior is unwelcome if basedir is a read-only filesystem. Adding
a property to allow this feature to be disabled in these environments.

Can you file an overarching JIRA? I think this change and previous would fit under "allow Beam to build on a read-only filesystem"

That said - you need to be able to write outputs somewhere. So wherever that "somewhere" is, you could first copy over the source tree then build with default settings. I concede that this is a bit distasteful.

I concede that this is not a generally useful patch in the way that steering the artifacts directory is. I'm closing this PR.


[Suggestion]Date and version

Just a little suggestion to put the report generation date and gpt-researcher version in the generated reports, say on the first page upper right corner. It's useful when one opens an old report, but you don't know when it was generated or how old it is; also useful when the reports are generated periodically, one can see the date and GPTR version.

Good call @petric3
We'll take it into consideration for the demo app

I have the same problem. I'd also put the GPTS version at the bottom of the main GPTS server page, one directly knows if it was successfully updated (many have automated update procedure).


About tests

What was the reasoning behind removing the config file for travis?
https://github.com/bear/parsedatetime/commit/0f8ce6d84005c45ddfa5ed4beba93bb43493b724

to be honest, probably just a mistake on my part thinking it wasn't being used


After successfully recharging the citrus wallet its showing html code.

After successfully recharging the citrus wallet its showing me html code. But amount is credited in my wallet. After recharging the wallet its not redirect to home screen. In both the environment we are facing this issue.
Th HTML code is as follows:

Redirect URL

This is because your redirectURL page could not communicate with SDK via jsinterface. If this communication is successful, SDK will close the webview and response will be returned to the app.
Please refer https://developers.citruspay.com/doc/integrations/  for returnURL page code.


failing with python3 + defusedxml

Hello,
tests (and the library) is failing when running on the python3 (3.11.6) and the defusedxml (0.7.1) is present (as this is the preffered xml library by the xlrd2). In defusedxml there is some strange situation where Element_has_iter is evaluated as false, but at the same time there is no getiterator() function.
self = <xlrd2.xlsx.X12Book object at 0x7f676b8b09d0>
stream = <zipfile.ZipExtFile name='xl\\workbook.xml' mode='r' compress_type=deflate>
heading = 'Workbook'

    def process_stream(self, stream, heading=None):
        if self.verbosity >= 2 and heading is not None:
            fprintf(self.logfile, "\n=== %s ===\n", heading)
        self.tree = ET.parse(stream)
        getmethod = self.tag2meth.get
>       for elem in self.tree.iter() if Element_has_iter else self.tree.getiterator():
E       AttributeError: 'ElementTree' object has no attribute 'getiterator'

xlrd2/xlsx.py:266: AttributeError

Full log from pytest:
error.txt
Workaround:
use xml.etree.cElementTree as default rather than defusedxml.ElementTree in xlrd2/xlsx.py

Related to this -
https://github.com/tiran/defusedxml/issues/48
https://github.com/python-excel/xlrd/pull/375


[Bug] [Master] Failed to pass global parameters between parent and sub process

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
When the parent process has global parameters, but the sub process does not have global parameters, a null pointer error will be triggered
[ERROR] 2021-12-31 14:06:44.898 org.apache.dolphinscheduler.server.master.runner.MasterSchedulerService:[215] - scan command error java.lang.UnsupportedOperationException: null         at java.util.AbstractList.add(AbstractList.java:148)         at java.util.AbstractList.add(AbstractList.java:108)         at org.apache.dolphinscheduler.service.process.ProcessService.joinGlobalParams(ProcessService.java:1025)         at org.apache.dolphinscheduler.service.process.ProcessService.setSubProcessParam(ProcessService.java:992)         at org.apache.dolphinscheduler.service.process.ProcessService.handleCommand(ProcessService.java:226)         at org.apache.dolphinscheduler.service.process.ProcessService$$FastClassBySpringCGLIB$$ed138739.invoke(<generated>)
What you expected to happen
Global parameters are passed normally
How to reproduce
Configure two processes, where the sub process has no global parameters, and the parent process  has global parameters
Anything else
No response
Version
2.0.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I will check this issue.


Other epaper support

Do you know if it is even possible to use the follow epaper-102 display on this board?  Thank you in advance!
https://github.com/Nicell/epaper-102
Here's the pin layout:

It's not pin compatible, no. But I wouldn't say it's impossible. I believe it requires one more pin to be wired beyond what's already available. So there would be some extra bodging necessary.
I have some of those boards and a few e-paper displays that I've tested with. If I'm being honest, the refresh rate is pretty frustrating. And the sharp memory displays are actually more power efficient. (Nice!View is a sharp memory display) so it's a tough sell for me.
 I just have a hard time recommending it.

Not much else to say about this. Closing the issue.

closing.