Code pane needs horizontal scrollbar

During an archboard review, I was often not able to (easily - my mouse does have a way to get a cursor but may not be intuitive to everyone) scroll horizontally on some of our longer method names. E.g.:

This makes reviews difficult. It would be great if a horizontal scrollbar was added as needed.

@heaths With a mouse that has a scroll wheel (which, I'm guessing, is almost all of them these days??) if you press and hold the scroll wheel down, and then move your mouse to the left or right of where your original click occurred, you should see horizontal scrolling occur. Does this work for you (it does for me)?

It does. I mentioned I did that, but not sure everyone would know about that. If we want everyone using this, having an intuitive and familiar UI may be a better user experience overall.

@JonathanGiles I'll take this one

@heaths If you have a moment, can you try installing a browser extension such as "Super Styles", so that you may make local modifications to websites. If you do, please go to the APIView website and apply the following CSS in the extension:
#review-right {
  overflow-x: visible !important;
}

The end result is an always-visible horizontal scrollbar, but I would argue that the usability is less than what we have today (so long as you know about the mouse wheel click trick to enable horizontal scrolling). Keen to hear your thoughts.

I saw a mockup of this by @cmcd22 yesterday that looked like this:

The problem with this image is that the API viewport becomes exceedingly small.
I still tend to prefer the suggestion that people horizontal scroll with their 'only up/down' mouse wheel by pressing the mousewheel down and moving the mouse horizontally. I wonder which mouse wheels do not support horizontal scroll (I would have assumed it was an operating system feature, rather than a hardware-specific feature)

The padding in that image definitely makes it works. I wouldn't add the padding and then it doesn't look so bad.
Not all mice support a middle click, though maybe in time that is solving itself.

@praveenkuttappan @chidozieononiwu are you please able to give your expert opinions on the approach taken in the screenshot above and if you believe this will aid usability? Cheers

FWIW, I'd get rid of the effective padding, though. Keep everything like it is, but make the divs scrollable. I came close to a mock-up using F12 dev tools but you have to use a fixed (or calculated, which is where my CSS 1.1 skills fail me) height/width for whichever axis you're trying to scroll.

Am not clear what is the ask here. It seems that this is already present either using the mouse wheel or the scroll bar itself.

Am not clear what is the ask here. It seems that this is already present either using the mouse wheel or the scroll bar itself.

Not all mouse wheels move horizontally, and there's no horizontal scrollbar.
What I'd like to see is vertical and horizontal scrollbars - as needed - for both the TOC and content.

Perhaps it's a browser compatibility issue. What web browser are you using. I ask because I see horizontal scroll bar in the main content in (Edge, Chrome and Mozilla). As for the table of Content a scroll is also present although not deployed yet. But it is fixed in staging right now.

I'm using Edge Dev and see no horizontal scrollbar:

@chidozieononiwu I know we went back and forth with this feature. What was the conclusion? and what is the state today?

The horizontal scroll is present even in Edge Dev.

Only if you scroll the main content down, though. It seems someone did already change the content regions: TOC has no scroll on overflow, but probably fine since it's typically long enough to see enough of a type; and th emain content does have a horizontal scrollbar but not properly taking into account the height above it seems. Here's me before scrolling the main content region (and doesn't matter if I maximize on a 2K resolution:

But now if I scroll the main content down, but losing all the useful toolbar features at the top:

It seems the height of the page about the main content area wasn't taken into account. Perhaps the height should be a CSS calculation.
Using Edge Dev version 107.0.1418.8 (Official build) dev (64-bit).

@chidozieononiwu the ask here is to make sure the horizatonal scroll bar is always available, not only if the user scrolls down

Do you want the toolbars and scrollbars to be always visible? that will require calculating the space to allocate for the content based on user screen size. That is possible but I suspect as someone else will complain about the space that will be eaten up by the toolbars and footer. I think the best thing here is to scroll with the mouse wheel which I think is the best experience.

Ohh right. I was one of those users complaining about all the space losted because of the scroll bars :|

The scrollbars themselves don't take much. In email discussions when the UI that we sort of have now, there's a lot of whitespace on the sides and bottom we don't need. Get rid of that, and the loss of space is none to negligible (none if the scrollbars aren't necessary). If the scrollbars aren't visible, there's no point in having them and it's probably just better off as it was.
If you know the height of the top and nix the bottom whitespace (like it was before), you can use height: calc(100% - {header height}).

Resolve in https://github.com/Azure/azure-sdk-tools/pull/5385


Fixed SSLv3 auth check

Input check for sslv3 auth options were wrong

Even with this fix, there was another problem. Can you paste the demo code/command in here?


Feat: Added swift 5 support

Hi please check, I have updated the code project to swift 5 compatible.


WIP: [CLI] Add new option -r to resolve local file references

When the CLI is used, the local path to the schema is known. Therefore, if some other files are referenced with the ref key, the program knows where to search for them.
In the documentation of json-schema, it is stated:

$ref can also be a relative or absolute URI, so if you prefer to include your definitions in separate files, you can also do that. For example:
{ "$ref": "definitions.json#/address" }
would load the address schema from another file residing alongside this one.

And:

The $id property is a URI that serves two purposes:
It declares a unique identifier for the schema.
It declares a base URI against which $ref URIs are resolved.
[…]
But be aware of the second purpose of the $id property: that it declares a base URL for relative $ref URLs elsewhere in the file. For example, if you had:
{ "$ref": "person.json" }
in the same file, a JSON schema validation library that supported network fetching would fetch person.json from http://foo.bar/schemas/person.json, even if address.json was loaded from somewhere else, such as the local filesystem.

I think that feature can be supported by the command line. I therefore added the -r option.

The main issue I have is that I don’t know how to make tests about that. My understanding of the current CLI tests is that there is no actual file at all, only fake openings. In that case, how to test this feature ? Maybe by redefining some of the os methods ?

Hi! Sorry, haven't gotten a chance to look at this yet but thanks again for the PR! Really appreciated.
Will try to give some guidance on how to test this (I suspect before digging to hard that an easy way to test it may be to rely on the metaschema files that are already shipped in the repo, and using those as the one you reference via relative references)
Also, on the option itself, I think my first reaction is I'd rather call this something like --base-uri? (Similar to what's in #669 which is about this essentially). What's being specified is a base URL that's used to resolve relative references, yeah?

wow I totally forgot that PR, sorry. I’ll try to continue it soon.


Redesign Meltano Hub

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/210
Originally created by @alexmarple on 2022-03-09 19:53:22

The current design does not make complete use of our new brand guide nor does it use its space particularly well.
Some examples of sites that the Hub should resemble post-refresh:

https://www.moderndatastack.xyz/
https://www.npmjs.com/

The NPM site is the main inspiration for the new site.
Additionally a Meltano Hub logo needs to be built (whether it's adding the correct wordmark or an entirely different emblem for the Hub is up for debate but probably the former).
The new Hub will require the following designs:

A new homepage (v1)

A large search bar
Links to loaders, utilities, and extractors
Links to a sign on/sign in area
Details on what the Hub is/does

A search results page (v1)
A 404 page (v1)
A single item page (loader, extractor, utility) (v1)

Displays any and all data we have on the item

An all items page (all loaders or all extractors or all utilities)

Can mostly be a clone of the search results page

A sign in/sign up page (v2)

An auth page to allow a user to log in or sign up

A dashboard page (v2)

This will display all of the items that the user has created/published
Bookmarked items?

Since we currently don't have a designer on staff I am not creating mocks for the above but simply working on them in the moment.

View 1 previous comment from the original issue on GitLab

@tayloramurphy How does this relate to https://github.com/meltano/hub/issues/205?

@DouweM we're going to convert this to a discussion and close the other in favor of a new issue that @aaronsteers will make


Cannot read views using geoparquet files while reading from s3 compatible storage

Describe the bug
Using 0.2.12 (Metabase v0.51.3 + DuckDB v1.1.3) Cannot read views using geoparquet files while reading from s3 compatible storage
It worked in version 0.2.9.  (duckdb 1.0.0 , metabase 0.50.33)
To Reproduce
Example view:
create or replace view my_schema.my_geo_table   as
SELECT OBJECTID, CODIGO, SHAPE_Length, SHAPE_Area, codigo_municipio,
CODIGO_DEPARTAMENTO, periodo_anio, unique_id, WKB, WKB_bbox
FROM read_parquet('s3://mybucket/bronze/geo/**/GEOPARQUET/my_geo_table.*.parquet',  hive_partitioning = false);

Example error:

2024-12-09 15:17:18,679 ERROR models.field-values :: Error fetching field values
clojure.lang.ExceptionInfo: Error executing query: java.sql.SQLException: Binder Error: Contents of view were altered: types don't match! Expected [BIGINT, VARCHAR, DOUBLE, DOUBLE, VARCHAR, VARCHAR, INTEGER, VARCHAR, GEOMETRY, STRUCT(xmin FLOAT, ymin FLOAT, xmax FLOAT, ymax FLOAT)], but found [BIGINT, VARCHAR, DOUBLE, DOUBLE, VARCHAR, VARCHAR, INTEGER, VARCHAR, BLOB, STRUCT(xmin FLOAT, ymin FLOAT, xmax FLOAT, ymax FLOAT)] instead {:driver :duckdb, :sql ["-- Metabase" "SELECT" "  \"bronze_cat_geo\".\"r_sector\".\"WKT\" AS \"WKT\"" "FROM" "  \"bronze_cat_geo\".\"r_sector\"" "GROUP BY" "  \"bronze_cat_geo\".\"r_sector\".\"WKT\"" "ORDER BY" "  \"bronze_cat_geo\".\"r_sector\".\"WKT\" ASC" "LIMIT" "  1000"], :params nil, :type :invalid-query}

Caused by: java.sql.SQLException: java.sql.SQLException: Binder Error: Contents of view were altered: types don't match! Expected [BIGINT, VARCHAR, DOUBLE, DOUBLE, VARCHAR, VARCHAR, INTEGER, VARCHAR, GEOMETRY, STRUCT(xmin FLOAT, ymin FLOAT, xmax FLOAT, ymax FLOAT)], but found [BIGINT, VARCHAR, DOUBLE, DOUBLE, VARCHAR, VARCHAR, INTEGER, VARCHAR, BLOB, STRUCT(xmin FLOAT, ymin FLOAT, xmax FLOAT, ymax FLOAT)] instead

Caused by: java.sql.SQLException: Binder Error: Contents of view were altered: types don't match! Expected [BIGINT, VARCHAR, DOUBLE, DOUBLE, VARCHAR, VARCHAR, INTEGER, VARCHAR, GEOMETRY, STRUCT(xmin FLOAT, ymin FLOAT, xmax FLOAT, ymax FLOAT)], but found [BIGINT, VARCHAR, DOUBLE, DOUBLE, VARCHAR, VARCHAR, INTEGER, VARCHAR, BLOB, STRUCT(xmin FLOAT, ymin FLOAT, xmax FLOAT, ymax FLOAT)] instead
        at org.duckdb.DuckDBNative.duckdb_jdbc_prepare(Native Method)

Environment

Docker :  FROM eclipse-temurin:23.0.1_11-jre-noble

Thanks for the report. Is there a sample geoparquet file with this schema you can share with us?

You can find an example in the following file:
dane_mgn_mpio_politico_2021_4326_15.zip


Data Labeling (exploration whether it's already done)

Full pictures ofp arking lot:
Labels saved in XML to every picture.
XML contains BoxId, Position, Height and Width, Classification.
Segmented pictures:
Classification in folder title. Id in the picture name. Same id as in the xml.


fixed error when not authenticated

Checking data.uid was causing an error because data: null.

Great, thanks for contribution


[UIFlow 2.0] [JP] System (additional)

Additional translation System category in UIFlow 2.0 into Japanese.

Translation has finished.


Telemetry work

Summary
Telemetry work, second review
Description
Removed design document docs
Addressed most feedback from 1st review
Additional Reviewers
@aaron-congo
@aaronchung-bitquill
@sergiyv-improving
@karenc-bq
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

There are some compilation errors due to the API changes.


nodePackages.imapnotify: init at 0.4.1

Motivation for this change
I'd like to add imapnotify to nixpkgs because it allows the user to be notified by the IMAP server upon new mail (push vs pull). It is a nice addition to isync (which is already part of nixpkgs). Also I could not find another tool providing this kind of functionality within nixpkgs. If there is anything missing, please let me know.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

this is now part of nixpkgs.


UI: Change Font Size of Creator Text

Category

[ ] Documentation
[ ] Resource Addition
[ ] Codebase
[x] User Interface
[ ] Feature Request

Description

We need to change the font size of the landing page creator text from 10px to 16px:

For Assignee:

Ensure that you submit a PR for this issue within 1 day. Otherwise, you will be unassigned from this issue and it will be passed on to the next contributor who asked to be assigned to it.

Note: This issue is for JWoC 2024 participants. If you are not participating in this event, please refrain from asking to get assigned to this issue.

@Sriparno08 please assign this issue to me under JWOC'24

Can you please assign me this issue under JWOC?

I have read the issue and I would like to work on this issue.
Please assign it to me under JWOC'2024.


System-tests: brokered: extend brokered jms TopicTest about shared/non-shared durable subscription

test shared durable subscription with max count of concurrent consumers
non-shared durable subscription - max count of consumers is set to 1

done: https://github.com/EnMasseProject/enmasse/pull/452


feat: is it possible to prevent re-renders?

WIP: Just an idea. Can we prevent unnecessary re-renders?

@alexpchin I reopened #41 and now I think it's going to really improve the performance. check out the new demo, it's super simple compared to the current implementation and much more bug-free. The downside is it does not come with a tab bar by default, but it's going to be super simple to provide a custom one.
This isn't my priority right now, but I'm going to try to update it in the next few days and make a reusable component.
I'm commenting here so maybe you don't waste too much time trying to improve the current implementation, preventing re-renders.
If you have something to add to the PR now or in the future, please let me know

@alexpchin I reopened #41 and now I think it's going to really improve the performance. check out the new demo, it's super simple compared to the current implementation and much more bug-free. The downside is it does not come with a tab bar by default, but it's going to be super simple to provide a custom one.
This isn't my priority right now, but I'm going to try to update it in the next few days and make a reusable component.
I'm commenting here so maybe you don't waste too much time trying to improve the current implementation, preventing re-renders.
If you have something to add to the PR now or in the future, please let me know

Closing as this is v2 focused.

Closing as this is v2 focused.


update to pnpm 8 and allow newer node versions

https://github.com/pnpm/pnpm/releases/tag/v8.0.0

FYI, some related work here https://github.com/vercel/turbo/pull/5030, but there were a couple issues with it that I didn't have time to resolve.

Current dependencies on/for this PR:

main

PR #5235   👈

This comment was auto-generated by Graphite.

@SocketSecurity ignore next@*


Add AWS first step in wizard

TODO:

Fix layout of 45 - ( it happens because of a CSS override by the Wizard)
Fix the aria labels to match the types
Can we cache the instance types list for now? because it is not dependent on a source only for a region and we only have one region.

@amirfefer  I think the font is bold, how can I change it?

I think the font is in bold, how can I change it?

which one?

Works well locally, I failed to deploy to ephemeral unfortunatelly, but no reason to block on that.

Does that mean you can see the instance types list? :)

Does that mean you can see the instance types list? :)

Yes, exactly! :)


Font Request: Pop!_OS

Pop!_OS is a desktop environment for Linux based on Ubuntu.
A PNG logo is available here. I'm afraid I do not know where to find an SVG logo. Hope it's alright.
Thanks. :smiley:

SVGs are available here.


Libmpdclient updates

Description of changes

Picking https://github.com/NixOS/nixpkgs/pull/275968 and expanding it
Things done

Built on platform(s)

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3141

Result of nixpkgs-review pr 276068 run on x86_64-linux 1

27 packages built:

ario
ashuffle
clerk
gjay
gmpc
libmpdclient
mpc-cli
mpd
mpd-notification
mpd-small
mpd-small.doc
mpd-touch-screen-gui
mpd.doc
mpd.man
mpdas
mpdcron
mpdscribble
mympd
ncmpc
ncmpcpp
polybarFull
sfwbar
vimpc
waybar
yambar
yambar.man
ympd

LGTM
protip: write closes #275968 in the OP and leave the other PR open or maybe put it on draft


option to write to file (if output is different)

Currently, the output is to stdout.  However with a timestamp or version info this might create different output every time.  This is fine for quick generation, but not good in other cases.
We want a flag to write to file only if the output is significantly different.  If the script is different in only comments or a timestamp, then godownloader should not overwrite the file.
Needed for #39
Blocked by #56

this works now.  closing.


Messaging: Cannot create multiple messengers in the same context anymore

Overview
I noticed this while working on #21 while updating the demo extension. You should be able to do this:
const messenger1 = defineExtensionMessaging();
const messenger2 = defineExtensionMessaging();

Messages sent to and from messenger1 should not trigger listeners for messenger2, and vice-versa.
Why is this important? Because each @webext-core/proxy-service sets up it's own listeners internally which should not effect any other proxy services or other messenger definitions.
Reproduction
Run the packages/messaging-demo extension, and press any of the buttons on the popup.
Todo

[x] Locate the regression https://github.com/aklinker1/webext-core/commit/8c536db4f5dbb8232b92516d5065855639eb9bf6#diff-4e590c2a32420547792e5d50f072fdd36ab78cade1ba59ea629c1ac44075a04cR87-R89
[x] Fix it 71c654da9a0fde64f9ce7eb59bea650632a1fa0d
[x] Add tests so this doesn't happen again 71c654da9a0fde64f9ce7eb59bea650632a1fa0d

Closed by https://github.com/aklinker1/webext-core/commit/71c654da9a0fde64f9ce7eb59bea650632a1fa0d


Make Android app shortcut use 'Home Assistant' as name.

Currently, when you create a shortcut to Home Assistant on Android, it suggests the name "Assistant" instead of "Home Assistant".
I think "Home Assistant" is the correct name for the app to clearly differentiate with Google Assistant or other Assistants.
Check out the animation on this page to see the "Assistant" name.

This should not have been merged and needs more discussion. The reason the name is "Assistant" is because otherwise the name will be cutoff on the home screen and it will be "Home Ass…"

(that's also why the property is called short_name)

Hmm, it's "Home Assi.." on default font size here, and "Home Assista.." on small font.
The downside is that Android always uses the short name, like in the task switcher where there is enough space for the full name.
I've gotten used to all the abreviations on the home screen that I haven't noticed it's abreviated. I did notice the "Assistant" name wasn't actually correct.


VideoFormatChanging change size of my video size

I have problem. Size of the video is changing. It narrows to near-minimum. with it I have event VideoFormatChanging.

I can't understand your problem, can you give me a snapshot?

Hello. I need feedback, please.

What's your VideoFormatChanging event handler?

http://blog.higan.me/content/images/2017/07/test.gif

Can you explain me, why my video size spontaneously changing?
and after or before that VideoFormatChanging event happened?


ViewPager alignment broken with more than 1 item

🐛 Bug Report
When setting just a single <Layout> item in my ViewPager, similar to how the documented example does, I see that the layout is correct. Example code:
function Component() {
    //snip
    return (
        <SafeAreaView>
            <ViewPager
                selectedIndex={selectedIndex}
                onSelect={handlePageSelect}
            >
                <Layout level="2">
                    <Card>
                        <Text>This is a sample card</Text>
                    </Card>
                </Layout>
            </ViewPager>
        </SafeAreaView>
    );
}

After I add more items, I get the following layout issue:
function Component() {
    //snip
    return (
        <SafeAreaView>
            <ViewPager
                selectedIndex={selectedIndex}
                onSelect={handlePageSelect}
            >
                <Layout level="2">
                    <Card>
                        <Text>This is a sample card</Text>
                    </Card>
                </Layout>
                <Layout level="2">
                    <Card>
                        <Text>This is another sample card</Text>
                    </Card>
                </Layout>
            </ViewPager>
        </SafeAreaView>
    );
}

To Reproduce
Steps to reproduce the behavior:

Create a view pager component
Add Layouts/Cards/Text as shown above

Expected behavior
Layout and scrolling should behave like the example on the website
UI Kitten and Eva version

Package
Version

@eva-design/eva
2.1.0

@ui-kitten/components
5.1.0

Environment information

  System:
    OS: macOS 10.15.7
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
  Binaries:
    Node: 14.16.1 - /usr/local/bin/node
    npm: 6.14.12 - /usr/local/bin/npm
  SDKs:
    iOS SDK:
      Platforms: iOS 14.4, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
  IDEs:
    Xcode: 12.4/12D4e - /usr/bin/xcodebuild
  npmPackages:
    react: ~17.0.1 => 17.0.1
    react-native: 0.64.1 => 0.64.1

User error.. sorry.


Unlock Person Credential by Business Card

Credential Definition IDs:

https://candyscan.idlab.org/tx/CANDY_PROD/domain/350
https://candyscan.idlab.org/tx/CANDY_PROD/domain/351

Hey team! Please add your planning poker estimate with Zenhub @wadeking98 @jleach @bryce-mcmath

@cvarjao Is this a circular dependency? To get a business card you need to do something with registry for which I believe a requirement is the person credential.

No. People only need BC Services Card App to login to BC Registries.

Noted. I must have misunderstood.

Blocked. Working to test test credentials.

Feature in 1.0.13 Build 1400

Conformed fixed by Peter in 1.0.13 1403.


Exception rasied if modified the class's property

Hi nick,
I am trying to adopt fastcoding lib. I found if the class property deleted or renamed will get a entire uninitialized object.
The reason I found setValue to a non existed property will raise an exception:
    for (__unsafe_unretained NSString *key in definition->_propertyKeys)
    {
            [object setValue:FCReadObject(decoder) forKey:key];
    }

The quick fix is add a try-catch to prevent the exception.
    for (__unsafe_unretained NSString *key in definition->_propertyKeys)
    {
        @try {
            [object setValue:FCReadObject(decoder) forKey:key];
        }
        @catch (NSException *exception) {

        }
    }

However if there is another better solution?

You could override setValue:forUndefinedKey: on your class and have it do nothing instead of crashing.

Wow, this feels better. Thank u.


Shorten QC messages

Fixes #46 by removing some redundancy in the QC messages.
Signed-off-by: pkruczkiewicz<EMAIL_ADDRESS>

@gcttong I cleaned up the QC messages a bit in 4a81526468b761d56404d13c4bfcf31b78b623ab.

ok alright, looks good!


Configure Reactotron in Storybook

Remember that in order to make it work you should set up in src/app/main.tsx
const SHOW_STORYBOOK = true

I discovered today that this solution breaks the storyshot tests. After some trial and error I came up with an alternate solution:
    if (typeof __TEST__ === "undefined" || !__TEST__) {
      const Reactotron = require("../src/services/reactotron")
      const reactotron = new Reactotron.Reactotron()
      reactotron.setup()
    }

And in package.json add:
  "jest": {
    "preset": "jest-preset-ignite",
    "testPathIgnorePatterns": [
      "/e2e/",
      "/node_modules/"
    ],
+    "globals": {
+      "__TEST__": true
+    }
  },

And in test/setup.ts add:
+ declare global {
+   var __TEST__
+ }

It's less pretty, but handles both use cases.

Closed by https://github.com/infinitered/ignite-ir-boilerplate-bowser/pull/87


Unexpected results?

I'm trying to do a FPS analysis of The Ascent on PS4. I'm getting weird results, 40-45FPS while I was expecting 30 or less. Is this something we could expect with parts of the screen being fully blacked out? Also seeing this with other scenes were there is color/sometihng happing without the black parts on the screen. I was wondering if someone is willing to have a look at the recording/run it through Trdrop. Btw seeing the same on PS4 Pro.
https://mega.nz/file/BYI3TRjb#DnRKTr_Kbp6FyP-OnRtaXMqBtVpzc8oRNqu8TmK-xSg

Can't download on Mega, file size too big + Quota limitations

Can't download on Mega, file size too big + Quota limitations

Thanks for the reply, fixed the quota issue. If still unable to download I’ll upload smaller samples.
Additional I noticed it was not really detecting some freezes, I believe you had similar issue before.
Appreciated!

Yea, upload smaller sample or another host like Google drive

Uploading atm, this is the url to the folder: https://drive.google.com/drive/folders/15CLRPLrh_sCBhv-9g0lu_ELLBHQuX_Bw?usp=sharing
Uploading two samples, first one should be visible/complete within 15 minutes. My upload speed is limited to 50Mb/s unfortunately. In the meantime I noticed you can disable Motion Blur in this title, will see if that makes any difference at all.

Update on this from my side:
Additional things tried:
Disable Montion Blur. Same results.
Played with Pixel-diff (just for the heck of it). No difference until you go in very high numbers and FPS drops to nothing.
Crop image to just process a piece of the screen. Same results.

Somehow it feels extremely sluggish but potential there is an unlocked FPS going on?
Freezes definitely not being detected or insufficient.
Trying to verify with the studio on Twitter (long shot) what the target FPS should be.

Atm no other ideas.


Framework website only works on Chrome based browsers

The site https://js.ozapp.io only works on Chrome based browsers due to its usage of attachShadow which is not implemented in other engines. It would be great if the page for a web framework worked on all major browsers.
Screenshot from Firefox:

Fixed?

Fixed for what ? Firefox nightly or another browser ?

@Banou26 this is firefox nightly 60, it is failing on attachShadow for me.

I'm closing this one since Firefox nightly(and soon normal Firefox) now supports the shadowDOM


Implement get_transaction method on sendMessage

While we're dealing with transactions, I though it would be great to expose sendMessage transactions.
This would allow storing job data in redis and publishing it to consumers in one atomic transaction and avoid "orphan" jobs that may occur on producer failures with non atomic transactions.

Thanks for your contributions. It seems like it may be worthwhile to implement this pattern across all the transactions for sake of consistency and flexibility

Thanks for your contributions. It seems like it may be worthwhile to implement this pattern across all the transactions for sake of consistency and flexibility

Makes sense.
However, this got me thinking: the transaction yielded here is somewhat non confusing - it uses the server time in the transaction creation, not execution.
That's ok, but might be confusing, since the visibility timeout would be relative to that time, and not to the message send time.
This is fine as long as you create the transaction right before you send it and prepares all the other actions ahead of time, but it's no very intuitive.
That might also be the case in some other methods. I'm not sure what we should do here and whether this can be improved.
Any thoughts?


quiet option to avoid "requesting list . done" in the output

Hi,
I have written a small shell script around docker-ls, and I would love to have a quiet -q option to avoid those characters "requesting list . done" in the ouput.
Thanks for the tool.

Just noticed that "-progress-indicator=false" does the job. Sorry.


Saved Data is outdated

https://github.com/neoforged/Documentation/blame/main/docs/datastorage/saveddata.md#L21

DimensionDataStorage#computeIfAbsent takes in three arguments: a function to load NBT data into a SD and return it, a supplier to construct a new instance of the SD, and the name of the .dat file stored within the data folder for the implemented level.

On 1.20.2, it takes in 2 arguments instead. The first argument is a record instead of function. The record in which takes in default constructor for the Saved Data, NBT deserializer, and nullable DFU Type.

Closed by #47


syncplay module: init

Motivation for this change
Working Syncplay server.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I'll merge after tests pass.

They are useful for certFile actually, in my configuration I have ACME-provided certificates which are readable by group ssl and syncplay is ran from this group. Sadly syncplay doesn't support dropping privileges, and of course I didn't want to allocate a dedicated system user for it.
On September 3, 2019 2:02:49 AM GMT+03:00, Silvan Mosberger<EMAIL_ADDRESS>wrote:

Infinisil commented on this pull request.

   default = null;

   description = ''

     TLS certificates directory to use for encryption. See

     <link

xlink:href="https://github.com/Syncplay/syncplay/wiki/TLS-support"/>.

   '';

 };

 user = mkOption {

   type = types.str;

   default = "nobody";

   description = ''

     User to use when running Syncplay.

   '';

 };

 group = mkOption {

Is there any point in making user and group configurable? You could
also try to use DynamicUser
--
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/NixOS/nixpkgs/pull/67972#pullrequestreview-282713203

--
Nikolay.


Problem in importing util

The "utils.py" file in the "bak" folder does not contain the attributes (dateGenerator, unify_word and padding) whereas you have used them in the "genFeatureMat_Glove.py" file to create feature matrix.
Where can I find the actual "util.py" file?
Thanks in advance

Hi, @Manoj98 , I changed the framework to a new version, it is much more powerful and easier to understand, you are welcome to take a look at it.

Hey, Thanks for the update on the new version. Will work on it.


maint: general structure

This PRs updates the general structure of ast_tree.py and fixes a issues related to the CustomFunctions class.
Some adjustments were needed due to the fact arguments can be different between the custom functions and the autogenerated one. In this case, the arguments within the custom functions are the ones used.

Codecov Report
Attention: Patch coverage is 76.06838% with 28 lines in your changes missing coverage. Please review.

Project coverage is 84.28%. Comparing base (67f530a) to head (c540a5e).

Files with missing lines
Patch %
Lines

src/pyconverter/xml2py/custom_functions.py
81.35%
11 Missing :warning:

src/pyconverter/xml2py/formatter.py
10.00%
9 Missing :warning:

src/pyconverter/xml2py/ast_tree.py
88.63%
5 Missing :warning:

src/pyconverter/xml2py/cli.py
25.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #356      +/-   ##
==========================================
- Coverage   84.55%   84.28%   -0.27%
==========================================
  Files          13       13
  Lines        2337     2418      +81
==========================================
+ Hits         1976     2038      +62
- Misses        361      380      +19

Flag
Coverage Δ

coverage
84.28% <76.06%> (-0.27%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


MockTracer use text map propag as default constructor

Resolves #177

Coverage increased (+1.5%) to 75.593% when pulling 7574312cc6c7fac4fbcec141c8a5dd7101cf56e1 on pavolloffay:mock-use-text-map-propag-be-default into a0fc80516b3ad1ce8fd45e7d0fc27529ea3b0c0b on opentracing:master.

If there are no issues I will merge in 48h.
cc @opentracing/otsc @opentracing/otiab

Coverage increased (+1.0%) to 74.086% when pulling b06ecb4598a598e6b59f188f0a99c71f969e3a12 on pavolloffay:mock-use-text-map-propag-be-default into e000eb0c62ee3149ae272c6c197bfc039d2c2749 on opentracing:master.

anybody from @opentracing/otsc to approve the PR?

Coverage increased (+0.9%) to 75.602% when pulling cfd8e1983d2fd6ce6cbc29c4433f341658da75af on pavolloffay:mock-use-text-map-propag-be-default into dd75c9dc3bb887ae90533ba22bba8d0ed51c94a3 on opentracing:master.

@pavolloffay Was thinking if we should re-target this to 0.31 directly (as that's the new defacto 'upstream' branch). Let me know, as I could re-target it myself (so we can enjoy it for RC 2 ;) )

As a reference, I re-targeted this patch for v0.31.0 - and the result is here: https://github.com/pavolloffay/opentracing-java/commits/master. @pavolloffay hope this helps to get it integrated soon ;)

Was thinking if we should re-target this to 0.31 directly (as that's the new defacto 'upstream' branch). Let me know, as I could re-target it myself (so we can enjoy it for RC 2 ;) )

@carlosalberto what about rebasing v0.31 against the master before publishing RC2?

@pavolloffay That's actually a good idea. So, lets merge this change and to the rebase later today.

+1 IIR there are more commits in master than in v0.31.. I will merge


feat: Add JavaScript code generation in Candid

This results in very similar output as didc, but it is officially part of the Candid library.

Closed in favor of https://github.com/dfinity/candid/pull/52


The title's a bit misleading - No Azure Storage Table API info

[Enter feedback here]
Hi, it seems that the page purely talks about Azure Cosmos API's and not about Azure Storage Table Storage API

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

ID: ef1295f9-eab5-8918-ab99-3a20b8723805
Version Independent ID: 0e551ce1-761a-d379-7b6c-4caf49c5bbb2
Content: Azure Cosmos DB Table API using .NET Standard SDK
Content Source: articles/cosmos-db/tutorial-develop-table-dotnet.md
Service: cosmos-db
Sub-service: cosmosdb-table
GitHub Login: @sakash279
Microsoft Alias: akshanka

I send this too soon, the article exactly says the API content is the same.  I think I scrolled to the source portions too quickly for my quick copy-paste instant gratification bug.   The title's not misleading at all,
Sorry about that :(

@ndrix We are glad you were able to find the information you needed.


Fix Centered Content Switcher

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #142
Test URLs:

Original: https://www.esri.com/en-us/about/about-esri/americas
Before: https://main--esri--aemsites.hlx.live/en-us/about/about-esri/americas
After: https://improve-centered-content-switcher--esri--aemsites.hlx.live/en-us/about/about-esri/americas
After: https://improve-centered-content-switcher--esri--aemsites.hlx.live/en-us/about/about-esri/mea-central-asia
After: https://improve-centered-content-switcher--esri--aemsites.hlx.live/en-us/capabilities/indoor-gis/capabilities/indoor-positioning-navigation

For this one there's a text-transform: uppercase that shouldn't be applied to the text. Maybe you can apply it only for p:first-child


Remove Invalid Tokens from cache when found

Description

Uncache our TI token if an InvalidToken is caught and found.

Issue
#1326
Checklist

[ ] I have added tests to cover my changes
[ ] I have added logging where useful (with appropriate log level)
[ ] I have added JavaDocs where required
[ ] I have updated the documentation accordingly

Note: You may remove items that are not applicable

/review

/describe

/improve


Add support for CAA record types

Reference https://en.wikipedia.org/wiki/List_of_DNS_record_types and
https://en.wikipedia.org/wiki/DNS_Certification_Authority_Authorization

I'm having to close this for now as it hasn't received any input in a while and it's unlikely this will get traction any time soon. Please comment/update if you think this is still relevant and we can reopen this :+1:
Thank you for your effort nonetheless, keep it up! :+1:


Move onnx related tools to umbrella

onnx-as and onnx-dis are related to onnx and not that tight to onnc.
For cleaner project architecture, moving them to umbrella will be better.

How umbrella build these tools since most configuration kept in the onnc's building system? Any proposal or PR in umbrella?

I'll create Makefiles of each tool, and related them to build sctipt.

I've added Makefiles and build commands to the build scripts in umbrella.


Included Support of OPA Gatekeeper

Signed-off-by: Nitishkumar Singh<EMAIL_ADDRESS>
Users can now install OPA gatekeeper using arkade
Description

Motivation and Context

[x] I have raised an issue to propose this change (required)
Closes #314

How Has This Been Tested?

Install Gatekeeper
arkade install opa-gatekeeper
Create OPA constraint template

kubectl apply -f https://raw.githubusercontent.com/open-policy-agent/gatekeeper/master/demo/basic/templates/k8srequiredlabels_template.yaml

Create rule based on above created constraint template

kubectl apply -f https://raw.githubusercontent.com/open-policy-agent/gatekeeper/master/demo/basic/constraints/all_ns_must_have_gatekeeper.yaml

The above steps ensure that gatekeeper is installed successfully.
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

[ ] I have tested this on arm, or have added code to prevent deployment

/set reviewer: Waterdrips

/set reviewer: Waterdrips

@nitishkumar71 Is the OPA Gatekeeper chart multi-arch? Has it been tested on arm64/armv7?

@nitishkumar71 Is the OPA Gatekeeper chart multi-arch? Has it been tested on arm64/armv7?

I haven't tested in arm64/armv7 personally. but they do support these arch. Please look at their dockerhub Images arch.
As well as their makefile include build release for these platforms

I would like to see OPA release a configurable chart, and that would make a great app. However these two static manifests don't look like a very generic example, it appears to just install a hard-coded sample.
Can you help me understand what the app does @nitishkumar71 ?

I would like to see OPA release a configurable chart, and that would make a great app. However these two static manifests don't look like a very generic example, it appears to just install a hard-coded sample.
Can you help me understand what the app does @nitishkumar71 ?

Gatekeeper allows to create templates for policies, which can be used to create multiple policies with different parameters. It helps in creation of generic rule across organization where different teams can pass different parameters as per their requirement.
Their are some standard templates being managed by community, you can have a look on them.

I would like to see OPA release a configurable chart, and that would make a great app. However these two static manifests don't look like a very generic example, it appears to just install a hard-coded sample.
Can you help me understand what the app does @nitishkumar71 ?

Not sure I understand @alexellis - This is using the OPA helm chart.

It did not seem to be when I reviewed it. Can you check if this helm chart is current and maintained?

This is the official OPA Gatekeeper chart - Same one I use in other projects.

Sounds good to me. Let me know if you are happy with the changes you requested and then let's get this merged.


Update web-and-mobile.md

The permission under microsoft.web/serverfarms/firstpartyapps/ would not be used to general users.
So we should remove this information to avoid granting unnecessary permissions for any custom roles.

@rolyon
Can you review this PR? Sign off or approve if you accept the changes.
Note: To sign off, add a #sign-off comment to the PR.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @atsushina
Thanks for your pull request. Closing because these changes were made in a different pull request.
https://github.com/MicrosoftDocs/azure-docs/pull/123153
thanks

#please-close


Government sources

Have all the governments listed on the outreach page (https://github.com/microsoft/Open-Maps/wiki/Government-Outreach---US-States) given explicit permission to use data for OSM or have they just been contacted?

We are only listed those which have given explicit permission.


Wrong file extension if the file name has spaces

Hello,
On Android, the method openDocb64 select a wrong file extension (.pdf) if the file name has spaces.
Here is a piece of code that doesn't work (it will try to open the file in the pdf format):
let payload = {
    base64: 'base64data',
    fileName: 'name with spaces',
    fileType: 'png'
};

if (Platform.OS === 'android') {
    payload.cache = true;
}

OpenFile.openDocb64([payload]);

I look at your code and it seems the problem is here
Maybe you could change your code a little bit like this?
private static String getMimeTypeByUrl(String url) {
    System.out.println("Url: " + url);
    String extension = MimeTypeMap.getFileExtensionFromUrl(url);
    return getMimeTypeByExtension(extension);
}

private static String getMimeTypeByExtension(String extension) {
    String mimeType = null;

    if (extension != null) {
        mimeType = MimeTypeMap.getSingleton().getMimeTypeFromExtension(extension);
    }

    if (mimeType == null) {
        mimeType = "application/pdf";
        System.out.println("Mime Type (default): " + mimeType);
    }

    return mimeType;
}

Thnaks for your hard work!

I m also facing same issue

Hi @philipphecht,
The problem here is due to MimeTypeMap.getFileExtensionFromUrl not working with spaces.
JRMC223


Updated

hey there, I assume you don't need that PR so i'll close it but feel free to reopen and tag me if you still need it.


allow padding HumanBytes Display output

I've updated the Display implementation for HumanBytes so it can accept formatting parameters.

So what exactly is this trying to solve? It seems unfortunate that this will need an additional allocation.

The goal was to be able to output HumanBytes with formatting parameters.
Ex:
#[test]
    fn test_human_bytes() {
        use std::fmt::Write;

        let mut s = String::new();
        write!(&mut s, "{:>10}", HumanBytes(10_000)).unwrap();
        assert_eq!(s, "  9.77 KiB");
    }

I agree though I would not want to incur an additional allocation cost. If there is a better alternative I'd go with that.


Add a check for bad GHSA entries.

Also add a best-effort fix up attempt for some of these. If fixing them is not possible, we'll just ignore them in the meantime.

friendly ping!


Issue 115 - Support return preference

issue #115 - refactor FHIRPersistence interface to return Response wrappers.
now the PersistenceLayer can return OperationOutcome AND Resource(s) at the same time which will be needed for honoring the return preference indicated by the client.
issue #115 - use return preference from Prefer header

add support for setting a default HTTPReturnPreference in fhir-client
introduce create and update tests for checking the response to returnPreferences
add OperationOutcome to FHIRRestOperationResponse
set FHIRRequestContext.HttpReturnPreference in the FHIRRestServletFilter (default="minimal")
update return entity based on HTTPReturnPreference in FHIRRequestContext
update BundleTest to pass return=representation since the tests depend on this

Does not appear upload logs happened on this PR.  I kicked off another build just in case.

Could this be refactored to remove the SuppressWarnings?
        public Builder<T> resource(@SuppressWarnings("unchecked") T... resource) {
            for (T value : resource) {
                this.resource.add(value);
            }
            return this;
        }

Fixed a bad merge

set DCO to pass.  The commit was a minor commit.


Unable to use same Id after task transform

Problem:

If I transform a task to another type the id will be changed and I can't change it back to the old one

Steps to reproduce:

Drag task from pallet and change id to "myId"

Transform task to User Task (the id will be automatically changed)

Try to change id back to "myId"

Expected Behavior:

I can use "myId" again for the transformed task id
(Optional) The id isn't changed during transformation

Ok, so the issue is that IDs are not released, after element deletion. Otherwise you'd be able to assign it back.
We went for the current approach (giving a new ID to the element) on purpose. Our thinking was: The element will be a new one so this should be indicated through an ID change.  It may be more convenient for users if the ID would stay the same though.
We'll probably need to iterate over that again and come up with something clever ... :wink;

Need fix / refactoring in bpmn-js (via bpmn-io/bpmn-js#421).

We would be very happy, if ids would not change, when a bpmn element is transformed.
We identify bpmn elements in our workflow engine by the id. If the id changes the means of identification is lost. Our engine does not recognize the elements anymore and the linked behaviors (scripts, screens, ...) are lost.
A solution could be, to introduce a permanent uuid as an extension to the current element id. But I think I would prefer the id simply to not change. (Though I can understand the idea, why the id changes.)

@alpheios We closed this issue, because the original bug is fixed. But we opened another feature request issue regarding the id retaining when morphing: https://github.com/bpmn-io/bpmn-js/issues/513


Fix color conversion from RGB to xyY

Description
Fix color conversion from RGB to xyY space.
Tests
Added testcase for xyY conversion in transformc testsuite.
Checklist:

[x] I have read the contribution guidelines.
[ ] I have previously submitted a Contributor License Agreement.
[ ] I have updated the documentation, if applicable.
[x] I have ensured that the change is tested somewhere in the testsuite (adding new test cases if necessary).
[x] My code follows the prevailing code style of this project.

I believe this need an update of test reference output.

Indeed! Didn't notice it wasn't automatically updated! Should be good now.

I also pushed a fix regarding test compilation on Windows, where casting nullptr to intptr_t is invalid.


rm , of jobProperties. jackson can not parse it

👍
@scusjs do you mind fillig out CLA here: http://druid.io/community/cla.html

👍

ping @scusjs

@fjy I have filled it


DHCP Start Address is <IP_ADDRESS>

Having tried the tool I have worked through and added all the tags, roles etc. I have now got it to attempt to sync a prefix into DHCP.
When it does it sets the start address as <IP_ADDRESS>.
Logs
2023-03-27 09:35:00 INFO netbox_windhcp::sync - Start sync from http://netbox/api/ to localhost
2023-03-27 09:35:00 INFO netbox_windhcp::sync - Found 1 Prefixes and 1 Ranges
2023-03-27 09:35:00 INFO netbox_windhcp::sync - Sync Prefix <IP_ADDRESS>/24 - TEMP_LAN
2023-03-27 09:35:00 INFO netbox_windhcp::sync::windhcp::subnet - Set range to <IP_ADDRESS> - <IP_ADDRESS>
2023-03-27 09:35:00 ERROR netbox_windhcp_sync - Error setting subnet range to : The parameter is incorrect.

PREFIX
{
	"id": 17,
	"url": "http://netbox/api/ipam/prefixes/17/",
	"display": "<IP_ADDRESS>/24",
	"family": {
		"value": 4,
		"label": "IPv4"
	},
	"prefix": "<IP_ADDRESS>/24",
	"site": {
		"id": 1,
		"url": "http://netbox/api/dcim/sites/1/",
		"display": "Test",
		"name": "Test",
		"slug": "Test"
	},
	"vrf": null,
	"tenant": null,
	"vlan": null,
	"status": {
		"value": "active",
		"label": "Active"
	},
	"role": null,
	"is_pool": false,
	"mark_utilized": false,
	"description": "",
	"comments": "",
	"tags": [
		{
			"id": 1,
			"url": "http://netbox/api/extras/tags/1/",
			"display": "dhcp",
			"name": "dhcp",
			"slug": "dhcp",
			"color": "9e9e9e"
		}
	],
	"custom_fields": {},
	"created": "2023-03-24T12:53:32.895043Z",
	"last_updated": "2023-03-27T08:50:18.852579Z",
	"children": 0,
	"_depth": 2
}

IP RANGE
{
	"id": 5,
	"url": "http://netbox/api/ipam/ip-ranges/5/",
	"display": "<IP_ADDRESS>-254/24",
	"family": {
		"value": 4,
		"label": "IPv4"
	},
	"start_address": "<IP_ADDRESS>/24",
	"end_address": "<IP_ADDRESS>/24",
	"size": 245,
	"vrf": null,
	"tenant": null,
	"status": {
		"value": "active",
		"label": "Active"
	},
	"role": {
		"id": 1,
		"url": "http://netbox/api/ipam/roles/1/",
		"display": "dhcp-pool",
		"name": "dhcp-pool",
		"slug": "dhcp-pool"
	},
	"description": "",
	"comments": "",
	"tags": [],
	"custom_fields": {},
	"created": "2023-03-27T08:54:11.520715Z",
	"last_updated": "2023-03-27T08:59:42.168995Z"
}

Hi @lewis-green
Are you able to share the current set range on the DHCP server with me? For example the output of the following command?
Get-DhcpServerv4Scope -ScopeId <IP_ADDRESS>

Regards Marius

@jiuka There is no scope. Does it not create them if missing?

@lewis-green it is created and then the range is set. So it should be there now, but with no or a wrong range. <IP_ADDRESS> as a start address. The setting of the range is a bit tricky as it is not possible to move a range only shrink or expand one. In this case my suspicion is that the <IP_ADDRESS> is returned from the DHCP Server as the start of the current range. Is there realy no Scope or just no range? Which version of Windows DHCP server are you using?
Regards marius

I've got this:
ScopeId          : <IP_ADDRESS>
Name             : <omitted>
Description      : <omitted>
SuperscopeName   :
SubnetMask       : <IP_ADDRESS>
StartRange       : <IP_ADDRESS>
EndRange         : <IP_ADDRESS>
LeaseDuration    : 00:00:00
NapProfile       :
NapEnable        : False
Delay(ms)        : 0
State            : Active
Type             : Dhcp
MaxBootpClients  :<PHONE_NUMBER>
ActivatePolicies : True

It's Server 2022 DHCP.

It seems running it in VS Code with the source code it works fine but the 0.9.4 Installer doesn't work right.

I added additional logic to limit the range to the scope 561541432a2fa0299f4ebbb778f967d2ba159bc8. However there seems to be some (maybe caching) problems if the installer does not work but the compiled version does. Still investigating this.

@jiuka It seems it all worked straight away with this 0.97 build :)


Altered the console to set the canonical hostname rather than an alias

The /etc/hosts man page describes a difference between the "canonical_hostname"
(fqdn) and the aliases for that hostname.
This difference is reflected in the behaviour of some external tools such
as cloud-init and freeipa.
These tools will retrieve the hostname (typically via the hostname
command or from /etc/hostname) then consult /etc/hosts to find the
fqdn by taking the name in the "canonical_hostname" place in the line
where the found hostname is an alias.
This can cause problems when the application acts differently based on
whether the fqdn is "localhost" or not.
https://bugzilla.redhat.com/show_bug.cgi?id=1286830

:+1: we need this.


(BKR-954) Add Ability to Disable Openstack Volume Management

Prior to this commit beaker openstack support assumed that the
volumes service was provisioned.  This service is not always
available in an Openstack deployment.  This commit gives you the
option to disable that functionality.  If you do not disable it
on an instance that does not publish this functionality, even if
you don't configure the volumes setting in CONFIG, beaker runs
will fail.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3735/

@tvpartytonight Updated variable name

@ferglor Corrected spelling

@kevpl Took a crack at simplifying that logic, please review.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3741/

@ipcrm looks lgtm to me right now. Please prepend the beaker ticket reference to all commits in the history to match our CONTRIBUTING standard.
Once that's done, we can look at merging this in.

@kevpl Commits updated.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3804/

@ipcrm thanks! looks like a transient failure, re-running now.
@puppetlabs-jenkins retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/3805/


iOS 14 default browser Linkin.canOpenURL issue

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
On iOS 14 we have the ability to change the default browser. So, after I change it to Chrome when I call Linking.canOpenURL with any URL it returns false. But if I did not check and call Linking.openURL function it works and the link is opened in Chrome.
React Native version:
System:
OS: macOS 11.0
CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Memory: 129.34 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 12.18.3 - ~/.nvm/versions/node/v12.18.3/bin/node
npm: 6.14.8 - ~/.nvm/versions/node/v12.18.3/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.9.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 14.0, DriverKit 19.0, macOS 10.15, tvOS 14.0, watchOS 7.0
Android SDK: Not Found
IDEs:
Android Studio: 4.0 AI-193.69<IP_ADDRESS>26763
Xcode: 12.0.1/12A7300 - /usr/bin/xcodebuild
Languages:
Java: 11.0.8 - /usr/bin/javac
Python: 2.7.16 - /usr/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: ^16.13.1 => 16.13.1
react-native: ^0.63.3 => 0.63.3
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

Change the default browser to Chrome
Call Linking.canOpenURL() with any URL
It will return false

Expected Results
When the default browser is changed Linking.canOpenURL() should return true

Did you check if this is an iOS bug? may be check ones.

add https & http to LSApplicationQueriesSchemes in Info.plist

add https & http to LSApplicationQueriesSchemes in Info.plist

This fixed the issue for us - thanks!

add https & http to LSApplicationQueriesSchemes in Info.plist

e.g. like so:
	<key>LSApplicationQueriesSchemes</key>
	<array>
		<string>http</string>
		<string>https</string>
	</array>

worked for us as well, thanks!

add https & http to LSApplicationQueriesSchemes in Info.plist

e.g. like so:
	<key>LSApplicationQueriesSchemes</key>
	<array>
		<string>http</string>
		<string>https</string>
	</array>

worked for us as well, thanks!


[Book] How to modularize & grow your code w/ 0.6

It's easy to end up with a very large src/main.rs / mod app with RTIC.  Having a next steps section with ideas on how to grow and modularize code would be a nice way to end the book.
Could also touch on creating structs for multiple shared resources, and using an impl block to keep their code together.

Yes, I think this is a very important point. Currently I see this as the biggest area where RTIC can improve.
It's very hard to structure a complex RTIC application.
I'm not sure if it was a bit hasty to already make a 1.0 release, because I believe that work in this area would involve a lot of breaking changes...

You can use the app just as a binding for entry points using the extern keyword. This allows for modularization of the code.
We are a bit limited by the current procmacro support in Rust. As RTIC needs to perform system-wide analysis all task/resource information needs to be in a  single item (the mod).
Food for thought:
You can think of RTIC just as cortex-m-rt, but with the (much needed) addition of race- and deadlock-free concurrency in safe user code.
The message passing, timers etc. are just icing on the cake. One can have other abstraction on top of RTIC, e.g., asynchronous actors or concurrent reactive objects (essentially actors with both synchronous asynchronous communication).
In this sense RTIC is just the lowest level of abstraction needed for building concurrent applications.

Related to this: Issue #639

The message passing, timers etc. are just icing on the cake. One can have other abstractions on top of RTIC, e.g., asynchronous actors or concurrent reactive objects (essentially actors with both synchronous asynchronous communication).
In this sense RTIC is just the lowest level of abstraction needed for building concurrent applications.

I would be very interested in the available higher level abstractions available the work nicely along RTIC. For example I could use a pub/sub (or event-broadcast) system that 'unlocks'/starts one or more tasks when some event has happened. Think Semaphores/Eventgroups as available in other RTOSses.
Having a (curated) list of available options (maybe even with some example code! 👍🏻) would be of great help to use RTIC in more advanced scenarios (i.e. real world embedded work).

There is an Actor API PR that provides higher level of abstraction (broadcasting).
https://github.com/rtic-rs/cortex-m-rtic/pull/537


Fixing nuspec for nuget generation.

Description
Fixing nuspec for nuget generation.
Fixes https://github.com/dapr/dotnet-sdk/issues/112

Out of curiosity @amanbha, why is Dapr using nuspec to generate .nupkg files? Is there something preventing you from packing  packages using the .csproj instead? For more information, please refer to:

https://docs.microsoft.com/en-us/nuget/create-packages/creating-a-package-dotnet-cli
https://docs.microsoft.com/en-us/dotnet/core/tools/csproj#nuget-metadata-properties

@loic-sharma  Nuspec allows more control and flexibility on how and where you want to put files, (we are putting icon files in the pkg itself) and provides more control specially if you decide to put multiple dlls form different projects into one nuget pkg.

@loic-sharma Nuspec allows more control and flexibility on how and where you want to put files, (we are putting icon files in the pkg itself)

Packing from a .nuspec is more flexible, but it is also significantly harder. NuGet has conventions for things like asset compatibility, you will continuously run into issues if you choose to manually pack with a .nuspec. Also, it is possible to pack arbitrary files and or to embed icons using .csproj.

and provides more control specially if you decide to put multiple dlls form different projects into one nuget pkg.

Out of curiousity, why do you need this? I believe you can do this too by including the assets from your project references. This seems like a weird scenario though...

@loic-sharma yes, simplicity is there with .csproj, we picked flexible over simple. We will definatley change it if we run into further issues.


feat: add doomModules option

Context
As a doom-emacs user
I want to distribute my doom modules as nix derivations
So that I can include them in multiple configurations of doom.d that do not require my personal config
Rationale
I would like to have a way to offer my personal doom-emacs modules as a git repository that later can be included in nix-doom-emacs installation. By using the new lib.mkDoomModules, I can easily create a derivation that contains the modules I want included on a doom.d folder that gets constructed using nix-doom-emacs.

The diff looks reasonable at a first glance, but frankly I am scared of even more things to maintain when I'm using a 2022 version of NDE, and @thiagokokada is using bare un-nixified Doom due to the various current issues with this project.
I like to blame #353 for the majority of problems, but Doom & Emacs are also just giant things to package correctly.
How has NDE been working for you? I'm surprised it hasn't exploded more for you.

The diff looks reasonable at a first glance

Thanks, I would be more comfortable if I added some testing. I have not dug into how the CI tests of this project work to see if I can add a new one; I'll check that out.

I am scared of even more things to maintain when I'm using a 2022 version of NDE

I understand. I can run my projects with my fork for a while with no problem and ask other interested parties to experiment with my branch as well. Luckily is simple to work with forks in nix flakes inputs. I wonder what is stopping you with working with the master version?

@thiagokokada is using bare un-nixified Doom due to the various current issues with this project.

Ooof

How has NDE been working for you? I'm surprised it hasn't exploded more for you.

On NixOS, I didn't have many issues. On macOS, I struggled a little, and that's why I created the evil-collections flake input PR (#466)

This seems cool! Can it be expected to help with https://github.com/nix-community/nix-doom-emacs/issues/421 ?

Tried building with my doom.d, with the following flake
{
  inputs = {
    nixpkgs.url = "github:nixos/nixpkgs";
    nix-doom-emacs.url = "github:roman/nix-doom-emacs/roman/feat/doom-modules";
  };
  outputs = { self, nixpkgs, nix-doom-emacs }:
    let
      system = "x86_64-linux";
      pkgs = import nixpkgs { inherit system; };
      doom-emacs = nix-doom-emacs.packages.${system}.default.override {
        doomPrivateDir = ./doom.d;
      };
    in
    {
      devShells.${system}.default = pkgs.mkShell {
        buildInputs = [ doom-emacs ];
      };
    };
}

got this:
error: attribute 'evil-collection' missing

This seems cool! Can it be expected to help with # 421 / #324 ?

@quinn-dougherty No, not directly. This comes before the bottleneck build steps.


[Hub Generated] Review request for Microsoft.ResourceHealth to add version stable/2022-05-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[x] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

Hey this request is to fix the following s360 items:
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=260F1CBF-7480-46E2-B8E1-2C5C518FD39A
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=7A661FA3-A486-491A-96A2-D68649102642
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=D0645A0D-77A7-4AB5-B734-D42B2E3BEBEB
https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=647F3368-7670-4ADF-9D8D-6A6F02006D3F
The properties seem to have been added to the code but never made it to swagger
. the following is the date they were added to the code:
events endpoints: added to response code on 5/16/2019 for all api version
properties.article.articleId (string)
properties.article.parameters (IReadOnlyDictionary<string, string>)
properties.reason (string)
Availability status endpoints:
properties.ArticleId (string)-> added in 1/13/2022 to all api versions
properties.Context (string) -> added in 12/8/2022 to all api versions
properties.Category (string)-> added on 12/8/2022 to all api versions
properties.recommendedActions[*]._ActionUrl.Comment (string)-> added on 11/14/2018 to all api version
(adding this last property seems to be triggering a linter exception for DefinitionsPropertiesNamesCamelCase but the name cannot be changed as it is been served to clients for years)
I sent an email to breaking change team asking for guidance and they said to create the pr and that they would approve the fixes

The main validation error I see is  "Property name should be camel case."
I assume that if this is a S360 fix, then that is by design?
I don't see any other problem from API POV.

The main validation error I see is "Property name should be camel case."
I assume that if this is a S360 fix, then that is by design?
I don't see any other problem from API POV.

@TimLovellSmith yeah its a fix for https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=647F3368-7670-4ADF-9D8D-6A6F02006D3F


Major performance issues with the sortable tree example

Hey,
I just want to start by saying thank you for this awesome library!
I am trying to implement something similar to the sortable tree example in https://examples.dndkit.com/ (it's on the bottom of the list).
It works great in the example with only a few items but once you get to a couple hundreds of rows it starts to get extremely laggy.
The problem is whenever I am dragging one item, every row is getting re-rendered,
(you can see that if you add a render counter using a ref on the component and increasing it with useEffect).
By the time I am finished dragging one item, all the rows have been rendered a few hundred times which causes a major performance hit.
By looking at the React dev tools it seems that all the rows are rendering because the dragHandle prop is changed
I tried memoizing some stuff but had no luck. Would love to get some help on this
Thanks!

Hey @idanlo! I don't have hundres, but I have a list with duzens of cards (which have some html, animations, svgs etc) and I'm also struggling with performance. I can't barely start dragging 1 card and it gets extremly laggy. I have useMemos everywhere to optimize things, but no luck so far...
Somebody help?

+1 on this, have a use case where I need to draw hundreds of items and virtualization is not possible. Dragging, updates, placeholder movement are all extremely laggy. This is reflected on the MultipleContainers example, by adding in a few hundred cards.

+1 tried to replicate the tree example as well with upto 1000 virtualized elements, but with only like 50 elements it starts to lag on lower end machines

+1

+1

I noticed the useSortable hook is triggering a re-render every time the order is being changed on EVERY sortable item (and therefore ignoring React.memo).

prepared a PR fixing this issue https://github.com/clauderic/dnd-kit/pull/1096.

+1


Change cp.round to cp.around

cupy.round is used in stain normalization and it is removed from the newer version of cupy (although I didn't find the changelog related to that). This PR simply change cp.round to cp.around, which should be backward compatible.

closing in favor of https://github.com/rapidsai/cucim/pull/508


Update blob quickstart code sample

Snippets from this code sample are used in the blob quickstart for Go. Once this code sample looks good, I'll update the quickstart.

@sandeep-sen / @jhendrixMSFT
Could either of you please review when you get a chance? This is the code sample we pull the blob quickstart snippets from. Thanks so much!


⚠ BCR Bot has degraded performance

In 044af1d, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 4322 ms

Resolved: BCR Bot performance has improved in 9c57c8e.


Move between using from node_modules to using from local (scoped packages)

related: https://github.com/sb-mig/sb-mig/issues/96

link: https://github.com/sb-mig/sb-mig/issues/96

closing - we will not have lock file, it's making project to complex, and not really needed so much.


This package must have Assembly Definitions (Editor and Runtime)

Hi!
First of all, great job with this library, it is really great and useful.

Unity Version: 2019.4
Problem: the lack of assembly definitions make it impossible to build a game with this asset.
Solution: just create and include the assembly the definitions on the Editor and Runtime folders.

I hope to create a PR later today.

Hi @truongnguyentungduy, I still don't know exactly the particularities of how Unity does its stuff, but, AFAIK, once you use assembly definitions in your project, you have to use it everywhere, including 3rd party packages.
Once you use asmdefs in your project, the magic folder seems to lose their meaning for building. So, since the Hierarchy2's editor scripts are not contained with a assembly definition, when you build the project they are included in the solution.

I can't comment about build errors but there are 2 other benefits of adding Assembly Definition Files:

they are mandatory for UPM support
they help reduce compilation times of other scripts because Hierarchy 2's scripts won't have to be recompiled with them

If you are interested in UPM support, I'd like to promote the repository structure I've been using for my packages which has worked well for me with minimal changes.


Powertoys Run ran into an issue

Microsoft PowerToys version
"0.45.0"
Running as admin

[ ] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
Got this error when my ASUS tuf with ryzen 4600 went to sleep and it did not woke up (this happens to me) so I had to shut down and then restart using the power button. After that I opened quicklook on a pdf and then after a while some errors came out of nowhere when the powertoys completed its startup loading.
There is two windows with the same message as well if that is anything important...
And more appears as I try to use Power Run.
✔ Expected Behavior
Power Run should work as expected
❌ Actual Behavior
powertoys run show me that it ran into some issue
Other Software
Name: QuickLook
Publisher: Paddy Xu
Version: <IP_ADDRESS>
No response

Could you attach a full bug report file? Right-click on the tray icon and click Report Bug. Then drag-and-drop the generated file into the GitHub comment to upload it. Thanks!

PowerToysReport_2021-10-03-00-13-48.zip

Thank you for the bug report. Your logs look strangely similar to this issue:
https://github.com/microsoft/PowerToys/issues/13532
Would you mind trying to disable the Program plugin in PowerToys Run, restart the computer, and then seeing if the error still reproduces?

I remember having it after the 'blue screen of death' , I have not had any blue screen since then
So it LGTM

Thank you for the feedback.
Please let us know if it happens again.


fixed cases on the soql query

the cases on the soql query should be fixed

Hi, can you sign the Salesforce.com Contributor License Agreement? Thanks


Validate workers max count vs max Nodes count allowed by the Pods CIDR

How to categorize this issue?

/area networking ops-productivity
/kind enhancement
What would you like to be added:
Validation whether the configured worker pools max Nodes count does not exceed max Nodes count allowed by the Pods CIDR. The Pod network calculations that has to be considered are described in https://github.com/gardener/gardener/blob/v1.38.3/docs/usage/shoot_networking.md.
Steps to reproduce:

Create a faulty Shoot cluster.

spec:
  provider:
    workers:
      - name: worker-bvl3g
        maximum: 3
        minimum: 3
  networking:
    pods: <IP_ADDRESS>/23

According to https://github.com/gardener/gardener/blob/v1.38.3/docs/usage/shoot_networking.md, we have:
Pod network: <IP_ADDRESS>/23 (this is `.spec.networking.pods`)
nodeCIDRMaskSize: /24
-------------------------

Number of podCIDRs: 2 (2 ^ 1, where 1 = /24 - /23) --> max. Node count
Number of IPs per podCIDRs: 256

Note: nodeCIDRMaskSize can be costomized with the field spec.kubernetes.kubeControllerManager.nodeCIDRMaskSize.

Make sure that the 3rd Node is unhealthy

kube-controller-manager:
2022-01-18 08:21:01 {"log":"Error while processing Node Add/Delete: failed to allocate cidr from cluster cidr at idx:0: CIDR allocation failed; there are no remaining CIDRs left to allocate in the accepted range","pid":"1","severity":"ERR","source":"controller_utils.go:260"}

Node condition:
    - lastHeartbeatTime: "2022-01-18T15:25:42Z"
      lastTransitionTime: "2022-01-18T15:25:42Z"
      message: RouteController failed to create a route
      reason: NoRouteCreated
      status: "True"
      type: NetworkUnavailable

Why is this needed:
To prevent users to configure a high Nodes count that actually cannot be supported/provider because of the network config.
It also worths mentioning that spec.networking.{pods.services.nodes} fields are immutable -> when user hits similar issue, there is no other option than recreating the Shoot cluster with updated spec.

/assign


python311Packages.add-trailing-comma: 3.0.1 -> 3.1.0

Diff: https://github.com/asottile/add-trailing-comma/compare/v3.0.1...v3.1.0
Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 252433 run on x86_64-linux 1

4 packages built:

python310Packages.add-trailing-comma
python310Packages.add-trailing-comma.dist
python311Packages.add-trailing-comma
python311Packages.add-trailing-comma.dist


CSS injected by UnoCSS incomplete on first load

UnoCSS version
0.61.3
Describe the bug
See https://github.com/slidevjs/slidev/issues/1705, which includes a detailed description. And https://github.com/slidevjs/slidev/issues/1573, which also reflects this bug.
When using UnoCSS with its Vite plugin, it seems that sometimes some UnoCSS class definitions are missing on the first load. Many Slidev users have encountered this, as well as antfu. However, I can't make this bug stably reproduced. Open this issue to track the bug and gather more information from a wider range of UnoCSS users.
Reproduction
Failed to make it stably reproduced.
System Info
No response
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I haven't used slidev much, can you provide a minimal demo of the occasional problem? And the steps to follow, what is normal and what is a problem?

I haven't used slidev much, can you provide a minimal demo of the occasional problem? And the steps to follow, what is normal and what is a problem?

Thanks for your concern. This bug happens very randomly at a low rate. There is a video in https://github.com/slidevjs/slidev/issues/1705, which shows the unexpected behavior.
In the video, the _uno.css wasn't fully loaded on the first load, but the other code was loaded properly. After reloading the page, _uno.css got fully loaded - you can see the response in the console became much longer.
Theoretically, this can be reproduced in any Slidev project, but at a super low chance (about 1/1000 I think).
There are too many factors inside Slidev, making it too hard to extract which usage of UnoCSS causes this. I think we may leave the issue there and wait for other people to report their findings in other projects.

This is still happening after #3984 is merged 😥 And on my local machine, I think this occurs more frequently than when I opened this issue.
For Slidev, I opened #1969 as a temporary workaround.


check for selinux already being disabled.

SE Linux returns a non zero exit code when already disabled.

Sorry it took me so long to have a look at this - I was doing some travelling and was busy. Thanks for raising this PR. The code looks good. I'm going to merge it and do a new patch release.


fix: ~~ and ~* these three operation expressions need to be improved

close #20

@Firstsawyou I can not sure if we can close issue #20 , please confirm


Proposal : Local-mode overlay driver

As of Docker 1.9 release, Overlay driver is a global mode driver which requires a KV-Store to synchronize distributed states. This proposal is to provide an optional way to operate overlay driver as a local-mode driver. That will essentially push the responsibility of distributed state management to higher-level mgmt layer.

@mavenugo It has been detected that this issue has not received any activity in over 6 months. Can you please let us know if it is still relevant:

For a bug: do you still experience the issue with the latest version?
For a feature request: was your request appropriately answered in a later version?

Thank you!
This issue will be automatically closed in 1 week unless it is commented on.
For more information please refer to https://github.com/docker/libnetwork/issues/1926


Create supervisord.md

@cody-clark can you update https://github.com/kubernetes/website/issues/5993 with terms you think should be included? I added supervisord because it was on that list


GetThreadDpiAwarenessContext return value is not correct

Actual behavior
The GetThreadDpiAwarenessContext API is generated as
		/// Gets the DPI_AWARENESS_CONTEXT for the current thread.
		/// <returns>The current DPI_AWARENESS_CONTEXT for the thread.</returns>
		/// <remarks>
		/// <para><see href="https://docs.microsoft.com/windows/win32/api//winuser/nf-winuser-getthreaddpiawarenesscontext">Learn more about this API from docs.microsoft.com</see>.</para>
		/// </remarks>
		[DllImport("User32", ExactSpelling = true)]
		[DefaultDllImportSearchPaths(DllImportSearchPath.System32)]
		[SupportedOSPlatform("windows10.0.14393")]
		public static extern winmdroot.UI.HiDpi.DPI_AWARENESS_CONTEXT GetThreadDpiAwarenessContext();

When the app is DPI_AWARENESS_CONTEXT_PER_MONITOR_AWARE_V2, the value is  0x00000022
Expected behavior
The value should be same to PInvoke.DPI_AWARENESS_CONTEXT_PER_MONITOR_AWARE_V2
		public static readonly winmdroot.UI.HiDpi.DPI_AWARENESS_CONTEXT DPI_AWARENESS_CONTEXT_PER_MONITOR_AWARE_V2 = (winmdroot.UI.HiDpi.DPI_AWARENESS_CONTEXT)(-4);

Repro steps

NativeMethods.txt content:

DPI_AWARENESS_CONTEXT_PER_MONITOR_AWARE_V2
GetThreadDpiAwarenessContext

NativeMethods.json content (if present):

{
   "$schema": "https://aka.ms/CsWin32.schema.json",
   "public": true
}

Any of your own code that should be shared?

app.manifest

  <application xmlns="urn:schemas-microsoft-com:asm.v3">
    <windowsSettings>
       <dpiAwareness xmlns="http://schemas.microsoft.com/SMI/2016/WindowsSettings">PerMonitorV2</dpiAwareness>
       <dpiAware xmlns="http://schemas.microsoft.com/SMI/2005/WindowsSettings">true</dpiAware>
    </windowsSettings>
  </application>

var dpiAwarenessContext = PInvoke.GetThreadDpiAwarenessContext();
if (dpiAwarenessContext == PInvoke.DPI_AWARENESS_CONTEXT_PER_MONITOR_AWARE_V2)
{
    ...
}

Context

CsWin32 version: [e.g. 0.1.619-beta]
Win32Metadata version (if explicitly set by project):
Target Framework: [e.g. net5.0-windows]
LangVersion (if explicitly set by project): [e.g. 9]

It looks I should use AreDpiAwarenessContextsEqual. And it works well.

Just to clarify, it's not in CsWin32's power to fix the numeric value being returned from the native extern call. What CsWin32 can do is make sure the return type is semantically right. Is the latter the problem?

I think it is not a bug even it is really confusing. GetThreadDpiAwarenessContext  returns correct type of DPI_AWARENESS_CONTEXT. Close this as not a bug.


fix(multiple): remove it

What: This removes the multiple props.

Why: Ref #93

How: Search for multiple in the project and remove it!

Checklist:

[x] Documentation
[x] Tests
[x] Ready to be merged
[ ] Added myself to contributors table N/A

Codecov Report

Merging #103 into master will increase coverage by 0.57%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #103      +/-   ##
=========================================
+ Coverage   15.12%   15.7%   +0.57%
=========================================
  Files           5       5
  Lines         271     261      -10
  Branches       62      59       -3
=========================================
  Hits           41      41
+ Misses        173     166       -7
+ Partials       57      54       -3

Impacted Files
Coverage Δ

src/downshift.js
17.34% <0%> (+0.84%)
:arrow_up:

Continue to review full report at Codecov.

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


Fixes #85 : Tab Reminder

Description
Added a Tab Reminder JS extension, which allows the users to set reminders to close specific tabs.
Fixes:  #85

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have made this from my own
[ ] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

You've crossed 200 points right? Keep checking if the PRs are invalid or not as per the guidelines.

@Sulagna-Dutta-Roy Okay mam


@mzikherman => Pass mediator to reaction

Related: https://github.com/artsy/reaction/pull/927

easy peasy


The problem of JVM GC in old gen

I setup an oauth2 system which is based on In-Memory token store. I tried the login stress test for my oAuth2 system. if the number of login users grows to 500K, the old gen is full. the server doesn't work well. i don't know why.
This is the JAVA_OPTS for Tomcat.
JAVA_OPTS='-Xms1024m -Xmx1024m -XX:PermSize=256M -XX:MaxNewSize=256m -XX:MaxPermSize=256m'
When the old gen is full, I checked the heapdump file, I found that the total memory of class LinkedHashMap$Entry(14$), ConcurrentHashMap$HashEntry(7%),HashMap#Entry  and Date(7%) is almost 35% of memory. Each memory of HashMap$Entry, LinkedHashMap, HashMap, HashMap$KeySet, LinkedHashSet, HashSet,Collections$UnmodifiableSet,DefaultExpiringOAuth2RefreshTOken,DefaultOAuth2AccessTOken, InMemoryTokenStores$TokenExpiry is 3.4%.
Because I close the other functions, I just test the function of creating the AccessToken, I have no idea.
Could someone give me some tips? Maybe, my configuration is wrong?
Please help me? If the data is not enough, I can give more data about my problem.

I think the reason is that the expire time of token is too long(30days), the token data is stored in Old Gen. It will be collected until the time expired. This is my assumption, I'm testing. But the following question is why the tokens is stored in Old Gen, they should be stored in token store?

Yes, the token data in Token_service is stored in Old Gen. But when the expired time passed, the data is still in Old Gen. Could you tell me why the data is still in Old Gen? or how to delete the expired token data in service?