Standardize code format, remove unused resources, and enhance HTML, C…

Description
This pull request fixes inconsistent codebases., including formatting improvements, removal of unused resources, refactoring of duplicate code, enhances maintainability & readability. Additionally, HTML enhancements ensure proper rendering, and outdated JavaScript variable declarations have been updated.

There have some conflict issue check it once.


Downloading a notebook has tables broken locally

URL to the relevant documentation
https://docs.quantum.ibm.com/guides/qedma-qesem
Please describe the UI problem.
When I view the notebook on the docs page, it looks fine:

But when I download and view in VSCode, it's not rendering properly:

I suspect this is because it's escaping the | character?

Yes it does look like it's escaping the pipe, here's some source from a downloaded notebook:
"source":["## Function parameters\n\n\\| Name |  Type | Description | Required ..."

Should be fixed now. The script we use to process notebooks before download was inserting extra newlines everywhere which was mostly harmless but did break the markdown tables.


[Serving Error] latest torchserve does not support torch<1.8.1

Describe the bug
Because torchserve uses torch.profiler since version 0.2.0, the current docker/serve/Dockerfile throws an error.
See https://github.com/pytorch/serve/issues/1548
Bug fix
There are three options below

Upgrade the version of torch>=1.8.1
Limit the version of torchserve==0.2.0
Wait for the latest torchserve to fix the issue.

I choose the second one for the PR in mmrotate https://github.com/open-mmlab/mmrotate/pull/160.

Should be fixed if torchserve is upgraded.


Flutter action not allowed in flutter/samples repository

Flutter action is failing to execute:
https://github.com/flutter/samples/actions/runs/1083944058

The failure message is:

subosito/flutter-action@v1, subosito/flutter-action@v1, and subosito/flutter-action@v1 are not allowed to be used in flutter/samples. Actions in this workflow must be: within a repository owned by flutter, created by GitHub, verified in the GitHub Marketplace or match the following: lee-dohm/no-response@9bb0a4b5e6a45046f00353d5de7d90fb8bd773bb, dessant/lock-threads@486f7380c15596f92b724e4260e4981c68d6bde6.

@RedBrogdon we need two small changes to fix the problem. We need to update the workflows to use a specific commit instead of the v1 tag to follow security recommendations and once we know the specific commit we want to use we'll added it to the allowed list.
Can you please help us with the PR to pin the action to a specific commit? I'll follow up with the other part.

Sure thing. Just created this PR: https://github.com/flutter/samples/pull/861/files.
There are two actions we're using that do not come from verified publishers:

subosito/flutter-action@4389e6cbc6cb8a4b18c628ff96ff90be0e926aa8
peaceiris/actions-gh-pages@068dc23d9710f1ba62e86896f84735d869951305

I did not pin the actions from GitHub (checkout, setup-java), but I can do so if needed.

Added those two deps to the allowed list. Github internal actions and trusted actions do not need to be added explicitly to the allowed list.

Tests are once again running smooth.


cnp-nist-sc-5-egress-deny.yaml

@yasin-cs-ko-ak since we cannot block class A private IP without a valid reason I'm closing this PR. Please re-raise it if you can find a use case that can be prevented using this policy.
cc: @salman-accuknox @raviknox


Plain imported css files and style tags are loaded twice

Describe the bug
Plain css files that are imported in __layout.svelte get duplicated in the resulting page.
In the demo app __layout.svelte contains import '../app.css', but the resulting index html page on the client has two corresponding <link rel="stylesheet"  href="…" /> tags.
First comes from the SSR or pre-render, second gets added by the router when client-side javascript takes over.
Disabling javascript or using export const router = false on the index page gets rid of the second stylesheet.
Moving the import from __layout.svelte to the index.svelte seems to still add 2 stylesheets, but declarations no longer appear twice inside the inspector.
Upd: Moving the styles from app.css to the <style /> tag inside the __layout.svelte does not seem to help. So it seems like it's a general css loading issue.
It seems that browsers are smart enough to not re-download the stylesheet, so this bug is only a minor annoyance when looking at the dev tools.
Reproduction

npm init svelte@next
Choose demo app; no; no; no
npm istall && npm run build && npm run preview
Visit the index page and inspect any element
:root declaration shows 2 times inside the Chrome's style inspector window
There are 2 stylesheets:

<link rel="stylesheet" href="/./_app/assets/pages/__layout.svelte-hash.css">
<link rel="stylesheet" href="/_app/assets/pages/__layout.svelte-hash.css">

Logs
Shouldn't be necessary.
System Info
System:
    OS: macOS 11.5.1
    CPU: (12) x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
    Memory: 14.40 GB / 32.00 GB
    Shell: 5.8 - /bin/zsh
Binaries:
    Node: 16.6.2 - /usr/local/bin/node
    Yarn: 1.22.11 - /usr/local/bin/yarn
    npm: 7.20.3 - /usr/local/bin/npm
Browsers:
    Chrome: 92.0.4515.131
    Firefox: 90.0.2
    Safari: 14.1.2
npmPackages:
    @sveltejs/kit: next => 1.0.0-next.147
    svelte: ^3.34.0 => 3.42.1

Severity
annoyance
Additional Information
Issues #1214 and #2130 might be related, but if I understood them correctly they only happen in dev, while I noticed the bug in prod when using adapter-static

I cannot reproduce this with 1.0.0-next.148. I'm guessing that Rich's recent improvement to asset handling fixed this

Yes, in 1.0.0-next.148 the first stylesheet no longer has a path starting with /./ and css gets deduplicated correctly

I'm actually getting the exact same issue with recent SvelteKit v1.0.0-next.236. Is the bug back for you as wel guys?

I can't seem to reproduce this exact issue on the demo app, but something might have been affected after the changes to base path handling (#3345, #3346)


runAfterTransition calls setupGlobalEvents before document is ready

🐛 Bug Report
I'm using the useButton hook in a project and experienced an error when the bundled javascript is included in the head of the page.

Uncaught TypeError: can't access property "addEventListener", document.body is null

🤔 Expected Behavior
Event setup side-effects happen when the component / hook is used.
😯 Current Behavior
Including useButton in the bundle leads to setupGlobalEvents being called, whether or not the component depending on useButton is active on the page.
Uncaught TypeError: can't access property "addEventListener", document.body is null
    $b3e8d5c5f32fa26afa6df1b81f09b6b8$var$setupGlobalEvents module.js:634
    node_modules 13-19d749bd3766408a4b8d.chunk.js:123131
    Webpack 28
module.js:634
    $b3e8d5c5f32fa26afa6df1b81f09b6b8$var$setupGlobalEvents module.js:634
    js module.js:638
    Webpack 29

💁 Possible Solution
Either call setupGlobalEvents when runAfterTransition is invoked, or update setupGlobalEvents to confirm the document is interactive (document.readyState === 'interactive') or wait until DOMContentLoaded` is fired.
if (document.readyState !== 'loading') {
  setupGlobalEvents();
} else {
  document.addEventListener('DOMContentLoaded', setupGlobalEvents);
}

🔦 Context
I'm trying to building an accessible component to act like a button for an internal design system for my company. @react-aria/button looked like a great solution for including all the attributes and behaviors to accomplish this task.
See "Possible Solution" section.
My current fix for the app is to defer the script for the bundle, but I still see this as a potential footgun for folks getting started with this project.
💻 Code Sample
https://gist.github.com/HipsterBrown/6480f7cc89afd083fb4aa827d6a89699
🌍 Your Environment
Rails 5.1 + Webpacker 4.0.2 + React 16.9.0

Software
Version(s)

react-spectrum
3.2.1

Browser
Firefox Developer Edition

Operating System
Macos 10.15.6

Good point! I like your proposed solution. Would you like to submit a PR?

@devongovett PR opened! https://github.com/adobe/react-spectrum/pull/1052


AZURE_SDK_GO_OMIT_READONLY not work after remove json tag

Introduced in this PR: https://github.com/Azure/autorest.go/pull/941. A simple workaround is we still generate it when the property is read only.

For some history, we used to omit sending R/O fields by default.  This behavior was later removed on 2022-10-07, and we added AZURE_SDK_GO_OMIT_READONLY as an escape hatch in case some services were misbehaving.  Since we haven't seen any bug reports, I'm inclined to say we leave this as is and remove the R/O fields omission code from azcore.  @JeffreyRichter thoughts on this?

The dead code has been removed from azcore.


Negative Monte Carlo pose loss during training

Dear authors,
Thanks for releasing your excellent work.
I'm training EPro-PnP-6DoF following your instructions provided. I noticed that loss_mc sometimes becomes negative, resulting in the overall loss being negative as well. I believe this is caused by the logarithm in the computation for L_pred, as the log of the number between 0 and 1 is negative. Is this behavior expected? Could you please elaborate a little on the logic behind it?
Much appreciated your reply in advance!

Hi! This is the expected behavior. Although the loss is derived from KL divergence in the paper, it is equivalent to cross-entropy up to an additive constant. Since the (usually positive) constant is not included in the simplified loss, the loss could well be negative.

Originally posted by @Lakonik in https://github.com/tjiiv-cprg/EPro-PnP/issues/36#issuecomment-1236793129

Thanks for the quick and informative response!


Permissions data for your user was not loaded during the pre-login stage

Hello, i am trying to make new server spigot 1.16.1. Installed essentialsx but i needed to add groups for perms, so i added luckperms. Unfortunately i cant join the server.

As the error suggests, check your console. If you can't figure it out on your own, please send your console log here, ideally via a pastebin site.

Are you using the latest version of LuckPerms from their CI server?

I am using the newest version 5.1.63. Is this enough? https://ctrlv.cz/WRpU

I am using the newest version 5.1.63. Is this enough? https://ctrlv.cz/WRpU

Don't use CraftBukkit, use Spigot instead.

looks like it solved my problem. Thank you very much. If i may ask my last question, do you know Worldedit that is already running on 1.16?

looks like it solved my problem. Thank you very much. If i may ask my last question, do you know Worldedit that is already running on 1.16?

It will take time for plugin authors to update to 1.16, as it only came out a couple days ago.
Please close the issue if your problem is solved. :)


Can we snap to (active and inactive) objects?

Often it is desirable to snap objects being drawn to existing objects in an inactive scrap.
The walls at the junction between two scraps is a common example.  This avoids having to manually invoke a scrap join elsewhere in ones Therion input data.
I might have missed something, but I cannot get the start or ends of lines, or new points to snap to control points (such as the ends of lines, line points or other points) on other objects.
The only snapping I can get at the moment is snapping to close the line that is currently being drawn, and snapping (tolerance) with the Inspect tool.  Am I having early morning brain fade perhaps? :)

No you're correct, there's no snapping anywhere, and I'm not planning on implementing it, it's a very large feature. The idiomatic way to join two scraps is with the join command, if you try to do it when drawing the TH2 file it could be disconnected with morphing (or other ways I don't know about).

My idea of best practice is to snap all conceptually coincident objects/points together, especially scrap/passage ends/joins, but also for any other point or line.  Avoids potential for hard to find issues that can arise when Therion tries to identify the interior of passages and areas.  Avoids gaps opening up due to morphing as well.
Explicit join statements are my option of last resort.  I use them only where implicit joining of snapped points is not possible, such as when the adjacent scraps are in different th2 files.
Therion's ability to morph snapped points together is rock solid (in my experience over 10 years of use), and is preferable to establishing sometimes complex join statements outside of the th2 file.
So I'd like to mark this as a feature request, to mimic the snapping functionality present in XTherion

Hi again!
I think I've implemented the minimum usable version of the functionality you were looking for, as now the Detail select tool can snap segments to other lines. This will only work if you're

using Detail select
have a single segment selected
dragging that segment without pressing shift
snapping to other lines (not to points)

I'm pretty sure this is enough for joining lines together, and in the future I may add this to the Select tool where it will work on points. But I'm still not sure how it would be possible to snap multiple segments or objects when moving them.


Xmc4xxx interrupt uart

Adds UART interrupt and FIFO support.

UART basic interrupt test is passing after commit
https://github.com/zephyrproject-rtos/zephyr/pull/48539

Please roll drivers: uart_xmc4xxx: Reindent after previous commit and drivers: uart_xmc4xxx: Improve code readability into previous commits. They can be considered part of the first commits introducing those changes.

Please roll drivers: uart_xmc4xxx: Reindent after previous commit and drivers: uart_xmc4xxx: Improve code readability into previous commits. They can be considered part of the first commits introducing those changes.

Thanks, I've squashed the commits.

@dcpleung I've rebased to origin/main in the latest push.

@dcpleung, without the fifo we can only call XMC_UART_CH_Transmit() once. After the first call, it will block until the byte before is shifted out, so I don't think the loop is needed.
The behavior with fifo_tx_size == 0 is the same as before the fifo commit.

@dcpleung, sorry... I needed to fix the indentation in the Kconfig.xmc4xxx file... could you re-ack, please?

@galak ping pls

@galak, could you remove the dnm label please?

ping


Clean up existing PRs

For PRs under ‘Not a problem:’ please check that the description of its state is correct and if so there is nothing to do.

For the other areas you need to let me know what is the path to get this PR checked in
Focus on getting these PRs cleaned up, get merge conflicts resolved, make sure they build in CI (obviously don’t worry about passing on devices) and then submit for final review.

Cleaning up the PRs is higher priority for right now than your other work for now.

Not a problem:

#957 - I believe this is still being worked on

Not Sure:

#902 - This is update wifiBasedNativeMock, I’m not sure why there is a PR since I thought the code is still being worked on, why do we have a PR?
#929 - What is going on with this one? It seems to have been stuck for a month.

We should be able to check these in once they pass CI build (not CI device since we can’t run device tests reliably anywhere at the moment):

#1093 - Approved and waiting for checkin by Artyom
3973 - If it passes build then it should be ready to submit into vNext_yarong_417_812, I already pointed the PR there
#889 - Please fix the merge conflicts with vNext_yarong_417_812 and once it passes build please submit to me for review
#851- This passes all tests and I gave it a LGTM a month ago! Please point the PR at vNext_yarong_417_812, fix merge issues and submit to me for review
876 - Please point this at vNext_yarong_417_812, resolve the remaining open issues, get it passing on build and submit to me for review

Please delete if not needed:

#1033 - Do we still need this?
#1014 - Do we still need this?

@yaronyg
#957 - still in a working state.