checkstyle fixes

Some fixes for #84

Coverage increased (+0.4%) to 78.673% when pulling 9d8433846b79c9adf5e5f7e610f004d23d93a79f on 84 into 6cee9b1752c944dff06372d506fba719990fc303 on master.

@rultor merge

@rultor merge

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@amihaiemil Oops, I failed. You can see the full log here (spent 4min)
[ERROR - 2017-01-10T15:30:26.702Z] Session [b5c463f0-d749-11e6-8e0a-83f587388ff8] - page.onError - msg: ReferenceError: Can't find variable: hljs
[ERROR - 2017-01-10T15:30:26.702Z] Session [b5c463f0-d749-11e6-8e0a-83f587388ff8] - page.onError - stack:
  (anonymous function) (http://eva.amihaiemil.com/index.html:16)
[INFO  - 2017-01-10T15:30:31.233Z] ShutdownReqHand - _handle - About to shutdown
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 72.408 sec - in com.amihaiemil.charles.GraphCrawlITCase
Running com.amihaiemil.charles.sitemap.SitemapXmlOnlineITCase
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.35 sec - in com.amihaiemil.charles.sitemap.SitemapXmlOnlineITCase
Running com.amihaiemil.charles.SitemapXmlCrawlITCase
Jan 10, 2017 3:30:33 PM org.openqa.selenium.phantomjs.PhantomJSDriverService <init>
INFO: executable: /usr/local/bin/phantomjs
Jan 10, 2017 3:30:33 PM org.openqa.selenium.phantomjs.PhantomJSDriverService <init>
INFO: port: 19012
Jan 10, 2017 3:30:33 PM org.openqa.selenium.phantomjs.PhantomJSDriverService <init>
INFO: arguments: [--webdriver=19012, --webdriver-logfile=/home/r/repo/phantomjsdriver.log]
Jan 10, 2017 3:30:33 PM org.openqa.selenium.phantomjs.PhantomJSDriverService <init>
INFO: environment: {}
PhantomJS is launching GhostDriver...
[INFO  - 2017-01-10T15:30:33.508Z] GhostDriver - Main - running on port 19012
[INFO  - 2017-01-10T15:30:33.702Z] Session [ba254d10-d749-11e6-9c3e-a3bb9fa766f7] - page.settings - {"XSSAuditingEnabled":false,"javascriptCanCloseWindows":true,"javascriptCanOpenWindows":true,"javascriptEnabled":true,"loadImages":true,"localToRemoteUrlAccessEnabled":false,"userAgent":"Mozilla/5.0 (Unknown; Linux x86_64) AppleWebKit/534.34 (KHTML, like Gecko) PhantomJS/1.9.8 Safari/534.34","webSecurityEnabled":true}
[INFO  - 2017-01-10T15:30:33.702Z] Session [ba254d10-d749-11e6-9c3e-a3bb9fa766f7] - page.customHeaders:  - {}
[INFO  - 2017-01-10T15:30:33.702Z] Session [ba254d10-d749-11e6-9c3e-a3bb9fa766f7] - Session.negotiatedCapabilities - {"browserName":"phantomjs","version":"1.9.8","driverName":"ghostdriver","driverVersion":"1.1.0","platform":"linux-unknown-64bit","javascriptEnabled":true,"takesScreenshot":true,"handlesAlerts":false,"databaseEnabled":false,"locationContextEnabled":false,"applicationCacheEnabled":false,"browserConnectionEnabled":false,"cssSelectorsEnabled":true,"webStorageEnabled":false,"rotatable":false,"acceptSslCerts":false,"nativeEvents":true,"proxy":{"proxyType":"direct"}}
[INFO  - 2017-01-10T15:30:33.702Z] SessionManagerReqHand - _postNewSessionCommand - New Session Created: ba254d10-d749-11e6-9c3e-a3bb9fa766f7
[ERROR - 2017-01-10T15:30:33.904Z] Session [ba254d10-d749-11e6-9c3e-a3bb9fa766f7] - page.onError - msg: ReferenceError: Can't find variable: hljs
[ERROR - 2017-01-10T15:30:33.904Z] Session [ba254d10-d749-11e6-9c3e-a3bb9fa766f7] - page.onError - stack:
  (anonymous function) (http://eva.amihaiemil.com/:16)
[INFO  - 2017-01-10T15:30:38.398Z] ShutdownReqHand - _handle - About to shutdown
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 5.824 sec - in com.amihaiemil.charles.SitemapXmlCrawlITCase

Results :

Tests in error:
  LiveWebPageITCase.retrievesLinksFromPage:71 » StaleElementReference {"errorMes...
  LiveWebPageITCase.retrievesLinksFromPageCname:56 » StaleElementReference {"err...

Tests run: 9, Failures: 0, Errors: 2, Skipped: 0

[WARNING] File encoding has not been set, using platform encoding UTF-8, i.e. build is platform dependent! The file encoding for reports output files should be provided by the POM property ${project.reporting.outputEncoding}.
[INFO]
[INFO] --- maven-failsafe-plugin:2.19.1:verify (default) @ charles ---
[WARNING] File encoding has not been set, using platform encoding UTF-8, i.e. build is platform dependent! The file encoding for reports output files should be provided by the POM property ${project.reporting.outputEncoding}.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 02:12 min
[INFO] Finished at: 2017-01-10T15:30:39+00:00
[INFO] Final Memory: 27M/391M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-failsafe-plugin:2.19.1:verify (default) on project charles: There are test failures.
[ERROR]
[ERROR] Please refer to /home/r/repo/target/failsafe-reports for the individual test results.
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
container ae3f621f61531816fe463a71d63b9e3e5ddf75130756a37493276327a0500034 is dead
Tue Jan 10 15:30:54 UTC 2017

@rultor try to merge again

@rultor try to merge again

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

@rultor try to merge again

@amihaiemil Done! FYI, the full log is here (took me 4min)


Personal Request List

I'm truly loving kanan's simplicity, and it's amazing how its left undetected for weeks while my friends who use abyss never stop complaining of the NGS detection as of late.
I hate being that guy, but kanan is missing a features I found very helpful, since you already reversed most of the useful ones, I'm just thinking, would it be simple to add the rest?
The ones I miss the most are:
Disable screen shake
Price value on items outside of trades
Commas in money string (Is that a feature in features.xml actually?)
Interface while dead
Minus HP (deadly) revealed
Clock ticking per minute
Cancellable Lance Counter
Name colors (Dark blue CP is hard to read)
And I remember before a certain update, true durability, item ID, dye codes used to function, are these still possible since the UI update?

Sure but most of the features I've been adding I've just been figuring out on my own so some of them might take a while unless another member helps.

I will keep that in mind for ya'll. I think the money string, clock
minutes, and name colors would be the trickiest.
I will keep ya'll posted if i come across anything in my daily activities^^
On Wed, Aug 17, 2016 at 9:55 PM, curse cat<EMAIL_ADDRESS>wrote:

Sure but most of the features I've been adding I've just been figuring out
on my own so some of them might take a while unless another member helps.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cursey/kanan/issues/40#issuecomment-240610567, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATu1XvyjhiruhbMyxVqyAcNZ1wlJsbJWks5qg8mcgaJpZM4JnDIc
.

Adding onto this. Is it possible to have the name colour for alt texts differ based on race? As abyss does it. I've found it useful. Or at least between elves/humans.

lol I guess while we're at it, you might also want:
infinite party time
self right click
talk to unequipped ego
targetable unagro'd mimics, sulfur golems, and strange books
view NPC equipment
party to housing board swap
default range swap
no minimum landing OR flying height while flying
no ending game ads
no combat mode timeout
preventing camera from zooming when talking to partners
allowing items to be hotkeyed
ignoring maintain frame rate limit in performance options so characters stay minimized
CPU reduction on minimizing mabi
It's all tricky territory to tread into...

Tricky, but gives something to do. I want to contribute to it as much a I can.
I spent 3 hours just trying to scan differences in Cheat Engine with Abyss' default range swap and all I ended up with is the skill icon value, sigh. I'll keep trying.

Finally done with the original requests :smile:


Add linting for docstrings

Now that we have an awesome setup due to @yadudoc that plugs code linting into Travis, I think it makes sense to add linting for our docstrings. It can be done via:
pep257 --ignore=D100,D101,D102,D103,D104

Or if you want more info:
pep257 --ignore=D100,D101,D102,D103,D104  --explain --source

The full list of errors can be found here. Resolving this issue should also include fixing the current things pep257 is complaining about. I can volunteer for this, since I'm currently going through the code to get familiar with it.

Installing this: pip install flake8-docstrings makes the existing flake8 CI call check docstrings for pep257 compliance. There are 2416 violation reports as of now.


Implemented --iidfile for podman commit

Added flag to Write the image ID to the file with podman commit command.
Fix to issue #5461
Signed-off-by: Sujil02<EMAIL_ADDRESS>

@sujil02 Looks like your tests are failing.
LGTM other then tests.

LGTM once tests are happy.  Looks like a real failure in the new test.

/lgtm
/hold

/hold cancel


WIP: enable pylint check E1111

@AlexeyKurakin I'm having some similar issues with two versions of validate_submissions in this PR too:
_target_class/attack_iter_target_class.py:106: [E1123(unexpected-keyword-arg), save_images] Unexpected keyword argument 'format' in function call examples/nips17_adversarial_competition/dev_toolkit/sample_targeted_attacks/iter_target_class/attack_iter_target_class.py:15: [C0411(wrong-import-order), ] third party import "import tensorflow as tf" should be placed before "from scipy.misc import imread" ************* Module validate_submission examples/nips17_adversarial_competition/dev_toolkit/validation_tool/validate_submission.py:49: [E1121(too-many-function-args), main] Too many positional arguments for method call


chore: migrate to owl bot

Attempt 4.
Obsoletes https://github.com/googleapis/nodejs-firestore/pull/1584

This looks like it needs another lint run, but it should otherwise be good.

Fixed!


Support Needed for VS CODE extensions

Though it's great and completes all of my needs it's better to have Extensions such as Prettier and Intellisense come to this.

Though it's great and completes all of my needs it's better to have Extensions such as Prettier and Intellisense come to this.

Yes, it needs some type of code formatter

Duplicate of my issue, #28

Though it resembles the issue #28 I am suggesting the Developer to Pre Install the Extensions like Prettier, Intellise and other popular Extensions

Though it resembles the issue #28 I am suggesting the Developer to Pre Install the Extensions like Prettier, Intellise and other popular Extensions

People my not want them, I certainly don’t like auto-formatters, and the user could install them themselves, if they had the option to, so it does resembles #28, not looks like it does.

I do plan on integrating prettier to the app, but you can actually install it by npm yourself. For intellisense, it's unclear what language you want.

For Intellisense I mean the vs code extensions which help to complete the code faster
https://code.visualstudio.com/docs/editor/intellisense
And also when I type code it does not show errors live.

For Intellisense I mean the vs code extensions which help to complete the code faster
It also completes the Code with Third Party Libraries
https://code.visualstudio.com/docs/editor/intellisense
And also when I type code it does not show errors live.

It still depends on what language you want...

Though it's great and completes all of my needs it's better to have Extensions such as Prettier and Intellisense come to this.

100%


[CI:DOCS] Packit: include ref to main branch builds

What type of PR is this?

/kind documentation
What this PR does / why we need it:
Links to rhcontainerbot/podman-next for main branch builds.
Which issue(s) this PR fixes:

None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

hmm, looks like this commit got used up in packit requesting builder perms on the copr because it didn't have them prior. I think the next one should trigger builds on commit. AFAICT, this should only be a 1-time hassle. Also, other repos should JUST WORK (TM) after this.


fix(start): echo command line

Only the command stdeout/stderr was being printed, which was difficult to decipher without the relevant command.
Before:
Logged in as test-81
=== Apps
test-238696780
test-435805123
test-755573488
Destroying test-238696780...
done in 0s

After:
$ deis login http://deis.<IP_ADDRESS>.xip.io:32762 --username=test-81 --password=asdf1234
Logged in as test-81
$ deis apps
=== Apps
test-238696780
test-435805123
test-755573488
$ deis destroy --app=test-238696780 --confirm=test-238696780
Destroying test-238696780...
done in 0s

Prints to GinkoWriter so everything will be silent unless a test case fails or the -ginkgo.v flag is given (which it is in make test-integration).

this is helpful :+1:


Add URL check in TRSS Grid view

Step1: set up TRSS environment locally https://github.com/adoptium/aqa-test-tools/blob/master/TestResultSummaryService/README.md
Step2: run tests https://github.com/adoptium/aqa-test-tools/blob/master/TestResultSummaryService/README.md#testing
Step3: add new test after https://github.com/adoptium/aqa-test-tools/blob/master/test-result-summary-client/cypress/integration/byPipeline.js#L14

assert URL contains /resultSummary

cy.url().should('include', '/resultSummary')

Hi @llxia  I've set up the local environment and would live to contribute to this. May I?

@olusanwo are you working on this issue? Do you have any questions?

@llxia. No, you may reassign

Thanks @olusanwo for letting us know.

Hello,I would like to work on this.

Sure, thanks @KESA24 !

Can i work in this ...

Tomorrow is the last date to submit project .

Can anyone help me to complete my project

Sorry Rajwin, I'm also currently working on the same project

No problem 🤗

@KESA24 are you still working on this? If not, I will unsign you from this issue.

Hello Lan,
Kindly reassign it.
thank you,
On Thu, 10 Feb 2022 at 20:01, Lan Xia @.***> wrote:

@KESA24 https://github.com/KESA24 are you still working on this? If
not, I will unsign you from this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/adoptium/aqa-test-tools/issues/570#issuecomment-1035169207,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALYZFFDSOQE3CNJRV5MP6VLU2PVPFANCNFSM5GHJYSMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Patricia

I can take this one!


cant connect my aasx file to the registry

I am using AASX v1 and the AAS server from Docker, as well as the registry server, but I can't connect  the AAS to the registry .

thank you for your help

Hi @yucef428 ,
Thanks for raising this issue.
Could you please attach the docker-compose file and all the configurations?


Multiple instances of Fragment detected warning

If I create a fresh sketch using any Three.js template and start it I get the following warning in the console:

This warning only happens when you must multiple tabs or windows pointing at the same URL opened at the same time. I would assume you opened the URL printed from the terminal while the previous one already opened reloaded in the background?

I thought the same but I've closed everything down, run the project via npx fragment and see the same warning.
Checking the network tab on the examples in the cloned fragment repo vs. the ones created in my project via fragment there is an additional request being made - I wonder if this has anything to do with it?
Fragment repo:

Mine:

Oh interesting 🤔 Might be related to that updated Vite version! I'll investigate

I was not able to reproduce the bug. I just release a new version v0.1.18 could you check if it still happens on your machine?


dNdScv analysis (1/N)

Purpose/implementation Section
What scientific question is your analysis addressing?
We will use the dNdScv R package to infer driver genes for the PBTA cohort. This analysis is part of the tumor clone inference analysis.
What was your approach?
What GitHub issue does your pull request address?
Closes https://github.com/d3b-center/pbta-tumor-evolution/issues/57
Directions for reviewers. Tell potential reviewers what kind of feedback you are soliciting.
Which areas should receive a particularly close look?
There is an issue when using the RefCDS_human_GRCh38_GencodeV39_liftover.rd a as the genome reference.
Is there anything that you want to discuss further?
Is the analysis in a mature enough form that the resulting figure(s) and/or table(s) are ready for review?
Results
What types of results are included (e.g., table, figure)?
What is your summary of the results?
Reproducibility Checklist

[ ] The dependencies required to run the code in this pull request have been added to the project Dockerfile.

Documentation Checklist

[ ] This analysis module has a README and it is up to date.
[ ] This analysis is recorded in the table in analyses/README.md and the entry is up to date.
[ ] The analytical code is documented and contains comments.

This is ready for the peer-review process. The genome reference has not been replaced with: example_output_refcds.rda and includes the most updated gene names.


TensorCoreTiledLayoutType -> TensorCoreTiledLayout

Context
What is the purpose of this PR? Is it to

[ ] add a new feature
[x] fix a bug
[ ] update tests and/or documentation
[ ] other (please add here)

TorchAO nightly from Oct 10 broke b/c they updated a variable name. See: https://github.com/pytorch/torchtune/actions/runs/11292390312/job/31408285709
Changelog
What are the changes made in this PR?

Update variable name

Test plan
Please make sure to do each of the following if applicable to your PR. If you're unsure about any one of these just ask and we will happily help. We also have a contributing page for some guidance on contributing.

[x] run pre-commit hooks and linters (make sure you've first installed via pre-commit install)
[ ] add unit tests for any new functionality
[ ] update docstrings for any new or updated methods or classes
[x] run unit tests via pytest tests
[x] run recipe tests via pytest tests -m integration_test
[ ] manually run any new or modified recipes with sufficient proof of correctness
[ ] include relevant commands and any other artifacts in this summary (pastes of loss curves, eval results, etc.)

UX
If your function changed a public API, please add a dummy example of what the user experience will look like when calling it.
Here is a docstring example
and a tutorial example

[x] I did not change any public API
[ ] I have added an example to docs or docstrings

cc @jainapurva


More speedups to section TOC rendering

follow-up to #1632
This PR

in toctree.py:

adds a bunch of comments
changes several variable names so it's clearer what they are
changes a few function names so it's clearer what they do (no deprecation needed, either they're private or are new enough that they're not in a released public API yet)
speeds up determination of whether we need to show the sidebar-nav-bs.html template on a given page. This is the main substantive change, and trims several minutes off the SciPy docs build time. We achieve this by exiting early if we found a relevant top-level ancestor page (without actually building out the toc-subtree rooted at that ancestor) and the site includes hidden TocTrees within its sidebar TocTree. The logic is that finding an ancestor and including hidden toctrees together guarantee that a non-empty toc subtree exists for the current page, and we can postpone the toctree.resolve() step until later when it's actually added to the page.
eliminates one intermediate function (get_unrendered_local_toctree) that was added in #1632 (no longer needed given the speedup in the prev. bullet). No deprecation needed, as it isn't in a released public API yet.

in layout.html:

updates the function called that determines whether to skip including sidebar-nav-bs

in __init__.py:

marks the theme as unsafe for parallel_write. This is based on extensive testing of the SciPy docs build, where the API pages were reliably getting the wrong left sidebar TOC when built in parallel, and getting the correct sidebar TOC when built with a single job. After marking the theme as parallel-write-unsafe, the SciPy docs now end up with the correct sidebar on all pages even when building in parallel (i.e. when pages are read in parallel but written serially). see https://github.com/scipy/scipy/pull/16660#issuecomment-1888055872

cc @tupui

team, I'm planning to self-merge this on Friday unless someone weighs in before then requesting more time to review. (or, feel free to review sooner than that and merge it yourself if you're happy).

thanks for reviewing @12rambau


Schedules should be accurate to local time

If I access the page for Polish Championship 2019, it appears to be a 4-day competition, starting late on Friday evening and finishing extremely early on Monday morning. This is displayed under "Schedule", and gives an inaccurate view of the actual schedule of the competition.
While I assume this is a feature designed to make it easier for the viewer to know when exactly events are taking place (relative to them), I think the progress bar that shows up under ongoing events is sufficient for that. Perhaps this could be an optional feature that can enable, to convert the schedule to their timezone, but I think the default should simply be to show the schedule in the local timezone of the competition, with the progress bar under ongoing events.

This is definitely intended and I have a strong opinion that's how it should work. You can always check out the schedule in competition's timezone on the WCA website (two clicks from WCA Live), whereas
a local schedule is the most reasonable and useful way of presenting the information to the user. Progress bar itself it's just a small part of the information, as you usually want to see at what time do some events start and doing timezone calculations manually is just ridiculous is a terrible user experience.

Hmm. Perhaps this is a labelling issue then? I agree that the information is useful for the viewer, but I still find it quite odd to look under "Schedule" and see events happening after midnight, or see the comp happening across more days than it's scheduled for.

Pretty sure it's just the matter of people getting used to it, as we've never really localized schedules before. Also when you think about it, such events are indeed happening after midnight for you and that's all that really matters ^^

Maybe yeah, we'll have to see how people feel about it after it's been live for a while I guess. Maybe it just feels unintuitive to me because I live in New Zealand, so I'm very used to thinking in terms of other time zones.

Maybe good idea to have a switcher? User lime zone / venue time zone? For example, I'll have competition in another time zone next week and it's a little bit hard for me (I have a problems with my mobile operator, it change time not always correct).


[release-1.12] Support distroless in piggyback test

This is an automated cherry-pick of #35872

@howardjohn Another cherry-pick that needs your CLA consent.


Disable SSH

Describe the solution you'd like:
It is possible to disable the SSH server in the cloud configuration so that the service does not run and it is not possible to log in via SSH. This means that updates and installations are only possible via the Elemental Operator.

Sure. You'd just run
systemctl disable sshd

on every boot


annotations workflow

The current annotations/ component has the following workflow:

download images to a local directory
run the annotation tool with reference to the local image directory
iterate through the images creating annotations

What we need is multi-user, multi image-set workflow. This ticket is for the design of that workflow, noting:

sometimes an image will have multiple headstamps in it. For example, if a collector uses a flatbed scanner to capture a collection of (potentially related) headstamps in a single scan.
sometimes a headstamp image will be captured in a situation that is outside the analysis context, e.g. field work.
The current tool is designed to annotate "the next non-annotated image" in a set, i.e. work through a set until it is annotated. The task of annotation is expensive in effort, the ergonomics are important.
When we have a multi-user system, each user may have a collection of image sets (a collection of collections of images), and each image may have multiple headstamps in it.
The same identical image may be cited in multiple sources - e.g. downloaded from multiple web sites. The same image may be in multiple collections.

Workflow may start with something like this (part 1: manage images):

authenticate (entity: User)
manage image collections (browse, create, edit, delete, search). Entity: ImageCollection
within an image collection, manage images (upload, delete/archive, manage image metadata). Entity: Image. Note:

m:n cardinality between images and collections
image metadata includes 1:n ImageCitation.url, ImageCitation.image_id <-- possibly identified by a content hash
do we want to associate taxonomic claims about all headstamps in all images in the collection (e.g. these images are of my collection of shell casings of this caliber, of era, or source, etc). i.e. so individual headstamps can inherit reified metadata from the collection to which they belong (this rule inferred that user made these assertions about this headstamp, because it's in that collection)...

the existing ML could identify the coordinates of headstamps within the image. Entity: HeadstampObservation. Note:

HeadstampObservation.image_id (same type as ImageCitation.image_id
HeadstampObservation.coordinates (where in the image is the headstamp. bounding box x/y? what about affine transformations/ellipses?)
which model-version observed this headstamp within the image... Is that something we want to keep track of? If not, how to we manage upgrades to headstamp detection models?

At the end of part 1, we will have a backlog of un-annotated HeadstampObservations, and a process for adding to this backlog. We will want some mechanism for deciding which headstamps to annotate next... (part 2: manage annotations):

create deterministically-ordered list of HeadstampObservations meeting some criteria
for each HeadstampObservation in the list:

view it (clipped to HeadstampObservation.coordinates)
manage annotations (view existing, add new, edit existing, delete)
navigate to the next HeadstampObservation in the list

Note:

the ability to create "lists of HeadstampObservations meeting some criteria" is a headstamp search. This might have structured features (e.g: from these collections, with/without annotations, of this caliber, manufacturer, age, country of origin, etc) or unstructured features (containing these characters, according to those types of annotation). This API service might be the basis of other UI features - collection browsing, etc.
In the future, we will have models making "annotations" on HeadstampObserations - e.g. this model-version thinks that glyph is an Arabic character, this person thinks that headstamp is actually the end of a pencil, etc. There will be multiple and various "annotation types", with structured data that's meaning is determined by the annotation type.

Latest pull request #110 caps off some work to enable collection sharing, and automated headstamp detection on upload.  There are a few outstanding UI annoyances but the e2e workflow is working.
I've probably broken things for the workflow where you don't have the prediction model handy- drop me a note and I'll share the model bits.

A good deal of this is addressed now so I'll go ahead and close.


Does SysDig use Prometheus? How do they integrate?

Describe the task
Does SysDig use Prometheus? How do they integrate?
Acceptance Criteria

[ ] what is required for this task to be complete?
what is the finishing point or end state of this task?
[ ] what is the output of this task?

SME/User Contact
(may want to use a persona to fill this in)
Additional context

any additional details that could not be captured above

Closed as duplicate.  See #37.


Can not find the input data

I can not find the imput data on the other project .....

+1

Sorry guys, but the owner of the data pull it off from the web.
It is really disappointing, but I hope the code will be somehow helpful anyway.

Will you be able to upload it?

Sadly I'm not the owner and I don't have the copy of the data. Maybe try to contact owner directly at his repo: https://github.com/liyongsea/challenge-data


V6. When using a condition on a relationship that uses STI, the ORM may choose the wrong table alias for the discriminator value

Describe the bug
On the version 6, when using a find with a condition on a relation that uses STI, the ORM will sometimes choose the wrong table alias for the discriminator value. It seems to depend on the type of relation and the owning side.
To Reproduce
Steps to reproduce the behavior:

create entities with relationships that use STI
use a find with a smart condition on the relation
on some occasions, the ORM will choose the wrong table alias for the discriminator value

https://github.com/YanDjin/mikro-orm/blob/bug/find-sti-relation/tests/issues/GH4422.test.ts
this link contains the test that replicates the issue.
Expected behavior
To choose the correct columns on the join and apply the discriminator condition on the right table.
Versions

Dependency
Version

node
18.16.0

typescript
5.0.4

mikro-orm
6.0.0-dev.82

your-driver
postgres/sqlite

maybe related to #4423

Fixed via 05e371b


Amend type hint of Collection::reduce

1. Why is this change necessary?
The type hint on the return value of Collection::reduce is just right on the only used occasion:
https://github.com/shopware/platform/blob/f9ff2d73cd975570128221b18fcaf4696d310614/src/Core/Framework/DataAbstractionLayer/Indexing/Indexer/TreeIndexer.php#L318
where it is used in a not really reducing way (following the definition of reduce as collection to scalar value).
2. What does this change do, exactly?
Removes the return value type hint. As php does not support generics (yet).
3. Describe each step to reproduce the issue or behaviour.

Use an entity collection and compute a scalar value without the API computation functionality.
Get a scalar is not array sort of exception

4. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

@JoshuaBehrens we merged your changes into the 6.1 branch.
It seems the auto detection didn't work here.
https://github.com/shopware/platform/commit/be708955a8383e65371b5b62a86595c6f1646f66

Auto detection? Detection of what in particular?

Auto closing of PRs seems to work only when commits get into master and not if they are in a release branch, probalby it looks for the target branch of the original PR.
Your PR targeted master, we've merged it into branch 6.1 and GitHub does not automatically closes your PR as you've obviously wanted your changes merged into master.

Ah now I get it. I didn't see the fixes #345 line at first. Makes sense. If a branch is not merged into the main branch it can't be fixed yet globally :D I guess you have to change the main branch to 6.1


document custom boot splash screen

Customizing splash
We configure all of our initial images to produce a resin logo at boot, shutdown or reboot. But we encourage any user to go and replace that logo with their own. All you have to do is replace the splash/resin-logo.png file that you will find in the first partition of our images (boot partition) with your own image. NOTE: As it currently stands plymouth expects the image to be named resin-logo.png.
 Front conversations

Need to probably add this to FAQs as well

Even if you do it that way, whenever you update your code the device loads the resin logo again isn't it?

@Seungja no, it should always load up whatever splash image you wrote over the resin-logo.png

I must be doing something wrong then, i replace the splash/resin-logo.png then boot a Intel NUC with that image and the custom splash loads fine, but just after Post Provisioning (when you must remove the USB drive and boot from disk) it loads default Resin Splash from there onwards. 😞

@Seungja ahh... I know what is going on. The issue is that this feature was broken on the flasher types (where resinOS is written to the internal storage, as in the case of the NUC) We have fixed that, but it has yet to be deployed on the intel NUC images. You can see the change log here: https://github.com/resin-os/meta-resin/blob/master/CHANGELOG.md#v19---2016-08-24
Hopefully in the next week or so we can get the latest version up on resin.io

@shaunmulligan Thanks for the info!, i'll have to wait for the version change then 😃

hey @Seungja , I believe that resinOS 2.0.4 is now available in production for the intel nuc. So in theory this feature should work for you. Let me know how the testing goes.
Cheers
Shaun

Hello @shaunmulligan, i just made some tests with the resinOS 2.0.4 version. and sadly, the behavior is the same :(
I even tried making a new application and starting anew but still just after post provisioning the splash resets to the default one.

Hey @Seungja I spoke to one of the device engineers (@Telphan) and he said it might be a regression, so he is going to look into what is going on. It might also be worth raising an issue on http://github.com/resin-os/resinos as it will have better visibility and tracking for the team there.

Is this still an issue? It looks like it may have already been resolved: https://github.com/balena-os/balena-os/issues/316

Looks like this is documented at https://github.com/balena-io/docs/blob/master/shared/overview/2.x.md#dev-vs-prod-images but perhaps this issue is suggesting it should be documented more prominently? (see also #1307 )

This is a very old issue so likely those docs above didn't exist then. I also added in a reference via https://www.balena.io/docs/reference/OS/advanced/#disabling-rainbow-splash-screen so I think I'll close this for now as it'll return at least a couple results searching for splash.


figure out a better approach to cli page aliases ahead of next cli release

ref: https://github.com/pulumi/docs/pull/9086
while we are in this area, we should make sure the changes i made will generate them in the right place; since the cli docs moved
maybe relevant https://github.com/pulumi/pulumi-hugo/blob/master/themes/default/layouts/shortcodes/pulumi-command.html

@sean1588 can you pick this up tomorrow?

verify change made here https://github.com/pulumi/pulumi/blob/master/pkg/cmd/pulumi/gen_markdown.go also fixes links in see also section
need to add aliases into markdown generation
test that generation, ahead of the next cli release
turn off auto-merge on next cli release; just to double confirm we didn't totally break cli docs
reach out to Justin with questions slash for review

aliases:
  - /docs/reference/cli/pulumi/


Insomnia Designer: how to I export my openAPI spec in a single standalone YAML file?

I run Designer 2020.5.2 on macOS Big Sur 11.0.1
I managed to write a working spec but now I'm willing to send it to colleagues who are not working with Designer.
So I basically need to send them the YAML file.
I connected Insomnia Designer to a git repo (a GitLab one); this worked, but now I've got 2 files inside the .insomnia folder:
ApiSpec/spc_<some_random_hexa_id>.yml
Workspace/wrk_<another_random_hexa_id>.yml
But the 1st one is not a "pure" openAPI spec because it starts with:
_id: spc_<the_random_hexa_id_above>
type: ApiSpec
parentId: wrk_<the_workspace_hexa_id_seen_above>
modified: <a_numeric_timestamp>
created:  <another_numeric_timestamp>
fileName: My API title + version
contents: >
 # and now goes the REAL content of my spec!
contentType: yaml

I can't send this "as is"?!!
I also tried the Preferences menu item > Data tab > "Export Data" button, and when checking "Current Document" I'm let choosing between the few API calls that were automatically generated in the main "Debug" tab (and if I uncheck ALL of them, I can't Export - button is greyed)
The only option I can resort on is a simple copy-paste, create a new file by hand and save it.
That's absurd?!

Hi @mxmlnglt! Fortunately you can do this using the companion CLI, inso. Here  are some docs for the export spec command.
You can install and run
npm i -g insomnia-inso
inso export spec <name> --output spec.yaml

Or you can use npx
npx insomnia-inso export spec <name> --output spec.yaml

Let me know how you get on, and thanks for using Designer! It is still relatively new, and I agree the export functionality built into the app needs to be expanded a little. 😄

Hi @mxmlnglt! Fortunately you can do this using the companion CLI, inso. Here  are some docs for the export spec command.
You can install and run
npm i -g insomnia-inso
inso export spec <name> --output spec.yaml

Or you can use npx
npx insomnia-inso export spec <name> --output spec.yaml

Let me know how you get on, and thanks for using Designer! It is still relatively new, and I agree the export functionality built into the app needs to be expanded a little. 😄

Ah! thanks for pointing me towards inso.
Just a last question: is there a way to convert the spec into a JSON format instead of the (default) YAML one?
They say both formats are acceptable (unfortunately, I'm asked for a JSON...) https://swagger.io/specification/#format

Ah! thanks for pointing me towards inso.
Just a last question: is there a way to convert the spec into a JSON format instead of the (default) YAML one?
They say both formats are acceptable (unfortunately, I'm asked for a JSON...) https://swagger.io/specification/#format

Awesome! 🎉

is there a way to export the document into a JSON format instead of the (default) YAML one?

Not right now, although this would be a pretty awesome (and simple) feature to add to inso. I can provide some pointers if you would be interested in adding it! Something like inso export spec <id> --format json --output spec.json.
For now you will need to resort to an external converter, like https://jsonformatter.org/yaml-to-json for example.

Awesome! 🎉

is there a way to export the document into a JSON format instead of the (default) YAML one?

Not right now, although this would be a pretty awesome (and simple) feature to add to inso. I can provide some pointers if you would be interested in adding it! Something like inso export spec <id> --format json --output spec.json.
For now you will need to resort to an external converter, like https://jsonformatter.org/yaml-to-json for example.

OK, will try to have a look into it if I can find some time.

OK, will try to have a look into it if I can find some time.

Not right now, although this would be a pretty awesome (and simple) feature to add to inso. I can provide some pointers if you would be interested in adding it! Something like inso export spec <id> --format json --output spec.json.

Our case is a little different so just thought I'd share our situation for the community:
We'd like to  inso export spec  to the native format stored in the Insomnia spec file without knowing which format was used.  We have distributed teams contributing Insomnia "workspace" artifacts back to a Centralizing Pipeline that publishes Specs that it doesn't otherwise know much about.


Invoke-Sqlcmd gets TypeInitializationException with 22.1.1

I upgraded from 21.1.18256 to 22.1.1, and now I get errors running Invoke-Sqlcmd. I have verified that the newest version has the same behavior.
PS C:\WINDOWS\system32> invoke-sqlcmd -query 'SELECT 1'
invoke-sqlcmd : The type initializer for 'Microsoft.Data.SqlClient.InOutOfProcHelper' threw an exception.
At line:1 char:1
+ invoke-sqlcmd -query 'SELECT 1'
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidResult: (:) [Invoke-Sqlcmd], TypeInitializationException
    + FullyQualifiedErrorId : ExecutionFailed,Microsoft.SqlServer.Management.PowerShell.GetScriptCommand

PS C:\WINDOWS\system32> $Error[0].Exception.ToString()
System.TypeInitializationException: The type initializer for 'Microsoft.Data.SqlClient.InOutOfProcHelper' threw an exception. ---> System.IO.FileNot
FoundException: Could not load file or assembly 'System.Runtime.InteropServices.RuntimeInformation, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken
=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified.
   at Microsoft.Data.SqlClient.InOutOfProcHelper..ctor()
   at Microsoft.Data.SqlClient.InOutOfProcHelper..cctor()
   --- End of inner exception stack trace ---
   at Microsoft.Data.SqlClient.SqlConnectionString..ctor(String connectionString)
   at Microsoft.Data.SqlClient.SqlConnectionFactory.CreateConnectionOptions(String connectionString, DbConnectionOptions previous)
   at Microsoft.Data.ProviderBase.DbConnectionFactory.GetConnectionPoolGroup(DbConnectionPoolKey key, DbConnectionPoolGroupOptions poolOptions, DbCo
nnectionOptions& userConnectionOptions)
   at Microsoft.Data.SqlClient.SqlConnection.ConnectionString_Set(DbConnectionPoolKey key)
   at Microsoft.Data.SqlClient.SqlConnection.set_ConnectionString(String value)
   at Microsoft.Data.SqlClient.SqlConnection..ctor(String connectionString, SqlCredential credential)
   at Microsoft.SqlServer.Management.PowerShell.ExecutionProcessor.CreateSqlConnection()
   at Microsoft.SqlServer.Management.PowerShell.ExecutionProcessor..ctor(GetScriptCommand sqlCmdCmdLet)
   at Microsoft.SqlServer.Management.PowerShell.GetScriptCommand.ProcessRecord()

Also interesting, I get error messages when Importing or Removing the module (even though Get-Module suggests that those commands are successful):
PS C:\WINDOWS\system32> import-module sqlserver
Import-Module : Could not load file or assembly 'netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' or one of its
dependencies. The system cannot find the file specified.
At C:\Program Files\WindowsPowerShell\Modules\sqlserver\22.1.1\SqlServer.psm1:61 char:25
+ ...       $binaryModule = Import-Module -Name $binaryModulePath -PassThru
+                           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Import-Module], FileNotFoundException
    + FullyQualifiedErrorId : System.IO.FileNotFoundException,Microsoft.PowerShell.Commands.ImportModuleCommand

PS C:\WINDOWS\system32> $Error[0].Exception.ToString()
System.IO.FileNotFoundException: Could not load file or assembly 'netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' or
 one of its dependencies. The system cannot find the file specified.
File name: 'netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'
   at System.Reflection.RuntimeAssembly.GetExportedTypes(RuntimeAssembly assembly, ObjectHandleOnStack retTypes)
   at System.Reflection.RuntimeAssembly.GetExportedTypes()
   at System.Management.Automation.Runspaces.PSSnapInHelpers.GetAssemblyTypes(Assembly assembly, String name)
   at System.Management.Automation.Runspaces.PSSnapInHelpers.AnalyzeModuleAssemblyWithReflection(Assembly assembly, String name, PSSnapInInfo psSnap
InInfo, PSModuleInfo moduleInfo, Boolean isModuleLoad, Dictionary`2& cmdlets, Dictionary`2& aliases, Dictionary`2& providers, String helpFile, Type&
 randomCmdletToCheckLinkDemand, Type& randomProviderToCheckLinkDemand)
   at System.Management.Automation.Runspaces.PSSnapInHelpers.AnalyzePSSnapInAssembly(Assembly assembly, String name, PSSnapInInfo psSnapInInfo, PSMo
duleInfo moduleInfo, Boolean isModuleLoad, Dictionary`2& cmdlets, Dictionary`2& aliases, Dictionary`2& providers, String& helpFile)
   at System.Management.Automation.Runspaces.InitialSessionState.ImportCmdletsFromAssembly(Assembly assembly, PSModuleInfo module)
   at Microsoft.PowerShell.Commands.ModuleCmdletBase.LoadBinaryModule(PSModuleInfo parentModule, Boolean trySnapInName, String moduleName, String fi
leName, Assembly assemblyToLoad, String moduleBase, SessionState ss, ImportModuleOptions options, ManifestProcessingFlags manifestProcessingFlags, S
tring prefix, Boolean loadTypes, Boolean loadFormats, Boolean& found, String shortModuleName, Boolean disableFormatUpdates)
   at Microsoft.PowerShell.Commands.ModuleCmdletBase.LoadModule(PSModuleInfo parentModule, String fileName, String moduleBase, String prefix, Sessio
nState ss, Object privateData, ImportModuleOptions& options, ManifestProcessingFlags manifestProcessingFlags, Boolean& found, Boolean& moduleFileFou
nd)
   at Microsoft.PowerShell.Commands.ImportModuleCommand.ImportModule_LocallyViaName(ImportModuleOptions importModuleOptions, String name)
   at Microsoft.PowerShell.Commands.ImportModuleCommand.ProcessRecord()
   at System.Management.Automation.CommandProcessor.ProcessRecord()

WRN: Assembly binding logging is turned OFF.
To enable assembly bind failure logging, set the registry value [HKLM\Software\Microsoft\Fusion!EnableLog] (DWORD) to 1.
Note: There is some performance penalty associated with assembly bind failure logging.
To turn this feature off, remove the registry value [HKLM\Software\Microsoft\Fusion!EnableLog].

PS C:\WINDOWS\system32> remove-module sqlserver
Remove-Module : An item with the same key has already been added.
At C:\Program Files\WindowsPowerShell\Modules\SqlServer\22.1.1\SqlServer.psm1:78 char:5
+     Remove-Module -ModuleInfo $importedModules
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Remove-Module], ArgumentException
    + FullyQualifiedErrorId : System.ArgumentException,Microsoft.PowerShell.Commands.RemoveModuleCommand

Version info: Windows Server 2012 R2 ; SQL Server 2014 (12.0.6449.1); Powershell 5.1.14409.2001

Hi @matt-gantz - this looks like an old problem that was addressed a while back.
What OS are you on? The issue back then was only for people running on some really old OS (Windows Server 2016 maybe?) without NetFx 4.7.2.
The workaround back then was to forcefully include that netstandard "shim". But since the new v22 module got modernized, NetFx472+ is required... and that should make the need for that DLL go away.

Hi @Matteo-T - Yes this was occurring on Windows Server 2016 and 2012 R2.
It does appear that dotnet 4.8 solves the issue. I feel that this dependency should be listed here:
https://www.powershellgallery.com/packages/SqlServer
Do you agree?

Uhm... I don't see anything specific for "OS and .Net dependencies".
Those OSes are so old that... I would think by now the world had moved forward. :)
If that's not the case and I keeep hearing about this problem, I'll think of something.
Thanks.


Weak .st2 config file permissions allow reading st2 creds by other Linux users

st2 login -w creates ~/.st2/config file with r-o permissions, meaning that unauthorized linux user can read that file and obtain st2 login credentials:
$ rm -rf ~/.st2
$ st2 login st2admin -p Ch@ngeMe -w

$ ls -la ~/.st2
total 16
drwxrwxr-x 2 vagrant vagrant 4096 May 24 14:59 .
drwxr-xr-x 8 vagrant vagrant 4096 May 24 14:59 ..
-rw-rw-r-- 1 vagrant vagrant   55 May 24 14:59 config
-rw------- 1 vagrant vagrant   77 May 24 14:59 token-st2admin

Additionally, ~/.st2/ dir permissions when created might be adjusted (currently 0755).
Adding setgid for st2 config dir will help to better handle sudo st2 cases when config or token file is created with root permissions (see https://github.com/StackStorm/packer-st2/issues/38 for example).
Wrap Up

~/.st2/ dir when created by st2 should have 2770 permissions (currently 0755)
~/.st2/config when created by st2 should have 660 permissions (currently 664)
~/.st2/token when created by st2 should have 660 permissions (currently 640)

Related
https://github.com/StackStorm/st2-packages/issues/558 and https://github.com/StackStorm/packer-st2/issues/38

Any changes needed on the ansible-st2 or puppet-st2 side?


[Android] Fix filepicker crash issue.

This patch is to fix the filepicker crash issue on some customized ROMs.
In some ROMs, "/sdcard/Pictures" was removed, verify this directory
before create image file.
BUG=XWALK-3983
(cherry picked from commit 52ab32b1fd637e6652d9250c290b904621f035ca)

@lincsoon Please merge this patch to crosswalk-13, thanks.

Done, please also cherry-pick this patch to crosswalk-14

Yes, please look at https://github.com/crosswalk-project/crosswalk/pull/3021


zariot - read cram from SIM for onboarding

Is your feature request related to a problem? Please describe.
New Zariot SIMs have cram loaded in the SIM card. Use this value for onboarding
Describe the solution you'd like
Based on a config, read cram secret from external element and use in onboarding flow.

With new SIMs signature verification is failing
https://gist.github.com/murali-shris/bc6203c0299aa39f24fc68c81057b215

With new SIMs signature verification is failing https://gist.github.com/murali-shris/bc6203c0299aa39f24fc68c81057b215

The root cause was difference in public key between old and new sim. Jerry from Zariot recommended to generate key pair by giving the private key id. This will update the public key in the sim corresponding to private key. This worked

@athandle Can you please create an atsign in staging environment for atsign -atsign00001 with a specific cram code.I will DM you the cram secret

Update from call with Chris, Gary and Ashish.

Derived value from secret in sim card, to be used as activation key
Pass the activation key to the registrar APIs and get the cram secret
Tried the  API given by Ashish.
murali@murali-Latitude-3460:~$ curl --location --request POST 'https://my.atsign.wtf/api/app/v3/get-atsign/' --header 'Authorization: ' --header 'Content-Type: application/json' --data-raw '{"atSign":"atsign00001", "ActivationKey":"<act_key>"}'
{"status":"success","value":{"atSign":"atsignatsign00001","ActivationKey":"<act_key"}}
Getting atsign twice in the response.Checking with Ashish

Invoked activate API and resolved the issue.
Able to onboard for atsign00001.

Computed activation key from a secret stored in SIM.Passed this activation key to generate cram in staging for atsigns. Tested onboarding flow with pkam signing using private key from SIM.
activation key  generation and calling registrar API yet to be automated since JACS use case is unclear.Moving this to upcoming sprint with 0 SP.


Update ktlint to 0.47.1

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #121 (57fc795) into develop (8886ec9) will increase coverage by 0.24%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop     #121      +/-   ##
=============================================
+ Coverage      87.19%   87.43%   +0.24%
- Complexity       168      169       +1
=============================================
  Files             21       21
  Lines            523      525       +2
  Branches          65       65
=============================================
+ Hits             456      459       +3
  Misses            42       42
+ Partials          25       24       -1

Impacted Files
Coverage Δ

...com/datadog/gradle/plugin/DdAndroidGradlePlugin.kt
75.00% <ø> (+0.86%)
:arrow_up:

...m/datadog/gradle/plugin/DdMappingFileUploadTask.kt
88.97% <ø> (ø)

...og/gradle/plugin/internal/GitRepositoryDetector.kt
97.06% <ø> (ø)

...m/datadog/gradle/plugin/internal/OkHttpUploader.kt
86.30% <ø> (ø)

...in/com/datadog/gradle/plugin/DdCheckSdkDepsTask.kt
84.78% <0.00%> (+0.69%)
:arrow_up:


REPL: Weird bindings with multiple components

This has resurfaced, or never went away. I can't figure out what I do to trigger it, but it's not hard. It may have something to do with putting something that causes a compilation error into a child component, even temporarily.
The REPL app gets into a weird state where changes to one component affect multiple components, and the options in the dropdown menu do not always match the current names of the files.

It might be triggered by the compiler throwing an exception that the REPL doesn't know how to deal with (e.g., because of a bug in the compiler, rather than an issue we found in the user's component). If I go to https://svelte.technology/repl?version=1.18.0&example=nested-components and type in something that makes the compiler crash (e.g. {{yield}} next to another element, bug), the REPL gets into a bad state and seems to stay that way until I refresh the page.


RuntimeError: CUDA error: CUBLAS_STATUS_INTERNAL_ERROR

Hi
When I upgraded from 2.0.1 to 2.2.0 (other things have not changed), the following error occurred, what is the possible reason?

THCudaCheck FAIL file=../aten/src/THC/THCCachingHostAllocator.cpp line=280 error=700 : an illegal memory access was encountered
main()
torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
File "/wenet/wenet/utils/executor.py", line 84, in train
RuntimeError: CUDA error: CUBLAS_STATUS_INTERNAL_ERROR when calling cublasSgemm( handle, opa, opb, m, n, k, &alpha, a, lda, b, ldb, &beta, c, ldc)
Variable._execution_engine.run_backward(
executor.train(model, optimizer, scheduler, train_data_loader, device,
File "wenet/bin/train.py", line 309, in
File "/usr/local/lib/python3.8/dist-packages/torch/_tensor.py", line 307, in backward
loss.backward()
Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/torch/autograd/init.py", line 154, in backward
File "wenet/bin/train.py", line 281, in main

Hi, Did you set spec_trim = True?

Hi, Did you set spec_trim = True?

Yes, I set spec_trim=true.

I also encounter this issue when using spec_trim=True (not sure why ...), u can try disabling it.

I disabled spec_trim, and after training for a while, the following error still occurred. I am training u2++_conformer, using the multi-machine multi-card method.
The training data was generated with 2.0.1, could this be related?

2.0.1 works perfectly even if spec_trim=True?

Hi
I feel that this error seems to have nothing to do with whether to use spec_trim, because spec_trim has not been implemented in 2.0.1.

I also encounter this issue when using spec_trim=True (not sure why ...), u can try disabling it.

Would you still encounter this error after disabling spec_trim?

I also encounter this issue when using spec_trim=True (not sure why ...), u can try disabling it.

Would you still encounter this error after disabling spec_trim?

No

I have solved the problem. There are two versions of CUDA drivers in the cluster. You just need to choose the higher version when installing PyTorch.


Modernize S2A frame protector tests

This PR modernizes the S2A frame protector tests to use the gtest framework.

Thanks for the quick review @cesarghali!
Squashing and merging now.


How can I compile the external_auth.proto and develope my own gRPC external auth service?

Hi, I am trying to develop an external auth service here, and I need to decide whether a request should be allowed or not by the token in the request body.  So I have to use a gRPC auth service because Content-length will be set to 0 with a http auth service: https://www.envoyproxy.io/docs/envoy/latest/api-v2/config/filter/http/ext_authz/v2/ext_authz.proto#envoy-api-msg-config-filter-http-ext-authz-v2-authorizationrequest.
So, I need to compile this proto file into java code first, right? (https://github.com/envoyproxy/envoy/blob/1faaed85740a97533484db3232796aef7973677f/api/envoy/service/auth/v2/external_auth.proto#L30)
But this proto file imports two other proto files which are not in the repo of envoy:
import "google/rpc/status.proto";
import "validate/validate.proto";
Where can I get these proto files?

https://github.com/envoyproxy/protoc-gen-validate/blob/master/validate/validate.proto
https://github.com/googleapis/googleapis/blob/master/google/rpc/status.proto

@yzq789 btw, it's not necessary to compile it by yourself. You can use already compiled stubs as a dependency https://mvnrepository.com/artifact/io.envoyproxy.controlplane/api

It works. Compiled stubs are so convenient, thank you !


MAINT: stats.wilcoxon: pass PermutationMethod options to permutation_test; document PermutationMethod support

Reference issue
gh-19770
What does this implement/fix?
During preparation of the release notes, I realized that I forgot to document the addition of PermutationMethod support for wilcoxon. I had also neglected to pass the options contained in the PermutationMethod object into permutation_test (bug). This PR corrects the mistakes and adds a test.
Additional information
If docs look good @tylerjereddy, I think this is safe enough to merge. (For comparison, see e.g. mannwhitneyu PermutationMethod support.) Sorry to catch this at the last minute.

I think the failing tests aren't related. I'm getting the same issue for https://github.com/scipy/scipy/pull/20265

Thanks @mdhaber !


Replaced old language service with Roslyn-based one

@Microsoft/fsharp-compiler
This will stay in the new branch (roslyn), and users can try it by installing the vsix produced from the CI build there.
The current version just loads the new language service, and has no UI features (bare text editor).

LGTM

thanks @cston and @vladima !

This is such a great start - super to see this.


Update reference to macOS SDK extraction instructions

This PR replaces the outdated gitian-building/gitian-building-mac-os-sdk.md with the link to https://github.com/bitcoin/bitcoin/blob/master/contrib/macdeploy/README.md#sdk-extraction

ACK 1381c21745c00f614389ff75eb1fbe49b82ee53e

Should https://github.com/bitcoin/bitcoin/blob/master/contrib/macdeploy/extract-osx-sdk.sh be deleted as well?

Should https://github.com/bitcoin/bitcoin/blob/master/contrib/macdeploy/extract-osx-sdk.sh be deleted as well?

Correct. But it is in another repo :)
I was going to submit a pr after merging this one.

On a second thought, should the previous instructions (or a link to them) be preserved, because they are still needed for gitian builds of old tags?

On a second thought, should the previous instructions (or a link to them) be preserved, because they are still needed for gitian builds of old tags?

Is it enough for a reader to just switch the branch: for example https://github.com/bitcoin/bitcoin/blob/0.20/contrib/macdeploy/README.md ?

ACK 0aebb726ba7abda71c796552a54401190e28a55e


Azure Monitor Query: Update method names

This PR addresses naming changes suggested in arch board review:

queryBatch to queryResources
MetricsBatchQueryResult to MetricsQueryResourcesResult
MetricsBatchQueryOptions to MetricsQueryResourcesOptions
resourceUris to resourceIds

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-monitor-query


bug in entrypoint - BASE_URL duplicates

If you set the BASE_URL  env value the following script will be executed in https://github.com/idoop/docker-nuget-server/blob/master/docker-entrypoint
  if [ -n "${BASE_URL}" ];then
    echo "Use base URL: ${BASE_URL}"
    sed -i -e "/fastcgi_temp_file_write_size.*$/a\        fastcgi_param BASE_URL ${BASE_URL};" /etc/nginx/conf.d/nuget.conf
  fi

This creates new entries on every reboot of the containers and eventually your nginx.conf file will look like this
    location ~ \.php$ {

        try_files $uri =404;
        fastcgi_split_path_info ^(.+\.php)(/.+)$;
        fastcgi_pass unix:/var/run/php/php7.3-fpm.sock;
        fastcgi_index index.php;
        fastcgi_buffers 4 1024k;
        fastcgi_buffer_size 1024k;
        fastcgi_busy_buffers_size 1024k;
        fastcgi_temp_file_write_size 5M;
        fastcgi_param BASE_URL https://domain.tld/;
        fastcgi_param BASE_URL https://domain.tld/;
        fastcgi_param BASE_URL https://domain.tld/;
        ...
        fastcgi_param BASE_URL https://domain.tld/;
        fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
        include fastcgi_params;

    }

    location = /index.php {
        dav_methods PUT DELETE;

        try_files $uri =404;
        fastcgi_split_path_info ^(.+\.php)(/.+)$;
        fastcgi_pass unix:/var/run/php/php7.3-fpm.sock;
        fastcgi_index index.php;
        fastcgi_buffers 4 1024k;
        fastcgi_buffer_size 1024k;
        fastcgi_busy_buffers_size 1024k;
        fastcgi_temp_file_write_size 5M;
        fastcgi_param BASE_URL https://domain.tld/;
        fastcgi_param BASE_URL https://domain.tld/;
        fastcgi_param BASE_URL https://domain.tld/;
        ....
        fastcgi_param BASE_URL https://domain.tld/;
        fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
        include fastcgi_params;

        # PHP doesn't parse request body for PUT requests, so fake a POST.
        fastcgi_param REQUEST_METHOD POST;
        fastcgi_param HTTP_X_METHOD_OVERRIDE $request_method;
    }

@Steinblock Fixed,pull latest tag image


task: RWX example changes should be made in longhorn/manager instead of longhorn/longhorn

Just noticed that the examples are not updated in longhorn/manager, which is the source of the truth for longhorn/longhorn.

Examples got moved and merged into longhorn/manager#625


My TFLite model works with 32-bit ARM-based architecture but doesn't work with 64-bit AArch64 architecture

Hi,

I've created a TFLite model and ran it on a device with 32-bit ARM-based CPUs and it worked fine.
I've done the same thing for 64-bit architecture and there seems to be a problem:
I built the libtensorflowlite.so files using Bazel build (used --cpu=arm64-v8a for 64bit) as your guides explains. I pushed the complied files to my Android platform and tried to run the model.
I don't get any errors, the model is allegedly terminated successfully but it reports 0 [ms] runtimes and no output.

Do you have any idea what can be the problem?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?


Add retries to Director calls

Description
Changes proposed in this pull request:

Retries added
Tests added

Related issue(s)

/retest

/retest

/retest

/retest


SIM-FE Alow horizontal scroll when there is syntax error in the contract

I can't seem to reproduce the issue @cristiam86 , any chance you can provide a little more context?

Here:
https://genlayer.slack.com/archives/C071DE6200M/p1723821700964359
Maybe you can ask Edgars how to reproduce it.
Cristiam Da Silva
On Wed, 21 Aug 2024 at 14:08, den @.***> wrote:

I can't seem to reproduce the issue @cristiam86
https://github.com/cristiam86 , any chance you can provide a little
more context?
—
Reply to this email directly, view it on GitHub
https://github.com/yeagerai/genlayer-simulator/issues/409, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABCW5U376TPAEEJPTU2SHJ3ZSR7MDAVCNFSM6AAAAABM34GL62VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ3TOOBXGMZTCNA
.
You are receiving this because you were mentioned.Message ID:
@.***>


New-Partition cmdlet 4th partition cannot be specified as primary

Request
The Powershell New-Partition cmdlet is creating an extended type partition when called when there are already 3 primary partitions on an MBR disk.
There is no option to create this 4th partition as Primary, with no parameter capable of specifying this information.
An old Stackoverflow post references such a need as well as this request
Environment data
PS C:> $PSVersionTable
Name                           Value

PSVersion                      5.1.14393.953
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.953
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

@imakiro The cmdlet is not in the Repo. Please report the Issue to https://windowsserver.uservoice.com/forums/301869-powershell


Yasuo's tornado and ult sounds not changing

His sound effects for tornado and ult remain the same on all skins.

#1

There is changed Q sound when tornado is charged and then there's mostly a sound effect that follows. Spirit blossom yasuo 3rd Q should be the most noticeable. The initial Q3 sound remains the same even after changing the skin.


ERC20 Enterprise README

Fixes #661  .
Changes proposed in this PR:

add tests for the functionality
create the .md file

I'm sorry to do this, but please apply the DRY rules defined and exemplified by Pablo in the latest merged PR.


Multi-token event triggers in ACE

Hi Xinya,
According to the data example, there is only one index provided for the trigger. Does this mean that in your preprocessing you removed all multi-token triggers?

Hi @raspberryice ,
I kept the first token (after the tokenization) of the trigger, similar what the BERT paper did for the NER experiment.


[synapse] Add lxml for URL previewing support

Ryan Rix (@rrix:kickass.systems)

ianweller: Pls add libxslt-dev libxml2-dev to BUILD_PKGS,apk add libxml2 libxsltand pip install lxml so that I can re-enable URL previewing in homeserver.yaml thanks
~OPEN SOURCE~
thanks

Should be fixed in 8bb2141 but pending build + your verification

Build done: https://hub.docker.com/r/ianweller/synapse/builds/bz3agloybddkyvjrvjwfhkh/
Can you pull and confirm that URL previewing support is there?

@rrix finally told me this works


Gallery Improvements

Instead of completely hiding locked images, draw placeholder text for each image like "Unlock in Snake" or "Unlock in Tiltrads"

Feature done, may update graphics


chore: Update Turing and Merlin charts with latest release specs

Motivation
The latest release of Turing (v1.16.0) and Merlin (v0.35.0) drop support for Python 3.7 altogether. This PR updates the chart specs accordingly.
Modification

Update app versions for Turing and Merlin
charts/merlin/templates/_config.tpl, charts/turing/templates/_config.tpl - Drop Python 3.7 from the helper template functions for the Pyfunc server images
Update unit tests

Checklist

[x] Chart version bumped
[x] README.md updated

Thanks for the quick review, @deadlycoconuts ! Merging.


Crash when running with Head Index

Running Minecraft 1.21.0 with Fabric Loader 0.16.9 and Fabric API 0.102.0 , running either NoClip 1.3.1 or Head Index 1.1.8 alone has no errors at all, but running both together causes a crash both on my PC and on my Aternos server. The first error thrown says that it's from sgui (part of Head Index) but the crash log says that NoClip caused the crash, so I am posting this in both places to try to get further clarity.
Full Log:
https://paste.ee/p/2wQ08

Appears to be a problem with a version mismatch, see the other issue for more details.


Shell escaping

I'm having a bit of a problem with shell escaping.
I use a variable to store a bunch of my default switches but no matter how I escape the quotes, don't use them, or use different quotes, it isn't interpreting like i'd expect.
Example:
STD_SWITCHES="--stats 5m
--config='$RCLONECONF'
--transfers=16 --checkers=32
--one-file-system
--bwlimit="07:00,6M 23:00,10M" "
-># echo $STD_SWITCHES
--stats 5m --config='/home/robert/.rclone.conf' --transfers=16 --checkers=32 --one-file-system --bwlimit="07:00,6M 23:00,10M"
root@HS: ~
-># rclone $STD_SWITCHES rmdirs
Error: invalid argument ""07:00,6M" for --bwlimit="07:00,6M: invalid time specification (hh:mm): ""07:00"
Usage:
...
...
2017/02/27 12:14:21 Fatal error: invalid argument ""07:00,6M" for --bwlimit="07:00,6M: invalid time specification (hh:mm): ""07:00"
I've tried all of the following:
STD_SWITCHES="--stats 5m
--config='$RCLONECONF'
--transfers=16 --checkers=32
--one-file-system
--bwlimit='07:00,6M 23:00,10M' "
STD_SWITCHES="--stats 5m
--config='$RCLONECONF'
--transfers=16 --checkers=32
--one-file-system
--bwlimit=07:00,6M 23:00,10M "
STD_SWITCHES="--stats 5m
--config='$RCLONECONF'
--transfers=16 --checkers=32
--one-file-system
--bwlimit=07:00,6M\ 23:00,10M "
Seems to be the parser in general. I've never run into this because I didn't have spaces in any of the variables until the bwlimit one was introduced but this doesn't work for the config variable either. Basically there is no way to escape it properly but it isn't the bwlimit parsing itself. It is the command line parser in general.

I think this is all to do with the shell.  The problem I think is the use of " within the quotes
This should work
STD_SWITCHES="--stats 5m
\"--config=$RCLONECONF\"
--transfers=16 --checkers=32
--one-file-system
\"--bwlimit=07:00,6M 23:00,10M\""

Note that when you do rclone --bwlimit="07:00,6M 23:00,10M" it is the shell parsing the quotes, not rclone.
So your original command passed the quotes to rclone which it doesn't parse - it is expecting the shell to do that.

I would have though so, but it doesn't.  here:

set -x
+ set -x
#STD_SWITCHES="--stats 5m \"--bwlimit=07:00,6M 23:00,10M\""
+ STD_SWITCHES='--stats 5m "--bwlimit=07:00,6M 23:00,10M"'
#rclone ls /tmp $STD_SWITCHES
+ rclone ls /tmp --stats 5m '"--bwlimit=07:00,6M' '23:00,10M"'
Usage:
  rclone ls remote:path [flags]

Now, if I take that exact command and run it without a variable:
#rclone ls /tmp --stats 5m "--bwlimit=07:00,6M 23:00,10M"
+ rclone ls /tmp --stats 5m '--bwlimit=07:00,6M 23:00,10M'
        0 dyndnslogin_output_***AzoO
        0 rclonefilelist.txt
...
...

rclone is doing something really non-standard when pulling in the variables.  This type of variable substitution is pretty typical.

I just verified rsync behaves the same way with switches with spaces in them.  I'll close this as it isn't an rclone problem.  I'm not sure how to deal with this but one way may be to use eval to eval the command before running it.  Pretty clumsy though.

You're being bitten by shell expansion.
When you do this:
$ cmd a b "foo bar"
You get:
argv[1] = "a"
argv[2] = "b"
argv[3] = "foo bar"
Which is what you want. But then you're setting this on a variable:
a="a b "foo bar""
cmd $a
cmd will actually see:
argv[1] = "a"
argv[2] = "b"
argv[3] = "foo"
argv[4] = "bar"
Notice how the space between foo and bar caused the expansion of foo and bar into two different arguments. This is not what you want. An alternative would be to avoid expansion by quoting $a:
$ cmd "$a"
But that will break in a different way. "cmd" will see:
$argv[1] = "a b foo bar"
The easiest way in your case would be:
eval "cmd $a"

Thanks Marco.  Makes total sense.

@calisro Did you solve this problem? I tried it in several ways, but I could not.

@adautoserpa I'd solve this by setting environment variables to set switches which eases the shell escaping pain.

@ncw @calisro Good solution.


Move navigation control to bottom right corner

The default position in the top-right corner makes them inaccessible because of the "Donate" banner.
See OSM US Slack thread. Closes #115.

Thanks for fixing this, @flother!


[Bug report] Typescript config files are still supported upstream by commitlint

✅ Checklist

[X] I am using the latest version
[X] I have searched FAQ

📄 System Info
not needed

🔎 Description
See https://commitlint.js.org/reference/configuration.html
So https://cz-git.qbb.sh/config/#typescript-template should be fixed
🔗 Reproduction link / repo:
No response

commitlint Replace ts-node with cosmiconfig-typescript-loader package https://github.com/conventional-changelog/commitlint/pull/3722
Although the speed has improved, adding cz-git and czg will double the size, and I am still considering it. 😶🌫

commitlint.config.ts is supported by commitlint directly, so the docs need to be corrected, there is nothing to do with your pacakge
Even if you want to have cz.config.ts supported, you should externalize cosmiconfig-typescript-loader, tools like rollup and tsup can do that easily

Besides, I think you have misunderstood my original issue complaining about the dep size.
Since your pacakge is strongly relying on commitlint, so you should try to reuse package that is required by it as much as possible while keeping the same major, this way you can externalize them safely as they are supposed to be installed to workspace anyway.
The main target for tool package like this is trying to avoid requiring any new package that is heavy. A package is heavy means that it has too many deps or a lot of unused features for your usage. You can try to bundle packages with cz-git, only if they are likely not required by any other packages, or in another word "unpopular"

Thank you for your reply.

However, I don't understand with my documentation not supporting commitlint.config.ts Please feel free to point it out.

Since your package is strongly relying on commitlint, so you should try to reuse package

Like @commitlint/load package have too many dep package.json. I have try but cjs project treeshakable effect is not well
Although I do not agree that a lightweight CLI tool, which requires the fastest startup speed, should resolve the ts configuration file at this time, I will keep an open opinion and will not add it in the short term.

However, I don't understand with my documentation not supporting commitlint.config.ts Please feel free to point it out.

No offense, but as I am pointing out the link in issue body, it should not be hard to find these.

Done
Btw, I will try to configure loading for TypeScript using node > 22 in the future, instead of adding extra dependencies.

Hi there.
Node.js LTS v22 has been released, and in v1.11.0, I will add the loading of ESM configuration and TS configuration.
Regarding the loading of TS configuration, it is done using the native Node loading method, but it is currently an experimental feature that requires additional environment injection.
It is recommended to first use the ESM JS method for transition.
docs preview: https://deploy-preview-197--cz-git.netlify.app/
I have released a next pre-release version 🌟(v1.11.0-beta.1)🌟 for testing.
I expect to officially release v1.11.0 next weekend.
Welcome to try it out! 👀 Let me know if there are still any issues. 🎉

I am testing in the repository https://github.com/vuepress-theme-hope/vuepress-theme-hope
Suggestion (ESM js)

rename commitlint.config.ts => commitlint.config.js

import { execSync } from "node:child_process";
import { readdirSync } from "node:fs";
import { dirname, join } from "node:path";
import { fileURLToPath } from "node:url";

import { defineConfig } from "cz-git";

const packages = readdirSync(
  join(dirname(fileURLToPath(import.meta.url)), "./packages/"),
);

const scopeComplete = execSync("git status --porcelain || true")
  .toString()
  .trim()
  .split("\n")
  .find((r) => ~r.indexOf("M  packages"))
  ?.replace(/\//g, "%%")
  ?.match(/packages%%((\w|-)*)/)?.[1];

export default defineConfig({
  extends: ["@commitlint/config-conventional"],
  rules: {
    "scope-enum": [2, "always", ["demo", "release", ...packages]],
  },
  prompt: {
    defaultScope: scopeComplete,
    customScopesAlign: !scopeComplete ? "top" : "bottom",
    allowCustomIssuePrefix: false,
    allowEmptyIssuePrefix: false,
  },
});

(TypeScript configure)
-     "commit": "pnpm git-cz",
+     "commit": "cross-env NODE_OPTIONS=\"--experimental-transform-types --disable-warning ExperimentalWarning\" pnpm git-cz",

The latest version 🌟(v1.11.0)🌟 released and can use Node v22 to loading ts configure, But it is recommended to use ESM js configure as a transition for the future TypeScript configuration file, and switch to it after Node.js runs TypeScript stably.
Docs Link: https://cz-git.qbb.sh/config/#typescript-template


fix: correct issues with mixin params

fixes issue csstools/postcss-advanced-variables#77 https://github.com/csstools/postcss-advanced-variables/issues/78

@armano2 This forked repo fixed issue with vars in mixins? How can I use your forked repo?


ueberzugpp support on wayland

ueberzugpp supports wayland, but I cannot use it in ctpv because there is an explicit check that disables ueberzug in wayland.
I recompiled with the change below and now image previews are working perfectly.
Probably some better fix could be added (check for ueberzugpp? introduce a forceueberzug flag? ...) but this at least seams to work fine.
diff --git a/sh/helpers.sh b/sh/helpers.sh
index fef8691..f194557 100644
--- a/sh/helpers.sh
+++ b/sh/helpers.sh
@@ -52,7 +52,7 @@ set_image_method() {
 	[ -n "$forcekittyanim" ] && is_kitty && is_anim_image && { image_method="$image_method_kitty"; return 0; }
 	[ -n "$forcechafa" ] && exists chafa && { image_method="$image_method_chafa"; return 0; }

-	[ -n "$DISPLAY" ] && [ -z "$WAYLAND_DISPLAY" ] && exists ueberzug &&
+	exists ueberzug &&
 		[ -n "$fifo" ] && [ -e "$fifo" ] &&
 		{ image_method="$image_method_ueberzug"; return 0; }

that would be nice


Inconsistent comparison of basic_fixed_string

See https://godbolt.org/z/WfPYa8.
I don't understand why units::basic_fixed_string{"abc"} == units::basic_fixed_string{u8"abc"} chokes GCC 10+. Even then, the portability of such constructs is dubious. However, these inter-specialization equality operators always return false, and yet their relational operators do their job. But what isn't equality of strings if not !(l<r)&&!(r<l)? Could the intention have been to allow these for equal sizes? Or maybe also disallow their ordering?

Thanks for raising this issue! For my excuse, I did not put too much attention to types in the external directory for now (just made them work somehow ;-) ) as I do not perceive them as the official interface of the library.
I hope that someone else will standardize them or we can define them as implementation details for now. Of course, we may try to push them through the Committee by ourselves and in such a case we should spend some more time on making them right. Thank you for doing this here. However, I am afraid that it will be hard to satisfy everyone with such types. Everyone has different expectations from the interface of such a type and I do not want to end up with yet another std::string monster 😉.
BTW, talking about std::string there is a high chance that we will switch basic_fixed_string to std::string really soon...

I just looked into the code. Did you mean the implementation old (pre <=>) comparisons? If yes, this is a bug and could be fixed (although this is a low priority as at some point in the future, we will probably not be able to keep compatibility with gcc-9.3 and all this code will vanish).
I think that C++20 comparison is defined correctly, right?

A similar inconsistency between C++20 and C++17 comparison operators is in basic_symbol_text.

BTW, talking about std::string there is a high chance that we will switch basic_fixed_string to std::string really soon...

But you use basic_fixed_string for NTTPs, whereas you can't with std::string in C++20.
The main issue is that, for strings of different char types, a == b is not equal to !(a<b)&&!(b<a), because a == b is always false. Either a < b should always be false, or a == b should be implemented as std::ranges::equal(a, b).

But you use basic_fixed_string for NTTPs, whereas you can't with std::string in C++20.

That is right but it seems that the rules will change for C++23 with some magic operator template. But we sill see how it goes.
My understanding is that there is no operator== for strings of different char types in C++20 part of the code. The templated operator== works only for different sizes of the same character type. Am I right?

My understanding is that there is no operator== for strings of different char types in C++20 part of the code.

There are operator==s, but the one for different characters always returns false, unlike operator<=>.

The templated operator== works only for different sizes of the same character type. Am I right?

No, it works for strings of the same size, too.
Please look at the end of https://godbolt.org/z/fbPjro.
constexpr units::basic_fixed_string c{"abc"};
constexpr units::basic_fixed_string u8{u8"abc"};

static_assert(c <= u8);
static_assert(!(c < u8));
static_assert(!(c < u8) && !(u8 < c));
static_assert(c != u8);

Neither string is less than the other, so they must be equal. Yet, operator== claims they are not equal. That's the problem.

I am confused. As I understand we talk about this hidden friend comparison operator:
template<typename CharT, std::size_t N>
struct basic_fixed_string {
  // ...
  template<std::size_t N2>
  [[nodiscard]] friend constexpr bool operator==(const basic_fixed_string&, const basic_fixed_string<CharT, N2>&) { return false; }
};

In my understanding, it requires the same character type CharT for both arguments and only they can differ with size. Isn't it true? Where is the conversion from char to chat8_t is being done?

Good question. The first time I failed to notice that CharT is actually the class template's parameter, and though it was the operator's. Somehow there's a conversion from basic_fixed_string<char8_t> to basic_fixed_string<char> or viceversa.

If you remove #ifdefs from your example you end up with: https://godbolt.org/z/sh74vE 😉 Is it correct behavior that you meant?
As I said I do not care about "before C++20" implementation too much and I know it is sometimes inconsistent with C++20 one. In fact, right now I struggle a lot to make new dimensionless quantities to work with gcc9.3. This compiler is so buggy that I really consider getting rid of support for it (I assume that people who use gcc-9.3 already will move to gcc-10 easily). Moreover, there are (and will be) no Conan docker images for gcc-9.3 which also makes it hard to keep it in the loop.

If you remove #ifdefs from your example you end up with: https://godbolt.org/z/sh74vE wink Is it the correct behavior that you meant?

Yes. I forgot that these macros are defined by the library, so they aren't defined in my example. I was wondering why the behaviour was different when including the header.

Good question. The first time I failed to notice that CharT is actually the class template's parameter, and though it was the operator's. Somehow there's a conversion from basic_fixed_string<char8_t> to basic_fixed_string<char> or viceversa.

It happens that the alternative implementation actually does allow comparisons of strings with different character types: template<typename CharT2, std::size_t N2>.
The fix: Drop support for GCC 9.3 or fix those overloads.

GCC 9.3 is dropped already. Sorry to everyone that did not upgrade to gcc-10 until now...

@johelegp, can we close this one now?

Yes. Thank you.

GCC 9.3 is dropped already. Sorry to everyone that did not upgrade to gcc-10 until now...

Sorry for posting to the closed ticket, but I was searching for the reason why gcc-9 support was dropped and found this. So I'm  one of those who have no easy way to move to gcc-10 as I'm from embedded SW world. On the other hand I was one of the people voting for dimension one implementation. As this feature is implemented now (thanks for this!) I tried updating mp-units in my project to cleanup all the tricks I had to use due to absense of dimension one and discovered it doesn't build now :-) I'm about to fork the library and try to add gcc-9 support back to it, but @mpusz mentioned here that he had to struggle to make dimension one work with gcc-9, so now I'm wondering if it really worth the effort for me to try bringing the support back. @mpusz, as you already faced these issues, could you please elaborate on how bad they are (I always have an option to stick with older gcc-9-compatible library version if cost of adding gcc-9 support appears to be higher than the cost of maintaining tricks to continue to live without dimension one)?
Sorry again, if it is not the right place to discuss this. If so, please point me to the proper place.
Thanks in advance!

First of all, I am really sorry to hear that :-( I was afraid that with such a move we will lose some users so it was not an easy decision to be made.
The biggest problem I had was the ambiguity of quantity and quantity_point operators after I made them hidden friends. I had to make them hidden friends to enable implicit conversions of dimensionless quantities so to make compile things like 4_q_m / 2_q_m + 1 without the need to explicitly construct a dimensionless quantity from 1.
Even though gcc-9 is really buggy and lacks a lot of support we need, I am really open to restoring gcc-9 if it will compile fine. So if you will be able to achieve that than great. The biggest problem will be probably the one I described above. However, you should also start with reverting this commit 4bb51586dcb3c23d20f16f933f01838d771a129d that basically removed all the "garbage" in the code connected to gcc-9 support. If you will look into this commit you will find out that getting rid of gcc-9 left us with a much better and easier to maintain code. This is why I made this call.
Again, I am really sorry about this...

@mpusz Thanks for the quick feedback! And no problem, I understand that keeping compatibility (especially with something buggy) is a big maintenance burden, so I totally understand your decision. It's just an irony that after my comment in the poll regarding dimensionless quantities you told you will implement them. But for the sake of their implementation it was required to sacrifice gcc-9 support. And now dimensionless quantity is there but I can't use it :-)
But it's kind of good news for me that the biggest issue is implicit conversions, cause in my particular case I can easily live without them. So I think I will give bringing back gcc-9 support a try then, but chances are I will end up with a solution which is not too generic (e.g. with lack of implicit conversions you mentioned)


Exception config for hiding specific sidebar icons

It would be nice if I can hide selected icons on the sidebar in exception config, similar to how I can hide tabs today.
I can see two use cases

to hide some tools to specific users - as the sidebar can contain a mix of admin tools and user tools (such as map, notification, user profile, maybe some others)
to hide tools that are useless on mobile device
Today, the only option is to disable the whole sidebar - but then people can't even change their password.
Thanks!

I won't be adding any features related to the sidebar. Since CH only operates inside of Lovelace I need to keep any interaction with the sidebar to a minimum. The sidebar is required for navigation outside of Lovelace, so any modification of it should be handled with an addon that is available across the entire frontend.

Ok, will look in there or write one. Can you please point me into the right direction?
Integrations are backend things, they do not run in the context of individual user sessions.
Python scripts and Appdaemon scripts the same.
Then there are plugins, are frontend customization, so they seem to be the way to go - but you say they run only for lovelace.
Then there is the addon store, but that is only for hass.io (and I am not even sure how wold that work).
Do you know any such addon you mention?
Thanks

So this would need to be added as an extra_module, I have yet to look into creating one myself, but you'd add custom JS by using extra_module_url in your config. Something like:
frontend:
  extra_module_url:
    - /local/NameOfScript.js

https://www.home-assistant.io/integrations/frontend/#extra_module_url

I've been toying with the idea of a Custom Sidebar addon like this for a while, but just haven't had the time to get into it. If you do end up making something like this, please let me know as I have some code that I scrapped from an earlier version of Custom Header you might be interested in. It allows the sidebar to be moved to the right side of the screen, but for the reasons mentioned above I kept it out of Custom Header.

Was looking for this, but finally found the addon that uses this method:
https://github.com/andrey-git/home-assistant-custom-ui

Thanks :)


Removed euclid dependency, while allowing for more math functions.

Resolves issue #208. This is a lot of code, so please review carefully. Thanks!

I'm confused what happened here - I thought we were only replacing the uses of Point2D and Size2D in surfman; there should be no need to import all the other unused types from euclid. The code in the examples that uses other euclid types can continue to use euclid; we can change the dependency to a dev-dependency in the Cargo.toml instead.

That makes much more sense. Sorry. I'll close this and fix some of the changes.


Plugin name
Sign Text Collar
download page
https://dev.bukkit.org/projects/signtextcollar
Source code URL
No response
Paid?

[ ] is paid for?

Fully compatible


Typo in CoffeeScript sample

Hi,
I noticed a small typo in the CoffeeScript code sample, so I've corrected it.

Oops, thanks!


init-common.sh expects group==user which may be false

init-common.sh expects that group==user exists which may be not true
sudo chown -R $USER:$USER $ROOT
Solution is to use sudo chown -R $USER $ROOT maybe?
Affected are security-altered Ubuntu systems and also non-Ubuntu systems. My case is openSUSE which has no personal groups created for the users.

We both (@synapticvoid and me) working on a Kubuntu, so some commands could be Debian oriented.
Ok for me to update this line if it can increase the host OS compatibility without breaking the behavior on a Debian host.

Thanks, I just can confirm the great tool works with openSUSE 42.3 so yes, it would be increasing compatibility.


⚠ Bofrost.lu has degraded performance

In 9d4b77b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1157 ms

Resolved: Bofrost.lu performance has improved in 0b8ebec after 30 minutes.


[fix] archive download URL

Just one more thing!
I used gobrew install
but don't download archive.
i fix it.

Thanks!


Adds C++ and nn quantization utilities

This PR adds the following:

C++ quantize/dequantize ops
QuantizedLinear which mimics Linear but performs a quantized matmul.

To implement QuantizedLinear I introduced a couple of additions in the base Module and tree_map utils.

Module.update_modules: Similar to Module.update that updates the parameters from a "tree" of parameters. This function updates the modules from a "tree" of modules. Makes it quite easy to do all kinds of module swaps like the one done by QuantizedLinear.
tree_map takes an optional is_leaf

The end result is quite nice (if I may say so myself) although we do have to figure out how this connects with the converters and all. Example usage for now:
model = load_my_big_model(...)
nn.QuantizedLinear.quantize_module(model, groups=64, width=4)
mx.eval(model.parameters())

# profit

Update the error tolerance so that CPU tests pass

That fix never fails me 🤣

Haha it is usually the other way around. "Update the error tolerance so the GPU tests pass".
 Given that the error was only marginally larger (about 2e-5 larger) and that we need to revisit that cpu kernel anyway... It could be the simd_reduce_add or it could be that the addition order is different.


Why this number 49 ?

Is there a specific reason about this number ?
https://github.com/sacs-epfl/decentralizepy/blob/a5d46be907bc745652d5482c34117be388324439/src/decentralizepy/node/DPSGDNode.py#L177

There is no specific reason, we just tried to reduce the logging load after 50 training rounds.

Ok thanks,
I think it's best if this value can be transformed into a configurable parameter.


Update containerd/cgroups to fix perf issue

Update one of our vendors to pick up an upstream performance fix

LGTM and happy green buttons.  @rhatdan PTAL

@rh-atomic-bot r+

@rh-atomic-bot r+

@rh-atomic-bot retry

Manually merging since homu seems sick.


HUDI-139 Compaction running twice due to duplicate "map" transformation while finalizing compaction

Jira: https://issues.apache.org/jira/browse/HUDI-139

@n3nash @vinothchandar : Found this when debugging compaction failures in deltastreamer continuous mode testing.

Eyeballed for similar pattern. Couldn't find any.  Will go ahead and merge


[CEM-18672]: Rmoved throwing panic code

Replaced throwing panic code with logging and returning error. Replaced logging using fmt with log

recheck

recheck

recheck ha

recheck ha

recheck ha

Agree but it's based on this https://github.com/Juniper/contrail-operator/commit/5840bd542dc0100c7b6a0c9939857490c194c233 changes and we need first merge those and then mine. I'll create one more pr with both changes for 2008

@lrebeja ok, create the other one fro R2008 and we will merge it there first.
Also, you need to rebase this PR onto the latest commit in the master branch as some fixes for the ha e2e tests have been merged recently.

In fact you don't need to make this PR into master anymore, because together with CaVA team we have a merging flow where we work on R2008 branch and then from time to time me or Piotr makes PR which merge R2008 changes into master in single PR.

Closing as this change is already merged into R2008 branch and will be merged into master with batched PR from R2008 branch


Documentation Error

When I review the documentation under Installation, step number 3, "See config-sample.json for examples." the link to the config-sample.json is broken.

And nobody cares for a year.
Apparently the correct link would be https://github.com/lprhodes/homebridge-broadlink-rm/blob/master/config-sample.json


🩺(coverage) add config and make rule to compute coverage

Purpose
Compute test coverage to make sure we test every line, especially upon adding a new feature.
Proposal

[x] Configure pytest-cov settings in pyproject.toml
[x] make rule in Makefile

Improvements

Should we add a run of coverage in the CI ? We could set a branch rule stating, say, that a branch cannot be merged if it makes coverage go under, say, 90% on main. WDYT ?

Nice 🤩 Maybe we will be able to compute coverage in the CI and add a requirement to make sure that each PR does not degrade the coverage score?

Absolutely ! I wanted to do that but maybe in another PR, because I'm not super fluent in github actions and would rather focus on shipping the domains. I'll open an issue.


[Pick up city UBS.User's personal cabinet] User cannot enter numeric characters in the 'Name' and 'Surname' fields on the 'User date' page

Environment: Windows 10 Pro v.21H2, Google Chrome 112.0.5615.13 (64-bit).
Reproducible: always.
Build found: 24/04/23
Preconditions

User is logged in as user.
User data tab is opened.
'Name' and 'Surname' fields are filled and contain alphabetic characters
e.g. Iryna
e.g. Mikhalska

Steps to reproduce

Click on the 'Edit Data' button.
Add numeric characters in the 'Name' field
e.g. Iryna1.
Add numeric characters in the  'Surname' field
e.g. Mikhalska2.

Actual result
Error messages in red are displayed when numeric characters are entered in the 'Name' and 'Surname' fields.

Expected result
User can enter numeric characters in the 'Name' and 'Surname' fields.
User story and test case links
User story #3811
Labels to be added
"Bug", Priority ("pri: Medium"), Severity ("severity:Trivial"), Type ("Functional").

We have no access to the Pick up city UBS


Please provide documentation

I have been trying to implement this library for a while now(close to a month now), have trying adding it as a library, as a module, but always get errors. For instance - it throws an error-> Unable to find ViewUpdater as soon as the app launches.
I will really appreciate any help.

I am getting same error.. Please how can we fix this ? @dvg4000

Maybe you use example's CardsUpdater? Check this line in your layour.xml file: app:viewUpdater="com.ramotion.cardslider.examples.simple.cards.CardsUpdater". If you have it, just remove it or change to your local CardsUpdater.

Yes, that was the problem


[alt] remove default iree-compile extra args from onnx-iree mode

Always using iree-input-demote-i64-to-i32 was causing numerics problems since it wasn't loading i64 inputs correctly. Additionally the flags for specifying the chip for hip is out of date, and should be removed. Users should specify -ica "iree-hip-target=gfx942" "other-arg" "other-arg" ... if compiling for Mi300.

@ScottTodd Should I revert the change to the cl config to use --iree-hal-target-backends instead of --iree-hal-target-device?

@ScottTodd Should I revert the change to the cl config to use --iree-hal-target-backends instead of --iree-hal-target-device?

That's probably safer for now. Since the new flag hasn't been fully adopted in the IREE repo and doesn't have much mileage on it, it may be buggy. That's going to be easier to reason about upstream first, and that sort of flag change can also be made independently from the rest of the cleanup here.


Reduce contention when updating TimeDistribution

This change implements a simple lock striping strategy to help
reduce contention when updating TimeDistribution from multiple threads.
TimeDistribution now maintains a set of QuantileDigests that get updated
independently and merged lazily every so often.

Any particular reason Distribution doesn't need this treatment as well?
Consider making a StripedQuantileDigest?

other than John's concerns, looks good

Can't you use QuantileDigest directly? Or are you leaning on the fact that this class is thread-safe and has the convenience methods to translate the time units?
Would adding a method like "toQuantileDigest" that does the merge and returns an instance of the quantile digest be sufficient?

@martint in the mean time, how about we land this?


EVM Staking - Nomination Flow

Summary of changes

EVM users can now bond tokens and nominate validators.

Proposed area of change

[ ] apps/bridge-dapp
[ ] apps/hubble-stats
[ ] apps/stats-dapp
[x] apps/tangle-dapp
[ ] apps/testnet-leaderboard
[ ] apps/faucet
[x] libs/webb-ui-components

Screen Recording (UI Implementation)
https://github.com/webb-tools/webb-dapp/assets/53374218/e3bae463-30a2-4e79-ad48-f384ebe13c1a

Closing this since the EVM Staking app will be migrated to a new React app.