[Bug] add md5 check to prevent image files from being damaged when network error packets occur

Search before asking

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

Version
master, 1.2-LTS
What's Wrong?
image files damaged when network error packets occur
What You Expected?
master fe send image file to none master correctly, if has network error should not show execute success
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

if (!masterImageMD5.equals(newFileMd5)) {
|   | 133 | +            LOG.error("The Master image md5 is different with copied image md5, fileName is: {}, "
  |   | 134 | +                    + "master md5 is: {}, copied md5 is: {}", masterImageMD5, newFileMd5);
  |   | 135 | +            throw new IOException("The Master image md5 is different with copied image md5, fileName is:"
  |   | 136 | +                + filename + ", master md5 is: " + masterImageMD5 + ", copied md5 is :" + newFileMd5);
  |   | 137 | +        }
  |   | 138 | +        LOG.info("The Master image md5 is same as copied image md5, fileName is :{} , md5 is: {}",
  |   | 139 | +                filename, newFileMd5);
  |   | 140 | +    }

Check image file md5sum between master and non-master fe when do checkpoint
--
  |   | 121 | +     * @param masterImageMD5 master fe md5 get in http header
  |   | 122 | +     * @param filename non-master fe image filename
  |   | 123 | +     * @param dir non-master fe image dir
  |   | 124 | +     * @throws IOException


[NEW] Serial Connection Step 1 (Mass storage)

🚀  Please describe your feature request

add the functionality that the program automatically detect the device if it's connected to the pc and copy the waveform csv files locale on the pc

✏ Describe the solution you'd like

detect the owon device if it's configured as usb mass storage
create a new directory with the name of the current date and place the file into this dirs (this prevents that already exsisting files will be overwritten)

Additional context

this is a intermediate step for the whole serial connection

:white_check_mark: Tasks

[x] libusb included https://github.com/AlexanderTonn/OWON-CSV-INSPECTOR/commit/30687a8f29454d0d044a3559a4427ffd7b809339
[x] search for the owon mass storage among all connected devices  https://github.com/AlexanderTonn/OWON-CSV-INSPECTOR/commit/fdd13836bec8139d8b58369174148a93e38161e9
[x] add a menu option for automatically detecting the OWON device if configured as mass storage device
[x] copy and save waveform files automatically
[x] after finishing of copy process eject the owon device safely
[ ] as the circumstances require check the Github Actions workflows

While doing the tests on Linux (debian) i encountered the problem that debian isn't working with volume name as macOs or windows it does. For this reason i will looking for the size of the mass storage for identifing it
See mentioned below at line of sdb/sdb1
alexander@alexDebian:~$ sudo lsblk
NAME   MAJ:MIN RM   SIZE RO TYPE MOUNTPOINTS
loop0    7:0    0     4K  1 loop /snap/bare/5
loop1    7:1    0 105,4M  1 loop /snap/core/16574
loop2    7:2    0   104M  1 loop /snap/core/16928
loop3    7:3    0  55,7M  1 loop /snap/core18/2812
loop4    7:4    0  63,9M  1 loop /snap/core20/2182
loop5    7:5    0  74,2M  1 loop /snap/core22/1122
loop6    7:6    0 164,8M  1 loop /snap/gnome-3-28-1804/198
loop7    7:7    0 504,2M  1 loop /snap/gnome-42-2204/172
loop8    7:8    0  91,7M  1 loop /snap/gtk-common-themes/1535
loop9    7:9    0   1,6G  1 loop /snap/kicad/10
loop10   7:10   0   1,6G  1 loop /snap/kicad/9
loop11   7:11   0  78,4M  1 loop /snap/office365webdesktop/5
loop12   7:12   0 431,9M  1 loop /snap/qt515-core20/31
loop13   7:13   0 186,1M  1 loop /snap/signal-desktop/640
loop14   7:14   0 186,4M  1 loop /snap/signal-desktop/642
loop15   7:15   0  12,9M  1 loop /snap/snap-store/1113
loop16   7:16   0  17,3M  1 loop /snap/whatsie/148
sda      8:0    0 931,5G  0 disk
├─sda1   8:1    0 920,9G  0 part /
└─sda2   8:2    0  10,6G  0 part [SWAP]
sdb      8:16   1     8M  0 disk
└─sdb1   8:17   1   7,8M  0 part /media/alexander/disk
sr0     11:0    1  1024M  0 rom


Scrolling nav scrolls twice

Hi. I love this theme, however I have a small issue that I cannot figure out. When I click on a link on the scrolling nav bar, it scrolls to that anchor twice. It will scroll to that section, then go back where you were on the page, then scroll to that section again. The website is at www.westberryautotransport.com. Can you please help me with this issue.

The trigger is happening twice, maybe due to some of the other js you've added?
Here's a workaround (credit to gajotres):
freelancer.js
// jQuery for page scrolling feature - requires jQuery Easing plugin
$(function() {
    $('body').on('click', '.page-scroll a', function(event) {
    	$(this).off('click', '.page-scroll a', function(event) {
			var $anchor = $(this);
			$('html, body').stop().animate({
				scrollTop: $($anchor.attr('href')).offset().top
			}, 1500, 'easeInOutExpo');
			event.preventDefault();
    	});
    });
});

Hope this helps,
Stephen

Thank you very much for your help. That workaround fixed the issue. Now I guess I just need to dig through the rest of my JS and find out what caused that to happen.


Need information about DFS-share and replication

Can you add a section about putting the DFS-Share and replication on the front-end itself ?  Is it possible, allowed, supported or not supported ?

Thank you for this feedback and for contributing to the wellness of the content! This is a good idea and we have added it into our backlog for triage and prioritization.

@tseward Is investigating this.

As it an internal trasfer from one system to other so can be closed


Flywheel crash whit Optifine (1.19)

Describe the Bug
Hello first I wanted to say, yes I know this is not a finished version of the mod. But I wanted to report a bug. The 1.19.2 version of the mod not compatible with Optifine and crashes as soon as the loading of the Game has begun. so that's it good luck furthermore :)
Reproduction Steps
1.Launch the Game in 1.19.2 + Optifine
2.wait until it crash
...
Expected Result
The Game does not Crash
Screenshots and Videos
No response
Crash Report or Log
https://pastebin.com/QTLpRdqP
Operating System
Windows 11
What is your GPU?
RTX 2070 Super
Mod Version
0.6.5
Minecraft Version
1.19.2
Loader Version
Forge 43.1.25
Other Mods
Optfine HD U H9  --  (HD U I1 also crash)
Create 1.19
Additional Context
No response

This is not a Flywheel issue. Your version of Optifine does not work with your version of Forge, so you can downgrade Forge, update Optifine if there is an update, or uninstall Optifine.

ok, now i use the correct version of Forge and the Game starts now. But if you now loade a world it crash as soon as the Loading is complete.


simple-preavis:0.1.0

I am submitting

[x ] a new package
[ ] an update for a package

Description: The package is a template for writing a french move out letter. I am a student and need to write a letter to move out from my appartement typst help me make it standard, and be able to write it multiple times, I think it might be helpful for other french people that are in my case.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

[x] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

Thank you :)

You still need to upload a thumbnail for your template and specify that in your typst.toml, but then it should be good to merge.

Thanks!


Add WSS support

Needed when connecting from HTTPS host.

Thank you!  I'll cut another build this evening.

https://github.com/reactotron/reactotron/releases/tag/v0.9.0 is now out.  Thank you sir.


Expose classes and set nested type names

Required for non-JSON serialization.

@martint: please take a look

There are build failures

Sorry about that - I didn't realize you were going to merge it. Just wanted you to check the other changes I made to have stuff be consistent after changing the field type to be Optional. Fixed the build failures.

looks good


Update nlog.xsd

for NLog 5

One might consider to have a dedicated NLog4.xsd, since I have changed so obsolete properties are not included in the NLog.xsd.

Ow maybe we need a nlog5.xsd then?
(Or nlog.xsd + nlog4.xsd. doubting...)

I prefer that NLog.xsd is the latest official version. But not going to cry if you prefer having NLog3.xsd + NLog4.xsd + NLog5.xsd
But most documentation recommends using NLog.xsd with latest NLog-build.

Ok nlog.xsd + nlog4.xsd then :)

Resolved by https://github.com/NLog/NLog.github.io/pull/191


String.prototype.matchAll polyfill sometimes repeats a match incorrectly

here is an example I have in a test suite that breaks with the polyfill for matchAll
>>> var s = "<test><foo2/></test>"
>>> var p = p = /(?=<(?<tag>\w+)\/?>(?:(?<text>.+?)<\/\k<tag>>)?)/gu
>>> [...s.matchAll(p)]
[
    ["", "test", "<foo2/>", index: 0, input: "<test><foo2/></test>", groups: {…}],
    ["", "foo2", undefined, index: 6, input: "<test><foo2/></test>", groups: {…}],
    ["", "foo2", undefined, index: 6, input: "<test><foo2/></test>", groups: {…}], // I shouldn't be here
]

The final match is incorrect. When using the native javascript implementation the final match is not present.
I hope that's enough to go on.

Thanks for the report


test: Skip TestVolumePersistence

This test fails quite regularly in CI, we know the fix, and this area of code (i.e. volume management) isn't currently being worked on, so skip for now until we fix the underlying issue.
Ref #1125.

LGTM


feat(atomic): add formats to result value

https://coveord.atlassian.net/browse/KIT-475
Tell me if you think these options are enough currently. Tests will need to be added as well.

I was thinking of multiple components for these instead of adding options to one, I guess Im convinced I should now!

Proposition:
<atomic-text-field-value field="the_field_identifier" />

<atomic-date-field-value field="the_date_field_identifier_otherwise_default_is_date" format="specify_the_date_format_otherwise_sane_default" />

<atomic-currency-field-value field="the_price_field_identifier_otherwise_default_is_default_price_commerce_field?" currency="current_to_use_otherwise_default_is_$" />

<atomic-number-field-value field="the_field_identifier" format="sane_default" />

@olamothe I'd rather name them atomic-result-text-value etc. with property instead of field. The reason is because it fetches the property on the result object, and if it doesn't exist, it gets it from the raw object. Maybe I'm just confused as to what fields are 🤔


Segmented 3DGS

The first Figure is bitmap after prompt_segment;
The first Figure is segmented 3DGS.
Why is it different?

The rendered mask has some difference with the segmented Gaussians. This is because that we use a little threshold on the rendered mask.

Hi. The 0.5 in the notebook

plt.imshow(rendered_mask_pkg['mask'].squeeze().detach().cpu() >= 0.5)


fix(challenges): Replace camper leaderboard project

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #17450

@scissorsneedfoodtoo See the following screenshot for some strange formatting for at least one row.

Sorry about that! Should be fixed now:

And that's a good question. Let me ask around and see if there's an official one.

@scissorsneedfoodtoo When browser width is reduced.

We do have an official account that we use and share pens from. @RandellDawson @scissorsneedfoodtoo when you guys have finalised the pen is good to go, I will fork it and share the link here for you to update.

We do have an official account that we use and share pens from.

What is https://codepen.io/freeCodeCamp/ then?

@RandellDawson, worked on this a bit over the weekend. Could you take another look?

@scissorsneedfoodtoo This looks great!  I did notice one thing - the activity column is displaying the time since the last post, rather than since the last activity.  If you change
<CountField className="activity">{moment(topic.last_posted_at).fromNow(true)}</CountField>

to
<CountField className="activity">{moment(topic.bumped_at).fromNow(true)}</CountField>

that should sort it out.

@ojeytonwilliams, great catch, thank you! I didn't notice that. I'll fix that now.

You'll have to make the changes to testable-projects-fcc as well

i

Fixed another bug where clicking on the profile picture opened a dead link. Here's the latest version: https://codepen.io/scissorsneedfoodtoo/pen/BEjvom

Looks good to me - nice job @scissorsneedfoodtoo! Should we change this line: You can get feedback on your project by sharing it with your friends on Facebook. to point to the forum instead of facebook?
Looks like the pen is ready to be put on the fCC codepen account @raisedadead.

@moT01, thanks! I can't take credit for this project, though. It's really a fork of @jgibson02's Pen that I added some styling to.
Pointing to the forum rather than Facebook is a great idea. I'll make that change now.

The new pen is available at https://codepen.io/freeCodeCamp/pen/JqdoMV


Doc Issues Rectification and Updation

Describe the bug
There are a couple of instances where the docs are giving misleading information, also one who would like to work on the project but does not have experience working on an extension should get a basic understanding of how the file system and all work for this project. This section should be added to the doc.

This issue has been resolved. The pull request has been linked.


Rework Mikrotik device scanning following Unifi

Breaking Change:
This PR adds support for configuring Mikrotik integration through config flow. Importing from config yaml is also supported.
Description:

This PR reworks the Mikrotik integration to follow Unifi implementation and add config flow support.
If capsman or wireless interface is detected it will scan it and add/update devices status automatically. Otherwise it will fallback to DHCP list as source of devices.
The following options are included:

Force DHCP: Force using DHCP for scanning devices
ARP ping: Ping devices that have an active lease but are not in the wireless list. (Default is False)
Consider home interval: A device will change state to away if not detected after the set period. (Default: 300 seconds)

Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

@MartinHjelmare I made a new pull request because I somehow messed up the commits in the previous one (#27093) . I have improved the device scanning algorithm and added tests.
Please take a look and let me know your comments

Hi @MartinHjelmare , any chance you can look at this PR and let me know your comments.
Thanks

Sorry, this week is busy for me. I'll try to find some time.

Testing this Mikrotik integration for about a week now. Two concerns:

It doesn’t seem to handle disabled entities properly
consider_home doesn’t seem to work as expected. Or should we use discover_interval instead?

Testing this Mikrotik integration for about a week now. Two concerns:
* It doesn’t seem to handle disabled entities properly

* consider_home doesn’t seem to work as expected. Or should we use discover_interval instead?

Can you provide more details:
What is happening to disabled entities?
Are you testing with wireless or wired devices?
How much did you set the consider_home value and after how long is it setting the device as away?

Can you provide more details:
What is happening to disabled entities?
Are you testing with wireless or wired devices?

I am testing with wireless devices.
2019-11-10 21:39:53 WARNING (MainThread) [homeassistant.helpers.entity] Entity device_tracker.hs110 is incorrectly being triggered for updates while it is disabled. This is a bug in the mikrotik integration.
2019-11-10 21:40:09 WARNING (MainThread) [homeassistant.helpers.entity] Entity device_tracker.harmonyhub is incorrectly being triggered for updates while it is disabled. This is a bug in the mikrotik integration.
2019-11-10 21:40:25 WARNING (MainThread) [homeassistant.helpers.entity] Entity device_tracker.vlads_ipad_pro is incorrectly being triggered for updates while it is disabled. This is a bug in the mikrotik integration.
2019-11-10 21:42:31 WARNING (MainThread) [homeassistant.helpers.entity] Entity device_tracker.vladimiplewatch is incorrectly being triggered for updates while it is disabled. This is a bug in the mikrotik integration.

How much did you set the consider_home value and after how long is it setting the device as away?

This one cleared up by itself. It is possible I had an inconsistent wifi connection. Sorry, my bad.

@velaar , Thanks for the feedback and testing. Updating disabled entities can be fixed I will update the code.
Can you please confirm that the functionality of the scanning is working as expected. Any concerns?

@engrbm87 scanning functionality works as expected for me. Thank you for fixing the disabled entities. Wish there was an easy way of managing entities but this is obviously out of scope for this integration.

@engrbm87 scanning functionality works as expected for me. Thank you for fixing the disabled entities. Wish there was an easy way of managing entities but this is obviously out of scope for this integration.

ًWhat do you mean "easy way"?

Hello @balloob . For this PR I am setting the system option enable_newly_added_entities using an option in the config flow initial setup. Is that OK or how shall we let the user decide if they want to enable new entities by default or enable manually later?

@engrbm87 Can you rebase this PR?
Also, while at it, feel free to pin the librouteros version to 3.0.0.

I'm closely following this component and after the most recent update got this:
2020-01-03 11:55:19 ERROR (MainThread) [homeassistant.setup] Setup failed for mikrotik: Unable to import component
As I'm running HASS in docker not sure how to diagnose. The only component to import seems to be librouteros

The coverage is below 95% in the device tracker and hub modules. It's ok to exclude these from coverage calculation for now.


Is this repo dead?

Last merge was 3 months ago. Is everything being done on the tensorflow repo now?

@ns3284
Please refer to https://discuss.tensorflow.org/t/keras-project-moved-to-new-repository-in-https-github-com-keras-team-keras/1999
Moving this issue to closed status as there has been no activity, in case you still face the error please create a new issue.


[FE] Create a custom cursor

Objective:
Introduce a custom cursor design to enhance the visual and interactive experience of the application.
Task:

Create a custom cursor design and implement it across the application.
The cursor should look like the dot from the logo. When we are in the Blogs Section it should transform and be bigger. Check out Figma file for the details on how it will look.

Inspiration:
https://www.scalzodesign.be/

i have made a pull request heres's the link of the pr https://github.com/learnhubmkd/app/pull/84

@edichoska can you please share the Figma file link? I wasn't able to find the design for the cursor in the existing file, and I am having some doubts regarding the 'Next' text in the cursor when hovering over the blog section.

@stoimenov95  I don't have a Figma example of how this would look.

I was imagining something like this:
If it is looking "weird" maybe we can only have the arrow.

@edichoska only the arrow looks much better instead of the 'Next' text IMO.

I cand change like this to be if you want ?
and also i change the name of the section .large to .largeCursor


github-runner: 2.286.0 -> 2.287.1

Motivation for this change

github-runner: disable telemetry in createDepsFile
github-runner: 2.286.0 -> 2.287.1
github-runner: refactor disabledTests
nixos/github-runner: use --disableupdate flag

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Triggered a new runner registration. Seems to work fine.


xargs behaves weirdly

When you compile curl using MSVC and then do this command it launches a lot of xargs processes. It should launch 1-2 MAX.
seq 1<PHONE_NUMBER>000 | xargs -n1 -P16 c:\bin\curl

Which Windows version are you using?

Windows 10 LTSB.

@joeparkerz
Good News: Interop is much-improved in the most recent insider builds
Bad News: LTSB will not be receiving an update for two years.

@fpqc Good point.

@joeparkerz - Could you provide some more complete repro steps please?  Trying the above I'm only seeing a single xargs process being launched.
What build are you running?


SAK-42806 - Samigo > Restore Assessments > Need to update Modified Da…

…te when deleting

I just changed this so it updates the modified date when deleting. It wouldn't work for old assessments deleted but the alternative possibility of a new column to reflect the deleted date wouldn't work for that either. So this seems like lowest risk change to indicate when this was deleted.

I used the methods on AssessmentData rather than updating the query here too. Also looking at this it seems like the "retry" loop that's used through this whole method could probably be refactored and cleaned up. But there's probably a lot of redundant code in Samigo.

Based on the suggestions from John on the jira I changed the restore to also set the modified date.

I think I missed updating the PublishedAssessmentFacadeQueries though. I'll check that out and update that too.


Number of training steps

How can I define the number of training steps? In my attempt, the code has stopped running after 16000 steps and a final accuracy is shown. how can I increase or decrease these steps?

@montaser85
total training steps is changed by batch_size and num_epochs
It is calculated by the below equation.
total steps = num_epochs * (size of dataset / batch_size)
https://github.com/roomylee/rnn-text-classification-tf/blob/35a8a48d163a75f2c3bc3bfd1c42e3999c866e95/train.py#L28


Add is_valid_letter() to TextServer

Fixes https://github.com/godotengine/godot-proposals/issues/7656
Test project:
letter_test.tar.gz
related PR:

https://github.com/godotengine/godot/pull/75846

The code works, but the list of ranges is not complete. I used https://www.unicode.org/Public/UCD/latest/ucd/PropList.txt, but it's  only a subset. https://unicode.org/Public/UNIDATA/DerivedCoreProperties.txt seems to be the correct file for this…

@adamscott: ping! you wanted to review this PR?

@adamscott: reminder of this PR

Thanks!


Repackage class in metrics-lookout.

Modification:

Repackage class in metrics-lookout.

Codecov Report

Merging #146 into master will increase coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #146      +/-   ##
============================================
+ Coverage     68.57%   68.64%   +0.06%
- Complexity      943      944       +1
============================================
  Files           339      339
  Lines         14156    14156
  Branches       2268     2268
============================================
+ Hits           9708     9717       +9
+ Misses         3281     3276       -5
+ Partials       1167     1163       -4

Impacted Files
Coverage Δ
Complexity Δ

...a/rpc/metrics/lookout/RpcAbstractLookoutModel.java
100% <ø> (ø)
15 <0> (?)

...ofa/rpc/metrics/lookout/RpcClientLookoutModel.java
90.9% <ø> (ø)
7 <0> (?)

...om/alipay/sofa/rpc/metrics/lookout/RpcLookout.java
77.08% <ø> (ø)
14 <0> (?)

.../alipay/sofa/rpc/metrics/lookout/RpcLookoutId.java
76.74% <ø> (ø)
9 <0> (?)

...ofa/rpc/metrics/lookout/RpcServerLookoutModel.java
80% <ø> (ø)
3 <0> (?)

...a/com/alipay/sofa/rpc/event/LookoutSubscriber.java
87.17% <ø> (ø)
14 <0> (ø)
:arrow_down:

...ay/sofa/rpc/client/AllConnectConnectionHolder.java
53.91% <0%> (+0.55%)
0% <0%> (ø)
:arrow_down:

.../alipay/sofa/rpc/registry/local/LocalRegistry.java
57.83% <0%> (+0.6%)
26% <0%> (+1%)
:arrow_up:

...alipay/sofa/rpc/registry/zk/ZookeeperRegistry.java
57.08% <0%> (+0.78%)
0% <0%> (ø)
:arrow_down:

...ava/com/alipay/sofa/rpc/config/RegistryConfig.java
36% <0%> (+1%)
0% <0%> (ø)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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


Fix nacos config namespace

Describe what this PR does / why we need it
when using spring.cloud.nacos.config.namespace=public  , set 'public' to ''
Does this pull request fix one issue?
refer #2872 #2869

Why don't solve it in com.alibaba.cloud.nacos.NacosConfigProperties#assembleConfigServiceProperties directly?

Why don't solve it in com.alibaba.cloud.nacos.NacosConfigProperties#assembleConfigServiceProperties directly?

Ok , i fixed it in NacosConfigProperties

LGTM.

Please create a same PR to branch of 2.2.x.


Error from number of outputs

Within your YoloV8::detectObjects function, your code checks to make sure the number of outputs == 1 for a detection model; however, when I take my custom yolov8-trained model for detection, it has 3 outputs...thus it thinks its a segmentation model and errors out.
Here's the command I use to train my custom model:
yolo task=detect mode=train model=yolov8m.pt data=custom.yaml plots=True epochs=200 imgsz=640,480
Once trained, I use your pytorch2onnx.py script to convert it to an onnx file.
This all seems odd to me since if I take one of the yolov8 provided models, e.g. yolov8m.pt, and convert it using your script, it only shows 1 output and runs fine.
Not sure what's going on.  Any ideas??
Thanks!

Looks like it was a error in the way I converted the model to onnx.  I think I've fixed it.
Sorry!
Thanks for this repo!

@rbgreenway can you expand more on what changes you made to the conversion script that fixed this problem, as to help people in the future if they have the same problem.

@cyrusbehr I somehow converted it from a pt, or downloaded it from somewhere...I can't seem to recreate!  However, i did find a similar yolov8 onnx file with 4 outputs.  You can download from
https://github.com/kn1ghtf1re/yolov8-deepstream-6-1
Here's an image of output of the network taken from Netron.  As you can see, there are 4 outputs.

Not sure it's important now, but I'm very curious how they took the native .pt file from the results of training a yolov8 network, and converted it to such an onnx file with multiple outputs.  This looks like the "old style" outputs of yolo.

@cyrusbehr
Figured it out.
The conversion tool provided at:
[(https://github.com/marcoslucianops/DeepStream-Yolo/blob/master/utils/export_yoloV8.py)]
creates multiple outputs that is compatible with his popular Deepstream plugin.
The ultralytics model.export function collapses that into a single output, which is the format that you use in your TensorRT interface.
I now know how to use both approaches. Thanks!

I'm not certain, but it's possible you exported to onnx as end2end, which means that the model contains bbox decoding and nms (whereas my implementation does this using C++ code external to the model).

I believe the "classic" yolo (i.e. Yolo 2/3/4) output is scores, classes, and boxes...like this:

For integration with DeepStream, the popular plugin linked in my previous comment decodes the bounding boxes outside of the model, and is compatible with most, if not all, the yolo variants, including yolov8.  Also in that repo, is a pytorch-to-onnx conversion script that creates an onnx file compatible with the Deepstream plugin...but not compatible with your TensorRT code.  However, I am now putting your code into a new Deepstream plugin mainly because it gives me better control over some of the special things I want to do with an image before inference.


#0: Move kernels out of tt_eager/tt_dnn/kernels

awesome!
where do we keep softmax dataflow kernels?

awesome!
where do we keep softmax dataflow kernels?

tt_eager/tt_dnn/op_library/softmax/kernels/dataflow
There were some already there and then a couple in the parent directory. I just moved them into the dataflow subfolder as well

@muthutt they are moved not deleted/added.
The only ones not correctly tracked are sm and ln compute kernels due to git not determining move vs change correctly. Only way to make it correctly track those is to split the commits but I don't think that's necessary.


Comments fix

Just a tiny comment fix because I happened to see it, fun project!

OMG good job past me. Good job.


Roll Flutter Engine from e9ca7b2c457b to fafea2be8e43 (3 revisions)

https://github.com/flutter/engine/compare/e9ca7b2c457b...fafea2be8e43
2023-03-10<EMAIL_ADDRESS>Roll Skia from bd14d5f42db9 to af3cd5046449 (1 revision) (flutter/engine#40205)
2023-03-10<EMAIL_ADDRESS>Roll Dart SDK from 7240b35cc401 to c522cbe4fb12 (5 revisions) (flutter/engine#40203)
2023-03-10<EMAIL_ADDRESS>Add platform channel System.exitApplication and System.requestAppExit support (flutter/engine#40033)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Variables Evaluate is NOT Working for Struct Pointer of Function Variable

If submitting a bug please make sure

[Y] If you are using gdb

[Y] gdb --version >= 7.7.1
[Y] it works on the command line with gdb
[Y] cwd and target are properly set

[ ]  If you are using lldb

[ ] lldb --version >= 3.7.1
[ ] it works on the command line with lldb-mi
[ ] cwd and target are properly set

Screenshots are helpful but not required
All extension versions matter.
My Gdbs with verison 10.1/9.x have same problem

This PR
https://github.com/WebFreak001/code-debug/pull/414/files
Deal with this problem, but maybe not the best solution

Thank you for both raising this and providing a PR for the fix. LGTM.
Can you please rebase the PR and add a Changelog entry, as well as providing an "after" picture?


Update displaying-verification-statuses-for-all-of-your-commits.md

Why:
Closes [issue link]

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[x] This pull request impacts the contribution experience

[x] I have added the 'writer impact' label
[x] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

mian


How to remove gutter gap in reading view?

@mugiwara85 Thank you for creating this excellent plug-in. I discovered it just yesterday and find very useful.
One small visual quirk I would like to change are the gutter gaps in reading view, as shown here:

These gutter gaps are not present in the editing view, and I think it looks much nicer:

Can you please tell me how I can make the reading view look the same as the editing view for the gutter?

Hi @helicos6, just to let you know this has been addressed and is scheduled to be fixed in the upcoming release!

@mayurankv Thank you. 🙂

@helicos6 As mayurankv said, in the next release this will be fixed. Unfortunately the release is delayed a little, because the plugin is undergoing major changes (from the point of how and which CSS classes are used). The next release will also include some smaller new features.

@mugiwara85 Thank you, looking forward to the update. 😃

It should be fixed now. Should there be any problem, please get back to me.


Unable to reclaim old stack names after move from socket to tcp for docker host

Before you start, we need a little bit more information from you:
Use Case (delete as appropriate): Using Portainer at Home
Have you reviewed our technical documentation and knowledge base? Yes
Question:
I recently started implementing Docker Socket Proxy for my various containers. Unfortunately with Portainer I made the mistake of deleting the original Endpoint and then adding the new one. Now all Stacks (e.g. Compose files, not using Swarm) show as Limited control. I can delete the attached Container, which makes the old Stack disappear, but when I try to recreate them I get a message that Stack already exists with that name. Since I'm not in Swarm mode, I cannot impact whatever Portainer is referencing from the Docker side.
Looking for any help as to how I can reset things so that Portainer has full management using the original Stack names I used.
Thanks!

Follow up, I found a way to edit the Portainer database directly and move the stacks from the old endpoint to the new one. This error was not intuitive, and the GUI gave no method to resolve the issue without completely blowing away Portainer's database. It would be nice if there was either a way to intentionally move stacks from one endpoint to another, or at least be able to delete orphaned ones.


[BUG] Dasel crash when reading an empty file

Describe the bug
Dasel will crash if called on an empty file, even if it's by specification in most implementations interpreted as  a valid file for that format (yaml and toml).
(Yes, I just read both TOML and YAML specs, and it's ambiguous as in both case an unquoted empty string is corresponding to nil but the lexer seems to assume at least one identifier)
To Reproduce
touch /tmp/bug.toml
dasel -f /tmp/bug.toml

Will fail:
panic: reflect: call of reflect.Value.Interface on zero Value

goroutine 1 [running]:
reflect.valueInterface({0x0?, 0x0?, 0x0?}, 0x0?)
	reflect/value.go:1495 +0xfc
reflect.Value.Interface(...)
	reflect/value.go:1490
github.com/tomwright/dasel/v2.Value.Interface({{0x0, 0x0, 0x0}, 0x0, 0x0, 0x0})
	github.com/tomwright/dasel/v2/value.go:55 +0x6c
...

Expected behavior
Such files should be handled like most parsers interpreted it, so nil. This way creating a new toml file can be as simple as:
touch /tmp/state.toml
dasel put -f /tmp/toml .state.last-execution -t string -v $(date -Iseconds)

Desktop (please complete the following information):

OS: macOS 14.1
Version: dasel version 2.4.1

Thanks for highlighting this @Larandar .
Dasel has an issue with nulls at the moment, stemming from the fact that it uses reflection internally. That said, it shouldn't be too hard to get this working. I'll work on this soon.

I think I accidentally broke this by commenting this out: https://github.com/TomWright/dasel/blob/3bfb5dd1b1415b009ebf06660225a9e2de772523/internal/command/select.go#L107
I'll try and verify this tonight

This has been fixed in the development release and will be released soon.

Ha, the good old "I'm sure this code is not applicable anymore" 👍

Hey @Larandar,
This was included in the last release.
$ dasel -v
dasel version 2.5.0
$ touch empty.toml
$ dasel -f empty.toml


Allow to apply an opaque when drawing widget

It will be useful when making animation like fade in/out.

I'm not sure I understand this question. An opaque background color?

I'm not sure I understand this question. An opaque background color?

Yes, but apply on widget(Like #1631 I guess?), which means we won't need to use a rectangle with same background color to cover the widget we need to make a fade-in or fade-out animation on.
I'm a little poor in English, reply me if you still confusing. :D


simple flake8 updates

Removed a few imports that didn't seem to be needed and then just updated comment formatting.
I left alone other suggestions like old style exception raising  and != None vs is not None suggestions.  For another PR.

Ready for review @somexpert @ldko

Are we concerned about having all the first words in the comments capitalized? There are some that aren't.

For now I was just trying to push through the "passes flake8" without digging into the actual content.  Might be something we can focus on after a first deployment.  Might be time spent better in other places?

Makes sense. Well in that case, LGTM :thumbsup:

works for me :+1:


fix(spanner): update optimizer_statistics_package to latest

@dazuma Can I get an approval from you? Thanks.


restart docker before running tests on CI

What this PR does:
Trying to fix the:
Step #4 - "cassandra-3.11": ERROR [main] 2022-01-19 19:06:27,600 DockerClientProviderStrategy.java:208 - Could not find a valid Docker environment. Please check configuration. Attempted configurations were:
Step #4 - "cassandra-3.11": ERROR [main] 2022-01-19 19:06:27,600 DockerClientProviderStrategy.java:210 -     UnixSocketClientProviderStrategy: failed with exception TimeoutException (Timeout waiting for result with exception). Root cause IOException (native connect() failed : Permission denied)
Step #4 - "cassandra-3.11": ERROR [main] 2022-01-19 19:06:27,600 DockerClientProviderStrategy.java:212 - As no valid configuration was found, execution cannot continue
Step #4 - "cassandra-3.11": [ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 30.417 s <<< FAILURE! - in io.stargate.it.cql.JwtAuthTest
Step #4 - "cassandra-3.11": [ERROR] io.stargate.it.cql.JwtAuthTest  Time elapsed: 30.417 s  <<< ERROR!
Step #4 - "cassandra-3.11": java.lang.reflect.InvocationTargetException
Step #4 - "cassandra-3.11": 	at io.stargate.it.cql.JwtAuthTest.buildParameters(JwtAuthTest.java:53)
Step #4 - "cassandra-3.11":

/gcbrun

DSE build failed with Codacy push.. So all green again..


chore: upgrade packages and adjust tests

Pull Request Test Coverage Report for Build 450

6 of 7 (85.71%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.297%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/test.js
6
7
85.71%

Totals

Change from base Build 431:
0.0%

Covered Lines:
137

Relevant Lines:
141

💛  - Coveralls

Hi @rimiti ,
Could you review that and eventually release as 5.0?
Thanks!


build: use 'node-version-file' to read '.nvmrc'

Checklist

[x] the [contributor license agreement][] is signed
[x] commit message follows [commit guidelines][]
[ ] tests are included
[ ] screenshots are included showing significant UI changes
[ ] documentation is changed or added

Description of change
The GHA action setup-node now provides a property which is used to read .nvmrc. So update our CI config to use that.

There we go, storybook failing as expected.

Aha! And now rest-module-testing is failing as expected too:
FAIL test/Heartbeat.test.ts
  ● checkHeartBeat › returns a resolved promise if server responds correctly

    expect(received).resolves.toBeTruthy()

    Received promise rejected instead of resolved
    Rejected to value: [Error: connect ECONNREFUSED ::1:8080]

      21 | describe('checkHeartBeat', () => {
      22 |   it('returns a resolved promise if server responds correctly', async () => {
    > 23 |     await expect(checkHeartbeat(TC.API_PATH)).resolves.toBeTruthy();
         |           ^
      24 |   });
      25 | });
      26 |

I'm thinking that perhaps as part of this PR we should also revert 0c6dd7701ea80965312739cfa8c1d987393c065c. That should then produce a green build and then we can merge it in.
What do you think @PenghaiZhang ?

Aha! And now rest-module-testing is failing as expected too:
FAIL test/Heartbeat.test.ts
  ● checkHeartBeat › returns a resolved promise if server responds correctly

    expect(received).resolves.toBeTruthy()

    Received promise rejected instead of resolved
    Rejected to value: [Error: connect ECONNREFUSED ::1:8080]

      21 | describe('checkHeartBeat', () => {
      22 |   it('returns a resolved promise if server responds correctly', async () => {
    > 23 |     await expect(checkHeartbeat(TC.API_PATH)).resolves.toBeTruthy();
         |           ^
      24 |   });
      25 | });
      26 |

I'm thinking that perhaps as part of this PR we should also revert 0c6dd77. That should then produce a green build and then we can merge it in.
What do you think @PenghaiZhang ?

I agree. And we can work on the node 18 update in another PR.

Okay, the build has already got past the pain points:

I'm going to merge.


Request xb1 Controller support for flashlight

Any chance we could get Xbox One controller support for the flashlight?  User could still pick what button to bind to.

Also for VR controllers please.
(I'm using a command voice to press F button as a workaround)


Docs – Automatic Compaction config is not a runtime property

With its focus on Druid configuration files, the Configuration Reference contains information about the dynamic configuration of automatic compaction set by calling an API.
Affected Version
0.22.0
Description

Relates to https://druid.apache.org/docs/latest/configuration/index.html#compaction-dynamic-configuration
It includes information about the table datasource that will be compacted, tuningConfig (#11936) , granularitySpec and ioConfig (#11980), including the amount of parallelism (maxNumConcurrentSubTasks) – and thus has more in common with the concepts of ingestion than of runtime properties

Proposal
Move this section of the documentation into a dedicated page closer to that on how to do reindexing / compaction.

@techdocsmith another section of the the Configuration Guide that could be moved out.


[Documentation]: Improve get-started document of data collection

Is there an existing issue for this?

[X] I have searched the existing issues

What kind of documentation would you like added or changed?
Provide useful code examples in the get-started documents to help the users to finish their daily work.

[ ] a list of daily tasks for machine learning researchers and data scientists;
[ ] polish core APIs and towhee operators for these tasks;

Why is this needed?
No response
Anything else?
No response

I put an initial list of tasks in the get started doc: https://github.com/towhee-io/towhee/blob/main/docs/get-started/data-collection.md#examples
We need to complete this task with @shiyu22, @fzliu and @soulteary


Persistence Storage

I have deployed the K8s cluster on Google clour platform using Kismatic
How do I provision persistence storage (gce-pd) or LoadBalancer.
I had created the storage class and tried to provision using kubectl, but it's showing pending and not creating.
kubectl apply -f ssd-storageclass.yaml
kubectl apply -f ssd-claim.yaml
https://cloud.google.com/kubernetes-engine/docs/concepts/persistent-volumes
or should we want to build storage like rook and manage as our self.

Kindly take this some one and respond to this query

@coolpalani when you deployed did you specify the options below:
  # Kubernetes cloud provider integration.
  cloud_provider:

    # Options: 'aws','azure','cloudstack','fake','gce','mesos','openstack',
    # 'ovirt','photon','rackspace','vsphere'.
    # Leave empty for bare metal setups or other unsupported providers.
    provider: ""

    # Path to the config file, leave empty if provider does not require it.
    config: ""

I didn't specify.
Could you let me know the procedure to specify.

@dkoshkin
I had even found sample cloud config file for Azure.
https://raw.githubusercontent.com/alexbrand/ket-on-azure/master/azure-cloud-provider.conf.sample
I don't find yet for google.
could you share some sample config file

Looks like you found a @techjw project already, he has successfully used https://github.com/techjw/k8s-gcp to setup cluster with cloud provider enabled.


Is axpy function needed?

COO spmv kernels need to works on the output vector c first.
COO cuda kernels only do c = A*b + c
Thus, in spmv (c = A*b = A*b + 0), we set zero in c for ensuring kernel works correctly
In Hybrid format, its spmv function is composed of ELL and COO spmv functions.
ell_mtx->apply(b, x); // x = (ELL) * b
coo_mtx->apply(1, b, 1, x); // x = 1 * x + 1 * (COO) * b

The second command spends time to multiply x with 1, which is redundant.
Does It have any performance issue?
If it does, maybe we need the axpy function like c = A*b + c instead of calling advanced spmv.
ell_mtx->apply(b, x); // x = (ELL) * b
coo_mtx->axpy(b, x); // x = x + (COO) * b

To clearify:  this is only a performance issue: the COO kernel sets the output vector to zero first, in the hybrid kernel we don't need/want that. Is it reasonable to have a separate kernel for that?

I have tested some matrices from Florida matrices to compare two hybrid spmv.
The Hybrid matrix strategy is automatically (imbalance with 0.8)
Hybrid spmv with Coo advanced spmv: It is the same flow as this repo now.
Hybrid spmv with Coo axpy: the flow is similar, but it calls the coo kernel without multiplying the output vector by 1.
The multiplying operation has the large effect on the performance when the nnz is small (< 1e6).

Thanks for looking into it. Looks like we should indeed go with axpy.

I also implement another Hybrid implementation.
It sets zero to the output vector and then applies Ell and Coo kernels simultaneously.
Ell and Coo kernels use atomic_add to add resulting value to the output vector.
Compare it with Hybrid with Coo apply2 function.

This can be closed, I think. As the problem with this issue is solved.


Fix issues with #871 when using Android Gradle Plugin 3.1

The changes in #871 have caused issues when using older versions of the Android Gradle Plugin, i.e. 3.1.x which is used by default with React Native.  This is reported in #904 and also #661
With 3.1.x, the current approach doesn't copy any of the font files specified.
This change creates two separate tasks, one to copy to the 3.1.X compatible path and one to copy for the 3.2.x compatible path.
I've tested with both 3.1.4 and 3.2.0 of the Android Gradle Plugin and both work with the applied change.

Still having issues with gradle plugin 3.1.3 and upgrading to 6.2.0. The only thing that worked for me was https://github.com/oblador/react-native-vector-icons/issues/741#issuecomment-443255565.


Middleware rewrite does not work

This issue was mentioned on discord and there is an open PR about it but I thought it would be good to open an issue to track it.
Basically middleware rewrite does not work when the nextjs site is deployed with sst.
To confirm this I forked open-next, added sst drop in mode to the example nextjs project and deployed it. Here is the deployed address for rewrite https://dooy830n0j9e.cloudfront.net/middleware-rewrite . The page does not work when accessed directly and is missing the check mark when you navigate to it. You can compare it to the deployed vercel one and see the green check mark https://open-next.vercel.app/middleware-rewrite
Here is the diff of me adding sst to the example https://github.com/serverless-stack/open-next/compare/main...mathisobadia:open-next:main. On a side note, I can open a PR to add it to the open-next example. It would be a good example of how to deploy with sst.

So I just did some digging into next code and the reason this does not work is because of the minimal mode
https://github.com/vercel/next.js/blob/e91de8e0ddfd5b4a996c0e5df6549fe3981e1f6d/packages/next/src/server/next-server.ts#L1862

Hey @mathisobadia, we re-architectured how middleware is deployed in v0.8.0. Details are shared here — https://github.com/serverless-stack/open-next#will-my-nextjs-app-behave-the-same-as-it-does-on-vercel
It's a lot more robust now. Give v0.9.0 a try!


How to setup networks (use ethernet)

I use Cyclone V SoC that has an Arm Cortex-A9 core
I'm sure that my linux kernel and device tree are correct
But I can't get the board ip address when it power up
root@armhf:~# ifconfig
lo        Link encap:Local Loopback
          inet addr:<IP_ADDRESS>  Mask:<IP_ADDRESS>
          inet6 addr: ::1/128 Scope:Host
          UP LOOPBACK RUNNING  MTU:65536  Metric:1
          RX packets:0 errors:0 dropped:0 overruns:0 frame:0
          TX packets:0 errors:0 dropped:0 overruns:0 carrier:0
          collisions:0 txqueuelen:1000
          RX bytes:0 (0.0 B)  TX bytes:0 (0.0 B)

It seems to lack dhcp service...
root@armhf:~# ls /etc/init.d/
README			checkroot.sh  killprocs		     mountdevsubfs.sh	    procps    rmnologin  udev	       urandom
bootlogs		dbus	      kmod		     mountkernfs.sh	    rc	      sendsigs	 udev-finish
bootmisc.sh		halt	      motd		     mountnfs-bootclean.sh  rc.local  single	 umountfs
checkfs.sh		hostname.sh   mountall-bootclean.sh  mountnfs.sh	    rcS       skeleton	 umountnfs.sh
checkroot-bootclean.sh	hwclock.sh    mountall.sh	     networking		    reboot    ssh	 umountroot

Should I recompile rootfs again?
Thanks

use my rootfs generator jk
You can simply install it in the environment using the following command:
# apt-get install isc-dhcp-server


Merge the package owners, align entries

package/* is associated with mkolesnik, /package/* with skitt, this
merges both entries so that shared ownership actually works.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

I think the package owner overlap is no longer there? But the file could still use some alignment help imho.

It was handled here: https://github.com/submariner-io/submariner/pull/812/files


Scheduling Algorithms of Operating System

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is.
In an OS there are FCFS ( First Come First Serve), SJFS (Shortest Job First Scheduling) and a lot more scheduling algorithms which are of utmost importance to learn about Operating Systems workings.
Describe the solution you'd like
A clear and concise description of what you want to happen.
I will make the different scheduling algorithms using C which is very important in Operating System.
Do you want to work on it
Mention the one language you want to be assigned for this issue.
I would be working on C programming Language

I would like to contribute to the above problem in java as I am a participant of GSSOC'21. Kindly assign this to me.

Working on the issue and submission already made.


DropPlan only Shows 1 User Story

It seems that DropPlan only displays 1 user story from our current sprint.
Later:  I see now that it only works with Task Level WI's.  Ok....

The drop plan is a tool to manage tasks within user stories, are any of your tasks missing? can you send me a screenshot or some more details to help fixing it?


[Bigtable] GCRules.GCRule Javadoc fix for createTable

Fixes #4056 and FIxes #4090 (Both of these issues are related to GCRule).

Have added a sample code for GCRule in BigtableTableAdminClient's createTable() & createTableAsync().
Have added one line note about GCRule execution.

@igorbernstein2 Please have a look.
Q: Should we consider to adding this note in client Javadoc as well?

@igorbernstein2
I have addressed feedback comments, Please have a fresh look.

@sduskis @igorbernstein2
I have rebased this PR with the latest master. Please have a look.


StrawberryShake 13.0.5 is not including GraphQLException Error Details in Reponse

Is there an existing issue for this?

[X] I have searched the existing issues

Product
Strawberry Shake
Describe the bug
This appears to be related to #5802, but is still happening in 13.0.5.
Background: We were running Hot Chocolate and Strawberry Shake version 12.16.0. We have an IHttpRequestInterceptor for authentication that throws a GraphQLException when our JWT is invalid. We have covered this with several integration tests.
After following your migration documentation to get everything updated to 13.0.5 in both Hot Chocolate and Strawberry Shake, our integration tests around this are now failing because the error is not hydrated in the same was as it was in 12.16.0. You can see in the below screenshots that before the update we got back our specific Error details from the GraphQLException we threw in our interceptor. The Code and Message are populated. After the update, the exact same test only has a generic 500 error message with no code populated.
When we hit the same endpoint with a bad JWT from Banana Cake Pop, we still get all the errors details we expected.
Before (12.16.0) - with Error Code and Message

After (13.0.5) - generic message without code

Steps to reproduce

Write an IHttpRequestInterceptor that throws a GraphQLException something like this:

public class TestInterceptor : DefaultHttpRequestInterceptor
{
    public override async ValueTask OnCreateAsync(
        HttpContext context,
        IRequestExecutor requestExecutor,
        IQueryRequestBuilder requestBuilder,
        CancellationToken cancellationToken)
    {
        await base.OnCreateAsync(context, requestExecutor, requestBuilder, cancellationToken);
        throw new GraphQLException(new Error(
            "Authentication Failed",
            code: "401",
            exception: new Exception("Failed to authenticate")
        ));
    }
}

Use .AddHttpRequestInterceptor(...) to register the interceptor.
Make an HTTP request using StrawberryShake and inspect the result. Notice the details from the Error provided to the GraphQL exception are not present.

Relevant log output
No response
Additional Context?
No response
Version
13.0.5

Yes, we need to implement the new transport spec for strawberry shake.

Thanks for looking at this @michaelstaib. I do one question to help my understanding. The website indicates that version 13 is now the stable build, but if I'm understanding you correctly, it sounds like version 13 isn't quite to feature parity with version 12. Did I misunderstand that version 13 is the current stable build?

I'm having this same issue with graphql requests that are failing.  I get a generic 500 internal server error.

Errors in our AggregateError returned by the IErrorFilter are also not included.
We also get the generic 500 error

Hi @michaelstaib may I know if you have any timeline on this issue? Is StrawberryShake v13.0.5 ready for production? Thanks.

Yes, around 4 weeks until 13.1

Hi @michaelstaib  any updates on this? Thanks.


Nothing showing

Hy to all and thanks for your work.
Can someone help, because I don't see the calendar module at all.
Here is my config:
			module: 'MMM-CalendarExt2',
			config: {
				calendars : [
				{
	name: "Mihai",
	icon: "noto-v1:beaming-face-with-smiling-eyes",
	url: 'https://calendar.google.com/calendar/ical/blabla/private-blablabla/basic.ics',
				},
				{
	name: "Romania",
	icon: "twemoji:flag-for-romania",
	url: 'https://calendar.google.com/calendar/embed?src=ro.romanian%23holiday%40group.v.calendar.google.com&ctz=Europe%2FBerlin',
				},
				{
	name: "Kita",
	icon: "noto:family-man-woman-girl",
	url: 'https://calendar.google.com/calendar/ical/blblbla.calendar.google.com/private-blablabla/basic.ics',
				},
					],
				views: [
				{
    name: "Mishu",
    mode: "upcoming",
    position: "bottom_bar",
    calendars: ["Mihai", "Kita"]
				},
				{
	name: "Rom",
    mode: "upcoming",
    position: "bottom_bar",
    calendars: ["Romania"],
				},
					],
				scenes: [
				{
        name: "Default",
	views: ["Mishu", "Rom"]
				},
					]
				}
},```

first, upcoming view looks not so pretty on bar position. Or you should modify CSS as your wish.
I think you might have plenty of events on the calendars. set maxItems of view to bigger values. For the performance issue, it is limited with 100 by default. Enlarge it to fit your calendars.

Thank you and I'm sorry for writing the code like that. I copyied through VNC Viewer. How can I post an image?

just drag and drop under section of  the textbox

Still nothing.

		module: "MMM-CalendarExt2",
			config: {
			scenes: [
				{
		name: "DEFAULT",
		views: ["Mishu", "Rom"]
				},
				],
			views: [
				{
		name: "Mishu",
		mode: "upcoming",
		position: "top_left",
		maxItems: 5,
		maxDays: 1,
		hideOverflow: false,
		filterPassedEvent: true,
		calendars: ["Mihai", "Kita"],
				},
				{
		name: "Rom",
		title: "Sărbători România",
		mode: "daily",
		maxItems: 100,
		scanInterval: 1000*60*30,
		type: "row",
		position: "bottom_bar",
		slotCount: 5,
		hideOverflow: false,
		filterPassedEvent: true,
		calendars: ["Romania"],
				},
			],
			calendars: [
				{
	name: "Mihai",
	icon: "noto-v1:beaming-face-with-smiling-eyes",
	url: 'https://calendar.google.com/calendar/yyy/basic.ics',
				},
				{
	name: "Romania",
	icon: "twemoji:flag-for-romania",
	url: 'https://calendar.google.com/calendar/embed?src=ro.romanian%23holiday%40group.v.calendar.google.com&ctz=Europe%2FBerlin',
				},
				{
	name: "Kita",
	icon: "noto:family-man-woman-girl",
	url: 'https://calendar.google.com/calendar/ical/xxx/basic.ics',
				},
				   ],
				},
		},

@mishu28nmv you are sure you dont have another module above it in your config file that doesn't have a location.
for some reason when i was testing my calendar and i would just have it like this. it also wouldn't show.
i think it is bacause the module above in this case clock doesn't have a position.
so i put the MMM-CalendarExt2 module above it and it worked.
	modules: [                         {                         module: "clock", 			position: "", 			config: { 			    timeFormat: "24", 			    displaySeconds: "true", 			    clockBold: "true", 			    displayType: "both", 			    anologFace: "face-006", 			    timeZone: "Europe/Amsterdam", 			} 		}, 		{   		module: "MMM-CalendarExt2", 		  config: {....},

I too am getting nothing and everything has a position.
{
module: 'MMM-CalendarExt2',
config: {
scenes:[
{
name: "DEFAULT",
views: ["upcoming"],
},
],
views:[
{
name: "upcoming",
mode: "upcoming",
position: "bottom_center",
maxItems: 5,
maxDays: 1,
locale: "en",
hideOverflow: false,
filterPassedEvent: true,
calendars: ["Steve"],
},
],
calendars: [
{
name: "Steve",
url: "https://calendar.google.com/calendar/ical/sabard%40gmail.com/private-817ab860d8166392a841503f0cd9d6ed/basic.ics",
},
],
},
},

You've set maxItems too small. maxItems:5 means letting your upcoming view load at most 5 events from calendars. but usually, your calendar would have more than 5, but the view will load only 5. And unless you filter or sort, the events loaded might not match your filterPassedEvent:true and maxDays:1.
Give enough value to maxItems, by example, 100 or more.

If what you want was to show 5 of LOADED EVENTS, use slotCounts.

Well, I'm using your config and there seems nothing wrong.
Are you using page-changing/sliding or show/hide controlling module together with this module?

No I am not.  that is very interesting.  So then there is nothing wrong with the code.  Thanks.

@Lotusvball are you sure you installed it correctly because thats also a thing i did wrong to at first. i did start with javascript and css 2 months ago.

"cd ~/MagicMirror/modules
git clone --depth=1 https://github.com/eouia/MMM-CalendarExt2
cd MMM-CalendarExt2
npm install"

Are there any log files I can look at to see what is going on.
I am getting this in the terminal:
Connecting socket for: MMM-CalendarExt2
Sockets connected & modules started ...
Launching application.
[CALEXT2] calendar:Steve >> Scanning start with interval:1800000
[CALEXT2] calendar:Steve >> Scanned: 35, Selected: 35
But nothing.  I have commented out all other modules.

@Lotusvball are you sure you installed it correctly because thats also a thing i did wrong to at first. i did start with javascript and css 2 months ago.

cd ~/MagicMirror/modules
git clone --depth=1 https://github.com/eouia/MMM-CalendarExt2
cd MMM-CalendarExt2
npm install

I will try again just to make sure.

What does this mean?
created a lockfile as package-lock.json. You should commit this file.

@Lotusvball
If you are using pm2 to manage MM process, first stop it.
Then, go to MagicMirror directory then try this;
npm start dev

It will show you the front-dev panel on your MM screen. (usually right side)
Then naviage to console tab of that panel. If you find some error(by red color), report me.

I used npm start dev and I see not error and I now see the calendar.
When I use this to start it I see nothing still:
cd ~/MagicMirror && DISPLAY=:0 npm start
Is there an issue with this command?

I just started it using pm2 start MagicMirror and no Calendar.

Not sure what changed, but I am now seeing the calendar.  Thanks for your patience.  Very strange.

I do have one more question if you don't mind.  What does the (an hour) mean?

Maybe you need some delay to show the calendar. the interval of drawing and the interval of parsing are independent so, if your parsing is not finished on drawing time due to slow connection or too many events to parse, you cannot see anything. (It will be refreshed on next drawing interval)
To avoid this, you can use firstDrawingDelay. you can give some delay before drawing first time to finish parsing calendars.
https://github.com/eouia/MMM-CalendarExt2/wiki/2e.-Configuration:Others
And..
(an hour) is a duration of events. Read this document also
https://github.com/eouia/MMM-CalendarExt2/wiki/Event-Time

Great.  Thanks so much.

Hi, I seem to be having a similar issue. I followed the installation directions and the fix suggestions described in this issue. 1. Reinstall plugin 2. start in dev mode 3. Increase firstDrawingDelay. I see nothing on the screen (i am able to see other modules).
I have this as my config.
		module: "MMM-CalendarExt2",
		config: {
			 rotateInterval: 90*1000,
			 firstDrawingDelay: 1000*10,
			 scenes: [ // `scenes` could have several scenes as elements of array
			 {
				   name: "My kids schedule",
				   views: ["Daily", "Weekly"],
				   //className: "CX2",
			 },

			],
			views: [
				{
				  name: "Daily",
				  mode: "upcoming",
				  calendars: ["Schedule"],
				  position: "top_center",
				  maxItems: 5,
				  maxDays: 1,
				  locale: "en",
				  hideOverflow: false,
				  filterPassedEvent: true,
				},
				{
				  name: "Weekly",
				  mode: "weekly",
				  position: "middle_center",
				  calendars: ["Schedule"],
				  type: "row",
				  slotCount: 5,
				  locale: "en",
				  hideOverflow: false,
				  filterPassedEvent: true,
				},
			  ],
			  calendars : [
				{
				  name: "Schedule",
				  icon: "noto-soccer-ball",
				  url: "https://calendar.google.com/calendar/ical/1d4a1djtgmai0fpapjvtr6cdhs%40group.calendar.google.com/public/basic.ics",
				  //url: "webcal://p23-caldav.icloud.com/published/2/MTU3Mzc5NzU3MTU3Mzc5N7QDyWhkoQhUDUPID--ipr-ZOrn-Q4_1aVyt-0MWrJp7YNRJozQcvp5txgCkR1Oa9aPtBd5BXqQ48ly68Sx7T98/"
				},
			  ],

		},
		  },

Help appreciated.


Name Validation Error Message - Add more explanation of valid input

Is it possible to add additional context to understand why the name validation fails? How different are the validations for the different pages?
In Services, I tried creating a service with a "denise:1" or "denise_" in the name. I just got the validation name failure, but it doesn't tell me what is wrong.

Will now say name contains invalid character: '_".  There aren't many other resource types that have valid characters listed in the API, but this works for any types that do.

v0.22.0-rc1
Verified fixed


Input is invisible in a narrow device

I can't see an input field in a narrow device:

Hi @soniccat, same as the other issues, if you're still having this problem let us know and we'll look over it.
Thanks 🙏


Update README.md

Update with correct values of type argument, returned for the timeout event.
Reflects on the actual values returned by the event as of right now, but you might wanna change the implementation, such that it matches the README, instead of the opposite.

Thanks @cinjoff!


HOSTEDCP-1046: Add ImageDigestMirrorSet to Config API comment

What this PR does / why we need it:
Add ImageDigestMirrorSet to Config API comment
Which issue(s) this PR fixes:
Fixes HOSTEDCP-1046
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.

Address PR comment on 4.13 backport #2863 - https://github.com/openshift/hypershift/pull/2863#issuecomment-1660305424.

/cherry-pick release-4.13

/approve
/lgtm


Update tutorial with the new dataset

Hey @bentrevett, Thanks for your tutorial. Since torchtext has updated the datasets with the new abstraction, I'm wondering if you plan to update the tutorial here. One of the users open an issue to follow your tutorial with the new dataset. Thanks. https://github.com/pytorch/text/issues/913

Hi @zhangguanheng66. Yes, I'm planning to update the tutorials to the new abstractions. They are currently in progress in the experimental folder. Hopefully will have some time to put significant work into them during this week - should hopefully get all of the code done by then and I can focus adding the in-line explanations later.
Would be great to get your feedback on them when they're coded up, especially in terms of how my implementations align with your planned usage of the abstractions.


username / password

is it possible to add username and password to the commands?

I like this idea.  I didn't know anyone was running STUN with credentials,
but this makes sense.  Probably a good idea to offer long-term and
short-term credential options on the command line.
On Sun, Dec 25, 2022 at 3:02 AM mstrolo @.***> wrote:

is it possible to add username and password to the commands?
—
Reply to this email directly, view it on GitHub
https://github.com/jselbie/stunserver/issues/49, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHNSF5R7YPPHNPWV5XXKMTWPASVBANCNFSM6AAAAAATI5XJMU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


⚠ Processors has degraded performance

In 09f07bd, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 36 ms

Resolved: Processors performance has improved in b21d112.


Content misplacement on iPhone X when PanModalHeight is set to intrinsic

Describe the bug
If I set the bottom content constraint to safe area (rather than superview, in that case everything works fine) and return PanModalHeight(type: .intrinsic, height: 0) for longFormHeight, I am able to drag the content up higher, leaving the gap at the bottom (which seems to be equal to safeAreaInsets.bottom).
To Reproduce

Patch ShoppingCartViewController with these changes:

        bgView.snp.makeConstraints { (maker) in
//            maker.edges.equalTo(view)
            maker.leading.equalToSuperview()
            maker.trailing.equalToSuperview()
            maker.top.equalToSuperview()
            if #available(iOS 11.0, *) {
                maker.bottom.equalTo(self.view.safeAreaLayoutGuide)
            } else {
                maker.bottom.equalToSuperview()
            }
        }

...

extension ShoppingCartViewController {

    override func longFormHeight() -> PanModalHeight {
        return PanModalHeight(type: .intrinsic, height: 0)
    }

Present the viewcontroller
Drag it higher

Expected behavior
Just like in .content mode, I expect the viewcontroller will not move higher that it's intrinsic content size.
Screenshots

Screencast

hi, Sega-Zero
What version do you use HWPanModal
The latest version is 0.6.5
I use your code, and testing, it seems correct.

Caught that on 0.6.4. Made a retest on 0.6.5 and reproduce the very same behaviour.
The bug is a bit tricky, somehow I catch this only on first present of ShoppingCartViewController and only if I drag it up first, without dragging down

ScreenShot

@Sega-Zero
yeah, I have reproduced this bug. just happened first time.
Strange.
Give me some time to fix it.
Current solution:
Do NOT use intrinsic， you should calculate height by yourself.
Thank you.

@Sega-Zero
hey guys, I'm busy on my work, sorry for there is no time to fix it.
I will fix it as soon as I have time to do.


<Lifetime as ToTokens> is missing the apostrophe

If you use quote! on a syn::Generics, it gives <a, T>. It should be <'a, T>.

This syn::Generics includes a syn::Lifetime that you created yourself right? It looks like the Lifetimes parsed by syn always include the apostrophe.
I want the apostrophe to be included in the Lifetime name because this is how libsyntax does it. Here is what I did to check:
Create main.rs:
fn f<'a>() {}

Run rustc -Z ast-json main.rs. The following appears in the output:
"lifetimes": [
  {
    "attrs": {
      "_field0": null
    },
    "lifetime": {
      "id": 5,
      "span": {
        "lo": 5,
        "hi": 7
      },
      "name": "'a"
    },
    "bounds": []
  }
],

The lifetime is "'a" not just "a".
Can you think of a better way to handle this? Maybe it should panic if there is a lifetime that does not start with apostrophe? Or maybe add the apostrophe if one is missing? Or maybe just document that user is responsible for the apostrophe?

Ah! I never noticed the lifetime name included the '. syn behavior seems legit then.
Some documentation would be enough I guess. Maybe a panic in syn::Lifetime::new if there is no '.
Thanks!


Chore: remove unused options from the configs

What it solves
Remove unused configuration options: safe apps URL, safe relay service URL
How this PR fixes it
Literally removes them and also fixes tests
How to test it
Everything should work just fine as it used to

I will merge this because this is a chore task and it removes only unused code and this is very nicely covered by typescript


Question: Json File Parameters

I would first like to that you for making this application,  it truly has saved me MANY hours of work and frustration.
I was wondering if you by changes know (or know how to determine) the "names" / "reasons" for the data exported in the Json files.  For example the characters.json  (below is an example of what I Think might be correct, but am still missing a lot of the fields).  Have you ever gone through and looked into these files?
Side note: If this is a Data Mining (Manual) process that you have not looked into as of yet - I would be happy to document and share my findings, if where is somewhere in the Repo/Wiki you would like to aggregate this information.
"1": [
    // character string (matches assets folder)
    "alk",
    // ???
    "1",
    // Stars
    "4",
    // ???
    "0",
    // Race
    "Human",
    // ???
    "",
    // ???
    "0",
    // Gender
    "Male",
    // character string (matches assets folder) Duplicate ???
    "alk",
    // ???
    "1",
    // Leader (Ability)
    "Path to the Stars",
    // Ability 1
    "11",
    // Ability 2
    "12",
    // Ability 3
    "13",
    // Ability 4
    "14",
    // Ability 5
    "15",
    // Ability 6
    "16",
    // Ability (MB2 ???)
    "2",
    // Role
    "Balance",
    // Character ID
    "1",
    // ???
    "(None)",
    // ???
    "1",
    // ???
    "false",
    // ???
    "true",
    // ???
    "1",
    // ???
    "1",
    // ???
    "",
    // ???
    ""
],

To ascertain the meaning of each data in master table (aka orderedmap) files, you need to first look into the SWF scripts that import and actually uses them. So you need to pull the SWF and fully decompile it and extract all the needed .as files from decompilation. (FYI: You can do this with my tool, but doing this for JP client with my tool currently requires rooted device although its feasible without root permission. If you don't have rooted device and still want to decompile SWF out of JP client, you'll first need to unpack apk and find the SWF file packed inside APK and then you can manually decompile it with FFDEC)
anyway, back to the point, we need to locate the specific .as file that references your designated master table file. there's not many ways to do this since all the .as files are obfuscated and files names are in really messy format.
So here's the way I do it when needed.
Let's say we want to look for /box_gacha/box_reward.orderedmap file. I first look into the scripts/boot_ffc6.as file which contains all the master table imports and class matching. Then I search the master table file name within boot_ffc6.as so I can get the obfuscated name of the class that imports the initiator class. You'll get the result like below.

you can see the field clazz: §㆒§.§―§.§⁃§.§⼀§.ᅳ.§⁃§.§–§.§‑§.§⼀§.§‑§.§⎯§.§‒§.§⼀§.ｰ.ᅳ.int. This is the specific class where they import your orderedmap file and throw them into initiator classes, and these initiator classes are precisely the implementation we want to look at.
Now lets locate to the .as file for above class we found. For this example you want to search package §㆒§.§―§.§⁃§.§⼀§.ᅳ.§⁃§.§–§.§‑§.§⼀§.§‑§.§⎯§.§‒§.§⼀§.ｰ.ᅳ in your own IDE so we can find the file has this string.

As a result, you'll find the .as file that looks like above image. Focus on the get_data() method of the class. We want to copy the last-appearing class name in the return statement, which is name of initiator class where data refinement will happen inside.

Like we did before now we want to search for it again package ー.§−§.§‑§.§–§.§‒§.§㆒§.§−§.§—§.§‑§.§⁃§.§‐§.§–§.§−§.§—§.§‐§
and you'll get the file again, with the int() method somewhat look like below image.

Voila! int() method will be longer or shorter than the example depending on the cases. what we want to see is how they're parsing the data inside this method. you can see some variable names are in actual human readable format. So we can safely assume they're mapped like this

Sometimes initiator classes use more complex way of parsing and you may need to locate into several more referenced scripts but basics are same.
Also, contribution for documentation is always welcome! you can make PR with improved documentation if you want to contribute :) I recommend you to split such documentation and put it on the separated .md file in repository root and link them into README.md so we can locate to them more easily.

  I have time set aside this weekend for this, and will respond here if I have any follow-up questions.
Once I have more understanding and results on my side, I will work on getting this into a .md file and create a PR. :+1:

Apparently I don't have self control... And I took a look into character.character.  Thank you for the guide above 🎉
With character/character done... Now it is time to track down all these references.  Feel free to close this issue.


About separation  MSB and LSB and quantity input？

After reading your paper, I have some doubts and hope to get your help.
First of all, what is the role of delta in the quantization process of the quantSign function? (Because your comment is to quantize the input into arbitrary bits, the result obtained by using delta here is still a float).
Then use FastSign in PGBinaryConv2d to binarize the input (based on quantSign).
At this time, have the high bits of the binarized input been separated? Why is the result of out_msb multiplied by two thirds.
Moreover, in the process of calculating FracBNN, I did not understand the separation process of MSB and LSB. Whether it is the shift operation described in the paper or the sparse operation of LSB, there is no clear expression in the code. The end result seems to be just choosing between out_msb and out_full。

Thank you very much for your answer, I have some new questions.
In the hardware design, the input {-1, -1/3, 1/3, 1} is represented as {00, 01, 10, 11}, so should the result calculated by the hardware be multiplied by 1/3?

Yes correct. See here:
https://github.com/cornell-zhang/FracBNN/blob/main/xcel-cifar10/source/pgconv.h#L71
msb_scale = 1/3 << 1, so essentially the msb_conv is multiplied by 1/3.

Thanks for your prompt response, how did you solve the problem of slow training caused by temperature encoding. Is it a configuration problem? My gpu is 3070ti

How slow did you observe with/without thermometer encoding?
The released InputEncoder is fairly efficient in terms of GPU latency in our experiment. I can imagine the two sources of slow training: (1) expanded input channel due to encoding; (2) other quantization functions, but not related to thermometer encoding. For both we currently do not have a good solution.

Thanks for answering my doubts. Because I used it on 3DCNN, the delay is more obvious. I train almost 70 times slower after adding temperature encoding. Perhaps I can increase the training speed by increasing the size of the R parameter to reduce the number of expanded channels.
Thank you very much for your recent help


Can't decompress .xz file with lzma2_decompress

Hi,
I tried to decompress .xz file using lzma2_decompress function, but I didn't manage to do it.
I read .xz file as a  byte array and passed it to function as parameter. The function returned "undefined" and it wasn't possible to see decompressed file data. Is it possible to decompress .xz file using this library?

Sorry for the delay, my GitHub notification settings must be wrong...
And no, the client only requested decompressing raw LZMA2 files, and it was complete evil to work with raw bytes in JS, so I'm not willing to port it myself; this was only to satisfy a client. In addition, there's a wrapper I gave to the client which downloads the JS hosted on GitHub via XmlHttpRequest; so it's both open-source for others to not do the work I've done for LZMA2, and because my apps actually rely on a public URL - altho they try local site first.
If you're trying to reverse-engineer, in my scenario I ended up opening a Java LZMA2 program in Visual Studio Code (since the original LZMA-JS was based on a Java program), and stepping through the JS and Java together, comparing each line; I also uploaded an expected-result byte array and checked for match each time a byte was written.
If it helps, it is possible to pass LZMA2 chunks inside XZ using this app; LZMA2 has a single-byte property header which is all you should need. The chunks themselves are either uncompressed LZMA2 with size, or compressed, in which case both compressed and uncompressed sizes are in the chunk's header as 16-bit integers, if memory serves.
Hope that helps you with your quest, sorry I'm not in a position to do this for ya. I'll accept any pull requests if you do defeat the dragon though.

First, thank you so much @SortaCore for this effort! I was ripping my hair off until I realized that the data inside an xz archive is in fact LZMA2, which is not supported by the base LZMA-JS. So this one is a lifesaver.
@wojtaszg — xz is a container format and as such has a few headers around the actual LZMA{,2} data. I have created this gist demonstrating a (very crude) way to extract a block of data and decompress it with lzma2_decompress. It only supports archives that have exactly one block; files intended for streaming may have multiple. It should not be hard to adapt the code to support any number of blocks.


[20.09] slack: 4.8.0 -> 4.10.3

release-20.09

closed in favor of https://github.com/NixOS/nixpkgs/pull/103789


Typo in steps 2 and 3 - PowerShell Commands

Typo in step 1.3 PowerShell Command (no S at the end of the command)
Original: Get-AzureADApplications
Fixed:  Get-AzureADApplication
Typo in step 2.1 - PowerShell Command (no 'App')
Original: Get-AzureADApplication -AppObjectId
Fixed:  Get-AzureADApplication -ObjectId

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

thanks!

@wheatiesthecereal : Thanks for your contribution to the Azure documentation! The author, @kgremban, has been notified to review your proposed change.

Thanks for the edits @wheatiesthecereal
#sign-off


feat(rpc): adding daser stub for friendly API error

Closes #1299

Codecov Report

Merging #1308 (59809a5) into main (f5f5adb) will increase coverage by 0.24%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1308      +/-   ##
==========================================
+ Coverage   54.83%   55.07%   +0.24%
==========================================
  Files         176      176
  Lines       10552    10545       -7
==========================================
+ Hits         5786     5808      +22
+ Misses       4194     4159      -35
- Partials      572      578       +6

Impacted Files
Coverage Δ

nodebuilder/das/mocks/api.go
26.66% <ø> (ø)

nodebuilder/fraud/mocks/api.go
12.50% <ø> (ø)

nodebuilder/header/mocks/api.go
12.50% <ø> (ø)

nodebuilder/node.go
60.31% <ø> (ø)

nodebuilder/share/mocks/api.go
8.16% <ø> (ø)

nodebuilder/state/mocks/api.go
11.45% <ø> (ø)

nodebuilder/das/daser.go
100.00% <100.00%> (ø)

nodebuilder/das/module.go
95.12% <100.00%> (+0.52%)
:arrow_up:

nodebuilder/rpc/module.go
91.66% <100.00%> (-2.93%)
:arrow_down:

nodebuilder/rpc/rpc.go
100.00% <100.00%> (ø)

... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Make Release from Testing Branch on Stop Build Day

This should be done for all tested code following stop build day so we have a checkpoint we can revert to if needed. Ashwin knows how to this and Blake/Derrell should too if I'm right, it's pretty straightforward.

is this done? @ashwinc12


Measure the time for reporting TF final status and reformulate related SLO

Related to #1803: we want to reformulate the Testing Farm related SLO in a following way: „Test runs have set success/failure in X (TBD) minutes, after the test has finished“ so that it is consistent with the Copr build related SLO.

when reporting the final state of the testing farm run, measure the time from the test finished time
change the SLO (docs, related graphs, alerts)

Still relevant.


Sort the languages alphabetically at the footer

The available languages are sorted haphazardly, seemingly in the order they were completed by translators, which is not helpful for users.
A smarter sorting should be implemented, that takes into account the multiplicity of scripts. At Wikimedia Commons, we sort like this, for example.

Currently they are indeed sorted by when they were completed, as a rough proxy for popularity.
I think I'd be OK with sorting these differently if it's a) not too burdensome to implement and b) will be able to automatically handle new additions.


BasicStatistic into template

Statistic will collect data from entity header. E.g. (Crarrier failure,error counter, recipe counter)....

/cib


AssertionError: Gather function not implemented for CPU tensors

Hello! Thank you very much for such a new and excellent work! I have a question to ask you. when I run the training file train.py, there will be an error:
"AssertionError: Gather function not implemented for CPU tensors".
However, I don't know what went wrong. I followed the steps in your README step by step. I am using a dual-card machine. the GPUs are NVIDIA RTX 4090, and the memory of a single card is 24GB, hope to get your reply!

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Thank you for your reply! How does this work? Can you provide further explanation? Thank you very much!
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2024年8月27日(星期二) 上午10:33
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [ahayler/s4c] AssertionError: Gather function not implemented for CPU tensors (Issue #6)
to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Hey,
Thank you for your interest in our work.
It is a little hard to debug your error with the limited information you provided.
Could you check whether the GPUs are really available in your conda/micomamba environment? (https://stackoverflow.com/questions/48152674/how-do-i-check-if-pytorch-is-using-the-gpu).
All the best,
Adrian

Given the error it seems like some tensor should be mapped to the GPU, but isn't. I would try e.g. setting a breakpoint here:
File "/data/GPT/s4c-main/utils/base_trainer.py", line 294, in train_step
data = model(data)
and checking which tensors are not on the GPU (and try mapping them to the GPU).
I could also imagine that this is caused by your dual GPU setup. You could try using only one GPU setting:
import os
os.environ["CUDA_VISIBLE_DEVICES"] = "0"
At the top of your training script.

Thank you very much for your reply!
I just tried to add the setting of using a single GPU in the header of the train.py file as you suggested, but I still get the same error.
I will further try another way to set up breakpoint as you suggested!
Thank you sincerely!

Hello!
I did it again step by step as instructed in README, and still reported this error. Then I used the only one GPU setting according to your suggestion:
import os
os.environ[“CUDA_VISIBLE_DEVICES”]=“0”
The code can run normally, but only a single card is used for training, which affects the training speed. I hope to be able to train with multiple GPUs, so I set it to a dual GPUs setting:
import os
os.environ[“CUDA_VISIBLE_DEVICES”]=“0,1”
As a result, I got the same error report as before!
I noticed that I had no problem with the multi GPUs training using the code from BTS .
Can you help me to solve the error reported by the multi-card operation? Thank you very much !!!
Hope to get your reply!
best wishes!

Hello!
I had the same problem as you (one of) and I solved it by adding following lines in the end of BTSWrapper forward() function in bts/trainer.py:
cpu_keys = [k for k, v in data.items() if isinstance(v, torch.Tensor) and v.device == torch.device("cpu")]
for k in cpu_keys:
data[k] = data[k].cuda()
return data
I had a few problems so this could be a fix for something else. Try it.

I decided just to reduce complexity of model to be able to run with only 1 gpu (similar setup as you , 2x 24 gb). At the start of training, it seems like both of my GPU's are doing work but shortly afterwards, all the work goes to one gpu.
Good luck with your work.

Thank you very much! Your help means a lot to me!
Good luck with your work!


GuestConfiguration SDK update to support ARC senario

Update GuestConfiguration SDK to match newest GuestConfiguration Service Api interface.
Swagger change approved here:
https://github.com/Azure/azure-rest-api-specs/pull/9595

Hello @gehuang-dut please
* Check in the metadata TXT file generated when you run `generate.ps1`[(details)](https://github.com/Azure/adx-documentation-pr/blob/master/engineering/adx_netsdk_process.md#sdk-generation-from-updated-spec)

* Update release notes in csproj

* Update versions in csproj and `assemblyinfo.cs` if you are going to release

* Write test cases to cover new operations

Thanks

Addressed. Thanks.
It will be great if the documentation bellow can be added to SDK repo front-page directly =)
https://github.com/Azure/adx-documentation-pr/blob/master/engineering/adx_netsdk_process.md#sdk-generation-from-updated-spec

It will be great if the documentation bellow can be added to SDK repo front-page directly =)
Good point. Thanks

This "license/cla" status is not passing. I suppose it's because the PR is raised by you but the commits are authored by another user @msft-gehuan (looks like it's your account), so please

Make sure @msft-gehuan joins Azure orgnization: https://aka.ms/AzureGithub
Or author the commits by your current account @gehuang-dut

This "license/cla" status is not passing. I suppose it's because the PR is raised by you but the commits are authored by another user @msft-gehuan (looks like it's your account), so please
* Make sure @msft-gehuan joins Azure orgnization: https://aka.ms/AzureGithub

* Or author the commits by your current account @gehuang-dut

kick off a new build and the license error is gone. But looks the auto check is extremely slow today. Hitting the run-out-of-time error twice. Looks the system is in unstable status.

Thank you.


Send link as submitter to confirmMethod. Fixes #811

Since a[data-turbo-confirm] dynamically generates a form, there is no way to access the original element.
This PR stashes the original element on the generated form and passes it as the submitter to mimic form submitters.
Delete

window.Turbo.setConfirmMethod((message, element, submitter) => {
  // submitter is Delete
})

Fixes #811

Thank you for making this change! Could we add a test to the suite to exercise this behavior and guard against future regressions?

@excid3 what about incorporating the submitted ID into the confirm message?
diff --git a/src/tests/functional/form_submission_tests.ts b/src/tests/functional/form_submission_tests.ts
index 56cd951..5d1184a 100644
--- a/src/tests/functional/form_submission_tests.ts
+++ b/src/tests/functional/form_submission_tests.ts
@@ -971,6 +971,24 @@ test("test link method form submission inside frame with confirmation cancelled"
   assert.notOk(await hasSelector(page, "#frame div.message"), "Not confirming form submission does not submit the form")
 })

+test("test link method form submission with custom confirmation", async ({ page }) => {
+  page.on("dialog", (dialog) => {
+    assert.equal(dialog.message(), "Submitter: #link-method-inside-frame-with-confirmation")
+    dialog.accept()
+  })
+
+  await page.evaluate(() =>
+    window.Turbo.setConfirmMethod((_message, _form, submitter) =>
+      Promise.resolve(submitter ? confirm(`Submitter: #${submitter.id}`) : false)
+    )
+  )
+
+  await page.click("#link-method-inside-frame-with-confirmation")
+  await nextBeat()
+
+  assert.equal(await page.textContent("#frame div.message"), "Link!")
+})
+
 test("test link method form submission outside frame", async ({ page }) => {
   await page.click("#link-method-outside-frame")
   await nextBody(page)

@excid3 with the issue of testing resolved, I'd like to take this opportunity to get up on my soapbox and vent a little bit.
With the re-introduction of the  based <form> submissions, Turbo is responsible for:

creating a <form> element
transforming each query parameter in the a[href] value into an <input type="hidden"> element, and inserting that element into the <form>
transforming the a[href] value into a String without and query parameters, and assigning that to the <form> element's [action] attribute
making sure the form declares [data-turbo="true"] so that it is handled by Turbo, just in case it's inserted into the document in an area that has otherwise has Turbo disabled
making sure the <form> is visually hidden with [hidden] so that it doesn't flash visually during submission
transforming a[data-turbo-method] into form[method]
ensuring the presence or absence of form[data-turbo-action] based on a[data-turbo-action]
ensuring the presence or absence of form[data-turbo-confirm] based on a[data-turbo-confirm]
ensuring the presence or absence of form[data-turbo-stream] based on a[data-turbo-stream] which normally only affects GET requests in case a[data-turbo-method="GET"], which is an entirely counter productive yet possible use case
ensuring the presence or absence of the Turbo-Frame header in the request based on the  elements targeted Turbo Frame
appending the <form>, submitting it, listening for its next turbo:submit-end, then removing it

Now, with the addition of this change, passing the  element to another UJS-inspired configuration as a SubmitEvent.submitter, despite the fact that they are usually only ever <button> or <input type="submit"> elements.
I've always struggled to understand how Turbo owning the entire (probably incomplete and ever-growing) burden of responsibility outlined above is preferable over client applications using <form>+<button> elements for requests that are not GET.
What is gained by Turbo providing this support? What is lost by requiring/forcing client applications to use forms?
@excid3 I know you are not the inventor of this feature, so I want to be clear that this criticism isn't being leveled at you!

@excid3 what about incorporating the submitted ID into the confirm message?

Oh smart!

I also agree with this. Using links as forms that perform an action other than get should not be done for accessibility reasons, unless the link specifies the button role, which many people realistically ignore. In fact, this is mentioned in the Turbo handbook. Other needs include performing actions using buttons that are inside a form, in which case, instead of using a link, you could use a button with the formaction and formmethod attributes instead.

@excid3 Thank you for your work on this! May I ask why this hasn't been merged yet? Is there any other workaround (except using buttons, which breaks ex. button group bootstrap styling)?