Remove deprecated updateSemantics API usage.

The tree broke after https://github.com/flutter/engine/pull/36675 which @jonahwilliams fixed by adding ignores. This PR migrates the deprecated framework APIs.
Continues resolving https://github.com/flutter/flutter/issues/112221
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

It would also be good to add some tests, e.g.:

if you provide a PipelineOwner with an onSemanticsUpdate, it is actually called during flushSemantics
if a pipelineOwner is created without onSemanticsUpdate you get an error when turning on semantics
if a SemanticsOwner is created, onSemanticsUpdate is called during sendSemanticsUpdate.

With tests from aab180c643ab7f466b8c0482afa72f2b2ffd5744 and e1bc066d0b7edead42fb5df336dbf7472bcdb202, this one is looking ready to go @goderbauer.


review block structure

Since the release of the first Rivine-based chain, the Threefold Chain (tfchain in short) the block structure of such chains can be seen as a structure with the following properties in the order as defined:

ParentID
Timestamp
BlockStakeOutputIndexes
MinerPayouts
Transactions

The BlockStakeOutputIndexes is defined by the following structure:
// BlockStakeOutputIndexes groups the block height, the transaction index and the output index to uniquely identify a blockstake output.
// These indexes and the value are required for the POBS protocol.
BlockStakeOutputIndexes struct {
	 BlockHeight      BlockHeight
	 TransactionIndex uint64
	 OutputIndex      uint64
}

Each miner payout can be seen as a pair of a Value and an UnlockHash (the address which received the paired Value.
For the sake of this issue each transaction can be seen as an opaque binary object with the first byte indicating its version. How the object is to be decoded is defined by the chain, but not relevant for this issue.
Compared to other chains we make no difference between Header and Body. However given that you could see the first 3 properties as the header, with the next 2 being the body you could go from a conceptual concept to an implemented one. It would require at the very least one adaptation:

We do not have a property to uniquely and safely identify the transactions of the body. Other chains use often a MerkeRoot hash for this. Nothing prevents us from adding this property once we go for a header-body separation;

Besides this we might also want the following adaptations:

We could replace the BlockStakeOutputIndexes property with an opaque binary object, which could be called the settlement or however you wish to call it. The decoding of which depends on a version, either unique to this object or coupled directly to a block version as proposed in the next bullet point;
We currently do not have a block version, making it a lot more difficult to upgrade the block structure (again) in the future.

On top of that some chains also store a Nonce in the header. At the moment I am however not sure why this would be desired, given a Merkle Root hash together with the other properties (e.g. the parent ID) should be enough to make the header unique. If not, I would think you have an issues with your cryptographic primitives used. I might however mis something here.
In Memory we do already have a BlockHeader structure defined as follows:
// A BlockHeader, when encoded, is an 96-byte constant size field
// containing enough information to do headers-first block downloading.
// Hashing the header results in the block ID.
BlockHeader struct {
	 ParentID
	 BlockStakeOutputIndexes
	 Timestamp
	 MerkleRoot
}

This header is derived from the block structure listed earlier. As we already calculate the MerkleRoot based on the transactions, it wouldn't be hard to add it to an actually implementation used block header. Again I would propose to turn the BlockStakeOutputIndexes object into an opaque binary object linked to a version, so we can change the consensus algorithm when needed without hardcoding all these things in such a transparent manner. Further we again miss the version here.

The header is currently already transferred separately from the actual block between peers,
to allow the receiving peer to decide whether or not the actual block is desired.
It it is thus not unreasonable to go the extra mile here and also allow clients to actually
download headers-only by default, and downloading actual blocks only when desired.
That mile is however a long one.

Thus so far a proposal for new structures would be as follows:
type (
    BlockHeader struct {
        Version     byte
        ParentID    BlockID
        Settlement  []byte // find a better name
        Timestamp   Timestamp
        MerkleRoot  crypto.Hash
    }

    // Some chains also add a magic No in the front of a block when encoding,
    // to make sure during the decoding one can know for sure it is a block,
    // irrelevant on its actual version or content. Not sure about
    // if we really want this though.
    Block struct {
        Header BlockHeader
        // it is probably even better to remove the miner payouts property,
        // and turn them into a transaction (or transactions).
        MinerPayouts []MinerPayout
        Transactions []Transaction
    }
)

Papers such as the following one can help with going from this request for decent review towards an actual decent implementation that would allow Light clients to be the normality:
Superlight – A Permissionless, Light-client Only Blockchain with Self-Contained Proofs
and BLS Signatures, by Roman Blum and Thomas Bocek from the University of Applied Sciences Rapperswil

thanks @GlenDC

On top of that some chains also store a Nonce in the header

That's because of the Proof of work, I removed that in the early rivine days when implementing the proof of blockstake.


Streamlit to Dash

Replace Streamlit UI with Dash UI
Move "Random Seed" setting to config file
Move "Save Input to File" to Input tab
Move "Color by Case ID" to Results tab
Update "Upload File" to use dcc.Upload

Note: It is expected for test-osx to fail because this demo uses mip

Is it possible to have the upload-problem-file browser open at the input folder directly?

I don't think this is possible


ccz roundtrip in OpenQASM 2 broken

@ryahill1 notices that roundtrip in OpenQASM2 for several gates is broken (Originally posted in https://github.com/Qiskit/qiskit-terra/issues/9559#issuecomment-1424824806)
One of those cases is ccz:
import qiskit

qc = qiskit.QuantumCircuit(3)
gate = qiskit.circuit.library.CCZGate()
qc.append(gate, qargs=qc.qubits)

qasm = qc.qasm()
print(qasm)
qc_from_qasm = qiskit.QuantumCircuit.from_qasm_str(qasm)
print(qc_from_qasm)

qiskit.qasm.exceptions.QasmError: "Cannot find gate definition for 'ccz', line 4 file "

(Copied over): this doesn't need a separate issue.  I commented the fix for #9559 at the bottom, and it's trivially the same for the three concerned gates.


License?

This is a great guide (been following from here https://xem.github.io/articles/webgl-guide.html)
I notice that a license file is missing, are you intending to add it at some point?
If the license is permissive, it would be a great resource to build from for teaching creative coding and extending new works from.
(similarly having a clear license for https://github.com/xem/MiniShadertoy would be great since it could be useful to build & test examples from)
Thanks!

Hello,
Thanks a lot for getting in touch and enjoying my content!
I added CC0 license to both repos :)
Be aware that my webgl guide "part 1" (the one you read) is not finished yet, I'm adding some extra tricks and will be announce it shortly on my Twitter (@maximeeuziere, I'm following you there and on instagram, nice content! 👍 )
I'm also looking for proofreaders, so if you find errors or things that are not well explained in my guide, I'd be happy to hear about them and fix them.
Cheers,
Max

Nice, it's quite generous to share all this knowledge.
If you have some free time on your hands, I would encourage you to add licenses to your other repositories. Gives people the green light to do cool stuff with them.
Thanks for the kind words! Looking forward to part 2.

I wish Github repos were CC0 by default :(
But I rarely have licence requests like these, so I'll add licence files when someone demands it.
You can consider all my work CC0 by default, and feel free to ask licenses on any other repo(s).
I'd love to see what you do from these!

Great, thanks for the clarification!
Closing the issue.


Clean output of makefiles to output only what is strictly necessary, while enabling simple debugging

Done. The compilation output is now saved in a log file inside the build directory. The output is only shown on the command line when the 'verbose (-v)' option is used.


[SSoC'23] Maximum Product Subarray

Given an integer array, find the subarray that has the maximum product of its elements. The solution should return the maximum product of elements among all possible subarrays.
Input:  { -6, 4, -5, 8, -10, 0, 8 }
Output: 1600
Explanation: The maximum product subarray is {4, -5, 8, -10} having product 1600
Input:  { 40, 0, -20, -10 }
Output: 200
Explanation: The maximum product subarray is {-20, -10} having product 200

@Kumar-laxmi assign this to me under SSoC'23.

@Kumar-laxmi  please assign me this issue so that i can start contribute in ssoc'23.


cmd/kube-spawn: bring back help message for KUBECONFIG

This help message for setting $KUBECONFIG at the end of kube-spawn start used to exist, but it was removed by https://github.com/kinvolk/kube-spawn/pull/161.
We should bring it back for better user experience.
Fixes https://github.com/kinvolk/kube-spawn/issues/230

It does not seem to work properly (at least on @blixtra's computer)


No serial output

A user reported that he is getting no serial output with master#0236da9f1bfe3a29b9cdda1a7ca4cbfa6370da11 (exact hash is a guess, but definitely with serialport 5, probably 6).
git checkout eee06d8407769410764a1714b1d473705482a108 && npm prune && npm install, in effect downgrading to a version with serialport 4, solved the problem.

Closing as inactive.


DDEV issue with Terminus

eheza-app  ddev ssh
aaron@eheza-app-web:/var/www/html/server/www$ terminus

Sorry, your PHP version (7.3.33-1+0~20211119.91+debian11~1.gbp618351) is no longer supported.
Upgrade to PHP 7.4 or newer to use Terminus 3. For PHP versions prior to 7.4, downgrade to Terminus 2.x.

aaron@eheza-app-web:/var/www/html/server/www$

@adamhstewart @anvmn We should robustly downgrade Terminus inside DDEV, likely by adding the phar binary to the repository, to not rely on API limits, etc for the download, I started the process, then I realized that it's not strictly related to my assignments, so here's the issue for this.

Actually it became obsolete as PHP is now generally upgraded to 7.4 , and that's enough for terminus.


fix #322 by fixing incorrect e2e-ids for tags

This pull request fixes #322 if we merge it by changing the e2e id of a sound's tags from data-e2e-id="Soundlist-track-url" to data-e2e-id="Sound-tag-url" to reduce bugs during end-to-end testing.

Ah! Excellent question. It's because our testing suite doesn't look for the first soundlist track url on any pages containing tags.
If it did, it would look at the tag url, be confused why it didn't point to /sound/123, and exit.
The reason I saw this bug is because I was testing pagination of pack and similar sound search results, and Cypress kept giving me errors instead of running smoothly, even though everything was working fine through manual testing on localhost:3000.


"Why can't I select a different C# version?" not addressed

When trying to change the C# version on my project, in the Advanced Build Settings dialog box I found the language version drop-down greyed out, with a link saying "Why can't I select a different C# version?"  This link directed me to this page, which doesn't address this question at all.

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

ID: 5694e62b-c756-1f62-4014-2ceafa54c011
Version Independent ID: 3f01c378-22e4-197c-a862-ab4a924506c7
Content: C# language versioning - C# Guide
Content Source: docs/csharp/language-reference/configure-language-version.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for pointing this out @jwfxpr
We should be more clear about the reasoning behind this. I've added it to our backlog to address in the near future.

You could at least give us an informal answer rihght now.

@kwaazaar I'm checking with some folks internally to make sure I have the correct explanation. I will post an answer as soon as I can.

I suspect that you're going to find that the answer is actually "Due to a bug affecting the latest version of VS2019
Microsoft Visual Studio Community 2019
Version 16.4.2
@kwaazaar, if you are trying to figure out why your .NET Framework project is not in fact using the listed C# language version (C# 7.3) and has downgraded you to 6, and the methods listed here for manually changing the language version are being ignored by VS, I had to download the latest .NET Framework (4.8) from https://dotnet.microsoft.com/download/visual-studio-sdks, change my project to target this framework (when in fact I need to be targeting 4.0), and only then would VS compile my project in a C# version higher than 6.

The document intends to answer the question in the tooling in the first two paragraphs:

The latest C# compiler determines a default language version based on your project's target framework or frameworks. This is because the C# language may have features that rely on types or runtime components that are not available in every .NET implementation. This also ensures that for whatever target your project is built against, you get the highest compatible language version by default.
The rules in this article apply to the compiler delivered with Visual Studio 2019, or the .NET Core 3.0 SDK. The C# compilers that are part of the Visual Studio 2017 installation or earlier .NET Core SDK versions target C# 7.0 by default.

Is that insufficient?

@cartermp, entirely insufficient, yes.
"determines a default language version" et cetera would explain why a default language was selected. That would be fine, if one wanted an explanation of why a certain default were chosen.
That is not the question under consideration. It is not at all clear why no choice other than that default version is permitted to the user, either upwards nor downwards.

In this case, further in the paragraph is the explanation - to ensure you get a highest compatible language version. Adjusting this default is explained further below. Does that work?
On Fri, Dec 20, 2019, at 09:33, jwfxpr wrote:

@cartermp https://github.com/cartermp, entirely insufficient, yes.

"determines a default language version" et cetera would explain why a default language was selected. That would be fine, if one wanted an explanation of why a certain default were chosen.

That is not the question under consideration. It is not at all clear why no choice other than that default version is permitted to the user, either upwards nor downwards.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dotnet/docs/issues/16335?email_source=notifications&email_token=ABQEJTTMCGFVTJVXCAPZVQLQZT6XPA5CNFSM4J5A3B2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHNSWSI#issuecomment-568011593, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABQEJTUBHSGDZ5H7AWBMVQ3QZT6XPANCNFSM4J5A3B2A.

I... believe you're trying to be helpful, @cartermp, but repeating the text of the document we've already read many times with increasing frustration that didn't sufficiently answer our questions any of those tones is simply not helpful.
So, to answer your redundant question, no. That doesn't work, for two reasons.
Firstly, those options (Edit the project file, Configure multiple projects) do not work, and it isn't clear that they are supposed to work in the event that I can't select a different C# version. The wording in Visual Studio literally says can't. It is reasonable to assume that can't means can't. And then, if one assumes those solutions are in fact intended to be some kind of answer, one tries manually editing and reloading ones project again and again in different combinations to find that the advice you repeated is not a solution and is not a way to change the language which, to repeat, Visual Studio tells you you can't do.
Secondly, the explanation your describe is still explaining why the default has been chosen, not why the default cannot be changed.
It would be just great if any further comments on this issue were focused on enhancing or correcting information, not redundantly and slavishly repeating what I have read again and again. Thank you for your help, but next time make it helpful.

If the stated ways to override a default does not work for you, that is a product issue which should be fixed. I haven’t seen an issue about this yet, but if you have a link to an open issue could you point me at it? Thanks!
The answer to your question is stated in this document: the default is chosen to ensure everything works for your environment, which was not the case prior to VS 16.3. This is simply a product change: the way you might have done things in the past is now different.
On Fri, Dec 20, 2019, at 10:30, jwfxpr wrote:

I... believe you're trying to be helpful, @cartermp https://github.com/cartermp, but repeating the text of the document we've already read many times with increasing frustration that didn't sufficiently answer our questions any of those tones is simply not helpful.

So, to answer your redundant question, no. That doesn't work, for two reasons.

Firstly, those options (Edit the project file, Configure multiple projects) do not work, and it isn't clear that they are supposed to work in the event that I can't select a different C# version. The wording in Visual Studio literally says can't. It is reasonable to assume that can't means can't. And then, if one assumes those solutions are in fact intended to be some kind of answer, one tries manually editing and reloading ones project again and again in different combinations to find that the advice you repeated is not a solution and is not a way to change the language which, to repeat, Visual Studio tells you you can't do.

Secondly, the explanation your describe is still explaining why the default has been chosen, not why the default cannot be changed.

It would be just great if any further comments on this issue were focused on enhancing or correcting information, not redundantly and slavishly repeating what I have read again and again. Thank you for your help, but next time make it helpful.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dotnet/docs/issues/16335?email_source=notifications&email_token=ABQEJTTDMWCFYBHBUD5ICSDQZUFMXA5CNFSM4J5A3B2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHNY5AI#issuecomment-568036993, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABQEJTQLOXAYJM7GTKGB75TQZUFMXANCNFSM4J5A3B2A.

@BillWagner, does @cartermp represent the official response to this? I'm tired of explaining to him why he's wrong, and his passive-aggressive, counter-productive "responses".

I’m sorry you feel that the answer is not satisfactory, but this is the official position. If the product does not behave as documented, then that is an issue that will be resolved. Is the problem of overriding a default occurring with the latest Visual Studio?
On Fri, Dec 20, 2019, at 14:14, jwfxpr wrote:

@BillWagner https://github.com/BillWagner, does @cartermp https://github.com/cartermp represent the official response to this? I'm tired of explaining to him why he's wrong, and his passive-aggressive, counter-productive "responses".

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dotnet/docs/issues/16335?email_source=notifications&email_token=ABQEJTXPNAA4B75A7RZXNI3QZU7TNA5CNFSM4J5A3B2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHOKCIY#issuecomment-568107299, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABQEJTTEICNHCGR4JSEXU7TQZU7TNANCNFSM4J5A3B2A.

Yes. It is. I thought I made that clear.
I'll be unsubscribing from this now, since I don't want to interact with you any more.

Still awaiting a resolution to this problem.  3 weeks seems more than enough time to ask your coworkers.

@dannyagle The rationale for why a default language version is documented, as are workarounds. To the question of why one cannot toggle something in the UI that used to be toggle-able, the answer is that it was a deliberate product change to support how C# is evolving. If the stated workarounds aren't working for you, please do file an issue because that would indicate a product bug that needs resolving.

@cartermp Why do I have to manually set LangVersion to latest in the csproj when I'm using netcoreapp3.1 ? It was defaulting to using 7.3 and saying I can't change the version in properties. Isn't it supposed to default to version 8 since I'm using .netcore 3.0+

@ckrausko You shouldn't need to. Was it already set manually to C# 7.3 before, or are you using a preview SDK? There was a preview SDK version with a bug that defaulted to C# 7.3 but that was resolved before the release.

So how this problem was addressed finally? I am asking because all I can see is more explanation why the DEFAULTS are this or that way.

hi all,
this is a massive mess; right now I have code that is "stuck" in C#5 and telling me to upgrade to C#7.  When I use the Lightbulb Box to "switch this project to C#7 I get an exception error from VS and nothing gets done.  The .sln file doesn't have a .csproj to modify either, so it's a dead end for now.
'CSharpUpgradeProjectCodeFixProvider' detectó un error y se ha deshabilitado. Mostrar seguimiento de la pila Habilitar Habilitar y omitir futuros errores

@gadget00 I think this is an older issue so maybe no one else is looking at this. If you are working with a console app, ASP.NET Core or Windows Forms app, you should have a csproj file where you can change the TargetFramework or the LangVersion.
What is your app about, and what version of VS are you using?

FWIW I think this is an older bug you're running into and likely not related to the issue discussed several years ago. It also sounds like you're using an older IDE version, and so I suspect that if it's not already fixed, you'll be asked to upgrade your IDE.

@gadget00 looks like you're working on a web project using ASP.NET under .NET Framework (maybe 4.7.2 or 4.8) in Visual Studio 2022 (version 17.7.4 was released some days ago, so you're not using an older IDE as @cartermp said).
For older ASP.NET web projects under .NET Framework, the C# version can't be modified, because newer C# versions rely on feature available in the SDK.
If you got warnings about C# 5, maybe your project is targeting .NET Framework 4.5 or older, so you can actually upgrade in the Properties of the project to .NET Framework 4.8 and use C# 7.3. You should have a Properties window and follow the steps here: https://learn.microsoft.com/en-us/visualstudio/ide/visual-studio-multi-targeting-overview?view=vs-2022#change-the-target-framework
If you need more help, I'm happy to help via email (see my profile page). I think this is not the correct issue to ask for help.

hi all,
this is a massive mess; right now I have code that is "stuck" in C#5 and telling me to upgrade to C#7. When I use the Lightbulb Box to "switch this project to C#7 I get an exception error from VS and nothing gets done. The .sln file doesn't have a .csproj to modify either, so it's a dead end for now.
'CSharpUpgradeProjectCodeFixProvider' detectó un error y se ha deshabilitado. Mostrar seguimiento de la pila Habilitar Habilitar y omitir futuros errores

If you have no .csproj, then you are doing a Website project instead of Web Application Project. That's absolutely ancient. Microsoft shouldn't spend any time supporting that old Web Forms tooling at this point. You need to upgrade to ASP.NET Core , where all new feature development is taking place. There's lots of great paths for you to take there. Blazor, MVC, Minimal API or Web API combined with a SPA framework etc.

Hi @mason-mcglothlin
can you open an issue in the roslyn repo to report this? The folks there will have a better idea what's wrong in this scenario.
/cc @CyrusNajmabadi

Hi @mason-mcglothlin
can you open an issue in the roslyn repo to report this? The folks there will have a better idea what's wrong in this scenario.
/cc @CyrusNajmabadi

@BillWagner  I think you meant to direct that reply to gadget00.

Our position would be the same: this is an ancient combo of vs and asp, and we're not supporting that anymore. If the same issues arise on asp.net core and the latest VS, please let us know.

well, I indeed am using the latest VS version, which launched yesterday if I'm not mistaken; so this issue is still a problem to this day.  I appreciate the time of those who answered with alternatives rather than whining and berating. What is this? SO? LOL
On the other hand, if you don't want to give support to certain combinations of technologies or types of projects, just drop them from the IDE. Back in the early 2000s MSFT didn't bat an eye when they dropped perfectly working tech like VB6 and VFP just because "we're moving in a different direction now" burying many commercial products and developers in the process.  Nowadays if this is 'bothersome' to attend, just drop the ax instead of a tantrum LOL If it's still in the product, you should support it I believe.
Had to reinstall VS Community 2022 5 times today and copy/paste the whole code from another colleague (pulling from Git was giving problems as well) to be able to work. Tooling for web apps development shouldn't be such a troubling endeavor in 2023, but here we are.  Thanks again to those that tried to help

@gadget00 No one here is berating you. It is important though that you understand the reality of the situation so that you can make appropriate plans. You're using tech that's nearly two decades old. That's pretty much an eternity in web development times. Expecting Microsoft to keep this tooling up to date is not realistic. If you want to have solid features and expect support for those features, you need to be on a reasonably up to date piece of tech.
I know that may be frustrating to hear. But fortunately migrating is generally not too difficult, and even if you don't want to do it yourself there's plenty of others who can assist. I myself am quite active on Stack Overflow and still willing to answer questions about Web Forms, and there's lots more who are familiar with ASP.NET Core and the more recent frameworks it includes. I'd encourage you to upgrade if this app is something you must maintain, it will reduce your frustration.

@mason-mcglothlin thank you for the observation.  I wish I had a say in the project to tell them to upgrade, but I'm not the owner of the code so I have to work with what is in production right now.  Maybe in the future we could migrate to .net core but for now this 'old (and not well understood I see) bug' was a problem in an active codebase today and that shouldn't be imho

well, I indeed am using the latest VS version,

If you can repro this issue in the latest version of VS, please file a repro over at dotnet/Roslyn so we can investigate. Thanks @gadget00

and that shouldn't be imho

What bug are you referring to specifically?  I want to make sure I understand and the we're not talking about different things @gadget00   thanks!

@CyrusNajmabadi  I guess at least 3 bugs happened:
1- the lack of selection of the C# version in the project (that's more of a design decision I think)
2- the automated deletion of the entire <system.codedom> tag after updating the "Microsoft.CodeDom.Providers.DotNetCompilerPlatform"  package
3- the CSharpUpgradeProjectCodeFixProvider getting disabled after detecting a stack error in the IDE, when trying to use the Lightbulb Box recommendation to upgrade the C# language of the project to the recommended version

@gadget00 can you file '2' and '3' (as separate issues) over at dotnet/roslyn, with repros, so we can investigate/fix.  '1' is not something we will change and is an explicit design decision.
I was unable to repro the other issues, so there is something specific about your project.  Having a repro case will help us find/fix asap.  Thanks! :)

This thread is old and I dont mean to raise it from the dead... but I am having a serious issue with this too.  My code has a "$" in it that it doesn't like:  Feature 'interpolated strings' is not available in C# 5. Please use language version 6 or greater.
I have the latest, and only have ever had the latest VS, latest .NET etc.  I've tried to change the target to different .NET versions with absolutely no success.  I have spent hours on this already and I'm virtually ready to find the developers that made forced the option out of "advanced" a good punch in the face.  Whats more frustrating is that the documentation about the csproj file is super vague.  do you just make any file with an arbitrary name with an extension csproj? or does it have to be named something specific? does it just need to sit in the same directory as my project files??  I also have a huge number of projects in the same directory that would not appreciate seeing a friggen csproj file telling it to compile from a different version.  Whoever failed the rational test on this one should be given an old fashioned floggin.  can someone differentiate these problems for me pls?

Experience dev new to VS and .net. I still don't get why I can switch it haha


Update into Mojave

Hey, thanks for all the good work !!
I have installed high sierra on my Hackintosh and I wanted to know if I can update to Mojave without any new issues.
Now everything works except wifi (I use dongle that works fine) and bluetooth.
Thank you !

@Raphencoder Hi, if you are using 1.2.8 issue, then you can update to Mojave without new issues. Personally I suggest to create a new partition to experience Mojave for a few days. If it's OK, then update.

My every update executes in such a way that it overwrites the whole EFI folder which I download from the repository, but earlier to the notebook I have entered the mac data generated numbers during the first installation, because icloud later sees the next device with new serial numbers. So it's better to write them down somewhere. The data I have in the notebook is ROM number in RT variables, custom uuid is same as smuuid, system and board serial number.
After overwriting the entire BOOT and CLOVER folder in the EFI folder, then entering my serial data, and selecting the same Mac model from the list, I have not yet done so that the system did not get up and had some problems with icloud.
Greetings.


Added my Name "Umar Khatab" in the list

What changes does this PR do?
Added my Name "Umar Khatab" in the list
Screenshots (If any)

I have already joined buddy ! Thanks


Full screen webcam?

A full screen webcam feature would be amazing, I’m currently just zooming in on the image but this enhancement would be nice!

Very good suggestion! Will get to it at some point!

Awesome! Thank you so much

Came here to ask for the same, would be very useful.  Double click video for fullscreen, for example, similar functionality to Octoprint-Fullscreen plugin: https://github.com/BillyBlaze/OctoPrint-FullScreen

Thx @JD2005L  I will take a look at that plugin to see how it's implemented

@JD2005L @Quintin9801 full screen turns out to be non-trivial to do. But now the image is zoomable (https://github.com/kennethjiang/OctoPrint-Anywhere/issues/54). I hope this at least solve part of your problem.

Yep works for me.  Thanks!

@JD2005L The new 25fps video streaming will go full screen on a tap. Closing this issue.


fix: resolve bug in code

Changed the import path for ClipboardJS to "clipboard". Removed the spread syntax from copyButton.classList.add("copy-iconbutton"). Fixed the aria-label attribute name by changing copyButton.ariaLabel to copyButton.setAttribute("aria-label", "copy to clipboard").
IMPORTANT: Please do not create a Pull Request without creating an issue first.Any change needs to be discussed before proceeding.
Closing issues

closes #ISSUE_NUMBER_GOES_HERE
...
Description

...
Changes

High level
changes that
you made
...

Unclear that these fix any bugs, so likely not worth time to review. Lots of syntax formatting. Upstream conflicts at this point as well.


Bootstrap 5 support

This is a (multiple allowed):

[ ] bug
[ ] enhancement
[x] feature-discussion (RFC)

Is there a roadmap for Bootstrap 5 support?
Bootstrap 5 is currently in Beta2 and seems to be pretty decent. The complete drop of jQuery makes it more flexible.

So Bootstrap 5 is out now... I'll take a look into it when I have a little more free time in the next few weeks, and see what would need to be done, but I'm pretty sure that it's not possible to support both 4.x and 5.x simultaneously, ie we'd most likely need a new major and maintain them separately.

we'd most likely need a new major and maintain them separately.

Yup

I've opened a new issue that tracks problems and tasks.

Closing in favor of linked ticket


fix: CORS simple request for all browser builds

We were not setting the right headers previously
Now setting

accept: application/json and
content-type: 'application/x-www-form-urlencoded' (POST)
content-type: 'application/json' (PUT)

fixes #111

cc @speedblue @redox

LGTM if the integration tests are saying so :)

Yes, waiting for them, integration tests ran on my machine OK, now onto travis.
Integration tests are not checking if there's an OPTION request being made, unit test does.
Because you cannot intercept an OPTION request nor test it :-)

:+1:


Added kv! macro to provide a KeyValue shorthand

Added a kv! macro to allow a flexible way to define KeyValue attachments:
https://github.com/knox-networks/bigerror/blob/e5402869404d409155206a14932aa068354e275e/src/attachment.rs#L228-L240

just printerr attach_variant can be run to show the output of the combination of kv!(ty: other) with .attach_printable_lazy


Speech+Transcript conditioned phoneme recognition as an alternative to G2P

Hi @wookladin ,
While creating the training data, G2P gives phonemes based on how a particular word is supposed to be pronounced, but the audio might have a slightly different pronunciation due to various accents. I understand that you've used proprietary G2P for better results. But g2p models only utilize transcript information.

A speech+transcript conditioned phoneme recognizer would give better results wouldn't it?
Phoneme error rates are still high in the latest ASR acoustic models. Usually, ASR acoustic models predict not-so-accurate phonemes and ASR language models predict the transcript from the phonemes. But here we want to improve the accuracy of the phonemes given audio and transcript. I couldn't find any literature around that. Any leads/ideas?

Hi, sorry for the late response.

Yes. It would be better if there is a phoneme recognizer that gets speech and transcript as an input. But I've never seen such an approach.
I think it could be better if the language model of ASR and the other model using the ASR model's output to reconstruct original speech are learned jointly. However, this training process will be unstable. I think using RL might help in this case.


Infer a tuple type instead of array type when possible

Suggestion
I'd prefer typescript to be more aggressive about inferring tuple types, instead of always defaulting to Array.
This is related to https://github.com/microsoft/TypeScript/issues/6574. I'd like to open up a discussion about this again since that thread is locked, and conversation stopped after a workaround was provided without completely addressing the general issue.
🔍 Search Terms
Typescript, tuple, tuples, map, inference
✅ Viability Checklist
My suggestion meets these guidelines:

[*] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[*] This wouldn't change the runtime behavior of existing JavaScript code
[*] This could be implemented without emitting different JS based on the types of the expressions
[*] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[*] This feature would agree with the rest of TypeScript's Design Goals.

Existing typescript code could start emitting type errors from this change, so it would make sense to provide this as an option in tsconfig.
⭐ Suggestion
Typescript should infer a type to be a tuple instead of an array when it's possible.
📃 Motivating Example
The specific example I have in mind is for Array.prototype.map(), but there are probably more.
const tuple: [number, number, number] = [1, 2, 3];
const result = tuple.map(value => value + 1);

Here, typescript infers the type of result to be number[] rather than [number, number, number]. This could result in some wonkiness when trying to access values inside the tuple:
const [a, b, c, d] = result; // Typescript doesn't detect an error since result is an Array type.
const e = result[10]; // Typescript doesn't detect an error since result is an Array type.

Both of these errors can be caught if Typescript instead inferred result to be the tuple type [number, number, number].
💻 Use Cases
The above example is a use case. I want typescript to tell me when I make errors in my code.

Related #32758

Typescript should infer a type to be a tuple instead of an array when it's possible.

What's your rough definition of "possible" ?

@RyanCavanaugh I don't have a strict definition. But for me, Array.prototype.map() is a very clear example where Typescript should be able to infer a tuple type, since map() preserves the length of the input.
Another one that comes to mind is x.concat(y), if x and y are both tuple types. If you'd like me to compile a list of array methods where I think this would useful, I'd be happy to do that.
I'm more curious about why this isn't a feature in Typescript yet, and whether this is something that we can/should introduce.

I'm more curious about why this isn't a feature in Typescript yet,

I'm not sure how to answer that; we have thousands of open suggestions, all which were opened by people who think they are good ideas and would like to have them 🙂

I don't have a strict definition. But for me, Array.prototype.map() is a very clear example where Typescript should be able to infer a tuple type, since map() preserves the length of the input.

So for TypeScript, map, concat, and pretty much everything else are totally indistinguishable from any other method on any other object you could call. We'd need some better way of understanding what this means.
As an example of why this just isn't the default already, I wouldn't expect this code to be an error:
const p = [1, 2, 3].map(n => -n);
p.push(-4);

but it sounds like you're proposing that it should be (since p would be a [number, number, number]).

Just for sake of exploration: that question is whether to have typescript infer the type of an array literal to be an Array or tuple. One way to go about it is to default to whatever is the most common usage. And for the other use case, people can always explicitly add the type in.
// An option if typescript infers the tuple type
const p: number[] = [1, 2, 3].map(n => -n);
p.push(-4);

Either way, I'm fine with typescript inferring the type of [1, 2, 3] to be number[] instead of [number, number, number], so the code snippet above wouldn't produce an error.
I would, however, want this code to produce an error:
const p = ([1, 2, 3] as [number, number, number]).map(n => -n);
p.push(-4);
console.log(p[3]);

So for TypeScript, map, concat, and pretty much everything else are totally indistinguishable from any other method on any other object you could call. We'd need some better way of understanding what this means.

Hmm, if there's a hard technical limitation, then I understand. I don't know anything about how Typescript is implemented underneath.

It seems https://github.com/microsoft/TypeScript/pull/11252 have fixed it, and it's status is "Merged". Is there any regression?

@whzx5byb seems like there may have been a regression.
const myTuple: [number, number] = [1, 2];

// result is inferred to be number[]
const result = myTuple.map(v => v + 1);

Playground link


Rename condition set for ingress resources and check if SetAddress function is set

Fixes https://github.com/knative-sandbox/eventing-kafka-broker/pull/1295#discussion_r726077268
Proposed Changes

Rename condition set for ingress resources
Check if SetAddress function is set

/test pull-knative-sandbox-eventing-kafka-broker-unit-tests


imdb example fails to index with latest images

Steps to reproduce

Install latest qlever script from pip
qlever setup-config imdb
qlever get-data
qlever index

Expected result
Built index
Actual result

2024-11-25 09:18:33.205 - ERROR: Assertion false failed. Unsupported escape sequence found in RDF literal "10th Story: Dirty Joke 1 \ Random Run". Please report this to the developers. In file "/app/src/parser/RdfEscaping.cpp " at line 144

Complete log

user@host:~/qlever/qlever-indices/imdb$ qlever setup-config imdb

Command: setup-config

cat /home/user/.local/share/uv/tools/qlever/lib/python3.11/site-packages/qlever/Qleverfiles/Qleverfile.imdb | sed -E 's/(^ACCESS_TOKEN.*)/\1_1uZ2haDltqkG/'> Qleverfile

Created Qleverfile for config "imdb" in current directory

user@host:~/qlever/qlever-indices/imdb$ qlever get-data

Command: get-data

(echo "@prefix imdb: <https://www.imdb.com/> ."; FILE=title.basics.tsv.gz; curl -sLO -C - https://datasets.imdbws.com/${FILE}; zcat ${FILE} | sed 1d | awk -F'\t' '{ gsub("\\\\", "\\\\", $3); gsub("\"", "\\\"", $3); printf "imdb:%s imdb:id \"%s\" ; imdb:type \"%s\" ; imdb:title \"%s\" .\n", $1, $1, $2, $3 }'; rm -f ${FILE}; FILE=title.ratings.tsv.gz; curl -sLO -C - https://datasets.imdbws.com/${FILE}; zcat ${FILE} | sed 1d | awk -F'\t' '{ printf "imdb:%s imdb:averageRating %s ; imdb:numVotes %s .\n", $1, $2, $3 }'; rm -f ${FILE}) > imdb.ttl

Download successful, total file size: 1,182,943,627 bytes

user@host:~/qlever/qlever-indices/imdb$ qlever index

Command: index

echo '{ "ascii-prefixes-only": false, "num-triples-per-batch": 1000000 }' > imdb.settings.json
docker run --rm -u $(id -u):$(id -g) -v /etc/localtime:/etc/localtime:ro -v $(pwd):/index -w /index --init --entrypoint bash --name qlever.index.imdb docker.io/adfreiburg/qlever:latest -c 'cat imdb.ttl | IndexBuilderMain -i imdb -s imdb.settings.json -F ttl -f - --text-words-from-literals --stxxl-memory 5G | tee imdb.index-log.txt'

2024-11-25 09:18:30.700 - INFO: QLever IndexBuilder, compiled on Thu Nov 21 22:49:03 UTC 2024 using git hash 5f28e8
2024-11-25 09:18:30.701 - INFO: Locale was not specified in settings file, default is en_US
2024-11-25 09:18:30.701 - INFO: You specified "locale = en_US" and "ignore-punctuation = 0"
2024-11-25 09:18:30.701 - INFO: You specified "num-triples-per-batch = 1,000,000", choose a lower value if the index builder runs out of memory
2024-11-25 09:18:30.701 - INFO: By default, integers that cannot be represented by QLever will throw an exception
2024-11-25 09:18:30.701 - WARN: Implicitly using the parallel parser for a single input file for reasons of backward compatibility; this is deprecated, please use the command-line option --parse-parallel or -p
2024-11-25 09:18:30.701 - INFO: Parsing triples from single input stream /dev/stdin (parallel = true) ...
2024-11-25 09:18:30.702 - INFO: Parsing input triples and creating partial vocabularies, one per batch ...
2024-11-25 09:18:33.205 - ERROR: Assertion `false` failed. Unsupported escape sequence found in RDF literal "10th Story: Dirty Joke 1 \ Random Run". Please report this to the developers. In file "/app/src/parser/RdfEscaping.cpp " at line 144

Docker images used:

adfreiburg/qlever-ui latest 3e0d2c4eb96d   14 hours ago    236MB
adfreiburg/qlever latest 9783e69d8767   3 days ago      792MB

@mhutter Thanks for the feedback. I just tried it myself and did not experience that error. In the imdb.ttl produced by qlever get-data I find the following line:
imdb:tt0921351 imdb:id "tt0921351" ; imdb:type "tvEpisode" ; imdb:title "10th Story: Dirty Joke 1 \\ Random Run" .
Can you search for the same line in your imdb.ttl and whether it has only a single \. If yes, it's a problem of the GET_IMDB_BASICS command in the Qleverfile, which uses awk to convert the IMDb TSVs into TTL. It doesn't really have anything to do with QLever, just a convenience function to get the dataset (which is not available as TTL on the Web).
Which shell do you use? If you don't use bash, try bash.


Create Train_Model_Forecast_Local.ipynb

Created new Train_Model_Forecast_Local.ipynb, similar to Train_Model_Forecast.ipynb, to train the model locally (no remote option).
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I close this to focus on updating the Train Model Forecast with Remote (preferred) and Local train option

I close this to focus on updating the Train Model Forecast with Remote (preferred) and Local train option


fix: Hide Edit Account Button feature

Explanation of Bugfix/Feature/Modification:
The selector used for the edit account icon must have changed in a recent YNAB update. This fixes the Hide Edit Account Button feature.

that class definitely suggest it's probably something in a new update 😅 thanks!


Try using macos-13 for CI

Motivation:
Explain why you're making this change and what problem you're trying to solve.
Modifications:

List the modifications you've made in detail.

Result:

Closes #. (If this resolves the issue.)
Describe the consequences that a user will face after this PR is merged.

Happy if this works, but didn't work when I tried this 😅
https://github.com/jrhee17/armeria/commit/58d4ab9e839364e90d7806b3cee4e9ca33db469d
https://github.com/jrhee17/armeria/actions/runs/5464882513/jobs/9947526536

Oops... I didn't know you already tried it. 😅

The build on macOS has passed. Hmm... is there a hidden bug on io.netty:netty-transport-native-kqueue? 🤔

io.netty:netty-transport-native-kqueue worked well when Unix domain socket tests were excluded.

The build ran on macos-11 has passed the CI tests.
It seems some new changes on macos-12 are incompatible with kqueue + unix domain sockets.

The root cause of the build failure of macOS turned out to be https://github.com/netty/netty/pull/13503
These comments and commit history do not help to understand the build error issue.
Let me close this and create a new PR to fix the failure.

Thanks a lot for your investigation. 🙇


Could not find module 'SplunkOtel' for target 'x86_64-apple-ios-simulator'; found: arm64, arm64-apple-ios-simulator

I am using cocoapod and added Splunk through SPM in my project target. But after importing the framework in Appdelegate or anywhere it throws this error ->  Could not find module 'SplunkOtel' for target 'x86_64-apple-ios-simulator'; found: arm64, arm64-apple-ios-simulator.
I am using M1 chip Mac book
My deployment target is 11.0
Could you please suggest a way to accomplish this? Or is it not supported?

Hi we also encountered this issue. Any updates on this?

Hello, is this still a problem you're encountering? If so, can you tell us more details about your setup so that we can better diagnose the problem? And if not, can you let us know what you did to solve the problem so we can help other users who may also run into this? Thanks!

Current workaround we did is to not run splunk related codes in simulator
#if !targetEnvironment(simulator)
// Splunk related codes
#endif

I'm glad you found a workaround. I could not reproduce this on my M1 machine. If you have additional details for reproduction we can re-open and see if we can come up with a solution that doesn't require a workaround. In particular, can you tell me the settings you have under the Architectures section in Build Settings, as well as what you have set for "Always Embed Swift Standard Libraries"? Feel free to include any other settings you may think relevant as well.


Support for multiple flavor dimensions

Fixes #137. Very similar to PR #138 but with regression test (wrote this because there appears to be no update to #138).

This is awesome! Thanks @frownlee


Interpolation in @error not working as docs suggest

This example of @error in the sass_lang docs isn't working as expected. Interpolating the variables in the @error strings seems broken (http://sass-lang.com/documentation/file.SASS_REFERENCE.html#_7).
Works fine in sassmeister.
Also the interpolation works fine for @warn.
@mixin adjust-location($x, $y) {
    @if unitless($x) {
        @error "$x may not be unitless, was #{$x}.";
    }

    @if unitless($y) {
        @error "$y may not be unitless, was #{$y}.";
    }

    position: relative;
    left: $x;
    top: $y;
}

p {
    @include adjust-location(4, 5);
}

Doesn't interpolate the variables in the @error strings. Producing this in terminal:
Message:
    style.scss
Error: $x may not be unitless, was 4.
        on line 76 of style.scss
>>         @error "$x may not be unitless, was #{$x}.";
   --------^

When reporting an bug, you must provide this information:

NPM version (npm -v):
3.8.6

Node version (node -v):
v5.1.0

Node Process (node -p process.versions):
{ http_parser: '2.6.0',
node: '5.1.0',
v8: '<IP_ADDRESS>',
uv: '1.7.5',
zlib: '1.2.8',
ares: '1.10.1-DEV',
icu: '56.1',
modules: '47',
openssl: '1.0.2d' }

Node Platform (node -p process.platform):
darwin

Node architecture (node -p process.arch):
x64

node-sass version (node -p "require('node-sass').info"):
node-sass	3.10.1	(Wrapper)	[JavaScript]
libsass  	3.3.6	(Sass Compiler)	[C/C++]

npm node-sass versions (npm ls node-sass):
<EMAIL_ADDRESS>/Users/mk/Sites/planit
└─┬<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

Thanks, can you check/report this upstream in libsass since any change will need to happen there?

Error: $x may not be unitless, was 4.
on line 76 of style.scss

    @error "$x may not be unitless, was #{$x}.";

The error message looks interpolated to me.


check_service more generic with -state and -user options

Added options to check service in several different ways
Check for:
Existence of a service
check_service.exe -service audiosrv
A service in a specified state
check_service.exe -service audiosrv -state running
A service ran by a specified user
check_service.exe -service audiosrv -user "NT AUTHORITY\LocalService"
A service in a specified state and ran by a specified user
check_service.exe -service audiosrv -state running -user "NT AUTHORITY\LocalService"
Note that there are three breaking changes

The check is no longer called check_service_running - it's check_service.
No options no longer begin with service_ as I felt it was redundant.
The default is no longer to check for a running service, it is to check for existence of a service (state doesn't matter).

LGTM. Thanks @quaggoth


A few iperf updates

default to Kbytes/sec to get more precision in bitrate data (Gbps
limited to .00)
always convert bitrate to "Gbps" regardless of native-benchmark
output, so comparisons can always be easily made even if user
chooses a different iperf output

I like that this PR increases the resolution. Do we need 'k'?, Or 'm' is big enough for today and a few more years.

'k' gives us the most resolution and should have no disadvantages, so I recommend we use 'k'.  Regardless of 'k' or 'm', we are converting to Gbps for crucible metrics anyway, to the usage of 'k' is completely invisible to the user; they will still see the more familiar Gbps in their reporting.


Allow passing classes to govukRadio label

Currently there is 'govuk-radios__label' applied but you can't pass your own class in, it would be useful to allow for a bold label and a nonbold hint underneath
https://github.com/alphagov/govuk-frontend/blob/master/src/components/radios/template.njk#L58

released in v1.2.0


/enroll does not check against plex.tv usernames/emails already in-use

also check if the enroller is already enrolled


Nudge tooltip above metadata line

When working on a test with Go, there is a handy run test | debug test metadata line above the test function.
113 }
114
115 run test | debug test
      func TestUpdateRefresh(t *testing.T) {
116     mock = createMock()

(There are lots of other ways the metadata is used besides this Go test example.)
When mousing over the function name, an intellisense tooltip appears, and it ends up covering the metadata. This makes it a struggle to click the button in the metadata. And what's worse, sometimes the tooltip appears with a small delay, so you go to click run test, and end up clicking the suddenly appearing tooltip. The tool tip can have action links such as Peek Test Output, which often scrolls the editor. Frustrating.
Could the tooltip position be nudged above the metadata line instead of on top of it?

@alexdima You have a creative idea how to reposition the hover in the presence of code lens?

@jimsnab You can try to use "editor.hover.above": false.
IMHO making "editor.hover.above": false the default would be the best here, as that would align with almost all other code editors. However, we have a ux problem with the quick pick actions which are now hard-coded to render at the bottom of the hover. If the hover is rendered above, this is helpful as these actions are close to the mouse pointer, but if the hover is rendered below, these actions end up being pretty far away.

@alexdima thanks for the workaround suggestion. I've been using it and it greatly helps the test runner usage. But it degrades in other places. For example the below mode has been annoying me inside structs, that are usually at the top of the file. Overall though it is much more usable.
I think the best thing would be to nudge the 'above' mode properly above the whole line.


Hide the my location outer circle for higher zoom levels

Hide the my location outer circle at higher zoom levels.
Mapbox sample and Google maps both hide the outer circle for higher zoom levels

This makes sense to me.


As an employer, I want to view a list of all my existing job postings in one place.

Viewing a list with all existing job postings is in progress.

job lists can be viewed in dashboard


Migrate Vault helm chart in e2e tests from ad-hoc version to official chart from hashicorp

Signed-off-by: Martin Schimandl<EMAIL_ADDRESS>
Pull Request Motivation

addresses: https://github.com/cert-manager/cert-manager/issues/5375
Unfortunately this does not yet work with TLS.
Ideas welcome how to get TLS working again
Kind

kind:cleanup
Release Note

NONE

Would you mind updating the PR title and description? It's not particularly clear what you're trying to do here 😄 why are we switching to HTTP instead of HTTPS?

@munnerz
Hi!
The change from HTTPS to HTTP is only because I could not yet figure out how to inject the TLS certificates into the Helm chart.
Do you have some pointers/ideas?
will also update the MR title.

@munnerz now it works with HTTPS again

/ok-to-test


feat: added Chart with Text example

Closes #438
Description

Added example of displaying text in the center of charts.
Current behavior (updates)

NIL
New behavior

NIL
Is this a breaking change (Yes/No):

No
Additional Information

@108yen
Hi, thanks for the PR.
Getting an error in the Vercel build.
./components/navigation/tree.tsx:250:14
Type error: Expression produces a union type that is too complex to represent.
  248 |   switch (icon) {
  249 |     case "compass":
> 250 |       return <CompassIcon fontSize="2xl" {...rest} />
      |              ^
  251 |
  252 |     case "panels-top-left":
  253 |       return <PanelsTopLeftIcon fontSize="2xl" {...rest} />
 ELIFECYCLE  Command failed with exit code 1.
Error: Command "pnpm run build" exited with 1

@hirotomoyamada
Fixed!


Cannot use color string in text Button as in the IconButton

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
with IconButton i can use color string as the following example:
<Diversity3Icon color="action"></Diversity3Icon>
Since the API is similar, i should be able to use the same in the Button
<Button color="action">Text</Button>
Current behavior 😯
i get the following error:
Type '"action"' is not assignable to type '"inherit" | "success" | "primary" | "secondary" | "error" | "info" | "warning" | undefined'
Expected behavior 🤔
it should work.
Context 🔦
in the Button API documentation: https://mui.com/material-ui/api/button/ the color is
'inherit'
| 'primary'
| 'secondary'
| 'success'
| 'error'
| 'info'
| 'warning'
| string

but the compiler is complain that it doesn't match to type '"inherit" | "success" | "primary" | "secondary" | "error" | "info" | "warning" | undefined'
Your environment 🌎
System:
    OS: macOS 12.6
  Binaries:
    Node: 18.17.1 - /opt/homebrew/opt/node@18/bin/node
    Yarn: Not Found
    npm: 9.6.7 - /opt/homebrew/opt/node@18/bin/npm
  Browsers:
    Chrome: 119.0.6045.199
    Edge: Not Found
    Safari: 16.6.1
  npmPackages:
    @emotion/react: ^11.11.1 => 11.11.1
    @emotion/styled: ^11.11.0 => 11.11.0
    @mui/base:  5.0.0-beta.19
    @mui/core-downloads-tracker:  5.14.13
    @mui/icons-material: ^5.14.18 => 5.14.18
    @mui/lab: ^5.0.0-alpha.150 => 5.0.0-alpha.150
    @mui/material: ^5.14.13 => 5.14.13
    @mui/private-theming:  5.14.15
    @mui/styled-engine:  5.14.15
    @mui/system:  5.14.15
    @mui/types:  7.2.7
    @mui/utils:  5.14.15
    @mui/x-tree-view:  6.0.0-alpha.1
    @types/react: ^18 => 18.2.25
    react: ^18.2.0 => 18.2.0
    react-dom: ^18.2.0 => 18.2.0
    typescript: ^5 => 5.2.2

@palminha Did you follow this doc to add your custom color? https://www.mui.com/material-ui/customization/palette/#provide-tokens-manually
For TS you'll need to use module augmentation: https://www.mui.com/material-ui/customization/palette/#typescript

This is expected because Button does not support color="action" as you pointed out from https://mui.com/material-ui/api/button/.
If you want to add action color to the Button, you can follow the theme component variant.


IndexLink doesn't 'active' in nested routers configuration. (^1.0.0-rc3)

My router configuration is like below.
  <Router history={history}>
    <Route component={App}>
      <Route path="/login" component={Login} />
      <Route path="/" onEnter={requireAuth} component={Home}>
        <IndexRoute component={Dashboard} />
        <Route path="/about" component={About} />
      </Route>
      <Route path="*" component={NotFound} />
    </Route>
  </Router>

    return (
            <IndexLink
              to="/"
              activeClassName="active">Dashboard
            </IndexLink>
            <Link
              to="/about"
              activeClassName="active">About
            </Link>
    );

In 1.0.0-rc1, I only need to use Link to active each  tag. But in the 1.0.0-rc3, I have to use IndexLink component. But the IndexLink is not active unless I change the router config lilke:
  <Router history={history}>
      <Route path="/login" component={Login} />
      <Route path="/" onEnter={requireAuth} component={Home}>
        <IndexRoute component={Dashboard} />
        <Route path="/about" component={About} />
      </Route>
      <Route path="*" component={NotFound} />
  </Router>

It seems like the InexLink only work for the root index. How to config a sub-index?

I'm sorry but I'm unable to reproduce the bug you're describing. In d9053829722509dae07826fb86b5e51918963aab I verified that IndexLink is working correctly on deeply nested IndexRoutes. Can you take a look at that example? If there's something you're doing that we're not doing there, I'm happy to re-open and keep discussing.

I have upload a demo project to https://github.com/CrisLi/react-router-demo
Please have look at it. When I click the 'Dashboard' link, it would not become 'active'. Let know if there anything I was wrong. Thanks a lot.

@CrisLi @mjackson Have you resolved this issue? I have the same issue here.
I've also created very simple repo here https://github.com/jepezi/react-router-rc3-activeclass

@jepezi Not yet.
@mjackson I think maybe we should re-open this issue. It seems like this is a defect.

Investigating further. I've found that when the route's path is NOT '/', then IndexRoute is active but not when path is '/'. Consider this route and nav.
const routes =
{component: App, childRoutes: [
  // path: '/website' here causes Home route active
  {path: '/website', component: Wrapper, indexRoute: {component: Home}, childRoutes: [
    {path: 'product_category', component: ProductCategory},
  ]},
  {path: '/login', component: Login},
  {path: '*', component: NotFound},
]};

// IndexLink here is active

<IndexLink to="/website" activeClassName="active">Home</IndexLink>

<Link to="/website/product_category" activeClassName="active">Category</Link>

<Link to="/login" activeClassName="active">login</Link>

When we change path: '/website' ... to path: '/' ..., then Home route is not.
const routes =
{component: App, childRoutes: [
  {path: '/',                //  <---------------- change path to '/'
...

// With this new link (/website removed), IndexLink here is not active.

<IndexLink to="/" activeClassName="active">Home</IndexLink>

<Link to="/product_category" activeClassName="active">Category</Link>

...

Here is the simple repo to illustrate the issue. https://github.com/jepezi/indexlink

There seems to be some strange behavior at /. We should have a test for a deeply nested IndexRoute that renders at /.

This was fixed by #2237.


Get Assembly name using Property descriptors provided by DynamicTypeB…

…rowseObject
During the computation of the Xml Documentation Path, DynamicTypeBrowseObject was never consulted for AssemblyName. This change introduces the check to get the property value from DynamicTypeBrowseObject
@dotnet/project-system for review
Customer scenario
Enabling XML Documentation file, generates invalid path
Bugs this fixes:
#1280 and VSO bug
Workarounds, if any
Users have to manually hand edit files to a valid name
Risk
With this fix, we might be fixing lot other broken cases which we might have missed out. The risk here though is, we might hijack the correct code flow. But I think this is highly unlikely.
Performance impact
Low
Is this a regression from a previous update?
No
Root cause analysis:
We are discovering these issues as we are looking at the interaction between Property pages and CPS more closely
How was the bug found?
Adhoc testing

Though this PR is fixing the Property pages like the PR #1267, I want to open a separate PR since the fixes are different

The risk here though is, we might hijack the correct code flow. But I think this is highly unlikely.

Can we do some more testing to verify this doesn't happen? The fix here looks correct, but given that this is in the base AppDesigner code, I'd prefer additional testing for scenarios where this code path is taken and we don't regress anything.

@mavasani I bashed around my change and I was able to fix the gap.

I should not, in my build from today, we correctly get the AssemblyName in this case:

@Srivatsn @davkean

Why are we reading from the Properties now when we didn't have to before? What does this do to old csproj projects?

Also, I would like to understand - why we do not run into this for desktop projects?

In the old csproj, AssemblyName used to be part of the common properties, CCSharpProjectProperties. So this change was not needed.

In the new project system, we have most of the property value in the DynamicTypeBrowseObject and hence the change I made.

The old csproj with the change, tries to see if the property value is defined in its Configuration specific browse object. If it is not present, then it falls back into its original behavior.

I should note, in my build from today, we correctly get the AssemblyName in this case:

I don't know what is happening. I will have to debug the code to see what is happening with the newer build.

CCSharpProjectProperties and DynamicTypeBrowseObject are just implementations of the same "effective" object - which is why I don't understand this change.

CCSharpProjectProperties and DynamicTypeBrowseObject are just implementations of the same "effective" object

These objects may provide the same properties but they are used as different objects based on their implementation.
CCSharpProjectProperties is used to provide the Property Descriptors for Common Project Properties. Since AssemblyName  was one of the Property provided by CCSharpProjectProperties, the existing code worked for desktop projects.
In the case of CPS, DynamicTypeBrowseObject is not a common project properties provider. Project Automation object, CPS's OAProject, is the common project properties provider and it does not have the information about the AssemblyName.
With this change, I am asking the pages to look into the non-common project properties to look for the property before looking into the common project properties.

Okay, I did some investigation to better understand this:
The real bug is; if you open the Build property page and have never opened the Application page, the XML property gets the wrong value when you check the box.
The repro is:

Open Properties and choose Build tab
Close Properties
Open Properties and check XML documentation file

The "common" properties from the OAProject aren't returning the correct values, CPS added an extension point for us to override this, we were supposed to implement IDispatch/ICustomTypeDescriptor and return that. That never occurred. This is the actual correct fix and what we should do long term. We need a bug on that.

You've made two changes here; GetCurrentProperty and one to bind the AssemblyName and add it as a list of PropertyControlData's that we keep track of.  Given that GetCurrentProperty already falls back to the later - I'm not sure I understand why you need the former change if the later is present? Can you help me understand that?

The current changes to GetCurrentProperty worry me a lot - this basically affects every page including ones that we don't own and affects their main path (not a fallback path).

@davkean. Yes, I just verified the behavior that you have explained above.

The "common" properties from the OAProject aren't returning the correct values, CPS added an extension point for us to override this, we were supposed to implement IDispatch/ICustomTypeDescriptor and return that. That never occurred. This is the actual correct fix and what we should do long term. We need a bug on that

Agreed

You've made two changes here; GetCurrentProperty and one to bind the AssemblyName and add it as a list of PropertyControlData's that we keep track of. Given that GetCurrentProperty already falls back to the later - I'm not sure I understand why you need the former change if the later is present? Can you help me understand that?

Yes, you are correct. The former change was the first one I made and the second change was made to satisfy the condition to get property values belonging to the page. With the second change, the first change is not required. I shall update the PR accordingly

There is also an assert on opening of the page under DBG due the AssemblyName binding - please fix that.

Is this the assert, you are talking about?

No, it was around Set not being handled for AssemblyName.

I don't see any Asserts being thrown with my debug bits for the change this PR has. Can you please mention what you did to see the assert?

Tested the new change and we are returning the AssemblyName as expected. Old csproj works as expected as well. As discussed we should look to implement ICustomTypeDescriptor/IDispatch for OAProject. Will open an issue shortly.

@davkean could you please review this again. Thanks

With this current state of the VS, we are providing the expected value. I am assuming changes made by @mattwar has fixed this. Hence dropping this PR.


Show location URL by default, but allow manager to switch that off

In Mapcontrib, you can generate a link for the current position. But many users expect to be able to just copy the URL of the current view and the share link option is a bit hidden.
So suggested solution:

by default, always show the position in the URL
if there is no position in the URL, show the location the manager defined
the manager can switch of "show position in URL" in the instance settings

Related to this old issue

The repository moved to https://gitlab.com/mapcontrib/mapcontrib.next.


syntax error (int_adyen_SFRA/cartridge/controllers/Checkout.js#14)

Syntax error
SFRA version 5.3.0.
Adyen cartridge version 20.1.3.
To reproduce
Steps to reproduce the behavior:

Call 'Checkout-Begin' controller

Logger:
ERROR PipelineCallServlet|1098367910|Sites-MYSITE-Site|Checkout-Login|PipelineCall|ECIa5wZNBG system.core Sites-MYSITE-Site STOREFRONT ECIa5wZNBG uMieIHg3U2CMAAAK-0-00<PHONE_NUMBER>115172352 - Exception occurred during request processing: ScriptingException syntax error (int_adyen_SFRA/cartridge/controllers/Checkout.js#14)
Screenshots

Additional context
Code chunk:
"use strict";
var server = require('server');
var csrfProtection = require('*/cartridge/scripts/middleware/csrf');
var consentTracking = require('*/cartridge/scripts/middleware/consentTracking');
var _require = require('./middlewares/index'),
checkout = _require.checkout;
server.extend(module.superModule);
server.prepend('Begin', server.middleware.https, consentTracking.consent, csrfProtection.generateToken, checkout.begin);
module.exports = server.exports();
Im pretty sure that this issue is related to the transpilation process but Im not sure on how to solve it. Any help/hint would 've highly appreciated

UPDATE: After trying the transpile process after installing dependencies manually this is the new transpiled Checkout controller:

Now we are getting a new error which is this one:
ERROR PipelineCallServlet|1409601472|Sites-MYSITE-Site|Checkout-Login|PipelineCall|nvsP3IBmBL system.core Sites-MYSITE-Site STOREFRONT nvsP3IBmBL uMhvN4F7U2CMAAAK-0-00<PHONE_NUMBER>669318656 - Exception occurred during request processing: ScriptingException Error: Module "*/cartridge/scripts/adyenGetOriginKey" not found. (int_adyen_SFRA/cartridge/controllers/Checkout.js#9)
System
And I've noticed that there is not such script in the transpiled folder

Maybe again something related to the transpile process

Missing .project in transpiled cartridge was the issue


Preview1: Where did Microsoft.WindowsAppSdk.Bootstrap.dll go?

~Sending up a flare for DrusTheAxe~ Closed: it's in the NuGet as Microsoft.WindowsAppRuntime.Bootstrap.dll now.

Not yet ready for action. There's a bug in the implementation so it won't do what it should. Will be fixed in preview2.

@DrusTheAxe Thanks, any details? Am just curious if it's related to my WinUI failures.


menuActions outside of the scope?

I'm trying to add a navbar with a button that should toggle the menu, but can't find where menuActions are exposed.
I tried various ideas with this.toggleMenu and having toggleMenu as a function in the menu class, but don't get it to work.
Any ideas?
    var menu = <Menu navigator={nav}/>;

    return (
      <View>
        <View>
          <TouchableOpacity onPress={() => menu.toggleMenu()}>
            <Text>Click</Text>
          </TouchableOpacity>
          <View>
            <Text>
              MyApp
            </Text>
          </View>
        </View>
        <SideMenu menu={menu}>
          <ItemView navigator={nav} />
        </SideMenu>
      </View>
    );

I don't see any way to expose this functionality outside of SideMenu component. It can't be a static method as far as it use state to store current menu state. Why do you have this case? I mean isn't it enough to have an access to it in Menu component via prop?

It's a header/navbar outside of the sidebar and I don't want it to be pushed to the side like the content.

But if you have content outside of the SideMenu, it's not gonna work: when you open/close menu, it move only the layer inside. It'll looks like your navbar has position: fixed.

The content is within SideMenu. It's just one button (hamburger icon) which is in the header to open/close the SideMenu.

Okay, the only 1 think I can do for you (and what actually I've made in 5.1 version) - is to pass props to the first-level children. I'd like to set up context, but it works weird now, so I'd like not to relay on it. On first-level components you can use this.props.menuActions to access API. Does it solve your request?

Yes, that should work. If menuActions are exposed there are lots of possibilities to control the menu. Thanks!


Add ring2fxy

This PR adds ring2fxy and calls it from ring2nest. I think that's easier to understand, making it clear that his package always goes via fxy.
There is a second commit, changing the return in tu2fxy to order fxy like in all other functions. Apparently there was no caller relying on this, at least no test is failing.
Overall I wonder if we should create V3 and FXY objects when returning things?
Note how vec2ang shows a V3 object going in, but ang2vec doesn't show one coming out, just "Object".
@michitaro - Do you think we should make a change here, or keep as is e.g. for performance reasons?

Thank you for the PR.

This PR adds ring2fxy and calls it from ring2nest. I think that's easier to understand, making it clear that his package always goes via fxy.

I agree with that.
fxy is an internal thing, so I unexported it.
https://github.com/michitaro/healpix/pull/13/commits/61267d82645c3158db8d1884ebb110da499dce35#diff-f41e9d04a45c83f3b6f6e630f10117feR69

fxy is an internal thing, so I unexported it.

I think there could be value in exposing more from your library.
E.g. note that your za2tu and tu2za, i.e. the spherical projection is useful to have (see e.g. https://github.com/substack/healpix or WCSLib).
XY is used e.g. in HiPS (see Fig 4 in PDF here).
There it's XY within a tile, but that would be easy to compute I think if one has the global XY within the base pixel.
So overall I'm +1 to expose more functions, to make them appear in the docs and easily accessible.

OK, I understand some internal functions are useful.
To export them, we have to clarify notations or have some document.

Like I said in #11, I think we should start with that list and make it more visible in the docs.
I'm not familiar with the TS docs system. Should we put this in the README? Or are there "docstrings" we can put at the top of the file and then it will appear at https://michitaro.github.io/healpix/typedoc/modules/index.html ?
I'd be happy to help fill the documentation a bit, to continue to learn about HEALPix.

I really appreciate your help.
I added notations on typedoc.

We can add comments on typedoc something like this.


Ruby 2.7 warnings in Execute resource

    * execute[set hostname to chef-bk-ci.chef.io] action run/opt/chef/embedded/lib/ruby/gems/2.7.0/gems/chef-16.0.245/lib/chef/provider/execute.rb:52: warning: Using the last argument as keyword parameters is deprecated; maybe ** should be added to the call
/opt/chef/embedded/lib/ruby/gems/2.7.0/gems/chef-16.0.245/lib/chef/mixin/shell_out.rb:54: warning: The called method `shell_out!' is defined here

@lamont-granquist fixed this

Forgive my ignorance, but has the fixed been released yet?  I still encounter it using Chef Infra Client 16.0.257.

This fix has not yet been released. Most likely we'll ship a bug fix for 16.0 next week which will include the fix in https://github.com/chef/chef/pull/9762


Links in note are incorrect

The note in the README file refers to github.com/container-orchestrated-devices/container-device-interface/pkg/cdi while the correct reference is https://github.com/cncf-tags/container-device-interface/tree/main/pkg/cdi.

@CsatariGergely thanks for calling this out. We recently migrated our repos and the docs still refer to the old address. Would you be up to creating a PR that addresses the inconsistencies?

hey @elezar, can I work on this issue?

@ATREAY you're welcome to submit a PR.
We were going to wait on a "vanity URL" for this cncf-tags repos, but this does not have to be a blocker. (see https://github.com/cncf/toc/issues/1152).

Hey @elezar, I've made the necessary changes. Please take a look when you get a chance.

As far as I understand this has been addressed. Closing this issue. Please reopen if the links are still invalid.


Heuristic to choose a default for disable_caps_lock

Set to true when caps lock key is overwritten and there no other key is remapped to VK_CAPITAL.

replaced by #44


First-minute UX: user is flooded with errors if they forget about adding @SerializabilityTrait

That's what I've got while testing on Laguna project... I've activated ash plugin on the main module, and got like 100 errors of ......Response is used as Akka message but does not extend a trait annotated with org.virtuslab.ash.annotation.SerializabilityTrait. Passing an object of class NOT extending SerializabilityTrait as a message may cause Akka to fall back to Java serialization during runtime..
This is technically correct... but in fact I'd rather expect an error like there is no class annotated with @SerializabilityTrait on this module's classpath or sth like that, b/c that's the actual error to be addressed first.

search all files in client's source code directory

Huh... so basically add an extra phase of compilation, dedicated to just finding whether there's any @SerializabilityTrait anywhere, right? 🤔 or sth even more hackish (like accessing source files directly "on the disk"?)

Huh... so basically add an extra phase of compilation, dedicated to just finding whether there's any @SerializabilityTrait anywhere, right? 🤔 or sth even more hackish (like accessing source files directly "on the disk"?)

Yes, that's right

Hmmmm sounds like quite a lot of engineering... and a performance overhead for a very short-lived benefit (only really matters in the developer's first ever contact with this tool). Not closing the issue, but marking as nice to have and unassigning you then

Note: maybe this could be done by using the https://www.scala-lang.org/api/2.13.0/scala-compiler/scala/tools/nsc/Global.html#findMemberFromRoot(fullName:Global.this.Name):Global.this.Symbol - to be checked and tested.


Shortcut for interupting/restarting the kernel

JupyterLab's shortcut for interrupting the kernel is i, i and restarting the kernel is 0, 0. Can these be added to this extension?

Apologies for the delay, the command for interrupting kernels is jupyter.notebookeditor.interruptkernel & jupyter.interruptkernel for Notebooks and Interactive window, respectively.

Thanks! Not sure how I missed seeing those in the list of commands.
I added the "interrupt kernel" shortcut in pull request #10. These shortcuts match what is used by JupyterLab.
{
  "key": "0 0",
  "when": "notebookEditorFocused && !inputFocus",
  "command": "jupyter.notebookeditor.restartkernel"
},
{
  "key": "i i",
  "when": "notebookEditorFocused && !inputFocus",
  "command": "jupyter.notebookeditor.interruptkernel"
}

Useful !
Came to this repo to ask exactly this. Thanks for suggesting. I hope it will be implemented soon

FYI for anyone else who comes here...it looks like something changed in the extension. I had to update my keybindings
{
    "key": "0 0",
    "when": "notebookEditorFocused && !inputFocus",
    "command": "jupyter.restartkernel"
},
{
    "key": "i i",
    "when": "notebookEditorFocused && !inputFocus",
    "command": "jupyter.interruptkernel"
}


Xcode 12 use_frameworks compatibility

Reference https://github.com/facebook/react-native/issues/29633#issuecomment-694187116

Summary
Explain the motivation for making this change: here are some points to help you:

What issues does the pull request solve? Please tag them so that they will get automatically closed once the PR is merged
What is the feature? (if applicable)
How did you implement the solution?
What areas of the library does it impact?

Test Plan
Demonstrate the code is solid. Example: The exact commands you ran and their output, screenshots / videos if the pull request changes UI.
What's required for testing (prerequisites)?
What are the steps to reproduce (after prerequisites)?
Compatibility

OS
Implemented

iOS
✅❌

Android
✅❌

Checklist

[ ] I have tested this on a device and a simulator
[ ] I added documentation in README.md
[ ] I updated the typed files (typescript)
[ ] I added a test for the API in the __tests__ folder

Sorry, didn't see that there was a PR already for this. I created an issue for the same. Should I close it? https://github.com/react-native-svg/react-native-svg/issues/1510

@karvulf may I didn't get your comment, is there an issue when you try to install? I made the change locally in my machine and it works as expected.

and after pod install, React.framework is linked.

When will this be merged?

Since https://github.com/react-native-svg/react-native-svg/pull/1604 has been merged, I will close it. Feel free to comment here if something is wrong and we can always reopen it. Thanks for your contribution :tada:


pipeline failed to build final report in conda enviroment

Trying to run the pipeline on conda enviroment with Ubuntu22.04 in python = 3.11.5. When running pipeline with example file and local data, it manages to generate numerous files output folder, but failed to generate final report.
args.csv
clusters.csv
labels.csv
overview.txt
result.json
status.json
takeaways.csv
translations.json

Duplicate from #16 ?

I tought its a different problem, but its also fixed after updating node.js to v21. Thanks!


TraceParserGen error on Microsoft-Windows-Kernel-Network

Repro steps:
C:\>PerfView.exe /nogui userCommand DumpRegisteredManifest Microsoft-Windows-Kernel-Network

C:\>TraceParserGen.exe Microsoft-Windows-Kernel-Network.manifest.xml
Reading manifest file Microsoft-Windows-Kernel-Network.manifest.xml
Error: Error Microsoft-Windows-Kernel-Network.manifest.xml(32): Undefined Id Datasent.
Use /? for help.

Looking into the xml it might be that the bug is in the xml generation by PerfView, as there are 2 spurious '>' chars - but fixing them manually does not fix the TraceParserGen error.

Addition: the TraceParserGen copy on oneDrive  seems to succeed - with many duplicate-name warnings.


[Feature Request] Auto save after day end

Right now it appears I must resave a run before I start a new one. This isn't the biggest issue, but was wondering if there was a way to autosave a run after a day is completed

So you mean to save a run, after each day completes? The game does that automatically. Before starting another one you can save the current.
Or do you mean a system that makes a backup of the current run in case you start a new one?
Or making a new save file after each day (and overriding the old)?
Autosave isnt't the biggest priority, but i'll add that to the ideas.

So, lets say I make a save on Pizza Prep Day 2. Lets call it save 01/01/1969 00:00:00 (EPOC)
I then continue to play.
Now, my friends want to play. Sweet! So I make a new run.
Now it's the next day and I want to go back to my solo. If I load the save, it will be Prep Day 2. I tested this, but maybe I did something wrong?

In a similar vein, I had expected more like a “save slot” system; where you name a slot (e.g. “solo” for solo runs), and then the mod keeps the save file for that around in that slot. Now you want to host multiplayer and create a new slot. The latest state of the “solo” slot is saved for returning to it later.
So more like different profiles than a classic “save” system. I might fork this at some point if that’s OK :)

Now it saves after each day. But only if you have already made a save for that run before.
v1.3.3


chore(main): release 0.14.9

:robot: I have created a release beep boop
0.14.9 (2024-03-03)
Bug Fixes

Favorites: ensure full display of favorites content (#323) (38bab5e)
Home: resolve home tabs drifting issue(#349) (e63ead9)
TopBar: center the right side items (858e381)
WatchLater: correct watch later jump url (f303a29)
搜索框用中文输入法输入单词回车不应该直接触发搜索 #334 (6adfe9c)
深色模式适配 (#341) (9a6eceb)

Docs

update chrome web store link (#352) (6fca372)

CI

only sync branch in source repo (#353) (61d3354)

Miscellaneous

update version number (c14fd95)
为 adaptedStyles 中的正则匹配页面添加中文对应描述 (288ba26)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/hakadao/BewlyBewly/releases/tag/v0.14.9 :sunflower:


Added the script to generate eval-topX-recall_at_10_415.csv file

Added the script to generate the csv file for better replicability.

Add description of author and file purpose.


9.7 release code change

@zhaoqin-github
WIP 9.7 release code change
This patch try to enable 9.7 release on github

code looks good to me . but it's probably better to leave the commit which modifies init.py as a separate pull request, and only merge that one until the very last minute.(in case we have any more commits at all)


Add history to cohort registry

The cohort registry manages information about the active members of the cohort.  It would be useful to extend this to maintain a history of the membership so that it is possible to look up the originator of any metadata found in the repository.
Should we also add details of open metadata archives loaded too?  This way we have a complete record of the "home repositories" of all metadata types and instances.
(Do nto need to include external instances because have "replicatedBy" value to tie it back to owning open metadata repository.

It would be useful to know cohort 'members' that have no repository - ie cocoMDS4 for coco pharma. This aids in understanding all the collaborators in the cohort. Even though these servers do not participate in federated queries & updates, they do host services ie OMAS etc.

Still need this

Still need this

I am closing this since the DEREGISTERED_REPOSITORY support was enabled without this capability.


Revert "pick one DNS cleanup codepath and use it everywhere"

This reverts commit 65fb4af148ca96acae23681bb6865b2724cca417.
Causes a panic.
/cc @joelddiaz @akhil-rane

/lgtm

ok, e2e, I've waited long enough for you.
Forcing...


Automatic error report

<!---   *** BEFORE SUBMITTING: PASTE CLIPBOARD HERE TO COMPLETE YOUR REPORT ***   ---!>
Variable explorer is not working properly

This is fixed in our latest version. Please update by opening the Anaconda Prompt and running there
conda install spyder=3.3.4


Add style & type checking action

Adds style & type checking action

Codecov Report

Merging #33 (ca4cac4) into master (d0563e6) will decrease coverage by 3.86%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
- Coverage   42.69%   38.82%   -3.87%
==========================================
  Files          12       11       -1
  Lines         260      273      +13
==========================================
- Hits          111      106       -5
- Misses        149      167      +18

Impacted Files
Coverage Δ

tasks.py
0.00% <0.00%> (ø)

twitter_video_tools/utils/youtube_dl_wrapper.py
63.63% <0.00%> (ø)

twitter_video_tools/tests/test_sample.py

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Service Page Changes - list

Remove 2 Colums - Add Recurring On /Off and Recurring Scheduler Count - 5 to service list
demand by @vinpahilwani

Remove Status Update Column and Group Column


Request for Releasing Pretraining Code

Thank you for your wonderful work first! I am wondering are you planning to release the pretraining code so that people could use customized data to pretrain the scFoundation model?

I have the same question, whether it will be released?

Me too, I also wonder is it possible pretrained code can be released?

Following as well! Specifically wondering about this:
sys.path.append("/nfs_beijing/minsheng/scbig/bioinfoDownStream/pretrain/")
in line 12 of https://github.com/biomap-research/scFoundation/blob/main/enhancement/run_embedding_sc.py
Was wondering since a lot of the functions (eg load_model_frommmf, gatherData, pretrainmodel) seem to not exist in the code. I can only assume they come from some file in the pretrain/ folder...

Hi everyone, we now provide an API for users to infer the embeddings of their own data. Please find more information here: https://github.com/biomap-research/scFoundation/blob/main/README.md#api


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

AWS CodeBuild CI Report

CodeBuild project: DDKIntegrationTestsD8472609-5SCHWjldmWcf
Commit ID: 0dad8212b1ad6bf46d09c7ad43fe1c1ab2de00d3
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Adding University of Maine at Augusta file

https://www.uma.edu/
https://www.uma.edu/academics/programs/cis/

@philipto that's unfortunate it has been abused :(.  I have applied, and been approved, for the GitHub Student Developer Program as of today. Once my GitHub account is updated with the status I will apply again with JetBrains through GitHub. Thanks.


docker run -it mcr.microsoft.com/azure-cli does not work on Windows Mode Docker Container

Is your feature request related to a problem? Please describe.
I tried to install Azure CLI on docker with Windows Mode not Linux mode using: docker run -it mcr.microsoft.com/azure-cli and it throws error: C:\Program Files\Docker\Docker\resources\bin\docker.exe: image operating system "linux" cannot be used on this platform.
Describe the solution you'd like
Any command line for installing Azure CLI on Windows mode container?
Describe alternatives you've considered
Any other way to install Azure CLI on Windows mode container
Additional context
N/A

HI @fengzhou-msft could you pls have a  look? thanks.

@Astersss the image is for linux. For windows, we have the dockerfile, but we don't release the image built on that. You can build it by yourself.