Unable to narrow union to Record in TS 4.8

Bug Report

🔎 Search Terms
union record narrow type guard
🕗 Version & Regression Information

This changed between versions 4.7.4 and 4.8.2

⏯ Playground Link
Playground Link
💻 Code
function isObject<T>(it: T): it is T & Record<string, unknown> {
    return Object.prototype.toString.call(it) === "[object Object]";
}

declare function test(arg: Record<string, any>): void

declare const arg: (string | number | { a: "b" });

if (isObject(arg)) {
    // arg is:
    // - TS 4.7: {a: "b"}
    // - TS 4.8: (string | number | { a: "b" }) & Record<string, unknown>
    test({ ...arg }) // ERROR: Spread types may only be created from object types.(2698)
}

🙁 Actual behavior
arg stays as some intersection type, which TS does not recognize as an object type.
🙂 Expected behavior
arg gets narrowed to {a: "b"}

Bonus attempt using conditional types, not much better though:
type ExtractObject<T> =
	T extends readonly unknown[] ? never
	: {} extends T ? Record<string | number | symbol, unknown>
	: T extends Record<string | number | symbol, unknown> ? T
	: never;

function isObject<T>(it: T): it is T & ExtractObject<T> {
	return Object.prototype.toString.call(it) === "[object Object]";
}

declare function test(arg: Record<string, any>): void

declare const arg: (string | number | { a: "b" });

if (isObject(arg)) {
    // arg is:
    // - TS 4.7: {a: "b"}
    // - TS 4.8: (string | number | { a: "b" }) & { a: "b" }
    test({ ...arg })
}

Playground Link

Note that narrowing arrays broke in a similar fashion, but here I was able to work around it with a conditional type.
Works in 4.8:
type ExtractArray<T> =
	T extends readonly unknown[] ? T
	: {} extends T ? (T & unknown[])
	: never;

export function isArray<T>(it: T): it is T & ExtractArray<T> {
	if (Array.isArray != null) return Array.isArray(it);
	return Object.prototype.toString.call(it) === "[object Array]";
}

Used to work until 4.7:
export function isArray<T>(it: T): it is T & unknown[] {
	if (Array.isArray != null) return Array.isArray(it);
	return Object.prototype.toString.call(it) === "[object Array]";
}

function isObject<T>(it: T): it is T & object & Record<string, unknown>
Does this work for you?

Looks like a step in the right direction. Still need to find a way to remove arrays from the type though.

Not just unions are affected:
Not only unions. Here is a simpler example:
https://www.typescriptlang.org/play?ts=4.8.2#code/GYVwdgxgLglg9mABAZQKZgCYCEA2cBGAFBgIZQkBci4A1mHAO5gA0i6EcGMYA5lbfSYBKRAG8AsAChEMxDGCJC7Ttx6IAhAF5N1TKmDdUGRADITbSCt4btiAOQgowABx2RE6bK9QAFgCdGRDBUBkQAUT8Av0I7ZS5eNwBuKS8AXykU2QB6LMQAWTgQAGdURB84ADdUP0QoOEQS0oxUKFRoI1qATwAHUsyZHMQAdR90aiLVRAAVZEQKgBYAOmdFgCY5KDkisVTEAB9dZoNgjH7EQZGx4smZuaWAdkX5ja3EACJHFzf9w-1DU88MjiqmSklSoKAA
function SendBlob(data: unknown, encoding: unknown) {
    if (encoding !== undefined && encoding !== 'utf8') {
        throw new Error('encoding');
    }

    // Mouse hover to see detected type
    // When using TS v4.8.2 it is {} | undefined
    // When using TS v4.7.4 it is "utf8" | undefined
    encoding;
};

@AlCalzone can you provide some test cases that show why @whzx5byb’s solution of T & object isn’t sufficient? It works fine for the only case in your repro.

@andrewbranch Sure thing:
Playground Link
function isObject<T>(it: T): it is T & object & Record<string, unknown> {
    return Object.prototype.toString.call(it) === "[object Object]";
}

declare function test(arg: Record<string, any>): void

declare const arg1: (number | any[] | { a: "b" });
declare const arg2: (number | any[] | { a: "b" });

if (isObject(arg1)) {
    // spreading works:
    test({ ...arg1 });
    // this is not an error:
	const t = arg1.concat;
	//    ^?
	// t is unknown
}
// ↑ broken
//====================================================
// ↓ works

function isObjectFixed<T>(it: T): it is {} extends T
	? // Narrow the `{}` type to an unspecified object
	  T & Record<string | number | symbol, unknown>
	: unknown extends T
	? // treat unknown like `{}`
	  T & Record<string | number | symbol, unknown>
	: T extends object // function, array or actual object
	? T extends readonly unknown[]
		? never // not an array
		: T extends (...args: any[]) => any
		? never // not a function
		: T // no, an actual object
	: never {

	// This is necessary because:
	// typeof null === 'object'
	// typeof [] === 'object'
	// [] instanceof Object === true
	return Object.prototype.toString.call(it) === "[object Object]";
}

if (isObjectFixed(arg2)) {
    // spreading works:
    test({ ...arg2 });
    // this is now an error
    arg2.concat; // concat does not exist on type {a: "b"}
}

Sorry, had to edit this a few times, I messed up in between.

Ah. Yeah, since functions and arrays are assignable to our object, I’m inclined to say this is working as intended, and what you’re checking for is indeed more complex, hence the several conditions you need in the predicate. Perhaps object’s name is a bit unfortunate. What it really is is the non-primitive type. (In fact that’s what we name it in the checker.) But for what it is, not what it’s named, I think the behavior is expected.

Strangely T & Record<...> did work just fine in 4.7 and narrowed T to its Record-compatible subset.
Just like T & unknown[] narrowed T to its array subset and no longer does now.
Is it really supposed to be this complex to make sure one is dealing with an actual Record (or an actual array)?

What’s an “actual Record”? Lots of things that are not made out of object literals have named properties that can be accessed, which is what the {} and interface syntaxes mean in TypeScript. () => {}, [], and "hello" are all examples of a Record<"length", number>. Such is the nature of a structural type system.

I’m honestly shocked that the definition of isObject worked before. It’s totally inconsistent. Even in 4.7:
type T1 = (string | number | { a: "b" }) & Record<string, unknown>; // (string | number | { a: "b" }) & Record<string, unknown>
declare let arg2: T1;
test({ ...arg2 }); // error

I just inlined the intersection you’re performing in the type predicate. If you expect to get { a: "b" } out of that intersection, it means you think string & Record<string, unknown> is never, which has never been true. I’m actually totally baffled that the type guard acted like it was.

distinguish those from other (structurally compatible) things

Philosophically, this is just going to be a heavy lift, if not impossible, in a structural type system. TypeScript reasons much more about what you can do with a value than what it is. What’s returned by a value’s toString() is very nearly outside the scope of the language.

If you expect to get { a: "b" } out of that intersection, it means you think string & Record<string, unknown> is never, which has never been true. I’m actually totally baffled that the type guard acted like it was.

Heh, I guess I just went along with what TS gave me when I wrote this function that way. That behavior was pretty stable since 3.3 (or earlier, dunno).

@andrewbranch Technically I guess it’s not accurate to call object the non-primitive type either since I recently realized primitives can be assigned to { toString(): string } which in turn is assignable to object.  I don’t really know what to call this odd duck of a type anymore, heh.  Maybe just “{} with extra steps” 😆

Regardless of if this expected behaviour, it is a breaking change. Perhaps it should be rolled back and re introduced in a v5?
in our case we are comparing primitives and getting this same error:
type ValueType = (string | number | bigint | Date) | undefined | null;

const isEmpty = <T extends ValueType>(val: T) => {
    return val === undefined || val === null || val === '';
};

export const customComparer = <T extends ValueType>(a: T, b: T, locale: string) => {
    if (isEmpty(a) || isEmpty(b)) {
        return compareWithEmptyValues(a, b);
    }

    if (typeof a === 'string' && typeof b === 'string') {
        return a.localeCompare(b, locale, { sensitivity: 'base' });
    }

    return a < b ? -1 : a > b ? 1 : 0;
};

on v4.7 it worked fine but on v4.8.2 TS complains that a and b could be null on the last line.

@alebrozzo The TypeScript project here do not use semantic versioning, so you need to consider each minor version as most likely "breaking" and not just major ...


Add unit test for multiple onboards between issuing.

---
aries start PSM: send payload: sec pipe pack: indy pack message: Error: Wallet item not found
  Caused by: Item not found
  Caused by: Query returned no rows

Error is caused when existing agents try to execute protocols (issue credential) when new onboards has happened since their latest action. It seems that agent is not able to access (the correct?) indy wallet. The error may lie in the closing and opening logic for wallet manager, since it is repro only by onboarding multiple agents.

we continued together with the PR 230. I think this can be closed 🤔

Merged with #230


Add power statistics to AME UI

About the PR
The AME Controller UI now shows how much power the AME is currently supplying to the grid and how much it could supply at the current settings.
The numbers update on each "UpdateUi" of the controller, which is worst case every 10 seconds during AM injection. This will lead to a slight delay between increasing AME fuel and more power being shown, which reflects the internal behavior of the AME.
Why / Balance

There have been various discussions about making the AME more engaging and/or usable, and to improve the power distribution system in general for the engineering crew. This helps with both efforts. It clears up how much power is actualy produced right now, how much is used, and therefor if the AME is currently over-/undersupplying.
Technical details
Added variables all the way from the AME Controller System to the window definition to supply the numbers. The values are drawn directly from the attached PowerSupplierComponent of the AME Controller.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: The AME Controller now shows the amount of Power drawn from it and the maximum it can currently produce below the injection settings.

I'll take suggestions on what exactly the label in front of the numbers should say for maximum clarity for the players. Some ideas:

"Power draw" (simple, maybe too simple)
"Power used / available" (long, available might be misleading as "unused" power?)
"Power supplied / produced" (even longer, uses words that are also used in the code)

should be on separate lines imo

should be on separate lines imo

I thought about that too and decided for this version to keep the interface from becoming too cluttered

it needs a redesign anyway like the apc which now looks epic

I found an issue that the power readings were not updating properly after shutting down the AME. During the fix I used the opportunity and made a minimal effort "redesign" of the UI

maybe change "Increase" "Decrease" to "+2 cores" "-2 cores"?

its injection not cores
either +-2 injection or +- 1 core

maybe change "Increase" "Decrease" to "+2 cores" "-2 cores"?

I won't do that because I don't like that it changes the value by two. For experienced engineers that artificially limits how good you can finetune the AME. Since changing  that is on my list for possible upcoming changes to the AM I'm against hardcoding this number at more places.

having ame at double cores - 1 injection is extremely niche
you dont save much fuel at higher cores and you never need an ame with 1 core producing 40kw so theres no point

There are lowpop stations where Solars and a 2-core 3-fuel AME is viable. So 25% fuel savings.
A 1-core 1-fuel AME does 20kW
At least for me part of the Space Station 14 experience is the sandbox feeling - I can do lots of stuff even if they don't inherently serve a well defined gameplay purpose.
And all of that is not really related to this PR and I suggest we discuss this in depth when the time comes

well in that case having -2 -1 +1 +2 would be good so for most cases you click +2 but you can do +1 if you really want

Feel free to leave feedback under #20191

Thank you for the contribution. The delay between changing injection and updating "Max. power supply" is confusing. The power update delay is also not great.
I think it would be better if max power supply updated instantly after increasing/decreasing injection, and if the power update was increased to once every 3 seconds or so.

I agree with the part to update the readings every 3 seconds.
I don't like showing the players "false" numbers, but I can live with it when we rename it to "Targeted max. Power" or something to differentiate it from the Max. Power you see when Multitooling the HV cable. The difference is only there in edge cases (time until next injection, time of the last injection if not enough fuel is available) but still - it's something different.

I don't like showing the players "false" numbers, but I can live with it when we rename it to "Targeted max. Power" or something to differentiate it from the Max. Power you see when Multitooling the HV cable.

I think this is unnecessarily confusing for players and lightly insist that this be fixed.
Is there a technical reason that implementing this isn't worth it? It's a matter of updating the UI update function whenever the injection is changed, in addition to the automatic update every N seconds.

Ok somehow i have drastically messed up my branch, i'll put this as draft until that is resolved

Is there a technical reason that implementing this isn't worth it? It's a matter of updating the UI update function whenever the injection is changed, in addition to the automatic update every N seconds.

So the issue was, that the AME does not change the power output immediately when the injection settings are altered. The actual maximum output only changes at the next injection. So up to 10 seconds after a player changes the injection settings, the max.supply stays unchanged.
Thats what I meant with "false numbers" - a player turns up the AME, the UI says a number, but that number is not yet correct. As mendtioned above I resolved this by renaming the field "Targeted supply", to indicate that this is the intended and not necessarily the current maximum.
The UI now updates everytime something in the AME changes, or at least every 3 seconds to update the power draw readings.

Well I broke my local branch... again. Gonna fix this some time this weekend

Sorry for the force push, but now everything is as it should be. The review nitpicks have been addressed in the last three commits.


Update alert-dialog.mdx

Update alert-dialog.mdx
Pull Request Template
Description:
I have added an example of calling an alert dialog from outside.
Fixes Update alert-dialog.mdx
Tested Platforms:

[x ] Web
[x] iOS
[x] Android

Hi @psyrenpark,
Unfortunately, I cannot add the example to the documentation website. Although it looks like a valid way to get it to work, there are many ways to update the state of most components (ex: in a useEffect). In order to keep a standard in the documentation, it would require updating all other examples as well. This would mean too many examples which would make reading the documentation more difficult.
Thank you for taking the time to submit a pull request.


Bump versions

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6598013877

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6598013877
Status: success


Mention that Creating a binding for a CocoaPod dose not work with Version 1.8

I had CocoaPod v 1.8.x and I was stuck at step
**** Searching for requested CocoaPods**
after downgrading to CocoaPod 1.7.5 everything worked like a charm,
I believe that sharpie is not supported on 1.8.X and this should be mentioned in the documentation
this was discussed in this link
https://github.com/MicrosoftDocs/xamarin-docs/issues/173

Looking into it. This is the doc, correct? https://docs.microsoft.com/en-us/xamarin/cross-platform/macios/binding/objective-sharpie/examples/cocoapod?context=xamarin/ios

Yes, that is the documentation. I've been trying to get objective sharpie to work for the past 2 days with an updated library. pod install worked and installed the latest, while leaving sharpie pod init ios {NameOfLibrary} does not work and hang for 6+ hours.

Any update on this?
I have CocoaPods version 1.9.1 and running sharpie pod init ios AFNetworking takes forever. The installation hangs on this:
** Setting up CocoaPods master repo ...
   (this may take a while the first time)
Setup completed
** Searching for requested CocoaPods ...

Thanks for submitting an issue and we apologize for the delayed response. The requested updates haven't been made since the creation of the issue, and the timeline for resolution may vary based on resourcing, so the issue is being moved to an internal repo to better track it.


Upload Bxtcoin

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

Also, you can use these graphics to say that your ERC20 token fully compatible with Trust Wallet (https://trustwallet.com/logo-assets). Tokens will show up automatically once the user receives them.
Feel free to share this on social media and tag Trust Wallet:

Twitter: @trustwalletapp

Facebook: @trustwalletapp

When it is merged, does it mean that the logo of our coin will now display on trust wallet?

How do I verify that?


feat(profile): resolve all controls in profile

Description
Library functions to resolve all controls from a provided profile. Returns a map of source UUIDs to the respective controls. Intent of this structure is to properly map referenced source (from comp-defn) to the respective controls that are dictated by the profile.
Important Additions/Modifications:

Refactored the extraction of controls from a provided profile in component.go - the addition of this PR should allow for generation of component directly from a profile, HOWEVER, note the below constraint on profiles using internal refs
Updated network functionality to read from local files - should handle multiple path types, I think it was limited before on "file://" definitions which aren't always picked up for local paths
Unit tests for resolving catalog and profile controls

Important Note -
Including profiles with references to internal UUIDs (e.g., the way NIST does this profile) will not resolve.
Related Issue
Fixes #813
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Other (security config, docs update, etc)

Checklist before merging

[X] Test, docs, adr added or updated as needed
[ ] (n/a) Schema Updates applied
[X] Contributor Guide Steps followed

Temporarily pulling this down to address a bug I found in component generate - will also update those tests accordingly.

Temporarily pulling this down to address a bug I found in component generate - will also update those tests accordingly.

Realized that the AddControl logic was breaking the control search loop before child controls could be checked, so needed to re-order some of that functionality. Also updated the debugging logs in ComponentFromCatalog to accurately reflect controls not-found. Updated the TestComponentFromCatalog to check for an additional control that is nested under an excluded control.


Linux

You should make a Linux version

If you had looked at the README you would have found this.

LLL

BD is available for Linux already as noted by @qwerasd205


Don't clear sessions sessions when sessions expire

https://www.pivotaltracker.com/story/show/84382322
User gets 422 if they have 2 tabs open, one of which has an expired session, and they then submit a claim.
Make sure sessions are not cleared when they expire.

Fixed by @stevegraham in #535


If a problem occurs when update settings, the possibility which the user loses vim setting files exists.

If a problem occurs when update settings, the possibility which the user loses vim setting files exists.
The solution is needed...
An example of problems is "a user types Ctrl+C when update or switch".
https://github.com/taeguk/vimsw/blob/72e4cd88ba6674b769c0a47b0352e5d763f3576e/vimsw#L197
If a user types Ctrl+C in Line 197, vim setting files disappear..

Signal Handling can be a solution??
Or do backup before delete vim setting files.


Updated of Change Tracking

Changed "Change Data Tracking" to Change Tracking

@denzilribeiro : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off
Can the reviewer quickly edit this line and remove 'Data' from the description on the same updated line?
| Change Tracking | You will not be able to use Change Data Tracking with Hyperscale databases. |

correction to comment: "Data" was the word removed.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_ca500c1d-0226-447b-9804-1ab3bfe37c56/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_ca500c1d-0226-447b-9804-1ab3bfe37c56.git

Docs Build status updates of commit d856859:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_main_2022-02-16-06-03-42 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Tm669 frontend onboarding

Summary
I finished the frontend onboarding tasks.
Testing
I checked how my code was altering the frontend of the website.

Great work on frontend onboarding!


Could a SciPy-wide Reader protocol work with napari?

🚀 Feature
I have written a proposal for an overhaul of the lazy image I/O library PIMS. Starting from this idea, I see a path for establishing a new protocol across the SciPy ecosystem for "readers" that take some resource at rest (including but not limited to files) and return a suitable SciPy/PyData structure. This can be achieved in a distributed way without a dependency on or reference to any particular library. Perhaps it could be formalized as an informational NEP.
I discussed this over a private channel recently with @jni. I would like to explore whether the idea is compatible with https://github.com/napari/napari/pull/937 and napari's needs in general.
Motivation
It would be great if PIMS, napari, are-detector-handlers, intake, imageio, and so on could could use one another's I/O code. We all have our library-specific needs and design preferences, and I don't think it's possible or desirable to strip those away, but we might be able to make the fundamental "Turn this pile of bytes into a suitable SciPy/PyData structure," code mutually reusable.
Pitch
Here's the full version of the pitch: https://blog.danallan.com/posts/2020-03-07-reader-protocol/
Alternatives
Of course, we could continue with our parallel, semi-overlapping development. But it would be great if we could find some meaningful overlap.

Thanks a lot for opening this issue @danielballan!  I read through your full pitch and I like it a lot, we share lots of similar philosophies and ideas!  Though I can't speak for the whole team, I would venture to say "yes", this all is very compatible with what we started implementing in #937, and how we envisioned reader "plugins" working.  Of course, some of our implementation details would need to change, but not that many, and there are many ways in which what we started with is quite similar to your proposal (but less generalizable).  some scattered thoughts:

the general concept of a protocol that doesn't require any specific dependencies or libraries, and is easily discoverable with entrypoints is very attractive, and not dissimilar from what we tried to do with our reader plugins (which is captured in napari.plugins.hook_specifications.napari_get_reader. We also have a two-step "check first then read" process as you describe in your "why not just a function section".  Should something like your proposal pickup steam in the community, I don't think it would be difficult to modify our internals slightly to use it.
we also use plugin discovery using entry_points, which I think is great... and we could easily add your TBD.readers to the groups that we discover.  I do think we'd probably want to keep using pluggy as our "manager" of discovered implementations, due to the additional conveniences it provides around validating and calling plugin functions.
probably the most difficult part here is the return type.  I like your reader.container proposal (but I think that's the part that is still the "haziest" for me, in terms of general adoption).  Since napari internally can receive data as either numpy, dask or zarr, we didn't need to impose any restrictions on the type of container that plugins return... (but I do think some community standard would need to).  I suspect you and @jni talked a bit about the imtypes idea, which I gather has been in discussion here and other places long before I knew about it.  That might be more domain-specific than what you're picturing here. (If you're curious about the specifics of our current return type, this response I gave to another question is probably a decent place to start).

not directly related to readers, but worth mentioning, we have also had some discussions where we've allowed ourselves to dream a bit of a "community specification" for a variety of commonly applied analysis steps: wherein a set of function signatures and return types are declared, and then libraries can offer up implementations via entry_points.
Anyway, yes I think it's great what you've written!  We all should agree on a PathLike -> DataLike specification, and then use entry_points to discover them.  Let me know if you were looking for anything more specific than those (scattered) thoughts, or just curious whether we think we'd be able to use such a proposal within napari (yes i think so!).

Great, thank you. I agree with

probably the most difficult part here is the return type. I like your reader.container proposal (but I think that's the part that is still the "haziest" for me, in terms of general adoption).

It's the only part of the API that diverges from the analogy to built-in with open(...) as f: f.read(). Exactly as you said, for a community-wide standard we'd need some way to differentiate types so that the next layer up can impose type stability.
An exchange with @shoyer on Twitter convinced me that the sketch of some_library.open(...) in the post is not useful; it would be more useful to build type-stable functions like some_library.open_dask_array(...) that import the available Readers for a specific MIME type and either find an exact match (Reader.container == 'dask.array.Array') or something it can downcast (Reader.container == 'xarray.DataArray`).
It's not obvious to me that a special container attribute is the best way to declare this. Other ideas welcome!
One possible next step would be to build this in PIMS 2.0-alpha and area-detector-handlers (two projects I am a core dev on) in coordination with napari and demonstrate whether we find it useful.

I see new momentum on the numpy mailing list around embracing type annotations. In the blog post I suggested that it might premature to make that core to how Reader would work, but maybe that's not true. If we could remove the 'container attribute and rely on a type-annotation, we end up with API that looks just like built-in open, but with type-annotated read().
class Reader:
    def __init___(self, ...):
        ...
    def read(self) -> container:
        ...
    def close(self):
        ...
    def __enter__(self):
        return self
    def __exit__(self, *exc_details):
        self.close()

I'd love love love type annotations to be the main interface here. =) I seem to have killed the discussion over on the mailing list though! :sweat_smile:

Hey @danielballan!
I think I remember reading your blog post when it first came out but I guess it never hit me until recently.
I am one of the maintainers of aicsimageio. We try to act as something described as above but with handling for delayed chunking of microscopy images by dimensions, additional support for access and manipulation of metadata, multiple scene / multiple position management, and a few more bits and bobs useful to microscopy.
The reason I say "it never hit me until recently" was because we just put out a list of reasons for our next major release and a lot of the reasons on that list are largely your post above!
I guess I am now wondering, after a year and change, have you changed your mind on some of these ideas? I would love to incorporate them into whatever aicsimageio 5.0 becomes.
Thanks for all the work you do already, really appreciate your insight.


failed to install all extensions - it  shows a non-stopping progress status

Issue Type: Bug
VS code installed on Windows 8.1. But in is not installing extensions. It shows a contineous installing status. Tried all suggestions from forums with no success.
VS Code version: Code 1.28.2 (7f3ce96ff4729c91352ae6def877e59c561f4850, 2018-10-17T00:23:51.859Z)
OS version: Windows_NT x64 6.3.9600

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4702MQ CPU @ 2.20GHz (8 x 2195)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.93GB (10.79GB free)

Process Argv

Screen Reader
no

VM
50%

Extensions: none

@Yidy Can you please do following steps:

Set log level to trace using set log level command from Command Pallette
Open Shared Process logs in Output panel
Retry the install operation
Share us the logs


End effector frame in the provided urdf for the Jaco7 does not match the urdf(xacro)  from Kinova

I'm working with the Jaco7 and noticed that the end effector frame described in the urdf provided does not match the one you get from kinova's ROS repo https://github.com/Kinovarobotics/kinova-ros/blob/3078ec6f00225e0ec5b80aae800bd3d94a59c369/kinova_description/urdf/j2n7s300.xacro
The one obtained from the relaced_ik_core doesn't include the pi/2 rotation for Yaw on the end effector. (first image is of the relaxed_ik and the second is the one from Kinova)
My question is if I change this value so it matches kinova's xacro/urdf will that have a negative effect on IK?
Will I have to retrain the neural network?

Good question.  This depends on whether or not the changed values in the urdf would change the resulting robot pose given the same robot state vector.  For example, if joint values [0,0,0,0,0,0] had the robot going straight up with the old urdf values and now [0,0,0,0,0,0] has the robot facing forward in a different pose, the neural network would have to be retrained.  In this case, it looks like you're only shifting the end effector frame, so my guess is that retraining would not be necessary.  Feel free to give it a try, and if it seems like it is leading to errors, feel free to follow up here.  Thanks


Compute bucket level cardinality

Currently there are no statistics for bucket level cardinality in Platform. We have the mechanism to track this in the tsm1.MeasurementStats type, but it only tracks the tsm1.FileStore (snapshotted TSM files). We need to also track writes before they're snapshotted, e.g., still in the tsm1.Cache.

[ ] Add MeasurementStats to the tsm1.Cache.
[ ] Expose combined FileStore and Cache metrics via a storage.Engine method.
[ ] Ensure that when Cache snapshotted to an L0 TSM file, we don't double count stats.

don't close.


Mega base blueprint and naming

Hi, thanks for all the hard work of the blueprints! Do you also plan to add the mega factory here too? I would like to propose some minor naming cleanups, but wasn't sure where you keep the sources. Thanks!

They're in the "self-building-factory" subdirectory.
-- Brian
(sent from my Android; expect typos and brevity)
On Wed, Jun 22, 2022, 08:57 Yuri Astrakhan @.***> wrote:

Hi, thanks for all the hard work of the blueprints! Do you also plan to
add the mega factory
https://www.factorio.school/view/-MNZWdWosuqr3vtaC2hD here too? I would
like to propose some minor naming cleanups, but wasn't sure where you keep
the sources. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/bcwhite-code/brians-blueprints/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACLEM3D5MO7QZHEIURY4UZ3VQMESTANCNFSM5ZQEDT5Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


⚠ Bofrost.de has degraded performance

In 3e1b12b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1779 ms

Resolved: Bofrost.de performance has improved in 14dd49d after 2 hours, 20 minutes.


Update license

Type: Incomplete
Summary:
Update license
Details:
Update the license according to the information here https://bitbucket.org/pypa/distlib/src/master/LICENSE.txt
Resolution:
Update license
Affected definitions:

distlib 0.3.3

This should be OTHER since the license is the full stack, not just the CRNI portion, and is not a full match to the Python-2.0 license.
@AE49 Do you agree?


Fix posix no libc wasi debug interp build

This change supports building with WAMR_BUILD_LIBC_WASI=0 and WAMR_BUILD_DEBUG_INTERP=1, otherwise the os_socket_* functions will be undefined.

Hi @wenyongh, yes, the intention is for the else() clause to be different. The reason is that when we are building with WAMR_BUILD_DEBUG_INTERP=1, we want to compile posix_socket.c, and we also want to use platform_common_libc_util.cmake regardless of the setting of WAMR_BUILD_LIBC_WASI. I tested all for combinations of WAMR_BUILD_LIBC_WASI and WAMR_BUILD_DEBUG_INTERP before submitting this pull request, so unless I made a mistake I think this is behaving as intended.

Hi @wenyongh, yes, the intention is for the else() clause to be different. The reason is that when we are building with WAMR_BUILD_DEBUG_INTERP=1, we want to compile posix_socket.c, and we also want to use platform_common_libc_util.cmake regardless of the setting of WAMR_BUILD_LIBC_WASI. I tested all four combinations of WAMR_BUILD_LIBC_WASI and WAMR_BUILD_DEBUG_INTERP before submitting this pull request, so unless I made a mistake I think this is behaving as intended.

OK, got it, thanks.


Build error. OS X 10.11.5

[ 98%] Building CXX object CMakeFiles/souperweb-backend.dir/tools/souperweb-backend.cpp.o
souper/tools/souperweb-backend.cpp:36:53: error: use of undeclared identifier
'getGlobalContext'
parseAssembly(MB->getMemBufferRef(), Err, getGlobalContext())) {
^
1 error generated.

getGlobalContext was removed from LLVM a while back, so this probably broke in the last LLVM bump.

Yeah, I think this got missed since souperweb only gets built when Go is installed or something like that -- I'll look into it tonight.

Should be fixed-- thanks for the bug report!

Built fine for me on 10.12 beta. I think this issue can be closed.


Make flow for creating Linux executable

Make a script to create a Linux distribution for Autographa which is callable from within package.json.

Fixed in release Autographa Lite 1.2.1


Re-order sections on front page

Currently the order of sections is:

Get in touch
Community apps
Accountability
Bots
Friends of tech bloc

Ideally this would be rearranged so it becomes:

Accountability
Community apps
Bots
Get in touch
Friends of tech bloc

The latter ordering feels a bit more natural and showcases some of our biggest efforts at the top of the page.

Hi could I take this one?

Hi @Myrfion, please go ahead! I'll assign it to you 🙂

@AetherUnbound could u please check my PR? Thanks

Just wanna make it count toward the hacktobefest and the deadline is tonight

@Myrfion gave you some feedback on the PR 🙂

@AetherUnbound Thank you for getting back to me, I made an update

hey @AetherUnbound ,can i work on it if it still need changes?

Oh sorry, this was addressed in #22!


Add Unit Testing and E2E testing framework; Modals

I'm submitting a ...
[ ] bug report
[X ] feature request
[ ] question about the decisions made in the repository
Do you want to request a feature or report a bug?
Feature

Hi - I've added a simple modal to replace the window.confirm in the smart tables and unit test and e2e frameworks  (I've also added toastr)- would you like me to push these up?
No worries either way.

Hey @tonyhayes, we would love if you submit the changes you made as pull requests. One thing - will you be able to submit them separately per the functionality added? Thanks!

I've separated my changes into individual commits .. but how do I submit these changes - I can't just push them up into master ?

@tonyhayes please take a look at this explanation https://yangsu.github.io/pull-request-tutorial/

I got this message;
Switched to a new branch 'pull-request-1'
DCIs-MacBook-Pro:ng2-admin thayes$ git push origin pull-request-1
Username for 'https://github.com': tonyhayes
Password for<EMAIL_ADDRESS>remote: Permission to akveo/ng2-admin.git denied to tonyhayes.
fatal: unable to access 'https://github.com/akveo/ng2-admin.git/': The requested URL returned error: 403

@tonyhayes that isn't the way that the pull requests work.  Then you can use a command line or better GitHub to create a pull request. Pull request compares your fork and upstream (this repository). Only owners of the repository are allowed to merge.  Take that lesson, it's a standard, will help you a lot in the future. Best

Closing as unfortunately it's too old.

Hi @ramsunvtech, thank you for your suggestion, but unfortunately I don't think we want to use Jest, since we already use the default Angular setup (Karma + Jasmine) in Nebular (which is a UI library for ngx-admin) and it doesn't make sense to introduce additional library into the environment.

No Problem. If that is the case, then i'm fine with Karma + Jasmine

@StanislavChankov You need to fork repository and then push changes to your fork. Then you'll be able to submit pull requests to this repository.
https://help.github.com/en/articles/fork-a-repo
https://help.github.com/en/articles/creating-a-pull-request


Array handlers are skipped when @marko/run is used with eBay/arc

I have a handler +handler.ts and a variant of the handler +handler[variant].ts, both of which contains array handlers. Eg,
export const GET: MarkoRun.Handler[] = [
  () => { ... },
  () => { ... }
]

When I run this in production mode, the handlers are not executed.
I traced it down, and patch my local version with the following:
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 2f4b874..a95c96f 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -106,7 +106,7 @@ function compose(handlers) {
 function normalize(obj) {
   if (typeof obj === "function") {
     return obj;
-  } else if (Array.isArray(obj)) {
+  } else if (Array.isArray(obj) || obj.length) {
     return compose(obj);
   } else if (obj instanceof Promise) {
     const promise = obj.then((value) => {

When used with Arc, obj is a Proxy for the array and so will fail the Array.isArray branch. Adding the 2nd condition works, at least for my code-base. Not sure if there is a more elegant solution.

Thanks for the report and investigating the cause. I've made a PR to Arc which will preserve arrays when they are proxied so the Array.isArray check will work.

Fixed in<EMAIL_ADDRESS>


Monomorphic processing of TopN queries with 1 and 2 aggregators (key part of #3798)

Monomorphic processing: add HotLoopCallee, @CalledFromHotLoop, RuntimeShapeInspector, StringRuntimeShape, SpecializationService
Specialized topN queries with 1 or 2 aggregators
Added Cursor.advanceUninterruptibly() and isDoneOrInterrupted() for exception-free query processing, to avoid deoptimization of compiled code.
Made some stateless classes singletons, such as NoopAggregator and NoopBufferAggregator
Optimization of DistinctCountBufferAggregator: use Int2ObjectOpenHashMap instead of HashMap<Integer, ..>
Removed several unused classes

The key part of #3798.
This PR has minor intersection with #3883 (EmptyIntIterator added), but they can be merged in any order.

Ok, it depends a little more on #3883 than I expected, so #3883 should be merged first.

@leventov, nice work. I looked over the patch and am still reviewing.
So far, the patch looks good and I have only one question. How large is the runtime inspection overhead?

@jihoonson thank you.
Runtime inspection overhead is about 150 ns per PooledTopNAlgorithm.scanAndAggregate() call.

@jeffs thanks for review. Also regarding @CalledFromHotLoop - decided to annotate only interface methods, not implementation methods, because IntelliJ doesn't copy the annotation automatically and it's too tedious to insert it manually everywhere.

Going to add some tests.

@jihoonson addressed comments, testing specialization now.

@himanshug could you please review this?

@leventov sure, I'll have some time to start reviewing this tomorrow

I started with reviewing the classes in monomorphicprocessing, looks good so far, will continue the review

@jon-wei thanks for review

@himanshug could you please review this PR?

@leventov sure, will review this week likely tomorrow.

@leventov ok, i started reading the PR .
does all the speed up come from cloning a class and use that dedicated cloned class for a given shape because hopefully JIT will be able to generate optimized code for that specific shape ?

@himanshug yes, JIT inlines monomorphic calls.

@leventov how do you determine what methods in BufferAggregator and DimensionsSelectors should be annotated with @CalledFromHotLoop .
I see that BufferAggregator.init() , BufferAggregator.aggregate(), DimensionSelector.getRow() are called from within scanAndAggregate(..) so they should be annotated.
However, I see that following methods are annotated instead..
BufferAggregator.aggregate(), DimensionSelector.getRow() which are fine...
but DimensionSelector.lookupName() is annotated but not really called from the hot loop and
BufferAggregator.init() is called in hot loop but not annotated as such.

@himanshug annotated BufferAggregator.init() with @CalledFromHotLoop. DimensionSelector.lookupName() is used in some ValueMatcher implementations. ValueMatcher.matches() is a part of hot loop processing

@himanshug any other comments?

@leventov LGTM aside from https://github.com/druid-io/druid/pull/3889#pullrequestreview-26149557
However, this PR updates BufferAggregator to implement HotLoopCallee , BufferAggregator is a druid extension point and many users have custom aggregator extensions which will break with this. Given that change, this can only be released in 0.10.0 or 0.11.0
given that, we haven't released 0.10.0 yet, so we can try and pull this one into 0.10.0 milestone.

I think this is too big to pull into 0.10.0, we branched it off already and the branch should get bug fixes only. We could jump directly to 0.11.0 if we need to I guess.
Although, would this really be incompatible with existing extension jars? HotLoopCallee has no methods, so does adding it to BufferAggregator require extensions to be recompiled?

@gianm HotLoopCallee has method inspectRuntimeShape().
Another way is to move to Java 8 in 0.10.1 and provide a default implementation of inspectRuntimeShape() in BufferAggregator.

Oh, I missed that. A default implementation sounds good if that means an extension compiled for 0.10.0 would work in 0.10.1 (sorry, I'm not that familiar with what interface changes will and will not require recompiles).

@gianm
https://docs.oracle.com/javase/specs/jls/se8/html/jls-13.html#jls-13.5.6

In other words, adding a default method is a binary-compatible change because it does not introduce errors at link time

@leventov prior conversations were about requiring a java8 JVM, not about compiling all of druid for java 8 target. Requiring a java8 target for all code would need community discussion.

@drcrallen why compiling Druid for Java 8 target is more risky, than requiring Java 8 JVM?

@himanshug now BufferAggregator.inspectRuntimeShape() has default implementation.

@himanshug could this PR be merged now?

:+1:


Unable to log in

Unable to log in & get past this screen. Was working fine about an hour ago.
Running OTD v4.0.4 with BTD v4.11.3

ofc as soon as I post this, it starts working again...


Property is "Undefined" on Multistep Form

I'm using the latest master and have a problem with the multistep form code posted in the wiki. I'm using it exactly as posted with no alterations.
I guess the latest master might make the multistep javascript a little outdated because it does work with the 3.1.0 stable. Here is the javascript console errors that I get:
Cannot read property 'pending_account[first_name]' of undefined

validatorsFor
$.fn.isValid
e.extend.each
e.fn.e.each
f.event.handle
f.event.add.i.handle.k

Any help would be greatly appreciated!

I believe I solved the issue. I thought I would post it here if anyone else has the problem or if anyone more knowledgeable at javascript had some changes I should make.
Change:
var settings = window[this.form.id];
To this:
var settings = window['ClientSideValidations']['forms'][this.form.id];
To the multistep javascript code given in the wiki.
If anyone reads this and thinks I overlooked something or could have done it better please let me know since I am not amazing at javascript!

Thank-you for documenting your solution :)


Attempting to pay to an address with an empty assets object

Hi, I am trying to use pay.ToAddress but without any assets for a transaction
lucid     .newTx()     .pay.ToAddressWithData(       address,       {         kind: "inline",         value: Data.void(),       },       {},       validator,     )
This was allowed in lucid-cardano, but I get an error Attempting to pay to an address with an empty assets object with lucid-evolution. Can you guys support this? Currently we having to add additional ada while doing this just to satisfy the condition

You can set it to undefined, but we could add an empty object as well

Thanks, it works


Replacing an interactive message button when it is clicked.

I can't work out via issues or docs how to achieve the behaviour (replacing a button with text after it's actioned) in the following gif from the slack docs.

If I use bot.replyInteractive it replaces the entire attachment instead of just the button text.
According the documentation found here, it says:

Responding immediately with a JSON message body will replace the current message in its entirety by default. If you explicitly indicate that you want to create a new message instead, specify false in the replace_original field.

I still however cannot work out how to achieve this behaviour.
Has anyone else managed to replace an interactive button with a message upon it being clicked?

@alexframe As the docs state, the only option available is replacing the entire message.
Luckily, Slack sends the original message back as part of the button click payload, so you can use that as the basis for your response.

This skill, included in the Botkit Starter Kit for Slack, demonstrates this in action:
https://github.com/howdyai/botkit-starter-slack/blob/master/skills/interactive_messages.js#L8

So, that's exactly what I am doing. Or at least I thought so.
I send back the exact same message but Slack responds with a HTTP 500.
All I am doing is going through the blocks, change one text field of a button and write back the same message I received. Am I missing something?
Question on stackoverflow: https://stackoverflow.com/questions/57033773/respond-to-user-interaction-how-to-update-message


Feat/Fix typo in macros.yml

There's a typo in the macros.yml file. This raises a warning when a project that uses this package is invoked.

[WARNING]: Found patch for macro "grant_ownershop_on_schema_objects" which was not found

What is the plan to get this PR merged?  Could this go into release version 0.6.2 since it's just docs & non-breaking?

@JFrackson ^ asking about plan for releasing this PR, and forgot to "@" you in previous comment

Thanks for this fix and sorry for the delay with the review! We'd definitely like to add better CI/CD to this project and this is just one part of that


Feature: Show top contributors that I am not following

Type of feature
🍕 Feature
Current behavior
This top contributors feature shows the same contributors every time. I'd love to make the list more dynamic and encourage the logged-in user to follow new users. Until the active users increase, we should attempt some of my suggestions below.

Suggested solution

Check the logged-in user's following and remove individuals they are already following.
OpenSauced employees dominate the list. We need to exclude those folks as well.

Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

I totally agree with the first suggestion. Solving the first also means solving the second, you can't follow a user twice unless you unfollow them. So my proposal here is we exclude followed users from the API before sending them to the front-end. It would be fairer to approach this feature from the API side of things.

Update: We will have to update the fetch call to include the logged-in user-id once https://github.com/open-sauced/api/pull/275 is merged


not support xhr('/path', ...)?

In the xhr.open wrap code
 ...
 var parsedUrl = url.parse(urlStr, true, true);
 // do not forward relative or local request, beacuse the server can not resolve
 if (!parsedUrl.host) return urlStr;
 ...

If urlStr is /path like (without host), the wrap will fail.

Right. Just like the comment says, the mock server can not resolve the host correctly. Users may modify their hosts file, or visit a website of private networks.

Refer to https://github.com/chinesedfan/parrot-mocker/pull/4.


add to column menu

item: Filter...
it opens a text field field above the column, absolutely positioned above the input side of the cell above.  that search term applies to just this column.
it can be an expression like >10000
ie if it begins with > < >= <= or = then does numerical comparison.
otherwise does string match.

so if the string is "> 1000" then it's a numerical comparison as described above.
please support another syntax as well, &&, so they can write:
">1000 && < 1010"


Get-AzConsumptionPriceSheet

Hello again,
Utilizing azure us government, with Get-AzConsumptionPriceSheet in Recommend-AdvisorCostAugmentedToBlobStorage
noticed that get-azconsumptionpricesheet returns:
Get-AzConsumptionPriceSheet: Operation returned an invalid status code 'NotFound'
Id          :
Name        :
PriceSheets : {}
Tag         :
Type        :
hmnm, ran it with -debug
got a snippit
"error": {
"code": "404",
"message": "Cost management data is not supported for subscription(s) xxx in the provided api-version. Please use api-version 2019-10-01 or later. (Request ID: yyy)"
going to do more research on rights to subscription cost management and azure us government
Update: https://github.com/MicrosoftDocs/azure-docs/issues/38660
Changing the API version to the previous stable release (2019-01-01) allows the filter to work.
This is an example of the full URI: https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/providers/Microsoft.Consumption/usageDetails?api-version=2019-05-01&$filter=properties/resourceGroup eq 'dev-rg'
where the -debug of the command reported using
Absolute Uri:
https://management.usgovcloudapi.net/subscriptions/xxx/providers/Microsoft.Consumption
/pricesheets/default?$top=1000&api-version=2018-01-31

hmm, can't seem to find a way (not powershell command guru) to change the URL for this command to update to a newer or the specified api version...

Unfortunately, the API version is hardcoded in the PowerShell module and cannot be changed from outside. We'll have to wait for the module to update to latest API version. A work-around can be to abandon the Az.Billing PowerShell module/wrapper and implement the REST conversation directly with the API. However, not accessing the Pricesheet shouldn't break the runbook. You should see a "Consumption pricesheet not available, will estimate savings based in cores count..." log and the runbook would proceed normally. Can you confirm?

yes, it continues on, with the estimation.


Flashing Blue Spinner on 1.3.15

Windows Version
Microsoft Windows [Version 10.0.19045.3208]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 22.04
Other Software
No response
Repro Steps
Downloaded the 1.3.15 pre-release package, and on every startup of WSL the blue spinner next to my cursor flickered indefinitely. The only way to stop it from flickering that I found was to go into Task Manager and stop whatever WSL related executable was running. One of them was wslhost.exe, another was the full name, etc...
Unfortunately I immediately uninstalled the package and went to 1.2.5, and that solved the issue. I know 1.3.15 is a pre-release though so I thought I'd give a heads up here just in case, I just don't have the time to test more myself.
Expected Behavior
I expected it to not have a flashing blue spinner
Actual Behavior
There was a flashing blue spinner that kept flashing indefinitely.
Diagnostic Logs
No response

I think you want to refile this issue under https://github.com/microsoft/wslg/issues. Make sure to fill out the template and provide all the logs that they ask for.


Regarding issue 1698: Analyze current path and log invalid items as soon as extension loads

Hi,
Issue #1698 was resolved, but I checked my Path var and cannot find any errors. The supplied screenshot in that issue show the percent sign. I do have that one in my Path var since I am on Windows 10. But when I open VS Code the message displayed does not show the percent sign. So I am unsure if that is a problem or not.

Not following what you are saying.
We do not believe % cause issues with the extension loading, hence not displayed and not checked.
Now my questions is:

Is the extension crashing on your PC because of the % symbols?
If yes, have you tried removing them and testing
If no, then what is the problem you are experiencing

I realize it was a bit fuzzy. My question consists of two parts basically.

When I start VS Code I get this notification:

So the main question is, why am I getting this when I have none of those characters in my Path variables? See my screenshot in the first post. Or am I actually having one of those characters but I cannot see it?

The second part was that I reasoned about why the error could have happened. I took a wild guess to what the problem could be after looking at the image you posted here: https://github.com/Microsoft/vscode-python/issues/1698#issuecomment-397756306

As you can see, your comment/screenshot of the notification contains the % sign as a character. Whereas when I myself start VS Code do not see the percent sign.
To answer your question(s):

It does not crash, I only get the notification seen above.

Please print your path Marianne from command line and paste contents here.
The decision was made to ignore %, and the comments were made before that decision.

I was about to post the same issue,
C:\Users\2c>set PATH
Path=C:\Program Files\Docker\Docker\Resources\bin;C:\ProgramData\Oracle\Java\javapath;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0\;C:\Program Files\Git\cmd;C:\WINDOWS\system32\config\systemprofile\.dnx\bin;C:\Program Files\Microsoft DNX\Dnvm\;C:\Program Files\Microsoft SQL Server\120\Tools\Binn\;C:\Program Files\Microsoft SQL Server\130\Tools\Binn\;C:\Program Files\erl8.0\bin;C:\WINDOWS\system32\config\systemprofile\AppData\Roaming\Python\Scripts;C:\Program Files (x86)\Heroku\bin;C:\Program Files (x86)\git\cmd;C:\Program Files (x86)\Git\cmd;C:\Users\2c\AppData\Local\Android\sdk\platform-tools;C:\Program Files (x86)\Skype\Phone\;C:\Go\bin;C:\Program Files\Geth;C:\HashiCorp\Vagrant\bin;C:\PostgreSQL\pg96\bin;C:\Program Files (x86)\Elixir\bin;C:\WINDOWS\system32\config\systemprofile\.mix\escripts;C:\Program Files (x86)\Common Files\Intuit\QBPOSSDKRuntime;C:\Program Files (x86)\GtkSharp\2.12\bin;C:\WINDOWS\SysWOW64\WindowsPowerShell\v1.0\Modules\TShell\TShell\;C:\Program Files\Git LFS;C:\Program Files\nodejs\;;C:\WINDOWS\System32\OpenSSH\;C:\Users\2c\AppData\Local\Programs\Python\Python36-32\Scripts\;C:\Users\2c\AppData\Local\Programs\Python\Python36-32\;C:\Users\2c\Anaconda3;C:\Users\2c\Anaconda3\Scripts;C:\Users\2c\Anaconda3\Library\bin;C:\Users\2c\AppData\Local\Microsoft\WindowsApps;C:\Users\2c\AppData\Local\atom\bin;C:\Users\2c\AppData\Local\GitHubCLI\bin;C:\Program Files\GPAC;C:\Program Files (x86)\Microsoft VS Code\bin;C:\Users\2c\AppData\Local\Microsoft\WindowsApps;C:\Users\2c\AppData\Roaming\npm;C:\Program Files\Bandizip\;
PATHEXT=.COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH;.MSC;.PY

do you see anything wrong here that would provoke the message?

Yes, you have ;;
As mentioned, you can invite this message, it's a warning, and your extension loads.

@DonJayamanne That was @2c2c answering in the last comment and not me, who opened the issue.
I suppose I could chose to ignore the message completely, but I opened the issue in hopes of avoiding confusion for multiple other users of VS Code.
On Windows 10, the Path env is combined by two parts. The system variables and the user variables. In my original post for this issue the screenshots shows the system variables
Here are my user variables:

And here is my Path from command line:
Path=C:\Program Files (x86)\Intel\iCLS Client\;C:\Program Files\Intel\iCLS Client\;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0\;C:\Program Files\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files\Intel\WiFi\bin\;C:\Program Files\Common Files\Intel\WirelessCommon\;C:\Program Files (x86)\ctags58;C:\Program Files (x86)\Windows Kits\8.1\Windows Performance Toolkit\;C:\Program Files\Git\cmd;C:\WINDOWS\System32\OpenSSH\;C:\Users\UNKNOWN\AppData\Local\Programs\Python\Python36\Scripts\;C:\Users\UNKNOWN\AppData\Local\Programs\Python\Python36\;C:\Users\UNKNOWN\AppData\Local\Programs\Python\Python35\Scripts\;C:\Users\UNKNOWN\AppData\Local\Programs\Python\Python35\;C:\Users\UNKNOWN\AppData\Local\Microsoft\WindowsApps;C:\Program Files\Intel\WiFi\bin\;C:\Program Files\Common Files\Intel\WirelessCommon\;C:\Users\UNKNOWN\AppData\Local\Microsoft\WindowsApps;C:\Program Files\Microsoft VS Code\bin;;C:\Program Files\Docker Toolbox
As can be seen in the command line, there is two semicolons. However, as can be seen in the system variables and user variables there is no semi-colon there. There is no way I can remove those semicolons. I am unsure where they come from. I am "guessing" it is the current path?
Is the only solution for me to chose "Always ignore"? In any case, this problem should happen to a lot of other users?
(Thank you for clarifying the decision about the percent sign.)

you can modify your path directly through SET PATH=modifiedpath
I agree with your sentiment about the error being confusing for most people
>Paths containing (';', '"', or ';;')
Everyone's path variable contains ;, which made me think this was an issue involving a stray character mangling a filepath. It turns out I had a ;; that wasnt doing anything to any filepaths, and was invisible in the windows gui path editor

@2c2c Yes, I think it is the sentiment I am after. Of course I can ignore or force the Path var into something else. But since I had not specifically changed the Path var myself to include double semicolons, I assumed many more people might experience that issue. In the end maybe there is not much to do about it. But on Windows two double semicolons is legit.
I edited it via the regex editor.

Perhaps we can lighten the warning to not include this? Or does the extension legitimately not load when it encounters ';;' in the PATH for some instances?
@DonJayamanne it is common for folks to see ';;' in their paths, now that @thernstig and @2c2c have commented on it here. Here's a fairly common scenario:
User either creates an env var for a dev task, or install some software that creates an env var on their behalf. Let's say:
FANCY_DEV_PATH_SEGMENT=C:\Path\to\Fancy
To make life easier, the following is added to the system path:
PATH=%PATH%;%FANCY_DEV_PATH_SEGMENT%
In the meantime, other software is installed/further env vars are added, and the PATH variable gets loaded up with other segments.
Now, the user finished their dev task and cleans up their environment. The user/app in question cleans up the added variable, but neglects (or chooses not do attempt) to clean up PATH.
The user will see this in their PATH:
> set PATH
PATH=blah;blah;blah;C:\SomePath\To\Somewhere;;C:\SomeOther\Path

...and since Windows thinks this is just fine (for this reason, and probably a ton of others) life simply goes forward.

I guess we could relax detection of ;; completely.
The problem is we're not entirely sure of what causes segfaults in the extension. What we have observed is, users with paths containing ;; and " in individual paths have caused segfaults.
Reopening to limit the check to just "

I got the same problem.
I installed JDK in my system and I wrote Java's path like this:
CLASSPATH:.;%JAVA_HOME%\lib;%JAVA_HOME%\lib\tools.jar;
JAVA_HOME:   C:\Program Files\Java\jdk1.8.0_162
Then in PATH I add two line: %JAVA_HOME%\bin  and   %CLASSPATH%
When I check my path in cmd, the %CLASSPATH%  being replaced to
.;C:\Program Files\Java\jdk1.8.0_162\lib;C:\Program Files\Java\jdk1.8.0_162\lib\tools.jar;;
I delete the last ";" in CLASSPATH and I didn't got that notification again.
Hope helps.


Add uprn analysis script

Codecov Report

Merging #20 into master will decrease coverage by 6.92%.
The diff coverage is 4.54%.

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
- Coverage   55.94%   49.02%   -6.93%
==========================================
  Files          12       14       +2
  Lines         311      359      +48
==========================================
+ Hits          174      176       +2
- Misses        137      183      +46

Impacted Files
Coverage Δ

...ailability/lr/management/commands/analyse_uprns.py
0% <0%> (ø)

...ndavailability/lr/management/commands/importers.py
33.33% <0%> (+0.6%)
:arrow_up:

...ability/lr/management/commands/analyse_polygons.py
0% <0%> (ø)
:arrow_up:

...andavailability/lr/management/commands/__init__.py
25% <25%> (ø)

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 c1e2036...625d7f7. Read the comment docs.


Cilium network policy is not being applied

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
We are running a kubeadm k8s cluster with Cilium 1.15.2. I installed cilium with policy enforcement mode as always, and also enabled policy audit mode to monitor the traffic in the cluster.  I can see request being audited
Policy verdict log: flow 0x0 local EP ID 1935, remote ID 7704, proto 6, ingress, action audit, auth: disabled, match none, <IP_ADDRESS>:52096 -> <IP_ADDRESS>:1024 tcp SYN
I'm trying to apply a cilium network policy to whitelist this traffic flow
apiVersion: "cilium.io/v2"
kind: CiliumNetworkPolicy
metadata:
  name: "haproxy-network-policy"
  namespace: ingress-controller
spec:
  description: "haproxy network policy"
  endpointSelector:
    matchLabels:
      k8s:io.cilium.k8s.policy.serviceaccount: haproxy-ingress-controller-kubernetes-ingress
  ingress:
  - fromEndpoints:
    - matchLabels:
        k8s:io.cilium.k8s.policy.serviceaccount: monitoring-kube-prometheus-prometheus
    toPorts:
    - ports:
      - port: "1024"
        protocol: "TCP"

After applying the policy I still see traffic between this endpoints being audited.
I got the labels based on this command output
src:
kexec cilium ip list | grep -A 10 <IP_ADDRESS>

<IP_ADDRESS>/32                              k8s:app.kubernetes.io/instance=monitoring-kube-prometheus-prometheus
                                          k8s:app.kubernetes.io/managed-by=prometheus-operator
                                          k8s:app.kubernetes.io/name=prometheus
                                          k8s:app.kubernetes.io/version=2.50.1
                                          k8s:io.cilium.k8s.namespace.labels.kubernetes.io/metadata.name=monitoring
                                          k8s:io.cilium.k8s.namespace.labels.name=monitoring
                                          k8s:io.cilium.k8s.namespace.labels.prometheusMonitoring=enabled
                                          k8s:io.cilium.k8s.policy.cluster=default
                                          k8s:io.cilium.k8s.policy.serviceaccount=monitoring-kube-prometheus-prometheus
                                          k8s:io.kubernetes.pod.namespace=monitoring
                                          k8s:operator.prometheus.io/name=monitoring-kube-prometheus-prometheus

dest:
<IP_ADDRESS>/32                              k8s:app.kubernetes.io/instance=haproxy-ingress-controller                                     custom-resource
                                          k8s:app.kubernetes.io/name=kubernetes-ingress
                                          k8s:io.cilium.k8s.namespace.labels.kubernetes.io/metadata.name=ingress-controller
                                          k8s:io.cilium.k8s.namespace.labels.name=ingress-controller
                                          k8s:io.cilium.k8s.policy.cluster=default
                                          k8s:io.cilium.k8s.policy.serviceaccount=haproxy-ingress-controller-kubernetes-ingress
                                          k8s:io.kubernetes.pod.namespace=ingress-controller

How can we reproduce the issue?
Install cilium with helm with enforcement mode as always,  with policy audit mode as enabled and hostFirewall as enabled, and then install haproxy and prometheus.
Cilium Version
Client: 1.15.2 7cf57829 2024-03-13T15:34:43+02:00 go version go1.21.8 linux/amd64
Daemon: 1.15.2 7cf57829 2024-03-13T15:34:43+02:00 go version go1.21.8 linux/amd64
Kernel Version
4.18.0-513.24.1.el8_9.x86_64 #1 SMP Thu Apr 4 18:13:02 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Kubernetes Version
Client Version: v1.29.3
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.29.3
Regression
No response
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Cilium Users Document

[ ] Are you a user of Cilium? Please add yourself to the Users doc

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi!

Client: 1.15.2 https://github.com/cilium/cilium/commit/7cf5782920423b07e7a4e6e67fafd81e1d4a1d33 2024-03-13T15:34:43+02:00 go version go1.21.8 linux/amd64
Daemon: 1.15.2 https://github.com/cilium/cilium/commit/7cf5782920423b07e7a4e6e67fafd81e1d4a1d33 2024-03-13T15:34:43+02:00 go version go1.21.8 linux/amd64

We're at 1.15.7 now, could you please update to the latest patch level (or ideally even 1.16) and check if the problem still reproduces? Thank you.


Bump version to 2.1.7

Per my understanding of how VS picks the version packages to add as a PackageReference, this is still required everytime we patch aspnetcore 2.1.&, right @vijayrkn ?

@seancpeters can correct me if I am wrong.
Scaffolding looks at a json file to determine which all package versions to install for a given version of .NET core app. so, for Core app 2.1.7 verson,  if the json file list scaffolding package version 2.1.6, then it should ideally work provided the package closure makes sense.
If we dont have an entry for 2.1.7, then it probably tries to match the version of scaffolding package and .NET core app package. Not sure if this logic is still there.
@seancpeters ?

So, VS uses the package version of Microsoft.AspNetCore.App, regardless of whether it’s implicit or not? In that case, yes we need this.

Correct, it doesn't matter whether AspNetCore.app is implicitly or explicitly referenced.


222 technical logbuffer should be send somewhere

Description
Changed the logBuffer to send its content to the backend when in production or when the send_buffer_ logs env is set.
Added the postBufferContent function for that purpose.

Type of changes

[ ] Bugfix
[x] Feature
[ ] Refactoring
[ ] Optimization
[ ] Documentation
[ ] CI/CD
[ ] Other

Requirements

[ ] Everything build out of Material UI components
[ ] New Material UI components capsulated via dependency injection in "@common/components"
[ ] Provide basic information pieces in comments
[ ] Establish interface to backend (if necessary)
[ ] Use Skeleton components if something has to be fetched (if necessary)
[ ] Error management: Components should handle undefined inputs
[ ] Components should be resuable (if possible)
[ ] Component Requirements are met

add empty userid or -1 userid to the backend for logs that are sent before a user is logged in
allow postLogbuffer to send logs without userid


SILOptimizer: IWYU CommandLine.h (NFC)

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test and merge

@swift-ci please test and merge

@swift-ci please test macOS platform

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 7e64abd48d95f98eff699f8c1daa1271bbbcdaf0
Test requested by - @compnerd

@swift-ci please test macOS platform

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 7e64abd48d95f98eff699f8c1daa1271bbbcdaf0
Test requested by - @CodaFi


deprecate BaseCommand, merge into Command

As part of rewriting the parser in #2205, I was looking at the complexities of Click's definitions and processing. BaseCommand seems to exist to represent a command without the help, parse, and invoke behaviors implemented by Command. A test shows implementing parse_args to use optparse, where the parameters are defined there instead of as the command.params collection. This not documented, and subclassing Command instead doesn't prevent writing a custom parse_args method.

@janluke @kdeldycke any feedback on this is appreciated

A test shows implementing parse_args to use optparse, where the parameters are defined there instead of as the command.params collection.

I'm fine with the removal, especially if that's the only purpose of BaseCommand (using something different than command.params). Personally I've never made use of BaseCommand. In cloup I had a BaseCommand too but I quickly removed it because it didn't serve any purpose.

Yes, same as @janluke, never had to use it and was quite confused by its existence. Nothing prevent merging that back to Command.
So yes, I agree BaseCommand could be removed. 👍


fix #1204: keep screen on during GPX-import

This fixes the screen-off for all GPX/LOC file imports (also import from browser downloads and email). It
is achieved by keeping screen on for all progress dialogs using Progress class.
It does not fix 'Import from Web' because cgeocaches uses own ProgressDialogs instead of the Progress class.
I could try to replace ProgressDialog by Progress but want to hear your opinion first.

@stephanme : Thanks for implementing it.


[REQ] phpMyAdmin

Which addon?
phpMyAdmin
Home Assistant does have a phpMyAdmin addon but it is locked for a single server and restricts the ability to modify config.user.inc.php.
It would be nice to have an unrestricted phpMyAdmin instance.
Thanks.

Hi thanks for the proposal. I'm quite busy for the moment and not adding new addons but I'll keep it here.
Looks like a simple fork might do the trick? If you fork it on your repo, I can guide you through it if you want. Do you know what would be required for it to have the feature you want? To access "config.user.inc.php" it should be a simple matter or moving the file to an accessible location (like /config) and make a symlink with its original location. We should be able to do that by just adding some code lines at the start of the addon


Preceding a confirm dialog with a spinner causes the first input keystroke to be ignored

Describe the bug
As the title indicates, preceding a confirm dialog with a spinner causes the first input keystroke to be ignored.
To Reproduce

func main() {
	spinnerErr := spinner.
		New().
		Title("retrieving data...").
		Action(func() {
			time.Sleep(1 * time.Second)
		}).
		Run()
	if spinnerErr != nil {
		log.Fatal(spinnerErr)
	}

	var confirmed bool

	confirmErr := huh.NewConfirm().
		Title("Fix?").
		Value(&confirmed).
		Run()
	if confirmErr != nil {
		log.Fatal(confirmErr)
	}
}

Expected behavior
first keystroke should be recognized
Desktop (please complete the following information):

OS: [e.g. iOS] Windows
Browser [e.g. chrome, safari] -
Version [e.g. 22] v0.5.2

So, I am seeing a similar behaviour, but different initial setup (ie, no spinner then confimation)...
OS: Windows (Via powershell)
(Within nix and bash I can not reproduce)
Version: v0.5.1 (when upgraded from v0.3.0) which doesnt have the issue
(it could also be related to another package, as I did multiple updates this commit)
The first huh component I use is a huh.NewConfirm(), which seems to accept the first input / initial Enter key press.
I then have a huh.NewInput() which always has it's first keypress / input ignored. e.g to register the input foo I have to type ffoo
This then also happens for subsequent huh.NewInput() calls also

I managed to reduce my package changes and making the following change to my go.mod seems to fix this issue for me.
huh is one part of the changes...
But bringing back many combinations of these charmbracelet packages seems to reintroduce the issue, or other issues on powershell for Windows.
-       github.com/charmbracelet/bubbles v0.19.0
+       github.com/charmbracelet/bubbles v0.18.0

-       github.com/charmbracelet/bubbletea v0.27.0 // indirect
-       github.com/charmbracelet/lipgloss v0.13.0 // indirect
-       github.com/charmbracelet/x/ansi v0.2.2 // indirect
-       github.com/charmbracelet/x/exp/strings v0.0.0-20240722160745-212f7b056ed0 // indirect
-       github.com/charmbracelet/x/term v0.2.0 // indirect
+       github.com/charmbracelet/bubbletea v0.25.0 // indirect
+       github.com/charmbracelet/lipgloss v0.9.1 // indirect

-       github.com/charmbracelet/huh v0.5.3
-       github.com/charmbracelet/huh/spinner v0.0.0-20240821193529-5fd70815c13f
+       github.com/charmbracelet/huh v0.3.0
+       github.com/charmbracelet/huh/spinner v0.0.0-20240404200615-66118a2cb3cf

-       github.com/dustin/go-humanize v1.0.1 // indirect
-       github.com/erikgeiser/coninput v0.0.0-20211004153227-1c3628e74d0f // indirect
-       github.com/mitchellh/hashstructure/v2 v2.0.2 // indirect
-       google.golang.org/genproto/googleapis/api v0.0.0-20240730163845-b1a4ccb954bf // indirect

+       github.com/containerd/console v1.0.4-0.20230313162750-1ae8d489ac81 // indirect
+       github.com/muesli/reflow v0.3.0 // indirect
+       golang.org/x/term v0.23.0 // indirect

@addshore I have the same problem under Windows 10 with GitBash. I have huh.Confirm followed by huh.Confirm or huh.Select, the second one ignore the first pressed key. The two questions are not in a Form.

@addshore I have the same problem under Windows 10 with GitBash. I have huh.Confirm followed by huh.Confirm or huh.Select, the second one ignore the first pressed key. The two questions are not in a Form.

Related: https://github.com/charmbracelet/huh/issues/417


Lb custom configuration methods

Part of fix for gh-660.

Closing in favour of https://github.com/spring-cloud/spring-cloud-commons/pull/751.


stone-soup: fix sqlite link error

A kind of hot fix for sqlite linker error on Sierra (#691) by making it build its own static version of sqlite. Seems to work fine at least on my local machine. Yet, we'd better need to understand the root cause of the problem.

Merged. Thank you for your contribution to Homebrew!


will the Bluetooth (ble) sample have a c++ version?

c# can't be using in my project

Okay, I ported it for you. Like I said, not hard. A literal translation of the C# sample.


Typo, Update 6-add-storage.md

Corrected "Nest steps" too "Next steps"

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you!


{Keyvault} Remove useless AAD endpoint

Description

In #22432, we migrated AAD Graph to MS Graph. Some related code in Keyvault module was also modified from https://github.com/Azure/azure-cli/blob/1451bd570fd0fe54771cedaecdbd49cdb4b8d69c/src/azure-cli/azure/cli/command_modules/keyvault/custom.py#L681-L689 to https://github.com/Azure/azure-cli/blob/cac3f06cf483d8c81f93c9cd556116e27aff2182/src/azure-cli/azure/cli/command_modules/keyvault/custom.py#L679-L684
Before migration, we need to call profile.get_login_credentials to get cred and tenant_id info.
After migration, we don't need the cred anymore but didn't drop profile.get_login_credentials just to get tenant_id info.
But there's better way to get tenant_id, it's a waste to call profile.get_login_credentials just for tenant info. Besides, the existing profile.get_login_credentials is still passing resource with AAD Graph endpoint which is deprecated. This PR aims to clear these useless code.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Keyvault


Correction for raw affiliation Department of Computer Science, De Montfort University, Almaty, Kazakhstan

Correction needed for raw affiliation Department of Computer Science, De Montfort University, Almaty, Kazakhstan
raw_affiliation_name: Department of Computer Science, De Montfort University, Almaty, Kazakhstan
new_rors: 0312pnr83
previous_rors:
works_examples: W4394936057
contact: 31d6b127e038f79cfd9750b156c900e9:640128 @ dmu.ac.uk

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Fusion 360 in workspace - Linux Mint 20.2 solved

Hello
i got a new laptop with AMD Ryzen 5 5500u run with Mint 20.2.
i cant use F360 because i see only the fusion360 logo in workspace:

I have change to Directx9.
have alo tried with D11 and OpenGL.
but still same issue.
Maybe any idea?

I have update to kernel 5.11 (kernel 5.4 default with Mint 20.2)
with Directx9 & OpenGL works now.

@tome9111991 Good job. :+1:


Binding to bool shows Windows.Foundation.IReference`1

Describe the bug
Binding to a bool dependency property will show Windows.Foundation.IReference`1<Boolean> after the value was changed.
I bind a TextBlock Run object against a bool dependency property. The same property is also bound against a checkbox.
Initially the value is displayed correctly as a string. After changeing the Value with the checkbox the IReference string si displayed
Steps to reproduce the bug

Steps to reproduce the behavior:
Use the Sample Project and uncheck one of the checked checkboxes.
Expected behavior
The string changes from ItemX True to ItemX false.
Screenshots

An Checkbox with Initial state checked and a checkbox after it was unchecked and the Label is displayed incorrect

Version Info

NuGet package version:

[Microsoft.WinUI 3.0.0-preview4.210210.4]

Windows app type:

UWP
Win32

Yes

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

October 2020 Update (19042)
Yes

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

@Scottj1s and @stevenbrix is this a cswinrt issue?

Any update on this?

Binding to a List<Windows.System.VirtualKey> shows Windows.Foundation.IReference`1<Windows.System.VirtualKey>

@duncanmacmichael  any update?

I am seeing this when binding to a bool in a ViewModel from the View, it is as simple as IsEnabled={x:Bind ViewModel.IsEnabled}, the initial value bind ok, but any further changes to the bool results in a Windows.Foundation.IReference`1.


Support for font-awesome 5 support

Hi,
Can you add font-awesome 5 support for this plugin?

@sergiu-popa Not entirely true. More is needed.
First: read the page on FontAwesome 5.com how to use pseudo-elements: https://fontawesome.com/how-to-use/svg-with-js#pseudo-elements
This explains what to do.
This comes down to 2 steps:

Add a

Font Awesome 5 Pro support:
.br-theme-fontawesome-stars .br-widget {
  height: 28px;
  white-space: nowrap;
}
.br-theme-fontawesome-stars .br-widget a {
  font: normal normal normal 20px/1 'Font Awesome 5 Pro';
  font-weight: 500;
  text-rendering: auto;
  -webkit-font-smoothing: antialiased;
  text-decoration: none;
  color: #edb867;
  margin-right: 2px;
}
.br-theme-fontawesome-stars .br-widget a:after {
  font-family: "Font Awesome 5 Pro";
  font-weight: 500;
  content: '\f005';
  color: #d2d2d2;
}
.br-theme-fontawesome-stars .br-widget a.br-active:after {
  font-family: 'Font Awesome 5 Pro';
  font-weight: 900;
  content: '\f005';
  color: #edb867;
}
.br-theme-fontawesome-stars .br-widget a.br-selected:after {
  font-family: 'Font Awesome 5 Pro';
  font-weight: 900;
  content: '\f005';
  color: #edb867;
}
.br-theme-fontawesome-stars .br-widget .br-current-rating {
  display: none;
}
.br-theme-fontawesome-stars .br-readonly a {
  cursor: default;
}
@media print {
  .br-theme-fontawesome-stars .br-widget a:after {
    font-family: 'Font Awesome 5 Regular';
    font-weight: 500;
    content: '\f005';
    color: black;
  }
  .br-theme-fontawesome-stars .br-widget a.br-active:after,
  .br-theme-fontawesome-stars .br-widget a.br-selected:after {
    font-family: 'Font Awesome 5 Solid';
    font-weight: 900;
    content: '\f005';
    color: black;
  }
}

@sergiu-popa @hepabolu @eriktelepovsky
I tried all your fixes for FA5 support. None worked for me :(
Would love to see this added in... I had a try at doing it myself, but honestly I had no idea where to start.

Hi @hepabolu,
Does your fix work with fractional star ratings?

I didn’t have the need to select half stars, only to display them. And that
worked.
But eventually I solved it differently.
On Mon, 11 Jun 2018 at 13:05, Erdem Derebaşoğlu<EMAIL_ADDRESS>wrote:

Hi @hepabolu https://github.com/hepabolu,
Does your fix work with fractional star ratings?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/antennaio/jquery-bar-rating/issues/103#issuecomment-396206038,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHIOqpMclTBaX48-58Qb07rNn6g1kVLJks5t7k77gaJpZM4RY-vt
.

--
Bye, hepabolu

The following worked for us, including support for partial-stars:
`/* Font Awesome 5 polyfill */
.br-theme-fontawesome-stars-o .br-widget {
height: 28px;
white-space: nowrap;
}
.br-theme-fontawesome-stars-o .br-widget a {
font: normal normal normal 13px/1 'Font Awesome 5 Pro';
font-weight: 500;
text-rendering: auto;
-webkit-font-smoothing: antialiased;
text-decoration: none;
color: #FFB600;
margin-right: 4px;
}
.br-theme-fontawesome-stars-o .br-widget a:after {
font-family: "Font Awesome 5 Pro";
font-weight: 900;
content: '\f005';
color: #d2d2d2;
}
.br-theme-fontawesome-stars-o .br-widget a.br-active:after {
font-family: 'Font Awesome 5 Pro';
font-weight: 900;
content: '\f005';
color: #FFB600;
}
.br-theme-fontawesome-stars-o .br-widget a.br-selected:after {
font-family: 'Font Awesome 5 Pro';
font-weight: 900;
content: '\f005';
color: #FFB600;
}
.br-theme-fontawesome-stars-o .br-widget .br-current-rating {
display: none;
}
.br-theme-fontawesome-stars-o .br-readonly a {
cursor: default;
}
.br-theme-fontawesome-stars-o .br-widget a.br-fractional:after {
font-weight:900;
content: '\f089';
color: #FFB600;
}`


Option to reposition Kourend favor

The kourend favour bars often overlap the XP globes when you get XP for more than 2 skills. Just wondering if it is possible to move it

https://github.com/runelite/runelite/pull/5064

Well #5064 actually just adds possibility to mave the widgets layoutable but it is not adding new widgets to the list, so someone would need to also add the kourend favour widget id to it


feat: Wayland support

Closes https://github.com/marc2332/freya/issues/65
Changes

Rename the x11 feature to linux
Make the linux feature also enable the wayland feature in skia-safe

I will wait until rust-skia v0.58.0 is released


AD Site Boundaries are not supported for non domain joined machines

As per MS Ticket# 17257511, the information which I got is from Support Escalation Engineer is AD Sites boundaries in SCCM are not supported during OSD before client joins to domain.
Kindly update this in documentation.

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

ID: e6115449-e733-5bb3-9b9d-b9b39ca33602
Version Independent ID: 0fa0910c-af20-e914-bd12-4f4514a3135c
Content: Use boundaries and boundary groups - Configuration Manager
Content Source: sccm/core/servers/deploy/configure/define-site-boundaries-and-boundary-groups.md
Product: configuration-manager
Technology: configmgr-other
GitHub Login: @mestew
Microsoft Alias: mstewart

Thanks again for the feedback. I made a change to this article to note the behavior. Although after speaking with engineering, the behavior is different. There is a SQL stored procedure specifically for this process to convert AD site boundaries to IP subnets for Windows PE devices during OSD.
This issue will auto close when my change merges. It will then go live during the next daily publishing cycle, which should be later today.


Store dev dependencies under the [project.optional-dependencies.dev] key?

This is what I have been doing for my projects so far; it means that dev dependencies can be installed via pip as well!
I'm unsure about the practical and/or semantic downsides, would love to learn more here 👍
Thanks a ton for the work you're doing on Rye!

This is something I have been considering. The main reason it's not today is that optional-dependencies.dev cannot contain any pip args and the current hackjob lets you do things such as -e foo. I am however quite in favor of trying to abuse already existing standards liberally :)
One thing that makes this a bit tricker is that dev/non dev is currently not a persisted thing. We store separate lock files for either and autosync just assumes dev (there is no RYE_ENVIROMENT=production or something similar today). If we were to just move this into [dev] it might need some changes to how lock files and synching is handled.

Happy to hear it, and good points!
Allowing python devs to install with pip lowers the barrier to entry on contributing to Rye-based projects, which makes me much more likely to recommend Rye 👍

There is ongoing PEP about this topic;
Python Discuss Forum
PEP 735 - Dependency Groups in pyproject.toml

Just throwing in my vote for development dependency groups! This would be a huge readability win! Being able to separate development dependencies into test/lint/local types of groupings is a flow that I'm already using in PDM, and makes everything much easier to manage

I'd like to use something like dependency groups not just for development but also to build wheels for cpu and gpu versions of a python package that mostly share the same core dependencies but have some package versions that have both (or either) a cpu or gpu variant (pytorch, torchvision, triton). related issue: https://github.com/astral-sh/rye/issues/421


v0.4.5 to staging

should we set up the new idea for deploying these containers via actions as we do this release?

yes, I think we should

Ok I'll look into it later today -- we should set that up before merging into staging, I guess

ok I've moved it into the new org, and everything is set up and running correctly...

let's bump this to 0.5.0 since it adds the new s3 capability