Build our own golang-build container

Use our own golang-build container for building go code into a
binary. The original idea came from felix project [1]
[1]. https://github.com/projectcalico/felix/blob/master/Makefile

So --build-args in docker goes from 1.9.0 version, but on circleci it seems
Docker
We're using a build of Docker which is patched to work in the CircleCI build environment.
The current version is 1.8.2.
We also pre-install Docker Compose 1.5.1

the question is should I re-work my patch, or there is some change to update circleci test image

@SergK Does this enhancement solve a specific problem?
I agree that it is sensible for Calico repositories to build a golang build container in similar ways; but on the other hand we don't want to enforce uniformity across our repos.  So I'd like to understand if you are addressing a specific problem here.

For now one can have build issues for different calico components running on the same node, because of incorrect permissions in $HOME/.glide directory:

Try to build calico-containers, which for now build code under the root (https://github.com/projectcalico/calico-containers/blob/master/Makefile#L359)
Then try to build containers from felix repo and it fails, because in felix we run build under CI user
https://github.com/projectcalico/felix/blob/master/Makefile#L284

Of course, if CI user is root - no issue exists ... but on desktop, the issue is still exists, since in the most
cases users don't work under root
Definitely we can add simple chown UID:GID -R /.glide command in container
The rest issues are less important but I'd like to name them:

For example, we have a rather good feature https://github.com/projectcalico/calico-containers/pull/1342 and it can be implemented on other calico projects without any additional efforts, since we will have the same go build way
Moreover, probably we can move/create some component like calico/build-go-code container (put it for example in felix) and deliver to all calico components but simple pulling, so we will have the single point to configure calico/build-go-code
We don't need rely on upstream dockerepo/glide and can have our own fully customizable (with openssh out of the box, etc) container

@tomdee this sounds like your area of expertise?

I have a couple of comments, but I this more @tomdee and @fasacx area of expertise.

Fixing up the permissions issue with the $HOME/.glide is a good thing - it's certainly something I've run into in the past!

We generally name our containers calico/  so calico/build-golang or something akin to that would be preferable.

This container would also be used for testing I believe, so we'd need to get rid of the test container that is also in this repo (See TEST_CALICOCTL_CONTAINER_NAME in the Makefile)

Is it necessary to specify the WORKDIR in the docker file since you are now passing that in as a Docker run argument?

@robbrockbank FYI, we're using a convention of calico-build/<name> for build containers in felix to distinguish them from containers that we push.  That may or may not be a good distinction.

In case anyone needs the background as to why I made the Felix build jump through some hoops to run its docker containers as non-root in the first place:

It avoids leaking root-owned files and breaking permissions (i.e. breaking the glide cache apparently).  I didn't know about the glide cache explicitly, but I knew that if I ran the build as non-root I wouldn't run into that type of issue.
It protects you against some mistakes (rm -rf is a tad more powerful as root)
It reduces your attack surface (principle of least privilege); go build doesn't need root permissions so we shouldn't give it to it.  This is a bit academic, if someone's managed to sneak code into our build scripts then they could run as root via docker anyway.

The glide permission issue is definitely a problem that we should fix ASAP.  But I'm not sure (at least in the short term) whether than means we just stop mapping the .glide dir into builds in the short term, or fix permissions with a chown or move to a build container with correct users.
I'm cautiously in support of a single build container, but if it's going to be shared but a bunch of repos it's important that we get it right.

Where do we build it? In each repo or in one and then push it?
Do we use it just for building? Or for vendoring too? Or for running tests too? What about when different repos need different test utilities or libraries, will it end up getting bloated?

I'll keep pondering the best way forward but I am keep to keep this moving forward.

So AFAICT, this PR is just focused on fixing the vendoring permissions issue whereas there seems to be an expectation that the "build" container will be used not just for vendoring but all building and testing.

Ok folks, try to summarize from what we have.
The issue I'm trying to resolve:

Permission issues in ~/.glide cache
Allow to provide pre-populated /vendor directory. In isolated environment (no access to 3rd party repos) /vendor directory can be pupulated
outside of calico build process, then we can just put it and update target touch vendor/.up-to-date manually,e.g.:

cp -r $CALICO_PROJECT/prepopulated_vendor $CALICO_PROJECT/vendor
touch $CALICO_PROJECT/vendor/.up-to-date
cd $CALICO_PROJECT
make build-containerized

Simple way:

Add simple chown
Update targets, to support vendor/.up-to-date

Long way:

Implement single build-go container for all calico projects, but this looks for me not so simple and obvious, since for example GID and UID can be different for CI user in different systems.
Implement build-go container per project.

So if Simple way is acceptable for now I can prepare small change that will have less impact on current build-go way, but will resolve the above issues

I've created less destructive change https://github.com/projectcalico/calico-containers/pull/1387

I think we can close this PR in favor of https://github.com/projectcalico/calico-containers/pull/1387


AnimationTree with two states and expression-driven auto transitions prints _transition_to_next_recursive warning

Tested versions
I've tested the following Godot version:

v4.0.stable.official [92bee43ad] (works)
v4.1.stable.official [970459615] (works)
v4.1.1.rc1.official [e709ad4d6] (immediate warning)
v4.1.1.stable.official [bd6af8e0e] (immediate warning)
v4.2.1.stable.official [b09f793f5] (immediate warning)
v4.2.2.rc1.official [c7fb0645a] (immediate warning)
v4.2.2.rc2.official [c61a68614] (immediate warning)
v4.2.2.rc3.official [16a8334b8] (delayed warning)
v4.3.dev6.official [89850d553] (delayed warning)

System information
Godot v4.3.dev6 - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1070 Ti (NVIDIA; <IP_ADDRESS>22) - Intel(R) Core(TM) i7-4790K CPU @ 4.00GHz (8 Threads)
Issue description
An AnimationTree with a state machine and only two states/animations that use transitions with advance_mode set to "auto", will report a "_transition_to_next_recursive" warning if the advance_expression allows transition, even if the switch_mode is set to "At End".

Going from "idle" to "fire" happens when the advance_expression of triggered evaluates to true. This is driven by a simple button press in the UI:

Going from "fire" back to "idle" uses the expression not triggered or automatic.
If automatic is true, this results in "automatic fire", continuously switching between the "idle" and "fire" states/animations, and works as expected.
Main:<CanvasLayer#28168946939>::_fire() -> pew pew! (1983)
Main:<CanvasLayer#28168946939>::_fire() -> pew pew! (2750)
Main:<CanvasLayer#28168946939>::_fire() -> pew pew! (3516)

Godot still prints the warning:
W 0:00:02:0237   _transition_to_next_recursive: AnimationNodeStateMachinePlayback: parameters/playback aborts the transition by detecting one or more looped transitions in the same frame to prevent to infinity loop. You may need to check the transition settings.
  <C++ Source>   scene/animation/animation_node_state_machine.cpp:909 @ _transition_to_next_recursive()

This was originally introduced by PR 79141: Fix infinite loop state check in AnimationStateMachine in 4.1.1-rc1 and would print an immediate warning.
PR 89575: Move the line of infinity loop checking in AnimationStateMachine added in 4.2.2-rc3 modified the behavior, and the warning is only printed if the button is held down.
It seems the recursion detection doesn't take the switch_mode "At End" into consideration.
The issue can be "worked around" by introducing an intermediate state/animation ("trigger") and updating the advance_mode for the transitions to use that one instead:

It also doesn't occur if the idle transition is set to "At End":

However, that means the "idle" animation has to be very short for it not to delay the switch back to "fire".
Previous related issues:

issue #82935: Spurious animation transition warning: "aborts the transition by detecting one or more looped" for switch_mode = at_end
issue #89520: AnimationTree abort transition to stop infinite loop

Steps to reproduce
The MRP is setup to reproduce the issue. Hold down the "fire" button (brief button presses will not trigger it in 4.2.2-rc3 and later) to see the warning
Minimal reproduction project (MRP)
The MRP repository: animation_tree_transition_issue.
The MRP sets up a minimal AnimationTree state machine entirely controlled by the transitions' advance_expression.
The transitions' advance_mode is always set to "auto".
It's a firing mechanism for a gun, using triggered and automatic properties as part of the advance_expression.
The state machine defaults to the "idle" state/animation, until the triggered property results in a transition to the "fire" state/animation.
If the trigger is released or automatic is true, the state will transition back to "idle".
Holding down the "fire" button in the UI will repeatedly print "pew pew".
This functionally works but results in a _transition_to_next_recursive warning printed by Godot starting with 4.1.1-rc1.
A change introduced in 4.2.2-rc3 results in the warning only if the button is pressed for longer.

For me, taking off the looping for all my animations partly solved this. The animation didn't play the first time in-game but on the 2nd and always the 2nd time it would always play normally (except for one finger being offset which is weird because it's not offset in AnimationPlayer, only offset in AnimationTree). But now all the animations I want looped are no longer looped.


Added patient on device1, App kill required to updated patient list on device2

Describe the bug
Added patient on device1, App kill required to updated patient list on device2
Component
demo app
Precondition
Patient added on device1
To Reproduce
Steps to reproduce the behavior:

On device2, Go to 'demo app'
Search for the patient added on device1
3.Observe behavior
Kill app and relaunch
Search for patient added on device1
Observe behavior

Expected behavior
Patient should get reflected on device2 without app kill
Smartphone (please complete the following information):

Device: Lenovo tab
Android version: 10

@PallaviGanorkar , could you please try and recreate this bug with a gap of 3 to 4 minutes before using the second device.

This is expected behavior as Sync API is called in application when

on every launch
after launch poll is configured on every 15min.


chore: update go work example go toolchain

Description
Align with go.mod

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against the correct branch (see CONTRIBUTING.md).
[ ] Linked to GitHub issue with discussion and accepted design, OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/).
[ ] Added relevant godoc comments.
[ ] Provide a conventional commit message to follow the repository standards.
[ ] Include a descriptive changelog entry when appropriate. This may be left to the discretion of the PR reviewers. (e.g. chores should be omitted from changelog)
[ ] Re-reviewed Files changed in the GitHub PR explorer.
[ ] Review SonarCloud Report in the comment section below once CI passes.

Lgtm, one side question is, does anyone currently use this? (Guess tho it doesnt hurt having it around either way!)

I use it!


[FEATURE REQUEST] Allow users to prioritize challenges with cooperative work

Moved from maproulette/maproulette2#919
Let us say you have a challenge where some tasks with fixes that are almost certain to be right, and then some tasks where the user has to make a decision on their own.
An example would be a challenge fixing New Zealand waterway directions (the LINZ import done for them didn't properly directionalize the ways).
The challenge generator uses SRTM data (~90m GSD, IIRC), which means that most of the time the new waterway direction is correct, but there may be edge cases. Often, the user will be able to overlay the LINZ topo map and just apply the change in JOSM after finding the relevant contour lines.
I would like to be able to prioritize those since they are (a) low hanging fruit and (b) the mapper can do them fairly quickly.

If you're able to add a property to the tasks that would distinguish the 'low hanging fruit' from the rest, you could filter on that. See docs. But I may be misunderstanding what you're trying to accomplish?

I think I addressed your question, so I am going to close this, but apologies if I haven't and feel free to re-open in that case.


Update Maven Deploy URL

We recently moved some things around and as a result several URLs
changed.
Redirects are in place and most users will not need to make any changes.
We have already tested and confirmed that maven access works just fine
via the redirects.
However, maven will not allow you to deploy through the redirect so
projects using old deploy URLs will need to be updated.

Awesome, thanks!


fix test case

Codecov Report

Merging #60 (1a582b8) into main (85ad6a5) will increase coverage by 0.15%.
The diff coverage is 82.35%.

@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
+ Coverage   32.80%   32.95%   +0.15%
==========================================
  Files          37       38       +1
  Lines        5457     5474      +17
==========================================
+ Hits         1790     1804      +14
- Misses       3667     3670       +3

Flag
Coverage Δ

rust
32.95% <82.35%> (+0.15%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/error/mod.rs
22.91% <0.00%> (-1.53%)
:arrow_down:

src/lib.rs
0.76% <ø> (ø)

src/etl_node/openrpc_parser.rs
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Landing Page Inspiration

https://www.figma.com/file/34YX1uk6ZVhQWVzLyjzUdG/Twindle?node-id=0%3A1
Team, This issue is to encourage use of figma for our design part and make a thread for landing pages of other websites which we can be inspired.

Amazing work @JoelVinayKumar 🔥. Also look Dribbble for some landing page inspiration. Keep up the good work.

This is lit 🔥 I'm on board!

This is dope.  I can implement this design with React

@Mar-code We are not using React or any big library. Just plain HTML, CSS, and JavaScript.

Ohh,  alright that makes sense.

This is very light-weight UI. Doesn't take much to code in HTML or CSS.

https://dribbble.com/shots/13955707-Candidate
https://dribbble.com/shots/14267126-Travel-Landing-Page-UX-UI-Design
https://dribbble.com/shots/14375856-landing-page-homepage
https://dribbble.com/shots/14359951-Fitness-Landing-Page
https://dribbble.com/shots/13934987-Ekokubki-Landing-Page
https://dribbble.com/shots/13023346-AI-Dashboard-Website-Design
https://dribbble.com/shots/14241343-marketing-web-ui-landing-page

https://www.figma.com/file/34YX1uk6ZVhQWVzLyjzUdG/Twindle?node-id=0%3A1
Team, This issue is to encourage use of figma for our design part and make a thread for landing pages of other websites which we can be inspired.

I loved your 'dark mode' icon!

I am late for this but here is my design. 😅

I am late for this but here is my design. 😅

Updated the colours.

Updated colours

Landing page finalized by @JoelVinayKumar


Autocompletion improvements. Shorter first. Case matching priority.

Shorter First
If I type "EM" I get suggestions:

EntityManagerExtensions
EntityManager

A definitively better way would be to suggest in the opposite order here. Why? Because in the case above if I need the "EntityManager" I can't get to it by typing another letter, I have to move down with an arrow or type in the whole word "EManager" before it prioritizes "EntityManager" over "EntityManagerExtensions".
And if the shorter matching word is prioritized in the suggestions, we will be getting this:

EntityManager
EntityManagerExtensions

And this means that we can go immediately to what we need in any case. "EM" gives EntityManager. "EME" gives EntityManagerExtensions. So this is in my opinion always the preferred way to order the suggestion, shorter first.
Case Matching Priority
If I type "VI" (with capital i) I get very unpredictable results:

ADBannerView
NetworkView
SceneView
VectorInt

Logically what should come first is VectorInt otherwise I wouldn't type "VI" if I would need views I would type "Vi" (with lowercase i) and I should get this:

SceneView
NetworkView
ADBannerView
VectorInt

Why? Because logically case matching "Vi" appears first in "SceneView" (characters 6 and 7), then in NetworkView (characters 8 and 9), then "ADBannerView" and then not case matching "VectorInt"  But instead I'm getting this again:

ADBannerView
NetworkView
SceneView
VectorInt

But the main point is case matching. If I want "NetworkView" I go "NV". If I want "VectorInt" I should be able to just type "VI". So for "VI" this should be the order:

VectorInt
SceneView
NetworkView
ADBannerView

Thank you. Please let me know what you think. I've been sitting on this suggestion for years hoping that someone else will take it to the team, but here I am finally posting. Wasn't that hard :)
One last thing: I just noticed that suggestions, after moving away from a word and then coming back and retyping one letter, are different then if I wouldn't move or would retype two letters. Same word "VId" but different suggestions. Slight inconsistency there.

The sort order of recommendations is controlled by the extension that provides completions.
For what language should the above recommendations apply?

I would apply it to other languages too, although I personally don't use them.

Then can you please report the issue against the C# extension, this extension defines the sort order for c#.


Remove containerNoExploding

This PR fixes the alignment issues with revoked device and coin icons by removing the containerNoExploding styles. It also fixes issues with those message icons clipping on mobile by removing the marginLeftTiny style.

does this look correct on desktop/mobile/with exploding/with revoked/ orange line?

@chrisnojima desktop mostly isn't affected by these changes but the padding stuff doesn't change how desktop behaves. mobile looks fine

@keybase/react-hackers bump


feat: ESLint and Prettier added

ESLint and Prettier added to the project.

@all-contributors please add @syedmuhammadabid  for code


Bug: Navigating with keys in the Column Layout crashes the program

Description
While in the Column layout, using left and right arrow keys to traverse the file structure causes the program to exit with a 'Something went wrong!' error, and induces a catastrophic failure.
This bug is present on the Preview build (v <IP_ADDRESS>) and not on the stable release (v <IP_ADDRESS>)
Steps To Reproduce

Enter the Column layout
Enter a folder
Select either a subfolder or a file within that folder
Press the right arrow key
The program will exit with a catastrophic error

Alternatively, if you are in a subfolder and wish to return to a parent folder, pressing the left arrow key will crash the program.
Steps to fix

Fix crash that occurs when navigating the folder structure using the directional keys in the column layout.

I would expect this functionality to mirror that of Finder's (Mac OS) column view.
In the case that the selected item is a file, and not a folder, I think nothing should happen when pressing the right arrow key when it is selected.

Files Version
<IP_ADDRESS> (Preview)
Windows Version
Windows 11 22H2 22621.819
Relevant Assets/Logs
debug.log

Merging with #10109


Update environment-variables.md

Refactors to add setx syntax for SPEECH_REGION as well.
https://learn.microsoft.com/en-us/azure/cognitive-services/speech-service/get-started-speech-to-text?tabs=windows%2Cterminal&pivots=programming-language-csharp
I am following this tutorial and it references the SPEECH_REGION environment variable we should have set. Unless I'm missing it, we are missing the step to actually set the SPEECH_REGION environment variable there and then subsequently telling people to rely on it. That's causing my code to fail with a somewhat obscure error.
Please make sure to go and update the website I've linked to also include instructions to better understand the SPEECH_REGION variable and what the expectation is there.

Learn Build status updates of commit 3504e62:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/cognitive-services/Speech-Service/includes/common/environment-variables.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@eric-urban,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@eric-urban I commented on the PR somewhere, I thought, but I seem to have lost it. We do seem to be telling the user to set the variable in the comment that's clearly defined on the PR (which I didn't add, but the PR seems to indicate I did somehow - the comment is already on the production site).
I think all this PR needs is a decision on whether or not you guys want to add the setx command or any clarity.

@willWashington - Thanks for the recommendation! I've created a new PR for multiple platforms in parallel. https://github.com/MicrosoftDocs/azure-docs-pr/pull/223052

#please-close


Add option to set mod note

This will only support the Native Notes feature. If Toolbox support is necessary, the mods can either enable Toolbox's beta Native Notes or use the Notes Bridge app.

Supported as of commit 02c3f15b59d3d81464dc6e221df984f3de48c628, still needs config generator support.


Can't execute any requests to db

InfluxDB shell version: 1.6.0
I have 10 databases , all working fine execept one DB, when i try:
use pft_pnl
show measurements
or
drop database pft_pnl
I see no errors or results - just empty console in waiting process
DB size in data folder ~1GB
Other DB have size more then 300GB and all working fast
and i can't make http request to this db , i see - timeout error (only for this DB)

How can i fix this?
How can i delete this db by hand (from data and meta)?

Maybe problem in idx or tsm files?

if you run a show databases command does pft_pnl show up?
you can use the influx_inspect tool to inspect the shards containing your data
https://docs.influxdata.com/influxdb/v1.6/tools/influx_inspect/#sidebar


disabel update docker form https://download.docker.com/linux/ubuntu

Every time I run apt update, it tries to connect to https://download.docker.com/linux/ubuntu server. Doesn't snap docker have an entry from snap that receives updates from the main address?
By the way, how can I disconnect from https://download.docker.com/linux/ubuntu?
so it return duplicate source
Target Packages (multiverse/binary-all/Packages) is configured multiple times in /etc/apt/sources.list:34 and /etc/apt/sources.list.d/original.list:34
i solved it with this
https://askubuntu.com/a/762815/968481

Hi, thanks for reporting this.
Actually, snap services don’t use the apt tool or the Docker archive.
Did you install Docker from the Docker archive or as a snap?
It seems you installed Docker from a deb package, and your system has some residual configuration (such as the apt repository for Docker).


Waybar no longer appers after Arch update

Today I ran a system update on Arch and restarted the machine. When I logged back in the waybar was no longer visible. I have no clue which of the updates did this, but I don't think I saw sway or waybar among the updates. I updated my packages on my laptop and the same thing happened.
Waybar itself does not seem to crash. I've tried to start waybar from the console with --log_level trace but it does not print any errors.
[~] $ waybar --log-level trace
[2020-04-03 23:24:23.563] [info] Resources files: /home/magnus/.config/waybar/config, /home/magnus/.config/waybar/style.css
Unknown option --log-level
[2020-04-03 23:24:23.566] [debug] Output detected: HDMI-A-1 (Unknown U3277WB)
[2020-04-03 23:24:23.569] [warning] For a functional tray you must have libappindicator-* installed and export XDG_CURRENT_DESKTOP=Unity

My style and config files as per now.
https://gist.github.com/CodeGradox/4fee8406a676c9b2fcd745bc4c21e633
I'm running Linux version 5.4.30-1-lts with Waybar version v0.9.1

Must be related to https://github.com/Alexays/Waybar/issues/642.

Same here, I thought I am going mad here ...

Same here, I thought I am going mad here ...

:D me too

Dups of #642


Rust UTP wip - DO NOT MERGE

Pull request purely for people to watch the Rust UTP port progress

r? @chandraprakash
(maidsafe_highfive has picked a reviewer for you, use r? to override)

utp_connections::test_small_stream unit test is now finally passing and is stable. Yay!

Excellent news Niall - have a beer :-)
On Thu, Jun 25, 2015 at 6:23 PM, Niall Douglas<EMAIL_ADDRESS>wrote:

utp_connections::test_small_stream unit test is now finally passing and is
stable. Yay!
—
Reply to this email directly or view it on GitHub
https://github.com/maidsafe/crust/pull/163#issuecomment-115332497.

--
David Irvine
twitter: @metaquestions
blog:  http://metaquestions.me

Nice one Niall !!!


spring jpa

spring jpa

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
13

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Many tests timing out flakily on macos chrome 44

These tests all timed out once on chrome 44, but none have been seen to time out twice.  Not suppressing them yet, since none have repeated.
  1 html/custom/attribute_changed_callback_test/fully_supported  Pass   Timeout
  1 html/custom/created_callback_test  Fail   Timeout
  1 html/custom/entered_left_view_test/disconnected_subtree  Pass   Timeout
  1 html/custom/js_custom_test  Pass   Timeout
  1 html/cssstyledeclaration_test  Pass   Timeout
  1 html/node_validator_important_if_you_suppress_make_the_bug_critical_test/DOM_sanitization  Pass   Timeout
  1 html/xhr_test/xhr  Pass   Timeout
  1 language/async_star_test/02  RuntimeError   Timeout
  1 language/async_star_test/none  Pass   Timeout
  1 lib/async/run_zoned7_test  Pass   Timeout
  1 lib/async/stream_periodic2_test  Pass   Timeout
  1 lib/async/timer_cancel_test  Pass   Timeout

Many of these seem to have something to do with async callbacks, although not all of them. stream_periodic2_test seems the smallest. But I don't see anything obvious, and I can't get any of them to fail. Maybe stream_periodic2 ought to be returning a future from the test function, but if that's required then it's not clear why it wouldn't have been erratically timing out all along.

There is a change in tools/testing/dart/test_configurations.dart to reduce the number of browser processes to processors/2 in this case, but it doesn't seem to help.

This is a browser from years ago.


Bump up Binarry Logger file format version

Due to new BuildEventContext.EvaluationID
It is okay to break people because the binary logger has not RTMed yet. After 15.3, each version bump needs to introduce some dispatching code so that the current version is backwards compatible. Worst case design is forking the reader / writer for each version number.

Includes changes from #2135 and #2129

@dotnet-bot test Windows_NT Build for CoreCLR please

@dotnet-bot test Windows_NT Build for CoreCLR  please


lowercase 'vfsStream' in composer files

Description
Composer gives warnings about uppercase letters in paths. It looks like this one was missed at some point - fixing across 3 package locations.
Checklist:

[X] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[X] Conforms to style guide

Looks good, thanks.
For future reference, commits need to be GPG-signed. You checked that box, but did not securely sign your commit:-/

Yes sorry about that, all my previous ones were and I’m confused why this one wasn’t. I was just researching and found git config commit.gpgsign true - I’ll give it a go for next time. Thanks for the fast response!

just setting the git config isn't enough ... the signing keys need to be registered with your github account, and yout local git user.email needs to match that on your github account too

Yes I have both of those in place already which is why I was confused that it didn't show up as signed - I think this setting was the missing piece. Will test now.

Okay you're right that was not as straightforward as I thought! Got it all set though:
https://github.com/MGatner/CodeIgniter4/commit/961ca68869f840e70af531a370f0dee906e3d7d4
I'll be sure to be verified next time. Cheers!


Lets stick to the react 16 version as it will be downward compatible too

@nitish24p Your thoughts?

Umm yes but what if someone has not updated to React 16

Cool then when we use enzyme we need to use both the adapters (enzyme adapter for version 16 & 15 both cool ) .
Yeah we should support both the version


HTML Interactive Form Validation support

https://webkit.org/blog/7099/html-interactive-form-validation/ & the demo http://codepen.io/cdumez/full/zoOZmZ/ offers additional cues to help fill in forms.
It would be a fantastic addition to this project to help people complete forms.

Yes, that does look nice, I would be happy to review a PR.


Added NuoDB to Databases

Pre-submission checklist:
Please check each of these after submitting your pull request:

[X] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
-- Closed source - linked to GitHub organization

[X] Is your project closed source or, if it is open source, does your project have at least 250 GitHub stars?
-- Not open source

[X] Have you picked the single best (existing) category for your project?
-- Database

[X] Does it follow the other guidelines from the new entries section?

[X] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
-- Added to Hosted Logos

[X] Does your logo clearly state the name of the project/product and follow the other logo guidelines?

[X] Does your project/product name match the text on the logo?

[X] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?

[ ] ~5 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Deploy preview for landscape failed.
Built with commit e29e78e5998b4da675e3af12828f7c37b46d2b69
https://app.netlify.com/sites/landscape/deploys/5d09476dcccc330008c060db

Implemented here: https://landscape.cncf.io/selected=nuo-db
Let me know if you need any changes.

Hi Dan,
Thanks for helping with this! The profile itself looks good.
Curious if you could remove the "-" between nuo and db in the URL path to
make it =nuodb. We keep it one word. Let me know if thats possible.
Thank you!
[image: Screen Shot 2019-06-19 at 9.39.25 AM.png]
Samantha Kelly
Digital Marketing Manager
www.nuodb.com
p: +1<PHONE_NUMBER>
c: +1<PHONE_NUMBER>
t: @nuodb https://twitter.com/nuodb
l: linkedin.com/company/NUODB https://www.linkedin.com/company/nuodb
The information contained in this email is confidential and proprietary and
intended for the sole use of the intended recipient. If the reader of this
message is not the intended recipient, you are hereby notified that any
dissemination, distribution or copying of this communication is strictly
prohibited. If you have received this email in error, please notify the
sender and delete all copies.
On Tue, Jun 18, 2019 at 9:29 PM Dan Kohn<EMAIL_ADDRESS>wrote:

Implemented here: https://landscape.cncf.io/selected=nuo-db
Let me know if you need any changes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cncf/landscape/pull/1280?email_source=notifications&email_token=AHF2JW2GYUTRWDI5NT6R6PLP3GDW3A5CNFSM4HZDQ4FKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYANEHI#issuecomment-503370269,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHF2JW3ARA54Z4OOKWTSWATP3GDW3ANCNFSM4HZDQ4FA
.

You're welcome. Sorry, can't help with the hyphen. We're using
https://www.npmjs.com/package/param-case which sets it that way
automatically.
Dan Kohn<EMAIL_ADDRESS>Executive Director, Cloud Native Computing Foundation https://www.cncf.io
+1-415-233-1000 https://www.dankohn.com
On Wed, Jun 19, 2019 at 9:43 AM Sam Kelly<EMAIL_ADDRESS>wrote:

Hi Dan,
Thanks for helping with this! The profile itself looks good.
Curious if you could remove the "-" between nuo and db in the URL path to
make it =nuodb. We keep it one word. Let me know if thats possible.
Thank you!
[image: Screen Shot 2019-06-19 at 9.39.25 AM.png]
Samantha Kelly
Digital Marketing Manager
www.nuodb.com
p: +1<PHONE_NUMBER>
c: +1<PHONE_NUMBER>
t: @nuodb https://twitter.com/nuodb
l: linkedin.com/company/NUODB https://www.linkedin.com/company/nuodb
The information contained in this email is confidential and proprietary and
intended for the sole use of the intended recipient. If the reader of this
message is not the intended recipient, you are hereby notified that any
dissemination, distribution or copying of this communication is strictly
prohibited. If you have received this email in error, please notify the
sender and delete all copies.
On Tue, Jun 18, 2019 at 9:29 PM Dan Kohn<EMAIL_ADDRESS>wrote:

Implemented here: https://landscape.cncf.io/selected=nuo-db
Let me know if you need any changes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/cncf/landscape/pull/1280?email_source=notifications&email_token=AHF2JW2GYUTRWDI5NT6R6PLP3GDW3A5CNFSM4HZDQ4FKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYANEHI#issuecomment-503370269
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AHF2JW3ARA54Z4OOKWTSWATP3GDW3ANCNFSM4HZDQ4FA
.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/cncf/landscape/pull/1280?email_source=notifications&email_token=AAXQYBUMGBBLXCKS342JR3LP3IZW3A5CNFSM4HZDQ4FKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYB5AAI#issuecomment-503566337,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXQYBT4J6BL46N6URZZRXDP3IZW3ANCNFSM4HZDQ4FA
.

Ok. No worries! Good to go then.
Thank you
Samantha Kelly
Digital Marketing Manager
www.nuodb.com
p: +1<PHONE_NUMBER>
c: +1<PHONE_NUMBER>
t: @nuodb https://twitter.com/nuodb
l: linkedin.com/company/NUODB https://www.linkedin.com/company/nuodb
The information contained in this email is confidential and proprietary and
intended for the sole use of the intended recipient. If the reader of this
message is not the intended recipient, you are hereby notified that any
dissemination, distribution or copying of this communication is strictly
prohibited. If you have received this email in error, please notify the
sender and delete all copies.
On Wed, Jun 19, 2019 at 9:52 AM Dan Kohn<EMAIL_ADDRESS>wrote:

You're welcome. Sorry, can't help with the hyphen. We're using
https://www.npmjs.com/package/param-case which sets it that way
automatically.
Dan Kohn<EMAIL_ADDRESS>Executive Director, Cloud Native Computing Foundation https://www.cncf.io
+1-415-233-1000 https://www.dankohn.com
On Wed, Jun 19, 2019 at 9:43 AM Sam Kelly<EMAIL_ADDRESS>wrote:

Hi Dan,
Thanks for helping with this! The profile itself looks good.
Curious if you could remove the "-" between nuo and db in the URL path to
make it =nuodb. We keep it one word. Let me know if thats possible.
Thank you!
[image: Screen Shot 2019-06-19 at 9.39.25 AM.png]
Samantha Kelly
Digital Marketing Manager
www.nuodb.com
p: +1<PHONE_NUMBER>
c: +1<PHONE_NUMBER>
t: @nuodb https://twitter.com/nuodb
l: linkedin.com/company/NUODB https://www.linkedin.com/company/nuodb
The information contained in this email is confidential and proprietary
and
intended for the sole use of the intended recipient. If the reader of
this
message is not the intended recipient, you are hereby notified that any
dissemination, distribution or copying of this communication is strictly
prohibited. If you have received this email in error, please notify the
sender and delete all copies.
On Tue, Jun 18, 2019 at 9:29 PM Dan Kohn<EMAIL_ADDRESS>wrote:

Implemented here: https://landscape.cncf.io/selected=nuo-db
Let me know if you need any changes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<

https://github.com/cncf/landscape/pull/1280?email_source=notifications&email_token=AHF2JW2GYUTRWDI5NT6R6PLP3GDW3A5CNFSM4HZDQ4FKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYANEHI#issuecomment-503370269

,
or mute the thread
<

https://github.com/notifications/unsubscribe-auth/AHF2JW3ARA54Z4OOKWTSWATP3GDW3ANCNFSM4HZDQ4FA

.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<
https://github.com/cncf/landscape/pull/1280?email_source=notifications&email_token=AAXQYBUMGBBLXCKS342JR3LP3IZW3A5CNFSM4HZDQ4FKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYB5AAI#issuecomment-503566337
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AAXQYBT4J6BL46N6URZZRXDP3IZW3ANCNFSM4HZDQ4FA
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cncf/landscape/pull/1280?email_source=notifications&email_token=AHF2JW4XEAA3YO4EYZMQ54TP3I2Z5A5CNFSM4HZDQ4FKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYB547Y#issuecomment-503570047,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHF2JW76SUS4GTTSK7CGCETP3I2Z5ANCNFSM4HZDQ4FA
.


allow to pass pathlib based Paths in save and load APIs

Closes https://github.com/iterative/mlem/issues/428

@skshetry Unfortunately the above suggestion regarding os.fspath and os.PathLike leads to a lot of issues (around 35 of them) with mypy which is weird since one can pass str to variables defined with os.PathLike.
No other issues, tests are unaffected. So keeping this PR as it is for now.

@skshetry Unfortunately the above suggestion regarding os.fspath and os.PathLike leads to a lot of issues (around 35 of them) with mypy which is weird since one can pass str to variables defined with os.PathLike.

I meant to only replace pathlib.Path with os.PathLike, not the union typehint: Union[str, os.PathLike].


Issue running RNN Encoder Decoder

When running single layer RNN I got this error
python3 -m sockeye.train -s English-train.csv -t French-train.csv -vs English-train.csv -vt French-train.csv --use-cpu  -o eng-fr --encoder rnn --decoder rnn --num-embed=16:16 --num-layers=1:1
FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type.
from ._conv import register_converters as _register_converters
[ERROR:main] Mismatch in arguments for training continuation.
[ERROR:main] Differing arguments: num_embed, encoder, num_layers, decoder.
No matter what I change i get the same error with more arguments
python3 -m sockeye.train -s English-train.csv -t French-train.csv -vs English-train.csv -vt French-train.csv --use-cpu  -o eng-fr --encoder rnn --decoder rnn --num-embed=16:16 --num-layers=1:1 --rnn-num-hidden=16 --num-words=16:16 --batch-type=sentence --batch-size=1 --max-updates=4 --checkpoint-frequency=2
FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type.
from ._conv import register_converters as _register_converters
[ERROR:main] Mismatch in arguments for training continuation.
[ERROR:main] Differing arguments: decoder, num_words, rnn_num_hidden, batch_size, max_updates, num_embed, checkpoint_frequency, batch_type, encoder, num_layers.
I want the fastest single layer RNN with attention with embeddings and few hidden units to see attention scoring impact.

Solved it by changing output folder


drive: fix escaped chars in documents during list

Use the right variable when constructing document file stems in list.
Fixes #2591

Thanks @B4dM4n :-)


Request timeout is to small

I'm debugging my API server with breakpoints, and your app is complaining
Abort / timeout

The request has been aborted manually or because of the conection timeout. There were no response from the server but the connection wasn't closed.

Versions
App: <IP_ADDRESS>-stable
Chrome: 50.0.2661.75
Platform: Windows (10)

Duplicate of #502


Team 15 Hazard Analysis Review: Ambiguous Effects of Failure for ML Annotation Pipeline

For your ML Annotation Pipeline component, your two hazards have the same ambiguous effects of failure. I think you can identify more specifically what and how users' learning outcomes are impacted by the failure modes. For example, "latency in annotation", can have an effect of failure related to how the annotations do not sync up to the video. Overall I think there should be a bit more clarity in the effects of failure.
Team 15 TA: @Lyuyang0130

resolved in: 68fbfae53dbfda0980220b49e69f32be92e37994


Removed type id from comments

This should fix issue #173. Because the UDI isn't available on the TypeModel, I've just removed the ID for now...
I also made the TypesMap dictionary case insensitive, so it doesn't have to call ToLowerInvariant() on every key lookup!

Haven't merged the PR directly, have split it into two commits - but it's merged, thanks !


Bookstore: connect to API

🤸🏻♂ Make use of the Bookstore API as  backend.
🤸🏻♂Use axios with createAsyncThunk to make HTTP requests to the backend
🤸🏻♂Fetch books from the Bookstore API after the component has rendered
🤸🏻♂Refactor add book and remove book features to persist changes in the server
🤸🏻♂application behaves the same way after the refactoring

Thank you


circular dependency in next\dist\lib\router\index.js

I am trying to build a zip file only containing the code I need for production deployment.  Smaller builds benefit nearly all production users: cheaper to store indefinitely, quicker to roll out after build finish, faster to load cold start in serverless, and for large projects it is easy to go over serverless dist limits without optimized distribution.    I believe rollout is the best option for getting an optimized build.
The good news, running rollup turns my 60m dist file into 6m.  I believe this concept will be a huge win for people who want their next.js app to start as quick as possible.
Bad news: When I try to run rollup on my next.js build, I get a circular dependency issue that I believe stops the result from running:
(!) Circular dependency: node_modules\next\dist\lib\router\index.js -> node_modules\next\dist\lib\router\router.js -> node_modules\next\dist\lib\router\index.js
(!) Circular dependency: node_modules\next\dist\lib\router\index.js -> node_modules\next\dist\lib\router\router.js ->  commonjs-proxy:<FULL_PATH_TO_HERE>\node_modules\next\dist\lib\router\index.js -> node_modules\next\dist\lib\router\index.js

Would it be possible to not have a circular dependency between index.js and routes.js?
Here are the files I am using for testing this, hopefully this might help someone more experienced carry on from where I am stuck:
rollup.config.js
import babel from "rollup-plugin-babel";
import commonjs from "rollup-plugin-commonjs";
import resolve from "rollup-plugin-node-resolve";
import replace from "rollup-plugin-replace";
import json from 'rollup-plugin-json';

const NODE_ENV = process.env.NODE_ENV || "development";
const outputFile = NODE_ENV === "production" ? "./lib/prod.js" : "./lib/dev.js";

export default {
    input: "./server.js",
    output: {
        file: outputFile,
        format: "cjs"
    },
    plugins: [
        replace({
            "process.env.NODE_ENV": JSON.stringify(NODE_ENV)
        }),
        json(),
        babel({
            exclude: ["node_modules/**"]
        }),
        resolve({
            preferBuiltins: true
        }),
        commonjs()
    ],
    external: ["webpack", "readable-stream", "glob", "source-list-map", "through2" ]
};

(external has a few exclusions due to circular dependencies that need to be taken up with those package owners.)
"devDependencies": {
    ...
    "rollup": "^0.66.4",
    "rollup-plugin-babel": "^4.0.3",
    "rollup-plugin-commonjs": "^9.1.8",
    "rollup-plugin-json": "^3.1.0",
    "rollup-plugin-node-resolve": "^3.4.0",
    "rollup-plugin-replace": "^2.1.0"
  }

create rollup (note circular dependency issue)
NODE_ENV=production rollup -c

to run
node lib/prod.js

error I get but I suspect it is because of circular dependency
C:\Users\Adam\code\gell\gell-client-west\lib\prod.js:135129
const childModule  = commonjsRequire.resolve('./child/index');
                                     ^

TypeError: commonjsRequire.resolve is not a function
    at Object.<anonymous> (lib\prod.js:135129:38)
    at Module._compile (internal/modules/cjs/loader.js:689:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
    at Module.load (internal/modules/cjs/loader.js:599:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
    at Function.Module._load (internal/modules/cjs/loader.js:530:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:742:12)
    at startup (internal/bootstrap/node.js:279:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:752:3)

Can anyone weigh in on:

how hard it would be to rewrite next\dist\lib\router\index.js -> next\dist\lib\router\router.js without circular dependency?  Could the router.js code just be included directly into index.js?
Is "commonjsRequire.resolve is not a function" error due to the circular dependency or something else?

Is this issue will be fixed with next-server ? @timneutkens

Closing this in favor of #5927

The current size of my built next directory is over 500m because of some huge node_modules containing unnecessary code.  I currently run the next build result on Amazon Beanstalk.  I am confused about how serverless target is relevant to creating a small traditional next build.  Can serverless target be run on any deployment solution?

@timneutkens I'm setting up rollup for next-i18next and have noticed the same circularity warning. I'm on Next v7.0.2. Is this something that's coming in the 8.x release?

@isaachinman you should mark the next imports as external, otherwise rollup will bundle them up and you end up with duplicate code.

Ah makes sense, thanks.


not found file https://node-inspector.s3.amazonaws.com/debug/v1.0.1/node-v57-win32-x64.tar.gz

<Error>
NoSuchKey
<Message>The specified key does not exist.</Message>
<Key>debug/v1.0.1/node-v57-win32-x64.tar.gz</Key>
<RequestId>60422E2FE58F1918</RequestId>
<HostId>
oZAFxY2/sLrp3mSFgI5URhiePdU7RN0FlInY7U4xoOgYf3vv0y4k42MFM3V+t+0bvL86c4z9UIk=
</HostId>
</Error>

macOS too.
npm install -g node-inspector
node-pre-gyp ERR! Tried to download(404): https://node-inspector.s3.amazonaws.com/debug/v1.0.1/node-v57-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v57 ABI, unknown) (falling back to source compile with node-gyp)

Node 8 brings a big break change in the debug api which is too difficult to implement in Node Inspector. Besides it provides a built-in DevTools-based debugger which mostly deprecates Node Inspector. So basically you can just use the built-in debugger tools. https://nodejs.org/en/docs/inspector/
you can get use the same results by running the --inspector option
$ node --inspect server.js

npm install -g<EMAIL_ADDRESS>maybe ok

ou.it's confused

reason close: I closed this issue because I don't see the error anymore. if you encounter an error, create a new issue.


can't call route api

Laravel Version: 8.x
PHP Version: 7.4

Description:
I create a SPA , define a route catch all
Route::any('{catchall}', function () {     return view('app'); })->where('catchall', '.*');
and create all api route in api.php, but when call api , all request go to route web, and return view app
I know that i can use  where('any', '^(?!api).*$') , but in laravel 7.x , it work fine

Can you post your routes.php and api.php please. So I can try to replicate the problem.

Can you post your routes.php and api.php please. So I can try to replicate the problem.

My api.php

use Illuminate\Http\Request;
use Illuminate\Support\Facades\Route;
Route::middleware('auth:api')->get('/user', function (Request $request) {
return $request->user();
});
Route::apiResource('tasks', \App\Http\Controllers\TaskController::class);
My web.php
.

use Illuminate\Support\Facades\Route;
Route::any('{catchall}', function () {
return view('app');
})->where('catchall', '.*');

Laravel 8 changes the RouteServiceProvider behaviour, but it's not yet documented in the upgrade guide, take a look at the "Routing Namespace Updates" section here: https://laravel.com/docs/8.x/releases
took me a bit too to fix this during the upgrade

@JeffreyWay had mentioned this https://twitter.com/jeffrey_way/status/1303728676434063361

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Forwarding Arguments to Derived Node Constructor

It would be useful to have the option to forward constructor arguments to the derived class when performing addNode to get more reusable behaviour out of user defined node types without having to pass them in through some alternate mechanism.

Could you provide a small example? I'm not sure if I understood your message correctly

Something like
struct A : BaseNode {
A(..., int numIns) : BaseNode(...)
{
   // Do something with numIns like dynamically sizing number of inputs
}
// Rest of methods
};

addNode<A>("A", 2); // not sure if this exact syntax is possible


Async call for fmpeg which should be switchable

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

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

What is the purpose of your issue?

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

Hello there,
Could you do the fmpeg call async to the download process? As the download is way faster than the convertion to mp3 for example. I know that I could use the --exec flag, but a native support would be great. This should be switchable for slower connections via a config option or a command parameter.
Best regards,
KernelDev

Clarify what do you mean by "fmpeg call async to the download process".

Hello,
I mean, that the current workflow is: 1. Download 2. Convert 3. start at 1.
The optimized flow of the programm should be: 1. Download +start a demon to convert the file. While the first file is converting, the other pending downloads can run an start their own converter demon

Out of scope.


[Docs] Outdated nodejs docs

The official "Bot Builder for Node.js" documentation is really outdated. It does not contain information about changes included in botbuilder 3.5, 3.6 and 3.7.
Even if there are code examples in github, this causes a lot of confusion and may lead to using bad practices or deprecated code when building a bot with the framework.

Hi @palmerabollo, first of all, thank you for your feedback! 👍
We are currently working on providing new and up-to-date documentation for the SDK and hope to have them out soon.
To help us provide a better set of documentation (and also improve the SDK) for our developers, do you have anything you wish to see in the future, or perhaps have any recommendations or other thoughts?

@palmerabollo Good news! Our latest documentation has been released and can be found at https://docs.microsoft.com/en-us/bot-framework/


Security groups fail in etcd.

On my test install, everything goes wrong. iptables makes it clear that parsing the rules hasn't gone well:
Chain felix-p-_1f35a710c69e75a-i (1 references)
 pkts bytes target     prot opt in     out     source               destination
    0     0 DROP       all  --  any    any     anywhere             anywhere             /* ERROR failed to parse rules DROP: */

Chain felix-p-_1f35a710c69e75a-o (1 references)
 pkts bytes target     prot opt in     out     source               destination
    0     0 DROP       all  --  any    any     anywhere             anywhere             /* ERROR failed to parse rules DROP: */

With lots of this in the logs:
2015-04-02 16:08:13,030 [ERROR][25] calico.felix.frules 158: Failed to convert rules to fragments: [{'ip_version': 4, 'dst_net': u'<IP_ADDRESS>/0', 'dst_tag': None, 'dst_ports': None}, {'ip_version': 6, 'dst_net': u'::/0', 'dst_tag': None, 'dst_ports': None}].  Will DROP!
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/calico-0.14-py2.7.egg/calico/felix/frules.py", line 152, in rules_to_chain_rewrite_lines
    on_deny=on_deny))
  File "/usr/local/lib/python2.7/dist-packages/calico-0.14-py2.7.egg/calico/felix/frules.py", line 187, in rule_to_iptables_fragment
    assert not unknown_keys, "Unknown keys: %s" % ", ".join(unknown_keys)
AssertionError: Unknown keys: ip_version

Here's an example of some rules:
ubuntu@calico-st02:~/calico$ etcdctl get /calico/policy/profile/20834313-58bc-4867-80c9-876c4c0f8ffa_7184c0df-294c-4932-8504-e5e1dd82b0cc/rules | python -m json.tool
{
    "inbound_rules": [
        {
            "icmp_type": null,
            "ip_version": 4,
            "protocol": "icmp",
            "src_net": null,
            "src_ports": null,
            "src_tag": "20834313-58bc-4867-80c9-876c4c0f8ffa"
        },
        {
            "ip_version": 4,
            "protocol": "tcp",
            "src_net": "<IP_ADDRESS>/32",
            "src_ports": [
                22
            ],
            "src_tag": null
        },
        {
            "ip_version": 6,
            "src_net": null,
            "src_ports": null,
            "src_tag": "7184c0df-294c-4932-8504-e5e1dd82b0cc"
        },
        {
            "ip_version": 4,
            "src_net": null,
            "src_ports": null,
            "src_tag": "7184c0df-294c-4932-8504-e5e1dd82b0cc"
        },
        {
            "ip_version": 4,
            "protocol": "tcp",
            "src_net": "<IP_ADDRESS>/32",
            "src_ports": [
                22
            ],
            "src_tag": null
        }
    ],
    "outbound_rules": [
        {
            "dst_net": "<IP_ADDRESS>/0",
            "dst_ports": null,
            "dst_tag": null,
            "ip_version": 4
        },
        {
            "dst_net": "::/0",
            "dst_ports": null,
            "dst_tag": null,
            "ip_version": 6
        },
        {
            "dst_net": "::/0",
            "dst_ports": null,
            "dst_tag": null,
            "ip_version": 6
        },
        {
            "dst_net": "<IP_ADDRESS>/0",
            "dst_ports": null,
            "dst_tag": null,
            "ip_version": 4
        }
    ]
}

This seems to be a Felix bug: the etcd data model doc does list an ip_version key. Annoyingly, no-one told me, which means that #295 is now out of date.

Yep, bug; but at least you tested the "fail safe, DROP packets and write a clear error to the iptables config" feature... ;-)

Fixed by 85b15fb4ac5e7a83008030dea5ec63ffcf7014ce


tests with colors

Provide GREEN/RED colors when the tests pass or fail.

Thanks, but I consider this more as an enhancement request.  Actually your solution ignores the situation (on Travis this was the case once) that the output is connected to a Terminal.  I will at one point copy the code from Kissat for colors, but was too lazy for that.

I added a test to add the colors only if the script is ran on a terminal. Was that the problem?

You want to use ' if [ -t 1 ] ' as in
https://github.com/arminbiere/cadical/blob/master/scripts/colors.sh

Now I see that you also have a -t 1 test in the script.    In any case I do not do pull request because I am not developping on GitHub but only use it for releases.

Probably should add a CONTRIBUTING file ...

Actually, you do not need the else part in your script :)

Right .... Thanks.


Impossible to customize window size

The window can be neither resized nor set to full-screen.

These are all "bugs" associated with the game - not the snap. I am not the developer of the game, I just packaged the snap for consumption.

So the window size is impossible to resize by default? If so, is there a file that I have to edit to set it to full-screen by default? Probably I just have to set a value to full-screen or 1366×768 (my actual max resolution).

Again, this is not the fault of the snap but the application being snapped. Some games have a preset window size created by the original developer (not me) and the snap just respects those settings to allow the game/application to be played as the developer intended. You're welcome to reach out to the original developer to raise this issue.


[BEAM-3326] Add SdkHarnessClientControlService

This is the higher-level Control Service implementation which vends
SdkHarnessClient instances.
Implement AutoCloseable on SdkHarnessClient to close the underlying
control client when the high-level service is shut down.

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

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand:

[x] What the pull request does
[ ] Why it does it
[ ] How it does it
[ ] Why this approach

[x] Each commit in the pull request should have a meaningful subject line and body.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Filed tickets for the flakes.

run java gradle precommit


change PyModule::add_class to return an error if class creation fails

Related to #2942
At the moment there are panics deep in the #[pyclass] machinery when initialising the type fails. This PR adjusts a number of these functions to return PyResult instead, so that we can handle the error more appropriately further down the pipeline.
For example, take the following snippet:
#[pyclass(extends = PyBool)]
struct ExtendsBool;

#[pymodule]
fn pyo3_scratch(_py: Python, m: &PyModule) -> PyResult<()> {
    m.add_class::<ExtendsBool>()?;
    Ok(())
}

Currently, importing this module will fail with a panic:
TypeError: type 'bool' is not an acceptable base type
thread '<unnamed>' panicked at 'An error occurred while initializing class ExtendsBool', /Users/david/Dev/pyo3/src/pyclass.rs:412:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Users/david/.virtualenvs/pyo3/lib/python3.10/site-packages/pyo3_scratch/__init__.py", line 1, in <module>
    from .pyo3_scratch import *
pyo3_runtime.PanicException: An error occurred while initializing class ExtendsBool

After this PR, this import still fails, but with a slightly cleaner, more Pythonic error:
TypeError: type 'bool' is not an acceptable base type

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Users/david/.virtualenvs/pyo3/lib/python3.10/site-packages/pyo3_scratch/__init__.py", line 1, in <module>
    from .pyo3_scratch import *
RuntimeError: An error occurred while initializing class ExtendsBool

On reflection I've renamed StaticTypeObject to LazyTypeObject (was LazyStaticType before this PR). It's got nothing to do with "static" any more now that we use heap types, and the &'static reference is encoded elsewhere in the PyClassImpl implementation (but this would function perfectly fine without being static).
I've also pushed a follow-up commit which switches this to store Py<PyType> instead of *mut ffi::PyTypeObject to track ownership properly, and adjusts the internals to make use of this and &PyType so there's less unsafe.

Thanks! I've adjusted the comment around the code you've questioned, replied, and will proceed to merge.
If we think there's follow-up work I'll open a new PR.

bors r=adamreichold

bors retry

bors r+

bors retry


[AzureStack][Admin API]  Adding disk list api in a new api version

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ X] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[X ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[X ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ X] I have reviewed the documentation for the workflow.
[ X] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[ ] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@bganapa Hi Bala, hope you are doing well. Any updates for this PR?

@FireDefend there are swagger validations errors, could you please look in to them? Can you pls also ping me at bganapa internally?

@leni-msft  pls do the needful for getting this reviewed by ARM team

@FireDefend  Please fill out the checklist thoroughly.  No review can proceed until you have read and filled out the checklist.

@NetStorm-E  please ping me at bganapa. The commits from @TheOnlyWei  should be omitted from your PR. Those commits are in this PR
https://github.com/Azure/azure-rest-api-specs/pull/13283

@FireDefend  Can you pls look at the cross version breaking changes and make sure that are expected? This seems to be anew thing in the PR validation

@FireDefend Can you pls look at the cross version breaking changes and make sure that are expected? This seems to be anew thing in the PR validation

Hi Bala, I have checked, those changes are expected.

closed cause another same PR got merged
https://github.com/Azure/azure-rest-api-specs/pull/14426


IOS - unable to recieve incoming call notification when the app is in background.

I used callkeep to show incoming call notification for my video call app.
for the android part, it is working fine. I am receiving incoming call in foreground and background.
but for ios, The app shows incoming call screen when it is opened. if I touched the home button and the app is minimized. I am not receiving call notifications, the incoming call screen will  displayed as soon as I click the app icon.
I am using callkeep: ^0.3.2.
How to resolve this issue?.

@cloudwebrtc This is an urgent situation.  I am asking you maybe you can look at my Issue and help fix the problem. I have been struggling with this problem for a long time.

@cloudwebrtc This is an urgent situation. I am asking you maybe you can look at my Issue and help fix the problem. I have been struggling with this problem for a long time.
first of all please configure Voip Push server, without Voip push server on ios is not working (or use commercial voip push servers). Or you can send push as message to your App and then call CallKeepDidDisplayIncomingCall for the show CallKit screen.

Hey, did you find any solution or fix for this issue?

Hey, did you find any solution or fix for this issue?

I didn't find a solution this functionality is pending

Hey, did you find any solution or fix for this issue?

I didn't find a solution this functionality is pending

Hey, i solved this background/terminated calling issue. You can use this if your still looking for a solution for this issue:

Currently Callkeep package does not support showing callkit call in background/terminated state after receiving a Voip push, maybe they are working on a solution for this and patch this in the near future
Use flutter_callkit_incoming package to handle incoming voip push notifications, make sure to copy and add the appdelegate.swift file into your projects appdelegate file, doing so you will be able to handle the voip pushes
I have tested this approach and can verify that the solution is working when the app is in background and terminated mode


Decode does not work without callback!

https://github.com/birdofpreyru/audio-loader/blob/b316b1db94da4f32cb33200c2f4d44f87b0d4247/src/load.js#L53
At least in iOS WebView the inner code of this method must read
return new Promise((resolve, reject) => {
    options.decode(array, (error, result) => {
      if (error) reject(error);
      else resolve(result);
    });
  });

Double-check, maybe just making the function async also helps?


Add Azure IPI install

WIP thus DNM, however please review when time is available.

@chaitanyaenr  @jmencak PTAL this should be ready

@jmencak PTAL again

Didn't test this version, but LGTM.  Thanks!  @chaitanyaenr, I think this is ready to merge unless you have objections.


Added source of 3D models to the Resources.

Hum3D is a largest library of vehicle 3d models and blueprints in blend file format.

merged, thanks. ^^


Deployment Guide for EKS and ECR

Following - https://github.com/CrowdStrike/falcon-operator/tree/main/docs/deployment/eks, since the title is Deployment Guide for EKS and ECR, should we not be expecting a
registry:
type: ecr
entry in the node sensor yaml file and IAR yaml files?
https://raw.githubusercontent.com/crowdstrike/falcon-operator/main/config/samples/falcon_v1alpha1_falconnodesensor.yaml
https://raw.githubusercontent.com/crowdstrike/falcon-operator/main/config/samples/falcon_v1alpha1_falconimageanalyzer.yaml
type: ecr is in falcon admission and falcon container, but not node sensor and IAR.

Not needed.

not needed.


iter-KUI Canary + A/B/n rollouts with TFserving models

Identify and run canary + A/B/n experiments with 3 TF serving models
Status stripe
Create video centered blog article
Create video centered tutorial for MLOps
Go to appropriate slack channels with 1000s of users and slack them about MLOps with iter8-KUI
Structure the blog in a way that will get us answers to the following question

What would MLOps users like in terms of automated experiments with iter-KUI?
What would MLOps users like in terms of new metrics?
What would MLOps users like as part of HIL capabilities?

Need to revisit exact,y what this campaign entails.


Depth Map

Please make sure that this is a feature request.
System information (Please provide as much relevant information as possible)

MediaPipe Solution (you are using):
Programming language : C++/typescript/Python/Objective C/Android Java
Are you willing to contribute it (Yes/No):

Describe the feature and the current behavior/state:
Will this change the current api? How?
Who will benefit with this feature?
Please specify the use cases for this feature:
Any Other info:

Can we find depth maps of face from mediapipe facemesh solution?

Hi @showbit01 ,
Can you please check this documentation for depth.

Hi ,I have checked but this is about landmarks not depth

Hi @showbit01,
Could you please elaborate  this issue to investigate further.

I want to find the depth from the cropped face obtained from mediaipe library,the depth is a 2d matrix composed of z values

Hi @showbit01 ,
z: Should be discarded as currently the model is not fully trained to predict depth, but this is something on the roadmap.

Hi @showbit01 ,
z: Should be discarded as currently the model is not fully trained to predict depth, but this is something on the roadmap.

Thanks for replying.
Okay got it.


Add docs on the lifecycle of buffers in Turms

The lifecycles of the buffers related to HandleServiceRequest and SendNotificationRequest are very difficult to trace because we slice, retain, release, etc on the same buffer without duplicating to send it to multiple clients and servers, which makes the buffer really difficult to trace.
So we need to document it.

Done.


Null-safety and DIO 4.0.0 support

Description
This PR adds null-safety and DIO 4.0.0 support. Fixes #67
This PR is based on #82 so that one needs to be merge first, afterwards I can rebase. That will make the diff clearer.
This should be published as 1.0.0 or 1.0.0-nullsafety.1 preview.
CC @LukaGiorgadze @GiorgiBeriashvili @campfire5

Sounds exciting! #82 is even more important now. I will focus on it immediately and will provide feedback soon.

Any news on this? I have a migration blocked by this :(

You can use an override pointing to this branch for now.

I merged the PR (#82) the changes were based on.

This should be published as 1.0.0 or 1.0.0-nullsafety.1 preview.

Could you please explain why is that the case? I was thinking on either 0.2.0 or 0.3.0.

I merged the PR (#82) the changes were based on.

This should be published as 1.0.0 or 1.0.0-nullsafety.1 preview.

Could you please explain why is that the case? I was thinking on either 0.2.0 or 0.3.0.

Well this a breaking change and according to semantic versioning this should be a major version. And it would automatically be pulled in for anyone with ^0.1.0 in the pubspec where as 1.0.0 would not get automatically updated.
I rebased it on develop.

I merged the PR (#82) the changes were based on.

This should be published as 1.0.0 or 1.0.0-nullsafety.1 preview.

Could you please explain why is that the case? I was thinking on either 0.2.0 or 0.3.0.

Well this a breaking change and according to semantic versioning this should be a major version. And it would automatically be pulled in for anyone with ^0.1.0 in the pubspec where as 1.0.0 would not get automatically updated.
I rebased it on develop.

Understandable, though the package is still under 1.0.0, in which case it could be decided how the version will be raised: https://dart.dev/null-safety/migration-guide#package-version

@GiorgiBeriashvili That is up to you, since it is "only" a test dependency either way is probably fine.

I addressed the paths and CI change. The version is up to you.
Not exactly sure why the build fails now, need to look into this.

Ok dio was updated again. This might happen again until dio 4.0.0 is released.
Also I noticed that the format test in CI doesn't fail - set-exit-if-changed not working . See https://github.com/dart-lang/sdk/issues/44582

Update to latest Dio 4.0.0-prev1 and implemented the required interceptor API changes.

Update to latest Dio 4.0.0-prev1 and implemented the required interceptor API changes.

Thanks @kuhnroyal , i added @GiorgiBeriashvili and @campfire5 as a (re-)reviewers.

@kuhnroyal Could you please resolve the conflicts (two PRs have been merged in the meantime) and change the package's version to 0.2.0 in pubspec.yaml?
Thereafter, after re-reviewing once again, merging will most probably be in order.

On it.

@kuhnroyal First of all, thank you for your time and effort! Thank you for being actively contributing in open source world!
We checked this PR and approved, but one small thing is that some of PRs merged before and we have conflicts. I would like to resolve it for you, but i'm worrying to change something else.
Please, help us resolving this conflicts and we will merge it ASAP.
Thank you again!

You seemed to have force pushed over the develop and main when trying to release 0.1.6. This is really bad practice, now all forks are out of sync. The last shared ancestor commit is on the the 0.1.4 tag (1931051).
The commit of the 0.1.5 tag (ff0b5b1) is not even part of the repository anymore.
I strongly recommend reverting all the changes and resetting main and develop to before the force-push.
Additionally something got messed up during the rebase, the conflicts make no sense.

@kuhnroyal develop is reverted and ready to merge. After this merge, we need to resolve conflicts between develop and main.


Add PR template

Which issue(s) this PR fixes:
N/A
What this PR does / why we need it:
Add pull request template
Docs Changes:
.github/PULL_REQUEST_TEMPLATE.md
Release Note:
N/A

Looks good. Thanks!


Vanta Tool

In order to better understand and change the background for better frontend optimization, I have been learning how to use Vanta. However, I have been having difficulties converting the files into the same format as the examples.
In the meantime, I was able to create a scrolling backdrop using a custom background.
https://rliao569.github.io/Frontend-CSP/c5.0/c7.0/c7.6/2023/10/07/Background_Working.html
https://www.google.com/url?sa=t&rct=j&q=&esrc=s&source=web&cd=&cad=rja&uact=8&ved=2ahUKEwjn7Y2N6_mBAxWlSDABHYJECeoQFnoECBQQAQ&url=https%3A%2F%2Fgithub.com%2Ftengbao%2Fvanta&usg=AOvVaw0MRRamF0XagUPQl5Q-DEOe&opi=89978449
https://www.youtube.com/watch?v=yo1fJTfV4Mw

Converting the codes into a min js file does not allow it to work. There is a cloudflare server that has different links. In order to call the files on the basic one, I might need multiple code cells with different languages. The tutorials mostly use the cloudflare site. I still need to learn how to add a small scrolling image to help with formatting on our homepage.

https://www.youtube.com/watch?v=mptrAVnh-vw

I think I broke the flask server background.

https://jsfiddle.net/2e5ugftv/1/
The cells is not showing

https://jsfiddle.net/5a4jsfcb/
The waves works

VANTA CLOUDFLARE LIBRARY
https://cdnjs.com/libraries/vanta

https://jsfiddle.net/gcdpa45y/

https://jsfiddle.net/aktrooper/895ge0pk/1/

I think I can get the Vanta up on the flask server. The cloudflare can show me the min.js file but it is not copy and pasting in.


Cannot find TagsEditText im my xml layout

@Pirokar @anonymous-ME Can you guys provide more description of the issue?


Ready for merge

this is a test to check if travis have the correct data

Pull requests create the following web site  : http://yakchat-<PULL_REQUEST_NUMBER>.s3-website-us-west-1.amazonaws.com.


Update sbt to 1.2.8

Updates org.scala-sbt:sbt from 1.1.6 to 1.2.8.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "org.scala-sbt", artifactId = "sbt" }]

labels: semver-minor

Codecov Report

Merging #1133 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1133   +/-   ##
=======================================
  Coverage   80.51%   80.51%
=======================================
  Files          54       54
  Lines        1016     1016
  Branches       46       46
=======================================
  Hits          818      818
  Misses        198      198

Continue to review full report at Codecov.

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


Building against OpenSSL-enabled liboqs fails

If liboqs is built with OpenSSL support (e.g. ./configure --enable-openssl), it won't work with this wrapper.
I'm assuming we need additional LDFLAG entries. This can be done without touching the source code, via the CGO_LDFLAGS environment variable. However, I've failed thus far to get that working.
I've tried:
CGO_LDFLAGS="-L/usr/lib/x86_64-linux-gnu/ -lcrypto" go test .
CGO_LDFLAGS="-L/usr/lib/ -lcrypto" go test .

to no avail. The error is usually undefined symbol: BN_CTX_get. (I modify the source to print dlerror()). This symbol is indeed defined in libcrypto:
$ nm --dynamic /usr/lib/x86_64-linux-gnu/libcrypto.so | grep BN_CTX_get
00000000000a5830 T BN_CTX_get

This is a linking oversight in liboqs, see https://github.com/open-quantum-safe/liboqs/issues/485.
In the interim, the following additional LDFLAGS flags are needed to work with an OpenSSL build of liboqs: -Wl,--no-as-needed -lcrypto.
For instance:
CGO_LDFLAGS="-Wl,--no-as-needed -lcrypto" go test -v

open-quantum-safe/liboqs#485 now resolved.


feat: add theme switch on website home page

Fixes #13028

Looks good! Might want to make it a toggle button like the one on the bridge at the top of the page? Could make it easier to find and access.

Looks good! Might want to make it a toggle button like the one on the bridge at the top of the page? Could make it easier to find and access.

I agree, it should be on top of the page, it's way too hard to find right now

Looks good! Might want to make it a toggle button like the one on the bridge at the top of the page? Could make it easier to find and access.

I agree, it should be on top of the page, it's way too hard to find right now
@RogerLamTd @wolfderechter
The Nextra theme that we are using places it like so and thus I have replicated it.


Enable bail option to webpack prod configuration

Hi! This should solve https://github.com/vuejs-templates/webpack/issues/775.
Thanks for ur work!

Will make a quick test run with this to see the behaviour when it fails, then I'll merge. Thanks.


[gui] Smali debugger: Failed to init debugger

Debugger won't work on any app. When I click "Launch App", it first logs "Starting debugger..." and then "Failed to init debugger"
In the debugger:

jdwp handshake failed

(Yes the app is debuggable and in "Waiting for debugger" mode)
Jadx version: 1.3.4
OS: Ubuntu

@Mino260806 as discussed in #1412 downgrading ADB to previous version may help.
@jpstotz, @RebootSTR can anyone post a version of ADB which still work and/or which version don't work? So I can add to the debugger wiki page.

@skylot I tested some adb versions, but it seems like the adb commands like adb -L tcp:5037 start-server and some of the protocol commands don't allow an adb version that is too old. In the end I found out that connecting to the debugger via adb seems to be an exclusive operation. That means if you have one other program running that keeps the connection open Jadx will fail to connect to the app.
For example I had once installed the Android plugin into IntelliJ, that plugin seems to automatically connect to all attached devices and thus block them. Even killing adb server had no effect as the plugin automatically restarts adb and performs a reconnect. An running instance of Android Studio (with an open project) also has the same effect.
So at the moment I am running adb 33.0.0-8141338, v33.0.1-8253317 also works fine for me as long as the Android plugin is disabled and Android Studio closed.

@skylot Unfortunately I have not understood yet how Android Studio establishes the jdwp connection. At least it does not use the adb forward way Jadx uses.  May be I look into Google's ddmlib library source code. I would assume that it contain the code to establish the adb connection used by Android Studio.

I had this problem too, and switching to latest v33.0.1 adb solved the problem

Solved this problem by closing Android Studio, killing the ADB server, and using the ADB binary from the android-tools package instead of the one provided by the SDK.

I used version ：
Android Debug Bridge version 1.0.41
Version 33.0.3-8952118
but it still not work


Fix lint step

Following the conversation here: https://github.com/golangci/golangci-lint/discussions/2983
Signed-off-by: Pranshu Srivastava<EMAIL_ADDRESS>

https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/red-hat-storage_mcg-osd-deployer/71/pull-ci-red-hat-storage-mcg-osd-deployer-main-lint/1554014883962425344#1:build-log.txt%3A24

/lgtm


error #5102: Cannot open include file 'zmumps_struc.h'

Hello,
i am using your doc to include mumps into my project with FetchProject, as you described it.
But got this error when compiling:
: error #5102: Cannot open include file 'zmumps_struc.h'
    INCLUDE 'zmumps_struc.h'
------------^
/tmp/ifortVCXhzJ.i90(757): catastrophic error: Too many errors, exiting

but this file exists, in the build directory:
$ find . -name "zmumps*.h"
./_deps/mumps_proj-src/include/zmumps_c.h
./_deps/mumps_proj-src/include/zmumps_root.h
./_deps/mumps_proj-src/include/zmumps_struc.h
./_deps/mumps_proj-src/src/zmumps_gpu.h

Could you help, please?
Thanks in advance

I would recommend building MUMPS as ExternalProject in general. We've come to find FetchContent isn't the best solution for this type of project.


Autofill content types based on location picked.

Do you have a nice solution to autofill content types like: city, houseNumber, country, postalCode, etc. Based on the google location autocomplete return (field: address_components)?

I do not have plans to implement that feature in particular any time soon, but it would be really cool!


Jadx decompiler fails to return the source code

Describe what you wanted to do
When analyzing some apks, I figured that jadx is giving more suitable source-code. Thus, I wanted to use Androguard with the jadx decompiler and access the source code of classes. Sadly, it always remains empty. I followed the documentation at https://androguard.readthedocs.io/en/latest/intro/use_jadx.html. Fails on all apks. Minimal working example:
    a = APK('examples/android/TestsAndroguard/bin/TestActivity.apk')
    d = DalvikVMFormat(a)
    dx = Analysis(d)
    decompiler = DecompilerJADX(d, dx, jadx='/path/to/my/jadx')
    d.set_decompiler(decompiler)
    #d.set_vmanalysis(dx)

    for cls in d.get_classes():
        print(cls.source())

Describe what you expected
I expected the source code to be returned/printed. upon calling cls.source()
Describe what actually happened
None is being returned/printed.
System Information

Androguard Version: 3.3.5
Python Version: 3.6
MacOS
Tried with old jadx 1.10 and with the newest

Further Log Files and Output
During the decompilation, the following appears on the stdout
... (more lines before)
Found a class called sources/android/support/v4/content/ModernAsyncTask, which is not found by androguard!
Found a class called sources/android/support/v4/content/ContextCompat, which is not found by androguard!
Found a class called sources/android/support/v4/content/ContextCompatHoneycomb, which is not found by androguard!
Found a class called sources/android/support/v4/content/IntentCompatHoneycomb, which is not found by androguard!
... (more lines after)
Found a class called android/support/v4/view/VelocityTrackerCompat$BaseVelocityTrackerVersionImpl which is not decompiled by jadx
Found a class called android/support/v4/view/VelocityTrackerCompat$HoneycombVelocityTrackerVersionImpl which is not decompiled by jadx
... (more lines after)

Is my code composed correctly? Am I trying to emuleate behaviour that is not meant to be supported?

Oh, silly me, I rushed a bit. The cause is that the paths of Androguard and Jadx don't match by a single sources/ directory. I'll try to fix that and create PR, hopefully I manage -- never saw that code...

I try to use your method is not available, I use Windows OS, I choose to use regular expression to find the path.
        for root, dirs, files in os.walk(tmpfolder):
            for f in files:
                if not f.endswith(".java"):
                    log.warning("found a file in jadx folder which is not a java file: {}".format(f))
                    continue
                # as the path begins always with `self.res` (hopefully), we remove that length
                # also, all files should end with .java
                # path = os.path.join(root, f)[len("sources/") + 1:-5]
                path = os.path.join(root, f)
                path = path.replace(os.sep, "/")
                path = findall(".*/sources/(.*)\.java", path)
                path = path[0] if len(path) > 0 else ""

                # Special care for files without package
                # All files that have no package set, will get the
                # package `defpackage` automatically
                if path.startswith("defpackage"):
                    path = path[len("defpackage/"):]

Androguard still does not find the source files decompiled by JADX, because they are placed under tmpfolder/sources like @adamjanovsky already pointed out.
@adamjanovsky @1eafy and #869 already attempted to fix this issue, but it didn't find its way into the main version yet. Those attempts adjust the location where Androguard searches for the source files. Another approach would be to adjust the JADX call, so it places the sources to the expected location tmpfolder directly. This can be achieved, by changing the -d flag to -ds in decompiler.py#L652
cmd = [jadx, "-ds", tmpfolder, "--escape-unicode", "--no-res", tf.name]

I used this Merged，it don‘t show this error，but i also can't get method’s source code
https://github.com/androguard/androguard/pull/899
Found a class called sources/android/support/v4/content/ModernAsyncTask, which is not found by androguard!


Support for cross-platform RPM package generation

Please consider a patch aimed to add proper support for constructing RPM packages aimed for the CPU architecture other than the host machine. This addresses https://github.com/bazelbuild/rules_pkg/issues/727 issue.
The current version of the "pkg_rpm"-rule exposes an "architecture"-option, which is meant to indicate the package's intended platform. Unfortunately, this option is being incorrectly utilized. It is being used to define the "BuildArch"-option in the RPM spec file. However, as per RPM packaging guide (https://rpm-packaging-guide.github.io/):
"'BuildArch' should be used if the package is not architecture dependent. For example, if written entirely in an interpreted programming language, set this to BuildArch: noarch. If not set, the package automatically inherits the Architecture of the machine on which it is built".
It appears that BuildArch is intended to represent either architecture of host machine or "noarch". Any efforts to designate an "alien" platform as a value for this field lead to a failure in the "rpmbuild"-utility, accompanied by an error message stating: "error: No compatible architectures found for build."
The purpose of this patch is to maintain backward compatibility while provide the solution for the described issue.
It preserves the existing "architecture"-attribute of the "pkg_rpm" rule, as it might already be in use by someone.
It introduces the "target_architecture"-attribute to the "pkg_rpm" rule, aiming to utilize it as a "arch"-suffix for the package name and to provide it to the "rpmbuild"-utility as the value for the "--target" option.

Can we figure the right behavior out in the issue before jumping to code.
I'll comment there.

Can we figure the right behavior out in the issue before jumping to code. I'll comment there.

Thank you, Tony, I answered in the issue.

Friendly ping

Any plan to ever merge this? It is pity that this very convenient rule cannot be used for general rpm builds, while a solution is just sitting here in limbo.

Any plan to ever merge this? ...

Good question.  I keep seeing it in the overview as waiting on the original author, so I have been ignoring it.
It does seem OK though. I can look over the weekend.

Any hope off seeing this merged?

Any hope off seeing this merged?

Yes. If you can work with me on this.  Have you patched it in and tried it?  Does it have the right behaviors?
The original author went cold on answering my questions.

Any hope off seeing this merged?

Yes. If you can work with me on this. Have you patched it in and tried it? Does it have the right behaviors? The original author went cold on answering my questions.

Awsome, I will gladly try this and get back to you!


Crash single player on 1.14.4

https://paste.ee/p/eZtDI
https://github.com/AllOfFabric/AOF-Strawberry/issues/15

Sorry, I forgot to close this issue. This was because the mod wasn't yet updated to 1.14.4


Add ARM packages to your repository

I have an Odroid with Ubuntu ARM installed on it. I would love to be able to use this program on it. Unfortunately, your PPA only has i386 and amd64 packages available. At this point, this means that my only options are to either run the i386 version through QEMU's emulation, which is slow and I'm not convinced is 100% reliable, or compile it, which I'm not comforable enough to do.
Would it be possible to compile ARM packagess and make them available alongside the i386 and amd64 versions?

I think you can try installing OCaml (https://github.com/jrcharney/rigel/wiki/OCAML) and then install google-drive-ocamlfuse with OPAM.

I have one question though: How much effort would be required to add ARM binaries to your PPA repository? The reason I ask is because I would much rather use Ubuntu's built-in package manager to manage my software rather than installing another software manager just to install yours.
Otherwise, I could certainly try it. I do however notice that the installation instructions for OCAML has the words "TODO: REDO THE INSTALLATION INSTRUCTIONS!" at the top so I'm already worried. Furthermore, they state that building OPAM and OCAML will take a very long time (hours, judging by the suggestions provided for what to do in the meantime).

I didn't know you can enable ARM compiling from PPA settings (http://askubuntu.com/questions/371339/how-to-make-packages-for-arm-in-launchpad/765149#765149). I can enable these targets:

ARM ARMv8 (arm64)
ARM ARMv7 Soft Float (armel)
ARM ARMv7 Hard Float (armhf)

Which one do you need?

That's good to hear. Anyway, the Samsung Exynos 5 SOC used by the Odroid XU3 uses the armhf architecture.

OK, I published arm packages for Ubuntu 16.04. Let me know if you need packages for another version.

I can't believe it's been a month already.
I just managed to install your program on my Odroid and it seems to be working very well. Thank you! I am having one small issue but I'll create a seperate ticket for that.
As for the other two ARM architectures:
I'm not aware of any computer that requires armel and, from what I've read, armel is slower and older than armhf. I don't think you need to worry about this unless someone requests it.
ARMv7 is still the most popular architecture for ARM computers though as time goes by, ARMv8 is only going to become more popular. In this list, I count 7 computers that can use arm64 binaries:
https://en.wikipedia.org/wiki/Comparison_of_single-board_computers#CPU.2C_GPU.2C_memory

Well, thanks! So I'm going to select arm64 too.

Can you to provide a static binary for ARMv7, I will use it on Android.
Thanks.