Module for fetching and caching metadata from the aggregation server

Context

See:

https://jira.iohk.io/browse/ADP-258
#1595
#1596

Decision

Write a self-isolated module that can offer a simple interface to fetch and refresh metadata from an aggregation server. We'll likely cache these data in a database or on-disk in a second step.
Acceptance Criteria

Development

QA

Btw, did you know about managerRetryableException? Maybe it could have been used for cardano-wallet-jormungandr, or can be used here, if we want retrying.

Nope I didn't! Sounds like indeed something that is pretty handy. I've seen the servant client automatically retry on 500 errors. I wonder if it's manual or using this setting from the manager.

LGTM.


initial attempt at pretty toml strings

this is my initial working method for making toml strings "prettier."
This method is BAD -- it currently makes a completely separate Printer object called PrettyPrinter when really only a few lines need to be changed.
The issue I can't get around is that Printer has a Formatter object attached to it... and I have literally no idea how to get one of those without using the format! or related macros and overriding fmt::Display (I can't do that because I need to insert the pretty options in somehow!)
Do you know how to get a Formatter object so I can use it?
Other tasks:

move the tests to the tests folder

Other comments would be appreciated

closes #119

one possibility would be to attach the formatting options to the TomlTable object... not my favorite option, but possible.

Closing due to inactivity, but feel free to resubmit!


⚠ Community has degraded performance

In 64349ad, Community (https://community.wrk.com/health) experienced degraded performance:

HTTP code: 200
Response time: 219 ms

Resolved: Community performance has improved in 0c04e7c.


extended.external fails due to parallel SubDir childJobNum exceeds 20

Extended.external uses parallel=SubDir, but this now fails as the number of sub dirs exceeds 20:
https://ci.adoptopenjdk.net/view/Failing Test Jobs/job/Test_openjdk11_hs_extended.external_x86-64_linux/31/console
This limited and asserts the job here: https://github.com/AdoptOpenJDK/openjdk-tests/blob/1be48796f2a31912cd53cc55a6f27978445bd050/buildenv/jenkins/JenkinsfileBase#L202

An update to that childJobNum will land when https://github.com/AdoptOpenJDK/openjdk-tests/pull/2093 is merged.

This issue is now addressed via #2093


Does this expand an already existing service
No
Link to a game or software
https://taiga.io/
Links for the download
https://github.com/taigaio/taiga-docker
Links for the install docs
https://community.taiga.io/t/taiga-30min-setup/170

This can not be done as it needs a docker compose stack what wont work as it need multiple images


Annoyance about Augmented Weapons

Whenever I try to input an augmented weapon it automatically switches to the non-augmented version. This is starting to be come extremely irritating every time I attempt to do so. Could you correct this in the future?

No. It's literally not possible. They have the exact same model data and variant. You do not need to care about it. There's no way to distinguish between them from model information alone.


Database may use unmounted host path

Currently we use internal storage to mount a file called storage.ext4 to /mnt/paiInternal on the master node. Then postgresql uses /mnt/paiInternal as its data root.
Sometimes the path could be unmounted. If postgresql starts before internal-storage, it will use unmounted host path.

TODO: add a barrier to postgresql service


Swipe gallery component

Component to display pictures on mobile, with optional captions

Should we use this package? What are the considerations before deciding to add a UI package?

Considerations:

Does it have the features we want
Popularity (stars are a good indicator)
How active is development
Do they merge PRs from other people
How many open issues are there (and how serious are they, how quickly are issues closed) (noting that popular packages get more issues)
How much will it add to the package size  (not a massive deal as long as it's not crazy)

react-material-ui-carousel doesn't appear to be swipeable, I think that's important for the mobile design

What about swiper.js? It's a bit hefty though.

On second thought, maybe we should just start with a super simple one. It's only a couple of lines of css.

Interesting. I think that particular solution is no longer supported. But we can definitely skip the library and write our own component.
Otherwise, how about react-multi-carousel?

If rolling our own is very easy, it's probably a better idea, if not, that does seem like a good library!

And let's only worry about modern browsers until there's demand for supporting older ones.

This is closed in #658 with HorizontalScroller. It works for mobile only. For now it is basic with no caption support or lightbox, etc.


[next] replace useCursor with Hovering

Replaces useCursor with a runes-based class, Hovering.
Why the rename?
the functionality that useCursor provides is primarily about hovering over an object and then adjusting the cursor style. This is evident in its usage. Hovering has a current prop that when true adjusts the style of the cursor to the provided hoverStyle and sets it to unhoverStyle when current is false.
The logic is similar to the following
if (current) {
  element.cursor.style = hoverStyle
} else {
  element.cursor.style = unhoverStyle
}

Actually it might make more sense to call this Cursor or HoverStyle. Either way we cant call this UseCursor.
Why the change to a class?
Svelte seems to be favoring classes in their implementations. For example there's now class versions of Spring and Tweened, there's MediaQuery which is a class and there's others. I think we should implement with classes where we can for hooks and utility things.
Getter Parameters
both hoverStyle and unhoverStyle can now be getter functions which means we can get rid of the writables since now you can do something like this
const style = $state('move') // cursor style
const hovering = new Hovering(() => style)

in the snippet above, when hovering is true, the style inside the state is accessed, so the cursor style is set to move. because its a getter, anytime the style updates, the style updates with it. There's an example of this in the new mdx page.
I renamed the parameters to indicate better what the do. Before they were onPointerOver and onPointerOut which felt like they were event callbacks when in reality they are strings or, now, getters of strings.
Removing onPointerEnter and onPointerLeave
One big difference is the removal of the onPointerEnter and onPointerLeave methods. both of these were just convenience methods that essentially did the following
const onPointerEnter = () => {
  hovering = true
}

const onPointerLeave = () => {
  hovering = false
}

these are so simple that i don't think there's anything to be gained from putting them on the Hovering class. The user can create their own onPointerEnter and onPointerLeave callbacks if they need reusability. Something like the following:
<script>
import { Hovering } from '@threlte/extras'

const hovering = new Hovering()
const onPointerEnter = () => {
  hovering = true
}

const onPointerLeave = () => {
  hovering = false
}
</script>

<T.Mesh onpointerenter={onPointerEnter} onpointerleave={onPointerLeave} />

This means that everywhere there was a destructure and usage of onPointerEnter/onPointerLeave is now replaced with inline functions that do the same thing.
All the examples have been updated.
Other Changes
I made a couple of changes to some examples to modernize them. I tried not to do too much, just cleaned up some things that were easy to clean up.

oops forgot to pull next before branching. will fix that.

gonna close this one and make a new one with better ideas


Alertmanager integration with Discord

Discord is the new Slack/Hipchat for many organisations, including for one of my clients.
This ticket is to cover adding Discord support, based on the Slack/Hipchat implementations.

I've done most of the work here...
https://github.com/rossigee/alertmanager/tree/add-discord-support
It largely works, and when configured will send a brief message to Discord via the configured webhook URL, but the notification generation code is still lacking, so the main detail from the alerts aren't being passed through.
This is my first time using Go. It seems pretty straightforward so far, but It's taken me a few hours to get this far (mostly toolchain issues), and now I'm struggling with simple stuff, like debugging/developing the part that iterates the array of alerts to produce a set of embeds for the discord notification. If anyone with a better understanding of Go (and alertmanager) could help me figure out how best to generate the notification content, that would be massively appreciated.
Only other thing of note is that it's also failing one of the retry tests for HTTP response 429, but I haven't been able to figure out why.

Unfortunately as stated in the documentation, no new receiver is being added to AlertManager for the time being. The recommended approach is to use a standalone service that will get notifications from the webhook receiver and push it to the third-party service. It seems that https://github.com/benjojo/alertmanager-discord already does this for Discord. If it suits your needs, it can be listed in the Prometheus documentation on the integrations page.
Again thanks for the effort you've put into this.

Thanks for bringing that to my attention @simonpasquier.
Unfortunately benjojo's service doesn't suite me too well. As it stands, it has no run-time configurability, no build script/docs, no packaging etc. It's a bit too raw, and my lack of experience with the Go toolset means that I won't be in a position to finish cooking it any time soon.
For now, I've created a simple NodeRED flow which works well for us using the webhook notification mechanism.
To be honest, if every organisation that uses Discord and wants to use Alertmanager has to find/write and maintain yet more webservices on their infrastructure just so Alertmanager can send them alerts, that's going to add up to a lot of technical man hours that could be put to better use, plus a lot of people that probably wouldn't bother with alertmanager at all. Especially if users of Hipchat and Slack can achieve the same goal with a much simpler setup, but the door is closed for Discord users to be able to use it in the same way.
Is there a particular reason you don't want to accept any more mechanisms? It seems to me it might be because to implement a mechanism for every new IM platform that ever popped up would involve making the monolithic 'notify/impl.go' and the template file even more bloated/kludgy? If it's not already obvious, you should really probably consider breaking the different notification mechanisms etc in that file out into their own files, folders (or repos) to make it all more modular and generally easier to work with and extend.

Is there a particular reason you don't want to accept any more mechanisms?

Every new receiver increases mechanically the maintenance burden. The history has shown that while people are willing to contribute new receivers, it is often difficult for them to follow up. And it isn't only about implementing the notification code but also extending the AlertManager's configuration as well as writing the documentation. If you look at the sheer number of IM platforms, it would be impossible to support all of them and a line has to be drawn somewhere. You check #701 and #1259 for previous discussions on the topic. Again I definitely understand and share your frustration.

@rossigee just wanted to point out that discord supports slack compatible webhooks. Any service that allows you to specify a (complete) slack webhook endpoint allows you to use discord too. In your discord channel create a webhook and retrieve the url; then just append /slack to the end.
This should work with the current prometheus integration as well.

@luceos do you have a working an example?

Has anyone gotten this working? I have used @luceos suggestion in the past, however, it does not seem to work with the slack_config in alertmanager.

this shit does not work at all


Add RaycastResult sorting on RigidBodyComponentSystem#raycastAll

Fixes #5162
Implements RaycastResult distance sorting for RigidBodyComponentSystem#raycastAll. As API was never documented as having the results sorted by any manner I don't think that would count as breaking. I also exposed hitFraction which was used for the sorting and is a very useful information to know where the hit occurred.
Changes to RaycastResult public API:

+ hitFraction: The fraction the hit happened. Can be used to know which one came first based on distance.

I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

Looks great to me, but now that the results are sorted, we should document that somewhere so that users can depend on it. Otherwise we're sorting for no reason.

Done, thank you!

Nice! 🙌

Just noticed this. but do we have an option not to sort it? I mean its an extra step that not always needed.

Nope, but is it really a big deal? I was actually working at the same time on #5180 which is adding a new parameter to the function so I wanted to keep it quite short and simple rather than adding an extra parameter for this.

I would prefer to go for the simplest and most performant as a base, and any additional functionality that adds a cost would be optional. I do not see a justification to have a sorting enabled by default. We pass additional options to all sorts of methods across the engine, it should be fine to do so here as well. Can it not be part of the options object that adds filtering?

Makes sense, there you go then: #5181

I'm with LeXXik, sorting isn't always desirable so now there is extra unwanted cost to raycastAll


gpg invocation not working

for some reason the gpg invocation suggested here: http://backup.github.io/backup/v4/encryptor-gpg/#decrypting
doesn't work:
$ gpg -d github.tar.gpg -o github.tar
usage: gpg [options] --decrypt [filename]

even though the manpage for my gpg version does say that the -o flag can be used for specifying the output file.
This is successful:
gpg -d github.tar.gpg > github.tar

Hello John,
Could you specify the OS and GPG versions that you used to encrypt and decrypt?

Ubuntu 14.04.5
gpg 1.4.16

Thank you for that.
It seems that the order of the flags is significant, at least on that version of GPG. This should work:
gpg -o github.tar -d github.tar.gpg
I'll investigate a bit further, and put in an update to the documentation.

I've amended the documentation. Better late than never, I suppose.


[feature update]I want to upgrade a few features to make them more friendly

The features of the upgrade are as follows：
1. Support for cross anchor, sidenote of props of InlineAnchor supports for assignment array

click cross anchor can activate the included side notes in turn.

Usage

2. When the size of the side note changes, it will be positioned automatically.
before ↓

after ↓

3. Similar to the previous point，when the position of the anchor changes, the position of the side notes will also change.（This is very necessary in the editor.）
before ↓

after ↓

4. When all anchors are removed, the corresponding side notes are also hidden.

5. When initializing, sidenodes is positioned to the top anchor by default, not at the bottom.
before ↓

after ↓

TODO:
Optimize the lookup and delivery of docId; Or search for custom docId


Improve CUDA 'no NVCC' warning

https://github.com/deeplearning4j/nd4j/blob/master/nd4j-backends/nd4j-backend-impls/nd4j-cuda/src/main/java/org/nd4j/linalg/jcublas/JCublasBackend.java#L53
"CUDA backend can't find NVCC within PATH environment variable"
This warning could be improved:

Let users know what NVCC is (users without CUDA experience might not know what NVCC actually is)
Tell users they need to have CUDA installed
Tell users that NVCC needs to be on the system path, and is usually found in locations X or Y on their system.
Ideally: add a website link that explains exactly how to do this.

I have better proposal.
We don't need nvcc anymore since rc3.9, maybe let's just get rid of that check? We have javacpp which checks for cuda libraries availability anyway. So if there's no cuda libraries - we'll see it.

I thought maybe we might want to compile kernels on the fly at some point in the future, in which case we would need NVCC? Then again, maybe not, wouldn't be practical for users anyway...

I would be ok with raver's suggestion.

@saudet we had that before rc3.9, we were compiling them once. But we got rid of that before that huge release.
So ok, since everyone is happy i'll remove it :)
P.s. i'll be missing this complaints in general chat :)

Epoch gone.


URLs should preserve the filesystem filename

Right now the URLs to articles come from the title of the article - in this way when you edit the article its URL changes, and links break.
Instead, the links should keep the filesystem filenames.

@edne is it right?

Right!


Custom RBAC role / Permissions required to view VM Insights metrics

We are using Centralized Log Analytics setup , multiple teams logs their data into single workspace.
We would like to setup custom RBAC role to allow the view VM Insights metrics for their VM alone . Log Analytics reader role gives them permission to view logs from all resources , we want to restrict custom RBAC role for VMInsights alone.

@kgopi05
Hi Gopi,
Please help with the link of document on which above document feedback request is created.
In case your query in not related to document content, open a new forum thread in Microsoft Q & A or Stack Overflow as it could be better place to get help on your scenario. These forum community members could provide their expert guidance on your scenario based on their experience. Thanks.

@kgopi05
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.


[Question] What is clEntityId value for Get closed list entity info in LUIS API

Issue Description
My HTTP request URL is https://westus.api.cognitive.microsoft.com/luis/api/v2.0/apps/5290faa2-2579-4547-815c-ae52ae35d7d6/versions/0.1/closedlists/DetailsType
where I used clEntityId as a custom entity name of a type List. When I run this using API testing Console I get the error as "Message": "The request is invalid."
May I know where I can find this id or there is something else that is causing this error?
Reproduction Steps

Expected Behavior
I'm expecting Entity list in JSON format in my node js application in below format [sample]

Actual Results

@GauravDhavale, you're receiving this response because the API is expecting a GUID like e58c####-####-####-####-############, not the List Entity's name. To get the GUID for your DetailsType entity, you can use this programmatic API to retrieve all the entities' GUIDs for your current version.

Hi, @stevengum97 I tried the given API reference but it gives me the GUID of the only list of entities of simple type. I want the list Entity type closed list entities. How can I find the GUID for closed list entities?
What I got :

@GauravDhavale sorry about that, try this API instead models - Get application version closed list infos.

Thanks. Got the GUID as well as the closed entity list.


[ParseableInterface] Hack: prefer module names over type names

...in the parts of a .swiftinterface file that are pretty-printed.
The effect of this is to dodge a longstanding issue (SR-898) where if you have a module and type with the same name, the compiler always prefers the type. This is a problem for swiftinterface files, which are pretty-printed to use qualified types everywhere (to minimize the chance of collisions when a library changes). We know that one of these types exists in the wild, too: XCTest, the base class of XCTestCase.
This does not fix SR-898; it only works around it in certain limited cases by making the opposite assumption: preferring a module name instead of a type name. We also considered having some special syntax that indicates that the first component of a qualified name is a module, but felt that that might actually be a feature of the language some day, and wouldn't want to end up with two syntaxes for the same thing (or worse, have subtly different semantics for the one used in swiftinterfaces, or have it conflict with something else).
Builds on #22830.

@swift-ci Please test

The API digester changes are part of #22830, not something additional here. I asked Xi what he thought about that over there but he hasn't replied yet.

Ah, okay. Thanks!

Build failed
Swift Test Linux Platform
Git Sha - c218f5f0d9d0be0ea132e6d7b530b778b3a365e2

I wonder if we can come up with a more universal rule here. If you have a single type component, like Foo, we treat it as a type. If you have Foo.Bar, we treat Foo has a module. It would be interesting to see if that passes source compat.
I'd much prefer a single rule over adding yet another *%(#%& flag to NameLookup...

This rule would definitely break source compat. All it takes is a nested type in the type that has the same name as your module. (Also, I don't want to tie parseable interfaces to a language change.)
I share your distaste, though. I suspect the real answer will be that some day we'll have an actual syntax for module qualification and then this will only be legacy support.

Build failed
Swift Test OS X Platform
Git Sha - c218f5f0d9d0be0ea132e6d7b530b778b3a365e2

I just finished a refactoring of UnqualifiedLookup. I haven’t had a chance to look at the code yet, but I’ll chime in to agree with Jordan’s discomfort and Slava’s reservations. If there is any other way to meet the larger goals, it would be better to avoid more options in that code, which already has much that seems inexplicable or opaque, particularly at that module scope level.

David

On Feb 28, 2019, at 8:08 PM, Jordan Rose<EMAIL_ADDRESS>wrote:
This rule would definitely break source compat. All it takes is a nested type in the type that has the same name as your module. (Also, I don't want to tie parseable interfaces to a language change.)
I share your distaste, though. I suspect the real answer will be that some day we'll have an actual syntax for module qualification and then this will only be legacy support.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I haven't thought of anything that doesn't require inventing syntax, though, and that definitely seems worse (even less maintainable).

@jrose-apple I've added myself as a reviewer, since I just refactored some of the name lookup code. I'll be happy to help figuring out how to make the change in the refactored world, assuming you go through with it. Does it make sense to split up the PR, so the nameLookup part goes in first?
Also, FYI, in a day, I'll be submitting a PR to remove the legacy-cross-check. The change in this PR will be easier to put in after that happens.

Thanks! I have the PR split up the other way ("print everything qualified"), and that first part still has bugs in it, so I might not get back to this right away.

My pleasure! Good luck!!


chore: update binary urls

Switch to current version (0.24.12)

More updates coming and it seems this version doesn't use Core 19.2 image


Development grid stack

Related Issue

#655

Work Done

Updated the full vm grid stack guide as requested in the issue mentioned above

@A-Harby LMK if it's ok! Some stuff Bert will update on the grid_deployment's side, e.g. updating scripts to show if a file has been properly downloaded.

Last thing, but could you just add a note anywhere before the script about the downloading and the extracting time of the 3 files, just to let the user know it takes a long time to download and extract each?
It takes hours with nothing shown, so the note is very necessary to let the user know before starting the script.

Last thing, but could you just add a note anywhere before the script about the downloading and the extracting time of the 3 files, just to let the user know it takes a long time to download and extract each?

Is this sufficient?


Restructure benchmark framework, split runners and measurments

What
Restructure the bench framework.

Split up responsibilities ownership between the host and benches
Clean up module tree in the benches
Add a couple of new host cost measurement types

This PR splits the previous HostCostMeasurement into two places:

soroban-env-host/src/cost_runner: contains the runner of each cost type, as well as the structural definitions of the sample data.
soroban-env-host/benches: contains the data generator for each cost type.
Information dependency goes from 2-->1 but not the other way around.

Why
The main reason is to run non-public functions in the host for calibration.
The current calibration/benchmark framework consist of roughly three parts:

Generation of data, whose structure requirement is defined by a particular cost type.
The bench harness, dispatchers for kicking off a measurement for a (bench type, cost type, iteration) combination.
Running a cost entity in the host.

Currently these all sit with the "benches", which is outside of the host crate.  However, the host runner should belong to the host, since it needs access to the (non-public) host components. Same for the data structural definitions.
Known limitations
The actual cost measurements are not finalized:

Missing many cost types.
Many cost type measurements do not go through the host (which is fine for experimental but not long term). Some measurements do not truly reflect what the host does.

@graydon I pushed another change to construct and pass in Vec<SampleType> instead of &mut SampleType to the runner. Mainly because we want to remove internal cloning from the measurements. Now it constructs a vector by cloning sample data from the outside and then passes it into the runner to run the host, without accounting for sample cloning.
During this change I discovered an issue. Previously the the mem_tracker start happened right before calling the HCM::run, and it was underflowing the AtomicU64 counter after I made the above switch. I believe it's because the memory deallocation of the sample data container was being counted while the allocation (which happens before the tracker started) was not. I moved that call before the data setup, and it seem to give the correct results. I've left a comment in the code. Please let me know if my assessment is correct and if there is a better pattern to solve such issue (rearranging the code this way feels too subtle and error-prone).

@graydon I pushed another change to construct and pass in Vec<SampleType> instead of &mut SampleType to the runner. Mainly because we want to remove internal cloning from the measurements. Now it constructs a vector by cloning sample data from the outside and then passes it into the runner to run the host, without accounting for sample cloning.
During this change I discovered an issue. Previously the the mem_tracker start happened right before calling the HCM::run, and it was underflowing the AtomicU64 counter after I made the above switch. I believe it's because the memory deallocation of the sample data container was being counted while the allocation (which happens before the tracker started) was not. I moved that call before the data setup, and it seem to give the correct results. I've left a comment in the code. Please let me know if my assessment is correct and if there is a better pattern to solve such issue (rearranging the code this way feels too subtle and error-prone).

Addressed in https://github.com/stellar/rs-soroban-env/pull/561#discussion_r1019815009


(Question) configuring dismiss options for custom modal

Hi,
I have implemented a custom bootstrap modal but I can't see how to configure the options to allow/disallow pressing escape and/or clicking outside the modal to dismiss it. Hopefully this is straightforward but I haven't been able to find it.
Thanks

I think you are looking for .isBlocking(false)
example:
this.modal.confirm().message("Are you sure?").isBlocking(false).open();
pressing ESC did not dismiss ,but clicking outside the modal did.

Thanks but this is a custom modal using this.modal.open() which doesn't have the isBlocking option

In version 2 you create a custom modal using an OverlayConfig as a parameter.
See it in the demo
You can then easily add the isBlocking:
          const builder = new BSModalContextBuilder<CustomModalContext>(
            { num1: 2, num2: 3 } as any,
            undefined,
            CustomModalContext
          );

          let overlayConfig: OverlayConfig = {
            context: builder.isBlocking(true).toJSON()
          };

          return this.modal.open(CustomModal, overlayConfig);

Thank you.  A very useful library  :+1:


Fixed error thrown by a configuration with a relative path to models

I pushed a new commit on my fork to fix the issue #20.

@derbaeuerle Thanks for the fix! LGTM!
We recently added the generate-changelog module to this repo so that we don't have to worry about manually creating the changelog. We'll be updating the README so that it's more clear when contributing, but until then, can you amend your commit message to follow the format?
fix(paths): fixed error thrown by a configuration with a relative path to models

I can also do this if @derbaeuerle is unavailable? Thanks!

Not sure, but it's fine by me. If you want to fix the message @slester, I'll happily merge and publish a new version!

Thanks for doing the work to get a fix for this @derbaeuerle. @slester put up another PR with your fixes with a fixed commit message so now the fix has been released in version 1.7.1.


pythonPackages.pandoc-xnos: init at 2.5.0

Description of changes
Add pandoc-xnos the Python library for the pandoc-fignos, pandoc-eqnos, pandoc-tablenos and pandoc-secnos Pandoc Python filters. I plan to package those filters once this library is added.

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

There is already an open PR (#164328) which seems to cover the same set of packages.

Woops I missed it. Thanks.


How to set schedule for each rule

Hi member, I want to run my different rules in different schedules.
e.g.
ruleA : every 5 min
ruleB : every 1 day
ruleC : everey 7:00 o'clock
...
but I find it not use in config run_every in each rule.yaml ---- only the one config in config.yaml is effective.
I want to edit some code to implement my requests. Could u please give me some information about the process of program execution.

This feature is not merged https://github.com/Yelp/elastalert/pull/1298

May I know that when will this be deployed ?
I want to mimic the below watcher alert schedule in elastlaert. Thank you .
{
"trigger": {
"schedule": {
"interval": "2m"
}
},


add field-specific debouncing

This PR address #186 and adds the possibility of adding field-specific debounce times via
# set a debounce time of 10 ms
@debounce myfield 10

# set a debounce time of 10 ms for an explicit app
@debounce MyApp myfield 10

# remove the field-specific debounce time for the field `myfield`
@clear_debug myfield

# remove the field-specific debounce time for a field of an explicit App
@clear_debug MyApp myfield

# remove all field-specific debounce times
@clear_debug

# remove all field-specific debounce times from `MyApp`
@clear_debug MyApp

# In the good old function way ...

# set a debounce time of 10 ms
Stipple.debounce(MyApp, :myfield, 10)

# remove the debounce time
Stipple.debounce(MyApp, :myfield, nothing)

tests and docstrings are provided

Awesome, thanks! Looks good to me! (I see that it's @clear_debounce not @clear_debug :) ).

Awesome, thanks! Looks good to me! (I see that it's @clear_debounce not @clear_debug :) ).

I'll change this, as it will appear in the release notes ...


hello world js failing

Hello, first time arango user (experienced admin/dev). Arango 3.1.1 starts fine. Trying to get a hello world js script going but a confusing error results. Can someone explain what the problem here is? There are no databases created, just trying to get an empty list or some sort of positive response from arango.
arango.js
'use strict'
var arango = require('arango')

let db = arango.Connection()

console.log('db list')
db.database.list().done(function(res){
  console.log("collections: ");
})

db list

/home/donp/code/cointhink/superpoll/node_modules/arango/node_modules/micropromise/node_modules/microtask/index.js:60
                    throw e;
                    ^

Try
var arango = require("arangojs");

Can you clarify where you found a tutorial that tells you to use Connection? I'm guessing it's extremely out of date. The API has changed between the major versions of arangojs in backwards-incompatible ways (hence the major version changing) and there are still examples out there for older versions that don't explicitly pin the version number of arangojs.
Also it looks like you're following a tutorial for arango not arangojs. The former was developed by a member of the community and doesn't seem to be maintained any longer: https://www.npmjs.com/package/arango

I was following https://www.npmjs.com/package/arango#quick-start
Thanks for the help. I'm using arangojs now. Perhaps an arango dev could contact the maintainer of arango to add a big README warning about how that client has been abandoned.

It actually says:

which in turn states:

Yeah, the message could be clearer and it would be nice if it used the deprecation feature of npm.
But @andelo doesn't seem to be active on GitHub anymore and I'm not sure whether he's still doing anything with ArangoDB.


[BUG]: pip install -r requirements.txt

The following error is occurring when running the command 'pip install -r requirements.txt'
error: subprocess-exited-with-error
× python setup.py bdist_wheel did not run successfully.
│ exit code: 1
╰─> [12 lines of output]
running bdist_wheel
running build
running build_py
creating build
creating build\lib.win-amd64-cpython-310
creating build\lib.win-amd64-cpython-310\pysqlite3
copying pysqlite3\dbapi2.py -> build\lib.win-amd64-cpython-310\pysqlite3
copying pysqlite3_init_.py -> build\lib.win-amd64-cpython-310\pysqlite3
running build_ext
Builds a C extension linking against libsqlite3 library
building 'pysqlite3._sqlite3' extension
error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for pysqlite3
Running setup.py clean for pysqlite3
Successfully built document PyPika
Failed to build pysqlite3
ERROR: Could not build wheels for pysqlite3, which is required to install pyproject.toml-based projects

You can try pip install -e . instead.

You can try pip install -e . instead.

I tried. However, it didn't work either.

You can try pip install -e . instead.

I tried. However, it didn't work either.

It seems there is an issue with your local environment. You can try running the following command first：
pip install --force-reinstall pysqlite3
Then run pip install -e . . hope this method can solve your problem~


bad filename with colon, stops windows users from cloning

bad filename with colon, stops windows users from cloning

Hi @shaneholloman, thank you for your contribution! Would we not also want to change the 2:4_w4a16_group-128_recipe.yaml file?
From my grepping, I only see these two files as being files with colons in the name
./examples/quantization_24_sparse_w4a16/2:4_w4a16_recipe.yaml
./examples/quantization_24_sparse_w4a16/2:4_w4a16_group-128_recipe.yaml

Also note that we'll want to change all references to these file names in the following files
grep -rl '2:4_w4a16' src examples tests docs utils
examples/quantization_24_sparse_w4a16/README.md
examples/quantization_24_sparse_w4a16/llama7b_sparse_w4a16.py
tests/examples/test_quantization_24_sparse_w4a16.py

yes, any file whith colon will stop windows users from propoer git functionality

#846


Initial code for example attestation signing

Per request from Weiwu I implemented code for attestation signing and verification in the lib folder. I made a test-class that should easily show how to use it

@jot2re I will rebase this, the PR has gotten stale. I think your changes are mostly local to one new file, right?

@jot2re I will rebase this, the PR has gotten stale. I think your changes are mostly local to one new file, right?

The changes I made were only additions (modulo my IDE updating .gradle files, but Weiwu reverted these unintended changes). However, I am not sure how pervasive the changes made in this branch, before my additions, are.

@jot2re I cherrypicked & placed into a new PR, all good.


ModuleNotFoundError: No module named 'models'

I get the following error:
parsely_slackbot
Traceback (most recent call last):
  File "C:\Program Files (x86)\Python37-32\Scripts\parsely_slackbot-script.py", line 11, in <module>
    load_entry_point('parsely-slackbot==1.1.0', 'console_scripts', 'parsely_slackbot')()
  File "c:\program files (x86)\python37-32\lib\site-packages\pkg_resources\__init__.py", line 487, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "c:\program files (x86)\python37-32\lib\site-packages\pkg_resources\__init__.py", line 2728, in load_entry_point
    return ep.load()
  File "c:\program files (x86)\python37-32\lib\site-packages\pkg_resources\__init__.py", line 2346, in load
    return self.resolve()
  File "c:\program files (x86)\python37-32\lib\site-packages\pkg_resources\__init__.py", line 2352, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "c:\program files (x86)\python37-32\lib\site-packages\parsely_slackbot\__init__.py", line 1, in <module>
    from .slackbot import SlackBot
  File "c:\program files (x86)\python37-32\lib\site-packages\parsely_slackbot\slackbot.py", line 8, in <module>
    from parsely import parsely, models
  File "c:\program files (x86)\python37-32\lib\site-packages\parsely\parsely.py", line 1, in <module>
    from models import Post, Author, Section, Topic, Tag, Referrer, Shares
ModuleNotFoundError: No module named 'models'

Windows Server 2016, python 3.7 32bit

This is because our underlying Python API was not updated for Python 3 compatibility. Parsely/python-parsely#8 should fix that once it's merged and we cut a new release.

Not quite released yet. Will close when it is.

python-parsely 1.5.0 is out now, so this should no longer be a problem if you upgrade it.


roblox not opening

it fully broke when i opened the 2020 menu
if you may ask the log
i cant find it

i cant find it

There's a button that says "Locate log file".

Issue has been awaiting clarification for over a month, closing as stale. If you have anything further to add, just respond and I'll reopen.


linting broken when adding resolve.alias to webpack

I added resolve.alias:
// webpack.config.base.js
resolve: {
  extensions: ['.js', '.jsx', '.json'],
  modules: [
    path.join(__dirname, 'app'),
    'node_modules',
  ],
  alias: {
    '@fortawesome': path.join(__dirname, 'app', 'vendor', '@fortawesome'),
  },
},

The alias is working, and app is working fine, but the eslint is broken:
$ npm run lint

><EMAIL_ADDRESS>lint /Users/goldy/apps/zenwallet
> cross-env NODE_ENV=development eslint --cache --format=node_modules/eslint-formatter-pretty .

Cannot find module '/Users/goldy/apps/zenwallet/app/webpack.config.eslint.js'
Error: Cannot find module '/Users/goldy/apps/zenwallet/app/webpack.config.eslint.js'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:548:15)
    at Function.Module._load (internal/modules/cjs/loader.js:475:25)
    at Module.require (internal/modules/cjs/loader.js:598:17)
    at require (internal/modules/cjs/helpers.js:11:18)
    at Object.exports.resolve (/Users/goldy/apps/zenwallet/node_modules/eslint-import-resolver-webpack/index.js:68:25)
    at v2 (/Users/goldy/apps/zenwallet/node_modules/eslint-module-utils/resolve.js:94:23)
    at withResolver (/Users/goldy/apps/zenwallet/node_modules/eslint-module-utils/resolve.js:99:16)
    at fullResolve (/Users/goldy/apps/zenwallet/node_modules/eslint-module-utils/resolve.js:116:22)
    at Function.relative (/Users/goldy/apps/zenwallet/node_modules/eslint-module-utils/resolve.js:61:10)
    at remotePath (/Users/goldy/apps/zenwallet/node_modules/eslint-plugin-import/lib/ExportMap.js:379:30)

on Atom, linting on files that import '@fortawesome' are broken as well.
Any ideas why is that happening, and how can I fix that?
Do you have an example of a project that uses this boilerplate and has resolve.alias in webpack.config.base.js?

I spent quite a while on this and found no way to get resolving of aliases to work. If you do find a solution though, I would be very glad to hear it.

@Slapbox I prefer to explicitly import it wherever I'm using it, and not have <FontAwesomeIcon /> out of nowhere in my app.

@amilajack have you tried babel-plugin-module-resolver?
I also tried resolving webpack path in .selintrc and didn't work for me (need to convert to .eslintrc.js file to try this)
I did NOT try re writing webpack file in ES5
Don't know if it's related: in our webpack.config.eslint.js we export module.exports = require('./webpack.config.renderer.dev'); and NOT the webpack.config.base.js file. Why is that?

The base config doesn't have the configuration for styles

If you use .eslintrc or .eslintrc.yml please rename it to .eslintrc.js and change webpack resolve function to something like it:
webpack: {         config: resolve('./webpack.config.eslint.js'),       },
In my case it fixed the problem

@Mati365
this works, but breaks atom-linter, see issue, tho it has a workaround I tested and it works.
so might be worth while to combine the two approaches?
You can see how I implemented it for atom here

A word of warning: if you're using VSCode the .eslintrc.js method seems to break the eslint extension because the eslint server does not have NODE_ENV set, and the CheckNodeEnv('development') in the webpack.config.renderer.dev.js file referenced by eslint-import-resolver-webpack requires it. My temporary fix is just to delete that line, but a better solution might just be to stop using the eslint-import-resolver-webpack plugin entirely, and replace it by defining the same resolution options as in webpack in the import/resolver settings of the .eslintrc. I know that this method seems repetitive, but we already have to do that for the .flowconfig so I don't think it's that bad. Plus, it gets rid of a dependency :)


Updated httpfs documentation

Added some updated documentation for the httpfs extension.

@samansmink can you add the new pages to the menu json?


Pass --local always when in GitHub Actions environment

See discussion in: https://github.com/Homebrew/homebrew-core/pull/113506
Closes: https://github.com/Homebrew/homebrew-core/pull/113506

MacOS CI fails, can't figure out why.

@dawidd6 it's trying to write logs somewhere it doesn't have permission to:
    Errno::EPERM: Operation not permitted @ rb_sysopen - /usr/local/Homebrew/Library/Taps/homebrew/homebrew-test-bot/logs/testbottest/post_install.01.testbottest

(it's also a bad idea to be writing logs to e.g. inside /usr/local anyway)

@dawidd6 We shouldn't be putting bottle files there. They should be in e.g. $HOME

I'm a bit concerned with the compatibility of this change. Particularly since most taps will be using brew test-bot (we configure it in brew tap-new).
If this is requiring changes to workflows then I don't think it's worth it and could be passed in Homebrew/core like we do with many other things like --junit. Having to change the workflow here is already an indicator.

Honestly the main reason --local is forced on this side is because of the non-ephemeral nature of our self-hosted runners, so that conditional can probably go when fully ephemeral.
Beyond that it's only useful because of the specific way we upload artifacts on Homebrew/core, so probably makes sense controlled there.

Yea I agree, we should fix this on the core tap level rather than here. Imma close this PR and go back to the previous one there.

@dawidd6 Could you reopen just to remove the TODO? Thanks 🙇🏻

https://github.com/Homebrew/homebrew-test-bot/pull/845


Shorthand version of (e == null) ? null : f(e) as e ?: f(e)

From Pavel Minaev comment  February 25, 2014 10:40 pm on http://visualstudio.uservoice.com/forums/121579-visual-studio/suggestions/3990187-add-operator-to-c-
While we're at it, please also add a shorthand form of ?: where the null check is implied one branch is implicitly null. So that e.g. instead of:
(e == null) ? null : f(e)

we can write:
e ?: f(e)

So, basically, a generic "bind" for the nullable monad.

Now C# has this with the ?? operator and also ??=


Clean up geo-dash.ftl

geo-dash.ftl:
Lines 9-27: Why is this Javascript code in the FTL file rather than in geo-dash.js?
Line 9: let -> const
Line 10: var -> const (don't we already call this.getParameterByName("projectId") in geo-dash.js?)
Line 11: This function needs to be rewritten to use const, ?:, and not overwrite the input arguments to the function (impure).

The vars on 9, 10 dont look like they are being used.
Is this function still needed or is it done in the geo-dash.js file?

@sirmspencer None of that is needed, it was legacy code from pre-react time.  It is now removed in PR #359


False positive detected on virus total and ESET NOD32, etc

Virus detected. Novice users are unable to determine if it is safe. Also you seem to have an imposter, bitcore.io

Can you give more infos? Thank you.

This also includes Sparta based Bowsers and Sophos

Maybe you can upload to virustotal and provide a link.


Fixed the init_test by comparing

Fixed the main_init_test.go
Description

What does this PR do?

It just fixed the uncertainty error we get while testing
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New chore (expected functionality to be implemented)

Checklist:

[ ] My change requires a change to the documentation.

[ ] I have updated the documentation accordingly.

[x] I've signed off with an email address that matches the commit author.

after running the or i in {1..10}; do                                    echo "Run #$i" >> test.txt     go test -timeout 30s -run ^TestInitWithProvisioners$ -count=1 github.com/score-spec/score-k8s >> test.txt 2>&1 done
it passed

Thanks, @7h3-3mp7y-m4n!
We have been seeing random issue in the CI with this unit tests, like here for example:https://github.com/score-spec/score-k8s/actions/runs/11355384862/job/31584604052.
So if this is fixing this issue, thanks!
I would love your review on this, @delca85, any feedback/comment on this?

I ran it with the script and tested it 15 times and voila, no errors :)


Microphone not working

Microphone is not working. I get an error whenever i generate it. Please fix it.

ValueError: An event handler (transcribe_mic) didn't receive enough output values (needed: 2, received: 1).
Wanted outputs:
[<gradio.components.textbox.Textbox object at 0x7fd4a8932a10>, <gradio.templates.Files object at 0x7fd4a8932bc0>]
Received outputs:
[None]

Hi @tingtongdpnw.
I could not reproduce the same bug with the latest version of the web ui.
Can you try recording with the latest version of the web ui and see if the same bug happens?
You can update the web ui with:
git pull origin master


[Enhancement] Add standard unit of time check for batch build partition

Signed-off-by: Astralidea<EMAIL_ADDRESS>What type of PR is this：

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

Which issues of this PR fixes ：

Fixes #17320
Problem Summary(Required) ：

Checklist:

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

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

Bugfix cherry-pick branch check:

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

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

[FE PR Coverage Check]
:heart_eyes: pass : 39 / 45 (86.67%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/ast/MultiRangePartitionDesc.java
34
40
85.00%
[192, 199, 206, 214, 222, 226]

:large_blue_circle:
com/starrocks/common/Config.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/ast/RangePartitionDesc.java
4
4
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test


Fine-tuning meta-llama/Llama-3.2-11B-Vision-Instruct - ValueError: The number of images in each batch should be the same. Yes, the model does not  support having a different number of images per batch.

I tried using the fine-tuning script given in HF  for the model "meta-llama/Llama-3.2-11B-Vision-Instruct" but I am getting an error "ValueError: The number of images in each batch [1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1] should be the same  [1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1] should be the same. Yes, the model does not                         support having a different number of images per batch.".
As you could see above both arrays are equal and same but it still throws value error with line 308 in transformers\models\mllama\processing_mllama.py:308 file
Also when i print
n_images_in_text = [t.count('<|image|>') for t in texts]
n_images_in_text[0] - it prints 1
n_images_in_images = [len(sample) for sample in images]
n_images_in_images[0] - it prints 1
When i give single rows it works like
batch = processor(text=texts[90], images=images[90], return_tensors="pt", padding=True).to(model.device) ## This Works
batch = processor(text=texts, images=images, return_tensors="pt", padding=True).to(model.device) #just trying with 1st 100 rows ##This doesnt work
My First row in texts and images at index 0 are as below
texts=['<|begin_of_text|><|start_header_id|>user<|end_header_id|>\n\n<|image|>What type of tissue is shown in the image?<|eot_id|><|start_header_id|>assistant<|end_header_id|>\n\nThe image shows a histopathological section of an endonasal mass, which is a tissue sample taken from inside the nasal cavity.<|eot_id|><|start_header_id|>user<|end_header_id|>\n\nWhat is the magnification level?<|eot_id|><|start_header_id|>assistant<|end_header_id|>\n\nThe magnification level of the image is 200x.<|eot_id|><|start_header_id|>user<|end_header_id|>\n\nWhat is the diagnosis based on the histopathological features?<|eot_id|><|start_header_id|>assistant<|end_header_id|>\n\nThe diagnosis is a b-cell lymphoma. b-cell lymphomas are a type of cancer that originates from b lymphocytes, which are a type of white blood cell. the image shows solid sheets of large round cells diffusely obscuring the lamina propria underlying the respiratory epithelium. this finding, along with other clinical and laboratory data, led to the diagnosis of b-cell lymphoma.<|eot_id|>',...]
images - [<PIL.Image.Image image mode=RGB size=1457x1000>,...]
Pls assist

This was an issue with the understanding. Ignore this


Module polyfills

🐛
Current Behavior
Screenshot is from webpack-bundle-analyzer. As you can see, simplebar.esm.js imports all necessary polyfills (thanks to preset-env). But it becomes an issue if we use preset-env for our code too, many polyfills get added twice. Gzip compresses it quite nicely when sent over the network (for those of us who use compression, ~3Kb difference) but it's still ~16Kb more JS to parse.

Expected behavior
Im not 100% sure what's the correct way to handle this because Webpack is still largely a black box for me but it makes sense that if project uses esm bundle, it probably already polyfills other files and simplebar shouldn't import them. App's build process should run babel-loader on every .js file like big projects do (e.g. create-react-app), at least for production build.
simplebar.js and simplebar.min.js are already around 13Kb (after my own uglify and gzip), I presume both already include polyfills and that's okay because these are used in different environments.
Suggested solution(s)
Don't import polyfills in simplebar.esm.js. Or should there be another bundle? Again, black box..

Hi @adjourn !
The esm package just declares its dependencies (using import ...) so in theory the bundler should not import these dependencies twice! If it does, it means the bundler doesn't load the esm file but the .js :/

many polyfills get added twice.
I don't know why this is happening to be honest. I need to take a deeper look but in theory the esm version should still be pre-transpiled "ready to use" and the user shouldn't have to use Babel. Usual Webpack setup like create-react-app does not  transpile node_modules but only your source files. Every node_modules is supposed to be pre-transpiled.

The biggest deps for SimpleBar is the resize observer polyfill, I know it's bloating the size of the bundle and it's unavoidable. That's another issue but I was thinking that we could provide an option to load SimpleBar without it.
Could you confirm that some things are included twice in your final bundle? If so it's a bug. Otherwise it's just about performances optimisation.

Yes, I can cofirm that 5.56 Kb (gzip) of polyfills is included into simplebar. esm is definately being used because if I comment imports out, none of the polyfills get added twice to final bundle.
node_modules/simplebar/dist/simplebar.esm.js
/**
 * SimpleBar.js - v3.1.3
 * Scrollbars, simpler.
 * https://grsmto.github.io/simplebar/
 *
 * Made by Adrien Denat from a fork by Jonathan Nicol
 * Under MIT License
 */

// import 'core-js/modules/es6.regexp.replace';
// import 'core-js/modules/es6.function.name';
// import 'core-js/modules/es6.regexp.match';
// import 'core-js/modules/web.dom.iterable';
// import 'core-js/modules/es6.string.iterator';
// import 'core-js/modules/es6.array.from';
import scrollbarWidth from 'scrollbarwidth';
import throttle from 'lodash.throttle';
import debounce from 'lodash.debounce';

What's weird is that other dependencies get hoisted to bundle global scope (where they could be reused) but core-js dependencies stay inside simplebar scope (like shown in image above).
I don't mind resize observer polyfill that much, it's only 2.4Kb gzip. Especially compared to this issue.

I tried to find a solution for this but I'm kinda stuck.
I think it's related to that issue https://github.com/zloirock/core-js/issues/385
I need to see how other libraries/components are doing it and see if there is a solution.

@Grsmto I have some great news. I had permission issues in my folder (hello Linux), so I deleted yarn.lock file, modules directory and ran yarn command, now I don't have this issue. :tada:
I think it's safe to say that the problem was somewhere else and I have no idea where.

That's great news.
However if you check here you can see that core-js is taking a huuuuge part of the library size.
Which is weird because I don't use that many "new" functions. Feels like it's way too big, so I believe something wrong is definitely happening!

I think it might make sense, when I copy-pasta simplebar.js and compile it myself:
Code is ~4.8kB, observer 2.4kB, lodash ~2.9kB total, core polyfills ~5kB + others ~0.45kB. These numbers are probably not very accurate (webpack-bundle-analyzer) but ratios should be kind of right.
["@babel/preset-env", {
  "useBuiltIns": "usage",
  "debug": true
}]

"last 2 version", "not dead", "> 0.2%", "ie 10" (what I usually use)

Added following polyfills:
  es6.array.from { "android":"4.2", "chrome":"49", "ie":"10", "ios":"8" }
  es6.string.iterator { "android":"4.2", "ie":"10", "ios":"8" }
  web.dom.iterable { "android":"4.2", "chrome":"49", "edge":"17", "firefox":"52", "ie":"10", "ios":"8", "opera":"56", "safari":"11.1" }
  es6.regexp.match { "android":"4.2", "chrome":"49", "edge":"17", "ie":"10", "ios":"8" }
  es6.function.name { "ie":"10" }
  es6.regexp.replace { "android":"4.2", "chrome":"49", "edge":"17", "ie":"10", "ios":"8" }
  es6.object.keys { "android":"4.2", "ie":"10", "ios":"8" }
  es6.array.iterator { "android":"4.2", "ie":"10" }

"> 0.25%", "not op_mini all" (browserlist used by simplebar)

Added following polyfills:
  es6.array.from { "android":"4.4.3", "chrome":"49", "ie":"11", "ios":"8" }
  es6.string.iterator { "android":"4.4.3", "ie":"11", "ios":"8" }
  web.dom.iterable { "android":"4.4.3", "chrome":"49", "edge":"17", "firefox":"64", "ie":"11", "ios":"8", "opera":"57", "safari":"11.1" }
  es6.regexp.match { "android":"4.4.3", "chrome":"49", "edge":"17", "ie":"11", "ios":"8" }
  es6.function.name { "ie":"11" }
  es6.regexp.replace { "android":"4.4.3", "chrome":"49", "edge":"17", "ie":"11", "ios":"8" }
  es6.object.keys { "android":"4.4.3", "ie":"11", "ios":"8" }
  es6.array.iterator { "android":"4.4.3", "ie":"11" }

If it's something you want to address, I did some research:
If you replace Array.from (this has multiple listed dependencies) and find an alternative to current regexp (older version perhaps), you should only end up with web.dom.iterable (should be around 468B).

ref: https://medium.com/@WebReflection/avoiding-babels-production-bloat-d53eea2e1cbf


Confusion Matrix for far, near dataset for both training

Can you provide the confusion matrix for far and near dataset .
I wanted to check since the profiles are distance variant (from far F40 will look like F20) . It is visible in the confusion matrix.

Confusion matrix images for far and near constraints for Crossentropy, Evidential, Dropout and ensembles models.
Confusion matrix for far and near constraint.pdf
confusion_matrix_distance.zip


[Draft]make gc support multi-threads

What is changed and how it works?

Issue Number: Close #16101
What's Changed:

do parallel region gc and expose the gc thread configuration.

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)

Side effects

Performance regression

Consumes more CPU

Release note
Support configurable gc thread count.

The code above did not work. Because today's worker/runner model does not support native multi-threads as the underlying channel is MPSC. So there's only one thread to consume the tasks. We can dispatch tasks inside the GcRunner, but that requires huge code change to pass GcRunner's states to the actual workers.

The rust book has an example of where this can work: in their case, they used a Mutex around a std::sync::mpsc::Receiver (though in our case, UnboundedReceiver is not an mpsc::Receiver, but is a different implementation), and have the threads essentially race for control of receiver: https://doc.rust-lang.org/book/ch20-02-multithreaded.html

tested and it showed the GC worker could consume more than 1 core.

/test

/merge

/cherry-pick release-6.5

/cherry-pick release-7.1


Error if cursor is at the last character (empty new line) of a file

If your cursor is located at the very last line of your function, which happens to be an empty line, you can get an error from within ConnectButtonPushed function, on this line of code:
codeLine = txt(pos1:pos2);

In my case, pos2 was 61 but the size of txt was only 60 characters, so I got the error: "Index exceeds the number of array elements. Index must not exceed 60."
I can't tell if the bug is actually within positionInLineToIndex (which is returning a value past the end of the file), but ConnectButtonPushed should have a check for out-of-bounds positions.

Thanks for reporting this. It is puzzling that positionInLineToIndex will return something that is outside the bounds of text. I have protected against that now.


tenantName from config not propagated to passports OIDCStrategy

I had to put add tenantName: config.creds.tenantName to the setup of the OIDCstrategy in app.js to make it working.
passport.use(new OIDCStrategy({     callbackURL: config.creds.returnURL,     realm: config.creds.realm,     clientID: config.creds.clientID,     clientSecret: config.creds.clientSecret,     oidcIssuer: config.creds.issuer,     identityMetadata: config.creds.identityMetadata,     skipUserProfile: config.creds.skipUserProfile,     responseType: config.creds.responseType,     responseMode: config.creds.responseMode,     tenantName: config.creds.tenantName   }, ....
Also it might be useful to add the Info to your README, that you may to customize the SignIn and SignUp policy names (in index.ejs) to your B2C policy names.

This has been fixed.


Daylight saving time issue

The function endOf of 2017-10-15 returns a value greater than startOf of 2017-10-16

GMT-3h, Brazil
Chrome, Firefox and Edge - Windows 10
Chrome - Arch Linux
version 2.18.1

The code we've done as fix:
    let endOf = dates.clone().endOf('day')

    const startOfDayAfter =
      dates.clone().add(1, 'days').startOf('day')

    if (startOfDayAfter.isBefore(endOf)) {
      endOf = startOfDayAfter.add(-1, 'milliseconds')
    }

This issue was originally found on airbnb/react-dates/issues/776

Yeah, this is an old bug and it's particularly acute in Brazil because of the midnight DST. See e.g. #1990, #4164, #2749, and most importantly #3132

@icambron, after opening the issue I looked into the code and fixed it: https://github.com/moment/moment/pull/4254


Crash on save new file 1.77 on Ubuntu 16.04

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
1.77.0
7f329fe6c66b0f86ae1574c2911b681ad5a45d63
x64

OS Version:
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.6 LTS
Release:	16.04
Codename:	xenial

Steps to Reproduce:

Start vscode
Create new file (ctrl+n)
Save file (ctrl+s)

Please update to 1.77.1 and re-test.

Well..with 1.77.1 it doesn't crash.. however it does not save either. when
creating new file with ctrl+n .. save and save as on that file do not
work. I guess that is ONE way to solve the crash on save problem :)
On Wed, Apr 5, 2023 at 12:59 PM John Murray @.***>
wrote:

Please update to 1.77.1 and re-test.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/179280#issuecomment-1497829834,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAP4XB6BL7OCBFBOWQ3BDMDW7WQGDANCNFSM6AAAAAAWUJV6QA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Marius Giurgi

are you guys fixing bugs using ChatGPT ? :p
VSCode-Dev: Hey, solve the crash on saving new files problem
ChatGPT: here is the code fix (disabling the save new files feature)
On Wed, Apr 5, 2023 at 12:59 PM John Murray @.***>
wrote:

Please update to 1.77.1 and re-test.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/179280#issuecomment-1497829834,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAP4XB6BL7OCBFBOWQ3BDMDW7WQGDANCNFSM6AAAAAAWUJV6QA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Marius Giurgi

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

Here’s a recording. I disabled all extension just to make sure this lack of save is due to some extensions bugs.
The OS is Ubuntu 16.04 running in Parallels (VM) on a MacBook Pro 15 in Monterey 12.6.4. VSCode is 1.77.3
If creating the new file via CTRL+N it does not allow saving. either via CTRL+S or the menu save/saveas .. or even when selecting save on the dialog box when trying to close the unsaved file.

The VSCode 1.77.3 running natively on mac works fine.

Can you check if the issue persists with our latest insiders https://code.visualstudio.com/insiders ?

Actually it seems to be fixed now in 1.78 .. files save fine in Ubuntu 16.04 (without crashing :)

FYI.. now that I got save to work and got  a save dialog box .. I also file another (somewhat unrelated) issue (see link below).. since the dialog box VS uses in Ubuntu 16.04 sucks (hard to read) :)
https://github.com/microsoft/vscode/issues/181532


Removes redundant check

Some of the child addons (the ones that consume ember-browserify) start breaking after upgrading to 2.7.0. This fixes failures.

@ef4  / @asakusuma r?

@stefanpenner lgtm, but isn't this the same general approach as https://github.com/ef4/ember-browserify/pull/93? And if so, does the same deduping question need to be answered?

@asakusuma I somehow missed that PR. I feel like deduping is a pandora's box and requires a different PR/conversation all together.
/cc @stefanpenner

@twokul I agree that this is a separate issue, but I defer to @stefanpenner since he raised the issue in the first place.

@asakusuma 👍 👌

@twokul did you do the deeper investigation we discussed in-person. Basically the scenario described most likely does not work in the first case, this merely prevents it from exploding but does not address the underlying issue.


Upgrade trader accounts to Taproot/MuSig2

Depends on https://github.com/lightninglabs/pool/pull/370.
Pull Request Checklist

[X] LndServices minimum version has been updated if new lnd apis/fields are
used.

I addressed all comments and got almost all itests working on the server side.
This is now officially not WIP anymore :tada:

@positiveblue: review reminder

@roasbeef: review reminder
@positiveblue: review reminder

@positiveblue: review reminder

I addressed all comments except for the additional unit tests. I agree they should be added, but I'm going to work on them next week.

might need to change unit test frame work

I added a set of unit tests for the new spend paths. And I also ran the fuzzer locally for almost 2 hours and didn't find anything so far.

Ready for deployment modulo when the subasta PR is merged.  Do not deploy until then

⚠ PR will need to go back to sprint as changes have been made to MuSig2 spec regarding x-only keys
Deliverable outcome:

[ ] Modify the implemented MuSig2 spec to incoporate recent changes which will promote ISP

warning PR will need to go back to sprint as changes have been made to MuSig2 spec regarding x-only keys
Deliverable outcome:
* [ ]  Modify the implemented MuSig2 spec to incoporate recent changes which will promote ISP

Are we sure we want to wait for that? Updating to the latest spec means:

Implement new API in btcd, merge.
Implement changes to MuSig2 API in lnd, merge, release
Use new API in Pool.

I think it would make more sense to instead add a version parameter to the MuSig2 API when we implement this, so we can upgrade later.

PR will need to go back to sprint as changes have been made to MuSig2 spec regarding x-only keys

No, it can proceed as defined. As mentioned, those changes themselves are still in a draft state, with no other implementations currently tracking the change. Correctness is still retained without the changes. It would matter more in a p2p setting as you need cross implementation compatibility.

@guggero agreed, iiuc we already have the versioning in the proper places as well. An account version is the most apt, since the serialization changes will change the actual aggregate key, so making the jump a future version would be an actual account version bump.


Fix bug extend_self with private methods

Fix #6370. Cop extend self to turn into module_function should
not be invoked in case there are private methods. Because of that, the
flow of the code gets broken.
My idea: Since the switch from extend self into module_function forces you to change the code in the scenario you have private methods, I just added a small validation to check if there's any private declaration before. In case there are no private methods, proceed with the offense.
It works as well for the correction ( I am not sure if I need tests there since the warning is not written at all)
This is a very first draft PR. All refactors or wrong code is more than welcome 😄

Before submitting the PR make sure the following are checked:

[ ] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[x] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

@Drenmi I hope it looks good now 😄

Thanks for working on this! I really appreciate it! 🙇


Proactively notify user about not enough sats when attempting to Reply

Allowing the user to compose a message that cannot be sent, due to no sats, can be frustrating.
While right click permits the wallet to be opened in a new tab, most users won't think of doing that, and end up having to navigate away from the page where they had just composed their reply.
The top-level reply does not require an action, a person can just start typing.  So not sure what to do there other than perhaps detect typing in that textbox and notify if the user has insufficient balance in their wallet.   For any replies below the top level, that requires a click on Reply link. so that gives an opportunity to warn of No Sats at that point.
Did I see somewhere that the SN app remembers the message composed but not sent?   If that is true, then simply the warning about not enough sats could be modified to simply add that the message will be there after topping up.   I'm not sure this is the behavior or if I am misremembering that being how it works.

Did I see somewhere that the SN app remembers the message composed but not sent? If that is true, then simply the warning about not enough sats could be modified to simply add that the message will be there after topping up. I'm not sure this is the behavior or if I am misremembering that being how it works.

Yes, afaik it always saves them in local storage

JIT funding solves this afaict


no-unused-modules "missingExports" returns false positives

I'd expect both of these cases to count as exports:
module.exports = {} // Should probably not trigger
export * from './someFile' // Should almost definitely not trigger

module.exports = {} should definitely count.
export * from './someFile' should treat the module as if it had the same exports as someFile.

Similar issue:
foo/index.js
  export * as someName from './bar'

index.js
import { someName } from './foo';

results in "someNamed was not found in './foo'"

I just removed an eslint-directive that I noted we'd needed due to this issue, but unless ESLint is doing something weird right now, it seems like this issue may be fixed in 2.25.x
If anyone else hits this, give the issue a bump and I'll re-open it.

module.exports = {} should definitely count.

export * from './someFile' should treat the module as if it had the same exports as someFile.

Unfortunately it seems neither of these cases are actually resolved as of 2.26.0. I don't know what led me to believe they were. Perhaps I failed to re-enable the rule properly.
@ljharb any thoughts about possible causes and solutions?

Our settings key in .eslintrc.js looks like this - I wonder if it's possibly related to resolution issues? But it seems to work fine in other capacities, so I'm doubtful.
settings: {
    // 'import/resolver': 'webpack',
    'import/cache': { lifetime: 0 },
    'import/external-module-folders': ['node_modules', 'packages'],
    'import/resolver': {
      'babel-module': {},
      webpack: {
        config: './webpack_config/webpack.config.resolve.js',
      },
    },
  },

@Slapbox it might be that you're missing the node resolver - add that last, after webpack.

Sorry for the lengthy absence here.
I finally got to give that a try but unfortunately it makes no difference.
I added
      node: {
        extensions: ['.js', '.jsx', '.ts', '.tsx'],
      },

And installed eslint-import-resolver-node


Underground Jungle & Vault holy mountain items not shown correctly

The underground jungle's holy mountain shop items and spell refresh, and the vault's holy mountain shop items do not update when purchased.
The Archipelago tracker does show them as checked.  https://archipelago.gg/tracker/E368JBVURp6z5IhsKJI82A/0/29

Are other shops updating properly?

Are you using the latest release? It looks like this issue was solved in the update on May 2nd, 2023.

Yes, other locations are updating correctly, including other holy mountains.
This is the URL I downloaded from: https://github.com/ScipioWright/Noita-poptracker/releases/download/1.1.0/Noita-poptracker.zip

It has been almost a year since this release and nobody has ever brought this up until now lmao.
But yeah, the release is using the wrong version of the code.
I'll make a new release now fixing it.
Please confirm that the new release fixes your issue so I can close this after.

:D I just started playing Archipelago a week ago (although I've been playing Noita since early access), and just noticed it on a large multiworld run I'm doing with a bunch of other folks.
The underground jungle is fixed, but the vault is not.

Whoops, my bad -- forgot a few slashes. I'm at work and can't really test it before pushing it, unfortunately.
I updated the release, it should work now. Please confirm whether it does, thank you!

Unfortunately still no change, the vault's holy mountain is still showing green despite everything bought.

OK this is weird.  The tracker says I have all 5 checks, but poptracker is showing I have 4.

...I missed another slash. I really should have waited until I was home to do all this.
Should be fixed in a minute or so when I upload the release

Bingo!  Thanks for the fix. :)


🔖 release version<EMAIL_ADDRESS>

TL;DR
Bumped tauri-app version to 0.0.1
What changed?
The package.json file was changed, updating the version of tauri-app from 0.0.0 to 0.0.1.
How to test?
Check the package.json file and verify the updated version.
Why make this change?
This update marks the progression of the project and helps track changes and versions.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#120  👈
#119
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mingi3314 and the rest of your teammates on  Graphite

Merge activity

Apr 14, 11:20 AM EDT: @mingi3314 started a stack merge that includes this pull request via Graphite.


Form submitHandler endless loops

I'm facing an error in Windows 11 x64 environment, Firefox <IP_ADDRESS>
It seems there is the same error with Chrome 119.0.6045.200
With this very simple code:
$(document).ready(function () {
    $("#form").validate({
        rules: {
            "name": {
                required: true
            }
        },
        submitHandler: function (form) {
            $("#form").append("
OMG it's looping
");
            $(form).submit();
        }
    });
});

The submitHandler causes an infinite loop while sending the submit. The loop is visible only for a few if the page load is not fast.
Here you can find an example fiddle: https://jsfiddle.net/dvqczf85/

Yes you're right. My bad, I changed it because the real issue was related to a double submit element in the form, sems that if you have two submit buttons with same name (but different values), the form.submit() throws an error just after the submit.
I solved using two different names of submit buttons.


'Exception: Bad state: Arena no longer in use' in findFirst query

Stacktrace from console:

[VERBOSE-2:dart_vm_initializer.cc(41)] Unhandled Exception: Bad state: Arena no longer in use, releaseAll(reuse: false) was called.
#0      Arena._ensureInUse
package:ffi/src/arena.dart:105
#1      Arena.allocate
package:ffi/src/arena.dart:51
#2      QueryImpl.findInternal. (package:isar/src/native/query_impl.dart)
package:isar/…/native/query_impl.dart:1
#3      IsarCommon.getTxn
package:isar/…/common/isar_common.dart:94
#4      QueryImpl.findInternal
package:isar/…/native/query_impl.dart:46
#5      QueryImpl.findFirst
package:isar/…/native/query_impl.dart:33
#6      QueryExecute.findFirst
package:isar/src/query_builder_extensions.dart:197
#7      UsrDetailsDbService.getKbUser
package:kb_qa1/…/db/logged_in_usr_details_db_service.dart:141

This is happening in certain flow, when reading data from Firestore listener.
Also I have introduced few seconds delay before calling findFirst.. else the app was crashing without even printing this stack trace.
          await Future.delayed(Duration(seconds: 3));
          kbUserDetail = await kbUsersColl.where().usrIdEqualTo(kbUserId).findFirst();

If I trigger from UI - the same method calling findFirst query, this Exception did not occur and code ran fine.
Will try to get more detail around the Exception scenario and update in comments.
Isar Version used
isar: ^3.0.4
isar_flutter_libs: ^3.0.4
isar_generator: ^3.0.4
(However the app crash happened with isar 2.5 also.. for same scenario. I did not try few seconds delay with v 2.5... so this exception was not seen)
Device:
ios simulator iPhone 11 (iOS 13.7)

Thanks a lot for this detailed report 💜
Do you have a small code sample triggering this issue? Do you use the Future.delayed within a transaction?

hi @leisim,
I did not use Future.delayed within transaction.
Called it just before .findFirst(), as wanted to see if anything else happening their.. as app was crashing without printing any exception.
Will try to come up with small code to reproduce and share.
Thanks.

Fixed in 3.0.5 🙂

I tried now with 3.0.5, but still getting same exception in that scenario.
Tried below :

pubspec.yaml -> updated all 3 isar libs to 3.0.5
flutter clean
flutter pub get
flutter pub run build_runner build --delete-conflicting-outputs
ios simulator -> Erase all contents and settings
ran the app in ios simulator

In my code I am having multiple isar db instances, and at time of exception I believe there are atleast 2 isar-db-instances having simultaneous read/write. Not sure if that would cause such issue.
So far I could not reproduce this issue in smaller code.

Oh no :( Let's reopen then.
If you find a way to reproduce that would be awesome 💜

Hi, the issue occurred with below scenario:
another function is called within writeTxn on isar-db-instance-1
this other function uses other instance of isar (isar-db-instance-2)
Code sample with issue
  funcA(Msg msg) {
    isarInstance?.writeTxn(() async {
      insertedId = await isarInstance_1.msgs.put(msg);
      if (insertedId != null) {
        funcB(msg);
      }
    });
  }

  funcB(Msg msg) {
    User? user = await isarInstance_2.users.where().usrIdEqualTo(msg.usrId).findFirst();
  }

Fix:
move the funcB call out of writeTxn
  funcA(Msg msg, Function callback) {

    isarInstance?.writeTxn(() async {
      insertedId = await isarInstance_1.msgs.put(msg);
    });

    if (insertedId != null) {
      funcB(msg);
    }

  }

Earlier you asked if some call was within transaction, and I missed noticing the callback happening - involving different isar instances.
This workaround works for me. Thank you.

Ohhh you nest transactions of different instances. That's not allowed. I'll add a check to throw a better error.


Add custom volume mount documentation

one example for in-memory
and one example for NVME backed storage, also pointing out all the flaws/risks
in theory other volume mount types should be possible. Like a glusterfs mount and whatnot :-)

Haven't found the time to properly investigate the issues regarding leaving data behind.
And when stuff is left behind. So that's why I marked it as a draw, because I haven't done the research yet (not easily observed while folks are running builds constantly).
We could simply reword it a little to bring it to the users attention that these are aspects of concern and leave it at that.

@mumoshu I've checked the current state of things.
nvme setup:

backend volume fills up over time due to no cleanup.
with emphemeral: false i believe it leaves files behind on /tmp. I can see the on the backend volume and I have no reason to suspect that files in /tmp are not visible between buillds. they probably are, even with epmhemeral: true.

ramdisk setup:
here i believe nothing changes. i believe it's safe with emphemeral: true and flawed with emphemeral: false which is probably the case for any setup. emphemeral: true is probably better anyway to reclaim memory/disk space. We are at the moment running it with emphemeral: false :innocent:, though i will probably change it.
k8s will come with size limited ramdisks, so then we will be able to even have multiple workers on a very large node without overcommitting memory.
that sounds so awesome, can't wait. I think the current statis is alpha or beta.
but I think with these docs it's safe to show the new capabilities without neglecting due dilligence

i just tested and verified.
ramdisk with emphemeral: true is safe.
but i fear that data from the workdir is left behind with emphemeral: fasle and persistent storage like nvme.
so I'm investigating that right now

data in workdir is even left behind with emphemeral: true and nvme backend volume.
that is evil and we are running it like that right now :see_no_evil:
so perhaps we should only document ramdisk setup.

okay after some testing. with ephemeral: true data is cleared in the nvme setup on the work dir between job runs.
otherwise data even leaks inside the workdir :facepalm:

I thinks it's good to go, for real now :innocent:

@mumoshu I think this can be merged.
If you have suggestions for improvements, tomorrow I'm working and then I got a nice sweet holiday :palm_tree: :innocent:
Word on the street is there will be a release soon and this documentation should perhaps be included


basic IME supporting

General
This PR fix #979,#1544.
iced ime support was broken due to winit 0.27 update .
This PR add event to fit new winit IME event model and introduce new event support for text_input.
how to test
download app from my repo https://github.com/t18b219k/iced_text_input_sample/
Phase 1 : Can we input CJK characters?
tested on

[x] windows MS-IME
[x] windows Google IME
[x] linux ibus x11
[x] linux ibus wayland(gnome/kde)
[x] linux fcitx x11
[x] linux fcitx wayland(gnome/kde/sway/hyprland)
[x] macos IME

Phase 2 : Candidate position is near the TextInput?
You maybe failed to reproduce due to IME version.
candidate positon on X11 require LibX11 1.8.2 or above.
Wayland requirement.

input_method_popups_v2

tested on

[x] windows MS-IME
[x] windows Google IME
[x] Linux IBus X11(on Manjaro 22.0)
[x] Linux IBus gnome wayland
[x] Linux Fcitx5 X11(on Manjaro 22.0)
[x] Linux Fcitx5 gnome wayland
[x] Linux Fcitx5 kde wayland( can't move to the position )
[x] Linux Fcitx5 sway(supported on sway-im AUR package.)
[x] Linux Fcitx5 hyprland
[x] macos IME(setting candidate window position delay. maybe macos issue?)

Phase 3 draw underline and cursor
behavior
https://user-images.githubusercontent.com/48007646/205664366-ea71ddaf-a778-4f1a-9d0e-1b5cb557e70c.mp4
test
not required.
We render underline and cursor correctly if winit correctly report it.

[x] windows
[x] macos google japanese IME(not report boldlining),
[x] macos pinyin
[x] IBus X11 (no boldlining)
[x] IBus gnome Wayland(pinyin,mozc,and anthy incorrect report)
[x] Fcitx5 X11 (no boldlining)
[x] Fcitx5 gnome Wayland (mozc and anthy incorrect event report)
[x] Fcitx5 sway
[x] Fcitx5 hyprland
[x] Fcitx5 kde Wayland

tasks

[x] enable winit IME support
[x] translate winit IME event to iced event
[x] text_input editing support
[x] tell IME to put candidate window
[x] underline

I don't have apple devices please test someone .

we should move IME window to correct position to provide native look and feel
where we should implement this logic.

if alacritty  works well this should work as well.
please test alacritty works well.

On my mac the candidate box is showing but it's in the wrong place (macOS Monterey 12.6, MacBook Pro (16-inch, 2019))

In alacritty it works well. (alacritty 0.11.0)

we need to tell IME candidate window position to winit but I can't find good way.

@hecrj  I want to move IME candidate window to correct position but I can't find the way to send window Action from widget.
I think we need to extend Widget interface like clipboard implementation.
Is there any opinion to this problem.

now candidate window placed near TextInput .
this positioning is not best but much better than nothing,

@Cupnfish @someoneinjd  please test candidate window position is correct.

Thanks for your hard work. It works in most cases. But when I use the input method for the first time since the application started, the position is wrong, and after that the position is correct.

Alacritty also has this problem, so it might be a winit problem?

Thanks for your hard work. It works in most cases. But when I use the input method for the first time since the application started, the position is wrong, and after that the position is correct.   Alacritty also has this problem, so it might be a winit problem?

This is also the case on my laptop. Not only that, switching to another app and then switching back triggers the bug again.

Maybe we need to notify IMEEnabled Event from IME to TextInput .
I will try to fix this problem today .

@Cupnfish  @someoneinjd  please test condidate window is correct position even first time.

if we set IME position when we receive click event waybe we can achieve correct positioning.

@someoneinjd  @Cupnfish
please test if it works.
set_ime_position when TextInput clicked.
If this doesn't work we need to talk with alacritty team.

@someoneinjd @Cupnfish please test if it works. set_ime_position when TextInput clicked. If this doesn't work we need to talk with alacritty team.

It still comes up with that problem.

Unfortunately the problem still exists

shall we release this PR or not.
if you fixed this problem please open PR to mine repo.

I find that set_ime_position in IMEEnabled doesn't work, maybe we need to set the candidate box position before receiving this event.
But this may not be easy to achieve, because when I use the Chinese input method to start todos and type, IMEEnabled is the first event received by the update function.

Unfocus text_input then type korean, preedit character(expect) inserted in text_input.
It can delete manually but isn't deleted automatically.
Used font: NanumGothic.ttf
OS: Windows 10

IME handling is very platform dependent issue so feel free to PR to my repo .

@hecrj
This PR is very platform issue but no one tried to improving this PR.
I think it better to basic support than nothing.

I want to help, but I don't know where to start. Can I get a hint?

@zmtq05
here is my repo https://github.com/t18b219k/iced
it seems to me connecting to IME is ok .
so I want to you to dive to native/src/widget/text_input.rs update function.
before you making PR please run CI to build test binary.
if you can please install CJK font for easy test.

I resolve my issue by does nothing when receive preedit event and unfocused.
but japanese and chinese candidate window still appear when unfocused then input.
sorry for bad english.

I'm concerned about the case where the string contains a variation selector when input with the IME. It seems iced doesn't handle the variation selector at all.

I will resolve X11 issue by directly talk to ibus and fcitx  using dbus API.

Thanks for your hard work. It works in most cases. But when I use the input method for the first time since the application started, the position is wrong, and after that the position is correct.

Alacritty also has this problem, so it might be a winit problem?

Related X11 bug
https://bugs.freedesktop.org/show_bug.cgi?id=1580

directory talk to ibus and fcitx is not easy.
I'm now improving looking.

underlining when using IME.
Bold line means this section text is converting.
https://user-images.githubusercontent.com/48007646/198844635-dd08f4e4-47b4-4ec2-8a3f-d580096cc5f4.mp4

@zmtq05 Thank you for good idea.
My implementation is buggy when we use IME and open many TextInputs.
Your idea implemented in my latest patch.
https://github.com/t18b219k/iced/commit/fbc9c595c994831e4c18030b4d9d09a2bea3a516

underline position is weird.

Cursor must be located after the character.

There is no IME event standard even in MS.
Korean always returns indicator position 0,0 means cursor should place before editing character but Japanese and Chinese IME returns correctly.

ibus-hangul returns correctly.
ibus-pinyin same as MS Korean but correctly report.
if we apply MS Korean will break ibus-pinyin.

diff --git a/native/src/widget/text_input/ime_range.rs b/native/src/widget/text_input/ime_range.rs
index f66b8f72..a31eaef4 100644
--- a/native/src/widget/text_input/ime_range.rs
+++ b/native/src/widget/text_input/ime_range.rs
@@ -55,6 +55,8 @@ impl IMERange {
         if let Some((start, end)) = self.bold_line_range {
             if end == text.len() {
                 Some(text)
+            } else if start == 0 && end == 0 {
+                Some(text.split_at(end).1)
             } else if end == start {
                 Some(text.split_at(end).0)
             } else {

I fixed issue by this. Could you test japanese and chinese?

My developing branch is ime_support.
same Korean fix implemented in ime_support branch.
your impl not break JP and pinyin but not tested on chewing.

@hecrj please merge winit 0.27.5 to support candidate window position on x11 platform.
candidate window position setting works on fcitx5 5.0.19 + winit 0.27.5.
but not works on winit 0.27.2.

X11 and Wayland IME issue will fixed by using ibus directly.
I'm implementing winit patch to use ibus .

@hecrj
I did best but there is bug in Desktop enviroment but current iced text_input status is same as alacritty and also better than egui.

Cursor position issue again. :joy:

maybe reintroduced by merge conflict resolve or IMEState refactoring.

please someone test macos japanese IME to find bug in underline.

@Cupnfish
I made some change to this PR.
please test underlining if you can input Japanese please test it.

@someoneinjd @Cupnfish
IME enable and disable logic changed to support Password input .
Windows and linux works correctly but macos is correctly working?

https://user-images.githubusercontent.com/61791143/206238705-05654eb8-8303-4ca4-b69c-d4a68c5613f9.mov
looks like it works fine on intel macOS Ventura 13.0.1

I am using the latest version of Pop os with Mozc-ibus (version 2.26.4220.102) as the input method.

@Drodofsky
IBus mozc can display preedit but can't move Candidate window near the TextInput due to libx11 limitation.
very recently libx11 merge request  enable placing candidate window to  user specified position.  but ibus ,fcitx, and winit update needed to support it.

@Drodofsky
if you can try Manjaro , this issue fixed but unfortunately no ibus-mozc only ibus-anthy.

@Drodofsky if you can try Manjaro , this issue fixed but unfortunately no ibus-mozc only ibus-anthy.

Thank you. It works on Wayland. Unfortunately, Wayland is not yet stable on Pop os.

Closes #1544

fortunately close bug #1544 but why #1544 closed?

Winit generates no event when you hit ^, and IME is disabled.

This is the PR I'm waited for soooooo long! Iced could become a solid GUI framework after this is merged.

@someoneinjd
some changes may affect to macos so we need to test.
I have question about Sway.

how to display candidate window?

https://user-images.githubusercontent.com/61791143/209459596-19f33056-316c-45c6-9813-9236aa110542.mov
works fine!
AFAIK, displaying the candidate window requires the wayland server to support input_method_popups v2, which is not supported by Sway. #7266 tries to add support for it. So you can build sway from this source or just install the aur package sway-im.
What's more, hyprland also supports input_method_popups v2, you can use it to test.

thanks for testing

due to very large change made in advanced text so I rebased .


Policy for what to do if all destinations are unhealthy

If the result of health checks is that all destinations are unhealthy, then what should YARP do?
Currently we reject the request
An alternative would be to have panic routing and to pick a destination.
Should we have the concept of backup destinations which are used when the primary(ies) are unhealthy?

Duplicate of #83


Unicode

Hi.
I have some problems during training my own model on Persian dataset. It gave me error at the beginning of training phase. My dataset is in UTF-8 format. Does Glample support utf-8? If yes, what else can be the problem? My dataset is in CONLL2003 format.
The Error: "file loader.py", line 43, in update_tag_scheme
'Please check sentence %i:\n%s' % (i, s_str))
Exception: <exception str() failed>
"
Thanks

You might need to change the encoding scheme in loader.py from 'utf8' to your string encoding format e.g., I used 'latin-1' for Spanish and German.

Hi Sir i am also having the same issue with English Data set. My data set stanfordSentimentTreebank is encoded in UTF-8 and i am using GoogleNews Pretrained Word embedding that is a .gz file....
https://drive.google.com/file/d/0B7XkCwpI5KDYNlNUTTlSS21pQmM/edit
Kindly guide me as i am stuck with this error.

@dungtn  can you please help me solving the issue?


[lightbox-media-viewer - with carousel]: QA: Screen reader is not identifying the slide headings

Engineering info:

Innovation Team engineer:
Innovation Team JIRA Ticket: https://jsw.ibm.com/browse/HC-2520
DPO consulting engineer: N/A

Description
Page Element:
Headings in the carousel slide
Issue Description:
Screen reader does not identify the heading tag for the slide headings. they are announced only as text
Expected:
"Use Heading Level 1 for the main content of the page. Heading Level's should be in logical order, Starting from H1, H2, H3, H4, H5 and H6 respectively.
don't use Headings for font, For increasing font &
style use CSS."
Do not skip Heading levels
User Impact:
If the screen reader does not identify page headings, users are unable to skim through various sections in the page and understand the content.
Component(s) impacted
lightbox-media-viewer - with carousel
Browser
Safari, Firefox
Carbon for IBM.com version
canary
Severity
Severity 3 = The problem is visible or noticeable to users but does not impede the usability or functionality. Affects minor functionality, has a workaround.
Application/website
Carbon for IBM.com
Package
@carbon/ibmdotcom-web-components
CodeSandbox example
https://ibmdotcom-webcomponents.s3.us-east.cloud-object-storage.appdomain.cloud/deploy-previews/8757/iframe.html?id=components-lightbox-media-viewer--with-carousel
Steps to reproduce the issue (if applicable)
No response
Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[ ] I checked the current issues for duplicate issues

@andy-blum @kennylam Screen reader is announcing the headings in each slide now. Issue is fixed. Tested in Web components carousel default https://ibmdotcom-webcomponents.s3.us-east.cloud-object-storage.appdomain.cloud/deploy-previews/9149/index.html?path=/story/overview-getting-started--page


[en] Use apiversion networking.k8s.io/v1 for ingress

There is a warning deprecation message for kubernetes v1.19:
Warning: networking.k8s.io/v1beta1 Ingress is deprecated in v1.19+, unavailable in v1.22+; use networking.k8s.io/v1 Ingress

Switched apiversion in the example to networking.k8s.io/v1
Related PRs for #24224  and  #24075

/lgtm
/approve


update detection readme and meta

Signed-off-by: Can Zhao<EMAIL_ADDRESS>Fixes # .
Description
Update detection readme to follow the format of monai-toolkit
Status
Ready/Work in progress/Hold
Please ensure all the checkboxes:

[x] Codeformat tests passed locally by running ./runtests.sh --codeformat.
[ ] In-line docstrings updated.
[ ] Update version and changelog in metadata.json if changing an existing bundle.
[ ] Please ensure the naming rules in config files meet our requirements (please refer to: CONTRIBUTING.md).
[ ] Ensure versions of packages such as monai, pytorch and numpy are correct in metadata.json.
[ ] Descriptions should be consistent with the content, such as eval_metrics of the provided weights and TorchScript modules.
[ ] Files larger than 25MB are excluded and replaced by providing download links in large_file.yml.
[ ] Avoid using path that contains personal information within config files (such as use /home/your_name/ for "bundle_root").

/build