Update Readme to show and describe the new interface

Robert, great work on updating the ReadMe; it just misses the description for these buttons:

Add files from a list
Export list
Show only files with errors
Delete all
Update

Thanks for that.  I'll get those added probably today.
I'm having a lot of trouble when I edit Form1.Designer.    It's driving me crazy.  Do you have any ideas on what I might try to fix it?

We could get rid of the "fake items" on the listview, I have added them just for preview.
And then create the 6 needed groups during the form load
It is important to create them in sequence like they are now
Files sources
Excluded files
Assemblies
Parts
Sheetmetals
Drafts

OK.  It's worth a try.  I'm almost afraid to use the Form Designer at the moment.  Do you want me to try it on this end first?

Just remove everything and create groups at runtime

OK.  I'll try it this afternoon.
On Mon, Dec 19, 2022 at 11:21 AM Francesco @.***> wrote:

Just remove everything and create groups at runtime
—
Reply to this email directly, view it on GitHub
https://github.com/rmcanany/SolidEdgeHousekeeper/issues/26#issuecomment-1357993376,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APKPB4A2DBS6K7NHCAOWZILWOCKRLANCNFSM6AAAAAATB772OI
.
You are receiving this because you were assigned.Message ID:
@.***>

Before I start breaking things, let me summarize what I think
I'm going to do:
In Form1.vb (Design), ListViewFiles object:
Remove all entries in the Groups and Items collections.
In Form1_Load():
Add this stuff:
    Dim ListViewGroup1 As System.Windows.Forms.ListViewGroup = New System.Windows.Forms.ListViewGroup("Files sources", System.Windows.Forms.HorizontalAlignment.Left)
    Dim ListViewGroup2 As System.Windows.Forms.ListViewGroup = New System.Windows.Forms.ListViewGroup("Excluded files", System.Windows.Forms.HorizontalAlignment.Left)
    Dim ListViewGroup3 As System.Windows.Forms.ListViewGroup = New System.Windows.Forms.ListViewGroup("Assemblies", System.Windows.Forms.HorizontalAlignment.Left)
    Dim ListViewGroup4 As System.Windows.Forms.ListViewGroup = New System.Windows.Forms.ListViewGroup("Parts", System.Windows.Forms.HorizontalAlignment.Left)
    Dim ListViewGroup5 As System.Windows.Forms.ListViewGroup = New System.Windows.Forms.ListViewGroup("Sheetmetals", System.Windows.Forms.HorizontalAlignment.Left)
    Dim ListViewGroup6 As System.Windows.Forms.ListViewGroup = New System.Windows.Forms.ListViewGroup("Drafts", System.Windows.Forms.HorizontalAlignment.Left)

Don't add:
    Dim ListViewItem1 As System.Windows.Forms.ListViewItem = New System.Windows.Forms.ListViewItem(New String() {"FakeFile number 1", "C:\Documents\Projects\FakeProject"}, "Unchecked")
    Dim ListViewItem2 As System.Windows.Forms.ListViewItem = New System.Windows.Forms.ListViewItem(New String() {"FakeFile number 2", "C:\Documents\Projects\FakeProject"}, "Unchecked")
    Dim ListViewItem3 As System.Windows.Forms.ListViewItem = New System.Windows.Forms.ListViewItem(New String() {"FakeFile number 3", "C:\Documents\Projects\FakeProject"}, "Unchecked")
    Dim ListViewItem4 As System.Windows.Forms.ListViewItem = New System.Windows.Forms.ListViewItem(New String() {"FakeFile number 4", "C:\Documents\Projects\FakeProject"}, "Unchecked")
    Dim ListViewItem5 As System.Windows.Forms.ListViewItem = New System.Windows.Forms.ListViewItem(New String() {"FakeFile number 5", "C:\Documents\Projects\FakeProject"}, "Checked")
    Dim ListViewItem6 As System.Windows.Forms.ListViewItem = New System.Windows.Forms.ListViewItem(New String() {"FakeFile number 6", "C:\Documents\Projects\FakeProject"}, "Error")
    Dim ListViewItem7 As System.Windows.Forms.ListViewItem = New System.Windows.Forms.ListViewItem(New String() {"FakeFile number 7", "C:\Documents\Projects\FakeProject\Drawings"}, "Checked")
    Dim ListViewItem8 As System.Windows.Forms.ListViewItem = New System.Windows.Forms.ListViewItem(New String() {"FakeFile number 8", "C:\Documents\Projects\FakeProject\Drawings"}, "Checked")
    Dim ListViewItem9 As System.Windows.Forms.ListViewItem = New System.Windows.Forms.ListViewItem("C:\Documents\Projects", "folder")
    Dim ListViewItem10 As System.Windows.Forms.ListViewItem = New System.Windows.Forms.ListViewItem("C:\Documents\Projects", "folders")

Do add:
    ListViewGroup1.Header = "Files sources"
    ListViewGroup1.Name = "Sources"
    ListViewGroup2.Header = "Excluded files"
    ListViewGroup2.Name = "Excluded"
    ListViewGroup3.Header = "Assemblies"
    ListViewGroup3.Name = ".asm"
    ListViewGroup4.Header = "Parts"
    ListViewGroup4.Name = ".par"
    ListViewGroup5.Header = "Sheetmetals"
    ListViewGroup5.Name = ".psm"
    ListViewGroup6.Header = "Drafts"
    ListViewGroup6.Name = ".dft"
    Me.ListViewFiles.Groups.AddRange(New System.Windows.Forms.ListViewGroup() {ListViewGroup1, ListViewGroup2, ListViewGroup3, ListViewGroup4, ListViewGroup5, ListViewGroup6})

Don't add:
    ListViewItem1.Group = ListViewGroup3
    ListViewItem2.Group = ListViewGroup4
    ListViewItem3.Group = ListViewGroup4
    ListViewItem4.Group = ListViewGroup4
    ListViewItem5.Group = ListViewGroup5
    ListViewItem6.Group = ListViewGroup5
    ListViewItem7.Group = ListViewGroup6
    ListViewItem8.Group = ListViewGroup6
    ListViewItem9.Group = ListViewGroup1
    ListViewItem10.Group = ListViewGroup1
    Me.ListViewFiles.Items.AddRange(New System.Windows.Forms.ListViewItem() {ListViewItem1, ListViewItem2, ListViewItem3, ListViewItem4, ListViewItem5, ListViewItem6, ListViewItem7, ListViewItem8, ListViewItem9, ListViewItem10})

I have removed the preview items that causes problems during pull request

It's working so far.  Much less scary to tweak things!
I've been updating the Readme.  When you get a chance, please check the File Selection section.  I'm sure there are things to add/remove/fix/clarify.

I saw the new Icon, and it looks good; in my opinion, it needs a few tweaks:

"handle" color should be a bit more bright, it almost disappear on my blackish windows toolbar
"handle" should be a bit longer to be visible

The Readme is getting closer.  If you have a chance, please look it over.  Especially the File Selection and Filtering part.

It looks good to me, just miss my latest addition on RMB menu item to find ASM links

Good.  I've been looking at it for so long, I can't tell up from down.
I don't think we need the RMB button for ASM links any more.  I integrated
that functionality into the Update Button code.  I didn't delete anything,
just added a MsgBox to BT_FindLinks handler.
I'm pretty sure it works the same, but if you get a chance, please try it
out.  I feel it's more consistent with your overall design direction --
keep the user's eyes on the Selection Toolbar.
On Tue, Dec 27, 2022 at 11:21 AM Francesco @.***> wrote:

It looks good to me, just miss my latest addition on RMB menu item to find
ASM links
—
Reply to this email directly, view it on GitHub
https://github.com/rmcanany/SolidEdgeHousekeeper/issues/26#issuecomment-1366059394,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APKPB4HDXNTEJYJIG2BNDJ3WPMQSLANCNFSM6AAAAAATB772OI
.
You are receiving this because you were assigned.Message ID:
@.***>

I would leave the RMB find links there, it could be usefull in some situations and its pretty self explanatory.
But hey, you are the master branch :)

OK, let's give it a try.  I'll remove my MsgBox and update the Readme.  If it causes any confusion, I can just blame you :)

Done for now.  Closing.


Set column order

Hi,
I would like to set the column order for the tabular table. Is there a way to do that?
Many thanks,
Richard

Try use the "order" option from datatable (https://datatables.net/examples/basic_init/table_sorting.html) but failed.
const FooTable = new Tabular.Table({
  name: 'FooTable',
  collection: Foo,
  order: [[0, 'asc']],
  columns: [
.
.
.
.

The columns are in the order you define them in the array. Just define them in a different order.

try
columns: [],
order: [[0, 'asc']]


GO Portal is restart service.

GO Portal is restart service.


deleting depfile: Permission denied

I'm Building roughly 500 C++ files with Ninja Build. From time to times I have this error failing my build:
e:/tools/clang/clang++.exe -Wno-nonportable-include-path -MMD -MF [...] MyCPPFile.o
deleting depfile: Permission denied

If I restart the build right away it works (or at least fails on a different file).
It completely random and may happens on one file for a build and in a next build on another one.
Have you any idea what can induce this behavior?

Could be anti-virus. Which Windows version?

I have Windows 10 Pro with the Antivirus provided by this system.
Using the flag -d keepdepfile obviously workaround the situation.
I started a build with "Process Monitor".
The build log reported:
[31/595] Building C file e:/src/[...]/spawner.cpp
ninja: error: remove(e:/bin/logs.o.d): Permission denied
[32/595] Building C file e:/src/logs.cpp
FAILED: e:/bin/logs.o
e:/tools/clang/clang++.exe [...] -o e:/bin/logs.o
deleting depfile: Permission denied

In Process Monitor:
09:31:06,1016146	ninja.exe	18308	CreateFile	E:\bin\logs.o.d	SUCCESS	Desired Access: Generic Read, Disposition: Open, Options: Sequential Access, Synchronous IO Non-Alert, Non-Directory File, Attributes: n/a, ShareMode: Read, AllocationSize: n/a, OpenResult: Opened
09:31:06,1016597	ninja.exe	18308	ReadFile	E:\bin\logs.o.d	SUCCESS	Offset: 0, Length: 1 561, Priority: Normal
09:31:06,1016900	ninja.exe	18308	ReadFile	E:\bin\logs.o.d	END OF FILE	Offset: 1 561, Length: 65 536
09:31:06,1017064	ninja.exe	18308	CloseFile	E:\bin\logs.o.d	SUCCESS
09:31:06,1018156	ninja.exe	18308	CreateFile	E:\bin\logs.o.d	SHARING VIOLATION	Desired Access: Read Attributes, Delete, Disposition: Open, Options: Non-Directory File, Open Reparse Point, Attributes: n/a, ShareMode: Read, Write, Delete, AllocationSize: n/a

If I'm not wrong. The file is only opened once in Ninja, Read, closed and then deleted. So the log would happened outside.
I'll try deactivating my antivirus.

Deactivating the Antivirus did not solved the situation.

I don't know if this is relevant for you, but if you have dropbox turned on in the folder you're building in, then it does this if you don't pause dropbox before running a build, because dropbox sees the new file, starts syncing it with the file locked, then boom, can't delete the file.

Is this still effecting you with the latest release of ninja?

The same happens when using MEGA to synchronize files. Moving the folder to unsynced space helps to solve the problem with build🙌


refactor and fix TM init set

this PR makes a (much needed) reorganisation of the implementations of AbstractFlowpipe and AbstractReachSet into separate files. moreover, the initialization of TMJets with an initial TaylorModelReachSet is completed.

Pull Request Test Coverage Report for Build 652149604

128 of 637 (20.09%)  changed or added relevant lines in 16 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-3.4%) to 13.237%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Algorithms/TMJets/reach.jl
12
13
92.31%

src/Algorithms/TMJets/post.jl
2
5
40.0%

src/ReachSets/ShiftedReachSet.jl
0
4
0.0%

src/Flowpipes/MappedFlowpipe.jl
0
8
0.0%

src/ReachSets/ReachSet.jl
9
18
50.0%

src/Flowpipes/setops.jl
0
11
0.0%

src/ReachSets/SparseReachSet.jl
1
15
6.67%

src/Flowpipes/AbstractFlowpipe.jl
13
30
43.33%

src/ReachSets/AbstractReachSet.jl
17
42
40.48%

src/Flowpipes/MixedFlowpipe.jl
0
28
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/Flowpipes/setops.jl
1
10.72%

src/Algorithms/TMJets/reach.jl
2
27.54%

Totals

Change from base Build 650426911:
-3.4%

Covered Lines:
577

Relevant Lines:
4359

💛  - Coveralls


Add Sampler objects (prototype)

Sampler objects draw samples from a distribution via a generator returned by their "sample" method
_OnlineStatistics objects enable to track statistics of a Sampler object (moments, expected values, ...) in an online fashion (without storing the history of the samples) by feeding the samples via the "update" method

Nice initial prototype.
See my comments in branch _thomas_sampler_comments.


Stl 2233 banner for incidents

Short Description

Add new routes for the configuration
Add fake redis to test the configuration

Feedback

Any?

[ ] Add github secret

How to review this Pull Request
Link to Confluence (internal)

@SannyNguyenHung  you want to review the sonarcloud analysis before I do the review?


[FEATURE]: Navigation properties + projection for relational mode.

Describe what you want
One feature I'm missing from .NET's EF Core is having a possibility to specify navigation properties in projection. This would automatically create join between the two entities
export const users = pgTable('users', {
	id: serial('id').primaryKey(),
	name: text('name').notNull(),
	userAddress: //navigation property
});

export const userAddress = pgTable('userAddress', {
	id: serial('id').primaryKey(),
	userId: integer('userId').references(() => users.id),
	address: text('address'),
});

//Query
db.select({
	id: users.id,
	address: users.userAddress.address
}).from(users)

This is currently possible with the relational query builder. https://orm.drizzle.team/docs/rqb
Feel free to reopen the issue if the feature you're requesting is different.


Make install fails on Sierra for PHP 5.6.40 (and below)

Hello and Thanks for carrying on PHP 5.6 on brew!
I do have an issue installing these deprecated packages on Sierra. Compilation seems to run fine and make finishes without errors.
However, make install stops after an error. Here is a log:
Last 15 lines from<EMAIL_ADDRESS>make: [install-sapi] Error 1 (ignored) make<EMAIL_ADDRESS>Permission denied make: *** [install-cgi] Error 1 make: *** [install-cli] Error 1 mkdir<EMAIL_ADDRESS>File exists Installing shared extensions<EMAIL_ADDRESS>/usr/local/opt/httpd/lib/httpd/build/instdso.sh SH_LIBTOOL='/usr/local/opt/apr/libexec/build-1/libtool' libs/libphp5.so<EMAIL_ADDRESS>/usr/local/opt/apr/libexec/build-1/libtool --mode=install install libs/libphp5.so<EMAIL_ADDRESS>libtool: install: install libs/libphp5.so<EMAIL_ADDRESS>Warning!  dlname not found in<EMAIL_ADDRESS>Assuming installing a .so rather than a libtool archive. chmod 755<EMAIL_ADDRESS>Installing PHP FPM config:        /usr/local/etc/php/5.6/ Installing PHP FPM man page<EMAIL_ADDRESS>Installing PHP FPM status page<EMAIL_ADDRESS>Is anyone else having this problem? Was anybody able to solve it?
Cheers,
Pierre

Hi!
We do not maintain bottles for Sierra and we do not test for this platform. We only do so for Mojave.
Let me see if I can pop a new VM to test the install on Sierra. I'll let you know what I find.
Regards.

Anything? I have that problem too, but i think my problem is in ./configure
`==> Installing<EMAIL_ADDRESS>from exolnet/deprecated
==> Downloading https://php.net/get/php-5.6.40.tar.xz/from/this/mirror
Already downloaded: /Users/Nicox/Library/Caches/Homebrew/downloads/ab2ea146f7d8c5258e6d33e9522dac5528cbc3e966bc6e5689fa57860dbb5441--php-5.6.40.tar.xz
==> Patching
patching file acinclude.m4
Hunk #1 succeeded at 444 (offset 3 lines).
Hunk #2 succeeded at 459 (offset 3 lines).
Hunk #3 succeeded at 494 (offset 3 lines).
Hunk #4 succeeded at 506 (offset 3 lines).
Hunk #5 succeeded at 2500 with fuzz 2 (offset 81 lines).
patching file Zend/zend_compile.h
patching file Zend/zend_execute.h
==> ./buildconf --force
==> ./configure<EMAIL_ADDRESS>--localstatedir=/usr/local/var --sysconfdir=/usr/local/etc/php/5.6 --with-config-fi
Last 15 lines from<EMAIL_ADDRESS>checking for zlib version >= <IP_ADDRESS>... 1.2.8
checking for gzgets in -lz... yes
checking whether to enable bc style precision math functions... yes
checking for BZip2 support... yes
checking for BZ2_bzerror in -lbz2... yes
checking whether to enable calendar conversion support... yes
checking whether to enable ctype functions... yes
checking for cURL support... yes
checking for cURL 7.10.5 or greater... libcurl 7.64.1
checking for SSL support in libcurl... yes
checking how to run the C preprocessor... clang -E
checking for openssl support in libcurl... no
checking for gnutls support in libcurl... no
checking for curl_easy_perform in -lcurl... no
configure: error: There is something wrong. Please check config.log for more information.
If reporting this issue please do so at (not Homebrew/brew or Homebrew/core):
https://github.com/exolnet/homebrew-deprecated/issues`
in config.log i have a lot of errors

Hi!
We do not maintain bottles for Sierra and we do not test for this platform. We only do so for Mojave.
Let me see if I can pop a new VM to test the install on Sierra. I'll let you know what I find.
Regards.

Thanks for looking into this!

Hi!
We won't be able to provide bottles for Sierra and High Sierra.
Sorry about this.
Regards.


Car Doors

To load/unload cars and cargo, I'd suggest to look at systems similar to how air cargo is loaded. So a sort of tray that can be prepared and rolled into the actual pod.
Cars could drive onto the tray, be secured and then the tray is rolled into the pod (sideways). Upon arrival, the tray is retrieved from the pod, the car is released from the tray and can drive off.
Imagine even if the cars are Tesla's and can be battery swapped (to reduce weight) in that process!!

Yes, that would be a quite good solution.
I don't know programming though.

Or even better self-producing tesla's that charge and restore the life of other vehicles present, suficient use of sustainable electric tesla's self sufficiency is always good when other task are needed to be carried out.:)


move from SHA1 to a decent hash algorithm

Please move to a decent hash algorithm.
( https://www.schneier.com/blog/archives/2005/02/sha1_broken.html )
yes, your hashes are salted, but SHA1 sould be exchanged with something safer, e.g.  SHA512 (like SHA512_256). The code change would be minimal, just replace sha1.New with Sha512_256.New.
To make migration easy I would suggest to prepend the hash with the hashing algo used like
{SHA1}oldhashes
{SHA512_256}newhashes
and add a migration script that adds {SHA1} to all old hashes.

there might be safer other hash algorithms that might be available with other go dependencies

@tronicum
Thanks for pointing out, but based on by investigation a couple of months ago, the collision issue of SHA1 has little impact on  the security of HMAC-SHA1?

https://mathiasbynens.be/notes/pbkdf2-hmac
http://crypto.stackexchange.com/questions/26510/why-is-hmac-sha1-still-considered-secure
it is complicated, true. I still would recomend not using is, as the amount of crypto attacks on sha1 is growing, it's like using MD5 forever because it is so easy to implement.

Based on the above link, even PBKDF2-HMAC-SHA256 might be bad.

I guess something like SHA3 would be nice https://godoc.org/golang.org/x/crypto/sha3

I added an example with sha3.ShakeSum256 @ PR #1060

My suggest is Blake2 it's very fast
https://godoc.org/golang.org/x/crypto/blake2b
See more details
https://research.kudelskisecurity.com/2017/03/06/why-replace-sha-1-with-blake2/


[Feature, Backends, DA] Semi-local addressing for Pulser backend

Support semi-local addressing patterns on the Pulser backend as well since the initial implementation only worked for PyQTorch backend.
This should be done after refactoring the Interaction building procedure as detailed in #116

@jpmoutinho @vytautas-a It feels like this one can be closed ?


RangesDownloaded & GetDownloadedRanges()

Why Microsoft.NETCore.UniversalWindowsPlatform 6.1.7 does not have RangesDownloaded & GetDownloadedRanges()?

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

ID: 20a23550-5126-e2b0-f2a0-0aa4da29bcc8
Version Independent ID: 07e6eec6-10e2-09e0-abbb-1a338a25a5bb
Content: DownloadOperation Class (Windows.Networking.BackgroundTransfer) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.networking.backgroundtransfer.downloadoperation.yml&version=GBlive&_a=contents
Product: windows
Git Login: wdg-dev-content
Microsoft Alias: wdg-dev-content

Hi, @omansak, thanks for filing this issue. I've started investigating, and I'll get back to you when I have more information.
~Steve

Sorry about the delay in responding, @omansak. I don't have any progress to report, but I haven't forgotten about the issue. ~Steve

@stevewhims Thank you for investigating. I waiting

Hi, @omansak, again I apologize for the delay in getting info to you. Here's some info from the product team, which I hope will help.
Proposed solution: set the app’s Target Version to be at least the Windows 10 Fall Creators update (build 16299).
I can only reproduce the issue in one way: by setting the program’s properties>Application>Targeting Target Version to something less than the Windows 10 Fall Creators Update (Build 16299). Otherwise, it works correctly for me.
Original question: was “Why Microsoft.NETCore.UniversalWindowsPlatform 6.1.7 does not have RangesDownloaded & GetDownloadedRanges()?”
Some background: The NETCore.UniversalWindowsPlatform package is an “wrapper” so that .NET Core UWP applications can use the UWP set of APIs that are part of Windows and the Windows SDK. Different versions are made as the set of available APIs to a user’s program depends on the project target version (but not the min version).
The Ranges was added in the Fall Creators Update last year.
If you build an app with a target of at least the Fall Creators update and a lower min version, the app will compile but any attempt to use the new APIs will not work.

@stevewhims Thank you so much , this infos help to me

Based on the info in the product team's reply, it looks like you're not supposed to set the min version lower than the Fall Creators update.


Improve compare page loading and fix date/time selector

Description:
(from Lauri's report)
Time selector doesnt work on compare page.
My sensors had only 2 days of data and 1 week was selected and it showed me 2 days. Also, I expected it to update if I changed the date, same was as normally 🤔 or do we have a reason not to update (zoom if data is downloaded already)?

I think this is fixed now

QA passed


feature request: make temporary highlight show across panes, just like persistent ones.

In your screenshots in README, it looks like it highlights across panes. But this doesn't work for me, and I don't see any setting to enable it.

debug info with package info
{
  "atom": "1.27.1",
  "platform": "darwin",
  "release": "17.5.0",
  "vmpVersion": "1.32.0",
  "vmpConfig": {
    "automaticallyEscapeInsertModeOnActivePaneItemChange": true,
    "keymapYToYankToLastCharacterOfLine": true,
    "wrapLeftRightMotion": true,
    "blackholeRegisteredOperators": [],
    "startInInsertModeScopes": [],
    "charactersToAddSpaceOnSurround": [],
    "highlightSearchExcludeScopes": [],
    "flashOnOperateBlacklist": []
  },
  "activeCommunityPackages": [
    "atom-material-syntax: 1.0.8",
    "atom-vim-like-tab: 1.5.1",
    "hey-pane: 1.1.0",
    "keystroke: 0.4.0",
    "minimap: 4.29.8",
    "minimap-quick-highlight: 1.0.1",
    "one-tab: 0.10.0",
    "teletype: 0.13.2",
    "vim-mode-plus: 1.32.0",
    "vim-mode-plus-keymaps-for-surround: 0.2.1",
    "language-pug: 0.0.22",
    "platformio-ide-terminal: 2.8.1",
    "minimap-highlight-selected: 4.6.1",
    "highlight-line: 0.12.0",
    "quick-highlight: 0.13.0"
  ]
}

Oh, I see!!! I have to call the Toggle command to make it explicit, while selecting a word only does it temporarily in the current file. Got it.
Maybe the preview should also highlight across panes, that'd be very helpful temporarily.

Basically, the temporary feature would be much like the highligh-selected plugin, where you can double click a word and see it highlighted across panes immediately. This is helpful for temporary visual inspection.


Add syntax highlighting #2

Now submit the updated version to zed-extensions 🥳

What version of Zed were you testing it on? I'm not seeing the extension show up at all on Zed Preview now - looks like they've changed how extensions work and there's more work to be done.

The zed-preview there is a button to test

Or a more hard-core test
cd ~/Library/Application\ Support/Zed/extensions/installed/make/languages/make/
rm -rf *

wget https://github.com/caius/zed-make/raw/main/languages/make/config.toml
wget https://github.com/caius/zed-make/raw/main/languages/make/highlights.scm
wget https://github.com/caius/zed-make/raw/main/languages/make/injections.scm
wget https://github.com/caius/zed-make/raw/main/languages/make/indents.scm

I think I'd gotten it confused, neither reload nor uninstall worked and removing it from the installed dir didn't make it vanish after restarting Zed Preview either.
Eventually tricked it into functioning again 😅

Raised upstream at https://github.com/zed-industries/extensions/pull/405


feat: Multiplication of convex function

We prove that the product of nonnegative monovarying convex functions is convex. We take the opportunity to golf the various proofs of the convexity of x ↦ x ^ n.

[x] depends on: #7648

bors merge


[Concurrency] Reorder constructor initializers to avoid warning

Small warning fix for TaskLocalItem
swift/ABI/Task.h:285:13: warning: field 'valueType' will be initialized after field 'next' [-Wreorder-ctor]
            valueType(valueType),
            ^
1 warning generated.

oh fixed in https://github.com/apple/swift/pull/35874


Refactor

Folder more idiomatic/cleaned up a bug or two
Reorganize the user module as part of the server module. Separated out user::session.
Rename session to imap and Session to ImapSession
Refactor listen loops in main
Rename Command to FetchCommand and remove the CommandType because only one type - Fetch - exists.
Minor cleanup here and there inside command submodules

This change is

I'm a little bit unsure on the removal of CommandType because we'll probably need it once we start adding support for other commands. Do we want to take it out just to put it back in later? Then again, I guess since we're not actually using it now there's no need to have that functionality as it just complicates the code right now. I assume that was your reasoning?
Other than that, PR looks good.

Reviewed 19 of 19 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

I decided to just go ahead and merge this. We're not using CommandType functionality right now, and if necessary we can just add it back in later.


Some thoughts on new select functionality (1.6.[01])

Hi @lekoala. I have some feedback regarding one of the new features you've added in 1.6.0 ("make this work more like a select element"). With no disrespect intended, I do not find it very user-friendly or intuitive. It seems you are trying to both achieve the traditional tag select functionality as well as a dropdown, but the result has landed somewhere in between that is not quite either of those two. This makes it (in my opinion) a bit confusing and a difficult to use.

I think the dropdown to display all options (on focus) should be opt-in (or at least possible to disable) - to revert back to the original way it worked whereby the dropdown is only shown when a user starts to type something. I am thinking of situations whereby there may be 100 or 200 different available tags. By presenting the full list on focus, the user is inclined to start scrolling rather than searching which can be counter-productive. I think the option for a "full dropdown" (display all options) could still be achieved in this case if the "drop icon" on the right was a button rather than a background image (a bit like "show all available tags").

I think the new dropdown should also act more like a traditional dropdown and extend the full width of the virtual select field to prevent jumping around. Currently you have to chase the dropdown when selecting options:

Github itself has a fairly good example of what I mean when editing topics of a project (sorry, I can't get it to work when recording to show you what I mean).

I'm not sure what your feelings are, or whether this feedback is of any use to you, but hopefully it makes sense? Thanks again for this awesome module!

@axllent you can have a look at the latest file on master, it should improve things quite a bit :-)

fullwidth by default
you can hide the drop icon if you don't like it (showDropIcon)
fullWidth menu positioning has been improved to follow more closely what bootstrap does (2px offset below input)

Hi @lekoala. You're absolutely right, I hadn't considered suggestionThreshold which definitely addresses my first point and works exactly as I had intended.
I've just tried the latest commit and I think this replicates a common dropdown much more accurately (well done), and the ability to set data-full-width="false" together with the suggestion threshold works nicely to restore the original functionality. For my use-case (I use this module in Mailpit) the "select" field is almost the full width of the page, which just looks strange to have a full width dropdown, so using data-full-width="false" data-suggestions-threshold="1" will give me what I was looking for. For projects with narrower multiselects (eg: like your demo) I think the full-width dropdown will work very nicely :+1:
I think these (unreleased) changes have now the right balance between the two features, and allows enough flexibility to adjust depending on the user-case / preference without confusion. Thanks again! Do you want me to leave this open until you release the next version containing these changes?

https://github.com/lekoala/bootstrap5-tags/releases/tag/1.6.2
released with many other tweaks :-)
thanks again for your feedback. i'm closing the issue, but let me know if you have any other feedback :-)

@axllent by the way, if you want to use this in silverstripe, i've recently made this
https://github.com/lekoala/silverstripe-form-elements

@lekoala - that's awesome, I actually noticed (over the weekend) your Silverstripe modules! I'll definitely take a look at those some time in the near future :)


fixed bug with refreshing not working

Refresh button on the project panel wasn't working, so fixed it. The problem was that ProjectListManager was changed to more object based and singleton a bit earlier, but some methods had been left static, which caused problems with the instance variables getting messed up.

Coverage remained the same at 22.749% when pulling 762a9119b143a8d67d0308407a176742fb4fcd12 on bug-fixes-1 into c980c3cca3545e5a467595c8cb8d1aaeca1cd372 on master.


Closure in job chain is not called if preceded by job batch

Laravel Version
11.11.1
PHP Version
8.3.8
Database Driver & Version
No response
Description
Closure in job Bus::chain is not called if it is after Bus::batch - exception is thrown but there is nothing in failed jobs.
Call to undefined method Closure::getClosure() {"exception":"[object] (Error(code: 0): Call to undefined method Closure::getClosure() at /home/eldair/code/bus_chain_test/vendor/laravel/framework/src/Illuminate/Queue/CallQueuedClosure.php:107)
Steps To Reproduce
Batch setup
$batch = [];
for ($i=0; $i < 5; $i++) {
    $batch[] = new BatchJob($i + 1);
}

This will throw the exception and the closure will not be called
Bus::chain([
    new ChainedJob,
    Bus::batch($batch),
    function () {
        logger('after batch');
    }
])->dispatch();

This works fine
Bus::chain([
    new ChainedJob,
    Bus::batch($batch),
    new ChainedJob,
    function () {
        logger('after batch');
    }
])->dispatch();

Here is the reproduction repo https://github.com/eldair/bus_chain_test/tree/main, nothing was added except two jobs and console command to call the code above.

In my opinion, the problem is that the serializable-closure library incorrectly handles the case when SerializableClosure is used as a property type inside another SerializableClosure.
This case is happening here https://github.com/laravel/framework/blob/94df3d60c9a6c2e6fdb47202d979fc92d339a5f8/src/Illuminate/Bus/ChainedBatch.php#L131
I fixed the issue in this pull request https://github.com/laravel/serializable-closure/pull/86
I think this bug will be fixed when my pull request is accepted


Support unicode range

for example:
/[\u4e00-\u9fa5]{2,4}/.random_example
# => "廢仅弙"

Coverage remained the same at 100.0% when pulling d97ab9d09766480702aa1e1e0235f71c85cdfef4 on xiejiangzhi:master into 90229731daf8a40bf34b739a12e403d2bc45d272 on tom-lord:master.

Coverage remained the same at 100.0% when pulling d97ab9d09766480702aa1e1e0235f71c85cdfef4 on xiejiangzhi:master into 90229731daf8a40bf34b739a12e403d2bc45d272 on tom-lord:master.

Coverage remained the same at 100.0% when pulling d97ab9d09766480702aa1e1e0235f71c85cdfef4 on xiejiangzhi:master into 90229731daf8a40bf34b739a12e403d2bc45d272 on tom-lord:master.

Coverage remained the same at 100.0% when pulling d97ab9d09766480702aa1e1e0235f71c85cdfef4 on xiejiangzhi:master into 90229731daf8a40bf34b739a12e403d2bc45d272 on tom-lord:master.

The CI is failed before my changes.

CI should be fixed now, sorry.
I think we should add spec/examples.txt to the .gitignore before merging; I'll look through these change more carefully later if CI goes green.
Thanks for the contribution 😃

Hi @tom-lord , I removed spec/examples.txt.

:+1: Thanks!


Add new providers and alphabetically reorder for readability

Context
We have some new providers wanting to join the service.
Changes proposed in this pull request
This PR adds the following:

Alliance of Leading Learning
Ashton on Mersey SCITT
Nottinghamshire Torch SCITT
Oriel High School
Shotton Hall SCITT
Somerset SCITT Consortium
Trent Valley Teaching School Alliance

to providers_to_sync.yml and just alphabetically reorders them with the name of the provider beside for readability.
Guidance to review
Nothing in particular.
Link to Trello card
617 - Add December providers to support console

@tijmenb some of them start with numbers, maybe that's why?


(background.scan): only init values if they do not exist

Not yet tested. Script works.
Just came across this. The _background.scan.sh always inits the values, thus resetting them every time and afterwards sets them again.
Thus I updated _cache.sh to enable the NX parameter (see https://redis.io/commands/set/) to only set these initial values if they do not exist already.
Since saving these values is not a really expensive operation, I'm not sure if this change is meaningful enough.
The _cache.sh update is kinda hacky, don't know how to make it better without changing too much.

I rewrote the NX extra parameter to "_cache.sh init" - merged for testing.


Info about # of transactions

Add a prompt in between signing claim/claim all transactions about how many there are and that if you have ledger you will have to sign all.

This is for the old governance ui, closing as stale


Galaxy catalog creation on snapshots

Run Galacticus on AlphaQ (underlying N-body simulation) merger trees to generate one Galacticus output per time snapshot

Completed 10/18/17.
Next step to make a list of nodeIndexes of all Galacticus galaxies with halo mass > ~ 1e11 completed on 10/20/18 and passed to next phase of catalog production (generate the corresponding core catalog).


Correct some OSSL_PARAM documentation

The documentation wasn't quite in sync with the implementation.

Merged to master so #9404 can have conflicts fixed early.


Decoding firefox Login info with utf8 encoding

Firefox uses UTF8 for login info storage. Since utf8 can decode ASCII characters(without extended codes) properly, the decoding method was changed to UTF8 for compatibility.
I have test it with my own Firefox profile which contains both English and Chinese usernames and it works just well.

Thanks for the fix!


Feature Request : Character limit on tags.

I think we can have a character limit on tags that are being added.

+1 for this

That issue can be resolved using the following library:
https://www.jqueryscript.net/form/limit-characters-maxlength.html
$(".tagging input").maxLength({ maxChars: 10  });


MIDIUSB.h: No such file or directory

I cannot compile your Sketch because i get this error, although the MIDIUSB.h exists in C:\Users\name\Documents\Arduino\libraries\USBMIDI\src
In file included from C:\Users\tills\Documents\Arduino\AtomS3_USB-BLE_MIDI_Bridge\AtomS3_USB-BLE_MIDI_Bridge\AtomS3_USB-BLE_MIDI_Bridge.ino:34:
C:\Users\tills\Documents\Arduino\libraries\Arduino-USBMIDI\src/USB-MIDI.h:26:10: fatal error: MIDIUSB.h: No such file or directory
 #include <MIDIUSB.h>
          ^~~~~~~~~~~
compilation terminated.
exit status 1

Compilation error: exit status 1

do you have a tip how to solve this?

@tilllt I think you use master branch of Arduino USB-MIDI Transport.
This project needs multi-cable-support branch on my FORKED version. please chekcout the branch git checkout multi-cable-support
About Arduino BLE-MIDI Transport, it also need connection_compatibility branch on my FORKED version. Please be careful.

@tilllt  sorry, this is my typo. please try git checkout connection_compatibility

Thank you, finally it is working.


feat!: Update to Capacitor 5

Updated the plugin to Capacitor 5
changed the web error to return the default UNIMPLEMENTED error code ERR_PLATFORM_NOT_SUPPORTED, but can change that back.
Also changed some other things in the .ts files that are no longer needed but are not related to Capacitor 5.
closes https://github.com/moberwasserlechner/capacitor-sms/issues/20
closes https://github.com/moberwasserlechner/capacitor-sms/pull/18
closes https://github.com/moberwasserlechner/capacitor-sms/pull/19

Thanks for the PR and the cleanup. I did not know this.unimplemented() existed buts makes total sense to use it.


Add firebase-functions to devDependencies

It's inside ./functions => sorry just starting to use those :)


add support for webpack4

Please review carefully, Im not sure the PR is complete. I have no deep knowledge of webpack plugin internals.
I tested it in a small reference app and got a manifest generated.
Feedback appreciated.
Fixes #111

Codecov Report

Merging #118 into master will decrease coverage by 5.74%.
The diff coverage is 72.22%.

@@            Coverage Diff             @@
##           master     #118      +/-   ##
==========================================
- Coverage     100%   94.25%   -5.75%
==========================================
  Files           2        2
  Lines          78       87       +9
==========================================
+ Hits           78       82       +4
- Misses          0        5       +5

Impacted Files
Coverage Δ

lib/plugin.js
94.18% <72.22%> (-5.82%)
:arrow_down:

Continue to review full report at Codecov.

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

Thank you very much for that! :+1:
Can you edit the .travis.yml so test are running for v4

I updated it but I think we will have to wait until webpack-contrib/extract-text-webpack-plugin#707 to improve webpack 4 testing strategies.

Cool test are passing, I will merge that this week-end (remind me if I don't... ;) )

@mastilver Ping also on making a stable 2.0 release :) :)

@mastilver When can we expect this to get merged? 😬

Also, https://github.com/webpack-contrib/extract-text-webpack-plugin/pull/707 has been merged 🎉

For me its totally OK to upgade all dependencies, cut a 2.0 release of the plugin only working with webpack 4 and clean up old plugin API but I guess most people want to have a transition period.

@andriijas Thank you for the great work, let me know if you want ot be more involved into this project! :)
@all sorry for the delay, I didn't have much time. My boss just granted me permission to work on open source for the day, so I'm going to make the most out of it :)

@mastilver  Thank you for letting me, I honestly dont have any deep knowledge  of webpack I tried my best using the migration guide: https://medium.com/webpack/webpack-4-migration-guide-for-plugins-loaders-20a79b927202
Im still not sure how callbacks changed with the new plugin API

hum... weird, the build is now failing... maybe a dependency changed?

Found where it's coming from, a test :)

the test suite seems to run on webpack 3.x and maybe it should for a while. I read somewhere webpack 5 will be out in roughly  6 months, by then we can deprecate old plugin format.
Maybe we should use tapAsync though if we want that callback support   ref: https://github.com/facebook/create-react-app/pull/4077#discussion_r170622660

No, I'm talking about our unit tests and also I think we are missing: webpack-manifest-plugin-after-emit now on webpack 4

@mastilver check new PR

v2 is now available :tada:
Thank you @andriijas for your work, and all for your patience

no worries, just been learning while helping.   thanks finishing up!


ORC-1306: Fixed indented code style for Java modules

What changes were proposed in this pull request?
This pr is aimed at fixing all indentation style issues with java modules. And add indentation checking rules to checkstyle.xml.
    <module name="Indentation">
      <property name="severity" value="error"/>
      <property name="basicOffset" value="2"/>
      <property name="braceAdjustment" value="0"/>
      <property name="caseIndent" value="2"/>
      <property name="throwsIndent" value="2"/>
      <property name="lineWrappingIndentation" value="4"/>
      <property name="arrayInitIndent" value="2"/>
    </module>

Why are the changes needed?
Automated checks save more time and nothing is missed.
How was this patch tested?
Checking with maven-check-plugin.

Thank you for fix it @guiyanakuang

It's merged to main.
Could you make a PR to branch-1.8 too?


Bump version to 0.4.15

Releases https://github.com/vinted/elasticsearch-dsl-rs/pull/222.

Codecov Report

Merging #223 (b97a2cf) into master (95e26e2) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##           master     #223   +/-   ##
=======================================
  Coverage   85.36%   85.36%
=======================================
  Files         159      159
  Lines        9809     9809
=======================================
  Hits         8373     8373
  Misses       1436     1436


Destroyed savegames after Xenia updates

This is just a warning that the newest Canary version bricks savegames. When launching Xenia Manager I automatically clicked update on the newest Canary version but all my Perfect Dark Zero savegames are not loading anymore. I tried the old Canary version and I can access them again. I feel like some kind of warning would be appropriate. I know it should be obvious that any new update can have unwanted effects but users (me) are dumb and it would have reminded me to be more cautious.
EDIT: I used a completely new Canary folder without Xenia Manager to make it work. When I try to replace the Canary Version for my old folder structure Xenia Manager starts and shows the window at around 50% opacity and shuts down again. It seems inaccessible after deleting the Xenia Canary folder and replacing it with an older version. That means I can't use older Canary versions with Xenia Manager, only the newest version because that is the one that is automatically installed with Xenia Manager.

Because of the new update, you need to create a profile by manually launching Xenia (Xenia Manager folder/Xenia Canary/xenia_canary.exe), which should prompt you to create profile and merge data. This will migrate your content folder into the new structure and everything (should) be working just fine (Alongside you refreshing the game's configuration file with the "Copy Default Settings from the Default Profile").
Sadly, this is not something I can fix easily and it's on the users to migrate properly for everything to work properly and for that I'm sorry. This was a necessary change and it's just something we have to work around with since I'm working on this on my own and it's not affiliated with Xenia itself (although I am present on their Discord and talk with the contributors/maintainers when I have questions about the emulator or to help with some Xenia Manager issues people come with). With that, I don't expect another big update that will break Xenia Manager like this (especially v2.0.0) unless it's Xenia getting proper UI, which is when I'll archive this repository and stop working on Xenia Manager since it won't be needed anymore.

Here's a guide by the contributor for migrating to the new profiles system
If you're using a manager and want to restore savefiles the procedure is simple.
If you have a profile with XUID that starts with B13: Just right-click on it and select login
If you don't have any profile available:
Run Xenia from executable (without manager)
There will be a prompt with option to migrate. Select create profile and migrate
If you're not using manager that's it. If you're using manager you need to run it from manager, open Profile Menu and right-click on profile and select Login
If you don't and have different profiles:
Remove these profiles
Run Xenia directly, there should be a window with button to "Create profile & migrate data"
Click it and create profile, then close Xenia after receiving notification that migration is done
Run Xenia from manager. You can check if everything is ok by opening Profile Menu and checking if that profile exists and has icon on the left side

Thanks for posting this. Unfortunately this instance of Xenia Manager seems to be completely broken for me. I can create the profile and migrate. I can even play the game via Xenia Canary and use my savegames. But I can't start Xenia Manager at all anymore. I think it's because I deleted the Xenia Canary folder and manually dropped an older version of Xenia Canary in there instead.
I had to create a new instance of Xenia Manager in a new folder, install Xenia Canary with Xenia Manager, start Xenia Canary without creating a profile (immediately close it again) and then copy over the migrated files from the previous Xenia Canary instance.

Damn, I can't be thankful enough for the info in this thread, I thought my saves were lost forever! I use custom shortcuts to launch my games, and never seen the prompt to convert my profile into the new folder structure, so I was at a loss when suddenly I needed to create a new profile, with all saves and installed content being seemingly lost. I even stopped using Xenia because of this bummer!
Yesterday I decided to search for how to convert or resign Xenia saves one last time - and I'm glad I found this issue report! I just deleted (actually moved out, but doesn't matter) the newly created profile folders in the content folder, and launched Xenia itself - received the prompt, the profile was converted to the new folder structure successfully, and I have my Forza Horizon and GoW 2-3 saves back!
My only concern is that now there is that "Title update signature doesn't match" warning whenever I launch Forza Horizon. Probably because of my tinkering when I tried to make it work. Is it something I should be concerned about? Should I reinstall the title updates for the game?
And again, seriously, thank you people! 🙇♂

Fixed in v2.0.0


Added Mercator Projection

Added a class to calculate the coordinates in the Mercator projection.

Wonderful!
So now I have to think about the next steps....

@Kibnet I may be a tiny bit late with this question 😳 but, which epsg code is this?


Check nil column type when when converting redshift data api row

Not sure the column TypeName will ever be empty, but this is just a defensive check added to the code.

There is an undetected conflict with your other PR. If you remove here this line:
https://github.com/grafana/redshift-datasource/blob/main/pkg/redshift/driver/rows_test.go#L367
You should get a green.


Fix not working selects and weird choice behavior; add defer for components and fix responding to components

About this Pull Request

Fixes Error when creating selectoptions

Seperates Button and Select when sending

Converts options to dict before sending

when syncing options are called twice, while choices are a dict the second time not having the ._json attribute

Also, I added defer to components and fixed the problem that always the original message was updated instead of sending a new

Checklist

[x] I've run the pre_push.py script to format and lint the change(s) made.
[x] I've checked to make sure this pull request runs on Python 3.6.X.
[ ] This fixes/solves something from the project's Issues.

Issue (if referenceable):

[x] This adds something new.

[ ] This is not a code change.
[x] (If required) Relevant documentation has been updated/added.
[ ] There is/are (a) breaking change(s).

Additionally, resolve merge conflicts with gateway.py being updated (from #376) so once changes are made, the PR may be merged.

should be all done now

Merging this pull request, thank you for your contribution! Welcome to the contributors table. 😃


Add retry to networkpolicy e2e test to avoid flakes

NetworkPolicy e2e test involves dns resolving and dns resolving and
service loadbalancing which relies on coredns and kube-proxy. It's
observed that the test sometimes failed on dns resolving. This PR adds
retries when testing connection to avoid flakes due to coredns and
kube-proxy data syncing delay.
Besides, it corrects out-of-date function names and comments, and
removes connection tests against pod IPs directly as they have been
covered by service tests and can extend the testing time.

/test-e2e

/skip-conformance
/skip-networkpolicy


fix(checkbox): update checkbox colors and props

Closes https://github.com/carbon-design-system/carbon/issues/11268
Update checkbox color var and props
Changelog
Changed

updated override color token
added checked modifier to live demo

Removed

removed a prop

Hey @laurenmrice the border around the box seems to be an issue with the component when its is both checked and disabled and not the website. Would you mind opening an issue in the carbon repo please.

@aledavila Ok! Issue has been made here: https://github.com/carbon-design-system/carbon/issues/11403

@tay1orjones one more review please 😄


Update separators

Change "," to "\n" and remove sep = "|"
Ref #59

Hi @gitizenme i will close this pr, since from version 1.3.12 you can use public_install_page_url_map_format input, to specify the output format.

👍


Support a concatenated gzip data

Currently requests/urllib3 doesn't support a concatenated gzip data. Only the first gz content is extracted. Here is an reproducible test case.
1.Create a text file abc.txt including three lines of data:
1
2
3

2.Update GzipSimpleHttpServer
https://github.com/smtakeda/GzipSimpleHTTPServer/commit/68506651620840ba15cc6f60b4d15a3e7719b8c9
3.Launch GzipSimpleHttpServer
python GzipSimpleHTTPServer.py 8001

4.Run this:
import requests
ret = requests.get('http://localhost:8001/abc.txt')
print(ret.text)

You'll get 1 only, since zlib.decompress stops decoding at the end of the first gz data.
The proposed fix is feed https://docs.python.org/3/library/zlib.html#zlib.Decompress.unused_data into a new Decompress object until no unused_data is observed.

ah ok, the test case should be included... will work on it.

This is a tricky one.
The concatenated gzip format is...unusual. Primarily, it's unusual because it is used as a hack for concatenating multiple bits of unrelated output: for example, when used from the command line it allows you to essentially build an archive that preserves the filenames of each compressed file.
Certainly, I can't find evidence of support for this format in any other user-agent: curl doesn't support it (it prints 1 and then exits), Chrome doesn't support it (it gets the 1 but then basically writes out the remaining unused data), Safari doesn't support it (it prints 1).
This makes me disinclined to want to add support for the format into urllib3. In particular, it makes us prone to attacks on the zlib compression logic by including unused gunk at the back of the file that we will then attempt to decompress. That seems like an unnecessary vector to me.
Is there any reason that incremental gzip compression wouldn't be more useful in this case? (i.e. using zlib.compressobj)

Hi, thanks for a quick response. Yes, our company's DB engine compresses data in gzip in parallel, concatenates them and stores it in S3 in a special mode. I've implemented Python connector that downloads data using requests, and we discovered the second and further gzip data were ignored.
As you pointed out, Chrome, Safari don't support it, yet IE does according to my colleague who implemented the server code. Java has some library that can have the fix. So far I've done monkey patch to make this work.
In my understanding, if the data are compressed in parallel, it would be faster DB engine to send back to the client, i.e., compress each and concat multiple gz files should be faster than compressing a large file.

@smtakeda Given the use-case you have, which is presumably encrypting quite a lot of files, it seems like it may be better to simply not use the transparent gzip decryption that urllib3 provides, and instead to handle this with a custom decoder. You can disable the transparent decompression that urllib3 provides when using requests by using response.raw directly.
Another alternative would be to use something like gzjoin at the end of your compression pipeline to correctly concatenate those files together: this should resolve the problem of only seeing one chunk, and works across all user-agents rather than relying on each one to independently make a change that works for this use-case. This does have the downside of needing to decompress the data in order to correctly build the CRC32, but should otherwise be ok.
I remain reluctant to add support for this in urllib3: this is such an uncommon use-case that I don't know it justifies the maintenance burden of the code.

ah ok, response.raw worked. I thought it would not, because iter_content didn't:
http://docs.python-requests.org/en/latest/user/quickstart/#raw-response-content
This is a test code, and I can live with this in my application.
import requests
import zlib

ret = requests.get(
    'http://localhost:8001/abc.txt', stream=True)

chunk_size = 8192

data = []
d = ret.raw.read(chunk_size)
while d:
    data.append(d)
    d = ret.raw.read(chunk_size)

all_data = ''.join(data)

idx = 0
max_len = len(all_data)

obj = zlib.decompressobj(16+zlib.MAX_WBITS)
output = ''
while idx < max_len:
    if max_len - idx < chunk_size:
        end_idx = max_len
    else:
        end_idx = idx + chunk_size
    print("> start {0}, end {1}".format(idx, end_idx))
    file_content = all_data[idx:end_idx]
    output += obj.decompress(file_content)
    while len(obj.unused_data) > 0:
        print('>> unused_data: {0}'.format(len(obj.unused_data)))
        unused_data = obj.unused_data
        obj = zlib.decompressobj(16+zlib.MAX_WBITS)
        output += obj.decompress(unused_data)
    idx += chunk_size
    print('>>> {0}'.format(output.split('\n')))
output += obj.flush()
print(output)

Great suggestion about the compression end. I'll let the colleague know and see if we can incorporate it.
Thank you very much!

This is a very interesting use case, @smtakeda would you be up for writing a little recipe tutorial to be included in our docs?

sure, @shazow. Would the doc be in requests or urllib3? Btw, I came up with a simpler code where the parameter is response.raw. This assumes the data encoding is text UTF-8 and returns a Unicode data.
CHUNK_SIZE = 8192
def decompress_raw_data(raw_data_fd):
    obj = zlib.decompressobj(16 + zlib.MAX_WBITS)
    output = []
    d = raw_data_fd.read(CHUNK_SIZE)
    while d:
        output.append(obj.decompress(d))
        while obj.unused_data != b'':
            unused_data = obj.unused_data
            obj = zlib.decompressobj(16 + zlib.MAX_WBITS)
            output.append(obj.decompress(unused_data))
        d = raw_data_fd.read(CHUNK_SIZE)
    output.append(obj.flush())
    return b''.join(output).decode(u'utf-8')

@smtakeda It'd be much better to have the doc in urllib3's docs, I think. =)

Yea, feel free to add a new Recipes section in our docs/ and toss it in there (and add yourself to the contributors file!).


Enables multiple outputs for conversion to tensorflow

solve the problemhttps://github.com/microsoft/MMdnn/issues/759

@rainLiuplus
I am sorry that this PR did not pass the automatic test, but I looked at the test case that was not passed, and it is related to keras and mxnet, but my modification is only related to tensorflow, and it is the last step of the conversion process, based on the py generating tensorflow. Processing after the file.
I would like to ask you to see if there is a problem with this test case.

Hi @SmokerX, thanks for your contributions!
The test cases you mentioned have been restarted. They are often out of memory and out of time due to the limitations of Travis platform.

@rainLiuplus
I notice that this automatic has finished 14 minutes ago, but you said those test cases had been restarted 34 minutes ago. The status of automatic test is finished now.  Did they really restart?

@SmokerX, sorry, I do not understand your meaning. I manually restarted the test cases.

@rainLiuplus
OK, these test cases are running now.

@rainLiuplus
It seems to be out of memory again...

@rainLiuplus
There are still 2 test cases left this time.

The tests are all passed now.


Better Error Message On 'docker-machine active' When No Hosts Found

I believe we need a better error message and error code if there are no active hosts found with 'docker-machine active'.
e.g. I see this when I have no active hosts:
% docker-machine active
Error getting active host: Active host not found
% echo $?
1

That seems fairly clear to me, but I'm biased  Do you have any suggestions?

Didn't realize there is no more than 1 active host at a time. I was thinking there could be more than one and thus thought this error message was off. Apologies. Closing.

No problem - there actually can be more than one "active" host in a sense, just not more than one per shell - this is because the active host is dictated by the environment variable DOCKER_HOST and others, which can change per-shell.


Update al2 dockerfile for knn build

Description
Update al2 dockerfile for knn build.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Related issue:

https://github.com/opensearch-project/opensearch-build/issues/4379

https://build.ci.opensearch.org/job/docker-build/5319/console


Feature/merge data stream: WIP

Just for people to have a look at :) Not completely finished yet - there's a few tweaks that definitely need doing before it's pulled, most tasks are listed at the top of the dataInterface class

If you pan to the far left it makes a call for new data, and live data streams in all the time, altering the last candlestick. You can change periods in the main.js file where dataInterface is initialized


example fails

Following your instructions, I was able to do SignUp. When I try SignIn I'm getting the following:

I'm using all default settings. In my app setup, I see only default Redirect URL http://localhost:9000/callback
My JS config:
const config = {
  serverUrl: "http://localhost:8000",
  clientId: "<clientID>",
  organizationName: "test",
  appName: "test-front",
  redirectPath: "/callback",
};

How do I setup my app to

I found that my Casdoor Test SignIn Page doesn't work, returning this result. Still confused by this example, since an "example pet project" doesn't deliver any clear picture of using Casdoor SDK

Hi @tetra12 the issues you created these days are mostly simple questions or not Casdoor issues. Plz stop making new issues and try to ask questions via channels like Gitter: https://casdoor.org/help

@hsluoyz Hi you didn't answer any of my questions. The issue is THE issue. If one can't make it work from YOUR example, than make sure your thing works
The support of your project is poor at best, getting closer to NON-EXISTENT. Do you develop your project so that only your fellow Chinese students could you it ?

@tetra12 you need set up a "Redirect URLs" on the Casdoor. "http://localhost:8080/callback" and "http://localhost:5000/api/signin" both exists in The Casdoor setting(Application on Casdoor). in this way, the "sign in" would be redirect to correctly URL.


Update go1.17

What does this PR do?
This PR:

Updates the Go version to 1.17
Updates the Traefik dependency version to v2.5.6

I'm adding the bot/no-merge label because the semaphore workflow is not triggered. Maybe that's because this pull request modifies the semaphore.yml. Any ideas?

/sem-approve

/sem-approve

/sem-approve


[build-toolchain] Add the ability to specify that a toolchain should only build macOS.

This is useful when one only has to repro something on macOS, needs a toolchain, and wants to save some build time.

@swift-ci smoke test


tweaking profile pics

need to adjust profile pics to a consist size.  check to see if there exists a library for image sub-selection (cropping) and resizing...

https://stackoverflow.com/questions/6136588/image-cropping-using-python/8696558


[Tab]sub-level tabs can not be removed

Expected behavior
User can click on Trash icon in the action bar to delete dashboard components
https://user-images.githubusercontent.com/67837651/119538546-0f77d400-bd40-11eb-88a1-b39944b5a996.mov

@geido

I am having a look at this

@junlincc I can't reproduce. However, there is an annoying issue when a sub-tab is empty and it has another sub-tab down below. You can never reach the trash icon with your mouse :D
Check the video. I am solving that one
https://user-images.githubusercontent.com/60598000/119869024-735cf100-bf28-11eb-8d82-b52f76a3f147.mp4


openvswitch: fix systemd integration assertion

This is a commit which fixes https://github.com/NixOS/nixpkgs/pull/53033.
The commit was missing in the PR.
It disables the now obsolete assertion about vswitches being incompatible with systemd-networkd

Motivation for this change
Enable use of systemd and openvswitch managed by nixos.
Things done

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

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

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

What can we do to get this merged ?


[manager/dispatcher] Fix shutdown deadlock because of new RW rpc lock

Nevermind, this is actually not a problem.


sysconfig does not preserve existing kernel params

sysconfig charm wipes existing kernel params after the node has been rebooted.
Workaround:

Set grub-config-flags so that it includes required kernel param, e.g:
juju config sysconfig grub-config-flags='GRUB_CMDLINE_LINUX_DEFAULT="$GRUB_CMDLINE_LINUX_DEFAULT nvme_core.multipath=0"'.

Run update-grub on sysconfig unit.

Reboot node.

After the reboot, the node comes back online with correct kernel params:
$ cat /proc/cmdline
BOOT_IMAGE=/boot/vmlinuz-4.15.0-72-generic root=/dev/mapper/vg0-lvroot ro console=tty0 console=ttyS0,115200 console=ttyS1,115200 raid=noautodetect pti=off intel_iommu=on iommu=pt nvme_core.multipath=0

Imported from Launchpad using lp2gh.

date created: 2019-12-05T10:57:53Z

owner: phausman

assignee: None

the launchpad url

(by xavpaice)
There are a couple of ways to approach this.  The config item can either take ownership of the entire grub config flags, and remove any that were added manually prior to the setting being used, or we can make it adapt to not edit existing flags and add new ones.  I tend to prefer the former, as we are using this to model the desired state for the setting rather than simply using the charm as a means to make small edits.
Given that, the documentation is inconsistent, and we need to fix that urgently in order to avoid confusion and unexpected outcomes from the setting.


Initial usage stl2ascii???

The readme has an initial usage that says stl2ascii your_binary_stl_file.stl new_ascii_stl_file.stl, but there are no options to do this in the package and no example files to run this command. Am I missing something?

Are doing a pip install numpy-stl it should be a command that you can run just like the pip command.
Specifically, this part of the setup.py script takes care of that:
https://github.com/WoLpH/numpy-stl/blob/b2e71ccacf7260cbbc77d8d092dd375437e46589/setup.py#L115-L119
If it doesn't work for you, that probably means you don't have the Python scripts path in your shell's PATH environment variable.


Add emoji support to Windows Console

Please support emoji within Windows Console.
Very useful when you code in vim newsletters for startups or when you categorize stuff by emoji.

Already on the backlog :)

Sweet! Another use case: I have a command line app that outputs warnings using ⚠.

@zadjii-msft will this include support for non-latin unicode characters? i.e. can Arabic or Japanese characters not found in the currently deployed console font be displayed with a different font face?

Hypothetically, yes.
Arabic is kinda it's own problem though - there's pretty much no chance we'll be implementing left to right language support any time soon, bt the characters might be able to render correctly.
@adiviness can speak more on the topic if there's more to share.

Arabic is a difficult one because of the additional support required for bidirectional text and ligatures. As far as I’m aware not many terminal emulators currently support it.
There is also a difference between the console being able to render emoji and supporting font fallback in the case of the current font not having a glyph. They are both items on our backlog.

+1 Our build and CI scripts have emojis for success ✔, warning ⚠ and errors ❌ for quick, productive eyeballing of logs.

Can we please not +1 issues? Please use reactions or subscribe to notifications.

@SidShetye - appreciate your comment - your use-case is not unusual, but is useful to hear about.
In general, we ask people not to +1 because we'd like to avoid people +1-ing (esp. with no further comment), which just ends up as noise, and makes threads more difficult to parse and manage.
Sharing additional comments, context, observations, issues, etc. is FAR more valuable than a +1 ;)

Weird that nobody mentioned it, but the Yarn package manager uses Emoji's and it's a bit annoying that they are only displayed as squares :/

Thanks @destructive-dragon - there are many tools that contain/emit emoji, but the Console is not yet able to render them.

@bitcrazed You mentioned in this twitter thread on the conpty release that we still need to wait for a new buffer and a new renderer (DirectWrite). Are those the only two major blockers left?

@kavdev Essentially, yes. To display emoji glyphs, we first have to be able to store (potentially compound) Unicode code-points for each glyph (E.g. Ninjacats), but we also have to be able to render them, which requires font-fallback, which GDI doesn't support.
We'll be working on improvements to both the Console's text buffer implementation, and also the Renderer in future releases.

@bitcrazed technically font-fallback isnt required. even if:

current font has glyphs for some 4 byte characters
is monospace
your input data is using glyphs that the font has

it still wont work, because of legacy GDI renderer

@bircrazed
Many Emoji are composites, i.e., having multiple code points joined together (using ZWJ or VS, or whatever), and in most cases they do not fit in one Console cell. So your problem is not "1 cell to n characters" but "m cells to n characters"...

FWIW, my terminal emulator and iTerm2 both render emoji by basically treating them as "full width" (2 cell) CJK style characters. I don't know about iTerm2 but I don't make any attempt to support unicode "modifiers" on emoji or any other characters. Each character has to be one unicode code point although it may be normal width or full width.

@sedwards2009
The ultimate solution should be a Unicode or whatever specification that tell us how to handle complex script on a character grid.

The closest idea may be applying some concepts from justification (like inserting spaces between East Asian characters, and inserting Kashida when fitting Arabic characters into the grid), but justification implementation is a total mess. AFAIK DWRITE does the best job so far, but some implementations (like Kashida) is still very hacky.

@Ben-Hope @noxabellus are saying Same here.
I updated Windows 10 to 1809 and

Yeah I miss that little rocket when starting the vue-cli 😢

@bitcrazed @zadjii-msft is there an issue tracking the regression found in 1809? I'm not talking about full-on emoji support, just getting back the basic foreign language/unicode glyphs that were supported under previous releases. Do you know if the issue persists in later beta releases?

Same here. I updated my Win10 from 1803 to 1809 several days ago, and now all characters >= U+10000 (UTF-8 with 4 bytes or more) no longer display. I have also tried the newest insider version(Windows 10 Insider Preview 18358.1 (19h1_release)), unfortunately, this bug still exists.
Since 19H1 will be released soon, could you please fix it, or report it because it could be a bug in other project?

Same =(

Emoji rendering seems to have improved.
I tried a recent build of CascadiaPackage (Windows; x64) and get this (see image):

Running Windows 10, Build 1903.

Is there any merit to having an option to disable colour font emoji?
So any emoji used will be single colour using the font colour settings?

@mdtuak yep, in fact thats a setting @miniksa and I have previously discussed adding. I just filed #956 to track that work :)

@MartinMa that's an entirely different thing than the existing conhost.exe, if you tried to run OpenConsolePackage (which is the OSS conhost) you should still have the issue.

@MartinMa that's an entirely different thing than the existing conhost.exe, if you tried to run OpenConsolePackage (which is the OSS conhost) you should still have the issue.

Don't be so sure! The DirectWrite renderer that's used in Windows Terminal is also part of OpenConsole! You just need to set a registry key (HKCU\Console\UseDx=DWORD(1)) before it'll use it.

@DHowett-MSFT The console throws an exception when pasting an emoji string.
It's here.
https://github.com/microsoft/terminal/blob/2fdcb679ab1f1f1edc542e3b86327dacea78f7ac/src/buffer/out/CharRowCellReference.cpp#L15

@DHowett-MSFT Dustin Howett FTE The console throws an exception when pasting an emoji string.
It's here.
https://github.com/microsoft/terminal/blob/2fdcb679ab1f1f1edc542e3b86327dacea78f7ac/src/buffer/out/CharRowCellReference.cpp#L15

I am 80-90% sure that either @adiviness or I have a bug covering that already somewhere around here.

Yeah, the work I'm doing on the /dev/austdi/NewCookedRead probably affects the crash there. We don't support emoji's being pasted (or typed) quite yet in all shells.

@MartinMa that's an entirely different thing than the existing conhost.exe, if you tried to run OpenConsolePackage (which is the OSS conhost) you should still have the issue.

Don't be so sure! The DirectWrite renderer that's used in Windows Terminal is also part of OpenConsole! You just need to set a registry key (HKCU\Console\UseDx = DWORD(1)) before it'll use it.

I didn't even know that I was going home to try it. But does this registry affect the default conhost?

Yes, it's almost certainly because of #2053.

@DHowett-MSFT Will my commit fix be appropriate, if there is no problem, I will create a PR.
https://github.com/fcharlie/terminal/commit/4c6280ca35fff9eac0041c94385574bedc5f2a27

Is this fixed? I saw this Youtube video where @cinnamon-msft showed emojis support.

@innovoix That’s the Windows Terminal, this is about the Windows Console.

@ExE-Boss Ah OK, my bad. Thanks.

When I setup a Windows workstation for the first time in 10 years, now very used to OSX, I was very disturbed to lose the beloved and also extremely useful emojis. Sort of funny it's not already a long ago default

@jasonhargrove thanks for the feedback. You may want to check out Windows Terminal, which is (incidentally) also built out of this repository. It does support the things you're looking for.

@jasonhargrove We do plan on integrating many of the improvements to the core engine underlying both Windows Terminal, and Windows Console back into Windows Console in the future. Such areas include the internal text buffer, text rendering engine, etc. - things that won't generally impact backward-compatibility.
Why the "backward compat'" caveat? Windows Console's job (along with Cmd), is to remain backward-compatible wherever possible. Thus, there will be many features (e.g. tabs, split panes, etc.) that will only be available in Windows Terminal, and will never make it back into Console.
We strongly recommend that users start using and testing Windows Terminal, file any unexpected issues here so that we can triage and fix them as quickly as possible as we drive Terminal towards v1.0 (~Q2 2020).
FWIW, Windows Terminal DOES support emoji, split panes, a huge number of configuration options, etc.

One thing that drives me crazy is the inability to Shift+Insert paste in the Windows Terminal.

@jsilvermist you can add shift+ins as a key binding in your Terminal settings.

@jsilvermist Polite request - please keep issues to relevant threads. The issue you describe relates to key-bindings, not emoji ;)

@bitcrazed True, my bad! Also, thanks @DHowett-MSFT!

@jsilvermist No problem - we all do it now and again - many thanks :)

Ok Christmas comes early! Great start to this and what a major leap forward, congrats. And thanks for letting me know about it 📈

Ok Christmas comes early! Great start to this and what a major leap forward, congrats. And thanks for letting me know about it 📈

is it normal to see ?? if i paste the emoji. please confirm. thanks in advance.

@AmericanY yeah, this is an issue with PowerShell 5.1.

@DHowett-MSFT that's happens on CMD, PowerShell and windows new terminal .

Is the src code for fmt_test.exe available somewhere? (screenshot by @fcharlie earlier in this thread)

@luke1961 see https://github.com/fcharlie/bela/blob/master/test/fmt/main.cc

Thanks!

Hi,
To add on the comment of @AmericanY , what pasting in the Windows Terminal an emoji (say from a preexisting output) it is broken in the input and correct in the output.
Also if doing an up arrow right after, then it is correct in the input this time.
Is this expected ?

Windows Terminal
Powershell Core 6.2.1
Meslo LG M for Powerline

@AmericanY yeah, this is an issue with PowerShell 5.1.

Besides that one, the space between two emojis get converted to linefeed in posh 5.1, but not in other shells like ash, bash and so on:

@remidebette You'll want to see #1503

@zadjii-msft , this seems hard, but will be really nice once you crack it.
Hang in there!

I've tested emoji on a few shells under Windows Terminal (Version: 0.11.1191.0) and here are results:

CMD

Powershell

Cmder

WSL with ZSH

Cygwin with ZSH

Looks like the only shell that works perfectly so far is WSL (both bash and zsh)...

I’m working with an old set of information, but it at least used to be that the console stores the prompt line in a separate buffer from the output buffer and that it didn’t support more than ucs2 encoding. WSL works bc it’s encoded in utf8. I was working on a branch at one point that fixed this, if I find the time this weekend I’ll take a peek. Hmm

How can i fix the emoji?
https://i.imgur.com/cOx12WA.png

I'm encountering the same problem; it seems no an up-to-date solution afaik.

Sweet! Another use case: I have a command line app that outputs warnings using ⚠.

Output of Emoji works fine in my experience.

So it can be sure that the problem is input characters not rendering correctly in windows terminal. I have tried Git Bash(one kind fo shell), It works well with Git Bash GUI, but not working with windows terminal same as the bugs mentioned as others.

Git Bash GUI

Git Bash with Windows Terminal

My windows terminal version is 1.1.2233.0.
Hope could fix it as soon as possible! 👍 And is there an estimated schedule?

Please file a separate bug for this issue and include the contents of your Windows Terminal settings file.

Please file a separate bug for this issue and include the contents of your Windows Terminal settings file.

I have the same issue:
{         "guid": "{00000000-0000-0000-ba54-000000000002}",         "acrylicOpacity": 0.75,         "closeOnExit": true,         "colorScheme": "Campbell",         "commandline": "\"%PROGRAMFILES%\\git\\usr\\bin\\bash.exe\" -i -l",         "cursorColor": "#FFFFFF",         "cursorShape": "bar",         "fontFace": "Consolas",         "fontSize": 10,         "historySize": 9001,         "icon": "%PROGRAMFILES%\\Git\\mingw64\\share\\git\\git-for-windows.ico",         "name": "Bash",         "padding": "10, 0",         "snapOnInput": true,         "startingDirectory": "%USERPROFILE%",         "useAcrylic": true       },
Windows Terminal: v1.3.2651.0
Git: v2.16.2.windows.1

Emoji's dont show up in an ssh session as well, not sure if you know about it.

Any update on this?

Nope. We'll make sure to update this thread when there is. In the meantime, might I recommend the Subscribe button?

That way you'll be notified of any updates to this thread, without needlessly pinging everyone on this thread ☺

Emojis😊😋🥵 works very well on VS CODE Terminal!

Did #10478 improve this for the console? I think it did. Everything except the cooked edit line itself should probably work with the Uniscribe flows/fallback that ExtTextOutW offers.

Is this still on the backlog? Unicode 13 was released in March 2020...

Is this still on the backlog? Unicode 13 was released in March 2020...

Seems to work for me at the moment in Windows Terminal Preview 1.12.3472.0 in both PS 5.1 and WSL2/Ubuntu, although #1472 bug still there.

Under profile settings > advanced, 'Enable experimental text rendering engine' has significantly improved things for me like the garbled text in ncurses applications that were rendering certain emojis. Running Preview 1.13.10983.0

Hey guys, is this the official thread about emoji support?
I've just tried Version: 1.15.2874.0 running a simple python script that uses rich:
from rich._emoji_codes import EMOJI
from rich.console import Console

if __name__ == "__main__":
    console = Console()
    for k, v in EMOJI.items():
        console.print(f":{k}:")

And everything has rendered correctly on win10... Problem comes if I try to add emojis manually by pressing win+.,
in that case I'll just get an incorrect render, ie:

Anyway, is this one already reported? Is this the right backlog work-item to suscribe to? I've read another ones pinging to this one so I'm assuming it's the "official" one... 🙄
Evolving really nicely the software btw, congrats guys!

To clarify:

Outputting emoji to the Terminal generally works quite well today. Especially in an environment like WSL, where everything is already in UTF-8.

There's a long tail of bugs remaining here. Some sizing issues.
Then, there's also the case of emoji made from m a n y codepoints, which we're tracking in #8000 (more or less).

Inputting emoji really what's missing here. That's the visible part of "trying to type an emoji into CMD.exe". We're largely tracking that in https://github.com/microsoft/terminal/issues/7777, and the many threads linked to that.

This has kinda become the mega thread for both, though, most of the remaining work remains in #7777.

@zadjii-msft Hey Mike, thanks, cool stuff. So I guess the issue happening when copy/pasting emojis from other software such as SublimeText and not being rendered properly is also a related work-item from this one, ie:

You can see how in SublimeText the emojis are rendered properly, similar in the good old clipbrd.exe software but when i copy/paste into wt the render will be broken.
You can find the emojis.txt file attached
here

It seems to me that cmd.exe does not play any role here. Between cmd.exe and Windows Terminal there is conhost.exe (openconsole.exe), it is responsible for such handling of emoji. If we replace it with something of our own, then there will be the expected result
https://user-images.githubusercontent.com/11535558/210380863-d8061bd5-c4e0-4a3a-963f-cb245e8e845b.mp4

Will there be support for country flags at some point?

That's outside of Terminal's control, the Segoe UI Emoji font just doesn't include flags.
#2664 would allow customising the fonts Terminal tries to use, you could then include something like "Noto Color Emoji", which does include graphical flags.

Has anyone here tried to edit the Segoe UI Emoji font file and add in / replace the country flag emojis from another font? The Terminal doesn't support images so printing out GIFs or SVGs is a non-starter.

When will we get Nerd Font Icons(Glyphs) support in conhost.exe?

When will we get Nerd Font Icons(Glyphs) support in conhost.exe?

You should be able to install a "nerd font" font and set it as your font today. Also last year, and perhaps some years prior to that one.

With #16916 merged and with all the work that led up to it, support for complex Unicode, including Emoji, is effectively done. There are some smaller issues left to clean up, but I expect them to be a rare encounter. As such I'll close this issue for now.
This will ship in Windows Terminal 1.22 this year. If you want to try it out right now, please feel free to download our Canary (nightly) build here: https://github.com/microsoft/terminal/discussions/16121
Please note that PowerShell does not have support for complex Unicode yet, but that's expected to change in the foreseeable future (no exact date yet).
I'll keep this issue locked since a lot of people are subscribed to it and would get notifications every time someone posts here. If find any issues with our implementation, please report a new issue. If you have any other feedback, please feel free to start a new discussion.