Frigate is using deprecated API call in 2021.12 (dev version)

Version of the custom_component

v2.1.0
Configuration
configured through UI.
Describe the bug
A clear and concise description of what the bug is.
In Home Assistant 2021.12.0.dev20211106, the api async_timeout.timeout was deprecated, to be removed in 2022.2.  This must have occured in the last 2 days because I last updated my dev unit to 2021.12.0.dev202.11103.  Anyways, this warning is spamming the logs every 5 seconds.
Debug log

2021-11-07 05:16:53 WARNING (MainThread) [homeassistant.helpers.frame] Detected integration that called async_timeout.timeout with loop keyword argument. The loop keyword argument is deprecated and calls will fail after Home Assistant 2022.2. Please report issue to the custom component author for frigate using this method at custom_components/frigate/api.py, line 138: async with async_timeout.timeout(TIMEOUT, loop=asyncio.get_event_loop()):

Confirmed. For others facing this issue, a temporary means of preventing the logspam could be to only allow logging errors (not warnings), although this should be strictly temporary as other deprecations/warnings will be hidden.
logger:
  logs:
    homeassistant.helpers.frame: error


Allow anonymous github issues to be added to linear as app integration

Summary
Previously when a non synced user on linear was submitting an issue on GitHub it is not synced to linear. At Novu we use linear to triage issues with linear triage mechanism. And issues submitted by users were not synced.
This functionality follows a similar path with anonymous comment by creating the linear issue as an app and not a particular user.
This will only work if the app token env variable is available in the system.
I'm not sure if this is a behaviour everyone will want, but we really needed this internally.
Test Plan
Creating issues with anonymous users, and working with this for the last few days on our repo at novu.
Related Issues
None
Issues that created will look like this:

@tedspare I would actually generate a LINEAR_APPLICATION_ADMIN_KEY for the the Linear application you have created that generated oauth keys on linear. And then the issues created will be actually written as created by "SyncLinear.com" which makes total sense to me. For self-hosting the users need to generate this key for themself. Wdyt?

save to merge?

LGTM, sorry for late response @scopsy. Are you able to merge in this repo?

Yes sure, we've been working with for a while now and it works great so merging this


Ensure markdown links don't get messed up in language packs by using "Locked" comment

from https://github.com/microsoft/vscode/issues/126614#issuecomment-977181372

add a dev comment like {Locked="]({1})"} should be able to enforce there is no space added between.
There will validation error if space added. (previous plane text comment won't cause validation error).
Assume translation violate the rule still got checked in. Build process will revert the translation back to English because violate the loc rule. Hence there won't be function break.

so adding
'{Locked="]({0})"}'

to the comment array here:
https://github.com/microsoft/vscode/blob/4947eba7a5e7160d937115ee4030671182d8aca7/src/vs/workbench/contrib/workspace/browser/workspaceTrustEditor.ts#L834
so it looks like:
comment: ['Please ensure the markdown link syntax is not broken up with whitespace [text block](link block)', '{Locked="]({0})"}']

should enable this validation.
I need to find all of these comments and change them accordingly.
cc @danyeh

It's also possible to do this for package.nls.json files like so:
{
	"view.workbench.scm.disabled": {
               "message": "If you would like to use git features, please enable git in your [settings](command:workbench.action.openSettings?%5B%22git.enabled%22%5D).\nTo learn more about how to use git and source control in VS Code [read our docs](https://aka.ms/vscode-scm).",
               "comment": [
			"Please do not add a space between ] and (. It is a special link syntax.",
			"Please do not translate the 'command:....' text. It is a special link syntax.",
			"{Locked=\"]({0})\"}",
			"{Locked=\"(command:\"}"
		]
        }
}

from https://github.com/Microsoft/vscode/issues/126408#issuecomment-862205952

I'm going to close this and fix these as they come up


Recent update, Not working

Recently updated App and i now cannot load the app when trying to search through ingame
All i get is this message when i try to search for something

You can try reinstall completely. You'll need to also delete the %appdata%/sidekick folder.

You can try reinstall completely. You'll need to also delete the %appdata%/sidekick folder.

I have reinstalled, and deleteed everything in the %appdata%/Sidekick folder and still having the same issue

Sidekick_log20241224.log

Maybe fixed in #392
We are releasing the fix soon. After the release, can you try again?

After the recent update release it is now working again.
Thankyou for the quick responses.


Missing index on table group_membership for postgres (2)

In addition to https://github.com/cloudfoundry/uaa/issues/647 another index is missing on
delete from group_membership where group_id in (select id from groups where identity_zone_id = ?)
This causes a delete of an identityzone to spend 1.4 seconds in the DB in one of our systems.

@mwdb I'm unable to find the query
delete from group_membership where group_id in (select id from groups where identity_zone_id = ?)

We will still add an index to this column, and we will investigate #647 at the same time

this query comes when an identity zone is deleted.

https://www.pivotaltracker.com/story/show/149310593


Merge two sorted Linked List

Greetings sir/Mam,
I would like to contribute the code for this issue in C++/CPP.
The code is ready. I have added the code , along with proper explanation of the algorithm , test cases , time complexity and space complexity of the same, as per the guidelines.
Kindly assign it to me.
This is regarding GSSOC'21.
Regards,
Hitain Kakkar

my name is Nikita Jangid, i would like to contribute the code for this issue in cpp, please assign it to me.

present at #468


Change $this parameter in closure to $that to prevent memory leak.

Using $this as a parameter name in a closure under PHP 7 causes a memory leak under some specific circumstances which are present in this component.
Took me a while to track this down! ;)
Relevant PHP issue: https://bugs.php.net/bug.php?id=71737
More information: https://gist.github.com/jmalloc/e3db5842c2c4ab2a1edf

Great work :+1:, tagging in a few moments! How did you even find this one? Really impressed that you managed to track it down to this.

Lots of bisecting an existing codebase, a good set of eyes on @ezzatron, and the dumb luck of accidentally misusing call_user_func as though it were call_user_func_array:
call_user_func($closure, [$this])
Thanks for tagging it up so quickly :)

Awesome, very well done you two :+1: .
It is a rather nasty bug that had to be fixed and tagged quickly, plus it is a very simple fix and requires less time then feature PR's.


when i pip insall tiny-cuda-nn, raise error?

raise EnvironmentError('CUDA_HOME environment variable is not set. '
OSError: CUDA_HOME environment variable is not set. Please set it to your CUDA install root.
Building PyTorch extension for tiny-cuda-nn version 1.7
Obtained compute capability 61 from PyTorch
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.

Please follow the instructions on how to install tiny-cuda-nn in the following repo:
https://github.com/NVlabs/tiny-cuda-nn
If you experience any errors installing it, open an issue there.


Sort on IP address incorrect

Bug description
I've got networks in several different subnets, but the IP adress column under Network does not sort correctly. E.g. It treats <IP_ADDRESS> > <IP_ADDRESS> - which from a subnet point of view isn't correct - 172.16.2 is in the 172.16.0/20 subnet, while 172.16.18 is in the 172.16.16/20 subnet.
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

Expected behavior
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
Steps to reproduce the issue:
Create several networks with the subnets above and go to the "Networks" tab and sort on IP address column.
Technical details:

Portainer version: 1.20.2
Docker version (managed by Portainer): 18.09.6
Platform (windows/linux): Linux

Seeing now this is a duplicate of https://github.com/portainer/portainer/issues/1802 with an open PR https://github.com/portainer/portainer/pull/2334. Maybe this question on SO can help solve it: https://stackoverflow.com/questions/48618635/require-sorting-on-ip-address-using-js @deviantony @pantegrow

Duplicate of #1802


Add the Gentoo Linux guide

Your Gentoo guide was very good, thank you!
I hope your website becomes successful!
I wrote an article about the Gentoo guide because I really wanted to see that as a website.

I am currently verifying if the article works well

Finished verifying, guide works well.

To me this looks like its pretty much just the audio rip from the video, and I think that just makes it sound really weird for a website guide.

In this case I have 10GB of RAM and five CPU threads. So I will use the -j5 and -l5 options.

This is an example of what I mean. In the video it makes sense to word it like this, but in a guide it sounds weird to say "In this case". You also skip over the fact of why 10GB of Ram and 5 CPU threads makes you pick -j5 -l5, others might think its just a number you get a feeling for.
I think its a good starting place, but I think some of the wording might need to change to make it fit a text guide more.

Great guide! As @arthurmelton says it might need some cleanup in some places, just to make the tone more appropriate. I'll merge it in the meantime as a draft.

@arthurmelton I agree, I just wanted to make a draft for people to add on top of it (I am not an expert in explaining stuff)


Improve the documentation regarding the Deploying WSO2 API-M in a Distributed Setup

Description:
In order to share the deployment artifacts, Earlier we recommended to share the entire <API-M_HOME>/repository/deployment/server directory. But now its not necessary and sharing synapse-configs and executionplans are enough. Shall we improve the documentation including that part.

The doc [1] and [2] needs to be corrected.
[1] - https://apim.docs.wso2.com/en/latest/install-and-setup/setup/distributed-deployment/synchronizing-artifacts-in-a-gateway-cluster/
[2] - https://apim.docs.wso2.com/en/latest/install-and-setup/setup/reference/common-runtime-and-configuration-artifacts/#common-runtime-and-configuration-artifacts

This has been fixed in 3.2.0 docs.
[1] - https://apim.docs.wso2.com/en/latest/install-and-setup/setup/distributed-deployment/synchronizing-artifacts-in-a-gateway-cluster/#artifact-synchronization-with-a-shared-file-system
[2] - https://apim.docs.wso2.com/en/latest/install-and-setup/setup/reference/common-runtime-and-configuration-artifacts/#persistent-runtime-artifacts


Improvements.

Hello @jameschch thanks for a great platform you have created. As I wrote you I have taken your existing project as a base, and elaborated it little further. And here are the thigns I think could be imrpoved ( I am giving just a brief overview sorry if do not make every point as clear as it could be):

One of the most important things is walking forward optimiation. That has to be intuitively clear and easy to set up. There are not too much information on a subject on the internet but these two fine articles I have found that I would like to share with you:
https://profsoftware.com/bt/wfo.htm
http://www.codefortraders.com/Walk-Forward_Analysis/BTWFMgr.htm
these are the documentation for two ready WFO implementations
I tried to implement it similiar to what they have got:
https://github.com/IlshatGaripov/OptimizationLeanJames/blob/master/Optimization/OptimizerManager/WalkForwardOptimizationManager.cs

Cloud computing. To solve complex optimization problem I could see no better solution that to enable some sort of cloud computing. Depending of conditions of WFO there is a need sometimes to do thousands of  backtests per an experiement. So it lead to implementation of :
https://github.com/IlshatGaripov/OptimizationLeanJames/blob/master/Optimization/Common/AzureBatchManager.cs
The module does enable to expand computing power in a cloud - create compute nodes - download all the historical data to the storages and then to every node - do the backtest save the result to storage than translate it to the local PC and so on. This works in union with :  https://github.com/IlshatGaripov/OptimizationLeanJames/blob/master/Optimization/GeneticSharpExtensions/TaskExecutorAzure.cs
The drawback it takes a time to create the nodes - aroudn 20 min - and to uplaod the data to every node. Probably more convenient to have a multiprocessor architecture at home - 100 or more PCs ;)

Genetic optimization algorithm itself and oprationts that it performs were overwritten at many parts and results ot this work  were even highlighted in a separate folder
https://github.com/IlshatGaripov/OptimizationLeanJames/tree/master/Optimization/GeneticSharpExtensions
which makes me think that genetic optimizatino SDK could be in whole overwridden for the purposes of optimizing trading strategies. Because most of mutation selection operation etc that GeneticSharp has available are not evry applicable for the trading task and are more suitable for many classic academic optimization problems - like shift mutation or some binary mutation and so on are not applicable here at all.. on the other hand, other sort of mutations must be implemented and present.. by the way it feels like GA is a powerful solution for trading params search optimization problems - it requires more thoughtful study - so it ended up I used only the few files frmo GeneticSharp - rewrote all the rest.

All
https://github.com/jameschch/LeanOptimization/tree/master/Optimization/Fitness
classes I have joinged in a single functionality inside:
https://github.com/IlshatGaripov/OptimizationLeanJames/blob/master/Optimization/Common/StatisticsAdapter.cs
The static method CalculateFitness does all the job depending on fitnessFunction variable set by config file. But this is a minor thing..

These are all valuable suggestions. Sorry if I've not picked up exactly what you are trying to put across. Some of these issues are quite subtle so feel free to clarify where necessary.

A full walk forward optimizer it going to be my top priority. I did already put together a very simple measure that works on a similar time-step pattern:

NestedCrossSharpeMaximizer
This is based on my integration with the optimizers from SharpLearning, which extends the capabilities beyond Genetic to Bayesian, Particle Swarm, Random and Grid Search, Smac and a few others.
Regardless: correct walk forward will be next.

Yes your work integrating with Azure is impressive. I had not considered the compute as a service angle, if I have used a remote host I just deployed and ran the entire optimizer process. You would need a resource pool rather than waiting for spin-up. I have been looking into this for AWS and expect Azure would not be very different.

I think you're saying the repo can be restructured into folders. I completely agree and have finished a restructure a little while ago. In conceptual terms, I agree the genetic optimization has limited utility which is why I have integrated with the SharpLearning optimizers. The GA pattern does remain quite convenient even for the non-genetic optimizers.

There might be a little consolidation possible with the fitness classes.

James,

I have not peered in detail at your optimizers from SharpLearning, have been playing around with genetic optimization only - but they look promising! And this should be something I need to work with in the next experiments.

What concerned to Azure.. You say: To do this properly, you would need a resource pool rather than waiting for spin-up. The way the things are designed they actually create a pool at a first start. And then we can use it in the consecutive experiements with no time overhead required to create pool again. But upon completion of the calculations, the program will ask whether to delete the resource pool - we can choose no - if want to reuse the pool again. But after  the last experiment we still have to remove it as Microsoft charges for the resource useges for the fact the nodes are deployed - no matter whether or not they are currently in use. it’s certainly not the same as deploying resources exactly before each experiment but still there is a need to wait for the pool to be created and data be uploaded to the nodes - can take up to 40 min - once before the experiment session - which I am not sure how to optimize rather then supporting own multiprocessor architecture at home/office.

What I mean by that is that Genetic Sharp lib source files in my opinion had to be imporved in many parts to solve the problems and sometimes it
it was not possible to do due to some restrictions such as library interfaces, etc. So I decided to support own GeneticSharp clone inside the project.. Here how it looks renewed :
https://github.com/IlshatGaripov/OptimizationLeanJames/tree/master/Optimization.Genetic
And within the bounds of inner project I am free to many any changes that I feel useful..

You expressed interest before in Walk-forward optimization. Thanks to your encouragement, I have just merged my effort at this into the master branch of my .net core port of the Lean Optimizer available here:
https://github.com/jameschch/LeanParameterOptimization
I'm going to be shifting my focus over to developing this other codebase. I already have a great deal more features including some Blazor interfaces etc. Next step will probably be to follow your suggestion of stripping the genetic patterns into a separate channel and focusing more on the other optimizers such as Random, Grid, Bayes etc.
The whole cloud scale out problem is something I've given a lot of thought but have not yet had the nerve to tackle.

Hi @jameschch  that sounds interesting I would be definitely interested to study the new implementatio, as for the cloud computing, I managed to do everything smoothly enough and by the way the problem I wrote about with long deployment has been also resolved - it takes now 5 min to deploy all the computing powers in Azure to start the experiments.. probably this could be re-used and integrated in new version.. it's been busy time for me lately with new project sorry but I would be glad to follow the progress. Thanks!

I've been reflecting on the current version of the walk forward and think the current version is suitable for walk-forward validation but needs a multi-layer cost function over the cost of each out sample fold and the variance between the optimal parameters for each fold. I have been looking closely at this very promising approach to multi-level cost functions: https://arxiv.org/pdf/1808.06846. However, I suspect this goes too far and maybe all that's needed here is a simple weighted cost function. Maybe you could take a look and tell me what you think?
The next thing to move onto is going to be seamless cloud integration I think.

Hi @jameschch sorry it's been a busy time for me recently I have taken a look and I think this is probably an interesting approach if we take cost functions that based on different factors and which can vary greatly in time from each otherr - this could help to find optimum that will be potentially good (stable) in terms of all factors that have been taken into account. that's interesting. as for the search for a global maximum, maybe any of the heuristics from this book could come in help https://cs.gmu.edu/~sean/book/metaheuristics/Essentials.pdf  I have read in long time ago - and only few parts what are the genetic algorithms related - and one of the approaches that one can use I believe that is suggested in this book is to add at each step a sufficient number of new random points to search for new extremums that will help to prevent stagnation over the local min/max.


Update electron demo

Continuing from #1980 fixing the commented issues. Closes #1981. Addresses #1994 as well.

Ran the demo and got an error when I tried to click the "click here to select a file" button:
index.js:68 Uncaught (in promise) TypeError: Cannot read property 'showOpenDialog' of undefined
    at HTMLButtonElement.handleF (index.js:68)

@garrettluu what OS / electron version are you using?

@garrettluu what OS / electron version are you using?
Electron 9.0.5, Ubuntu 20.04

Ran the demo and got an error when I tried to click the "click here to select a file" button:
index.js:68 Uncaught (in promise) TypeError: Cannot read property 'showOpenDialog' of undefined
    at HTMLButtonElement.handleF (index.js:68)

@garrettluu I pushed in a fix for this. Can you try again?

Ran the demo and got an error when I tried to click the "click here to select a file" button:
index.js:68 Uncaught (in promise) TypeError: Cannot read property 'showOpenDialog' of undefined
    at HTMLButtonElement.handleF (index.js:68)

@garrettluu I pushed in a fix for this. Can you try again?

@srijonsaha Works for me now! Thanks!


Updated docker hash to match latest image

Description
This PR adjusts the hash referenced in the CircleCI config and other files to point to the latest one from the transcom/circleci-docker repository.  The docker image was recently changed from using circleci/python:3.9.1-buster-node to circleci/python:3.9.2-buster-node as its base image.
Reviewer Notes
Did I get the correct hash?  Did I miss changing any hashes in this repo?
I also changed the hash in Dockerfile.cypress because this base image change affects all the different images, not just a single one.
Setup
Verify that all CircleCI checks run successfully.
Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
PR from transcom/circleci-docker

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 54fd0e306116d5e800d722fa6f99623c5b07a53c


cagebreak: 2.2.3 -> 2.3.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/project-repo/cagebreak/releases.
meta.description for cagebreak is: A Wayland tiling compositor inspired by ratpoison
meta.homepage for cagebreak is: https://github.com/project-repo/cagebreak
meta.changelog for cagebreak is: https://github.com/project-repo/cagebreak/blob/2.3.0/Changelog.md
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.3.0 with grep in /nix/store/lz9z148s6chmydrzl4a1yr6xhjlxlm5j-cagebreak-2.3.0
found 2.3.0 in filename of file in /nix/store/lz9z148s6chmydrzl4a1yr6xhjlxlm5j-cagebreak-2.3.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cagebreak

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lz9z148s6chmydrzl4a1yr6xhjlxlm5j-cagebreak-2.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cagebreak https://github.com/r-ryantm/nixpkgs/archive/4eaef9d98b43d5e643cf61eda40c7ba8c15325f8.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/4eaef9d98b43d5e643cf61eda40c7ba8c15325f8#cagebreak

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lz9z148s6chmydrzl4a1yr6xhjlxlm5j-cagebreak-2.3.0
ls -la /nix/store/lz9z148s6chmydrzl4a1yr6xhjlxlm5j-cagebreak-2.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cagebreak

Maintainer pings
cc @berbiche for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/project-repo/cagebreak/releases.
meta.description for cagebreak is: A Wayland tiling compositor inspired by ratpoison
meta.homepage for cagebreak is: https://github.com/project-repo/cagebreak
meta.changelog for cagebreak is: https://github.com/project-repo/cagebreak/blob/2.3.1/Changelog.md
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.3.1 with grep in /nix/store/fyxg1pmixpkf0lfqk4z028jc7yskjyp1-cagebreak-2.3.1
found 2.3.1 in filename of file in /nix/store/fyxg1pmixpkf0lfqk4z028jc7yskjyp1-cagebreak-2.3.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cagebreak

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fyxg1pmixpkf0lfqk4z028jc7yskjyp1-cagebreak-2.3.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cagebreak https://github.com/r-ryantm/nixpkgs/archive/6bd7e67a49fe93103769b4b381587fe31c774ceb.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/6bd7e67a49fe93103769b4b381587fe31c774ceb#cagebreak

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fyxg1pmixpkf0lfqk4z028jc7yskjyp1-cagebreak-2.3.1
ls -la /nix/store/fyxg1pmixpkf0lfqk4z028jc7yskjyp1-cagebreak-2.3.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cagebreak

Maintainer pings
cc @berbiche for testing.


Generate report output

markdown file containing:

link to out.png diagram
summary table of each region:

region name
colour
region origin (mem address)
region size (bytes)
remaining space (bytes)
region collision (bool)

614a77cab812b0c48ff42b2593dd91e2f6aebe5e


feat: add key events to the editor.

Adds the following key events:
while alt key is pressed: treat everything like move mode
1: move tool
2: area tool
3. brush tool
4: mark tool
5: token tool
shift: toggle reveal/hide mode
CMD/Ctrl + S: push map to players

@n1ru4l Feel free to merge this when you're ready.


[AutoPR batchai/resource-manager] chore: jsonfmt batchai

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5919

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/2927


Add new ID for Teamfight Tactics Choncc's Treasure Game Mode

Added new "Pengu's Party" tft queue. Please considere a merge @stelar7


Ranks and Permissions Disappearing after restart

Hello,
I have a auto restart system setup, every time when the server restarts all the ranks and permissions have been removed from every player. Every time when that happens we need to manually restart it again to restore it. It's been happening for a while now. I believe it's a luckperm bug.

Check for startup errors. If data isn't being saved it's because LP can't properly initialize the data storage.


[Bug] grafana-operator rbac is missing routes/custom-host permission

Describe the bug
With version 4.1.0 a regression was introduced where the grafana-operator cluster role is missing the routes/custom-host permission. This breaks deployments on top of OpenShift and the following error can be observed when checking for events in the namespace:
14m         Warning   ProcessingError         grafana/grafana                                             Route.route.openshift.io "grafana-route" is invalid: spec.host: Forbidden: you do not have permission to set the host field of the route

Version
v4.1.0
To Reproduce
Steps to reproduce the behavior:

Install operator
Deploy grafana with ingress

Expected behavior
Deployment succeeds
Suspect component/Location where the bug might be occuring
grafana-operator/config/rbac/role.yaml

@NissesSenap sounds like this might require a 4.1.1?

Yes, for sure.

This is solved in https://github.com/grafana-operator/grafana-operator/pull/644 and it will be part of the next release.


FIX: Undo caused crash when inserting newline at the end of text

Line 195 causes "vector subscript out of range" when undoing a newline on the last line.
Unsure if this is a proper fix or if it could be made better.

Whoa... that's a big list of changes. Is it possible to break it down into smaller ones? I would like to merge most of them, but not all, at least not at once. Can we discuss it please?

Whops. I thought i made the fix based on the origin, must have been tired when i made the request. My current version have been heavily modified to fit my purposes (Focus on Lua only). And now it have some tight coupling to my private project making it unfit for sharing. I could try to split out the general parts into a public fork, but that will require some work reorganizing the commits.
The inclusion of imgui_internal.h was because I'm using ImRect.

Ok, that would be great, I think it is enough to split it into a few separate commits, grouped by topic. Just let me know if you have done it.
I keep avoiding using inclusion of imgui_internal.h because I do not want to have any extra dependencies on ImGUI's internals, which might change every now and then and might cause extra work necessary every time the code is compiled against the new version of ImGUI.


Anndata

Closes #7, closes #8

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

44 of 47 (93.62%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.6%) to 92.364%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/harissa/core/model.py
13
14
92.86%

src/harissa/core/dataset.py
31
33
93.94%

Totals

Change from base Build<PHONE_NUMBER>:
-1.6%

Covered Lines:
1460

Relevant Lines:
1529

💛  - Coveralls


Introduce shellcheck to lint shell scripts

This PR fixes a number of issues in several shell scripts that shellcheck complains about. It also introduces a new CI job to ensure shellcheck compliance in the future.
Issues fixed:

https://www.shellcheck.net/wiki/SC2145
https://www.shellcheck.net/wiki/SC2027
https://www.shellcheck.net/wiki/SC2034
https://www.shellcheck.net/wiki/SC2046
https://www.shellcheck.net/wiki/SC2124
https://www.shellcheck.net/wiki/SC2164
https://www.shellcheck.net/wiki/SC2294
https://www.shellcheck.net/wiki/SC2059
https://www.shellcheck.net/wiki/SC2086
https://www.shellcheck.net/wiki/SC2295
https://www.shellcheck.net/wiki/SC2002
https://www.shellcheck.net/wiki/SC2005
https://www.shellcheck.net/wiki/SC2006

Issues ignored:

https://www.shellcheck.net/wiki/SC2013

Follow-up to #15139

It will be nice to have this 🙂


Update README.md

Add RaspberryPi docs to setup WIFI on boot.

Updated PR with Official link. Wifi setup instructions are nice to have front and center.

Updated PR with Official link. Wifi setup instructions are nice to have front and center.


Bug: We are able to deploy devtron applications in env which is not created from cluster and environment section.

📜 Description
We are able to select env which is not created from cluster and environment section during deploying devtron applications.
but in backend it is going to create helm release name app name with environment name.(these env name contain underscore).
Drop down list

Error screenshot

error logs
Invalid value: "sh.helm.release.v1.external_apache.v1": a lowercase RFC 1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is 'a-z0-9?(.a-z0-9?)*')
👟 Reproduction steps

Install devtron without CI CD mode
Install ci cd integration
create the ns from kubectl cli.
create a devtron app and in workflow configuration , we will see the env names(created by kubectl cli) for deploying with drop down.

👍 Expected behavior
It should only show the environments which are created from cluster and environment section including devtron-demo.
👎 Actual Behavior
Right now we are seeing the dropdown with environments list which contains all.
☸ Kubernetes version
1.22
Cloud provider

Azure

🌍 Browser
Chrome
🧱 Your Environment
microk8s
✅ Proposed Solution
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find any similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

@ajaydevtron please verify.


Provide Path Mappings for debugging

Discussed in https://github.com/php-perfect/ddev-intellij-plugin/discussions/54

Originally posted by nico-loeber May 19, 2022
To allow zero configuration debugging from a browser a path mapping is required.
It looks like a deployment configuration is required to do so. A "local or mounted folder" deployment with a path mapping project directory => /var/www/html could be added to do so.

I've added a deployment configuration containing a path mapping for the mounted ddev container.
Sadly PHPStorm refuses to use this mapping. Hope to get some feedback on this issue: https://youtrack.jetbrains.com/issue/WI-55140


Create Helping Chef.c

Related Issue
Solved issue number: #248
Describe the changes you've made
added solution of the helping chef problem
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Language used for coding
C
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] New and existing unit tests pass locally with my changes.

add some comments.

add some comments.

Please have a look,
I have done required actions


feat(node): reimplement node SDK in wasm

Re: #373
POC re-implementing the node SDK with wasm instead of FFI
This will make it platform independent!
TODO

[ ] Cleanup/update documentation
[ ] Implement periodic fetching in NodeJS of flag state?
[ ] Add back browser tests (need to split up the call to wasm-pack to allow for both nodejs and web
[ ] Update client to return XResponse instead of Result types
[ ] Update packaging workflow for Node client

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.34%. Comparing base (c130095) to head (199e16d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #378   +/-   ##
=======================================
  Coverage   88.34%   88.34%
=======================================
  Files          10       10
  Lines        3699     3699
=======================================
  Hits         3268     3268
  Misses        431      431

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Deleting a point in edit mode will not remove symbol from map.

Add a point,
Put point in edit mode,
Delete symbol
point stays on map.
It will also stay on map regardless if editing is complete.

Discovered when cancelling or completing an edit, a link is established back to map root.  Once this occur and the link to the parent overlay is removed, there is no command being sent to the engine to remove.  Investigating what is causing this link.

Method emp.editingManager.edit() is creating a emp.typeLibrary.Feature class through copying the object in the data store.  The method used is feature.getObjectData(mapInstance, parentCoreId).   The mapInstance is being passed in as the parentCoreId, and that is what creates that extra link later on.   The parentCoreId is currently required by Cesium as a V2 holdover, and was not available from the emp.storage.findFeature method, so we thought that getObjectData would fix this.  without the parentCoreId the cancel fails to restore to original state.  Looking at other methods for retreiving the correct parentCoreId and potentially removing the need for it in Cesium since it is no longer required in our v3 baseline.

Jimmy,
to test, add point or line.  edit feature.  test cancel, and delete line/point  it should remove.   do the same for complete and delete.   Items should delete.
then test polygons make sure they still work too.

Reintegrate


BUG: nbinom.cdf is very slow

Describe your issue.
I noticed that scipy.stats.nbinom.cdf() is much slower than the most naive pure python implementation.  I was wondering why that was.
Reproducing Code Example
In [13]: %timeit nbinom.cdf(16, 26, 0.5)
110 µs ± 2.68 µs per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

This  is 110 µs for a simple call.

Now consider the following naive pure Python implementation.

def nbinom_pmf(k, n, p):
        return comb(k+n-1, n-1)* p**n * (1-p)**k

def nbinom_cdf(k, n, p):
        return sum(nbinom_pmf(x, n, p) for x in range(k+1))

We now get:

%timeit nbinom_cdf(16, 26, 0.5)
10.5 µs ± 64.2 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)

This is about 10 times faster than the scipy call.
Error message
This not an error report.

SciPy/NumPy/Python version information
blas_mkl_info:   NOT AVAILABLE blis_info:   NOT AVAILABLE openblas_info:     libraries = ['openblas', 'openblas']     library_dirs = ['/usr/local/lib']     language = c     define_macros = [('HAVE_CBLAS', None)]     runtime_library_dirs = ['/usr/local/lib'] blas_opt_info:     libraries = ['openblas', 'openblas']     library_dirs = ['/usr/local/lib']     language = c     define_macros = [('HAVE_CBLAS', None)]     runtime_library_dirs = ['/usr/local/lib'] lapack_mkl_info:   NOT AVAILABLE openblas_lapack_info:     libraries = ['openblas', 'openblas']     library_dirs = ['/usr/local/lib']     language = c     define_macros = [('HAVE_CBLAS', None)]     runtime_library_dirs = ['/usr/local/lib'] lapack_opt_info:     libraries = ['openblas', 'openblas']     library_dirs = ['/usr/local/lib']     language = c     define_macros = [('HAVE_CBLAS', None)]     runtime_library_dirs = ['/usr/local/lib'] Supported SIMD extensions in this NumPy install:     baseline = SSE,SSE2,SSE3     found = SSSE3,SSE41,POPCNT,SSE42,AVX,F16C,FMA3,AVX2     not found = AVX512F,AVX512CD,AVX512_KNL,AVX512_KNM,AVX512_SKX,AVX512_CLX,AVX512_CNL,AVX512_ICL

This is because the distribution infrastructure is extremely slow at slow at checking for NaNs, arguments outside the support, etc. If you use the _cdf method, which is private, all of that is bypassed, and the result is much faster.
from scipy.stats import nbinom
%timeit nbinom.cdf(16, 26, 0.5)  # 72.7 µs ± 634 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)
%timeit nbinom._cdf(16, 26, 0.5)  # 1.97 µs ± 17.4 ns per loop (mean ± std. dev. of 7 runs, 1000000 loops each)

The need for the distribution infrastructure to be more efficient and for all the exception case checks to be optional are both tracked in gh-15928.

I tested again and ._cdf is much faster but still slower than the naive python implementation for small parameter values. For example:
In [22]: %timeit nbinom_cdf(1, 26, 0.5)
978 ns ± 30.3 ns per loop (mean ± std. dev. of 7 runs, 1,000,000 loops each)
In [23]: %timeit nbinom._cdf(1, 26, 0.5)
3.08 µs ± 221 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)
The difference is even more stark for ._pmf, e.g.
In [24]: %timeit nbinom_pmf(1, 26, 0.5)
282 ns ± 1.61 ns per loop (mean ± std. dev. of 7 runs, 1,000,000 loops each)
In [25]: %timeit nbinom._pmf(1, 26, 0.5)
2.03 µs ± 6.55 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)

The same could be said of many SciPy/NumPy functions:
import math
%timeit math.sin(3.5)  77.2 ns ± 0.196 ns per loop (mean ± std. dev. of 7 runs, 10000000 loops each)

import numpy as np
%timeit np.sin(3.5)  # 516 ns ± 2.02 ns per loop (mean ± std. dev. of 7 runs, 1000000 loops each)

They are optimized for array input so that they are fast when the computation would otherwise be slow.
x = np.linspace(0, np.pi, 10000)
%timeit np.sin(x)  # 39.9 µs ± 241 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

%timeit [math.sin(xi) for xi in x]  # 1.07 ms ± 5.07 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)

But yes, there is some overhead, so pure Python may be faster in cases that take very little time.


train FCOS

Hello
I tried to train FCOS but I get this error:
INFO:croptrain.data.datasets.visdrone:Loaded 548 images in COCO format from /home/mahilamoghadami.mut/CZDet/dataset/VisDrone/annotations_VisDrone_val.json
INFO:croptrain.engine.inference_fcos:Start inference on 548 batches
INFO:croptrain.engine.inference_fcos:Inference done 11/548. Dataloading: 0.0010 s/iter. Inference: 0.1016 s/iter. Eval: 0.0006 s/iter. Total: 0.1032 s/iter. ETA=0:00:55
INFO:croptrain.engine.inference_fcos:Inference done 59/548. Dataloading: 0.0017 s/iter. Inference: 0.1027 s/iter. Eval: 0.0006 s/iter. Total: 0.1051 s/iter. ETA=0:00:51
INFO:croptrain.engine.inference_fcos:Inference done 107/548. Dataloading: 0.0020 s/iter. Inference: 0.1026 s/iter. Eval: 0.0006 s/iter. Total: 0.1053 s/iter. ETA=0:00:46
INFO:croptrain.engine.inference_fcos:Inference done 155/548. Dataloading: 0.0020 s/iter. Inference: 0.1026 s/iter. Eval: 0.0006 s/iter. Total: 0.1052 s/iter. ETA=0:00:41
INFO:croptrain.engine.inference_fcos:Inference done 202/548. Dataloading: 0.0020 s/iter. Inference: 0.1027 s/iter. Eval: 0.0010 s/iter. Total: 0.1057 s/iter. ETA=0:00:36
INFO:croptrain.engine.inference_fcos:Inference done 249/548. Dataloading: 0.0020 s/iter. Inference: 0.1031 s/iter. Eval: 0.0009 s/iter. Total: 0.1061 s/iter. ETA=0:00:31
INFO:croptrain.engine.inference_fcos:Inference done 296/548. Dataloading: 0.0020 s/iter. Inference: 0.1032 s/iter. Eval: 0.0009 s/iter. Total: 0.1062 s/iter. ETA=0:00:26
INFO:croptrain.engine.inference_fcos:Inference done 344/548. Dataloading: 0.0020 s/iter. Inference: 0.1033 s/iter. Eval: 0.0008 s/iter. Total: 0.1061 s/iter. ETA=0:00:21
INFO:croptrain.engine.inference_fcos:Inference done 392/548. Dataloading: 0.0020 s/iter. Inference: 0.1033 s/iter. Eval: 0.0008 s/iter. Total: 0.1061 s/iter. ETA=0:00:16
INFO:croptrain.engine.inference_fcos:Inference done 439/548. Dataloading: 0.0020 s/iter. Inference: 0.1034 s/iter. Eval: 0.0011 s/iter. Total: 0.1066 s/iter. ETA=0:00:11
INFO:croptrain.engine.inference_fcos:Inference done 486/548. Dataloading: 0.0021 s/iter. Inference: 0.1035 s/iter. Eval: 0.0010 s/iter. Total: 0.1066 s/iter. ETA=0:00:06
INFO:croptrain.engine.inference_fcos:Inference done 533/548. Dataloading: 0.0021 s/iter. Inference: 0.1035 s/iter. Eval: 0.0010 s/iter. Total: 0.1067 s/iter. ETA=0:00:01
INFO:croptrain.engine.inference_fcos:Total inference time: 0:00:57.919741 (0.106666 s / iter per device, on 1 devices)
INFO:croptrain.engine.inference_fcos:Total inference pure compute time: 0:00:56 (0.103500 s / iter per device, on 1 devices)
Traceback (most recent call last):
File "train_fcos.py", line 89, in
launch(
File "/home/mahilamoghadami.mut/miniconda3/envs/CZDET/lib/python3.8/site-packages/detectron2/engine/launch.py", line 82, in launch
main_func(*args)
File "train_fcos.py", line 80, in main
return trainer.train()
File "/home/mahilamoghadami.mut/miniconda3/envs/CZDET/lib/python3.8/site-packages/detectron2/engine/defaults.py", line 484, in train
super().train(self.start_iter, self.max_iter)
File "/home/mahilamoghadami.mut/miniconda3/envs/CZDET/lib/python3.8/site-packages/detectron2/engine/train_loop.py", line 150, in train
self.after_step()
File "/home/mahilamoghadami.mut/miniconda3/envs/CZDET/lib/python3.8/site-packages/detectron2/engine/train_loop.py", line 180, in after_step
h.after_step()
File "/home/mahilamoghadami.mut/miniconda3/envs/CZDET/lib/python3.8/site-packages/detectron2/engine/hooks.py", line 552, in after_step
self._do_eval()
File "/home/mahilamoghadami.mut/miniconda3/envs/CZDET/lib/python3.8/site-packages/detectron2/engine/hooks.py", line 525, in _do_eval
results = self._func()
File "/home/mahilamoghadami.mut/CZDet/DroneDetectron2/croptrain/engine/trainer_fcos.py", line 234, in test_and_save_results
self._last_eval_results = self.test_crop(self.cfg, self.model, self.iter)
File "/home/mahilamoghadami.mut/CZDet/DroneDetectron2/croptrain/engine/trainer_fcos.py", line 278, in test_crop
results_i = inference_fcos.inference_with_crops(model, data_loader, evaluator, cfg, iter)
File "/home/mahilamoghadami.mut/CZDet/DroneDetectron2/croptrain/engine/inference_fcos.py", line 124, in inference_with_crops
results = evaluator.evaluate()
File "/home/mahilamoghadami.mut/miniconda3/envs/CZDET/lib/python3.8/site-packages/detectron2/evaluation/coco_evaluation.py", line 194, in evaluate
self._eval_predictions(predictions, img_ids=img_ids)
File "/home/mahilamoghadami.mut/miniconda3/envs/CZDET/lib/python3.8/site-packages/detectron2/evaluation/coco_evaluation.py", line 228, in _eval_predictions
assert category_id < num_classes, (
AssertionError: A prediction has class=10, but the dataset only has 10 classes and predicted class id should be in [0, 9].
while I trained FaterRCNN with your code without any error. I mean that data structure , annotations and classes are ok and are used without any changes compared to training FasterRCNN.
this is my train script:
python train_fcos.py --num-gpus 1 --config-file configs/FCOS-CROP.yaml OUTPUT_DIR outputs_FCOS_crop_pre
appreciate for help.
Thanks

I think we already discussed that this is a class index mismatch error that comes due to the "ignore" or "other" category present with the Visdrone dataset.  If I remember correctly, we fixed it in one of the issues you raised but I don't know how you are getting the same error again (probably using an old version of the code or json file?). In my case, I am ignoring it while loading the JSON file here (for training and testing):
https://github.com/akhilpm/DroneDetectron2/blob/main/croptrain/data/datasets/visdrone.py#L236
and while evaluating here:
https://github.com/akhilpm/DroneDetectron2/blob/main/croptrain/engine/trainer.py#L172
With this info, you may check what wrong you are doing in your case.


Prevent out of bounds panic when value size exceeds c.config.MaxEntrySize

panic: runtime error: slice bounds out of range

goroutine 17 [running, locked to thread]:
github.com/allegro/bigcache/queue.(*BytesQueue).allocateAdditionalMemory(0xc8200b6a08)
	/home/dolf/Projects/ClueGetter/src/github.com/allegro/bigcache/queue/bytes_queue.go:70 +0x512
github.com/allegro/bigcache/queue.(*BytesQueue).Push(0xc8200b6a08, 0xc821871300, 0x3b, 0x3b, 0xd)
	/home/dolf/Projects/ClueGetter/src/github.com/allegro/bigcache/queue/bytes_queue.go:54 +0x407
github.com/allegro/bigcache.(*BigCache).Set(0xc820072930, 0xc3abe0, 0xd, 0xc8217f56a0, 0x1c, 0x20)
	/home/dolf/Projects/ClueGetter/src/github.com/allegro/bigcache/bigcache.go:105 +0xb12

Coming to think of it, I'm not sure about this PR. Obviously a panic is bad, but the solution may not be ideal either. In my case I've got objects that can be as large as 48 MiB. However, given that most items will only be a few kb I'm not sure I should actually set the MaxEntrySize to 48 MiB.

Hi @Freeaqingme,
Thank you for the first open PR in this project. Currently there are no limits on value size and I prefer to keep it in that way. You should be able to put whatever you like to the cache (MaxEntrySize is only used to calculate initial size of the cache) Therefore what you have described is a bug. I have submitted fix for it: #2 Please, verify if it solves your problem. If it does then I will merge it to the master.

@druminski Tnx! I don't seem to be able to trigger this panic anymore, so I'd say it's good to go.
=> Closed

@druminski Consider renaming MaxEntrySize to InitialCacheSize

@Freeaqingme awesome, merged.
@janisz ok, InitialCacheSize does not explain that this need to be size of an entry. I have added comment to that field, so I hope it is clearer.


IPNetwork Parse/TryParse and common networks

Parse and TryParse methods
Static props for common networks like rfc1918 and link-local

Hi @moander, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

IPAddress.Parse behaves differently on OS X.
https://github.com/dotnet/corefx/issues/11843

Please update the samples to show the new APIs

We're not going to take a dependency on immutable collections for what seems like a such an obscure use case.

@moander Could you sign the CLA if you agree with the changes proposed?

Hi @moander, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Closed/reopend to trigger the CLA bot

@moander, thanks for sending this PR. Closing this one as this is quite stale right now. Please file a bug if you'd like to bring this back to discussion.


Add Nikita to authors

Add Nikita to the authors list in the Cargo.toml files for the crates. Per Anthony's suggestion. I'm not sure if you meant for just credential or for the other crates as well. I can modify the PR to only add my name to credential if that's what you meant.

I believe he meant all of the crates so the PR is good to go. 😄


Change exit code when ParserException ?

Actually many people have already reported problems with PHP 7.1.
Me too I've actually theses problems.
But I think the exit code must be 1, like describe in your documentation.
Here it is on this line, it add errors but it is not catches here for return 1.
So CircleCI returns that all right, and in fact, it is not.
Yes it is a temporary problem until you update library, but I think it will be a good things if we have a wrong exit code to see them.

Good  point, it is a problem in the Command and PHPMD classes that don't check if we have any error.


"Updated pyproject.toml to fix installation issue"

Fixes _MissingDynamic:'readme' error on pip install.
...
Getting requirements to build wheel ... error
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> [51 lines of output]
C:\Users_\AppData\Local\Temp\pip-build-env-0zbhnviw\overlay\Lib\site-packages\setuptools\config_apply_pyprojecttoml.py:75: _MissingDynamic: readme defined outside of pyproject.toml is ignored.
...
https://docs.github.com/github/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax

fixed from dev branch


Enable the Ukrainian version of the website

Enable the Ukrainian version of the website
Description
Сhanged the default language to Ukrainian.

Summary by CodeRabbit

New Features

Introduced support for dynamic language selection through the NEXT_PUBLIC_MAIN_LANGUAGE environment variable, enhancing localization.
Implemented auto-release workflow documentation to streamline release processes via GitHub Actions.
Updated workflows to enhance security and flexibility, allowing for comprehensive changelog generation and release creation capabilities.

Bug Fixes

Improved localization in various components and tests, ensuring accurate translations for UI elements and messages.

Refactor

Replaced hardcoded strings in multiple components and tests with dynamic translations using the t function for improved internationalization.

Chores

Updated project name in package.json, indicating a shift in branding.
Added localization initialization scripts to manage translations effectively across the application.

@coderabbitai full review

@coderabbitai review

@coderabbitai full review

@coderabbitai full review


As VALIDATOR, I want to cross-check my study data against the study define.xml for variable length mismatch so that I can stay compliant with SD1231

This check applies to SEND, SDTM, and ADaM data.
This check is IG-version-agnostic.
This check needs to work with both variable and value level metadata.

SD1231 in FDA VR v1.6:

Rule message: Variable value is longer than defined max length<EMAIL_ADDRESS>when value-level condition occurs
Rule description: Value level length must not exceed the length as specified define.xml.
per define-xml-2-1:

The variable length in characters.
Business Rules:
Length is the maximum expected variable length. It is not based on the length of the data collected.
Length should be present only when DataType is "Text", "Integer" or "Float".
For ItemDef elements referecend from a ValueListDef, the Length must be less than or equal to the length of the parent variable.
furthermore,
for when DataType is integer, Length is the largest allowable integer width; note that for negative integers, the length will include the minus sign.
for when DataType is float, Length is the largest allowable whole number width plus the maximum number of decimal digits.

The acceptance criteria for the issue have been successfully fulfilled, indicating that all the required conditions and expectations have been met. The engine has effectively identified instances of variable length mismatch for integer, text, and float data types. This means that the engine accurately detects cases where the length of variables does not align with the expected values.
Moreover, the engine has demonstrated its capability to incorporate erroneous values into the generated report. This ensures that any discrepancies or inconsistencies regarding variable length are clearly highlighted and documented. As evidence, I have attached one of the reports generated during the validation process, which provides a comprehensive overview of the identified issues and their corresponding details.
CORE-Report-2023-07-14T14-21-54.xlsx

Validation Summary
Validation Methodology
The validation process incorporated rigorous testing methodologies to ensure the accuracy and reliability of the system. This included:
Manual Testing:
Manual Testing was done for validating the issue. The rules engine test validation was run against a dataset having positive and negative cases for all the points in Acceptance criteria. The reports were generated and then validated that each correct negative case was identified successfully by the engine.
Unit Testing:
Unit tests were run to validate the implementation against edge cases. The tests were also reviewed for:

Test Coverage
Test Design and Structure
Test Data and Inputs
Assertions and Expected Results
Test Efficiency

Code Review:
The code changes in the code files were reviewed and were found to be in accordance with the Acceptance criteria. Also, the code was reviewed for following:

Code Quality Evaluation
Logic and Functionality Review
Error Handling and Exception Handling Assessment
Security Considerations

Regression Testing:
Regression tests were run to validate that the changes did not affect any functionality of the overall system.
Test Execution:

The Tests were run by executing test validation against different datasets. Following command was used
python -m core test -s sdtmig -v 3.4 -r rule/file/path -dp dataset/file/path
The regression tests were executed using the following command:
python -m pytest tests/QARegressionTests

Results:
After successful running of manual and regression tests and careful validation, the issue was found to be good to close as the Acceptance criteria was fulfilled and all validations passed.
Deviations and Non-conformances:
No deviations or non-conformances were observed.
Summary of Findings:
Test Results: All tests conducted during the validation process passed successfully, indicating that the system functions as intended and meets the specified requirements. The tests covered various scenarios and edge cases, providing confidence in the system's reliability and correctness.
Unit Regression: The unit regression tests were executed, and no regressions or issues were identified. This indicates that the changes made during the development or modification of the system did not introduce any unintended side effects or break existing functionality.
Code Reviews: The code reviews conducted by both development and QA teams were completed, and no major issues or concerns were identified. The code adheres to coding standards and best practices, demonstrating good code quality, readability, and maintainability.


Freedom ipsum

In order to help #271 get merged into main I fixed the last issues.
cc @dlx55

Hi @AlexWayfer because you probably reviewed recently #271 I would like to get a review. Because I helped this guy getting his PR merged into main. 🤣


fix role "switch" requiring "aria-checked" when used with native input element

Closes #932
As described in the issue #932 and the Switch ARIA pattern, the aria-checked attribute is not required when the target element is a native <input /> element with type="checkbox".

Thank you for resolving the issue!


Gain calculation incorrect for Nprof > 1 cycles

When there is more than one profile in a single cycle (Nprof > 1), float data for that cycle all gets lumped together for the gain calculation as cycles are used as the index for each profile. This is not an issue for the WOA data, as it uses time as the index and when Nprof > 1, the cycles will be the same but time will be different.
for example, take float 4902480:
>>> df = pd.Dataframe(data=track, columns=['cycle', 'sdn', 'lat', 'long'])
>>> df.head()
Out:
   cycle           sdn        lat       long
0    1.0  18066.659028  58.497711 -53.264366
1    1.0  18068.222917  58.526173 -53.361496
2    2.0  18078.216667  58.873486 -52.927937
3    3.0  18088.231944  59.413994 -51.272209
4    4.0  18098.230556  58.972908 -50.491405

Proposed solution: change gain calculation to accumulate/index data by time instead of cycle. Be aware that this could have adverse effects on data that have unique timestamps for each datapoint, but for now, where only complete profiles have a timestamp, this should work.

Indexed by time now, so correction should be correct. Comparison script to audit should be double checked.

Comment from Henry Bittig:

Every profile will/must have one unique JULD time, even if there are measurement times associated with individual samples. Those measurement times are available in the b-profile's MTIME parameter, not in the s-profile.


Update to 4.0.12443

A new version is now published (4.0.13615) : https://github.com/boubou/chocolatey/tree/master/packages/jabra-direct.
My pull request has expired.


Not able to initialise the values in forms????

What is your issue?

I think I have the same issue at version 6.4.1
<Field   name="test"   component={MyComponent}   value="test" />
in MyComponent in props the initial value this.props.input.value is empty string

@gustavohenke so ok, I pass it with initialValues and use it in MyComponent. But on validation, or submit, there are no my values, till I call onChange there. Did I miss something?
I have similar issue with redux-form-material-ui, how can I pass current value to SelectField for instance?

@theturboboy Take a look here you need to use connect you can't just do <Field component={MyComponent} initialValues='foo' /> which I assume you are doing now.

was this feature ever supported? eg doing <Field value="bla" />?

No, that has never been a feature.


add ability to select cloud providers comma separated

Awesome tool! It would also be great if we could add the ability to select different cloud providers comma separated

You can add your own provider in this format: https://raw.githubusercontent.com/Ne00n/Looking-Glass/master/data/everything.json

How is it? How to choose provider?


Open up a little the authentication delegation

For a customization of the DelegatedClientAuthenticationAction, I have needed to use the isLogoutRequest method.

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


Remove the CompositeDevice.tuple() method

IMHO having a public method called tuple() just makes things too confusing - the first time I saw self.tuple(some_args) I automatically assumed it was a typo of just tuple(some_args). And the way it was used in CompositeDevice.value as return self.tuple(some_args) when it was defined as being def tuple(self): just looks weird. And it's not included in the API documentation either.
I'm not sure I can see a use for it being a public property anyway - why would a user actually want/need to get at the underlying namedtuple type (rather than just relying on whatever the value property returns) ?
OTOH if you do want to keep it as a public property (and reject this PR), perhaps it should be renamed to namedtuple or device_tuple_type and then added to the documentation.

Current coverage is 72.33%

Merging #272 into master will decrease coverage by -0.26% as of 4bd189b

@@            master    #272   diff @@
======================================
  Files           28      28
  Stmts         4062    4060     -2
  Branches         0       0
  Methods          0       0
======================================
- Hit           2949    2937    -12
  Partial          0       0
- Missed        1113    1123    +10

Review entire Coverage Diff as of 4bd189b

Powered by Codecov. Updated on successful CI builds.

The main reason for having the function was so the user could get hold of the tuple type used by the CompositeDevice in order to manually construct objects to feed to value. For example, imagine you've got some device with the sub-components foo and bar and you have a function which yields values for those components. It'll work perfectly well without names:
class MyDevice(CompositeOutputDevice):
    def __init__(self):
        super(MyDevice, self).__init__(foo=LED(4), bar=LED(5))

def my_source():
    v = True
    while True:
        yield tuple(v, not v)
        v = not v

my_device = MyDevice()
my_device.source = my_source()

But I think it's a little nicer with them:
class MyDevice(CompositeOutputDevice):
    def __init__(self):
        super(MyDevice, self).__init__(foo=LED(4), bar=LED(5))

def my_source(target):
    v = True
    while True:
        yield target.tuple(foo=v, bar=not v)
        v = not v

my_device = MyDevice()
my_device.source = my_source(my_device)

Still, I take your point that it probably shouldn't be called tuple. namedtuple sounds a lot better to me, and it should be added to the docs (that's purely an oversight on my part).
I'm just working on a re-org of the docs after some experiments last night. I think I've got something that's a little nicer, so I'll rename tuple to namedtuple for that and include it in the docs.

Okay, we've gone with namedtuple in the 1.2 release, so I'll close this - but thanks for the suggestion - it was definitely worth doing something about it!

The reason I didn't close this myself after the release of 1.2, was because you haven't added namedtuple to the docs yet... ;)

Doh! Doc-string's missing. I'll get to this in a bit (too busy with picamera stuff right now)


adding cors header to api doesn't respond to options for non-simple requests

Laravel Version: 5.5
PHP Version: 7.0
Database Driver & Version: n/a

Description:
I have a simple cors middleware which is responsible for sending the correct headers (listed below) and I've added a class reference to the 'api' key under the route middleware groups. For simple requests (GET), this works fine and a non-local host can access the api. However, for PUT / POST, the correct OPTIONS headers are not returned preventing the POST/PUT to continue. However, if I move the reference to the CORS middleware from the groups to the middleware array, it works as expected.
// cors middleware code:
{
header('Access-Control-Allow-Origin: *');
$headers = [
	'Access-Control-Allow-Methods' => 'GET, POST, PUT, DELETE, OPTIONS',
	'Access-Control-Allow-Headers' => 'X-Requested-With, Content-Type, Accept, Origin, Authorization',
	'Access-Control-Allow-Credentials' => 'true',
];

if ($request->getMethod() == "OPTIONS") {
        return Response::make('OK', 200, $headers);
    }

$response = $next($request);

foreach ($headers as $k => $v)
	$response->header($k, $v);

return  $response;

}
// code added to 'api' key of middlewaregroups array in Kernel.php
\App\Http\Middleware\Cors::class,

Check the order of the middleware.

Use / see https://github.com/barryvdh/laravel-cors

@Dylan-DPC  There's only one middleware in the api key
@Kyslik The Cors middleware above should be fine? I did look at that middleware but couldn't see any difference ultimately

@ChangePlaces just read package's readme it will help you out. And yes I use that package and it works for my use.

Using a package shouldn't be a solution to a problem

The problem here is that for "simple" cors requests like GET, the middleware is activated, however for complicated requests (POST/PUT/PATcH) the OPTIONS reques is not / doesn't appear to be activated. I think the real, underlying problem is that, for some reason how laravel handles the api requests doesn't work properly when assigning middleware to those methods.
Assigning CORS to all routes is not desirable and is not a solution.

Please ask on the forums, this repo is for bug reporting only. You can use https://laracasts.com/discuss or https://laravel.io/forum which are forums with a very large community of developers helping each other.

@themsaid this is a bug

You can search this repo for similar issues, there has been a lot of discussions around that already.


Expose local RenderingDevice creation to RenderingServer

Marked as draft because I don't think that this is the way that @reduz wants to expose RenderingDevice. Also I am not sure why this works, I feel like it shouldnt.
Anyway, I am opening this PR in case others want to play around with compute shaders from GDscript.
Here is a minimal example of a compute shader.
in a GDScript _ready() function
var shader_file = load("res://compute.glsl")
var shader_bytecode = shader_file.get_bytecode()
var rd = RenderingServer.create_local_rendering_device()
var shader = rd.shader_create(shader_bytecode)
var pipeline = rd.compute_pipeline_create(shader)

var storage_buffer = rd.storage_buffer_create(64)
var u = RDUniform.new()
u.type = RenderingDevice.UNIFORM_TYPE_STORAGE_BUFFER
u.binding = 0
u.add_id(storage_buffer)
var uniform_set = rd.uniform_set_create([u], shader, 0)

var compute_list = rd.compute_list_begin()
rd.compute_list_bind_compute_pipeline(compute_list, pipeline)
rd.compute_list_bind_uniform_set(compute_list, uniform_set, 0)
rd.compute_list_dispatch(compute_list, 8, 1, 1)
rd.compute_list_end()

In a file named "compute.glsl"
#[compute]

#version 450

layout(local_size_x = 8) in;

layout(set = 0, binding = 0, std430) restrict buffer ColorBuffer {
  float data[];
}
color_buffer;

void main() {
    color_buffer.data[gl_GlobalInvocationID.x] = 1.0;
}

Thanks!

Hi fellow devs.
Maybe I am missing something but I thinks this is broken right now (don't know if I should answer here or open an issue).
I added this after your _ready() example:
var data = rd.buffer_get_data(storage_buffer)
print("size:" + str(data.size()))
print("data[0]:" + str(data[0]))
print("data[63]:" + str(data[63]))

Output:
size:64
data[0]:0
data[63]:0

If I debug, all the 64 elements of the PackedByteArray are filled with zeros.
I tried to change the size of storage_buffer and it did change the size of my PackedByteArray but still filled with zeros.

@ChristopheClaustre
I can see 2 potential issues:

The GPU may not have finished writing the data before you read it back. You may need to add some idle time in between when you submit the GPU command and read it. Remember, GPU and CPU are async, so you can't count on a result being available immediately
GDScript has received a lot of changes and I'm not sure what the implicit byte -> float conversion looks like. If this is the problem, you can try using integers in the shader to see if the conversion goes smoother.

For the GPU and CPU async, I have the code on my PC and the rd.compute_list_end() seams to wait for GPU (here).
I will try to wait before fetching results and to use integers.
I will keep you informed.

Some news : It works !
First, I was having trouble with the godotscript byte decoding API (reading the C++ directly was simpler).
Then, you were right about timing, waiting one frame before fetching seams mandatory. I suppose that Godot submit all the orders to Vulkan at the end of the frame and so I can't get my result earlier (what about the barrier ?). If I am right, I feel it must not work that way for compute shader but Godot must have its reasons (alpha ?).
Thank you for your time @clayjohn.

@ChristopheClaustre Great to hear!
Indeed, it is still difficult to deal with bytes directly in GDScript, I think there have been some improvements recently, but we will need to do more work to make it easy.
Regarding the frame timing. The GPU and CPU run out of sync with one another. This means that the CPU code will continue running without waiting for the GPU code to finish. This is one of the benefits of using compute shaders, but it means you need to be careful about when you retrieve your data from the GPU.
In the future, we will be adding documentation to make this clear, and we may add more to the API to do things like force the CPU to stall until data is returned.

Hello!
I'm trying to figure out why my output is wrong while running this example. I'm using most current sources. Only changes I have made was adapting to API changes and replacing get_bytecode() with get_spirv() and shader_create with shader_create_from_spirv. I've also changed u.type with u.uniform_type. Now the problem is, my output of 64 bytes is filled with repeating pattern being four values: [0, 0, 128, 63]. When debugging C++, I can see the same exact values on C++ side, so it's not any conversion issue, I guess. What are my options now, can I use RenderDoc on exported project to debug this further?

@tito91 it sounds like you need to convert groups of 4 bytes into a float! One way to do that is use the function to_float32_array on your packedbytearray

@clayjohn Oh wow, that's a bummer. Thanks!


Feature Request: Disable Upgrade Process per OS

I would love this model to be extended to more OS in order to track golden firmware drift.
Would you have any interest in being able to allow a user to disable the upgrade process and track software versions for more platform/NOS'?
Given standard NAPALM drivers already leveraged in NetBox and included in the netbox-docker, you could use NAPALM to get the software version from the device using the already set NAPALM user and password settings in NetBox.
With those model, more users could leverage the plugin for SoT type functionality, but leverage their existing workflows to upgrade their devices (ansible, python, salt, vendor specific platforms).

By the way, thanks for contributing this!  Awesome job!

hi, did not completely get idea. you asked about assignment gold image to PID without uploading, just for tracking purpose?

Sorry I apparently missed the comment!
I think your process of tracking and upgrading is a good idea.
Use cases where it would be valuable to be able to track in a data model the "standard OS version" per model/device-type without the upgrade process being enabled:

Depending on who have access to netbox, some people would be hesitant to allow the upgrade process to even be enabled, but would.
Where an OS upgrade process is not supported.  (but again valuable to pull the version number and associated link from the GUI and API for their existing upgrade workflow via Ansible, Nornir, etc.)

Let me know if you need a better write up

Hi, i got the idea, will try to add option just to set target (golden) SW version without uploading image. For tracking purpose only.


Few struct parsing failures

Here's a minimal example:
typedef struct test {
    int *test1, *test2;
} test;

I also noticed that this doesn't compile either
typedef struct {
    int (*const hash)(const void *, int);
} PyHash_FuncDef;

unsigned long PyThread_start_new_thread(void (*)(void *), void *);

Thanks for reporting, now generating:
test :: struct {
    test1 : ^_c.int,
    test2 : ^_c.int,
};

PyHash_FuncDef :: struct {
    hash : #type proc(unamed0 : rawptr, unamed1 : _c.int) -> _c.int,
};

@(default_calling_convention="c")
foreign  {

    @(link_name="PyThread_start_new_thread")
    PyThread_start_new_thread :: proc(unamed0 : #type proc(unamed0 : rawptr), unamed1 : rawptr) -> _c.ulong ---;

}

For your test cases.

You sir are a gentleman and a scholar, thank you very much! Hopefully I won't catch many more issues


Enable some passing bihu proofs

Switch these proofs over to using #gas (infinite gas).
Allows the  cell to change state in these specs.
Add no-evaluators to function symbol with no rules.

Looks good to me, waiting on CI to finish


chore: credo and oca to gitignore

Add two additional projects to the .gitingore so that local development can follow a similar pattern to Bifold.

aries-oca-bundles/
credo-ts/

@cvarjao I think I fixed the issue with the requirement for the iOS to build and run. The issue I have now is its not waiting for the entire workflow:
Native Build & Test / check-secrets (pull_request) Successful in 3s

This step is still running and takes several minutes.


Add VectorStackFrame to reduce memory usage in train_dqn_batch_ale.py

This PR adds

VectorEnvWrapper: an VectorEnv analog to gym.Wrapper
VectorStackFrame: an VectorEnv analog to chainerrl.wrappers.atari_wrappers.StackFrame

to reduce memory usage in train_dqn_batch_ale.py.
The current example wraps envs by StackFrame then by MultiprocessVectorEnv. This is actually memory inefficient because when LazyFrames is passed between processes via Pipe, it becomes not lazy anymore due to pickling. This PR solves the issue by stacking frames in the main process.
TODO:

[x] add experimental results to validate this change

I ran python3 examples/ale/train_dqn_batch_ale.py --num-envs 8 --steps 10000000 for Breakout and SpaceInvaders with three random seeds each. VectorFrameStack and FrameStack corresponds to this PR and the master branch, respectively.
The achieved scores are similar.

Below are memory usage 10 minutes before the training finished (seed=0, Breakout). VectorFrameStack recuded memory usage a lot.

VectorFrameStack: 9691M
FrameStack: 31006M

Thanks for your review! I followed your suggestions.


feat: Updated snippets and added JavaScript code snippets with Fetch

/update-dist

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

v1.9.0
GitHub release

Your semantic-release bot :package::rocket:


add support for plural name in annotation

Description
This change adds the argument pluralName to the annotation. In cases
where the name is not specified it defaults to the original behavior of
adding an "s" to the end of the entity name.
Related Issue
none
Types of changes

[X] Updated docs / Refactor code / Added a tests case / Automation (non-breaking change)
[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

tested this with spark-operator


AlphaFold Colab: Minimisation failed after 100 runs - disable amber relax - prediction does not download

I have had this issue for a while now using AlphaFold Colab (for some reason not AlphaFold2 Colab though) where I try and run a prediction, it tells me minimisation has failed after 100 runs which leads me to disable run relax and then it creates the prediction but doesn't download it
(I am connected to a custom GCE VM)
Anyone got any ideas about what the issue might be?

Thanks for raising this! This duplicates https://github.com/deepmind/alphafold/issues/483 and https://github.com/deepmind/alphafold/issues/550 so closing this issue.


Save the state for multiple projects in the same folder

The state of multiple projects in the same folder needs to be saved with a different name in the workspaceContext,  I added the name of the folder to distinguish between the two.
This better be checked in before PR #2993

@bobbrow  I have tested this branch on multi-PR


eslint: wrong autofix for multi-line queries

the following query auto-fixes correctly when using @tanstack/query/prefer-query-object-syntax:
// input
const query = useQuery(['key'], () => Promise.resolve('data'))

// output
const query = useQuery({ queryKey: ['key'], queryFn: () => Promise.resolve('data') })

However, if you have the same query in a multi-line setup, the output is wrong:
// input
const query = useQuery(
  ['key'],
  () => Promise.resolve('data'),
)

// output

const query = useQuery(
    ,
{ queryKey: ['key'], queryFn: () => Promise.resolve('data') })

Gif of working version:

Gif of non-working version

@Newbie012 FYI

I'll take a look at it


Finish translation -> tooltips

Well done for translanting @flossmanualsfr  !
but I think it still miss the translations of the tooltips, and one or two corrections...  do you do it, or I try ?

Try because i don't have seen where to do this translation, please :)
Elisa
2016-08-29 18:21 GMT+02:00 Camille Bissuel<EMAIL_ADDRESS>
Well done for translanting @flossmanualsfr
https://github.com/flossmanualsfr !
but I think it still miss the translations of the tooltips, and one or two
corrections... do you do it, or I try ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flossmanualsfr/escoria/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/AIBKBrDtOmX4k_nXJ9XKLUe79j05jOIBks5qkwcdgaJpZM4JvpIN
.

--
pour Floss Manuals Francophone
site : http://fr.flossmanuals.net
boutique : http://flossmanualsfr.net

                                                                                Hi :) the tooltip ids are explained in the i18n chapter, they should probably go in the general.csv file                                                                                Sent from my BlackBerry 10 smartphone on the Orange network.                                                                                From: Flossmanuals francophoneSent: Monday, August 29, 2016 6:03 PMTo: flossmanualsfr/escoriaReply To: flossmanualsfr/escoriaSubject: Re: [flossmanualsfr/escoria] Finish translation -> tooltips (#3)Try because i don't have seen where to do this translation, please :)

Elisa
2016-08-29 18:21 GMT+02:00 Camille Bissuel<EMAIL_ADDRESS>
Well done for translanting @flossmanualsfr
https://github.com/flossmanualsfr !
but I think it still miss the translations of the tooltips, and one or two
corrections... do you do it, or I try ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flossmanualsfr/escoria/issues/3, or mute the thread
https://github.com/notifications/unsubscribe-auth/AIBKBrDtOmX4k_nXJ9XKLUe79j05jOIBks5qkwcdgaJpZM4JvpIN
.

--
pour Floss Manuals Francophone
site : http://fr.flossmanuals.net
boutique : http://flossmanualsfr.net
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/flossmanualsfr/escoria","title":"flossmanualsfr/escoria","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/flossmanualsfr/escoria"}},"updates":{"snippets":[{"icon":"PERSON","message":"@flossmanualsfr in #3: Try because i don't have seen where to do this translation, please :)\n\nElisa\n\n2016-08-29 18:21 GMT+02:00 Camille Bissuel \u003cnotifications@github.com\u003e:\n\n\u003e Well done for translanting @flossmanualsfr\n\u003e \u003chttps://github.com/flossmanualsfr\u003e !\n\u003e\n\u003e but I think it still miss the translations of the tooltips, and one or two\n\u003e corrections... do you do it, or I try ?\n\u003e\n\u003e —\n\u003e You are receiving this because you were mentioned.\n\u003e Reply to this email directly, view it on GitHub\n\u003e \u003chttps://github.com/flossmanualsfr/escoria/issues/3\u003e, or mute the thread\n\u003e \u003chttps://github.com/notifications/unsubscribe-auth/AIBKBrDtOmX4k_nXJ9XKLUe79j05jOIBks5qkwcdgaJpZM4JvpIN\u003e\n\u003e .\n\u003e\n\n\n\n-- \npour Floss Manuals Francophone\nsite : http://fr.flossmanuals.net\nboutique : http://flossmanualsfr.net\n"}],"action":{"name":"View Issue","url":"https://github.com/flossmanualsfr/escoria/issues/3#issuecomment-243186036"}}}

I added a few strings in the game that will need to be translated. l’ll probably take care of that in the next two days.

I added strings for the credit scene and the tooltips in the genereal.csv file... and started to translate...
@flossmanualsfr or @punto-  do you feel like doing the last translations in spanish, plus the new one that will be added to strings.csv, and we generate everything again ?

So everything is ready for the final translation ;)

Done for French and English ! I removed Spanish until we can have some significant contribution.


open-eid

WIP
Things done

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

[x] NixOS

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@aszlig @bluescreen303 (top guys from google-talk plugin) please help me with firefox plugin. It doesn't seem to work :/

Current problems: qdigidoc doesn't compile, see [1]; firefox plugins seems not to work, see [2].
Any help is much appreciated!
[1] https://github.com/open-eid/qdigidoc/issues/9
[2] https://github.com/open-eid/browser-token-signing/issues/3

Current situation. qdigidoc compiles and works.
Firefox Plugin now sees the library but doesn't work anyway.

Now everything works!
Please note, that you need to do in firefox:
Settings->Advanced->Certificates->Security devices->Load
and add onepin-opensc-pkcs11.so from opensc pkg.

Going to merge in a day if no objections.

cc @joachifm for review :)

@jagajaga I'm afraid I know too little about this software to provide much useful feedback, but looks fairly straight-forward to me.

@joachifm thank you.

Actually, I just realized that certs.patch is absolutely gigantic. I don̈́t suppose it's possible to fetch it from somewhere instead of checking it into the repo?

@joachifm no, we can't fetch it from anywhere in easyway. Because it's located on private servers and downloaded at build time. We can't download during build time so that's why I've created a patch.


Make video plugin ready for republication

Removed use of List.cast to avoid an awkward Dart SDK constraint.
Bit of code cleanup.
Bump version for republication.

Are you going to publish this latest version of the video player package, we need it to be able to run the flutter gallery in strong mode. See my PR https://github.com/flutter/flutter/pull/14662.... Thanks.

Sorry, I forgot that step. Silly me.


[windows] remove pip2 declaration to reduce pip version confusion

What does this PR do?
removes the pip2 definition dependency, so that we don't have multiple pip installs

On Windows, is it required (or more generally, what are the reasons) to ship pip with python directly, instead of installing it separately from python?
Asking because installing it separately is what's done on Linux/macOS and makes things more flexible, as it makes it easy to upgrade pip (which is quite frequent) separately from python upgrades (which are less frequent).

After 6.13, we will repackage python such that it no longer includes pip itself.  Then the installed pip will become the correct, only version

Superseded by https://github.com/DataDog/datadog-agent/pull/3924


[FEATURE REQUEST] Ability to run a command before calculating cache key?

I have a situation where it would be useful to cache data, but the key needs to be a checksum of a file that isn't on the filesystem, yet.
An arbitrary example:
  - label: "Run Cached Step"
    command:
      - wget -O remote-config https://example.com
      - ./script-that-does-something.sh
    plugins:
      - gencer/cache#v2.4.10:
          key: "{{ id }}-{{ checksum 'remote-config' }}"
          paths:
            - .cache-dir

Running the wget command as a separate step won't work, as the file system gets cleaned, and even if there weren't, we run multiple agents.
Would it be possible/make sense to have a pre-run-command argument that I could have, Or perhaps be able to invoke the plugin manually?
Examples:
  - label: "Run Cached Step"
    command:
      - ./script-that-does-something.sh
    plugins:
      - gencer/cache#v2.4.10:
          key: "{{ id }}-{{ checksum 'remote-config' }}"
          pre-restore-command: wget -O remote-config https://example.com
          paths:
            - .cache-dir

or perhaps:
  - label: "Run Cached Step"
    command:
      - wget -O remote-config https://example.com
      - gencer-cache-restore
      - ./script-that-does-something.sh
    plugins:
      - gencer/cache#v2.4.10:
          key: "{{ id }}-{{ checksum 'remote-config' }}"
          defer_restore: true
          paths:
            - .cache-dir

This is probably a very niche/weird request.

This is probably a very niche/weird request.

Normally, Yes. But not in this situation because of:

Running the wget command as a separate step won't work, as the file system gets cleaned, and even if there weren't, we run multiple agents.

This makes this request meaningful.
There are 3 open requests incl. this one and I still not yet finished any of them. I'll try to implement ASAP. However, I am fully open for PRs (Generally speaking).

not stale

not stale


add option to listen on <IP_ADDRESS> instead on specific interfaces

should fix #38

I could merge this, but eventually I would like to make it a bit more automatic. Using <IP_ADDRESS> would work for most users, so maybe that should be the default and if it doesn't find any players it could try and bind to specific IPs instead. That is of course a bigger rewrite.

Hmm other question, whats the reason to search on all interfaces anyway? whats the advantage?
another option which would be a bit more generic might be that the option contain the listening ip, this might be more generic? I don't know if the automatic solution solve the issue, maybe you start sonos discovery before the sonos device is online, then it wouln't work.

The reason was because in some scenarios, when binding to "all" interfaces, it only listened to all interfaces, but the outgoing traffic was only sent on a single interface, and not necessarily the "correct" one. Binding to all interfaces individually gave me the possibility to send on all interfaces and just act on the one that received a response. This was especially common when you had virtualization software installed on your computer.
One could also try and loop through the interfaces and search for players (and one for <IP_ADDRESS>), but the discovery would be slower probably.

OK make sense. But what do you think about adding a specific IP address which could be used when the automatic discovery fails (like in my case)?

Well, a specific address won't work in your case since you have the same IP for tun0 and bond0. The <IP_ADDRESS> probably works because the OS itself will select the most appropriate interface to use (probably the one with the lowest metric), and in your case this is the best option (opposite to some other observations I've made).
I can merge this for now if you like, but I want to rewrite the logic later on. I also want to make this library Promise-based now that node 4.x.x has been released and there is native support for it.

Well <IP_ADDRESS> is a specific address too ;) I think thats fine, merge now and add more features later on.
BR and thanks


tf.reduce_sum not consistent with documentation when axis==[ ]

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  Linux Ubuntu 16.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.7.0
Python version:  3.5.2
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:  9.0.176 / 7.0.5
GPU model and memory: GeForce GTX 1050 Ti/PCIe/SSE2 4GB
Exact command to reproduce: custom script

Describe the problem
documentation says:
If axis has no entries, all dimensions are reduced, and a tensor with a single element is returned.
In practice:
If axis == []: nothing is reduced (the output is the input itself)
Source code / logs
import tensorflow as tf
constant = tf.constant([[1., 2.], [3., 4.]])
reduce = tf.reduce_sum(constant, axis=[])
with tf.Session() as sess:
print(sess.run(reduce))

The current behavior of tensorflow matches numpy:
# python
Python 2.7.12 (default, Dec  4 2017, 14:50:18)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import numpy as np
>>> x = np.asarray([[1., 2.], [3., 4.]])
>>> np.sum(x, ())
array([[1., 2.],
       [3., 4.]])
>>>

So I think this is a documentation (not implementation) issue. Created a PR #19509 for the documentation fix.


Use https for sparse checkout command

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Fix child selector issue in ornament

lambdaisland/ornament ignores the class names generated by girouette
and substitutes its own. This causes issues when girouette returns not
only a class name but also some additional selector. In this case, the
family of tailwind utilities like space-x-2 was causing issues because
it uses a common CSS selector pattern of: .space-x-2 > * + *.
To fix the issue, I've modified the implementation of several tw
utilities to have one additional layer of nesting in the garden output.
See the changes to the unit tests for an example.
See this discussion over on the ornament repo: https://github.com/lambdaisland/ornament/issues/5

Thank you, your pull request looks correct. Unfortunately, the selector has a problem with Garden during CSS generation:
(garden/css [".space-x-2" [#garden.selectors.CSSSelector{:selector "& > * + *"} {:margin-left "0.5rem"}]])
=> ".space-x-2 & > * + * {\n  margin-left: 0.5rem;\n}"

There might be a bug in Garden with the & selector at https://github.com/noprompt/garden/blob/31066ed2d7d769819c2d1d95460bd20818bc9561/src/garden/compiler.cljc#L176-L189
I found this ugly hack which works:
(garden/css [".space-x-2" [#garden.selectors.CSSSelector{:selector "&>*+*"} {:margin-left "0.5rem"}]])
=> ".space-x-2>*+* {\n  margin-left: 0.5rem;\n}"

Let's see if we can have it fixed in Garden before using the workaround.

@Vynlar On a second thought, waiting for Garden might take too long. Feel free to add the workaround as a new commit.

@green-coder I've updated this PR to use the workaround you found above. I've created a new helper between-children-selector to keep this hack in just one place. If the issue in garden is addressed, we could update between-children-selector to be more idiomatic.

Thank you !
Is this PR alone is sufficient for fixing https://github.com/lambdaisland/ornament/issues/5? If yes, I am going to make a new release.

@green-coder Yes, it should be!


shadowed:0.1.0

I am submitting

[x] a new package
[ ] an update for a package

Description: shadowed adds box shadows to Typst using SVGs and a Gaussian blur filter.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

Thanks!


runtime: Handle implementation of Enumerable::each

Proposal to fix #1547.
Motivation
The following code will pass sorbet-static but raise with sorbet-runtime:
class Foo
  include(Enumerable)
  extend(T::Sig)

  sig do
    implementation
      .params(blk: T.proc.params(arg0: Integer).returns(Integer))
      .returns(T::Enumerable[Integer])
  end
  def each(&blk)
    [1, 2, 3].each do |x|
      yield x
    end
  end
end

foo = Foo.new
foo.each { |x|
  puts x
}

sorbet-runtime will raise the following:
Error in `validate_non_override_mode': You marked `each` as .implementation, but it doesn't match up with a corresponding abstract method. (RuntimeError)
  Either check for typos and for missing includes or super classes to make the parent method shows up
  ... or remove .implementation here: Foo at file.rb:XX

Approach
The problem here is that sorbet-static tries to validate the implementation clause by looking at the super method. In this case to validate Foo::each it will look for Enumerable::each which doesn't exist at runtime.
The proposed solution is to include a definition and its signature for Enumerable::each at runtime. I simply included it in model_ext.rb.
At first I tried to add a special case for that at https://github.com/sorbet/sorbet/blob/058f4dab45e71d254e9a08edf116faa4dead6e5c/gems/sorbet-runtime/lib/types/private/methods/signature_validation.rb#L45. But the approach needed the Enumerable::each method and signature to be created by hand which seemed brittle.
Since all the work to parse signatures is already done, I just added the definition and let sorbet-static parse and check it. Not sure this is the correct approach. Seemed the simplest and safest to do that.
Test plan
Added two new tests to show the expected behaviour: https://github.com/sorbet/sorbet/pull/1563/files#diff-1ce8121afb699ae03063b08cca1be7adR1

Hi Alexandre,
How pressing is this for you?
We've seen quite a few problems caused by 'implementation' and we're considering condensing it with 'override' that woundn't require parent method to be abstract.
Would this help here?
WDYT?

@DarkDimius :

We've seen quite a few problems caused by 'implementation' and we're considering condensing it with 'override' that woundn't require parent method to be abstract.
Would this help here?

Yes, sounds like a really good idea, should also fix this problem.

How pressing is this for you?

I can find a temporary workaround if the above mentioned fix is coming shortly.

@DarkDimius any chance we can merge this one until the new override is ready?
Even if it's not the right direction, it still fixes a bad behaviour in sorbet-runtime until we have the new override ready.

any chance we can merge this one until the new override is ready?

Unfortunately, this fix will break Stripe :-( we check that classes aren't monkey patched by our gems and this will make it impossible for us to bump Sorbet at Stripe.
I'll bring this to our discussion on Monday to see what would be best course of action.


Searching

Added character count function
Formatted date with Momentjs
Still working on filtering messages to display

looks good!