After changing the arrow position, there are already resolutions where the arrow does not appear in the right position (Any workaround handle this problem?)

Hi! I'm using this really cool tooltip and since I'm using a custom style the default arrow position wasn't right because it would hang in the box if I didn't have the background color set, so I thought I'd rewrite it in CSS and it worked, but there are some resolutions where it slips a few pixels.
Could this be fixed somehow in the current tooltip, I assume it slips because of the internal calculation, but I can't override that.
I made a stackblitz where you can test it, in the picture above you can see that the arrow is out of alignment, in the second picture you can see the correct expected look.
However, this only occurs at uncertain resolutions, so it is interesting.
https://stackblitz.com/edit/vitejs-vite-bdqp7e?file=src%2FApp.tsx

Thanks for your reply in advance.

The issue you're experiencing is due to setting a border attribute directly through the CSS. This breaks the position calculations.
You should use the border prop on the Tooltip component:
<Tooltip border="1px solid black" />

Btw, not sure if you're doing the same thing on your project as you've done on the sandbox you've linked, but copying the tooltip.css from the dist/ folder is not recommended, since those auto-generated class names are likely to change on another version.
If you really need to disable the style injection, follow the steps here(mainly about importing the correct CSS file).
But from what I can tell, you could probably achieve your custom style by simply overriding the existing styles. You can probably do that with just the className and classNameArrow tooltip props (but remember not to change border or opacity there, use the component props for those).

The issue you're experiencing is due to setting a border attribute directly through the CSS. This breaks the position calculations.
You should use the border prop on the Tooltip component:
<Tooltip border="1px solid black" />

Btw, not sure if you're doing the same thing on your project as you've done on the sandbox you've linked, but copying the tooltip.css from the dist/ folder is not recommended, since those auto-generated class names are likely to change on another version.
If you really need to disable the style injection, follow the steps here (mainly about importing the correct CSS file).
But from what I can tell, you could probably achieve your custom style by simply overriding the existing styles. You can probably do that with just the className and classNameArrow tooltip props (but remember not to change border or opacity there, use the component props for those).

Hi! Thank you for the detailed reply, it is now clear why it was wrongly positioned.
I still don't understand the classNameArrow part, is this an internal prop to get the style of the arrow? I searched the documentation and couldn't find such a part.
Thanks for the reply in advance, I appreciate this react-tooltip package.

I still don't understand the classNameArrow part, is this an internal prop to get the style of the arrow?

It's on the "options" page on the docs (third one from the top on the "Available props" section).
Usually library components expose a className prop for each "sub-component" (HTML elements) so you have more control over the styling. In this case, the tooltip has two "sub-components", the tooltip itself, and the arrow.
That way, you can do something like this:
JSX
<Tooltip className="my-tooltip-styles" classNameArrow="my-tooltip-arrow-styles" />

CSS
.my-tooltip-styles {
  background-color: ...;
  border-radius: ...;
}

.my-tooltip-arrow-styles {
  background-color: ...;
}

That said, you don't need to use className and classNameArrow. You can use the default selectors .react-tooltip and .react-tooltip-arrow, which we already set on the correct elements internally.
.react-tooltip {
  background-color: ...;
  border-radius: ...;
}

.react-tooltip-arrow {
  background-color: ...;
}

@gabrieljablonski Ohh nice, thanks for your help.

@gabrieljablonski Hi!
It works as you wrote in my example, but in the actual place where I use it, the arrow position is still wrong.

Do you have any more tips I should try, it's strange because it works at some point, so it's not permanent and if it's wrong and then I hover over it again it'll fix it at some point.
I deleted all the custom css so I actually use the original version, what I noticed after each build is that the class you wrote will still regenerate.
Could this be related?
Thanks for your reply in advance, I just can't figure out what the problem could be because I couldn't reproduce stackblitzen, but maybe you have some idea what the problem could be.
Bad

Good

I deleted all the custom css so I actually use the original version, what I noticed after each build is that the class you wrote will still regenerate.

I don't understand what you mean by this. Where is the class "regenerating"?
From your screenshots, you're clearly applying some custom styles. How are you doing that?
Please provide a snippet of your .css file and of your JSX where you're using the Tooltip component. Event better if you can provide a sample git repo with the issue reproducible.

@gabrieljablonski Hi! https://stackblitz.com/edit/vitejs-vite-bdqp7e?file=src%2FApp.tsx
This is specifically how the prod code is written, but for some reason the arrow positioning is wrong. I have not been able to reproduce the problem in stackblitz, but as I test I see that when I resize the page it is sometimes right and sometimes wrong, so it still seems to me that something is messing up the positioning of the arrows, although unfortunately I have not been able to reproduce the problem in stackblitz, BUT maybe you can tell me something about this that I have experienced.
Really all where I deleted the css from. I don't use the border property anywhere in the css and it's completely random when the arrow appears on the right and when it doesn't.
Thanks for your reply in advance.

Sorry for the delayed response. From the sample code you've sent, I technically can reproduce it, but only if I zoom out a lot.

Unfortunately, I don't think there's much that can be improved here. When you're working with zoomed in/zoomed out pages, the browser has to round some numbers when calculating how to render the elements.
Especially when zooming out, sometimes the calculation rounds up, when it should've rounded down, for better "visual fidelity".
We could add a "correction factor" to the arrow position based on the level of zoom on the page (for instance, moving the arrow 1 extra pixel down at 50% page zoom fixes it for me), but that doesn't seem worth it to me.
This is something we just kinda have to live with for the sake of simplicity: some styles look weird when zooming in/out. And that's ok, >99% of your users will never notice this.

Feel free to comment with anything else you'd like to add, but I don't think there's much we can improve here.

@gabrieljablonski Hi!
Thanks, I waited for your answer, but I didn't close it, but we decided to use it this way, thank you very much for your research to solve the problem.
It will be absolutely fine for us, thanks again for your help and for making this library.


Fix crash in WebCore::StyleResolver::loadPendingImages()

Version: 2.0.1 (2015.06.26) | net.artsy.artsy
Reason
No reason found. Full stack trace includes WebCore, WebKitLegacy, DataDetectorsUI, CoreFoundation, libsystem_pthread.dylib.
Link to HockeyApp

https://rink.hockeyapp.net/manage/apps/37029/crash_reasons/38385383

Closing this in anticipation of #606.


Data access vocabulary

I'm interested in using the data access vocabulary when validating schemas. I'm planning on writing a PR for jsonschema-rs to implement this. Is this functionality the maintainers would be interested in merging and if so are there any particular concerns that would make an implementation a good fit for this codebase?

Hi @alexjg
Thank you for raising this issue! This library doesn't support Draft 2019-09 (#44) yet, so the notion of custom vocabularies wasn't yet considered from the design point of view :(
Though the feature is awesome, and I'll be happy to have it here :) I believe that it could be implemented as is a keyword just inside the library, but it would be nice to have it pluggable. Also, I am not sure if it would be better to implement support for Draft 2019-09 first to have some API for adding custom vocabularies.
Let me know what do you think about it :)

Glad to hear this is of interest to you :slightly_smiling_face: . Are you imagining a kind of plugin system for implementing vocabularies or are you thinking more that each vocabulary would be implemented in-crate and we would just check that we have an implementation of the vocabulary when we load the schema?

I think that there could be some built-in vocabularies (like this one) + an API to plug them. Something like (just an idea):
let compiled = JSONSchema::options()
    .with_vocabulary(vocabularies::Data)
    .compile(&schema)

In this example, vocabularies::Data is a built-in implementation of this vocabulary. Probably it will require a new trait, and the actual with_vocabulary signature might look like this:
pub fn with_vocabulary(&mut self, vocabulary: impl Vocabulary) -> &mut Self {
    // Store it in options
}

Though, I don't know how this trait might look like yet. Likely it will be more clear once there will be some experiments with the implementation.


903 - [Geo Sites] Page section tabs for products/applications

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #903
This is implementation option 1 , where we change the page tab block to have a list of links with the correct hash in the content and for which the link text will be translated.

The code is written in such a way that if a section is added without the link in the page tab, it will still be added as a best effort, just that in the translated websites it will remain in english. This functionality can be of course removed if we don't like it*.
➕ directly in the word document - super clear where it is used and what for
➕ we already use that for other static filters (e.g. card list filters block)
➖ needs to be done for each page individually, because links need to be set for that page specifically - can be error prone
➖ if you forget one, the content is not displayed (*only if we do the removal)
Test URLs:

Before: https://main--moleculardevices--hlxsites.hlx.live/products/microplate-readers/multi-mode-readers/spectramax-id3-id5-readers1
After: https://translated-tabs1--moleculardevices--hlxsites.hlx.live/products/microplate-readers/multi-mode-readers/spectramax-id3-id5-readers1

@andreituicu Is there a way to test this on a site in another language?

@alexcarol unfortunately, no... there is slack thread with a proposal to create a branch where we cherry-pick commits that we want to test for translations: https://adobe-dx-support.slack.com/archives/C04QNDDUADT/p1689341254252569?thread_ts=1689325128.566399&cid=C04QNDDUADT.

Closing in favour of https://github.com/hlxsites/moleculardevices/pull/913


dts: stm32f302: Delete spi1 node from STM32F302X8

STM32F302X8 SoCs don't support SPI1 port,
so delete SPI1 node for these SoCs in DT.
Signed-off-by: Yannis Damigos<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master   #10742   +/-   ##
=======================================
  Coverage   53.26%   53.26%
=======================================
  Files         215      215
  Lines       25827    25827
  Branches     5695     5695
=======================================
  Hits        13758    13758
  Misses       9753     9753
  Partials     2316     2316

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 c229126...8ea914f. Read the comment docs.


Gltf vertex color is not imported correctly.

Godot version:
3.2.3 stable
Issue description:
I am using Blender 2.92 to export a simple cube with vertex color.
Steps to reproduce:

Make a cube with vertex color
Export it to glb
Import it to Godot
make a saptial material with vertex color as albedo
the color is not matched in blender

The first one is from Blender.

The second one is from Godot.

Minimal reproduction project:
test-vertex-color.zip
I made a simple project. It contains the blender file and a godot project

cc @fire

I believe this is fixed in the 3.2 release candidates. Can you test?

I believe this is fixed in the 3.2 release candidates. Can you test?

oh yeah. It is fixed. Thanks


Update omero-web-apps role, update mapr (IDR-0.3.1)

This includes https://github.com/openmicroscopy/infrastructure/pull/220 to avoid conflicts when cherry-picking.

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1127. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1128. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1129. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1130. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1131. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1132. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1133. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1134. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1135. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1136. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1137. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1138. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

PR #222 manics 'Bump IDR omero to 0.3.1-rc2, and idr-notebooks to HEAD (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1139. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Upstream changes

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1140. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Upstream changes

ansible/idr-playbooks/group_vars/omero-hosts.yml

Conflicting PR. Removed from build MANAGEMENT_TOOLS-merge#1141. See the console output for more details.
Possible conflicts:

PR #220 manics 'Install JDK on idr OMERO for jstack (IDR-0.3.1)'

ansible/idr-playbooks/group_vars/omero-hosts.yml

Upstream changes

ansible/idr-playbooks/group_vars/omero-hosts.yml

Rebased

ome101 run passing. Merging for a final test of origin/master


What training data was used for GSII model?

Thank you for releasing this useful tool.
I was wondering which AMR release was used for training the Parse GSII?

LDC2020T02 (aka AMR-3), although I've noticed that, for some reason, scores end up being about the same as with LDC2017T10 (aka AMR-2).

LDC2020T02 (aka AMR-3), although I've noticed that, for some reason, scores end up being about the same as with LDC2017T10 (aka AMR-2).

Great, thanks for the prompt response.
On Sun, Jan 17, 2021, 1:51 PM Brad Jascob<EMAIL_ADDRESS>wrote:

LDC2020T02 (aka AMR-3), although I've noticed that, for some reason,
scores end up being about the same as with LDC2017T10 (aka AMR-2).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bjascob/amrlib/issues/14#issuecomment-761815762, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AC54L5TYBL3KMLXA7DAK54TS2LTNRANCNFSM4WGBLLIQ
.

Great, thanks for the prompt response.
On Sun, Jan 17, 2021, 1:51 PM Brad Jascob<EMAIL_ADDRESS>wrote:

LDC2020T02 (aka AMR-3), although I've noticed that, for some reason,
scores end up being about the same as with LDC2017T10 (aka AMR-2).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bjascob/amrlib/issues/14#issuecomment-761815762, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AC54L5TYBL3KMLXA7DAK54TS2LTNRANCNFSM4WGBLLIQ
.


Result from optimizers

Hi!
I was working with optimizers implemented in Qibo and I think it would be better if the optimizers returns more than only the value of the function and optimal parameters for example why the minimisation process has stoped, number of function evaluation etc...
Methods like "Powell" have also the possibility in SciPy to set bounds (not in dictionary "options") but this is not possible in Qibo.
Let me know what do you think about it. Thank you all!

@lorenzoconfalonieri thanks for reporting this, could you please have a look at this PR https://github.com/Quantum-TII/qibo/pull/358 and let me know if is sufficient?

I think it's perfect, thank you!


updated README.md

Signed-off-by: Phil Helmling<EMAIL_ADDRESS>

Signed-off-by: Phil Helmling<EMAIL_ADDRESS>


Temperature measurement in T1S

My smartwatch T1S also measures body temperature. After connection to Your app I have temperature in watch switched from Celsius to Farenheit. Wearfit 2.0 has Celsius / Farenhait option in Settings next to option kilometers / miles. Could You add this option setting in app and visualisation of temperature?

Hello, I'm not sure about the commands related to temperature measurements.
What I need is the commands related to that but I can't test it since my watch doesn't have those features.
You can send me the watch ID from the DT78 app so I can have a look
Example commands: DT78 Commands

This is how user data is sent to the watch
byteArrayOfInts(0xAB, 0x00, 0x0A, 0xFF, 0x74, 0x80, user.step, user.age, user.height, user.weight, units, user.target/1000, 0x01)
AB 00 0A FF 74 80 3C 1A  8C 50 00 05 01
I think the last part is for the temperature option, can you check because my watch doesn't have that. Change to 00 and send to the watch

Hello great work with this app, the current wearfit app is so cluttered and a data farm it makes one uneasy. I was going to write the same as the original poster. If you could change from Fahrenheit to Celsius or create an option for that. The watch that I have is called "i29" found here and when I use your app, it switches to Fahrenheit but I need it in Celsius.

In the watch settings, click on the watch type and copy the ID

If you don't have such long ID, ensure the watch is not connected in the Bluetooth settings before connecting from the app

Thank you very much. This is the ID that I got: AB-00-11-FF-92-C0-04-01-00-85-0F-00-00-00-00-28-00-E2-07-80

I'm unable to get it working, can you share the watch data logs?

This is the data log: data.txt

Also, in the future, would it be possible change the weather source? This watch I have is using a weather source that is not very accurate for my location. If somehow we could set it to openweathermap, I have my own API key which I could use, that would be awesome.

I'm using an ESP32 board to simulate the watch but the Wearfit app is still not giving me the settings.

Maybe you can try sending commands from nrfConnect app.
Send these commands
AB000AFF74803C1A8C50000500

AB000AFF74803C1A8C50100500

Here is how to send the command.
https://github.com/fbiego/DT78-App-Android/issues/2#issuecomment-695774020
after sending, check if the units have changed

I sent only the first command and it worked, now I have the temperature in Celsius. I didn't try the second one ->"AB000AFF74803C1A8C50100500" I'm using the wearfit pro app from the play store. Thanks for your help on this!

Actually, I decided to try the second command to see if it would change it back Fahrenheit but nothing happened, it's still Celsius. Oh well, the first issue is mainly fixed. So thanks a lot for that. But do you by any chance know how to send weather information from openwheathermap instead of the default one from wearfit?   :D

I sent only the first command and it worked, now I have the temperature in Celsius. I didn't try the second one ->"AB000AFF74803C1A8C50100500" I used to use the "wearfit pro" app from the play store. Thanks for your help on this!

Perfect I'll try to integrate it into the app & also the weather feature

Hello, this app is no longer being maintained. The weather feature has been added to the new app Chronos, available on playstore

I downloaded the app today it works and connect to my watch fine and is able to change the temperature from F to C or vice versa but as for pulling the temperature data, it is unable to do so. I have to use the old annoying app wearfit pro to do it


Add support for referencing current parition

This makes it possible to reference the current partition from within a CDK stack (using Aws.PARTITION). This is accomplished by adding a defaultPartition attribute to the PolicyStatementBase class. In the CDK this is the Aws.PARTITION variable, otherwise, it retains the current default of 'aws'.
Making defaultPartition an instance variable seemed odd, so it's instead a protected static member. I'm also not sure if there's a great way to detect whether the generation is for the CDK and I wasn't sure of an elegant way to modify that one line of the JSDoc within mkcdk.ts.
Finally, I wasn't exactly sure how to test this. I ran make generate and make cdk-all; however, there was a CDK build failure both on this branch and on the main branch around the '@aws-cdk/core:enableStackNameDuplicates' feature flag.
Implements #127

Thanks Kyle,
that's a great addition. Sorry for the delay. I'm quite busy these days... was now finally able to test it.

I'm also not sure if there's a great way to detect whether the generation is for the CDK

Actually the generate is only run once. There are no separate builds for CDK/non-CDK. The CDK variant is build from the non-CDK-variant by modifying some files. It's pretty ugly but I haven't come up with a better solution yet. That also means there is currently no way to have differing parameter description for the two variants. I hadn't have the need so far, this is the first time it would make sense but think your static desc is good for now.
Regarding testing... there is actually no CDK testing implemented that does actually deploy a stack. All the tests (everything inside the examples dir) only create statements locally and compare them to what is expected.
I'll merge this now but won't trigger a new release. There's probably going to be an automated release tonight when AWS changed the IAM docs.
Thanks,
Daniel


std.net.curl doesn't handle exceptions in callbacks

code (@MartinNowak) reported this on 2014-10-04T08:05:45Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=13573
CC List

bugzilla (@WalterBright)
moonlightsentinel

Description
cat > bug.d << CODE
import std.net.curl;

void main()
{
    auto http = HTTP("dlang.org/non-existing-url");
    http.onReceiveStatusLine = (HTTP.StatusLine s) {
        if (s.code != 200)
            throw new Exception("Request failed.");
    };
    http.method = HTTP.Method.head;
    http.perform();
}
CODE

dmd -L-lcurl -run bug

----

This program segfaults because the D exception handling cannot unwind through the C stack of libcurl. What's needed here is either a nothrow requirement on the callbacks or an exception catch/rethrow mechanism.

moonlightsentinel commented on 2019-12-17T11:50:14Z
Can reproduce it locally but only for 64-bit builds.
dmd -g -m64 -run bug.d segfaults but dmd -g -m32 -run bug.d prints the expected stack trace

bugzilla (@WalterBright) commented on 2019-12-17T12:27:59Z
For me -m64 works and with -m32 I get a linker error: Scrt1.o cannot be found...

bugzilla (@WalterBright) commented on 2019-12-17T08:48:25Z
I'm getting "object.Exception@test.d(8): Request failed." Feel free to reopen, when the bug still exists on your computer.

moonlightsentinel commented on 2019-12-17T12:30:09Z
Wow. SHould've mentioned that this was tested on Win 10, 64 Bit with v2.089.0


Add e2e test for Azure Service Bus Queue using Workload Identity and Pod Identity

Proposal
We need to add e2e test coverage for the scenario
Use-Case
No response
Anything else?
No response

This is already done https://github.com/kedacore/keda/pull/4030


Runtime Error on dragging nodes quickly.

It seems that there's some async issues on managing target IDs while rendering node / placeholders.
==== Environment ====
Mac mini M2 | macOS 14.3 | Firefox 118.0.1 & Safari 17.3 | React 18.2
(It's device and OS dependent and works fine when viewing from my iOS devices.)
==== Ways to Reproduce ====
Placeholder failed to render and dragging a node rapidly over a tree could possibly throw a run-time error (which is impossible to catch since it's raised from the React-dnd bundle).
==== Error Message ====
Expected targetIds to be registered.
invariant@http://localhost:3000/static/js/bundle.js:1610:24
checkInvariants@http://localhost:3000/static/js/bundle.js:10076:68
hover@http://localhost:3000/static/js/bundle.js:10053:20
@http://localhost:3000/static/js/bundle.js:10375:43
@http://localhost:3000/static/js/bundle.js:141623:31
==== Media ====
https://github.com/nosferatu500/react-sortable-tree/assets/48004590/64b13272-bc25-4271-8331-c94eedee64b8

having same issue,
    "react-dnd": "13.0.1",
    "react-dnd-html5-backend": "12.1.1",

changing to this version works


tikv-ctl: support get range properties under remote mode

What is changed and how it works?

Issue Number: Close #14620
What's Changed: support get range properties under remote mode
Related changes

PR to update pingcap/kvproto:

Check List
Tests

No code

Release note
Make get_range_properties command support remote mode.

/cc @bufferflies

/run-test

/merge


Crash should not result in (code=exited, status=0/SUCCESS)

When for example running out of disk space, Routinator reports:
# sudo systemctl status routinator
● routinator.service - Routinator 3000
     Loaded: loaded (/lib/systemd/system/routinator.service; enabled; vendor preset: disabled)
     Active: inactive (dead) since Wed 2022-01-12 10:56:40 UTC; 6min ago
       Docs: man:routinator(1)
    Process: 2186 ExecStart=/usr/bin/routinator --config=/etc/routinator/routinator.conf --syslog server (code=exited, status=0/SUCCESS)
   Main PID: 2186 (code=exited, status=0/SUCCESS)
        CPU: 46.584s

Jan 12 10:56:17 debian11-4vcpu-8gb-ams3 routinator[2186]: Fatal: failed to write to file /var/lib/routinator/rpki-cache/stored/tmp/98601c4f: No space left on device (os error 28)

From the manpage:

Upon success, the exit status 0 is returned. If any fatal error happens, the exit status will be 1. Some commands provide a --complete option which will cause the exit status to be 2 if any of the rsync commands to update the repository fail.


(659) Apply - Prison behaviours

This adds the Prison behaviours screen to the Apply journey. There's been some tweaking to the async methods that pages optionally use (loads of context in the commit messages), and we've also had to make some changes to check your answers to support embedding definition lists inside definition lists, so the whole note is visible on Check Your Answers, but I feel it's pretty comprehensive!
Screenshots

Check Your Answers

 Prison behaviours screen

Thanks for taking this the distance!


chore: add remote config for CPV

Description
As the title, this PR just adds the remote config value.
Other changes
N/A
Tested
N/A
How others should test
N/A
Related issues
Relates RET-384
Backwards compatibility
N/A

Hey @MuckT since NA is mentioned in how other should test section ,we are skipping this PR
CC: @kathaypacific
Thanks.!


VSCode Crashes on saving java file

Issue Type: Bug
when we create a new UNTITLED file and right click on it (located in right pallete) and save it by clicking Save As, after giving a name to the file and clicking SAVE. The IDE crashes. It is happening all the time.
VS Code version: Code 1.21.0 (9a199d77c82fcb82f39c68bb33c614af01c111ba, 2018-03-07T11:04:09.969Z)
OS version: Windows_NT x64 10.0.15063

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6300U CPU @ 2.40GHz (4 x 2496)

Memory (System)
15.83GB (8.62GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

java
red
0.20.0

vscode-java-debug
vsc
0.6.0

vscode-java-pack
vsc
0.3.0

vscode-java-test
vsc
0.3.0

vscode-maven
vsc
0.5.2

Reproduces without extensions

Can you provide the file you are working on?

I created a new file - an untitled file - using code editor itself. and it
crashed. so I do not have a file working at hand
On Fri, Mar 9, 2018 at 1:12 AM, Peng Lyu<EMAIL_ADDRESS>wrote:

Can you provide the file you are working on?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/45259#issuecomment-371600680,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIPKgBgRvc2_4X7MfReUJu-juToCGRNhks5tcYm2gaJpZM4SiMaL
.

@prasoonnaik sorry for coming back on this issue for too long. Can you still reproduce this issue?


Seed public_port to proxy header

When running the proxy on a non-standard port (8443), when a user shut down their server the hub trips over a CORS check with the error
Blocking Cross Origin API request.  Referer: https://myserver:8443/hub/home, Host: myserver/hub/

I remember running into issues with this earlier - I had originally included it, but removed it with cdf93c984ee06ec99e1d594ccea856598cc19c83. I agree it makes sense to include it, but I was running into issues with this when testing. I'll try this out again!

on a bit more digging, the problem that manifested itself happens in the following case:
user -> edge proxy (listens on port 80) -> nchp (listens on port 8080) -> jupyter singleuser notebooks.
In this case, the Origin header from the client side js will be set to hostname:80 (since that's the edge proxy's port, and that's what the user connects to), while Hostname will be set to 8080 (since that's what nchp is listening on!).
So the proper fix would be to find a way of making sure that the Host header that gets passed to the notebook matches what the user sees, rather than what the proxy is bound to. Multi host http proxying is fun :8ball:
I feel like this definitely needs some diagramming at least to figure out the proper solution.

@ryanlovett can you check out #7?

Works for me!

Ok, I'm going to close this one then. Thanks a lot for reporting and testing this through!


Added the additional permissions in the pipeline runner file for Ray

The issue resolved by this Pull Request:
Resolves RHOAIENG-880
Description of your changes:
Added the additional permissions which was missing and is required to create ray clusters.

Testing instructions

Create a Data Science project with namespace "pipelineskfptekton1"
Create a valid pipeline server
Import the pipeline mentioned below
Run the pipeline
It should work without any errors.

pipeline.yaml
apiVersion: tekton.dev/v1beta1
kind: PipelineRun
metadata:
  name: ray-integration-test
  annotations:
    tekton.dev/output_artifacts: '{"ray-fn": [{"key": "artifacts/$PIPELINERUN/ray-fn/Output.tgz",
      "name": "ray-fn-Output", "path": "/tmp/outputs/Output/data"}]}'
    tekton.dev/input_artifacts: '{}'
    tekton.dev/artifact_bucket: mlpipeline
    tekton.dev/artifact_endpoint: minio-service.kubeflow:9000
    tekton.dev/artifact_endpoint_scheme: http://
    tekton.dev/artifact_items: '{"ray-fn": [["Output", "$(results.Output.path)"]]}'
    sidecar.istio.io/inject: "false"
    tekton.dev/template: ''
    pipelines.kubeflow.org/big_data_passing_format: $(workspaces.$TASK_NAME.path)/artifacts/$ORIG_PR_NAME/$TASKRUN_NAME/$TASK_PARAM_NAME
    pipelines.kubeflow.org/pipeline_spec: '{"description": "Ray Integration Test",
      "name": "Ray Integration Test"}'
  labels:
    pipelines.kubeflow.org/pipelinename: ''
    pipelines.kubeflow.org/generation: ''
spec:
  pipelineSpec:
    tasks:
    - name: ray-fn
      taskSpec:
        steps:
        - name: main
          args:
          - '----output-paths'
          - $(results.Output.path)
          command:
          - sh
          - -c
          - (PIP_DISABLE_PIP_VERSION_CHECK=1 python3 -m pip install --quiet --no-warn-script-location
            'codeflare-sdk' || PIP_DISABLE_PIP_VERSION_CHECK=1 python3 -m pip install
            --quiet --no-warn-script-location 'codeflare-sdk' --user) && "$0" "$@"
          - sh
          - -ec
          - |
            program_path=$(mktemp)
            printf "%s" "$0" > "$program_path"
            python3 -u "$program_path" "$@"
          - |
            def ray_fn():
                from codeflare_sdk.cluster.cluster import Cluster, ClusterConfiguration
                import ray

                cluster = Cluster(ClusterConfiguration(
                    name='raytest',
                    # namespace must exist, and it is the same from 432__data-science-pipelines-tekton.robot
                    namespace='pipelineskfptekton1',
                    num_workers=1,
                    head_cpus='500m',
                    min_memory=1,
                    max_memory=1,
                    num_gpus=0,
                    image="quay.io/project-codeflare/ray:latest-py39-cu118",
                    instascale=False,
                    ingress_domain='apps.upaptest-n-pool-ds2m8.rhods.ccitredhat.com'
                ))
                # workaround for https://github.com/project-codeflare/codeflare-sdk/pull/412
                cluster_file_name = '/opt/app-root/src/.codeflare/appwrapper/raytest.yaml'
                # Read in the file
                with open(cluster_file_name, 'r') as file:
                    filedata = file.read()

                # Replace the target string
                filedata = filedata.replace('busybox:1.28', 'quay.io/project-codeflare/busybox:latest')

                # Write the file out again
                with open(cluster_file_name, 'w') as file:
                    file.write(filedata)
                # end workaround

                # always clean the resources
                cluster.down()
                print(cluster.status())
                cluster.up()
                cluster.wait_ready()
                print(cluster.status())
                print(cluster.details())

                ray_dashboard_uri = cluster.cluster_dashboard_uri()
                ray_cluster_uri = cluster.cluster_uri()
                print(ray_dashboard_uri)
                print(ray_cluster_uri)

                # before proceeding make sure the cluster exists and the uri is not empty
                assert ray_cluster_uri, "Ray cluster needs to be started and set before proceeding"

                # reset the ray context in case there's already one.
                ray.shutdown()
                # establish connection to ray cluster
                ray.init(address=ray_cluster_uri)
                print("Ray cluster is up and running: ", ray.is_initialized())

                @ray.remote
                def train_fn():
                    return 100

                result = ray.get(train_fn.remote())
                assert 100 == result
                ray.shutdown()
                cluster.down()
                auth.logout()
                return result

            def _serialize_int(int_value: int) -> str:
                if isinstance(int_value, str):
                    return int_value
                if not isinstance(int_value, int):
                    raise TypeError('Value "{}" has type "{}" instead of int.'.format(
                        str(int_value), str(type(int_value))))
                return str(int_value)

            import argparse
            _parser = argparse.ArgumentParser(prog='Ray fn', description='')
            _parser.add_argument("----output-paths", dest="_output_paths", type=str, nargs=1)
            _parsed_args = vars(_parser.parse_args())
            _output_files = _parsed_args.pop("_output_paths", [])

            _outputs = ray_fn(**_parsed_args)

            _outputs = [_outputs]

            _output_serializers = [
                _serialize_int,

            ]

            import os
            for idx, output_file in enumerate(_output_files):
                try:
                    os.makedirs(os.path.dirname(output_file))
                except OSError:
                    pass
                with open(output_file, 'w') as f:
                    f.write(_output_serializers[idx](_outputs[idx]))
          image: registry.redhat.io/ubi8/python-39@sha256:3523b184212e1f2243e76d8094ab52b01ea3015471471290d011625e1763af61
        results:
        - name: Output
          type: string
          description: /tmp/outputs/Output/data
        metadata:
          labels:
            pipelines.kubeflow.org/cache_enabled: "true"
          annotations:
            pipelines.kubeflow.org/component_spec_digest: '{"name": "Ray fn", "outputs":
              [{"name": "Output", "type": "Integer"}], "version": "Ray fn@sha256=e224348a70f708acdcee6dc324d56466087027edc04954a409c66a602978e1c7"}'

Checklist

[X] The commits are squashed in a cohesive manner and have meaningful messages.
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

A new image has been built to help with testing out this PR: quay.io/opendatahub/data-science-pipelines-operator:pr-598
An OCP cluster where you are logged in as cluster admin is required.
To use this image run the following:
cd $(mktemp -d)
git clone git@github.com:opendatahub-io/data-science-pipelines-operator.git
cd data-science-pipelines-operator/
git fetch origin pull/598/head
git checkout -b pullrequest 4707c96e913724752e30025b63eff5e0c6bcbdf9
oc new-project opendatahub
make deploy IMG="quay.io/opendatahub/data-science-pipelines-operator:pr-598"

More instructions here on how to deploy and test a Data Science Pipelines Application.

Change to PR detected. A new PR build was completed.
A new image has been built to help with testing out this PR: quay.io/opendatahub/data-science-pipelines-operator:pr-598

once this is moved out of WIP please have folks from DW review this as well

Change to PR detected. A new PR build was completed.
A new image has been built to help with testing out this PR: quay.io/opendatahub/data-science-pipelines-operator:pr-598

/lgtm
/approve


[BUG] Fix failing FaultDetectionMetricsCollectorTests test

What is the bug?
FaultDetectionMetricsCollectorTests.testFaultDetectionMetrics is failing and not reproduce-able locally. It is failing since this commit - https://github.com/opensearch-project/performance-analyzer/pull/214
org.opensearch.performanceanalyzer.collectors.FaultDetectionMetricsCollectorTests > testFaultDetectionMetrics FAILED

Suite: Test class org.opensearch.performanceanalyzer.collectors.FaultDetectionMetricsCollectorTests
  2> java.lang.AssertionError: expected:<1> but was:<7>
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.failNotEquals(Assert.java:835)
        at org.junit.Assert.assertEquals(Assert.java:647)
        at org.junit.Assert.assertEquals(Assert.java:633)
        at org.opensearch.performanceanalyzer.collectors.FaultDetectionMetricsCollectorTests.testFaultDetectionMetrics(FaultDetectionMetricsCollectorTests.java:44)
    java.lang.AssertionError: expected:<1> but was:<7>
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.failNotEquals(Assert.java:835)
        at org.junit.Assert.assertEquals(Assert.java:647)
        at org.junit.Assert.assertEquals(Assert.java:633)
        at org.opensearch.performanceanalyzer.collectors.FaultDetectionMetricsCollectorTests.testFaultDetectionMetrics(FaultDetectionMetricsCollectorTests.java:44)

https://github.com/opensearch-project/performance-analyzer-rca/runs/6457175983?check_suite_focus=true

The issue seems not to be happening for as far as the GitHub logs allow us to go back. Marking as closed.


Add min_x, max_x ,bin_width to circular_average function in roi module

Uncovered Suggestions

+0.57% via skxray/core/utils.py#119...131
+0.48% via ...kxray/io/avizo_io.py#119...129
+0.48% via ...kxray/io/avizo_io.py#45...55
See 7 more...

Powered by Codecov. Updated on successful CI builds.

@tacaswell travis is working changed the lmfit version to 0.8.3. Ready for review

@tacaswell documented the API changes ready for review

@tacaswell thanks!


[type:test][ISSUE #4540] add plugin test cases for admin

[type:test][ISSUE #4540] add plugin test cases for admin.(#4540 )
Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

hi，pls check ci

Codecov Report

Merging #5231 (5843375) into master (fcb671f) will increase coverage by 0.08%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #5231      +/-   ##
============================================
+ Coverage     61.70%   61.79%   +0.08%
- Complexity     8597     8605       +8
============================================
  Files          1249     1249
  Lines         37955    37954       -1
  Branches       3602     3602
============================================
+ Hits          23419    23452      +33
+ Misses        12480    12441      -39
- Partials       2056     2061       +5

see 5 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@moremind I found a code error in PluginHandleChangedEvent. The dataType in the contrast() method takes the dateCreated field of before.PluginHandleChangedEvent

@moremind I found a code error in PluginHandleChangedEvent. The dataType in the contrast() method takes the dateCreated field of before.PluginHandleChangedEvent

can you fix it in another pr？

@moremind I found a code error in PluginHandleChangedEvent. The dataType in the contrast() method takes the dateCreated field of before.PluginHandleChangedEvent

can you fix it in another pr？

of course!

pls add my wechat: yijia1949


Get-GeoFromIpAndTagIncident won't tag multiple IPs, fails on subsequent tags

When executing the playbook on an incident with multiple IPs, it parses the IPs just fine and labels the first one, but subsequent labels fail with the following example:
{
"error": {
"code": "Conflict",
"message": "Newer version of resource '193f0487-f0fc-4edc-a7b6-1796944dc97a' exists. Data was not saved"
},
"debugInfo": "clientRequestId: 0fc69fa4-c2a4-4d90-b8fd-ec29b7d71823"
}
I'm guessing this is because it's trying to write subsequent tags to Tag1. Can we easily increment IP2 to write to tag2?

@richlilly2004 good catch... taking this with the connector team.

@richlilly2004 #924 PR was just submitted. We changed the concurrency to 1 for the loop.  could you try the new template?

@richlilly2004 #924 PR was just submitted. We changed the concurrency to 1 for the loop. could you try the new template?

Looks good now


[elemental_tv] Add elemental_tv extractor

This pull request adds an extractor for a Bulgarian website https://elemental.tv
mp@mp-X230:/mnt/ramdisk$ yt-dlp "https://play.elemental.tv/channel/573f5a14761973ec1d502507" --verbose --netrc -F
[debug] Command-line config: ['https://play.elemental.tv/channel/573f5a14761973ec1d502507', '--verbose', '--netrc', '-F']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip)
[debug] Python 3.11.2 (CPython x86_64 64bit) - Linux-6.1.0-17-amd64-x86_64-with-glibc2.36 (OpenSSL 3.0.11 19 Sep 2023, glibc 2.36)
[debug] exe versions: ffmpeg 5.1.4-0 (setts), ffprobe 5.1.4-0, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2022.09.24, mutagen-1.47.0, pyxattr-0.8.1, sqlite3-2.6.0, websockets-11.0.3
[debug] Proxy map: {}
[debug] Extractor Plugins: ElementalTVIE
[debug] Plugin directories: ['/home/mp/.config/yt-dlp/plugins/test/yt_dlp_plugins']
[debug] Loaded 1858 extractors
[ElementalTV] : Downloading JSON metadata
[ElementalTV] Extracting URL: https://play.elemental.tv/channel/573f5a14761973ec1d502507
[debug] [ElementalTV] Channel ID: 573f5a14761973ec1d502507
[ElementalTV] 573f5a14761973ec1d502507: Downloading m3u8 information
[ElementalTV] 573f5a14761973ec1d502507: Downloading JSON metadata
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[info] Available formats for 573f5a14761973ec1d502507:
ID EXT RESOLUTION │   TBR PROTO │ VCODEC      ACODEC
───────────────────────────────────────────────────────
LO mp4 512x288    │  800k m3u8  │ avc1.640029 mp4a.40.2
ME mp4 768x432    │ 1600k m3u8  │ avc1.640029 mp4a.40.2
HI mp4 1024x576   │ 2400k m3u8  │ avc1.640029 mp4a.40.2

Please let me know if there are improvements to be made to the code before merging!
Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense
[x] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Fix or improvement to an extractor (Make sure to add/update tests)
[x] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Please do not force push

Do not force push again

Do not force push again

I've just read the message. Thank you for the information.

General preliminary review:

Extractor methods should be private: _get_stream_metadata() not get_stream_metadata() etc
Class vars are being overused. Avoid class constants that are only used once in the code. channel_id shouldn't be a class var
Separating blocks of code with an empty line is acceptable, but here almost every other line is empty and it makes it harder to read.
ExtractorError is being raised a lot. Shouldn't some of these be expected=True?
Use relative imports: from .common import InfoExtractor etc
Are these livestreams? Return is_live or live_status in the info dict if so.

Thank you for reviewing the code. I think I've fixed most of the issues (if not all of them).

* Extractor methods should be private: `_get_stream_metadata()` not `get_stream_metadata()` etc

Fixed.

* Class vars are being overused. Avoid class constants that are only used once in the code. `channel_id` shouldn't be a class var

I've left the access_token and channel_id as constants as they're being used in multiple methods: getting stream URL, stream metadata, logging in and extracting m3u8.

* Separating blocks of code with an empty line is acceptable, but here almost every other line is empty and it makes it harder to read.

I think I've fixed it. I've just left those around ifs and returns for code readability reasons.

* `ExtractorError` is being raised a lot. Shouldn't some of these be `expected=True`?

After further analyzing the API I've added handling login error (wrong username or password) with expected=True with the error message being passed to the user. Rest of them is related with more critical issues so I think I wouldn't be able to get rid of them.

* Use relative imports: `from .common import InfoExtractor` etc

I'm testing the plugin by sideloading the elemetal_tv.py file in ~/.config/yt-dlp/plugins/test/yt_dlp_plugins/extractor/ and making imports relative breaks it up. I don't know a workaround for it.

* Are these livestreams? Return `is_live` or `live_status` in the info dict if so.

I've fixed it.
I've also moved exception raises to method bodies related with loading the particular data the raise refers to to keep the main _real_extract method as clean as possible.


I can not find  the python file of the definition model, only the official training of good models such as imagenet, vgg16, vgg19

https://github.com/dmlc/mxnet-model-gallery
http://data.mxnet.io/models/imagenet/

I want to have a python symbol Corresponding to the official pre-training model in the imagenet models for exmaple vgg16 vgg19

At /example/image-classfication/symbol/vgg

@lilhope  it is not vgg19, it is vgg16, and it is not Corresponding to the official pre-training model in the imagenet models(http://data.mxnet.io/models/imagenet/vgg/vgg16-symbol.json and http://data.mxnet.io/models/imagenet/vgg/vgg19-0000.params)


[FLINK-5418] [table] Estimated row size does not support nested types

This adds an estimate for array, composite and generic types.

A better estimate for ROW will be merged with #3039. I think the best we can do for array is to estimate the size of one array element and have some default array size.

Hi @twalthr, I have merged PR #3039 yesterday. Can you update this PR? Thanks

I will update this PR with the code from #3039 as a fix for the 1.2 release branch. What do you think is a good estimate for an array size? 16 elements? How do we want to estimate the size of a generic type?

Good question, 16 sounds OK to me. For generic types I don't know, maybe 32 bytes?

I have chosen 128 now. Because generic types should be very expensive. If they have 4-8 fields with arrays and strings we should already be in that region.

+1 to merge

Merging this...


CASSANDRA-16902 trunk: A user should be able to view permissions of role they created

Sorry, I missed the fact that there was some DTest changes.


Bundle assets - expose meta changes

resolves https://github.com/relative-ci/bundle-stats/issues/4769

close in favor of https://github.com/relative-ci/bundle-stats/pull/4816


Unable to view demo game

I attempted to watch the Ke Jie vs. AlphaGo game today (created as a demo game by the USGO2 account), but it would not load up. The console displayed these errors:
XML Parsing Error: syntax error Location: https://www.gokgs.com/json-cors/access Line Number 1, Column 1:  access:1:1
TypeError: e.players.white is undefined
I'm guessing the second error is the more important one.

Sorry about that. I have a fix I'll be pushing soon.

Fixed in commit 8923e2d02189


Refactoring/Organizing Code

Just setting this up as an issue.  The amount of commands and code has grown quite large and it may be time to split these out into separate python modules.

Refactoring/Organization achieved in 1.4.0


What's the plan regarding UrqlCombinedError.t?

Hello, all!
I have been using reason-urql since hooks got in, and one of the pain points has been around error handling. Right now, there's no way to inspect the errors that happened during a request, as the error type is simply an UrqlCombinedError.t with no helper function to extract more information from that type.
What's the plan regarding error handling, and is this error API what's gonna ship with 1.0?

@gugahoa the UrqlCombinedError API is definitely rough (I actually think it's just broken right now). I think I see where the issue is – there's a binding to support newing up a CombinedError, but users are never actually doing that, they're just receiving a CombinedError instance from urql. I'll work on a prototype this afternoon, but the short answer is no, we'll fix this API before shipping a formal v1. The beta I mentioned will still go out with it so folks can use the hooks now ✅, and I'll probably release another beta when this API gets fixed. I just don't want to release a v1 and then need to start doing lots of semver releases that disrupt folks.

Closed by #89.

Released in v1.0.0-beta.2.


Vendor in latest projectatomic/buildah

buildah fixed its probelm where it was not pulling in
the ENV of the base image. This pulls that change into
libpod as well.
Signed-off-by: umohnani8<EMAIL_ADDRESS>

@rhatdan @mheon @baude PTAL

LGTM pending tests

@mheon tests pass


Develop bridge schema

@jaimehgb coming up with v1 of schema after 11/10/2022 meeting. We will then be meeting again sometime next week

see https://github.com/messari/subgraphs/pull/1438#


cleanup: remove grpcHostname option in istio_ca

remove grpcHostname option in istio_ca as it's not used anywhere anymore
/assign @incfly

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: wattli
If they are not already assigned, you can assign the PR to them by writing /assign @wattli in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS
security/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

I think @costinm added this..

@rshriram Thanks for reminding
/assign @costinm

@lichuqiang: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


use type hint classes for backwards compatibility

When specializing type for type hints, the type[...] notation causes
problems for older version of Python. Using the type hint classes from
the typing module fixes this.
list -> List
tuple -> Tuple
dict -> Dict

Oops, I made #2 earlier because I thought you weren't going to make a PR. It's fine though, I just undid it. Thanks!


Progress bar? Output expectation?

Is there a progress bar or something that can tell me how it's going? I told it to train & it took less than 1 minute before it appears to have stalled. The [STOP] button is still there but what should I expect as a result or completion verification?
Where does the CKPT get created/filed?

Same here...get this: Then just hangs...no clue what it's doing...if anything....
Unable to find image 'smy20011/dreambooth:latest' locally
latest: Pulling from smy20011/dreambooth
40dd5be53814: Already exists
0645c48e225b: Already exists
8889d023c18e: Already exists
74115353f57d: Already exists
9e0ea72fe76c: Already exists
13f51f3f80fd: Pulling fs layer
c4ba9103d17d: Pulling fs layer
8d9b8d71f868: Pulling fs layer
97badaa6a776: Pulling fs layer
63041bc7286a: Pulling fs layer
5f237510596a: Pulling fs layer
e84ba40a3ba2: Pulling fs layer
3514d699b3b1: Pulling fs layer
389cef77d3f2: Pulling fs layer
454b4e6dabd9: Pulling fs layer
08c0b6423a55: Pulling fs layer
08c0b6423a55: Waiting
c4ba9103d17d: Download complete
97badaa6a776: Download complete
5f237510596a: Waiting
389cef77d3f2: Waiting
e84ba40a3ba2: Waiting
3514d699b3b1: Waiting
454b4e6dabd9: Waiting

I think the first run is slower because you need to full all the docker image. Wait after it finished pulling and rerun it should give you correct result.

Awesome! Thank you so much! Waiting...
Just curious...Why does it need a huggingface token? What is communicating with them for?

You need that token to download model from huggingface website.
On Fri, Nov 4, 2022, 11:15 PM Ryan Jones @.***> wrote:

Awesome! Thank you so much! Waiting...
Just curious...Why does it need a huggingface token? What is communicating
with them for?
—
Reply to this email directly, view it on GitHub
https://github.com/smy20011/dreambooth-gui/issues/4#issuecomment-1304419826,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAL4433TXAMZVU5H3EMXV2LWGX3PXANCNFSM6AAAAAARHJLSZU
.
You are receiving this because you commented.Message ID:
@.***>

But If I already have the model locally...
can't I just reference it like this /SDModel/sd-v1-4.ckpt in the Model Text box?
My input vars are this:
docker run -t --gpus=all -v=D:\Users\ryanp\Dream\Train:/instance -v=C:\Users\ryanp\AppData\Roaming\smy20011.dreambooth\Mariejones:/class -v=D:\DreamBooth\Output:/output -v=/SDModel/sd-v1-4.ckpt:/input_model -v=C:\Users\ryanp\AppData\Roaming\smy20011.dreambooth:/train -e HUGGING_FACE_HUB_TOKEN=XXXXXXXXXXXXXXXXXX smy20011/dreambooth:latest /start_training /train_dreambooth.py --pretrained_model_name_or_path=/input_model --instance_prompt=sks --instance_data_dir=/instance --class_data_dir=/class --with_prior_preservation --prior_loss_weight=1.0 --class_prompt=Mariejones --max_train_steps=600 --learning_rate=1e-5 --lr_scheduler=constant --lr_warmup_steps=0 --resolution=512 --output_dir=/output --use_8bit_adam --mixed_precision=fp16 --train_batch_size=1 --gradient_accumulation_steps=1 --use_8bit_adam --resolution=512 --gradient_checkpointing --train_text_encoder
Errors i'm getting now are
Traceback (most recent call last):
File "/train_dreambooth.py", line 695, in
main()
File "/train_dreambooth.py", line 376, in main
args.pretrained_model_name_or_path, torch_dtype=torch_dtype, use_auth_token=True
File "/opt/conda/lib/python3.7/site-packages/diffusers/pipeline_utils.py", line 372, in from_pretrained
revision=revision,
File "/opt/conda/lib/python3.7/site-packages/diffusers/configuration_utils.py", line 279, in get_config_dict
raise EnvironmentError(f"It looks like the config file at '{config_file}' is not a valid JSON file.")
OSError: It looks like the config file at '/input_model' is not a valid JSON file.
I'm guessing that text field is not expecting the MODEL but a JSON config file about a model?
Any docs on the expected values for the various inputs?

Ahhh I see
Allow advanced customization
Load local model.
Local is part of the road map...
So I updated it for get it from hugginface and it seems to be working....at least it's downloading...

Hey BTW got it all working...yeah it just had to download the model and other stuff.  Works GREAT now. With people I don't even use a CLASS...just leave it blank and it still works great....DID have to change the SKS classname to t@y as SKS is a gun and I kept getting GUN images...
Would love to try to take the OUTPUT Model.ckpt and put it back IN to add another instance image...but can't seem to find where to put it...Is this even possible? Does the downloaded model come down as a ckpt file or is it more complicated than that?
If it's compenents that are coming down from hugginface....is it possible to take the OUTPUT model.ckpt and break it up into the components?


Refactor and Correct Navbar Alignment 🐞

🐞 Bug Report
Describe the bug

The current navbar has misaligned items and could use some improvement for better visual consistency and usability. The theme toggle button on the navbar is also not optimally positioned or styled, affecting user experience.
Expected behavior

A visually balanced and neatly aligned navbar, with a well-placed theme toggle button, improving both aesthetics and functionality across the website.
Screenshots or Video

Environment
Please provide the following details:

OS: Windows
Browser/Software: Chrome, Node.js_

Logs & Error Messages

@dhairyagothi Done


[Event Request] CU 1303 Correct Posted Sales Invoice SetTrackInfoForCancellation

Hello,
can we please add:
`
[IntegrationEvent(false, false)]
local procedure OnBeforeSetTrackInfoForCancellation(var SalesInvoiceHeader: Record "Sales Invoice Header"; var IsHandled: Boolean)
begin
end;
local procedure SetTrackInfoForCancellation(var SalesInvoiceHeader: Record "Sales Invoice Header")
var
SalesCrMemoHeader: Record "Sales Cr.Memo Header";
CancelledDocument: Record "Cancelled Document";
IsHandled: Boolean;
begin
OnBeforeSetTrackInfoForCancellation(SalesInvoiceHeader, IsHandled);
if IsHandled then
exit;
`
To instroduce custom code.
Thank you,
Bartek

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Status bar color differs from example screenhots

The background color of the status bar is different from screenshots in README. Instead of being dark it's some sort of dark magenta. Looks totally fine on web version of VSCode though, and the README screenshot seems to be on macOS, so this might be a problem with VSCode for Windows instead. I have not experienced this problem with other themes before.
Also, the color the status bar is the same that the default VSCode dark theme uses for the status bar.
Screenshot on Windows:

This appears to be happen due to some missing properties in the theme JSON file.
After adding the following properties to settings.json the theme looks as it should.
"workbench.colorCustomizations": {
    "statusBar.debuggingBackground": "#080808",
    "statusBar.debuggingForeground": "#ddd",
    "statusBar.noFolderBackground": "#080808"
  }

I'm assuming those are required for the theme to function properly?


add medley

A followup to https://github.com/electron/electron.atom.io/pull/603

@zeke  thanks a lot for taking care of this Zeke! really appreciate it!
from what I read on #PR 616, there will be a script that automatically update the newly added apps on the list? Or is there anything else i still have to do on my end?
Thanks!

Still working out the details for https://github.com/electron/electron.atom.io/pull/616, but yeah you're all set. Your app will be on the site soon! ✨

Thanks Zeke! Grateful for the help :)


Can't install 1.8.2

When I try npm i -g yo :
node v4.4.4 (also try with 4.3.1)
npm  v3.9.0
deprecated<EMAIL_ADDRESS>this package has been reintegrated into npm and is now out of date with respect to npm
npm ERR! Darwin 15.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "-g"<EMAIL_ADDRESS>npm ERR! node v4.4.4
npm ERR! npm  v3.9.0
npm ERR! path /usr/local/lib/node_modules/yo/lib/cli.js
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall chmod

npm ERR! enoent ENOENT: no such file or directory, chmod '/usr/local/lib/node_modules/yo/lib/cli.js'
npm ERR! enoent ENOENT: no such file or directory, chmod '/usr/local/lib/node_modules/yo/lib/cli.js'
npm ERR! enoent This is most likely not a problem with npm itself
npm ERR! enoent and is related to npm not being able to find a file.
npm ERR! enoent

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/jaumard/Downloads/testTrails/npm-debug.log

If I install npm i -g<EMAIL_ADDRESS>all is working

Wrong repo sorry it was for yo


Sc wrap retry and delivered on

2 fixes in this pull request (see first 2 commits)
first commit fixes bad wrapping of commit retries that was causing duplicate vertices in amberdb
second adds the 'deliveredOn' relationship to the loading of works into memory
@yetti
Yetrina, would you take a look at the second one at least ?

That looks alright to me :+1:


Add GraphQL scripts

The web-components library includes a dev/ directory that demonstrates how to use the components. Specifically, it demonstrates how to load data from the LIS GraphQL server and shim it into structures compatible with the web components. This code has served as a template for the various sites that use the LIS Jekyll theme. For this reason, this JS code should be copied into the assets/js/ directory of the theme AND DOCUMENTED. As with the web components, including these scripts in the head of a page should be opt-in via the page's Jekyll front matter so it's only loaded on pages that need it. Additionally, the URI of the GraphQL server should be configurable via the _config.yml file.
Edit: It's worth noting that the GraphQL query function underlying the specific queries uses the native fetch function. This function returns a promise so modifying the data being sent to a component to include links and whatnot is as simple as calling .then() on the returned promise and applying whatever modifications you want to the data.

Oh, I get it, because right now we have those scripts locally in the site, not theme:
[shokin@jekyll-dev-vm jekyll-legumeinfo]$ ls assets/js/graphql/
get-gene-form-data.js  get-genes.js  graphql.js  linkout-function.js

I can do this as part of the use-theme branch on the site (plus a new branch on the theme), shall I?

That would be great. Thanks!
An implementation detail I didn't explicitly note above is that there should be an assets/js/graphql/index.js barrel so that adding the scripts to a Jekyll page is a matter of including that single file.

Alright, I'm new to barrels and don't see an example in web-components. Is the deal that I change, for example in graphql.js,
function graphqlQuery(uri, query, variables={}, abortSignal=undefined) {
to
export function graphqlQuery(uri, query, variables={}, abortSignal=undefined) {
and then in index.js I use
import { graphqlQuery } from './graphql.js'
???

Alright, I'm new to barrels and don't see an example in web-components. Is the deal that I change, for example in graphql.js, function graphqlQuery(uri, query, variables={}, abortSignal=undefined) {  to export function graphqlQuery(uri, query, variables={}, abortSignal=undefined) {  and then in index.js I use import { graphqlQuery } from './graphql.js' ???

Correct, the individual files will need to export their functions. index.js, however, will just re-export the contents of the other files:
export * from './graphql.js';

When including the barrel in the page head you'll need to specify that it's a module:
<script type="module" src="{{ "assets/js/graphql/index.js" | relative_url }}"></script>

Hrmph. The functions are not getting picked up. Here's my index.js and I put export in front of the functions and constants in the four js files. This includes the hack to get a _config.yml variable into a JS file. Maybe you know a better way, that's what I found.
 ---
layout: blank
---
// query the LIS GraphQL server given by graphqluri in _config.yml.
// https://github.com/legumeinfo/graphql-server
// we can use system variables in a js file when we have front matter as above!
const uri = "{{ site.graphqluri }}"

export * from './graphql.js';
export * from './linkout-function.js';
export * from './get-gene-form-data.js';
export * from './get-genes.js';

I did check that the individual JS files are being read.

Hrmph. The functions are not getting picked up. Here's my index.js and I put export in front of the functions and constants in the four js files. This includes the hack to get a _config.yml variable into a JS file. Maybe you know a better way, that's what I found.
 ---
layout: blank
---
// query the LIS GraphQL server given by graphqluri in _config.yml.
// https://github.com/legumeinfo/graphql-server
// we can use system variables in a js file when we have front matter as above!
const uri = "{{ site.graphqluri }}"

export * from './graphql.js';
export * from './linkout-function.js';
export * from './get-gene-form-data.js';
export * from './get-genes.js';

What file is this in? The graphql.js file needs front matter to force evaluation of liquid template syntax but it shouldn't specify a layout. See the uikit-theme.scss file for an example.

Ah, OK, but that's not what's causing it to not work. (Before it just gave a warning that blank does not exist, so it's nice to not have that line). The issue remains: Uncaught ReferenceError: getGeneFormData is not defined so I'm doing something wrong in my chain-o-exports.
I've punched my changes into two new branches: jekyll-legumeinfo:graphql-js-in-theme and jekyll-theme-legumeinfo:assets-js-graphql. Hopefully it's obvious to you what I'm doing wrong here.

What file is this in? The graphql.js file needs front matter to force evaluation of liquid template syntax but it shouldn't specify a layout. See the uikit-theme.scss file for an example.

Oh, yeah, that was in index.js. Moved to graphql.js.

Note: I don't think this is fatal, but simply having
---
# This YAML front matter ensures Jekyll will pass the site variables in.
---

at the top of graphql.js leads to a boatload of HTML being put into _site/assets/js/graphql/graphql.js on the Jekyll site. Which is unattractive but perhaps not fatal. (I tried removing the above and hardcoding uri and still got the functions not being picked up by the gene search HTML.)

Just a gentle reminder @alancleary I'm still stumped as to why the site doesn't pick up the gene search functions from the theme's assets.
Regarding the import, I forgot that module content isn't added to the global scope (working with the web components library has muddied my memory!). So the import should actually be something like:
{% if page.graphql %}
<script type="module">
  import * as graphql from "{{ "assets/js/graphql/index.js" | relative_url }}";
  window.graphql = graphql;
</script>
{% endif %}

This takes all of the GraphQL code and sticks it in the global graphql variable. Since all of the exported functions are now going to be accessed via this variable, it probably makes sense to rename the graphqlQuery function to just query so calling it will become graphql.query(...) instead of graphql.graphqlQuery(...).

Ok, this is a bit messy (to me). assets/js/graphql/index.js exports FOUR files:
export * from './graphql.js';
export * from './linkout-function.js';
export * from './get-gene-form-data.js';
export * from './get-genes.js';

Those files contain references to themselves, for example linkout-function.js references the function formerly known as graphqlQuery now query which was defined in graphql.js. So presumably now that it's query I reference it in linkout-function.js as just query, right?
That being said, graphql is not being defined with the following code:
---
layout: full-width
title: Gene Search
web_components: true
graphql: true
---

{% if page.graphql %}
<script type="module">
  import * as graphql from "{{ 'assets/js/graphql/index.js' | relative_url }}";
  window.graphql = graphql;
</script>
{% endif %}

I get Uncaught ReferenceError: graphql is not defined. However, the JS files are indeed being loaded. (And I checked by removing the page.graphql conditional, same result.)

That Uncaught Reference Error occurs here:
      const geneSearchElement = document.getElementById('gene-search');
      geneSearchElement.formDataFunction = graphql.getGeneFormData;

At this point I think it would be more productive for me to just implement this. Thanks for trying!

Yup, I agree and will look most eagerly at the implementation. :) I'll delete my branches.

@sammyjava I added the GraphQL scripts in the graphql-scripts branch. Interestingly I ended up using an import map to make the JavaScript module available in <script> tags that will be used to configure web components. There are other ways to do this but import maps seemed to reliably get the scripts loaded in time for the components to automatically search if query string parameters are present.
I'm going to hold off on opening a PR since the imminent trait association search component will require more GraphQL code. In the meantime free to take it for a spin and let me know what you think.

I'm wondering why it's always loaded in head.html rather than having a page.graphql conditional.

And why did you not include the linkout function, which is also a graphql query? Forgot it, perhaps?

I'm wondering why it's always loaded in head.html rather than having a page.graphql conditional.

The import map maps the import path to the lis-graphql JavaScript module but the code isn't actually loaded until the module is used, i.e. something is imported from the module. So I decided not to introduce an opt-in front matter variable since having the import map there when you're not using the module has virtually no overhead.

And why did you not include the linkout function, which is also a graphql query? Forgot it, perhaps?

I sure did. I'll be sure to include it before opening a PR.

FWIW I'm still having trouble getting functions resolved, but I'm probably not running the right code on the calling side.
Uncaught (in promise) Error: No form data function provided
Uncaught (in promise) Error: No search function provided

Also, query.js is still getting all the HTML stuffed in front on the dev site (unlike in your case, apparently) due to the empty front matter. I wonder if there's some _config difference on the dev site I'm running. If I yank the front matter and hardcode url in query.js the HTML goes away and I don't get the error about the unexpected ">" in query.js. I'd like to figure that out, it's pretty messy.

Ahhh interesting, testing on my older setup. Recall that I had
layout: blank
That DID NOT insert HTML into the page. But if I have EMPTY front matter:

empty front matter

I reproduce the HTML getting stuffed in with the Uncaught SyntaxError: expected expression, got '<' error.

Recall that I had gotten the "layout: blank" trip from the interwebs. It seems to prevent the HTML from getting in there. Perhaps we have different default layouts? If the default were blank, you'd be able to use empty front matter.

This sounds like an environment/jekyll-legumeinfo issue. The theme provides a default layout, i.e. _layouts/default.html, and it has never interfered with anything, such as the UIkit SCSS theme which is also using empty front matter. I develop the theme as a submodule of jekyll-starter-legumeinfo and run the site with Docker so the environment is consistent and reproducible. The only issue I tend to have is Firefox caching stuff in the theme but that's easy enough to work around.

Should I not have the default layout set in _config.yml as follows? Not sure what else in the enviro is an issue, but I'm not going to worry about it, I don't think it's a fatal error.
# Site-wide defaults
defaults:
  -
    scope:
      path: ""
    values:
      layout: "default"
  -
    scope:
      path: "blog/"
      type: "posts"
    values:
      layout: "blog-item"
  -
    scope:
      path: "news/"
      type: "posts"
    values:
      layout: "news-item"

Ah yes the starter site does NOT have that stuff in _config.yml (and neither does the theme).

Should I not have the default layout set in _config.yml as follows?

The default entry is redundant with the theme and since it doesn't specify a type for the scope it's being applied to everything that's not under blog/ or news/, including SCSS and JS files under assets/ that have front matter.

So, back to this. I've fixed the issue with HTML being spit out with the JS file. And this seems to be working  fine, running my  graphql-js-in-theme branch of jekyll-legumeinfo against the graphql-scripts branch of jekyll-theme-legumeinfo.
However, and this is probably OCD, I still get error messages in the console:
Uncaught ReferenceError: graphql is not defined
    <anonymous> https://jekyll.dev.lis.ncgr.org/tools/search/gene.html?page=1&genus=&species=&strain=&identifier=&description=photosynthesis&family=:685
[gene.html:685:7](https://jekyll.dev.lis.ncgr.org/tools/search/gene.html?page=1&genus=&species=&strain=&identifier=&description=photosynthesis&family=)
Uncaught (in promise) Error: No form data function provided
    formDataFunction https://cdn.jsdelivr.net/npm/@legumeinfo/web-components@1.0.0/dist/web-components.min.js:145

Is this due to a race condition, the web-components tag being read before the JS module is fully loaded? Do we care if it works?
I'd like to get all this stuff (theme, LIS site) merged into main soon, along with getting graphql-server:<IP_ADDRESS> merged in as I'm about ready to promote GlycineMine and LegumeMine.

BTW as you'd expect, those errors are only thrown during initial page load, not during subsequent searches.

As far as I know, the thing left on this issue is to add the linkouts code to the theme. Unless you think that quick flash of the pink error box on the page as the code loads is an issue (I suppose it is, but that sure seems tricky to solve!).

It looks like you haven't updated legumeinfo to use the syntax I documented in the README. Maybe try that?

No, I did. Here, I'll commit it for your viewing pleasure.
https://github.com/legumeinfo/jekyll-legumeinfo/blob/graphql-js-in-theme/tools/search/gene.html
It's working fine (other than the pink box flash) but I'm not getting links on the gene results. And I don't see the linkouts code under jekyll-legumeinfo/_themes/jekyll-theme-legumeinfo/assets/js.

So I should disregard the error you reported an hour ago? If I follow the link in that error message the code is clearly not up to date...
I still need to add the linkout code. Probably won't get to it until next week. Feel free to tackle it if that's not soon enough.

So I should disregard the error you reported an hour ago? If I follow the link in that error message the code is clearly not up to date...

Yeah, I had left the old script tag underneath the new module import which was throwing the graphql is not defined. :) Sorry about that!
But I do still get the initial error Uncaught (in promise) Error: No form data function provided on a fresh page load which is resolved in a fraction of second (here at work at least) and the form data function is fine.
https://jekyll.dev.lis.ncgr.org/tools/search/gene.html
You have to look very carefully to see the red "Failed to load data" error flash on the page, perhaps it lasts longer on a slow connection. Unless there's something else I should be doing but I just copied the code chunk from the current graphql-scripts README.

I still need to add the linkout code. Probably won't get to it until next week. Feel free to tackle it if that's not soon enough.

I'll give it a try, once I have examples like you've provided in /assets/js/graphql I can usually sally forth.

I do still get the initial error Uncaught (in promise) Error: No form data function provided on a fresh page load which is resolved in a fraction of second (here at work at least) and the form data function is fine.

That's a harmless bug in the gene search component. Apparently giving the form data function property a default value triggers the auto-search code, which is ironic because the default function throws the error you're seeing :upside_down_face:

Well, I thought adding the linkouts would be straightforward, but I failed. No problem importing the linkoutFunction as far as I know, and no new errors, but I'm not getting any a tags in the results. Here's my gene.html:
---
layout: full-width
title: Gene Search
web_components: true
---

<lis-gene-search-element id="gene-search"></lis-gene-search-element>
<lis-modal-element modalId="modal">
  <lis-linkout-element id="linkouts"></lis-linkout-element>
</lis-modal-element>

<script type="module">
  import {getOrganismsFormData, geneSearch, linkoutFunction} from "lis-graphql";
  const geneSearchElement = document.getElementById('gene-search');
  geneSearchElement.formDataFunction = getOrganismsFormData;
  geneSearchElement.searchFunction = geneSearch;
  const linkoutElement = document.getElementById('linkouts');
  linkoutElement.linkoutFunction = linkoutFunction;
  // wait for the modal slot to load before adding an event listener
  window.onload = (event) => {
      const modal = document.getElementById('modal');
      modal.addEventListener('toggle', (event) => {
          const [{$el: link}, ...uikitComponents] = event.detail;
          const {gene, location, start, end} = link.dataset;
          let type = '';
          let variables = {};
          if (gene !== undefined) {
              type = 'gene';
              variables = {identifier: gene};
          } else if (location !== undefined && start !== undefined && end !== undefined) {
              type = 'location';
              variables = {identifier: location, start: parseInt(start), end: parseInt(end)};
          }
          linkoutElement.getLinkouts({type, variables});
      });
  }
</script>

OK, I get it, never mind, the LINKS themselves are not implemented in get-genes.js so that's why the results aren't linked. I need to add something akin to https://github.com/legumeinfo/jekyll-legumeinfo/blob/6f8374feca6fe8371ca86078f3285367d165e90c/assets/js/graphql/get-genes.js#L68C1-L73C32

I just realized I should clarify that I'm working on this now since you weren't at the stand-up this morning.
Combing modals with linkouts and having them shown when a gene or location is clicked is a pretty specific use-case for the gene search component. So while I think we should provide as many pre-programmed pieces of this recipe as possible, I don't think it should actually be the preprogrammed behavior of the geneSearch function provided by the lis-graphql scripts in the theme. If we do it right, though, putting the pieces together to achieve that functionality should be easy.

Yup, I gave up on Friday after committing what I'd done and making that comment on the commit. I'll be interested to see how close I got. :)

Closing since we can't properly support the trait association search until the next Intermine release. I'll open a different, more specific issue for those scripts.


rubydoc.info site does not have GAPIC/GRPC entries on the left navigation panel

http://www.rubydoc.info/gems/google-cloud-logging
The low-level gapic/grpc documentation shows up on the 'index' page: http://www.rubydoc.info/gems/google-cloud-logging/index
but not on the left navigation panel.

We believe the site is using some cached data on the navigation view on the left hand side. I am investigating how to rebuild the gems to get the correct navigation to show. Hope to have this fixed soon.

Compare the following URLs:

http://www.rubydoc.info/gems/google-cloud-logging/index
http://www.rubydoc.info/gems/google-cloud-logging/0.23.0/index

They should both be the same, but they are not. The default is to show the most recent gem version in both the navigation and content panes, but the first URL is showing a different navigation pane than what is in the content pane.

The left nav is loaded in an iframe: http://www.rubydoc.info/list/gems/google-cloud-logging/class
I don't believe we can do much to fix this.

Informed the maintainers and they have cleared the app's cache and things work correctly now. Thanks for the heads up!
http://www.rubydoc.info/gems/google-cloud-logging/index


Show localized names in Groups tree, without parents path

This pull request improves (and supersedes) #2288 by translating group names

Great! Thanks, @mlocati!
-Steve


travis-nox-review-pr: Try to detect if the build ran out of memory

Travis builds sometimes fail because Travis doesn't have much memory.
When it happens, the failure reason is often hard to see in the logs and
is confusing (especially for new contributors).
Try to detect OOM errors in dmesg and give a warning when it happens.

Ok, here's a working version. An example build failure (deliberately rigged to OOM) is at: https://travis-ci.org/NixOS/nixpkgs/builds/67677096

:+1:


docs: add readme to all packages

Packages' README are optimized for being displayed on NPM. NPM doesn't have a dark theme, so the banners are always light mode.


How to run the container from vscode

Hi.
I would like to run the different project as you mentioned using vscode. When I run from my linux terminal de container using the following
docker run --gpus all -it --rm --net=host --privileged -v /tmp/.X11-unix:/tmp/.X11-unix -e DISPLAY=$DISPLAY -w /opt/nvidia/deepstream/deepstream-6.1 deepstream-6.1.1-devel:v1
The container run properly and I can launch the composer. Nevertheless, when I lanuch from vscode de container, it works except that is not possible to launch the composer. My devcontainer.json is as follows
{
"name": "UbuntuGraphComposer",
"image": "deepstream-6.1.1-devel:v1",
"runArgs": [
"--gpus=all",
"--name=elvis-demo",
"--net=host",
"--rm",
// "--add-host=host.docker.internal:host-gateway", /* To allow container to access host /
"--privileged=true",
"-it",
"-v","/tmp/.X11-unix:/tmp/.X11-unix",
"-v", "/home/sruap/Documents/Applaudo/Repositories/elvistechgo/elvistech:/home/",
"-v", "/var/run/docker.sock:/var/run/docker.sock", /  Needed for Container Builder in Graph Composer */
"-e","DISPLAY=$DISPLAY"
]
}
Additionally, since I have docker engine and docker desktop, I have to put the context of vscode to

in order to launch docker using engine. Do you have any clue of what can be this issue?

Hi Santiag,
Do you use your own Machine with a GPU or something like an AWS instance ?
If the composer runs fine from bash, then maybe the DISPLAY environment variable is not defined in VS Code terminal.
Do you have error details?

I am using my own machine with a RTX 3080 GPU. I put in the runArgs the -e and the display.
I also test running with the devcontainer.env in which i put the same as you
DISPLAY=:10.0

Look, I did my whole experiment using an AWS EC2 VM with a GPU because I have a Mac. Hence I was actually SSHing into the VM from my Mac and the display variable I got was 10. 10 is probably not a valid value for someone doing that setup on their local machine.
In your case, try to echo $DISPLAY in bash and in VS Code and see what you get. Also make sure to xhost + in both just in case.

You were right. In my case the DISPLAY variable was different. Thanks. Now it runs

Great news!


Outputing ".wav" files instead of ".mp3"?

Anywone has an idea?

Not exactly a solution, but you could pipe the ouput to another "wav to mp3" program.
You could make your own with Python (with pydub's AudioSegment), or you could use ffmpeg (fastest, as it uses CPU acceleration): ffmpeg -i <out_file>.wav <desired_name>.mp3


Does MPTCP support a global proxy in the middle?

For the application scenario in question, let's assume A represents common devices such as smartphones or computers, B and C are Ubuntu 20.04 servers, and D serves as the server backend or the source server. Currently, B functions as a gateway router for A. There are multiple channels between B and C, such as tun1, tun2, and tun3. The question is whether it is possible to deploy MPTCP between B and C to accelerate A's access to D.
The primary objective is to solve the issue where the bandwidth of each of the tunnels like tun1 between B and C is only 10M, with tun2 and tun3 each also at 10M. However, if my smartphone or computer needs a bandwidth of 30M to access resources from D, can MPTCP be implemented between B and C to merge these into a single TCP flow? This would effectively give me a real bandwidth of 30M. Then, when A’s request passes through B and reaches C, C would perform NAT to access D normally. When D sends resources back to C, C could split the response through one TCP connection into multiple MPTCP subflows back to B. Subsequently, B reassembles and sends it back to A via a regular TCP connection. The question is whether it is feasible to achieve this kind of bandwidth aggregation.

The exact same issue has been opened just before in https://github.com/multipath-tcp/mptcp_net-next/issues/488
Please do not create duplicated issues.


Satellite collision event

The aim was to detect if two satellites (with two different orbits) collide with each other or not. One proposal was to use the "Close Approach Analysis" by calculating the miss distance, as described in Vallado (Fourth edition) Section 11.7.1.
As I remember, we preferred using an array of position and velocity vectors themselves rather than using ephemerides (similar to what was done in the LOS event). Then, the miss distance could be calculated. We had also discussed that since such an event is used with many (eg: thousands) satellites, it must be fast enough.
There was #1003 but focused on detecting whether something collides with its attractor, which is already done by the Lithobrake event.

@Yash-10 now I remembered that we had https://github.com/poliastro/poliastro/issues/1350. Do you think we could merge both issues and continue the discussion there? Or do you think this one is different?

After reading a bit more carefully, yes, #1350 includes the miss distance + the minimum orbit intersection distance. Closing this one.

Oh, yes, let's continue there. Sorry for the duplication!


Implement React Redux

Using React Redux to handle most of the state will probably result in a faster site.

hey @Chris5613 assign me this i wanna work on it

Ok
On Tue, 9 May, 2023, 8:49 pm Christian Wu, @.***> wrote:

@nazrul7711 https://github.com/nazrul7711 done! take your time on this.
no rush :)
—
Reply to this email directly, view it on GitHub
https://github.com/Chris5613/RankRiddler/issues/41#issuecomment-1540365984,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALZW3AQXV3TWWQC7FU2OAMLXFJN6JANCNFSM6AAAAAAXZKBQOQ
.
You are receiving this because you were mentioned.Message ID:
@.***>