python3Packages.icmplib: 3.0.1 -> 3.0.2

Motivation for this change
Update to latest upstream release 3.0.2
Change log: https://github.com/ValentinBELYN/icmplib/releases/tag/v3.0.2
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

3 packages built:

home-assistant
python38Packages.icmplib
python39Packages.icmplib


frame data not displaying

The window shows up in game but the frame data only shows up after i throw an attack 5-6 times as if the program is lagging or something. I've tried running as admin and starting it before or after i start tekken and nothing seems to help. Any ideas as to why this is happening and how to fix it?

Sounds like a performance issue. TekkenBot is pretty poorly optimized right now and requires a decent chunk of cpu. Are you getting 60 fps in Tekken 7?

yeah i get 60 fps but i will mess with some setting and see if that helps thanks for answering


minio: 2020-03-25 -> 2020-05-01, minio-client: 2020-04-04 -> 2020-04-25

Motivation for this change
Things done

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

[X] NixOS
[X] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg test minio

@GrahamcOfBorg test minio


Fix: Cross-namespace RequestAuthentication

Please provide a description of this PR:
fixes #54051
adds searching svc namespace to enable cross-namespace reqauthn

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@ilrudie: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
947f17cdb808e84da5747077067b8222894c5af7
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Should we consider a 1.24 cherry-pick? I think its fair to expect this would work in 1.24

cool cool, if we like the general direction I'll take a look at getting more tests in place, ty

@ilrudie: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
a759772a54f62f5f5f32d4122165c647367dbb06
link
true
/test lint

release-notes_istio
a759772a54f62f5f5f32d4122165c647367dbb06
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@ilrudie: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
a759772a54f62f5f5f32d4122165c647367dbb06
link
true
/test lint

release-notes_istio
1eca9730fe6c2bc82ea651ac6b0c132fcf19167f
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@ilrudie: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
bbdc5121161546b991adf53cde3901b5cacf93b8
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@ilrudie: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
04959054c73221bb783c1f39b3cea9dc9a4558c1
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Got some TestWaypointTelemetry style tests for RA and x-namespace RA

Hold incase you want to change the test, but I am good as is as well

Hold incase you want to change the test, but I am good as is as well

Thanks, I like the suggestion. I'll take a look at getting that added.

@ilrudie: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
b53998b4e9a32a39e3ea1a34c912a8d8f001e729
link
true
/test lint

gencheck_istio
b53998b4e9a32a39e3ea1a34c912a8d8f001e729
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

In response to a cherrypick label: new pull request created: #54144


Add option to disable automatic request logging

There should be an option to disable the automatic request logging - i.e. "request finished" and "request errored" - as we want to customize the output. Currently we get two logs, our custom and the automatic logging.

Would you like to send a PR?

@mcollina PR submitted #70


Adds name amalia.

Hi @kaxcode,
Thanks for your contribution. Can you please address the requested changes?
Tag me here when it's done and I'll happily merge this for you :)

Changes made.

Fantastic! Thanks @kaxcode.
Merged #1


$scope.form.$valid is returning false even it is true.

Hi,
Some times $scope.myform.$valid is returning false even it is valid. i don't understand why it is happening please help me out.
Thanks,
Rajesh.

Unless you provide a Plunker that I can troubleshoot with, I can't really help you.

Hi,
I had created a plunker. you can check it here.http://plnkr.co/edit/y6HqYhxoqWSlj8aWqEz7?p=preview
try entering single character i will alert you the onchange  $valid value. Some times it is displaying false and some times it is displaying true.

I see the effect as you said, but it seems to happen only within the Controller. It's probably a timing issue, there's a few timer and also the $digest that could affect, so I'll see what I can do.
On the other side or testing, disabling the button with form1.$invalid inside the html code is always working correctly.
<button type="submit" name="btn_ngDisabled" class="btn btn-primary" ng-disabled="form1.$invalid" >Save</button>
I'm also curious, why are you looking at form1.$valid inside the Controller? I usually have a call to checkFormValidity() with the validationService to do just that. But actually, I'll have to look at that function too since it seems to have the same issue there too.

Even it is not working properly with checkFormValidity(). When there is no data it is returning true, when i enter 1 character it is returning false and then true. Please check the updated plunker.
http://plnkr.co/edit/y6HqYhxoqWSlj8aWqEz7?p=preview

I know since I've already tried it. I think the problem is around the fact that you use ngChange, there seem to be a timing issue, possibly with the $digest. What is the particular reason that you use ngChange?

When there is a change in input field i need to enable save and cancel buttons (by default they are disabled). if user clicks on save or cancel button i need to call corresponding function and again i need to disable the buttons until there is a change in input fields.

Ok so I found what the problem is. It might not be clearly visible to you but the conflict is simply by the fact that I use a timer of 1sec (by default) to even start doing validation, but in your controller you use ngChange right away without waiting for my validation to even start.
If you add a timer, which is over my 1sec timer, into your onChange(), function it does work (even if that doesn't look like nice code)
$scope.onChange = function(){
   $timeout(function(){
     $scope.result = $scope.form1.$valid;
     $scope.checkFormValidity = new validationService().checkFormValidity($scope.form1);
   },1100);

On the other hand if you try removing the timer by setting the debounce to 0, you still need to use a $timeout simply because using form1.$valid inside the Controller is just too fast and too early in the $digest to get correct result. For example, this would work:
 var vs = new validationService({ debounce: 0 });

 $scope.onChange = function(){
   $timeout(function(){
     $scope.result = $scope.form1.$valid;
     $scope.checkFormValidity = new validationService().checkFormValidity($scope.form1);
   },5);

but if you try making it under 5ms, you'll have timing issue again.
So let's resume all that, the way you want to check the form1.$valid in the Controller is simply too fast and too early in the $digest to work correctly with my Directive. You can counter that problem with a $timeout even if ugly, it does work. I'm not sure how else to fix your current implementation. Hope that helps... if you have other ideas then say it.

Actually this makes me think that there might be another option but that would require code change on my side. Inside my Directive, I use a function named attemptToValidate() which return a promise. I use this for other purposes, but if you want, I could possibly create an attribute for a callback when the promise is finished.
Please note that this is just an idea, it might not work. But let's take an example of this possibly.
<input name="firstname" ng-model="firstName" validation="required:alt=First name is required"
validation-callback="onChange">

So basically you would use a new attribute of validation-callback instead of ng-change, that could possibly work since I already deal internally with a promise.

This would be greatful with out using above ugly code. can you please test it by making change on your side by adding new attribute. if that works it will be very helpful to me.

The concept works, I just tried it now. I'll have to look and see if I can add it to the Service as well. I'll have it done by the end of the week or maximum next week.

ok Thank you.

Added the validation-callback like I told you before. So you could use it this way
<input name="firstname" ng-model="firstName" validation="required:alt=First name is required"
validation-callback="onChange()">

Pushed in latest version 1.4.14, please see the Wiki - Validation Callback
You could also look into the folder more-examples/validationCallback for a full demo. Enjoy it...

If you like the project, you could also click on Star on top, that's all I ask... enjoy :)

Thank you so much for your help... Its done i can give to multiple stars...:)


BlackboardEntry references are gone after adding an entry to Blackboard

When to remove an entry from the Blackboard, some references on Nodes are gone.
How to reproduce:

Create some nodes and conditionals, add references from Blackboard.
add another entry to blackboard

Then some (not all) references from nodes and conditionals are gone.
Have a look at the video:
https://github.com/acdamiani/schema/assets/20422129/dc7782b0-79c2-469c-b817-1a0d93ce38ec

Additional Info:
If you do that again and again after you've added 2-3 more entries to Blackboard, all references are gone. I only have a few nodes but it's a lot of work to wire all again.
I happened to me with v1.0.5, too.

One more: When you add one in global Blackbaord, all references in all Graph assets are gone. :-/

I'm having trouble reproducing this issue. In the video linked, it looks like you're referencing some Blackboard variables that are not included in the local Blackboard ($AttackDistance for example). Are they global Blackboard variables? Dynamic variables? I'd hazard a guess that it's related to the typechecking code that's run when new entries are created.

Hi @acdamiani,
yes I have local and global Blackboard variables.

I'm not sure if I'm using dynamic variables.
For instance: is the 100 radius a dynamic variable?

Arrgh. I've had a working Tree and wanted to test how to reproduce it with a new tree.
When I've added a variable to the local Blackboard in the new tree, everything was fine. When I've added it to the global variables the reference in the test tree AND the working ai tree were gone :-(
Next time I create a blank project for it.
Here my global Blackboard Asset:

I cannot find the Assets inside the Entries List. Could that be a possible reason?
I need to cry a bit in my bed now since my references are gone.

Glad you had a backup! Yes, this almost definitely a Global blackboard issue; I'll fix it ASAP.

Hi @acdamiani
Do you have any updates on this issue? It's currently blocking me. I can temporarily work without using the BlackBoard, but I'd really like to use it. 😄
Thanks in advance. I hope you're doing well and not too stressed right now.

Sorry about that! I've been busy recently and have not had much time to work on this project. This issue turned out to be a simple fix through--see #20.

I could test it today and it works. Finally I can go on with may AI.
Thank you very much. :)


Bump identity.governance.version and identity.event.handler.notification.version

Purpose

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7110946905


Run all three source code checks in Pylint job

@tisnik Your image is available in the registry: docker pull quay.io/openshiftio/rhel-fabric8-analytics-f8a-api-gateway:SNAPSHOT-PR-32


Update accessibility contrast test coverage

Update the accessibility test to cover all Elements with same semantic label to get better coverage on contrast test.
#109465 Improve flutter contrast test coverage
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

CC @goderbauer for secondary review

Won't this pick up text that has been excluded from semantics?

Won't this pick up text that has been excluded from semantics?

Hasn't happened in any test in GP3. The fix has identified 470 tests however where we needed to manually handle failures.

Also, if we're going to look at every text element anyway - then why bother looking at the semantics nodes at all. Shouldn't we just traverse the element tree to gather every text element?

@jonahwilliams has a good point regarding element traversal...

Won't this pick up text that has been excluded from semantics?

Hasn't happened in any test in GP3. The fix has identified 470 tests however where we needed to manually handle failures.

Can you add a test case to verify that this isn't happening, please?

Added another test with a contrast failing Text wrapped in ExcludeSemantics and a duplicate Text with contrast within guidelines.

You probably have to rebase this with the latest master to get tests to pass again.

Closed this since it caused a bad merge with master and somehow went ahead of flutter/master.
Created a new PR in https://github.com/flutter/flutter/pull/109784


New version: Impute v0.2.0

Registering package: Impute
Repository: https://github.com/invenia/Impute.jl
Created by: @rofinn
Version: v0.2.0
Commit: 4f21ce4473be02def6cfee43763b83871bf97d5f
Reviewed by: @rofinn
Reference: https://github.com/invenia/Impute.jl/commit/4f21ce4473be02def6cfee43763b83871bf97d5f#commitcomment-34191012

Please put back constraints on julia and DataFrames.


Bump github-api to 1.90

Life critical fix because of kohsuke/github-api#387

Thanks! Is it sufficient to manually upgrade the GitHub API Jenkins plugin as part of your Jenkins installation, or do I need to release a new version of this plugin too?

This has been fixed in 1.2.4. Please allow a few hours for the update to propagate. You should see it in your Jenkins update center.


Add new SL for worker mountpoint filling up.

This is usually something the customer has to take care of.

/lgtm

/approve


Add correct datetime literal format for maxcompute

See https://github.com/tableau/connector-plugin-sdk/issues/683

 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.Jianan Sun seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Issues accessing intellisense / argument completion with screen reader

While writing code, there are two tooltips - intellisense and argument completion. The latter can be triggered automatically or manually, depending on settings.
I'm looking at the accessibility of these tooltips with a screenreader, there are 5 different types of content to discuss - see screenshots below.
A - intellisense autocomplete. This plays nicely with NVDA, reading the list of suggestions if you pause typing.
B - intellisense docs. I have not managed to get this read out by NVDA
C - intellisense links. I have not managed to select these with NVDA
D - argument completion docs. I have not managed to get this read out by NVDA
E - argument completion links. I have not managed to select these with NVDA
Are there any shortcuts I am missing, or navigation strategies?
Are any/all of these intended to be screen reader accessible, or is the information available elsewhere? (in which case it would be good to have a deliberate decision and this be documented)

The information is a subset of the information exposed via the API tab. However, accessing it in a contextual way is very useful.
I think we'd welcome suggestions.
Some of these behaviours are from CodeMirror 6 with various degrees of customisation.
It might be interesting to review Visual Studio Code's approach.

My understanding from looking at VSCode (which could be wrong), is that the text in the right hand box is placed in a div outside of the visible area which is read out by the screen reader.
Despite there being a close button and link in the right hand box, I can't find a shortcut to access this area using the keyboard only.


Adjust Font Size and Styling in 'Latest News' Section news page

[ ] Decrease the font size of the 'Latest News' heading.

[ ] Apply correct font style to the news title.

[ ] Align the date and "written by" on the same line with the same font size as the text.

[ ] Update the 'Read More' button with correct styling and font.

Website that needs to be changed:

Figma/Design:

@ankiiin  this task is the same as #638 which is already complete. :)


Is it safe to remove the old CoginitoUserPool was created by AmplifyCLI ?

** Which Category is your question related to? **
auth
** What AWS Services are you utilizing? **
AppSync + Lambda + Cognito + APIGateway
** Provide additional details e.g. code snippets **
Not until recently if I change the CognitoUserPool's attributes, Amplify CLI throw the error.
the reason why it throws the error is here: https://github.com/aws-amplify/amplify-cli/issues/1845
but, I noticed today that it is now creating the new CognitoUserPools with new attributes but leaving the old one behind.
Is it safe to remove the old userpool?

@monaye If you don't have any active users in it and don't are about the data, then yes you can go ahead and get rid of it.

@monaye
Thanks, I just wanted to make sure if the resources are somehow still refereeing in the CloudFormationStack or somewhere.


Exception includes name of partial

Exception includes now name of partial
Use as instead of is and then as

I added a few changes which makes finding the problem as a user of the library a bit simpler plus a small "optimization"

Awesome! Thanks


Lookup failure exceptions

Currently, lookup returns None when it doesn't find anything, which we've known might be a problem.
https://github.com/SynBioDex/pySBOL3/blob/0b87e64dad642099c36153d02a68f7085f411666/sbol3/refobj_property.py#L19
In fact, this does tend to lead to opaque errors, like iGEM-Engineering/iGEM-distribution/issues/222
I would like to have it raise exceptions instead, of two classes:

Missing child objects - this is a serious error, and should never happen
Missing toplevel objects - this is legal state, but needs to be handled as an exception situation, allowing the calling function to decide on a context-appropriate action to take (e.g., record the outgoing line, fetch the missing object, abort its operation, declare an error)

Both of these should be child of an exception class that covers both, so that one can decide whether to catch either or both.
I am divided on whether to make this backward compatible with an optional or not. On the one hand, I'm certain that not making it backward compatible will cause lots of things to break. On the other hand, this is our last chance to rip off the bandaid before 1.0 deployment.

An alternate approach is to document the lookup method as returning either an instance of sbol3.Identified or None if the object is not found. Callers should be responsible for checking the returned value before proceeding with it.
The stack trace in https://github.com/iGEM-Engineering/iGEM-distribution/issues/222 shows code that is using the value returned from lookup without checking to see if an object was found. That feels like an application error. And without proper method documentation on lookup, it is understandable.
I think what we have to decide is whether the inability to lookup a URI via the lookup method is an exceptional condition or simply a possible outcome. Since there is nothing in the SBOL 3 specification that requires a reference to resolve within a document I think it falls into the latter camp of "possible outcome", rather than an exceptional condition. Others may view the situation differently.

Planned approach: two SBOL utilities functions, get_child and get_toplevel (so similar)


no typescript supported

npm install @types/assets
npm ERR! code E404
npm ERR! 404 Not Found: @types/assets@latest

npm ERR!      /Users/me/.npm/_logs/2018-12-07T08_42_30_384Z-debug.log

@moffsugita I'm not really a TypeScript pro. Could you provide a pull request for this, please?


Cannot save entropy plot in the absense of X

vagrant@vagrant:/code/Misc/Trimble/MSMobile5_503_1250_OperatingSystem_TSC2⟫ binwalk -E -J wat.txt

DECIMAL       HEXADECIMAL     ENTROPY
--------------------------------------------------------------------------------
0             0x0             Falling entropy edge (0.198120)
QXcbConnection: Could not connect to display
Aborted

wat.txt is a text file containing the three bytes wat.
This is a VM without any UI components. I understand that displaying the file entropy is probably not reasonable, but there's no reason saving a file should fail.
GDB StackDump (for a different file, but that shouldn't make a difference):
vagrant@vagrant:/code/Misc/Trimble/MSMobile5_503_1250_OperatingSystem_TSC2⟫ gdb --args python3 /usr/local/bin/binwalk -E -J FWV5_0_3WWE.img
GNU gdb (Ubuntu 7.11.1-0ubuntu1~16.04) 7.11.1
Copyright (C) 2016 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "x86_64-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
<http://www.gnu.org/software/gdb/documentation/>.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from python3...(no debugging symbols found)...done.
(gdb) r
Starting program: /usr/bin/python3 /usr/local/bin/binwalk -E -J FWV5_0_3WWE.img
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
[New Thread 0x7fffea651700 (LWP 30060)]
[New Thread 0x7fffe9e50700 (LWP 30061)]
[New Thread 0x7fffe564f700 (LWP 30062)]

DECIMAL       HEXADECIMAL     ENTROPY
--------------------------------------------------------------------------------
0             0x0             Falling entropy edge (0.756011)
172032        0x2A000         Falling entropy edge (0.605149)
540672        0x84000         Rising entropy edge (0.955983)
1294336       0x13C000        Rising entropy edge (0.974166)
2883584       0x2C0000        Falling entropy edge (0.842440)
2969600       0x2D5000        Rising entropy edge (0.996469)
24100864      0x16FC000       Falling entropy edge (0.625845)
QXcbConnection: Could not connect to display

Thread 1 "python3" received signal SIGABRT, Aborted.
0x00007ffff7825428 in __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:54
54      ../sysdeps/unix/sysv/linux/raise.c: No such file or directory.
(gdb) bt
#0  0x00007ffff7825428 in __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:54
#1  0x00007ffff782702a in __GI_abort () at abort.c:89
#2  0x00007ffff3f96d9e in QMessageLogger::fatal(char const*, ...) const () from /usr/local/lib/python3.5/dist-packages/PyQt5/Qt/lib/libQt5Core.so.5
#3  0x00007fffe0d8625e in QXcbConnection::QXcbConnection(QXcbNativeInterface*, bool, unsigned int, char const*) ()
   from /usr/local/lib/python3.5/dist-packages/PyQt5/Qt/plugins/platforms/../../lib/libQt5XcbQpa.so.5
#4  0x00007fffe0d8b505 in QXcbIntegration::QXcbIntegration(QStringList const&, int&, char**) ()
   from /usr/local/lib/python3.5/dist-packages/PyQt5/Qt/plugins/platforms/../../lib/libQt5XcbQpa.so.5
#5  0x00007fffe108d43d in ?? () from /usr/local/lib/python3.5/dist-packages/PyQt5/Qt/plugins/platforms/libqxcb.so
#6  0x00007ffff473bbfb in QPlatformIntegrationFactory::create(QString const&, QStringList const&, int&, char**, QString const&) ()
   from /usr/local/lib/python3.5/dist-packages/PyQt5/Qt/lib/libQt5Gui.so.5
#7  0x00007ffff4749436 in QGuiApplicationPrivate::createPlatformIntegration() () from /usr/local/lib/python3.5/dist-packages/PyQt5/Qt/lib/libQt5Gui.so.5
#8  0x00007ffff4749edd in QGuiApplicationPrivate::createEventDispatcher() () from /usr/local/lib/python3.5/dist-packages/PyQt5/Qt/lib/libQt5Gui.so.5
#9  0x00007ffff417d7d6 in QCoreApplicationPrivate::init() () from /usr/local/lib/python3.5/dist-packages/PyQt5/Qt/lib/libQt5Core.so.5
#10 0x00007ffff474b6ab in QGuiApplicationPrivate::init() () from /usr/local/lib/python3.5/dist-packages/PyQt5/Qt/lib/libQt5Gui.so.5
#11 0x00007fffee31ed09 in QApplicationPrivate::init() () from /usr/local/lib/python3.5/dist-packages/PyQt5/Qt/lib/libQt5Widgets.so.5
#12 0x00007fffeec41ce9 in sipQApplication::sipQApplication(int&, char**, int) () from /usr/local/lib/python3.5/dist-packages/PyQt5/QtWidgets.so
#13 0x00007fffeec41db9 in init_type_QApplication () from /usr/local/lib/python3.5/dist-packages/PyQt5/QtWidgets.so
#14 0x00007fffef5f861b in sipSimpleWrapper_init () from /usr/local/lib/python3.5/dist-packages/sip.so
#15 0x000000000055d17c in ?? ()
#16 0x00000000005b7167 in PyObject_Call ()
#17 0x0000000000528d06 in PyEval_EvalFrameEx ()
#18 0x0000000000528814 in PyEval_EvalFrameEx ()
#19 0x000000000052d2e3 in ?? ()
#20 0x0000000000529332 in PyEval_EvalFrameEx ()
#21 0x0000000000528814 in PyEval_EvalFrameEx ()
#22 0x0000000000528814 in PyEval_EvalFrameEx ()
#23 0x0000000000528814 in PyEval_EvalFrameEx ()
#24 0x0000000000528814 in PyEval_EvalFrameEx ()
#25 0x0000000000528814 in PyEval_EvalFrameEx ()
#26 0x000000000052d2e3 in ?? ()
#27 0x0000000000528eee in PyEval_EvalFrameEx ()
#28 0x000000000052d2e3 in ?? ()
#29 0x0000000000529332 in PyEval_EvalFrameEx ()
#30 0x0000000000528814 in PyEval_EvalFrameEx ()
#31 0x000000000052d2e3 in ?? ()
#32 0x000000000052dfdf in PyEval_EvalCode ()
#33 0x00000000005fd2c2 in ?? ()
#34 0x00000000005ff76a in PyRun_FileExFlags ()
#35 0x00000000005ff95c in PyRun_SimpleFileExFlags ()
#36 0x000000000063e7d6 in Py_Main ()
#37 0x00000000004cfe41 in main ()
(gdb) q
A debugging session is active.

        Inferior 1 [process 30056] will be killed.

Quit anyway? (y or n) y

This is a known issue, addressed in the FAQ. Unfortunately, the pyqtgraph graphing library relies on QT functionality that requires an X server, even if just generating an output file and not displaying it (there is a StackOverflow discussion on the matter here).
The FAQ describes how to use X virtual frame buffers to satisfy this X server requirement on GUI-less systems.

Whoops, I missed that in the FAQ. Derp.
FWIW, it's not that I don't know how to use a XVFB to work around the issue, it's that installing any windowing environment makes a huge mess and installs a giant number of additional packages, and I'm running headless to (among other reasons) keep the linux install small.
Have you considered another graphing backend? If you're just throwing up simple static graphs (rather then dynamic stuff), matplotlib is quite nice, and doesn't have the attachment to QT that pyqtgraph does.

For this, and various other reasons, the latest commits of  binwalk have switched to using matplotlib instead of pyqtgraph. This should alleviate any X server issues!

Fancypants! Matplotlib is really nice too.


Shot won’t stop in micra

Hi Tate - I finally successfully installed the board into my Micra this weekend and it appears to be connected well. My acaia lunar is taring and starting its timer when I start a shot but I am not getting the 3 sec beep and when I turn the lever to the off position at 3 sec, the shot stops.
I connected with light blue as in the iOS instructions and was able to set/confirm 36 grams setting.  I’m not that familiar with troubleshooting Arduino but am willing to give it a try? I’m a Mac user.

Hi @chcha , moving this discussion to the discord. We can troubleshoot there. If there ends up being a bug in the firmware, we can open a case on the git repo. Thanks!


Hocs 4206 add home sec interest

HI Laurie, you need to merge (manually) the four bpmn diagrams - the tests fail and the workflow is broken until this is resolved.

Hi Laurie, the build is still failing because the top level bpmn process tests all fail. I think I can see why; the HomeSec stage on the top level bpmn diagram is floating, as in not connected to anything!?! I think the line that looks like it is connected actually passes underneath it!


Multiple TS files for Find All references , Peek Definition

Is it possible to register multiple files , like file1.ts , file2.ts and files3.ts, then use "Peek Definition" or
"Find All references" commands  in File1.ts and   able to see  the result from all the 3 files?
Currently, it works for loaded code in editor.
Obviously , I am asking to have similar functionality of  VSCode.
Thanks

That's something the standalone editor doesn't support and there are no plans to do so. It would require to introduce the notion of a project and to have access to arbitrary files of that project.

I was able to make this work across files with just a few hacks. First add all the files with both of these functions:
monaco.languages.typescript.typescriptDefaults.addExtraLib
monaco.editor.createModel

Then I hacked into the require function of the loader to monkey patch how the internals find the other file reference.
                var oldRequire = require
                function newRequire() {
                    if(arguments.length >= 2) {
                        if(arguments[0][0] == 'vs/editor/browser/standalone/simpleServices') {
                            var cb = arguments[1]
                            arguments[1] = function(m) {
                                m.SimpleEditorService.prototype.findModel = function(editor, data) {
                                    var mod = monaco.editor.getModel(data.resource);
                                    var s = (<any>new Error()).stack
                                    if(s.indexOf('openEditor') >= 0) {
                                        editor.setModel(mod)
                                    }
                                    return mod
                                };
                                if(cb) {
                                    cb(m)
                                }
                            }
                        }
                    }
                    oldRequire.apply(this, arguments)
                }
                newRequire['config'] = require.config
                newRequire['getConfig'] = require.getConfig
                newRequire['reset'] = require.reset
                newRequire['getBuildInfo'] = require.getBuildInfo
                newRequire['getStats'] = require.getStats
                require = newRequire

Great!, But I could not make it work. Do you have a working sample?
I added :
function xhr(url) {
var req = null;
return new monaco.Promise(function(c,e,p) {
req = new XMLHttpRequest();
req.onreadystatechange = function () {
if (req._canceled) { return; }
		if (req.readyState === 4) {
			if ((req.status >= 200 && req.status < 300) || req.status === 1223) {
				c(req);
			} else {
				e(req);
			}
			req.onreadystatechange = function () { };
		} else {
			p(req);
		}
	};

	req.open("GET", url, true );
	req.responseType = "";

	req.send(null);
}, function () {
	req._canceled = true;
	req.abort();
});

xhr('file1.ts').then(function(res) {
  monaco.languages.typescript.javascriptDefaults.addExtraLib(
                       res.responseText, 'file1.ts');
       });

monaco.editor.createModel("file1.ts","typescript");
});

xhr('file2.ts').then(function(res) {
  monaco.languages.typescript.javascriptDefaults.addExtraLib(
                       res.responseText, 'file2.ts');
       });

monaco.editor.createModel("file2.ts","typescript");
});

It has to load all the files into addExtraLib first before createModel. Also your params on createModel are wrong.
setProject(infos: { [name: string]: string }) {
        for(var k in infos) {
            monaco.languages.typescript.typescriptDefaults.addExtraLib(infos[k], k)
        }
        for(var k in infos) {
            monaco.editor.createModel(infos[k], 'typescript', new monaco.Uri().with({ path: k }))
        }
    }

I'll be making my app public soon.

I got it partially working by adding :
new monaco.Uri().with(.....)
Now it shows  the list of files  in overlay result from "Find All References".  But you cannot select the result files and navigate to them.

Check for errors. The default implementation crashes which is why that monkey patch above is needed.

I added the monkey patch in different places and added the breakpoint to it, but it seems, the code is never been called.  I added your code at the end of loader.js and also in the page before calling this:
require.config({ paths: { 'vs': 'node_modules/monaco-editor/dev/vs' }});
Also, I added the code after " var RequireFunc = (function ... " line 1807 in loader.js, but I got some error.
where is the place of monkey patch?

Yes setup the patch before require.config then call require on the specific modules that need to be patched. Here's what I have right now as a React component.
https://gist.github.com/pierogitus/e5d68605c9f014e51dad0df47515b4d7

Thanks, I used the code, and now it  breaks in "SimpleEditorService.prototype.findModel " patch.
It does not work yet, one of the error is :
[UriError]: If a URI does not contain an authority component, then the path cannot begin with two slash characters ("//").

Here's my full app http://run.primoca.com
Use the "fork this" link at the top to explore.

Great, I checked it without  local ipfs.


for testing whether we skip the error prone check

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
70
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ HBASE-22174 Compile Tests _

+1
mvninstall
314
HBASE-22174 passed

+1
compile
31
HBASE-22174 passed

+1
checkstyle
41
HBASE-22174 passed

+1
shadedjars
340
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
80
HBASE-22174 passed

+1
javadoc
28
HBASE-22174 passed

_ Patch Compile Tests _

+1
mvninstall
313
the patch passed

+1
compile
32
the patch passed

+1
javac
32
the patch passed

+1
checkstyle
39
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
330
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
644
Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.

+1
findbugs
87
the patch passed

+1
javadoc
27
the patch passed

_ Other Tests _

+1
unit
237
hbase-client in the patch passed.

+1
asflicense
15
The patch does not generate ASF License warnings.

2706

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-114/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/114

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux bdfecbd1ac96 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
HBASE-22174 / 86f9443eeb

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-114/1/testReport/

Max. process+thread count
293 (vs. ulimit of 10000)

modules
C: hbase-client U: hbase-client

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-114/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

Seems worked. There is no '-PerrorProne'
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-114/1/artifact/out/branch-compile-hbase-client.txt
cd /testptch/hbase/hbase-client
/usr/share/maven/bin/mvn --batch-mode -DHBasePatchProcess clean test-compile -DskipTests=true


make an autocomplete-plus provider for supercollider

Possibly this should be a separate package, or maybe even somebody else could do this.
https://github.com/saschagehlich/autocomplete-plus/wiki/Tutorial:-Registering-and-creating-a-suggestion-provider
Build an index of class and method names from the currently configured supercollider library + quarks. This could be saved as as JSON file file so it could be used also by sublime, emacs, vim etc.

https://github.com/atom/autocomplete-plus/wiki/Provider-API


[Triangle] Non-pseudo triangle

Status: Work in progress
Why?
Using a pseudo element for the tooltip triangle is convenient, but comes with all the short-comings of using a pseudo element. For example, the lack of box-shadow support forces the current implementation to use filter: drop-shadow which has some cross-browser incompatibilities. This could also open up the opportunity to add/remove the arrow from the constructor since the arrow would need to be appended from libraries like drop.

:100: thanks @geekjuice

please can this be resolved?

This PR doesn't actually remove the pseudo-elements. It just replaces :before pseudo-elements with :after pseudo-elements. I don't see any advantage to this change.

(From our offline conversation) For clarification, the non-pseudo was indeed a misnomer :grimacing:. Rather the intend was to wrap the triangle pseudo element with a non-pseudo element which can be styled more appropriately e.g. box-shadow instead of having the pseudo element directly on the tether'd  elements.
That said, this was a hacked up solution so it'll probably be better to come up with a more robust one.

I haven’t looked into how they do it, but we might want to look at https://github.com/FezVrasta/popper.js which essentially seems like an attempt to solve this issue and some of the other annoying issues with working with Tether/Drop.

@adamschwartz I've been taking a look at Popper. It looks like a very high-quality library. They do indeed avoid using pseudo-elements for arrows, instead using real elements.
However, I want to clear up a point of confusion on this ticket: Using real elements instead of pseudo-elements does not solve the box-shadow problem. They're still rectangular elements, and if you give them box-shadow, they get a rectangular shadow:

From an API standpoint, there may be advantages to having real elements for arrows instead of pseudo-elements—real elements can be accessed and manipulated with JavaScript, for example—but from a styling perspective, real elements and pseudo-elements are equivalent.

@TrevorBurnham I’m mostly inclined to agree, but I think there might one thing you’re missing. Using a real element gives one at least the advantage of having two more pseudo-elements. Stylistically more boxes gives you more control. With three pseudo elements, for example, one could draw three triangles, one for the arrow, one for a border around the arrow:

While this could be accomplished with an SVG background image on one pseudo-element, changing the color of the arrow or its border would require creating specific SVGs for each color combination. So more elements does at least yield more flexibility, which practically speaking I think lends itself to more stylistic options.
I think this argument is made even stronger when you consider arrow placement, which is a much harder problem to solve with pseudo-elements since JavaScript needs to communicate the placement to CSS in some way, and a pseudo-element does not have its own style attribute. Therefore any sort of alignment of a pseudo-element must be done through rules (as opposed to direct values) passed through from its owner or parent elements’ attributes (as we do with .tether-attached-left, etc.).
Point being, it’s probably fair to say that much could be accomplished keeping the arrows as pseudo, but there’s no shortage of reasons to switch to a proper element. Being the person who made them pseudo’s in the first place I’m happy to admit I was short-sighted on this one. :stuck_out_tongue:

One final point: at the time I implemented this, one thing I thought might catch on was -webkit-filter: drop-shadow(Npx) (which I’m sure as you know handles border-width-hack arrows just fine). This is actually how the shadow works on Shepherd’s welcome page:

And it is one of the ways I envisioned shadows working. Of course, it turns out that three years later drop-shadow still isn’t really a thing (afaik), so we still have to resort to SVG blur or other tricks to draw non-rectangular shadows.


i18n(fr): update guides/rss.mdx

Description (required)

Adds changes from #9601
Removes type: recipe from the frontmatter (see #8239)
Removes additional blank lines and reorganize import to more easily compare changes in the file
Fixes some typo (e.g. fr-fr should be lowercase)
Rewords some sentences

Related issues & labels (optional)

Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

fr
guides/rss.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


[Feature] - Game Details Page

Description
Design and implement the Game Details Page to display comprehensive information about each game, including title, description, genre, publisher, release date, system requirements, and cover art.
Motivation
A detailed game page provides users with all necessary information to make informed decisions about the games they are interested in, enhancing user engagement and satisfaction.
Proposed Solution

Use Bootstrap cards and grid layouts to organize game information effectively.
Implement tabs or accordions using Bootstrap components to categorize details like system requirements and additional information.
Ensure that cover art images are responsive and optimized for different devices using Bootstrap's utility classes.
Use HTMX to dynamically load additional game details if necessary.

Acceptance Criteria

[ ] Feature is clearly defined
[ ] Meets proposed solution
[ ] Tests considered

Additional Info

Include interactive elements such as user ratings and reviews to enrich the game details.
Ensure accessibility features are in place to support users with disabilities.
https://interfaceingame.com/

Samuel finished this,


[FLINK-31510][yarn] Use getMemorySize instead of getMemory.

What is the purpose of the change
JIRA: FLINK-31510. Use getMemorySize instead of getMemory
In YARN-4844, use getMemorySize instead of getMemory, because using int to represent memory may exceed the bounds in some cases and produce negative numbers. This change was merged in HADOOP-2.8.0, we should use getMemorySize instead of getMemory.
Resource#getMemory
/**
   * This method is DEPRECATED:
   * Use {@link Resource#getMemorySize()} instead
   *
   * Get memory of the resource. Note - while memory has
   * never had a unit specified, all YARN configurations have specified memory
   * in MB. The assumption has been that the daemons and applications are always
   * using the same units. With the introduction of the ResourceInformation
   * class we have support for units - so this function will continue to return
   * memory but in the units of MB
   *
   * @return memory(in MB) of the resource
   */
  @Public
  @Deprecated
  public abstract int getMemory();

Resource#getMemorySize
/**
   * Get memory of the resource. Note - while memory has
   * never had a unit specified, all YARN configurations have specified memory
   * in MB. The assumption has been that the daemons and applications are always
   * using the same units. With the introduction of the ResourceInformation
   * class we have support for units - so this function will continue to return
   * memory but in the units of MB
   *
   * @return memory of the resource
   */
  @Public
  @Stable
  public long getMemorySize();

Because the unit is the same, both are MB, we can directly replace.
Brief change log
Use getMemorySize instead of getMemory.
Verifying this change
Manually verified.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? no

Thanks @slfan1989 for creating this.
How did you manually verify this issue, could you describe it in detail?
By the way, would you mind resolving conflict in YarnClusterDescriptor?

@reswqa Thank you very much for your suggestion, I will modify the information as soon as possible

@reswqa Can you help review this PR again? Thank you very much! I explained the reason for possible negative numbers.

Thanks @slfan1989 for the update, would you mind squashing all to only one commit with the message like:

@reswqa Thank you for your help to review the code! I will rebase and submit.


Is it possible to use without singleton?

Since singleton approach will affect possibilities to do a clean unit test I was wondering if this would just work just injecting IPermissions as a singleton instance on constructor.
Thanks!

The best way to do testing is simply is to implemented your own IPermissions
Then, in your shared code to test do something like this:
public static IPermissions Permissions
{
   get
   {
     //if in unit testing return your implementation, else return
      return CrossPermissions.Curren;
    }
}


[Bug] Single quote shown as comma when applying keystrokes

Describe the bug
When showing keystrokes, if you press the single quote key ( ' ) key, it is shown as a comma ( , ).
To Reproduce
Steps to reproduce the behavior:

Start recording
Press '
Stop the recording
Apply keystrokes

Expected behavior
The key ' is shown as '.
Screenshots

Desktop (please complete the following information):

OS: Windows 10 (10.0.19044 N/A Build 19044)
Version 2.37

Additional context
N/A

@reduckted What keyboard layout are you using? I'm not able to reproduce this using my configuration. I'm also in Windows 10 build 19044

"English (United States)" (the QWERTY layout). I'm not sure if it helps, but GetKeyboardLayoutName() reports it as 00000409.

I just tried a few other things. I can also reproduce it with the the on-screen keyboard in Windows.

And it also occurs when editing the key strokes. Here's a gif of me pressing ; then ' then . then ,:

Hi, that's weird.
For me, it's displaying correctly the ' and , chars.

@NickeManarin I just found out what's happening and opened a PR with the fix. We can discuss further details there.
@reduckted Can you build ScreenToGif using that branch, repeat your test cases and get back in this thread?


Never initiates connection on Windows

Hey love this concept. I got it working great on Linux but I have a use case where I need to forward Meraki logs out of a network where only Windows is allowed.
When I start the session, the first packet never emits. If I use nmap, it emits just fine as per the Wireshark here.
Any idea what the issue might be? Might be something simple as it is my first attempt.
https://github.com/SenseUnit/dtlspipe/assets/15258962/3605564e-c7b4-4cdb-87aa-04250a2128a1

Didn't realize I needed to initiate traffic to get a negotiation to start, I'll submit a PR for a README clarification.


Update upload speeds to show Mbps instead of MB/s

Motivation
Most people use speed tests to get their speed in Mbps. This is deceivingly larger than the MB/s we show in the CLI, so people assume our CLI is slow.

I don't know if its most people, I personally use MB/s on steam and prefer to see such units on other speed tests as well


Reduce leader imbalance

When performing a rolling restart, one of the conditions for bubuku to
restart a broker is that the cluster is in a green state. This means
no under replicated partitions and no preferred replica imbalance.
Kafka deployment by default already enables automatic check for
imbalance and in our case this is configured to happen every 100
seconds. But by default it triggers only when 10% of partitions are
imbalanced for a broker (that's my understanding).
From Kafka docs:
leader.imbalance.per.broker.percentage
The ratio of leader imbalance allowed per broker. The controller would trigger a leader balance if it goes above this value per broker. The value is specified in percentage.

Type:	int
Default:	10
Valid Values:
Importance:	high
Update Mode:	read-only

This change is an atempt to cause it to trigger more often in order to
avoid the rolling restart from getting stuck and requireing manual fix
before it can continue.

Although it seems that it should help, I am a bit skeptical about this change. Each broker has ~2500 partitions. 1% = 25 partitions. I a mean that maybe 0 is the proper value, but I am not sure (

True. I didn't see if they have an even smaller threshold. But let's focus on finishing upgrade then we can look into this.


[Dubbo-5004] Performance tuning: Reduce object allocation

What is the purpose of the change
Reduce object allocation when get Method returnTypes.
The original code allocate 2 Optional objects, 2 Consumer objects and 1 Iterator object each time it gets Method returnTypes, it will increase GC pressure.
Brief changelog

Use  != null instead of  Optional.ifPresent(Consumer)
Use Map<String, List<MethodModel>>  instead of  Map<String, Set<MethodModel>> in ServiceModel

Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Optional is more readable, but it may cause performance issue.
Some related info:
https://stackoverflow.com/questions/54443944/gc-overhead-of-optionalt-in-java
https://stackoverflow.com/questions/34696884/performance-of-java-optional
I also did a search in github, spring and hibernate use optional
https://github.com/spring-projects/spring-boot/blob/21f1b3a79d1c2168aa0088e45368f3f75957b22c/spring-boot-project/spring-boot-actuator-autoconfigure/src/main/java/org/springframework/boot/actuate/autoconfigure/web/servlet/CompositeHandlerAdapter.java#L56
https://github.com/spring-projects/spring-framework/blob/ad6231ad2989ca5428bb51afdf48a9112710ce1d/spring-core/src/main/java/org/springframework/core/annotation/MissingMergedAnnotation.java#L103
https://github.com/hibernate/hibernate-orm/blob/79a8f43ba57d9e4a1269c5027dfd063b18d4f7d9/hibernate-core/src/main/java/org/hibernate/bytecode/enhance/internal/bytebuddy/ByteBuddyEnhancementContext.java#L88
It's more like a trade off.
And also if we choose not use Optional, I suggest add related rule in checkstyle check

Yeah, it's a trade off. We can use Optional in non-critical paths to improve code readability. But, using Optional in critical paths increases the pressure on the GC.
Spring, Hibernate and Dubbo are different types of frameworks, and their workloads are very different. Dubbo carries thousands of requests per second, which requires higher performance stability, so maybe ! = null is a better choice.

 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.

 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.

what？

I agree with @LinShunKang on all the improvements made in this PR, I think we should prefer performance instead of readability in this case.
The performance-2.7.x branch has been merged into master, could you please open a new one

@chickenlj I was already open new PR in here.


fix: use open -t to open text editor

This fixes #3284

Superseded by #4004. This addresses the comments that @liby made as well, incidentally. Thank you!


Initial DenyListedApi

Task/Issue URL: https://app.asana.com/0/1198194956794324/1208235055272415/f
Description
Add a deny list API lint rule where we can specify what APIs we don't want devs to use.
The PR adds some few rules for now and also regenerates the baselines to allow existing usages.
Steps to test this PR
Build passes

#4990  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @aitorvs and the rest of your teammates on  Graphite


Update README.md to clarify specifying orientation from Exif data

Clarified how to use the orientation option, and changed wording regarding the Exif parser.
Regarding the orientation option, from the original wording I thought the meaning was that you could simply enable the orientation option (e.g. by setting it to true), but from reading the source code for the demo, I see you have to determine the orientation separately with the loadImage.parseMetaData function.
On a side note, it might be worth making it an option to correct the orientation in a single function call. Fixing the orientation is the single reason I'm using this library (HTMLImageElement provides decent functionality out of the box), and I'm guessing the same could be for many others users of this library.
Thoughts on my proposed wording changes? I'm not completely happy with the wording I cam up with, but it seems clearer to me than the original wording.

Thanks @Nateowami, I've also just added the new options meta:true and orientation:true:
https://github.com/blueimp/JavaScript-Load-Image#options

Thanks, that's great!


Cannot read property 'env' of undefined

Hi,
I use EnvPlugin and I have an error in first call to process.env : Cannot read property 'env' of undefined
In the top of bundle I have a __process_env__ object defined with all values
Modules have a row  at its top : var process = require("process");
What I am missing ?

Could it be related to HMR? Can I refresh the page? That's a known bug on the list

@JiDai  order of plugins matter, could you show your fuse config ?

Here is my config :

const fuseBox = new fsbx.FuseBox({
    homeDir: "static/app/",
    outFile: "./build/bundle.js",
    modulesFolder: "node_modules",
    plugins: [
        fsbx.EnvPlugin(env),
        fsbx.JSONPlugin(),
        [
            fsbx.SassPlugin({
                includePaths: [
                    path.resolve('./static/sass'),
                    path.resolve('./static/images')
                ]
            }),
            fsbx.CSSResourcePlugin({
                dist: "build/images",
                resolve: (f) => `/images/${f}`
            }),
            fsbx.CSSPlugin(),
        ],
        fsbx.SVGPlugin(),
        fsbx.BabelPlugin({
            config: {
                sourceMaps: true,
                presets: ["es2015", "react", "stage-0"],
                plugins: [
                    "react-hot-loader/babel",
                    "transform-class-properties"
                ]
            }
        })
    ]
});

``

It happens at page load, not after HMR reload.

Modules have a row at its top : var process = require("process");

Think this is invalid. process is a node global https://www.google.com.au/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=nodejs+process. Remove the line  var process = require("process"); and try again :rose:

My target is the browser. That's why I use this plugin, and this is the plugin that add these lines

My target is the browser. That's why I use this plugin, and this is the plugin that add these lines

my bad. Sorry :heart:

I can actually use process.env in browser without require('process'). At least I can with webpack / browserify, is fusebox different here?

I suppose, with wepack I have not this behaviour but I use the DefinePlugin to get process.env on browser

@basarat  we have envplugin for this and that is what @JiDai  is trying to do :)

So this test does not work for you? https://github.com/fuse-box/fuse-box/blob/master/test/process_variable.js

The test pass.
I tried to debug in chrome on 2 similar project :
On the first one, it fails and I have in compiled bundle process module contents :
// for now just expose the builtin process global from node.js
module.exports = global.process;

On the second it works and I have the shim module
// shim for using process in browser
if (FuseBox.isServer) {
    if (typeof __process_env__ !== "undefined") {
        Object.assign(global.process.env, __process_env__);
    }
    module.exports = global.process;
    ...

It helps ?

Hm, which modules are you trying to run?

I don't understand. I just run node fuse.js

Could you show the code please? Some test repo.

https://github.com/fuse-box/fuse-box/pull/235 may be related

I can't, it is a private project. I update all my deps to last version, I re-run and and the error is no more. But the process.env object is empty. I have find this in bundle.js
FuseBox.pkg("process", {}, function(___scope___){
___scope___.file("browser.js", function(exports, require, module, __filename, __dirname){

// shim for using process in browser

var process = module.exports = {};

I suppose another version of process module was resolved

@JiDai yes! that's the problem, you have something that fusebox does not support. Any ideas on how to solve it?

This is what I was looking for, I assumed that there were other dependencies to process module. I found 2 but after removing them, I still have the problem.

I finally find it : I set modulesFolder to value 'node_modules' in fuse file. This causes  this.context.customModulesFolder to be true in PathMaster.getNodeModuleInformation and resolving fail. I removed the setting

@JiDai  can we close this now :)

Thanks!


fix: re-use validators and identity from state

Problem
When producing a block, we redundantly fetch the validators again after just
having fetched it to build the node state. This is very slow and unnecessary.
Solution
Use the validators and identity that are part of the State.t that we build to produce the block.

Probably going to close this PR with a better solution.

This is a small improvement but it's an improvement - let's merge it after all.


Map markers disappear at minimum zoom level

When fully zoomed out, markers disappear in the map visualization (SCORE Seasonal, SCORE Mozambique, UMSP)
Presumably a problem with the longitude bounds being more than a "whole world" but will investigate.

I will 'up' the version level requirement for this repo at web-eda in one of my related tickets.

@aaronwlsong if something on the site is preventing you from QA'ing the ticket, you can put a pause label & note as to why QA is paused, but please leave the ticket in "ready for testing". thanks!

When I zoom out, looks like that the marker is there and the same as other studies.

This wasn't completely fixed.


Unable to download a tweeted video from Twitter on the web

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.06.02. 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.06.02

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)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

$ youtube-dl --verbose https://twitter.com/AlifAilaan/status/1001523575386341376
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://twitter.com/AlifAilaan/status/1001523575386341376']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.06.02
[debug] Python version 3.6.5 (CPython) - Linux-4.16.12-1-ARCH-x86_64-with-arch
[debug] exe versions: ffmpeg 4.0, ffprobe 4.0, rtmpdump 2.4
[debug] Proxy map: {}
[twitter]<PHONE_NUMBER>386341376: Downloading webpage
[twitter:card]<PHONE_NUMBER>386341376: Downloading webpage
[twitter:card]<PHONE_NUMBER>386341376: Downloading JSON metadata
ERROR: Unable to download JSON metadata: HTTP Error 429: Too Many Requests (caused by <HTTPError 429: 'Too Many Requests'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 579, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2211, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python3.6/urllib/request.py", line 532, in open
response = meth(req, response)
File "/usr/lib/python3.6/urllib/request.py", line 642, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python3.6/urllib/request.py", line 570, in error
return self._call_chain(*args)
File "/usr/lib/python3.6/urllib/request.py", line 504, in _call_chain
result = func(*args)
File "/usr/lib/python3.6/urllib/request.py", line 650, in http_error_default
raise HTTPError(req.full_url, code, msg, hdrs, fp)

Description of your issue, suggested solution and other information
The issue only occurs with Twitter ; Facebook and Youtube work fine.  All tweet urls containing a tweeted video give the same error.

I confirm the problem. I'm also using the latest youtube-dl under x64 Windows 7. I suppose /assume Twitter changed something during this week, because I could download some videos from Twitter three or four days ago and reverting to the previous build(s) of youtube-dl doesn't work.

FWIW....if you run YouTube-DL against the direct video mentioned:
https://video.twimg.com/amplify_video/1001498432052350976/pl/1280x720/dG4ypLu7EKJV5o6E.m3u8
above it does download correctly.
So I guess it is an embed issue?

I'm seeing the issue as well.
E:\USER\temp_ytdl>youtube-dl --verbose "https://twitter.com/electrolemon/status/
971208968323280896"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://twitter.com/electrolemon/statu
s/971208968323280896']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2018.06.02
[debug] Python version 3.4.4 (CPython) - Windows-2008ServerR2-6.1.7601-SP1
[debug] exe versions: ffmpeg N-89595-g40d4b13228, ffprobe N-89595-g40d4b13228, r
tmpdump 2.4-20151223-gfa8646d-OpenSSL_1.0.2n-x86_64-static
[debug] Proxy map: {}
[twitter]<PHONE_NUMBER>23280896: Downloading webpage
[twitter:card]<PHONE_NUMBER>23280896: Downloading webpage
[twitter:card]<PHONE_NUMBER>23280896: Downloading JSON metadata
ERROR: Unable to download JSON metadata: HTTP Error 429: Too Many Requests (caus
ed by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sur
e you are using the latest version; type  youtube-dl -U  to update. Be sure to c
all youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp2pwxs9zy\bu
ild\youtube_dl\extractor\common.py", line 579, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmp2pwxs9zy\bu
ild\youtube_dl\YoutubeDL.py", line 2211, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_defau
lt

Well, HTTP Error 429: Too Many Requests, should be a temporary issue then.

Hrxn wrote:

Well, HTTP Error 429: Too Many Requests, should be a temporary issue then.

Keep in mind that error messages often do NOT mean what they say.

youtube-dl --verbose https://twitter.com/AlifAilaan/status/1001523575386341376
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'https://twitter.com/AlifAilaan/status/1001523575386341376']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.06.02
[debug] Python version 2.7.13 (CPython) - Linux-4.9.0-6-amd64-x86_64-with-debian-9.4
[debug] exe versions: ffmpeg 3.2.10-1, ffprobe 3.2.10-1, rtmpdump 2.4
[debug] Proxy map: {}
[twitter]<PHONE_NUMBER>386341376: Downloading webpage
[twitter:card]<PHONE_NUMBER>386341376: Downloading webpage
[twitter:card]<PHONE_NUMBER>386341376: Downloading JSON metadata
ERROR: Unable to download JSON metadata: HTTP Error 429: Too Many Requests (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 579, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2211, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 435, in open
response = meth(req, response)
File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python2.7/urllib2.py", line 473, in error
return self._call_chain(*args)
File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
result = func(*args)
File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

Have updated frequently, no go.


Render Cities special achievement cards

We need the special icons for each of the 5 achievements, plus the proper icons for the other side of the achievement (the arrows, flags, and fountains).


Better type definitions for renderers prop

Currently, renderers prop object keys are expanded to string, which can cause bugs if the user types the wrong NodeType as a key.
Using Record<NodeType, ReactType>, the object's keys must be valid NodeType. Partial is needed to allow for partial renderer override.

See #417 and #417

Hi, thanks for your patience!
I’m gonna go with https://github.com/remarkjs/react-markdown/pull/428, which will result in lots of merge conflicts that it’s probably better to close this and take a new look if there still is something missing.


Fetching Spotify IDs and Profiles

Need to figure out a way to fetch other people's Spotify ID using my API

Not relevant anymore.


Show better error message when internal error occures.

Folks were worried about what detail was returned to the client and that the server operator should check the logs for detail; did the correct detail not make it to the logs?

yes, backtrace and message is in log.

Okay, I'll have to reject this one for now, sorry. I do remember that returning just any exception message could result in leaking more info than desired to the client in some cases (to be honest I don't remember any specific example though). If it's pretty important to you, I could certainly see it as an option if said option was off by default. Thanks a bunch for your work on Swauth!

i agree and thank you.


Justifying Flex-Growing Item with given Aspect Ratio is not correct

Hi.
Here is my Problem:
https://gist.github.com/LeoSchleicher/d972ca621c54decfba38a161b642984d
I want to center a box both horizontally and vertically within other box, filling all available space, bit preserving aspect ratio (ie zoom photo centered on full screen). It works as expected as I fix box size. But when I set it to flexGrow, it calculates size of box correctly, but looses vertical centering.
I made it with flexDirection=column on parent element, alignItems=center and justifyContent=center.

I see this issue too.


Remove nulls during parse_file

Question | Feature Request

Python version import sys; print(sys.version): 3.7
Pydantic version import pydantic; print(pydantic.VERSION): 0.26

With regard to the following code and outputs:
class SomeModel(BaseModel):
    a: Optional[str]
    b: Optional[str]
    c: Optional[str]
...
myjsonfile.json:
{
     "a" : "mya"
     "b" : "myb"
}

sm = SomeModel.parse_file(myjsonfile.json)
print(sm.json())
{
     "a" : "mya"
     "b" : "myb"
     "c" : null
}

Would it be possible to include a feature in the parse_file or perhaps add another helper function that would return the json without null values - this would need to apply across deeply nested structures including lists and dicts? Any advise greatly appreciated.

This is the same as #587


Priority queue in python

Is your feature request related to a problem? Please describe.
Describe the solution you'd like
A clear and concise description of what you want to happen.
It's an algorithm in queue data structure which I want to implement in python language.
Please assign it to me under lgm-soc.
Programming language

[ ] C
[ ] C++
[ ] Java
[x] Python

please specify the questions @Khushi-agarwal

Implementing priority queue data structure @ravikr126

@Amit366 kindly assign me this issue.

@Saurabh4626 @Ayush7614 please assign me this issue.


iOS: Update network activity indicator as needed

The network activity indicator has been deprecated as of iOS 13.


Can it send different info to different clients?

I have such scenario, in the python server, it uses class to update and calculate the instance' s state, then sends such state to corresponding client; in the client side, more than one clients need to communication with the server to get its one result and not interfere by others. Can your tool distinguish with different clients?

There are 2 very common networking patterns: Request/Reply and Publish/Subscribe.
What you are talking about sounds like a Request/Reply pattern which is what this repo does.
In request/reply pattern, the server acts as a function. Any client can call this function and the function output will not get interfered from other clients.
For example, suppose the server is a square function. You send 5, it gives you 25. You send 10, it gives back 100. Server can handle any number of clients. Each client can request different values and no interference would happen because the server handle each request one by one.
ZeroMQ does this common pattern perfectly.

There are 2 very common networking patterns: Request/Reply and Publish/Subscribe. What you are talking about sounds like a Request/Reply pattern which is what this repo does.
In request/reply pattern, the server acts as a function. Any client can call this function and the function output will not get interfered from other clients. For example, suppose the server is a square function. You send 5, it gives you 25. You send 10, it gives back 100. Server can handle any number of clients. Each client can request different values and no interference would happen because the server handle each request one by one. ZeroMQ does this common pattern perfectly.
I think what you need to do is to think how to fit your code into request/reply pattern or pub/sub pattern. It's usually possible to do that.

Thanks for your fast reply. I understand your example, but my scenario might be more complicated.
Suppose in server side, there is a class with initial value self.i =0, then each time the client calls the class' s update function that do self.i=self.i+1 and return self.i. Actually there are two clients call such update function simultaneously, like when client1 calls update at third time, client2 calls update at first time. Can server distinguish between them and send correct value to corresponding client?

Do you want to store one i value for each client? That's a stateful system. What you can do is to send client name along with the request and in the server create a dictionary pointing to each client's i value.
Another solution is to just let client stores their own state and when requesting, you need to request with your state.
I think it will be better if you tell your specific use case because there might be a better way.

I don't know whether your tool can let me know the client's address. Then we can create threading for each client in the server side and don't need to store the i value.

You might be able to read the client's address but that's probably not the
pattern that zeromq suggests. It prefers no identity when communicating. I
use zeromq in this repo. You can go learn more about it.
On Sat, Nov 20, 2021, 17:01 ll @.***> wrote:

I don't know whether your tool can let me know the client's address. Then
we can create threading for each client in the server side and don't need
to store the i value.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/off99555/Unity3D-Python-Communication/issues/18#issuecomment-974625814,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADUCY5BSQB5CTHMHCY5NX7DUM5WV7ANCNFSM5IMLLYWQ
.

Thanks a lot!!!


Add metric at director to monitor number of origin/cache in the federation

As part of the project to implement monitoring metrics we discussed during the kickoff meeting, we want to introduce a new metrics collected at director level to show # of registered (advertised) caches and origins in the system.
Let's try to name the metric pelican_num_server with labels

type: origin or cache (or staging)
organization/institute: This one also needs to be obtained from external sources or we need to change origin/cache advertisement message to include this
namespace: We want to include namespace per meeting notes, although I don't think we want to add the whole collection of namespaces it contains as a label

Naming is hard, so if @bbockelm or anyone else has better idea of how we should name this metric, please leave your comment : ).

With #264 to implemented soon, it's a bit unclear where should we gather this information, given that registry will have more information that the director needed for this metric, but registry are unware of if an origin/server is live / running in the system.

Not started as of 01/05/2024. Given the other PRs on my todo list, I won’t be able to work on it until 1/8, so it’d better slip into 7.5

I figured that we already have the ability to count number of servers in the federation, just run the query prometheus/query?query=count(up{server_type="Origin"}) for number of origins or prometheus/query?query=count(up{server_type="Cache"}) for caches. The only thing that's missing is the institution label where I plan to add via an extra call to registry to get when origin/cache register at the director
I propose to add a new metric to count # of namespaces in the federation, labeled by server name.

Moved to 7.6 with pending questions and also the advertisement struct change.

After talking to Brian, we agree to proceed with #1006 . Close this issue


Using Jquery steps with Flask

I am using Jquery steps, I want to send the data to back-end flask and return the results on last steps. Is it in the scope of jquery steps.

@asgar-hussain You need to be able to submit the form for that. You can do that within onFinished
var form = $("#example-form");
form.validate({
    errorPlacement: function errorPlacement(error, element) { element.before(error); },
    rules: {
        confirm: {
            equalTo: "#password"
        }
    }
});
form.children("div").steps({
    headerTag: "h3",
    bodyTag: "section",
    transitionEffect: "slideLeft",
    onStepChanging: function (event, currentIndex, newIndex)
    {
        form.validate().settings.ignore = ":disabled,:hidden";
        return form.valid();
    },
    onFinishing: function (event, currentIndex)
    {
        form.validate().settings.ignore = ":disabled";
        return form.valid();
    },
    onFinished: function (event, currentIndex)
    {
        alert("Submitted!");
    }
});

More examples here http://www.jquery-steps.com/Examples#advanced-form


Fix service name lookup on TCP routers

What does this PR do?
Fix the service name resolution to IP addresses for TCP services referenced by hostname/DNS.
Fixes #7167 #7354
Motivation
Currently, for TCP routers the service name resolution is done during the router setup (Proxy object) and never updated again, in this case, any changes to the actual addresses behind that hostname will have no effect unless the Traefik instance is restarted.
Moving the resolution to the connection handling allows for a fast reaction upon address changes, every connection to the server will trigger a lookup of the hostname.
Pros of this approach:

Fast reaction to changes in addresses (per connection)
Not dependent on pooling (either by the provider config or independent check)
Should work for all providers (tested with File and Kubernetes)

Cons:

Adds an address lookup per connection

More

[ ] Added/updated tests
[ ] Added/updated documentation

Additional Notes
I didn't notice any meaningful impact in latency while testing (basic checks with ncat), compared with v2.2, but a more intensive performance check with thousands of short-lived connections could be done if needed.

in #7167 traefik updates IP when config is reloaded, it just ignores some of k8s Endpoints events. It explained here. The problem will disappear once reloading of the config in k8s provider is fixed.

@kopaygorodsky yes, but a config reload is not a solution when working with a DNS name external service, as kubernetes will never trigger a config update since the object never changes (service definition without endpoints). In this case you would have to rely on restarting Traefik or deleting and creating the router again to update the actual addresses.

I thought we can watch for Endpoints resource, can't we?

@kopaygorodsky an external name service generates no endpoints in kubernetes. You can create them manually but it would be ignored by Traefik in this case anyway.
For external name services Traefik resolves the externalName key to an actual TCP address that will be stored at the router service servers list. That was done only during the router setup and this PR changes it to the router's proxy server inside Traefik responsible for handling requests, thus resolving the name for each new request.
TCP based services will usually connect once and keep the connection until the work is done, like databases for example, and for that reason I guess it won't have a negative impact. Even for multiple distinct requests I did not see a noticeable increase in latency, but I would like validation from more community members or senior maintainers.

oh, I got where I was wrong. Thanks a lot for answering :)

@ddtmachado Thanks for your PR, It looks promising!
I wonder if it wouldn't be a more understandable behavior to not have a default value for addrLookupCache. If the user don't specify the addrLookupCache option, it would just resolve the address for every connection. If this option gets configured, the users chose its value and is aware of the side effects. As it's an optimization, it doesn't have to be enabled by default.
Also, what do you think about changing the addrLookupCache into a duration? If there's no default value it doesn't have to be a pointer to an int and we don't need to allow negative values.

@ddtmachado After discussing with the team, we think having a cache adds more complexity than it helps. Right now the current implementation has some concurrency issues around the p.target and p.resolvedAt. Fixing concurrency while still avoiding simultaneous resolves and connection bottle neck is doable but would make the code even more complex. The only cases where you want to use this resolution is when using the file provider or with ExternalName services on Kubernetes. As it is an edge case, not caching resolutions is less of a problem.
What do you think about doing the resolution only if the given address is a service name? The most common use case won't be impacted. If users want to avoid stressing too much there DNS server it is still possible to rely on a local DNS cache using Kubernetes nodelocaldns or more generally with dnsmasq.

@jspdown yes, it is not synced but it does not cause any race condition as well, in the worst case scenario it will trigger more lookups than necessary. Being so I agree that the complexity to add synchronicity is not worth in this case but I wouldn't mind keeping it like that.

What do you think about doing the resolution only if the given address is a service name?

I wouldn't mind doing this instead, but then I would prefer to do it outside the Proxy to properly resolve the name into multiple ip addresses and avoid relying on the DNS round robin to load balance, wdyt?

@jspdown Actually forget my suggestion on doing it outside as we would have to spawn another routine to keep it updated through time and it would be more complex than fixing concurrency 😅

@ddtmachado You are right there's no race condition. It's just that this concurrency issue shows us that implementing a DNS caching mechanism is not that trivial. Instead of committing right now on how we want to configure the cache we would like more user inputs. The workaround (installing a DNS cache) currently provides a more capable cache than ours, so we should let users open issues describing their use-case and iterate on it.

woohoo! Thx for your effort!


LROs default to Location instead of Azure-Async-Operation (which is the correct behavior according to the docs)

Describe the issue or request
(Note this issue applies to recent track 2 C# SDKs.)
According to the autorest docs here the default value of final-state-via should be "azure-async-operation" if not specified, but it looks like it's defaulting to "location" for C# operations. Is there a reason for this discrepancy? This will generate a lot of unnecessary ARM reads, which are already precious due to the low limits.
A teammate of mine hit this while testing out the compute track 2 SDK and resizing several scalesets. The response has the right headers:
Azure-AsyncNotification:Enabled
Azure-AsyncOperation:https://management.azure.com/subscriptions/12345/providers/Microsoft.Compute/locations/southcentralus/operations/f7608d88-97bf-4d78-9a4b-78e8f2b7b389?p=c24205a0-e7ae-41d0-9985-0ea829801910&api-version=2021-03-01

But the operation is set to OperationFinalStateVia.Location, so it ends up polling the VMSS every second instead of polling the operation. (See this line in GitHub)
Describe your ideas for solutions
The default should be changed to match the docs, or the docs to match the actual behavior. If the latter, service teams will need to be better about setting final-state-via, as failing to do so will generate potentially large numbers of extra ARM reads.

This is an issue in autorest, the specific API you are talking about has no lro options defined.  This causes it to default to Location in autorest from here.  @AlexanderSher do you know whether the docs are incorrect or if the implementation is incorrect?

It seems that the default header to check should be azure-async-operation. See the spec here:

azure-async-operation - (default if not specified) poll until terminal state


correcting a minor typo with section name

Ah, thank you for the correction !

Fix released on PyPi, thanks !

Thanks Eliot.


Renaming alert doesn't change it

Renaming an alert reports back as successful but doesn't actually change the resource.
e.g.
module.********: Modifying... (ID: 1234)
  name: "name-NEW" => "name"
module.******: Modifications complete after 9s (ID: 1234)

You discovered what I discovered around the same time. Check out https://github.com/appoptics/terraform-provider-appoptics/pull/45/commits/e6decca5a87f03b23d75c8f2ab099d33f1450dbd#diff-67f991c94bb36946aa55d5c4d54ec3a7R403.

This should be fixed from @pcn's latest release: https://github.com/appoptics/terraform-provider-appoptics/releases/tag/0.4.0-beta.1 - can you give it a shot and let us know?


stop inserting envelope hash for intoto:0.0.2 types into index

Fixes: #1164
Signed-off-by: Bob Callaway<EMAIL_ADDRESS>

should cherry-pick this back into release-1.0 branch


Beam provider added

Requested by #2

Looks great - I'll merge it in now - thank you very much for your contribution!


Add interactive plots with Plotly

Description
This pull request adds a new module to the dashboard for showing interactive plots in the browser.
Motivation and Context
See issue #162
Checklist:

[x] I am familiar with the Contributing Guidelines.
[x] I agree with the terms of the Contributor Agreement.
[x] The changes introduced by this pull request are covered by existing or newly introduced tests.
[x] The package documentation and framework documentation in signac-docs are up to date with these changes.
[x] I have updated the changelog and added any related issue and pull request numbers for future reference.

Do you consider that a new test needed for the module?

Do you consider that a new test is needed for the module?

Explicit tests aren't needed -- the testing framework is pretty weak right now, and this should be automatically covered by the "kitchen sink" tests. However, an example would be good to include.

I agree with Bradley. This seems very useful. Thank you for the implementation! If you could include an example, I'll also test it locally on my machine.

I'm not familiarized with Flask, so when it came to add the routes and assets I just used other modules as reference (I think it was the DocumentEditor module).
When adding the Plotly CDN URL I realized that the "filename" attribute in line 104 was not used, apparently, so I removed it. Is this ok?


Extract layers module from rpc crate

Closes #8161

@mattsse any updates on this?

@emhane any idea why some checks are failing?

try locally
rm Cargo.lock
cargo clean
cargo c

then commit the updated lock file, and push. that should do it. @rupam-04

try locally
rm Cargo.lock
cargo clean
cargo c

then commit the updated lock file, and push. that should do it. @rupam-04

worked, ty


Adding all-contributions

@all-contributors please add syauqi for code

@all-contributors please add  for

@all-contributors please add syauqi for code

@all-contributors please add syauqi for code

@all-contributors please add arsandev for design

@all-contributors please add msazzuhair for code

@all-contributors please add arsandev for design

@all-contributors please add arsandev for design


Add Logical Type for Age and AgeNullable

Closes #778 by adding a logical type for age.

How do we want to test this logical type?

@jeff-hernandez We can do minimal test for this and make sure we can init with Age as well as making sure we can set a Logical Type of Age.

I'm not 100% versed on when we do and don't need a relationship in our type system between two logical types, but I think we might want Integer to be a parent to Age and IntegerNullable to be a parent to AgeNullable in our DEFAULT_RELATIONSHIPS


Create an Implementation of 2048 using coffeescript

use the coffeescript programming language to create a basic 2048 program.

yes please do! that would be excellent!

Hello i finished an implementation , could you tell me how to push ?

I'm having an error i think i need your permission ?


[MM-17603] Added null check to label generator on suggestion list

Summary
This PR fixes a possible null on the current label for the suggestion list, causing a JS error. It stops the toLowerCase() call if the object is null.
Ticket Link
https://mattermost.atlassian.net/browse/MM-17603

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3443.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Test server destroyed

@devinbinnie
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
!!! 'upstream/release-5.15' not found. The second argument should be something like upstream/release-0.21.
    (In particular, it needs to be a valid, existing remote branch that I can 'git checkout'.)


Work with 1st client to prepare opportunity statement

Cloud Operations has confirmed they can be the first client.

Need to have the opportunity template completed so they can help test this piece.


Do the model of object detection support on mobile?

As the title mentioned, do it support on Android?I saw the segmentation model works on ios, do anyone tried the object detection model Android mobile?Thanks

I have not tried it myself. But we welcome pull requests.


Can't install Gadael

Hello.
I cannot install the software. I've tried with Ubuntu 17.10, Ubuntu 16.04, Ubuntu 16.10, Debian, Fedora and CentOS.
None of the installation methods works.
The main problem is that nodejs-legacy package has dissappeared, now is integrated with nodejs. I've tried to force installation but initialization scripts don't work.

@nOundO I got passed that error by using a base docker image preinstalled with npm.
docker run --name gadael -it -p 3000:3000 node:9.11.1 bash

# inside the container...
apt install mongodb git g++ gyp gcc make
npm install -g bower

adduser gadael
su - gadael
git clone https://github.com/gadael/gadael
cd gadael
npm install
bower install

however i get a different issue later on, bower stumples on one of the dependencies: moment-duration-format doesn't have a dev branch anymore it seems

follow-up: after editing bower.json to a valid version for moment-duration-format such as 2.2.2 then bower install completes
However the service fails to start...
$ node install.js gadael "ACME" EN
internal/modules/cjs/loader.js:550
    throw err;
    ^

Error: Cannot find module '../config'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:548:15)
    at Function.Module._load (internal/modules/cjs/loader.js:475:25)
    at Module.require (internal/modules/cjs/loader.js:598:17)

you can copy the config.example.js in config.js and update the file with your settings, i thought "node install.js" should have done this for you, maybe this is an access right problem?

Hello,
I tried deploy in aws ,i got the error as cannot find the module.
And when i tried in heroku, the error is push rejected failed to compile node js app.

Could you help me to resolve these errors.

Also hit the Error: Cannot find module '../config' error. As @polo2ro said, just copied config.example.js as config.js and it worked (dispite having to kill the script):
(node:28464) DeprecationWarning: current Server Discovery and Monitoring engine is deprecated, and will be removed in a future version. To use the new Server Discover and Monitoring engine, pass option { useUnifiedTopology: true } to the MongoClient constructor.
(node:28464) DeprecationWarning: collection.remove is deprecated. Use deleteOne, deleteMany, or bulkWrite instead.
Database gadael created, please use `node app.js 3000 gadael` to start your Gadael instance

^C

After that, when i run node app.js 3000 things won't work with this error message:
node:28873) DeprecationWarning: current Server Discovery and Monitoring engine is deprecated, and will be removed in a future version. To use the new Server Discover and Monitoring engine, pass option { useUnifiedTopology: true } to the MongoClient constructor.
Error: createDb: Database already initialized with company object: gadael
    at gadael_loadMockModels.then.then.count (/home/nemesis/dev/oss/gadael/api/Company.api.js:268:12)
    at process._tickCallback (internal/process/next_tick.js:68:7)

I've tried commenting the config.mongodb.init: true option in config.js with no luck, still getting the same message. I'ts due to the saveCompanyDoc function counting if company already exists.
Any idea on this?

Ok, just got it. After copying config.js and running the node install.js [...] command, you have to set config.mongodb.init specifically to false, isn't enough to comment it out.
Now it's working :)