Upgrade to Elm 0.19

switch to andre-dietrich/parser-combinators which has been upgraded to 0.19
tests modified slightly to remove precending "+" which elm doesn't seem to support any more
no parser modifications for 0.19, just changes to compile and pass existing tests

Is the example working for you? it's just blank for me when I load it

Yes example/index.html works for me in Firefox and Chrome


Loosen requirements

Loosen the requirements to allow more current versions of packages (mostly for cryptography).

Thanks for the PR; poetrys weird default upper version pinning bit me again here. Can you tell my why cryptography = ">=37.0.4, <43" and not cryptography = ">=37.0.4"?

I'm fine with no upper bounds on these, I just added them with the concern of major releases potentially causing backward comparability problems but it does seem pretty safe


Document and rename bevy::scene::Entity

How can Bevy's documentation be improved?
This type (docs,  source) appears to be used to store reflected entities, along with their components in scenes. However, it is a) completely lacking documentation b) very confusingly named: Entity is one of our most important types, and this is closely-related but distinct in important ways.
At a glance, this seems to be a reflection-powered serializable representation of an entity and its components. The entity: u32 field  is likely for use with Entity::new() (the standard Entity type), and the entire DynamicScene type is used to spawn groups of entities which contain a number of entities, each with their own run-time defined set of components. These DynamicScenes can be written to file, but not read from.
Later in the same file, EntityMap is used, which contains a map between entities, in the standard bevy_ecs sense. This naming convention seems to be fine.
My first impulse would be to call this DynamicEntity, but please, bikeshed me if you disagree.

Yup I generally try to treat Bevy as a "global namespace" and this very clearly breaks that rule of thumb.

DynamicEntity seems very appropriate.

These DynamicScenes can be written to file, but not read from.

We get a Handle<DynamicScene>>  when loading a serialized scene from file:
https://github.com/bevyengine/bevy/blob/ccee658436be3d8bd721bbdda4a02f47235799da/examples/scene/scene.rs#L53
(However, we get a  Handle<Scene> type when loading a glTF scene, which might be confusing).


megapixels: 1.8.2 -> 1.8.3

https://gitlab.com/megapixels-org/Megapixels/-/tags/1.8.3
This fixes Megapixels crashing when trying to take a picture, see https://gitlab.com/megapixels-org/Megapixels/-/issues/26.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks!


week 2 group and individual reports

no styling yet. just want to get content merged in first.

lgtm:)


How to use the 'done' event?

Hey there, thanks for this plugin.
I am having trouble getting a simple callback to work. When pace has finished, I want to execute a function. The docs mention that options and events should be placed before Pace executes. Where exactly? In a script tag before the file is included?
Pace.on('done', function() {
  alert("loaded!");
});

Could you also help me out if this is the wrong syntax?
Cheers. :)

<script src="pace.min.js"></script>
//after the including

Pace.on('done', function() {
  alert("loaded!");
});

helps a lot


Menu graphical issues

When running netbox on a vertical monitor, one is forced to use the menu button. This is fine, however, the background doesn't extend to fit all the menu items:

I did a little searching and couldn't find something related to this. Not a huge deal for us, but just want to make the right people aware of it

this is the problem I'm having with https://github.com/digitalocean/netbox/issues/511


How to generate baseline file with specific Codesmell_Signature

Hi, I'm trying to generate a baseline.xml using ./gradlew detektBaseline and define my detekt task :
detekt{
     baseline = projectDir.resolve("src/main/resources/reports/baseline.xml")
     input = files("src/main/java", "src/test/java", "src/main/kotlin", "src/test/kotlin", "$name.gradle.kts")
     config = files("kotlincheck/kotlin-common-style-config.yml")
 }

But generated baseline.xml don't have Codesmell_Signature.
Here is my baseline.xml:
<?xml version="1.0" ?>
<SmellBaseline>
  <Blacklist timestamp="1556192176797"></Blacklist>
  <Whitelist timestamp="1556192176797">
    <ID>ImportOrdering:</ID>
    <ID>Indentation:</ID>
    <ID>MaximumLineLength:</ID>
    <ID>NoItParamInMultilineLambda:</ID>
    <ID>SpacingAroundOperators:</ID>
  </Whitelist>
</SmellBaseline>

I'm using detekt RC14.
How can I generate a file with format like <RuleID>:<Codesmell_Signature>, I hope Codesmell_Signature to be a specific file name

my baseline file doesn't haveCodesmell_Signature too。

please take a look at the linked issue for an answer. If you still have any troubles, please reopen this issue.


ci(travis): remove greenkeeper

:tada: This PR is included in version 6.4.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


docs: Add initial docs

Same order as the FuncMap for now. Splitting or re-ordering can be done later.
I feel like some of them aren't quite right...

I'll merge this, but I'm still not quite sure how to achieve what you outlined in https://github.com/chezmoi/templatefuncs/issues/1#issuecomment-1384679715, both how to set up embed, and how to consume it within chezmoi, having never really done anything like this before.

how to achieve what you outlined in https://github.com/chezmoi/templatefuncs/issues/1#issuecomment-1384679715

chezmoi uses this for its documentation and examples. The rough outline is:

The markdown documentation is included in chezmoi's binary using the Go standard library's embed package.
chezmoi includes a markdown parser and renderer and uses this to extract relevant bits of documentation.

The relevant chezmoi code is:

https://github.com/twpayne/chezmoi/blob/4504bc3b4fd221d4a20e29a2b95ee9cffebadb10/assets/chezmoi.io/docs/reference/commands/commands.go makes all the command *.md files available in a io/fs.FS. This is in the Go package github.com/twpayne/chezmoi/v2/assets/chezmoi.io/docs/references/commands.
This package is imported and, on startup, the command *.md files are parsed and the help text extracted: https://github.com/twpayne/chezmoi/blob/4504bc3b4fd221d4a20e29a2b95ee9cffebadb10/internal/cmd/cmd.go#L80-L91. The actual parsing happens here: https://github.com/twpayne/chezmoi/blob/4504bc3b4fd221d4a20e29a2b95ee9cffebadb10/internal/cmd/cmd.go#L152. It's a crude regexp-based state machine that extracts sections of the markdown files based on section titles.
The extracted markdown sections are rendered as plain text and stored in strings: https://github.com/twpayne/chezmoi/blob/4504bc3b4fd221d4a20e29a2b95ee9cffebadb10/internal/cmd/cmd.go#L199-L210.
Later, the strings are used in chezmoi's help output.

For template funcs, what's needed it for the docs subdirectory to contain a .go file that:

Embeds templatefuncs.md.
Includes an init() function that parses templatefuncs.md and populates an exported map[string]Reference, something like:

package docs

import (
    _ "embed"
)

//go:embed templatefuncs.md
var templateFuncsStr string

type Reference struct {
    Title   string
    Body    string
    Example string
}

var References map[string]Reference

func init() {
    // Parse templateFuncStr and store the results in References.
}

References then contains something like:
References["contains"] = Reference{
    Title: "`contains` *substring* *string*",
    Body:  "`contains` returns whether *substring* is in *string*.".
    Example: "```text\n" +
             "{{ "abc" | contains "ab" }}\n" +
             "\n" +
             "true\n" +
             "```\n",
}

Then, finally, a tool in chezmoi's build system imports github.com/chezmoi/templatefuncs/docs and generates Markdown files for chezmoi's website from the contents of the References variable.


PostGIS RouterDB?

Hi,
Is it possible to hook up a running PostGIS server to Itinero in stead of loading data through shape or pbf files?
Cheers, Johan

You can use pgRouting support with PostGIS http://pgrouting.org. Or you export data to shapefile with sample http://docs.itinero.tech/docs/itinero/data-sources/shapefiles.html

Converting to shapefile is exactly what I'm trying to avoid.
To be more specific: I'm trying to use Itinero's OpenLR implementation.

No, it's not possible at the moment.
You could try reading the data directly from the database. There used to be a stream source for PG from OSM data but it's unmaintained at the moment and hasn't been updated in a while (meaning it won't work anymore out-of-the-box with the latest OsmSharp version):
https://github.com/OsmSharp/data-providers/blob/master/OsmSharp.Data.PostgreSQL/Osm/PostgreSQLDataSource.cs
That would at least bypass shapefiles, that's a win, but in the end this comes down to the same thing because creating a routerdb is always needed, and it streams over all the data.
You could also create a version of this:
https://github.com/itinero/geo/blob/master/src/Itinero.IO.Shape/Reader/ShapeFileReader.cs
To read directly from PG, perhaps it's a good idea to create an NTS IO project that just uses NTS linestrings as it's source, that would support any source NTS supports.
If you are always decoding/encoding OpenLR strings in the same area or you don't care about performance that much you could write something that filters a bounding box around the area and creates routerdb's on the fly.
The roadmap for Itinero's next version is to make the data management in the routerdb's easier by using a tiled scheme, that could work in this scenario by syncing it with the database.
It was a deliberate choice at some point to move away from any database dependency because it's usually too slow for high volume fast routing.
We also do consulting on projects, if you need direct help from us that's also possible.

Closing this, feel free to reopen if there are more questions.


🚀 Feature Request: user message should be added before awaiting onSubmitMessage

🎤 Tell us your idea
Currently, onSubmitMessage is being awaited, then the user message is appended/visible
Change this behavior so that the message is already visible before calling onSubmitMessage

@mme i would love to work on it


APM ignore examples

Waiting for : https://discuss.atom.io/t/npmignore-for-apm-apmignore/14985

The contents of the repository are the contents of the APM package.
So this task is now to : Move the examples out of this repository.

moved to https://github.com/TypeStrong/atom-typescript-examples


MM-14196 Add virtualised list for posts

Summary
P1 To Do for next Spinmint for QA, in priority order:

[ ] fix overlapping issue when making new posts in a channel that has a lot of posts loaded
[ ] logic tweak for bottom of the channel so we never see unmounted posts (now shows 10 max)
[ ] Manual loadtest of posts to verify this issue os only related to the /test command: Fix loading indicator stuck and showing blank posts above (see image below)
[ ] Fix jumping to permalink when posts are not loaded yet
[ ] Scroll to bottom when adding reaction or editing most recent post
[ ] Youtube previews causing minor incorrect scroll correction
[ ] Don't scroll center channel to bottom when posting in the RHS

QA testing cases:

busy channel with many incoming posts - verify no white space when scrolling or sitting at the bottom of the channel
Scroll near the top of a channel with 5000+ messages and then try to switch channels. Examine perf
Post a message when scrolled near the top of a channel with 5000+ messages
Click new messages indicator at the top of a channel with losts of posts.
Editing posts and adding reactions. Verify no flickering or scroll pop
Posting images, link previews and youtube previews. Verify no pop during post, scrolling or refresh
Scroll quickly in one direction then quickly switch scrolling direction
Permalinks to posts in channels that aren't loaded yet

P2 after QA starts testing:

[x] Loader can get stuck at the top of the screen if we can repro
[ ] First team switch does not reliably work because of channel sync issue.
[ ] ES: IE11 scroll correction to be tested after addressing above issues
[x] minor scroll bar adjustment when entering the channel on subsequent visits

Completed:

[x] Scroll bar position should be based on posts that are currently mounted or have been mounted in your current channel view session
[x] Trigger loading of posts at 500px from top of loaded posts
[x] Link preview and SVG can throw off scroll correction a little. That will be addressed with https://github.com/mattermost/mattermost-webapp/pull/2334 and https://github.com/mattermost/mattermost-webapp/pull/2469
[x] youtube previews cause LARGE pop
[x] removing posts can cause blank space at the top of the channel
[x] channel header missing on channels with few posts
[x] In progress: Fix issue of flicker when a a new reaction is added (Does not happen always most likely related to the way style cache is busted).
[x] /test posts command then tying more messages causes blank space at the top of the screen
[x] scroll channel down to bottom when current user make a new post
[x] Permalink view should move post to middle of screen and highlight
[x] Remove padding below channel header

Ticket Link
MM-14196
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[ ] Added or updated unit tests (required for all new features)
[x] Has UI changes
[x] Touches critical sections of the codebase (auth, posting, etc.)

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-072e78d2e9599b8c4.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-072e78d2e9599b8c4

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0eb1e6a4c9a13c65d.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-0eb1e6a4c9a13c65d

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-04c8bfa4f5f47bf3b.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-04c8bfa4f5f47bf3b

Thanks Sudheer, this is looking really good. Some things I noticed:

White bar at the top:

Some channels are infinitely stuck in the loading more posts state (try in "autem" and "doloremque"):

Edit: actually all channels seem to be doing this at some point far enough back in the channel

The above ^ makes it a bit hard to test but I feel like we are loading much too early. Notice how the scroll bar starts to shrink almost immediately as I scroll up:

As far as I know the rules we put in place last time involved loading more posts at X% from the top. If you test Slack it has a much nicer feel because they seem to only load when you are almost at the top of the loaded posts (however I would say that they are loading a bit later than I would like us to do).

I did not notice any unmounted posts (white space). Is the virtualized post list active or is it just done well so I'm not noticing it?

Added new to do list items to PR description ^

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-050935d578db9c81e.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-050935d578db9c81e

Spinmint test running for more than 7 days. This test server was terminated.

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0a50dd8a42b8589d0.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-0a50dd8a42b8589d0

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-00f3b7a36dfba35ca.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-00f3b7a36dfba35ca

Updated PR description with new P1 to dos before QA testing

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0681245bedad4ca4b.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0681245bedad4ca4b

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0cab4638cf074be87.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0cab4638cf074be87

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0bf9abb191e282d3f.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0bf9abb191e282d3f

Updated the PR description with new fixes required

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0162ca479a2f5c5dc.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0162ca479a2f5c5dc

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0235ef264b2695299.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0235ef264b2695299

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-036fd3fc91c83f30a.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-036fd3fc91c83f30a

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0c8c159d1c2f9e7de.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0c8c159d1c2f9e7de

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-01801045d4a5aa843.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-01801045d4a5aa843

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0d400a82635f11e08.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0d400a82635f11e08

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-07213acea2d9c591d.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-07213acea2d9c591d

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-008032a64c672fb74.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-008032a64c672fb74

@sudheerDev When I open a channel with a collapsed image attachment and expand it, the position breaks. Seems to only happen near the top of a channel though. https://i-008032a64c672fb74.test.spinmint.com/ad-1/pl/fozizd4yz7yh8pkgebmqonhgxo

@sudheerDev Chrome - Expanding a youtube link on the bottom of the page positions the preview out of view, the view isn't scrolled to the bottom (not an issue on Firefox where the view is scrolls to the bottom)

@sudheerDev Firefox - glitch when expanding a youtube link that's not on the bottom of the channel.
Note in the gif how the preview shows up on top of the url before taking it's correct position below the url:

@sudheerDev Clicking on a permalink opens a white screen.

Discussed with QA. Before merging to master we want to fix high priority issues, then work on the other known issues as it soaks on community-daily.
P1 (before merging to master - needs new spinmint):

Fix regression with jumpiness on loading new posts
Fix permalink view + jumping to post from search

P2 (after merging to master):

Investigate lag on safari
Small delay on clicking New Messages below bubble (1 second)
https://github.com/mattermost/mattermost-webapp/pull/2447#issuecomment-475445716
https://github.com/mattermost/mattermost-webapp/pull/2447#issuecomment-475456037
https://github.com/mattermost/mattermost-webapp/pull/2447#issuecomment-475462621

cc// @sudheerDev @DHaussermann @ogi-m

Spinmint test server destroyed

@saturninoabril @enahum completed functional changes. This is up for review. I am adding some testes for post_list.jsx but other than that feel feel to review.

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-075bf94edd781ccb0.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-075bf94edd781ccb0

@esethna @DHaussermann @ogi-m
P1 (before merging to master - needs new spinmint):

Fix regression with jumpiness on loading new posts - Done
Fix permalink view + jumping to post from search - Done

P2 (after merging to master):

Investigate lag on safari - Partially done but we can improve this
Small delay on clicking New Messages below bubble (1 second)
https://github.com/mattermost/mattermost-webapp/pull/2447#issuecomment-475445716
https://github.com/mattermost/mattermost-webapp/pull/2447#issuecomment-475456037 - Done
https://github.com/mattermost/mattermost-webapp/pull/2447#issuecomment-475462621 - Done

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-02b6fca036f5fa301.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-02b6fca036f5fa301

Spinmint test server destroyed

@amyblais removing the docs needed label. In my mind infinite scroll/virtualized post list is expected functionality from a users perspective, rather than something that needs to be called out in docs. Feel free to re-add if you think there's something we should cover specifically.


Don't Ignore all files or directories in output directory?

Hello,  Thank you for useful nuxt module!
I want to gitignore only file such as icons.svg not *.
For example, I set options to nuxt.config.js like:
svgSprite: {
  input: "~/assets/svg/icons",
  output: "~/assets/svg",
  defaultSprite: "icon-sprite"
}

All icon files in ~/assets/svg/icons will be also ignored because .gitignore has *.
Only I can do is change directory structure?
thank you

Only I can do is change directory structure?

Yes.
For now, there isn't any option to customize .gitignore on project. although we recommend setting the output path inside .nuxt directory, for example:
svgSprite: {
  input: "~/assets/svg/icons",
  output: "~/.nuxt/svg",
  defaultSprite: "icon-sprite"
}

In next realese, default output path will change to something like this and module will not generate .gitignore

@farnabaz
Thanks for your response.

output: "~/.nuxt/svg",

In next realese, default output path will change to something like this and module will not generate .gitignore

Sound good 👍 Thank you.


MOB 101 support typescript and convert Button to typescript

also made styled-components peerDependency
this is a pre-step for importing the button component in the portal (MOB-101)

wrong ticket number 🤦


cron_expressions. not being returned in ScheduleSpec

Expected Behavior
        schedule = await self.client.create_schedule(
            "my_schedule_id",
            Schedule(
                action=ScheduleActionStartWorkflow(
                    YourSchedulesWorkflow.run, id="my_id", task_queue="default"
                ),
                spec=ScheduleSpec(cron_expressions=["0 0 * * *"]),
            ),
        )

        schedule = self.client.get_schedule_handle(id=schedule.id)
        scheduleDesc = await schedule.describe()
        print("schedule", scheduleDesc.schedule.spec)

Should return
ScheduleSpec(calendars=[ScheduleCalendarSpec(second=(ScheduleRange(start=0, end=0, step=1),), minute=(ScheduleRange(start=0, end=0, step=1),), hour=(ScheduleRange(start=0, end=0, step=1),), day_of_month=(ScheduleRange(start=1, end=31, step=1),), month=(ScheduleRange(start=1, end=12, step=1),), year=(), day_of_week=(ScheduleRange(start=0, end=6, step=1),), comment=None)], intervals=[], cron_expressions=["0 0 * * *"], skip=[], start_at=None, end_at=None, jitter=None, time_zone_name=None)

Actual Behavior
        schedule = await self.client.create_schedule(
            "my_schedule_id",
            Schedule(
                action=ScheduleActionStartWorkflow(
                    YourSchedulesWorkflow.run, id="my_id", task_queue="default"
                ),
                spec=ScheduleSpec(cron_expressions=["0 0 * * *"]),
            ),
        )

        schedule = self.client.get_schedule_handle(id=schedule.id)
        scheduleDesc = await schedule.describe()
        print("schedule", scheduleDesc.schedule.spec)

returns
ScheduleSpec(calendars=[ScheduleCalendarSpec(second=(ScheduleRange(start=0, end=0, step=1),), minute=(ScheduleRange(start=0, end=0, step=1),), hour=(ScheduleRange(start=0, end=0, step=1),), day_of_month=(ScheduleRange(start=1, end=31, step=1),), month=(ScheduleRange(start=1, end=12, step=1),), year=(), day_of_week=(ScheduleRange(start=0, end=6, step=1),), comment=None)], intervals=[], cron_expressions=[], skip=[], start_at=None, end_at=None, jitter=None, time_zone_name=None)

Steps to Reproduce the Problem

Create schedule with cron_expression
Get handler for schedule by id
Display spec for schedule

Specifications

Version:^1.5.1
Platform:Python

Thanks for reporting this issue! It looks like you are experiencing a bug where the cron_expressions field in the ScheduleSpec is not being returned correctly when using the Python SDK.
This issue is being tracked in Temporal's GitHub repository https://github.com/temporalio/temporal/issues.
We are actively working on a fix for this bug. In the meantime, you can work around it by using the intervals field of the ScheduleSpec to specify your desired schedule.
Let us know if you have any other questions or issues!

Thanks for reporting this issue.
It seems that you are experiencing a known issue. The issue is that the spec returned from describe() does not contain the cron_expressions field even though the schedule was created with cron.
For more information and possible workarounds, please refer to the Temporal CLI command options reference documentation.


Uncaught TypeError: Cannot read property 'uploadFile' of undefined

[Enter steps to reproduce below:]

...
...

Atom Version: 1.6.2
System: Unknown Windows Version
Thrown From: activate-power-mode package, v0.4.1
Stack Trace
Uncaught TypeError: Cannot read property 'uploadFile' of undefined
At /C:/Users/Qi/.atom/packages/remote-sync/index.coffee:80

TypeError: Cannot read property 'uploadFile' of undefined
    at handleEvent (file:///C:/Users/Qi/.atom/packages/remote-sync/index.coffee:42:3)
    at atom-workspace.disposables.add.atom.commands.add.remote-sync:upload-file (file:///C:/Users/Qi/.atom/packages/remote-sync/index.coffee:85:40)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (C:\Users\Qi\AppData\Local\atom\app-1.6.2\resources\app.asar\src\command-registry.js:260:29)
    at C:\Users\Qi\AppData\Local\atom\app-1.6.2\resources\app.asar\src\command-registry.js:3:61
    at CommandPaletteView.module.exports.CommandPaletteView.confirmed (C:\Users\Qi\AppData\Local\atom\app-1.6.2\resources\app.asar\node_modules\command-palette\lib\command-palette-view.js:183:32)
    at CommandPaletteView.module.exports.SelectListView.confirmSelection (C:\Users\Qi\AppData\Local\atom\app-1.6.2\resources\app.asar\node_modules\atom-space-pen-views\lib\select-list-view.js:338:21)
    at space-pen-div.atom.commands.add.core:confirm (C:\Users\Qi\AppData\Local\atom\app-1.6.2\resources\app.asar\node_modules\atom-space-pen-views\lib\select-list-view.js:109:19)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (C:\Users\Qi\AppData\Local\atom\app-1.6.2\resources\app.asar\src\command-registry.js:260:29)
    at C:\Users\Qi\AppData\Local\atom\app-1.6.2\resources\app.asar\src\command-registry.js:3:61
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (C:\Users\Qi\AppData\Local\atom\app-1.6.2\resources\app.asar\node_modules\atom-keymap\lib\keymap-manager.js:580:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (C:\Users\Qi\AppData\Local\atom\app-1.6.2\resources\app.asar\node_modules\atom-keymap\lib\keymap-manager.js:388:22)
    at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeydown (C:\Users\Qi\AppData\Local\atom\app-1.6.2\resources\app.asar\src\window-event-handler.js:97:36)
    at HTMLDocument.<anonymous> (C:\Users\Qi\AppData\Local\atom\app-1.6.2\resources\app.asar\src\window-event-handler.js:3:61)

Commands
     -1:25.5.0 intentions:highlight (atom-text-editor.editor.is-focused)
     -1:25.5.0 core:paste (atom-text-editor.editor.is-focused)
     -1:17.5.0 intentions:highlight (atom-text-editor.editor.is-focused)
     -1:17.2.0 core:save (atom-text-editor.editor.is-focused)
     -0:43.7.0 intentions:highlight (atom-text-editor.editor.is-focused)
     -0:43.3.0 window:run-package-specs (atom-text-editor.editor.is-focused)
     -0:42.5.0 intentions:highlight (atom-text-editor.editor.is-focused)
     -0:42.3.0 command-palette:toggle (atom-text-editor.editor.is-focused)
     -0:22.1.0 core:move-down (atom-text-editor.editor.mini.is-focused)
     -0:21.4.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:21.3.0 remote-sync:upload-folder (atom-text-editor.editor)
     -0:12.1.0 intentions:highlight (atom-text-editor.editor.is-focused)
     -0:12 command-palette:toggle (atom-text-editor.editor.is-focused)
     -0:08.9.0 core:backspace (atom-text-editor.editor.mini.is-focused)
     -0:04.4.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:04.4.0 remote-sync:upload-file (atom-text-editor.editor)

Config
{
  "core": {
    "disabledPackages": [
      "Ensime",
      "Ensime",
      "Ensime",
      "Ensime",
      "Ensime",
      "Ensime",
      "Ensime",
      "Ensime",
      "Ensime",
      "vim-mode"
    ],
    "themes": [
      "two-light-ui",
      "one-light-syntax"
    ]
  },
  "activate-power-mode": {
    "particles": {
      "size": {},
      "spawnCount": {},
      "totalCount": {}
    },
    "screenShake": {
      "maxIntensity": 2
    }
  }
}

Installed Packages
# User
browser-plus, v0.0.54
hyperclick, v0.0.35
intentions, v1.0.5
language-scala, v1.1.6
linter, v1.11.4
linter-scalac, v1.4.0
remote-sync, v3.5.0
two-light-ui, v1.0.0

# Dev
No dev packages

It seems that this error is caused by the remote-sync package. I don't know why atom shows the activate-power-mode instead.

Thanks. I also noticed this. Very
excited to hear from you
Sent from my Mi phoneOn Daniel Alejandro Cast<EMAIL_ADDRESS>Apr 21, 2016 6:34 AM wrote:It seems that this error is caused by the remote-sync package. I don't know why atom shows the activate-power-mode instead.
—You are receiving this because you authored the thread.Reply to this email directly or view it on GitHub


Provide a task that can show the origin of any dependency, across all projects

Expected Behavior
I'd like to be able to find out where a dependency comes from
Current Behavior
I can't find out where a dependency comes from.
The dependencyInsight task requires that the name of a configuration is provided. I don't know the name of a configuration of a transitive dependency. This task also doesn't work across subprojects.
Context
I was trying to work on #19028 and replace com.jcraft:jsch with com.github.mwiede:jsch, but the dependency I'm trying to replace is still showing up.
IntelliJ tells me that the dependency I'm trying to replace is still downloaded, but I can't tell where it's coming from.

I'd like to be able to find that out, but as described above dependencyInsight isn't suitable. I've tried creating a custom task, but this also doesn't work.

Task dependencyReport from project-report plugin (and propagating it to submodules) gives me desired results.

I was reading "show the origin of any dependency" and "find out where a dependency comes from" as "give me the URL / repository the dependency was resolved from". That's unfortunately something the Project Report Plugin still does not seem to be able to do.

Probably, I understood incorrectly. My understanding of "origin" was "where this dependency is located in dependency tree across all configurations".
Could you please describe exactly what do you mean under "origin"?

Could you please describe exactly what do you mean under "origin"?

I would have thought you mean an URL like https://repo1.maven.org/maven2/com/jcraft/jsch/0.1.55/jsch-0.1.55.jar.

@sschuberth I'm not sure that download urls is a responsibility of dependencyInsight as data has been taken from a gradle cache.
Download URLs can be found in build logs (assuming debug output is on)
The OP tells, that dependency is shown up, and here dependencyReport is quite useful.


restructure knn

Just pushed a fix

I fixed the treading. There was no performance change for some reason but at least its more correct.


Too many alerts when critical water level is reached.

If reading interval is set to a low number you will get an alert on each and every critical level reading.  Need to add a way that if an alert is triggered it sleeps for a period of time before initiating the next reading, or you could get bombarded with texts\emails.  Perhaps a 5 minute alert interval would be sufficient but will look to add it as an option to the configuration file.

Version 0.7.0 addresses this problem.
Added 'alert_interval =' to the config file to rate limit alerts.  If value is not present then it is set to 5 minutes.


Coverage Question

When I use JQF-2.1 to test James-3.8.1, I encounter a situation where the coverage information is 0.
`Semantic Fuzzing with Zest
Test name:            org.apache.james.jmap.draft.methods.SetMessagesCreationProcessorTest#assertUserCanSendFromShouldThrowWhenSenderIsNotTheConnectedUser
Instrumentation:      Janala
Results directory:    /home/xmq/Downloads/james-project-3.8.1-source-release/james-project/server/protocols/jmap-draft/target/fuzz-results/org.apache.james.jmap.draft.methods.SetMessagesCreationProcessorTest/assertUserCanSendFromShouldThrowWhenSenderIsNotTheConnectedUser
Elapsed time:         3s (max 10s)
Number of executions: 95,009 (no trial limit)
Valid inputs:         0 (0.00%)
Cycles completed:     0
Unique failures:      3
Queue size:           0 (0 favored last cycle)
Current parent input:
Execution speed:      31,971/sec now | 25,595/sec overall
Total coverage:       0 branches (0.00% of map)
Valid coverage:       0 branches (0.00% of map)
Fuzzing stopped due to guidance exception: Too many trials without coverage; likely all assumption violations
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  4.520 s
[INFO] Finished at: 2024-12-28T15:00:57+08:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal edu.berkeley.cs.jqf:jqf-maven-plugin:2.1-SNAPSHOT:fuzz (default-cli) on project james-server-jmap-draft: Internal error: Too many trials without coverage; likely all assumption violations -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
`

You can see the error message: Too many trials without coverage; likely all assumption violations and 3 unique failures in the status screen. You likely have assumeTrue() statements where the predicate is always false (i.e., no randomly generated input is satisfying the predicate) or some assumption in your input generator that is never true.
You can debug by running mvn jqf:repro with the input files saved in the failures directory, and seeing what the error stack trace looks like. Maybe also add some print statements at the start of your test method to see what your inputs look like in order to debug the issue.


Fix warnings leftover from VFS deprecation

Fix warnings leftover from VFS deprecation
This addresses warnings introduced recently and generally improves the
overall context class loader feature and related code.
Fixing warnings:

Deprecate some classes that exist only to support deprecated code
Reduce references to deprecated classes by putting them behind more
utility methods (especially in ClassLoaderUtil)
Remove references to deprecated classes in javadoc @see tags
Remove unused variables
Remove Deprecated annotation when it would propagate and create too
many warnings for internal classes that aren't user-facing and don't
need to advertise their deprecation status
Suppress unavoidable compiler warnings due to deprecations
Prevent new compiler warnings from being introduced, by adding a
flag for the compiler plugin to fail the build on warnings. Since we
keep untriaged (unsuppressed) warnings at 0, this should not cause a
problem for most builds, but can be suppressed during development with
-Dmaven.compiler.failOnWarning=false.

Other improvements:

Deprecate old table.classpath.context in favor of new
table.class.loader.context, and update Property docs
Update ContextClassLoaderFactory javadocs and remove declared
RuntimeExceptions from interface definition
Consolidate ContextClassLoaders and ClassLoaderUtil utility classes
Rename LegacyVFSContextClassLoaderFactory to
DefaultContextClassLoaderFactory
Use more functional and streams, wherever possible, and use generics
where previously rawtypes or excessive casting was used
Remove unneeded initialize method from factory interface
Push down IOException handling into utility methods in
AccumuloVFSClassLoader that were previously propagated and handled
outside the class to simplify calling code

All ITs passed except for known flaky tests #1574 and #1791


add dependabot config script

Inspired by https://github.com/kubeflow/pipelines/issues/4682  I created a script that will create a config file for depandabot so that it knows what directories to scan. It will scan the repository for files named *ockerfile*, package*.json, *requirements.txt and go.*.  It is setup for dockerfiles, npm packages, pip dependencies and gomod at the moment. It is trivial to further customize what folders are selected if further customization is needed. It also parses the closest OWNERS file for a given dependency listing file, and assigns the relevant approvers and adds the relevant reviewers to the PRs it creates.
This is a sibling PR to https://github.com/kubeflow/pipelines/pull/5015, https://github.com/kubeflow/kubeflow/pull/5542, https://github.com/kubeflow/kfserving/pull/1309, https://github.com/kubeflow/arena/pull/403, https://github.com/kubeflow/testing/pull/855, https://github.com/kubeflow/fairing/pull/550, https://github.com/kubeflow/kfp-tekton/pull/432, https://github.com/kubeflow/katib/pull/1420, https://github.com/kubeflow/tf-operator/pull/1224, https://github.com/kubeflow/kfp-tekton-backend/pull/28, https://github.com/kubeflow/mpi-operator/pull/319, https://github.com/kubeflow/pytorch-operator/pull/315, https://github.com/kubeflow/metadata/pull/255, https://github.com/kubeflow/xgboost-operator/pull/107, https://github.com/kubeflow/fate-operator/pull/26, https://github.com/kubeflow/mxnet-operator/pull/87, https://github.com/kubeflow/website/pull/2459, https://github.com/kubeflow/kfctl/pull/479, https://github.com/kubeflow/examples/pull/843, https://github.com/kubeflow/code-intelligence/pull/198, https://github.com/kubeflow/gcp-blueprints/pull/192, https://github.com/kubeflow/common/pull/107, https://github.com/kubeflow/blog/pull/64, https://github.com/kubeflow/kubebench/pull/215, https://github.com/kubeflow/internal-acls/pull/429, https://github.com/kubeflow/community-infra/pull/34 and https://github.com/kubeflow/caffe2-operator/pull/4
I am not yet sure how many PRs will be created with this configuration, as dependabot is not yet finished scanning my fork.
For reference, the PRs that will be created can be found here: https://github.com/DavidSpek/crd-validation/pulls

/assign @carmark

/assign @carmark

I am holding the PR to have some control over when it gets merged so that the optional test infra doesn't get overloaded if all the repo's were to merge this at the same time.
/hold

I am holding the PR to have some control over when it gets merged so that the optional test infra doesn't get overloaded if all the repo's were to merge this at the same time.
/hold


Use +genclient for the configuration object

The description of the HiveConfig type has the
+genclient:nonNamespaced comment, but not +genclient. Both need to
be present, otherwise the code generation tools don't generate a client
for the type.

@dgoodwin @nimrodshn please review.

/retest

/test e2e

/test e2e

@jhernand LGTM :+1:

/lgtm


OOD train test split question

Hi!
I am using CellOT in the OOD setting and I was wondering why you were using a test_size > 0 for split_cell_data_train_test in the ood setting? My thinking is that in OOD you would want to use all the data you can that's outside of the holdout group. Is it for evaluating the performance of the non-ood tasks or maybe something else I am missing?
Thank you so much!

Hi Dan,
We wanted the test set to be the same for all experiments, iid and ood
Best,
Stefan


Update shortwave aerosol lookup tables.

Purpose
Update shortwave aerosol lookup tables.
Updated shortwave aerosol lookup tables are temporarily added directly to the repository. This file is expected to be available directly from the artifact source repository (https://github.com/earth-system-radiation/rrtmgp-data) in their upcoming release.

[x] I have read and checked the items on the review checklist.

@szy21 : Please confirm if the updated documentation citing the source for this new file is accurate!

@szy21 : Please confirm if the updated documentation citing the source for this new file is accurate!

Yes, it's good.


Add guide on how to build ROS 2 with tracing instrumentation

Closes #2805
This guide will become invalid/unneeded if we do https://github.com/ros2/ros2/issues/1177
Signed-off-by: Christophe Bedard<EMAIL_ADDRESS>

The guide feels too straightforward to me, but maybe I'm jumping over things that are not obvious to other people. Feedback would be really useful.

@christophebedard Should we backport this to all of Foxy, Galactic, and Humble?

yep!


Add the new HA border radius prop

Seemed like a nice quick/easy one to add.
fixes https://github.com/nathan-gs/ha-map-card/issues/57

Thanks again for fixing this one!

Thank you guys


loco-cli should auto-install sea-orm-cli if missing

Probably wise to also ask before installing it, and explain why it is needed.

Can I pick this issue?

@jondot maybe we can fix the seaorm cli crate and make the generator entry to use via the crate instead of the cli installation?
what do you think?

The problem is that the sea-orm is needed only on specific templates (saas).
We could do something smarter and check in the lib that:
Check if the user uses the DB feature and the migration commands in the configuration are enabled (migration, truncate, reset..) and then check if seaorm-cli is installed. and if not, stop the service and prompt a pretty message.
What do you think?

yes sounds good!

@AbhilashGade you can feel free and take it.
you can implement it in the lib instead of the CLI.
see that the validation should be only when with-db feature is enable

@kaplanelad yeah on it

just make sure that you not installing automatically sea-orm. i think it is better to return an pretty error with explanation

Okay how do I test this

@AbhilashGade you can use (check_seaorm_cli)[https://github.com/loco-rs/loco/blob/master/src/doctor.rs#L132] which doing the checks that you needs

So do I do something like this
`fn check_sea_orm() {
println!(" SeaORM is not installed. SeaORM is a powerful ORM for Rust that provides convenient database abstractions. It is required for this feature. Would you like to install it? (y/n)");
let mut input = String::new();
std::io::stdin().read_line(&mut input).unwrap();
let input = input.trim().to_lowercase();

if input == "y" || input == "yes" {
    println!("Installing SeaORM... ");
    std::process::Command::new("cargo")
        .arg("install")
        .arg("sea-orm")
        .output()
        .expect("Failed to install SeaORM");
} else {
    println!("SeaORM is required for this feature. Please install it to continue.");
}

}
`

@kaplanelad yeah on it

just make sure that you not installing automatically sea-orm. i think it is better to return an pretty error with explanation

Did you see my previous message? I believe there's no need to install the sea-orm-cli. Handling the installation process might be unnecessary and open to potential problems.
Prompting the user with the necessary details should suffice.
You can take a look at this PR: #212. if you think we can to it better or prettier you are more then welcome to advise


Bluetooth: host: Allow GATT client to restore subscription info without resubscribing

Is your enhancement proposal related to a problem? Please describe.
When a GATT client subscribes to notifications from a server, the client host layer stores some subscription info in bt_gatt_subscribe_params structs (and possibly elsewhere depending on PR #21682). If the client loses this subscription info (e.g. due to a reboot), there is currently no way to restore it without resubscribing (i.e. resending the subscription requests over the air to the server). In the case of bonded devices, the server will preserve subscriptions (BT Core spec v5.1 vol 3 part G section <IP_ADDRESS>), so in this case resubscribing is a waste of time and power.
Describe the solution you'd like
The host should provide a way to restore the subscription info on the client side without resubscribing. One solution might be to persist this info via the settings subsystem, but this might be unnecessary as most of the info (handles, subscription values, and peer addresses) would normally already be known and persisted (or hardcoded in the application logic) by the application layer or host in the context of service discovery and bonding, and the remaining info (node, ...) should be fairly easy for the host to recreate. So it would be preferable if the host would provide an API allowing the application layer to collaborate with the host to restore the subscription info - without the need for resubscribing or persisting duplicate info.
Describe alternatives you've considered
-
Additional context
This issue was originally part of issue  #21103.

@Vudentz or @joerchan any feedback?


Exception while Subcommands Parsing

Hi,
I took your example for subcommands and and changed a little detail, just added a default:
val output by parser.option(ArgType.String, "output", "o", "Output file").default("any_file")
And I added a println in excute(), just to use this option, call the getter.
        override fun execute() {
            result = addendums.sum()
            result = if (invert!!) -1 * result else result

            println("result is: $result and will output to $output")
        }

Calling $ demoprogram summary 1 2 4
leads to the exception:
Exception in thread "main" java.lang.IllegalStateException: Value for argument output isn't set. ArgParser.parse(...) method should be called before.
	at kotlinx.cli.ArgumentSingleValue.getValue(ArgumentValues.kt:121)
	at kotlinx.cli.ArgumentValueDelegate$DefaultImpls.getValue(ArgParser.kt:59)
	at kotlinx.cli.ArgumentSingleValue.getValue(ArgumentValues.kt:118)
	at DemoKt$main$Summary.execute(Demo.kt:18)
	at kotlinx.cli.ArgParser.parse(ArgParser.kt:455)
	at kotlinx.cli.ArgParser.parse(ArgParser.kt:375)
	at DemoKt.main(Demo.kt:33)

Using kotlinx-cli-jvm-0.2.1.jar
PS: General arguments must be set before the  subcommand keyword. Would be nice to mention it somewhere, it took me some time to figure it out.

As I understood exception was caused by order of arguments. We are planning to fix this just with removing this restriction connected with order of arguments.

Just to make it clear, the exception is thrown after using the default() statement. And the related option was not mentioned in the command line.
This means, in the moment you can't define default() values while using subcommands.

Fixed


[5.3] Type narrowing regression with property comparison to boolean false

🔎 Search Terms
narrowing, strict equals, property access, boolean false, undefined
🕗 Version & Regression Information

This changed between versions 5.2 and 5.3
This also happens in the nightly

⏯ Playground Link
https://www.typescriptlang.org/play?ts=5.3.1-rc#code/JYOwLgpgTgZghgYwgAgGIHt3IN4ChnIBGcUAXEZgDYRwi4C+uuC6IAzmMgOYRgZYBeZAAoAlOX7IAPsgCuIACYQYoCAuQCAfDnzIovWVBDIAsnDAALAHRRaC9AFsxybQAYrAVmQB+HERLk8JRsKPTI5PJKKiBqDEws7JwwmBrcvPxiuMAwIsno3lbEUBoCQkEhojoECWzo1FaU6FzCeaJxuAD0HcgAAmBsALQQAB4ADhAIYENQUOjFA8h5yMBsyKPobGzAhJQAnnKKyqoKzKy19Y3NeYUkbUA
💻 Code
interface Foo {
  bar: boolean
}

const getFoo = (): Foo | undefined => {
  return Math.random() > 0.5 ? { bar: false } : undefined
}

const foo = getFoo()
if (foo?.bar === false) {
  console.log(foo)
}

// @ts-expect-error - foo is possibly undefined
console.log(foo.bar)

🙁 Actual behavior
In line 13 and following, foo is incorrectly narrowed to just Foo (whereas it was previously Foo | undefined), although we learned nothing about it from the preceding if as that did not influence control flow.
Unused '@ts-expect-error' directive.(2578)

🙂 Expected behavior
TS should report an error when accessing foo.bar as foo can potentially be undefined.
Additional information about the issue
I originally discovered this issue in JSX with a snippet similar to:
function Component () {
  const foo = getFoo()
  return (
    <div>
      {foo?.bar === false && 'foo'}
      {foo.bar ? 'true' : 'false'}
    </div>
  )
}

There as well, the access foo.bar is allowed due to the preceding check, although they are completely separate.
Note that changing it to === true or removing the checks entirely fixes the problem.

It looks very much related to my change: https://github.com/microsoft/TypeScript/pull/53714 . I'll investigate this soon. Thanks for the report!

Here's a different reproduction resembling the one we have in our project, maybe you find it helpful:
export interface Options {
    a?: boolean;
    b?: boolean;
}

function foobar(options?: Options) {
    // @ts-expect-error 'options' is possibly 'undefined'
    if (options?.a === false || options.b) {
    }
}

https://www.typescriptlang.org/play?ts=5.3.2#code/KYDwDg9gTgLgBASwHY2FAZgQwMbDgeTBgQiQGc4BvAKDjrkwH4AuOAIwggBthMkBuWvTYt2nHn0EBfatXQBXJNmKk46Tm0xQAFBCIlyowivIBKKkLoB6K3AACMMgFpQYYMpdQo0OAHI9JmS+iBSQZGQIbFwAnn6KACbA6MjA8b6WiOhwuvqkZIwAdJhwALxlaphcZHgAPjVwAQZkBWzmNPRwMlJAA

This issue has a lot of thumbs up; out of curiosity, are people finding this naturally? Or is there some Twitter thread that's directing people here?
The original PR didn't fail any of the extended test suites, so it's somewhat surprising to me to have a regression like this.

My guess is that it was posted on a company’s Slack channel or smth and a couple of devs from that channel +1ed this 😉

Looks like this issue still exists for some specific cases
https://www.typescriptlang.org/play/?#code/CYUwxgNghgTiAEkoGdnwILwN4CgCQAksugBQCUAXPAC4AWAlmoxgNw4C+OOoSciA9gDtk1eAAco1aiBiCqmAD7wAroNAAzeoJDA261WGr0h8KGLEQAngAVJ02eWz566+CQlSZggPwA6IqRk8AC8ofDqUBDIIEG4eHge9oJs8QD0qfAAet74eHDUyrIpnAl2XinpWTnsQA

Please file a new issue


feat: provide a Span<T> to Array<T> method

Feature Request
Span<T> is a wrapper around @Array<T>.
It should be possible to get the value out of this wrapper, either as @Array<T> or Array<T>. Or both.
Right now the only way to build back an Array from a Span is to iterate over it through pop_front. It is not a good DevX.
Describe Preferred Solution
A counterpart to the Array::span() method. Span::consume -> Array and Span::snapshot -> @Array maybe.
Or implementations of the Into trait.
Describe Alternatives
Implementations of the Into trait.
Related Code

If the feature request is approved, would you be willing to submit a PR?
(Help can be provided if you need assistance submitting a PR)
No, it's an important design decision, it should most likely be done by the project maintainers.

Do note that:
let mut arr = array![];
arr.append_span(span);

Should provide what you requested.
Not sure about adding a Into implementation - as Into does not sound like a heavy operation - and this action is definitely expensive.

Also - do note - that since (at least currently) a span is not the owner of its elements - consumption of a span into an array isn't actually possible.

Not sure about adding a Into implementation - as Into does not sound like a heavy operation - and this action is definitely expensive.

I have to disagree about Into not being for heavy operation. You have no way to enforce it once the trait is pub and there is no general understanding of what is or is not a heavy operation. So this should not be a criteria when thinking about implementing Into.
As you can see in the When to implement From section of the rust language, the complexity of the operation is not a criteria.
Having From<Array<T>> of Span<T> and From<Span<T>> of Array<T> being implemented as part of the corelib doesn't go against any of the criteria listed in the rust doc and most importantly enhance the programer ability to create complex generic trait with bouds like +<Into<Array<T>>>

Also - do note - that since (at least currently) a span is not the owner of its elements - consumption of a span into an array isn't actually possible.

You consume the Span, not its elements.
This should do the work:
let mut arr = array![];
arr.append_span(span);
drop(span);

But I note that those implementation would only exist where +Copy<T> or +Clone<T>

You do not consume the span, as you cannot consume a copyable object (as any access to it would not consume the object)

I accept the into claim (mostly as there is a Into for &[] to Vec in rust).
I still am strongly against calling any function that accepts a span being called consumed - given that it doesn't consume it in any way that would make sense to the function caller.

I still am strongly against calling any function that accepts a span being called consumed - given that it doesn't consume it in any way that would make sense to the function caller.

From the caller's pov, because of Copy, it looks like it is not consumed.
But into does consume it if you look at the actual Into impl. pop_front is called by append_span onto span. Leaving span` empty at the end of the execution, then dropping it.


UI:Create Error Suppression

As an ECMPS user, I want to be able to create an error suppression via the error suppression module in order to perform admin needs
Given a user has navigated to the error suppression module
When a user clicks to add or clone and saves the details in the pop up
Then an error suppression is created and will be viewed in the error suppression table

Add Suppression Clone

User is able to Add and Clone an Error Suppression via the UI


CMatchingGenerator misses some obvious matches

For some inputs CMatchingGenerator generates suboptimal matchings. Certain objects are effectively left unmatched, even though they have a match in the other set.
Here is a test I wrote that should pass but is currently failing. I found a few "bad" examples, this is one of the smallest. Here, two checks are passing, but the last one is failing: EXPECT_EQ( 7, FindMatchForLeft( 5 ) ). Instead, left no.5 is matched to a different element with a score of 0.
TEST( CMatchingGenerator, UniqueMatches6x8 )
{
    struct CMyPair {
        typedef int Quality;

        int LeftIndex = -1;
        int RightIndex = -1;
        Quality Score = 0;

        Quality Penalty() const { return 1 - Score; }
    };
    const int numLeft = 6;
    const int numRight = 8;
    const int pairScores[numRight][numLeft] = {
        {0, 0, 0, 0, 0, 0},
        {0, 0, 0, 0, 0, 0},
        {0, 0, 0, 0, 0, 0},
        {0, 0, 0, 0, 0, 0},
        {0, 0, 0, 0, 0, 0},
        {0, 0, 0, 1, 0, 0},  // Left #3 matches right #5
        {0, 0, 0, 0, 1, 0},  // Left #4 matches right #6
        {0, 0, 0, 0, 0, 1}   // Left #5 matches right #7
    };
    NeoML::CMatchingGenerator< CMyPair > generator( numLeft, numRight, 0, INT_MAX );
    for( int leftInd = 0; leftInd < numLeft; ++leftInd ) {
        for( int rightInd = 0; rightInd < numRight; ++rightInd ) {
            CMyPair& pair = generator.PairMatrix()(leftInd, rightInd);
            pair.LeftIndex = leftInd;
            pair.RightIndex = rightInd;
            pair.Score = pairScores[rightInd][leftInd];
        }
    }
    generator.Build();
    CArray<CMyPair> matching;
    generator.GetNextMatching( matching );
    const auto& FindMatchForLeft = [&matching]( int leftInd )
    {
        for( const CMyPair& pair : matching ) {
            if( pair.LeftIndex == leftInd ) {
                return pair.RightIndex;
            }
        }
        return NotFound;
    };
    EXPECT_EQ( 5, FindMatchForLeft( 3 ) );
    EXPECT_EQ( 6, FindMatchForLeft( 4 ) );
    EXPECT_EQ( 7, FindMatchForLeft( 5 ) );
}

This test added

https://github.com/neoml-lib/neoml/pull/1112

and works fine


Various fixes to gitinterface

This PR introduces various fixes to gitinterface, some of which were previously seen in #115.

AbsoluteReference can now handle HEAD by resolving it to the target and returning that absolute path
Previously, there was some code to create the remote tracking ref for some ref, this has been broken out into a reusable component
Fetch had a trackRemote option to choose whether the fetch was for the remote tracker or not, this behavior has been updated to always fetch to the specified target ref (so refs/heads/main:refs/heads/main) and the corresponding remote tracker
Some Fetch* tests missed a symbolic reference for HEAD, this is going to become an issue as we implement other sync interfaces alongside #134
Some tests were using manual tmp dir handling, this has been updated to use t.TempDir() based on a prior review

Of these, I think only number 3 is a controversial change as this is slightly different from Git's own behavior. However, we're the only consumers for this and I think we're more likely to want and use the updated behavior. LMK what you think, @wlynch!


[Benchmark] Modify run.sh to accept config as param

Please describe the feature you'd like to see
We use config path which is currently hardcoded in the codebase. Since we want to run different configs for CI, Weekly jobs we cannot do it right now.
Describe the solution you'd like
We need to modify the CI to accept the config to run benchmarks on as a param.
Are there any alternatives to this feature?
Is there another way we could solve this problem or enable this use-case?
Additional context
Add any other context about the feature request here.
Acceptance Criteria

[ ] All checks and tests in the CI should pass
[ ] Unit tests (90% code coverage or more, once available)
[ ] Integration tests (if the feature relates to a new database or external service)
[ ] Example DAG
[ ] Docstrings in reStructuredText for each of  methods, classes, functions and module-level attributes (including Example DAG on how it should be used)
[ ] Exception handling in case of errors
[ ] Logging (are we exposing useful information to the user? e.g. source and destination)
[ ] Improve the documentation (README, Sphinx, and any other relevant)
[ ] How to use Guide for the feature (example)

This is not required anymore


c34b4ad0-50e0-4987-9aab-81ea5a6d64e5

this is for automated testing

1997c6b8-b231-4b53-8e8e-f4b1a3b94aac


Adding Offline info

Please see offline docs
I also reorganized the usage docs generally, because the usage doc was getting long. I can redo it if we don't like it.
Addresses #515

It could be good to also list the paths that OSV-Scanner uses to look for the database, and some basic instructions on how to manually download that database themselves.
The paths right now:

The value returned by os.UserCacheDir, if possible
The value returned by os.TempDir

To download themselves, follow osv data dumps doc, then place the all.zip files in this directory:

{local_db_dir}/
  osv-scanner/
    npm/all.zip
    PyPI/all.zip
    …
    {ecosystem}/all.zip

(Copied from the design doc, see it for more context)

Codecov Report

Merging #517 (02f4c62) into main (7f014fb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #517   +/-   ##
=======================================
  Coverage   68.17%   68.17%
=======================================
  Files          71       71
  Lines        4701     4701
=======================================
  Hits         3205     3205
  Misses       1414     1414
  Partials       82       82

:loudspeaker: Have feedback on the report? Share it here.

@G-Rath @another-rex
Please see updated documentation:

Offline mode
Usage

Rex, if you could specifically check the manual download section

@another-rex @G-Rath Thanks for your comments. Please review.

@another-rex please see changes. I think this may be ready.


Checkobx not working on ios

Hi everyone,
I just installed that, it works well on android but I get an error on IOS.
Invariant Violation: requireNativeComponent: "RNCCheckbox" was not found in the UIManager.
here is My react native versions:
react: 16.8.3 => 16.8.3
react-native: 0.59.0 => 0.59.0
"@react-native-community/checkbox": "^0.5.7"
after I do pod install on ios folder, I get this:
npx : 1 installé(s) en 2.332s
Scanning for pods...
1.9.1
> pod install
Analyzing dependencies
[!] Unable to find a specification for `React-Core` depended upon by `RNCCheckbox`

You have either:
 * out-of-date source repos which you can update with `pod repo update` or with `pod install --repo-update`.
 * mistyped the name or version.
 * not added the source repo that hosts the Podspec to your Podfile.

Aborting run
An unexpected error was encountered. Please report it as a bug:
{ CocoaPodsError: The command `pod install` failed
└─ Cause: Error: pod exited with non-zero code: 1

help please it's urgent. Thank you for you all!

I'm using it with expo sdk 40 and I have the same problem... Android works fine but in IOS I can't even start the app because of this... Can anyone help?

They mention on the front page that V0.2 is the latest version which supports RN 0.59 and below (Android only).
I'm not a maintainer. If possible, try to upgrade to RN 0.60+. This tool might help, though it definitely depends on how old your dependencies are.

hello @rakicluca , for me I solved the problem for ios by creating a checkbox component using native elements. so on ios I called the created component and android this one and that's it. without upgrading react native version

hi @hanaechahid, thanks for reply! I see that with react native elements but I want to use this one... For the moment I will use the react native elements version... Anyway thank you for the advice ;)


Support building just the libraries.

The current Makefile doesn't support building the libraries and not the tools, whereas the install targets provide for installing just the libraries.
The initial commentary is misleading (it doesn't mention the tools).
This change adds a target 'build-libs', to build just the libraries.

Makefile (initial commentary): Document the actual effects of the current targets, and add targets for just the libs. (all): use build-libs. (build-libs): New target, follows design of the install-libs target.

This PR was an experiment to check that there were no issues.


The clip_feature

When I run the tool/clip_prepro_feats.py, the shape of clip_feature is 2048, but each value is the same.

Hi @Timon0327, can you try the updated repo and share with us your log so we can better help you?


I can't find my env in the console

App Id: d3u1esrvizobbf
Region: us-east-1
Step:
Status:

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
A clear and concise description of what the issue/question is.
I can't find my env in the console, I can only see older ones. The url provided from amplify-console gives this error App with id: d3u1esrvizobbf not found. I'm able to make changes to the env from the cli at least but I need to access the resources manually.
Error message
If there is an error message, please include it here.
Note: Be sure to check the message for sensitive information.
App with id: d3u1esrvizobbf not found
Additional information
Please add any other relevant information. Please feel free to include screenshots.
additional sample code or a sample repository to help us reproduce the issue. (Be sure to remove any sensitive data)

Are you sure you're in the right region from which you deployed?

Oh wow thats embarrassing 😭I was not in the right region. Thanks for the quick response!


Prep for 0.1.1 release

These changes have been sitting in master for 11 months without a Forge
release, while they are needed for some use cases. About time to just
roll them in.

@reidmv
Can you hold off on releasing these changes? I've got a major rewrite in progress right now that I'm testing.


[FR] Cancel enqueued task

Is your feature request related to a problem? Please describe.
Not applicable.
Describe the solution you'd like
I need to be able to cancel a task that was enqueued, like this pseudo-code:
const task = await queue.enqueue({ foo: true })

// Then...
await queue.cancel(task.id)

Describe alternatives you've considered
Maybe there's some way to cancel the task using @google-cloud/tasks directly, but I didn't try yet.
EDIT: Apparently, it's not possible because the task ID is not returned by firebase-admin.
Additional context
I've also asked on StackOverflow, but I think it's really a missing feature: https://stackoverflow.com/questions/72549309/how-to-cancel-a-task-enqueued-on-firebase-functions

Internal feature request number 235998431

Hey, I'd like to add here that being able to specify the name of the task would be better for me than getting an auto-generated name.
Use-case: I want to have a Firestore-triggered Cloud Function that will enqueue the task when a document is created. If I can specify the task name, I just create the document with a taskName field and pass it to queue.enqueue(). Otherwise, if the task name is always automatically generated, I'll need to update the document to store it and then protect against infinite recursion because changing the document will retrigger the function. I'd rather not have to deal with a possible infinite recursion.

Both features have been submitted for API review, which is a formal step before any implementation can begin. One thing I realized during my research though is that autogenerated document IDs make horrible task IDs. Task queue IDs are used for sharding, so sequences of increasing numbers (like timestamps) cause scaling problems. Automatic document IDs are based on timestmaps and will cause problems. A solution is to reverse the string with mystring.split("").reverse().join("")

Indeed. That code is for truly unique IDs. I thought Firestore was like the Realtime Database, which uses base64 encoded timestamps along with an anti-collision nonce.

Somewhat related: I wanted to set task name myself, so I can use the Cloud Tasks automatic deduplication.

I am running into the same issue.
I want to prevent the tasks in the queue from being duplicated. It seems the only solution is to manually set a task name in the code. When this is supported than I wont need to clear or cancels tasks by myself

They just released v11.10.0 which provides a way to cancel enqueued tasks: https://github.com/firebase/firebase-admin-node/releases/tag/v11.10.0 🥳


Object and Embed elements are JS functions in Firefox and Chrome 47, so dart2js does not find interceptors for them

Same root cause as https://github.com/dart-lang/sdk/issues/23466 but more general.
@rakudrama

Note: tests that fail because of this are marked Pass, RuntimeError while we have some bots on 46 and others on 47. This should be fixed once they're all updated.

All the bots are on 47 now and the tests are properly marked RuntimeError.

@alan-knight what else remains here? I noted the bug is not assigned to anyone.

This high priority bug should be assigned to someone on the dart2js html library team, or it shouldn't be high priority.

The high priority original problem now has a workaround, so we can probably remove this from high priority.

Moving to 1.15

@alan-knight could you give the link or example of this workaround?

As I remember, the issue was setting some content through Angular, so it was going through setting innerHtml, which meant that it was getting touched by Dart code, although it didn't need to be. So there are a few possible workarounds. One is don't use that element :-). Another is that if it's in static HTML and you're not programmatically manipulating it through Dart it shouldn't be a problem. I think that even if you go through setting innerHtml/appendHtml/etc, if it's HTML that you control and you can use NodeTreeSanitizer.trusted, then that should work. And I guess the last-ditch workaround is write a fragment of JS to do the work you need to do on that element and invoke it from Dart.

possible duplicate https://github.com/dart-lang/sdk/issues/25785 ?

Yes, I think that's the same underlying issue.

Clearing out 1.15 milestone as the last full push to dev has happened. If any changes are required before 1.15 is shipped, please file a merge request


vCore: add an extended confirmation dialog for "risky" operations

Currently it's used in vCore only, but it also adds a new "Setting":

A confirmation prompt when option is DISABLED:

A confirmation prompt when option is ENABLED:

BTW: I just noticed that Azure Resources has a similar setting in place, however, they'd ask the user to input the name of the resource being deleted.

Couple of thoughts:

what is the default setting for azureResourceGroups.deleteConfirmation? Seems to be Enter Name? I'm wondering if it'd make more sense to reuse that to have a centralised setting, why would users want to have different preference for databases?
Data Explorer is always asking for the name:

If we stick with the existing setting, we could still do the number puzzle for Click Button preference depending on the severity of the operation.
Enter Name is not applicable for bulk operations such as bulk deleting documents, so for these operations we could always show the number puzzle.
In your example the "default" button seems to be the correct number:

So that hitting "Enter" would be enough to confirm with the keyboard. We should make sure the "required" number is not the first/default keyboard selection.

Thank you @sevoku ! I wasn't aware of azureResourceGroups.deleteConfirmation existence. This is the best way to go.
Also, Dmitrii made good remarks over chat about testing and e2e scripts. I'll update this code to use the deleteConfirmation that's already implemented and tested.

as I wrote, I'm not advocating for the old experience, I like the puzzle especially for operations without an available name (we can't expect folks to enter document ID + partition key, and not what for bulk operations). I think it makes a lot of sense to have this "simplified" experience for the database context. However there are some more caveats:

The Resources setting is speaking of "resource groups":

That's not obvious that it also affects other operations, so we should PR a wording change to Azure Resources
The puzzle dialog should also have a link to the settings page, since it's mentioning that the behaviour can be changed in settings.

Updated settings, the "quiz" is not in there, just the two basic ones as in "Azure Resources":

This is the behavior. I'm using "Drop Container" from vCore:
Word Confirmation

Button Click

We have a simple helper function that takes care of reading settings and showing an appropriate warning:
/**
 * Prompts the user for a confirmation based on the configured confirmation style.
 *
 * @param title - The title of the confirmation dialog.
 * @param message - The message to display in the confirmation dialog. This message will be suffixed with instructions for a specific prompt.
 * @param expectedConfirmationWord - The word that the user must type to confirm the action when the confirmation style is set to 'Word Confirmation'.
 * @returns A promise that resolves to a boolean indicating whether the user confirmed the action.
 */
export async function getConfirmationAsInSettings(
    title: string,
    message: string,
    expectedConfirmationWord: string,
): Promise<boolean> {
    const deleteConfirmation: string | undefined = vscode.workspace
        .getConfiguration()
        .get<string>(ext.settingsKeys.confirmationStyle);

    if (deleteConfirmation === 'Word Confirmation') {
        return await getConfirmationWithWordQuestion(title, message, expectedConfirmationWord);
    }
    return await getConfirmationWithClick(title, message);
}

to be continued here: https://github.com/microsoft/vscode-cosmosdb/pull/2377


crash after unity logo

It looks like many people have this problem. Idk what causes it. Maybe it's because of GPU. But I can't recreate it in my laptop.
Possible problem:

wrong player setting
wrong graphic settings
openxr issue
serial issue
if you have same problem, pls sent your PC specs here (CPU, GPU, Windows Version, Headset model)

It looks like many people have this problem. Idk what causes it. Maybe it's because of GPU. But I can't recreate it in my laptop. Possible problem:

wrong player setting
wrong graphic settings
openxr issue
serial issue
if you have same problem, pls sent your PC specs here (CPU, GPU, Windows Version, Headset model)

I have the same problem, but I found a solution

If you start sinmai first and then maidxr, you need to close the port binding and open the port mapping after maidxr enters the game
(recommended) start maidxr first, and then it usually gets stuck. Then start sinmai, and maidxr will automatically return to normal
When I run the project in the unit editor, the editor will always report an error that cannot link the port. I speculate that the process is blocked due to the failure of port communication

It looks like many people have this problem. Idk what causes it. Maybe it's because of GPU. But I can't recreate it in my laptop. Possible problem:

wrong player setting
wrong graphic settings
openxr issue
serial issue
if you have same problem, pls sent your PC specs here (CPU, GPU, Windows Version, Headset model)

I have the same problem, but I found a solution

If you start sinmai first and then maidxr, you need to close the port binding and open the port mapping after maidxr enters the game
(recommended) start maidxr first, and then it usually gets stuck. Then start sinmai, and maidxr will automatically return to normal
When I run the project in the unit editor, the editor will always report an error that cannot link the port. I speculate that the process is blocked due to the failure of port communication

Thank you for the solution. I think I know what is the issue. Can you try enable buffers in com2com and see if the problem is still Herren or not. If it still crash, i will rewrite all serial code.

Ok, I found the issue, You must enable the enable buffer option in com2com on both ports of all pairs. Otherwise, your MaiDXR will crash after the Unity logo.


Need help on working with two type parameters

Hi,
this is great work!
I am trying to use your library for an Either<E,A> like type, but it looks as if two type parameters are not possible (because there is only one associated type for it in Higher.
What would you suggest to do to work around this?
Jan

This is something I'm still trying to figure out the best way to go about. Check out the pull request, aprosian implemented some higher types for two type parameter structures, but there were some sacrifices made. The pull request has the code and discussion, so it should be a good jumping point!

Thanks!


OCPBUGS-44957: Use fleet manager configmaps to determine placeholder pod node antiaffinity

What this PR does / why we need it:
Before this commit, the placeholder controller looked at nodes with hosted cluster labels to determine which fleet manager labels it should exclude for placeholders. However, in the case when a hostedcluster control plane is cleaned up, but the HostedCluster CR itself has not been deleted due to the controlplane namespace not getting cleaned up, there are no longer any nodes with the hostedcluster label, but the hostedcluster is still associated with a particular pair label. Placeholder pods are not prevented from landing on nodes with this same pair label. These nodes cannot be used to allocate to a new HostedCluster because they are still associated with previous HostedCluster. This change modifies the placeholder controller to use the pair association configmaps in the placeholder namespace to determine which pair labels to avoid. This makes it consistent with the scheduler which also looks at the same configmaps to determine whether a fleetmanager pair label is available to use for a new cluster.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #OCPBUGS-44957
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[x] This change includes unit tests.

/jira refresh

/lgtm

Previous failure on e2e-aws-417:
  - lastTransitionTime: "2024-11-26T16:04:01Z"
    message: "Throttling: Rate exceeded\n\tstatus code: 400, request id: 693a651c-2a52-4ed6-b15d-e58f398fe8e3"
    reason: AWSError
    status: "False"
    type: AWSEndpointAvailable

On awsendpointservice resource


Drop pointer on remaining fields

What changed?

Why?

How did you test it?

Potential risks

Coverage decreased (-0.0004%) to 64.533% when pulling aa611a683a3da5e9b62b17548770cfc6f1b7b900 on vytautas-karpavicius:drop-ptr-remaining-fields into cf5c9cc786863002b0cfbfcd4760de43569ba820 on uber:master.


[SHIRO-777] remove powermock.

This test will now actually start hazelcast. If this is not feasible, just add a corresponding comment to this PR.

Added enforcer plugin
Does not actually start  hazelcast anymore due to overriding the create method (which is now untested).

retest this please

@fpapon restest:

JDK 8: https://builds.apache.org/view/S-Z/view/Shiro/job/Shiro-pr/267/
JDK 11: https://builds.apache.org/view/S-Z/view/Shiro/job/Shiro-pr-jdk11/101/
JDK 14: doesn find the correct commit: https://builds.apache.org/view/S-Z/view/Shiro/job/Shiro-pr-jdk14/21/


Error  on ATOM version 1.18.0 x64

Just tried to install go-debug and ATOM is giving an error message after the restart.  Is there a possible fix for this?
Some installed packages could not be loaded because they contain native modules that were compiled for an earlier version of Atom.
go-debug 1.5.2

binding@unknown – Cannot find module '~/.atom/packages/go-debug/node_modules/fsevents/lib/binding'
binding@unknown – Cannot find module '~/.atom/packages/go-debug/node_modules/fsevents/lib/binding'
binding@unknown – Cannot find module '~/.atom/packages/go-debug/node_modules/fsevents/lib/binding'
binding@unknown – Cannot find module '~/.atom/packages/go-debug/node_modules/fsevents/lib/binding'
binding@unknown – Cannot find module '~/.atom/packages/go-debug/node_modules/fsevents/lib/binding'
binding@unknown – Cannot find module '~/.atom/packages/go-debug/node_modules/fsevents/lib/binding'

have a look at this issue: https://github.com/lloiser/go-debug/issues/110


Hidden project shows up on people's personal page

The unpublished projects show up on people's personal page. For example, the screenshot shows that kinetic fab study project appears on Jessica's personal page.

Somewhat related to https://github.com/jonfroehlich/makeabilitylabwebsite/issues/833
Can solve both of them at same time.

Fixed.
https://makeabilitylab.cs.washington.edu/member/jessicachin/


Locally host fonts

Fixes #201
Across the board seems to increase performance numbers especially on docs page where we actually use these fonts (all screenshots are clickable and lighthouse scores vary more wildly than WPT):

remote
local

home WPT 3g

docs WPT 3g

docs WPT 4g

lighthouse docs (vary more – give less weight to these)

Since Andrew is out, @augustjk, you're getting my lit.dev reviews in his place

Did you look at https://github.com/lit/lit.dev/pull/529? The blocker there was just that the Manrope from Google Fonts is kind of broken on some people's screens, so we needed to update it to fetch Manrope from GitHub instead.
An advantage of the other PR is that we can auto-update the fonts. It has some handling for licenses too.

What is the status of that PR? What can we do to move that forward?

Is having a way to update the open sans and roboto fonts important like we currently have for manrope? I can see this being a stop gap until we move forward with the other PR. But if the other PR is close enough maybe it's worth pursuing that directly.

Yeah, I'm fine with landing this. Updating fonts is something we probably want to do only very rarely.
You may want to look into a LICENSE file for the fonts folder, just for GitHub. I think that may be required. The script in the other PR created a combined one, I believe.

suggestions in. @aomarks what should do about max-age cache headers? Should I emulate what you did in the other?
I just looked at what other framework sites set for their cache headers and copied the most-common ones

@augustjk also removed from CSP as asked in DM


Update Settings “general”

Automatically generated by Netlify CMS

👷 Deploy Preview for safehavenaccounting processing.

Name
Link

🔨 Latest commit
dae60460b0d71c401bbc23d7d2d2877280028ac9

🔍 Latest deploy log
https://app.netlify.com/sites/safehavenaccounting/deploys/64755f9006a41100089cf05d


Tabular: Updated training priority for multiclass

Issue #, if available:
Description of changes:
Updated training priority for multiclass so tree algorithms train last.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-676-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-676/1/index.html

Benchmark Results (Filtered to only multiclass datasets):
                                framework  > AutoGluon_best_4h_2020_09_21_multi_pri  < AutoGluon_best_4h_2020_09_21_multi_pri  = AutoGluon_best_4h_2020_09_21_multi_pri  % Less Avg. Errors Than AutoGluon_best_4h_2020_09_21_multi_pri  time_train_s  metric_error  time_infer_s  loss_rescaled   rank  rank=1_count  rank=2_count  rank=3_count  rank>3_count  error_count
0  AutoGluon_best_4h_2020_09_21_multi_pri                                         0                                         0                                        16                                           0.000000                13137.97500      0.450248    439.848160          0.375  1.375            10             6             0             0            0
1            AutoGluon_best_4h_2020_09_17                                         6                                        10                                         0                                          -4.801281                13017.81875      0.494969    461.309315          0.625  1.625             6            10             0             0            0

As can be seen, this change is a major improvement.


Teams for Education Policy Wizard

[Enter feedback here]
I am following this guide but the Teams for Education Policy Wizard is not yet available?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0ebe30d6-2b8f-b8fd-3a22-0cbb1f7fabcb
Version Independent ID: 99e96c91-d183-f794-0825-db150fefa7aa
Content: Teams for Education Policy Wizard to easily apply policies for safe learning - Microsoft Teams
Content Source: Teams/easy-policy-setup-edu.md
Service: msteams
GitHub Login: @cichur
Microsoft Alias: v-cichur

Hi @tlstock,
I've shared your comments with our SME. -Cindy

Thanks so much @cichur

Hi @tlstock,
Thanks.
Currently the feature is in TAP and only selected tenants have the access. The feature will be available in Jan.
But if any tenant wants to be a part of TAP, then they can share their tenant id with us and we will enable the feature for them.
Thanks
Shashank

Hi Shashank
We would like to be part of TAP our tenant id is 59459c4b-e3c3-452a-a12e-d99164b2de43
Thanks
Theresa


[hipcc] Skip warning on gfx000.

The known target checking should skip gfx000 as well as it won't be
used in real compilation command formation. The avoid generating
annoying warning on gfx000.

So far, rocm_agent_enumerator always output gfx000 as the rocm agent. Since we already skip generating compilation option for gfx000, we should also avoid generating warning on it. That's quite annoying.


New release ?

7.3.0 was released Jan 7, 2017 but since, ember-macro-helpers has been updated a few times.
Wouldn't it be appropriate to cut a new release ?
PS: on a side note, looks like tags have not beed added to the repo since 7.0.1
Ju

@kellyselden any chance you can help bring ember-moment up to the latest API?

@jasonmit done #238

Included in 7.3.1


Child route renders the container of its parent

Issue Type

[x] Bug (https://github.com/mxstbr/react-boilerplate/blob/master/.github/CONTRIBUTING.md#bug-reports)
[ ] Feature (https://github.com/mxstbr/react-boilerplate/blob/master/.github/CONTRIBUTING.md#feature-requests)

Description
After adding a child route as described here, the container specified in the parent of the child route route is rendered instead of the one specified in the child route.
Steps to reproduce
In the routes.js file, for example add a child route to the /features route:
...
    {
      path: '/features',
      name: 'features',
      getComponent(nextState, cb) {
        System.import('containers/FeaturePage')
          .then(loadModule(cb))
          .catch(errorLoading);
      },
      childRoutes: [
        {
          path: '/features/test',
          name: 'home2',
          getComponent(nextState, cb) {
            System.import('containers/HomePage')
              .then(loadModule(cb))
              .catch(errorLoading);
          },
        }
      ],
    }
...

Now, after running npm start the /features/test route should display the HomePage container, but instead its parent, the FeaturePage container, is rendered. The same thing happens when using the code from the example from the documentation.
I was able to reproduce this in both the master and dev branch, however I've seen other people using it so maybe I'm making a mistake somewhere.

in the case of your example, you have to

provide reducer/sagas for HomePage
render children somewhere in FeaturePage container so RR has target for nested routes

//...
    {
      path: '/features',
      name: 'features',
      getComponent(nextState, cb) {
        System.import('containers/FeaturePage')
          .then(loadModule(cb))
          .catch(errorLoading);
      },
      childRoutes: [
        {
          path: '/features/test',
          name: 'home2',
          getComponent(nextState, cb) {
            const importModules = Promise.all([
              System.import('containers/HomePage/reducer'),
              System.import('containers/HomePage/sagas'),
              System.import('containers/HomePage'),
            ]);

            const renderRoute = loadModule(cb);

            importModules.then(([reducer, sagas, component]) => {
              injectReducer('home', reducer.default);
              injectSagas(sagas.default);

              renderRoute(component);
            });

            importModules.catch(errorLoading);
          },
        }
      ],
    }
//...

somewhere in FeaturePage container...
export class FeaturePage extends React.Component {
  //...
  render() {
    return (
      <div>
        {React.children.only(this.props.children)}
         //...

Ah, thanks! Number one was my bad. Should have known about number two, I see that the same happens in the index of the App container now that I know what it does. My mistake after all...