fix(DDI-1321): CSS Changes

Updated border on Hover to a dashed line
Changed padding to 32px (--goa-spacing-xl) and top padding to 28px on vertical version
Updated spacing between icon and text (Drag and Drop files…) to --goa-spacing-s (12px)

:tada: This PR is included in version 4.0.0-alpha.151 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.190 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-alpha.17 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.21 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.11.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.11.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


refactor: pull docker image earlier

some cases when docker run ... is ran before the image is pulled

merge with master please @ANGkeith


remove jenkins warehouse data sync

Context
This is postgresql wharehouse data sync no longer needed because the data sync is done via the govuk_env_sync python code that runs as a cron job on the database machines themselves.
This will hopefully solves the issues seen during the Jenkins jobs: Copy Data to Staging and Copy Data to integration
Decisions

remove all postgresql wharehouse data sync from Jenkins in the puppet code

Hmm, I guess these changes should have happened at the same time as https://github.com/alphagov/env-sync-and-backup/pull/140
I didn't realise the two things were quite so tightly coupled.

Unfortunately yes they are. I need to delete the password in hiera secrets but not a blocker.

no worries, thanks :-)


[WIP] Fix derive of PySequenceProtocol trait

As shown in #421, implementing PySequenceProtocol is not possible in v0.6.0 because the custom derive macro shuffles the argument types. This PR fixes the method types, and adds some tests to check a PySequenceProtocol implementor compiles as intended.
TODO:

[x] Fix argument types of __getitem__, __setitem__, __delitem__, __repeat__, __concat__
[ ] Fix return types of __inplace_concat__ and __inplace_repeat__: it should be something like PyResult<&'mut Self> or PyResult<PyMutRef<Self>> since these methods do not have ownership over the Rust struct.

Rebased on latest master. CI fails because of 9a9b519.


Miscellaneous fixes

Coverage decreased (-0.26%) to 74.22% when pulling ffc886429900e3892005b39595b69982883b5ccf on Neob91:fixes into bf362ce3e000dba43b24f583ce49b0772a6f0af5 on bodbdigr:master.

Coverage decreased (-0.26%) to 74.22% when pulling ffc886429900e3892005b39595b69982883b5ccf on Neob91:fixes into bf362ce3e000dba43b24f583ce49b0772a6f0af5 on bodbdigr:master.


"Warning: parameter method=adaptive is unrecognized and unused."

I get this warning alot in stdout output when running the script.

Thanks. Apparently pymbar changed their options. I just submitted a fix.


Create custom resolvers for interpreted object types

As mentioned in #45, we recently made object types default to string. The logic is that, for whatever reason the OAS specifies an unsupported type, we can also interpret that type as a string. We need to create a resolver to handle this case.

Is this about stringifying non-scalar response bodies whose schemas are not fully described in the OAS? I believe we handle this case now.


Fix passing key through find

RELEASE_NOTES=[BUGFIX] Fix optional key passed through find
Signed-off-by: Dominik Schulz<EMAIL_ADDRESS>

Codecov Report

Merging #1440 into master will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1440      +/-   ##
==========================================
- Coverage   64.28%   64.25%   -0.02%
==========================================
  Files         203      203
  Lines       11603    11607       +4
==========================================
  Hits         7458     7458
- Misses       3334     3338       +4
  Partials      811      811

Impacted Files
Coverage Δ

internal/action/show.go
46.09% <0.00%> (-1.49%)
:arrow_down:

Continue to review full report at Codecov.

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


The cast action casts all MagicalEntity spells

The handle_spell_casting method works by checking for the 'cast' action but that means whenever the player presses the key to cast all MagicalEntities will cast their spell.

Fixed in 080c937.


Degraded volume not replicating

After restarting some of the nodes there is only one healthy replica remaining. This replica is not being replicated to the other nodes. This may be related to #412 or #350. I attempted to delete the unhealthy replicas, detach and attach the volume and change the engine image without success. Would appreciate some advice before I try anything else and the data is gone.
2/26/2019 6:11:20 PM time="2019-02-26T17:11:20Z" level=warning msg="Error syncing Longhorn engine longhorn-system/pvc-f7218fa5ec2a11e8-e-c1f4c4d5: fail to sync engine for longhorn-system/pvc-f7218fa5ec2a11e8-e-c1f4c4d5: fail to start rebuild for pvc-f7218fa5ec2a11e8-r-ad6a7753 of pvc-f7218fa5ec2a11e8-e-c1f4c4d5: timed out waiting for the condition"
2/26/2019 6:11:20 PM time="2019-02-26T17:11:20Z" level=info msg="Event(v1.ObjectReference{Kind:\"Engine\", Namespace:\"longhorn-system\", Name:\"pvc-f7218fa5ec2a11e8-e-c1f4c4d5\", UID:\"f7f809ad-ec2a-11e8-af7d-000c29dc70ce\", APIVersion:\"longhorn.rancher.io/v1alpha1\", ResourceVersion:\"27622422\", FieldPath:\"\"}): type: 'Normal' reason: 'Rebuilding' Start rebuilding replica pvc-f7218fa5ec2a11e8-r-fce6bde9 with IP <IP_ADDRESS> for pvc-f7218fa5ec2a11e8"
2/26/2019 6:11:20 PM time="2019-02-26T17:11:20Z" level=error msg="Failed rebuilding <IP_ADDRESS> of pvc-f7218fa5ec2a11e8: failed to add replica address='tcp://<IP_ADDRESS>:9502' to controller 'pvc-f7218fa5ec2a11e8': Failed to execute: /var/lib/rancher/longhorn/engine-binaries/rancher-longhorn-engine-v0.4.0/longhorn [--url http://<IP_ADDRESS>:9501 add tcp://<IP_ADDRESS>:9502], output time=\"2019-02-26T17:11:20Z\" level=info msg=\"Adding replica tcp://<IP_ADDRESS>:9502 in WO mode\" \ntime=\"2019-02-26T17:11:20Z\" level=fatal msg=\"Error running add replica command: Bad response: 500 500 Internal Server Error: {\\\"actions\\\":{},\\\"code\\\":\\\"Server Error\\\",\\\"detail\\\":\\\"\\\",\\\"links\\\":{\\\"self\\\":\\\"http://<IP_ADDRESS>:9501/v1/replicas\\\"},\\\"message\\\":\\\"Replica must be closed, Can not add in state: open\\\",\\\"status\\\":500,\\\"type\\\":\\\"error\\\"}\\n\" \n, error exit status 1"
2/26/2019 6:11:20 PM time="2019-02-26T17:11:20Z" level=info msg="Event(v1.ObjectReference{Kind:\"Engine\", Namespace:\"longhorn-system\", Name:\"pvc-f7218fa5ec2a11e8-e-c1f4c4d5\", UID:\"f7f809ad-ec2a-11e8-af7d-000c29dc70ce\", APIVersion:\"longhorn.rancher.io/v1alpha1\", ResourceVersion:\"27622422\", FieldPath:\"\"}): type: 'Warning' reason: 'FailedRebuilding' Failed rebuilding replica with IP <IP_ADDRESS>: failed to add replica address='tcp://<IP_ADDRESS>:9502' to controller 'pvc-f7218fa5ec2a11e8': Failed to execute: /var/lib/rancher/longhorn/engine-binaries/rancher-longhorn-engine-v0.4.0/longhorn [--url http://<IP_ADDRESS>:9501 add tcp://<IP_ADDRESS>:9502], output time=\"2019-02-26T17:11:20Z\" level=info msg=\"Adding replica tcp://<IP_ADDRESS>:9502 in WO mode\" \ntime=\"2019-02-26T17:11:20Z\" level=fatal msg=\"Error running add replica command: Bad response: 500 500 Internal Server Error: {\\\"actions\\\":{},\\\"code\\\":\\\"Server Error\\\",\\\"detail\\\":\\\"\\\",\\\"links\\\":{\\\"self\\\":\\\"http://<IP_ADDRESS>:9501/v1/replicas\\\"},\\\"message\\\":\\\"Replica must be closed, Can not add in state: open\\\",\\\"status\\\":500,\\\"type\\\":\\\"error\\\"}\\n\" \n, error exit status 1"
2/26/2019 6:11:20 PM time="2019-02-26T17:11:20Z" level=error msg="Removed failed rebuilding replica <IP_ADDRESS> of pvc-f7218fa5ec2a11e8"

I can send the support bundle if needed. If so where should I send it?

@NchemIcaLS You can send the support bundle to me at my email (listed in my profile), or on Slack @yasker

@NchemIcaLS I've received the support bundle.
There is an issue in that volume: "Too many snapshots created". The snapshot limit is around 200. There was a related issue we fixed in #253 . But since you're running v0.4.0, this can caused by something else.
2019-02-26T17:12:18.203809066Z time="2019-02-26T17:12:18Z" level=error msg="Error in request: Too many snapshots created"
2019-02-26T17:12:18.204134364Z <IP_ADDRESS> - - [26/Feb/2019:17:12:18 +0000] "POST /v1/replicas HTTP/1.1" 500 171
2019-02-26T17:12:48.321996606Z time="2019-02-26T17:12:48Z" level=info msg="Connecting to remote: <IP_ADDRESS>:9502"

Can you try the following steps:

Make a backup of the current volume.
Reduce the replica count of the volume to the number of current healthy replicas.
2.1 If I read it correctly, you have only one replica pvc-8cfc741eec1a11e8-r-61c3c623 in RW state of the volume. So you can reduce the number to 1.
Delete the replicas stuck in the rebuild state. Now you should have only healthy replicas running.
Delete the snapshots manually except for the latest e.g. 25 snapshots.
Increase the replica count of the volume by 1. Wait for the rebuild to complete.
Repeat step 5 till you get desired replica count number for the volume.

There was a limited amount of log before Too many snapshots created message. If anything abnormal happened with the volume?

I didn't realize there were that many snapshots. The backup settings are set to retain 7 for the past week (backup 1x each day) and 12 for the year (backup 1x each month). This volume has only existed for 3 months so even if I made a snapshot every day that would only be around 100 snapshots. Is there a reason there are this many snapshots? Do I need to configure something differently?
The snapshots also only appear on the volume details. Only 8 snapshots are showing under the backup page when selecting the same volume.
After manually deleting several backups I was able to backup the current state. However, I can't reduce the replica count because of the following error:
volume must be healthy to update replica count

I have 4 more volumes that are in the exact same state (they are not attached since I was concerned about the data). Nothing abnormal happened except that the Kubernetes cluster is on premise and the servers were restarted causing individual nodes to temporarily drop out; however longhorn has survived this in the past, even a power outage.

I managed to get 3/4 volumes back to healthy by deleting snapshots and then de/re attaching them. However pvc-85d77fd9ea6f11e8 is still not rebuilding even after deleting some snapshots. I tried to analyze the logs but couldn't find out why. I sent you another email with a new support bundle.

@NchemIcaLS
Sorry, I forgot about the replica count update limitation...
Seems pvc-85d77fd9ea6f11e8-r-e5e50d90 is in trouble. We have seen this before we haven't found the root cause of this. I would file a separate issue for this.

2019-02-26T21:42:21.051086295Z time="2019-02-26T21:42:21Z" level=error msg="Error in request: Invalid name volume-snap-c-wyw6oz-f0590eb8.img does not match pattern: volume-head-(\d)+.img"

If you delete the replica pvc-85d77fd9ea6f11e8-r-e5e50d90(after backup the volume), the rebuild should resume.
Regarding snapshots and backups, there is nothing wrong with your setup. Snapshots are different from backups in Longhorn. Snapshots colocated with the volume data, so if volume lost, the snapshots of the volume would be lost with it as well. But backups are located outside Longhorn, so if volume lost, there is no impact on backups since it's stored outside Longhorn. Longhorn can still retrieve the backups and restore the volume.
The number limitation only applies to the snapshots, not backups. You can have as many backups as you want.
If you've created the volume before Longhorn v0.3.3, there was a bug (#253) that would continuously create system snapshot for rebuilding purpose, so it can result in Too many snapshots error. That state will be with the volume until you delete the snapshots and free up the slots. There also can be another bug we haven't found yet result in the same thing. I will perform more test in the lab to see if it can happen in any other situation.

Thank you for explaining the difference between snapshots and backups. That makes sense now.
Unfortunately that is replica is the only healthy one so after deleting it the volume is essentially gone as there is no data now. I also couldn't make a backup of that last replica as the entire volume would immediately crash and go into a faulted state. But thanks to the backups I had scheduled every day I was easily able to restore to a backup from the previous day. The data lost was negligible so this wasn't a problem. I was pleasantly surprised how well the backups worked and a great feeling that there are so many fail-overs even if something happens.
Thanks again for the support!

@NchemIcaLS Oh, sorry, I didn't notice pvc-85d77fd9ea6f11e8-r-e5e50d90 is the only RW replica in the volume. Well, that explains it.
The replica "Invalid name" issue will be tracked at #412 .


Make indirect variable explicit for PHP 7

Fixes two failing tests (ActiveRecordTest::test_delegate_set_attribute and test_delegate_setter_gh_98) on master under PHP7 by making the interpretation of $this->$item['to']->$delegated_name explicit.
PHP 7 interprets this as ($this->item)['to']->$delegated_name, we want it to mean $this->{$item['to']}->$delegated_name.
See http://php.net/manual/en/migration70.incompatible.php for the change in PHP behaviour

Can you update this branch so Travis runs with php 7 too?

Also, there might be more locations which need the more explicit syntax


How can one restart the library/stack with a similar effect to killing and restarting the app?

Hello friends,
first thanks for this wonderful library - its mostly a breeze for me and working with iOS and Android properly.
What happens to me now that more people are using my apps and sometimes we hit reliability issues - for example, in some devices in some situations, users just click all around frantically... and the app just stops scanning properly. (Of course this can also be just logic on the app side- but this happens regularly and in different apps..)
What I usually do in those times is to "kill" the app, which always works but I can't really ask users to do.
Is there any way to achieve a similar behaviour without closing the app ? I tried closing and opening the BLE manager but that didn't seem to trigger a low level restart.
Thanks in advance!

I'm excited to inform you that we've released a new version of react-native-ble-plx. Many of the concerns and bugs have been addressed in this update.
If you encounter any issues or have further suggestions, please don't hesitate to open a new issue.


Change ARM build image for ripgrep

VSCode version - 1.65.2, Windows x64.
Im controlling RPI via Remote-SSH for over a year now.
on latest version, global search is not working on remote machine anymore.
when opening a local folder, global search works great.
when opening remote folder, global search is only working on open files and only if you have viewed this file 1 time at least!
what i tried:

not use ignore files
remove all ignore patterns
delete user JSON settings file
verify that the "search only in open editors" button is not selected

im not sure what else to try

Please do the following

Run the command "Set log level" and change it to "Trace"
Run the search again
Show the search-related messages in the "Log (Window)" and "Log (Extension Host)" output panel

Please do the following

Run the command "Set log level" and change it to "Trace"
Run the search again
Show the search-related messages in the "Log (Window)" and "Log (Extension Host)" output panel

thanks, here is the Log(Window) trace:
https://pastebin.com/wJ0SpUrF
===============================
Log (Extension Host) output show no log from this action, but Log (Remote Extension Host) does show and here it is:
https://pastebin.com/AGgugcDf

I guess this is the problem
[2022-03-22 12:35:46.467] [exthost] [debug] RipgrepSearchEH#search /home/pi/.vscode-server/bin/c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1/node_modules/@vscode/ripgrep/bin/rg: /lib/arm-linux-gnueabihf/libc.so.6: version `GLIBC_2.28' not found (required by /home/pi/.vscode-server/bin/c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1/node_modules/@vscode/ripgrep/bin/rg)

If this used to work, I guess it would be from https://github.com/microsoft/ripgrep-prebuilt/commit/f764a98a23560f7a5ad36bbde25e0372ff276253
@deepak1556 do we have a certain GLIBC requirement for vscode? Should I build ripgrep in the same image we use for vscode?

I haven't addressed this for our arm server builds yet but we will target glibc >= 2.24 based on this table https://github.com/nodejs/node/blob/master/BUILDING.md#platform-list.

Should I build ripgrep in the same image we use for vscode?

Yup that sounds good and in your case it only needs to be done for the arm flavors of ripgrep which rely on the gnu C library, intel flavors are built using musl so they should be fine.
I can share the image details once I address https://github.com/microsoft/vscode-remote-release/issues/4069

Sorry forgot to update here, our arm64 server uses the following image to build https://github.com/microsoft/vscode-linux-build-agent/blob/main/centos7-devtoolset8-arm64/Dockerfile which provides support for glibc >= 2.17
armhf server uses https://github.com/microsoft/vscode-linux-build-agent/blob/main/stretch-armhf/Dockerfile which supports glibc >= 2.24

@roblourens @deepak1556
is there any plan to solve it?

Example of an issue that will be addressed: https://github.com/microsoft/vscode-remote-release/issues/6930
We technically support glibc 2.17 in vscode, but ripgrep is not built for it

We are building a statically-linked version of ripgrep now, this should be resolved.


Add show-on-mobile

There is already have hide-on-mobile, I plan to add show-on-mobile class.

Inspirited by https://github.com/cobyism/gridism/blob/master/gridism.css#L120

hide-on-mobile show-on-mobile
or
hide-on-mobiles show-on-mobiles?

Added show-on-mobile


Cherry-picked commits for the v1.62-lcm branch

Closed and re-opened PR to force Travis to re-run the build.


Rephrase View Street View button

[x] Rephrase View Street View because it does not translate well and is confusing to people
suggested: "See Street View" or "Browse Street View"

^Here is how the Google hover words it. doobneek suggests "See how the entrance of this location looks" or something like that.
(this issue was moved from #56)

@adambard1
Fixed. and deployed  here for test


Backward compatibility for VS2015

Greetings,
we use VS 2015 for internal company reasons which doesn't have the most up-to-date C# compiler features. We tried installing the MS CodeDom compiler nuget package but with no success.
We want to be able to have the MLAPI source code in our project and successfully compile it. But it seems it was using a few newer C# features like inline out declare and inline 'is' checking. They were in a few spots so it wasn't a big deal to remove them but it would be nice to have that compatibility built-in so when we update MLAPI we don't have redo things.
In NetworkedBehaviour.cs we changed (in a couple of spots):
if (stream is BitStream bitStream)

to:
BitStream bitStream = stream as BitStream;
if (bitStream != null) ...

And in RelayTransport.cs we changed all instances of the inlined 'out byte b' to be declared separately. 'byte b' 'out b'
Same thing in EnetTransport.cs with 'out uint peerId'
Would be appreciated to have older C# support built-in.
Thanks.

Wait what C# version does Unity support? C# 6?

Just to clarify, We're using Unity 2019, it compiles the code fine. It's just that VS2015 isn't familiar with the inlined 'is' and 'out' declarations so it throws out compile errors. I believe VS2015 is C# 6 correct.

Just to clarify, We're using Unity 2019, it compiles the code fine. It's just that VS2015 isn't familiar with the inlined 'is' and 'out' declarations so it throws out compile errors. I believe VS2015 is C# 6 correct.

If the MLAPI is not actually using many of the C# 7 features, we might aswell downgrade to C# 6.

Yep as far as we saw it was just those outs and is checks. That would be awesome thanks!

Yep as far as we saw it was just those outs and is checks. That would be awesome thanks!

There is ~50 compiler problems when downgrading  to C# 6. I will work on it straight away.

All the errors we had were fixed by applying the changes I mentioned. The rest are just "Couldn't find identifier X" because of the actual 'out' and 'is' errors. If you scroll down the error list all the way to the bottom you should see the real errors.

All the errors we had were fixed by applying the changes I mentioned. The rest are just "Couldn't find identifier X" because of the actual 'out' and 'is' errors. If you scroll down the error list all the way to the bottom you should see the real errors.

Well I need to migrate things like local functions, the new set => and get => attributes and more. Not just is and out. I'm not sure what language version you are targeting. I am targeting C# 6 atm.

Language Version 6 and Target Framework Version v4.7.1 in the .csproj file.
We do use the => on some functions and getter only properties (e.g. ActorState State => (ActorState)StateNumber;) but I don't think we ever used set => and get => yeah I think that was added to C# 7 later.

Also it seems like Unity spits out 'latest' as the Language Version tag in their generated project files. But we explicitly use Language Version 6 in our custom project. (we use a custom hotreloading setup where Unity loads an external game DLL that we compile from an external 'game code' csproject)

https://github.com/MidLevel/MLAPI/pull/283 There is a C# 6 compliant version. I'm not sure it's worth it if im honest. Code gets quite dirty.

Right. Code gets a bit more verbose but more userbase coverage :D
Thanks for the fast replies btw!

Closing this as it is solved and shouldn't be an issue anymore for anyone once we switch to package version.


Add es_mx.json

Note: Spanish localisation is very inconsistent across spanish speaking countries so I made the choice to use neutral spanish, mostly used in Latin America.
Also this is not a literal translation. I made some changes to the sentences/words but the meaning remains the same.

Will include in the next release.


Added assertion failure for tests that run in record mode

Tests running in recordMode should not count as successful tests, cause then you can't see that they are not comparing the snapshots by just running the tests.
Especially after an "accidental" commit even the build server won't notice.
Cause of that I added a little assert that recordMode ist disabled.

Did this get change get reverted? We are using 1.6 and tests are marked as "passing" when run in record mode


Replace categories with tags

Categories are quite static. I’d rather prefer a tag system similar to the one Newsboat has implemented.

Sorry, I'm not planning to change this.


Bug: Eeprom_read function is sending wrong status for read operations

Product: (OC-SDR, OC-LTE, OC-LTE, OC-Power, others)
OC-SDR
Category: (Hardware, Software/stack, Firmware/bootloader, Documentation)
Software/stack
Describe the issue found
eeprom_read function always sends status as OK whether it will read or not properly.
Please provide all necessary steps to reproduce the issue
Run eeprom UT with error case for eeprom_read function.

Code is merged into master


Add links of new websites

Feel free to add links of new websites

Assign it to me

sure @Codefreak69

Hi @Gilbishkosma , can I also work on this? Or should I wait until @Codefreak69 finish first?

@dissyulina sure, feel free to submit a PR.


Schema $Id

We need to define a stables identifier ($id) for the schema. Preferably, it should be possible to retrieve the (combined) schema using the URI used as $id.

Yes - waiting on a stable base address from eHealthNetwork deployment -> issue raised last Friday

@dirkx, please see if we get get a redirect in place so that $Id/valuesets/XXX are redirected (302) to https://raw.githubusercontent.com/ehn-digital-green-development/ehn-dgc-schema/main/valuesets/XXX. Perhaps the same could be used for the entire $Id namespace?

So we can do two things

We wait for T-Systems/SAP to dicuss this with DG-Connect and then push something operational at their facility with the first versions of the DGCG.
Will take 3+ weeks most likely.

Quick 'hack' that maps id.uvci.eu -> raw.github.io/.......
And we prefix it our URLS with http(s)://id.uvci.eu/
And we later had over control of the URL to the LUX team that runs the DGCG.

As above - but we quickly register some other name like 'dgs-schemas.com' or something.

I'd go with id.uvci.eu

The URL https://id.uvci.eu/ no points to anyt something in MAIN.
The URL https://id.uvci.eu/v1/ no points to anyt something in the branch v1 (which we do not yet have)
The URL https://id.uvci.eu/next/ no points to anyt something in the branch 'next'
If this works - will arrange /work with T-Systems.

Very good.

Can we close this @dirkx?


Added Bengali  Translation for README.

🤔 This is a

[ ] Feature
[ ] Bug
[ ] Documentation Update
[x] Translation

✍ Description

Write a short description
Completed Bengali translation of readme file

Added to documentation

[x] readme file
[  ] No documentation needed

🐱💻 Use a meme/gif to describe how you feel for contributing (optional)

@x3shat, go through the Contribution md file first. Then create another pull request.


Add testing

We want to test the following:
Graph computations

[x] Subtyping.
[x] Supremum. (some tests exist)
[x] Infimum

Critical term examples

[ ] algebraic expressions => sensitivity computations

Currently running the tests produces lots of debugging print statements. It would be nice if we had a proper logging framework, and could disable logging for the tests (#1).

Closing this issue as it makes more sense to track things which need to be tested using individual issues.


Not being recognized as a 1.14.4 mod

I have downloaded BBOutlineReloaded-1.0.13-1.14.4-forge.jar and forge seems to not recognize it as a 1.14.4 mod. There are is an error while loading the mod and it says that it requires minecraft 1.14.3 and that currently running 1.14.4 (which i am). I am using twitch launcher and forge 28.0.45, which as of the moment is the latest version.
I also have 11 other mods and a few data packs

This is now resolved. Please download a new version from curseforge.


globals.load(new PackageLib()) is  necessary?

Using the lua script in java code, and to reduce the risk, I want to just use the base module,(not all of JsePlatform.standardGlobals()),
it's the config code:
`public class LuaConfig {
private final Globals globals;
public LuaConfig() {

    globals = new Globals();
    globals.load(new JseBaseLib());

    // exclude the PackageLib   (管理 Lua 模块和包。允许通过该require()函数加载外部 Lua 模块)
    //globals.load(new PackageLib());

    globals.load(new Bit32Lib());
    globals.load(new TableLib());
    globals.load(new StringLib());
    globals.load(new JseMathLib());

    LoadState.install(globals);
    LuaC.install(globals);
}

public LuaValue executeLuaScript(String script) {
    // load and exec the script  (加载并执行 Lua 脚本)
    LuaValue chunk = globals.load(script);
    return chunk.call();
}

}`
and throw expection Caused by: org.luaj.vm2.LuaError: attempt to index ? (a nil value) ,
after deubg I found only the JseBaseLib moudle can be loaded without PackageLib;
If load PackageLib , how could I only use the base function  and avoid the risk of PackageLib moudule (avoid the  Injection Attacks )?
could it be safety completed by next code ?
`
// load other  ...
globals.load(new PackageLib());
// don't use require
globals.set("require", LuaValue.NIL);
// load other ...
`

the risk scene could be looked at :  script as param in  http request


Input Box close when clicking on other software

Expected behavior
The input Box should stay open when clicking elsewhere
Actual behavior
The inpu box close as soon as we click somewhere else
Steps to reproduce the behavior
Start a combo with input box and click elsewhere
Beeftext version (e.g. Beeftext v7.0 Portable Edition).
Operating system version (e.g Windows 10)

Hi. This is not a bug. Beeftext relies on the fact that the application it was called from is the one that will become active again we the input box closes. If you can click elsewhere, this does not work anymore, that's why beeftext closes the input box automatically.


NcML for climate simulation

Description
NcML for a historical and future climate model simulations.

See https://pavics.ouranos.ca/twitcher/ows/proxy/thredds/catalog/datasets/simulations/cmip5/atmos/catalog.html?dataset=datasets/simulations/cmip5/atmos/day_MPI-ESM-LR_historical.ncml
@richardarsenault please confirm this is what you were looking for.
Future simulation coming up.

@huard yes, that EXACTLY. It's perfect, thanks. Even includes 3 members, which is even better. Can't wait to get the future one and a Bias correction method :)

We'll close when the future simulation is available.

Ah, good point. My bad!

https://pavics.ouranos.ca/twitcher/ows/proxy/thredds/catalog/datasets/simulations/cmip5/atmos/catalog.html


add operation name in async method

Please answer these questions before submitting pull request

Why submit this pull request?

[ ] Bug fix

[ ] New feature provided

[x] Improve performance

Related issues
https://github.com/apache/skywalking/issues/4012

New feature or improvement

Describe the details and related test reports.
add async span to modify OperationName method

I am thinking how to avoid using setOperationName through API , Because there are two ways to set operation name , Make user choose

Codecov Report

Merging #4015 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #4015      +/-   ##
==========================================
- Coverage   27.55%   27.55%   -0.01%
==========================================
  Files        1122     1122
  Lines       24337    24339       +2
  Branches     3527     3527
==========================================
  Hits         6707     6707
- Misses      17033    17035       +2
  Partials      597      597

Impacted Files
Coverage Δ

.../agent/core/context/trace/AbstractTracingSpan.java
59.82% <0%> (-0.54%)
:arrow_down:

...walking/apm/agent/core/context/trace/NoopSpan.java
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@cyejing Any update?

Now updated, please check


[Chapter 8] Comparison between original text and (ko) translated.

This PR is made for comparison and making comments.

related: doomspork/elixir-school#411

Looks... pretty good! :sweat_smile::sweat_drops:

Looks pretty nice! :smile_cat:


Enhance object logging

What issue does this PR address?
Objects being serialized as strings instead of sending pure objects with @ suffix to Serilog.
It makes it impossible to filter/enrich by the request/response data in serilog, and to analyze in the logging systems.
Does this PR introduce a breaking change?
No
Please check if the PR fulfills these requirements

[x] The commit follows our guidelines

Other information:
I've tested it locally, before the change those logs arrived to the Serilog enricher as strings in the Properties.
After the change, the Serilog enricher receives real structured data that we can work with.
In addition, I've fixed the logging models namespace, so you can see multiple irrelevant files changed using statements.

How does this behave with non-Serilog logging systems?

IOW - is the @ decomposition also a feature of plain ASP.NET Core logging?

I still can't find a specific documentation about @ in standard .NETCore logging,
But I ran a test locally without Serilog and with this line of code:
Logger.LogInformation(nameof(ValidatedAuthorizeRequest) + Environment.NewLine + "{@validationDetails}", details);
This is the output:
      ValidatedAuthorizeRequest
      {
        "ClientId": "SomeClientId",
        "ClientName": "SomeClientName",
        "RedirectUri": "https://some.redirect.com/auth",
        "AllowedRedirectUris": [
          "http://localhost:3000/auth",
        ],
        "SubjectId": "anonymous",
        "ResponseType": "id_token token",
        "ResponseMode": "fragment",
        "GrantType": "implicit",
        "RequestedScopes": "openid",
        "State": "4a6413622e6144179b80f606a679118e",
        "Nonce": "2008bd7b234849c1aa703ee9bdaaadf9",
        "AuthenticationContextReferenceClasses": [
          "idp:Google"
        ],
        "Raw": {
          "client_id": "SomeClientId",
          "redirect_uri": "https://some.redirect.com/auth",
          "response_type": "token id_token",
          "scope": "openid",
          "state": "4a6413622e6144179b80f606a679118e",
          "nonce": "2008bd7b234849c1aa703ee9bdaaadf9",
          "acr_values": "idp:Google"
        }
      }

IMO it looks good even without Serilog.

This is a very good idea, however it's worth mentioning that this is sort of a quiet breaking change.
If you forwarded these Identity Server logs into some log index before, the objects were simply strings, however now they would get destructured into objects, causing a type mismatch. This can lead to all kinds of mapping exceptions or lost logs in i.e. logstash / elastic search, unless you manually resolve that or rollover the index. While not a huge deal, I encountered that when upgrading to Identity Server 2.3.

yes.. you are right.


Add to readme how to install brax from pypi

Hello! I remember somebody telling me that brax can be installed from pypi. However, this is not documented here in the readme. Is this not the recommended way to install it? I would suggest it is, no?

Yes, it is probably time we let people know pypi is available     :-)
In fact I think we can switch the order and make the installing from source be the "Alternatively, if you want to..."
WDYT?

Absolutely, this is what I meant by "is this not the recommended way". Pypi first, source for the tinkerers and developers.

Looks great, thanks!


[Backport 1.x] Add sysctlInit to OpenSearch Chart

Signed-off-by: Peter Zhu<EMAIL_ADDRESS>Description
[Backport 1.x] Add sysctlInit to OpenSearch Chart
Issues Resolved
https://github.com/opensearch-project/helm-charts/pull/278
Check List

[x] Commits are signed per the DCO using --signoff

For any changes to files within Helm chart directories:

[x] Helm chart version bumped
[x] Helm chart CHANGELOG.md updated to reflect change

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

Hi @DandyDeveloper @TheAlgo please approve this backport to 1.x.
Thanks.


Please host docs

It would be nice if you hosted rustdoc and linked to it from the repo and crates.io.
Thanks!

:+1:

:+1:

Note that you now can use https://docs.rs/crate/rust-crypto instead.


Work around xlC 13.1.4 compiler bugs.

Addresses in part issue #113.

Make compatibility wrappers/functions that dance around xlC bugs. Wrappers are provied in util/compat.hpp and live in the compat namespace.

compat::end reimplements std::end but in a way that apparently does not trigger the xlC bug.
compat::compiler_barrier_if_xlc_leq() inserts a compiler reordering barrier if the compiler is xlC and the version less than or equal to that specified. Name is deliberately verbose.
compat::isinf() is an inline wrapper around std::isinf(), which apparently is sufficient to defuse an evaluation order bug with std::isinf() in switch statements.

Use compat::compiler_barrier_if_xlc_leq() in util::unitialized reference access methods to avoid improper reordering with -O2.
Use compat::isinf() in test_math.cpp to defuse improper reordering within EXPECT_EQ gtest macro of std::isinf().
Use compat::end() in util::back() and util::cend() to avoid incorrect std::end() behaviour with -O2.
Use util::cend() in algorithms::sum(), again to avoid incorrect std::end() behaviour with -O2.

Looks good to me. Thanks for doing this Sam!


build: add codecov support

You need to modify the .travis.yaml to tell it to report coverage, right?

And I just saw the other TBR :)


net: UDPConn.WriteToUDP not sending anything on Windows

What version of Go are you using (go version)?

$ go version
go version go1.11.4 windows/amd64

Does this issue reproduce with the latest release?
N/A
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\jmarkel\AppData\Local\go-build
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\repo
set GOPROXY=
set GORACE=
set GOROOT=C:\Go
set GOTMPDIR=
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\jmarkel\AppData\Local\Temp\go-build202622878=/tmp/go-build -gno-record-gcc-switches

What did you do?
Trying to run an mdns query using the hashicorp/mdns client
https://github.com/hashicorp/mdns

What did you expect to see?
Expected to see a list of devices with mDNS services.
What did you see instead?
No results.
Please see the attached call stack.
With Wireshark running, I was not seeing any packets when the program was invoked.

It's as if the call into the dll (WSASentTo()) to send the UDP message is not working correctly, though no errors are thrown when stepping through the code.

Without code to reproduce this (ideally a minimal reproduction), it is extremely difficult to diagnose what is going on. Do you have an example client/server setup you could provide? Perhaps this would also be more appropriate for https://github.com/hashicorp/mdns/issues/80 and once you’ve determined it’s an issue with the Go toolchain, you could then revisit this issue.

package main

import (
        "fmt"
        "github.com/hashicorp/mdns"
)

func main() {
        // Make a channel for results and start listening
        fmt.Println("searching for new entries")
        entriesCh := make(chan *mdns.ServiceEntry, 4)
        go func() {
                for entry := range entriesCh {
                        fmt.Printf("Got new entry: %v\n", entry)
                }
        }()

        // Start the lookup
        mdns.Lookup("_ssh._tcp", entriesCh)
        close(entriesCh)
}

Please see the call stack for verification of the Golang toolchain.

Does this work as expected on platforms other than Windows?

yeah, works great on a Pi. see attachment.

Thanks. That’s helpful.
Can you attach text instead of images, please? Images are harder to read and hurt accessibility.

@mikioh @alexbrainman

Sure.  Will do.

Tentatively marking this a release blocker for 1.15 while we gather more info.

Could be because of any of the known bugs outlined here: https://golang.org/pkg/net/#pkg-note-BUG

Just wanted to note I've updated Go (same problem, however)
go version go1.14.1 windows/amd64

Any updates on this or any potential workarounds?

@jmarkel44 it’s been two days. Please be patient.

There's a Timeout in the Lookup function. The code above is the recommended way to use the library too, based on github.com/hashicorp/mdns.
As noted, this works perfectly on a RPi.
Thanks.

I'm sure you're right, but when I look at that program it looks clearly incorrect.  Humor me.  Thanks.

What results are you getting when running the program?

I don't have a Windows system.

But on my GNU/Linux system I get
searching for new entries
2020/04/03 13:09:02 [INFO] mdns: Closing client {0xc000010030 0xc000010038 0xc000010040 0xc000010048 1 0xc0000181e0}

so if you do something like
$ dns-sd -B _workstation._tcp
are there any mDNS devices on the network?

I don't have dns-sd installed.
It would not surprise me if there are no mDNS devices on the network.

@jmarkel44
I never used this package github.com/hashicorp/mdns - I don't know what it does. But I tried running your program on Linux:
searching for new entries
2020/04/05 16:52:10 [INFO] mdns: Closing client {0xc000010030 0xc000010038 0xc000010040 0xc000010048 true 0xc000068180 {1 0}}
2020/04/05 16:52:10 [ERR] mdns: Failed to read packet: read udp4 <IP_ADDRESS>:55783: use of closed network connection

and on Windows
searching for new entries
2020/04/05 16:54:47 [ERR] mdns: Failed to bind to udp6 port: listen udp6 [ff02::fb]:5353: setsockopt: not supported by windows
2020/04/05 16:54:48 [INFO] mdns: Closing client {0xc00014c020 0xc00014c028 0xc00014c030 <nil> true 0xc000108180 {1 0}}
2020/04/05 16:54:48 [ERR] mdns: Failed to read packet: read udp4 <IP_ADDRESS>:54096: use of closed network connection

And 2 outputs looks very similar to me.
If you think the problem is in UDPConn.WriteToUDP, please, create small program that we can run on our computers to investigate. Do not assume we have Wireshark installed or other special tools available.
Thank you.
Alex

So this came down to a set-interface issue on the windows platform. traversing the entire list of possible interfaces resolves the issue we were seeing with mdns package.
Thanks for the help!
Closing the ticket.

So this came down to a set-interface issue on the windows platform. traversing the entire list of possible interfaces resolves the issue we were seeing with mdns package.
Thanks for the help!
Closing the ticket.

Please, could you provide a code example of the solution?


JavaScript to get cookie in ReadMe is broken?

While logged in, running the script is returning an empty string.

I managed to find the cookies using Firefox's dev tools, but it seems that for some reason the a and b cookies vanish after page load? I don't get it but they don't show up.

That code only works on the "classic" version of the site.  You'll need to log into an account that has beta mode disabled.

@boothale So I just found out where the setting for beta vs classic is and I have never used the beta before. I was getting this problem on classic. I've just tried it again in both, and still have this error.

Did more testing and confirmed that this happens in Chrome.  As a work around, if you open the requests tab and look at the primary request, the cookies should be listed:

The javascript still seems to work on my version of Firefox.

Since I haven't heard back in a while I'm closing this.  Please reopen if you still need help.

I believe I confirmed that worked as well but I do not remember as it has
been a while since I worked with it.
On Jul 18, 2017 21:12, "Erra Boothale"<EMAIL_ADDRESS>wrote:

That code only works on the "classic" version of the site. You'll need to
log into an account that has beta mode disabled.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/boothale/faexport/issues/17#issuecomment-316267543,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOZPP4KhatupxHJ6HK_fqL6XbFE3_Rbcks5sPYI7gaJpZM4OZeLA
.

Any chance of providing an example for the beta site?
Or barring that, I have my a and b cookies, how would I turn those into an FA_COOKIE string?

@ShaunDreclin The ReadMe shows exactly how to assemble the string. b=<stuff>; a=<stuff>

Ooh gotcha. I was reading /docs, didn't think to check both pages. Thanks ^^

I added an example to /docs as well so people don't have to go to the readme.


Fix regression with path overrides

If an override points to a path dependency then that's not locked, so we're
missing information for that, but it's already warned about, so no need to
worry.
Closes #3288

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

ping r? @brson
Probably too late to fix this regression in 1.13, but hopefully we can get it in 1.14

ping r? @brson

:umbrella: The latest upstream changes (presumably #3283) made this pull request unmergeable. Please resolve the merge conflicts.

@bors r+

:pushpin: Commit a0f0932 has been approved by brson

:hourglass: Testing commit a0f0932 with merge d1bfc01...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: brson
Pushing d1bfc0196fb4476171060cfc3ee347642f6d25e4 to master...


Shrink LLVM artifact to minimal size

Rust compiler does not need whole llvm-project artifact.
Smaller artifact can make images and deployment smaller which leads to quick container spin-up and smaller footprint in registries.
Research options to shrink the artifact.

LLVM could be reduced down to following items:
./lib/clang/14.0.0/include/*
./lib/libclang.so.14.0.0

Testing implementation done for Linux x86_64

Shrinked. Expect Windows. For some reason LLVM was not working properly.
Windows will be handled in separate issue.


on redis 3.2.6.  modify value of Json string from list. but the Json string change to binary when save the value to list again

before the value  like this: {"status":1,...}".
modify value of 'status' field to 2 and save to list by 'lset' comm . after the vlaue like this: {"status":2\x00\x00\x00
what is happen ?  anyone know how to solve this problem? thx.
PS : json object to json string  i use JSON.parse and JSON.stringify, node_redis i use 2.6.5 version.

try convert created_time's value to string

@AbnerGC your example does not seem to be complete. You could activate the debug mode to see what data is actually transferred. This is very likely not an issue with node_redis though and I'm closing this as question.
@SkymaxPlay data as a whole would have to be stringified in the example. The number property as such is not the issue.


Add a suppress-unk option

During runtime, we sometimes don't want to output <unk> tokens.

Ropened - see my comment on PR #701

Nuda na LRECu dela svoje :D


config.User in image config

I created an oci image by following the image-spec. While converting my 'image' into a runtime-bundle using oci-image-tool convert, I received the error:
config.User: unsupported format
github.com/opencontainers/image-tools/image.runtimeSpec
        /c/Users/mattm/repo/personal/go/src/github.com/opencontainers/image-tools/image/config.go:103
github.com/opencontainers/image-tools/image.createBundle
        /c/Users/mattm/repo/personal/go/src/github.com/opencontainers/image-tools/image/image.go:342
github.com/opencontainers/image-tools/image.createRuntimeBundle
        /c/Users/mattm/repo/personal/go/src/github.com/opencontainers/image-tools/image/image.go:289
github.com/opencontainers/image-tools/image.CreateRuntimeBundleLayout
        /c/Users/mattm/repo/personal/go/src/github.com/opencontainers/image-tools/image/image.go:235
main.createAction
        /c/Users/mattm/repo/personal/go/src/github.com/opencontainers/image-tools/cmd/oci-image-tool/create.go:74
github.com/opencontainers/image-tools/vendor/github.com/urfave/cli.HandleAction
        /c/Users/mattm/repo/personal/go/src/github.com/opencontainers/image-tools/vendor/github.com/urfave/cli/app.go:485
github.com/opencontainers/image-tools/vendor/github.com/urfave/cli.Command.Run
        /c/Users/mattm/repo/personal/go/src/github.com/opencontainers/image-tools/vendor/github.com/urfave/cli/command.go:193
github.com/opencontainers/image-tools/vendor/github.com/urfave/cli.(*App).Run
        /c/Users/mattm/repo/personal/go/src/github.com/opencontainers/image-tools/vendor/github.com/urfave/cli/app.go:250
main.main
        /c/Users/mattm/repo/personal/go/src/github.com/opencontainers/image-tools/cmd/oci-image-tool/main.go:63
runtime.main
        /usr/local/go/src/runtime/proc.go:195
runtime.goexit
        /usr/local/go/src/runtime/asm_amd64.s:2337

My image config came straight from the spec's documentation with some minor changes such as the user name, home directory, etc.
{
    "schemaVersion": 2,
    "created": "2019-06-29T04:43:42.455040700Z",
    "author": "Bob T. Hacker<EMAIL_ADDRESS>    "architecture": "amd64",
    "os": "linux",
    "config": {
        "User": "root",
        "ExposedPorts": {},
        "Env": [
            "PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
            "FOO=oci_is_a",
            "BAR=well_written_spec"
        ],
        "Entrypoint": [
            "/bin/sh"
        ],
        "Cmd": [],
        "Volumes": {},
        "WorkingDir": "/",
        "Labels": {
            "com.example.project.git.url": "https://example.com/project.git",
            "com.example.project.git.commit": "45a939b2999782a3f005621a8d0f29aa387e1d6b"
        }
    },
    "rootfs": {
        "diff_ids": [
            "sha256:ec3da7fb5f709a1ce912e6e31fccc5588420c5f1dcecc362c72c989532c1917a"
        ],
        "type": "layers"
    },
    "history": [
        {
            "created": "2019-06-29T04:43:42.455040700Z",
            "created_by": "/bin/sh -c #(nop) CMD [\"sh\"]",
            "empty_layer": true
        }
    ]
}

The code responsible for throwing the error is on lines 87-104 in the image/config.go file
	if uid, err := strconv.Atoi(c.Config.User); err == nil {
		s.Process.User.UID = uint32(uid)
	} else if ug := strings.Split(c.Config.User, ":"); len(ug) == 2 {
		uid, err := strconv.Atoi(ug[0])
		if err != nil {
			return nil, errors.New("config.User: unsupported uid format")
		}

		gid, err := strconv.Atoi(ug[1])
		if err != nil {
			return nil, errors.New("config.User: unsupported gid format")
		}

		s.Process.User.UID = uint32(uid)
		s.Process.User.GID = uint32(gid)
	} else if c.Config.User != "" {
		return nil, errors.New("config.User: unsupported format")
	}

Going by the code, I changed the config.User value to "0:0" (i.e: uid:gid) and the command oci-image-tool convert command completed successfully.
Is the documentation wrong? Are developers not supposed to enter an actual username in the config.User field?
UPDATE
After running the runtime-bundle in docker, I get the following error:
docker: Error response from daemon: linux spec user: unable to find user 0/0: no matching entries in passwd file.

Is there any information/update on this? Is it even an issue or am I configuring something wrong?


add sync workflow

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Wrong repo!


Add support for TablePlus app on Mac

Simple addition to support TablePlus on mac. Handles if installed by Setapp or just installed as normal.

Thank you for the update @joelsteidl!


ci: Fix auto release

Auto release workflow tries to download Node 24 which doesn't exist yet

🎉 This change has been released in version 1.4.0


[bug] Problem with dynamic variables

Describe the bug
The use of dynamic variables only seems to work in the outer level.
To Reproduce
Steps to reproduce the behavior:

Having the following folder structure

config.py:
from dynaconf import Dynaconf

settings = Dynaconf(
    envvar_prefix="DYNACONF",
    settings_files=['settings.yaml'],
)

print(settings.a) # ok (A)
print(settings.b) # ok (A)
print(settings.c.d) # ok (D)
print(settings.h) # ok (D)
print(settings.c.e) # dynaconf.vendor.box.exceptions.BoxKeyError: "'Settings' object has no attribute
print(settings.c.f) # dynaconf.vendor.box.exceptions.BoxKeyError: "'Settings' object has no attribute
print(settings.c.g) # dynaconf.vendor.box.exceptions.BoxKeyError: "'Settings' object has no attribute

Having the following config files:

settings.yaml:
a: "A"
b: "@format {this.a}"
c:
  d: "D"
  e: "@format {this.a}"
  f: "@format {this.c.d}"
  g: "@format {this.d}"
h: "@format {this.c.d}"

Executing under the following environment

pipenv install dynaconf
Expected behavior
I would not expect to have an exception raised on quering settings.c.e or settings.c.f. I would expect it on settings.c.g because this refers to the whole config.
Environment (please complete the following information):

OS: MacOS
Dynaconf Version: 3.1.2
Frameworks in use: None

This has been fixed in https://github.com/rochacbruno/dynaconf/pull/446 and will be included in the next release 3.1.3

Fine! Thanks for your work!


10128 DEV Add migration query to create new columns prJobsCount, prJobsSuccessful, prJobsFailed, prJobsTimedOut, prJobsCancelled, prJobsSkipped, prJobsUnstable to projectDailyAggs

https://github.com/Shippable/pm/issues/10128

Verified in RC


((k,v) for (k,v) in dictVar) causes error

does not work:
d = {'a':5}
x = ((k,v) for (k,v) in d)

does work:
d = {'a':5}
x = ((k,v) for (k,v) in d.items())
x = [(k,v) for k,v in d.items()]
x = {k:v for k,v in d.items()}
x = ((k) for k in d)

⚠ There has been a error when trying to display your variables. Sorry :(

Traceback (most recent call last):
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\pythonEvaluator.py", line 77, in exec_input
returnInfo['userVariables'] = jsonpickle.encode(userVariables, max_depth=100) # any depth above 245 resuls in error and anything above 100 takes too long to process
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\jsonpickle_init_.py", line 135, in encode
numeric_keys=numeric_keys)
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\jsonpickle\pickler.py", line 44, in encode
return backend.encode(context.flatten(value, reset=reset))
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\jsonpickle\pickler.py", line 157, in flatten
return self._flatten(obj)
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\jsonpickle\pickler.py", line 161, in _flatten
return self._pop(self._flatten_obj(obj))
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\jsonpickle\pickler.py", line 177, in _flatten_obj
return flatten_func(obj)
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\jsonpickle\pickler.py", line 429, in _flatten_dict_obj
flatten(k, v, data)
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\jsonpickle\pickler.py", line 498, in _flatten_key_value_pair
data[k] = self._flatten(v)
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\jsonpickle\pickler.py", line 161, in _flatten
return self._pop(self._flatten_obj(obj))
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\jsonpickle\pickler.py", line 177, in _flatten_obj
return flatten_func(obj)
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\jsonpickle\pickler.py", line 235, in _ref_obj_instance
return self._flatten_obj_instance(obj)
File "C:\Users\anon\Downloads\arepl-win32-x64\arepl-win32-x64\resources\app\src\python\jsonpickle\pickler.py", line 367, in _flatten_obj_instance
data[tags.ITERATOR] = list(map(self._flatten, islice(obj, self._max_iter)))
File "", line 2, in
ValueError: not enough values to unpack (expected 2, got 1)

ported this over to AREPL-backend


ActionFailed. An action failed. No dependent actions succeeded.

@russbasiura Please can you provide some more info

I'm having the same error message in the logic app.  During the check template type routing block, specifically Clone team.  The errors are bad requests.  I did add the beta Teams switch to the install script.  Here are the inputs:
{
"uri": "https://graph.microsoft.com/beta/teams/standard/clone",
"method": "POST",
"headers": {
"content-type": "application/json"
},
"authentication": {
"tenant": "<redacted>",
"audience": "https://graph.microsoft.com",
"clientId": "<redacted>",
"secret": "sanitized",
"type": "ActiveDirectoryOAuth"
},
"body": {
"classification": null,
"description": "Desc",
"displayName": "TM-ATP Demo 7",
"mailNickname": "TMATPDemo7",
"partsToClone": "apps,tabs,settings,channels",
"visibility": "Private"
}
}
Here is the output/return values:
My apologies if redacting the ID numbers is a Bad Thing; obviously I'm a newb to this. The request and client-request IDs were/are the same.
{
"statusCode": 400,
"headers": {
"Transfer-Encoding": "chunked",
"Vary": "Accept-Encoding",
"Strict-Transport-Security": "max-age=31536000",
"request-id": "<redacted>",
"client-request-id": "<redacted>",
"x-ms-ags-diagnostic": "{"ServerInfo":{"DataCenter":"Central US","Slice":"E","Ring":"2","ScaleUnit":"002","RoleInstance":"DS3PEPF00001664"}}",
"Date": "Tue, 02 Mar 2021 22:05:49 GMT",
"Content-Type": "application/json",
"Content-Length": "296"
},
"body": {
"error": {
"code": "BadRequest",
"message": "teamId needs to be a valid GUID.",
"innerError": {
"date": "2021-03-02T22:05:50",
"request-id": "<redacted>",
"client-request-id": "<redacted>"
}
}
}
}

@tsstahl @BBenedikt I'm afraid the beta logic app (provisioned using the UseMSGraphBeta param) has not been updated for some time. I'm going to deprecate this one as the only functionality it provided as extra was the ability to clone private channels, all it really did was re-create them anyhow.
So it will be removed shortly.
Therefore I would suggest re-deploying with the UseMSGraphBeta flag set to false.


Not Placing orders

When placing an order the following message is produced and it does not place the order. This happens whether buying or selling.
unsuccessful
[11:18 AM]
The order verification produced the following messages: ['message not found', 'Please note that the issuer of this security is deficient in meeting the Exchange\x92s continued listing requirements. You may want to consider doing further research before placing this order.  (DO840)', 'Please Note:  %SYMBOL% has a special maintenance requirement of %2%%.  (AC157)']
[11:18 AM]
unsuccessful

An update on this -- Schwab's web app calls a new API endpoint that didn't exist when I first created this package.
I spent some time today to understand how we can call it (because there's been some changes in authorization) and believe I've figured out how to get the package to place orders on the new API. I don't use Schwab anymore to actively trade, so I'll have to move some money around and test it in the coming days, but I'm optimistic this could solve this problem (or, offer more transparency in identifying the solution)!

Awesome!

I just pushed version 0.3.0, which features a trade_v2() function. I suspect this alone will not resolve the issue, but I can more clearly reproduce the trade in the UI now to identify what needs to be done to verify the order.
Could you all give me an example stock that generates this message?

For me I get this message on all stocks I try to buy with this. An example could be SYTA

Hi, could you please try placing an order using both the regular api.trad() and api.trade_v2() functions? I was able to successfully buy SYTA using both accounts. cc: @NelsonDane

On api.trade() I get
unsuccessful The order verification produced the following messages: ['message not found'] ([''], False)
On api.trade_v2() I get
unsuccessful The order verification produced the following messages: ['message not found'] ([''], False)
I also have multiple accounts and the api is only showing that I have one account now. I also tried this with INTC

@MaxxRK, can you identify where in trade_v2() it returns? I suspect it's one of the if r.status_code != 200 sections, in which case we could learn more if you print out r.reason. I encountered some "request entity too large" errors that I was able to get around, but I'm still trying to understand why I get the error and why the workaround works.

I ran this the first time with just example.py and schwab_api==0.3.2 installed. I put some prints in after each if response and ran it from the folder itself schwab_api.schwab_api
this is what I got
The order verification was successful The order verification produced the following messages: []
There was quite a bit of json underneath this. I did not see this order in my order status page still only seeing one account when I should see multiple, but it does seem to output something.

@MaxxRK, to be clear, the orders are going through as expected then?

They went through if I just download the repo from GitHub and run the example through that. If I run it from the package it did not seem to go through.

An update on this -- Schwab's web app calls a new API endpoint that didn't exist when I first created this package.
I spent some time today to understand how we can call it (because there's been some changes in authorization) and believe I've figured out how to get the package to place orders on the new API. I don't use Schwab anymore to actively trade, so I'll have to move some money around and test it in the coming days, but I'm optimistic this could solve this problem (or, offer more transparency in identifying the solution)!

Schwab acquired TD Ameritrade and closed the TD API, resulting in the original python tda-api (https://tda-api.readthedocs.io/en/latest/getting-started.html#important-new-about-the-charles-schwab-transition) no-longer work any more.  I think that's where this new API endpoint come from.


Scope out Analytics

How do we want to do analytics? Should that be defined at this level? Should it not?

https://github.com/react-ga/react-ga

Things we want to be able to track:

What page is being viewed? How long?
What Buttons are being pressed?
How many people are currently viewing a page?
How long are people taking to register? What "registration" questions are taking the longest?


could not use multiple gpus in MICN

RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cuda:1! (when checking argument for argument tensors in method wrapper_CUDA_cat)

Hi. Could you please provide more detailed error information to help us fix the corresponding bug? Such as your experimental parameters and error location, etc.

Hi, we have fixed this problem. Thanks for coming up with this issue.


tests/kernel/device/kernel.device.pm fails on bt510/bt6x0

./scripts/twister -s tests/kernel/device/kernel.device.pm -p bt510 -p bt6x0
gets:
[ 77%] Building C object zephyr/drivers/sensor/si7055/CMakeFiles/drivers__sensor__si7055.dir/si7055.c.obj
In file included from /home/galak/git/zephyr/include/kernel_includes.h:21,
                 from /home/galak/git/zephyr/include/kernel.h:17,
                 from /home/galak/git/zephyr/include/init.h:11,
                 from /home/galak/git/zephyr/include/device.h:29,
                 from /home/galak/git/zephyr/include/drivers/sensor.h:23,
                 from /home/galak/git/zephyr/drivers/sensor/si7055/si7055.c:10:
/home/galak/git/zephyr/twister-out/bt510/tests/kernel/device/kernel.device.pm/zephyr/include/generated/devicetree_unfixed.h:7232:35: error: 'DT_N_S_soc_S_i2c_40003000_S_si7055_40_P_wakeup_source' undeclared here (not in a function); did you mean 'DT_N_S_soc_S_i2c_40003000_S_lis2dh12_18_P_wakeup_source'?
      |                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

@galak A bit more information given that the twister tool doesn't run on our development platform (windows) would be nice - what is failing exactly? There is no power management in the Si7055 driver, if the test tool is expecting it then it seems like a bug in the test tool, just like there is no power management in the Si7006 or Si7060 drivers.

We run nightly builds and these boards fail building the test.  If power mgmt isn't supported on the boards than the feature should be disabled or the test excluded for those platforms.
Here are links to the test runs:
https://buildkite.com/zephyr/zephyr-daily/builds/424#92f62569-80b0-4106-bfea-78947595ad64
https://buildkite.com/zephyr/zephyr-daily/builds/424#202597a2-9449-486e-b6b3-1ee67b6bd46d

We didn't change/add any tests or enable/disable anything so I'm not sure how this is our problem? Various Laird Connectivity board files were recently updated to change a Kconfig for the crystal error rate and that's it, which is unrelated to this issue.

This might be a more generic PM issue and not specific to the bt510/bt6x0 boards.


Added missing closing parens in language mapping description

Fix a very minor typo where the closing parens was missing.
Before

After

Thanks!


Size different from 128

Hi,
I am trying to feed as argument image size=300 but I'm getting this error (my images are indeed of size 300)
assert h == img_size and w == img_size, f'height and width of image must be {img_size}'
AssertionError: height and width of image must be 300
When I put image_size=128 everything works just fine, is there a way to run the model and training with size different than 128 ?
Thank you

@ethancohen123 Hi Ethan, can you paste your script?

Hi, here is my script (I followed the instruction in the readme.) When using image_size=128 it works fine, otherwise it does not.
import torch
from denoising_diffusion_pytorch import Unet, GaussianDiffusion, Trainer
model = Unet(
dim = 64,
dim_mults = (1, 2, 4, 8)
).cuda()
diffusion = GaussianDiffusion(
model,
image_size = 300,
timesteps = 1000,   # number of steps
loss_type = 'l1'    # L1 or L2
).cuda()
trainer = Trainer(
diffusion,
'/projects/synsight/ethan/corona/corona_png/',
train_batch_size = 32,
train_lr = 2e-5,
train_num_steps = 700000,         # total training steps
gradient_accumulate_every = 2,    # gradient accumulation steps
ema_decay = 0.995,                # exponential moving average decay
amp = True                        # turn on mixed precision
)
trainer.train()

@ethancohen123 are you sure you are using the latest version?

Sorry bout that, I was using an older version, thanks.
I now can generate higher size but 300 doesnt seems to work but that okay size 256 is fine

An other not related question, is conditional generation supported?
Thanks a lot

@ethancohen123 yup, you'll want to use 384 rather than 300
do you wish to condition on an image or text? if you want text, i recommend https://github.com/lucidrains/imagen-pytorch

Actually I want to get something as in High-Resolution Image Synthesis with Latent Diffusion Models since I want to be able to encode from any source of data. I guess denoising diffusion is not the way to go then aha

@ethancohen123 no you can still use this library
afaik, latent diffusion is just denoising diffusion with predict x0 objective

oh thats awesome ! any idea how I can do that ? If I have a condition x and I create an encoder T(x) and I want to generate images from this how can I do that ? Thanks a lot !

@ethancohen123 i'd recommend https://github.com/lucidrains/dalle2-pytorch i actually have latent diffusion built in as an option, though it is untested
most of the code is there though

yes i saw this repo ( which is awesome btw ) but only thing is that I cannot do any clip training as I dont have text conditionning but features condition. Is there any way to use it without doing anything related to CLIP and just feed the diffusion with the condition ?
Thanks for your time again


fix: bug where was not smart enough to detect type 2 transactions

What I did
When using max gas and stuff, it didnt realize you were wanting type 2 so itd fail
How I did it
we actually dont need to override create_tx method anymore, so deleting it fixes it.
this is because we have a setting now that sets the default tx type
How to verify it
see new tests
Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

This requires a core release to work and for tests to pass


Vulkan GPU/CPU synchronization is wrong

Godot version
4.2.x master [16a93563bfd3b02ca0a8f6df2026f3a3217f5571]
System information
Godot v4.2.dev (262d1eaa6) - Ubuntu 20.04.6 LTS (Focal Fossa) - X11 - Vulkan (Forward+) - dedicated AMD Radeon RX 6800 XT - AMD Ryzen 9 5900X 12-Core Processor (24 Threads)
Issue description
"OOOFFF"
That's the only way I can describe this ticket.
It starts as a simple CPU to GPU synchronozation bug but there is also another set of wrong assumptions and bugs which, when combined, "more or less" synchronization ends up being correct.
There are so many things going on I don't know where to start, so I'm going to write this in the order in which I discovered these problems.
At worst this is just a waste a VRAM, at best suddenly many of those DEVICE_LOST start making sense.
A lone validation error that shouldn't have happened
It all started with a random validation error saying a vkDestroyRenderPass got called while it was still in use while trying to repro #80286 on AMD, an error that I could not reproduce ever again.
So I went looking for vkDestroyRenderPass calls in the codebase and luckily they weren't many.
I'm still evaluating everything to understand what's going on. But it is clear it's wrong (there's no way to sugar coat it).
That's when I noticed the function RenderingDeviceVulkan::swap_buffers does the following:
void RenderingDeviceVulkan::swap_buffers() {
	context->swap_buffers();
	frame = (frame + 1) % frame_count;
	_begin_frame();
}

// And inside _begin_frame we call _free_pending_resources( frame )
void RenderingDeviceVulkan::_free_pending_resources(int p_frame) {
	// Free in dependency usage order, so nothing weird happens.
	// Pipelines.
	while (frames[p_frame].render_pipelines_to_dispose_of.front()) {
		RenderPipeline *pipeline = &frames[p_frame].render_pipelines_to_dispose_of.front()->get();

		vkDestroyPipeline(device, pipeline->pipeline, nullptr);

		frames[p_frame].render_pipelines_to_dispose_of.pop_front();
	}

      // .. and lots of other vkDestroy* calls
}

Where's the wait?.
That's wrong. The code should've been something along:
context->swap_buffers();
frame = (frame + 1) % frame_count;
vkWaitForFences(device, 1, &fences[frame]);
_begin_frame();

I was about to shout 'Eureka!', but went into swap_buffers() just in case there was already a wait there (although it would be some weird math because it would have to be vkWaitForFences(device, 1, &fences[frame + 1])).
That's when it got worse.
An inverted wait
I couldn't find anything in swap_buffers, but I was wondering where is the vkWaitForFences call???
I found it in VulkanContext::prepare_buffers. But sadly it performs the following:
vkWaitForFences(device, 1, &fences[frame_index], VK_TRUE, UINT64_MAX);
vkResetFences(device, 1, &fences[frame_index]);
vkQueueSubmit(graphics_queue, 1, &submit_info, fences[frame_index]);

frame_index += 1;
frame_index %= FRAME_LAG;

This is wrong.
In case someone wonders, for data that is used to communicate CPU from/to GPU we are supposed to have at least 2 pools of memory:
CPU writes to pool 0 while GPU is reading from pool 1, and CPU writes to pool 1 while GPU is reading from pool 0. If the GPU is still reading from pool 0 and CPU wants now to write to pool 0 (it's done writing to 1), it needs to stall and wait.
This is a standard double buffer scheme. A triple buffer scheme is the same, but there is an extra pool to minimize potential stalls.
Unfortunately with Godot's current code the following sequence is what will happen after running for several frames.

Action
Remarks

vkWaitForFences on 0

vkResetFences on 0

vkQueueSubmit on 0

frame_index = 1;

Fill gpu buffers on 1

vkWaitForFences on 1

vkResetFences on 1

vkQueueSubmit on 1

frame_index = 0;

Fill gpu buffers on 0
Oops!! We never waited to see if fence[0] is done

vkWaitForFences on 0

vkResetFences on 0

vkQueueSubmit on 0

And thus we're not actually synchronizing anything! The CPU keeps writing to data that may be in use by the GPU.
A simple possible solution is the following:
vkResetFences(device, 1, &fences[frame_index]);
vkQueueSubmit(graphics_queue, 1, &submit_info, fences[frame_index]);

frame_index += 1;
frame_index %= FRAME_LAG;
vkWaitForFences(device, 1, &fences[frame_index], VK_TRUE, UINT64_MAX);

For best performance it's often suggested to delay the vkWaitForFences() as much as possible (i.e. until it's actually needed for us to write to a buffer from CPU, or destroy a resource, etc). But it would seem given Godot's code chaotic complexity this is the only safe solution that won't result in more hazards.
Sadly, the report doesn't end here. There is more. Godot doesn't have 2 pools, it's got 4.
Wrong assumptions on Swapchains
RenderingDeviceVulkan::frame_count is set to the following:
// Always need one extra to ensure it's unused at any time, without having to use a fence for this.
frame_count = p_context->get_swapchain_image_count() + 1;

Note: frame_count is almost always 4

That comment is HUGE. This is a false assumption.
Neither vkAcquireNextImageKHR nor vkQueuePresentKHR have an obligation to synchronize the CPU.
They use VkSemaphores, which synchronize GPU work with other GPU work (e.g. so that a window isn't presented to the screen while the GPU is still working on it, producing a similar effect to when VSync is off).
However in practice most (all???) drivers do stall the CPU due to limitations in SW or HW architectures. Linux+X11 is notorious for stalling inside vkAcquireNextImageKHR, but other drivers prefer to stall inside vkQueuePresentKHR instead.
Since we are debunking false assumptions, we also don't have the guarantee that vkAcquireNextImageKHR will return swapchain indices in order (e.g. 0, 1, 2, 3, 0, 1, 2, 3). It could return them out of order or even return the same set multiple times (e.g. 2, 0, 2, 0, 1).
We could set the value frame_count to as high as 30, and we still won't have the guarantee that CPU and GPU work will be synchronized. We need a fence.
Swapchains should not dictate the number of pools
As I said earlier, on a double buffer scheme, we should have 2 pools of memories (FRAME_LAG = 2).
But because of the wrong assumption on swapchains, Godot keeps 4 pools of memories (because frame_count = 4 on most machines).
Swapchain count should not be used for anything else other than tracking swapchain related data like VkRenderPass.
Combine these 2 bugs and it works
So we have 2 competing implementations and 2 bugs:

VulkanContext::prepare_buffers uses vkWaitForFences but the calls are out of order

This is what's causing the CPU to wait for the GPU.
This should be used to track the 2 pools.

VulkanContext uses swapchain count to use 4 pools instead of 2.

I've ran simulations in Spreadsheets of what happens when we run our current vkWaitForFences code with just 2 values and a frame_count = 4 and it would seem that this just happens to synchronize correctly, a happy accident.
Because Godot will stall when starting the 3rd frame, and it keeps 4 pools of data, the data should be safe for access on the 3rd frame.
However the complexity of all this is too large, so it's possible that I missed something and we may still get a race condition (that lone vkDestroyRenderPass error that sparked this invegistation must've come from somewhere!), just a rare one.
I also noticed that if, for some God-forsaken reason, frame_count = 3 or lower, then we will have race conditions.
I analyzed the surrounding code behind frame_count setup, and this is possible but it is improbable.
We set desiredNumOfSwapchainImages = 3 and I don't think there are implementations that have surfCapabilities.maxImageCount = 2 however I could be wrong. Also, if the driver returns 2 in vkGetSwapchainImagesKHR despite us asking for at least 3 (which AFAIK would be a driver bug), we're still in trouble.
However it would make sense that those users who can repro DEVICE_LOST with high frequency happen to have frame_count <= 3.
I didn't analyze what happens when separate_present_queue == true. In fact, this is so rare that I don't think this code has been tested at all, and we have no way to verify yet if the bug reports have been using this.
Steps to reproduce
This bug is rare to repro, but it should be much easier to repro if we force frame_count to a value lower than 4.
Minimal reproduction project
None.

We should lower the default frame_count in the long run (and possibly expose a project setting to adjust it), as people have reported latency issues in Vulkan that don't occur with OpenGL.

We should lower the default frame_count in the long run (and possibly expose a project setting to adjust it), as people have reported latency issues in Vulkan that don't occur with OpenGL.

I saw the other thread about latency and I forgot to comment on this. The current implementation is too wrong to be discussing it.
Godot right now is doing double buffering (FRAME_LAG = 2). But because of the bug, it is actually somewhere between triple and double buffer (because we write to buffer[2] and right afterwards we wait on fence[0]).
Double buffering is the lowest latency that can be reached with reasonable performance. Also frame_count doesn't matter much as long as frame_count > FRAME_LAG.
In fact in my PR I will have rename frame_count because it's wrong and confusing.
Perhaps after the problem is fixed the latency problems are gone. But if not, we will have the proper code to test other values (for instance, FRAME_LAG should not be hardcoded).

Is this issue still on the blocks for 4.3, or is this now moved to 4.x? This seems deeply related to the AMD issues we're seeing with frame shuffling when vsync is enabled, especially on 5700 XTs.

I tested darksylinc's build of 4.2 tagged 'binary for testers' and still saw the stutter and shuffling issues on my Radeon 5700 XT.
The Vulkan renderer also frequently crashes and corrupts the entire display leading to an irrecoverable driver crash that requires a complete system shutdown. Unsure if it's related.

@DarioSamo @darksylinc Can this be considered fixed by https://github.com/godotengine/godot/pull/87340, or only partially? If so, what's the current status?

The issue described here is solved, as the new synchronization order is what Matias described as the correct solution.


Reduced integration cards in homepage

Description
Reduced integration cards in homepage.
The number of integration cards is reduced to 30 in the homePage-integrations.html

This PR fixes
#1371
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@Ishwar-Zatke , Thanks for your contribution however Please ensure that you went thorough the issue comments.
We follow assignment in chronological order . Please take care of these things from near future

Since this is your first Pull Request ,
Let's discuss it on the websites call.
Please add this as an agenda item in the meeting minutes, if you would :)
https://docs.google.com/document/d/1XczAHXVe2FIWPqiF57ospJ43zw5cZQ7ui8mn39v5EvA/edit#heading=h.lohhtewfwima

@Ishwar-Zatke Thank you for your contribution you did great but make sure like @Ghat0tkach said ask for an assignment, get assigned then start working on the issue.
Will you shift the "See all integrations" below the integration cards?

@ritiksaxena124 , Instruction noted, will follow the flow from here onwards. I had also updated the changes u asked for, Can you review it?

Looks good to me @Ishwar-Zatke 🙌 well done

let's discuss this item on today's website meeting,
please add this agenda in the docs if you would :): https://docs.google.com/document/d/1XczAHXVe2FIWPqiF57ospJ43zw5cZQ7ui8mn39v5EvA/edit#heading=h.69qskjv558bk

@Ishwar-Zatke Was this PR discussed in the website meeting?

Hey @Ishwar-Zatke Let's discuss this on website call today at 5:30 PM IST / 7 AM Central time. Please add this as an agenda item in the meeting minutes

@sudhanshutech Ready to merge, It's been stale for a long time.

Looks good to me @Ishwar-Zatke 🙌 well done

@sudhanshutech Ready to merge, It's been stale for a long time.

// @vishalvivekm @sudhanshutech

@abhijeetgauravm https://layer5io.slack.com/archives/C015QJKUMPU/p1697478343561099

This issue has been solved in PR #1467


FogDensity does not work in 1.16.5 Multiplayer.

OTG Plugin or Mod

[X] OTG Mod for Forge

OTG Version
OpenTerrainGenerator-1.16.5-0.0.17r1.jar
Forge or Spigot Versions
forge-1.16.5-36.2.2  (Both client as Server)
What other plugins are you running?
None
Description
Fog color, foilage color etc works but FogDensity: 1.0 does not work for Multiplayer and results in the picture shown below. Using the same values on single player will result in zero visibility like it should do.

Fixed in next dev release.


make: *** [../node-addon-api/nothing.target.mk:109: Release/obj.target/nothing/../node-addon-api/nothing.o] Error 127

Did you see the documentation relating to installation?
Yes
Have you ensured the architecture and platform of Node.js used for npm install is the same as the architecture and platform of Node.js used at runtime?
Yes
Are you using the latest version? Is the version currently in use as reported by npm ls sharp the same as the latest version as reported by npm view sharp dist-tags.latest?
No
bash-5.1# npm ls sharp
<EMAIL_ADDRESS>/usr/src/project
`-- (empty)

bash-5.1# npm view sharp dist-tags.latest
0.29.3

What is the complete output of running npm install --verbose sharp? Have you checked this output for useful error messages?
npm verb stack Error: command failed
npm verb stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/@npmcli/promise-spawn/index.js:64:27)
npm verb stack     at ChildProcess.emit (node:events:520:28)
npm verb stack     at maybeClose (node:internal/child_process:1090:16)
npm verb stack     at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5)
npm verb pkgid<EMAIL_ADDRESS>npm verb cwd /usr/src/project
npm verb Linux 4.19.130-boot2docker
npm verb argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "--ignore-scripts=false" "--verbose" "sharp"
npm verb node v17.4.0
npm verb npm  v8.3.1
npm ERR! code 1
npm ERR! path /usr/src/project/node_modules/sharp
npm ERR! command failed
npm ERR! command sh -c (node install/libvips && node install/dll-copy && prebuild-install --runtime=napi) || (node-gyp rebuild && node install/dll-copy)
npm ERR! make: Entering directory '/usr/src/project/node_modules/sharp/build'
npm ERR! make: Leaving directory '/usr/src/project/node_modules/sharp/build'
npm ERR! info sharp Detected globally-installed libvips v8.12.1
npm ERR! info sharp Building from source via node-gyp
npm ERR! gyp info it worked if it ends with ok
npm ERR! gyp verb cli [
npm ERR! gyp verb cli   '/usr/local/bin/node',
npm ERR! gyp verb cli   '/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js',
npm ERR! gyp verb cli   'rebuild'
npm ERR! gyp verb cli ]
npm ERR! gyp info using<EMAIL_ADDRESS>npm ERR! gyp info using<EMAIL_ADDRESS>| linux | x64
npm ERR! gyp verb command rebuild []
npm ERR! gyp verb command clean []
npm ERR! gyp verb clean removing "build" directory
npm ERR! gyp verb command configure []
npm ERR! gyp verb find Python Python is not set from command line or npm configuration
npm ERR! gyp verb find Python Python is not set from environment variable PYTHON
npm ERR! gyp verb find Python checking if "python3" can be used
npm ERR! gyp verb find Python - executing "python3" to get executable path
npm ERR! gyp verb find Python - executable path is "/usr/bin/python3"
npm ERR! gyp verb find Python - executing "/usr/bin/python3" to get version
npm ERR! gyp verb find Python - version is "3.9.7"
npm ERR! gyp info find Python using Python version 3.9.7 found at "/usr/bin/python3"
npm ERR! gyp verb get node dir no --target version specified, falling back to host node version: 17.4.0
npm ERR! gyp verb command install [ '17.4.0' ]
npm ERR! gyp verb install input version string "17.4.0"
npm ERR! gyp verb install installing version: 17.4.0
npm ERR! gyp verb install --ensure was passed, so won't reinstall if already installed
npm ERR! gyp verb install version is already installed, need to check "installVersion"
npm ERR! gyp verb got "installVersion" 9
npm ERR! gyp verb needs "installVersion" 9
npm ERR! gyp verb install version is good
npm ERR! gyp verb get node dir target node version installed: 17.4.0
npm ERR! gyp verb build dir attempting to create "build" dir: /usr/src/project/node_modules/sharp/build
npm ERR! gyp verb build dir "build" dir needed to be created?  System Linux 4.19.130-boot2docker
npm ERR! gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
npm ERR! gyp ERR! cwd /usr/src/project/node_modules/sharp
npm ERR! gyp ERR! node -v v17.4.0
npm ERR! gyp ERR! node-gyp -v v8.4.1
npm ERR! gyp ERR! not ok

What is the output of running npx envinfo --binaries --system?
  System:
    OS: Linux 4.19 Alpine Linux
    CPU: (4) x64 Intel(R) Core(TM) i5-7600K CPU @ 3.80GHz
    Memory: 6.95 GB / 7.78 GB
    Container: Yes
    Shell: 1.34.1 - /bin/ash
  Binaries:
    Node: 17.4.0 - /usr/local/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 8.3.1 - /usr/local/bin/npm

This looks like https://github.com/lovell/sharp/issues/2810#issuecomment-886692938 which was a problem with Alpine Linux syscall permissions within Docker and unrelated to sharp.

@lovell
Thanks for the advice, but
bash-5.1# scmp_sys_resolver faccessat2
439

That is means the faccessat2 is supported.

npm verb pkgid<EMAIL_ADDRESS>
Please can you upgrade to the latest version, currently v0.29.3.

npm ERR! info sharp Detected globally-installed libvips v8.12.1

Is your use of a globally-installed vips intentional? Prebuilt binaries are available and should "just work" without compilation.
If you're still having problems, please can you share a Dockerfile that allows someone else to reproduce.

I tried to reproduce the issue from scratch and remove package.json file at all
Dockerfile:
FROM node:current-alpine

RUN apk add --upgrade --no-cache \
    vips-dev fftw-dev gcc g++ make libc6-compat imagemagick bash pngcrush optipng=0.7.7-r0 file ffmpeg runc && \
    apk add --upgrade graphicsmagick ghostscript

RUN mkdir -p /usr/src/project/tmp
WORKDIR /usr/src/project

then (for testing reason) just open a container shell and npm install sharp --verbose
the error part of log I have bump

Ok, solved.
I specified the tag for node from node:current-alpine to node:17-alpine3.14, and everything started to work. So, something wrong with the current alpine implementation.

I cannot reproduce this using the provided Dockerfile (Alpine 3.15) running the latest docker v20.10.12 on Ubuntu.
I note the log you've provided mentions boot2docker, which I believe has been unmaintained for a couple of years.

@lovell thank you for the comments, investigation and your time.
I will try to find an accurate replacement for the boot2docker and back to you with accurate logs if needed.


making top_navbar more clear

**- top informative bar is made more visible.

Added a glow hover effect on cards**

After change:

Before:

Please add this under hacktober fest.


DeleteArtifactsNotInTemplate - Orchestration Sequence Issue

Issue Description: current process sequence of section 'DeleteArtifactsNotInTemplate' in 'orchestrator.ts' file is ahead of deployment. Deployment fail in following scenario:
Step 1:  'Pipline1' has an dataflow activity 'dataflow1', hence 'dataflow1' is calling from 'Pipeline1'. Initial deployment succeed
Step 2: Updated 'Pipeline1' artifact by remove activity 'dataflow1'. Then delete dataflow 'dataflow1' since it is no longer needed
Step 3: Deploy again, it fails, error message is
2022-05-12T17:33:14.9428660Z The document cannot be deleted since it is referenced by Pxxxxxxn.
2022-05-12T17:33:14.9433308Z Error during execution: Error: For Artifact: Bxxxxxxg deletion failed. {"status":"Failed","error":{"code":"DeleteDataFactoryResourceOrchestrationError","message":"The document cannot be deleted since it is referenced by Pxxxxxxn."}}
2022-05-12T17:33:14.9471695Z ##[error]Encountered with exception:Error: For Artifact: Bxxxxxxg deletion failed. {"status":"Failed","error":{"code":"DeleteDataFactoryResourceOrchestrationError","message":"The document cannot be deleted since it is referenced by Pxxxxxxn."}}
Work around: run deployment twice, first time with 'DeleteArtifactsNotInTemplate' set to 'false', and deploy again with 'DeleteArtifactsNotInTemplate' set to 'true'
Suggest Solution: In orchestrator.ts file, move 'DeleteArtifactsNotInTemplate' after deployment

Hello @jeannejin
The team will look into it and will come up with a fix.

Hi,
Thanks for the message. This is my first time to raise an issue in GitHub. I was wondering what is going to happen.
When it is fixed, would you please let me know? I will update my deployment pipeline to remove the walkaround. It double the deployment time now. 🙂
Thanks
Jeanne

From: Aaditya Sharma @.>
Sent: May 16, 2022 11:01 AM
To: Azure/Synapse-workspace-deployment @.>
Cc: jeannejin @.>; Mention @.>
Subject: Re: [Azure/Synapse-workspace-deployment] DeleteArtifactsNotInTemplate - Orchestration Sequence Issue (Issue #56)
Hello @jeannejinhttps://github.com/jeannejin
The team will look into it and will come up with a fix.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/Synapse-workspace-deployment/issues/56#issuecomment-1127785253, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALRUT4ULRTSINA4EDW2QR6LVKJPMTANCNFSM5VZOYNOQ.
You are receiving this because you were mentioned.Message ID: @.***>

Hello @jeannejin
Sure, I'll update it here.
Thanks.

The fix has been released. Please use V1.6.0


Request For Enhancement: flexible Camera selection

Expected Behavior
Expect some button / drop down menu to change camera to my standard Raspberry camera. USB camera (webcam) works. I would really appreciate this feature as I would like to use this cam.
Actual Behavior
taking image does not work, no camera found

We have added a Camera selection drop-down in the Device widget on the Devices page. In it you can select the Raspberry Pi Camera.


Any way to make the output file zipped?

Using this command to make archives and find out that executable.phar is just a large plain text file without zipping.
php -d phar.readonly=off ./phar-composer.phar build . ./bin/executable.phar

PRs are very much appreciated! :shipit:
Compression make a whole lot of sense for many use cases. Though keep in mind that this means that both the creator and the user of the archive need to have ext-zlib enabled (which is not always available). Accordingly, this may limit how the resulting archive can be used. I'm fine with adding an option to control this, but this should probably default to being off.
As much as I'd love to work on this myself, there are currently no immediate plans to build this from my end (no demand at the moment and more important outstanding issues currently). If you need this for a commercial project and you want to help sponsor this feature, feel free to reach out and I'm happy to take a look.

Closed via #3


Add Code Snippets Section

Is your feature request related to a problem? Please describe.
we don't have code samples in the repository
Describe the solution you'd like
create a folder called code-snippets and add a file as an example with any solidity code.

I would like to take on this task.

fixed by #189


Bug: broken ci

the publish task will fail, it seems the lock file is generated and lerna does not like uncommited changes.
I will try to find the reason why it's generated, "dirty fix" would be to ignore uncommited changes when lerna is running.

I think it's due to the new version of pnpm. When I use pnpm install with pnpm version 9.0.6. it reproduce the whole lockfile.
I opened a PR for add --frozen-lockfile in #216

So I think there are two option.

add --frozen-lockfile in ci command.
commit new lockfile from new version of pnpm
or both


Water supply/demand by basins in allocation (new node type TargetLevel)

Fixes https://github.com/Deltares/Ribasim/issues/564.

Hi @SouthEndMusic, just a small check. When using the vertical fluxes from the simulation, I assume you take the average or accumulation and not the instantaneous values

@gijsber, good point, I currently take instantaneous values. I am somewhat aware of efforts to obtain integrated values (also requested for coupling: https://github.com/Deltares/Ribasim/issues/1084). As far as I am aware the ball for this one is in @visr's court.

To do for adding flows into basins to the objective function:

[ ] Define absolute value variables for the flows into the basins
[ ] Define constraints on these variables so that they act as the absolute value of the right expression
[ ] Finish add_basin_term! by passing the proper constraints to add_objective_term!
[ ] Properly call add_basin_term! add the end of set_objective_priority!

@gijsber in this PR I will keep using instantaneous values, I made a new issue for using averaged values: https://github.com/Deltares/Ribasim/issues/1121.

I added some tests with this model:

The idea is that basin 2 starts with its level below the minimum level, and so user 3 will only abstract after the basin has been filled. This does work:

There are 3 stages:

Before the first allocation solve, where the user abstracts fully (identical to when allocation is not used);
Between the first allocation solve and when the basin reaches its maximum level, where the user does not abstract;
After the basin has reached its maximum level, where the user abstracts fully again.

All this is qualitatively as expected, but when I writing the tests I found some weird but structured errors. Here are the errors per stage (expected - simulated):

See also the code for the tests in allocation_test.jl.


Start markdown

[ ] Testando
[ ] Turn on GitHub Pages
[ ] Outline my portfolio
[ ] Introduce myself to the world


For prometheus, where to scrape metrics from metrics-server?

Hi,
After setup, I only get this apis endpoints from https://localhost:443/:
# curl -k -XGET -H "Authorization: Bearer $TOKEN" https://localhot:443/
{
  "paths": [
    "/apis",
    "/apis/metrics",
    "/apis/metrics/v1alpha1",
    "/healthz",
    "/healthz/healthz",
    "/healthz/ping",
    "/healthz/poststarthook/generic-apiserver-start-informers",
    "/swaggerapi/"
  ]
}

Where does metrics-server expose metrics in Prometheus format?
I want to config my prometheus to scrape metrics from metrics-server, because I need information from kubelet /stat/summary endpoint.

metric server does not expose the underlying metrics -- it's not intended to be used for this purpose.  You'll need to scrape them directly from the nodes.  In 1.7.3+, they should be available at $KUBELET/metrics/cadvisor.

please provide more info on what metrics-server is, if not a server for exposing metrics.

I found some documentation around metrics-server on the official docs page: https://kubernetes.io/docs/tasks/debug-application-cluster/core-metrics-pipeline/

@kfox1111 see #21

@cofyc   have you got answer of your query ?  I  am also at the same point , new in this domain , any help appreciated

I was a little confused. When I heard of heapster's deprecation I recall not pointing directly to any server named heapster or seeing it anywhere in my prometheus scrape config.
When I look at metrics-server docs, I see the new URL is /apis/k8s.metrics.io/
I currently scrape the below to get node and pod metrics. Is this good or should I be using /apis/k8s.metrics.io ?
https://kubernetes.default.svc:443/api/v1/nodes//proxy/metrics/cadvisor
https://kubernetes.default.svc:443/api/v1/nodes//proxy/metrics

Prometheus does not scrape heapster.  It goes directly to the nodes, or via the API server proxy to the nodes.  What you're doing is correct.
/apis/metrics.k8s.io is intended for consumption by kubernetes system components so that we're not tied to a particular TSDB.  It's not designed as a way for metrics pipelines to gather metrics.

we made a metrics-server-exporter for metrics server, can be useful for those who miss the heapster metrics

could you tell me which endpoint should i check to see metrics provided by kublet on nodes ?

Since metrics-server doesn't supply metrics about itself, is there a good way to monitor it?


Cannot retry an upload in the demo app.

If uploading a website fails in the demo app, it's impossible to retry: It will immediately stop because it cannot create public/www.

I'm getting something similar to this.   See this thread.   https://github.com/maidsafe/safe_examples/issues/95


There are no commands defined in the "tmg" namespace.

RT

@sevendiscovery Sorry, since this package was originally used in our project, the namespace has not been changed, this issue has been changed in v1.0.1, you can use v1.0.1


added go mod tidy to the linting env to fix import errors

This PR will fix the Undefined issues when running CI linting tests. You can see the errors in this issue: #2785
What does this PR do?
This PR will add another step to the lint phase which will run a 'go mod tidy'. This way all of the packages will be loaded into the gocache and the linter will be able to reference these objects.
Testing:
To test this simply you can follow the steps of the linter on a local machine:

clone the epinio repo
run the linter as specified via the test: golangci-lint run --timeout=10m --skip-files docs.go
note the failed import
run go mod tidy to update the local go cache
rerun the lint and notice the import errors are gone.

Note the passing CI tests above ^

Note that the CI tests are now passing except for the release which is related to https://github.com/epinio/epinio/issues/2784


Do not add profiling time traces that are shorter than 250microseconds

Please only review last 2 commits. All others are part of #1043
Motivation
I'd like to be able to cheaply add tracing to functions that are normally very fast but have bad worst-case behaviors. Prompted by #1043
Test plan
Ran on pay-server, ran with -e 1, in both cases interesting parts of traces remained.
Putting @jvilk-stripe as a reviewer, as this might affect some of your metrics as "super fast" requests will no longer be included in distributions.

My approval only applies to the last two commits.