Added libpcap-dev dependency

The go get command fails because go cannot find libpcap. Apt-getting it after grabbing golang fixes it.

👍 Thanks @afraisse !

Happy to contribute Thanos !
Aurproxy looks great for my use case (which is deploying applications with inter-dependencies onto Aurora).
I am struggling a bit though and would really value your input, if you have some time :)
I am trying to go through the getting started tutorial (on an Aurora cluster I deployed on AWS), but it seems that the service discovery announcement in ZK falls through. Have you ever heard of a similar issue ?
Here is Aurproxy’s stderr output :
016-05-12 13:55:15,527 [INFO] kazoo.client: Connecting to <IP_ADDRESS>:2181
2016-05-12 13:55:15,537 [INFO] kazoo.client: Zookeeper connection established, state: CONNECTED
2016-05-12 13:55:15,538 [INFO] tellapart.aurproxy.source.sources.serverset.[//aurora//www-data/devel/hello_world]: TellApart ServerSet initializing on path //aurora//www-data/devel/hello_world
2016-05-12 13:55:15,539 [WARNING] tellapart.aurproxy.source.sources.serverset.[//aurora//www-data/devel/hello_world]: Path //aurora//www-data/devel/hello_world does not exist, waiting for it to be created.
2016-05-12 13:55:17,570 [INFO] tellapart.aurproxy.backends.nginx.backend: No update required.
2016-05-12 13:58:17,695 [WARNING] tellapart.aurproxy.source.manager: No endpoints returned by regular sources.
2016-05-12 13:58:17,696 [WARNING] tellapart.aurproxy.source.manager: No endpoints returned by regular sources.
2016-05-12 13:58:17,696 [INFO] tellapart.aurproxy.backends.nginx.backend: No update required.
I digged into Zookeeper’s nodes and indeed couldn’t find any relevant ServerSet.
Thanks for your help.
Adrian
P.S. Also, to have Thermos working properly I had to install libcurl4-nss-dev into the Aurproxy container. Seems like an Aurora issue though.

Le 10 mai 2016 à 16:19, Thanos Baskous<EMAIL_ADDRESS>a écrit :
👍 Thanks @afraisse https://github.com/afraisse !
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/tellapart/aurproxy/pull/35#issuecomment-218172009

Hi Adrian,
Are you trying to point at a job that is based on the example in this project (examples/hello_world.aur) or at yoru own job? If it is your own job, can you confirm that it has named ports (configuration-level rendered command line contains something like "{{thermos.ports[http]}}") and is configured to announce itself into Aurora service discovery (service definition contains something like "announce = Announcer()")?
See https://github.com/tellapart/aurproxy/blob/master/examples/hello_world.aur for an example.

Hello Thanos,
Thanks for your reply. I am indeed using examples/hello_world.aur. I figured that my problem was related with Aurora configuration and not with Aurproxy, so I asked away on the aurora user list and got this reply :
I didn’t set the —announcer_ensemble flag in Thermos executor - a step that is already done when using the vagrant demo env (https://github.com/apache/aurora/blob/master/examples/vagrant/upstart/aurora-scheduler.conf#L45 https://github.com/apache/aurora/blob/master/examples/vagrant/upstart/aurora-scheduler.conf#L45).
Now everything is working ! :)
Btw, did you also experience any issues regarding libcurl4-nss-dev when running the Aurproxy image ?
Cheers
Adrian

Le 12 mai 2016 à 19:08, Thanos Baskous<EMAIL_ADDRESS>a écrit :
Hi Adrian,
Are you trying to point at a job that is based on the example in this project (examples/hello_world.aur) or at yoru own job? If it is your own job, can you confirm that it has named ports (configuration-level rendered command line contains something like "{{thermos.ports[http]}}") and is configured to announce itself into Aurora service discovery (service definition contains something like "announce = Announcer()")?
See https://github.com/tellapart/aurproxy/blob/master/examples/hello_world.aur https://github.com/tellapart/aurproxy/blob/master/examples/hello_world.aur for an example.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/tellapart/aurproxy/pull/35#issuecomment-218822298

We have not seen problems with libcurl4-nss-dev. One possibility is that a dependency on it has emerged in the latest version of Aurora (we are currently running one version behind).

(Also, glad to hear that you got it working, let me know if you have any further trouble!)

Hi Thanos,
Thank you for your concern ;) I use Aurora 0.12.0 with Mesos 0.25.0. Maybe the issue with libcurl comes from Mesos itself.
I took some time to dive deeper into Aurproxy before getting back to you. First, I wanted to report an issue with the example aurproxy job from the tutorial :
Because Aurproxy’s service is defined with a « host » constraint, the job scheduling results in status :  PENDING • Constraint not satisfied: host.
Indeed, in the .aur file :
base_aurproxy_service_template = Service(
container= Container(
docker = Docker(
image='{{docker_url}}/{{docker_library}}/{{docker_package}}:{{docker_image_version}}')
),
constraints={
'host': 'limit:1'
}
)
Unfortunately, the Mesos slave process started within Vagrant doesn’t have any attributes, let alone a host attribute (see http://<IP_ADDRESS>:8081/slaves). I didn’t make any pull request because I am unsure whereas you would rather remove the host constraint from the configuration file or ask users to update the slave's start-up flags in the tutorial.
This leads me to a bunch of questions.
For context, my company develops an open-source application management software for cloud infrastructures (http://alien4cloud.github.io). We use TOSCA to create cloud application blueprints (we call them topologies) and are able to deploy them onto multiple cloud providers using cloudify.
I would like to use Aurproxy to allow multiple jobs to discover each other (say, for exemple, an apache-powered web-app interacting with a database), in order to be able to define TOSCA-based cloud topologies and deploy them onto an Aurora/Mesos cluster, as seamlessly for the user as if it was deployed onto a IaaS.
First, about scaling :  do you use only one instance of Aurproxy or rather have one Aurproxy instance for each job you want to proxy to ? (From my understanding, only one instance should be set up per slave (I believe this is what your host constraint does) so this could quickly ramp up the number of slaves required in the cluster).
Second, about host routing : I wanted to be able to serve different jobs through a single port via Aurproxy. I’ve looked up the nginx.conf.template and noticed that I can set server names and used that in that end (see the .aur file attached). It worked perfectly when setting the Host HTTP header, but is against your recommendation for Proxy server objects : «  ports: Required list of one or more ports to listen on. Shouldn't collide with other ports in use »
Is this approach best practice in your opinion ?
Thank you for this great piece of software and sorry for the uber-long mail !
Best regards,
Adrian

Le 12 mai 2016 à 20:59, Thanos Baskous<EMAIL_ADDRESS>a écrit :
(Also, glad to hear that you got it working, let me know if you have any further trouble!)
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/tellapart/aurproxy/pull/35#issuecomment-218853386

Hi Adrian,
Sorry for the delay in my response. Thanks for noting the issue - I will test it out when I get a chance and update if necessary.
To answer your questions:
"First, about scaling : do you use only one instance of Aurproxy or rather have one Aurproxy instance for each job you want to proxy to ? (From my understanding, only one instance should be set up per slave (I believe this is what your host constraint does) so this could quickly ramp up the number of slaves required in the cluster)."
We have aurproxy deployed several ways:

For large HTTP services (hundreds of thousands of QPS), we run a cluster of aurproxy nodes per service.
For small HTTP services, we run a small cluster of "common" aurproxy nodes and use HTTP Host Header to route.
For internal proxying of L4 (TCP) services, we run a small cluster of aurproxy nodes with hard-coded, globally distributed ports. Port 9000 -> Service A, Port 9001 -> Service B, etc.

We use Aurora to schedule aurproxy onto a dedicated group of mesos workers. The workers are EC2 instances, and the aurproxy Aurora tasks are sized such that they and only they fit on the EC2 instance. This isn't very elegant, but it has worked for us. If you can find a way to guarantee that your "fixed" ports won't collide, you can run multiple aurproxy instances per host, but it isn't something we've done for any production services.
"Second, about host routing : I wanted to be able to serve different jobs through a single port via Aurproxy. I’ve looked up the nginx.conf.template and noticed that I can set server names and used that in that end (see the .aur file attached). It worked perfectly when setting the Host HTTP header, but is against your recommendation for Proxy server objects : « ports: Required list of one or more ports to listen on. Shouldn't collide with other ports in use »
Is this approach best practice in your opinion ?"
As mentioned in (2) above, we run a small aurproxy cluster that does host-level routing for many small jobs.
I hope those answers help! Let me know if I can expand on either.
-Thanos

Hi Thanos,
Many thanks for your thorough response. I have found your approach quite efficient and I think we can pick one for each of our topologies depending on how much we scale things.
However as supporting Docker is a strategic objective for my company, focus has shifted to integrating Marathon to our product, so Aurora is on hold at the moment…
I will use your answer to write guidelines for our next iteration. We will provide feedback on our Aurproxy usage once it’s in production :)
Thanks again
Best regards,
Adrian

Le 25 mai 2016 à 01:57, Thanos Baskous<EMAIL_ADDRESS>a écrit :
Hi Adrian,
Sorry for the delay in my response. Thanks for noting the issue - I will test it out when I get a chance and update if necessary.
To answer your questions:
"First, about scaling : do you use only one instance of Aurproxy or rather have one Aurproxy instance for each job you want to proxy to ? (From my understanding, only one instance should be set up per slave (I believe this is what your host constraint does) so this could quickly ramp up the number of slaves required in the cluster)."
We have aurproxy deployed several ways:
For large HTTP services (hundreds of thousands of QPS), we run a cluster of aurproxy nodes per service.
For small HTTP services, we run a small cluster of "common" aurproxy nodes and use HTTP Host Header to route.
For internal proxying of L4 (TCP) services, we run a small cluster of aurproxy nodes with hard-coded, globally distributed ports. Port 9000 -> Service A, Port 9001 -> Service B, etc.
We use Aurora to schedule aurproxy onto a dedicated group of mesos workers. The workers are EC2 instances, and the aurproxy Aurora tasks are sized such that they and only they fit on the EC2 instance. This isn't very elegant, but it has worked for us. If you can find a way to guarantee that your "fixed" ports won't collide, you can run multiple aurproxy instances per host, but it isn't something we've done for any production services.
"Second, about host routing : I wanted to be able to serve different jobs through a single port via Aurproxy. I’ve looked up the nginx.conf.template and noticed that I can set server names and used that in that end (see the .aur file attached). It worked perfectly when setting the Host HTTP header, but is against your recommendation for Proxy server objects : « ports: Required list of one or more ports to listen on. Shouldn't collide with other ports in use »
Is this approach best practice in your opinion ?"
As mentioned in (2) above, we run a small aurproxy cluster that does host-level routing for many small jobs.
I hope those answers help! Let me know if I can expand on either.
-Thanos
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/tellapart/aurproxy/pull/35#issuecomment-221436772


python310Packages.einops: 0.6.1 -> 0.7.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.einops is: Flexible and powerful tensor operations for readable and reliable code
meta.homepage for python310Packages.einops is: https://github.com/arogozhnikov/einops
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.7.0 with grep in /nix/store/igafr0fzrqv2q8awk0gs6kn3cw4kcwy2-python3.10-einops-0.7.0
found 0.7.0 in filename of file in /nix/store/igafr0fzrqv2q8awk0gs6kn3cw4kcwy2-python3.10-einops-0.7.0

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

25 total rebuild path(s)

25 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.arviz
python310Packages.augmax
python310Packages.bambi
python310Packages.dalle-mini
python310Packages.einops
python310Packages.elegy
python310Packages.encodec
python310Packages.k-diffusion
python310Packages.pymc
python310Packages.rotary-embedding-torch
python310Packages.treex
python310Packages.xarray-einstats
python311Packages.arviz
python311Packages.augmax
python311Packages.bambi
python311Packages.dalle-mini
python311Packages.einops
python311Packages.elegy
python311Packages.encodec
python311Packages.k-diffusion
python311Packages.pymc
python311Packages.rotary-embedding-torch
python311Packages.treex
python311Packages.xarray-einstats
tts

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/igafr0fzrqv2q8awk0gs6kn3cw4kcwy2-python3.10-einops-0.7.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 python310Packages.einops https://github.com/r-ryantm/nixpkgs/archive/d41a5acab479352f6d85a099b72b64b404297f02.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d41a5acab479352f6d85a099b72b64b404297f02#python310Packages.einops

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/igafr0fzrqv2q8awk0gs6kn3cw4kcwy2-python3.10-einops-0.7.0
ls -la /nix/store/igafr0fzrqv2q8awk0gs6kn3cw4kcwy2-python3.10-einops-0.7.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

4 packages marked as broken and skipped:

python310Packages.elegy
python310Packages.treex
python311Packages.elegy
python311Packages.treex

3 packages failed to build:

python310Packages.dalle-mini
python311Packages.dalle-mini
python311Packages.k-diffusion

18 packages built:

python310Packages.arviz
python310Packages.augmax
python310Packages.bambi
python310Packages.einops
python310Packages.encodec
python310Packages.k-diffusion
python310Packages.pymc
python310Packages.rotary-embedding-torch
python310Packages.xarray-einstats
python311Packages.arviz
python311Packages.augmax
python311Packages.bambi
python311Packages.einops
python311Packages.encodec
python311Packages.pymc
python311Packages.rotary-embedding-torch
python311Packages.xarray-einstats
tts

Maintainer pings
cc @yl3dy for testing.

Result of nixpkgs-review pr 265768 run on x86_64-linux 1

8 packages marked as broken and skipped:

python310Packages.elegy
python310Packages.elegy.dist
python310Packages.treex
python310Packages.treex.dist
python311Packages.elegy
python311Packages.elegy.dist
python311Packages.treex
python311Packages.treex.dist

4 packages failed to build:

python310Packages.dalle-mini
python310Packages.dalle-mini.dist
python311Packages.dalle-mini
python311Packages.dalle-mini.dist

38 packages built:

python310Packages.arviz
python310Packages.arviz.dist
python310Packages.augmax
python310Packages.augmax.dist
python310Packages.bambi
python310Packages.bambi.dist
python310Packages.einops
python310Packages.einops.dist
python310Packages.encodec
python310Packages.encodec.dist
python310Packages.k-diffusion
python310Packages.k-diffusion.dist
python310Packages.pymc
python310Packages.pymc.dist
python310Packages.rotary-embedding-torch
python310Packages.rotary-embedding-torch.dist
python310Packages.xarray-einstats
python310Packages.xarray-einstats.dist
python311Packages.arviz
python311Packages.arviz.dist
python311Packages.augmax
python311Packages.augmax.dist
python311Packages.bambi
python311Packages.bambi.dist
python311Packages.einops
python311Packages.einops.dist
python311Packages.encodec
python311Packages.encodec.dist
python311Packages.k-diffusion
python311Packages.k-diffusion.dist
python311Packages.pymc
python311Packages.pymc.dist
python311Packages.rotary-embedding-torch
python311Packages.rotary-embedding-torch.dist
python311Packages.xarray-einstats
python311Packages.xarray-einstats.dist
tts
tts.dist


Sortable on Range products_view broken

JQuery UI was removed here: https://github.com/django-oscar/django-oscar/commit/1af6521ab5a6a1891e09f78f6cfd90a55f5e8020
If you go to products_view (/dashboard/ranges/3/products/) you get a javascript error: $(...).sortable is not a function

@john-parton thanks for the report. I'll prepare a fix.

@mvantellingen please review.

Hi @sasha0,
This results in the following exception for me in a new sandbox env:
[2015-11-22 11:14:20,355] Internal Server Error: /en-gb/dashboard/ranges/2/reorder/
Traceback (most recent call last):
  File "/usr/local/opt/pyenv/versions/oscar/lib/python2.7/site-packages/django/core/handlers/base.py", line 132, in get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/local/opt/pyenv/versions/oscar/lib/python2.7/site-packages/django/utils/decorators.py", line 145, in inner
    return func(*args, **kwargs)
  File "/usr/local/opt/pyenv/versions/oscar/lib/python2.7/site-packages/django/contrib/auth/decorators.py", line 22, in _wrapped_view
    return view_func(request, *args, **kwargs)
  File "/usr/local/opt/pyenv/versions/oscar/lib/python2.7/site-packages/django/views/generic/base.py", line 71, in view
    return self.dispatch(request, *args, **kwargs)
  File "/usr/local/opt/pyenv/versions/oscar/lib/python2.7/site-packages/django/views/generic/base.py", line 89, in dispatch
    return handler(request, *args, **kwargs)
  File "/Users/mvantellingen/projects/opensource/django-oscar/src/oscar/apps/dashboard/ranges/views.py", line 204, in post
    self._save_page_order(order)
  File "/Users/mvantellingen/projects/opensource/django-oscar/src/oscar/apps/dashboard/ranges/views.py", line 213, in _save_page_order
    entry = RangeProduct.objects.get(range=range, product__pk=item)
  File "/usr/local/opt/pyenv/versions/oscar/lib/python2.7/site-packages/django/db/models/manager.py", line 127, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/usr/local/opt/pyenv/versions/oscar/lib/python2.7/site-packages/django/db/models/query.py", line 334, in get
    self.model._meta.object_name
DoesNotExist: RangeProduct matching query does not exist.

Actually, there's more than that. Found another one - there're duplicated products for range with included categories:

Also, if you delete product from this range, it will still remain in the list, because it's still has given category and included to the range within it.

@sasha0 Thanks for the problem description. I think the most pragmatic solution for now is to only allow re-orderingen the products if they are explicitly added

@mvantellingen thanks for the suggestion - I used it. Also included fixes for product duplication and product removal from range, explained in my previous comment. Please take a look.


chore(security-issues): fixed springboot security vulnerabilities and added codeql

Why we create this PR?

Added CodeQL workflow to analyze the codebase as a requirement from the release guidelines
Removed veracode workflows from git actions (replaced by CodeQL).
Fixed spring security vulnerabilities by upgrading the springboot version from 3.2.2 to 3.2.4

What we want to achieve with this PR?
To meet the trg-8 requirements
What is new?
Added

Added CodeQL code analysis to the workflow to comply with the release guidelines

Deleted

Removed veracode workflows from git actions (replaced by CodeQL).

Security Fixes:

Fixed the following spring security vulnerabilities:

CVE-2024-22234
CVE-2024-22257
CVE-2024-22243
CVE-2024-22259

Linked PR(s):
#289

@RoKrish14  please check and re approve

If all dependencies are approved and the workflow changes propose by @RoKrish14 are solved then I approve

@matbmoser, I fixed now the issues requested by @RoKrish14. I hope there should be no issue left behind.


Param / ini file don't seem to be taking.

I can't seem to get any Params to take effect. I also tried the .ini file but my Server Admin Password never changes or the server name or anything else. Could you please help. I'm not sure what i am missing.

edit config and add server start param -adminpassword=""

I'm going to try again today but that's where we put it and it doesn't seem to want to take. I can always give you remote access to our server so that you can see what I'm talking about if you're interested.

try the new version it is working on my end


Add support to track web sockets

It might not be possible to do automatic tracking but it would be nice to provide an interface for manual logging.
Here is what should be supported

websocket connect. With possibly a request object
Message sent
Message received
Websocket disconnect.

All of this can be tracked within one single line item with a list of back and forth messages. Alternatively each entry could be its own line item.

This feature is in the backlog. It should support both URLSessionWebSocketTask and other technologies, e.g. Network framework.

All of this can be tracked within one single line item with a list of back and forth messages. Alternatively each entry could be its own line item.

It'll be great to give users control over how it's tracked and/or displayed.

hey @kean, do you have an idea on the implementation for this? We're looking to add this support for easier debugging of graphql subscriptions. We'd be happy to help by contributing!

Hey, @alexjameslittle. I've been ramping up the development, and there is a large backlog. I haven't prioritized web sockets and GraphQL yet because I don't use these in my workflow. I'll be happy to implement it, and it would be helpful if you could share some expectations of how you would need it to work, maybe including some mockups.

Great, at the very minimum it would be great to have a way to track all websockets without specific graphql support, click into their streams and see the messages sent & received, potentially connect/deconnected events for those streams too. We've not put too much thought into how it could look within the pulse UI.


labhub_test: Ignore test temporarily

Ignoring test for creating private issue
until a fix for the ACL failure has been
found.
Closes https://github.com/coala/corobo/issues/526
Reviewers Checklist

[ ] Appropriate logging is done.
[ ] Appropriate error responses.
[ ] Handle every possible exception.
[ ] Make sure there is a docstring in the command functions. Hint: Lookout for
botcmd and re_botcmd decorators.
[ ] See that 100% coverage is there.
[ ] See to it that mocking is not done where it is not necessary.

@meetmangukiya ping

I changed the line in the tests to ACCESS_CONTROLS: {'LabHub:create_issue_cmd': {'allowprivate': False}} , didn't worked.
commit 55509e9 is working but partly. create_issue command from user leads to bot giving results twice instead of once. https://prnt.sc/jm7l8y Here bot made 2 API calls to make issue instead of one but shows a single result for nevermind(nm). This happened in coala/cobot-test
But when I tried to create issue from private with the ACL allowprivate:false the filter worked fine for the first result but failed for the second time. https://prnt.sc/jm7nhp

Since we are inheriting the LabHub plugin in GitStats plugin, the bot is creating duplicates of the inherited bot commands hence the above mentioned behaviour.

"review/gitmate/commit" not responding. turning it off

@gitmate-bot ff


[WIP] GH1281: Add a "Prepend" extension for the ProcessArgumentBuilder

See issue #1281.
This PR adds a corresponding Prepend extension method for all of the existing Append extension methods for the ProcessArgumentBuilder class.
This allows the script to insert new arguments at the start of a tool command for instances where the final argument of a tool has special significance (such as SignTool where the final argument is the file(s) to sign) and adding new arguments to the end can cause an issue.

Hi @antwilliams, 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;

Currently a WIP, the code is implemented but test coverage is required.

Test cases added (based on the existing Append tests)

@antwilliams looks good to me!  I will merge after the CI finish.


Rog ally up tdp to 45?

Hello can we please have the ally upped to 45 watt or tell us what file to modify?. The ally has great cooling and I can run at 50 watts in windows ?

if you unlock you can.
how to:
https://www.youtube.com/watch?v=pTvWJey9ku8

I have smokeless already the problem is power control does not work on nobara steam :) new issue

Yes I can change tpd with ryzenadj up to 50 watts, I think I with nobara it’s a permissions issue MarcOn 11 Nov 2023, at 19:08, Marcilio Junior @.***> wrote:
This appears to be a RyzenAdj issue. Can you make the changes directly with RyzenAdj?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Identifier not found "TRawByteStringStream"

Hi!
I'm getting the following error when compiling "Identifier not found "TRawByteStringStream"

Hi! Which version of FreePascal are you using? This type exists in standard rtl in FreePascal. Weird

This is the version I'm using:
Free Pascal Compiler version 3.0.4+dfsg-18ubuntu2 [2018/08/29] for x86_64

Is there any valid reason why you are using an outdated version of FPC? Current stable FPC is 3.2.2.
Of course, I can try to add some hack to the code for backward compatibility of older versions of FPC. But You'll have to wait until I get home few days

Well, it's the version that is coming in my linux ubuntu (18.04). Knowing that it is just a problem of fpc version I will try to install a more updated version, no need backward compatibily. Thanks for your support!

Wow. I can see there is no office package for debian/ubuntu for the version you told me:
https://www.freepascal.org/down/i386/linux-hungary.html#linuxdeb

I'll think about a hack for backward compatibility.
As for getting a more recent version of FPC, and generally a convenient installation, I recommend using fpcupdeluxe to install Lazarus

So, I added defines for the outdated FPC to work. Try it now https://github.com/Al-Muhandis/fp-telegram/tree/deprecated-fpc. Message here if it helped, I will apply the changes to the main branch as well

Great!! It works like a charm! Thanks!!!

So, then I will add to the main branch

#8


echo -e doesn't work on Mac when using bash

Describe the bug
The command echo -e "Some text" writes -e Some text to the screen.
To Reproduce

Run dotman

Expected behavior
The -e is not included in the output
Desktop (please complete the following information):

OS: Mac OSX 10.15.5
Bash Version: GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin19)

Additional context
Consider using printf instead

Hi @smenzer :wave: Thanks for reporting this.
Would fix this in the next release

@all-contributors please add @smenzer  for bug

Fixed in 0.2.0


Remove index on invitations_count column as it's probably not needed by everyone

I don't think this index is needed unless someone wants to build some leaderboard kind of functionality. For most of the cases it will lie unused in the database. More details of the conversation can be found here (on the original PR where it was added):
https://github.com/scambra/devise_invitable/pull/425/files#r487461684

Not sure about the test failures. They look totally unrelated to my changes.


fix: erc20 test setup

There's no need to set the name and symbol of the token within the arguments.

Nice catch. Should not get merged after #89 though.

#89 resolves this


Solve python2 and python3 requirements

Solve #130

OS: Ubuntu
OS Version: 16.04
Python Version: 2.7.4

thanks for your PR.


Minor edit to installation.md

Minor edit: the docker container name does not have a hyphen. Although, you may prefer to change this such that it pulls the singularity image directly.
I ran into this while testing a pipeline with singularity.

This files will be deleted from the pipeline anyway when the automated syncing PR is merged (https://github.com/nf-core/rnaseq/pull/194/files).
They have been relocated to the website to make them more centralised:
https://github.com/nf-core/nf-co.re/tree/master/markdown/usage
Would you mind double-checking the docs there please @jburos , and creating a PR here if you spot anything:
https://github.com/nf-core/nf-co.re
Thank you!

Looks like #194 needs merging asap - lots of merge conflicts already 😱


Invalid get index 'hide_in_list' (on base: 'Dynamic Font')

When refreshing the asset manager window error appear in the Godot output, one of the errors is
res://addons/asset-manager/LibraryCategory.gd:100 - Invalid get index 'hide_in_list' (on base: 'Dynamic Font')
This error is triggered when my folder structure was incorrect (missing game_object/deco/levels folders in asset-manager/assert_manager_resources). A more meaningful error could help troubleshooting (I saw there there is a check for the presence of these folders, but the corresponging message in case they are missing was never triggered)

Thanks for letting me know!
I will add a button to add the basic folder structure to the selected folder!
In the meantime I added the correct folder structure to the repository


Patch

R<!--

Please provide a clear and concise description of what the bug is.
Include screenshots if needed.
Please test using the latest React Native release to make sure your issue has not already been fixed: http://facebook.github.io/react-native/docs/upgrading.html
-->
React Native version:

Steps To Reproduce

Describe what you expected to happen:
Snack, code example, screenshot, or link to a repository:

Hi @elrussord1,
Your issue seems empty. Could you please either elaborate on what problem you're facing or close it?
Thank you 😊


D11.2 Barcode Reader

BarcodeReader Demo ,Compile and run without problems, But no results are returned from scanning the barcode
Delphi Version: D11.2
Platform(s) :Android

Can you attach a picture of the barcode here, please?

Can you attach a picture of the barcode here, please?

The barcode is the BarcodeScanExample.png in the Screenshots directory of the demo, other barcodes are also the same scan code did not return

What model device do you have and what version of Android is on it?

My device is Honor Magic 4 with Android version 12

Same for other Android devices, Android 10+

..and this is using the demo project? I'm having no trouble here on Pixel 6 Pro (Android 13), Pixel 3a (Android 12), Nexus 5X (Android 8.1). Can you attach your .apk here?

Download link to the apk generated by compiling the demo with d11.2
https://pan.baidu.com/s/1YjMHf98BYvyVf5XkZn32TA?pwd=1234

I'm unable to download the apk from that link. Please zip your .apk and attach it (by drag/drop) here.

Okay, here it is.
BarcodeReaderD11.zip

The .apk works OK on my Nexus 5X (Android 8.1).
It does not install on my Pixel 6 Pro (Android 13) because the manifest does not have the android:exported attribute for the activity, even though the targetSdkVersion is set to 32. This implies you are using Delphi 11.2, so you will need to delete the AndroidManifest.template.xml file and allow the IDE to recreate it by doing a Clean, then Build. I doubt this is the cause of your issue, though - the out of date template file would prevent the app from being installed onto devices with Android 12 or higher, so it would not even be able to run.

BarcodeReaderD11.zip
I added in AndroidManifest.template.xml
android:exported and recompiled it, but the demo does not recognize the barcode on my phone

I think you may need to follow the instructions, here: https://qr.ae/prWNDG, then retry the BarcodeReader demo.

Thank you very much, I'll try again


Fix tooltips in chat not showing on ctrl hover.

Removes some dead code that prevented tooltips on badges etc. from showing while holding the control/alt modifiers.
Fixes #1292

Everything seems to work as expected using your branch. However, before I merge this PR, I would like to try to understand what it does/did.

I've looked back to the original commit to understand what this section is supposed to do. It seems like it was something which allowed you to alt+click a split to drag it around. This functionality isn't implemented in any other part anymore.
The same code segment existed in the mousePressEvent and mouseReleaseEvent where it was removed some time ago.
As the split dragging that is currently implemented works in a different way so this seems like it was just some forgotten part left in.
Maybe @fourtf knows what this part was supposed to do as he originally commited this here.

ctrl+alt movement is something I tested as well, and that works on master and on your branch (hold ctrl+alt, press the "move" icon, move split around, release mouse)

The alt+ctrl overlay is now implemented as a widget that's above the split so this is (as stated) not used anymore.


About page doesn't look consistent with rest of UI

Information is there but it just looks weird (maybe font color or something). Low priority but would be great if it could work in before release.

Done #85


fix(table): schematic generates code that throws an exception

fix exception when instantiating MatPaginator before dataSource defined
fix ExpressionChangedAfterItHasBeenCheckedError caused by above

first initialize the template w/o dataSource & w/ a sort and paginator
then get the references to the template's MatSort and MatPaginator
then set the data source's sort and paginator
then set the table's data source from the component

Fixes #15788
Not the same issue, but this is also related to https://github.com/angular/material2/issues/15791.

@andrewseguin PTAL

Any fixes? The problem is still relevant.

@Splaktar can you rebase?
@andrewseguin can you review?

Rebased ✅

It seems it would be easier to just add [length]="dataSource?.data.length" as the fix here and we can have another PR that changes the data source structure if we want to improve how it flows.
I'm not a fan of providing the data source as an input to the table in the code rather than the template, and the data source code in general is confusing because it tries to be a mix of client-side and server-side data.
Also, this is particular is confusing to read and doesn't seem to be the best way to show users how to use the datasource:
return merge(...dataMutations).pipe(map(() => {
      return this.getPagedData(this.getSortedData([...this.data]));
    }));

@andrewseguin I tried to explain it a bit in the commit, but it may be too brief. I had discussed this a good deal with Paul while working on it.
Just making the [length]="dataSource?.data.length" change causes an ExpressionChangedAfterItHasBeenCheckedError to be thrown. That can be worked around by using ChangeDetectorRef.detectChanges() but that didn't seem like a good practice to be using in a schematic.

Yeah, people should extremely rarely call detectChanges is real code

@andrewseguin I agree that the snippet of RxJS that you posted is not an easily readable piece of code as it uses a lot of concepts that may not be well known by every developer. However, that wasn't introduced or changed in this PR. I had considered doing so, but I didn't want to widen the scope of the PR, so I tried my best to keep that intact.

@andrewseguin on the topic of client vs server side paging, I agree that this is confusing to developers who just want to try out our schematic and may only be considering one of those options. I commented in https://github.com/angular/components/issues/15791#issuecomment-482763138 that we should probably have an option to generate the table with client or server side paging. I also mentioned that we should probably have an option to generate a table w/o paging or sorting. Should I open issues to track that?

@crisbeto is also working on a related change to "Rename MatTableDataSource to ClientArrayTableDataSource" (https://github.com/angular/components/issues/14378) since as Jeremy noted in the issue, it causes a lot of confusion. The PR is https://github.com/angular/components/pull/15432.

What about just removing [length]="dataSource.data.length" and changing the lifecycle hook from ngAfterViewInit to ngOnInit?

@andrewseguin if I make only those changes, then the original error from #15788 remains: TypeError: Cannot read property 'page' of undefined due to
https://github.com/angular/components/blob/dd7f4eab0d3b931e04740f25cdd1940575bc5821/src/material/schematics/ng-generate/table/files/__path__/__name@dasherize<EMAIL_ADDRESS>

Hm, well hopefully we could figure out some better syntax to the whole schematics. For now this looks fine if the other solutions aren't panning out

How to find out which version contains this fix?

@felikf it's in the commit linked above: https://github.com/angular/components/commit/301371aadc3d2c33c3e37cde9dfdf97ae0602e74.
8.0.0-rc.2 is the first version with this fix. 8.0.0 is the first stable version with the fix.


Is Qwen being used in OneChart or not ?

I found the Qwen module in side the code structure. But nowhere in the paper has mentioned it.

OneChart does not use Qwen. Qwen appears in the code because OneChart borrowed code from Vary, which does use Qwen. These Qwen references in OneChart's code are not active or functional.


[Bug?] Gradient Synchronization for DDP

According to no_sync function description in  https://github.com/pytorch/pytorch/blob/main/torch/nn/parallel/distributed.py#L1424
.. warning::
    The forward pass should be included inside the context manager, or
    else gradients will still be synchronized.

The current code does separate forward and backward pass in no_sync, therefore will still trigger gradient synchronization

Code reference of forward no sync with ddp: https://github.com/pytorch/PiPPy/blob/main/pippy/_PipelineStage.py#L425
And backward no sync with ddp: https://github.com/pytorch/PiPPy/blob/main/pippy/_PipelineStage.py#L444
Since this is separate call, the no_sync will not take effect and still trigger gradient synchronization

Hi thanks for reporting this.
You are right that the forward and backward passes are called in separate context managers.
It seems to me that both context managers would control the same flag: self.require_backward_grad_sync, where self here refers to the DDP module. (I found this from the code of the no_sync manager.)
Thus, I wonder if calling the forward and backward passes in separate managers might be okay? Please correct me if I missed something. Thanks!

Thanks for your quick response! I don't have exact root cause, but see other users also reported calling fwd and bwd separately in no_sync context still triggers grad_sync. https://discuss.pytorch.org/t/whats-no-sync-exactly-do-in-ddp/170259. I am not sure if this is still an issue today so would like to confirm here.


[Bug 1.19.2]: Spawners being placed not respecting config

Minecraft Version
1.19.2
Fabric Version
0.14.13
Mod Version
4.2.4
Details

This is set to false however when placing the monster farm all the spawners are still generating.

Please try this in 1.19.3.
1.19.2 is no longer supported.

Aqm2 is still on 1.19.2, as well as most modpacks seeing how not many major
mods are on 1.19.3
On Sat, Feb 11, 2023, 4:10 PM Brian Wuest @.***> wrote:

Closed #78 https://github.com/Brian-Wuest/MC-Prefab-Fabric/issues/78 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/Brian-Wuest/MC-Prefab-Fabric/issues/78#event-8497508679,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIL5CKEZFRS664ZPH2CZNU3WXAS7TANCNFSM6AAAAAAUY5LJ7I
.
You are receiving this because you authored the thread.Message ID:
@.***>


Adding my solutions in solutions_from_participants dir

 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.


ez80: fix several bugs in emac driver

Summary
IRQs cannot be individually disabled on the eZ80, so using
up_disable_irq() had no effect. This left the IRQ handler being
constantly triggered without the lower half handler running.
The macro for EMAC stats was incompatible with Clang. The simplified
form gives identical results under ZDS-II.
The MII clock speed must be set before trying to read MII registers.
It's now done before resetting the PHY using the Mode Control Register.
A few typos, unused variables, and other miscellaneous issues were also
fixed.
Impact
These changes permit building the eZ80 EMAC driver and enable basic Ethernet functionality.
Testing
Successful ICMP echo receive and response:
ez80emac_receive: rxnext=0xffdd00 {ffdd80, 102, 8020} rrp=0xffdd00 rwp=0xffdd80 blkslft=003e
ez80emac_receive: rxnext=0xffdd80 {653734, 14128, 3166} rrp=0xffdd80 rwp=0xffdd80 blkslft=0040
ez80emac_receive: IPv4 frame
icmp_input: Outgoing ICMP packet length: 84 (84)
ez80emac_transmit: txnext=0xffc6c0 {000000, 0, 0000} trp=06c0
ez80emac_transmit: txdesc=0xffc6c0 {ffc740, 98, 8000}
ez80emac_transmit: txnext=0xffc740 {000000, 0, 0000} trp=0740
ez80emac_txinterrupt_work: txhead=0xffc6c0 {ffc740, 98, 0000} trp=0740 istat=01
ez80emac_txinterrupt_work: txhead=0xffc740 {000000, 0, 0000} trp=0740
ez80emac_txinterrupt_work: No pending Tx.. Stopping XMIT function.

There are further issues with networking on the eZ80 to resolve:

Large frames put the EMAC driver into an endless loop failing to process the invalid message
A DHCP lease cannot be obtained, but I don't yet know why - it may not be related to the EMAC driver
Auto-negotiation is successful on the DP83848C but not correctly detected as begun or complete

I can continue to update this PR if it's preferable to merge any further changes into one.

@codebje is this patch ready for merge?

I think so - I've traced the DHCP issue far enough to know that it reaches udp_input with the right data, so whatever's causing the DCHP offer to be lost is probably not in the EMAC.

Ok, let's merge it.


Link to grading view from submission

Low priority

Some other nice links:
submission -> search (to view scores)
final submission -> search (to view scores)


ModuleNotFoundError

I have install the all environment and download and install the DOTA_devkit，but when I run python main.py --data_dir dataPath --epochs 80 --batch_size 16 --dataset dota --phase train 。This error happend：“ModuleNotFoundError： No module named 'datasets.DOTA_devikit' ”.
could you ask me how to solve this question？

You may put DOTA_devkit under the folder datasets

Thanks for your answer, I have forget change the name of DOTA_devkit folder.
But there is another mistake that " No module named 'dota_kit'". May I not install DOTA_devkit correctly?

Thanks for your answer, I have forget change the name of DOTA_devkit folder.
But there is another mistake that " No module named 'dota_kit'". May I not install DOTA_devkit correctly?

I have the same problem. Have you solved it

You can comment on the lines in ResultMerge_multi_process.py, I don't know why the authors for DOTA_devkit do not provide the dota_kit folder.
`# try:
from . import dota_utils as util
except:
from .dota_kit import dota_utils as util`

You can comment on the lines in ResultMerge_multi_process.py, I don't know why the authors for DOTA_devkit do not provide the dota_kit folder.
# try:
from . import dota_utils as util
# except:
# from .dota_kit import dota_utils as util

Thank you very much.


docs(Documentation): Add paraview version compatibility table

Ping @jourdain I didn't think we wanted a new version, so I went with a doc commit.


perf: neighbours

I always believe performance changes should be backed up by metrics: have you timings to establish the performance impact of this change?


Update autofill to 8.1.1

Task/Issue URL: https://app.asana.com/0/1205223917466045/1205223917466045
Autofill Release: https://github.com/duckduckgo/duckduckgo-autofill/releases/tag/8.1.1
Description
Updates Autofill to version 8.1.1.
Autofill 8.1.1 release notes
What's Changed

Add shared creds by @GioSensation in https://github.com/duckduckgo/duckduckgo-autofill/pull/361

Full Changelog: https://github.com/duckduckgo/duckduckgo-autofill/compare/8.1.0...8.1.1
Steps to test
This release has been tested during autofill development. For smoke test steps see this task.

Will land 8.1.1 as part of https://github.com/duckduckgo/Android/pull/3397


Improve performance of the BitDropdown component

Improve performance of the BitDropdown component:
There are some complex codes that need to be simplified.
There are some redundant codes that need to be removed.
There are some complicated execution flows that need to be refactored.

this issue is not planned for now.


[SPARK-26118][BACKPORT-2.3][WEB UI] Introducing spark.ui.requestHeaderSize for setting HTTP requestHeaderSize

What changes were proposed in this pull request?
Introducing spark.ui.requestHeaderSize for configuring Jetty's HTTP requestHeaderSize.
This way long authorization field does not lead to HTTP 413.
How was this patch tested?
Manually with curl (which version must be at least 7.55).
With the original default value (8k limit):
$ ./sbin/start-history-server.sh
starting org.apache.spark.deploy.history.HistoryServer, logging to /Users/attilapiros/github/spark/logs/spark-attilapiros-org.apache.spark.deploy.history.HistoryServer-1-apiros-MBP.lan.out

$ echo -n "X-Custom-Header: " > cookie
$ printf 'A%.0s' {1..9500} >> cookie

$ curl  -H cookie http://458apiros-MBP.lan:18080/

Bad Message 431

reason: Request Header Fields Too Large

$ tail -1 /Users/attilapiros/github/spark/logs/spark-attilapiros-org.apache.spark.deploy.history.HistoryServer-1-apiros-MBP.lan.out
18/11/19 21:24:28 WARN HttpParser: Header is too large 8193>8192

After:
$ echo spark.ui.requestHeaderSize=10000 > history.properties

$ ./sbin/start-history-server.sh --properties-file history.properties
starting org.apache.spark.deploy.history.HistoryServer, logging to /Users/attilapiros/github/spark/logs/spark-attilapiros-org.apache.spark.deploy.history.HistoryServer-1-apiros-MBP.lan.out

$ curl  -H cookie http://458apiros-MBP.lan:18080/
<!DOCTYPE html>
      ...
         <link rel="shortcut icon" href="/static/spark-logo-77x50px-hd.png"></link>
        <title>History Server</title>

...

(cherry picked from commit ab61ddb34d58ab5701191c8fd3a24a62f6ebf37b)

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Test build #99153 has finished for PR 23114 at commit 01bd63b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99153/
Test PASSed.

@attilapiros .  I followed the verification steps in the PR description, but I cannot reproduce this issue at Spark 2.4.0 and 2.3.2 History Server. I'm using Mac like the following. Is there something I missed?
$ ls -alh cookie
-rw-r--r--  1 dongjoon  staff   9.3K Nov 21 22:19 cookie

$ curl --version
curl 7.62.0 (x86_64-apple-darwin18.0.0) libcurl/7.62.0 SecureTransport zlib/1.2.11
Release-Date: 2018-10-31
Protocols: dict file ftp ftps gopher http https imap imaps ldap ldaps pop3 pop3s rtsp smb smbs smtp smtps telnet tftp
Features: AsynchDNS IPv6 Largefile NTLM NTLM_WB SSL libz UnixSockets

Apache Spark 2.4.0
$ curl --cookie cookie http://localhost:18080 | grep version
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3165  100  3165    0     0  30728      0 --:--:-- --:--:-- --:--:-- 31029
                  2.4.0

$ curl  -H cookie http://localhost:18080 | grep version
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3165  100  3165    0     0   343k      0 --:--:-- --:--:-- --:--:--  343k
                  2.4.0

Apache Spark 2.3.2
$ curl  -H cookie http://localhost:18080 | grep version
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3082  100  3082    0     0   752k      0 --:--:-- --:--:-- --:--:-- 1003k
                  2.3.2

$ curl --cookie cookie http://localhost:18080 | grep version
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3082  100  3082    0     0  1003k      0 --:--:-- --:--:-- --:--:-- 1003k
                  2.3.2

Oh, I got it. The PR description is wrong in this PR.
The current one.
$ curl  -H cookie http://458apiros-MBP.lan:18080/

The correct one needs '@'.
$ curl  -H @cookie http://458apiros-MBP.lan:18080/

Apache Spark merge script removes '@' from the commit message in order to avoid pinging people.

Merged to branch-2.3.

This is merged now. Please close this PR.
Thanks!


feat(menu): expand support for custom branding

SUMMARY

Expands support for custom branding in superset via config options:

LOGO_RIGHT_TEXT: text that should appear next to the logo
LOGO_TOOLTIP: tooltip that should appear when hovering over logo

Additionally, a watermark Powered by Apache Superset is added if the logo (APP_ICON) is changed from the default superset logo superset-logo-horiz
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

After:

TESTING INSTRUCTIONS

Manually tested by setting configs

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up LOGO_RIGHT_TEXT=true LOGO_TOOLTIP=true


Organize actions by category

Feature Request
Was thinking that it would be much easier for the user if actions were only available to that category selected. For example, if I have the category drop down selected as "order", then the actions just pertaining to that category would be available. Then, there could be a selection for the category drop down that says "all" if the user once to see all available actions.  I realized this may be a good idea, once importing the premade made categories.

This isn't possible given the structure of Articulate. Categories are really just there for organization. And when you start thinking multi-actions this doesn't make sense. For example greet may belong to Small Talk and Order Pizza may belong to Food, but we want to be able to support phrases like hello, can I get a pizza?


coplay is not wokring currently on Youku

the control bar does not show when click on the extension fyi

Fixed in v0.6.0.


Site goes down when rate limited by Github

@justinelsberry experienced today a case while navigating. All the sudden he saw a github rate limit message and the whole site went down. looks like we still have some lingering github accesses happening on the site post compilation causing this issue. He reloaded the site in incognito mode with the same results.

This is a release blocker as with the increasing number of users in the announcement we can see ourselves easily in the same situation. Are we perhaps calling github auth on each request instad of caching the user's credentials in the cookie?

Just hit refresh about 20 times on the homepage and then that error was displayed

I've detected a couple calls to GitHub without using the user's access token, thus unauthenticated and subject to rate limiting. Wil fix this tomorrow morning.


additional.py

verified operator

Your checklist for this pull request

[x] I have committed only those files that are required.
[x] I have tried commit messages concise and reflective of the code committed.
[x] I have used a branch name reflective of the issue.
[ ] If there is relavant issue for the PR, I have used closes #<issue number> to auto-close the issue once PR is merged.

Detailed description

Test plan (required)

Closing issues

"+" operator verified

fixed the error

fixed error


Use html-webpack-plugin

removes index.html
creates container in index.js

I'm not sure is this a good practice.
It saves an 'index.html' though ;)

Thanks. I would rather keep it simple and avoid plugins when possible as they may conflict with other plugins.

Yep :+1:


5.0 Router does not provide React Context

Version
5.0.0 (with React 16.8.5)
Test Case
TypeScript code:
import React from 'react';
import ReactDOM from 'react-dom';
import PropTypes from 'prop-types';

import { BrowserRouter as Router } from 'react-router-dom';

class App extends React.Component {

  static contextTypes: React.ValidationMap<any> = {
    router: PropTypes.shape({
      history: PropTypes.shape({
        push: PropTypes.func.isRequired,
        createHref: PropTypes.func.isRequired
      }).isRequired
    }).isRequired
  }

  constructor(...args) {
    super(...args);

    // this will log "[{},{}]"
    console.log(JSON.stringify(args));
    // this will log "{}"
    console.log(JSON.stringify(this.context));
  }

  public render() {
    return (

Router {this.context.router ? 'defined' : 'is not defined'}

    );
  }
}

// Wrap the `App` in the `Router` so that it provides it the `router` as context
ReactDOM.render(
  <Router>
    <App />
  </Router>,
  document.getElementById('root')
);

Steps to reproduce
Attempt to render the above page and observe that the router in not defined.
Expected Behavior
this.context.router to be defined.
Actual Behavior
this.context.router is not defined, which also causes warnings due to the contextTypes expecting it.
The exact same code works with react-router-dom 4.3.1.

This looks like it changed because the way that the React Context is supplied did change between 4.x and 5.0:
4.x:
https://github.com/ReactTraining/react-router/blob/3d233bf0b6dd5bf68d9bac9c94273ae25646b207/packages/react-router/modules/Router.js#L19-L21
5.0:
https://github.com/ReactTraining/react-router/blob/ea8eba843bf899daf8a51c2617d05c179b38369d/packages/react-router/modules/Router.js#L55-L63

Directly accessing the context was never part of the public API; the recommended way to access the context has always been to use withRouter.

Well then, that explains it! Thanks for the quick response @pshrmn!


chore(deps): update dependency compression-webpack-plugin to v6.0.3

This PR contains the following updates:

Package
Type
Update
Change

compression-webpack-plugin
devDependencies
patch
6.0.2 -> 6.0.3

Release Notes

webpack-contrib/compression-webpack-plugin
v6.0.3
Compare Source

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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

v6.1.2
GitHub release

Your semantic-release bot :package::rocket:


Updated validations.py python script.

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

What should i do now


[youtube] "Last message repeated XXX times" (or more)

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.05.18. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.05.18

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

I can not bring the log, since this does not happen with all streams! besides, these are living streams and I can not reproduce this problem! in addition, some streams are very time-consuming. At the end only this line is "Last message repeated XXX times". Nothing special except this.
Some live streams do not end correctly. I get the message "Last message repeated N times". "N" can be really big (N > 1000) and it really downloaded each time last message from Youtube. Such hanging streams heavily load the network when there are a lot of them.
For download streams from Youtube i use last version yutube-dl and use command string like this:
youtube-dl.exe --hls-use-mpegts https://www.youtube.com/watch?v=XXXXXXXXXXX
Why some streams not ended correctly and how fix it? I can set some limit for this?
I REALLY tried to find the answer to my question and did not find it! If you think that this problem is described and solved - then PLEASE post here a link to this solution. It is very difficult for me to write in English, which does not detract from the importance of my communication.
P.s. I have a suspicion that this happens to streams that have been deleted

Only i have this problem? No any ideas how fix it?

There is not enough context for any reasoning. Anyway Last message repeated comes from ffmpeg so you may ask there. Closing for now.

I think this is a problem with YouTube, not ffmpeg or youtube-dl. but you can somehow automatically stop recording after a certain number of repetitions ???
if you want to reproduce this problem - just record any random streams of users. It often happens that the broadcast is complete, but YouTube continues to play a static picture. I think that this is what leads to this behavior.


Error if output format is text/html with text/html input

The same call as in issue #76, however this time we ask to output the result using text/html. However, we get the following error

Whitelabel Error Page

This application has no explicit mapping for /error, so you are seeing this as a fallback.

Wed Feb 17 16:02:27 CET 2016

There was an unexpected error (type=Internal Server Error, status=500).

String index out of range: -14

What exactly is the problem here?
P.S. According to the w3 validator our document is valid (beside some deprecated elements, however, that should not make a difference I think).

this issue is related to e-Internationalization service so I moved it https://github.com/freme-project/e-Internationalization/issues/35


Disable publishSigned in private projects

Our examples projects shouldn't be publishing.  I think it's biting us now because we started signing our snapshots.
If this fixes it, I'll add the fix in rig and remove it from our private project plugin.

Appveyor picked up a bad merge of ServletIo that was subsequently fixed.

Okay, that wasn't it.  My understanding is that overwrite is supposed to be true when it's a snapshot.  I don't know why this doesn't work.


Change the link to "poster size graphic"

Currently the Microsoft fwlink points to a non-existent page. It should point to https://github.com/Microsoft/sql-server-samples/blob/master/samples/features/security/permissions-posters/Permissions_Poster_SQL_Server_vNext_and_SQLDB.pdf.

Hi @ojasp, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

#please-close


[Documentation] How to implement matter audio products？

Documentation issues
Hi, I want to develop and manufacture matter audio products. I have some questions below:

I want develop a matter wireless speaker that has the features of play, pause, adjusting volume, previous track, next track, audio input selection, audio output selection...... How to use Matter SDK to implement? I can build chip-tools and linux-x64-all-clusters app in repo now , but I don't know how to implement audio product using SDK? I found some clusters in media device type, but I don't know how to use these clusters to implement my products.
I want develop a matter amplifier that has the feature of adjusting gain, adjusting DSP parameters...... This is not a standard device type in Matter document. How can i implement it? and if I implement a custom device type, does it mean that other ecosystems(Alexa, Siri, Google Nest Hub...) can't control my product?

Platform
IMX8
Anything else?
No response

I want develop a matter wireless speaker that has the features of play, pause, adjusting volume, previous track, next track, audio input selection, audio output selection

There's no such device type defined in the device library right now; you would probably need to get that to happen...  You could perhaps pretend like it's a video player with no video, but previous/next track are not things anywhere in there.

does it mean that other ecosystems(Alexa, Siri, Google Nest Hub...) can't control my product?

Correct, unless they have some sort of out-of-band knowledge of what your non-standard clusters are...

@bzbarsky-apple
Thanks for your quick reply.
We are an audio product manufacturer. According to what you said, we are unable to develop and manufacture a fully featured audio products that support matter at current stage because no such device types.
If we want Matter to add speakers and amplifiers device types, could you tell me what are the ways to achieve? If our company joins CSA members, is it possible to allow us to propose corresponding standards? Or submit an application to expedite the development of this standard?

https://csa-iot.org/become-member/
There is some description about membership at a glance, including things like: Participant Members contribute to and develop standards they will later adopt and use. Participants can access Alliance Working Groups, are hands-on in developing specifications, and gain early access to draft specifications and testing, with opportunity to develop and get to market faster.


[ISSUE] Git decorations no longer overwriting the assigned colours

Using VS code 1.85.1
"It's important to note that when a file is under source control and there are changes to be committed, Git will overwrite the file's color with its own color scheme."
The above is no longer the case in the version I'm using. Colors remain as assigned by the extension.
I still need my IDE to highlight the files I've modified in the file explorer and tab titles.

I'm having this issue as well. I'd much prefer the git color override to easily distinguish which files have been modified.


Steam Overlay not working while using Forward+ renderer

Describe the bug
When using the Forward+ renderer I don't get the usual Steam popup in the bottom right corner when running from the editor. When the renderer is changed to Compatibility it works perfectly.
To Reproduce
Steps to reproduce the behavior:

Create new project with Forward+ renderer
Create basic Steam initialization script. 'steamInit' and 'run_callbacks()' are more than enough.
Running the project does not activate Steam overlay.
Change renderer to Compatibility and run project again.
Steam overlay works normally.

Desktop (please complete the following information):

OS: Windows 11
Version: 22H2

Version of Godot & GodotSteam:
Godot 4 beta 12 (pre-compiled) from the GodotSteam release page.
Thank you for all your hard work. :>

Ahh, checked previous closed issues and saw that this was already addressed. :)

Hey there!  I have not heard of Forward+ but if it has compatibility issues with Steam's overlay there isn't anything we can do it about it.  Out of curiosity, what does the "Compatibility" renderer use?

If I understood correctly, Compatibility uses GLES3.

Interesting.  Yeah, I tested it yesterday when someone else brought this up.  Forward+ is definitely breaks overlay but I found that Compatibility severely lags it even though it does work.  The responsiveness was pretty non-existent and it took the friends list about 4 minutes to properly populate.

So just to confirm this is still an issue as of RC 2, having compiled the GDExtension branch. I'm not entirely sure where/who to open an issue with. It seems that it was an issue on Valve's side at some time which from what I can see should be fixed? Therefore is this a Godot issue? If so should it be brought up there to see what's going on exactly?

Hey there.  Yeah, as far as I can tell this is 100% a Godot issue and should be reported in their repo.  Steam overlay works with Vulkan, DirectX, OpenGL, Metal, etc.  if the engine is using them properly.  I don't know much about how Godot is using Vulkan but it indeed does not work  And, as I mentioned before, Compatibility barely works.  I don't know how this gets fixed but surely someone can figure it out.
Thank you for confirming it is still broken.  Hopefully they can get this ironed out by stable or else Godot games on Steam are about to lose a bunch of functionality.

My thoughts exactly especially when Godot 4 is nearing release. I'll open an issue and try and explain as best as possible. Many thanks

Sounds good.  Keep us posted!

Just for reference issue opened here: #73532

Thanks for the link.  I am now involved in the thread.  Sadly, the suggested changes don't seem to make any difference so I'm not sure what else to do at this point.  Waiting to hear back if they have some other suggestions perhaps.
Still odd it would work in OpenGL / Compatibility with as is but not Forward+.  I also admittedly know very little about graphics rendering and related technologies.

No worries. Thanks for taking the time to take a look. Also to add to this I tried a C# Steam implementation using Godot Mono to see if that had any differences. Again no, same outcome Forward + didn't work, compatability did. Looking at this FP Wiki

Returns true if the overlay is running and the user can access it. The overlay process could take a few seconds to start and hook the game process, so this function will initially return false while the overlay is loading.

Could be not hooking to the process in the way OpenGL does, again I'm similar, no idea what it could be to be honest without knowing more about the underlying Vulkan rendering.

Indeed!  I've looked into it a few times already but I've never found any solutions; which is why I assume it was Godot's issue. Especially since we ran into the problem during Godot 4's alpha and beta stages.  I guess if it doesn't work with Facepunch either then it very well could be Forward+.
Hopefully someone on the Godot team will take a closer look because none of the suggestions in the thread you linked work and there isn't really much else we can do on our end.  OpenGL is now really smooth in RC2, so that's good.

OK, so far further testing I have found that GodotSteam with Forward+ actually does work if the project is run through the Steam client itself either as a non-Steam game or a Steam game.  However, it will not work when run through the editor or as an exported executable outside of the Steam client.
I don't know what this means or how to resolve it but changing the module initialization levels doesn't make any difference.

I tried testing with older Nvidia drivers since one user said they do have overlay working with Forward+ on an Nvidia 1070.  However, that driver set with my EVGA 1070 still doesn't work with overlay and Forward+.  I'm not entirely sure how that person has it functional using one of our pre-compiled builds from beta 11 though.

I'm currently looking at how/if it’s possible to run the Steam Init before the render loop begins and maybe it might get Steam to hook to it. Obviously, this is a workaround but I'll update this if I get any results.

Allegedly moving it up in the module initialization process was supposed to work but does nothing.  And, according to my tests, running it through the Steam client works perfectly fine even if it doesn't work in-editor with Forward+.  Driver changes didn't work either.  With my limited knowledge on the topic, I'm not sure what else to check on but it still seems odd.

Definitely keep us posted on whatever you find out!  I'd love to get this solved somehow.

Will do! I also want to solve this as it seems like a trivial feature to have working.

Still hoping someone on the Godot team will look at this a bit more or at least provide some additional information.  Given what we can access, and from what I understand, there might not be much more I can do on my end.

I hope so. I think the most likely scenario will be more people will eventually work in Godot 4 (using Forward+)  and then notice the same issue occuring when they go to implement Steam. Then maybe this will give this issue some more attention. Again thank you for spending the time to take a look, completely understand you are limited with what you can do.

True.  I'm not really sure how many developers are really using Godot 4 at this time but the number will indeed grow after stable release.  I know I don't have any plans to use it until sometime after stable releases.  And welcome!  I've been working on this project for about seven years or so now so I like to keep it in good working order.  I will keep looking for solutions to this provided it is something we can fix on our end.

Unfortunately, no luck in a workaround. I tried exploring the idea of binding the Steam Init before the RenderDevice or RenderServer starts. However, I do not believe there is an exposed way of doing this or a way to override the default RenderDevice class to imbed the Init in the initialization of the RenderDevice (At least from what I could see)
What I did find however is more proof of the GameOverlayUI.exe  found in the default Steam installation not hooking to the debug process. In that folder it has a GameOverlayRenderer.log file that shows the process hooking to running instances. In compatibility mode it clearly shows this link happening.
In Forward+ the log just stops, and nothing happens.
On a closed Steam group discussion, I saw a suggestion of enabling ENABLE_VK_LAYER_VALVE_steam_overlay_1=1 but I think this is already enabled in SteamOverlayVulkanLayer64.json So I don’t think that is a lead.
I think I’ve exhausted all my options from what I can do to find a workaround. If anyone else comes across this in the future I hope any of this information helps. I think for now I’ll have to live with the overlay not functioning and just talk to Steam for achievements (which for now is all I really need)

Thanks for that thorough update!  Yeah, the tools they give us for moving the module up in initialization don't work for this; either they don't work at all (which is unlikely) or they have no effect on this issue.  Which, after everything I've seen and what little knowledge I have, seems to still be a Godot issue.  I'm not sure what the difference between OpenGL and Vulkan is under-the-hood for Godot.
Again, it works in Steam client but not in the editor, so whatever overlay functions you need will still work for the end-user.  Not great but at least this bug has no effect on the actual players.

Hello all,
I have a custom build with mono for Godot 4.0 stable and same behavior can be seen on my side(steam overlay not starting).
But if you minimize the debug window and maximize it again, the overlay it starts to work as expected... hope this helps the developers, and also it can be used as a workaround for the time being...

Just to add to my findings. If you export and upload a game to Steam and then play said game through Steam the overlay will work. This is just an editor bug which I was unware of. (I thought the debug preview would be representative of the exported game) Still doesn't really explain the issue though.

Just to add to my findings. If you export and upload a game to Steam and then play said game through Steam the overlay will work. This is just an editor bug which I was unaware of. (I thought the debug preview would be representative of the exported game) Still doesn't really explain the issue though.

Try my finding above, it behaves the same for you? minimize debug window and then open it

Just tried that and it didn't work. I am using the GodotSteam GDExtension with Godot 4 Stable mind you.

@miketgv That is pretty interesting behavior.  I'll try it out this afternoon and see what happens.
@Defaultsound  Yeah, it works uploaded and seems to be an editor problem.  As you say, I'm not sure what that means or why.

I can see the same behavior using Facepunch.Steamworks(minimize the debug window and maximize it again, the overlay starts to work as expected) so it's and engine issue.

I think @Defaultsound might have found this out too, or it was someone else.  I have suspected it was Godot but I don't know enough about display renderers to technically argue the case.  We haven't been able to get traction with the Godot devs on this though; they just bounced the issue back to us.
You are using Windows?

I think @Defaultsound might have found this out too, or it was someone else. I have suspected it was Godot but I don't know enough about display renderers to technically argue the case. We haven't been able to get traction with the Godot devs on this though; they just bounced the issue back to us. To be fair, I think the issue got raised over there just before Godot 4 hit stable so they were super busy.
You are using Windows?

Yes on Windows

Hmm.  I can't get it to start the overlay with that.  That being said, I'm using Windows 11 and it doesn't seem like it actually properly minimizes the window ever.
What GPU are you using, by the way?

Hmm. I can't get it to start the overlay with that. That being said, I'm using Windows 11 and it doesn't seem like it actually properly minimizes the window ever.
What GPU are you using, by the way?

I use an AMD Ryzen 9 5900HX  laptop with AMD Radeon 6800M GPU (Advantage edition), you can see the behavior in the video bellow.
https://www.youtube.com/watch?v=En5_vgJB9PM

Interesting.  Yeah, very flicker-y too.  I'll try it out on my laptop which is AMD.  My desktop is AMD Ryzen but with an NVidia GPU.

Interesting. Yeah, very flicker-y too. I'll try it out on my laptop which is AMD. My desktop is AMD Ryzen but with an NVidia GPU.

Maybe this will help also:
The issue is persistent in the build-ed version of the game for Facepunch.Steamworks, but not for GodotSteam, so maybe what is done in the build for GodotSteam should be also for Debug?

I'm not sure I 100% follow. You mean exported builds with Facepunch or GodotSteam? I tried this on my Linux laptop and still don't get overlay after minimize / maximize.
That is an interesting error code though. Yeah, let me know what you find in VS.
So I've debugged the engine and for some reason the  VK_ERROR_UNKNOWN = -13 is returned as a result on vulkan_context.cpp:1970 line when the function vkCreateSwapchainKHR(fpCreateSwapchainKHR inside the engine) is run after the game is maximized... I'll try to debug it further, but I'm not a driver developer(I have some c++ experience but not that in depth)

I'm not sure I 100% follow. You mean exported builds with Facepunch or GodotSteam? I tried this on my Linux laptop and still don't get overlay after minimize / maximize.
That is an interesting error code though. Yeah, let me know what you find in VS.

So I've debugged the engine and for some reason the  VK_ERROR_UNKNOWN = -13 is returned as a result on vulkan_context.cpp:1970 line when the function vkCreateSwapchainKHR(fpCreateSwapchainKHR inside the engine) is run after the game is maximized... I'll try to debug it further, but I'm not a driver developer(I have some c++ experience but not that in depth)

Hmm, yeah, while this is impressive progress I do not understand it.  Haha.  About how many times do you have to minimize/maximize for this to kick in?  Also huge thanks for getting more information on this.

Hmm, yeah, while this is impressive progress I do not understand it. Haha. About how many times do you have to minimize/maximize for this to kick in? Also huge thanks for getting more information on this.

In my case after first minimize -> maximize i have to wait 2-3 seconds to see the steam popup, after that if I minimize->maximize again the error occurs, it's a strange behavior but I can replicate it each time and on both Facepunch.Steamworks and GodotSteam

You can try adding your findings to the issue on Godot's Github page: https://github.com/godotengine/godot/issues/73532  They closed it but maybe this will ignite some interest in it.

Gonna play around with it more and see if I can get the min/max thing to work on one of my Linux systems.

OK gang, we finally have an official answer to all of this.  Steam overlay will never work when running your game from the editor using Forward+ due to how and when it activates Steamworks.  It works fine with Compatibility and when you export your game to upload to Steam.
Making it work with Forward+ in the editor actually causes more issues than it is worth.  Thank you everyone for pitching in on solving this mystery!


Added support for multiple transformers and validations

Closed as there was some weird diff issue

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

Project coverage is 100.00%. Comparing base (c312206) to head (b94a2ec).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main        #9   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          862       874   +12
=========================================
+ Hits           862       874   +12

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

Closed as there was some weird diff issue

Reopened this issue as the reason for diff was because didn't merge branch after squash


SR-12300: URLSession.dataTask doesn't follow redirects with lowercased location

When initialising the HTTPURLResponse, store the header fields names
capitalised except for X- headers. This matches Darwin.

This allows responses with lowercase fields names to be read correctly
and redirects to works if the field name is 'location' instead of
'Location'.

Add missing function HTTPURLResponse.value(forHTTPHeaderField:).

@swift-ci test linux


Find queries with rejectOnNotFound should return a type and not type | null

Bug description
Return type for find queries with rejectOnNotFound should not be something | null, and instead just something
How to reproduce
Take the following example schema:
datasource db {
  provider = "sqlite"
  url      = env("DATABASE_URL")
}

generator client {
  provider = "prisma-client-js"
}

model User {
  email String  @id
  name  String
}

This:
function exampleFn(): { name: string, email: string } {
    const user = await prisma.user.findUnique({
        where: { email<EMAIL_ADDRESS>},
        rejectOnNotFound: true
    })
    return user
}

results in:
Type '{ name: string; email: string; } | null' is not assignable to type '{ name: string; email: string; }'.
  Type 'null' is not assignable to type '{ name: string; email: string; }'.ts(2322)

Expected behavior
function exampleFn(): { name: string, email: string } {
    const user = await prisma.user.findUnique({
        where: { email<EMAIL_ADDRESS>},
        rejectOnNotFound: true
    })
    return user
}

works without errors
Prisma information
See above.
Environment & setup

OS: macOS 11.1
Database: PostgreSQL
Node.js version: v15.0.1
Prisma version: 2.15.0

today's release solves this issue :) (also tracked [here])

Thanks @spencer-brown for pointing that out :smile:


Send chunks from different files

Using simultaneousUploads option you can send multiple chunks to the server on parallel. Chunks are picked in sequence so if you upload 3 files it will first process all the chunks of the first file.  The server then can either, create partial files and merge them (Node sample) or (if possible) use multiple threads each one writing to a random access file (Java sample).
Is there any workaround to make flowjs picks chunks from different files (round robin)?
I am happy implementing an option to change the strategy of the "uploadNextChunk" but I wanted to know if it is possible already.
Reason why this option would be handy in my case:
The backend of the application I am working on, streams the data to an external distributed system which creates the files in different nodes. This system doesn't allow creating partial files or writing multiple chunks to the same file concurrently. Setting simultaneousUploads higher than 1 is useless in this case.
However, the distributed system allows writing different files at the same time very efficiently, as different files are processed by different nodes.
Cheers.

This was a point of confusion for me.  I was expecting to see multiple files upload at once as well.
Theoretically, uploading simultaneous chunks from one file is no faster or slower in total than simultaneous chunks from multiple files.  But there is a perceived slowness when you see things happen synchronously.  I would like to see a feature to allow this.  It would also be nice to have some level of extensibility to allow for priority file uploads.  For instance, if the files list has 10 small files and one large file, I may want to start the large file and let the small files upload at will.


Create dry-advection model for flight trajectories

Description
Based on flight trajectory, create a model of where a contrail "might be" based on advection.
We'd have to make assumptions about spreading and sedimentation, but these could be model parameters.
Alternatives
Additional context

I think the team at Google may have already implemented a model like this as their "baseline"
This could be baseline for a slight more nuanced model that takes into account SAC and ISSR

Model parameters could include:

Dispersion rate (constant, linear in time, or more complicated function)
Sedimentation rate (vertical)

Completed in 0.46.0


Converted SendToAll to transaction output

fixed incorrect fee in UseForChange txs

Codecov Report

Merging #96 (f61070d) into master (d30dd7a) will increase coverage by 0.07%.
The diff coverage is 88.09%.

@@            Coverage Diff             @@
##           master      #96      +/-   ##
==========================================
+ Coverage   51.05%   51.12%   +0.07%
==========================================
  Files         108      108
  Lines       11277    11301      +24
==========================================
+ Hits         5757     5778      +21
- Misses       5138     5140       +2
- Partials      382      383       +1

Flag
Coverage Δ

unittests
51.12% <88.09%> (+0.07%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

model_transaction_config.go
83.06% <ø> (ø)

model_draft_transactions.go
74.60% <88.09%> (+0.50%)
:arrow_up:

Continue to review full report at Codecov.

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


fix the license header on the spirv-new tests

fixes #1471
The source files for the spirv-new tests were using the older Khronos license instead of the proper Apache-2.0 license.  This PR fixes the license header in all source files.
Note, I removed the license header from CMakeLists.txt instead of replacing it because none of the other CMakeLists.txt seemed to have a license header.  If we add a license header to CMakeLists.txt we should add it to all of them.
No functional changes.

Merging as discussed in the January 16th teleconference.


Fix bom/README.md

@googlebot I signed it!

@kanterov The email used for the commits in this PR (you're github no-reply email) isn't registered with your CLA signing. I'm going to ping a teammate offline to see what we can do here.

@googlebot I signed it!

@BenWhitehead I fixed the email. As far as I know, my employer has signed CLA, but it didn't seem to work as well. I'm fine with closing this PR if you can do this 1 char change yourself because it doesn't seem to worth our time :)

Thanks for the contribution @kanterov and being flexible. I've opened #372 with the fix and passing cla check.


ADS-B Weather Failover

Stratux version: v1.5b2

Stratux config:
SDR

[ ] single
[X] dual

GPS

[X] yes
[ ] no
type:

AHRS

[X] yes
[ ] no

power source: 12v adapter
usb cable: about the fourth one I tried and was finally able to provide stable power.

EFB app and version: ForeFlight 11.4
EFB platform: iOS 12.3
EFB hardware: iPhone XR, iPad Air

Description of your issue:

I’ve configured my stratux so that my iPad connects via Ethernet to the stratux, thus leaving the WiFi free to connect to my phone. As such, I’m often in a spot where I can receive ADS-B based weather as well as Internet based weather. However, on a recent flight I realized that this isn’t quite as helpful as it might seem since when the ADS-B reception was bad (a condition that will hopefully be alleviated by my recently purchased v3 UAT radio) I still wasn’t getting internet weather. I didn’t want to disconnect the stratux, because I still wanted a good GPS signal, but by doing that I was stuck with poor weather data. I realize that this is something that could/should probably be handled by ForeFlight, I was wondering if it might be feasible to have an option that would basically just disconnect the ADS-B based weather thus forcing ForeFlight to automatically fail over to internet based weather.

(...) any data sent from stratux triggers ForeFlight to use it as its only data source. So if the connection from your receiver were dropped when there were no UAT towers, that would mean losing the GPS and AHRS data feeds.

Issue speaks of reception problems that are likely the result of some other issue with the receiver. Leaving this to troubleshooting on Reddit.


Improvement/player

Massive improvements of the player package:

Refactored to be able to extend the code with local blob storage/caching
Changed blob retrieval method to native reflector calls instead of using S3 storage directly
Changed the way binary data is read from blobs physically, now using default ioutil.CopyN 32K buffer instead of hacked version using 2M buffer
Implemented local storage for downloaded/decrypted blobs (aka cache)

@lyoshenka,

huge PR. I skimmed it because @tiger5226 already did a great job with the in-depth review.

This is a common complaint but I don't know what to do with it. Regardless of PRs size, by the time they are usually reviewed, my work on any particular piece of code has long moved forward and/or is deployed already.
Case in point, there's just 60 lines of code and it's been open for 9 days now. I'm not complaining, I know we are all very busy but that's a chronic pain point in our code reviews.

I am going to start to make it a priority to check lbry.tv PRs brother. I have never really looked for PRs to review here which is a big part of the problem. I only look when you ask me to.
As to the size of the PRs, it's definitely a problem. I can assure you, that you will get a less thorough review from 99% coders as a result.
As a general practice, I require my team to submit PRs for units of work that take 0.5, 1hr, 2hrs, 4hrs, or 8 hrs. No one is allowed to submit 16 hrs of work for review. In other words, if you worked a day without sending a PR in to be merged. One of the following things happened:

you forgot to send in the PR at the end of the day for work completed.
you did not spend enough time breaking your work out into logical working units.
you ignored the rules

In addition to this, the flip side is that a PR is never to be outstanding for more than 24 hrs, and most should be reviewed within 2-6 hrs depending on the time of day it is submitted. The first task in the morning for those on my team is to review PRs, then code. The thought process is that any code review left outstanding is a timeline cost. timeline for projects, timeline for feature releases, timeline for phases etc.
While I think these rules are great for overall productivity, I am not trying to dictate, but at the very least, I should be dogfooding my beliefs and get your PRs reviewed. I will do a better job of that.


Integrate CI build output with Slack

As a: Developer I want receive slack notifications on successful and failed build results from PRs so that I can maintain real-time PR activity in our team slack.
Integrate Azure Devops Builds with Bedrock Slack or Github PR activity with team slack.

Location is in #cloud-factory-git of epicstuff slack workspace
Azure Pipelines APP [11:55 AM]
@nrose has added subscription to *Build completed* notification for Microsoft.bedrock


Update dead links in DefaultPlugins docs

Objective

Fix dead links on this docs page: https://docs.rs/bevy/0.9.0/bevy/struct.DefaultPlugins.html

Solution

Point links to the imported versions instead of the versions in external crates.

Thanks! These links make much more sense.

bors r+


Mongolian (Mongol bichig) produces error

On both the dev and prod versions, I experience an error when attempting to convert using the "Mongolian (Mongol bichig)" language.  A simple example is converting the text "Mongol" using roman-to-script.  On dev, the error message is "WARNING: An internal error occurred. If the error persists, contact the technical support team."  On prod, it is "Error: SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data".  I do not experience these errors when the same conversion is run with the language set to "Mongolian (Cyrillic)".

This is caused by a malformed YAML file. I have it fixed in 44fa98a but that's part of a much larger set of changes in test. I can short-cut that fix into main if you need it soon.

Fixed in #130.


Laziness on Lucene indexes

implement laziness on lucene index engines, full-text and spatial

feature backported to 2.2.24, massive import/read/delete tests done.


Add husky pre-push hook

Add a husky pre-push hook that prevents pushing if the branch has one of the following:

Failed tests
Linting errors

@degrammer please have a look at my PR. I left it with just checking Eslint because I feel like adding unit test to the pre commit hook could slow down ones workflow and is sometimes not needed, e.g. if there are changes to the readme or other parts of the code base that aren't impacted by unit tests. That said, I think we can be always smarter what we want to do and this could be a good start.

Merged, so closing.


Permission settings for save_config() function

When running robot.save_config() you get:
PermissionError: [Errno 13] Permission denied: '/usr/local/lib/python3.6/dist-packages/dorna-1.3-py3.6.egg/dorna/config.yaml'
This goes away if you run sudo or sudo -s before running the script, but maybe you could change the permissions when downloading everything?

This is a problem with Linux and Mac.
I actually don't know about your suggestion: "maybe you could change the permissions when downloading everything". If you can explain it more to me.
One option is to the make sure that the user has a sudo access when they are initializing the Dorna object.

Yes,
These suggestions all make sense!


Added a travis.yml configuration.

Added a travis.yml configuration.

You will need a requirements.txt to get the ipython/jupyter dependencies. See the one in jupyter_nbconvert for an example.
I'm not sure if this is going to work because apt-get install pyqt doesn't make pyqt available to the virtualenv where I think the tests are run.

One way we could do this is to install Miniconda and use that to install pyqt. It's kind of a hack, but it works. Here's an example that does it: https://github.com/jhamrick/dbtools/blob/master/.travis.yml

What did you do to install pyqt before this repo was split out?  Still a little stuck on the best approach and was digging through old versions of github.com/ipython/ipython

We didn't ;-) - the Qt console wasn't tested on Travis. We also run tests on our Jenkins server, where we can install Qt. But now that the Qt console is a separate repo, doing this doesn't hold up tests for the rest of IPython, so it makes sense.

@takluyver Finally got the tests to pass! Lmk what you think

Does it uses docker on travis ? Or should we set sudo: false ?
Also sidenote, I guess on docker build we could use cached dirs to speed-up rebuild.

Ah good catch--I was previously installing pyqt via apt-get. sudo can be set to false.  It's not using docker, although that would be a good thing to do as a follow-up

depends also on when the repo was created, I think they might default to sudo:false now.

decreased the time by ~30 sec /run roughly 1min15 total.

Great, thanks for getting this started. Since it's using conda, you can probably remove the use of travis-wheels, and install a few dependencies with conda install pyzmq sphinx pygments.

Cool--went ahead and made that change

:+1:

Great!


[move-prover] Avoiding unnecessary writebacks.

This PR extends the borrow analysis to track which references actually have been written to and therefore need to be written back when the reference dies.
Before ths PR, if we have the following Move code:
fun f(r: &mut T) {
    let x = r.x;
    *x = 1;
}

The generated bytecode would be something like:
   x := borrow_field(r, x)
   LocalRoot(r) <- .. // r dies, but unnecessary write back because r has not been written to
   WriteRef(x, 1);.
   LocalRoot(r) <- ..

This PR removes this redundancy, which appeared to be quite common in the code generated from the framework.
There seems to be a small performance benefit coming out of this PR, but I have not systematically measured it. More importantly perhaps, the generated code is cleaner, and it is step for me to understand borrow and write back analysis better.
Note: This PR is based on unmerged #5927 which is separately reviewed; please focus on the 2nd of the two commits.
Motivation
Towards refining pack/unpack.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Existing tests
Related PRs
#5927

/land


10/08/2018 01:53 : Create Pull Request for branch master -> prTesting1_2018100801

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Checklist for 2020.12

Corresponds to changes and updates done in the 2020.12 release
Removals:

[ ] Deprecated method candidates (subbuf(Any:U) on Buf, chdir(Str(), :$!test) on IO::Path)
and indir(IO() $path, &what, :$test!) subroutine candidate that were
throwing an exception instead of a deprecation warning for a long time
were removed from CORE -9040318c

Additions:

[ ] Add new method deterministic to Iterator role -87fc041f-b83b1b34-b63c0e02-c37a88e8-96285afb
[ ] Introduce %*SUB-MAIN-OPTS<coerce-allomorphs-to> setting -bd5eba4b-49eecd68
[ ] Add a new candidate to spurt routine and method.
It does not have an argument taking content to write,
making it similar to the touch utility -f2ea0a65
[ ] Add :emit-timed to Supply.batch method -492651ea
[ ] Make is DEPRECATED trait introspectable on Routine instances -0d1c8a8e

Changes:

[x] Improve output of Attribute documentation when rendered with Pod::To::Text -a0a8a515
[ ] Increase sensitivity of Supply.batch(:seconds) x 1000 -aecfc9b3
[ ] The cas subroutine now accepts Mu as both its target and values -998cae57
[ ] Defined List instances no longer return True when calling ACCEPT
with an undefined List (i.e. List ~~ () returns False now) -9fd79f9e
[x] Mark the base native array class as Positional -d1d2546e

subbuf candidates that have been added now need an example.