[BUG] Colors not showing on the new character sheet

Module Version: v0.3.21
Before open any issue

Enable the module setting "Enable debugging"
Click F12 go to the console tab
make the test you want and replicate the error
Go to the tab console open on point 2) and just right click and click 'Save as' and 'Save' or send a screenshot of the exception on the console.
Attach the text file on the github issue with all the logs related tot he module, or send a screenshot of the messages on the console.

Describe the bug
Colors are not appearing on the new character sheet, in the new NPC's Sheets are working as should.
TypeError: Cannot set property message of  which has only a getter
at Hooks.onError (foundry.js:752:30)
at 🎁call_wrapped [as call_wrapped] (libWrapper-wrapper.js:507:22)
at 🎁Hooks.onError#lib-wrapper (listeners.js:138:11)
at 🎁Hooks.onError#0 (libWrapper-wrapper.js:187:52)
at #call (foundry.js:734:36)
at Hooks.callAll (foundry.js:687:17)
at ActorSheet5eCharacter2.renderPatch (application-render.js:78:15)
at async ActorSheet5eCharacter2._render (foundry.js:6532:5)
at async ActorSheet5eCharacter2._render (character-sheet-2.mjs:76:5)
To Reproduce
Steps to reproduce the behavior:

Go to 'Character sheet'
See Error

Expected behavior
Expected to see the colors in the features, spells and items
Screenshots

Browser:

Chrome

Foundry Version:
V11 Stable - Patch 13 (Build315)
Game System:
Dungeons & Dragons Fifth Edition (3.3.0)

Cannot replicate , can you pass to me the acro json data (rename the .json to .txt to attach the file on github)

Can you tell where i can find this acro json data?

Can you tell where i can find this acro json data?

A sorry "actor json data", just right click on the actor on the actor sidebar and click on the voice "Export Data"


PhysicalExpr Orderings with Range Information

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Merged upstream, closing.


Formatter: Raw strings containing double quotes

I'm not sure how Black handles this case, but Ruff wants to force this:
r'\"'

to instead be this:
r"\""

...Which just feels like a completely unnecessary drop in clarity.
Tested/confirmed with the following code, run through  Ruff 0.6.7 via ruff format --diff:
#!/usr/bin/env python3

input = '"Some string"'

output = input.translate({ord('"'): r'\"'})
print(f"{output=}")

Result:
$ ruff format --diff ./ruff_test.py
--- ruff_test.py
+++ ruff_test.py
@@ -2,6 +2,5 @@

 input = '"Some string"'

-output = input.translate({ord('"'): r'\"'})
+output = input.translate({ord('"'): r"\""})
 print(f"{output=}")
-

1 file would be reformatted

I'm not sure how Black handles this case, but Ruff wants to force this:

I tried it in Black's playground and Ruff's behavior matches Black's.
Pasting the different strings into the REPL produces the following output
>>> print(r"\"")
\"
>>> print(r'\"')
\"
>>> print(r'"')
"

What's important to note is that the \ is necessary to escape the ", but it also changes the string's value (the backslash is preserved, unlike in regular strings). Ruff can't remove the \ because of that, even when the string uses single quotes.
Based on this, I think Ruff's/Black's quote choice is consistent: They use the preferred quote style (most commonly " unless configured otherwise) except when using the preferred quote style requires more escapes. In this case, the \ isn't just an escape, it's semantically important. Because of that, using ' or "requires the exact same amount of quotes, in which case both formatters use the preferred quote style.

Thanks for looking at this, @MichaReiser.

What's important to note is that the \ is necessary to escape the ", but it also changes the string's value (the backslash is preserved, unlike in regular strings). Ruff can't remove the \ because of that, even when the string uses single quotes.

Correct. Actually, if we're being technical the \ doesn't escape the ", because there are no escapes in raw strings. While in a non-raw string, \" is an escaped double quote, in a raw string it's simply a backslash followed by a double quote. The non-raw equivalent of that string is '\\"' or "\\\"".

Based on this, I think Ruff's/Black's quote choice is consistent:

I don't entirely disagree, but it's consistent in the context of what I would argue are an incomplete set of criteria.

They use the preferred quote style (most commonly " unless configured otherwise) except when using the preferred quote style requires more escapes.

See, the way I look at it, the reason to use single quotes around a string containing double quotes isn't primarily to avoid extra escapes...

In this case, the \ isn't just an escape,

Not an escape at all.

it's semantically important. Because of that, using ' or "requires the exact same amount of quotes, in which case both formatters use the preferred quote style.

The main reason to use different quote styles around a string containing quotes, to me, is because it makes it easier to differentiate the outer quotes from the internal ones. It's the same reason we switch from double to single quotes in English, "When we're embedding 'quotations inside quotations'."
Which is why I think Ruff (and probably Black) is also wrong to want to turn this:
print('"He\'s in \'ere."')

into this:
print("\"He's in 'ere.\"")

Exact same number of escapes either way, but having escaped double quotes immediately adjacent to outer double quotes is much less easily parseable (for humans) than having escaped single quotes deep inside the string, but unescaped double quotes directly adjacent to unescaped single quotes at the outer extremes.

In this case, the \ isn't just an escape,

Not an escape at all.

Actually, I take that back. It is an escape, but only because the raw string rules make a special exception for escaped quotes. From the STRINGS help:

Even in a raw literal, quotes can be escaped with a backslash, but the backslash remains in the result; for example, r"\"" is a valid string literal consisting of two characters: a backslash and a double quote; r"\" is not a valid string literal (even a raw string cannot end in an odd number of backslashes).  Specifically, a raw literal cannot end in a single backslash (since the backslash would escape the following quote character).

The fact that, unlike in non-raw strings, r'\"' and r"\"" happen to be equivalent, is something they don't address.

Thanks for the detailed explanation.

The main reason to use different quote styles around a string containing quotes, to me, is because it makes it easier to differentiate the outer quotes from the internal ones. It's the same reason we switch from double to single quotes in English, "When we're embedding 'quotations inside quotations'."

I can see the reasoning behind this. I worry that it's too complicated for an auto-formatter.

It's important for auto-formatter that the result isn't surprising for users. Users should understand the heuristics. Too clever heuristics and users won't understand why the formatter formats their code in a specific way, which can be frustrating
Implementing your proposal would require coming up with a heuristic to a) match quotes: There's no formal grammar in strings that require that quotes are balanced and b) decide what are outer and what are inner quotes.
The style applied by Ruff/Black aligns with auto formatter in other ecosystems (e.g. Prettier). Consistent cross-ecosystem behavior has the advantage that the formatter output is less surprising to users and matches the code that most users read, making it easier for them to read the code.

That's why I prefer keeping the existing style.


[YouTube] ...

Extension – YouTube
Author: @tonka3000

Description

Error:

Steps To Reproduce

The current behavior
The expected behavior
Raycast version
Version: 1.39.2

What is the error? could not see the image you attached


Add CRC32 checksums as trailer

This change is made for the multipart parallel (non-seekable) stream
upload.

Needs https://github.com/minio/minio-go/pull/1764

@klauspost 👍🏾 Closing this one for now. Won't try to add this again, unless we can demonstrate at least some improvement.


Respawn Timer bugged

I've been comparing the respawn timer to the one that's in NPC Indicators and this one does not seem to be working. When I switch on the NPC Indicator respawn timer, it identifies the respawn timer after killing a few of the NPC (in this case I just tested it on a Man) but when I flipped plugins and tried it on this plugin it literally does not have a respawn timer. There has been cases where it will randomly appear after killing loads of NPCs but the timer usually bugs out

Why is this issue closed? I have this issue as well.

After a few updates to the plugin you can basically flip the NPCs now so the ones you want respawn timers for you put on the vanilla NPC Indicators plugin, and the ones you want all the fancy features on you can also put on this plugin. Also he didn't reply so I just closed it


Add support for windows container dir binding

This shouldn't apply to WSL which we're recommending now so closing this out


Add integrated test for zipkin plugin

Fixes #2739

Codecov Report

Merging #2767 (16cb71b) into master (99dfb84) will decrease coverage by 0.17%.
The diff coverage is n/a.

:exclamation: Current head 16cb71b differs from pull request most recent head 61be855. Consider uploading reports for the commit 61be855 to get more accurate results

@@             Coverage Diff              @@
##             master    #2767      +/-   ##
============================================
- Coverage     60.10%   59.92%   -0.18%
+ Complexity     4419     4413       -6
============================================
  Files           685      685
  Lines         19692    19690       -2
  Branches       1750     1750
============================================
- Hits          11835    11799      -36
- Misses         6863     6893      +30
- Partials        994      998       +4

Impacted Files
Coverage Δ

...yu/sync/data/http/refresh/AbstractDataRefresh.java
70.00% <0.00%> (-13.34%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...apache/shenyu/admin/utils/ShenyuDictH2Trigger.java
63.63% <0.00%> (-9.10%)
:arrow_down:

...ache/shenyu/admin/utils/PluginHandleH2Trigger.java
63.63% <0.00%> (-9.10%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
75.73% <0.00%> (-8.09%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
55.55% <0.00%> (-7.94%)
:arrow_down:

...in/response/strategy/NettyClientMessageWriter.java
69.56% <0.00%> (-4.35%)
:arrow_down:

...henyu/plugin/httpclient/NettyHttpClientPlugin.java
53.84% <0.00%> (-1.93%)
:arrow_down:

...gin/apache/dubbo/cache/ApacheDubboConfigCache.java
70.88% <0.00%> (-1.27%)
:arrow_down:

...n/alibaba/dubbo/cache/AlibabaDubboConfigCache.java
73.75% <0.00%> (-1.25%)
:arrow_down:

... and 1 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 99dfb84...61be855. Read the comment docs.

pls check ci , there may be some error in shenyu-integrated-test-agent-tracing-zipkin/docker-compose.yml

pls check ci , there may be some error in shenyu-integrated-test-agent-tracing-zipkin/docker-compose.yml

ok

@LeeGuoPing Thanks for your contribution, your code with @lxl910128 has been merged into master in #2824


what is the performance of ghostnet？

I did not try it in deep, it could be improved in the future.


Factor of 2 was missing inside log (see eq. 20 in Kraskov et al.)

Fixes #66.
I can't get the Makie example plots to run locally, so the convergence plots in the documentation build should be checked before merging this.

Upon testing on real distributions (locally), this is not a bug. The factor of 2 causes the entropy to converge away from the true values, whereas without the factor it doesn't. Either I'm misunderstanding a formulation in the paper, or the formulation in the paper is wrong. Closes #66 .


AbletonOSC paused on Live 12 main menu access.

"Hello!
Note: This issue is with Live 12. I can't test with Live 11 since I'm blind. Be very comfortable to close that issue since AbletonOSC is not officially supported for Live 12. That said, it really works fine with 12.
So, is it possible that the script doesn't work on the Ableton side when the main menu is open? I'm sending a request to identify the devices every 3 seconds, and when the main menu is open, these requests don't respond and accumulate. Then, they are sent back at a very high rate. Thank you!
The request that's being sent is:
/live/track/get/devices/name'"

Hi @Daretreble, thanks for the report. I've not seen this before, but have just hold of Live 12 so will be able to try to reproduce this (and will more generally be verifying performance with Live 12). Will get back to you and let you know what I find...

i'm gonna try this out with Live12 to see what happens.
Also, what's the Main Menu? Ableton Settings? Top Menu on macOS?

@Daretreble so hi, i booted up Live 12, entered the Ableton Settings, and sent a command to change tempo.
i also tried this
>>> /live/track/get/devices/name 1
1, EQ+Filter+chorus+DC+AutoPan

and got the result.
so.. .it's not Ableton Settings. maybe you're saying that while the main menu (topbar menus) is open, then nothing happens? is this it:
(an image of the top menu open, in this case, View)

i could fully understand that that doesn't work, but hopefully there's a workaround for this.


Load Json to persist data between deployments

Hello,
I see you used firebase to do persist data between deployments, would it be possible to add this feature with json?
Context:
In my case the streamlit app is deployed on a docker and we have no persistence of volume (security constraint), so I was thinking about uploading the json on a server at fixed frequency rate. For example a task that will upload the json file each 30 seconds.
Then if the docker dies, we could fetch this json and load it in your tracker to persist the data.

Merged with #10


LinkML does not currently support second-order grouping of Value syntax components

For example, we don't have a way to say that MIXS:0000322 values should be in one of these forms

some text, followed by a semicolon, a floating point number, and a unit
a floating point number, followed by a unit, followed by a semicolon, a floating point number, and a unit

Value syntax
Example
Requirement
Preferred unit
Occurrence
MIXS ID

{{text}|{float} {unit}};{float} {unit}

X

1
MIXS:0000322

Since anything will match {text}, I propose we express this Value syntax as {text};{float} {unit} and elaborate in the Expected value and or Example

Note that the term submitter did not provide an example


Update to 9.0.1

Depends on https://github.com/conda-forge/llvmdev-feedstock/pull/81

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

@conda-forge/compiler-rt Ready for review

Thanks


Replace uuid randomizer with source player uuid

Issue
When using at my server, all market entry cases was from "DummySeller" and /balance top displays null: [quantity] when something is bought. Because of the uuid randomization while storing, the getEntries function never finds by uuid
override fun getEntries(): List<StoredMarketEntry> {
        return cache.toList().sortedBy { it.expire }.also {
            it.forEach { entry ->                                             // here
                val seller = GunpowderMod.instance.server.userCache.getByUuid(entry.uuid)?.name ?: "DummySeller"

Changes
This should use the uuid from source player instead of the Randomizer (I think it was used for debug purposes).


🐛 [Bug report]: CONTRIBUTING.md contains the contents of Code of Conduct

Describe the bug
CONTRIBUTING.md is having the same content as that of CODE_OF_CONDUCT.md
Expected behavior
CODE_OF_CONDUCT.md is having the correct content, however CONTRIBUTING.md should ideally mention steps about local setup guide and other information that may be needed for contributing.
Screenshots

@supminn Good catch!
Do you want to take it up to fix?
We can model it on this one: https://github.com/TryShape/tryshape/blob/main/CONTRIBUTING.md

Sure, I'd love to take it up.
Thank you @atapas


docs-ref-services/speech-service.md: initial version

@sudeepku  - please have a look.

CC @fmegen @wolfma61

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-autogen/microsoft-cognitiveservices-speech-sdk/index.yml
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/speech-service.md
:white_check_mark:Succeeded
View (azure-node-latest)

docs-ref-autogen/microsoft-cognitiveservices-speech-sdk/index.yml

[Warning] Uid(microsoft-cognitiveservices-speech-sdk) has already been defined in docs-ref-services/speech-service.md.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-autogen/microsoft-cognitiveservices-speech-sdk/index.yml
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/speech-service.md
:white_check_mark:Succeeded
View (azure-node-latest)

docs-ref-autogen/microsoft-cognitiveservices-speech-sdk/index.yml

[Warning] Uid(microsoft-cognitiveservices-speech-sdk) has already been defined in docs-ref-services/speech-service.md.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-autogen/microsoft-cognitiveservices-speech-sdk/index.yml
:warning:Warning
View (azure-node-latest)
Details

docs-ref-services/speech-service.md
:white_check_mark:Succeeded
View (azure-node-latest)

docs-ref-autogen/microsoft-cognitiveservices-speech-sdk/index.yml

[Warning] Uid(microsoft-cognitiveservices-speech-sdk) has already been defined in docs-ref-services/speech-service.md.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


What is this Package For

Is this package to allow proxying of incoming or outgoing requests from the application
I am looking for a core compatible client proxy as WebProxy only exists in desktop.
Thanks

This implements a reverse proxy for forwarding incoming requests to back-end services. It is not a WebProxy implementation.
Ask about WebProxy in corefx, I've heard they are adding something there.


Make vscode-jest Official

Make vscode-jest Official, and allow it to share the majority of the code with a potential nuclide version.
TODO

[x] Decide on repo? (would this be a npm module as a lerna thing? @cpojer)
[x] TypeScript -> Flow
[x] Convert from mocha -> Jest
[x] Start to decouple VS Code from the codebase. This means only Extension.ts is allowed to use import {workspace} from 'vscode'; as a bare minimum.
[ ] Try to remove hacks/string parsing that could be fixed by officially supporting things inside Jest

Getting Jest Version
Getting Jest Config
actually that entire config reader is a tad hacky, make that all structured JSON data
add a "terse" message for any fail

Awesome.
Let's put all of these into packages/ inside of Jest. If you can reserve the package names on npm and add me (cpojer) and the "fb" user to it, that will be great. We can then start publishing from within this repo. We may need to figure out some permissions but that shouldn't be that big of a deal :)

So deployment of vscode packages isn't done via npm, ( that's a whole thing) so we don't need to claim it for vscode-jest, and from the look of it, we don't need to from atom (from a quick look at a tutorial.)
So, unless you think it makes more sense to have a jest-editor-support package that they both rely on, which I'm amiable to, then we don't need to claim anything?

Oh that's interesting, I was unaware. I definitely think we should split everything up into a jest-editor-support package (or many) and then find a way to publish the vscode package. For atom, we should do exactly the same: push most of the code into the editor support package and the atom specific code into a separate package that we deploy via apm. Does that make sense?

Yep 👍

This is done 👍


Textarea floating label position

Describe the bug
The label appears in the middle of the text area.
To Reproduce
Steps to reproduce the behavior:

Create a textarea input using the provided html markup on the floating label documentation
Add 5 rows rows={5}

Expected behavior
The label should appear at the top of the input
Screenshots

Desktop (please complete the following information):

OS: Mac
Browser chrome
Version 108.0.5359.98

Additional context
This is on NextJS

Fixed it by changing peer-placeholder-shown:top-1/2 to peer-placeholder-shown:top-[1.5rem]


Palisades as semaphore poles

theyre pretty cool

Manually merged as part of https://github.com/Layers-of-Railways/modpack/commit/87cb999e499e0fe37b20c49c0faa38b6e098fc8c, Thanks!


Official MS docs should contain up-to-date MSTest documentation

Summary
We should make sure to publish updated information on official MS docs website.

I'll close this ticket as we have started to do continuous update on the various learn.microsoft.com entries. Next stop for us is the update of the code APIs for which we are still waiting for some input.


Added Dockerfile

Added a Dockerfile for easy deployment

Nice work. One comment.
The downside to this docker file is it includes the entire go tool chain,
which defeats the point of using go which creates static binaries which
have no dependencies.
On Feb 15, 2015 1:32 PM, "Lorenz Brun"<EMAIL_ADDRESS>wrote:

Added a Dockerfile for easy deployment
You can view, comment on, or merge this pull request online at:
https://github.com/hoonmin/influxdb-collectd-proxy/pull/27
Commit Summary

Added Dockerfile

File Changes

A Dockerfile
https://github.com/hoonmin/influxdb-collectd-proxy/pull/27/files#diff-0
(9)

Patch Links:

https://github.com/hoonmin/influxdb-collectd-proxy/pull/27.patch
https://github.com/hoonmin/influxdb-collectd-proxy/pull/27.diff

—
Reply to this email directly or view it on GitHub
https://github.com/hoonmin/influxdb-collectd-proxy/pull/27.

Yes I know. I thought about it, but finally decided to go with a self-compiling dockerfile because the static binaries would have to be compiled outside of docker which means that autobuilds from the GitHub Repository aren't possible anymore. I could include a slim version which contains only precompiled images, but they would have to be compiled somewhere.

Maybe it's time to setup a Travis build someplace @hoonmin?
On Feb 16, 2015 9:30 AM, "Lorenz Brun"<EMAIL_ADDRESS>wrote:

Yes I know. I thought about it, but finally decided to go with a
self-compiling dockerfile because the static binaries would have to be
compiled outside of docker which means that autobuilds from the GitHub
Repository aren't possible anymore. I could include a slim version which
contains only precompiled images, but they would have to be compiled
somewhere.
—
Reply to this email directly or view it on GitHub
https://github.com/hoonmin/influxdb-collectd-proxy/pull/27#issuecomment-74544143
.

A Travis Build would be an excellent solution, I would still keep two versions, a self-contained one (my current one) for people that fork the repo and want to do stuff quickly and a Travis version, which would precompile the binary and import it into a FROM scratch Dockerfile.
For the official Docker Image I would make the default tag for each version the slim build, and add a version-dev version for the large self-contained one.
What do you think of that?

SGTM
On Feb 16, 2015 9:42 AM, "Lorenz Brun"<EMAIL_ADDRESS>wrote:

A Travis Build would be an excellent solution, I would still keep two
versions, a self-contained one (my current one) for people that fork the
repo and want to do stuff quickly and a Travis version, which would
precompile the binary and import it into a FROM scratch Dockerfile.
For the official Docker Image I would make the default tag for each
version the slim build, and add a version-dev version for the large
self-contained one.
What do you think of that?
—
Reply to this email directly or view it on GitHub
https://github.com/hoonmin/influxdb-collectd-proxy/pull/27#issuecomment-74546106
.

@lorenz Your dockerfile looks good but I think it would be better to export a VOLUME for types.db. And I prefer the base docker image(golang) you used because we have no binary release right now :-(
@yanfali Sure I agree with you. Please wait a sec :-)


[react-core] Integrated BNS to resolve .base names with useENS hook

Problem solved
Resolve .base names with useENS hook when user is on Base mainnet
https://github.com/thirdweb-dev/js/assets/13912728/03317c31-d1a5-418f-8a58-a807f7754385
Changes made

[ ] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[x] Manual tests: step by step instructions on how to test

Contributor NFT
Paste in your wallet address below and we will airdrop you a special NFT when your pull request is merged.
Address: jackjack.base | jackjack.eth

PR-Codex overview
This PR adds BNS support to resolve .base names in the codebase by integrating BNS registry and updating chain configurations.
Detailed summary

Integrated BNS registry to resolve .base names using useENS hook
Added Base object import for chain configurations
Updated chain configurations to handle BNS resolution

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

hey @iganbold - sorry for the delay here, we've been pretty heads down working on #2356 - which should make it much easier to achieve something like what you're trying to do here.
Once that is merged )hoping today) I'll circle back to this PR to see if we can:
a) port it to the new sdk version
b) also include it on top of the v4 sdk retroactively
I'll follow up with you as soon as I have clarity on it!
Thank you again for your contribution!

hi @iganbold, finally getting back to you here (sorry for the delay). We've built in a way to override both the ens resolver and the chain that it is being resolved from into v5:
resolveAddress now lets you specify options that include the resolverAddress and chain to resolve on.
the same is true for resolveAvatar, resolveName and resolveText
in line with our design principles for v5 all of these can be used in react by using the useContractRead hook.
I'd like to thank you again for your contribution, it helped us in designing the new version of the SDK with this flexibility in mind!


Update MongoDB quicstart yaml url

We came with a convention to use for yaml links. https://github.com/kubedb/docs/pull/21/files#diff-baa4f473410c72d9b66f40db520c52a7R130


Rename READY_FOR_MERGE to PULL_REQUEST_FEEDBACK

@bot-gradle test and merge

OK, I've already triggered a build for you.


执行 vue ui 命令时出现错误： ERROR  TypeError: Cannot read properties of undefined (reading 'on')

Version
5.0.4
Environment info
Environment Info:

  System:
    OS: Windows 10 10.0.22621
    CPU: (16) x64 AMD Ryzen 7 5825U with Radeon Graphics
  Binaries:
    Node: 16.15.0 - C:\Program Files\nodejs\node.EXE
    Yarn: Not Found
    npm: 8.10.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: Not Found
    Edge: Chromium (103.0.1264.44)
  npmGlobalPackages:
    @vue/cli: Not Found

Steps to reproduce
执行 vue ui 命令时出现以下错误语句：
Persisted queries are enabled and are using an unbounded cache. Your server is vulnerable to denial of service attacks via memory exhaustion. Set cache: "bounded" or persistedQueries: false in your ApolloServer constructor, or see https://go.apollo.dev/s/cache-backends for other alternatives.
ERROR  TypeError: Cannot read properties of undefined (reading 'on')
TypeError: Cannot read properties of undefined (reading 'on')
at ui (C:\Users\Administrator\AppData\Roaming\npm\node_modules@vue\cli\lib\ui.js:90:14)
at processTicksAndRejections (node:internal/process/task_queues:96:5)
What is expected?
出现的错误给出了解决·方案，但是我不知道如何寻找ApolloServer构造函数设置' cache: "bounded"或者' persistedQueries: false '
What is actually happening?
出现错误语句，浏览器没有打开可视面板

it's a new issue, not happened until yesterday.
same situation at 5.0.6, different error thrown at 5.0.7:
TypeError: server.installSubscriptionHandlers is not a function
    at module.exports (C:\Users\xxxxxx\AppData\Roaming\npm\node_modules\@vue\cli\node_modules\@vue\cli-ui\graphql-server.js:165:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async ui (C:\Users\xxxxxx\AppData\Roaming\npm\node_modules\@vue\cli\lib\ui.js:71:26)

btw, only affects ui command, npm run serve goes fine.

这是一个新问题，直到昨天才发生。 相同的情况5.0.6，不同的错误抛出5.0.7：
TypeError: server.installSubscriptionHandlers is not a function
    at module.exports (C:\Users\xxxxxx\AppData\Roaming\npm\node_modules\@vue\cli\node_modules\@vue\cli-ui\graphql-server.js:165:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async ui (C:\Users\xxxxxx\AppData\Roaming\npm\node_modules\@vue\cli\lib\ui.js:71:26)

顺便说一句，只影响 ui 命令，npm run serve一切正常。

这是一个新问题，直到昨天才发生。 相同的情况5.0.6，不同的错误抛出5.0.7：
TypeError: server.installSubscriptionHandlers is not a function
    at module.exports (C:\Users\xxxxxx\AppData\Roaming\npm\node_modules\@vue\cli\node_modules\@vue\cli-ui\graphql-server.js:165:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async ui (C:\Users\xxxxxx\AppData\Roaming\npm\node_modules\@vue\cli\lib\ui.js:71:26)

顺便说一句，只影响 ui 命令，npm run serve一切正常。

Yes, this caused all the 5.x versions I installed to be unusable

I am experiencing this issue after updating Vue-CLI as well. Downgrading back to an older version does not help.

更新 Vue-CLI 后我也遇到了这个问题。降级回旧版本没有帮助。

I can fix this by going back to 4.x, and maybe you can try it too.

That does work for me as a workaround for now, though isn't an ideal solution. Especially since the UI feature was working fine on 5.x earlier today.
From digging into vue-cli source a bit, they updated the @vue/cli-ui package's Apollo server today which seems to be causing the issue:
"apollo-server-express": "^3.9.0",
When I downgrade @vue/cli, the updated @vue/cli-ui package (which requires apollo-server-express) doesn't downgrade with it unless I go all the way back to 4.x. From npm list -g apollo-server-express on a downgraded @vue/cli, notice that apollo-server-express is still version 3.9.0:
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>

同样的问题。

如果出现当前的问题我目前的解决方案是：
方案—：
将版本回退到 4.x
方案二：
第一步：请将您的版本回退到 5.0.6 以下
第二步：删除 node_modules 依赖包文件夹
第三步：请修改 package.json 文件中的代码，以安装的版本 5.0.4为列：
@vue/cli-shared-utils": "^5.0.4"    修改为   @vue/cli-shared-utils": "5.0.4" @vue/cli-ui": "^5.0.4"    修改为   @vue/cli-ui": "5.0.4" @vue/cli-ui-addon-webpack": "^5.0.4"    修改为   @vue/cli-ui-addon-webpack": "5.0.4" @vue/cli-ui-addon-widgets": "^5.0.4"    修改为   @vue/cli-ui-addon-widgets": "5.0.4"
第四步：根据 package.json 文件重新安装依赖即可解决

If there are current problems, my current solutions are as follows:
Solution a:
Roll back version to 4.x
Solution b:
Step 1: Please roll back your version to 5.0.6 or below
Step 2: Delete the node_modules dependency package folder
Step 3: Modify the code in the package.json file as the installed version 5.0.4:
@vue/cli-shared-utils": "^5.0.4"    Modified to   @vue/cli-shared-utils": "5.0.4" @vue/cli-ui": "^5.0.4"    Modified to   @vue/cli-ui": "5.0.4" @vue/cli-ui-addon-webpack": "^5.0.4"    Modified to   @vue/cli-ui-addon-webpack": "5.0.4" @vue/cli-ui-addon-widgets": "^5.0.4"    Modified to   @vue/cli-ui-addon-widgets": "5.0.4"
Step 4: Reinstall dependencies from package.json

same problem today,, Roll back @vue/cli to 4.x will fix it


Can't see skybox on right eye.

Hello,
I've imported this plugin into my project today. After placing [CameraRig] prefab on my scene, I hit the play button.
Camera rotation and controller position worked correctly,  but There was a problem with the right eye camera.
It can't see the skybox material and there's an afterimage on the object.
It seems to be an issue about clear flag setting, but there's only one camera in the prefab and entire scene.
Now I'm a little confused. Any advice would be appreciated.

I'm still trying to solve this. It seems to be an issue about custom shaders. Because it doesn't appear when I use the default skybox.

https://github.com/ValveSoftware/steamvr_unity_plugin/issues/809#issuecomment-673780322
solved from this link. Many thanks for distributors.


Prevent Page breaks (keep with next)

Introduction
Normative references
Terms and definitions

Adding to this, as well for Titles of the above sections:

Terms and Definitions items should not break across pages (item, abbr, def, citation).

As noted in https://github.com/SMPTE/st430-5/issues/112#issuecomment-2129781224:

Cells in tables should not page break by default.


Host financial_phrasebank data on the Hub

Fix #4597.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


WIP: Polyphonic samplers

Hi,
I noticed a few things in v5 that were bothering me in v4 as well, so I though this would be a good moment to create a pull request with a suggestion. I haven't fully tested this but would like your opinion, if you agree with the direction I'm taking here.
I'm trying to make AppleSampler inherit PolyphonicNode instead of Polyphonic. Because I believe that works fine and is more useful. Is there a reason it was not polyphonic before?
Here's what I did:

I made AppleSampler inherit from PolyphonicNode
Then I noticed that stop and play methods of AppleSampler were throwing, but those of PolyphonicNode are not. I had 2 options here: (1) make methods on PolyphonicNode throw or (2) catch errors in AppleSampler and don't throw. I chose for the latter. But let me know if you think otherwise.
For the same reason, I added a channel argument to the stop method of Polyphonic. I think it's strange that play receives a channel but stop doesn't.

I will try to add a few tests for this as well.

I don't see a problem with your approach. Make sure its well tested and lets try to get it in before AK5 is more officially launched.

Without looking at the code I have a first question... Apple Sampler is polyphonic by default. So what exactly do you mean by making it polyphonic?

Most of your changes look fine and warranted tho - so good job. I just don't understand the Polyphonic thing

Thanks! I'll add tests.
It behaves polyphonic indeed, but it inherits from a Node instead of a PolyphonicNode. In my project I ran into an issue where I had a function that receives an instrumentNode: PolyphonicNode argument, but I couldn't pass an AppleSampler instance to it. So that's why I changed:
open class AppleSampler: Node {
to:
open class AppleSampler: PolyphonicNode {

I added two tests now, just for a simple sample start and stop. I'm not exactly sure how to test the inheritance of AppleSampler. Doing something like this would be a bit useless:
func testPolyphonicNodeInheritance() {     let polyphonic = AppleSampler() as? PolyphonicNode // Conditional cast from 'AppleSampler' to 'PolyphonicNode' always succeeds     XCTAssertNotNil(polyphonic) }

I now added a test that uses an inline function that needs a PolyphonicNode as its first argument. Let me know what you think of that. I'm not sure why the CI tests fail though, they succeed on my machine.
Also the Hound keeps biting me. 🐶 Which is a good thing of course.

I added some more tests. Do you have any idea why my the Resources/drumloop.wav file cannot be loaded? When I run the tests on my Mac it works fine, but somehow the CI tests can't find it. It's this line that fails:
let sampleURL = Bundle.module.url(forResource: "Resources/drumloop", withExtension: "wav")!

Recently this folder was renamed "TestResources", try that.


Move compute_tools from console repo (zenithdb/console#383)

Currently it's included with minimal changes and lives aside of the main
workspace. Later we may re-use and combine common parts with zenith
control_plane.
This change is mostly needed to unify cloud deployment pipeline:

build compute-tools image
build compute-node image based on the freshly built compute-tools
build zenith image

So we can roll new compute image and new storage required by it to
operate properly.

Fixed most of comments and put some TODOs for future. I removed the 100% dead code, but don't want to do a huge refactoring right now. I'll do another pass a bit later. Thanks a lot for the comments @LizardWizzard


Az VM repair-- Inplace upgrade

Hello Team,
We have a existing az Vm repair command to Create Rescue VM and attach the problematic disk, For performing in place upgrade through nested hyper-v, there should be add-on step for attaching ISO file with option to select Windows server 2016/ Windows Server 2019.
There is internal TSG , we wanted the same incorporated through AZ vm repair command.

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

ID: 355a594a-22ef-fd15-0b58-07701dd1815c
Version Independent ID: d8356a0a-9aaf-900b-ce7a-021c9a0af509
Content: az vm repair
Content Source: latest/docs-ref-autogen/vm/repair.yml
Service: virtual-machines
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Compute

Team wanted to check on the feedback which I had provided, curious to know if there is an ETA

@dkulkarni-ms Could you please take a look at the vm repair related issue?

This item is being discussed with engineering team

Moving Jayla's issues over to me.

Hi @yuvaraj-azurevm, we have released the ability to do an inplace upgrade directly on VMs without using VM repair. Please see the following documentation on this process: https://learn.microsoft.com/en-us/azure/virtual-machines/windows-in-place-upgrade.
With this new capability, do you still feel something is needed in VM Repair, or can I close this issue?

Moving this to closed now that in-place upgrades are generally supported for Windows Server.


fiber: fails to build on 32-bit x86 and arm

Regresed by #5459 (missed x86.rs and arm.rs). Found while packaging Veloren.
$ cargo build --target i686-unknown-linux-gnu
[...]
error[E0432]: unresolved import `wasmtime_asm_macros::asm_sym`
  --> crates/fiber/src/unix/x86.rs:13:37
   |
13 | use wasmtime_asm_macros::{asm_func, asm_sym};
   |                                     ^^^^^^^ no `asm_sym` in the root

For more information about this error, try `rustc --explain E0432`.
error: could not compile `wasmtime-fiber` (lib) due to previous error

Wasmtime doesn't work on 32bit x86 and arm32. It is not just a missing fiber implementation (that would be easy to fix), but also missing backends for both architectures in Cranelift, which is a lot harder to fix. See https://github.com/bytecodealliance/wasmtime/issues/1173 and https://github.com/bytecodealliance/wasmtime/issues/1980.

To add more on what @bjorn3 already mentioned, can you clarify what you mean by regressed by #5459? Wasmtime/Cranelift haven't had support for x86 and arm in quite some time so that PR shouldn't have regressed anything "in the large". If, however, you only need the wasmtime-fiber crate and nothing else then I can see that PR regressing the support there. Do you only need the wasmtime-fiber crate? Given the commit you linked it seems like you're packaging all of Wasmtime so while it doesn't seem like this is the case I wanted to ask to be sure. (if you need all of Wasmtime I'll close this in favor of the issues that @bjorn3 linked tracking 32-bit support)

Indeed. I've got confused by the cruft i.e., x86 and arm support in wasmtime-fiber which is trivial to fix by removing unused asm_sym. Veloren then failed in wasmtime-runtime due to missing assembly but otherwise built fine (I've stubbed the assembly using x86_64 as example to check for more errors but didn't find any).


Update to napari-yaml plugin definition

git-bob think about how to update the code in this repository to make use of napari's yml plugin definition. You find the documentation for this file format under the headline "npe2 contributions" here: https://github.com/napari/docs/blob/main/docs/plugins/advanced_topics/npe2_migration_guide.md and here: https://github.com/napari/docs/blob/142ac9a29857a5db5fcc5c3960c3d4b2c1c83073/docs/naps/6-contributable-menus.md
We basically need to convert all @register_function(menu="My menu") decorations in init.py into the mentioned yaml format, e.g.:
contributions:
  commands:
    - id: nsbatwm.my_menu
      title: My menu
      python_name: nsbatwm.my_menu

  menus:
    napari/layers/context:
      - my_menu: nsbatwm.my_menu

git-bob answer with the complete list of all functions in the yaml format.

Ok, currently the menu items are all in the same menu napari/layers/context, which should not be the case. Reaching out to @jni can you point us to the documentation of which menus are allowed?

They are listed in the NAP here:
https://napari.org/stable/naps/6-contributable-menus.html#complete-set-of-proposed-contributable-napari-menus
Actually the context menu itself is not yet contributable, sorry about that 😅
The correct syntax is
napari/[file/layers]/<lowercase_menu_name>
for each of the listed contributable menus:
File
├─ ...
├─ New Layer
├─ IO Utilities
├─ Acquire
Layers
├─ Visualize
├─ Annotate
├─ Data
├─ Layer Type
├─ Transform
├─ Measure
├─ Filter
├─ Register
├─ Project
├─ Segment
├─ Track
├─ Classify

So, two examples:

napari/file/io_utilities
napari/layers/measure

If your item doesn't fit in any of the boxes, see:
https://napari.org/stable/naps/6-contributable-menus.html#process-for-expanding-set-of-contributable-menus

git-bob think about how to revise the napari.yaml content you proposed above. Put every menu entry into a menu of the list @jni gave us.

Ok cool. git-bob implement this and make sure all register_function decorated functions in __init__.py appear in the new napari.yaml file, first as commands and second in the right menu.

Ok, let's try this again. Do not modify __init__.py keep the decorators as they are.
git-bob implement this

Note that napari/layers/utilities also doesn't exist.


memory for state

thanks Keon for your great code!
I have two questions:
1- What does [0] means in self.model.predict(next_state)[0] and return np.argmin(act_values[0])? Does this mean that first element of batch?
2-If in addition to batch, I need that my state is the state from K times before, what is the necessary change in order to do this? I want to send the state=state[i-k+1]....state[i-1],state[i] not only one state! How I can do this?
Thanks again

Keras Model API tell us that model.predict(predict_batch) returns a numpy array with an array of predictions for each element in predict_batch. Since model.predict is being called on a single array, next_state then we want the first and only element in that array. Hence '[0]'

Thanks for question 1- I did not understand how to do it?
How to have a memory on state? in nature paper the memory is used and it is equal to 4! and this is in addition to using batch

@fi000 can you provide a link to the said paper.

self.model.predict(next_state)[0] predicts on batch as @CarterEllsworth pointed out. It returns an array of predictions for each of the elements in the batch but since we're only predicting on one state element we only the first and only prediction, hence the [0].

You could somehow normalize over the last k states as I've implemented here. You'll need to adjust the dimensions according to whatever best suits your task.

You can refer to paper "Playing Atari with Deep Reinforcement Learning" section 4.1 last sentences of first paragraph! I have an state with 5 inputs but I have a problem in giving for instance 4 states in a frame!


Allow adding custom properties to layout items

Hello,
I am making widgets page, each widget has its own settings which I need to store in the database, to be able to re-render these widgets with their custom settings on a grid layout.
@STRML Your library is really good for this purpose but there is one thing that I can't handle.
I have no idea how could I store each grid item custom properties.
When I try to add custom properties to grid item in layout array onLayoutChange event is automatically removing these my custom properties.
For example, I tried to add type to grid item:
{"w":3,"h":3,"x":8,"y":0,"i":"n0", "type": "counter"},   {"w":3,"h":3,"x":3,"y":0,"i":"n1r", "type": "list"}
This would help me to determine which widget I should render on a grid because I want to have different widgets.
Do you guys have any ideas about how can I store my custom grid item properties in database if I cant add them to my layout json?
Thanks for your help.

+1
I have a similar case, my grid items can have a certain custom feature which needs to be indicated in the layout array, but it gets lost on layout change.
{ "w":3,"h":3,"x":8,"y":0,"i":"n0", "autoHeight": true }

It is not intended or supported to put arbitrary data here. Store a map
elsewhere correlating grid ids with arbitrary data.
On Wed, Apr 17, 2019, 2:55 AM Ran Byron<EMAIL_ADDRESS>wrote:

+1
I have a similar case, my grid items can have a certain custom feature
which needs to be indicated in the layout array, but it gets lost on layout
change.
{ "w":3,"h":3,"x":8,"y":0,"i":"n0", "autoHeight": true }
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/STRML/react-grid-layout/issues/929#issuecomment-483696730,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJFP-wdNp4hi9uNOOFzxo_sdER_xtSOks5vheRYgaJpZM4cmhw3
.

Fair enough 👍


Exception in thread "main" java.lang.UnsatisfiedLinkError: dev.whyoleg.ktd.TelegramRawClient.create()J

When I try to
import dev.whyoleg.ktd.Telegram
import dev.whyoleg.ktd.TelegramClientConfiguration
import kotlin.time.ExperimentalTime
import kotlin.time.seconds

@ExperimentalTime
suspend fun main() {

    val telegram = Telegram(
        configuration = TelegramClientConfiguration(
            maxEventsCount = 1000,
            receiveTimeout = 1.seconds //ms
        )
    )

    val client = telegram.client()
}

I am getting below error
Try to load tdlib as library
Exception in thread "main" java.lang.UnsatisfiedLinkError: dev.whyoleg.ktd.TelegramRawClient.create()J
	at dev.whyoleg.ktd.TelegramRawClient.create(Native Method)
	at dev.whyoleg.ktd.NativeClient.<init>(NativeClient.kt:8)
	at dev.whyoleg.ktd.Telegram.client(Telegram.kt:11)
	at MainKt.main(Main.kt:19)
	at MainKt$$$main.invoke(Unknown Source)
	at kotlin.coroutines.intrinsics.IntrinsicsKt__IntrinsicsJvmKt$createCoroutineUnintercepted$$inlined$createCoroutineFromSuspendFunction$IntrinsicsKt__IntrinsicsJvmKt$1.invokeSuspend(IntrinsicsJvm.kt:199)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
	at kotlin.coroutines.ContinuationKt.startCoroutine(Continuation.kt:114)
	at kotlin.coroutines.jvm.internal.RunSuspendKt.runSuspend(RunSuspend.kt:19)
	at MainKt.main(Main.kt)

Hi
Which version of lib do you use?
Also, is it android or jvm?

JVM ->     implementation "dev.whyoleg.ktd:ktd-api-coroutines-jvm:0.5.0-1.5.4" (which is given in the README)

Ok, are you sure, that there is no tdjni native lib in default paths? It looks, like it's even don't try to load it from artifact. If there are no line Try to load tdlib from artifact then something goes wrong. (seems will be fixed in next release)

As I am running with Ubuntu 16.04 and the library's built-in tdjni not supporting this version of Ubuntu, I had to build the so file myself and store in the default path. Reference

Hm, and how you built it?
You need to use cli module in that project to build it with right JNI file, which is different from that, which is in tdlib examples

I built the SO file using TDLib Build instruction generator

So, it's why it don't work...
You need to install it sudo apt install make zlib1g-dev libssl-dev gperf libc++abi-dev libc++-dev ccache ninja-build -y
And then, checkout that repository, in cli module there is file App.kt
change method main, so it will contain one line parser().parse("tdlib -v 1.5.4 -p jvm -t linux".cmd())
It will built right .so file in tdlib/jni/bin

Okay, I'll give it a try.

@theapache64 do you have any progress? May be some help needed?

Also built the tdlib from instructions for the macOS and after I got the lib I've also go Can't find class [org/drinkless/tdlib/Client] error.
Any solution for the Catalina 10.15.7?


feat(papi-v2): home both pipettes at once before specific h/s actions

Closes RSS-65
Overview
Switches pipette retraction before shake & open labware latch commands to use the hardware.home method in order to home both the pipettes at once instead of one at a time.
Changelog
Just the above change + test
Review requests
I've tested on a robot and verified this works. A quick code review should suffice.
Risk assessment
Low

We don't have a way to quickly retract both axes concurrently?

Ya the hardware controller API doesn't expose a method to do that although it seems like the motor driver is actually capable of "fast homing" multiple axes at once. Created RSS-72 to add capability to fast retract both pipettes at once.


Example for <.title /> is instead for <.user_avatar />

I don't know anything but I expected to see an example for <.title /> in the docstring.
https://github.com/livebook-dev/livebook/blob/8e6558a83a34012cc751a44ba17b991699830475/lib/livebook_web/live/page_helpers.ex#L11

Nice catch, thanks :)


Unable to install bee on Ubuntu running on WSL

Summary
bee installation failed with error
Error: dial unix /var/lib/bee-clef/clef.ipc: connect: no such file or directory
Steps to reproduce

Followed the steps to install Bee Clef https://docs.ethswarm.org/docs/installation/bee-clef
Followed the steps to get Bee wget https://github.com/ethersphere/bee/releases/download/v0.5.3/bee_0.5.3_amd64.deb
sudo dpkg -i bee_0.5.3_amd64.deb
Execution failed after running sudo dpkg -i bee_0.5.3_amd64.deb

Expected behavior
Bee installation succesful
Actual behavior
user@DESKTOP-S00B9E3:~$ sudo dpkg -i bee_0.5.3_amd64.deb (Reading database ... 32170 files and directories currently installed.) Preparing to unpack bee_0.5.3_amd64.deb ... Unpacking bee (0.5.3) over (0.5.3) ... Setting up bee (0.5.3) ... Error: dial unix /var/lib/bee-clef/clef.ipc: connect: no such file or directory dpkg: error processing package bee (--install):  installed bee package post-installation script subprocess returned error exit status 1 Errors were encountered while processing:  bee
Using Ubuntu 20.04.2 LTS running on Windows Subsystem for Linux (WSL).

It seems to me that bee-clef was not correctly installed. Can you confirm that bee-clef was running?
This to me does not seem as a problem with bee, or bee-clef , but rather a user error. We would be very happy to help you in our Discord!


Data import

Basic information

My operating system: windows 64bit
My MZmine version: 3.2.8

What happened: I can't import raw data file; they are data from Thermo wexactive in full MS
the following message appear : child processexited with code<PHONE_NUMBER>
Thanks for supporting

Please upload a .raw file that fails and we can take a look at it.

AA_T0_1a.raw
https://drive.google.com/file/d/1Zj8WygVpN_INqGJ_FGYHcrGVBf7kMF4f/view?usp=drive_web
Il giorno mar 11 ott 2022 alle ore 13:36 Tomáš Pluskal <
@.***> ha scritto:

Please upload a .raw file that fails and we can take a look at it.
—
Reply to this email directly, view it on GitHub
https://github.com/mzmine/mzmine3/issues/937#issuecomment-1274543840,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3RSACAJFRXTJNZZB24FOP3WCVGMXANCNFSM6AAAAAARCGDMUM
.
You are receiving this because you authored the thread.Message ID:
@.***>

I send the file by mail; I want to tell that I can upload one file but when I upload more that 10 file like this there are problem


Add couchers logo on unauthenticated pages

As in title - namely the couchers logo with the beta badge on the landing page, login page and sign up page.

Frontend checklist

[x] Formatted my code with yarn format && yarn lint --fix
[x] There are no warnings from yarn lint
[x] There are no console warnings when running the app
[ ] Added any new components to storybook
[ ] Added tests where relevant
[x] All tests pass
[x] Clicked around my changes running locally and it works
[x] Checked Desktop, Mobile and Tablet screen sizes

Sorry found another unused React import since you mentioned it so need the approval again


[Intermediate][Develop]add feature that showing required time when write output file

I would like to work on this issue.


Adding chest-xray-covid model

The COVID Chest X-Ray model is a lightweight Deep Learning model for image classification where the goal is to predict from a Chest X-Ray if a person might have COVID. Currently, only the TF Lite implementation of the model is added.

Hi @Rishit-dagli,
Since this is your first time contributing to the TFhub.dev repository, we just need you to read & agree to the TensorFlow Hub terms of service at https://tfhub.dev/terms. If you feel comfortable moving forward, please add a comment "I have read and agree to the TensorFlow Hub Terms of Service" to this thread.

Hi @maringeo ,
I have read and agree to the TensorFlow Hub Terms of Service

A query - Since, the validator does not support format: lite is there anything else I would need to do to specify so.
I also had another query about publishing for this same model which was not covered clearly in the publishing documentation. Let's say I also want to add TF 2.0 Saved Model as I understand I would have a model directory like this?
chest-xray-covid
├── default/lite
 |       └──  1.md
├── saved_model_2
 |      └──  1.md
├── 1.md

A query - Since, the validator does not support format: lite is there anything else I would need to do to specify so.
I also had another query about publishing for this same model which was not covered clearly in the publishing documentation. Let's say I also want to add TF 2.0 Saved Model as I understand I would have a model directory like this?
chest-xray-covid
├── default/lite
|   └──  1.md
├── saved_model_2
|   └──  1.md
└──  1.md

format is a confusing name. It can be either <!-- format: saved_model_2 --> if exported from TF2 SavedModel or <!-- format: hub --> if exported from TF1.
If you want to add a TF 2.0 SavedModel, you can delete the current Placeholder MD and set the parent-model of the TF Lite MD to be the TF2 SavedModel MD.
That would result in the following directory structure:
chest-xray-covid
-- default/lite
-- --1.md
-- 1.md (this is the SavedModel MD)

A query - Since, the validator does not support format: lite is there anything else I would need to do to specify so.
I also had another query about publishing for this same model which was not covered clearly in the publishing documentation. Let's say I also want to add TF 2.0 Saved Model as I understand I would have a model directory like this?
chest-xray-covid
├── default/lite
|   └──  1.md
├── saved_model_2
|   └──  1.md
└──  1.md

format is a confusing name. It can be either <!-- format: saved_model_2 --> if exported from TF2 SavedModel or <!-- format: hub --> if exported from TF1.
If you want to add a TF 2.0 SavedModel, you can delete the current Placeholder MD and set the parent-model of the TF Lite MD to be the TF2 SavedModel MD.
That would result in the following directory structure:
chest-xray-covid
-- default/lite
-- --1.md
-- 1.md (this is the SavedModel MD)

Thank you, this solves it but a quick follow up question on this let's say I want to add two more model types, how would I go about that, would I use something like this?
-- default/lite
-- --1.md
-- coral (let's say)
-- --1.md
-- 1.md (this is the SavedModel MD)

Thank you, this solves it but a quick follow up question on this let's say I want to add two more model types, how would I go about that, would I use something like this?
-- default/lite
-- --1.md
-- coral (let's say)
-- --1.md
-- 1.md (this is the SavedModel MD)

The typical structure is
--default/lite
-- -- 1.md (lite MD)
-- default/coral
-- -- 1.md (coral MD)
-- default/tfjs
-- -- 1.md (tfjs MD)
-- 1.md (SavedModel MD, or Placeholder MD if there is no SavedModel MD)
We're working on documenting this better (and also simplifying the structure). Sorry for the inconvenience.

Thank you, this solves it but a quick follow up question on this let's say I want to add two more model types, how would I go about that, would I use something like this?
-- default/lite
-- --1.md
-- coral (let's say)
-- --1.md
-- 1.md (this is the SavedModel MD)

The typical structure is
--default/lite
-- -- 1.md (lite MD)
-- default/coral
-- -- 1.md (coral MD)
-- default/tfjs
-- -- 1.md (tfjs MD)
-- 1.md (SavedModel MD, or Placeholder MD if there is no SavedModel MD)
We're working on documenting this better (and also simplifying the structure). Sorry for the inconvenience.

Thank you, this solves the query.

Hi @Rishit-dagli , sorry I missed this in the previous review iteration - is it possible to extend the Lite MD with a brief description of the model performance (for example ROC, precision-recall or another performance metric that you've used when training/evaluating the model)?

Hi @Rishit-dagli , sorry I missed this in the previous review iteration - is it possible to extend the Lite MD with a brief description of the model performance (for example ROC, precision-recall or another performance metric that you've used when training/evaluating the model)?

@maringeo  This would definitely be helpful to others trying to use this model, Thanks for pointing that out. I have added the model Precision/ Recall curve and the Confusion matrix for trained Lite model.

Hi @Rishit-dagli , thank you for contributing to TF Hub.
The model is live on https://tfhub.dev/rishit-dagli/lite-model/chest-xray-covid/1/default/1.
This PR has been merged in 47972a85945f356da62e1dea732b7ddabf937e03.

@maringeo I just noticed TF Hub also rendering the model metadata on the page, looks really cool!

@Rishit-dagli congratulations on this one. Some thoughts:

There's a minor typo I noticed - "Lightweight deep learning model for clasifying chest xrays as COVID or not."
If you have a demo app already it's probably better to have that linked from your TF-Hub model page.

@maringeo I see that the dataset property is missing from the model card. Is this issue known?

@Rishit-dagli congratulations on this one. Some thoughts:

There's a minor typo I noticed - "Lightweight deep learning model for clasifying chest xrays as COVID or not."
If you have a demo app already it's probably better to have that linked from your TF-Hub model page.

@maringeo I see that the dataset property is missing from the model card. Is this issue known?

Thanks, @sayakpaul for pointing out these errors

I will update this PR with the typo being fixed
I have a demo app but I would need to work on the UI part a bit to make it usable so I will open a PR in the future with these changes

About the dataset property, the TF Hub metadata mentions the dataset name as chest-xray-covid and I can see the model listed here from the dataset search but I can't see the dataset poperty on the model page, maybe that needs to be fixed.

Enlisting a not-fully-worked out APP (i.e. not so cool UI) usually works as long as it demonstrates the model usage in a systematic manner.

Glad to hear you like the TFlite metadata preview!

I have seen that TF Hub also supports having an Open in Android Studio Button to use with the model binding plugin, since the docs do not mention anything about that could you also share details about that.

This just means you can download this model and import it into Android Studio with ML Model Binding. Learn more at https://developer.android.com/studio/preview/features#tensor-flow-lite-models

I see that the dataset property is missing from the model card. Is this issue known?

Yep, known issue, we'll try to fix it for this model this week and get it fixed generally in the next few weeks since it happens frequently for new datasets

Cool @kempy. Looking forward to those updates. Metadata preview is definitely a game changer given the usefulness it offers.

Glad to hear you like the TFlite metadata preview!

I have seen that TF Hub also supports having an Open in Android Studio Button to use with the model binding plugin, since the docs do not mention anything about that could you also share details about that.

This just means you can download this model and import it into Android Studio with ML Model Binding. Learn more at https://developer.android.com/studio/preview/features#tensor-flow-lite-models

I see that the dataset property is missing from the model card. Is this issue known?

Yep, known issue, we'll try to fix it for this model this week and get it fixed generally in the next few weeks since it happens frequently for new datasets

@kempy Thank you! I had made an Android project with this model (a basic one) I will very soon add it.


[id] Fix mismatch in Labels and Selectors concept page

en PR https://github.com/kubernetes/website/pull/39205/files
/assign @girikuncoro

@Arhell any idea why the checks are failing?

@Arhell any idea why the checks are failing?

Nope, but i re-push commit


Correction of module location

Fix for error:
module declares its path as: github.com/helloeave/json
        but was required as: github.com/homelight/json

Do you know if this breaks older imports? There are a few people using this package https://github.com/search?p=1&q="github.com%2Fhelloeave%2Fjson"&type=Code. I'd have to test this scenario. Maybe it can be solved by adding a line to the readme explaining that the package was moved to a different organization.

I'll include this as part of the next release when I publish the latest releases for each Go version.


Make ProcessCommonJSModules, ProcessEs6Modules and ES6ModuleLoader public

For this year's GSoC project, we are planning to add CommonJS, ECMAScript 6 and AMD module support to ClojureScript and would like to use functionality that is already provided by the Google Closure Compiler. Instead of using this functionality indirectly through the Compiler class we would like to use the ProcessCommonJSModules, ProcessEs6Modules and ES6ModuleLoader classes directly to make it easier for us to support incremental compilation.
I have signed the CLA and David Nolen, who is my mentor for this project, asked about this change on the mailing list [1].
[1] https://groups.google.com/forum/#!topic/closure-compiler-discuss/C8yfc6ImDkI

ClojureScript is going to depend on the Closure Compiler? This is going to get confusing. I forsee hearing the phrases "Clojure with a J" and "Closure with an S" a lot more often :)

@MatrixFrog haha ClojureScript has actually always depended on the Closure Compiler, and yes we do have that problem :) @MNeise thanks for the work!

I've discovered recently that Google Voice will recognize "Clojure" if I pronounce it like "sojourn," ie /ˈkloʊ·dʒɜr/ (taking a stab there, I'm not much of a phonetics geek). That's valuable enough that it's tempting to shift toward that pronounciation...

@Mneise go for it!


Entity Inventory Type #22

Description
Entity Inventory Type should be created. Inventory has the following types: laptop, PC, phone, monitor, powerbank, headphone
Entity should have the following properties:

[x] 'name: String' -- Unique identifier of the Inventory Type.
[x] 'desc: text' -- Description of the Inventory Type

UI changes:

[x] Centre created for Inventory Type
[x] Master created for Inventory Type

Expected outcome
Inventory Type Entity is created and prepopulated in Database. Can be viewed and edited.

Done


IPP version definition change

IPP_VERSION_X100 was changed to:
IPP_VERSION_MAJOR * 100 + IPP_VERSION_MINOR*10 + IPP_VERSION_UPDATE
to manage changes between updates more easily;
IPP_DISABLE_BLOCK was added to ease tracking of disabled IPP functions;
Overall version usage was unified across library;

@lupustr3 Please fix build warning

:+1:


FIX: compiling bindtextdomain i386 on core20 snaps

fixes #31

huh, such a simple fix. well done for working this out, I was flummoxed!


Add support to ask for more types of variants (more complex InDels and duplications)

Proposal by Michael Baudis, please elaborate if insufficient.
For example:
INS[ATGC]+
DEL[0-9]*
DUP
Discussion on interpretation and use cases was already started in this document: https://docs.google.com/document/d/1PfSt0o0m59BRs92PtyDcP31fUl8QgMYSTiclHAXCG0s/edit?usp=sharing

Comments from the Google Doc for reference:
Miro Cupak (4:03 PM Mar 28): The description [of alternateBases] refers to the VCF spec. Is there ambiguity?
Michael Baudis (4:11 PM Mar 28): DEL (or <DEL>); DUP (or <DUP>) ...
Heinz Stockinger (8:02 AM Mar 29): ALT field looks ok. We might even consider to add the INFO field such as "ALT;INFO" (i.e use ";" to separate ALT and INFO). Then we can have examples such as: "<DUP>;SVTYPE=DUP;END=12686200;SVLEN=21100;CIPOS=-500,500;CIEND=-500,500"

+1 to include that in version 0.4

+1
Heinz Stockinger wrote:

+1 to include that in version 0.4
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ga4gh/beacon-team/issues/20#issuecomment-225908638,
or mute the thread
https://github.com/notifications/unsubscribe/AI_EVCKdQWRzPP7H4O182LALcUM5H8PGks5qLsEogaJpZM4IDB-p.

+1
On Tue, 14 Jun 2016 at 16:57 antbro<EMAIL_ADDRESS>wrote:

+1
Heinz Stockinger wrote:

+1 to include that in version 0.4
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ga4gh/beacon-team/issues/20#issuecomment-225908638,

or mute the thread
<
https://github.com/notifications/unsubscribe/AI_EVCKdQWRzPP7H4O182LALcUM5H8PGks5qLsEogaJpZM4IDB-p
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ga4gh/beacon-team/issues/20#issuecomment-225908985,
or mute the thread
https://github.com/notifications/unsubscribe/AHsiOlqtZEFzMel7QoN5TzSX44Fw4ZtCks5qLsFngaJpZM4IDB-p
.

+1

+1

Obviously +1 on this. Additional elaboration:
The CNV/CNA space (basic description of regional copy number imbalances vs. standard reference genomes, is a supremely suitable first extension of the current variant representation schema:

high biologic relevance, both in genetics and cancer
simple structure for basic query/representation
represented in VCF
(presumed) low impact for re-identification

I am not overly concerned regarding specific privacy issues. Obviously, any additional datapoint in principle can provide a point of attack for re-identification attempts. However, the number of rare CNVs per sample is comparatively low; it is not trivial to query base-specific CNV boundaries (and those may freq. be approximate); somatic CNV/CNA (e.g., cancer) are currently not considered critical (see e.g. ICGC, where computed copy number is fully open).
Anyway, the evaluation of possible re-identification issues is deferred to the implementer of the Beacon resource.
The only open issues right now are IMHO specifics, e.g. how overlap queries & imprecise boundaries are defined & implemented, as well as how to query/return CN levels (i.e. granular, integer options beyond DUP/DEL).

Hello Michael,
Could you please provide an example of what you mean by "... how to query/return CN levels (i.e. granular, integer options beyond DUP/DEL)." Thanks, Heinz

@heinzstockinger Copy number variations can have different quantitative levels. Based on a 2n allele count, deletions can lead to 1n or 0n (homozygous). For duplications there is no upper limit; in canver genomes, amplicons with hundreds of repeats of the same sequence can be found (sometimes including one or more complete CDRs; an example here is MYCN).
There are reasons to query specific  copy number levels, e.g. to find only homozygous deletions.
The VCF file format allows to provide this information through FORMAT => CN ("Copy number genotype for imprecise events"); see pp. 13/14 of VCF 4.3.
Calling numerically correct copy numbers is difficult (especially in cancer w/ mixed cellularity etc.), and frequently data contains just DUP/DEL information instead of integer count values, with the possible addition of HOMODEL (i.e. 0n) and AMP (i.e. passing a arbitrary threshold, e.g ≧ 4).
While there are clearly use cases for this kind of granularity, implementation adds some complexity which makes only sense when there are repositories actually providing this type of data & not only the theoretical urge to do so (e.g. while we work on this for arraymap.org, integer CN calls are not implemented yet).
Conclusion:

At least for 0.4 implement qualitative DUP/DEL calls.
Keep in mind a future extensibility towards integer CN thresholding.

In order to be consistent, we should have:
INS[ATGC]+
DEL[0-9]*
DUP[0-9]*

+1

I don't understand the issue
If I'm not the only one, perhaps chat it through in a Beacon TC ?
Tony
Heinz Stockinger wrote:

+1
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ga4gh/beacon-team/issues/20#issuecomment-230427033,
or mute the thread
https://github.com/notifications/unsubscribe/AI_EVAYvkq8YFxR2U54CKiLrn7ArH0RGks5qSiAzgaJpZM4IDB-p.

It's just a small change with respect to the proposal at the top of the page, i.e., we had:
INS[ATGC]+
DEL[0-9]*
DUP
we now propose to update it to:
INS[ATGC]+
DEL[0-9]*
DUP[0-9]*
i.e. only adding [0-9]* to DUP - so both, DUP and DEL have the possibility for integer values
(as it is already the case in the current v0.3 and earlier specifications).

Aha - thanks!
+1
T
Heinz Stockinger wrote:

It's just a small change with respect to the proposal at the top of
the page, i.e., we had:
INS[ATGC]+
DEL[0-9]*
DUP
we now propose to update it to:
INS[ATGC]+
DEL[0-9]*
DUP[0-9]*
i.e. only adding [0-9]* to DUP - so both, DUP and DEL have the
possibility for integer values
(as it is already the case in the current v0.3 and earlier
specifications).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ga4gh/beacon-team/issues/20#issuecomment-230438376,
or mute the thread
https://github.com/notifications/unsubscribe/AI_EVKHqydasshsiGIgqaiivo-RRUvuXks5qSixDgaJpZM4IDB-p.

Hi All
I'd like to reflect on some basics about 'what is a Beacon', and hence
thereafter decide on what to turn it into...
Currently Beacon asks about 'a single base allele'. Originally this meant
(1) "any subject-specific record where the query allele is present"
(whether heterozygous or homozygous)
But we now allow people to use Beacon to ask
(2) "any database record referring to the query allele" (could be
population frequency data, protein structure or pathogenicity
consequences, animal model correlates, etc)
We decided in Hinxton last week that both (1) and (2) are acceptable
Regarding queries that focus on records about the properties of human
subjects (as opposed to the properties of variants) we have never yet
tried to enable queries to distinguish between standard genotypes
(homozygous or heterozygous presence of the query allele), but if we did
this could quickly expand into asking about zygosity generally (e.g.,
hemizygous, Y chm markers or X markers in females, polyploidy, etc)
...and that would open a way into a general solution for genome counts
of an allele (which could be fractional, ranges, >, < etc)  ...which
then, if openned up to variants other than single base changes, provides
a way to handle copy number variation.
In parallel we'd also want a way to specify a query on a local haplotype
(i.e., one chm rather than one genome level)
SO I PROPOSE WE ENABLE QUERIES THAT SPECIFY CLEANLY AND SIMPLY

TYPE OF ALLELE
COUNT_IN_GENOME
COUNT_IN_HAPLOTYPE

Beyond this, I'd like to see the Beacon query language able to ask but
whether data/records exist that relate to a genome region defined by a
start and stop base (which could be one and the same), and how those
data/annotations match the target region
(exact|exceed|begin_between|end_between|begin_and_end_between|only_begin_between|only_end_between|begin_at_start|end_at_stop)
Plus a search option for specific sequence strings
Then one could easily imagine combinations of the above, eg:
'SNP allele' located 'exact' at 'Chm:2/start_base:6543' and
'Chm:2/stop_base:6543'
'SNP allele' located 'exact' at 'Chm:2/start_base:6543' and
'Chm:2/stop_base:6543' with 'Count_In_Genome > 0.5'
'TTAGGAGG' located 'begin_between' 'Chm:2/start_base:6543' and
'Chm:2/stop_base:6553'
'Copy number variant allele X' with 'Count_In_Genome > 4'
'Copy number variant allele X' with 'Count_In_haplotype > 2' where
haplotype at 'Chm:2/start_base:5,000' and 'Chm:2/stop_base:100,000'
Thoughts...?
Tony
Michael Baudis wrote:

@heinzstockinger https://github.com/heinzstockinger Copy number
variations can have different quantitative levels. Based on a 2n
allele count, deletions can lead to 1n or 0n (homozygous). For
duplications there is no upper limit; in canver genomes, amplicons
with hundreds of repeats of the same sequence can be found (sometimes
including one or more complete CDRs; an example here is MYCN).
There are reasons to query specific copy number levels, e.g. to find
only homozygous deletions.
The VCF file format allows to provide this information through
|FORMAT| => |CN| ("Copy number genotype for imprecise events"); see
pp. 13/14 of VCF 4.3.
Calling numerically correct copy numbers is difficult (especially in
cancer w/ mixed cellularity etc.), and frequently data contains just
DUP/DEL information instead of integer count values, with the possible
addition of HOMODEL (i.e. 0n) and AMP (i.e. passing a arbitrary
threshold, e.g ≧ 4).
While there are clearly use cases for this kind of granularity,
implementation adds some complexity which makes only sense when there
are repositories actually providing this type of data & not only the
theoretical urge to do so (e.g. while we work on this for
arraymap.org, integer CN calls are not implemented yet).
Conclusion:
* At least for 0.4 implement qualitative DUP/DEL calls.
* Keep in mind a future extensibility towards integer CN thresholding.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ga4gh/beacon-team/issues/20#issuecomment-226687963,
or mute the thread
https://github.com/notifications/unsubscribe/AI_EVDjBma1psFWBHbckkhpIsSSpoKOXks5qMjmigaJpZM4IDB-p.

@antbro Maybe you move this to a separate doc which can be edited/commented on? I think it would be best to have the specific use cases listed & discussed, which is tricky with this format here on Github.
(overall your examples are in my line of thinking)

For info linked here, a write-up of options for range queries, using a VCF:INFO approach (but pointing to alternative use of other attributes):
https://docs.google.com/document/d/1uePLlLMl0FzxZxDrsF9IxsC2nYvZ84029fzUD1ULNWI/edit#

I was in believe that in version 0.4 we will implement  complex variants like the ones in the document https://docs.google.com/document/d/1uePLlLMl0FzxZxDrsF9IxsC2nYvZ84029fzUD1ULNWI
Am I wrong?

I believe the decision made on today's call was to implement the first step as described in https://github.com/ga4gh/beacon-team/issues/20#issuecomment-230438376 and put off the changes proposed in the document above to a later time.

There would just be the additional field to add: alternateBasesInfo. It's an optional parameter so it is completely backwards compatible.
Details are in the following pull request: https://github.com/ga4gh/beacon-team/pull/65

To summarize related decisions made during the workshop yesterday:

Instead of using arrays for start and end, we use individual explicitly named fields for both sides of the intervals.
CNV positions should be separate from SNP positions. I.e. we'll use start, start_min, start_max, end_min, end_max. The min/max nomenclature received +1s in a follow-up email thread.
We'll extend existing allele request instead of implementing a separate endpoint.

We're going with https://github.com/ga4gh/beacon-team/pull/94 over https://github.com/ga4gh/beacon-team/pull/95 as the base for implementation.

@mcupak I've added a comment to the DUP,DEL... PR https://github.com/ga4gh/beacon-team/blob/develop-proto-structural_and_ranges/src/main/proto/ga4gh/beacon.proto#L50
Actually, on re-reading VCF the reference value can stay "required", since values of A,C,G,T,N are permitted (this is conceptually slightly different from a . as recommended for a missing value, but practically the same).
Is this sufficiently verbose?
  // Reference bases for this variant (starting from `start`).
  //
  // Accepted values: see the REF field in VCF 4.2 specification
  // (https://samtools.github.io/hts-specs/VCFv4.2.pdf).
  // When querying for variants without specific base alterations (e.g.
  // imprecise structural variants with separate variant_type as well as
  // start_min & end_min ... parameters), the use of a single "N" value is
  // recommended.
  string reference_bases = 8;

Closing since implemented in develop-proto branch.


logging changes

Changes to remove the use of the stdlib singleton logger.

I think log is the best name for the variable. Using l is too short and anything else is an attempt to not use log. Let's keep it.

Looks good on 1st pass


Add Discussion Notice in Celo CLI that invites user feedback

Component
@celo/celocli
Describe the feature you would like
when running a command in celo cli display message about discussion of future of cli and what commands are most valuable
Additional context
No response

@arthurgousset to confirm if we do now or later

I didn't get a chance to ask Mariano for input, but I propose we put together a draft PR that implements a version of this, and then show the impact this change would have using a beta (or other canary) from the PR.
That way he can suggest variations/discourage this entirely based on what it would look like.

assigning to me to remind me to create a draft pr


unocero ads

 unocero.com google ads showing up
they show up in small res, in pc you need to do f12, toggle device toolbar (ctrl + shift + M) select Pixel 2 XL, then refresh the page and just in case you're also using abp extension, disable it.

Bruno @brunoherrera I'm not seeing this problem here, regardless of device width or emulation, Pixel 2 XL or otherwise.

@StevenBlack ok, sorry I can't debug this further, but this doesn't happen to me when I use 1hosts.cf, only with your hosts

Closing.


Import any kind of geospatial files

https://observablehq.com/@neocartocnrs/geoimport

Now that we use DuckDB, we would use the spatial extension https://duckdb.org/docs/extensions/spatial


[Feature] Insert planner adapt to iceberg table sink

What type of PR is this：

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
https://github.com/StarRocks/starrocks/issues/21502
Problem Summary(Required) ：

we define partition(k1=1) in insert statement to KeyPartition which ref to the https://mariadb.com/kb/en/key-partitioning-type/
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:disappointed: fail : 47 / 94 (50.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/planner/IcebergTableSink.java
0
21
00.00%
[41, 42, 43, 44, 45, 46, 47, 48, 49, 50, 52, 53, 55, 56, 58, 62, 67, 72, 77, 82, 87]

:large_blue_circle:
com/starrocks/catalog/IcebergTable.java
0
2
00.00%
[134, 135]

:large_blue_circle:
com/starrocks/sql/InsertPlanner.java
19
41
46.34%
[126, 210, 211, 212, 257, 308, 516, 517, 518, 519, 520, 521, 522, 524, 525, 526, 527, 528, 529, 530, 531, 533]

:large_blue_circle:
com/starrocks/sql/ast/PartitionNames.java
6
8
75.00%
[92, 93]

:large_blue_circle:
com/starrocks/sql/parser/AstBuilder.java
10
10
100.00%
[]

:large_blue_circle:
com/starrocks/sql/analyzer/InsertAnalyzer.java
10
10
100.00%
[]

:large_blue_circle:
com/starrocks/sql/ast/InsertStmt.java
2
2
100.00%
[]


Adding an action and preset to leave the currently active Voicechannel

Resolving one part of my feature request  #17
I sadly haven't worked with ts ever before so I hope everything checks out.

Looks good to me, the only thing I'm going to add is that the interface for the action should have options: Record<string, never>, as even if the action doesn't require any options there'll still be an options object on callback that'll just be empty.


Use CenterFormer on other datasets

Hello, really impressive work!  I wonder whether I could use the method on other datasets apart from WOD and Nuscenes. I want to use my own dataset for which I already wrote a data pipeline that works. So if I want to make CenterFormer to have the cheering performance on my own dataset, is there any essential change that I should or it's ready for that?

Hi there! To use the code on your custom dataset, you'll need to write a new dataset class and converter. You can refer to either of these two examples: waymo and nuscenes. However, you should be careful about the coordinate system you use, as it may differ from the default used in the examples.

Hello, really impressive work! I wonder whether I could use the method on other datasets apart from WOD and Nuscenes. I want to use my own dataset for which I already wrote a data pipeline that works. So if I want to make CenterFormer to have the cheering performance on my own dataset, is there any essential change that I should or it's ready for that?

hello, have you ever tried centerformer on kitti dataset? thank you.


chore: Fix admission filters wip

Description
Fixing the Admission Filters WIP
End to End Test:
(See Pepr Excellent Examples)
Related Issue
Fixes #

Relates to #
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Other (security config, docs update, etc)

Checklist before merging

[ ] Unit, Journey, E2E Tests, docs, adr added or updated as needed
[ ] Contributor Guide Steps followed

fixed in #1396


It fails to import

import ValidateableForm from 'react-form-validate'; causes failure on
import vForm from './source/ValidateableForm.react.jsx';

export default vForm;
module.exports = vForm;

Library seems not properly packaged.

Thanks for the feedback...
I didn't realize that before when I use it in my own project...
Fixed that now, try again?

Problem not solved. I got same error.

Problem still exists. I got same error.
Am using React latest version v16.1.1


The Stop button doesn't work on test cases

Description
Hello,
I have a test project in which I am using a parameterized test method to which I am providing input using the TestCaseSource attribute. After I open the assembly containing my test in TestCentric I can see all the test cases but if I run them all (or just a subset) the Stop button does not work:
When clicking the button the execution does not stop when the current test case finishes; Furthermore, if I click the Kill button nothing happens. If I click the Kill button again the button becomes disabled, the test status changes to Failed and the UI freezes for a couple of seconds. Then the execution continues and at the end an exception occurs.
Steps to reproduce the behavior:

Load the TestCasesProject.dll from the attached folder into TestCentric
Click on Run All Tests
After a couple test cases pass click on the Stop button => the run doesn't stop
Click the Kill button => nothing happens
Click again the Kill button => the UI freezes for a couple of seconds, the test status changes to Failed and then the execution continues; an error appears at the end of the execution.

Expected behavior
When clicking the Stop button I expect the test execution to stop after the running test case ends.
When clicking the Kill button I expect the test execution to stop the running test case and the test execution.
Environment:

TestCentric Version 2.0.0-alpha4
OS: Windows 10
Target Runtime for your test assembly: .Net 5
Nunit Version 3.13.3
Nunit3TestAdapter Version 4.2.1

I attached a small project that can be used to reproduce the issue
TestCasesProject.zip

In the past, it has been the responsibility of the framework itself to cancel the test run when asked. With changes in .NET 5.0 and later, it's no longer possible to do this and the framework simply ignores force: true. See issue nunit/nunit#3276, which I filed in 2019.
In general, it's not possible for the engine  to know what a particular framework is capable of doing. However, such knowledge can (and should be) built into the driver for each framework it knows about, especially the NUnit V2 and 3 frameworks! So, in general, such knowledge belongs in a driver.
However, in this case, the engine runner should know that a forced stop is impossible under .NET 5.0 or greater, so this particular bit of knowledge could be built into the runner itself.
I'll experiment putting it in both places (runner and driver) and see which looks cleaner.

Blocked while waiting to here how framework team plans to handle this issue.

Tasks:
This change is most readily tested using the GUI, which uses the engine package. So I'll do it as a series of small changes in separate PRs. Each PR will be merged and the GUI will be modified to use the new dev build. It's tedious but safe!

[ ] Modify .NET Standard driver to throw an exception if StopRun(true) is called.

It turns out that the Kill button stopped working, even for .NET Framework, in alpha 5. This is an additional problem on top of the one reported here, but it's moot anyway, since we'll be using Process.Kill for all frameworks.

StopRun(true) in response to the Kill button was working in commit b85bec4, merged on April 17. Commit 1a49fbb broke it the following day. I've spent a bit of time on this and it's not easy to see exactly why. Since the plan is to re-implement StopRun(true) anyway, I'm not following up on the reasons for the failure any further.

Normally, I'd save the UI for last. However, the Stop/Kill buttons in the GUI needed some fixes in order to make it easier to run manual tests, so i did the UI first.

It's conceivable for a driver to handle StopRun(true) correctly. However, the plan at this point is to no longer expect any driver to do so. That simplifies things if we are cancelling a multi-assembly run with different drivers being used.
With that said, the approach I am implementing will work as follows...

Our built-in drivers will throw an exception if StopRun(true) is called. Note that we can't eliminate the argument because we make use of the nunit api, which includes it.

AssemblyRunner will terminate the process when the method is called. All other runners will continue to pass the call on to subordinate runners until AssemblyRuner is reached.

Hello,
Thank you for working on this bug with a high priority. I'll check the behavior as soon as the alpha6 release is ready.

:tada: This issue has been resolved in version 2.0.0-alpha6 :tada:
The release is available on:

GitHub Release
NuGet Package


OpenXR error

FS2020 will not transition to VR and I sometimes get an OpenXR error, mostly I do not. Installing PimaxXR seemed to have broken my SteamVR (I probably broke it) and remains broken despite reinstalls, beta versions and registry edits. I had transitioned from OpenXR Toolkit 1.1.0. It seems to work 1 out of 50 times. However DCS works 100% of the time as it has nothing to do with STEAM (it seems). When I launch FS2020, my VR home environment is jittery and unstable.
Pimax Vision 8kx
pimax-openxr 1.txt

PimaxXR seemed to have broken my SteamVR (I probably broke it) and remains broken despite reinstalls, beta versions and registry edits

PimaxXR doesn't touch anything SteamVR, so I believe your issue is elsewhere.

sometimes get an OpenXR error, mostly I do not
It seems to work 1 out of 50 times

I don't understand. You say "mostly I do not [have error]" then you say it works "1 out of 50". These two statements contradict each other. Does it work most of the time or not?

I had transitioned from OpenXR Toolkit 1.1.0

1.1.0 is not supported with PimaxXR.

When I launch FS2020, my VR home environment is jittery and unstable.

If you have not entered VR in the game, then this is 100%  not related to PimaxXR.
I'm sorry but your report is very very unclear, and there is nothing actionable for me at this time.

Maybe try disabling OpenXR Toolkit for now in case it's a bad setting causing the issue?

"sometimes get an OpenXR error, mostly I do not
It seems to work 1 out of 50 times"

It does not work at all. The error message appears in the FS2020 sometimes but mostly there is no error message. Either way nothing happens.

"sometimes get an OpenXR error, mostly I do not
It seems to work 1 out of 50 times"

By this I meant as above, however on two occassions it randomly worked in VR after I changed the runtime in the regedit. I had tried several times after but could not replicate.

"I had transitioned from OpenXR Toolkit 1.1.0"

I WAS using OpenXR 1.1.0 ToolKit with DCS for a few months. It works with the 8kx. I deleted it when I found PimaxXR

As for PimaxXR and STEAM, doesn't PimaxXR replace the OpenVR runtime as per the 2 tickboxes at the top of the PimaxXR Control Centre? In that case it is affecting STEAM.
A few questions from your answers.

Could Open Composite be conflicting with PimaxXR

Does the remove function on OpenXR supposed to purge every last trace of the ToolKit? I had reinstalled it to see if it was working and all my settings were intact.

3)Why does it run perfect with a non Steam game such as DCS?

As for PimaxXR and STEAM, doesn't PimaxXR replace the OpenVR runtime as per the 2 tickboxes at the top of the PimaxXR Control Centre? In that case it is affecting STEAM.

This is just one registry key, you can set back to SteamVR. This is not affecting SteamVR per sei, it's only telling apps like MSFS or OpenComposite to use PimaxXR instead of SteamVR. If you select SteamVR from the Control Center app, it restores the 1 registry key back to what it was before.
You can also do it from SteamVR itself, by going to the Settings menu, Developer (need Advanced Settings to be shown) and click Set SteamVR as OpenXR runtime.

Does the remove function on OpenXR supposed to purge every last trace of the ToolKit? I had reinstalled it to see if it was working and all my settings were intact.

If OpenXR Toolkit is not installed then those settings are not used anyway. So it doesn't matter that they are not purged.

Why does it run perfect with a non Steam game such as DCS?

You are mixing up Steam (which is a content store) and SteamVR (which is a runtime environment for VR).
You can run DCS from inside of Steam just like you can run MSFS from outside of Steam. In your case it looks like you might have purchased MSFS from Steam rather than the MS Store but nonetheless you can run MSFS without Steam (I do it all the time), just create a Desktop shortcut that points to steam://launch/1250410/VR.  I would recommend you try that and see if it makes a difference.

Ok thanks, I will look at creating that shortcut and will test with my other steam titles.
Also, the reason I didn't change the runtime in steamvr is that if it blocks the HMD due to a critical error (which I forgot to mention), you do not get access the 'Steam VR settings' window and cannot change it.
I will try all this tomorrow and will close the query shortly after.
Thanks again


z-index of video

any idea when will be available to control the z-index of the video ? to add overlayers, controls, etc...

Can't I customize anything else different of width/height ? I couldn't set width, height, zindex, opacity...

Have you ever read the README?

The plugin implements "display, opacity, visibility, z-index, object-fit and also horizontal mirror effect with -webkit-transform: scaleX(-1); or equivalent".
As the README clearly says, use the mailing list for questions.

@ibc I have read but as I said I tried to apply it and no one worked.. only width/height..
display, opacity, visibility, z-index are not working

If you are using the latest version of this plugin all those things do work as expected. Sure.
But, as the README clearly stats, "you cannot place HTML elements on top of the video". Said that, you can do a dirty trick by setting the video opacity to `08' or whatever, as in this [demo]https://github.com/eface2face/iOSRTCApp#changes-to-the-original-apprtc-html5-code).


Better support for multi-core AMP SoCs

Is your enhancement proposal related to a problem?
Zephyr's support for AMP SoCs... works, but has several well-known usability problems due to workarounds for deficiencies in our hardware support features at the following layers:

devicetree
hardware model
build system

This is an umbrella issue that tracks the combined usage of several related enhancements to Zephyr's hardware support which are described further in individual issues listed below.
Describe the solution you'd like
I would like Zephyr to:

~support system devicetree: see #51830~ : POSTPONED. This is still a long-term goal, but we propose an initial implementation with standard devicetrees.
define a new, future-proof hardware model which works for AMP SoCs: see #51831
allow using sysbuild as a convenient entry point to the above: see #51832

I would like the implementation to be backwards compatible in the sense that merging it shall not break existing users of our current hardware support features, and shall be opt-in at the SoC (and therefore board) level.
Describe alternatives you've considered
Nordic previously tried to solve these problems in #13672 in 2019. This was rejected upstream for (valid) reasons. We have learned from this initial work and believe the above proposal can address the limitations and issues that led to its rejection.
Change history

2022-10-31: initial proposal

Architecture WG:

Presentation by @mbolivar-nordic and @tejlmand
@nashif worries about a 2-tier system for both Devicetree and boards: System DT vs DT and hw model v1 vs hw model v2
@galak agrees with @nashif that it might be confusing for users to have to deal with multiple systems
@henrikbrixandersen thinks that both (old and new) hw models and sysdt vs dt should coexist

I've moved this to v3.5 and made a note that system devicetree is not expected as part of the initial implementation.


Hive: tdp_user no access to external tables

Currently tdp_user is currently not able to create external tables, only managed ones. To make it work multiple ranger policies have to be added (for both hive and tdp_user), but an alternative is:

Change hive.server2.enable.doAs to true in hive-site.xml (allows for impersonation; this is set in inventory/group_vars)
Add Ranger policy: tdp_user access to /warehouse/tablespace/managed/hive  in hdfs (this is needed for access to managed tables, not sure if there's an other way to do it)

Cheers,

Hi @gonzaloetjo, I think it is resolved with #196. Do not hesitate to reopen or open a new issue if the problem persists.


Update Dockerfile-on-prem

You should test the pipeline in the fork repository
https://help.github.com/en/articles/fork-a-repo


Domain activation is Failed - [BRIEF DESCRIPTION]

App Id: d329w4dtlrd6ec
Region: eu-west-2
Step: Domain activation
Status: Failed

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
A clear and concise description of what the issue/question is.
Error message
If there is an error message, please include it here.
Note: Be sure to check the message for sensitive information.
Additional information
Please add any other relevant information. Please feel free to include screenshots.

It looks like this has since successfully verified and been created, please reopen this issue if you're still experiencing a problem.


OSError in the PR Workflow Test

Hi, I just found an OS error in the PRs' workflow.
Collecting huggingface-hub<0.1.0
  Downloading huggingface_hub-0.0.8-py3-none-any.whl (34 kB)
Collecting sacremoses
  Downloading sacremoses-0.0.45-py3-none-any.whl (895 kB)
Collecting tokenizers<0.11,>=0.10.1
  Downloading tokenizers-0.10.3-cp37-cp37m-manylinux_2_5_x86_64.manylinux1_x86_64.manylinux_2_12_x86_64.manylinux2010_x86_64.whl (3.3 MB)
Collecting filelock
  Downloading filelock-3.0.12-py3-none-any.whl (7.6 kB)
ERROR: Could not install packages due to an OSError: [Errno 2] No such file or directory: '/opt/hostedtoolcache/Python/3.7.10/x64/lib/python3.7/site-packages/importlib_metadata-4.6.0.dist-info/METADATA'

Probably, somebody has some idea about this error that occurred in many PRs recently.
Thanks!

It seems already solved :) on the recent commits