Implement IRI Encoding

Excuse my poor understanding of this topic in advance.
Something that occurred to me is that it should be possible to exceed the compression we have via compressToEncodedURIComponent() using IRIs. The logic being simply that URIs only allowed for ASCII characters, but IRIs allow for the millions of different unicode characters, so by using these extra characters we should be able to encode some input more efficiently. Notably the goal for this kind of thing is to make the shortest URL, which isn't necessarily the same as making the smallest memory representation (because Unicode characters are larger than ASCII ones in memory). Also, this is different to the existing compressToUTF16 function, because not all unicode characters are legal in IRIs, although most are.
If this isn't particularly difficult, I'm happy to contribute this functionality if I'm able. Having looked at the existing implementation, I wonder if this is as simple as providing a new "conversion" function that encodes integers into strings. This could be done similarly to how you have done it with URIs:
https://github.com/pieroxy/lz-string/blob/2f749bc9fe3cc889fa9e32d2769a930d977596e6/libs/lz-string.js#L95
For reference, the IRI standard that defines the available characters is here: https://datatracker.ietf.org/doc/html/rfc3987#section-2.1. The part that is of relevance is the iquery rule, which defines the legal IRI query string:
   ipchar         = iunreserved / pct-encoded / sub-delims / ":"
                  / "@"

   iquery         = *( ipchar / iprivate / "/" / "?" )

   ifragment      = *( ipchar / "/" / "?" )

   iunreserved    = ALPHA / DIGIT / "-" / "." / "_" / "~" / ucschar

   ucschar        = %xA0-D7FF / %xF900-FDCF / %xFDF0-FFEF
                  / %x10000-1FFFD / %x20000-2FFFD / %x30000-3FFFD
                  / %x40000-4FFFD / %x50000-5FFFD / %x60000-6FFFD
                  / %x70000-7FFFD / %x80000-8FFFD / %x90000-9FFFD
                  / %xA0000-AFFFD / %xB0000-BFFFD / %xC0000-CFFFD
                  / %xD0000-DFFFD / %xE1000-EFFFD

   iprivate       = %xE000-F8FF / %xF0000-FFFFD / %x100000-10FFFD

Another use-case for a standardised dictionary for compressToCustom!


Idea: Family Tree?

Do you think family tree functionality would be within the scope of what you're attempting here? There currently aren't any real supported, working mods that accomplish this (the "Family Tree" mod is not being updated anymore by its author). It would be nice to see a mapping of the family members, colonists and non-colonist wise (i.e., different faction members) for all of your colonists.
Thanks for all the excellent work you do in general. As someone in the IT world, I appreciate your commitment to clean & high-performance code and feel a lot of your work substantially enhances the experience.

It's a neat idea I'll give you that, just outside of the scope of a colonist bar.


Allow Vault to use env vars.

For instances VAULT_SKIP_VERIFY

Can this have a test around it?

@relistan the underlying lib actually test the behaviour with os.Setenv(VAULT_SKIP_VERIFY, "true") but not too much to test from our side.


Test URL for Marketplace items are outdated

The URL to validate the published custom marketplace items still references the port 30015 URL that is no longer valid. Needs to be updated to galleryartifacts URL

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

ID: e401ae26-588f-124f-09a8-f1f6dc75da4e
Version Independent ID: 6a9509a1-f16c-f527-5df1-9dae2d8eeaed
Content: Create and publish a Marketplace item in Azure Stack Hub - Azure Stack Hub
Content Source: azure-stack/operator/azure-stack-create-and-publish-marketplace-item.md
Service: azure-stack
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@caz3542 Thank you for the feedback -- can you please include the name and/or path to the article in question?

https://docs.microsoft.com/en-us/azure-stack/operator/azure-stack-create-and-publish-marketplace-item?view=azs-2002 - Item (6)

@caz3542 Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@caz3542 Thank you for the report, we are currently looking into this.

#in-progress

@caz3542 Hi again and sorry for the delay. I have obtained the new information, as follows are the corrected gallery artifacts URLs. I will be updating the article with this information today:
https://galleryartifacts.adminhosting.[Region].[externalFQDN]/artifact/20161101/[TemplateName]/DeploymentTemplates/Template.json
https://galleryartifacts.hosting.[Region].[externalFQDN]/artifact/20161101/[TemplateName]/DeploymentTemplates/Template.json
#please-close


Java client: batch messages - do not create a batch message of 1 single message

I think that we could implement a small but effective enhancement to batching.
It may happen that even if you enable batching you come to create entries with 1 only message.
Processing batch messages requires a good amount of resources, both on the broker and on the client side.
Especially when you are using PIP-105, Broker side filtering, you have to unpack (and decompress) the whole entry in order to process the very single message.
So my proposal is to change the (Java) producer, to make it produce a regular message instead of a batch message if the batch contains only 1 message.
Testing should cover:

regular batch sending using the Java producer
KEY based batcher builder

I am working on that, thanks


New entries for Hong Kong, Panama, Sri Lanka, USA (Oregon), minor updates for Russia & Morocco

Many thanks as usual Paul!!


Eclipse compile time code generation with QueryDSL 4.1.4

Is there any support for eclipse compile time code generation with queryDSL 4.1.4, I could successfully generate QClasses with query dsl 3.3.2 but I am not able to generate QClass code with 4.1.4.
P.S. I am not using maven.

If you use javac directly, and have the apt classifier jar (querydsl-jpa:{version}:apt) on the classpath the annotation processors should be picked up as-is.
You can also use the querydsl-apt:{version}:{backend} classifier jars.
There are quite a few different classifiers, which you can manually download.
https://search.maven.org/#search|ga|1|g%3A"com.querydsl"
P.S. I recommend you use some kind of build tool, even if it's just to manage dependencies.

So, to be sure.
This release breaks annotation processing?
https://projects.eclipse.org/projects/eclipse/releases/4.7.3a
Can anyone get me a diff between those versions? (4.7.3 to 4.7.3a)
Want to see why they're throwing an exception with that message. The message itself doesn't tell me much.

I believe so. I will try to download an older oxygen release tomorrow. Not sure whether that will work though.
The current TypesImpl can be found here: https://git.eclipse.org/c/jdt/eclipse.jdt.core.git/tree/org.eclipse.jdt.compiler.apt/src/org/eclipse/jdt/internal/compiler/apt/model/TypesImpl.java?h=master
If I am seeing this correctly there was a validation step added with:
https://git.eclipse.org/c/jdt/eclipse.jdt.core.git/commit/org.eclipse.jdt.compiler.apt/src/org/eclipse/jdt/internal/compiler/apt/model/TypesImpl.java?id=0574db9360ba34dc60ca55c743cc86eb6aa06d51
I will try and see whether I can get a Eclipse PDE up and running tomorrow and set a breakpoint for more information.

Could the problem be that JPAConfiguration.java#L116 is being called for an ExecutableElement? Does it even make sense to try and call erasure() on `ExecutableElement.asType()?

I confirm that the problem is that erasure() is being called with an ExecutableTypeImpl. Is this a problem with ecj or with querydsl? I don't have a lot of experience with annotation processing. Should erasure maybe be called on the return type of the ExecutableElement?

I have just created an Eclipse Neon environment and checked what erasure returns for ExecutableTypeImpl:

I believe something like this would work:
TypeMirror erasure;
if (element instanceof ExecutableElement) {
  erasure = ((ExecutableElement) element).getReturnType();
} else {
  erasure = types.erasure(element.asType());
}

I have included my "fix" and could successfully execute mvn test.


Add screenshot based testing

It'd be nice to script screenshots to be taken of the site during testing and auto-post them to PR threads.

I don't understand this... what are we "editing online"? And what screenshots are we testing?

Ah sorry! Description updated

Aaaah... interesting.

This one is done, yes?


fix(multi-select): avoid cyclic dependency for Svelte 5 compatibility

Fixes #1986
Here’s an attempt to fix the issues in the MultiSelect component that caused incompatibility with Svelte 5 by creating infinite loops (https://github.com/carbon-design-system/carbon-components-svelte/issues/1986). It seems like the core issue was the circular dependency between the items and sortedItems variables:
items assigned tosortedItems in an afterUpdate block:
https://github.com/carbon-design-system/carbon-components-svelte/blob/ac38e9d3531af85b413ba9919fdc70bca62c2002/src/MultiSelect/MultiSelect.svelte#L261
sortedItems depends on items:
  $: sortedItems = items.map((item) => ({
    ...item,
    checked: selectedIds.includes(item.id),
  }));

The commit removes the sortedItems variable, which served an intermediate role, and changes are made to items directly.

I've pushed a 6fbedab that does the following:

Remove the cyclical dependency in afterUpdate
Consolidate around sortedItems internally to avoid the cyclical dependency

I've so far tested with both Svelte 4 and Svelte 5.

Thank for reviewing and catching the selectedIds issue. I think that your solution introduces one breaking change, though: the sorting logic doesn’t update the items variable, so the new sort order is not propagated to the parent (if items is bound reactively).

Arguably that was a bug; the property was not marked as being reactive and who really wants this?

It is marked as reactive in the docs (https://svelte.carbondesignsystem.com/components/MultiSelect#props), though, as you say, maybe it’s not useful for this prop to be reactive.

Arguably that was a bug; the property was not marked as being reactive and who really wants this?

Nice catch – MultiSelect items should not be reactive, and that was identified via this cyclic dependency fix. Generated docs need to be updated to reflect this – created #2042

Released in v0.86.0


Where is your mystery unit converter?

Hello developers,
This is a question related to your website, not iOS app.
May I ask where is your mystery unit converter(s)? And how can I summon it?
Yesterday, typing 1 foot to meters made your mystery unit converter show up. Today that's not working.
Would you please help make them available in easier way? The question to a good engineer is: if you have those things, why you make it extremely hard for the users to find them?

Hello,
Generally what you describe should be working. Reaching unit converter is as easy as searching for it: https://duckduckgo.com/?q=unit+converter It should also automatically appear for the related queries - like the one you mentioned.
If you have issues related to search results (like missing unit converter for a particular query), there is an option to send us feedback about it:

On the web, 'Send feedback' button located in lower-right corner.
On the mobile, 'Send feedback' button located at the very bottom of the results page.

List of all instant answers is available here: https://duck.co/ia
Hope that helps! Please ask if you have follow up questions (or reach out directly to website team by sending them feedback).

To @bwaresiak and DuckDuckGo developers:
I apologize for my question. I was not calm when I typed it. Then, for embarrassment, I deleted my account.
Thank you for your hard work.

Thanks for the follow up - worry not, no offense taken 😃
Since this issue is not related to iOS I'm going to close it. Fell free to reopen if needed.


Contact info in fabric.mod.json points to quilt loading screen

As stated in title, contact information is wrong for release 1.0.0

fixed in 1.0.2


[Feature] Rewind memory access loops

Issue
Currently the loops for accessing top level variables that are automatically generated with pipelining at II=1 which is great. However, in my testing this can still lead to 10x the theoretical runtime for 2d arrays.
Solution
Adding rewind to the end of the automatically generated pipeline pragmatism fully solves this performance issue while sometimes also reducing hardware usage.
Example - My matrix vector multiply program.
Without rewind (current setup):

78 cycle interval for buf1
With rewind manually added:

4 cycle interval achieved.

I'm sorry this is Zack, I'm just on the wrong account.


Preparing the JSHint set-up for the next release

"laxbreak" is deprecated as it relates more to code style, not correctness.
However, as it is a relaxing option, actually not setting it means I'm using a deprecated functionality. If I do not want to use the deprecated functionality, I need to set a deprecated option.
It seems that, at this point I actually am unable to get rid of using JSHint in a non-deprecated mode?

Related - #2625

Thank you for the reference!


Can't find math.h and other headers

I'm using OSX.  I've tried tracking this down, but have spent way too much time trying to figure out where the pre-processor is called, and how to resolve this.
Unitive-MacBook-Pro-1598:scaffcc gonzo$ ./scaffold.sh Algorithms/Binary_Welded_Tree/binary_welded_tree.n100s100.scaffold
Algorithms/Binary_Welded_Tree/binary_welded_tree.n100s100.scaffold
[Scaffold.makefile] Compiling binary_welded_tree.n100s100_merged.scaffold ...
binary_welded_tree.n100s100_merged.scaffold:14:10: fatal error: 'math.h' file not found
#include <math.h>
^
1 error generated.
make: *** [binary_welded_tree.n100s100.ll] Error 1
I assume the standard C++ headers are used.  If that's true, these are installed and used in my C++ code.  For the record, I can go to the scripts dir and run regression_tests.sh.  12 of 20 run fine... only the ones requiring headers math.h or stdio.h fail.

Guessing there's not a lot of OSX users here...
I gave up on this after also trying to get the QX simulator to load and run on OSX.  That crashed on startup.  With that, I switched to using LINUX.  Everything works fine.
Bummer that Mac QC tools seem to be orphaned; but happy LINUX appears to have support.


The C4 bomb's sound lack of sense of direction and distance

The C4 bomb's sound lack of sense of direction and distance.
Converting the sound to mono may help :D

From my experience, every sound in Minecraft has this issue. The subtitle feature helps to point out the direction.


[MINOR] Update known_translations

What changes were proposed in this pull request?
In the PR, I propose to update the file dev/create-release/known_translations by new contributors to 3.3.0
Why are the changes needed?
While preparing the list of contributors to 3.3.0 in the PR https://github.com/apache/spark-website/pull/397, I have found that some of contributors are not in the file known_translations.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
None.

Please remove me, must be a mixup.

I am closing this PR because some mappings are wrong. Sorry all for massive pings.


Changed URL

Not really a bug, but the installation section refers to the wrong github.
Unfortunnately I had to delete the old integration in HACS and add the new url to use it.
Keep up the good work!

Thank you for reporting, have already fixed that!
But yes, I had to transfer this integration when I've merged my work and private GitHub accounts. GitHub redirects automatically redirects transferred repo's, but HACS does not handle that well...

Thank you for reporting, have already fixed that!
But yes, I had to transfer this integration when I've merged my work and private GitHub accounts. GitHub redirects automatically redirects transferred repo's, but HACS does not handle that well...


Update PPP

I have updated my PPP with some new information regarding my contributions to tp.
However, the document is still WIP and will be ready by the deadline of v1.3a.

Looks good!

@JCSnap , @seantehds , @Sebtey , @maypfv , this PR is ready for merging. Thank you!


Correctness of Hyps.Delete

Did we prove strengthening somewhere? I actually thought it is false. @cangiuli

If strengthening were false, that would be very surprising & confusing... At least as far as the proof theory is concerned, it seems unlikely that there is any proof which can be done "without using" a hypothesis, but which cannot be done "without" that hypothesis being "existing".

I think we can prove a:Void >> whatever without mentioning a in the current implementation. Maybe it's time to rethink about Void-elim(a)?

Actually, I feel one can still work around it even with Void-elim....

Hmmmm, maybe so... Let me think about it a bit.

On Aug 8, 2017, at 8:51 AM, favonia (Sai-hong サイホン<EMAIL_ADDRESS>wrote:
I think we can prove a:Void >> whatever without mentioning a in the current implementation. Maybe it's time to rethink about Void-elim(a)?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@favonia I got confused; Hyps.Delete is not strengthening, but actually weakening! (The nature of backward inference gets my mind reversed sometimes.)
All that Hyps.Delete says is, "To show J assuming x:A, it suffices to show J without assuming x:A."

@jonsterling Oh, I see.


[Regression] Since v2.0.0 pagination buttons scroll to top of page

Since v2.0.0 and the move to href="#" for pagination buttons, the default behavior for such links takes place on click. I also think that those whould be buttons, not anchors, but that's another matter.
To get around this, I am doing this after list initialisation:
$list_container.find('.pagination').on('click', 'a', function(event) {
	event.preventDefault();
});

Not ideal, for somehting that shouldn't be there in the first place.
Also, of note, you seem to be preventing these events bubbling, so trying to be clever and doing something like this doesn't work:
$(document).on('click', '.pagination a', function(event) {
	event.preventDefault();
});

It doesn't seem to work for me either way, keeps going to the top of the page ☹

It doesn't seem to work for me either way, keeps going to the top of the page ☹

Put this after list initialization. Replace $list_container with you list container and make sure pagination is under the class you filter by .find('.pagination'):
$list_container.find('.pagination').on('click', 'a', function(event) {
	event.preventDefault();
});

I ran into this issue as well. I'm on version 2.3.1.
As you can see in the pagination.js source there is option to set the item. It's also in the docs.
I use it like this:
const options = {
  pagination: [{
    innerWindow: 10,
    item: '
<button class="page"></button>
',
  }],
};

That should fix your issue.

@rutger1140 Thank you so much for that info! I totally missed it, my bad! Closing.


Ref script fee (chang)

See update of CML https://github.com/dcSpark/cardano-multiplatform-lib/pull/349

Documentation of the changes: https://github.com/IntersectMBO/cardano-ledger/blob/lehins/fee-changes-adr/docs/adr/2024-08-14_009-refscripts-fee-change.md

Done in #47


Added "Nix Italia" NUG

Before accepting (hopefully :upside_down_face:) this we would like to know if it's possible having a notion page like liverpool.nix.ug
In that case I would change the url to that page.

I also hope that a png logo isn't a problem

If you have a website that I can create a CNAME for on DNS like italy.nix.ug -> your_server I can do that. I think it would be better to make an Elm program instead of a Notion page in future, that way I can template it in the code in this repository, and it could be easy to PR it. Sadly the Notion page was far more work than it's worth, and I wouldn't do this for people unless it could be automated like I suggested.

Also, it looks like this PR is just a png file that is named "default.nix" by accident, and you haven't created a default.nix which contains the details of the NUG like these other PRs which you can use to see how to correctly add a NUG:

https://github.com/nix-how/nix.ug/pull/1/files
https://github.com/nix-how/nix.ug/pull/2/files

it would be better to make an Elm program instead of a Notion page in future

It would be more cool, I agree.

I can create a CNAME for on DNS like italy.nix.ug -> your_server I can do that

We don't have anything right now, I would say to use the matrix room link for now. But thank you! We'll definitely reach you if we'll ever have a page somewhere.

looks like this PR is just a png file that is named "default.nix" by accident

Ooops, I force pushed it removing my second commit that overwrote default.nix.
For now I'll keep this PR as a draft since we were discussing a new logo, as soon as it's ready I'll mark it as ready and I'll ping you. Or if you prefer merge it and I'll make another PR when logo is ready.

@MatthewCroughan we are still discussing about the logo (someone also proposed to write a derivation that outputs the svg file). Things will take a long time probably, could we merge this as is, please?
We'll open another PR in the future for the logo.


ip space / pools

would it be possible to add a IP/prefix pool feature?
here's how I see this could be used:
admin: create a pool

define size of pool item (/32 for fixed ip or any other prefix length)
assign one or more IP ranges (prefixes) to the pool from which items can be requested
assign a role to the pool (infrastructure, dhcp, nat, ssl, etc..)

user: request an IP/prefix

choose an IP / prefix length
choose a role
provide description

ipspace/pool module:

pick the next free IP/prefix available from the pool matching prefix length and role and assign it
calculate the usage of a pool and send a notification when it reaches a certain threshold (80%?)

if the request mechanism is available via api then it will become very easy to script IP assignation without having to search through the whole ip space.
we could use the new 'is_pool' prefix type to mark prefixes used in pools
Thanks,

NetBox already has the ability to track pools of IP addresses, by creating IP addresses with the "DHCP pool" status. A pool must be defined as a set of IP addresses, because a pool is not necessarily bound to prefix size (e.g. you can create a pool of .10 through .100 inside a /24). Release 1.8.0 will support wildcard expansion during the creation of IP addresses to make this approach more practical.
Any interaction beyond what's currently supported will need to be accomplished via some external component; I want to avoid NetBox functioning as any sort of brokerage system. Once the API has been made fully writeable (#113) this should be much easier to accomplish.
Does this address your suggestion?

Hi
I had to play a bit more with NetBox to understand how to DHCP pool
currently works.
(using the v1.7.3 docker version as a sandbox)
First:

create a few IPs with DHCP status
created a new device.

At this point I have 2 choices:

edit device and add IP address manually (the IP address needs to be
looked up and reserved separately)
go  to IP addresses listings, find a DHCP marked IP address, select
it and assign it to the device

My suggestion is aiming at providing the following workflow:

add new device
add new IP address : choose a pool fitting the role/site/etc..
-> the next available IP is assigned out of the pool
Is this what you mean by brokerage system?

As the input and the output remains within NetBox I believe it would be
useful.
Once the new write API is available this could be done via external
scripts but we'd loose the benefit of the web interface.
It also helps to separate roles (IP resources provider and IP resources
consumer)
This way you could imagine an external script doing VM provisioning:
simply request an IP from a VM pool.
So I tried to play a bit more with custom fields:
By adding two fields: "pool" (text or selection) and "pool-assigned"
(boolean)
I can reproduce the structure suggested above but without the intelligence.
Prepopulate the database with IP addresses having the DHCP status and
assign them a pool and pool-assigned as false.
Via API it would be possible to retrieve information about IP addresses
belonging to a "pool"
and then calculate out the next available IP address and make callback
to the API to assign the IP (pool-assigned=true)
My suggestion was going a bit further:
If we can define a pool of /32s, then we could as well define a pool of /27s
In this case instead of filling the pool with a range of IPs (.10 to .100)
We'd choose some bigger ranges for example out of which we can pick /27
prefixes.
So in the end:
If the purpose is to use the NetBox web interface to manage the
different resources then it would make sense to implement the suggestion.
If practice shows that users prefer to use the API around the NetBox
data structure then it's not needed.
This is something I was badly missing in phpipam, and that exists in nipap.
Having something similar in NetBox would really bring the DCIM and the
IPAM part together in my point of view.
I hope this makes my suggestion clearer.
I guess if other users also see the benefit then they should voice their
interest.
Thanks
On 03.01.2017 16:21, Jeremy Stretch wrote:

NetBox already has the ability to track pools of IP addresses, by
creating IP addresses with the "DHCP pool" status. A pool must be
defined as a set of IP addresses, because a pool is not necessarily
bound to prefix size (e.g. you can create a pool of .10 through .100
inside a /24). Release 1.8.0 will support wildcard expansion during
the creation of IP addresses to make this approach more practical.
Any interaction beyond what's currently supported will need to be
accomplished via some external component; I want to avoid NetBox
functioning as any sort of brokerage system. Once the API has been
made fully writeable (#113
https://github.com/digitalocean/netbox/issues/113) this should be
much easier to accomplish.
Does this address your suggestion?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/digitalocean/netbox/issues/761#issuecomment-270138061,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXue71vz5-RPJM1QTTm4iEZChDLKIsrvks5rOmeWgaJpZM4LZH2l.

It sounds like you're conflating at least two ideas. The first is the creation of DHCP pools in NetBox. As explained, this is done by creating the desired number of IP addresses with a status of "DHCP." It is not done with prefixes because a pool does not typically comprise an entire prefix; rather, they are of arbitrary size within a prefix (e.g. 200 IPs out of a /24).
The other concept is the assignment of an available IP address from a pool. I invite you to read through #665 where this was discussed at length. I'm going to close this issue out as I believe what you're suggesting has already been proposed.

Hello moolinet,
I have the same problem as you.
I have a block 91.xxx.41.32/27 allocated for my VM and I try to be able to use the API to request an IP and assign it to a device (new VM)
For the moment, I do not see how to do because I only see GET methods in the REST API.
Or go through the PostGres database but it's not the goal when you have an API. 👎


Stage Selected Range Bug with Path Capitalisation

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

VS Code Version:

Version: 1.91.0 (user setup)
Commit: ea1445cc7016315d0f5728f8e8b12a45dc0a7286
Date: 2024-07-01T18:52:22.949Z
Electron: 29.4.0
ElectronBuildId: 9728852
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Windows_NT x64 10.0.22631

OS Version:
Edition	Windows 11 Pro
Version	23H2
Installed on	22/09/2022
OS build	22631.3810
Experience	Windows Feature Experience Pack 1000.22700.1020.0
Steps to Reproduce:
Hey team. Think I've uncovered a bug with the stage selected / stage selected range feature in the latest VSCode Update.
E.g. I've got a file /client/src/LetterWriter/toolbar/WriterToolbar.tsx where I want to stage just this change on line 88. See image below

When I click this + here, the following happens. For some reason VSCode thinks the capitalisation of the direct parent folder (actually toolbar with a lowercase "t") has changed to an uppercase. As seen below it duplicates the file in the working tree and the staged changes show the uppercase "T".

Is this a git bug somehow (Edit: I don't think so given my first comment)? On Github, the path is definitely lowercase and the same on my local disk. See below:

I've tried clearing git cache but this issue is persistent and I've encountered the same issue with other folders too :(. I haven't ever changed the capitalisation of this toolbar folder.
The change (line 88) does move from the working tree but only on the file with the incorrect path. Before a recent update this feature worked perfectly. Has something changed? I really like this feature and it would be good if it could be fixed.
A git status does also show the same as VSCode's src control tab so I'm wondering if the issue might be there instead.
Changes to be committed:
  (use "git restore --staged <file>..." to unstage)
        new file:   client/src/components/LetterWriter/Toolbar/WriterToolbar.tsx           <<<< (observe wrong path)
        renamed:    client/src/components/LetterWriter/getSelectedNode.tsx -> client/src/components/LetterWriter/utils/getSelectedNode.tsx

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        modified:   client/src/components/LetterWriter/LetterWriter.tsx
        modified:   client/src/components/LetterWriter/Toolbar/WriterToolbar.tsx            <<<<<
        modified:   client/src/components/LetterWriter/toolbar/WriterToolbar.tsx             <<<<< (observe duplicate file with different path)

Untracked files:
  (use "git add <file>..." to include in what will be committed)
        client/src/components/LetterWriter/plugins/AutoLinkPlugin.tsx
        client/src/components/LetterWriter/plugins/FloatingLinkEditorPlugin/
        client/src/components/LetterWriter/plugins/LinkPlugin.tsx
        client/src/components/LetterWriter/utils/canUseDOM.ts
        client/src/components/LetterWriter/utils/setFloatingElemPositionForLinkEditor.ts
        client/src/utils/url.ts

For what it's worth, doing this in the command line with git add -p then following the commands works without issue. Is VSCode using git add -p or doing something else?
D:\Redacted\Documents\Redacted\Redacted>git add -p   client/src/components/LetterWriter/toolbar/WriterToolbar.tsx
diff --git a/client/src/components/LetterWriter/toolbar/WriterToolbar.tsx b/client/src/components/LetterWriter/toolbar/WriterToolbar.tsx
index 4d71187..665840b 100644
--- a/client/src/components/LetterWriter/toolbar/WriterToolbar.tsx
+++ b/client/src/components/LetterWriter/toolbar/WriterToolbar.tsx
@@ -33,9 +33,10 @@ import {
     UNDO_COMMAND,
     REDO_COMMAND,
     LexicalEditor,
+    KEY_MODIFIER_COMMAND,
 } from "lexical";
 import { $setBlocksType } from "@lexical/selection";
-import { $isLinkNode } from "@lexical/link";
+import { $isLinkNode, TOGGLE_LINK_COMMAND } from "@lexical/link";
 import {
     $isListNode,
     INSERT_CHECK_LIST_COMMAND,
(1/6) Stage this hunk [y,n,q,a,d,j,J,g,/,s,e,?]? n
@@ -85,14 +86,17 @@ import IconButton from "../../Inputs/IconButton";
 import { RadioOption } from "../../Inputs/RadioInput";
 import { TbFlask, TbLetterA, TbLetterASmall, TbTextResize } from "react-icons/tb";
 import { GrDocument } from "react-icons/gr";
-import { getSelectedNode } from "../getSelectedNode";
+import { getSelectedNode } from "../utils/getSelectedNode";
 import { FaCircle, FaRegCircle, FaRegSquareFull, FaSquareFull } from "react-icons/fa6";
 import { TOOLBAR_ICON_SIZE } from "../../../../constants/sizing";
 import { INSERT_PAGE_BREAK } from "../plugins/PageBreakPlugin";
 import { PARSE_REFERENCES_COMMAND, TOGGLE_REFERENCES_COMMAND } from "../plugins/ReferencePlugin";
+import { Dispatch } from "../../../utils/record";
+import { sanitizeUrl } from "../../../utils/url";

 interface ToolbarProps {
     toolbarRef: React.RefObject<HTMLDivElement>;
+    setIsLinkEditMode: Dispatch<boolean>;
 }

 class InvalidTagError extends Error {
(2/6) Stage this hunk [y,n,q,a,d,K,j,J,g,/,s,e,?]? s
Split into 3 hunks.
@@ -85,8 +86,8 @@ import IconButton from "../../Inputs/IconButton";
 import { RadioOption } from "../../Inputs/RadioInput";
 import { TbFlask, TbLetterA, TbLetterASmall, TbTextResize } from "react-icons/tb";
 import { GrDocument } from "react-icons/gr";
-import { getSelectedNode } from "../getSelectedNode";
+import { getSelectedNode } from "../utils/getSelectedNode";
 import { FaCircle, FaRegCircle, FaRegSquareFull, FaSquareFull } from "react-icons/fa6";
 import { TOOLBAR_ICON_SIZE } from "../../../../constants/sizing";
 import { INSERT_PAGE_BREAK } from "../plugins/PageBreakPlugin";
 import { PARSE_REFERENCES_COMMAND, TOGGLE_REFERENCES_COMMAND } from "../plugins/ReferencePlugin";
(2/8) Stage this hunk [y,n,q,a,d,K,j,J,g,/,e,?]? y
@@ -89,7 +90,9 @@
 import { FaCircle, FaRegCircle, FaRegSquareFull, FaSquareFull } from "react-icons/fa6";
 import { TOOLBAR_ICON_SIZE } from "../../../../constants/sizing";
 import { INSERT_PAGE_BREAK } from "../plugins/PageBreakPlugin";
 import { PARSE_REFERENCES_COMMAND, TOGGLE_REFERENCES_COMMAND } from "../plugins/ReferencePlugin";
+import { Dispatch } from "../../../utils/record";
+import { sanitizeUrl } from "../../../utils/url";

 interface ToolbarProps {
     toolbarRef: React.RefObject<HTMLDivElement>;
(3/8) Stage this hunk [y,n,q,a,d,K,j,J,g,/,e,?]? d

D:\Redacted\Documents\Redacted\Redacted>git st
On branch 5-custom-email-templates
Your branch is up to date with 'origin/5-custom-email-templates'.

Changes to be committed:
  (use "git restore --staged <file>..." to unstage)
        modified:   client/src/components/LetterWriter/toolbar/WriterToolbar.tsx
        renamed:    client/src/components/LetterWriter/getSelectedNode.tsx -> client/src/components/LetterWriter/utils/getSelectedNode.tsx

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        modified:   client/src/components/LetterWriter/LetterWriter.tsx
        modified:   client/src/components/LetterWriter/toolbar/WriterToolbar.tsx

Untracked files:
  (use "git add <file>..." to include in what will be committed)
        client/src/components/LetterWriter/plugins/AutoLinkPlugin.tsx
        client/src/components/LetterWriter/plugins/FloatingLinkEditorPlugin/
        client/src/components/LetterWriter/plugins/LinkPlugin.tsx
        client/src/components/LetterWriter/utils/canUseDOM.ts
        client/src/components/LetterWriter/utils/setFloatingElemPositionForLinkEditor.ts
        client/src/utils/url.ts

@hugs7, could you try to reproduce this issue in a small, isolated repository? That would significantly help us debug this issue. Thanks!

Sure, let me see if I can replicate

Was struggling to replicate this, even in the same repo for a moment but I think I got it. I think it's something to do with Windows allowing folder names to differ only by capitalisation, where as this isn't the case on Linux/macOS.
This wasn't how I came across the issue in the first place but to replicate the steps are below. I went back in my git history to see if I ever changed the /toolbar folder's capitalisation in the app I'm working on but I never did. Must be even another way to replicate this issue but sorry I wasn't able to reproduce it.

Make git repo
For the purposes of recreating this issue, I made a simple Vite app with npm create vite@latest and followed the prompts but I'm sure this bug would occur regardless. (b741e89ca0e17baa083a8ca7c3d89ee80ff9d7b3)
Make a file (I made a component called Foo.tsx at path /src/components/Foo.tsx)
Commit this file within VSCode to git (24cf8662040785c09416671dd64e01aeeba56ffe)
Either within File Explorer or VSCode's Explorer tab, change the components folder to components_old
Commit this change to git. (1cb1809de2109ea59aee9979ed59679c9ad7c89c)
Not sure if this step is required but make a change to Foo.tsx. This can be any change. Commit this change to git. (37852d2c38825bb26239eb8ff2d2837c720b570a)
Rename components_old to Components with a capital C.
Commit the subsequent rename of the folder. (461b08bcbe8ac8dddad0aba8ac88bafb7becde5d)
Now for the bug, make a change to Foo.tsx, now within the /src/Components directory
Go to VSCode's Source Control tab and click Stage Block on (at least) one change. (I commited the incorrect change VSCode staged here: ea0e11cb1506cf8624cb6dac82a2c57be42513e8)
Observe in the Source Control panel, the same issue described in my OP

E.g.

If I then go to commit the next changes

The change with the lowercase components will be an empty change (f0d6e73c7a4e6b493690e45451cdb2f9a9d97112), and VSCode won't commit the other file, even if it's staged, but the change with the uppercase Components will produce an error and doesn't let me commit.

Logs below
2024-07-16 22:45:49.435 [info] > git add -A -- . [32ms]
2024-07-16 22:45:49.474 [info] > git -c user.useConfigOnly=true commit --quiet --allow-empty-message --file - [36ms]
2024-07-16 22:45:49.510 [info] > git config --get-all user.name [32ms]
2024-07-16 22:45:49.538 [info] > git config --get-all user.email [25ms]
2024-07-16 22:45:49.565 [info] > git config --get commit.template [24ms]
2024-07-16 22:45:49.569 [info] > git for-each-ref --format=%(refname)%00%(upstream:short)%00%(objectname)%00%(upstream:track)%00%(upstream:remotename)%00%(upstream:remoteref) --ignore-case refs/heads/main refs/remotes/main [25ms]
2024-07-16 22:45:49.603 [info] > git status -z -uall [31ms]
2024-07-16 22:45:49.637 [info] > git for-each-ref --format=%(refname)%00%(upstream:short)%00%(objectname)%00%(upstream:track)%00%(upstream:remotename)%00%(upstream:remoteref) --ignore-case refs/heads/main refs/remotes/main [32ms]
2024-07-16 22:45:49.673 [info] > git config --local branch.main.vscode-merge-base [34ms]
2024-07-16 22:45:49.673 [warning] git config failed: Failed to execute git
2024-07-16 22:45:49.707 [info] > git reflog main --grep-reflog=branch: Created from *. [31ms]
2024-07-16 22:45:49.735 [info] > git symbolic-ref --short refs/remotes/origin/HEAD [25ms]
2024-07-16 22:45:49.735 [info] fatal: ref refs/remotes/origin/HEAD is not a symbolic ref
2024-07-16 22:45:51.339 [info] > git config --get commit.template [30ms]
2024-07-16 22:45:51.342 [info] > git ls-tree -l HEAD -- D:\Hugo Burton\Documents\vscode-stage-selected\src\Components\Foo.tsx [30ms]
2024-07-16 22:45:51.346 [info] > git ls-tree -l HEAD -- D:\Hugo Burton\Documents\vscode-stage-selected\src\App.tsx [31ms]
2024-07-16 22:45:51.352 [info] > git ls-tree -l HEAD -- D:\Hugo Burton\Documents\vscode-stage-selected\src\Components\Foo.tsx [35ms]
2024-07-16 22:45:51.355 [info] > git for-each-ref --format=%(refname)%00%(upstream:short)%00%(objectname)%00%(upstream:track)%00%(upstream:remotename)%00%(upstream:remoteref) --ignore-case refs/heads/main refs/remotes/main [35ms]
2024-07-16 22:45:51.387 [info] > git show --textconv HEAD:src/App.tsx [35ms]
2024-07-16 22:45:51.387 [info] > git show --textconv HEAD:src/Components/Foo.tsx [39ms]
2024-07-16 22:45:51.392 [info] > git show --textconv HEAD:src/Components/Foo.tsx [37ms]
2024-07-16 22:45:51.395 [info] > git status -z -uall [36ms]
2024-07-16 22:45:51.427 [info] > git for-each-ref --format=%(refname)%00%(upstream:short)%00%(objectname)%00%(upstream:track)%00%(upstream:remotename)%00%(upstream:remoteref) --ignore-case refs/heads/main refs/remotes/main [29ms]
2024-07-16 22:45:51.453 [info] > git config --local branch.main.vscode-merge-base [24ms]
2024-07-16 22:45:51.453 [warning] git config failed: Failed to execute git
2024-07-16 22:45:51.485 [info] > git reflog main --grep-reflog=branch: Created from *. [29ms]
2024-07-16 22:45:51.511 [info] > git symbolic-ref --short refs/remotes/origin/HEAD [24ms]
2024-07-16 22:45:51.511 [info] fatal: ref refs/remotes/origin/HEAD is not a symbolic ref

I've performed these steps above on a test git repo for you. Please see: https://github.com/hugs7/vscode-stage-selected
I've also tried to provide the commit hash next to the relevant steps listed above to match my repo. Interesting how the git log in VSCode does have --ignore-case in a few of the commands - maybe something to go off? Regardless given the steps above, it seems this issue could affect Linux/macOS too given we are renaming, then renaming back.
If you would like any further clarification please let me know - happy to help however I can.

 See shell output below.
D:\Hugo\Documents\vscode-stage-selected>git st
On branch main
Your branch is up to date with 'origin/main'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        modified:   src/Components/Foo.tsx

no changes added to commit (use "git add" and/or "git commit -a")

D:\Hugo\Documents\vscode-stage-selected>git st
On branch main
Your branch is up to date with 'origin/main'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        modified:   src/Components/Foo.tsx

no changes added to commit (use "git add" and/or "git commit -a")

D:\Hugo\Documents\vscode-stage-selected>

This is a huge issue for me and resulted in unintended deletion of many hours of work. Needs fixing ASAP.

This is a huge issue for me and resulted in unintended deletion of many hours of work. Needs fixing ASAP.

Could you please elaborate on how exactly this behaviour has results in data loss? Thanks!

Please can this be reopened @lszomoru . I provided a repo and detailed steps on how this issue can be replicated.


Avoid gc of share ktab

#448 introduced a new feature to consolidate ktab amond all methods in a class. This PR avoids GC of the shared ktab, which is normally not an issue since this feature is primarily aiming at solidificating code.

@skiars could you please approve this PR? I can't merge otherwise


Write your own row template

Feature to use custom template for each row.

Use template-url.

The template example shows how to make your full directive template, is there a way to only make a template for the rows?

@jonathanwiesel Unfortunately there is no way, you need to provide the whole template.


Add UAA SP key generation to bosh-lite manifest generator

The bosh-lite manifest generation script now generates a key and self-signed cert for now-required manifest properties in the UAA job.
Signed-off-by: Helen Chung<EMAIL_ADDRESS>

@plfx, it looks like this adds an undocumented dependency on jq that up until now, the scripts haven't needed. Can you either do it without jq, or document the need to install jq to use the bosh-lite generation scripts? In particular, the section here: https://github.com/cloudfoundry/bosh-lite#deploy-cloud-foundry. The docs that section mentions would also need to be updated.

Since this is for people's CI environments, among other things, we'll
change it to not use the new utility.
On Wed, Nov 23, 2016 at 1:03 PM, Jesse Thomas Alford <
<EMAIL_ADDRESS>wrote:

@plfx https://github.com/plfx, it looks like this adds an undocumented
dependency on jq that up until now, the scripts haven't needed. Can you
either do it without jq, or document the need to install jq to use the
bosh-lite generation scripts? In particular, the section here:
https://github.com/cloudfoundry/bosh-lite#deploy-cloud-foundry. The docs
that section mentions would also need to be updated.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cloudfoundry/cf-release/pull/1108#issuecomment-262629358,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHS49irVHbPRIeek0blk11j_cKKdy9eOks5rBKopgaJpZM4K6-zE
.

--
Jeremy Coffield
Cloud Foundry Pivot

@JesseTAlford We looked into a concourse pipeline to see if the container has jq on a common stemcell, and it does. We'll document the jq dependency.

We've made a PR to bosh-lite to update the readme. We can make the docs team aware of the change as well. https://github.com/cloudfoundry/bosh-lite/pull/409

Does this generate a new keypair every time it generates the manifest? Sounds like a nasty side-effect, it's bad enough that this script requires BOSH-Lite to be targeted and then alters global BOSH CLI state.

@ematpl It does, however this is only in the case that you have not specified a key in your stubs.

I don't understand the purpose of this pr. First of all, why add a dependency on jq, a simple bash process substitution can do the trick. I'm also not sure why we cannot just add a stub that provides default values for the certs and can easily be overridden by user by providing their own stub to the generate-bosh-lite-dev-manifest

@jvshahid We have been advised by the Security team not to have any kind of defaults for Key and Password material. This is the reason for generating then per environment.
Are there any issues with adding this dependency if we properly document this.

@sreetummidi if that's the case, we should have a far more consistent policy across CF manifest generation about how we direct operators to generate these credentials, especially for dev environments such as BOSH-Lite where most of the time it does not matter. Hiding random key generation in the manifest-generation script is the wrong place for this. cc @dsabeti

I'd prefer a static default cred. Bosh lite is terribly, terribly insecure by default allready. cf-deployment based bosh-lites will be getting generated creds like everyone else; the current bosh lite manifest generation uses defaults for other secrets, so why not this?
@emalm, Sabeti's out this week, I'm all you've got.

@JesseTAlford @ematpl @plfx @luan The lack of this default property is currently blocking a build in CAPI's pipeline.  They may be able to provide more context, but it would be good to make a decision about the requirements/expectations for this script.  Who is driving the security discussion here?  Do we need to loop them in?

Its being take care of. We will be providing the defaults in the bosh-lite manifest without the need for the script

I'm closing this PR in favor of a new one without any generation.

I'mm'a delete the branch on this, then. Thanks for talking it through to an alternative solution, everyone!


Add David Picture

Add prokarius picture
Add *.un~ and *.swp to .gitignore.

Pull Request Test Coverage Report for Build 53

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.811%

Totals

Change from base Build 50:
0.0%

Covered Lines:
1433

Relevant Lines:
1544

💛  - Coveralls

Merge conflict with Irham's PR.


[issue] - type of 'browser' is any

Hi, there's a problem with browser variable, its type should not be any.
Steps to Reproduce

Clone the project & install dependencies.
Run pnpm dev.
Content of src/auto-imports.d.ts:

declare global {
  const browser: typeof import('webextension-polyfill')['default']
  // ...
}

The type of every browser variable is any.

I just ran into the same problem. The webextension-polyfill package exports a Browser type, so you can do this:
-    const browser: typeof import('webextension-polyfill')['default']
+   const browser: import("webextension-polyfill").Browser;

I'm not sending a PR for this since I'm not sure if this is the correct fix -- unsure why this has broken in the first place, or whether it's never worked correctly in the first place.


chore: clear Keychain accounts on reset + iOS auth prompt fix

Description
Builds on top of https://github.com/valora-inc/wallet/pull/2539

Clears Keychain accounts when resetting Valora via its settings menu
Fix iOS Face ID / Touch ID prompt on every app start (was a side effect of listing keychain items protected by the system auth)
Update logic in the account checker

Tested

Updated tests
Manually on iOS and Android

How others should test

Resetting the app via its settings menu should work
No unexpected iOS auth prompt shown on app start

Related issues

Part of https://github.com/valora-inc/wallet/issues/2392

Backwards compatibility
Yes

Hey @MuckT, We have verified above task on Latest Android Internal Release Build V1.34.0 & updated status in test rails.
Test rail links:
Android 10: https://valoraapp.testrail.io/index.php?/tests/view/41867
Device: Samsung Galaxy M31 (10.0)
CC: @jeanregisser
Thanks..!!

Hey @MuckT, We have verified above task on Latest iOS Test Flight Release Build V1.34.0 & updated status in test rails.
Test rail links:
iOS 15: https://valoraapp.testrail.io/index.php?/tests/view/41929
Device:  iPhone 13 mini(15.1.1)
CC: @jeanregisser
Thanks..!!


Documentation in Health Check about different scenarios of restarts

Hi,
Regarding to the description of Health Check documentation would it be nice to have documented the following:
-What happens if several App Services are running under the same App Service Plan? Do they all get assigned to a new instance?
-What happens if instead of Several App services, we have several slots? Do they all get assigned to a new instance?
Regards,
Ignacio

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

ID: 20c42a13-f6cb-1e60-0438-f7f32b86a336
Version Independent ID: 79e7dd15-24aa-2e6d-cee7-7714e1940c11
Content: Monitor the health of App Service instances - Azure App Service
Content Source: articles/app-service/monitor-instances-health-check.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

@IgnacioAlvarezArenas Thanks for the feedback! We will review and update accordingly.

@IgnacioAlvarezArenas Thanks again for the feedback! As this article is for Health check in the Azure portal to monitor App Service instances. Your ask is documented in this article, How does my app run and scale?
" When you create an app in App Service, it is put into an App Service plan. When the app runs, it runs on all the VM instances configured in the App Service plan. If multiple apps are in the same App Service plan, they all share the same VM instances. If you have multiple deployment slots for an app, all deployment slots also run on the same VM instances. If you enable diagnostic logs, perform backups, or run WebJobs, they also use CPU cycles and memory on these VM instances."
Let us know if you have further query on this.

Hi Sneha, it is not exactly what I was looking for, apologies as I am rereading my comment and it is unclear. Yeah all the App Services and slots run on the same VM instances on the App Service Plan. But my questions are a bit more specific.
Let's say Health Check fails for one of the apps, but it is still working for the others on the same App Servuce Plan. Will still generate a new instance, therefore restarting all the App Services that were running on the "unhealthy" instance?
That is the sort of behaviour I think it could be a good idea to have there.

@IgnacioAlvarezArenas Thanks again for the feedback! I have assigned the issue to the content author to review further and update the document as appropriate.

Is there any update on this documentation? I understand if it is rejected but it can be a great opportunity to bring clarity into this.

Hi @IgnacioAlvarezArenas thank you for your feedback. Let me review this and see how we can update the doc. Thank you.

ChrisClarkRs
Consolidating Health check feedback from https://github.com/MicrosoftDocs/azure-docs/issues/71978
The documentation for the "Authentication and Security" section for health check should include that when a new Web App is created the default listener will be created as secure "HTTPS" protocol. However when you setup the Web App health probe for your various instances you cannot set which protocol to use or the ability to update the probe to use a custom domain. Therefore the health probe will automatically fail unless you select "HTTPS-Only".

This is currently being worked up. Look for an update w/in the next couple weeks. Thank you.
#please-close


koordlet: remove nodeMetricLister in states_nodemetric plugin

Signed-off-by: Jason Liu<EMAIL_ADDRESS>Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/close


Dynamic Prompting extension?

Love the script! Question - is it possible to adapt Dynamic Prompting into an extension?

Glad you're enjoying it. It's on the to do list. Thanks for opening the issue.

Released as an extension - https://github.com/adieyal/sd-dynamic-prompting/tree/v0.11.0 - let me know if you encounter any issues.


Roll Flutter Engine from bb1ff84b6c4f to fa1b2dd4007d (3 revisions)

https://github.com/flutter/engine/compare/bb1ff84b6c4f...fa1b2dd4007d
2023-02-15<EMAIL_ADDRESS>Roll Skia from 21627ff455d0 to bb3d8185f067 (13 revisions) (flutter/engine#39661)
2023-02-15<EMAIL_ADDRESS>Roll Dart SDK from 5d17a336bdfe to a594e34e85b6 (1 revision) (flutter/engine#39656)
2023-02-15<EMAIL_ADDRESS>Revert "Revert "Add support for double tap action from Apple Pencil 2 (#39267)" (#39607)" (flutter/engine#39637)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


How to implement multimodal multi task learning in MMDetection Mask RCNN Model?

I have a dataset with 4 channel input imagery, 2 labels set - one label set is in MS COCO JSON Format and another labels format is in TIF format.
The dataset is designed for instance segmentation multi-modal multi-task.
How do I give input of 2 labels for maskrcnn model?
How do I tell the model to do multi-tasking?
Can someone please provide guidance on this task?

@Keiku @anurag1paul @hachreak


DISCUSS:Feature:Passive client

Hi,
First of all I find your project amazing!
Now I would like to discuss the possibility of having a true passive, firewall bypasser client that means:
-Disable keepalive request (I've already done that, but not in elegant way).
-Connect to rabbitmq via HTTP protocol
-Connect only when a check is performed
This isn't for spying purposes but, to request various data among servers which are in client DMZ/LAN. And many of them are bad when it comes to configure firewalls.
I think this can be a great feature, and unique one among monitoring solutions (despite of nagios agent which has bad licensing, and abilities)!

We've implemented one of these things in the latest release (version 0.24) via a new keepalives boolean configuration attribute.
See here for more info:

Release: https://sensuapp.org/blog/2016/06/07/sensu-core-v0-24-0.html
Docs: https://sensuapp.org/docs/0.24/reference/clients.html#client-attributes

The other two may be out of scope for Sensu, but I'll leave this open for discussion if anyone else has opinions about them. :)

Closing due to inactivity, and because the portions of this issue which aren't already implemented are out of scope for Sensu at this time. Please feel free to create a new issue if this matter is not resolved. #monitoringlove


Zig Build System

This PR updates diffz to use a zig.build.zon manifest, adds a zig build test step, adds .zig-cache to the .gitignore, and such miscellany.

superseded by #23


Color search returns artworks with no image

https://www.webumenia.sk/katalog?color=849573

fixed ✅


When sending put command with an inexistent id to /reading of Core Data API, 404 should be returned

OS version: ubuntu 16.04
EdgeX version: 1.0.0
Environment details: Docker container
Steps to reproduce:

start up EdgeX
send a put command with an inexistent id to /reading of Core Data API
for example, with the following request URL and body:

Method: PUT
URL: http://localhost:48080/api/v1/reading
Body:
{
    "id": "57e866d4e4b0ca8e6d73a412",
    "created":<PHONE_NUMBER>170,
    "modified":<PHONE_NUMBER>170,
    "origin":<PHONE_NUMBER>619,
    "pushed": 0,
    "name": "co2",
    "value": "12"
 }

Actual results: Got returned http status code 409 Conflict
Expected results: Got returned http status code 404 NotFound
RAML: https://github.com/edgexfoundry/edgex-go/blob/master/api/raml/core-data.raml#L331

Tested. Works as expected. Closing.


PRIMARY KEY AUTOINCREMENT not working

Hi, in the code below PRIMARY KEY AUTOINCREMENT does not work for me. There are only undefined values in the id column. Did I miss something?
Thanks for help!
alasql('CREATE localStorage DATABASE IF NOT EXISTS Investments');
alasql('ATTACH localStorage DATABASE Investments AS Investments');
alasql('SET AUTOCOMMIT ON');
alasql('CREATE TABLE IF NOT EXISTS Investments.Products (id INTEGER PRIMARY KEY AUTOINCREMENT, name STRING, isin STRING)');
alasql('SELECT * INTO Investments.Products FROM ?', [[product]]);

alasql('SELECT * INTO Investments.Products FROM ?', [[{ isin:'LU0147784465', name:'AXA High Yield Bond' }]]);
alasql('SELECT * INTO Investments.Products FROM ?', [[{ isin:'IE0004851808', name:'Baring HY Bond' }]]);

vm.products = alasql('SELECT * FROM Investments.Products');
console.log(vm.products);

Thanks for the issue.
You did not miss anything - Its a bug - also mentioned in #462
I will reference this issue in the #462 issue and close this one so we dont get duplicated issues.


Move voters.json to server directory

This way, we can view the json in-browser

@abrad1212 If you're still around, could you take a look at this and cast a vote?

I think voters.json currently exists as cron/voters.json, not in the root.

@PlasmaPower
python3 -c 'help(open)'

file is either a text or byte string giving the name (and the path
if the file isn't in the current working directory)

What makes you think the current working directory is cron?

Oops, I was looking at the execl code right below it which was using THIS_DIR. Perhaps this should too? I don't like relying on the CD very much.

Yeah, that'd probably be a good idea. I'd like to redo everything using pathlib at some point. I could also make everything repo root/__file__ relative at that point.


Prepare release 6.2.0

Updated the browser.js file for npm case to use test-fixture as JS module instead of html import.
Updated the integration tests to support running on Sauce via wct-sauce plugin.
No longer require ?npm=true url parameter, as web-component-tester and wct-browser-legacy versions of browser.js now contain explicit npm/non-npm directives of the form window.__wctUseNpm as a first step towards splitting out the client-side environment-specific config and behavior.
Added support for scoped package names under npm.
Updated polyserve to 0.22.0 for better compilation support and ES modules in HTML script tags.
[x] CHANGELOG.md has been updated

Will the Bower dependency to test-fixture also be updated to 3.0.0?

@hastebrot +1 also yes.

CI is demonstrating and helping verify a compile cache failure in polyserve 0.22.0 with specific ordering of remote browsers under test.  Pausing release to address https://github.com/Polymer/polyserve/issues/216

Fixed polyserve compile cache issue.  Rerunning CI.


Add tests to cover a bug where logs are forwarded in HTTP response headers

PHP-FPM sends logs in the FastCGI output under the key PHP message:. Those lines end up in the HTTP headers. They are parsed by Hoa/FastCGI as response headers.
In the end those logs are considered as part as the HTTP response headers but they are not.
PHP-FPM should not write those in the FastCGI connection. I cannot understand why this is happening and how to fix this.
Example of a raw HTTP response sent by PHP-FPM over FastCGI:
PHP Message: This is a test log from error_logX-Powered-By: PHP/7.3.1
Content-Type: text/html; charset=UTF-8
Content-Length: 12
Server: Werkzeug/0.14.1 Python/3.7.2
Date: Wed, 30 Jan 2019 21:29:43 GMT

Hello world!

As you can see in the first line the first HTTP header (X-Powered-By) is mangled with the log line.
See the tests for a scenario to reproduce it.
I tried several options (not daemonizing FPM, disabling logging in stderr, etc.) but no success.

https://github.com/hoaproject/Fastcgi/pull/19 will help solve this properly.


Evaluating Information Manifold Metrics

[ ]  Experiments 0.0.x Baselines
[ ]  Experiments 1.0.x
[ ]  Experiments 1.x.x
[ ]  Experiments 2.x.x
[ ]  Experiments 3.x.x

The statistical module was implemented. Some baselines were generated. However, we are waiting for all experiments to finish.

The statistical module was implemented. Some baselines were generated. However, we are waiting for all experiments to finish.

Getting started with  statistical analysis for those experiments that already finished ( still running sacp 32 and 128k)

[ ]  Evaluation 0.0.x Baselines
[ ]  Evaluation 1.0.x
[ ]  Evaluation 1.x.x
[ ]  Evaluation 2.x.x
[ ]  Evaluation 3.x.x

@danaderp Closing this issue an opening a new one for BPE and another one for statistical analysis


Enable image embeddings for vision models

I would love to be able to create embeddings for images with vision models like llama3.2-vision.
Creating image and text embeddings with a vision-capable model should allow creating image search and image categorization applications.
If my understanding of the shared semantic vector space of image models is correct, it should be possible to perform calculations like cosine similarity on text and image embeddings to, for example, find all the photos of puppy dogs in a random assortment of photos :)
At this time, the generate endpoint accepts an images parameter, but the embed endpoint does not. I tried passing an image as a base64 string to the input parameter of the embed endpoint, but the resulting embedding appears to be the vector of the text string and not of the image.
Would it be possible to expand the embed endpoint to accept an image parameter?

https://github.com/ollama/ollama/issues/5304


Refactor generate_neurons_blocks

The function generate_neurons_blocks was created todo all the heavy lifting for the multiprocessing block learner. However, this function is massive. It needs to be refactored into smaller functions. For instance, the subprocess pool should be refactored into a separate class and module with possible factory functions. Also generate_neurons_blocks  has several clear phases, which can be broken up into smaller functions.

This function is quite long. It includes components that could be refactored into several different functions. For instance, generating blocks shapes/offsets with a particular overlap. Submitting and monitoring job submission for different systems DRMAA, multiprocessing, maybe some ABC for them. Perhaps, merging of blocks should be a separate function. All of these will be more useful and general. Also, they will be easier to test.


Naming Cells in Hierarchical Components

Describe the bug
In gdsfactory v7, there is an alias option when component.add_ref(cell, alias="<name>") for improving readability, while it is deprecated in v8. As suggested by warning and migration guide, using name in a hierarchical cells may trigger the kdb existing cellname problem.
To Reproduce
I built a framework in v7 for fast schematic-layout transform and simulation:
# for simplicity, only show the relevant codes
class Circuit:
  def __init__(self):
    self.final_gds = gf.Component(self.__class__.__name__)
    self.instances = {}
    self.placement = {}
    self.exposed_ports = {}
    self.update()

For a customized MZI basic cell:
class MZI(Circuit):
  def update(self):
    self.instances = {
      'wg1': gf.components.straight(...),
      'wg2': gf.components.straight(...),
      'mmi1': gf.components.mmi(...),
      'mmi2': gf.components.mmi(...),
    }

    # here work well in v7 but trigger error in v8
    for k, v in self.instances.items():
      ref = self.final_gds.add_ref(v, alias=k)
      self.placement[k] = ref

For hierarchy circuits:
class MyChip(Circuit):
   def update(self):
     self.instances = {
       'mzi1': MZI(),
       'mzi2': MZI(),
     }

It works well in v7 but in v8, the error seems from kfactory:
File [/depends/miniconda3/envs/gds/lib/python3.11/site-packages/kfactory/kcell.py:787), in KCell.__init__(self, name, kcl, kdb_cell, ports)
    785 else:
    786     _name = name
--> 787 self._kdb_cell = kdb_cell or kcl.create_cell(_name)
    788 if _name == "Unnamed_!":
    789     self._kdb_cell.name = f"Unnamed_{self.cell_index()}"
...
   3658         f"Cellname {name} already exists. Please make sure the cellname is"
   3659         " unique or pass `allow_duplicate` when creating the library"
   3660     )

ValueError: Cellname MZI already exists. Please make sure the cellname is unique or pass `allow_duplicate` when creating the library

Expected behavior
Is there any method to manually reset the kdb or allow alias in kfactory? BTW, I often do optimization using my framework through Jupyter Notebook, and I found that any code rerun without resetting the python kernel will also trigger the cellname already exists problem. Any suggestions?
Suggested fix
It seems to require modification in kfactory.

This cannot be achieved with KLayout. The problem is that you create two cells with the same name.
You need to do some caching or similar to resolve that.
In general I wouldn't recommend building cells in classes. You needs to create unique names per Component() call or return an existing one. Caching would achieve the latter one.
You can try to use the @gf.cell decorator but I am not sure it will work in your case as I think the cache would be instance specific

hi @LIU-Yinyi, if you must build your cell from a class, i recommend looking at the discussion here: https://github.com/gdsfactory/gdsfactory/issues/2294
the key is to always build your cells within a cell decorator. building your cell without wrapping it in @gf.cell is not very safe...

@LIU-Yinyi the @cell decorator should already take care of that for you. Indeed dicts and lists are a bit a nuisance, but it casts lists to tuples (and I think tuples of tuples in case of list of lists, but I am not entirely sure of that) and dicts use frozensets (and back in the function) https://docs.python.org/3/library/functions.html#func-frozenset
therefore, I would suggest to directly try the cell decorator and if it fails, open a new issue with a case where it fails. Note though, if you pass too complex objects, e.g. something with the name attribute it should be unique (i.e. singleton like behavior wrt its name)

You can also treat the new KLayout-backend like a "global" cache if you are sure names are enough to identify a component:

try:
    c = gf.kcl[my_component_name]
except ValueError:
   c = gf.Component(my_component_name)
   # Construct component here ...

This is more or less a global cache as long as you know your componnent's name


Implement basic calculator functionality

Set up Flask routes to handle different calculator operations.
Create Python functions for addition, subtraction, multiplication, and division.
Connect the HTML buttons to trigger the appropriate Python functions.

I can try this