No feedback when an error occurs in the server in certain cases

With the way the tracing system is set in the server, there's no feedback provided to the user if an error occurs in certain cases. This is the case because for every tracing::error expression, we also require to send a request to display the message like:
https://github.com/astral-sh/ruff/blob/7571da877811100d88fd0a33c933835d05c7c2e4/crates/ruff_server/src/server/api/requests/execute_command.rs#L129-L130
But, it's missing in a few cases. For example, if the path specified in ruff.configuration is invalid, the user won't know that:
https://github.com/astral-sh/ruff/blob/7571da877811100d88fd0a33c933835d05c7c2e4/crates/ruff_server/src/session/index/ruff_settings.rs#L288-L291
Related to https://github.com/astral-sh/ruff/issues/12523

I looked at this and I think the reason this error message isn't displayed is because this code gets executed many number of times while building the settings index. We could either introduce a macro like show_err_msg_once but the logs would still contain multiple entries.


Can I import Plex glyphs with Carbon?

Summary
We recently ran into an issue on the "Let's Create" rebrand on IBM.com: https://www.ibm.com/lets-create/
On Safari, the design team noticed that the curly arrow glyph ↷, which appears on many new pages, does not render correctly in Safari.
We were able to fix this by adding the following to the <head>:
<link
      rel="stylesheet"
      href="//1.www.s81c.com/common/carbon/plex/plex-full.css"
/>

The only downside to this approach is that the font is loaded again. We are also using the following in the SCSS like so:
@import "~@carbon/type/scss/font-face/mono";
@import "~@carbon/type/scss/font-face/sans";
@include carbon--font-face-mono();
@include carbon--font-face-sans();

What is the best-practice approach to including glyphs on IBM.com?
Relevant information

Name and organization
Alex Motzenbecker
<EMAIL_ADDRESS>IBM/Ogilvy
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

@mstudio What version of Safari is being used in that screenshot? I'm on 15.3 (176<IP_ADDRESS>.8) and it renders correctly for me.

@kennylam The glyph rendering issue is seen on the latest version of Safari Version 15.4, when the normal plex.css CDN is being used, however, when the reference is changed to //1.www.s81c.com/common/carbon/plex/plex-full.css, the glyph is rendering fine on Safari as seen on the current https://www.ibm.com/lets-create/ and this CodeSandbox (https://codesandbox.io/s/goofy-snyder-ds0d0u?file=/index.html).
Looks to me like the plex-full.css is pulling from @ibm/plex package, which has all of the font families and including glyphs where the normal plex.css doesn't have? https://github.ibm.com/webstandards/plex-cdn/blob/main/scss/plex-full.scss#L9

@mstudio So after a little testing I can confirm these two imports should solve the problem – no need for plex-full
<link rel="stylesheet" href="https://1.www.s81c.com/common/carbon-for-ibm-dotcom/tag/v1/latest/plex.css"/>
<link rel="stylesheet" href="https://1.www.s81c.com/common/carbon/plex/sans.css"/>

It looks like certain fonts may be left out in plex.css when the CDN file is built. I'll need to do a little more digging and create a new issue for this. But in the meantime this should resolve your issue so you don't need to import the entire font family.
cc: @proeung

@kennylam Yay! Looks like the two imports that you've suggested are working correctly on Safari. See CodeSandbox example - https://codesandbox.io/s/xenodochial-chandrasekhar-75cn7m
I'll update the documentation of this component so that adopters like AEM can use the proper plex imports to show the correct glyphs! Thanks, @kennylam!

@mstudio @proeung New issue #8661 created to track this bug.


Stop showing diplomat modal on language change

fixes ENG-1244

Summary by CodeRabbit

New Features

Introduced methods for managing dynamic document head updates and integrating navigation components.

Improvements

Enhanced language change handling and achievement display logic for better user experience.
Refined user role management and subscription handling for improved clarity and efficiency.

Bug Fixes

Refined error handling for user actions to ensure clear communication of network issues.

Removals

Removed the DiplomatSuggestionModal functionality to streamline user interactions.

It had a long run! You can also get rid of app/templates/core/diplomat-suggestion.pug.


[Rancher2] (Continue) Add RKE2 as an installation method to Rancher Docs

Request Summary:

https://github.com/rancher/docs/issues/3056 only partially fulfils our desire to add RKE2 as an install method to the Rancher2 docs. Could we update https://rancher.com/docs/rancher/v2.x/en/installation/requirements/ so it will list RKE2 as an option?  For the cpu and memory just mirror the requirements rancherd outlines. You can reach out to me for any details needed.
Details:

Note that the currently-documented workaround to make ingress a daemonset on RKE2 isn't working:
Normal   Scheduled  6m59s                  default-scheduler  Successfully assigned kube-system/rke2-ingress-nginx-controller-lfg64 to oly-rancher-52d
Normal   Pulling    5m21s (x4 over 6m59s)  kubelet            Pulling image "index.docker.io/us.gcr.io/k8s-artifacts-prod/ingress-nginx/controller:v0.34.1"
Warning  Failed     5m20s (x4 over 6m58s)  kubelet            Failed to pull image "index.docker.io/us.gcr.io/k8s-artifacts-prod/ingress-nginx/controller:v0.34.1": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/us.gcr.io/k8s-artifacts-prod/ingress-nginx/controller:v0.34.1": failed to resolve reference "docker.io/us.gcr.io/k8s-artifacts-prod/ingress-nginx/controller:v0.34.1": pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed
Warning  Failed     5m20s (x4 over 6m58s)  kubelet            Error: ErrImagePull
Normal   BackOff    5m4s (x6 over 6m58s)   kubelet            Back-off pulling image "index.docker.io/us.gcr.io/k8s-artifacts-prod/ingress-nginx/controller:v0.34.1"
Warning  Failed     110s (x20 over 6m58s)  kubelet            Error: ImagePullBackOff
The issue is being addressed here: (https://github.com/rancher/rke2/issues/493
But in the mean time, the tweak documented at the bottom of this page doesn't work: https://rancher.com/docs/rancher/v2.x/en/installation/resources/k8s-tutorials/ha-rke2/

Closing because https://github.com/rancher/docs/pull/3166 was merged


OrangeHat - Assembly and Testing

Assemble and test the OrangeHat step by step.

[x] PCB #18
[x] Power Delivery
[x] Microcontroller
[x] Raspberry Pi
[x] Servo #22
[x] Display
[ ] IMU
[ ] RS485 Communication

Power Delivery

2021-09-18 Programming.

Powering the servo causes brownout #22  #24

Demo OrangeHat installed on Maze Runner


Added learning page and updated index.html of doc

Issue Number:
152
Issue Description:
Build an html page in the docs directory to provide links to helpful learning resources in an organized manner. Add a link to the index.html page in the docs directory leading to the new learning resources page.
Summary of solution:

Added a link to index.html
Created a new learning page
Added resources

Can this issue be closed?
Yes
Should any new issues be added as a result of this solution?
No
Have you named your branch in a descriptive way? Remember to name your branch in a unique and descriptive manner in order to properly reflect the issue or feature.
Yes
Thanks for contributing!

This is great Thanks!


Invalid regular expressions in grammar

👋 from github-linguist/linguist.
I'm in the process of creating the next release and I've seen our grammar compiler has detected the grammar has two invalid regular expressions which may impact the highlighting of some Nushell files:

repository vendor/grammars/nu-grammar (from https://github.com/hustcer/nu-grammar.git) (2 errors)

Invalid regex in grammar: source.nushell (in grammars/tmLanguage.json) contains a malformed regex (regex "(?<!\w)(?:(\^)|(?![0-9]|(?<=\)|\...": lookbehind assertion is not fixed length (at offset 42))
Invalid regex in grammar: source.nushell (in grammars/tmLanguage.json) contains a malformed regex (regex "(?<=\)|\}|\]|\$\w+)(?:\.(?:[\w-]...": lookbehind assertion is not fixed length (at offset 18))

I suspect these were reported in the output when you added the grammar to Linguistin your original PR but likely missed it.
I won't hold back the release for this to be fixed, but I will be adding this to the list of known grammar issues: https://github.com/github-linguist/linguist/issues/3924. Any fixes you make to this grammar will be pulled in when the next release is made.

OK，thanks, but how to detect invalid regular expressions for grammar?

The regex works in ECMAScript (Javascript) but doesn't work for PCRE and others. That's why it works in vscode but can't work somewhere else.
I'll try to work around this issue to make something which morks for PCRE.

@glcraft Thanks, the grammar file for linguist has been updated just now

Should be closed by https://github.com/nushell/vscode-nushell-lang/pull/137


Finetuned Weights Loading Error

Hello! I trained a LLM model and saved my finetuned weights.  When I tried to load the weights, the following error occurred.
RuntimeError: Error(s) in loading state_dict for HF_LLM:
        Missing key(s) in state_dict: "_LLM_model.encoder.embed_tokens.weight", "_LLM_model.decoder.embed_tokens.weight".

Could you give me some advice on how to solve this problem? Are there some ways to save the LLM model weights in the form of Hugging Face?

Hi,
Could you give more info on how you trained and saved weights please? Did you use one of the examples?
Our examples only save parameters with "requires_grad=True" and it may be possible that the two missing keys you show were not saved as they are frozen (i.e. they have "requires_grad=False"). If that's the case, please try setting , strict=False when loading the finetuned weights.

Awesome! I try to set strict=False and successfully load my finetuned weights. Thanks for your advice.

Perfect!
I am closing the issue. Please do not hesitate to open an other one (or repoen this one) if you face other troubles.


feat: blink.cmp source

Is your feature request related to a problem? Please describe.
The current completion system uses the built-in completions and does not support external completion engines
Describe the solution you'd like
Support for external completion engines such as blink.cmp and/or nvim-cmp
Additional context
Hey there, love the plugin! It's getting me to finally switch off my shell filled with aliases :smile: I'm the author of blink.cmp and was curious if you'd accept a built-in source for it. I believe I'll need to make some changes on blink's side which I'm more than happy to do.

Hey there, love the plugin!
Seen blink.cmp around but never had the time to test it out, just did and liking it a lot!
Would the end results be that we have an optional dependency to an external cmp engine but if not, we fallback on the built-in one?
I have not looked at the code for cmp sources so not sure what changes needs to be done on our side but I like the idea and let me know if you need me to do anything!

i think it's wise.
if you plan on implementing it, i think it must come with support for the most popular (nvim-cmp) and yours at the same time.

i think it's wise. if you plan on implementing it, i think it must come with support for the most popular (nvim-cmp) and yours at the same time.

If it's an optional dep with fallback on the current built-in I would say it's enough to make the implementation extensible so that nvim-cmp can be added separately


just reporting an error - Uncaught Error: code length overflow. (1756>1056)

this will generate an error. go here: https://codepen.io/davidshimjs/pen/NdBYrg
and enter the below

https://sdasda-asdasd-in-uat.appspot.com/?bid=0006300GG-UCA01&title=G-UCA01&name=Faculty of Information Technology&z=1&lat=-37.910976965731614&lng=145.13378548592536&fid=12143&cid=159&pid=1154825&buildingId=3387

error message:
qrcode.min.js:1 Uncaught Error: code length overflow. (1756>1056)
at Function.b.createData (qrcode.min.js:1)
at b.makeImpl (qrcode.min.js:1)
at b.getBestMaskPattern (qrcode.min.js:1)
at b.make (qrcode.min.js:1)
at QRCode.makeCode (qrcode.min.js:1)

Same here...

More details:
Does not work for 192 >= length >= 217

any update about this issue ? mine also got
Uncaught Error: code length overflow. (1604>1056)
at QRCodeModel.createData (qrcode.js:105:54)
at QRCodeModel.makeImpl (qrcode.js:90:54)
at QRCodeModel.getBestMaskPattern (qrcode.js:91:479)
at QRCodeModel.make (qrcode.js:88:127)
at QRCode.makeCode (qrcode.js:584:17)
any suggestion how to increase this code length? which part of qrcode.js / line number ?

Same here...

More details: Does not work for 192 >= length >= 217

I found that length 218 doesn't work, but 219 does.


Add List types for console API

What this PR does?
This PR adds missing registration for List types for console API.
It looks like a bug for me but please correct me if I'm wrong.

/ok-to-test

/lgtm

/assign @bparees
Just for approval.

/approve


can add the same name and same address to address book many times, even on accident.

Steps to reproduce
scan a QR
add it to the address book
screen goes back to the screen where there is the add button
add it again
add it again
now there are many versions of the same name and address.
Why would a user click the button more than once? because it is there, maybe they thought the first time is a bug or it didn't save, or they hit the wrong button so they try again.
Current behavior
the "add" button should not be clickable if an address is already added. If anything it should be an "edit" button.
Expected behavior
there should not be multiple entries for the same name and address.
If there is an entry with that name, then an error says that name already exists, if it is an address that is duplicated then there should be an error that address is already added, if both like this, then an error of one, both, or the other.
Affected Version
latest version of iOS

@jabo38 @cryptoBeliever fixed.

ok.


[Feature] ChessEnv

Stack from ghstack (oldest at bottom):

-> #2641

@kurtamohler I landed all the stack up until here.
If you want to give a shot at this feel free to do so!
You seem like a more skilled player than I am so feel free to edit things as you think it's best, I'll be happy to give it a look later!
Also we don't have tests but it'd be worth having some.
For the record here is the random forest maker I showed the other day
from torchrl.data import MCTSForest, Tree
import torch
from torchrl._utils import timeit
from torchrl.envs import ChessEnv
from tensordict import TensorDict, LazyStackedTensorDict, NonTensorData
env = ChessEnv()
f = MCTSForest(observation_keys=["hashing"], max_size=1000000,
               excluded_keys="fen")

def make_random_forest():
    r = None
    reset = None
    i = 0
    while r is None or r.shape[-1] > 1:
        with timeit("rollout"):
            r = env.rollout(1000, tensordict=reset, auto_reset=reset is None)
        # r_ext = r.exclude("fen", ("next", "fen"))
        with timeit("extend"):
            f.extend(r)
        if i == 0:
            idx = 0
        else:
            idx = torch.randint(r.shape[-1], ())
        i += 1
        reset = r[idx]
    timeit.print()
make_random_forest()
t = f.get_tree(env.reset())
t.plot()
# etc

If you want to give a shot at this feel free to do so!

Ok I'll give it a shot!
It might be easiest if we merge this as-is first and then I can make a new PR with updates. I looked into whether ghstack has collaboration features, and it looks like it doesn't

Done! Thanks for that


Different parse results with the same number

flutter_libphonenumber: ^1.2.4
When parsing a number with no regionCode, for example<PHONE_NUMBER>9 (An Egyptian number), sometimes it returns +201114377479, other times it fails to parse. Why would there be this inconsistency, how can i help?

Anybody has a hint on where to investigate ? Is it a google libphonenumber issue?


Getting "UnboundVariable: ': unbound variable'" while performing ingestion

Ingestion type: Snowflake
The ingestion yaml file is has the keys mentioned in the guide.
source:
  type: snowflake
  config:
    # Coordinates
    host_port: ****
    warehouse: "*****"

    # Credentials
    username: ****
    password: ****
    role: "****"

    # Options
    top_n_queries: 10

sink:
  type: "datahub-rest"
  config:
    server: "http://localhost:8080"

However I keep getting this error. Seems like I may be missing something. I also verified that the yaml is fine syntax wise. In the error log, I do see a bunch of env variables being assigned to values properly. I'm running this on a macOS Catalina via Docker.
UnboundVariable: ': unbound variable'

Thanks!

This can be closed. Thanks to this post in the datahub slack channel (https://datahubspace.slack.com/archives/CUMUWQU66/p1629381251390600?thread_ts=1629359928.380100&cid=CUMUWQU66)
My password had a $ character. Escaping it with a \ solved the issue.


Improve the annotation assistant to become a query builder assistant

Now that the annotation assistant is quite robust, the menu should be improved to become a general Query Building Assistant:
Instead of having all annotation foundries at the first context-insensitive level, the query builder assistant may have more general sections on the first level:

Annotations
opens the helper as it is now to add annotations to the search bar
Queries
opens bookmarked or older queries and macros to add to the search bar
Operators
opens a list of Operators (depending on the current query language) like
Query-Builder plugins may in the future be added to the first level as well.

Operators may be listed like

Or-Operator: A | B
Let either A or B be true
-> will add |
Positional Operator: startsWith(A, B)
Only find occurrences of B at the beginning of A
-> will add startsWith([], [])

By introducing containers, the annotation assistant now is able to store and retrieve query fragments.


Create "tasks" report or daily email

List of:

Carts with all items ready to ship but has ot been shipped

Carts with all items ready for pickup but has not been marked as ready for pickup

Cart lines that should have been in stock (I think not is back or preorder will work) to pick and mark ready.

If this takes too long to load on a page, then generate a report and email it.

Done in 2ee9993255a118d0e9c71b130569ed0be029427d and 433c01599488e1250deb6c90766a2a7701e565f9 with the exception of a placeholder for the release date of pickup orders


Remove redundant label on activity

Closes #325.

Unless I'm missing something, the label doesn't seem unnecessary as it's on the main launcher Activity.
I'm closing this, but feel free to reopen if the above isn't true.

Reopening bc I re-read the description in the issue you opened. I'll find out what the best practice is on this and let you know.

Reopening bc I re-read the description in the issue you opened. I'll find out what the best practice is on this and let you know.

Thanks. I would like to mention that there is a RedundantLabel lint rule for this and has a 5/10 warning priority.

Thanks for spotting and fixing this. I can confirm that the lint check in Android Studio states that this label is redundant.


V 2.0.0

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             master     #4   +/-   ##
=======================================
  Coverage       100%   100%
- Complexity      106    118   +12
=======================================
  Files             2      3    +1
  Lines           221    253   +32
=======================================
+ Hits            221    253   +32

Impacted Files
Coverage Δ
Complexity Δ

src/Bindings/MySQLBindings.php
100% <100%> (ø)
62 <62> (?)

src/Statement.php
100% <100%> (ø)
42 <4> (-50)
:arrow_down:

Continue to review full report at Codecov.

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


[NyaaTorrentsBridge] Add custom fields

This will fix https://github.com/RSS-Bridge/rss-bridge/issues/3418 and add all custom fields from the nyaa-torrents rss feeds. The problem are the : in the xml fields which lead to the whole fields getting removed by simplexml_load_string(). This is fixed by removing nyaa: from the feed result and then parsing everything as before.
@ORelio Feel free to comment or suggest changes as well!

A feed item will now include the following custom fields:
{
  "...",
  "_rssbridge": {
    "seeders": "0",
    "leechers": "6",
    "downloads": "0",
    "infoHash": "6fc9a10a9bd3e992aa3cc5041f5dcc4b575533aa",
    "categoryId": "1_4",
    "category": "Anime - Raw",
    "size": "81.9 GiB",
    "comments": "0",
    "trusted": "No",
    "remake": "No"
  }
}

as far as i understand, this will only affect the json output format?

I added the nyaa specific torrent id as well, so people can further use it to build e.g. the torrent download link and they do not need to fetch it from the url themself. Tihs would conclude the PR for me (if the linting won't again fail of course)

@dvikan I guess we can merge this now, as long as you are also fine with it, too.

Good job, @Jisagi 👍

there is an issue with this pr with the call to $this->collectRss2($rssContent, self::MAX_ITEMS);
it prevents me from refactoring FeedExpander.
this bridge is pretty close to not even needing to extend from FeedExpander. But it needs to parse xml i understand that.
can something be done?

We could also just hardcode the 20 as it was before this pr. I do not see any reason to keep it as constant.

See https://github.com/RSS-Bridge/rss-bridge/pull/3742

If that solves the problems you had with the FeedExpander, that would be perfect. I didn't know $newItem still has access to the custom namespace fields.


Create free-competitive-programming-book.md

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)
How do we know it's really free?
For book lists, is it a book?
Checklist:

[ ] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

duplicate


Add BEM to example gallery

Currently, in two tests we (only) create BEM surfaces, which end up in the respective FreeSurfer derivatives directory; no Report can be generated based on this. Therefore, these two tests cannot be used as examples in our example gallery. We should change that.

@hoechenberger I don't get it. If you have bem surfaces the report should be able to overlay them on the T1 MRI.

@hoechenberger I don't get it. If you have bem surfaces the report should be able to overlay them on the T1 MRI.

The issue I meant to point out here is this:
If one only runs the BEM-creation step and doesn't process any other parts of the data (i.e. doesn't touch the electrophysiological data at all), Report.parse_folder() won't find anything and will crash. Therefore, two of our examples where we only create BEM surfaces from T1 and FLASH images, respectively, don't show up in our examples gallery. But I suspect that's okay – we still have our ds000248 example, where we create the BEM surfaces AND process MEG data. Closing this issue.

The issue I meant to point out here is this:
If one only runs the BEM-creation step and doesn't process any other parts of the data (i.e. doesn't touch the electrophysiological data at all), Report.parse_folder() won't find anything and will crash. Therefore, two of our examples where we only create BEM surfaces from T1 and FLASH images, respectively, don't show up in our examples gallery. But I suspect that's okay – we still have our ds000248 example, where we create the BEM surfaces AND process MEG data. Closing this issue.


How to use light control with GPIO and voice

I needs to control the light by voice with GPIO and relay. How can I do ??

How to use light control with GPIO and voice
I needs to control the light by voice with GPIO and relay. How can I do ??


[Dubbo-3653] etcd as config center

What is the purpose of the change
This pull request adds the etcd support as config center. #3653
Brief changelog

Support get dubbo.properties from etcd
Support listen to updates to configurator, routers, and etc.

Implementation note:
The jetcd library(currently 0.3.0) does not support cancel a watch, which I've filed an issue here, so I choose to implement it with lower-level grpc api provided by etcd.
Verifying this change
Verified using this sample.
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #3663 into master will decrease coverage by 0.06%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #3663      +/-   ##
============================================
- Coverage     63.94%   63.88%   -0.07%
  Complexity       71       71
============================================
  Files           699      699
  Lines         30757    30768      +11
  Branches       4989     4991       +2
============================================
- Hits          19669    19657      -12
- Misses         8824     8842      +18
- Partials       2264     2269       +5

Impacted Files
Coverage Δ
Complexity Δ

.../apache/dubbo/remoting/etcd/jetcd/JEtcdClient.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../dubbo/remoting/etcd/jetcd/JEtcdClientWrapper.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ubbo/remoting/etcd/support/AbstractEtcdClient.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ache/dubbo/remoting/p2p/support/AbstractGroup.java
45.45% <0%> (-11.37%)
0% <0%> (ø)

.../apache/dubbo/remoting/transport/AbstractPeer.java
63.04% <0%> (-8.7%)
0% <0%> (ø)

...ng/exchange/support/header/HeartbeatTimerTask.java
73.68% <0%> (-5.27%)
0% <0%> (ø)

...dubbo/rpc/protocol/dubbo/CallbackServiceCodec.java
76.47% <0%> (-2.21%)
0% <0%> (ø)

...dubbo/remoting/exchange/support/DefaultFuture.java
70.94% <0%> (-2.03%)
0% <0%> (ø)

...he/dubbo/registry/multicast/MulticastRegistry.java
67.87% <0%> (-1.81%)
0% <0%> (ø)

...apache/dubbo/rpc/protocol/dubbo/DubboProtocol.java
65.78% <0%> (-1.13%)
0% <0%> (ø)

... and 5 more

Continue to review full report at Codecov.

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

I will check this week.

You need to add dubbo-configcenter-etcd dependency into this file https://github.com/apache/incubator-dubbo/blob/master/dubbo-all/pom.xml, otherwise, it will not be packaged into dubbo-all.jar.

I found another problem. If I shutdown etcd server, the connection will be lost. And when I start etcd server again, and modify the key, the listener will no longer be notified.

I have fixed the issue I found before, the new implementation will watch again when connection is established.

LGTM. My review focus on EtcdDynamicConfiguration.
I checked the following by running demos locally:

can recover listeners after reconnect
can get notified whenever updating or deleting happens
other parts in EtcdDynamicConfiguration looks ok

@zonghaishang I merged this pull request, but feel free to drop your review comments.

how we use this? where is the etcd sample ?


asyncData not called after vue-router hook beforeRouteEnter is called

In order to have a completely separate Nuxt page for search versus our homepage, we implemented a redirect if the query string parameter q is present. We use vue-router’s beforeRouteEnter hook to implement the redirect:
  beforeRouteEnter (to, from, next) {
    if (typeof to.query.q !== 'undefined') {
      next({ name: 'domain', query: to.query, replace: true })
      return
    }
    next()
  }

Everything works great, except, on the server, asyncData isn’t called by the domain route. It otherwise loads and renders correctly, sans asyncData.

Dug into it a bit further, and it turns out asyncData is called for the first component, but not the component (page) that ends up being rendered.

Update 2: I tried replacing the component (page)-level beforeRouteEnter hook with a global router.beforeEach hook. The same thing happened (asyncData on the first component was called).

Could you instead use a middleware inside your page (it can be a function) and use redirect instead? router.beforeEach does not work well on server-side.

The source and destination URLs differ only by query string. The router will resolve the first page after the redirect.

I see, so I suggest you to use instead one page but 2 Views components inside and use <component :is="viewComponent"/> inside your template depending if the user is searching or no.

Is this related to #2737 ?

After some debugging it turned out that asyncData gets called once again on params update, but new result from asyncData isn't persisted in data later.

I've tried to solve this problem like this:
Both asyncData and beforeRouteEnter can't access component instance, so I created a mixin that keep and restore the previous query in module scope by path.
https://gist.github.com/comfuture/2d40d2cc1d659614ec982a7613e0fe38
I think this issue is critical. @ydnar Please refer to this mixin to solve your problem.


Strap line

The current strap line is:
requirex is not only a zero-configuration module loader for browsers and Node.js. It's also a revolution
It feels a bit clunky, how about something like:
Web development, simple again
or
Web dev for notepad users
or
JavaScript, without the s**t

Maybe
Web Dev made grokable

Web Dev without node_modules


final_schedule

👷 Deploy Preview for rayko-research processing.

Name
Link

🔨 Latest commit
c8113fc7f76e10b39eb6a90f4a5be54b6ff3ca14

🔍 Latest deploy log
https://app.netlify.com/sites/rayko-research/deploys/666aad3a6900f10008537286


Feat/test env package replacement

Replacing older testing environment setup functions with @inrupt/internal-test-env
Adds the getNodeTestingEnvironement call to build the matrix version of the test we are running to ensure using the standardized Environment variables across multiple environments.
Checklist

[x] All acceptance criteria are met.
[x] Relevant documentation, if any, has been written/updated.
[x] The changelog has been updated, if applicable.
[x] New functions/types have been exported in index.ts, if applicable.
[x] New modules (i.e. new .ts files) are listed in the exports field in package.json, if applicable.
[ ] New modules (i.e. new .ts files) are listed in the typedocOptions.entryPoints field in tsconfig.json, if applicable.
[x] Commits in this PR are minimal and have descriptive commit messages.

@NSeydoux addressed the sonar scan issue


get-msisummaryinfo : A share violation has occurred

After installing from PSGallery I am getting this error when trying to use get-msisummaryinfo. I'm not sure what this means. Is there a step I am missing?
PS > Install-Package msi

Name                           Version          Source           Summary
----                           -------          ------           -------
MSI                            3.2.33           PSGallery        Exposes Windows Installer functionality to Windows ...

PS > get-msisummaryinfo
get-msisummaryinfo : A share violation has occurred
At line:1 char:1
+ get-msisummaryinfo
+ ~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotImplemented: (:) [Get-MSISummaryInfo], PSNotSupportedException
    + FullyQualifiedErrorId : NotSupported,Microsoft.Tools.WindowsInstaller.PowerShell.Commands.GetSummaryInfoCommand

A path argument is required but doesn't seem like it's enforced for parameter binding. While technically it's a breaking change, since this invocation only errs it should be okay to make a breaking change to require at least one argument.
To see how to use the cmdlets, please run help <cmdlet> or help <cmdlet> -full to see more detailed help including parameters.

Thanks, I'm used to help being displayed when no arguments are passed. This is traditionally how commands behave. "A share violation has occurred" doesn't explain that an argument is required. It sounds more like a bug in the code.

PowerShell doesn't work that way, unfortunately. You can pass -? as a sole argument which PowerShell translates to help <cmdlet>, but that's undocumented. Normally, help <cmdlet> or help about_<topic> is what you need to do. You can also run help about to see all available about topics.
So, just to be clear, this cmdlet (or any cmdlet, unless they handle it themselves which would be non-standard) will not display help when no arguments are passed.
Still, I can fix this specific problem since it shouldn't fail that way. It (and all other cmdlets) will prompt for required arguments, though.

Note to self: this happens with several cmdlets. Was there some reason why or was it just an oversight?

Get-MSIFileType
Edit-MSIPackage (one sharing violation, but then enumerated files in $PWD which is also incorrect)
Get-MSISummaryInfo

Digging in more, I'm going to leave all but Get-MSISummaryInfo because the others are designed to enumerate all files if no parameters. Not sure why I was getting the sharing violation on one machine, but was unable to reproduce on another.

PowerShell Gallery has been updated with MSI version 3.3.4 with a fix for this specific issue. If -Path isn't specified, PowerShell will prompt (unless started with -NonInteractive). Bear in mind, however, that cmdlets don't emit help if you don't pass required parameters. Use help <cmdlet> for more information.
Thanks for reporting!


Association edge_factory

A really weird thing that happened during development of Association factory:
Normally we define a special factory function and try to reuse the generic_factory, since this already gets almost everything done. Most special factories just deal with the labels that should appear.
With associations in class diagrams it's a bit different:
Here we have to deal with multiple(I think it's at maximum 2) labels that are specifying the rolenames. I tried following changes to the aird.diagram.Edge:

Edges now have labels that default to an empty sequence.
This wasn't explicitly handed over in the generic factory and caused repetition of all preceeding edge.labels in the newly constructed edge... almost like the edge wasn't constructed freshly, just reused.
I'd really like to understand how this could happen in my favourite programming language.
What fixed the issue:

labels: t.Sequence[Box] = [],

Here's the issue. If you use a mutable default value as function argument, the modifications persist, because it's the same list that gets passed in every time the function/method is called.
Instead, use None as the default and use something similar to this at the start of the function body:
if labels is None:
    labels = []

(It might be a good idea in general to enable pylint automatic checking, because IIRC it actually warns you about this exact issue.)


.off and .destroy fn's broke?

var monitor = scrollMonitor.create(someElement);
.on('enterViewport', onEnterFunction); //works great
.off('enterViewport') //doesn't work  //keeps calling .on('enterViewport', onEnter);
Same with destroy - doesn't work.
Thanks for any feedback

you have to tell it which callback you want to remove, it doesn't clear them all out like jquery does.
You can monitor.destory() if you want to destroy it.

So I have to pass in the onEnterFunction to the .off??
Got it thanks!

@stutrek
Hey I appreciate the feedback so far - just wanted to let you know that the above is still not working properly. - Off does so I'm good to go, thought i'd leave this here however.
   var monitor = scrollMonitor.create(someElement);     //monitor.off('enterViewport', onEnter); //works     monitor.destroy(); //doesn't work


how can I make a unique layer? is there such a possibility?

is it possible to assign some parameter to the layer so that it becomes unique and is used in the collection only once.

I'm sorry, but i think this feature is not implemented


The system cannot find the specified file

I am using python 3.x and never find the file. I am sure that my file is in the correct directory. what am I doing wrong?

Hi,
This issue is quite old but as I faced the same problem recently, I'll post my answer:
The file Windows doesn't find is probably the ffmpeg executable, not your audio file. In the 101st line of the script, in the convert_to_wav() function, a subprocess calls ffmpeg to convert the data to a .wav file. Here the argument is litteraly ffmpeg, meaning that the ffmpeg executable must be in your PATH.
From here you have two solutions:

add ffmpeg to your PATH (here is a tutorial on how to do that on Windows),
or modify the first argument of the subprocess function from ffmpeg to /your/path/to/ffmpeg/executable/.

I hope it will help you or anyone having the same issue !


ph7 engine added

Unfortunately, Ph7 looks like inactive now. I think despite Zend has a lot of problems, it is still the best choice for us. And I decided to support PHP-ng.


[New-Feature] Allowing for model weights to be loaded from the art.Classifier class

Description
Added a way to allow for loading model weights for art.classifer.Classifier.  This has the same type signature as the save and provides symmetry.
Fixes # (issue)
Type of change
Please check all relevant options.

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

Testing
Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.

[ ] Test A
[ ] Test B

Test Configuration:

OS : GNU/Linux (Ubuntu 18.04)
Python version: Python2.7/Python3.7
ART version or commit number: 1.0.0
TensorFlow / Keras / PyTorch / MXNet version: Pytorch version 1.0.0

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Please hold on pulling this one.  I broke the rest of the unit-tests barring the Pytorch one due to the addition of a new method in the ABC Classifier.

@AkashGanesan Are you still working on this?

Closing this, as inactive for long time. Please feel free to reopen it if you get the chance to work on the feature again.


Confusion about the data preprocessing

As far as I understood preprocessing steps for snare-seq are
atac-seq dataset-> cistopic -> unit normalization
rna-seq dataset-> unit normalization -> PCA-10 components
Is that correct?
I know this is a general machine learning question, but what did you use to choose the number of components when doing PCA for a different dataset? Which tool/settings do you recommend?

Hi!
For epigenomic datasets, such as scATAC-seq and scMethyl-seq, we use cisTopic when preprocessing our data. You can try a range of dimensions and cisTopic has a funcionality to choose the one with the highest likelihood.
For PCA, we checked the percentage of variance explained by the chosen number of dimensions and made a decision based on that (e.g. >=75%).
For both, our process was as follows:
data --> cisTopic/PCA --> unit normalization, so we applied unit normalization after dimensionality reduction for the datasets in our paper.
I hope this was helpful. Let me know if you have any questions :)

thanks, that makes it clear :)


Nifi 6583

Description of PR
NIFI-6583: Enables the feature that forwards nifi metrics to azure log analystics.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ X] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[X] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Any update on my pull request? The build fail with FR generated from web page. Please let me know what I am missing.


cquery: 2018-05-01 -> 2018-08-08

Motivation for this change
Update to latest version
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg eval

@GrahamcOfBorg  build cquery

Success on x86_64-linux (full log)
Attempted: cquery
Partial log (click to expand)
one
Enasdfd of search list.
asdfasdf
[doctest] doctest version is "1.2.7"
[doctest] run with "--help" for options
===============================================================================
[doctest] test cases:     69 |     69 passed |      0 failed |      0 skipped
[doctest] assertions:    430 |    430 passed |      0 failed |
[doctest] Status: SUCCESS!
/nix/store/hg1c0vfm0nxn8v1v0dyghlsssqf648kp-cquery-2018-08-08

Success on x86_64-darwin (full log)
Attempted: cquery
Partial log (click to expand)
one
Enasdfd of search list.
asdfasdf
[doctest] doctest version is "1.2.7"
[doctest] run with "--help" for options
===============================================================================
[doctest] test cases:     69 |     69 passed |      0 failed |      0 skipped
[doctest] assertions:    430 |    430 passed |      0 failed |
[doctest] Status: SUCCESS!
/nix/store/1fwpadq8ivmc9wpyzqd54k0k0j35z489-cquery-2018-08-08

Success on aarch64-linux (full log)
Attempted: cquery
Partial log (click to expand)
one
Enasdfd of search list.
asdfasdf
[doctest] doctest version is "1.2.7"
[doctest] run with "--help" for options
===============================================================================
[doctest] test cases:     69 |     69 passed |      0 failed |      0 skipped
[doctest] assertions:    430 |    430 passed |      0 failed |
[doctest] Status: SUCCESS!
/nix/store/bvv5z50cqpmd4z8l21xpzd1z1v2ppbv5-cquery-2018-08-08


Summarise approach and decisions for internationalisation in the Design System backlog

What
Write up a summary of our final approach for internationalisation and why we decided to follow that approach. Might also be a good opportunity to talk about the other things we tried and explain why those don't work. Also anything we've decided not to do (e.g: because it's not part of our MVP solution; because it's out of scope).
Why
We should try and be transparent about the decisions we make, to help people understand why we've done things a certain way.
This may also help as somewhere to refer people to if we get questions on support.
Who needs to work on this
Content, Developer
Who needs to review this
Content, Developer
Done when

[ ] On Github, add backlog comment in backlog, e.g https://github.com/alphagov/govuk-design-system-backlog/issues/99

From mid-sprint review:

Look through Google Drive for relevant documentation related to i18n. Pull information from docs we think will be useful to users
Check whether we have guidance in internal docs about how to translate components (if not, this feels like it's own card tbh)
Use decision summaries published previously as a template for recording this information (like Cookie Banner)

@claireashworth

This reads really well to me, thanks for sharing @claireashworth. @romaricpascal please can you also review.

That was some thorough gathering of all the I18n related issues! Cheers @claireashworth! Added a couple of comments on the doc so we can clarify a few things and then we should be in a good place.

I think the backlog issue makes as much sense as anywhere.
I have added a few (hopefully minor!) comments to the Google Doc, but generally think it's looking really good. Thanks for doing all the hard work to pull all of that together, @claireashworth! 🙌🏻

Comments completed and summary published on #99 'Supporting multiple languages'.


[Question] Which framework should I use for the module management/system to use MEAN.js

Hello I want to get into full stack web development so after making some research on frameworks (for web dev. obviously), I saw that the MEAN.JS (not the .io one) stack (MongoDB+Express.js+Angular 2+Node.js) was the way to go especially as I'm more oriented Front-End so DHTML (HTML+CSS+JS) over PHP&MySQL and that I'm not used to the LAMP stack (Linux+Apache+MySQL+PHP).
The ick is that there's several frameworks/libraries which take care of the module management/system:
-RequireJS (AMD)
-CommonJS (used by Node.js)
-SystemJS (used by Angular here: https://github.com/gocardless/angularjs-style-guide#third-party-dependencies)
-Browserify.js
-Webpack
And ES6 (official name ECMAScript 2016 aka JavaScript v6) has it's own module management/system
So I'm confused wether to stick with just ES6 (for the module management/system) or go for one of those frameworks to complete the MEAN stack.
What are the pros and cons of each of them and which one would have the best pro/con ration regarding their use with MEAN.js and what would you recommend.
Thanks in advance :).
Also (if that may help you understand why I'm asking all of this), I'm going to learn ES6, Bootstrap and probably jQuery despite:
-not being recommended by Angular developpers and that: http://dancancro.com/meanio_vs_meanjs.html
-I have own library that does mostly of what jQuery does (with a lot of other stuff too)

If you look in the config directory, you will see that the files are required() using Node.js (CommonJS), however you should be able use what suits you.
You'll probably have to do your own research on which is better than what - it will be different for each dev.
In Meanjs there's already a system in place to require the modules that you want and it works automatically if you follow the folder conventions. Look inside: https://github.com/meanjs/mean/tree/master/config/assets
to get a feel for what's needed.


#78 Increase playwright test timeout to make CI more reliable

Describe your changes
Increase playwright test timeout to make CI more reliable. This is going to make them slower as well.
We should fix the underlying issue.
Issue number
Closes #78
Checklist before requesting a review

[x] Performed a self-review of my code
[x] Formatted my code with pkgmt format
[x] Added tests (when necessary).
[x] Added docstring documentation and update the changelog (when needed)

Will this work in case of intermittent failures? I think the time-outs happen in these failure cases, and it's only going to increase the time for the CI to fail.

@neelasha23 Yeah it is unlikely to work, just wanted to try, was not sure how to run the CI without making a pull request

@neelasha23 it didn't work, will need to take a different approach https://github.com/ploomber/jupysql-plugin/actions/runs/6401180754/job/17378192452


Possible to disable the zoom out effect on menu pressed?

I want to know if it is possible to disable the zoom out effect when pressed on the menu button.

is it possible?


Cannot delete service from Table view of Service Catalog

Description:
Cannot delete service from Table view of Service Catalog. This works in Grid view.

Description:
Cannot delete service from Table view of Service Catalog. This works in Grid view.

What BD you aré using? If you use H2 recomend delete from table

Reduced priority for beta.


Numbers of peers past 1000 get cut off in the toolbar

Describe the bug
Any number of peers past 1000 gets cut of in the toolbar
To Reproduce
Steps to reproduce the behaviour:

Connect to over 1000 peers (Enabling AcceleratedDHTClient and DHT server mode helps with that)
Look at the extension icon

Expected behaviour
The number does not get cut off
Screenshots

Desktop (please complete the following information):

OS: macOS 12.0 Beta (21A5294g)
Browser: Firefox
Version: 92.0a1 (2021-08-05) (64-bit)

If we truncate it to 1.7k it takes the same amout of space and the cutoff happens.
I think rounding it to 1k be acceptable, as there is not much more we can do.
The line where you want to do the rounding is:
https://github.com/ipfs/ipfs-companion/blob/7aba6d9d436fe7a4c2ae62223cefe6ccc454a2bb/add-on/src/lib/ipfs-companion.js#L510
PR welcome :)

I can take this one!


Feature: Automatically add authors name and book description using api

Feature description
When the user enters the name of the book the authors name and book description will automatically fetched and added in the text field using an API.
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

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

I want to work on this issue under GSSoC'22

@mustali17  The back-end of this page is ready yet , if done will let you know .

@mustali17  Assigned , for any problem reach out to us on discord .

@mustali17  May I  get an update on the issue .

Hey, I am a contributor in GSSoC 22. I would like to work on this issue.
The API that we are talking about here is it already available in the backend folder? or do I need to create a custom API in the backend folder first? or can I use any online available API (if I can find one)

@mayonk-kumar-git Assigned , for any issues reach out to us on discord.


Specify heroku credentials via ENV var

As a user of the heroku gem I want to specify my username and API key via environment variable so that I can perform automated deployments from an environment on which I cannot easily set ~/.heroku/credentials.
It is not always appropriate to rely on the ~/.heroku directory to specify the current set of heroku credentials a command should use. For example a continuous integration environment might be deploying multiple heroku apps, each with different credentials, concurrently. Worse the build agents for a CI system may be on remote instances or VMs and it should not be necessary to install a credential set on all build agents in order to safely perform an automated deploy.

Please see https://github.com/heroku/heroku/pull/208

See #208


Publish Epoch in Nuget.org

Would be very useful if we can install epoch from nuget.org repository.

upvote

I agree. This would be very helpful.


Unexpected blinking!

I'm using the latest version of the plugin in attempts to create a demo version of the button, which, in the case of long text, the font size is reduced by the given algorithm. But instead of the expected result, I see...
https://codepen.io/7iomka/pen/RMyzyq

Looks to me like you have created a loop, rethink your queries/steps and it will work.

@7iomka You could break the loop by giving the btn an explicit width, rather than having its width shrink-to-fit the size of its descendants: https://codepen.io/eeeps/pen/NYzYdm
The loop happens because you're querying the same thing you're conditionally styling. The btn__content font size shrinks, and so it’s shrinkwrapped btn parent shrinks, which makes the query false, which makes the btn__content font-size grow again, which makes the btn grow again which makes the query true, which...[repeat forever]

eeeps, thanks for explanation and example! It works fine! But!
I just wanted to use this plugin for buttons as independent components, the width of which I would not want to limit, I just use only paddings and font-size.
I thought this plugin would help me do the following:
I want the buttons that have the size of content, for example, do not fit 300 pixels, changed their font-size to the one I set, in which the content would already fit in one line. Naturally, I would do this only on mobile breakpoint. This is a kind of improvement of the UX of an independent component, which I wanted to do every time not to make an exception in the form of a hardcode over a certain button, the content of which was too long to fit on a certain width of the mobile :(


Update Site settings “info”

Automatically generated by Static CMS

👷 Deploy Preview for dainty-llama-34c066 processing.

Name
Link

🔨 Latest commit
44887cd5776ce206a61922e10cf7103d9049247c

🔍 Latest deploy log
https://app.netlify.com/sites/dainty-llama-34c066/deploys/661a3443e1e971000803a724


BUG: Quaternion output (r.as_quat()) is wrong

Describe your issue.
Hello.
I'm not sure whether it is bug or not,  sorry for putting in Bug report.
if it is not right position , I will change to category.
I'm using scipy's rotation function well, and found out that output of quaternion's output is not neither Hamiltonian nor JPL.
The document of scipy says that
scipy.spatial.transform 's rotation.as_quat() should give quaternion as Hamiltonian, which is (X,Y,Z,W) form,
https://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.transform.Rotation.as_quat.html
but the actual output shows (Y,X,W,Z)
Am I Interpreting wrong? or is it truely mistake from scipy?
Thank you in advanced
Reproducing Code Example
import numpy as np
from scipy.spatial.transform import Rotation as R

r = R.from_matrix([[0, -2, 0],
                  [1, 0, 0],
                  [0, -1, 1]])

b =  r.as_quat()
print(b)

output :[ 0.         -0.26726124  0.53452248  0.80178373]
which is (Y,X,W,Z)

%But if it is right, scipy should return hamiltonian notation,
[ -0.2672612, 0, 0.8017837, 0.5345225 ] which is (X,Y,Z,W)

Error message
None

SciPy/NumPy/Python version and system information
C:\Users\7700X-2-1\anaconda3\envs\airsim_env\lib\site-packages\scipy\__config__.py:154: UserWarning: Install `pyyaml` for better output
  warnings.warn("Install `pyyaml` for better output", stacklevel=1)
{
  "Compilers": {
    "c": {
      "name": "gcc",
      "linker": "ld.bfd",
      "version": "10.3.0",
      "commands": "cc"
    },
    "cython": {
      "name": "cython",
      "linker": "cython",
      "version": "3.0.8",
      "commands": "cython"
    },
    "c++": {
      "name": "gcc",
      "linker": "ld.bfd",
      "version": "10.3.0",
      "commands": "c++"
    },
    "fortran": {
      "name": "gcc",
      "linker": "ld.bfd",
      "version": "10.3.0",
      "commands": "gfortran"
    },
    "pythran": {
      "version": "0.15.0",
      "include directory": "C:\\Users\\runneradmin\\AppData\\Local\\Temp\\pip-build-env-rikezz0v\\overlay\\Lib\\site-packages/pythran"
    }
  },
  "Machine Information": {
    "host": {
      "cpu": "x86_64",
      "family": "x86_64",
      "endian": "little",
      "system": "windows"
    },
    "build": {
      "cpu": "x86_64",
      "family": "x86_64",
      "endian": "little",
      "system": "windows"
    },
    "cross-compiled": false
  },
  "Build Dependencies": {
    "blas": {
      "name": "openblas",
      "found": true,
      "version": "0.3.21.dev",
      "detection method": "pkgconfig",
      "include directory": "/c/opt/64/include",
      "lib directory": "/c/opt/64/lib",
      "openblas configuration": "USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS= NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= SKYLAKEX MAX_THREADS=2",
      "pc file directory": "c:/opt/64/lib/pkgconfig"
    },
    "lapack": {
      "name": "openblas",
      "found": true,
      "version": "0.3.21.dev",
      "detection method": "pkgconfig",
      "include directory": "/c/opt/64/include",
      "lib directory": "/c/opt/64/lib",
      "openblas configuration": "USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS= NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= SKYLAKEX MAX_THREADS=2",
      "pc file directory": "c:/opt/64/lib/pkgconfig"
    },
    "pybind11": {
      "name": "pybind11",
      "version": "2.11.1",
      "detection method": "config-tool",
      "include directory": "unknown"
    }
  },
  "Python Information": {
    "path": "C:\\Users\\runneradmin\\AppData\\Local\\Temp\\cibw-run-ee1litsm\\cp39-win_amd64\\build\\venv\\Scripts\\python.exe",
    "version": "3.9"
  }
}

@scottshambaugh would you be commenting/interested in looking into this?

Can you try an example with a proper orthogonal rotation matrix? It's not obvious to me what rotation you are expecting from this non-orthogonal matrix.

I tested with valid DCM as follows;
import numpy as np
from scipy.spatial.transform import Rotation as R
r = R.from_matrix([[0.06714201,  0.40047534,  0.91384433],
[ 0.88294712, -0.45038622,  0.13250149],
[ 0.46464646,  0.7979798,  -0.38383838]])
b =  r.as_quat()
print(b)
========================
Quaternion from Matrix
[0.68945025 0.46537892 0.49985143 0.24130759]
and showed correct hamiltonian notation. (X,Y,Z,W). It seems when invalid DCM comes in, output is mixed up.
Thank you for your advice. I will close the issue. Thank you very much.


Contact us page

Link is in the footer, might be worth adding their phone number in the footer too

updated with address and google maps link.

updated with fixes


Broken pipe

I have this error working  on google collab
File "/content/drive/MyDrive/Colab Notebooks/R2Gen/pycocoevalcap/meteor/meteor.py", line 61, in _stat
self.meteor_p.stdin.write('{}\n'.format(score_line))
BrokenPipeError: [Errno 32] Broken pipe
Any help ?

Thank you for your reply.
I still have the same error; I checked the blank in the path.
File "/content/drive/MyDrive/Colab_Notebooks/Medical-Vision-Langauge-Transformer/run_report_generation.py", line 536, in
imageCaptionTaskUniLM(args)
File "/content/drive/MyDrive/Colab_Notebooks/Medical-Vision-Langauge-Transformer/run_report_generation.py", line 502, in imageCaptionTaskUniLM
eval_results = test(model, test_data_loader, args, search_mode)
File "/content/drive/MyDrive/Colab_Notebooks/Medical-Vision-Langauge-Transformer/run_report_generation.py", line 234, in test
MVlBertEval.evaluate()
File "/content/drive/MyDrive/Colab_Notebooks/Medical-Vision-Langauge-Transformer/pycocoevalcap/eval.py", line 60, in evaluate
score, scores = scorer.compute_score(self.gts, self.res)
File "/content/drive/MyDrive/Colab_Notebooks/Medical-Vision-Langauge-Transformer/pycocoevalcap/meteor/meteor.py", line 44, in compute_score
self.meteor_p.stdin.write(eval_line_encoded)
BrokenPipeError: [Errno 32] Broken pipe

What's the args when runing the code? And have you ever print the content to be written?
If both are ok, i am afraid it is the enviroment problem.
Many factors may lead to this error.  You may refer to this issues in the original coco-caption repo How to deal with IOError: [Errno 32] Broken pipe
where these evaluate files are from.

I am using google colab pro
Here is my running arguments
!python run_report_generation.py --batch 32 --conv swintransformer --pretrained --pretrained_path ./.ipynb_checkpoints/swin-rgc-roco-medicat --dataset mimic

It seems it is OK. Try the solutions mentioned in https://github.com/tylin/coco-caption/issues/17
If they still do not work, i am afraid you have to find the solution yourself.


Added content in freeCodeCamp/client/src/pages/guide/english/data-science-tools/scikit/index.md

[ x] I have read freeCodeCamp's contribution guidelines.
[ x] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

@ravish1729 Awesome! Thanks! Congrats on your first open source contribution to this repository!


Use "attrib", not "atrib"

I would say the generally used abbreviation for "attribute" in code is attrib - is your use of atrib just a typo maybe?

Hi! Thanks for pointing this out. We have fixed the typo. If you have any other feedback feel free to share it with us!
Best regards,
Amiq

Thanks for the quick fix
Do you still have the Doxyfile around (it's not in the repo) - then you (or the users themselves) could build the documentation, which still refers to atrib right now.

The Documentation will be updated next week.

Hi! We have regenerated the documentation and added the Doxyfile as well.
Regards,
Amiq


Create index.md

Thank you for contributing! Please confirm this pull request meets the following requirements:

[ ] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[ ] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[ ] Content (and my changes are in index.md)

Please replace this line with an explanation of why you think these changes should be made.

Curating a new topic or collection

[ ] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[ ] My folder contains a *.png image (if applicable) and index.md
[ ] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing topic or collection, or at least not a single one
[ ] My suggested edits are not about curating a new topic or collection, or at least not a single one
[ ] My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

👋🏻 Can you check boxes and edit your PR description to provide the requested information, please?

Looks like this PR is still missing information, so I'm going to close it out. Feel free to re-open if you think it's still relevant!

okk


Dependencies for a project's dependency should automatically be imported

Currently, if you add a dependency to a project that itself has a dependency, building the project will fail unless the other dependency is also manually added.
For example, the Ownership Standard is a dependency for the Ownership Library. When adding the library as a dependency to a user's project without the standard, compiling will fail as the library cannot be built. The standard must ALSO be added as a dependency separately. This should automatically be done.
Minimal Repro:
We have the following Forc.toml file and main.sw file.
[project]
authors = ["Fuel"]
entry = "main.sw"
license = "Apache-2.0"
name = "test_dependencies"

[dependencies]
ownership = { git = "https://github.com/FuelLabs/sway-libs", tag = "v0.13.0" }

contract;

use ownership::*;

storage {
    owner: Ownership = Ownership::initialized(Identity::Address(Address::from(0x0000000000000000000000000000000000000000000000000000000000000000))),
}

Although the ownership library implements the initialized() function for the Ownership struct, compiling fails as the SRC 5 Standard dependency is not found.
After adding the extra dependency, the following compiles:
[project]
authors = ["Fuel"]
entry = "main.sw"
license = "Apache-2.0"
name = "test_dependencies"

[dependencies]
ownership = { git = "https://github.com/FuelLabs/sway-libs", tag = "v0.13.0" }
src_5 = { git = "https://github.com/FuelLabs/sway-standards", tag = "v0.1.0" }

contract;

use ownership::*;
use src_5::*;

storage {
    owner: Ownership = Ownership::initialized(Identity::Address(Address::from(0x0000000000000000000000000000000000000000000000000000000000000000))),
}

As the Sway ecosystem grows and projects begin to build upon one another, this is more imperative than ever.

I think this is not the case in a generic way. forc already creates a dependency graph which contains dependency of a project's dependencies (and their dependencies, recursively) and fetches builds them all in the correct order. I think this is failing because dependency of a dependency is not in the namespace.
We use re-exporting for similar stuff in other languages. So that we can expose things from project2 to projects that depends on project1, in a scenario where project1 depends on project2 but the root project does not depend on project2 directly.
I am not sure if re-exporting is possible with Sway yet. cc @FuelLabs/sway-compiler


Embracing immutability

@WhiteBlackGoose Seems that modifications done to GitHub Actions workflow files by a user without write access cannot be reflected in pull requests - Checks show up as empty.

I still think that
private int _quackQuack;

is not as good as
private int quackQuack;


Exception in distributed configuration: com.orientechnologies.common.exception.OException: Cannot find best cluster for class

In 2.0.10 and 2.0.11 we are getting exception in distributed configuration: com.orientechnologies.common.exception.OException: Cannot find best cluster for class looks like issue happens only on Linux (Red hat, CentOS), Steps to reproduce: 1. We are setting up a two node cluster.

We start the node1 and then start the node2. Now the cluster is up.

3.Attempt save any records on node1 failed. We get the following exception:
com.orientechnologies.common.exception.OException: Cannot find best cluster for class 'platform-DataflowContents' on server 'node_SI-RH6-Cluster101.pbi.global.pvt_8080'. ClusterStrategy=local

did you drop your class on node1 ? (so it would also drop the node1 cluster for this class)
maybe you can do "ALTER CLASS  ADDCLUSTER <some_cluster_name>" and try to save new records on node1 again

We don't this before prior 2.0.x every class had only one cluster. How there are multiply clusters per node and it doesn't seems to be working correctly. Looks like it is related to order in server list: "servers": ["nodel","node2","<NEW_NODE>"] ... but I am not sure.

Issue appears to be related to the fact that we are setting server names as: "servers": ["node_SI-RH6-Cluster102.pbi.global.pvt_8080","node_SI-RH6-Cluster103.pbi.global.pvt_8080","<NEW_NODE>"]. Once we updated names to: "servers": ["node_Cluster102_8080","node_Cluster101_8080","<NEW_NODE>"] issue was fixed. Is there any regulations for server names in Orient DB? Why we are getting exception: "Cannot find best cluster for class" and not some IO error.

You're right: server names can't have dot in the middle. I'm going to put an additional check on server names.

Thank you looks OK we can close this issue.


handling pyxis version check errors to display preflight release url

Relates: ##1135

Handling the two possible 400 series error messages.

Displaying the latest preflight release URL to the user.

Sample Pyxis Responses:

/test 4.15-e2e

/test 4.12-e2e

/test all

@komish @bcrochet I'm not really sure what to do with this PR. IMO using string(body) everywhere is the clearest path forward, since the intent of the type being used is in the code, trying to add another variable intrudes confusion of value/type of any given line of code that needs both the byte value and the string value.


UseLet is reported for else if

Expected Behavior
Following code should not trigger UseLet rule:
val a: Int? = null
val b = if (true) { // Any condition can go here
   0 // Anything can go here
} else if (a != null) {
   a + 1
} else {
   null
}

Observed Behavior
Above code triggers UseLet rule on the else if line
Steps to Reproduce

Attempt to scan above code with detekt

Context
While it is true, that above code could technically be turned into let, maybe something like that:
val c = if (true) {
   0
} else {
   a?.let { it + 1}
}

I feel like this variant is less readable and inferior to the else if variant.
Your Environment

Version of detekt used: 1.23.6
Version of Gradle used (if applicable): 8.7

I'm not sure about this. I don't see why the else if case should be treated differently. Maybe on a more real use case it is more clear, but in your snippet I think that the let works prefect.

Because conversion from else if to let is much more messy than from if to let:

in if -> let, let replaces the entire let statement, so the complexity is about the same than before, it's just clearer to the user (because ?.let usually means null check)
in if else -> let, let does not replace the if statement. Instead it adds another nested statement to it. Which in my opinion makes it less readable and more complex (I think it's even  doubly more complex by the detekt's own cyclomatic complexity rule, because it's nested)

I would say not to add that on detekt but I would like to know what other @detekt/maintainers think about it.

I already voted for not add.

Then closing for the reasons explained before. In any case, thanks for the proposal :)


Use .gitkeep to preserve output folder structure

Right now, many scripts will choke because subdirectories in the output folder aren't under version control. Use wildcards and .gitkeep to make sure the output directory structure comes along with cloning/forking.

Addressed in b48f702


Make sure CEPH state is tested (or create an issue)

### Tasks
- [x] dalco stag
- [x] dalco prod
- [x] inhouse tip
- [x] inhouse master

Done


the draggable item outside the container can't be draggale in mobile

the draggable item outside the container can't be draggale in mobile,what should i do

I am also facing this issue. I have a requirement to implement drag from outside on touch devices as well. What should I do?

I am also facing this issue. I have a requirement to implement drag from outside on touch devices as well. What should I do?

i just use ontouchmove / ontouchstart/ ontouchend to implement it.but there is some flaws to be fixed

Has anyone found a solution?

Has anyone found a solution?

@youthereSun @code31415926535


NPM

Hey @maximzhurkin,
Are you going to add it on NPM? If you need any help, please let me know!

Hi! https://www.npmjs.com/package/jquery-scrolla added)


Update plan-cloud-voicemail.md

Based on some questions from field/customers, added not to provide clarity that when moving users from SfB 2015 to 2019, their UM voicemail will automatically migrate to CVM.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@dstrome Changes look correct. Could you please review and merge? Thanks!


import is incorrectly added to existing from statement

The functools import is added to an existing from typing ... import statement:

I'm expecting the following import to be created instead:
import functools

Selecting the second option in the drop-down works 🤦♀


Fix PPA URL, add tools_only parameter, add start/stop checks to unit file

I've added some more minor changes to this PR -- if you want me to split it out into seperate PRs just let me know.

if you squash the requested changes into the existing commits, I'm fine with keeping things in the same PR :-)

Hi @flokli -- anything left before this can be merged?

@runejuhl yes, somehow I didn't receive an email about the changes. Merged now, thanks!


k8s-prometheus-adapter authentication with AWS eks.2

Hey guys,
Is there any documentation on how to authenticate with EKS apiserver?
I was under the impression I'm suppose to use --requestheader-client-ca-file with the ca.crt file generated for the worker nodes. However, this isn't working.
1 wrap.go:42] GET /apis/custom.metrics.k8s.io/v1beta1: (209.558µs) 401 [[kubectl/v1.10.3 (darwin/amd64) kubernetes/2bba012] <IP_ADDRESS>:51048]
E1026 14:45:37.424947       1 authentication.go:62] Unable to authenticate the request due to an error: x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "kubernetes")

Thanks!

Bump on this instead of opening a new issue. I opened a ticket with AWS support which went all the way up to the service team for EKS. They were also not able to get prometheus adapter to work with EKS.

I am able to reproduce similar error message with custom prometheus adapter.
Upon debugging further, EKS core API server doesn't use certificate based authentication for it's clients i.e. client-ca-file flag isn't set in core API server then the extension API server fails to start with error message
Error: cluster doesn't provide client-ca-file
In order to fix this issue, prometheus adapter should patch their custom metric API server which should be able to start and not require that both client-ca and requestheader-ca be present in the extension-apiserver-authentication ConfigMap. This allows EKS clusters which don't use certificate based authentication for core API server to be able to dynamically discover the requestheader-ca instead of having to mount it as a secret.

How did you generate cm-adapter-serving-certs?

How did you generate cm-adapter-serving-certs?  Did you use CertificateSigningRequest?

I think we need to do a dependency update to pull in the latest dependencies that have the EKS fix.  @brancz FYI

Whoops, sorry, responded too fast :-).  Let me actually read through this one

Would it be helpful to have self signed certs when starting out?

So, for the "cluster doesn't provide client-ca file" we just need the updated API machinery deps that fix that issue with EKS.  It's just a matter of pulling in the most recent release of the API machinery, and making sure it didn't break anything.

@twittyc Did you figure out a way around this? I am currently blocked with the same issue. I have created a ticket with AWS as well

I haven't been able to work on this. My priorities have shifted. Looks like you would need to update the API machinery deps in the Gopkg.toml and rebuild as this was fixed in the upstream dependency and the app needs to be rebuilt with those changes.

This has been working for me on EKS since #110. Is everyone else still seeing this with v0.3.0+?

@q3aiml are you installing it via helm or using anything else ? Because i installed it using helm and still have this issue.

like i already said i use v0.4.0, but still get error with certificates

Sorry, I missed that edit. Email only has the original comment without the version edit so I should pay more attention to the web interface. Sorry for the trouble...

I have just deleted cache of helm and tried installing adapter again and it worked. So I'm pretty sure it works on eks now.

This works -- closing issue. Thanks guys!

I am seeing this error message logged repeatedly on the prometheus-adapter pod, but it seems to be working. I am able to access the metrics I wanted to be exposed through custom.metrics.k8s.io. Is this something I should try to fix, or is there a way to ignore this error message? It's filling my logs.


scrollTo doesn't work with a custom scroll component

Current behavior
ref.current.scrollToItem (or scrollToIndex) does not work when we pass a custom scroll component in the renderScrollComponent prop.

https://snack.expo.dev/@degroteverleiding/nervous-macaroni-and-cheese---fixed

That we still can scroll to the item or index.

Platform:

[x] iOS
[x] Android

Environment

1.4.0

Solution:
const CustomScroller = React.forwardRef((props, ref) => {
    return <ScrollView ref={ref} {...props}/>;
})

With typescript it gives a lot of errors and it still doesn't work. Weird that it works in the expo snack but not in my app.

  const renderScrollComponent = React.forwardRef((props, ref) => (
    <Animated.ScrollView {...props} onScroll={scrollHandler} ref={ref} />
  ));

This is what i do in my app

SOLUTION: THIS SHOULD BE OUTSIDE COMPONENT

Just had the same problem. For me it was even simpler because I needed the ScrollView from react-native-gesture-handler. The following code gives no TS errors
import {
  ScrollView,
} from 'react-native-gesture-handler';

...

<FlashList
   ...
   renderScrollComponent={ScrollView}


[9.x] add die directive

Hello 👋🏻
In this PR I have added a die directive so we can use @die() or @die($var) on blade

What is the advantage over the @dd() directive?

@rodrigopedra Actually we can use both the same.
I have just tried to use it after dumping on a loop and found that it doesn't exist, sooo I thought to add it to the framework

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Add file logger callback & export train loss json file

This PR adds a new callback, which is invoked on the logging event - if the most recent log contains specific keys, the subdictionary is extracted, and the train_loss.json is reexported.
Example using LLama 7b with the twitter complaints example (tested on single GPU with a v100).
python tuning/sft_trainer.py  --model_name_or_path $MODEL_PATH  --data_path $DATA_PATH  --output_dir $OUTPUT_PATH  --num_train_epochs 5  --per_device_train_batch_size 4  --per_device_eval_batch_size 4  --gradient_accumulation_steps 4  --evaluation_strategy "no"  --save_strategy "epoch"  --learning_rate 1e-5  --weight_decay 0.  --warmup_ratio 0.03  --lr_scheduler_type "cosine"  --logging_steps 1  --include_tokens_per_second  --packing False  --response_template "\n### Label:"  --peft_method pt --tokenizer_name_or_path $MODEL_PATH --dataset_text_field "output" --torch_dtype "float32" --use_flash_attn False

Produces a train_loss.json file in the output directory:
$ ls out
checkpoint-13  checkpoint-15  checkpoint-3  checkpoint-6  checkpoint-9  train_loss.json

Which looks as follows:
[
    {
        "epoch": 0.31,
        "loss": 5.4751
    },
    ...,
    {
        "epoch": 4.62,
        "loss": 5.6743
    }
]

(Probably needs to be tweaked a bit to handle multigpu, since we should only do this from the main process)

Hey @gkumbhat! Thanks for the review. Some thoughts on your broader questions

General question, should we follow the same pattern as used by caikit-nlp for consistency and compatibility perspective ?

I would greatly prefer to avoid subclassing the trainer just for logging. The actual logging events look similar either way, so any logic in the logging should be pretty portable between the two approaches, and this project is already using callbacks, so callbacks feels more natural. Plus the SFT trainer is already a trainer subclass and pretty complex, subclassing it again just to write an extra file seems like unnecessary complexity to me.

We'll need to see how the callbacks work for steps vs epochs. Since we may want to log individual steps, even if it is set to epoch wise decisions. Another scenario we'll need to understand better is multi-gpu case.
For steps vs epochs etc - the event being handled here just gives a parsed version of the logged object, so we can just handle those keys separately, I think. I'll look into making sure this behavior is sensible.

For multigpu, we probably need to make sure that the file is only written by the master process, but I'm not sure how rank is taken into account for each process. I can look into this as well, although do you know if we formally support multiGPU at the moment, or is it still experimental?

I would greatly prefer to avoid subclassing the trainer just for logging. The actual logging events look similar either way, so any logic in the logging should be pretty portable between the two approaches, and this project is already using callbacks, so callbacks feels more natural. Plus the SFT trainer is already a trainer subclass and pretty complex, subclassing it again just to write an extra file seems like unnecessary complexity to me, since it would basically be for the one function

But follow pattern, I actually meant the formatting of the output file. Sorry I was not quite clear there

do you know if we formally support multiGPU at the moment, or is it still experimental?
Ops, I don't know 🤷

Thanks everyone - updated the PR to write in jsonl format, and only dump logs from process zero to prevent clobbering when running multiprocess training

Hey @gkumbhat - updated the PR and description based on our discussions to match the legacy log format, should be ready for another look when you have a moment


[MicroWin] Screenshotting Broken Without Snipping Tool

Describe the bug
By default MicroWin strips out Snipping Tool which, at least on WIndows 11 23H2 is required for screenshotting. The older overlay still appears but there is no notification and the screenshot is never saved. Turning off the setting to use snipping tool when pressing print screen doesn't seem to help.
To Reproduce
Steps to reproduce the behavior:

Install WIndows 11 23H2 using MicroWin
Try to take a sreenshot
Open the Pictures folder
Install Snipping Tool from the Microsoft Store
Take another screenshot

Expected behavior
Screenshotting should work out of the box when using MicroWin.
Additional context
The advanced debloat tweak also removes it. Maybe that should be reconsidered too?

@Samq64, a fix is on its way


Add Issue type

/cc @codeclimate/review
This adds an issue type for messages sent to external services. I chose to add this new type over adding to quality, as quality is already used for 2 things (rating changes and refactoring hot spots).
I realize there is a fair amount of duplicated code here - I'm open to ideas on how to fix that, but I'm not familiar enough with this codebase to make that call right now.
I skipped the following services, as they only deal with transitions:

Campfire
Flowdock
Hipchat
Slack

Preview on each service:

LGTM


Version compatibility

@henry-spanka This is a question not an issue.
In the Readme, it is stated that SysAp V3.0 is supported.
What is with the other versions:

v3.1
v3.2

I seem to have error in a usecase of the npm module: https://github.com/mobilutz/ioBroker.freeathome/issues/654
Thanks

Should be supported. I am running latest firmware on SysAp and working so far.


Azure Notification Hub static instance or recreate

Good day, I made a SO question about this, but I have not any official MS response.
SO
What is the correct way to use the c# Azure Notification Hub Client?
We are using this in a very high usage server sending millions of pushes a day.
I would like to know if we should recreate the NotificationHubClient every time or if we should keep a static/singleton instance of it and then reuse that each time?
Currently, we recreate it every time we send a push notification. However, I know from personal experience we have had issues with the .net HTTP client and it not releasing tcp sockets fast enough. I was worried that this library could start having similar issues.

I am also experiencing issues with using the hub client whereby it is exhausting sockets, what is the best practice for creating the and reusing the NotificationHubClient

For anybody struggling with this, we solved it by passing in a MessageHandler from the
IHttpMessageHandlerFactory this prevents it from recreating the HTTP client and starving sockets
// Example code where _httpMessageHandler is a static property

`            if (_httpMessageHandler == null)
{
var serviceProvider = new ServiceCollection().AddHttpClient().BuildServiceProvider();
            var httpClientFactory = serviceProvider.GetService<IHttpMessageHandlerFactory>();

            _httpMessageHandler = httpClientFactory.CreateHandler();
        }

        var hub = new NotificationHubClient(nhub.EndPoint, nhub.HubName, new NotificationHubClientSettings
        {
            MessageHandler = _httpMessageHandler
        });

`

@tiaan-lg yes, that would be a recommended approach.  Please reopen this issue if this is still a problem.

@mpodwysocki Can you please clarify if this approach is correct or not, we are adding the following code in the Constructor of our Controller.
private NotificationHubClient _nhClient;
public NotificationsController(CMLogger logger,
                                       IConfiguration config, IHttpMessageHandlerFactory httpMessageHandler)
        {
            _logger = logger;
            _config = config;
            _nhClient = new NotificationHubClient(_config["NotificationsHub:ConnectionString"],
             _config["NotificationsHub:HubName"],
             new NotificationHubClientSettings
             {
                 OperationTimeout = TimeSpan.FromSeconds(10),
                 MessageHandler = httpMessageHandler.CreateHandler()
             })
        }


Assertion failure in tf.raw_ops.RFFT2D and tf.signal.rfft2d when the values of fft_length are 0.

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
tf 2.15
Custom code
No
OS platform and distribution
Linux Ubuntu 20.04
Mobile device
No response
Python version
3.10
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Assertion failure in tf.raw_ops.RFFT2D and tf.signal.rfft2d when the values of fft_length are 0.
Standalone code to reproduce the issue
https://colab.research.google.com/drive/1oXMVvT3ghxI4KF1kMd1_QRbvA1468oUb?usp=sharing

Relevant log output
Assertion failure
axis length mismatch

@SuryanarayanaY I was able to replicate this issue here.
Thank you!

Hi @xl1043237213,
The FFT length is often chosen to be a power of 2 for computational efficiency. However, it can be either smaller or larger than the corresponding dimension of the input data. So use fft_length as 16, 32, 64 like. Here is the gist.
Thank You


Update route describe decorator to use autoDescribeRoute

Relies on girder/girder#1701

Codecov Report

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

@@           Coverage Diff           @@
##           master     #570   +/-   ##
=======================================
  Coverage   62.03%   62.03%
=======================================
  Files          61       61
  Lines        2845     2845
=======================================
  Hits         1765     1765
  Misses       1080     1080

Continue to review full report at Codecov.

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

@TristanWright Hold off on review, looks like there is a whole test commented out!? not sure why ...

@TristanWright OK, fixed up.


Fix Crow Clock For Dark Mode and Fix Desktop Launcher Bangle.js 1 not showing white images.

#1532
#1539

Great - thanks for sorting this!


Missing support for cancelling out of Reveal 'close' event

It would be very useful if it was possible to cancel out of the close event of the Reveal dialog, for example when a form has not been submitted.
Something like this:
('#my-modal').on('close', function( e ) {
    var ok = confirm( 'Are you sure you want to close this?' );
    if ( ! ok ) {
        e.preventDefault();
    }
}

We added this to our "known issues" section in our planning board for the next version. A PR would be valuable to the community, otherwise It will be addressed then. You can see and vote on the proposed updates here: https://trello.com/b/QBPKlQqf/foundation-sites


Test problematic variants

added unit test module for "problematic" variants (anything which raises an exception when annotating variants in the wild, e.g. from TCGA MAF files)
Variant now trims any shared suffix or prefix strings from ref and alt and adjusts the variant position accordingly. The position field has been renamed to start. The original variant fields are preserved as Variant.original_ref, Variant.original_alt, Variant.original_start. For example, if a VCF contains "chr1 10 CAG CATG", then the corresponding Variant object will have ref = "", alt = "T", start = 11, original_ref = "CAG", original_alt = "CATG", original_start = 10.
Renamed top_priority_transcript_effect to top_priority_effect (since we now have effects like Intergenic which aren't tied to a particular transcript)
Added ExonLoss as an effect
Reorganized the effect annotation logic in Variant to deal with variants which span feature boundaries (e.g. deletions which affect both an exon and its neighboring intron).
renamed VariantCollection.variant_effects to just VariantCollection.effects
renamed VariantCollection.variant_summary to VariantCollection.summary_string
added unit tests for misc. VariantCollection methods

I didn't check all the logic here (really unit tests are the only way), and sounds like much of this will change if you move to the ref_position_before and ref_position_after way of localizing variants, but from looking at what you have so far I don't see any major flaws

Did anything stand out as particularly in need of more/any unit tests?


[Compiler Fix] Bring declarations and definitions to be in sync

This PR fixes the linking issue in our setup.


Modularize per-cycle tasking flow around CompletionStage And CompletableFuture

The op synthesis layer (op template parsing and normalization, op mapping and op dispensing) needs to create operations which are composable as functions, and which are easy to dispatch in different scheduling systems. This will allow us to chain key validation logic in places where it is specified, but to elide those which are not.
This is necessary to make adding modular validation logic more manageable without adding branching and conditions to the hottest code path in the engine.

Discussions #938, #946, #1988, and #1978


Usage in the Browser?

Greetings,
I'd like to use this plugin in a browser environment.
The only roadblock is the import var path = require('path').posix;, which is ending up as undefined in the browser build.
I wonder, would there be a simple way to avoid this import, so that the build can work in a browser environment?
One option appears to be https://github.com/suarasaur/node-path-posix

See my comment on PR #9:

Hmm, I'm not sure I want to pull in a very old duplication of a Node API in order to paper over a deficiency in some other duplication. I don't see any reason for whatever is supplying the path package in your browser build to omit the posix export.
If you're using rollup-plugin-node-builtins, there's an issue for this: https://github.com/calvinmetcalf/rollup-plugin-node-builtins/issues/33
Browserify, on the other hand, seems to include a correct posix export. Perhaps you could use a Rollup plugin to alias the path import to path-browserify.
I'm going to close this.

What's here already works in Node, should work in Browserify, and in Rollup with some tinkering. That's all I'm aiming for when it comes to compatibility, so I'm going to close this issue.
Feel free to reopen if you think any of those goals are unmet or if you have a reason for me to set a higher standard.

Thanks for looking into this!
I will look into this more as well. In particular

whatever is supplying the path package in your browser build

I don't know how this would be supplied, as it's a Node API rather than an NPM package, so I can research more on how it might be supplied for a browser build.
Fingers crossed it can work

in Rollup with some tinkering.

Will tinker and report back here if I do find a way to make it work.
Thanks for your work on this package!


Breakpoints linespec in Windows is case-sensitive

When adding breakpoints, if the casing does not match exactly "d:/go_dev/src/github.com/vornet/hello/hello.go" vs "D:/go_dev/src/github.com/vornet/hello/hello.go", the breakpoint will not be added, erroring out with: Command failed: Location "xxx" not found.
On other OSes, case sensitivity is expected, but on Windows, it should not be case-sensitive.
For full filepaths, the workaround is to make sure the casing in the GOPATH (delve uses it to match full paths) matches the breakpoint linespec. So if GOPATH="D:/Go_Dev", linespec should be "D:/Go_Dev/src/...."

I think this is the same as an issue I raised earlier: https://github.com/derekparker/delve/issues/370

Closing this as duplicate of #370.


Any instructions for running Tensorflow.js?

Hi,
Which version of tensorflow.js is currently supported?
Is there an example somewhere of tensorflow.js running on node-gles?
Last and most key question is: does this offer any benefit over using the C version of tensorflow for nodejs (I believe there is a tensorflow native module for node out there, based on my vague recollection) For instance, does node-gles allow tensorflow.js to fully exploit the GPU (if present on the system) whereas tensorflow nodejs native module is a CPU only version? Also, what if the system has multiple GPUs, does tensorflow.js have any plans to support local distributed training or distributed inference using multiple GPUs?
Would love to hear back soon. Thank you. This is EXCITING.

Some answers to your questions:

Which version of tensorflow.js is currently supported?
We currently don't have an official supported version of TensorFlow.js. Stay tuned this is something we're working on at the moment.

Is there an example somewhere of tensorflow.js running on node-gles?

Nothing official/stable/etc. I do have a test branch that I'm using to sort out some bugs with f16 textures: https://github.com/google/node-gles/compare/kreeger-rpi-fix?expand=1

Last and most key question is: does this offer any benefit over using the C version of tensorflow for nodejs (I believe there is a tensorflow native module for node out there, based on my vague recollection) For instance, does node-gles allow tensorflow.js to fully exploit the GPU (if present on the system) whereas tensorflow nodejs native module is a CPU only version? Also, what if the system has multiple GPUs, does tensorflow.js have any plans to support local distributed training or distributed inference using multiple GPUs?

We do have Node.js bindings with the C library of libtensorflow. Those bindings exist in the following two packages:

@tensorflow/tfjs-node (CPU libtensorflow)
@tensorflow/tfsj-node-gpu (CUDA-only libtensorflow)

We're thinking about the messaging here since there are many options for hardware execution paths. The existing Node.js bindings enable CPU support in libtensorflow and GPU w/ CUDA with libtensorflow. The headless WebGL backend (powered by node-gles) will enable GPU hardware acceleration on more devices than just CUDA enabled devices. Think of devices like IoT and mobile as well as some desktop/server integration strategies.
This is all preliminary work and hopefully we'll have something to showcase externally in the coming weeks.

Great info, thank you. The trajectory of this project looks amazing in that there will be great benefit if we could run TF with more than just NVIDIA GPUs, without a CUDA to OpenCL conversion.
For the time being, given the early stage of this project, I may be better off going with  tfjs-node-gpu on top of CUDA. But only on temporary basis until node-gles is fully ready!


Expose descheduler flags

/cc @aveshagarwal @sjenning
Added support for descheduler flags.
Note to myself: As of now, there is now way to add policy-config file flag.  We need provide capability to add one.
Fixes: #18

/retest

/retest

/retest

/test e2e-aws-operator

/test e2e-aws-operator

/retest

/retest

/retest

/lgtm
/approve

/test e2e-aws-operator

/hold
Tests have been solid red on this. Need to debug why, this passed previously and working fine locally on my system.

/hold cancel
@sjenning - While stress running the tests, I noticed that this test is flaking because of timeout. I increased the timeout for now for cron job. I am working on making the operator to make changes instead of requeuing it. Hopefully that will reduce the test time.

/retest

e2es are hitting VPC exhaustion atm

Yeah, I checked in installer channel and Clayton responded that they are working on it. So, I will wait for now.

/hold
Seems the increased timeout did not address the issue on ci provisioned cluster

/hold cancel