refactor(rust): avoid using non-borrowing cbor fields

Proposed Changes
Refactor ockam_command & ockam_api so that CBOR structs avoid using non-borrowing fields and make use of CowStr and CowBytes. This is all in aid of making the code base more consistent and, hopefully, memory efficient.
Also added impl From<&String> for CowStr to help reduce repeated code.
Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository.

Thanks @adrianbenavides. I've done the changes.
I have to admit, I don't 100% understand what's going on with the b/n attributes.

Thanks @adrianbenavides. I've done the changes.

Great, thanks @neil2468, I'll merge the PR shortly!

I have to admit, I don't 100% understand what's going on with the b/n attributes.

Their purpose is just to define whether a type should borrow or clone when decoding from a CBOR buffer. As a rule of thumb, If a type has lifetimes, then we should use b so they can be decoded from the CBOR input without allocating new memory.


Color fix

Fixes #34 Fixed the colors to be decided by series item rather than Y-Axis. Also added a KAYAK color theme.

Coverage decreased (-0.004%) to 97.438% when pulling 3edbd5c3a1dddd295fb64bdc0a9ae6492be0a11b on color_fix into ee6048538cdc79ec78e718758d5f8ab17d651a57 on master.


Interest isn't applied to bank's money

Image
It's applied to users' accounts though.
Idk if it's intended or nah, but if it's like this, wouldn't the bank go negative if people withdrew all their money?
Is that how banks work :thinking:💭?

@Programmix might have a look at this, since he wrote the initial code for it.

Um that's not really how interest works. Interest is only applied to the money in the bank; the ledger.

@Programmix It adds money to the bank balance you see when you do ?donuts, but not to the total amount of donuts in the bank.
Say I'm the only user.
I deposit 1000 donuts.
The interest rate is 0.1%.
The interest is applied.
I do ?donuts, it shows 1010 donuts.
I do ?bank, it says the bank has 1000 donuts in it.
Is that how it's supposed to be?

Yes, yes it is.

So does the interest go negative when the bank has negative donuts?

When would the bank have negative donuts?

... when you withdraw the 1010 donuts when the bank only has 1000, maybe?

Check the code, that can't happen. Just like in real life, the bank won't let you withdraw more money than the bank has.

If it's a credit account, you can withdraw more than you have, but you need
to pay it back otherwise dire consequences.
On 3 Feb. 2017 5:27 am, "Programmix"<EMAIL_ADDRESS>wrote:
Check the code, that can't happen. Just like in real life, the bank won't
let you withdraw more money than the bank has.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/WeebDev/Commando/issues/38#issuecomment-277040571, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGAACwP8gzuZtDm3zLgGv5URe83FxX3fks5rYiAlgaJpZM4LynuP
.

Credit accounts or loans arent a thing yet iirc, @Programmix planned on doing it, but so far its not implemented yet, so I don't see it happening.

@Programmix So you're saying the bank is supposed to have a lot less donuts in it than the sum of all bank accounts' balances, with the difference increasing every time interest is applied?

Yes... that's how banks in real life work.

We just got to a point on the server where most people had "money in the bank", none "in hand" but the bank itself was dry. The bank needs a way to make money or the whole Commando economy won't really work.

That is how banks work. If every person makes a run on a bank and withdraws
all their money, banks get fucked. This is what happened during the Great
Depression, and the reason we have the FDIC. Your bank shouldn't be
applying interest to its own money, because then it'd be actually printing
money, lol. What they actually do is invest the money and use the profit
from returns on investments to keep the operation going and profits flowing.
On Mar 29, 2017 04:04, "rbJARB"<EMAIL_ADDRESS>wrote:

We just got to a point on the server where most people had "money in the
bank", none "in hand" but the bank itself was dry. The bank needs a way to
make money or the whole Commando economy won't really work.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/WeebDev/Commando/issues/38#issuecomment-290056927,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL4Bc95Hmj61dwj_7HW1cfCKuEhdERLoks5rqjqjgaJpZM4LynuP
.

Yes, I know what they do to earn money, but they don't do it here. So is
there anyway for it to earn money?
On Wed, Mar 29, 2017 at 10:03 PM, Buluphont<EMAIL_ADDRESS>wrote:

That is how banks work. If every person makes a run on a bank and withdraws
all their money, banks get fucked. This is what happened during the Great
Depression, and the reason we have the FDIC. Your bank shouldn't be
applying interest to its own money, because then it'd be actually printing
money, lol. What they actually do is invest the money and use the profit
from returns on investments to keep the operation going and profits
flowing.
On Mar 29, 2017 04:04, "rbJARB"<EMAIL_ADDRESS>wrote:

We just got to a point on the server where most people had "money in the
bank", none "in hand" but the bank itself was dry. The bank needs a way
to
make money or the whole Commando economy won't really work.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/WeebDev/Commando/issues/38#issuecomment-290056927,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AL4Bc95Hmj61dwj_
7HW1cfCKuEhdERLoks5rqjqjgaJpZM4LynuP>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/WeebDev/Commando/issues/38#issuecomment-290099812,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AN3igqYZN5xJUv71yuXU8u6QJitUn7I3ks5rqmS6gaJpZM4LynuP
.

--
Thanks! Harry

Yes, there are plans for banks to invest into virtual stocks or whatever, to keep the profits flowing.


Change ic_launcher

Overview (Required)
Current icon is temporary I created before.

I’ll give it a try.

Cool! 🆒 Thank you so much!
I want to change to the icon @wariemon created.
http://wariemon.com/post/151555749659/droidkaigi2017

OK, I tried making icon. Is it nice?

Looks good! Go ahead 👍


Error

If I generate code it says
Input How Many Codes to Generate and Check: 5
Error | https://discord.gift/6At2EU5DBMRo7zeqzxwUtTB
Error | https://discord.gift/E6ukKJJI3FbXHGifCnbYLBD
Error | https://discord.gift/6tDEZEkUH2TqhfHCMWyhv0X
Error | https://discord.gift/m0tOw2DCQqMxEW1uZpGH7Iq
Error | https://discord.gift/FT3ZolxRhRMKQ93HLdz0bTD

Did you install all the requrements and made sure you added Them to your PATH?

Did you install all the requrements and made sure you added Them to your PATH?

Yea still error


Data files

Hi. Where is the data files listed in the example "pew Global attitudes Spring 2014"?

Hi @mp1960
Good shout, I didn't add it to the repo as I don't have permission. Sorry I should have said, I'll add this to the readme. In theory you can use any spss file but if you want to use this file then the link is below. You may need to set yourself up with a free account.
https://www.pewresearch.org/global/dataset/2014-spring-global-attitudes/
The file is chunky and creates a very large json file. It might be better to experiment with a smaller file.


[darwin] Use DNSServiceReconfirmRecord for A and AAAA records to miti…

…gate stale cache issues
Problem
Sometimes there are stale DNS A and AAAA records. This patch instruct the daemon to verify the validity of such records. It does not prevent stale records to be used but reduce their lifetime and may help for subsequent pairing attempts.

I advise against doing this. This is not the intended use of this API. If deployed, it would likely generate so much Wi-Fi multicast traffic that it will make Matter highly unreliable.
What problem is this expected to solve? The description talks about “stale records” but it doesn’t describe the scenario that leads to those stale records, or how calling this API would change that scenario. Is there a bug report describing the problem that this change is intended to solve?

Here is more background on the intended use of DNSServiceReconfirmRecord(), which is this:

On Wi-Fi, multicast is very unreliable.
On Wi-Fi, unicast is highly reliable.
Therefore, if a client fails to connect to a service via unicast then that’s a good indication that the service is probably no longer there, so DNSServiceReconfirmRecord() is feedback from the client saying that it has determined with high confidence that the Multicast DNS data is wrong, and should be deleted.
The Multicast DNS state machine will attempt to verify this hint, but due to multicast packet loss there is a good chance that it will fail to detect if a client is lying when it says that it has determined that service is unreachable.
This API contract only works if the client has in fact determined that a unicast connection attempt was unsuccessful. If the client wildly calls DNSServiceReconfirmRecord() for every record it knows about without actually determining via unicast that the data is stale, the end result is lots of valid data being deleted, causing lots of failures to connect.

I'd still be interested to see a bug report describing the problem that this change is intended to solve. It is always possible that we have some other bug here, and we can help diagnose that and maybe work out a real fix for the underlying cause.

I advise against doing this. This is not the intended use of this API. If deployed, it would likely generate so much Wi-Fi multicast traffic that it will make Matter highly unreliable.
What problem is this expected to solve? The description talks about “stale records” but it doesn’t describe the scenario that leads to those stale records, or how calling this API would change that scenario. Is there a bug report describing the problem that this change is intended to solve?

Thanks for this answer and the additional details about the intended use of the API.
In this context, "stale records" refers to A and AAAA records in the mdns cache for a node that has been rebooted.
The source of the reboot can be because of a hard reset (e.g power loss), or a soft reset (FW update) with the node not properly clearing up the mdns records.
Since a single "best" IP is tried when establishing a session over UDP (PASE/CASE) it may fails if the "best" IP is one of the stale entry. Also, even if all returned IPs are tried, the session establishment will be slow reliable UDP is used and we will need to wait for 5 messages failures before attempting to connect to the next one (the time between retries varies depending on some values emitted by the node in the mdns advertisement).
I have updated this PR such that DNSServiceReconfirmRecord is only done when PASE session establishment timeouts during commissioning.
It does not completely solve the issue, since the session establishment will still fails, but it may help to remove the stale entry for the cache such that a second try a few seconds later returns a valid IP.

@andy31415 I am re-requesting review since the code has changed a lot since your last review.


Add CopyCommand and CopyCommandParser

CopyCommand is a Command that represents the Copy command which users can use in the UI.
'CopyCommandParser' is a Parser that is responsible for the parsing of arguments when the Copy command is used.
Users can quickly copy information of a person to their clipboard using the command without manually selecting it in the UI.
Closes #40

Maybe as a followup, we can write test cases for the new copy command. After all, if they can write test cases for both the list command and the help command, we can probably write test cases for the copy command too.

Maybe as a followup, we can write test cases for the new copy command. After all, if they can write test cases for both the list command and the help command, we can probably write test cases for the copy command too.

Yep. I will add in the test cases in a separate commit.


Add Sacred citation to README

Citation and corresponding link to the Sacred publication added to readme.rst

Coverage remained the same at 83.426% when pulling 65af86dafb27c849d98bb0d2fa5a845abf6e0b37 on mukkla:add_citation into d5d1a1a763c83ffc1cd88e60ff9df79499b3b875 on IDSIA:master.

Thanks!


make documentid decoder overload internal

No-longer-breaking change for #9465.

@andrewheard I think you made a bunch of changes in this area that obsolete this PR. Can I close it?

Thanks for cleaning up old PRs, I need to do the same. I went ahead and closed it.


Improvements for Container class section

To be pulled after https://github.com/Illarion-eV/Illarion-Server/pull/85 has been pulled as it reflect those changes

Replaced with issue #27


Support Minecraft "local" coordinates

In 1.13, Minecraft has added Local coordinates, which is a third coordinate system (in addition to relative and absolute coordinates). It uses caret (^) notation, is centered on the player's head, and rotates with the head! Here is the wiki's 1-line explanation:

For caret notation, they are along axes originating in the center of the target's head and pointing leftwards, upwards, and forwards, respectively, changing with the head's rotation.

A few examples:

^0 ^0 ^0 is the player's head
^3 ^0 ^0 means 3 blocks to the left of the player's left "ear"
^0 ^0 ^5 means 5 blocks "forward" in the direction the player is currently looking

In my opinion, the best use of this is to change the default position to a block that says "In front of me" and have that map to something like ^0 ^0 ^5. That way, beginners don't have to learn relative and absolute coordinates to start placing stuff in an intuitive way. They could do things like "Place [dirt] [in front of me]" and that would make the block appear in a visible location no matter their position and orientation. Major win!
Wiki paragraph on caret notation
5-minute explanation video
And the good stuff... example of what you can do with this
Cowboom is in the past. This is the end game.

This is very nice indeed.

So great to see you on our repos Guillaume!!!  We miss you  :-)

Just merged in the block for this, will be supported after our next release :)


Value Error for ID Number starting with 000000

SouthAfricanIdentityValidate("0000000000000").validate() raises a ValueError. I would expect that is just returns False.
I will submit a PR soon

Thanks for catching that. I can also fix it. Busy adding a test for it

Thank you @c-goosen. I also have test cases in my PR


Major refactoring

This a large rework and cleanup:

JSON-LD and Hydra logic are now decoupled from the code classes (it should be possible to support other formats such as HAL now)
Filter and Operation are now classes instead of dirty arrays
See the services.yml file to see how operations and filters must be registered now
I've introduced a factory to build custom operations
The DataProvider system changed to be simpler and more extensible
Improvements and reorganization everywhere (removed some circular references, better SOLID respect)

However, this is also a huge BC break but I think that this cleanup work was necessary before the first release.
Speaking about releases, an option is to rename the current master branch to a 0.1.x branch and make this pr being the new master branch. That will allow existing projets to continue working but I'll prefer that everyone migrate its code. What do you think @sroze @deuzu @theofidry?

I've not updated README.md yet but the new syntax can be found here: https://github.com/dunglas/DunglasJsonLdApiBundle/blob/refactoring/features/fixtures/TestApp/config/config.yml

Currently we are using my fork (vcs repository) so even if there is any BC it won't disturb us for the time being. We will upgrade it later and given the current use we have, it should not take us too much time.

@theofidry cool. @sroze?

Btw, the Scrutinizr note has been greatly improved (+1.16 point): https://scrutinizer-ci.com/g/dunglas/DunglasJsonLdApiBundle/inspections/a603616f-a7b5-4b36-ad20-01d36f76b60e

Great ;)

Désormais : 1 ligne = 1 facture.

I left few comments but btw, it's a really great move :+1:

@sroze I've just pushed a refactoring of the filter system. And it remove the last circular dependency of the project :) Can you review it?

@dunglas Yeah, that's great with this filter implementation, I like it pretty much !

In you example you override existing endpoints, what becomes of adding a new one? In your last update of the doc there is no mention of registering the action in the services.yml (cf. Using a custom controller).

@theofidry what do you mind about adding new ones? Just create a new Resource to add a new endpoint or create custom Operation with a different path.
About registering custom controller, the way to do it has changed: https://github.com/dunglas/DunglasJsonLdApiBundle/pull/30/files#diff-04c6e90faac2675aa89e2176d2eec7d8R574

My bad, I read it too fast. Ok thanks, great enhancement!


Bug: Play from "Home  View"

Windows, Opera and Firefox, Latest Commits.
Good Afternoon
I have discovered that the play buttons doesn't work for the "Most Played Songs" & "Recently Played" section in the Home view once any song has been played.
To reproduce simply load the web page and play anything, then try to click the play button for any song in the "Most Played Songs" & "Recently Played" section and it wont work.
Note: if the song currently being played is in the "Most Played Songs" & "Recently Played" section you will be able to pause and continue playing that song from there but that song only.

Thanks for the report!


Remove version: CHIRPSoftware.CHIRP-next version 20240117

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://trac.chirp.danplanet.com/chirp_next/next-20240117/chirp-next-20240117-installer.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 912)


[pipeline] add check about api-version for multiapi

make sure each subfolder (like v2020_01_01) only has one kind of api-version

/check-enforcer override


Issue With saving Menu

Im having an issue where i create the first menu and save which is fine. then i create a nested item and save then it disappears. but when i look in the database it has saved it?
Any ideas why this might be? only i cant use the plugin right now with this behaviour.
Below are my versions:
node v14.19.1
npm 6.14.13
postgresSql 13
Dependencies
"@_sh/strapi-plugin-ckeditor": "^1.1.0",
"@strapi/plugin-graphql": "^4.2.3",
"@strapi/plugin-i18n": "^4.2.3",
"@strapi/plugin-sentry": "^4.2.3",
"@strapi/plugin-seo": "^1.7.4",
"@strapi/plugin-users-permissions": "^4.2.3",
"@strapi/provider-upload-cloudinary": "^4.2.3",
"@strapi/strapi": "^4.2.3",
"pg": "^8.7.3",
"pg-connection-string": "^2.5.0",
"strapi-plugin-navigation": "^2.1.0",
"strapi-plugin-placeholder": "^4.1.7",
"strapi-plugin-sitemap": "^2.0.8",
Any help greatly appreciated. I have also attached a video of what's happening. plus proof that it saved to the database but won't show in frontend admin UI
https://user-images.githubusercontent.com/25214970/179167209-a1894c59-a8e8-4f06-96fb-732901ea21bb.mov

• We are not receiving any errors
• I deleted the menus and tried again. here are the screen shots

• Yes it only happens with nested items NOT on top level menus
•It makes no difference if the menu is a wrapper/internal/external
What is interesting is that if i delete the main top level menu and save then the nested item thats in the database appears. please see below.
https://user-images.githubusercontent.com/25214970/179335435-16f8f69c-2571-428c-83e5-cda1470a75e5.mov
It seems to be also happening in SQlite

Whats interesting is I just completely created a new database from scratch using postgres and its not happening now. very strange

It looks like it was some kind of faulty database. I will assume that it was related only to the database issue and close this for now. If you ever happen to reproduce this bug again please feel free to reopen it 🙂.

I am having similar issue with v4.3.2 and mysql.

Choosing menu to edit (right top).
Creating first item.
Save.
Creating nested item. Click save in modal.
Save on the top right near chosen menu - disabled.
Reloading page - no saved items.

Console clean.

... tried to re-enable the topmost save button in console - no luck. looks like it has no click listeners binded to it.

Hi @rimallo.
This seems to be unrelated to the issue matt was having. I will look into your issue asap.
Could you please provide the plugin version that you are running?


Besu - Local permission (with Node Allowlist)

Hello Everyone,
I am using local permission concept to restrict the nodes. My environment is under AWS ECS and has DNS mapped. I have enabled --xdns-enabled & --xdns-update-enabled param in my network. Have a permissions_config.toml file with list of all enodes that can connect. While deploying the network, the permission file is got deployed into the container.  Even the DNS and IP mapping is also getting resolved after few mins of node start or when I do a permission file reload throught api call. Also I have the same permission config file for all the nodes.
When I pull the Allowlist through api call, it gives the result with all enodes along with DNS to connect with the node.
But if I do a net_peerCount, the result is 0x0, means the nodes are not connected with each other. There was no log with error. Then I have to a redeploy all the nodes to make it sync and then the peer count comes with exact number of nodes.
Any idea, why the behaviour is like this? Is anything I am missing here?
<@680245613258211336> We were able to check further on this and below is the scenario where discovery fails when we enable node local permisioning.
We are starting boot node with the initial list of peers having permission to network. The permission list is updated with Enode URL having hostname rather than IP. Also we have enabled BESU_DNS_ENABLED and BESU_XDNS_UPDATE_ENABLED parameters for this. As none of the Besu nodes are currently up and when we call the Perm method ( curl
-X POST - - data ‘{" jsonrpc": "2.0- "method":"perm getNodesAllowlist", "params": [], "id": 1}’ http://bootnode_hostname:8545 ) to list the permitted node list from boot node, its giving the enode addresses of all nodes with loopback IP instead of hostname which we passed in permission TOML file. In our environment, Besu nodes hostname is only resolvable after the node starts as we are using discovery to get the DNS configured.
Now if we start any of the besu nodes - it won't be able to communicate to bootnode for discovery as the permission list of bootnode is still with <IP_ADDRESS>. Post besu node starts and the corresponding hostname is resolvable - when we do a reload of permission list in bootnode - the enode URL of node will get updated with the hostname entry as we given in permission TOML file ( enode://enodeaddrss@DNS_NAME: p2p_port )
Even after the boot node permission list is updated with host entries

besu node is not able to do the discovery until the node is restated.

Following are our queries
How we can retrigger the connection to bootnode from besu nodes to do the discovery again?
Why the permission list is giving loopback IP when we are providing hostnames in the TOML file?
This has been raised in discord channel(Topic : Besu-Local permissioning issue) and as advised raising an issue here

Hey @gokulraj17687 ,
Can you also attach the logs for different scenarios.
Set log level to trace - https://besu.hyperledger.org/en/stable/public-networks/reference/api/#admin_changeloglevel
and then collect all the logs.
Thanks!

@macfarla There has been no reply from the reporter @gokulraj17687
Shall we close or wait for more time?

Closing issue, as no reply from the issue reporter.
@gokulraj17687 You can reopen the issue if you face the error again. Thanks!!


Added Frequency Plots

Pull Request Testing

[ ] Tested to see that it creates the correct output plot:

[ ] To test, check out branch feature_950_weather_regime_freq from METplus. The output should produce 4 plots, but only obs_freq.png is new. Run s2s/UserScript_obsEra_obsOnly_WeatherRegime.conf along with a system .conf file. Check that obs_freq.png matches the below plot. Should there be some kind of unit tests for this?

[ ] Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes or No]

[ ] Do these changes include sufficient testing updates? [Yes or No]

[ ] Will this PR result in changes to the test suite? [Yes or No]
No

[ ] Please complete this pull request review by [Fill in date].

Pull Request Checklist
See the METplus Workflow for details.

[x] Complete the PR definition above.
[x] Ensure the PR title matches the feature or bugfix branch name.
[ ] Define the PR metadata, as permissions allow.
Select: Reviewer(s)
Select: Organization level software support Project or Repository level development cycle Project
Select: Milestone as the version that will include these changes
[ ] After submitting the PR, select Linked issues with the original issue number.
[ ] After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
[ ] Close the linked issue and delete your feature or bugfix branch from GitHub.

@CPKalb I'm a little unclear one what is needed to do to test. Is there a use case in the METplus branch that calls the new function?

Hi George:
Yes.  feature_950_weather_regime_freq calls this.  It's still in it's own
branch because the plotting is needed for it to work.  You'll want to run
UserScript_obsERA_obsOnly_WeatherRegime.conf, along with a system.conf
file.  Does that help?  Do you need more information?
Tina
On Wed, Jul 14, 2021 at 10:54 AM George McCabe @.***>
wrote:

@CPKalb https://github.com/CPKalb I'm a little unclear one what is
needed to do to test. Is there a use case in the METplus branch that calls
the new function?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/138#issuecomment-880054053,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJHD4QB36POOL2ECNW4G2DTXW6ODANCNFSM5ALYW5ZA
.

--

Christina Kalb
Associate Scientist
National Center for Atmospheric Research
3450 Mitchell Lane
Boulder, CO 80301
@.***
303-497-2819

My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.

Sorry, that .conf file is in the s2s directory.
Tina
On Wed, Jul 14, 2021 at 10:59 AM Christina Kalb @.***> wrote:

Hi George:
Yes.  feature_950_weather_regime_freq calls this.  It's still in it's own
branch because the plotting is needed for it to work.  You'll want to run
UserScript_obsERA_obsOnly_WeatherRegime.conf, along with a system.conf
file.  Does that help?  Do you need more information?
Tina
On Wed, Jul 14, 2021 at 10:54 AM George McCabe @.***>
wrote:

@CPKalb https://github.com/CPKalb I'm a little unclear one what is
needed to do to test. Is there a use case in the METplus branch that calls
the new function?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/138#issuecomment-880054053,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJHD4QB36POOL2ECNW4G2DTXW6ODANCNFSM5ALYW5ZA
.

--

Christina Kalb
Associate Scientist
National Center for Atmospheric Research
3450 Mitchell Lane
Boulder, CO 80301
@.***
303-497-2819

My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.

--

Christina Kalb
Associate Scientist
National Center for Atmospheric Research
3450 Mitchell Lane
Boulder, CO 80301
@.***
303-497-2819

My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.

Thanks for the info. I set up the use case to run on seneca and it looks like the data required for the use case is not on that machine. I see the missing file on the DTC web server under the feature_950_weather_regime_freq directory. Is this data available on an NCAR machine where I can test this?

@CPKalb I obtained the missing file from the web server and tried to run it on seneca, but the file has hard paths to what looks like Cheyenne in it.
METplus_Data/feature_950_weather_regime_freq/model_applications/s2s/UserScript_obsERA_obsOnly_WeatherRegime/ERA/Daily/ERA_daily_files_lead000.txt
Is this file supposed to be generated by the use case?

Hi Geroge:
The file can be generated by the use case.  However, I have it currently
turned off as it takes a while to generate the file listing of 30 years.
You can turn it back on by adding UserScript(obs_wr_filelist) to the
process_list and making sure the following files point to your data:
OBS_WR_INPUT_DIR =
{INPUT_BASE}/model_applications/s2s/UserScript_obsERA_obsOnly_WeatherRegime/ERA/Daily
OBS_WR_INPUT_TEMPLATE = Z500_daily_{valid?fmt=%Y%m%d}_NH.nc
I'm not sure what to do about this in the long run because I think the run
time will be too long for the unit tests to have it always turned on, but
on different machines, the file paths will need to be edited.
Tina
On Wed, Jul 14, 2021 at 3:23 PM George McCabe @.***>
wrote:

@CPKalb https://github.com/CPKalb I obtained the missing file from the
web server and tried to run it on seneca, but the file has hard paths to
what looks like Cheyenne in it.
METplus_Data/feature_950_weather_regime_freq/model_applications/s2s/UserScript_obsERA_obsOnly_WeatherRegime/ERA/Daily/ERA_daily_files_lead000.txt
Is this file supposed to be generated by the use case?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/138#issuecomment-880219658,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJHD4UE4LOBNU3NJE3O77LTXX54PANCNFSM5ALYW5ZA
.

--

Christina Kalb
Associate Scientist
National Center for Atmospheric Research
3450 Mitchell Lane
Boulder, CO 80301
@.***
303-497-2819

My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.

@CPKalb OK, I will try that out. You could have the file include relative paths from the input directory so they will be the same regardless of which machine it is run on. Then your script can prepend the input directory path to the relative path to get the full path.

An enhancement to the logic for running UserScript should likely be made to obtain a full list of files that match a filename template for the full range of run times specified in the config file and provide that list to the user script for each call. Currently it just uses wildcard characters to get all files, but that isn't entirely accurate if there are files that exist outside the range of times. This enhancement would greatly speed up the time your script to generate the file list takes. I will create a GitHub issue for this work. I think it would be very useful in other applications as well.
I think for now it is OK to have the use case take a long time to run in the tests until we make improvements to speed it up. They are only run for pull requests and protected branches (develop, mainX, etc.) and it is not the longest use case. Also, the current logic has the text file written under the INPUT_BASE and we want to avoid that and only write files under the OUTPUT_BASE.

Hi George:
Another issue that I see with the script is that creating the listing of
input filles uses an append.  I do have logic in my weather regime python
code which will clean up the text file at the end, but if a user doesn't do
this, then the script will continue to append files to the input file
listing and it will grow to double its original size, which is not
desirable.  I couldn't come up with a better solution, as adding something
like a date stamp to the txt file containing the list would result in
multiple text files being written out (since it takes a while to process).
As you correctly noted above, I can't use a wildcard for all my
processing.  I could probably do this for the weather regime analysis as I
think it pulls all files in that directory.  But, I can't do it for the
blocking code as the IBL section of the blocking code requires a subset of
the available data.
I'm not exactly sure how to use relative paths as the text file expands the
environment variables.  How would I keep those environment variables in the
text listing output?  I find when I run my UserScript, OUTPUT_BASE and
INPUT_BASE are not visible to the script.  I have to add them as
environment variables in that section for my code to be able to see them
Tina
On Wed, Jul 14, 2021 at 4:03 PM George McCabe @.***>
wrote:

An enhancement to the logic for running UserScript should likely be made
to obtain a full list of files that match a filename template for the full
range of run times specified in the config file and provide that list to
the user script for each call. Currently it just uses wildcard characters
to get all files, but that isn't entirely accurate if there are files that
exist outside the range of times. This enhancement would greatly speed up
the time your script to generate the file list takes. I will create a
GitHub issue for this work. I think it would be very useful in other
applications as well.
I think for now it is OK to have the use case take a long time to run in
the tests until we make improvements to speed it up. They are only run for
pull requests and protected branches (develop, mainX, etc.) and it is not
the longest use case. Also, the current logic has the text file written
under the INPUT_BASE and we want to avoid that and only write files under
the OUTPUT_BASE.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/138#issuecomment-880239522,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJHD4UQ5HGTDTRHCBBB7MLTXYCT7ANCNFSM5ALYW5ZA
.

--

Christina Kalb
Associate Scientist
National Center for Atmospheric Research
3450 Mitchell Lane
Boulder, CO 80301
@.***
303-497-2819

My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.

If it's okay for the use case to take a while, I'll update the branch so it
creates the file listing and such, and commit that new version.  I can also
try to create relative paths.  Should I add INPUT_BASE to the user_env_vars
section, or is there a better way to do this?
Tina
On Wed, Jul 14, 2021 at 6:08 PM Christina Kalb @.***> wrote:

Hi George:
Another issue that I see with the script is that creating the listing of
input filles uses an append.  I do have logic in my weather regime python
code which will clean up the text file at the end, but if a user doesn't do
this, then the script will continue to append files to the input file
listing and it will grow to double its original size, which is not
desirable.  I couldn't come up with a better solution, as adding something
like a date stamp to the txt file containing the list would result in
multiple text files being written out (since it takes a while to process).
As you correctly noted above, I can't use a wildcard for all my
processing.  I could probably do this for the weather regime analysis as I
think it pulls all files in that directory.  But, I can't do it for the
blocking code as the IBL section of the blocking code requires a subset of
the available data.
I'm not exactly sure how to use relative paths as the text file expands
the environment variables.  How would I keep those environment variables in
the text listing output?  I find when I run my UserScript, OUTPUT_BASE and
INPUT_BASE are not visible to the script.  I have to add them as
environment variables in that section for my code to be able to see them
Tina
On Wed, Jul 14, 2021 at 4:03 PM George McCabe @.***>
wrote:

An enhancement to the logic for running UserScript should likely be made
to obtain a full list of files that match a filename template for the full
range of run times specified in the config file and provide that list to
the user script for each call. Currently it just uses wildcard characters
to get all files, but that isn't entirely accurate if there are files that
exist outside the range of times. This enhancement would greatly speed up
the time your script to generate the file list takes. I will create a
GitHub issue for this work. I think it would be very useful in other
applications as well.
I think for now it is OK to have the use case take a long time to run in
the tests until we make improvements to speed it up. They are only run for
pull requests and protected branches (develop, mainX, etc.) and it is not
the longest use case. Also, the current logic has the text file written
under the INPUT_BASE and we want to avoid that and only write files under
the OUTPUT_BASE.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/138#issuecomment-880239522,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJHD4UQ5HGTDTRHCBBB7MLTXYCT7ANCNFSM5ALYW5ZA
.

--

Christina Kalb
Associate Scientist
National Center for Atmospheric Research
3450 Mitchell Lane
Boulder, CO 80301
@.***
303-497-2819

My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.

--

Christina Kalb
Associate Scientist
National Center for Atmospheric Research
3450 Mitchell Lane
Boulder, CO 80301
@.***
303-497-2819

My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.

@CPKalb , yes it is fine to set INPUT_BASE in the user_env_vars section so it is available to the script.
I remembered that the GridDiag wrapper already has logic to create file lists based on the runtime frequency that would solve the issues were facing. I'm currently working on moving that logic up to the parent class RuntimeFreqWrapper so that UserScript wrapper can utilize that logic too. We can test the changes with your use case to make sure it works as expected. I wouldn't spend any time dealing with the appending to file issue since this fix will make that part unnecessary. With the new changes, a file list will be generated that contains all of the relevant files for the run that can be passed into the UserScript (or made available as an environment variable) so that you can use that text file to get the full list of files you need.
Also, I just realized that this discussion probably doesn't belong in this PR for METplotpy, so we should continue the discussion in the issue I created for this work: dtcenter/METplus#1002. I referenced this discussion in that issue so we can trace back the history if needed.


Keyboard Manager doesn't detect brightness up/down keys

Microsoft PowerToys version
0.62.0
Running as admin

[X] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce

Open PowerToys Settings
Navigate to Keyboard Manager
Click on "Remap a key"
Click on "+" in the "Remap Keys" window
Click on either one of the "Type" buttons
Press brightness up or brightness down key (as found on Microsoft Surface or other Microsoft keyboards)

✔ Expected Behavior

The button press being detected
"Brightness Up" or "Brightness Down" appearing in the "Keys Pressed" field

❌ Actual Behavior

No key stroke being detected by PowerToys, as if no button was pressed at all
Windows brighness change pop up appearing in top left corner (Default behavior for these buttons)

Other Software
No response

I'll need to check, but It might be that those keys are "hardware" keys, so Windows doesn't generate input events for them.

@yuyoyuppe Thanks for the reply! Not sure about Surface products, but I'm actually encountering this issue on my bluetooth Microsoft Designer Keyboard, so them being hardware keys shouldn't be possible right? Not a keyboard expert, thought this little piece of information might help in figuring that out.

Yes, this has been requested a lot. Issue #13200 points to #3350, foôsome reason.

I need this feature too, I wonder why Volume Up / Volume Down are supported and not Brightness Up / Brightness Down

Same here, I'd love to be able to remap this keys as my desktop monitor don't even have the ability to do that. They are just wasting space on my keyboard.

Same here, I'd love to be able to remap this keys as my desktop monitor don't even have the ability to do that. They are just wasting space on my keyboard.

If they add the ability to remap it to a shortcut, you can use apps that interface with DDC/CI like Twinkle Tray to change the external monitor brightness.


Getting React.createRef() to work with react-spring

In pull request #185, ref forwarding was added to react-spring—yay!
The problem
React-spring currently only handles ref forwarding if using the "callback ref" method. However, a newer method using React.createRef() is not supported. In this case, instead of a function, the ref retrieved via an object with the form {current: [node]}.
Discussion of the issue with error in this Spectrum post.
Proposed solution
Handle both callback refs and React.createRef() in react-spring.
Suggestion (please discuss)
In createAnimatedComponent on line 71, we could check whether the ref passed is a function or object and handle it accordingly:
<Component
  {...animatedProps}
  ref={node => {
    this.node = node
    if (forwardRef) {
      // If it's a function, assume it's a ref callback
      if(typeof forwardRef === "function")
        forwardRef(node)
      // If it's an object and has a 'current' property, assume it's a ref object
      else if(typeof forwardRef === "object" && forwardRef.current)
        forwardRef.current = node
      }
    }
  }
/>

It's fine with me, could you test and see if it works? Then just add the PR and i'll merge right away.

I added your code into the 6.x branch, it'll come out soon.

Thanks for putting it in!
I see you removed the check for whether forwardRef.current exists. Took me a moment but makes sense: it's better that it triggers an error instead of silently failing 👍

It actually didn't work that way b/c initially ref.current is null, so that condition will always return false and never get to set it.


Can't hotkey a period (dot)

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[X] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
Can't hotkey a period (dot).
Expected behavior
To be able to hotkey a period
Minimal reproduction of the problem with instructions
Attempted:
{ keys: 'control.\.', description: 'Go To Settings' }
{ keys: 'control..', description: 'Go To Settings' }
{ keys: 'control..', description: 'Go To Settings' }
{ keys: 'control.46;', description: 'Go To Settings' }

What is the motivation / use case for changing the behavior?
Using this specific hotkey to go to settings
Environment

Angular version: 13.x

Browser:
- [X] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

Even more, on Ukrainian and Russian layout the dot character and  "?" are on the same button, so you can't trigger the help window on those layouts.

@jherman can you be more specific about the issue not able to understand with "Can't hotkey a period (dot)."

Hi, @Tweniee
I am not the original commenter, but I will give a few more details (I am having the same issue):
It does not seem to be possible (or I am not able to find a way) to place a shortcut on the dot/period key (.)
As an example,this works fine: this.hotkeys.addShortcut({ keys: 'alt.,',  description: 'Foo' })
bus this does not work this.hotkeys.addShortcut({ keys: 'alt..',  description: 'Foo' })
(for obvious reasons since "." is the separator char)
The HotkeysShortcutPipe prints this shortcut (alt..) as:

So the question is: Is there any way to create a shortcut for the dot/period key (.) ?
(@jherman Listed a few ways they tried that weren't successful.)


Chef Server should allow creation and association of Organization

Chef Server API requires pivotal key to create and associate a user to an organization.
The API should allow server-admins to do the same operations as pivotal users.
This will avoid the need to distribute the Pivotal key for users.

webui key should also allow this

@kalroy could you please add some more context around why this change is required. Thanks!

@PrajaktaPurohit this would be so that a tool such as Terraform could be used to manage users as code.  I have extended the Terraform provider with many resources, however associating a user with an organization still requires the pivotal PEM.
I feel that a user with admin privileges should also be able to perform this action.


sql to csv task login fails

i have sql to csv task however it fails to login
<Workflow id="119" name="xyz" description="g360-wf" xmlns="urn:wexflow-schema">
  <Settings>
    <Setting name="launchType" value="trigger" />
    <Setting name="enabled" value="true" />
    <Setting name="period" value="00.00:00:00" />
    <Setting name="cronExpression" value="" />
  </Settings>
  <LocalVariables />
  <Tasks>
    <Task id="2" name="SqlToCsv" description="execute sql step" enabled="true">
      <Setting name="connectionString" value="Data Source=xyz,50001;Initial Catalog=test;Persist Security Info=True;User ID=xyz;Password=xyz" />
      <Setting name="sql" value="SELECT [STATUS] FROM tablexyz where [STATUS] in (1,3,5) AND [CS_CREATED_DATE] > CAST(GETDATE() + 1 AS DATE)" />
      <Setting name="type" value="sqlserver" />
    </Task>
    <Task id="3" name="FilesMover" description="Moving SqlTocsv files from temp folder" enabled="true">
      <Setting name="destFolder" value="C:\WexflowTesting\SqlToCsv\" />
      <Setting name="selectFiles" value="2" />
    </Task>
  </Tasks>
</Workflow>

2019-08-02 15:53:28,878  INFO [16] - [g360 / 1] Workflow started.
2019-08-02 15:53:28,880  INFO [16] - [g360 / 1] [SqlToCsv] Executing SQL scripts...
2019-08-02 15:53:28,915 ERROR [16] - [g360 / 1] [SqlToCsv] An error occured while executing sql script. Error: Login failed for user 'xyz'.
2019-08-02 15:53:28,915  INFO [16] - [g360 / 1] [SqlToCsv] Task finished.
2019-08-02 15:53:28,915  INFO [16] - [g360 / 1] [FilesMover] Moving files...
2019-08-02 15:53:28,915  INFO [16] - [g360 / 1] [FilesMover] Task finished.
2019-08-02 15:53:28,922  INFO [16] - [g360 / 1] Workflow finished.

This seems to be a problem related to your database access configuration.

Which version of Wexflow are you using?
Can you check if the connection string is correct and that you can connect to your database with the same connection string with another application (PowerShell for example)?

4.0.1, my bad read was not granted


Bug: tests are run inconsistently in CI pipeline

Describe the bug
Maybe not a real bug but more an issue I came across when I looked at how our tests are run in CI pipeline:

core: here we have a matrix covering Python versions from 3.9 - 3.12
community modules: we have a matrix without real purpose covering only Python version 3.11

So looking at pyproject.toml I guess we want to support Python versions from 3.9 - 4.0, so our pipeline needs to reflect this.
I also see that we do not pin the versions of our Ubuntu runners. I would say it's a better practice to pin version to get predictable CI results.
Also I am scratching my head what purpose that make file still serves there. I see no need to use it in the pipeline jobs.
Also we use coverage, but are not doing anything with that test coverage data.
I could spend some time fixing this up. But I would be very grateful for some feedback and thoughts. Not so sure if I miss something here.

@totallyzen Ok, so I will create a PR addressing these inconsistency issues and pin runner versions.
I just went through https://github.com/testcontainers/testcontainers-python/pull/460 and left some comments and suggestions.
I haven't used slack in ages. But I will check on that channel now.


Failed to instantiate module angularUtils.directives.dirPagination

Error: [$injector:modulerr] Failed to instantiate module angularUtils.directives.dirPagination due to:
Error: [$injector:nomod] Module 'angularUtils.directives.dirPagination' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.

It was working fin then suddenly! BOOM!

Did something in your code base change (apart from the huge boom)?

My bad! Sorry scripts where missing on the HTML

Same thing for me. I ass|u|me'd that the bower would add the script line in the html for me. Alas, I just added  and all is well with the world.

It looks like the pagination directive doesn't work for more than one table on a page at a time.


JWT integration (login with superhero)

JWT Auth service:

https://github.com/aeternity/jwt-auth

Superhero wallet

https://github.com/aeternity/superhero-wallet

The above are used for loging in in league.superhero.com with superhero ID.

ref to several PRs


Typo fixes in Samples: change instances of "Unkown" to "Unknown"

How ironic that I've got a typo in the name of my branch!

Fixes same typos as #3482

Whoops, I didn't realize someone else had caught this already. I noticed this independently while looking through the samples, but I should have searched deeper to see if it was a known issue!

Accepting this as @sphanley has already signed the CLA. Thanks for the contribution!


BigInt div only works for BigInt of size 4

The underlying issue is that expressions using const generics can't be used in a const context as of now.
Solution
A possibly "solution" would be to create an impl macro, and whenever a new size is needed, simply call the macro for the new size. I would have to decide if the macro is exposed in the public API or not.
If it is public, other users can make sure they can use division for any size. On the other hand, if I call the impl macro for a new size which a user is already using, this could cause breakage.
If it's private, there's no breakage, but users can't use custom sizes.
Because the main purpose of libcrypto is to support turtls, the macro should be kept private, at least for now. I can always make it public with now breakage, but I can't make it private with no consequences.

A macro has been added to solve this issue. However, it will hopefully soon be removed again. Using the macro is messy, and LSP is basically useless inside of it.


Cleanup containers

Use custom types in container definitions (fix #1153)

Renamings

deposit_index => eth1_deposit_index (more precise and future proof; consistent with eth1_data and eth1_data_votes in "Eth1" grouping)
previous_epoch_attestations => previous_attestations (consistent with previous_crosslinks, previous_justified_epoch and previous_justified_root; shorter)
validator_registry => validators (cleaner; more semantically correct—the registry is now both validators and balances; shorter)
Remove all latest_ prefixes (cleaner; shorter; the prefix does not add much other than clutter)

Semantically group BeaconState fields, following a logical progression (see item 14 in #1054)

    # Versioning
    genesis_time: uint64
    slot: Slot
    fork: Fork
    # History
    block_header: BeaconBlockHeader
    block_roots: Vector[Hash, SLOTS_PER_HISTORICAL_ROOT]
    state_roots: Vector[Hash, SLOTS_PER_HISTORICAL_ROOT]
    historical_roots: List[Hash]
    # Eth1
    eth1_data: Eth1Data
    eth1_data_votes: List[Eth1Data]
    eth1_deposit_index: uint64
    # Registry
    validators: List[Validator]
    balances: List[Gwei]
    # Shuffling
    start_shard: Shard
    randao_mixes: Vector[Hash, RANDAO_MIXES_LENGTH]
    active_index_roots: Vector[Hash, ACTIVE_INDEX_ROOTS_LENGTH]
    # Slashings
    slashed_balances: Vector[Gwei, SLASHED_EXIT_LENGTH]
    # Attestations
    previous_attestations: List[PendingAttestation]
    current_attestations: List[PendingAttestation]
    # Crosslinks
    previous_crosslinks: Vector[Crosslink, SHARD_COUNT]
    current_crosslinks: Vector[Crosslink, SHARD_COUNT]
    # Justification
    previous_justified_epoch: Epoch
    previous_justified_root: Hash
    current_justified_epoch: Epoch
    current_justified_root: Hash
    justification_bitfield: uint64
    # Finality
    finalized_epoch: Epoch
    finalized_root: Hash

validator_registry => validators

We debated this long ago and decided that validator_registry was more explicit -- it is the entire registry of validators rather than just a list of validators

previous_epoch_attestations => previous_attestations (consistent with previous_crosslinks, previous_justified_epoch and previous_justified_root; shorter)

There is a difference between previous_epoch_attestations and previous_crosslinks/previous_justified_epoch. previous_epoch_attestations are the attestations for the previous epoch. Whereas previous_crosslinks were the crosslinks during the previous epoch. Same with previous_justified_epoch -- this was the latest justified_epoch during the previous epoch.

We debated this long ago and decided that validator_registry was more explicit

I was in favour of validator_registry in that debate. Since then the situation has changed with the registry split being into two parts:
    # Registry
    validators: List[Validator]
    balances: List[Gwei]

@protolambda, @hwwhww Looks like there is a race condition in the linter CI. I didn't change phase 1 yet the CI is complaining about it.
@djrtwo I think I've addressed all issues, and I've added a bunch of descriptions to field containers. Feel free to edit/add/remove those as you feel appropriate :)

@JustinDrake I'll take a closer look. Best guess would be that you changed something in phase 0, and it wound up as part of phase 1.

@JustinDrake fixed linting in 75b4692 . Not a race condition, just one missing space. You can run make -B pyspec and make lint to check your changes locally. :+1:


feature of cooperation edit

amazing library.
I make it mutli-editable.
I dont know whether you need it.

Thank you for the PR. I am impressed.
But currently the project is a pure static website hosted on GitHub pages. You can even download it and open the index.html to execute it.
I do think server side code is useful. But it also makes hosting harder.
I will leave the PR open for now.

Close it. Since we won't support this feature in the near future.


.AddSigningCredential(certificate) is not working

When I was using ID Server 4 RC 3,  .AddSigningCredential(certificate) code was working fine on Windows Server 2012 R2 but after upgrade to ID Server 4 and .NET Core 1.0 to .NET Core 1.1, the .AddSigningCredential(certificate) is not working anymore.
But this code  .AddTemporarySigningCredential() is working fine in every condition.
But when I use this .AddSigningCredential(certificate), I start getting Bad Gateway issue for new controller which I have written. On class level I used [Authorize(Policy = "CRUD")].
Any help/pointers will be appreciated.

In what way is it not working?

I created one controller with two action methods. The controller is decorated with authorize attribute. From fiddler i invoke one of the method with bearer token. But it says bad gateway. Getting token works fine from id server.

maybe you are running into this:
https://github.com/dotnet/coreclr/issues/8148

I don't think so, because I am not getting any kind of exception in log and plus tried with below code.
        var fail = true;
        X509Certificate2 cert;
        if (!fail)
        {
            cert = new<EMAIL_ADDRESS>"demo-svc");
            cert = new X509Certificate2(cert.RawData);
            Console.WriteLine(cert.Thumbprint);
        }

        cert = FindInStore("demo-svc");
        Console.WriteLine(cert.Thumbprint);

        **var credential = new SigningCredentials(new X509SecurityKey(cert), "RS256");**

The last line is copied from ID Server code and it is not throwing any kind of exception. So I think problem is somewhere else.

I had the above mentioned problem - the exception was happening at some point and since the whole CLR broke down, there were also no logs.
Closing for now - feel free to re-open if it needs further discussion.

Yes, during setup, the code don't break (X509Certificate2). But once it receive request, it started giving the Bad Gateway issue. To overcome this we have to create a object of X509Certificate2 as mentioned in your pointers. You can also mentioned this in some documentation imho.


Resolve missing permission errors confusion

What
This PR fixes a typo in a CameraView.kt
Changes

Swapped permission errors to throw.

Tested on

iPhone 11 , iOS 14.3
Nokia 5.3, Android 10

Related issues

Resolves #162

Thanks!


Linux Editor seems broken

Describe the bug
Just tested the editor on Linux, the results are not that impressive. Not sure if this was ever tested properly before?

UI scaling isn't working correctly, setting it to something other than 1.0 breaks the whole UI
Creating simple game objects like a cube caused exceptions
I couldn't get the game to build, it caused various errors

To Reproduce
Steps to reproduce the behavior:

Build & run editor on Linux
Set UI scale to something other than 1.0
Notice how the whole UI breaks
Try to spawn a cube
Notice various exceptions in the console
Try to build the game after assiging a scene in the project settings
Notice various exceptions in the console

Expected behavior
A working editor.
Additional context
Tested on Ubuntu 24.04 with Nvidia 550 drivers, X11 display server on a 27" 4k monitor and .NET 8.0.0
I haven't tested Windows so far, but I assume it's working better there?

@Armynator may I ask what branch of Prowl you are working with? The development branch is where the latest changes are at.

Just tested the latest development branch which is even worse. Upon opening a project in the editor, tons of errors are spammed into the console. Example:
Failed to import /test/Library/AssetDatabase/8623525a-750e-406f-aa7c-eca654dfd043.serialized!
System.Exception: Failed to import /test/Library/AssetDatabase/8623525a-750e-406f-aa7c-eca654dfd043.serialized
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset(Guid assetGuid) in /Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 491
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset[T](Guid assetGuid, UInt16 fileID) in /Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 430
Error in EditorWindow: Something went wrong loading asset.
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset[T](Guid assetGuid, UInt16 fileID) in /Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 452
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset[T](FileInfo assetPath, UInt16 fileID) in /Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 413
   at Prowl.Editor.Assets.EditorAssetProvider.LoadAsset[T](String relativeAssetPath, UInt16 fileID) in /Prowl/Prowl.Editor/Assets/EditorAssetProvider.cs:line 17
   at Prowl.Runtime.RenderPipelines.DefaultRenderPipeline.ValidateDefaults() in /Prowl/Prowl.Runtime/Rendering/RenderPipeline/DefaultRenderPipeline.cs:line 38
   at Prowl.Runtime.RenderPipelines.DefaultRenderPipeline.Render(Framebuffer target, Camera camera, RenderingData& data) in /Prowl/Prowl.Runtime/Rendering/RenderPipeline/DefaultRenderPipeline.cs:line 48
   at Prowl.Editor.SceneViewWindow.Draw() in /Prowl/Prowl.Editor/Editor/SceneViewWindow.cs:line 144
   at Prowl.Editor.EditorWindow.ProcessFrame() in /Prowl/Prowl.Editor/Editor/EditorWindow.cs:line 235

For the first example error it looks like the path is wrong, shouldn't the "test" project be in a subfolder of the Prowl folder?
The main branch doesn't have that problem. The project at least opens and renders there.
Also when setting the UI scale to anything else than 1.0, this happens, on development and main branch:

You might have more luck in the discord I'm not actually that involved in the dev so I'm not sure why that is happening

Hey, thanks for trying out prowl and making an issue!
UI Scaling isn't too surprising, however, it is a bug and should be working, It appears ClipRects have gotten broken at some point.
Im not sure why exceptions are being thrown on the creation of a cube or exporting project...
The development branch is a lot further along and has more effort in it toward Linux support due to @sinnwrig a Linux user doing a lot of work on it.
One thing is that the Development branch and Main branch projects are incompatible with each other, which explains those errors.
Other than that development should work a lot better and shouldn't have exceptions on creating a cube or exporting the game (Export may not produce a viable executable though) Could you try development out again with a new project and let us know if there are any exceptions?

@Armynator apologies your experience with Prowl wasn't the smoothest - it's pretty rough around the edges right now, but the issues you described should be relatively simple fixes!
If it's alright, could you please verify if the import error still occurs even when opening a newly created project with the development branch? As @michaelsakharov mentioned above, the serialization formats have changed considerably from main and I can only seem to reproduce the same exceptions when opening a project on development that was created with main.
As for UI scaling, the scissor rects are broken on all platforms, although that should be the only place the UI is broken, as disabling scissor rects seems to alleviate almost all of the issue. It should be a relatively simple fix and I'll try to patch it up ASAP on development.

@Armynator UI scaling should be fixed in a806522 on development. However, I'll leave this issue open in case project importing is still broken.

UI scaling is fixed. Opening project still doesn't work. (Well, it opens, but the editor stays blank and errors are spammed into the console)

Clone repo, checkout development branch, init submodules
Open or build the SLN file
Start Prowl.Editor
Create project
Open project
Notice error spam in console

Failed to import /ProwlProjects/Test/Library/AssetDatabase/ddc9e4fb-07a9-4b76-a65c-a2be4d059533.serialized.
System.Exception: Failed to import /ProwlProjects/Test/Library/AssetDatabase/ddc9e4fb-07a9-4b76-a65c-a2be4d059533.serialized.
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset(Guid assetGuid) in /ProwlProjects/Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 491
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset[T](Guid assetGuid, UInt16 fileID) in /ProwlProjects/Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 430
Error in EditorWindow: Something went wrong loading asset.
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset[T](Guid assetGuid, UInt16 fileID) in /ProwlProjects/Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 452
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset[T](FileInfo assetPath, UInt16 fileID) in /ProwlProjects/Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 413
   at Prowl.Editor.Assets.EditorAssetProvider.LoadAsset[T](String relativeAssetPath, UInt16 fileID) in /ProwlProjects/Prowl/Prowl.Editor/Assets/EditorAssetProvider.cs:line 17
   at Prowl.Runtime.RenderPipelines.DefaultRenderPipeline.ValidateDefaults() in /ProwlProjects/Prowl/Prowl.Runtime/Rendering/RenderPipeline/DefaultRenderPipeline.cs:line 35
   at Prowl.Runtime.RenderPipelines.DefaultRenderPipeline.Render(Framebuffer target, Camera camera, RenderingData& data) in /ProwlProjects/Prowl/Prowl.Runtime/Rendering/RenderPipeline/DefaultRenderPipeline.cs:line 45
   at Prowl.Editor.SceneViewWindow.Draw() in /ProwlProjects/Prowl/Prowl.Editor/Editor/SceneViewWindow.cs:line 144
   at Prowl.Editor.EditorWindow.ProcessFrame() in /ProwlProjects/Prowl/Prowl.Editor/Editor/EditorWindow.cs:line 235
Failed to import Defaults/SkyDome.obj
Unable to load shared library 'libdl.so' or one of its dependencies. In order to help diagnose loading problems, consider using a tool like strace. If you're using glibc, consider setting the LD_DEBUG environment variable:
/ProwlProjects/Prowl/Build/Editor/Debug/net9.0/runtimes/linux-x64/native/libdl.so: cannot open shared object file: No such file or directory
/home/hannes/.dotnet/shared/Microsoft.NETCore.App/9.0.0-rc.1.24431.7/libdl.so: cannot open shared object file: No such file or directory
/ProwlProjects/Prowl/Build/Editor/Debug/net9.0/libdl.so: cannot open shared object file: No such file or directory
/ProwlProjects/Prowl/Build/Editor/Debug/net9.0/runtimes/linux-x64/native/liblibdl.so: cannot open shared object file: No such file or directory
/home/hannes/.dotnet/shared/Microsoft.NETCore.App/9.0.0-rc.1.24431.7/liblibdl.so: cannot open shared object file: No such file or directory
/ProwlProjects/Prowl/Build/Editor/Debug/net9.0/liblibdl.so: cannot open shared object file: No such file or directory
/ProwlProjects/Prowl/Build/Editor/Debug/net9.0/runtimes/linux-x64/native/libdl.so.so: cannot open shared object file: No such file or directory
/home/hannes/.dotnet/shared/Microsoft.NETCore.App/9.0.0-rc.1.24431.7/libdl.so.so: cannot open shared object file: No such file or directory
/ProwlProjects/Prowl/Build/Editor/Debug/net9.0/libdl.so.so: cannot open shared object file: No such file or directory
/ProwlProjects/Prowl/Build/Editor/Debug/net9.0/runtimes/linux-x64/native/liblibdl.so.so: cannot open shared object file: No such file or directory
/home/hannes/.dotnet/shared/Microsoft.NETCore.App/9.0.0-rc.1.24431.7/liblibdl.so.so: cannot open shared object file: No such file or directory
/ProwlProjects/Prowl/Build/Editor/Debug/net9.0/liblibdl.so.so: cannot open shared object file: No such file or directory

        In UnmanagedLinuxLibraryImplementation.dlopen at :0:0
        In UnmanagedLinuxLibraryImplementation.NativeLoadLibrary at :0:0
        In UnmanagedLibrary.LoadIfNotLoaded at :0:0
        In AssimpContext.CreateConfigs at :0:0
        In AssimpContext.ImportFile at :0:0
        In ModelImporter.Import at /ProwlProjects/Prowl/Prowl.Editor/Assets/Importers/ModelImporter.cs:68:13
        In AssetDatabase.Reimport at /ProwlProjects/Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:362:13

Failed to import /ProwlProjects/Test/Library/AssetDatabase/ddc9e4fb-07a9-4b76-a65c-a2be4d059533.serialized.
System.Exception: Failed to import /ProwlProjects/Test/Library/AssetDatabase/ddc9e4fb-07a9-4b76-a65c-a2be4d059533.serialized.
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset(Guid assetGuid) in /ProwlProjects/Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 491
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset[T](Guid assetGuid, UInt16 fileID) in /ProwlProjects/Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 430
Error in EditorWindow: Something went wrong loading asset.
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset[T](Guid assetGuid, UInt16 fileID) in /ProwlProjects/Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 452
   at Prowl.Editor.Assets.AssetDatabase.LoadAsset[T](FileInfo assetPath, UInt16 fileID) in /ProwlProjects/Prowl/Prowl.Editor/Assets/AssetDatabase.Core.cs:line 413
   at Prowl.Editor.Assets.EditorAssetProvider.LoadAsset[T](String relativeAssetPath, UInt16 fileID) in /ProwlProjects/Prowl/Prowl.Editor/Assets/EditorAssetProvider.cs:line 17
   at Prowl.Runtime.RenderPipelines.DefaultRenderPipeline.ValidateDefaults() in /ProwlProjects/Prowl/Prowl.Runtime/Rendering/RenderPipeline/DefaultRenderPipeline.cs:line 35
   at Prowl.Runtime.RenderPipelines.DefaultRenderPipeline.Render(Framebuffer target, Camera camera, RenderingData& data) in /ProwlProjects/Prowl/Prowl.Runtime/Rendering/RenderPipeline/DefaultRenderPipeline.cs:line 45
   at Prowl.Editor.SceneViewWindow.Draw() in /ProwlProjects/Prowl/Prowl.Editor/Editor/SceneViewWindow.cs:line 144
   at Prowl.Editor.EditorWindow.ProcessFrame() in /ProwlProjects/Prowl/Prowl.Editor/Editor/EditorWindow.cs:line 235

Interesting - it seems that the Assimp native library doesn't load correctly or can't be found. I'll investigate a bit more and try to fix the native dependencies, but this might be fixed once @michaelsakharov finishes porting Assimp to C# and removes the native dependency altogether.

@Armynator I've found a related issue to yours regarding Assimp here, and they seem to have a very similar problem with libdl not being found when loading Assimp. It appears that this line of code in the Assimp package is causing issues since it's looking directly for libdl.so (when the system might have libdl.so.2 or the like) and causes cascading exceptions for the editor when it fails to find it. If this is the case, completely fixing this issue might have to wait until the Assimp package is fixed or our C# port of Assimp is completed. In the meantime, you could potentially try a fix like this one provided in the stack exchange question and create a symlink to the libsdl version installed on the system, although it is quite hack-y:
$ whereis libdl.so.2
libdl.so.2: /usr/lib/libdl.so.2 /usr/lib32/libdl.so.2 (or similar paths)
$ sudo ln -s /usr/lib/libdl.so.2 /usr/lib/libdl.so (make symlink to 64-bit libdl)
$ sudo ln -s /usr/lib32/libdl.so.2 /usr/lib32/libdl.so (make symlink to 32-bit libdl)

Yup that worked.


Add TimeLimiter node

Description

Add TimeLimiter node that will fails the child that does not finish running after x seconds specified in export variable wait_time.
Clear running child of decorators and composite nodes after ran.
BeehaveRoot now call child's before_run if it was not running in previous tick.

Screenshots

Player entering an Enemy range will be chased for a set amount of time, it will then lose aggro if Chase action can not finish in time.

@Ohan17 thank you for your contribution.


Noting the deprecation of Dynamic Disks

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

#sign-off

Closing this PR to remove duplicate.


test azure pipelines

The first commit of the pr should fail, since no version bump.
The next should be ok with the version bump.
Should then publish after merge.

ok, something is wrong then!

ok, now we are on track


Change parameter name for getAllVersions call

Hi,
It's me again ;)
According to the specification (http://docs.oasis-open.org/cmis/CMIS/v1.1/os/CMIS-v1.1-os.html#x1-3440006) the getAllVersions service has 2 required parameters : repositoryId and versionSeriesId.
So I made and propose to you this little change (before the parameter was objectId).
BR.
Wiser.

ok, at least in Alfresco the call work with both values (they are almost the same thing), but you are correct about the specification


add missing printf anchor

Thanks for the fix.

Uhm, turns out it was the wrong fix. The right fix was to remove the parameter, since it is already listed below.


Hidden Widget does not ignore DefaultPosition and DefaultSize when set to floating

Changes

Whenever a widget has its default state as hidden and then later set to floating, it would float into the top left corner and ignore the defaultFloatingPosition and defaultFloatingSize of the widget. This will no longer be the case, it will float to the size and position of the default parameters, until the size is changed by the user. The problems were that the position was just not being passed in with this case and the userSized variable of the floating widget was not being set.

Testing
I tested by creating a new widget in one of the frontstages, the widgets default state is hidden. I then later changed its state to floating with the click of a button. The recommended code to test this is in the linked issue.

Can this be backported to 4.6.x ?

This fix will be included when we release 4.7 early next week @GintV

@joehenry9498 If I remember correctly, this was working on 4.5.x, so it would be good to get 4.6.x patch for this in case 4.7 will bring more regressions since we plan to release in upcoming weeks.

@mergifyio backport release/4.6.x


Version compare is not correct

app store version is 1.4.135,  Install the 1.4.135 on iPhone
NSString *currentVersion = [bundleInfo objectForKey:@"CFBundleShortVersionString"];
// 1.4
NSString *latestVersion = [appDetails objectForKey:@"version"];
//1.4.135
//This will be true
if ([latestVersion compare:currentVersion options:NSNumericSearch] == NSOrderedDescending) {
//This will be true
}

//This will be true

Are you assuming or are you actually experiencing a problem?
Because this was solved in issue #1

not exactly like #1 , [bundleInfo objectForKey:@"CFBundleShortVersionString"] just get short version, not include build number (like 1.4).  the verson returned from apple lookup api  include build bumber (like 1.4.5).
Suggest you join build number.
NSString *versionStr = [[[NSBundle mainBundle] infoDictionary] objectForKey:@"CFBundleShortVersionString"];
NSString *buildStr = [[[NSBundle mainBundle] infoDictionary] objectForKey:(NSString )kCFBundleVersionKey];
NSString currentVersion = [NSString stringWithFormat:@"%@.%@",versionStr,buildStr];

It's comparing the versions, not the build. If I do as you say above, it will always be greater than just the version number.
Have to compare apples with apples, not apples with apples + pears. ;)
But please feel free to do this in your own project :yum:


Update lhospitals_rule.qmd

some typos.

Thanks again. I am so appreciative of the time and care you have taken on this.


Update RBAC to the new version

Update RBAC to the new version https://istio.io/docs/setup/kubernetes/upgrade/steps/#migrating-from-rbacconfig-to-clusterrbacconfig

Working on that


Error when running User Defined Op Tutorial

For bugs/issues, please fill in the following.  The more information you
provide, the more likely we can help you.
Environment info
Operating System:  Tried both on OS X and Rhel7
If installed from binary pip package, provide:

Which pip package you installed.

sudo pip install --upgrade https://storage.googleapis.com/tensorflow/mac/tensorflow-0.7.1-cp27-none-any.whl
2. The output from python -c "import tensorflow; print(tensorflow.version)".
0.7.1
If installed from sources, provide the commit hash:
Steps to reproduce

Follow instructions from https://www.tensorflow.org/versions/master/how_tos/adding_an_op/index.html
Attempt to build zero out:

g++ -v -std=c++11 -shared zero_out.cc -o zero_out.so \
-I $TF_INC -l tensorflow_framework -L $TF_LIB \
-fPIC -Wl,-rpath $TF_LIB

Get Error:

 g++ -v -std=c++11 -shared zero_out.cc -o zero_out.so -I $TF_INC -l tensorflow_framework -L $TF_LIB -fPIC -Wl,-rpath /usr/local/lib/python2.7/site-packages/
Apple LLVM version 7.0.2 (clang-700.1.81)
Target: x86_64-apple-darwin14.5.0
Thread model: posix
 "/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang" -cc1 -triple x86_64-apple-macosx10.10.0 -Wdeprecated-objc-isa-usage -Werror=deprecated-objc-isa-usage -emit-obj -mrelax-all -disable-free -disable-llvm-verifier -main-file-name zero_out.cc -mrelocation-model pic -pic-level 2 -mthread-model posix -mdisable-fp-elim -masm-verbose -munwind-tables -target-cpu core2 -target-linker-version 253.9 -v -dwarf-column-info -resource-dir /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../lib/clang/7.0.2 -I /usr/local/lib/python2.7/site-packages/tensorflow/include -stdlib=libc++ -std=c++11 -fdeprecated-macro -fdebug-compilation-dir /usr/local/lib/python2.7/site-packages/tensorflow/include/tensorflow/core/user_ops -ferror-limit 19 -fmessage-length 158 -stack-protector 1 -mstackrealign -fblocks -fobjc-runtime=macosx-10.10.0 -fencode-extended-block-signature -fcxx-exceptions -fexceptions -fmax-type-align=16 -fdiagnostics-show-option -fcolor-diagnostics -o /var/folders/pg/fbr4jqk15zz3wx4b2h1pgzlr002cz4/T/zero_out-31d965.o -x c++ zero_out.cc
clang -cc1 version 7.0.2 based upon LLVM 3.7.0svn default target x86_64-apple-darwin14.5.0
ignoring nonexistent directory "/usr/include/c++/v1"
#include "..." search starts here:
#include <...> search starts here:
 /usr/local/lib/python2.7/site-packages/tensorflow/include
 /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1
 /usr/local/include
 /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../lib/clang/7.0.2/include
 /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include
 /usr/include
 /System/Library/Frameworks (framework directory)
 /Library/Frameworks (framework directory)
End of search list.
In file included from zero_out.cc:1:
In file included from /usr/local/lib/python2.7/site-packages/tensorflow/include/tensorflow/core/framework/op.h:22:
/usr/local/lib/python2.7/site-packages/tensorflow/include/tensorflow/core/framework/op_def.pb.h:9:10: fatal error: 'google/protobuf/stubs/common.h' file not
      found
#include <google/protobuf/stubs/common.h>
         ^
1 error generated.

Dupicate of #1270

The protobuf error was coming from a bad installation, and there was a previous error around the eigen that is now fixed.


crash when stress testing tab change

When I change between tabs at an abnormal speed, the app eventually crashes.
It is a Release variant on iOS and I haven't got Sentry source maps set up without EAS build/update, but here's what I got on Sentry:

https://github.com/user-attachments/assets/61e63e56-f087-4a03-a0ea-ff8155038092

Hey, would you mind sharing a stack trace?
Without it's going to be extremely difficult. I tested it in the example app in the repo and it's not happening.
Maybe it's something within your app that causes this

Not sure if there's something else you'd want from Xcode, let me know if this isn't sufficient

Thanks this is helpful, judging by the stack trace you are using new architecture?

Yep

Can you re-test this with 0.3.1? 🙏

Wasn't able to reproduce this on 0.3.1 🥳


Instant is not suitable for measuring time

A couple of hours ago the libs team decided that the time that the PC is suspended should not be counted, meaning that Instant as they decided is not entirely suitable anymore for measuring speedrun times (maybe your PC goes to sleep during some long segment while doing a speedrun on a console for example). I'm looking into forking it atm.
https://github.com/rust-lang/rust/pull/87907#issuecomment-901394948

Thanks for bringing this to my attention. I'll probably implement an Instant for mist then, or eventually borrow yours from livesplit-core I suppose.


$currentDate update operator

type QuerySelector is missing the $currentDate update operator.
For reference: MongoDB docs

The new version should already support


dotnet tool broken with 3.0-preview4 on openSUSE Tumbleweed

This works with preview3, hence it's a regression:
havana ~ > dotnet tool

Welcome to .NET Core 3.0!
---------------------
SDK Version: 3.0.100-preview4-011223

Telemetry
---------
The .NET Core tools collect usage data in order to help us improve your experience. The data is anonymous. It is collected by Microsoft and shared with the community. You can opt-out of telemetry by setting the DOTNET_CLI_TELEMETRY_OPTOUT environment variable to '1' or 'true' using your favorite shell.

Read more about .NET Core CLI Tools telemetry: https://aka.ms/dotnet-cli-telemetry

----------------
More about .NET Core CLI Tools telemetry: https://aka.ms/dotnet-cli-telemetry
Explore documentation: https://aka.ms/dotnet-docs
Report issues and find source on GitHub: https://github.com/dotnet/core
Find out what's new: https://aka.ms/dotnet-whats-new
Learn about the installed HTTPS developer cert: https://aka.ms/aspnet-core-https
Use 'dotnet --help' to see available commands or visit: https://aka.ms/dotnet-cli-docs
Write your first app: https://aka.ms/first-net-core-app
--------------------------------------------------------------------------------------
System.NullReferenceException: Object reference not set to an instance of an object.
   at System.Globalization.CalendarData.NormalizeDatePattern(String input)
   at System.Globalization.CalendarData.LoadCalendarDataFromSystem(String localeName, CalendarId calendarId)
   at System.Globalization.CalendarData..ctor(String localeName, CalendarId calendarId, Boolean bUseUserOverrides)
   at System.Globalization.CultureData.GetCalendar(CalendarId calendarId)
   at System.Globalization.CultureData.DateSeparator(CalendarId calendarId)
   at System.Globalization.DateTimeFormatInfo.InitializeOverridableProperties(CultureData cultureData, CalendarId calendarId)
   at System.Globalization.DateTimeFormatInfo.set_Calendar(Calendar value)
   at System.Globalization.CultureInfo.get_DateTimeFormat()
   at System.Globalization.CultureInfo.GetFormat(Type formatType)
   at System.Globalization.DateTimeFormatInfo.get_CurrentInfo()
   at System.Globalization.DateTimeFormatInfo.GetInstance(IFormatProvider provider)
   at System.Globalization.TimeSpanFormat.TryFormat(TimeSpan value, Span`1 destination, Int32& charsWritten, ReadOnlySpan`1 format, IFormatProvider formatProvider)
   at System.TimeSpan.TryFormat(Span`1 destination, Int32& charsWritten, ReadOnlySpan`1 format, IFormatProvider formatProvider)
   at System.Text.StringBuilder.AppendFormatHelper(IFormatProvider provider, String format, ParamsArray args)
   at System.TimeZoneInfo..ctor(Byte[] data, String id, Boolean dstDisabled)
   at System.TimeZoneInfo.GetTimeZoneFromTzData(Byte[] rawData, String id)
   at System.TimeZoneInfo.GetLocalTimeZoneFromTzFile()
   at System.TimeZoneInfo.CachedData.CreateLocal()
   at System.DateTime.get_Now()
   at System.DateTimeOffset.get_Now()
   at Microsoft.AspNetCore.DeveloperCertificates.XPlat.CertificateGenerator.GenerateAspNetHttpsCertificate()
   at Microsoft.DotNet.Cli.AspNetCoreCertificateGenerator.GenerateAspNetCoreDevelopmentCertificate()
   at Microsoft.DotNet.Configurer.DotnetFirstTimeUseConfigurer.Configure()
   at Microsoft.DotNet.Cli.Program.ConfigureDotNetForFirstTimeUse(IFirstTimeUseNoticeSentinel firstTimeUseNoticeSentinel, IAspNetCertificateSentinel aspNetCertificateSentinel, IFileSentinel toolPathSentinel, Boolean hasSuperUserAccess, DotnetFirstRunConfiguration dotnetFirstRunConfiguration, IEnvironmentProvider environmentProvider)
   at Microsoft.DotNet.Cli.Program.ProcessArgs(String[] args, ITelemetry telemetryClient)
   at Microsoft.DotNet.Cli.Program.Main(String[] args)

Looking at the backtrace, this might be due to ICU, here is the details of ICU packages installed:
havana ~ > rpm -qa | grep libicu
libicu64_1-ledata-64.1-1.1.noarch
libicu-suse64_1-64.1-1.1.x86_64
libicu-devel-64.1-1.1.x86_64

@tarekgh @dagood can you please help route it? Is it a known problem?

I'll try to look. Is there any more data here can help? like debugger dump or more info about the running environment (e.g. what is the locale on the failed machine)?
@dagood if we have a machine or image I can use to check that would be helpful if you point me to that. thanks.
CC @krwq

No repro with a from scratch install from tarball on a opensuse/tumbleweed container. 😕
docker run -it --rm opensuse/tumbleweed bash -c '
  zypper install -y wget tar gzip libicu
  wget https://download.visualstudio.microsoft.com/download/pr/26d4dc1f-f674-4902-9921-f287f65266f9/ef8a67939f0c3e5729b2674a5e013328/dotnet-sdk-3.0.100-preview4-011223-linux-x64.tar.gz
  mkdir -p $HOME/dotnet && tar zxf dotnet-sdk-3.0.100-preview4-011223-linux-x64.tar.gz -C $HOME/dotnet
  export DOTNET_ROOT=$HOME/dotnet
  export PATH=$PATH:$HOME/dotnet
  dotnet tool'

...

Welcome to .NET Core 3.0!
---------------------
SDK Version: 3.0.100-preview4-011223

Telemetry
---------
The .NET Core tools collect usage data in order to help us improve your experience. The data is anonymous. It is collected by Microsoft and shared with the community. You can opt-out of telemetry by setting the DOTNET_CLI_TELEMETRY_OPTOUT environment variable to '1' or 'true' using your favorite shell.

Read more about .NET Core CLI Tools telemetry: https://aka.ms/dotnet-cli-telemetry

----------------
More about .NET Core CLI Tools telemetry: https://aka.ms/dotnet-cli-telemetry
Explore documentation: https://aka.ms/dotnet-docs
Report issues and find source on GitHub: https://github.com/dotnet/core
Find out what's new: https://aka.ms/dotnet-whats-new
Learn about the installed HTTPS developer cert: https://aka.ms/aspnet-core-https
Use 'dotnet --help' to see available commands or visit: https://aka.ms/dotnet-cli-docs
Write your first app: https://aka.ms/first-net-core-app
--------------------------------------------------------------------------------------
Required command was not provided.
Usage: dotnet tool [options] [command]

Options:
  -h, --help   Show command line help.
...

I think it's reasonable to try to repro on that base container if we can get more info, though.

Locale is C.UTF-8, if you need more information you have to let me know :-)

Apologies, I see now that looks pretty passive aggressive. I was just expressing my hope that @tarekgh can think of more things to ask about to get to the bottom of this. 🙂 I don't know enough about this area to know what other stuff to gather.

@ismail @dagood  what you have sent both is very useful. I'll try to look more here. Thanks for your help.

I can repro this now.

Issue moved to dotnet/corefx #37098 via ZenHub


Feedback form error

@nikhilkotiya Can you please provide the description for the error so that I can assign you the issue

@khannakshat7  When i submitting Feedback  form then showing error csrf_token  is missing and when i check feedback.html template then  i found csfd token is missing

@nikhilkotiya You can fix the bug it is easy

@nikhilkotiya are u facing any problems?

yes i Push my code on branch nikhil and after that i create a pull from branch nikhil but i dont know what i have do next?

Yes ur part is done. I'll review the code and merge if there are no errors

Sry i dont know about git too much. I commit my code 3 time and pull request 2 times  so is there any eshu?

No there is no issue with that
On Tue, 9 Mar, 2021, 11:55 am nikhilkotiya<EMAIL_ADDRESS>wrote:

Sry i dont know about git too much. I commit my code 3 time and pull
request 2 times so is there any eshu?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/khannakshat7/Elektra/issues/7#issuecomment-793450547,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH3XXPGKGMVMBFKOWFAK433TCW5NPANCNFSM4Y2Y3G3Q
.

OK what i have do next?

We'll review and merge the code if it has no errors.
On Tue, 9 Mar, 2021, 12:00 pm nikhilkotiya<EMAIL_ADDRESS>wrote:

OK what i have do next?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/khannakshat7/Elektra/issues/7#issuecomment-793452940,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH3XXPE2KMUA7IVUMPKMLVDTCW6BPANCNFSM4Y2Y3G3Q
.

Ok sir


Close SSH connections in tests - to avoid spurious threads

Ref https://github.com/fractal-analytics-platform/fractal-server/pull/1728#issuecomment-2328849776

Closed within #1728, with respect to the offending test.
#1730 remains open.


include for class Pair missing

#include "capu/Config.h" missing
Missing PairTest

Is fixed in CAPU 0.19.0


decode优化

What problem does this PR solve?

decode优化，剩余包处理
group测试bug修复
其他代码、注释优化

What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to be included in the release note

@lonng

protobuf包地址已由github.com/golang/protobuf改为google.golang.org/protobuf，故而CI检查报错。先合PR并后统一修改。


gs get stuck full cpu and no popup window.

RPI 3B+ latest code， desktop got stuck, checked from putty. top suggested 100% CPU on each core.

Kill from putty

sudo -E DISPLAY=:0 ./gs

wlan1 & wlan2 support monitor mode, which is rtl8192cu card.
$ ifconfig
docker0: flags=4099<UP,BROADCAST,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        ether 02:42:51:af:e3:b7  txqueuelen 0  (Ethernet)
        RX packets 0  bytes 0 (0.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 0  bytes 0 (0.0 B)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

eth0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::8112:16ad:ead7:dc16  prefixlen 64  scopeid 0x20<link>
        inet6 2408:8240:610:61f0:f198:aad0:39d1:abfa  prefixlen 64  scopeid 0x0<global>
        ether b8:27:eb:f3:69:a6  txqueuelen 1000  (Ethernet)
        RX packets 3134  bytes 914784 (893.3 KiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 970  bytes 270711 (264.3 KiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

lo: flags=73<UP,LOOPBACK,RUNNING>  mtu 65536
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>
        inet6 ::1  prefixlen 128  scopeid 0x10<host>
        loop  txqueuelen 1000  (Local Loopback)
        RX packets 5  bytes 284 (284.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 5  bytes 284 (284.0 B)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

wlan0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::25d8:b6f9:e8b3:2573  prefixlen 64  scopeid 0x20<link>
        ether b8:27:eb:a6:3c:f3  txqueuelen 1000  (Ethernet)
        RX packets 7708  bytes 893984 (873.0 KiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 9031  bytes 6097881 (5.8 MiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

wlan1: flags=4099<UP,BROADCAST,MULTICAST>  mtu 1500
        unspec 14-CF-92-0B-EA-04-30-30-00-00-00-00-00-00-00-00  txqueuelen 1000  (UNSPEC)
        RX packets 98697  bytes 22437402 (21.3 MiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 123  bytes 25826 (25.2 KiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

it seems duplicated with HDMI monitor. currently, I add two wifi cards.
gs compiles ok, but it doesn't work properly #20


v2.1 or v3.0

It's not great to be needing to use dev-master to get ss4 compatibility.
Can the changes on master get a release? 🙏

Any possibility of getting this done @sheadawson or perhaps opening up the repo to other people?


tensorflow2.0 detected 'xla_gpu' , but 'gpu' expected

System information

Linux Ubuntu 16.04 :
TensorFlow installed from binary, tensorflow2.0-gpu-beta1:
Python version 3.6.4:
CUDA 9.1, cuDNN 7.0:
GPU: Titan Xp.

tensorflow detected 'xla_gpu' , but 'gpu' expected
train_model = keras.utils.multi_gpu_model(train_model, gpus=2)   File "/usr/local/lib/python3.6/site-packages/tensorflow/python/keras/utils/multi_gpu_utils.py", line 182, in multi_gpu_model     available_devices)) ValueError: To call multi_gpu_modelwithgpus=2, we expect the following devices to be available: ['/cpu:0', '/gpu:0', '/gpu:1']. However this machine only has: ['/cpu:0', '/xla_gpu:0', '/xla_gpu:1', '/xla_cpu:0']. Try reducing gpus.

In tf2.0, using distribute_strategy to utilize multiple gpus is a better way.

In tf2.0, using distribute_strategy to utilize multiple gpus is a better way.

Thank you , but
if I use
    mirrored_strategy = tf.distribute.MirroredStrategy(         devices=["/gpu:0", "/gpu:1"])
then:
RuntimeError: Error copying tensor to device: /job:localhost/replica:0/task:0/device:GPU:0. /job:localhost/replica:0/task:0/device:GPU:0 unknown device.
if I use     mirrored_strategy = tf.distribute.MirroredStrategy(         devices=["/xla_gpu:0", "/xla_gpu:1"])
then:
ValueError: Unknown attribute: 'xla_gpu' in '/xla_gpu:0'

seems that you are using a XLA compiled tf build. Not sure if it's a bug. Need some confirmation from the tf team members.

problem solved, after update CUDA to 10.0, CUDNN to 7.4

Does this problem solved for CUDA 10.0 and CUDNN 7.4
I have installed CUDA 10.1 and CUDNN 7.5, it does not work.
tf.distribute.MirroredStrategy( devices=["/xla_gpu:0", "/xla_gpu:1"])
ValueError: Unknown attribute: 'xla_gpu' in '/xla_gpu:0'

same error.

Exact same issue for me using
CUDA 10.2
cuDNN 7.6
Tensorflow 2.0
Ubuntu 18.04
It might be a compatibility issue with the latest CUDA and cuDNN with tf? Not sure.

Any news concerning this issue ?

@mbenguig from the sound of all the reading I've done, the latest version of tf has some bugs in it and doesn't work properly? Honestly, until they can fix some of these issues I've given up on tensorflow roulette and am piping some of my work to pytorch to try my luck there.

Finally, you can get rid of this issue by uninstalling / reinstalling (tested on Ubuntu 18.04):

Tensorflow 2.0
CUDA 10.0
cuDNN 7.6.4 (described as dedicated for CUDA 10.0)

https://www.tensorflow.org/install/source#tested_build_configurations. You will get xla devices with corresponding non xla devices.

same issue
CUDA 10.2
cuDNN 7.6
Tensorflow 1.15
Ubuntu 18.04

Same.
CUDA 10.1
CuDNN 7.6
Tensorflow 2.1.0

Same issue here:
CUDA 10.1
CuDNN 7.6
Tensorflow 1.15

Same.
CUDA 10.1
CuDNN 7.6
Tensorflow 2.1.0

Same.
CUDA 10.2
CuDNN <IP_ADDRESS>
Tensorflow 2.1.0

Same:

CUDA 11.0.1 RC
cuDNN 8.0.2 RC2
TensorFlow 2.2.0

Hi,
I had the same issue and I think I figured out a way around it. In my case, I am working on an HPC and I intalled keras on my /.local, whereas Tensorflow and CUDA are installed by the IT staff, anyway I encountered the same error above. I am using Tensorflow==1.15.0 and Keras==2.3.1
I noticed that the message error:

ValueError: To call multi_gpu_model with gpus=2, we expect the following devices to be available: ['/cpu:0', '/gpu:0', '/gpu:1'].
However this machine only has: ['/cpu:0', '/xla_cpu:0', '/xla_gpu:0', '/xla_gpu:1']. Try reducing gpus.

is located in the following file of keras, line 184:

/home/.local/lib/python3.7/site-packages/keras/utils/multi_gpu_utils.py

I solved this, by replacing the line 175 with the following:

target_devices = ['/cpu:0'] + ['/gpu:%d' % i for i in target_gpu_ids] (before)
target_devices = ['/cpu:0'] + ['/xla_gpu:%d' % i for i in target_gpu_ids] (after)

Moreover, I modified the following keras file:

/home/.local/lib/python3.7/site-packages/keras/backend/tensorflow_backend.py

so I replaced the line 510 with:

return [x for x in _LOCAL_DEVICES if 'device:gpu' in x.lower()] (before)
return [x for x in _LOCAL_DEVICES if 'device:XLA_GPU' in x] (after)

Long story short only to say that apparently this is a bug of Keras and not of some environmental setup.
After such modification my network was able to run with the xla_gpus, I hope this is somehow helpful.
Cheers,
Michele

@micbia
Tried your solution, but for me the files .../multi_gpu_utils.py and .../tensorflow_backend.py don't even exist.
I have tensorflow 2.3.1 though.
In general, is there any solution to tensorflow/keras being unable to use XLA-GPU that does not involve reinstalling CUDA+tf+cudnn?

@arb-git
notice that path for the files I mentioned above are for keras module, If you are not using keras then I can suggest you to use the tf.keras.
The location of the multi_gpu_utils.py module for TensorFlow2.3.0 (stable) is basically the same tf.keras.utils.multi_gpu_model, and the line 172 is to change as I suggested
In the case of tensorflow_backend.py it changes a little bit tf.keras.backend.backend and the line 783 is to change, but by replacing:

return [x.name for x in _LOCAL_DEVICES if x.device_type == 'GPU'] (before)
return [x.name for x in _LOCAL_DEVICES if x.device_type == 'XLA_GPU'] (after)

I have not tried personally on TensorFlow2.3.0. It would be interesting if you could share at let us know if it works in your case with this changes.
Cheers,
Michele


Add IP and port range handling for WebRTC ICE connections

Closes #16 .

Yep, I have been testing it in docker! This does not need any special networking mode.
I'm currently using it with this docker-compose.yml:
lightspeed_webrtc:
    image: lightspeed-webrtc-test
    container_name: lightspeed_webrtc
    restart: unless-stopped
    entrypoint: lightspeed-webrtc
    command: --addr=<IP_ADDRESS> --ip=<IP_ADDRESS> --ports=19050-19100 run
    ports:
        - 8085:8080 # WebRTC
        - 65535:65535/udp # RTP
        - 19050-19100:19050-19100 # WebRTC PeerConnection
        - 19050-19100:19050-19100/udp # WebRTC PeerConnection UDP

Yep, I have been testing it in docker! This does not need any special networking mode.
I'm currently using it with this docker-compose.yml:
lightspeed_webrtc:
    image: lightspeed-webrtc-test
    container_name: lightspeed_webrtc
    restart: unless-stopped
    entrypoint: lightspeed-webrtc
    command: --addr=<IP_ADDRESS> --ip=<IP_ADDRESS> --ports=19050-19100 run
    ports:
        - 8085:8080 # WebRTC
        - 65535:65535/udp # RTP
        - 19050-19100:19050-19100 # WebRTC PeerConnection
        - 19050-19100:19050-19100/udp # WebRTC PeerConnection UDP


add gitignore and update README file.

@maboss-YCMan/one
hi, two branch: master and development.
for different stability.
our work should be working on topic branch with descriptive name

@maboss-YCMan/one  use this sign for merge confirming
🤘

🤘

:metal:


docs: fix invalid syntax in some sources examples

Summary
TOML 1.0 doesn't support multi-line for inline tables, so those examples are invalid.

Oh this is tragic — these were intentionally split for readability otherwise you have to scroll to see the full example. Is there anything we can do about that?

Oh this is tragic — these were intentionally split for readability otherwise you have to scroll to see the full example. Is there anything we can do about that?

There's another way yes. Taking the first example, it could also be written as:
[tool.uv.sources.httpx]
git = "https://github.com/encode/httpx"
tag = "0.27.0"

Happy to do a follow-up PR if you think this would be better.

I'm not sure, to be honest. @charliermarsh do you have thoughts?


Windows XP Build

Hey, I downloaded the latest build and I'm using the Windows XP build on a virtual machine through VMware 17.5.2. When I try to run a game, I keep getting an error message about 'InitializeCriticalSectionEx' in KERNEL32.dll.

I was wrong to download version, sorry.