allow show icon in taskbar

allow user to specify if the prompt window is to display icon in taskbar or not
this is helpful on windows for when a user hasnt created a browserwindow but only a tray icon
because when you run prompt, no icon is displayed in taskbar and if user minimizes the prompt window by accident, you cant get prompt box back

Hi, thanks for the PR

Hi, thanks for the PR

please can you do a new release with this request in?


ref #339: add action style class (adds cursor pointer) to table list …

…cells if the cell has a function and therefore is clickable

Q
A

Branch
master

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
yes

Tests pass?
yes

Fixed issues
chameleon-system/chameleon-system#339

License
MIT

Thanks @pixeljunkie


NetInfo + AsyncStorage Dependency fix for latest version

Please Add this fix lot of people are requesting this fix

Sorry removing cause of typos


route: Allow route destination <IP_ADDRESS>/8 when route type is specified

According to RFC6890, <IP_ADDRESS>/8 and its subnet cannot be used as the route destination except for the route type is unreachable, blackhole or prohibit.

@cathay4t it seems that nispor should update the netlink-packet-route version into 0.19.0 ?

@cathay4t it seems that nispor should update the netlink-packet-route version into 0.19.0 ?

Done


Decrease flakes in Diagnostic integration tests.

Use a smaller buffer for trace buffer tests as this depends the stacktrace created being large enough.
Only check for filepath and line number on windows as we do in other tests

Thanks Jon!


Doesn't create MCPACK file

So I use Geyser to enable cross-play for me Java server.
To allow my Bedrock players to see my Java resource pack I have been looking for a tool exactly like this! You can imagine my delight when I came across your software. However, I need the java resource pack converted to a MCPACK file as this is how Geyser sends the resource pack to Bedrock users.
I have no use for the converted zip, I need a MCPACK file.
Please help!

I could probably write a solution to this quickly, but in the meantime, a .mcpack is essentially just a renamed ZIP file. so just rename the zip file to mcpack, and it should work!

The issue should be fixed in the latest alpha release!

Sheesh AITE TYVM
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Cory Borek @.>
Sent: Saturday, April 16, 2022 6:05:34 PM
To: agentdid127/RPC-Bedrock @.>
Cc: Kalamity @.>; Author @.>
Subject: Re: [agentdid127/RPC-Bedrock] Doesn't create MCPACK file (Issue #8)
I could probably write a solution to this quickly, but in the meantime, a .mcpack is essentially just a renamed ZIP file. so just rename the zip file to mcpack, and it should work!
—
Reply to this email directly, view it on GitHubhttps://github.com/agentdid127/RPC-Bedrock/issues/8#issuecomment-1100780503, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASGP3D77UQLGOHMQI5GHMLLVFNPV5ANCNFSM5TTDREPQ.
You are receiving this because you authored the thread.Message ID: @.***>


Make a separate category for Bonus Actions

Thanks for the great module! I just have a little quality-of-life suggestion, somewhat linked to issue #61.
If it's doable, I think it would be great if Bonus Actions could be displayed as their own category, as it is in statblocks in Tasha's Cauldron of Everything. As it is now, it can be hard to see at a glance which features can be activated as an action and which can be activated as a bonus action, as it needs to be written in the feature's description.

This is already a "maybe" feature. The issue is, that book is as far as I know the first instance of this format.
As you noticed, the current format is also wrong as issue 61 points out. But my concern is that 99% of Stat blocks with bonus actions don't list them that way, which will be weird for anyone expecting Monster Manual consistent formatting.
I will probably do this anyway, and use TCoE as an excuse. I do prefer it, as it's more clear.

Alright, thanks for the quick reply! I absolutely understand your point. I would just add that in my opinion - and the reason why I use your very good module - the main advantage is not so much that it looks like official statblocks, but that you have all the relevant info at a glance (which is however kind of the point of official stat blocks in the first place).
In this regard, having bonus actions as a separate category is, I think, very helpful. I'm surprised that the official statblocks didn't do it in the first place as it forces the GM to check every description to remember what monsters can use as a bonus action. And given that GMs are already likely to forget those in the first place (at least I know I am), I find this an unfortunate design decision.
I think however that it can be explained by the fact that in a lot of those earlier statblocks, bonus actions were conditional and mostly tied to specific traits, like "if the creature reduces someone to 0 hit points, it can attack again as a bonus action", or "if the creature moves more that 20 feet, it can attack as a bonus action". Those sort of make sense to list under "Feature" (although I still think it would be more practical to have them listed under a "Bonus Action" category). However, for bonus actions that monsters can use every round no matter what (like Fey Spirits in TCE), it really makes sense to have a specific category, especially if your monster has many bonus actions from which he can choose. And I think that people designing their own monsters will tend to give them more bonus actions that regular monsters to spice things up, following advice from people like Matt Colville with his action oriented monsters (at least that's what I do). For such monsters, having a dedicated Bonus Action category is invaluable in combat.
Finally, if you don't implement this new category, a helpful alternative would be to have the option for the type of action used to show up somewhere automatically at the beginning of the description, a bit like auto-generated attack description, from the dedicated field in Foundry. As it is now, and unless I am mistaken, you have to manually write in the description of the action that it uses a bonus action.
So that was my point of view, but I'll of course understand whichever decision you make! Sorry for the wall of text and thanks again for this great module!

Closing as duplicate of https://github.com/zeel01/MonsterBlocks/issues/61 to consolidate issues.


fix(e2e): Remove the PreExistingCRDOwnerIsReplaced e2e test case

The PreExistingCRDOwnerIsReplaced test case is no longer applicable
as the ownership replacement check has been removed from olm. This
test was introduced when olm only allowed one single ownership of
CRD.
Signed-off-by: Vu Dinh<EMAIL_ADDRESS>
Description of the change:
Motivation for the change:
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /doc
[ ] Commit messages sensible and descriptive

Is there any PR(s) that you could link to provide more context as when this functionality was removed?

Here is the original PR for CRD upgrade feature for single ownership that I did (PR). Then, Jeff did another PR to allow multiple ownership upgrade (PR). Since then, that check has been removed as if you look at the CRD upgrade code in step.go, there is not ownership check anymore as it has been removed with this PR.

Thanks, I think I'm convinced this test is no longer needed.
/lgtm


empty page token gives a 422

/v1/assets?pageToken=&keywords=

If we want to make it more likely that old Poly clients work with Icosa without tons of debugging, we should fix this.

This is done. It required parsing the pageSize and pageToken params as str first, then int and falling back to the default if the value could not be parsed as an int.
Not the nicest, but it works.


Add Translation! (ko-default.json)

Summary
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Docs
[ ] Build-related changes
[x] Other, please describe: Translating english to korean

If changing the UI of default theme, please provide the before/after screenshot:
Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

You have tested in the following browsers: (Providing a detailed version will be better.)

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge
[ ] IE

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature
[ ] Related documents have been updated
[ ] Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:

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


Feature/tg 88 programs delete

Summary
Implementation of the programs delete functionality
Details

Delete program endpoint
Database persistence for delete programs
Validation of program existence
Validation of classrooms non-dependencies with program
Unit testing for application layer
Unit testing for infrastructure layer
Unit testing for presentation layer
Update of http file for programs endpoints

please add some images as evidence. You can take this PR as an example:
https://github.com/JU-DEV-LatamBootcamp/Klockan-Backend/pull/12


Convert to template

There's nothing specific to Algorand here except the LinkableTransaction property, which feels out of place in the presentation layer anyway. Consider making it more generic, e.g. SvgQrCode.

Don't have the capacity to maintain a generic svg component, so will convert this into an Algorand specific example / template.


Proper Integration for Mystical Aggradditions

IMPORTANT: Feature requests will only be added to the latest supported Minecraft version.
Is your feature request related to a problem? Please describe.
Ex: I cannot add Insanium Essence seeds. Tier 6 is invalid because it only checks for Tiers in Mystical Agriculture or ones you define in the custom tiers thing. I can add the Insanium Essence seeds under Tier 5, but the problem is it just creates a new essence type all together which isn't usable for anything. I want the Insanium Seeds to produce the proper essence in Mystical Aggradditions.
Describe the solution you'd like
Full support for crops in Mystical Aggradditions and other mods like it, like in Mystical Agriculture.

The behavior with the essences is a work around since the generated items use the mysticalagriculture namespace and would therefor cause issues. I can add a special case for Insanium essence to make that behavior consistent.
As far as the Tier 6 crop tier, that's an issue with agradditions not registering the crop tier properly. That'll be fixed in the next agradditions update, whenever that comes out.

Customization 3.1.1 makes the insanium crop work like the others, and Agradditions 5.1.1 fixes the issue with the tier 6 crop tier.


Using the recommended GzipCodec results in files that gunzip cannot parse.

Using org.apache.hadoop.io.compress.GzipCodec with the Hadoop 2.7.1 and 3.0.0-alpha1 native libraries results in output files that gunzip cannot parse.  This is on x86_64 Debian Jessie with OpenJDK 1.8.0_111.
Hopefully I'm just doing something silly and this is easily resolved.


Keep public headers are headers only

Using IMPL to refactor all public headers, will keep the only declaration in headers.

/assign


Add "--enable-libharfbuzz" to default build to re-enable drawtext

drawtext is currently not working anymore under the current build args without libharfbuzz https://trac.ffmpeg.org/ticket/10570#comment:2

I noticed that too today, do you know when was the last working version with drawtext?

The library should probably be autodetected by FFmpeg, it's not nonfree or anything.

Confirmed - build N-112119-gda1d56776c works as expected. Thanks for the quick fix.

I see no reason why that would be the case.
6.0 configure does not even know about libharfbuzz: https://git.videolan.org/?p=ffmpeg.git;a=blob;f=configure;hb=refs/heads/release/6.0
6.0 should still work fine without it being explicitly enabled. In fact, trying to enable it would break the build.

$ ./ffmpeg-n6.0-latest-win64-gpl-6.0/bin/ffmpeg.exe -filters |& grep drawtext
 T.C drawtext          V->V       Draw text on top of video frames using libfreetype library.
$ ./ffmpeg-n5.1-latest-win64-gpl-5.1/bin/ffmpeg.exe -filters |& grep drawtext
 T.C drawtext          V->V       Draw text on top of video frames using libfreetype library.
$ ./ffmpeg-master-latest-win64-gpl/bin/ffmpeg.exe -filters |& grep drawtext
 T.C drawtext          V->V       Draw text on top of video frames using libfreetype library.

All versions got the drawtext filter enabled just fine. It was only an issue for a few days with the master builds, cause of the new libharfbuzz dependency. Nothing changed on all release branches. None of that got backported there.
So your issue must be something else.
You will need to be more specific than "does not work".

With 5.1.1 :
/var/www/ktt # ffmpeg
ffmpeg version 5.1.1-static https://johnvansickle.com/ffmpeg/  Copyright (c) 2000-2022 the FFmpeg developers
ffmpeg -i /var/www/ktt/var/upload/sharedMovie/192_movie.mp4 -filter_complex "[0]scale=1920:1080:force_original_aspect_ratio=decrease,pad=1920:1080:(ow-iw)/2
:(oh-ih)/2[b];[b]drawtext=fontfile=/var/www/ktt/web/fonts:text= : timecode='00:00:00:00': r='30': x=10: y=10: fontcolor=white: fontsize=50: box=1: boxcolor=0x00000000@1
" -f mp4 -vcodec libx264 -b:v 3M -preset medium /var/www/ktt/var/upload/sharedMovie/192ffmpeg -i /var/www/ktt/var/upload/sharedMovie/192_movie.mp4 -filter_complex "[0]scal
e=1920:1080:force_original_aspect_ratio=decrease,pad=1920:1080:(ow-iw)/2:(oh-ih)/2[b];[b]drawtext=fontfile=/var/www/ktt/web/fonts:text= : timecode='00:00:00:00': r='30'
: x=10: y=10: fontcolor=white: fontsize=50: box=1: boxcolor=0x00000000@1" -f mp4 -vcodec libx264 -b:v 3M -preset medium /var/www/ktt/var/upload/sharedMovie/192_movie_1.mp4
It runs flawless
Same with version 6 (wget https://johnvansickle.com/ffmpeg/builds/ffmpeg-git-amd64-static.tar.xz)
/var/www/ktt # ffmpeg
ffmpeg version N-67181-g984d0b6e6f-static https://johnvansickle.com/ffmpeg/  Copyright (c) 2000-2023 the FFmpeg developers
built with gcc 8 (Debian 8.3.0-6)
/var/www/ktt # ffmpeg -i /var/www/ktt/var/upload/sharedMovie/192_movie.mp4 -filter_complex "[0]scale=1920:1080:force_original_aspect_ratio=decrease,pad=1920:1080:(ow-iw)/2
:(oh-ih)/2[b];[b]drawtext=fontfile=/var/www/ktt/web/fonts:text= : timecode='00:00:00:00': r='30': x=10: y=10: fontcolor=white: fontsize=50: box=1: boxcolor=0x00000000@1
" -f mp4 -vcodec libx264 -b:v 3M -preset medium /var/www/ktt/var/upload/sharedMovie/192ffmpeg -i /var/www/ktt/var/upload/sharedMovie/192_movie.mp4 -filter_complex "[0]scal
e=1920:1080:force_original_aspect_ratio=decrease,pad=1920:1080:(ow-iw)/2:(oh-ih)/2[b];[b]drawtext=fontfile=/var/www/ktt/web/fonts:text= : timecode='00:00:00:00': r='30'
: x=10: y=10: fontcolor=white: fontsize=50: box=1: boxcolor=0x00000000@1" -f mp4 -vcodec libx264 -b:v 3M -preset medium /var/www/ktt/var/upload/sharedMovie/192_movie_1.mp4
-hide_banner
[AVFilterGraph @ 0x6832b80] No such filter: 'drawtext'
Failed to set value '[0]scale=1920:1080:force_original_aspect_ratio=decrease,pad=1920:1080:(ow-iw)/2:(oh-ih)/2[b];[b]drawtext=fontfile=/var/www/ktt/web/fonts:text= : timecode='00:00:00:00': r='30': x=10: y=10: fontcolor=white: fontsize=50: box=1: boxcolor=0x00000000@1' for option 'filter_complex': Filter not found
Error parsing global options: Filter not found

Both of those builds are not from here.

Sorry. :-(
I will try your build.
Many thanks for all your help.


Let's switch from just using GET/POST to allowing all HTTP request methods

I should be done now provided that testing goes smoothly...


Refactor experiment page bundle

We are going to simplify the logic and deployment of experiment page router component and trying to move the bundle to the front end component repository. The package has not been officially published yet, so in this PR, I only installed a local compiled package, you may test the refactoring results by doing the same.

Please add validations where you need before fetching data and consider isRequired props validations mandatory please.

I think that's not this PR about. That is one of your previous work isn't it? The validation?

Hi, I am still not sure what is the anatomogram property in the ExperimentPageRouter. I added a longer comment regarding to that one. Could you please clarify it? Thanks

Yes, I think you are right, we have anatomogram props under variable tabs, i.e., anatomogram is an entry of tabs, but here is a redundant or bug line, as we don't have anatomogram props under content. I removed this line, please take a further look.


Replace elasticsearch_plugin es_version calls with native calls

Pull request acceptance prerequisites:

[x] Signed the CLA (if not already signed)
[x] Rebased/up-to-date with base branch
[ ] Tests pass
[x] Updated CHANGELOG.md with patch notes (if necessary)
[x] Updated CONTRIBUTORS (if attribution is requested)

Historically the call to elasticsearch -version has caused lots of problems due to various underlying causes (JAVA_HOME, resource constraints, etc.). This change replaces it with code that calls the package[elasticsearch] Puppet catalog resource to retrieve the package version instead.
Related to #734

Does https://github.com/elastic/puppet-elasticsearch/pull/754 make still sense?

@lucabelluccini it does - although this PR will eliminate the need to call out to ES in es_version, all of the commands the provider calls to do things like install/uninstall will still benefit by having JAVA_HOME managed in the with_environment method (so #754 may need to be updated with this change, but it'll just remove a few lines from the PR).

I just stumble on this now. I have a masternode machine with total 16Gb RAM and gave Elastic  -Xms8g -Xmx8g. Everything works fine until I do a change and puppet rerun, it runs out of memory.
Error: /Stage[main]/Main/Node[uuc-logmn012-qa.its.uu.se]/Elasticsearch::Plugin[x-pack]/Elasticsearch_plugin[x-pack]: Could not evaluate: undefined method scan' for nil:NilClass`
If I log in to the machine and run /usr/share/elasticsearch/bin/elasticsearch --version
Java HotSpot(TM) 64-Bit Server VM warning: INFO: os::commit_memory(0x00000005d4cc0000,<PHONE_NUMBER>, 0) failed; error='Cannot allocate memory' (errno=12)
#
# There is insufficient memory for the Java Runtime Environment to continue.
# Native memory allocation (mmap) failed to map<PHONE_NUMBER> bytes for committing reserved memory.
# An error report file with more information is saved as:
# /root/hs_err_pid3438.log

free -mh
              total        used        free      shared  buff/cache   available
Mem:            15G        8.8G        4.4G         96M        2.3G        6.1G
Swap:            0B          0B          0B

If I add an extra 5Gb RAM it works again and puppet works.
So is there an easy way turning of the "duplicate elasticsearch run" in puppet?

@flysen thanks for confirming some suspicions about this. The call to elasticsearch in the plugin provider to determine the version will go away once this PR makes it into a release. As far as I know, that should be the only place the elasticsearch executable is directly called by the provider, so it should solve that specific problem - the provider only calls bin/plugin to perform plugin operations so hopefully it doesn't run into memory allocation problem.


XVG not receiving coins in my wallet

Hallo,
I have tried making some test transactions to my XVG wallet before sending all of my XVG.
I have done 3 test transactions and all 3 have not reached my Wallet.
I have done 2 from Binance and 1 from Cryptopia.
I contacted both Binance and Cryptopia for explanation and Binance said that the transactions where OK and send to the address that whas given. Craptopia like always didn't even respond.
I have 2 wallet addresses on my wallet, don't know why that is.
Wallet is synchronized.
Dates of transactions are: 25/26/30 - Januari 2018
Version: Windows.
Transaction was not a stealth transaction.
Version: v<IP_ADDRESS>-unk-beta.
Transaction-rf:
323fe06b0523cce89e929abee14ccf99213109f2999b7cb30d83cbaaa9824fbf.
f17ff6e092d7161afff2e6c33d0a07b48d3e6d85c89beaaad436fa05a6e3c0ed.
f454981b06d914b24bc54ba9f598e732b82f0ce0dfbf2da0bb7c10b1c9958d76.

can you confirm your wallet is on the same block number as the explorer?

Maybe stupid question but how do I check that?

go to the bottom right and hover the mouse, or go to help, debug console and type getinfo

Dear justinvforvendetta ,
I meet same issue, my wallet is synchronized, I follow your instruction and see my block explorer is 1831753, the block of my transaction id is 1850853
So how do I jump my wallet to above block?

same for me - transferred coins from cryptopia 2 days ago and they never hit my wallet version 4.0.1. HELP!

close the wallet, download the newest wallet, then delete the blockchain data and open it again

thank you, I am downloading newest blockchain data

open your Verge client .. go to help -> debug ... on the first screen , please list the details... post screenshot if possible.
dead giveaway:  Version: v<IP_ADDRESS>-unk-beta is the wrong version
if you are on 4.0.1 funky-beta ... and still have problems, come post more information.

You can download almost the full blockchain from:
https://verge-blockchain.com/down
...if you want to speed up the sync.

Can someone help me on how i must delete the blockchain data?
I think if we can put a DUMB PEOPLE GUIDE ON HOW TO UPDATE WALLET, it would help a lot of people.
First step is -close the wallet-.
Second step is -download the newest wallet from the official VERGE website.
Third step is -delete the blockchain data.
Forth step is -Dowload almost the full blockchain from https://verge-blockchain.com/down.

It's in your 'data directory'... i think default is:
C:\users<you>\appdata\roaming\verge
dont delete wallet.dat ,, that's your wallet, lose that and you are done!

also there is a guide out there on the Vergecurrency.com / FAQ... takes you to:
https://verge-blockchain.com/howto

I installed the most recent wallet (4.0.1-funky-beta) and blockchain, but my coins have still not yet appeared, and all blocks are installed. what should I do?

Help -> Debug Window -> Console
Type:
scanforalltxns
It might make it look like your verge client is crashing -- but don't quit out -- it will finish.  It might take 15-30 mins.

if it's a stealth transaction, do this afterwards as well:
scanforstealthtxns

Thanks for the help!

also, verify your wallet is at the same block height as vergechain.org or verge-blockchain.info

ok - i did have unk-beta and upgraded to funky-beta - now its stuck synchronizing network block 20238 remaining - I feel we are getting close - what next?

@highfi5  ... go to help -> debug window ... post a screenshot of that please.
The 'blocks remaining' SHOULD be moving ... just give it time (couple hours maybe)

You got a good amount of "connections".. is it still stuck at 20,238?  I would try closing verge-qt and re-opening if it's still not moving.

closed and reopen - now just says (out of sync) 2 active connections

That's fine -- give it some time, it will start syncing.

YES! perfect AND my last withdraw is now in my wallet - THANK YOU!


System.IO.FileLoadException: Could not load file or assembly 'System.Configuration.ConfigurationManager, Version=<IP_ADDRESS>,

Problem encountered on https://dotnet.microsoft.com/en-us/platform/upgrade-assistant/tutorial/install-upgrade-assistant
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
I have installed System.Configuration.ConfigurationManager, Version=<IP_ADDRESS> in every project under the solution and cannot figure out why it won't find the files.
I am on Windows 11, using VS 2022 v17.2.4 & the latest upgrade-assistant v.0.3.330701+fa00bab2d06aab9c710c344c0f32440ad4eb8410.
My goal is to update my Xamarin app to .NET 6 MAUI.
ERROR:
Unexpected error
Autofac.Core.DependencyResolutionException: An exception was thrown while activating Microsoft.DotNet.UpgradeAssistant.MSBuild.MSBuildWorkspaceUpgradeContext.
---> Autofac.Core.DependencyResolutionException: An exception was thrown while invoking the constructor 'Void .ctor(Microsoft.Extensions.Options.IOptions1[Microsoft.DotNet.UpgradeAssistant.MSBuild.WorkspaceOptions], Microsoft.DotNet.UpgradeAssistant.MSBuild.Factories, System.Func3[Microsoft.DotNet.UpgradeAssistant.MSBuild.MSBuildWorkspaceUpgradeContext,System.IO.FileInfo,Microsoft.DotNet.UpgradeAssistant.MSBuild.MSBuildProject], Microsoft.Extensions.Logging.ILogger1[Microsoft.DotNet.UpgradeAssistant.MSBuild.MSBuildWorkspaceUpgradeContext])' on type 'MSBuildWorkspaceUpgradeContext'.  ---> System.IO.FileLoadException: Could not load file or assembly 'System.Configuration.ConfigurationManager, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. Could not find or load a specific file. (0x80131621) File name: 'System.Configuration.ConfigurationManager, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'  ---> System.IO.FileLoadException: Could not load file or assembly 'System.Configuration.ConfigurationManager, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'.    at System.Runtime.Loader.AssemblyLoadContext.LoadFromPath(IntPtr ptrNativeAssemblyLoadContext, String ilPath, String niPath, ObjectHandleOnStack retAssembly)    at System.Runtime.Loader.AssemblyLoadContext.LoadFromAssemblyPath(String assemblyPath)    at System.Reflection.Assembly.LoadFrom(String assemblyFile)    at Microsoft.Build.Locator.MSBuildLocator.<>c__DisplayClass15_0.<RegisterMSBuildPath>g__TryLoadAssembly|3(AssemblyName assemblyName)    at Microsoft.Build.Locator.MSBuildLocator.<>c__DisplayClass15_0.<RegisterMSBuildPath>b__2(AssemblyLoadContext _, AssemblyName assemblyName)    at System.Runtime.Loader.AssemblyLoadContext.GetFirstResolvedAssemblyFromResolvingEvent(AssemblyName assemblyName)    at System.Runtime.Loader.AssemblyLoadContext.ResolveUsingEvent(AssemblyName assemblyName)    at System.Runtime.Loader.AssemblyLoadContext.ResolveUsingResolvingEvent(IntPtr gchManagedAssemblyLoadContext, AssemblyName assemblyName)    at Microsoft.Build.Evaluation.ToolsetReader.ReadAllToolsets(Dictionary2 toolsets, ToolsetConfigurationReader configurationReader, PropertyDictionary1 environmentProperties, PropertyDictionary1 globalProperties, ToolsetDefinitionLocations locations)
at Microsoft.Build.Evaluation.ProjectCollection.InitializeToolsetCollection(ToolsetConfigurationReader configReader)
at Microsoft.Build.Evaluation.ProjectCollection..ctor(IDictionary2 globalProperties, IEnumerable1 loggers, IEnumerable1 remoteLoggers, ToolsetDefinitionLocations toolsetDefinitionLocations, Int32 maxNodeCount, Boolean onlyLogCriticalEvents, Boolean loadProjectsReadOnly, Boolean useAsynchronousLogging)    at Microsoft.Build.Evaluation.ProjectCollection..ctor(IDictionary2 globalProperties)
at Microsoft.DotNet.UpgradeAssistant.MSBuild.MSBuildWorkspaceUpgradeContext..ctor(IOptions1 options, Factories factories, Func3 projectFactory, ILogger1 logger) in /_/src/components/Microsoft.DotNet.UpgradeAssistant.MSBuild/MSBuildWorkspaceUpgradeContext.cs:line 69    at lambda_method21(Closure , Object[] )    at Autofac.Core.Activators.Reflection.BoundConstructor.Instantiate()    --- End of inner exception stack trace ---    at Autofac.Core.Activators.Reflection.BoundConstructor.Instantiate()    at Autofac.Core.Activators.Reflection.ReflectionActivator.ActivateInstance(IComponentContext context, IEnumerable1 parameters)
at Autofac.Core.Activators.Reflection.ReflectionActivator.b__11_0(ResolveRequestContext ctxt, Action1 next)    at Autofac.Core.Resolving.Middleware.DelegateMiddleware.Execute(ResolveRequestContext context, Action1 next)
at Autofac.Core.Resolving.Pipeline.ResolvePipelineBuilder.<>c__DisplayClass14_0.b__1(ResolveRequestContext ctxt)
at Autofac.Core.Resolving.Middleware.DisposalTrackingMiddleware.Execute(ResolveRequestContext context, Action1 next)    at Autofac.Core.Resolving.Pipeline.ResolvePipelineBuilder.<>c__DisplayClass14_0.<BuildPipeline>b__1(ResolveRequestContext ctxt)    at Autofac.Core.Resolving.Middleware.ActivatorErrorHandlingMiddleware.Execute(ResolveRequestContext context, Action1 next)
--- End of inner exception stack trace ---
at Autofac.Core.Resolving.Middleware.ActivatorErrorHandlingMiddleware.Execute(ResolveRequestContext context, Action1 next)    at Autofac.Core.Resolving.Pipeline.ResolvePipelineBuilder.<>c__DisplayClass14_0.<BuildPipeline>b__1(ResolveRequestContext ctxt)    at Autofac.Core.Pipeline.ResolvePipeline.Invoke(ResolveRequestContext ctxt)    at Autofac.Core.Resolving.Middleware.RegistrationPipelineInvokeMiddleware.Execute(ResolveRequestContext context, Action1 next)
at Autofac.Core.Resolving.Pipeline.ResolvePipelineBuilder.<>c__DisplayClass14_0.b__1(ResolveRequestContext ctxt)
at Autofac.Core.Resolving.Middleware.SharingMiddleware.Execute(ResolveRequestContext context, Action1 next)    at Autofac.Core.Resolving.Pipeline.ResolvePipelineBuilder.<>c__DisplayClass14_0.<BuildPipeline>b__1(ResolveRequestContext ctxt)    at Autofac.Core.Resolving.Pipeline.ResolvePipelineBuilder.<>c__DisplayClass14_0.<BuildPipeline>b__1(ResolveRequestContext ctxt)    at Autofac.Core.Resolving.Middleware.CircularDependencyDetectorMiddleware.Execute(ResolveRequestContext context, Action1 next)
at Autofac.Core.Resolving.Pipeline.ResolvePipelineBuilder.<>c__DisplayClass14_0.b__1(ResolveRequestContext ctxt)
at Autofac.Core.Pipeline.ResolvePipeline.Invoke(ResolveRequestContext ctxt)
at Autofac.Core.Resolving.ResolveOperation.GetOrCreateInstance(ISharingLifetimeScope currentOperationScope, ResolveRequest request)
at Autofac.Core.Resolving.ResolveOperation.ExecuteOperation(ResolveRequest request)
at Autofac.Core.Resolving.ResolveOperation.Execute(ResolveRequest request)
at Autofac.Core.Lifetime.LifetimeScope.ResolveComponent(ResolveRequest request)
at Autofac.ResolutionExtensions.TryResolveService(IComponentContext context, Service service, IEnumerable1 parameters, Object& instance)    at Autofac.ResolutionExtensions.ResolveService(IComponentContext context, Service service, IEnumerable1 parameters)
at Autofac.ResolutionExtensions.Resolve(IComponentContext context, Type serviceType, IEnumerable`1 parameters)
at Autofac.ResolutionExtensions.Resolve(IComponentContext context, Type serviceType)
at Autofac.Extensions.DependencyInjection.AutofacServiceProvider.GetRequiredService(Type serviceType)
at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService(IServiceProvider provider, Type serviceType)
at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService[T](IServiceProvider provider)
at Microsoft.DotNet.UpgradeAssistant.UpgraderMsBuildExtensions.<>c__DisplayClass1_0.b__3() in //src/components/Microsoft.DotNet.UpgradeAssistant.MSBuild/UpgraderMsBuildExtensions.cs:line 46
at Microsoft.DotNet.UpgradeAssistant.MSBuild.MSBuildUpgradeContextFactory.CreateContext(CancellationToken token) in //src/components/Microsoft.DotNet.UpgradeAssistant.MSBuild/MSBuildUpgradeContextFactory.cs:line 27
at Microsoft.DotNet.UpgradeAssistant.Cli.ConsoleUpgrade.RunAsync(CancellationToken token) in //src/cli/Microsoft.DotNet.UpgradeAssistant.Cli/Commands/Upgrade/ConsoleUpgrade.cs:line 50
at Microsoft.DotNet.UpgradeAssistant.Cli.ConsoleRunner.StartAsync(CancellationToken token) in //src/cli/Microsoft.DotNet.UpgradeAssistant.Cli/ConsoleRunner.cs:line 61
[16:30:01 DBG] Hosting started
[16:30:01 DBG] Hosting stopping
[16:30:01 DBG] Hosting stopped
[16:30:01 DBG] UA_NuGetb43a8e6dd47847259a1582e5a0ed9c52 extension is unloading
[16:30:01 DBG] UA_Defaultb43a8e6dd47847259a1582e5a0ed9c52 extension is unloading
[16:30:01 DBG] UA_try-convertb43a8e6dd47847259a1582e5a0ed9c52 extension is unloading
[16:30:01 DBG] UA_VBb43a8e6dd47847259a1582e5a0ed9c52 extension is unloading
[16:30:01 DBG] UA_Webb43a8e6dd47847259a1582e5a0ed9c52 extension is unloading
[16:30:01 DBG] UA_windowsb43a8e6dd47847259a1582e5a0ed9c52 extension is unloading
[16:30:01 DBG] UA_MAUIb43a8e6dd47847259a1582e5a0ed9c52 extension is unloading

Duplicate of issue #1173


Write multiple holding registers: ERROR Invalid exception code

When we write 2 registers it works:
~$ mbpoll -a10 -b38400 -P none -t4 -r25 com4 2 1
mbpoll 1.5 - FieldTalk(tm) Modbus(R) Master Simulator
Copyright (c) 2015-2023 Pascal JEAN, https://github.com/epsilonrt/mbpoll
This program comes with ABSOLUTELY NO WARRANTY.
This is free software, and you are welcome to redistribute it
under certain conditions; type 'mbpoll -w' for details.

Protocol configuration: Modbus RTU
Slave configuration...: address = [10]
                        start reference = 25, count = 2
Communication.........: com4,      38400-8N1
                        t/o 1.00 s, poll rate 1000 ms
Data type.............: 16-bit register, output (holding) register table

Written 2 references.

But 3 it doesn't work!
~$ mbpoll -a10 -b38400 -P none -t4 -r27 com4 2 0 0 -v
debug enabled
Set mode to RTU for serial port
Set device=com4
3 write data have been found
Set data=2
Word[0]=0x2
Set data=0
Word[1]=0x0
Set data=0
Word[2]=0x0
mbpoll 1.5 - FieldTalk(tm) Modbus(R) Master Simulator
Copyright (c) 2015-2023 Pascal JEAN, https://github.com/epsilonrt/mbpoll
This program comes with ABSOLUTELY NO WARRANTY.
This is free software, and you are welcome to redistribute it
under certain conditions; type 'mbpoll -w' for details.

Opening com4 at 38400 bauds (N, 8, 1)
Set response timeout to 1 sec, 0 us
Protocol configuration: Modbus RTU
Slave configuration...: address = [10]
                        start reference = 27, count = 3
Communication.........: com4,      38400-8N1
                        t/o 1.00 s, poll rate 1000 ms
Data type.............: 16-bit register, output (holding) register table

[0A][10][00][1A][00][03][06][00][02][00][00][00][00][A4][7E]
Waiting for a confirmation...
<0A><86><03><73><A3>
ERROR Invalid exception code
Write output (holding) register failed: Invalid exception code

C:\Users\pasca>mbpoll -b38400 -a10 -t4 -r10  com1 20 21 22 23 24 25 26 27
mbpoll 1.5 - FieldTalk(tm) Modbus(R) Master Simulator
Copyright (c) 2015-2023 Pascal JEAN, https://github.com/epsilonrt/mbpoll
This program comes with ABSOLUTELY NO WARRANTY.
This is free software, and you are welcome to redistribute it
under certain conditions; type 'mbpoll -w' for details.

Protocol configuration: Modbus RTU
Slave configuration...: address = [10]
                        start reference = 10, count = 8
Communication.........: com1,      38400-8E1
                        t/o 1.00 s, poll rate 1000 ms
Data type.............: 16-bit register, output (holding) register table

Written 8 references.

C:\Users\pasca>mbpoll -b38400 -a10 -t4 -r10 -c8 -1 com1
mbpoll 1.5 - FieldTalk(tm) Modbus(R) Master Simulator
Copyright (c) 2015-2023 Pascal JEAN, https://github.com/epsilonrt/mbpoll
This program comes with ABSOLUTELY NO WARRANTY.
This is free software, and you are welcome to redistribute it
under certain conditions; type 'mbpoll -w' for details.

Protocol configuration: Modbus RTU
Slave configuration...: address = [10]
                        start reference = 10, count = 8
Communication.........: com1,      38400-8E1
                        t/o 1.00 s, poll rate 1000 ms
Data type.............: 16-bit register, output (holding) register table

-- Polling slave 10...
[10]:   20
[11]:   21
[12]:   16
[13]:   0
[14]:   16
[15]:   9
[16]:   8


I want to add codechef solutions kindly assign it to me

want to add solutions of some recent problems from the codechef contests

sure go for it😊

I want to contribute @tanus786. Please assign me so that I can complete hacktoberfest 🙏.

@SanjoySaha24 sure go for it👍


Tutorial for HMAC signing via C# with examples

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

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

Hi @alexandra142 - Thanks for your new content submission. A member of the content team will review it.

Hi @alexandra142 - Thanks for your new content submission. A member of the content team will review it.


Add a lint test checking for coverage of standardized variants

From @roozbehp:
I just found that the Noto Sans Symbols misses the support for standardized
variants (issue 321). Please add a test to noto_lint checking for the support.
The data table is at
http://www.unicode.org/Public/8.0.0/ucd/StandardizedVariants-8.0.0d2.txt.
There are five classes:

Emoji/text variation sequences ending in FE0E/FE0F should be ignored for
now. The glyphs will be in different fonts anyway.

CJK ones should be ignore for now, since noto_lint doesn't test the CJK
fonts yet.

Noto Sans Mongolian and Noto Sans Phags-pa seem to support the standardized
variant sequences through GSUB rules (they follow Microsoft's font, so it works
with HarfBuzz), although I didn't manually check all cases. A test should be
added to check that all the standardized sequences are supported in the fonts.

The symbol set is missing (and should probably be implemented in the font
using cmap subtable format 14). A test should be added to check they exist in
the font.

Moved from googlei18n/noto-fonts#322

cc @behdad @jungshik

@roozbeh, you mentioned Mongolian.  I've been developing a test for this, ran it on Mongolian, and there's one combination that doesn't seem to be supported:
Automatic testing for 'unhinted,NotoSansMongolian-Regular.ttf', Version 1.04 uh:
E <complex/gsub/variants> Expected GSUB to generate all standard variants, but 1 cases did not (182d 180b (fina): [u182D.fin=0+1358|space=0+0])
There's a web site https://r12a.github.io/scripts/mongolian/variants that seems to indicate that this might be intentional-- NotoSansMongolian is one of the fonts examined on the site.  I'm not sure how current the site is w.r.t. the version of StandardizedVariants.txt I'm using.  Do you know anything about this site?

Btw, the output indicates that Harfbuzz generated two glyphs: 182D.fin and space, when asked to generate output for 182d 180b with features -isol,fin.  There's no ligature data in NotoSansMongolian for 182d 180b (there is for 182d 182c).  All other combinations generate a single glyph as output, so this correctly indicates that this is a mismatch with StandardizedVariants.txt.  The question is whether it is intentionally differing from StandardizedVariants, or adhering to an earlier version of StandardizedVariants, or what.

Fixed in ec48d31c9349e61ca44998a8b9a70b0280a72351
The question of whether the Mongolian behavior is as intended is still open.  I'll file another bug for that.


.car files are not generated with lambdas

Hope this is reproducible.
Project in Eclipse contains only a one module with descriptor (module.ceylon):
native("jvm")
module tt "1.0.0" {
	import ceylon.interop.java "1.3.1";
	shared import java.base "8";
}

Just a one source file:
import java.lang {
	Thread,
	Runnable
}
import java.util.concurrent {
	ExecutorService,
	Executors,
	ThreadFactory
}

shared void run() {
	value f = Foo().fooLambda();
	print(f);
}

shared class Foo() {
	shared ExecutorService fooLambda() {
		variable Integer threadIndex = 0;
		return Executors.newFixedThreadPool (
			2, (Runnable runnable) => Thread(runnable, "pool - thread ``++threadIndex``")
		);

	}
	shared ExecutorService fooSAM() {
		variable Integer threadIndex = 0;
		return Executors.newFixedThreadPool (
			2,
			object satisfies ThreadFactory {
				shared actual Thread newThread(Runnable runnable)
						=> Thread(runnable, "pool - thread ``++threadIndex``");
			}
		);

	}
}

No errors, but .car files are not generated. If try to run the output is:
ceylon run: Module tt/1.0.0 not found in the following repositories: ... (a big list of repositories)...
If look to project folder/modules - there are only .src files but no .car files.
If remove labmda and use only way with object everything is OK.

related to #6749

This sounds pretty bad.

I can't reproduce this bug, it works fine for me on command line and in eclipse. I guess it has been fixed in the meantime. (There were lots of bugfixes for lambda conversions.)


Improve folder loading time

Ever since the version 1.0.0, the loading of folders inside macroParser kept increasing.
I think I located where the issue was, filtering macros while importing is causing performance drop.
Maybe we can filter macros in another thread and boost up performance ?
Maybe using hashing tables would boost performance ?

started to implement hashing tables to boost import time.
Very large folders (> 100'000 macros now loads in seconds !)
Still work still has to be done se that the look command is as fast as the import command.

done : https://github.com/ProSurfer73/Macro-Parser/commit/dd1935f6eb0c084d7df4214e3fb3001f4010b6d4


Add documentation on pipeline deletion

Add a warning about possible collisions when deleting pipelines.

Until https://github.com/gocd/gocd/issues/879 is resolved, we should document possible dangers with pipeline deletion.

Could you elaborate or provide a concrete example of what you mean by "may cause conflicts"

My team just hit the SQLException detailed at https://groups.google.com/forum/#!searchin/go-cd/sqlexception/go-cd/ftCO3UMrbrE/fulTbKqb7mQJ, which links to a related Google Groups discussion at https://groups.google.com/forum/#!topic/go-cd/LVnapi82dmg/discussion. It's a known issue that pipelines can't be re-created with the same name, but poorly communicated.

Just realized this was still open. Do we need more changes here or should this be merged?


Themes doen't uninstall in reloaded VSCode, cause by prev uninstalled theme

Hello,

VSCode Version: 1.18.1 (x64)
OS Version: Windows 10 (x64)

Steps to Reproduce:

Open up VSCode
Install two themes say A and B
Reload VS Code
Head to Extensions
Select one of installed theme, click on Uninstall button (_VSCode shows a confirm dialog)
Click on OK
Click on Relaod button
Now uninstall the second theme

Expected: Confirm dialog should be shown and the theme should be uninstalled.
Observed:  The Confirm doesn't show up and the theme will not Uninstalled even after 5 mins
How to fix?
Simply by closing and re-opening the VSCode
Reproduces without extensions: No, it's related to extensions I think
Thanks.

@dehghani-mehdi After step 5, does the uninstalled them is still shown in installed extensions list?

@sandy081 no, the first try is successful, but after reload (cause by successful uninstall) the issue happens.

@dehghani-mehdi So, you do not see the extension in the Installed extensions list after reload? So you search for the extension again and Install it again?

@sandy081 here is example/more details:
I installed 2 theme, (let's say) theme A and theme B. both installed successfully.
Then I decided to remove both, so in extensions tab/section select theme A and click on Uninstall button, confirm shows up, click on OK, after few seconds the Reload button shows up, click on that, after reload theme A doesn't exist anymore (thanks to VSCode), the problem is, if I select theme B and click on Uninstall, 2 things will be happened:

The uninstall confirm won't show up
The theme won't be removed and the Uninstalling stay for ever

Notice that if I close the VSCode (click on X button) and open it again, I can uninstall the theme B with no issue.
So, are we okay on this?

@dehghani-mehdi I cannot reproduce it. Please see the attached video if I am missing any steps.

Can you check if there are any errors in console. Use commands Toggle Developer Tools and Toggle Shared Process

Can you check to reproduce it on latest insiders - https://code.visualstudio.com/insiders/

@sandy081 I'll try, but can you click on the theme name and click on uninstall button in main area instead of selecting Uninstall from context menu?

Yeah, I tried from the editor too and it is working

It's weird, I could not reproduce the issue!
Sorry and thank for your time, if I faced to the issue again, I'll update this issue with more info.


Bump MSRV to 1.57

Codecov Report

Merging #230 (59082d1) into master (d4bd9c9) will decrease coverage by 0.17%.
The diff coverage is n/a.

:exclamation: Current head 59082d1 differs from pull request most recent head 8061956. Consider uploading reports for the commit 8061956 to get more accurate results

@@            Coverage Diff             @@
##           master     #230      +/-   ##
==========================================
- Coverage   55.44%   55.26%   -0.18%
==========================================
  Files          95       95
  Lines        8774     8750      -24
==========================================
- Hits         4865     4836      -29
- Misses       3909     3914       +5

Impacted Files
Coverage Δ

...gorithms/linfa-preprocessing/src/linear_scaling.rs
72.03% <0.00%> (-4.24%)
:arrow_down:

algorithms/linfa-bayes/src/base_nb.rs
62.96% <0.00%> (-3.71%)
:arrow_down:

algorithms/linfa-kernel/src/lib.rs
60.21% <0.00%> (-3.06%)
:arrow_down:

src/composing/multi_target_model.rs
64.44% <0.00%> (-3.00%)
:arrow_down:

algorithms/linfa-svm/src/permutable_kernel.rs
45.12% <0.00%> (-1.87%)
:arrow_down:

...linfa-clustering/src/appx_dbscan/cells_grid/mod.rs
49.12% <0.00%> (-1.76%)
:arrow_down:

algorithms/linfa-nn/src/balltree.rs
54.33% <0.00%> (-1.58%)
:arrow_down:

...lgorithms/linfa-clustering/src/optics/algorithm.rs
48.82% <0.00%> (-1.47%)
:arrow_down:

algorithms/linfa-hierarchical/src/lib.rs
41.66% <0.00%> (-0.59%)
:arrow_down:

algorithms/linfa-linear/src/glm/mod.rs
58.02% <0.00%> (-0.52%)
:arrow_down:

... and 22 more

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 d4bd9c9...8061956. Read the comment docs.


Issue with git config user.name

I tried the most recent version, both using copier and the PyPI package and I get an error when configuring the git user name. I suspect it is because my name is 3 words so there should be quotation marks somewhere.
Full error log:
 > Running task 1 of 3: git init -b main
Initialized empty Git repository in /home/brosaplanella/battery_modelling/supercapacitors/.git/
 > Running task 2 of 3: git config user.name Ferran Brosa Planella
error: no action specified
Traceback (most recent call last):
  File "/home/brosaplanella/.local/bin/copier", line 8, in <module>
    sys.exit(copier_app_run())
             ^^^^^^^^^^^^^^^^
  File "/home/brosaplanella/.local/lib/python3.11/site-packages/plumbum/cli/application.py", line 638, in run
    inst, retcode = subapp.run(argv, exit=False)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/brosaplanella/.local/lib/python3.11/site-packages/plumbum/cli/application.py", line 633, in run
    retcode = inst.main(*tailargs)
              ^^^^^^^^^^^^^^^^^^^^
  File "/home/brosaplanella/.local/lib/python3.11/site-packages/copier/cli.py", line 281, in main
    return _handle_exceptions(inner)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/brosaplanella/.local/lib/python3.11/site-packages/copier/cli.py", line 70, in _handle_exceptions
    method()
  File "/home/brosaplanella/.local/lib/python3.11/site-packages/copier/cli.py", line 272, in inner
    with self._worker(
  File "/home/brosaplanella/.local/lib/python3.11/site-packages/copier/main.py", line 228, in __exit__
    raise value
  File "/home/brosaplanella/.local/lib/python3.11/site-packages/copier/cli.py", line 279, in inner
    worker.run_copy()
  File "/home/brosaplanella/.local/lib/python3.11/site-packages/copier/main.py", line 832, in run_copy
    self._execute_tasks(self.template.tasks)
  File "/home/brosaplanella/.local/lib/python3.11/site-packages/copier/main.py", line 323, in _execute_tasks
    subprocess.run(task_cmd, shell=use_shell, check=True, env=local.env)
  File "/usr/lib/python3.11/subprocess.py", line 571, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'git config user.name Ferran Brosa Planella' returned non-zero exit status 129.

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Personally I think we should remove the git config commands entirely so that we can let them be inferred from the global config, what do you think, @santacodes?

Personally I think we should remove the git config commands entirely so that we can let them be inferred from the global config, what do you think, @santacodes?

That would be good, but in the case of a user who hasn't set up their credentials how would it work for them? I guess it wouldn't let them commit without a username and an email. Maybe we could do something like infer if they have a global username and email, if they don't we can configure them with the ones they prompted in copier.

@brosaplanella I don't believe it is a quotation mark issue because I remember I fixed it in this PR.

That would be good, but in the case of a user who hasn't set up their credentials how would it work for them? I guess it wouldn't let them commit without a username and an email. Maybe we could do something like infer if they have a global username and email, if they don't we can configure them with the ones they prompted in copier.

I would think anyone working on a project on a VCS platform would have credentials set up. If they don't, at that point, it lies outside the scope of pybamm-cookie as a "project generation" tool. I opine that removing the commands entirely would be better, since there are no extra steps needed for inferring their global defaults.


[DISCUSSION] Indent using tabs?

This is a kind-of selfish request, but would it bother you if I unified all the indentation to using tabs and fix the remaining codestyle checks?
I know that 4-spaces is preferred by PEP-8 over tabs, but I like the extra flexibility of choosing the current indentation size based on context rather than hard-coding some arbitrary size.
Would that be OK for you? (@AlibabasMerchant)

I don't mind the change, but then wouldn't it be better to just stick to the standard PEP-8?
(Btw, you got my username wrong in the @mention 😄 )

Damn! I noticed that first b… :grin:

I don't mind the change, but then wouldn't it be better to just stick to the standard PEP-8?

The great thing about using tabs is that your average Python developer can view them as 4 spaces just like he always did. I actually view them as 4 spaces most of the time, but when working on small screen (3-way merge anybody?) reducing tab size to 2 (or even 1) can be very helpful.
On the other hand, I'm still sometimes have a hard time reading 2-space indented (Google convention) deeply nested JavaScript code – It's often very difficult for me not to mix up returns statements of different levels and such, but apparently most JS devs either don't care or prefer it that way? Had they chosen tabs, I would have to care. Similarly some developers may prefer other indents at other times and with tabs we don't need to argue over the indent size as it can be both at the same time.
^ So that's why I much prefer tabs for pretty much any use-case. Judge for yourself. :stuck_out_tongue_winking_eye:

OK, we need a decision here. I did a lot of library restructuring for async/await and having first-class binary streams and it currently uses tabs.
Let's just vote on this then:

I said: We (or mostly me anyway) want this
@AliabbasMerchant says: Don't really care, but we should stick to the recommendations on PEP-8

@Seenivasanseeni please read the above and give your vote on this, thanks!

I vote for async/await and PEP-8.

Tabs or spaces?

spaces

@alexander255 I suppose we can close this huh? 😎


WIP: feat: add schematics for @ngxs/store

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/ngxs/store/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

https://github.com/ngxs/store/issues/35
What is the new behavior?
Installation
Install Angular CLI
You should be using<EMAIL_ADDRESS>or newer.
npm i -g @angular/cli

Install NGXS Schematics
npm i -g @ngxs/schematics

Usage
Creating a new project
To generate new Angular project with NGXS Store, you can use ng new with @ngxs/schematics specified as the schematics collection.
ng new --collection=@ngxs/schematics my-app

Create a NGXS Store
To generate store with @ngxs/schematics:
ng generate @ngxs/schematics:store --name todo

Result:
CREATE src/todo/todo.actions.ts
CREATE src/todo/todo.state.ts

Or with spec:
ng generate @ngxs/schematics:store --name todo --spec

Result:
CREATE src/todo/todo.actions.ts
CREATE src/todo/todo.state.spec.ts
CREATE src/todo/todo.state.ts

Create a NGXS State
To generate state with @ngxs/schematics:
ng generate @ngxs/schematics:state --name todo

Result:
CREATE src/todo/todo.state.ts

Or with spec:
ng generate @ngxs/schematics:state --name todo --spec

Result:
CREATE src/todo/todo.state.spec.ts
CREATE src/todo/todo.state.ts

Create a NGXS Actions
To generate state with @ngxs/schematics:
ng generate @ngxs/schematics:actions --name todo

Result:
CREATE src/todo/todo.actions.ts

NGXS Starter Kit
Usage
To generate store with @ngxs/schematics:starter-kit:
ng generate @ngxs/schematics:starter-kit

Result:
CREATE src/store/store.config.ts
CREATE src/store/store.module.ts
CREATE src/store/auth/auth.actions.ts
CREATE src/store/auth/auth.state.spec.ts
CREATE src/store/auth/auth.state.ts
CREATE src/store/auth/model/auth.model.ts
CREATE src/store/dashboard/index.ts
CREATE src/store/dashboard/states/dictionary/dictionary.actions.ts
CREATE src/store/dashboard/states/dictionary/dictionary.state.spec.ts
CREATE src/store/dashboard/states/dictionary/dictionary.state.ts
CREATE src/store/dashboard/states/dictionary/model/dictionary-response.model.ts
CREATE src/store/dashboard/states/user/user.actions.ts
CREATE src/store/dashboard/states/user/user.state.spec.ts
CREATE src/store/dashboard/states/user/user.state.ts
CREATE src/store/dashboard/states/user/model/person.model.ts

Does this PR introduce a breaking change?
[ ] Yes
[x] No

WIP

Hi, I have a question regarding to this feature. Would it be possible to have a sorter command line ? Something like ng generate ngxs (inspired from nrwl ). In my opinion, @ngxs/schematics is kind of long to type.

Create issue please


Request: disable retweets (and /quote tweets/comments/likes) at home timeline

Separate option for disabling at home timeline. Current retweets hider option disables retweets at user profiles only.
Also: exempt disabling retweets from users personal profile. As current options disables retweets from view in your own profile too.

OK!
I'll make the features.
(But perhaps I cannot make perfect feature.)

👌it will be good and development continues


Reduce overhead by changing alignment

Signed-off-by: AlanDiaz<EMAIL_ADDRESS>Commit Message: Reduce overhead via alignment
Additional Description: Moved smaller values like enums, int32 and bools to the end of the class to save on memory via alignment
Risk Level: Low

@adisuissa @yurykats

LGTM. Out of curiosity, do you have sizeof number before and afer?

@yurykats here's the size of comparison
New sizeof 2632
Old sizeof 2656

LGTM, thanks!

/retest


Fast running strangely

https://github.com/flaviovdf/klkmeans/blob/90bbe05483e13865afcdc1fd6e0ce89f07eccec3/klkmeans/kmeans.py#L59
I think the code above should be:
C = _compute_centroids(X, assign, num_clusters)

But then there is no convergence.

Hi,
I'll have a look, it does look like there is a bug.
--
Flavio
On Mon, Apr 23, 2018 at 6:11 AM, Aleksey Ropan<EMAIL_ADDRESS>wrote:

https://github.com/flaviovdf/klkmeans/blob/90bbe05483e13865afcdc1fd6e0ce8
9f07eccec3/klkmeans/kmeans.py#L59
I think the code above should be:
C = _compute_centroids(X, assign, num_clusters)
But then there is no convergence.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/flaviovdf/klkmeans/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAf08GKr3NYwIabgj2VZFsuYEVxGFtr7ks5trZrUgaJpZM4TfhI-
.


PubSub changes based on Issue feedback

Hello,
this should fix the pubsub issues users including me reported the last days
Regards
Syzuna

Hi! This looks good to me. Because it's a bit of a large commit and I haven't worked on pubsub recently, I'm going to leave it open for another day or two to allow others to review it as well and see if both you and I missed anything.

Dont worry. Take your time to review we both dont want bugs that could be solved beforehand ;)

looks good, sorry for delay


No media data if viewing with WHEP then stop then start too fast

It seems to has some bugs in key-value, where key-value event not received in case start too fast after stop

Same with https://github.com/8xFF/atm0s-media-server/issues/75


How to use 2D cartographer for the navigation of a robot

Hello, I have been running 2D cartographer successfully. Now I would like to use it in the navigation of my car. I have some questions about it. Could the cartographer use in the navigation? If so, how to do it in detail? I am new of the navigation, so I don't know how to do it. Any suggestions will be appreciation!
Thanks a lot.

@Duke-Allen, if you new to navigation you should go through the ROS navigation tutorial, http://wiki.ros.org/navigation/Tutorials, in detail (assuming you are using ROS).
It is possible to have Cartographer give out pgm/yaml maps like gmapping/hectorslam in the above tutorial. Save the Cartographer state in your mapping stage as a pbstream file and run cartographer_assets_writer to give you those files. An example is already here: https://github.com/googlecartographer/cartographer_ros/blob/master/cartographer_ros/launch/assets_writer_ros_map.launch

@precyon , thanks for your advice. I will try it, thanks again.

@Duke-Allen Did you have any success? I'm pretty much in the same situation as you, but can't seem to figure out a way to do it. Tried with navigation stack and move_base, which i've manage to get working by simulation. But can't seem to transfer it to my robot.

Closing as inactive.


[WIP] Enabling nested spans for trace mode OpenTelemetry

This PR enables nested spans for trace mode OpenTelemetry. Major changes are:

I moved InferenceTraceMode from server to core as TRITONSERVER_InferenceTraceMode
Added TRITONSERVER_InferenceTraceWithModeNew, to create a trace instance with provided mode + refactored TRITONSERVER_InferenceTraceTensorNew and TRITONSERVER_InferenceTraceNew to remove repeated code.
TRITONSERVER_InferenceTraceSetOpenTelemetryTracer and TRITONSERVER_InferenceTraceSetOpenTelemetryContext to pass OpenTelemetry's tracer and context to core's trace instance. These 2 objects are necessary for span creation (tracer) and storing info about currently active span and time_offset (created on the server side).
I tritonserver.h ran through cclang, so it has a lot of cclang related changes

Current logic : on the core side I create 2 spans: Request span, which has the same name as the model, this request is sent to, and compute span, which holds compute related timestamps and events and represents the amount of time compute took:
How it looks:

Note: this PR should be merged with server's PR: https://github.com/triton-inference-server/server/pull/5928

Working on the doc for discussions

Closing: moved this logic to server side


Cow: Watch topcashforcarscanberra.com.au

Cow requests the watch of the watch_keyword topcashforcarscanberra\.com\.au. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

topcashforcarscanberra\.com\.au has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by VLAZ in Charcoal HQ


Revert "Delete a template job after the job gets completed"

This reverts commit 61bfdd5fabe3530a16c8036d7cbc5aaa93f97c24.
We merged a fix for the original BZ we deleted any completed osd removal job after they were successful, So that while creating another osd removal job a check for the earlier completed job's deletion is not necessary. But we didn't know our docs had these steps where customer using encryption have a step where they depend on the logs of the successful job logs to get the PVC name for the replaced osd. They would use this PVC name to clean up the dmcrypt mapping on the node.
Updating the docs for so many types of platforms at so many places feels risky & anyway, this fix is not adding much value instead just creating more issues. So we should revert the fix.
Ref-https://bugzilla.redhat.com/show_bug.cgi?id=2248832

/cherry-pick release-4.15

/cherry-pick release-4.14


How to get updated endpoint of small Arcs in Elliptic Arc after changing origin

var makerjs = require('makerjs');

var arc = new makerjs.models.EllipticArc(289, 251, 75, 100);

var arc_0Path = ['models', 'Curve_1','paths', 'arc_0'];
var arc_0_1 = makerjs.travel(arc, arc_0Path).result;

console.log(makerjs.point.fromPathEnds(arc_0_1));

makerjs.model.move(arc, [10, 10]);

var arc_0_2 = makerjs.travel(arc, arc_0Path).result;

console.log(makerjs.point.fromPathEnds(arc_0_2));

this.models = {model: arc};

This is simple elliptic arc, path end coords for arc_0 is same even after changing the origin of Model. How to get the updated endpoints of small arcs after model is moved.

You may want to look at originating.

Thanks @danmarshall


Expose multiple merkle proofs

References https://github.com/tendermint/merkleeyes/issues/30

These have to be available through the abci_query interface.
Support non-existence for keys type
Add other paths- "/list", "/first", "/last" with proofs for other query types

Maybe we need to expand the abci interface to do so properly for range queries: https://github.com/tendermint/abci/issues/94

closed in favour of #341


Using DefinePlugin to define import throws an error

Bug report
What is the current behavior?
Have a webpack config that includes this in its plugins:
			new DefinePlugin({
				// defined to access the native browser import, which is ordinarily
				// prevented by (1) TS compiling import to AMD require(), and (2)
				// to its own native module loading. This bypasses both.
				__non_webpack_import__: 'import',
			}),

Upon compilation, you'll get an error
SyntaxError: Unexpected token (1:7)
    at Object.pp$4.raise (C:\Users\Connor\Documents\Github\vscode-dev\client\node_modules\acorn\dist\acorn.js:3460:15)
    at Object.pp$9.unexpected (C:\Users\Connor\Documents\Github\vscode-dev\client\node_modules\acorn\dist\acorn.js:756:10)
    at Object.pp$5.parseExprImport (C:\Users\Connor\Documents\Github\vscode-dev\client\node_modules\acorn\dist\acorn.js:2854:12)
    at Object.pp$5.parseExprAtom (C:\Users\Connor\Documents\Github\vscode-dev\client\node_modules\acorn\dist\acorn.js:2829:21)
    at Object.pp$5.parseExprSubscripts (C:\Users\Connor\Documents\Github\vscode-dev\client\node_modules\acorn\dist\acorn.js:2627:21)
    at Object.pp$5.parseMaybeUnary (C:\Users\Connor\Documents\Github\vscode-dev\client\node_modules\acorn\dist\acorn.js:2593:19)
    at Object.pp$5.parseExprOps (C:\Users\Connor\Documents\Github\vscode-dev\client\node_modules\acorn\dist\acorn.js:2520:21)
    at Object.pp$5.parseMaybeConditional (C:\Users\Connor\Documents\Github\vscode-dev\client\node_modules\acorn\dist\acorn.js:2503:2

If the current behavior is a bug, please provide the steps to reproduce.
See above

What is the expected behavior?
It should not throw an error.
This is due to the recursion check in the DefinePlugin

...the JavaScript evaluator tries to parse it as (import), which is not valid.

Perhaps if the parsing fails, the code should be emitted as-is?
Other relevant information:
webpack version:  5.68.0
Node.js version: v16.13.2
Operating System: Windows 10
Additional tools: N/A

import is a reserved word, not sure that this must works. you can use/define your own require


Very hard to see Geary mail list

Please check

Can you tell me where or which file I have to look in to fix this issue? I have spend time doing hit and trial but couldn't figure out any change happening.
Edit 1: There is no debug inside org>gtk>settings>debug in Freya to enable gtkinspector..I don't know how to get debug in the hierarchy.
I have included apps.css under gtkwidgets.css in the fashion done in default theme but haven't seen any change

To get the debugging app inside a GTK3 app, you have to run the app via the terminal with a special flag (something like GTK_DEBUG=true or something like this, don't remember now, better to google it), then press Ctrl+Shift+D IIRC.

#71


Updating eth_getTransactionReceipt to add the effectiveGasPrice

What

Updating the addition of the field effectiveGasPrice on the eth||_getTransactionReceipt RPC method, that it's the same value of the gasPrice on the transaction.

Why

We are updating the RSKj node adding this field and want to let the documentation updated and synchronised with this change.

Refs

Please, don't merge until we effectively release this feature with the RSKj, we will keep you posted. This is the PR associated, when we release the rskj v6.4.0, we will update this PR and convert it from a Draft to an effective one.

This devportal isn't used anymore, closing this pull request.


Subsets across greenwich meridian failing - need to explore

Subsets across greenwich meridian failing - need to explore

issue in clisops:
https://github.com/roocs/clisops/issues/35

issue in clisops:
https://github.com/roocs/clisops/issues/35

Duplicate

Duplicate


Allow MAPL to overwrite checkpoint restarts

I am running into a problem that leads me to believe that the checkpoint restarts cannot be overwritten … I recall there was a similar issue w/ history sometime back, and you pointed me to a flag in history that allows history to (use its old behavior) to overwrite its nc4 output files, this is the toggle:
Allow_Overwrite: .true.
I am wondering if the checkpoint writing suffers from the same restriction, or rule, that then cannot be overwritten by default. If that is so, is there an equivalent statement that can be put in AGCM.rc that tells MAPL to allow for overwrite?
CONTEXT: when the ADAS ensemble crashes, all that's typically needed is for the job to be re-submitted and the ensemble picks up from where it left;  it re-does only what hasn't completely successful in the original run. In case a member GCM crashes past the time it had written checkpoint files ... a re-submit will try to overwrite the checkpoint files previously written; with the present MAPL restriction of not allow restarts to be overwritten it is impossible to use the checkpoint mechanisms of the ensemble ADAS ... it requires re-running the who cycle from scratch.
The version of MAPL used in the 5.41.x ADAS is: v<IP_ADDRESS>  it would be nice to have the option added on top of this; and carried forward to more recent versions of MAPL.

@rtodling I'm implementing this, and once I've tested it, and do a PR into our develop branch I can back port this into a v<IP_ADDRESS>, I guess we would do a v<IP_ADDRESS>


View clipping error when zooming after moving graphic

Describe the bug
In the following video, notice that I can zoom in very close. After moving the graphic it then clips too soon as I zoom in.

Here is a simpler model that shows the problem with a cylinder and using only iTwinjs Edit Tools.

If I close that model and open another model or restart the app, zooming is correct again. Turning EditScope off and on again does not help.
To Reproduce
cylinder.zip has the bim file in the second example. I placed the cylinder using iTwin Edit Tools. I used the keyin "editor move elements" to move the cylinder.

Open the example bim with an App that has the Edit Tools enabled. I tried in Display-Test-App but it errors with "Key-in Failed to run"
Use the keyin "editor move elements" to move the cylinder to other locations
There is a certain distance from the original location when zooming in close clips too soon.

iTwin.js: 4.3.0-dev.25
iTwinUI: 2.11.10
AppUI: 4.6.1
React: 17.0.2
Node.js: 18.16.1
Electron: 26.0.0
Build Tools Commit: 842412e

Are you moving the element outside of the initial project extents?

It is within the project extents.

Notice project extents are displayed. I moved the right cylinder until zoom started clipping too soon. I then placed the right cylinder and recorded.

How small is your geometry? (meters)

About 8' tall with 2.5' radius. 1.6 m high with 0.77 diameter.

About 8' tall with 2.5' radius. 1.6 m high with 0.77 diameter.

I wonder if that is too small of a size for the frustum code. @pmconne what do you think? In the past when very tiny things (and only very tiny things) have been modeled in a .bim (sub-1-meter?), it is a known shortcoming that the the frustum can go bad (in a similar way to what is shown above).

@pmconne what do you think?

I think @matthaleky described a synchronization problem. He notes the problem begins after making an edit, and is resolved when reloading the iModel. Presumably the frustum is not being adjusted to account for the geometric changes he made to the model.

@matthaleky I cannot reproduce this currently with a move command that I implemented in DisplayTestApp.  Does this problem still occur in whatever App you were using?

Sounds promising, we are currently using version 4.7.0. What version are you using?

@MarcNeely, I  cannot run any editing commands in display-test-app.(except for "projectLocationTool"). Please tell me how to run the move command in the display-test-app. Also, please send me the imodel you are using.

https://github.com/user-attachments/assets/7ed7c33a-e76f-4aa8-a525-f020d7812288
In the video, I am using the Editing Studio-App, I can give you the steps to use it, if needed. I placed the two circles, moved the one on the right and now it disappears before the other circle when zooming in like the previous videos above. In the background are normal sized real world geometry and I am well within the project extents. Perhaps it looks subtle in this video, i only did one simple move, after doing more moves, the effect becomes more dramatic.

OK, I think that I've been able to reproduce this in display-test-app now.  When you do the zooming in, are you still in edit mode?

Yes, if "edit mode" = "edit scope".

OK, I think that I have tracked this down.  It looks to me like the bounding ranges are all OK, but the bounding sphere of the temporary tiles is not getting updated, so it's throwing the tile out if the un-updated bounding sphere is not in the view.  I'll look at updating the bounding sphere properly as the ranges are updated.

Thank you for the update.


[Bug] New-DbaConnectionString breaks at some point

Something is up here. I'm looking into the differences in connstring between Microsoft.Data and System.Data connstrings and this was the output
New-DbaConnectionString -SqlInstance sql2017 -SqlCredential blah -AccessToken adfasdf -ApplicationIntent ReadOnly -BatchSeparator "GO" -ClientName blah -ConnectTimeout 10 -Database beep -EncryptConnection -FailoverPartner test -LockTimeout 10 -MaxPoolSize 10 -MinPoolSize 10 -MultipleActiveResultSets -MultiSubnetFailover -NetworkProtocol AppleTalk -NonPooledConnection -PacketSize 10 -PooledConnectionLifetime 10 -SqlExecutionModes CaptureSql -StatementTimeout 10 -TrustServerCertificate -WorkstationId work -OutVariable string

;MultiSubnetFailover=True;Failover Partner=test;ApplicationIntent=ReadOnly;

This is true in dev as well.

gonna fix this in my SMO branch


added houmath edit

This is an edit of the excellent houmath notebook, adding in median filtered properties.
CV gives slight improvement so we shall see.

Oh, sorry, I went and merged this before realizing that your jovyan directory is in there too... the one time I don't check the files changed... Anyway, if that's a problem for you, maybe you can figure out how to purge them?

OK, thanks for this guys. The Prediction_houmath_edit.csv scores 0.599. This is not the stochastic version though... I will not get to that until the morning. Or if you want to implement --- just make a list of 100 y_pred vectors and dump it out to a NumPy file.

I dont even know what a jovyan is ^_^ Is this still an issue?

IDK, I just deleted it from my repo... I think it's something to do with Jupyter... or maybe this?

Possibly,  I wasn't aware that it is even in there. Thanks for dealing with it. I'll double check my git add next time.


Error when switch ENVIRONMENT_TEST to ENVIRONMENT_SANDBOX

Hi. I'm OK to run the quick-start app when using ENVIRONMENT_TEST mode.
But while I change to ENVIRONMENT_SANDBOX mode, the android pay says I didn't attach a vaild card . I did nothing except for changing to ENVIRONMENT_SANDBOX mode. Is there any other configuration I need to change?

I copied the most core payment code from quick-start to myself test app

You should always be using ENVIRONMENT_TEST, the SANDBOX environment is used more for internal testing and may not be reliable.


Issue 600 - JointJS

Description
This does not resolve the issue 600 completely but it provides a lot improvement

Editable JointJS object added
JointJs for state machine (from umple code to graph)

Tests
no test has been added.

I fixed most of the serious bugs that you mentioned. However, there are still some left (for example: zoom feature).  @vahdat-ab
I cleaned up the code and added the commented. JointJS is generally working and I think other bugs can be fixed as separate issues and now it is a good time to submit a PR. I will document the unfixed bugs/features in issue600 later today.

Thanks for the PR.
I see a major issue with paper (you used in jointJS).
It's not usable in many ways. I cannot move the paper to arrange big state machines. The state machines are not drawn automatically in the middle of paper so they are invisible partially. You can check them by just explore examples we have in Umpleonline.
Without these simple usability features, still, we cannot move to Joint.
Could you please check all examples and make sure the way to put elements on the paper is correct. If it's big so we should be able to got to right or left to see them. I could not make it. maybe it's there and I don't know.

I just checked and it happens to class diagrams as well. For example, Access Control

I just tested and I saw the problem. The object (especially the link) is touching the left edge of the paper, which make the edge harder to see. I will fix it.

I made some changes.
This time, I checked all Class Diagram using JointJS, they are all working fine except 'Geometric system' which does not work with Editable Class either. (But works with GraphViz Class)
For state machines, now the objects are centered, and the recursively nested state machines are also displayed.
However, I am still having problems with the layout, if the state machine is big, the auto-layout is not working properly, I tried different approaches but still could not fix it.
"Home Heating System", "Security Light", "Traffic Light B" are not working because the JSON received from the server are not valid. But this can be fixed as a separate issue.


 No the arrow will be printed by beginning of the task and be overwritten with an check mark. I think this is ok, because it only speed up the old implementation with lees pain.
Old:

set space or arrow, depending on verbositiy
perform task
set arrow or check mark, depending of written lines

New:

set arrow
perform task
set check mark if no new lines were written

The result is actually the same.

Now it will be printing arrow instead of space?

I adopted this change to the other pull-request https://github.com/deployphp/deployer/pull/580
But yes it will now print this arrow at the beginning of the line, because here is the output of the normal output of the old implementation:
  Executing task A
\033[k\033[1A^M➤
A✔ Ok
  Executing task B
\033[k\033[1A^M➤
\033[k\033[1A^M✔

It prints Executing … than it moves the cursor up to the beginning of the line and overwrites the first char (the space) with an arrow and that happens once before the next output will be written.
At the end it will do the same trick if no other output was written.
The same happens in the verbose output with one exception, it already prints an arrow as first character:
➤ Executing task A
\033[k\033[1A^M➤
⤷ on [localhost]
A⤶ done on [localhost]
✔ Ok
➤ Executing task B
\033[k\033[1A^M➤
⤷ on [localhost]
⤶ done on [localhost]
✔ Ok

And this also happens in some error-cases because of the ugly workaround.
So my change will always print an arrow at the beginning because it will do so anyway and removes the ugly extra line though.
So the output looks like this in the normal output:
➤ Executing task A
A✔ Ok
➤ Executing task B
\033[k\033[1A^M✔

And like this for the verbose one:
➤ Executing task A
⤷ on [localhost]
A⤶ done on [localhost]
✔ Ok
➤ Executing task B
⤷ on [localhost]
⤶ done on [localhost]
✔ Ok

I think that is much cleaner, actually in the verbose version shouldn't stay any cursor movements!
I used this recipe to test this:
<?php
task('A', function(){write('A');});
task('B', function(){});

task('AB', ['A', 'B']);


Manually bump catalyst-api version (to avoid Mist version bump)

We're keeping Mist pegged to a version for now

i'd recommend updating the release value to a tag (or branch name) instead. commit value will get updated by the bot


Error "Codex CLI error: Invalid request - The model: code-davinci-002 does not exist"

Edit 3: looks like OpenAI shut down their Codex API (https://news.ycombinator.com/item?id=35242069). Apparently there was an e-mail but I never received one.
Also apparently the API is available through Azure (https://learn.microsoft.com/en-us/azure/cognitive-services/openai/how-to/work-with-code). Perhaps an alternative would be to use Azure?
Original:

I'm getting this error on each attempt to use the codex:
Codex CLI error: Invalid request - The model: `code-davinci-002` does not exist

I haven't dug into the code at all but I noticed all of the OpenAI beta URLs have been removed. In the install instructions most redirect but the engines listing URL returns 404 (https://beta.openai.com/docs/engines/codex-series-private-beta). The new URL is https://platform.openai.com/docs/models/codex. It still shows it's in private beta on the new URL.
Edit:
When I query a list of OpenAI engines available to my account, code-davinci-002 is not listed. So I reran the install script and select a different engine available to me (gpt-3.5-turbo). After restarting PowerShell I'm still seeing the same error. I confirmed that the openaiapirc file was correctly updated with the new engine and verified that all instances of PowerShell had been shut down but am still seeing the same error.
Edit 2:
I also updated the current_context.config file with the updated engine. Now the error I receive is
Codex CLI error: Invalid request - This is a chat model and not supported in the v1/completions endpoint. Did you mean to use v1/chat/completions?

It seems code-davinci-002 and code-cushman-001 have been removed, but yes, the Codex CLI does not seem to support gpt-3.5-turbo at the same time.

This fork is intended to fix this, https://github.com/Lukas-LLS/Codex-CLI. A PR from @Lukas-LLS would be great.

Note:

As of March 2023, the Codex models are now deprecated. Please check out our newer Chat models which are able to do many coding tasks with similar capability

as per:

https://platform.openai.com/docs/guides/code
Use gpt-3.5-turbo for best results! Once you get API access to GPT-4, move up to that, no point in using Codex models, like davinci anymore.

I'm still getting "cannot find openAI model" errors, even with gpt-3.5-turbo as my model.

I can think of two possible causes for your problem:

You might have an old version of the fork, because at the time my fork was posted into this issue I was still working on it and at that point it was not operational (If you update to a newer version make sure to use a cleanup script and setup thereafter, because there a some changes that will break an older setup)
It could also be possible that you still have the setup from https://github.com/microsoft/Codex-CLI. If that is the case, you should run the cleanup script from the original project before running the setup from the fork. (I don't know how you migrated to the fork, so this is just a possibility)

And for gpt-4 make sure you have the model available to you, for that to be the case you must have either signed up on the waitlist: https://openai.com/waitlist/gpt-4-api (from the waitlist you gain access to the gpt-4 model) or you must have the ChatGPT Plus subscription: https://chat.openai.com/chat in the lower left corner the button Upgrade to Plus (from ChatGPT Plus you gain access to the gpt-4-32k model)
If your issue still persists after these steps, let me know and I will look further into it

@Lukas-LLS , could you consider submitting a PR to the official project?

I already have submitted a PR #131

@loftusa I have found and fixed your problem. It was limited to the zsh_setup.sh script, therefore I did not find it immediately. The problem was that in some cases the variable name of modelId was modelID. That was due to me carelessly refactoring the name without checking it afterwards. The problem should be fixed with the latest commit.

I also found out that the zsh_setup.sh does only work for macOS and not for zsh under normal Linux due to different implementations of the sed command under these two different operating systems.

Support for Chat models (GPT 3.5/4) now works on my fork!
Feel free to use it here:
https://github.com/AntonOsika/CLI-Co-Pilot
The required changes in the code were small but non-obvious.

@AntonOsika , for Bash, your fork is inserting an space at the beginning of the generated commands preventing them to be stored in the history. Can I change this behavior by configuration?

Thanks for pointing this out!
Would love if you submit a MR to make it depend on which shell is being
used.
On Sat, Apr 8, 2023 at 2:48 AM Jaime Hablutzel @.***>
wrote:

@AntonOsika https://github.com/AntonOsika , for Bash, your fork is
inserting an space at the beginning of the generated commands preventing
them to be stored in the history. Can I change this behavior by
configuration?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/Codex-CLI/issues/129#issuecomment-1500745745,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABCCSUK2OAAQL6VWASFMO6TXACYUNANCNFSM6AAAAAAWGZRAWQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Anton Osika

@loftusa I have found and fixed your problem. It was limited to the zsh_setup.sh script, therefore I did not find it immediately. The problem was that in some cases the variable name of modelId was modelID. That was due to me carelessly refactoring the name without checking it afterwards. The problem should be fixed with the latest commit.

I also found out that the zsh_setup.sh does only work for macOS and not for zsh under normal Linux due to different implementations of the sed command under these two different operating systems.

I found that after pressing ctrl+G, the generated commands will only be displayed after the issue I raised.like this:
what is running on port 3306sudo lsof -i :3306

That was the original behavior of Codex-CLI. I found that `# Your comment here` && had worked for this behavior, although I did not like that way of writing a prompt. I have now changed that way the command is insert in bash and zsh to match Powershell allowing to write normal comments then hitting Ctrl+G and then the command appearing in a new line below the comment. For this to take effect you must update your local repository.

That was the original behavior of Codex-CLI. I found that `# Your comment here` && had worked for this behavior, although I did not like that way of writing a prompt. I have now changed that way the command is insert in bash and zsh to match Powershell allowing to write normal comments then hitting Ctrl+G and then the command appearing in a new line below the comment. For this to take effect you must update your local repository.

Now it can run great. Thank you.🙏

Support for Chat models (GPT 3.5/4) now works on my fork!
Feel free to use it here:
https://github.com/AntonOsika/CLI-Co-Pilot
The required changes in the code were small but non-obvious.

getting this error in zsh:
Codex CLI error: Unexpected exception - module 'openai' has no attribute 'ChatCompletion'
do you know how to fix it?

do you know how to fix it?

Try updating the openai package with pip.

@hablutzel1 looks like it works, thanks!!!


Fix the origin handling in the pkg/utils/managedresources.NewForShoot func

How to categorize this PR?

/area quality
/kind bug
What this PR does / why we need it:
https://github.com/gardener/gardener/pull/7162 introduced the origin arg to the pkg/utils/managedresources.NewForShoot func.
https://github.com/gardener/gardener/blob/f32aca696063cfd75fdc8ea58dbb9be044ae9200/pkg/utils/managedresources/managedresources.go#L79-L90
But currently this origin arg is unused and origin=gardener is always passed as label to the underlying ManagedResource.
Which issue(s) this PR fixes:
See above.
Special notes for your reviewer:
N/A
Release note:

So far the `github.com/gardener/gardener/pkg/utils/managedresources.{NewForShoot,CreateForShoot}` funcs were ignoring the passed `origin` argument and were always using `gardener` as value. These funcs will now respect and use the passed `origin` value.

/cc @dimityrmirchev


Maintain peer list without developer maintenance

disonnected_peers and accept_new_connections are something you have to remember to call to maintain the peers list.
This could be handled a different way which is a lower cost for the developer.

Potential solution A: HashMap.
Proposal A
Change WebRtcSocket.peers from a Vec<PeerId> to a HashMap<PeerId, Instant>. This would actually just be a free optimization, since we are only using the .remove(), .push(), .clear(), and .extend() operations on peers.

Insert
Delete
Clear

Vec
O(1)
O(n)
O(1)

HashMap
O(1)
O(1)
O(1)

But wait - Instant? That would be the last "keep alive" transmission for such peer. :)
This would, however, move that overhead to connected_peers and disconnected_peers. Checking for disconnections would always be a O(n) task, filtering peers who have exceeded a timeout.
Proposal B
This proposal builds on Proposal A to reduce the time complexity of checking for connected/disconnected peers to O(1).
Start by implementing Proposal A.
Next, we'd augment WebRtcSocket with transmissions: BinaryHeap<Transmission>, a priority queue with a type envelope containing the PeerId and Instant we receive a transmission.
struct Transmission {
    peer: PeerId
    arrival: Instant
}

For each received transmission, we insert a Transmission { peer, arrival: Instant::now() } into the pq, prioritized by arrival. Insert is technically O(logn) in a pq, although we can assume Θ(1), since all cases except unordered arrival would be O(1).
At the time of insert, we dually peek the bottom of the heap, checking if its transmission was longer than the timeout. 1 insert and 1 potential erasure happen, amortizing the cost of maintenance. If the peeked element was beyond timeout, we double-check the peer hashmap to see if there was a more recent transmission. If there wasn't (or the peer DNE), the peer disconnected through timeout. We remove the peer.
In summary, we would need 2x the memory capacity for peers roughly, but we'd end up with with a nearly constant amortized total maintenance package for retrieving disconnected and connected peers.

@johanhelsing requesting thoughts on Proposal A?

Would this this completely replace the PeerLeft events way of hanlding disconnections?
I think it's important to think about how we present the various kinds of disconnection in terms of API. I.e. are we still reporting peers as disconnected if they lose their connection to the signalling server but keep their data channels to us open? Should peers drop all connections to their peers if they get disconnected from the signalling server?
For my use-case, run-time complexity doesn't matter much, since n = 5 at most. SmallVec would probably be more appropriate there, but either way, it doesn't matter much, and I'd much rather have ok performance across the board, so TL;DR: I'm fine with HashMap :)
Before work is started on this, maybe it makes sense to sketch out what the resulting API would look like?

PeerLeft would still exist, but only sent on explicit socket close.
Channels would not close unless there was a PeerLeft event.

PeerLeft would still exist, but only sent on explicit socket close.

Agree.

Channels would not close unless there was a PeerLeft event, or timeout was reached.

Instant would be the last "keep alive" transmission for such peer.

I'm curious about how you envision timeouts... What exactly is it we would be timing out here? Do we need to add new types of messages to send between peers through the data channel(s)?

FWIW, tower has a Discover service trait for maintaining lists of servers. Might be interesting to use or crib designs off of:
https://docs.rs/tower/latest/tower/discover/index.html

I have a disposition to close this because #117 is going in the right direction.
I think ultimately what would be nice is if there was only one tick method to receive events, explicitly, or the ability to add callbacks to those events.


Future feature documentation

Document has been created. I'd like to go over it with the whole team.

https://docs.google.com/document/d/1uaOfZml8I-TlJ18ZBR_ZOeLrwPmoz-PcRnaAxhd6pWk/edit#
https://docs.google.com/document/d/1LJulp01psc9EGTEUqdgHqXcWGXUyXL7274BsMFoh4_8/edit#heading=h.gne3t9kpg1ry