upgrade aes to use autoPadding and for legacy methods to skip the stream

adds autopadding options to ECB and CBC ciphers and implements update and final more in line with how node does it
currently cipher.on('data', function (d){console.log(d)}).update('foo') triggers the event here but not in node, this is fixed also errors due to padding and authentication now will only throw if you use them with .final() not with .end() (with .end they will cause an error event)

44ad3caff87f1db1bf2f4e267ef217d4e9c8edd9

awesome. okay back on line! one thing though: can you do version number bump in it's own commit? that being the way i've always done it with this module (and all my modules)

oh, and post the version number in the issue too.

yes can do
On Sun Jan 04 2015 at 1:57:48 AM Dominic Tarr<EMAIL_ADDRESS>wrote:

oh, and post the version number in the issue too.
—
Reply to this email directly or view it on GitHub
https://github.com/dominictarr/crypto-browserify/pull/92#issuecomment-68623494
.

@dominictarr I would say its better to do the version bumps outside of pull requests in any case?  Its unnecessary and can be done separately, not to mention it means pull requests can't be merged out of order, or asynchronously of each other (without producing a weird commit history).

Yeah I'd vote for outside as well, just run npm version (this one though
can't be merged in a different order though)
On Sun, Jan 4, 2015, 7:10 PM Daniel Cousens<EMAIL_ADDRESS>wrote:

@dominictarr https://github.com/dominictarr I would say its better to
do the version bumps outside of pull requests in any case? Its unnecessary
and can be done separately, not to mention it means pull requests can't be
merged out of order, or asynchronously of each other (without producing a
weird commit history).
—
Reply to this email directly or view it on GitHub
https://github.com/dominictarr/crypto-browserify/pull/92#issuecomment-68656204
.

@dcousens correct, the maintainer should make the final call on the version number, but they pull requester can state whether the change is a patch/feature/break

ok going forward I will use the npm version command after I merge it in and will post the version number in the issue.  Sound ok?

perfect :100:


Keyboard shortcuts disabled once the focus is moved to other widgets

Description
Unable to move focus back to Video once it's moved to other widgets, thus making keyboard shortcuts disabled.
Reproduce code
import 'package:flutter/material.dart';
import 'package:media_kit/media_kit.dart'; // Provides [Player], [Media], [Playlist] etc.
import 'package:media_kit_video/media_kit_video.dart'; // Provides [VideoController] & [Video] etc.

void main() {
  WidgetsFlutterBinding.ensureInitialized();
  // Necessary initialization for package:media_kit.
  MediaKit.ensureInitialized();
  runApp(
    const MaterialApp(
      home: MyScreen(),
    ),
  );
}

class MyScreen extends StatefulWidget {
  const MyScreen({Key? key}) : super(key: key);
  @override
  State<MyScreen> createState() => MyScreenState();
}

class MyScreenState extends State<MyScreen> {
  // Create a [Player] to control playback.
  late final player = Player();
  // Create a [VideoController] to handle video output from [Player].
  late final controller = VideoController(player);

  @override
  void initState() {
    super.initState();
    player.open(Media(
        'https://user-images.githubusercontent.com/28951144/229373695-22f88f13-d18f-4288-9bf1-c3e078d83722.mp4'));
  }

  @override
  void dispose() {
    player.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return Row(
      children: [
        Expanded(
            flex: 2,
            child: Video(
              controller: controller,
            )),
        Expanded(
            child: Container(
          child: TextField(),
        )),
      ],
    );
  }
}

Reproduce steps:

Test keyboard shortcuts upon build such as space to pause/play the video, it's normal at this step.
Click the TextField to move focus to it.
Click the Video widget again and try to use keyboard shortcuts, now they wouldn't work unless fullscreen mode is entered.

i have tested your sample code, and when foucs is on text field yes the shortcuts doesnt work, but when i tap on the video widget they work again, which looks like the excepted behaviour (Note: i am testing on macos)

I'm on Windows, and also tested it at https://flutlab.io/ using its default web build.

I had a problem with focus and flutter web before, can you test it using windows build? And feel free to suggest a fix in a pr

Windows build also has this issue, I'll try to look into it.


Cleaning Baseline

The baseline has a lot of comment code and should be organized.

Also adding a development group


login

Got the image up and running, but what's the default login ?

I guess there isn't any, until adduser.php has been used. Now I am in :-)

This container is not supported anymore, still works but in future, you may have PHP problems.

It is working fine, with PHP warnings :-) Do you know of any other ( supported ) container-versions of observium?

Search here: https://hub.docker.com/search?q=observium&type=image


Maven repo

Hi,
Are there any plans to migrate 1.4.0 to Maven?  Have I missed a discussion somewhere?  I cannot release an app that uses the cute Material SearchView because google won't accept libraries on jcenter() any longer.
Is there a simple fix for this?

looks like the library has been abandoned by contributors (5 years ago)

@MiguelCatalan @lgvalle lgvalle


Improvements to Help Mode Pie Menu Visuals

Changes

font for left-side keyboard command list and options on pie menu enlarged slightly for better legibility
pie menu pan option lowered so the center "1" button does not obscure it

Compare left (current) and right (revised)

Looks good then!


fix: typos in documentation files

This pull request contains changes to improve clarity, correctness and structure.
Description correction:
Corrected mainained to maintained x2.
Please review the changes and let me know if any additional changes are needed.

Fixed in #470


nodenv with Travis

Is there an example for how to use nodenv on Travis CI?
I've tired the setup written in the README.md but that doesn't seem to work, Travis isn't detecting the nodenv binary.

What method of setup did you follow?

I'm not sure how to help with this. Is nodenv cloning correctly? Is it in the correct location? Does travis allow usage of binaries in $HOME?
Can you verify that the clone is successful and the binary is where you expect it?
Can you verify that executing the binary via absolute path succeeds?
Can you verify that your $PATH is correct?
Show what which nodenv -a emits.
Though I'd take a step back and ask why you need to install nodenv on travis since they support node via nvm out of the box.

closing as stale. feel free to re-open with further details if you still want guidance.


Choose better sorting and priority queue algorithms

[ ] Gather statistics about the order of item arrays to better choose algorithms.
[x] Implement an unstable, O(n log n) time, constant space sorting algorithm that work fast on small and medium-sized arrays -- probably a combination of pattern-defeating quicksort and insertion sort.
[ ] Implement a special case for the priority queue that works fast on small arrays of tiny elements. For example, on up to 32 byte-sized elements.

The Rust standard library already uses efficient sorting algorithms.

Let's use a branchless binary search algorithm.


[New Port Request]

Library name: opencascade
Library description: a CAD geometry kernel library
Source repository URL: https://git.dev.opencascade.org/gitweb/?p=occt.git;a=snapshot;h=fd47711d682be943f0e0a13d1fb54911b0499c31;sf=zip
Project homepage (if different from the source repository): https://www.opencascade.com/
Anything else that is useful to know when adding (such as optional features the library may have that should be included):
opencascade has some dependencies
ffmpeg
freeimage
freetype
tcl/tk
vtk
qt5-base

Related issue: https://github.com/microsoft/vcpkg/issues/4968 .

Related issue: https://github.com/microsoft/vcpkg/issues/4968 .

Hi @silentmissile thanks for posting this request.
It's duplicated with #4968. Let's track this issue here.
@traversaro thanks for your info.


Building Ros2-desktop image

I am attempting to build Ros2-desktop by changing the the ROS_PKG argument from "ros_base" to "desktop" as done in #38 . I have attempted to do this with both Foxy and Galactic and am running into the following error.
--- stderr: sdl2_vendor
CMake Error at cmake/Modules/Findsdl2_custom.cmake:9 (target_link_libraries):
Cannot specify link libraries for target "SDL2::SDL2" which is not built by
this project.
Call Stack (most recent call first):
CMakeLists.txt:12 (find_package)

Failed   <<< sdl2_vendor [4.70s, exited with code 1]
I can see that rosdep successfully installs libsdl2-2.0-0 and libsdl2-dev.
This is the version of sdl2_vendor used for Foxy.
local-name: joystick_drivers/sdl2_vendor
uri: https://github.com/ros2-gbp/joystick_drivers-release.git
version: release/foxy/sdl2_vendor/3.0.0-2

I haven't been able to build Foxy desktop either, but Dan Pollock from the forums has:

https://forums.developer.nvidia.com/t/ros2-foxy-zed/183451
https://forums.developer.nvidia.com/t/script-to-install-ros2-foxy-desktop-on-agx-xavier-jetpack-4-5-1/182772

The ROS2-foxy-zed is only a ros_base image.
The script by Dan Pollock has some errors in it and doesn't work either.
I spent some time troubleshooting and was able to get foxy desktop to build in a docker container by combining your foxy Dockerfile with the script by Dan Pollock.
I also included some sensor drivers for Ouster, RoverRobotics and VectorNav in the ~/ros2_ws.
Works and tested on fresh install of Jetpack 4.5.0.
1.
Change Dokcer Default Runtime:
To enable access to the CUDA compiler (nvcc) during docker build operations, add "default-runtime": "nvidia" to your /etc/docker/daemon.json configuration file before attempting to build the containers:
{
"runtimes": {
"nvidia": {
"path": "nvidia-container-runtime",
"runtimeArgs": []
}
},
"default-runtime": "nvidia"

}
You will then want to restart the Docker service or reboot your system before proceeding.

Clone dusty-nv docker container scripts:
git clone https://github.com/dusty-nv/jetson-containers
cd jetson-containers
3.
Download files:
Ros2-foxy-desktop-CollinNHays.zip

Replace files:
In the jetson-containers directory replace "Dockerfile.ros.roxy" with the respective file downloaded above.
In the jetson-containers/scripts directory replace "docker_run.sh" withe the respective file downloaded above.
In the jetson-containers/packages directory replace "ros_entrypoint.sh" withe the respective file downloaded above.
5.
Build the container:
./scripts/docker_build_ros.sh foxy desktop
You may have to attempt to build multiple times if any of the downloads fail in the process.
This takes over an hour.
6.
Run the container:
./scripts/docker_run.sh -c ros:foxy-desktop-l4t-r32.5.1
Ctrl + d to exit and stop the container.


Adding profiles

Adding vertical profiles and temporal profiles for the emissions

I wrote an issue on the previous gitlab :
We want to generate the profiles of the emissions.
Requirements
Profiles are simply scaling factors to apply to the emission map.
Profiles must have dimensions:

time (for temporal)
level (for vertical)
Depending on the case, the scaling factors can depend once only on time and once only on level, or we could have a dual coordinate.
This means instead of having two vectors and selecting one value in each vector (time and level), we have a 2d array and we select one value at the time/level coordinate.

Profiles can have additional dimensions for which they vary:

substance
category (of the inventory)
grid cell

Data adding
This explains conceptually how the data will/can/must be added.
Vertical
Each source in the data can have a vertical value assigned to it.
This can be very easy in the gdfs to set a single value to each source.
For gridded emissions we need to assign a profile to the data.
Temporal
Each source must have a profile. we must assume that the sum of the values of the profile equals to one. considering a yearly profile.
However we might have daily, weekly, monthly profiles as well. These ones are not covering the full domain alone.
This means that their value don't sum to 1, but vary around 1.
Missing data
In the case of missing data it might be impossible to generate some kind of output.
In that case we want to make sure that we have a good way to specify the missing data.
Suggested solution
We need to adopt many parts of the code the this profiles and we don't want to change the existing framework too much.
In the inventory class, we want to add the following
Vertical data for the sources (gdfs)
gdfs should contain a 3d points or geometry such that we could extract the vertical data from them.
profiles attribute
This should be a xarray dataset containing all the profiles. A convention should be decided for how to treat exactly these things, but the use of multiple dimensions (with names following the conventions) should make it determinable what kind of scaling should be used for generating data higher than the 2-D maps.
general operations notes
When doing inventory operations, one should be careful how the profiles are implemented.
One should make sure that the results come as expected using testing on basic test inventories.
merging inventories
When putting two inventories together, it is possible to keep the original profiles of the two inventories, or to combine them. Combining profiles should be done using a weighted method scaling based on the emission values of the inventories.
If now we merge two inventories with different dimensions it should be possible to either downscale the dimensions of the profiles or to upscale them.
Test case
Roofs for zurich mapluft
For the heating sector we could use some of the height of the buildings to create the vertical profiles.
TNO profiles
TNO has profiles already shipped with it. We can use these to test reading to emiproc format and then exporting.


Not received airdrop

my username is joker31, if the distribution uses the alphabet then I should have got airdrop because it has arrived at the letter L now, I have the airdrop logo and also the spacedrop message but can't, why?

My usernam is mescouyes40 and i have the same problem. Can t received anything...

Same issue for my account  under username "marda". Please check the issue.

Keep in mind that there is still more than 40% of the XLM that needs to be distributed. https://stellar.expert/explorer/public/account/GDV4KECLSZLKRVH4ZTWVAS4I3W2LPAPV66ADFFUZKGIVOTK6GMKGJT53

I can look distribution still ongoing but goes to letter M and my username joker31 not received xlm

If your username letter M you can get today maybe

I don't think they distribute based on the alphabet, I think it's based on when you registered for the airdrop.

You can look distribution here sir
pacmam.keybase.pub


Cant access to my login web page

i have an issue when im trying to entry to my phishing adress (outlook) then there is no any
outlook login web site there is just an youtube video displays on my screen how can i fix this when im typing phishlets get-url outlook https://www.youtube.com (for example) then

lures create outlook
lures get-url 0
this is all
On Sat, Apr 13, 2019 at 1:29 PM asadsf5959<EMAIL_ADDRESS>wrote:

i have an issue when im trying to entry to my phishing adress (outlook)
then there is no any
outlook login web site there is just an youtube video displays on my
screen how can i fix this when im typing phishlets get-url outlook
https://www.youtube.com (for example) then
[image: Screenshot from 2019-04-12 23-58-18]
https://user-images.githubusercontent.com/49343531/56078039-eb82ef80-5d7e-11e9-9363-6a5c95fd8f14.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kgretzky/evilginx2/issues/276, or mute the thread
https://github.com/notifications/unsubscribe-auth/AuiNyvn0ehRbfu_LvVjGPzoYSXBPR0_kks5vgbFtgaJpZM4ct-N8
.

thanks for help btw how can i make a redirect url like victim when logs in after that how he can go to my website am i should make lures get-url www.redirecturl.com ?

just type help lures ...
On Sat, Apr 13, 2019 at 6:51 PM asadsf5959<EMAIL_ADDRESS>wrote:

thanks for help btw how can i make a redirect url like victim when logs in
after that how he can go to my website am i should make lures get-url
www.redirecturl.com ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kgretzky/evilginx2/issues/276#issuecomment-482825369,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AuiNyiC8yWSLEebatoAZ1HEKgPjCVR8_ks5vgfzzgaJpZM4ct-N8
.

thanks for everything really now working perfectly


[stable/node-problem-detector] Add relabelings to include node label in metrics

Signed-off-by: Patrick Harböck<EMAIL_ADDRESS>
What this PR does / why we need it:
Since #16125 node-problem-detector exposes metrics via a prometheus-operator ServiceMonitor.
These metrics did not include any labels to identify the relevant node.
This PR adds the necessary relabeling rules to include these labels (as suggested in https://github.com/kubernetes/node-problem-detector/issues/332).
Which issue this PR fixes

fixes https://github.com/kubernetes/node-problem-detector/issues/332

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @wangzhen127

@max-rocket-internet Is the behaviour of @k8s-ci-robot expected here? It does not seem to be able to assigned its suggested assignee.

/assign @andyxning

@max-rocket-internet Is the behaviour of @k8s-ci-robot expected here?

I never understand what the process is here. Users can be listed in Chart.yaml or in OWNERS but they can't be part of the process unless they are members of the Helm org I think?

/ok-to-test
/lgtm


lnd: 0.18.2 -> 0.18.3

Description of changes

version update

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

fyi @cypherpunk2140 @prusnak

@GrahamcOfBorg build lnd

backport in https://github.com/NixOS/nixpkgs/pull/341462


Update config for welcome app

This tweaks the configs for welcoming new users:

No longer point people towards probot/friction. I've seen multiple people open an issue here and then close it because the comment told them to. I think probot/friction is established enough now that we can just manually open issues over there as needed.
Point people toward https://probot.github.io/community/ to get more involved
Remove cat gifs, which will very funny, are a little intense, especially when you open your first PR and get it merged.
Use a slightly less intense (and more robot themed) gif to celebrated merged PRs

:robot: This image is excellent 🤖


qa_crowbarsetup: Only check vendordata if key is set

We only set the vendordata json blob in Cloud7+ so also do
the check only in Cloud7+.

+1

@aplanas can you have a look again? The fix was incomplete.

+1


Adding files and diffs for AMRL paper

@microsoft-github-policy-service agree [company="Microsoft"]

@microsoft-github-policy-service agree company=Microsoft


Msal-common post release January 2023 patch #2

Codecov Report

Merging #5597 (925a4dc) into dev (7472119) will not change coverage.
The diff coverage is n/a.

Flag
Coverage Δ

*Carryforward flag

msal-angular
96.50% <ø> (ø)

msal-browser
86.28% <ø> (ø)

Carriedforward from 7472119

msal-common
85.84% <ø> (ø)

msal-core
82.84% <ø> (ø)

Carriedforward from 7472119

msal-node
83.37% <ø> (ø)

msal-node-extensions
76.03% <ø> (ø)

Carriedforward from 7472119

msal-react
94.68% <ø> (ø)

node-token-validation
88.88% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.


Integration tests

We need to increase test coverage and extend integration tests to include networking, etc (see current integration tests here).
Work on removing any redundancy with current backend integration tests (ie, those that require starting amplifier), that can be defined by yaml test specs and added here.

Do you mind if I submit a PR for renaming the test files from:
00-invalid-sample-01.yml
00-invalid-sample-02.yml
01-create-service.yml
02-list-service.yml
03-curl-service.yml
04-remove-service.yml
05-create-stack.yml
06-list-stack.yml
07-stop-stack.yml
08-restart-stack.yml
09-remove-stack.yml
10-cpu-stats.yml
11-mem-stats.yml
12-io-stats.yml
13-task-stats.yml
14-container-stats.yml
15-net-stats.yml
16-node-stats.yml
17-service-stats.yml
18-stack-logs.yml
19-numbered-logs.yml
20-metadata-logs.yml
21-all-logs.yml

to
00-invalid-sample-01.yml
00-invalid-sample-02.yml
01-service-create.yml
02-service-list.yml
03-service-curl.yml
04-service-remove.yml
05-stack-create.yml
06-stack-list.yml
07-stack-stop.yml
08-stack-restart.yml
09-stack-remove.yml
10-stats-cpu.yml
11-stats-mem.yml
12-stats-io.yml
13-stats-task.yml
14-stats-container.yml
15-stats-net.yml
16-stats-node.yml
17-stats-service.yml
18-logs-stack.yml
19-logs-numbered.yml
20-logs-metadata.yml
21-logs-all.yml

?

@bquenin if you can wait a bit before renaming the yaml files, I have a PR in progress impacting those files, thanks.

I'd like them renamed, but there is no reason for them to be numbered sequentially, giving a false sense of them running in order (see discussion here: https://github.com/appcelerator/amp/issues/374#issuecomment-255879628). Just drop the numbers altogether.
Also, let's get rid of "invalid". That was copied over from the stack file parse tests where it made sense to indicate that the stack file format was invalid. In addition to having support for expected output (which can be an error message just as much as it can be anything else), we should also support an expected error code. It's not a bad idea to have a key for each test case that does indicated it is an negative ("unhappy") case test just so we can keep track and report in test results.
cc/ @ndegory @bquenin @JosephGJ @neha-viswanathan @generalhenry

IMO current status of CLI tests justifies that we close this PR (in time for 0.3.0)

We made good progress on testing. This is an ongoing effort, so I'm going to update the milestone.


Add functional test for UI Metric

Description
Add functional tests to validate the UI Metric Collector introduced in OpenSearch dashboards core. This PR is the same as
https://github.com/opensearch-project/opensearch-dashboards-functional-test/pull/1223 wit the updated changes based on comments which got closed by mistake.
Added test cases to validate server side batching of UI Metric reports introduced in https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6721.
Tests
https://github.com/opensearch-project/opensearch-dashboards-functional-test/assets/38322563/23f24815-0084-4448-bffa-dd92eb2753e1
https://github.com/opensearch-project/opensearch-dashboards-functional-test/assets/38322563/bbd3c5f8-ceaa-46d6-ba65-62bb28988668
https://github.com/opensearch-project/opensearch-dashboards-functional-test/assets/38322563/fa5d23c9-d542-4676-84ac-12ffdc0cf822
https://github.com/opensearch-project/opensearch-dashboards-functional-test/assets/38322563/0cf9d5f6-f52f-4f5c-9543-11c411b22b17
Check List

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

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

@LDrago27 Thanks for bringing this up. Is this PR targeting for 2.14?

@SuZhou-Joe These changes are not being targeted for 2.14 release.

will you handle the OSD CI to run these tests? with the new env variable set?

@kavilla I will update the OSD CI to include these tests. However i would like to wait till the changes in https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6721 are reviewed and merged , so that introducing these tests don't break the CIs.


diamond support

Following up on our conversation via email I have added diamond support.
To publish the package when you are ready, run the following commands:
Install diamond:
npm i -g diamondpkg
Register for an account:
diamond login
Publish the package (run in package dir)
diamond publish

I would not advise to merge this right now. As pointed out by another diamond user I will add support to pull package info directly from a package.json file for packages that don't use any of diamond's advanced features. This should not take more than a day.

On diamond v0.10.1, package info can be gathered from a package.json file if no diamond.json file is found. You can now run the above steps and not have to edit a diamond.json for stuff like new versions.

Hello, could you fix the conflict before in order for me to merge it. Thanks.

This PR can be closed. The Diamond project was abandoned in 2017 and their website is no longer up.


Relay New Issue

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Issue Description
<Describe your issue / question / feature request / etc..>

test

[Realsense Customer Engineering Team Comment]
Do you still need help on this?

No

test comment


Update air quality api

This PR updates the Air Quality clusters' API to match the latest pattern. This pattern detangles clusters from the zap generated code. The PR does the following,

Adds an Air Quality cluster server instance class. This class holds the attribute data and provides an API to get and update them.
Includes this cluster and its attributes to the list of AttributeAccessInterface manage attributes.
Updates the all-clusters-app to use this class.

closes #28498

Air-quality-sensor-app build for linux is broken by this commit

@UR6LAL Air-quality-sensor-app build for linux is broken by this commit.

This is known. @tcave-bb will be updating this example. This app is also currently broken by the recent changes to the ConcentrationMeasurement cluster.


get-room-id shows M_UNKNOWN: Expected RoomAlias string to start with '#', even when input starts with #

I'm trying to get the Room ID using get-room-id, so I can start sending messages to the right room only.
my $room-id = $client.get-room-id("#the-life-of-sjon:matrix.org");

However, I keep getting the following error message:
M_UNKNOWN: Expected RoomAlias string to start with '#'
  in method get at /home/tyil/projects/personal/perl6-Local-Sjon/../perl6-Matrix-Client/lib/Matrix/Client/Requester.pm6 (Matrix::Client::Requester) line 43
  in method get-room-id at /home/tyil/projects/personal/perl6-Local-Sjon/../perl6-Matrix-Client/lib/Matrix/Client.pm6 (Matrix::Client) line 316
  in sub MAIN at /home/tyil/projects/personal/perl6-Local-Sjon/lib/Local/Sjon.pm6 (Local::Sjon) line 27
  in block <unit> at bin/sjon line 4

This is confusing, as the string I'm passing does start with a #. Is there any way I can start looking to figure out where it's going wrong?

Hey @Tyil,
What's your version of Matrix::Client? I'm trying the same snippet without errors in the latest version (0.5.1)
% perl6
To exit type 'exit' or '^D'
> use Matrix::Client
Nil
> my $c = Matrix::Client.new(:home-server<https://matrix.deprecated.org>);
Matrix::Client.new(device-id => Str, home-server => "https://matrix.deprecated.org", access-token => "")
> $c.get-room-id('#deprecated:matrix.deprecated.org')
!CrhcYlwlZRJtxskYqv:matrix.deprecated.org
> $c.get-room-id('%23deprecated:matrix.deprecated.org')
!CrhcYlwlZRJtxskYqv:matrix.deprecated.org
> $c.get-room-id('#the-life-of-sjon:matrix.org')
!iwJJhLVhPkJqYLeaLg:matrix.org

The fix at the time was replacing # with %23.
Could you try with the latest version?

As an explanation of the error, the room alias is on the URI and not on the query params. Without escaping the # the room alias is taken as a fragment identifier.
% curl 'https://matrix.deprecated.org/_matrix/client/r0/directory/room/%23the-life-of-sjon:matrix.org'
{
    "room_id": "!iwJJhLVhPkJqYLeaLg:matrix.org",
    "servers": [
        "matrix.org"
    ]
}
% curl 'https://matrix.deprecated.org/_matrix/client/r0/directory/room/#the-life-of-sjon:matrix.org'
{
    "errcode": "M_UNKNOWN",
    "error": "Expected RoomAlias string to start with '#'"
}

And that's why you're seeing the M_UNKNOWN: Expected RoomAlias string to start with '#'

I'm not sure which version I have. I'll try to upgrade it to latest
version with zef tomorrow to ensure I'm on the latest release.
--
With kind regards,
Patrick Spek
www:  https://www.tyil.nl/
mail<EMAIL_ADDRESS>pgp:  1660 F6A2 DFA7 5347 322A  4DC0 7A6A C285 E2D9 8827
social: https://soc.fglt.nl/tyil
git:    https://gitlab.com/tyil/


init does not understand --frameworkversion

I'll do this next.

Cool, OK. My PR has changed a few things so might be worth checking everything still makes sense there first though.

Pr submitted

also added an arg for organisation

Fixed by #11


Sqlalchemy connection is missed when mysql restarts or the connection is renewed

Just notice an application that after some time starts to throws a 500 error instead of rendering the page correctly.
The root cause of this issue is the mysql connection is tunnelled and some times the daemon renews the connection. Pymysql connection is gone and a error similar to this one can be seem at the log:
WARNING - firenado.data[data:215] - 2016-07-30 22:49:44,773 - (pymysql.err.InternalError) Packet sequence number wrong - got 0 expected 1 [SQL: u'select now() as testtime;']
ERROR - sqlalchemy.pool.QueuePool[pool:643] - 2016-07-30 22:49:44,773 - Exception during reset or similar
Traceback (most recent call last):
  File "../venv/lib/python2.7/site-packages/sqlalchemy/pool.py", line 636, in _finalize_fairy
    fairy._reset(pool)
  File "../venv/lib/python2.7/site-packages/sqlalchemy/pool.py", line 774, in _reset
    self._reset_agent.rollback()
  File "../venv/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1563, in rollback
    self._do_rollback()
  File "../venv/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1601, in _do_rollback
    self.connection._rollback_impl()
  File "../venv/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 670, in _rollback_impl
    self._handle_dbapi_exception(e, None, None, None, None)
  File "../venv/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 1341, in _handle_dbapi_exception
    exc_info
  File "../venv/lib/python2.7/site-packages/sqlalchemy/util/compat.py", line 200, in raise_from_cause
    reraise(type(exception), exception, tb=exc_tb)
msm
  File "../venv/lib/python2.7/site-packages/sqlalchemy/engine/base.py", line 668, in _rollback_impl
session
    self.engine.dialect.do_rollback(self.connection)
  File "../venv/lib/python2.7/site-packages/sqlalchemy/dialects/mysql/base.py", line 2526, in do_rollback
    dbapi_connection.rollback()
  File "../venv/lib/python2.7/site-packages/pymysql/connections.py", line 771, in rollback
    self._execute_command(COMMAND.COM_QUERY, "ROLLBACK")
  File "../venv/lib/python2.7/site-packages/pymysql/connections.py", line 1054, in _execute_command
    self._write_bytes(packet)
  File "../venv/lib/python2.7/site-packages/pymysql/connections.py", line 1006, in _write_bytes
    raise err.OperationalError(2006, "MySQL server has gone away (%r)" % (e,))
OperationalError: (pymysql.err.OperationalError) (2006, "MySQL server has gone away (error(32, 'Broken pipe'))")
INFO - tornado.access[web:1946] - 2016-07-30 22:49:44,789 - 304 GET / (<IP_ADDRESS>) 18.22ms

This is not the original error but one I provoked trying to reproduce the issue.
The steps to reproduce the issue are:

Create an application with a mysql data source configured
Create a handler with some data coming from this mysql data source and render it on a template(or just print it on the request)
Start the application
Access the url of this handler and observe the data printed corrected on the response
Restart the mysql server
Refresh the page and observe the error on the console log

Here are some insight about the issue:

http://stackoverflow.com/questions/31121948/flask-sqlalchemy-cant-reconnect-until-invalid-transaction-is-rolled-back
http://stackoverflow.com/questions/23301968/invalid-transaction-persisting-across-requests
http://stackoverflow.com/questions/8645250/how-to-close-sqlalchemy-connection-in-mysql

This is happening on pgsql too. Let's fix it here.


Lighting breaks on window resize

The lighting techniques don't appear to work on window resize, even with the new preference listener system.

Fixed in https://github.com/Yattabyte/Delta/commit/485412d28ab1b2295f0183b26dfa596846da1363
Problem was the geometry buffer's resize() method https://github.com/Yattabyte/Delta/commit/485412d28ab1b2295f0183b26dfa596846da1363#diff-1d7eb0879a313c50cc1ce9714c23b9de
It did not use the correct internal format. It was using RGB for its textures instead of RGBA.


AppImageUpdate-Qt --version wants an X display

When running '--version' with AppImageUpdate-Qt I see the following issues:

Build 208: AppImageUpdate-Qt-208-012801f-x86_64.AppImage --version on a local machine returns this line:
   AppImageUpdate-Qt version 1-alpha (commit 012801f), build 208 built on 2018-02-15 21:55:43 UTC

This command does not use any GUI dialog to pop up, but it wants an X display to be present. Because the same command, when run via an SSH remote login returns this:
  AppImageUpdate-Qt version 1-alpha (commit 012801f), build 208 built on 2018-02-15 21:55:43 UTC
  qt.qpa.screen: QXcbConnection: Could not connect to display
  Could not connect to any X display.

So while it DOES give the required info through SSH as well, it still is ugly and does not feel right...

Build 212: AppImageUpdate-Qt-212-9c3a6d6-x86_64.AppImage --version does not work at all:
 execv error: No such file or directory

Note: the application itself seems to work just fine (though I didn't do any actual updates with it yet...).

duplicate of https://github.com/AppImage/AppImageUpdate/issues/70

Thanks for your notice, @antony-jr. I was about to close it as a duplicate. Please refer to #70, @KurtPfeifle.

I just discovered there's a second aspect in this issue that needs some answer.
I agree that the --version flag shouldn't require a UI. I am not a Qt expert, and I'll try to find out whether I can change the workflow in the code a bit so that there's no UI required for the CLI parsing.

I thought my chosen headline was indicating what I regarded as the FIRST aspect of this issue. The other thing ("execv error:...") I only found when I was looking for an more uptodate version to verify what I first had seen with build 208...

@TheAssassin The AppImageUpdate-Qt*.AppImage is for GUI , right ?
There is a appimageupdate-.AppImage for CLI usage , Then it is good to check for X Server when using AppImageUpdate-Qt.AppImage eventhough its just a version check. Am I wrong @KurtPfeifle ?

Yes, @antony-jr. But anyway, I do think any app shouldn't require an X server for non-UI work like displaying its version.
Also, nobody's explicitly checking for an X server. It's the Qt framework that implicitly checks for that. I'm interested in finding out the cause and fixing it.

@TheAssassin You can not do it qt , this is what happens when you force qt to compile.
QWidget: Cannot create a QWidget without QApplication
Aborted (core dumped)

QApplication always pulls the required resources when called. ( Like X Server ). The Workaround  is to use some custom command line parsing to exit the main function before you call QApplication::QApplication(int argc ,const char **argv);
From Qt Docs...

For any GUI application using Qt, there is precisely one QApplication object, no matter whether the application has 0, 1, 2 or more windows at any given time. For non-QWidget based Qt applications, use QGuiApplication instead, as it does not depend on the QtWidgets library.

Some GUI applications provide a special batch mode ie. provide command line arguments for executing tasks without manual intervention. In such non-GUI mode, it is often sufficient to instantiate a plain QCoreApplication to avoid unnecessarily initializing resources needed for a graphical user interface.

Thanks for your explanation. That was my guess, too. However, I'm not working on this with priority, as it isn't really preventing anyone from using the application.
IIRC you can use QCommandLineParser without a QApplication`. We'll have to see whether it's worth it to make these changes, though.

IIRC you can use QCommandLineParser without a QApplication

Yes thats true. @TheAssassin

@antony-jr:

"[....] its just a version check. Am I wrong @KurtPfeifle ?"

If I remember correctly, last I checked, almost ALL KDE and many Qt GUI applications could be run with certain CLI params (like '--help', '--help-kde', '--version')  which do not need an X server and which do not complain if an X server is missing and which print the requested info just to <stdout>...
One example of an AppImage which does not require an X server: my version of OpenSCAD which simply prints 'OpenSCAD version 2018.01.13' into the terminal for '--version', and a little help screen for '--help'.
Another example is LibreOffice: the AppImages there print a quite long output for '--help' and a one-line version string for '--version.
So my expectation regarding this topic isn't very exotic... I do not consider this issue to be very important, though. As my initial comment said, for me it just 'is ugly and does not feel right...'

for me it just 'is ugly and does not feel right...'

@KurtPfeifle  I Agree because the code is just in beta level , But I think @TheAssassin can fix this anytime. No need to rush. :+1:


First login post installation is a bit confusing

After following the steps to install Konflux and trying to login for the first time, the screen look confusing.
I would expect to see Konflux to verify I am in the right place
Same for the following screen

I see you point.
What we see in this screen is the adapters that are used in order to make use of different identity providers (i.e. when deployed in real envs, this will let you select google, github, etc.).
With this local deployment however, we use locally-defined dummy users, so it might look a little pointless.
Anyhow, this is something that we can look into, regardless.
This is something that is still new as we're still in the process of finalizing the integration of the new UI for the different downstream environments and this might also shape how it will look here.


Process Kill Experiment and cmdProbe modification for m-agent

Why We Need This PR
This PR adds the m-agent based Process Kill experiment and the inline cmdProbe modification for executing commands natively using m-agent
Special notes for your reviewer:

No breaking changes
Tested the experiment for positive and negative cases for Ubuntu, CentOS, RHEL, and SUSE Linux.

Checklist:

[ ] Fixes #
[ ] PR messages has document related information
[ ] Labelled this PR & related issue with breaking-changes tag
[ ] PR messages has breaking changes related information
[ ] Labelled this PR & related issue with requires-upgrade tag
[ ] PR messages has upgrade related information
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] E2E run Required for the changes

Experiment logs for successful cmd probe execution:
W0114 15:17:41.006930       1 client_config.go:541] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
time="2022-01-14T15:17:41Z" level=info msg="Experiment Name: process-kill"
time="2022-01-14T15:17:41Z" level=info msg="[PreReq]: Getting the ENV for the process-kill experiment"
time="2022-01-14T15:17:43Z" level=info msg="[PreReq]: Updating the chaos result of process-kill experiment (SOT)"
time="2022-01-14T15:17:47Z" level=info msg="[Info]: The process information is as follows" Process IDs=1608
time="2022-01-14T15:17:47Z" level=info msg="[Status]: Connecting to the agent"
time="2022-01-14T15:17:47Z" level=info msg="[Status]: Verify that the target processes are running"
time="2022-01-14T15:17:50Z" level=info msg="[Probe]: The cmd probe information is as follows" Phase=PreChaos Name=test-cmd-probe Command="cat /home/neelanjan/example.txt" Comparator="{string equal This is From GCP VM}" Source="{ false}" Run Properties="{5 2 1 0 0 false}" Mode=Edge
time="2022-01-14T15:17:50Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:17:50Z" level=info msg="[Probe]: {Actual value: This is From GCP VM}, {Expected value: This is From GCP VM}, {Operator: equal}"
time="2022-01-14T15:17:52Z" level=info msg="[Probe]: test-cmd-probe probe has been Passed 😄 " ProbeStatus=Passed ProbeName=test-cmd-probe ProbeType=cmdProbe ProbeInstance=PreChaos
time="2022-01-14T15:17:52Z" level=info msg="[Info]: Target processes list: [1608]"
time="2022-01-14T15:17:52Z" level=info msg="[Chaos]: Killing [1608] processes"
time="2022-01-14T15:17:52Z" level=info msg="[Chaos]: [1608] processes killed successfully"
time="2022-01-14T15:17:54Z" level=info msg="[Wait]: Waiting for chaos interval of 30s"
time="2022-01-14T15:18:24Z" level=info msg="[Confirmation]: process-kill chaos has been injected successfully"
time="2022-01-14T15:18:26Z" level=info msg="[Probe]: The cmd probe information is as follows" Mode=Edge Phase=PostChaos Name=test-cmd-probe Command="cat /home/neelanjan/example.txt" Comparator="{string equal This is From GCP VM}" Source="{ false}" Run Properties="{5 2 1 0 0 false}"
time="2022-01-14T15:18:26Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:18:29Z" level=info msg="[Probe]: test-cmd-probe probe has been Passed 😄 " ProbeType=cmdProbe ProbeInstance=PostChaos ProbeStatus=Passed ProbeName=test-cmd-probe
time="2022-01-14T15:18:29Z" level=info msg="[The End]: Updating the chaos result of process-kill experiment (EOT)"

Experiment run for unsuccessful cmd probe execution:
W0114 15:14:01.341103       1 client_config.go:541] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
time="2022-01-14T15:14:01Z" level=info msg="Experiment Name: process-kill"
time="2022-01-14T15:14:01Z" level=info msg="[PreReq]: Getting the ENV for the process-kill experiment"
time="2022-01-14T15:14:03Z" level=info msg="[PreReq]: Updating the chaos result of process-kill experiment (SOT)"
time="2022-01-14T15:14:07Z" level=info msg="[Info]: The process information is as follows" Process IDs=1452
time="2022-01-14T15:14:07Z" level=info msg="[Status]: Connecting to the agent"
time="2022-01-14T15:14:08Z" level=info msg="[Status]: Verify that the target processes are running"
time="2022-01-14T15:14:10Z" level=info msg="[Probe]: The cmd probe information is as follows" Comparator="{string equal This is From GCP VM}" Source="{ false}" Run Properties="{5 2 1 0 0 false}" Mode=Edge Phase=PreChaos Name=test-cmd-probe Command="cat /home/neelanjan/example.txt"
time="2022-01-14T15:14:10Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:14:10Z" level=info msg="[Probe]: {Actual value: This is A Different Text}, {Expected value: This is From GCP VM}, {Operator: equal}"
time="2022-01-14T15:14:10Z" level=error msg="the test-cmd-probe cmd probe has been Failed, err: {actual value: This is A Different Text} is not equal to {expected value: This is From GCP VM}"
time="2022-01-14T15:14:12Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:14:13Z" level=error msg="the test-cmd-probe cmd probe has been Failed, err: {actual value: This is A Different Text} is not equal to {expected value: This is From GCP VM}"
time="2022-01-14T15:14:15Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:14:15Z" level=error msg="the test-cmd-probe cmd probe has been Failed, err: {actual value: This is A Different Text} is not equal to {expected value: This is From GCP VM}"
time="2022-01-14T15:14:17Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:14:17Z" level=error msg="the test-cmd-probe cmd probe has been Failed, err: {actual value: This is A Different Text} is not equal to {expected value: This is From GCP VM}"
time="2022-01-14T15:14:19Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:14:20Z" level=error msg="the test-cmd-probe cmd probe has been Failed, err: {actual value: This is A Different Text} is not equal to {expected value: This is From GCP VM}"
time="2022-01-14T15:14:22Z" level=error msg="[Probe]: test-cmd-probe probe has been Failed 😢 " ProbeStatus=Failed ProbeName=test-cmd-probe ProbeType=cmdProbe ProbeInstance=PreChaos
time="2022-01-14T15:14:22Z" level=info msg="[Info]: Target processes list: [1452]"
time="2022-01-14T15:14:22Z" level=info msg="[Chaos]: Killing [1452] processes"
time="2022-01-14T15:14:23Z" level=info msg="[Chaos]: [1452] processes killed successfully"
time="2022-01-14T15:14:25Z" level=info msg="[Wait]: Waiting for chaos interval of 30s"
time="2022-01-14T15:14:55Z" level=info msg="[Confirmation]: process-kill chaos has been injected successfully"
time="2022-01-14T15:14:57Z" level=info msg="[Probe]: The cmd probe information is as follows" Run Properties="{5 2 1 0 0 false}" Mode=Edge Phase=PostChaos Name=test-cmd-probe Command="cat /home/neelanjan/example.txt" Comparator="{string equal This is From GCP VM}" Source="{ false}"
time="2022-01-14T15:14:57Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:14:57Z" level=error msg="the test-cmd-probe cmd probe has been Failed, err: {actual value: This is A Different Text} is not equal to {expected value: This is From GCP VM}"
time="2022-01-14T15:14:59Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:14:59Z" level=error msg="the test-cmd-probe cmd probe has been Failed, err: {actual value: This is A Different Text} is not equal to {expected value: This is From GCP VM}"
time="2022-01-14T15:15:01Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:15:02Z" level=error msg="the test-cmd-probe cmd probe has been Failed, err: {actual value: This is A Different Text} is not equal to {expected value: This is From GCP VM}"
time="2022-01-14T15:15:04Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:15:04Z" level=error msg="the test-cmd-probe cmd probe has been Failed, err: {actual value: This is A Different Text} is not equal to {expected value: This is From GCP VM}"
time="2022-01-14T15:15:06Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:15:06Z" level=error msg="the test-cmd-probe cmd probe has been Failed, err: {actual value: This is A Different Text} is not equal to {expected value: This is From GCP VM}"
time="2022-01-14T15:15:08Z" level=info msg="[Probe]: Executing Cmd Probe via agent"
time="2022-01-14T15:15:09Z" level=error msg="the test-cmd-probe cmd probe has been Failed, err: {actual value: This is A Different Text} is not equal to {expected value: This is From GCP VM}"
time="2022-01-14T15:15:11Z" level=error msg="[Probe]: test-cmd-probe probe has been Failed 😢 " ProbeType=cmdProbe ProbeInstance=PostChaos ProbeStatus=Failed ProbeName=test-cmd-probe
time="2022-01-14T15:15:11Z" level=info msg="[The End]: Updating the chaos result of process-kill experiment (EOT)"

Can we modify .github/workflows to push image with tag m-agent


Typo

In the table for the graph objects, there is a typo in the first row:
Uniquely enforced per partition. If a value isn't supplied upon insertion, and auto-generated GUID will be stored.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 34338360-13c3-e81e-9f64-86cbf4ab27ff
Version Independent ID: 6e84ff45-a77e-7b32-218c-ba3eeee2b349
Content: Graph data modeling for Azure Cosmos DB Gremlin API
Content Source: articles/cosmos-db/graph-modeling.md
Service: cosmos-db
Sub-service: cosmosdb-graph
GitHub Login: @LuisBosquez
Microsoft Alias: lbosq

@Tanguerjho Thank you for pointing out the issue, I have updated the doc accordingly.
The changes should be live EOD or tomorrow. We will close this issue now.
In future, if you see any minor bug fixes in the doc, feel free to click on the edit button and submit a PR to update.
We love taking contributions from the community.


pythonPackages.pytest-timeout: disable flaky test

Motivation for this change
ZHF: #80379
I could finally reproduce the failure on hydra by running the test in a i686-linux VM with stress -c 5. Only this test seems to fail reliably, so let's disable it!
Things done

[x] Tested using sandboxing (nix.useSandbox)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg build python27Packages.pytest-timeout python37Packages.pytest-timeout python38Packages.pytest-timeout

@GrahamcOfBorg build python27Packages.pytest-timeout python37Packages.pytest-timeout python38Packages.pytest-timeout

I think test_fix_setup_func_only, test_fix_finalizer_func_only, test_ini_timeout_func_only and test_timeout_marker_inheritance are also flaky (yes, that's a lot, but as I have said, "it's kinda the entire point of the module to make tests flaky").

I would say to try disabling this one first and see how hydra responds.

Ouch: mass rebuild.

Mass rebuild is fine, this is mostly Python code, so it should take an hour or so.

Do you think it's ok to push to master directly?

Yes.

Backported in 142dd6cb5dab. Let's see.


[D1] D1 API tutorial

Tutorial for creating an API to interact with D1 outside of Workers/Pages project

@ShaheenFattoe8 Can you run thru this tutorial and leave any user feedback as comments?

@harshil1712 We should host the full code example somewhere - to be paired with the tutorial.
@nevikashah this could make a good D1 template

@vy-ton agree that we should host the full code somewhere. Let check the process here. I'll update the tutorial with the link.

Made all the suggested changes and have added link to the finished GitHub repo.

Closing in favour of #16412


Clarify required React version?

I ran into some inscrutable ref errors trying to use these components on a project running React 16.2, and eventually realized that it relies on the createRef() API added in 16.3.
Assuming this is true across the board, it might be worth adding a note to the README that 16.3+ is required, or listing it as a peerDependency? (I can submit a PR if you can confirm that 16.3 is the intended min version, not sure whether there are other behaviors in later versions that it also relies on)

Thinking about this a bit more it seems like it might be preferable to make the min version 16.4.2, since that's what's currently being tested against - or to roll back the devDependencies to 16.3.0 - what do you think?

I think the earlier the version the wider the audience that can use the components. So my inclination is to rollback devDeps to 16.3.0. But like you said we've been using 16.4.2 as what has been tested against.  I think making the min version 16.4.2 is the safer bet.

Sounds good - it also wouldn't prevent people from using an earlier version, just warn.

Right, but they wouldn't be able to anyways with the different features we use in the repo. Or it would force them to update. So this at least will throw them a warning instead of them having sift through the changelog and figuring out why its not working (like you had to do).


ignore auction channel

Added config var for channel exclusion to bot listener

why is this needed? we can just remove bot access from the channel(s) via discord itself
in general, you rarely want to ignore a single thing, but rather an array or a list of channels (and then compare them with slices.Contains)


Set Error.prototype.cause type to unknown (#70)

Describe your changes
This PR fixes #45167 which changes the type of Error.prototype.cause from Error to Unknown
Testing performed
I added new tests to ensure that unknown type is recognised
Additional context
Could also potentially resolve #48098.  @DanielRosenwasser's suggested being more restrictive with the type of the cause property of the options param. I am open to discussions on this.
Fixes #45167
Credits to Bloomberg colleagues who helped with reviews @mkubilayk @dragomirtitian

Oh, apparently the Bloomberg CLA thing is being worked on by @RyanCavanaugh: https://github.com/microsoft/TypeScript/pull/49675#issuecomment-1165902970


FI-430 add include search tests

MedicationRequest and PractitionerRole both have _include search parameters that are "SHOULD" in US Core. You can test this with https://api-v8-r4.hspconsortium.org/czhour4/data and using Adams, Daniel X".
Alternatively, use the Community tests and the open endpoint https://api-v8-r4.hspconsortium.org/czhour4/open with patient-id SMART-1288992
Submitter:

[X] This pull request describes why these changes were made
[X] Internal ticket for this PR: https://oncprojectracking.healthit.gov/support/browse/FI-430
[X] Internal ticket links to this PR
[X] Internal ticket is properly labeled (Community/Program)
[X] Internal ticket has a justification for its Community/Program label
[X] Code diff has been reviewed for extraneous/missing code
[ ] Tests are included and test edge cases
[X] Tests/code quality metrics have been run locally and pass

Reviewer 1:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure
where appropriate, and accomplishes the task's purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Reviewer 2:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure
where appropriate, and accomplishes the task's purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Good catch. This bug was introduced for the delayed sequences because of an earlier PR that made inferno look through all returned resources for a search parameter. It should be fixed in this branch now.
Also the medicationrequest intent issue should be fixed in another PR, but you can test with PractitionerRole here. You need to run AllergyIntolerance first though.

Could you rebase @czh-orz

I merged with development and updated the test description to not include "SHOULD" wording.


Update mobile developer documentation to reflect release of mobile apps

The mobile developer machine setup documentation currently states that the Mattermost mobile apps are not yet released and will be released in March 2017.
Since the apps have now been released and are available on the Android and iOS stores, it makes sense to update the first line which says that the mobile apps are unreleased.

Thanks for the report @aditya-konarde :)  I've created a pull request to update the doc.

You're welcome @lindy65 :)
:+1: for the quick fix.


Change COMBINE import captions

Change the COMBINE import captions to:
Import COMBINE as Python
Import COMBINE as phrasedML

Fixed in sys-bio/spyderplugins@602a4cbb53bf3aa67e5c3c8c6dec517c2bcd095a


jdtls: add default data dir

https://github.com/pta2002/nixvim/pull/308/files is the better solution


How to install the addon?

I have some troubles with installing the addon. I download and open it as project in Studio, when i try to add it in my main project as Custom component, I get error that artifact not found

Check this guide: https://github.com/cuba-platform/cuba-dnd#step-by-step-guide
Use version that is compatible with your platform version: https://github.com/cuba-platform/cuba-dnd#compatibility-with-platform-versions
In the simplest case, you just add com.haulmont.addon.dnd:cuba-dnd-global:1.2.0 (for CUBA 6.8) as custom application component using CUBA Studio.

Yes it was a problem with Cuba version. But still can't use cuba-dnd. Because in my Controller i use DDVerticalLayout, which imported as fi.jasoft.dragdroplayouts.DDVerticalLayout, no options with com.haulmont.addon.dnd.components.


Introduce Cookie

Introducing cookies and removing Endpoint.embed (yay!).

Codecov Report

Merging #783 into master will decrease coverage by 1.18%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##           master     #783      +/-   ##
==========================================
- Coverage   80.95%   79.76%   -1.19%
==========================================
  Files          43       44       +1
  Lines         672      677       +5
  Branches       33       24       -9
==========================================
- Hits          544      540       -4
- Misses        128      137       +9

Impacted Files
Coverage Δ

core/src/main/scala/io/finch/Endpoint.scala
73.84% <ø> (-1.35%)
:arrow_down:

core/src/main/scala/io/finch/Endpoints.scala
66.66% <0%> (-33.34%)
:arrow_down:

core/src/main/scala/io/finch/internal/Rs.scala
87.5% <100%> (+0.83%)
:arrow_up:

core/src/main/scala/io/finch/endpoint/cookie.scala
45.45% <45.45%> (ø)

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 be8790f...ff284d8. Read the comment docs.


Fixing wrong type hint on CPUParticles2D's color ramp

Fixes #22795

Thanks, this should work fine but I'm wondering if it's intended that ParticlesMaterial and CPUParticles{,2D} use a different approach as mentioned in https://github.com/godotengine/godot/issues/22795#issuecomment-435868216
I'd like to have a comment from @reduz before merging.

As @mrcdk mentioned, "particles use the gpu so they need a texture, cpuparticles use the cpu so they don't", so it might indeed be purposely that CPUParticles{,2D} use Gradient, so this fix seems correct.
It should also be done for CPUParticles though, not only for 2D.

I amended the CPUParticles fix and merged manually as a8a92ec11. Thanks!


More complex phSetpoint calculation

The mode and expert setting is now taken into account.

This should fix #24


feat(Terra): CW-5718 Terra CW-20 Transfer

Terra CW-20 Transfer
Include supported/official tokens
Testing/Implemented in coin-tester

Task linked: CW-5718 [SDK] Implement LUNA to support CW-20 token transfer


Update plugin to 4.0.0 to work for<EMAIL_ADDRESS>

Thanks for your interest in palantir/grunt-tslint, @Izzmo! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

@palantirtech done! Thanks.

@nchen63 can you please take a look

@Izzmo are you still working on this PR?

@helfer yes sorry I'll finish it up tonight! The holidays got ahead of me.

@izzmo any updates?

Sorry guys, I'm having a heck of a time getting the tests to work. It would seem these haven't worked in a while though... so what should we do? @helfer @bondz

@helfer @adidahiya I updated the file to not create a program. Again, half the tests work.. half don't. I think this is basically because no one has kept them up to date with new dependency versions and node itself.

Ping @nchen63


Is there an established property name for the SDK?

Is there any way to find the name of the SDK during a build? for example $(SdkName) ? (This obviously doesn't exist - I tried it).

A project can import multiple SDKs so there is no one property that represents this.  But each SDK usually sets a property:
https://github.com/microsoft/MSBuildSdks/blob/66c96f32bae6d73faf05aa01aa10a56db750c4c6/src/Traversal/Sdk/Sdk.props#L10
https://github.com/microsoft/MSBuildSdks/blob/66c96f32bae6d73faf05aa01aa10a56db750c4c6/src/NoTargets/Sdk/Sdk.props#L10
https://github.com/dotnet/sdk/blob/53087be78307466344a83ac92ba596aeb45b435d/src/Tasks/Microsoft.NET.Build.Tasks/sdk/Sdk.props#L24
But not every SDK does this so it may be hit or miss.

@jeffkl - Thanks, this was of some help.
My problem remains though, a "legacy" .Net Framework project's nuget behavior in VS2019 is messed up by the mere presence of a Directory.Build.props file, even when I experimentally add conditions.
Here's the file:
<Project>
	<ItemGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'And'$(UsingMicrosoftNETSdk)'=='true'">
		<PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.0.0" PrivateAssets="All"/>
	</ItemGroup>

	<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'And'$(UsingMicrosoftNETSdk)'=='true'">
		<DebugType>embedded</DebugType>
		<DebugSymbols>true</DebugSymbols>
		<GitRepositoryRemoteName>principal</GitRepositoryRemoteName>
	</PropertyGroup>

	<Target Name="ConvertCommitIdToUpperCase" AfterTargets="InitializeSourceControlInformationFromSourceControlManager">
	  <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'And'$(UsingMicrosoftNETSdk)'=='true'">
		<SourceRevisionId>$(SourceRevisionId.ToUpperInvariant().Substring(0,16).PadLeft(17))</SourceRevisionId>
	  </PropertyGroup>
	</Target>
 </Project>

If I put that at the root of a repo then legacy projects in the repo suffer, they can build but in the nuget package manager it says that the only installed package is the SourceLink one and sees none of the actual packages in the packages.config.
I added that condition to see if this might help but no!
The only way around is for me to put a empty (just an empty Project node) Directory.Build.config in the solution's folder and that seems to "override" this and the legacy project behaves normally once more.

Man...
https://github.com/dotnet/project-system/issues/4175

I usually wrap my Directory.Build.props/targets with this if it becomes to annoying. (Old projects and visual studio not evaluating conditions for example 😂)
´´´

  ...

´´´


Add optional menu depending on config.

If relevant section is disabled, menu should stays hidden.
Relevant section title should be the text of nav bar.

Great job. Thank you.


Delayed output

Following up from https://groups.google.com/forum/#!topic/julia-users/nyb9qRJgzpo.
The issue is that the display of output to STDOUT is interrupted / delayed by computations. This test code
print(1:4); flush(STDOUT); svd(rand(1000,1000)); println("done");

prints "1:" and after the svd is done "4done", so flushing does not seem to help here.
This started to occur some time in the last weeks on Julia master, in 0.3.11 it works.

The issue is that the sending of stdout messages to Jupyter is handled by a separate green thread (it doesn't pre-empt svd), and exactly when that gets woken up is somewhat random.
Maybe IJulia could hook into flush to force it to wake up the stdout thread.

This may be an unrelated issue, but in 0.4 if you "run all cells" or similar you can end up with the output of anything that prints in the wrong cell:

@amellnik, that is #342.

Ah, apologies, I didn't look far enough back.

Actually, IJulia already hooks into flush.  It is just that it looks like the hooks broke in Julia 0.4.  Let me look into it further.

Okay, I fixed it so that the IJulia flush is actually being called.   However, at the time flush is called, after 1: has printed but before the 4 has printed, nb_available(STDOUT) is returning 0, so it thinks there is nothing to send.
@vtjnash, is this some kind of buffering?  How can the flush command get everything that has been written to the pipe at this point?

The only way to get the kernel to give you everything on the pipe (with certainty), is to close the write end of the pipe and redirect_stdout to a new pipe for each new cell.

@vtjnash, that's not really acceptable for us, because there might be an @async task from a previous input cell still running and printing I/O to STDOUT.
So, this is really a kernel thing and there's nothing we can do about it?

Can we get the kernel to treat the STDOUT pipe as a pseudoterminal, so that it won't buffer I/O?

I don't see how to have printing from @async to work at an arbitrary time later, but also expect to know when all output has been received.

@vtjnash, I just want to stop the kernel from buffering STDOUT.

The problem is that it's really expensive to ask the kernel for that data (and the kernel really dislikes actually buffering any data), so it is a slightly difficult problem to coordinate flushing everyone's buffers. On linux, we know that the libuv read size > default kernel buffer size, so calling yield() once after flush() has returned should suffice to return all of the buffered data. (make sure you have an @async task reading the data or flush() will deadlock however).
On windows, ReadFile doesn't have a non-blocking version (the non-blocking version, PeakNamedPipe, is documented to never block, except for when it does), so I think you would need to call ccall(:SwitchToThread, Void, stdcall, ()); yield() to have a pretty good chance to read all of the buffered data from the kernel (assuming you had a separate @async task that is doing the read created first)

"Pretty good chance" seems like an improvement.

In our case, there are multiple @async tasks going on, so just yield seems insufficient — couldn't it wake up one of the other tasks?  And there are separate tasks for STDOUT and STDERR.

it's a fair scheduler, so you can safely assume it will wake up all of them. come to think of it though, you may need to call yield() twice (in case the reader task was already queued with some of the data, because you would need yield to call process_events() with the reader task is actually waiting for the data)

Does anyone know what is the status of this issue? Based on recent closed issues, I assumed this was mostly fixed. However, after updating Julia (0.4.1) and IJulia, I found that I get a similar issue:
After executing @time sum(linspace(0,1,1e6)), there is a stream message after the execution_state: idle message is received:
In [25]: kc.execute('@time sum(linspace(0,1,1e6))')
Out[25]: 'da1ad3fc-1937-4b91-a719-cd9de44adeee'

In [26]: kc.get_shell_msg(timeout=1)
Out[26]:
{'buffers': [],
 'content': {u'execution_count': 3,
  u'payload': u'',
  u'status': u'ok',
  u'user_expressions': {}},
 'header': {'date': '2015-12-03T17:10:58.550513',
  u'msg_id': u'1aa0aa87-0cf7-4b1e-a114-aeafb5b3b823',
  u'msg_type': u'execute_reply',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'},
 'metadata': {},
 'msg_id': u'1aa0aa87-0cf7-4b1e-a114-aeafb5b3b823',
 'msg_type': u'execute_reply',
 'parent_header': {u'date': datetime.datetime(2015, 12, 3, 17, 10, 56, 260207),
  u'msg_id': u'da1ad3fc-1937-4b91-a719-cd9de44adeee',
  u'msg_type': u'execute_request',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'}}

In [27]: kc.get_iopub_msg(timeout=1)
Out[27]:
{'buffers': [],
 'content': {u'execution_state': u'busy'},
 'header': {'date': '2015-12-03T17:11:03.161155',
  u'msg_id': u'a0d3a2c7-2c76-4a9c-88ed-b3c8786b80b6',
  u'msg_type': u'status',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'jlkernel',
  u'version': u'5.0'},
 'metadata': {},
 'msg_id': u'a0d3a2c7-2c76-4a9c-88ed-b3c8786b80b6',
 'msg_type': u'status',
 'parent_header': {u'date': datetime.datetime(2015, 12, 3, 17, 10, 56, 260207),
  u'msg_id': u'da1ad3fc-1937-4b91-a719-cd9de44adeee',
  u'msg_type': u'execute_request',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'}}

In [28]: kc.get_iopub_msg(timeout=1)
Out[28]:
{'buffers': [],
 'content': {u'code': u'@time sum(linspace(0,1,1e6))', u'execution_count': 3},
 'header': {'date': '2015-12-03T17:11:04.063564',
  u'msg_id': u'2cf0d337-edc9-465a-9547-c551ec7f72ca',
  u'msg_type': u'execute_input',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'},
 'metadata': {},
 'msg_id': u'2cf0d337-edc9-465a-9547-c551ec7f72ca',
 'msg_type': u'execute_input',
 'parent_header': {u'date': datetime.datetime(2015, 12, 3, 17, 10, 56, 260207),
  u'msg_id': u'da1ad3fc-1937-4b91-a719-cd9de44adeee',
  u'msg_type': u'execute_request',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'}}

In [29]: kc.get_iopub_msg(timeout=1)
Out[29]:
{'buffers': [],
 'content': {u'name': u'stdout', u'text': u'  '},
 'header': {'date': '2015-12-03T17:11:05.140277',
  u'msg_id': u'931bf8b7-6d7e-4aff-bf76-d814adb73fd8',
  u'msg_type': u'stream',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'},
 'metadata': {},
 'msg_id': u'931bf8b7-6d7e-4aff-bf76-d814adb73fd8',
 'msg_type': u'stream',
 'parent_header': {u'date': datetime.datetime(2015, 12, 3, 17, 10, 56, 260207),
  u'msg_id': u'da1ad3fc-1937-4b91-a719-cd9de44adeee',
  u'msg_type': u'execute_request',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'}}

In [30]: kc.get_iopub_msg(timeout=1)
Out[30]:
{'buffers': [],
 'content': {u'data': {u'text/plain': u'499999.99999999994'},
  u'execution_count': 3,
  u'metadata': {}},
 'header': {'date': '2015-12-03T17:11:06.130795',
  u'msg_id': u'cce658a5-fa95-452a-adde-943f9d1bed95',
  u'msg_type': u'execute_result',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'},
 'metadata': {},
 'msg_id': u'cce658a5-fa95-452a-adde-943f9d1bed95',
 'msg_type': u'execute_result',
 'parent_header': {u'date': datetime.datetime(2015, 12, 3, 17, 10, 56, 260207),
  u'msg_id': u'da1ad3fc-1937-4b91-a719-cd9de44adeee',
  u'msg_type': u'execute_request',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'}}

In [31]: kc.get_iopub_msg(timeout=1)
Out[31]:
{'buffers': [],
 'content': {u'execution_state': u'idle'},
 'header': {'date': '2015-12-03T17:11:08.095832',
  u'msg_id': u'f172ff7c-d87e-45a6-8752-5ca2ee4527f8',
  u'msg_type': u'status',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'jlkernel',
  u'version': u'5.0'},
 'metadata': {},
 'msg_id': u'f172ff7c-d87e-45a6-8752-5ca2ee4527f8',
 'msg_type': u'status',
 'parent_header': {u'date': datetime.datetime(2015, 12, 3, 17, 10, 56, 260207),
  u'msg_id': u'da1ad3fc-1937-4b91-a719-cd9de44adeee',
  u'msg_type': u'execute_request',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'}}

In [32]: kc.get_iopub_msg(timeout=1)
Out[32]:
{'buffers': [],
 'content': {u'name': u'stdout',
  u'text': u'0.023915 seconds (6.76 k allocations: 280.694 KB)\n'},
 'header': {'date': '2015-12-03T17:11:09.247268',
  u'msg_id': u'34eda40c-821b-4686-a51d-0add17dbb970',
  u'msg_type': u'stream',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'},
 'metadata': {},
 'msg_id': u'34eda40c-821b-4686-a51d-0add17dbb970',
 'msg_type': u'stream',
 'parent_header': {u'date': datetime.datetime(2015, 12, 3, 17, 10, 56, 260207),
  u'msg_id': u'da1ad3fc-1937-4b91-a719-cd9de44adeee',
  u'msg_type': u'execute_request',
  u'session': u'4f3df0fa-34a0-480b-96fc-ffb4a3c7cc7e',
  u'username': u'user',
  u'version': u'5.0'}}

So basically, the pattern is the following:
'content': {u'execution_state': u'busy'},
'content': {u'code': u'@time sum(linspace(0,1,1e6))', u'execution_count': 3},
'content': {u'name': u'stdout', u'text': u'  '},
'content': {u'data': {u'text/plain': u'499999.99999999994'},
'content': {u'execution_state': u'idle'},
'content': {u'name': u'stdout', u'text': u'0.023915 seconds (6.76 k allocations: 280.694 KB)\n'},

cc @JobJob

@rsmith31415 are you on master?
Pkg.checkout("IJulia")

I have the same output on the current tagged version (the default one installed by Pkg.add("IJulia")) but it's fixed for me with the latest master.
If you are on the latest master - what OS are you using here?

Oh, sorry. I thought the changes had been made available in the tagged version. No, everything looks okay in master.

Great!


warning: comparison of integer expressions of different signedness

Platform: RPI 2
OS: Ubuntu 32 Bit
Uname: Linux raspberrypi 4.19.50-v7+ #896 SMP Thu Jun 20 16:11:44 BST 2019 armv7l GNU/Linux
CC:
➜  build git:(master) cc -v
Using built-in specs.
COLLECT_GCC=cc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/arm-linux-gnueabihf/8/lto-wrapper
Target: arm-linux-gnueabihf
Configured with: ../src/configure -v --with-pkgversion='Raspbian 8.3.0-6+rpi1' --with-bugurl=file:///usr/share/doc/gcc-8/README.Bugs --enable-languages=c,ada,c++,go,d,fortran,objc,obj-c++ --prefix=/usr --with-gcc-major-version-only --program-suffix=-8 --program-prefix=arm-linux-gnueabihf- --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --enable-bootstrap --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-libitm --disable-libquadmath --disable-libquadmath-support --enable-plugin --with-system-zlib --with-target-system-zlib --enable-objc-gc=auto --enable-multiarch --disable-sjlj-exceptions --with-arch=armv6 --with-fpu=vfp --with-float=hard --disable-werror --enable-checking=release --build=arm-linux-gnueabihf --host=arm-linux-gnueabihf --target=arm-linux-gnueabihf
Thread model: posix
gcc version 8.3.0 (Raspbian 8.3.0-6+rpi1)

[  3%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_asconf.c.o
[  3%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_bsd_addr.c.o
[  3%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_auth.c.o
[  4%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_asconf.c.o
[  5%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_callout.c.o
[  6%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_cc_functions.c.o
[  7%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_crc32.c.o
[  8%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_indata.c.o
[  8%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_input.c.o
[  9%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_auth.c.o
[ 10%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_output.c.o
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c: In function ‘sctp_lower_sosend’:
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c:13898:14: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘uint32_t’ {aka ‘unsigned int’} [-Wsign-compare]
   if (sndlen > asoc->smallest_mtu) {
              ^
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c:13933:15: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘u_int’ {aka ‘unsigned int’} [-Wsign-compare]
    if (sndlen > SCTP_SB_LIMIT_SND(so))
               ^
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c:14129:14: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘u_int’ {aka ‘unsigned int’} [-Wsign-compare]
      (sndlen > SCTP_SB_LIMIT_SND(stcb->sctp_socket))) {
              ^
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c:14158:16: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘uint32_t’ {aka ‘unsigned int’} [-Wsign-compare]
  if (((max_len <= local_add_more) &&
                ^~
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c:14312:17: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘uint32_t’ {aka ‘unsigned int’} [-Wsign-compare]
    if ((max_len > SCTP_BASE_SYSCTL(sctp_add_more_threshold)) ||
                 ^
[ 11%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_bsd_addr.c.o
[ 12%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_callout.c.o
[ 13%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_cc_functions.c.o
[ 14%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_crc32.c.o
[ 15%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_indata.c.o
[ 16%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_pcb.c.o
[ 17%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_peeloff.c.o
[ 18%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_sha1.c.o
[ 19%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_ss_functions.c.o
[ 20%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_sysctl.c.o
[ 21%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_timer.c.o
[ 21%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_userspace.c.o
[ 22%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctp_usrreq.c.o
[ 22%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_input.c.o
[ 23%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet/sctputil.c.o
[ 24%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/netinet6/sctp6_usrreq.c.o
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctputil.c: In function ‘sctp_sorecvmsg’:
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctputil.c:6447:23: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘unsigned int’ [-Wsign-compare]
        (copied_so_far >= (uint32_t)max(so->so_rcv.sb_lowat, 1)))) {
                       ^~
[ 25%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/user_environment.c.o
[ 26%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/user_mbuf.c.o
[ 27%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/user_recv_thread.c.o
[ 28%] Building C object usrsctplib/CMakeFiles/usrsctp.dir/user_socket.c.o
[ 29%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_output.c.o
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c: In function ‘sctp_lower_sosend’:
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c:13898:14: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘uint32_t’ {aka ‘unsigned int’} [-Wsign-compare]
   if (sndlen > asoc->smallest_mtu) {
              ^
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c:13933:15: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘u_int’ {aka ‘unsigned int’} [-Wsign-compare]
    if (sndlen > SCTP_SB_LIMIT_SND(so))
               ^
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c:14129:14: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘u_int’ {aka ‘unsigned int’} [-Wsign-compare]
      (sndlen > SCTP_SB_LIMIT_SND(stcb->sctp_socket))) {
              ^
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c:14158:16: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘uint32_t’ {aka ‘unsigned int’} [-Wsign-compare]
  if (((max_len <= local_add_more) &&
                ^~
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctp_output.c:14312:17: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘uint32_t’ {aka ‘unsigned int’} [-Wsign-compare]
    if ((max_len > SCTP_BASE_SYSCTL(sctp_add_more_threshold)) ||
                 ^
[ 30%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_pcb.c.o
[ 31%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_peeloff.c.o
[ 31%] Linking C shared library libusrsctp.so
[ 32%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_sha1.c.o
[ 32%] Built target usrsctp
[ 33%] Building C object programs/CMakeFiles/tsctp_upcall.dir/tsctp_upcall.c.o
[ 34%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_ss_functions.c.o
[ 35%] Building C object programs/CMakeFiles/tsctp_upcall.dir/programs_helper.c.o
[ 35%] Linking C executable tsctp_upcall
[ 35%] Built target tsctp_upcall
[ 36%] Building C object programs/CMakeFiles/tsctp.dir/tsctp.c.o
[ 37%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_sysctl.c.o
[ 38%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_timer.c.o
[ 39%] Building C object programs/CMakeFiles/tsctp.dir/programs_helper.c.o
[ 40%] Linking C executable tsctp
[ 40%] Built target tsctp
[ 41%] Building C object programs/CMakeFiles/test_libmgmt.dir/test_libmgmt.c.o
[ 42%] Building C object programs/CMakeFiles/test_libmgmt.dir/programs_helper.c.o
[ 43%] Linking C executable test_libmgmt
[ 43%] Built target test_libmgmt
[ 43%] Building C object programs/CMakeFiles/chargen_server_upcall.dir/chargen_server_upcall.c.o
[ 44%] Building C object programs/CMakeFiles/chargen_server_upcall.dir/programs_helper.c.o
[ 44%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_userspace.c.o
[ 45%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctp_usrreq.c.o
[ 46%] Linking C executable chargen_server_upcall
[ 46%] Built target chargen_server_upcall
[ 47%] Building C object programs/CMakeFiles/ekr_client.dir/ekr_client.c.o
[ 48%] Building C object programs/CMakeFiles/ekr_client.dir/programs_helper.c.o
[ 49%] Linking C executable ekr_client
[ 49%] Built target ekr_client
[ 50%] Building C object programs/CMakeFiles/discard_server.dir/discard_server.c.o
[ 50%] Building C object programs/CMakeFiles/discard_server.dir/programs_helper.c.o
[ 51%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet/sctputil.c.o
[ 52%] Linking C executable discard_server
[ 52%] Built target discard_server
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctputil.c: In function ‘sctp_sorecvmsg’:
/home/weinrank/github/usrsctp/usrsctplib/netinet/sctputil.c:6447:23: warning: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘int’} and ‘unsigned int’ [-Wsign-compare]
        (copied_so_far >= (uint32_t)max(so->so_rcv.sb_lowat, 1)))) {
                       ^~
[ 53%] Building C object programs/CMakeFiles/ekr_peer.dir/ekr_peer.c.o
[ 53%] Building C object programs/CMakeFiles/ekr_peer.dir/programs_helper.c.o
[ 54%] Linking C executable ekr_peer
[ 54%] Built target ekr_peer
[ 55%] Building C object programs/CMakeFiles/daytime_server.dir/daytime_server.c.o
[ 56%] Building C object programs/CMakeFiles/daytime_server.dir/programs_helper.c.o
[ 57%] Linking C executable daytime_server
[ 57%] Built target daytime_server
[ 58%] Building C object programs/CMakeFiles/rtcweb.dir/rtcweb.c.o
[ 59%] Building C object programs/CMakeFiles/rtcweb.dir/programs_helper.c.o
[ 60%] Linking C executable rtcweb
[ 60%] Built target rtcweb
[ 60%] Building C object programs/CMakeFiles/http_client_upcall.dir/http_client_upcall.c.o
[ 61%] Building C object programs/CMakeFiles/http_client_upcall.dir/programs_helper.c.o
[ 62%] Linking C executable http_client_upcall
[ 62%] Built target http_client_upcall
[ 63%] Building C object programs/CMakeFiles/client_upcall.dir/client_upcall.c.o
[ 64%] Building C object programs/CMakeFiles/client_upcall.dir/programs_helper.c.o
[ 65%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/netinet6/sctp6_usrreq.c.o
[ 65%] Linking C executable client_upcall
[ 65%] Built target client_upcall
[ 65%] Building C object programs/CMakeFiles/echo_server_upcall.dir/echo_server_upcall.c.o
[ 66%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/user_environment.c.o
[ 67%] Building C object programs/CMakeFiles/echo_server_upcall.dir/programs_helper.c.o
[ 68%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/user_mbuf.c.o
[ 69%] Building C object programs/CMakeFiles/client.dir/client.c.o
[ 70%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/user_recv_thread.c.o
[ 71%] Linking C executable echo_server_upcall
[ 72%] Building C object programs/CMakeFiles/client.dir/programs_helper.c.o
[ 72%] Built target echo_server_upcall
[ 73%] Building C object programs/CMakeFiles/echo_server.dir/echo_server.c.o
[ 74%] Linking C executable client
[ 75%] Building C object programs/CMakeFiles/echo_server.dir/programs_helper.c.o
[ 75%] Built target client
[ 76%] Building C object programs/CMakeFiles/ekr_loop.dir/ekr_loop.c.o
[ 77%] Linking C executable echo_server
[ 78%] Building C object usrsctplib/CMakeFiles/usrsctp-static.dir/user_socket.c.o
[ 78%] Built target echo_server
[ 79%] Building C object programs/CMakeFiles/ekr_loop.dir/programs_helper.c.o
[ 80%] Building C object programs/CMakeFiles/discard_server_upcall.dir/discard_server_upcall.c.o
[ 81%] Building C object programs/CMakeFiles/ekr_server.dir/ekr_server.c.o
[ 83%] Building C object programs/CMakeFiles/discard_server_upcall.dir/programs_helper.c.o
[ 83%] Building C object programs/CMakeFiles/ekr_server.dir/programs_helper.c.o
[ 83%] Linking C executable ekr_loop
[ 83%] Built target ekr_loop
[ 83%] Building C object programs/CMakeFiles/test_timer.dir/test_timer.c.o
[ 84%] Linking C executable ekr_server
[ 85%] Linking C executable discard_server_upcall
[ 86%] Building C object programs/CMakeFiles/test_timer.dir/programs_helper.c.o
[ 86%] Built target ekr_server
[ 86%] Built target discard_server_upcall
[ 87%] Building C object programs/CMakeFiles/daytime_server_upcall.dir/daytime_server_upcall.c.o
[ 88%] Building C object programs/CMakeFiles/ekr_loop_offload.dir/ekr_loop_offload.c.o
[ 89%] Linking C executable test_timer
[ 90%] Building C object programs/CMakeFiles/daytime_server_upcall.dir/programs_helper.c.o
[ 90%] Built target test_timer
[ 91%] Building C object programs/CMakeFiles/ekr_loop_upcall.dir/ekr_loop_upcall.c.o
[ 92%] Building C object programs/CMakeFiles/ekr_loop_offload.dir/programs_helper.c.o
[ 93%] Linking C executable daytime_server_upcall
[ 93%] Built target daytime_server_upcall
[ 94%] Linking C executable ekr_loop_offload
[ 95%] Building C object programs/CMakeFiles/http_client.dir/http_client.c.o
[ 96%] Building C object programs/CMakeFiles/ekr_loop_upcall.dir/programs_helper.c.o
[ 96%] Built target ekr_loop_offload
[ 97%] Building C object programs/CMakeFiles/http_client.dir/programs_helper.c.o
[ 98%] Linking C executable ekr_loop_upcall
[ 99%] Linking C executable http_client
[ 99%] Built target ekr_loop_upcall
[ 99%] Built target http_client
[100%] Linking C static library libusrsctp.a
[100%] Built target usrsctp-static

Any idea why this shows up now? I don't think the code has changed. So a new version of gcc?

I think this issue is related to the 32 bit architecture.
We currently don't have a 32 bit machine in the buildbot setup, I'll add an 32 Bit RPI in our buildbot setup.

Hmm, but the issues are not about size mismatch (sizeof(int) != sizeof(long)), but are all related to signedness issues. So are you using a different compiler flags or a new (or old) compiler?
Are you referring to a RPI4 (like the one sitting on my table)?


window.URL.createObjectURL doesn't work with gopher:// changes

I load user images from the web via XHR. This used to work when I would do this:
    var xhr = new XMLHttpRequest();
	xhr.open('GET', imageUrl, true);
	xhr.responseType = 'blob';
	xhr.onload = function(e) {
	  var img = document.getElementById('profileImage');
	  img.src = window.URL.createObjectURL(this.response);

	};

	xhr.send();

Now with the new gopher URLs in 0.7.3, I get the following error on Android:
I/chromium( 5960): [INFO:CONSOLE(0)] "Refused to load the image 'blob:file%3A///35b71821-e3d9-43cd-a560-6c29d499c012' because it violates the following Content Security Policy directive: "default-src file: data: chrome-extension: gopher: https://ssl.gstatic.com". Note that 'img-src' was not explicitly set, so 'default-src' is used as a fallback.

Looks like the problem was unrelated to the 0.7.3 changes, but rather from the latest Android WebView update changing Content-Security-Policy to block blob: urls (CSPv1 didn't have them blocked, but CSPv2 adds them to the block list).
I've published an update to add them to the CSP whitelist (v0.7.4)


Fixed broken email hyperlink in Contributing Guide

This pull request fixes the broken email hyperlink in the CONTRIBUTING.md file by replacing it with a valid mailto link. This ensures users can contact the mailing list without encountering navigation errors.

This PR is stale because it has been open for 30 days with no activity. Please remove stale label or add any comment to keep this open. Otherwise this will be closed in 5 days.

This PR was closed because it has been inactive for 35 days. Please re-open if this PR is still relevant.


Add St. Xavier's College to the list

Official website URL - http://www.sxc.edu.np
IT course offered by University - http://cs.sxc.edu.np/index.php

@npankaj365 Pull request merged. Thank you!


Error when running recent versions

Hello,
I am unable to run the Cecret pipeline versions 3.22.24345 or 3.22.24331. Up through version 2015.24324 the following code was working:
/Shared/SHL-BUG/software/nextflow/nextflow run UPHL-BioNGS/Cecret -r master  -profile singularity,artic_V5_3_2 --reads ${run_dir} --aci false --samtools_plot_ampliconstats false -c /Shared/SHL-BUG/singularity/Cecret/Cecret_WW_custom.config
However, when running with the new versions mentioned above I get the following error in the nextflow log:
Dec-11 09:49:28.119 [Actor Thread 53] INFO  nextflow.script.BaseScript - FATAL : No input files were found!
Dec-11 09:49:28.120 [Actor Thread 53] INFO  nextflow.script.BaseScript - No paired-end fastq files were found at .. Set 'params.reads' to directory with paired-end reads
Do parameters such as --reads need to be specified in a different way following the moderate update?

Oh no! This should work fine.
Can you share the entire error message?

This is puzzling. Perhaps it's a filename issue?
What are the names of your fastq files at /Shared/SHL-BUG/workspaces/whottel/scratch/Illumina_runs/IA-VL00618-241029/Data/FastQ?

2509183-SC2CL-IA-VL00618-241029_S2_R1_001.fastq.gz  2509554-SC2CL-IA-VL00618-241029_S3_R2_001.fastq.gz
2509183-SC2CL-IA-VL00618-241029_S2_R2_001.fastq.gz  2509555-SC2CL-IA-VL00618-241029_S4_R1_001.fastq.gz
2509554-SC2CL-IA-VL00618-241029_S3_R1_001.fastq.gz  2509555-SC2CL-IA-VL00618-241029_S4_R2_001.fastq.gz
And I did double-check any running with -r 3.15.24324 works without issue, except for the Nextflow warning
WARN: Include with addParams() is deprecated -- pass params as a workflow or process input instead

The latest updates were to avoid that warning.
This is the current problem (this is the glob that finds files):
.fromFilePairs(["${params.reads}/*{1,2}*.{fastq,fastq.gz,fq,fq.gz}"], size: 2 )

This is having problems parsing your filenames. This happens with filenames with 1s and 2s in them.
It used to be
.fromFilePairs(["${params.reads}/*_R{1,2}*.{fastq,fastq.gz,fq,fq.gz}",
                        "${params.reads}/*_{1,2}*.{fastq,fastq.gz,fq,fq.gz}"], size: 2 )

Reverting back to this should fix your issue. Hopefully.

I've made the changes and they're currently undergoing testing in this PR: https://github.com/UPHL-BioNGS/Cecret/pull/395
Once all the tests are successful (a few hours), I'll release a new version.
Sorry!!!!

No worries, thanks for promptly looking into this!

It should be working now! Let me know if it doesn't!!!

It looks to be mostly working for me.
When running, the plot_ampliconstats process fails. Not sure why this is the case as I include --samtools_plot_ampliconstats false in my run command. Nextflow log attached:
nextflow.log
One additional thing I had to do was change my custom config to refer to process "PANGOLIN" rather than "pangolin".

My apologies on this!!!
And, yes, all of the workflow, subworkflow, and process names are now capitalized.
It's part of nf-core's recommendations that I'm trying to follow so that their linter and schema creator work.
I made some changes with this PR here that should address this issue: https://github.com/UPHL-BioNGS/Cecret/pull/397
Once the changes pass all the tests, a new version will get get released that can be used.

https://github.com/UPHL-BioNGS/Cecret/releases/tag/3.25.24348 is out now! Let me know if you run into any other issues!

Thanks so much, all good now!


Deletion of series

When running a full reindexing, delete now-unused series
When receiving a delete event from cumulus, remove asset from series it was in, and if the series is now empty, delete it

good fricking stuff :)  LGTM


Set clientcert=verify-ca in pg_hba.conf when using SSL

Fixes #11

Set clientcert=verify-ca in pg_hba.conf when using SSL as clientcert=1 is deprecated

Also:

update the ansible-lint pre-commit hook
fix (or ignore) ansible-lint complaints

closing as these changes were merged in #13


[Feature][MASTER-SERVER] Periodic task characteristics

Describe the feature
A cycle task is executed once every 10 minutes, but it takes more than 10 minutes to execute. I hope it can't be executed next time. Is the community considering adding this feature

we don't have assigned this feature in the coming 3 months.
would you like to implement it?


nginx frontend: fix for urls containing slashes

In production, the / was causing some 404 not found... I believe that /index.html$is_args$args was expanding to /index.html?foo=bar/blah with the second / interpreted as a directory separator?
In any case, we don't need to pass arguments to the static index.html.

Thanks @art-w!


switch openssl & nasm because openssl depends on nasm

To fix this error from the Windows playbook:
TASK [OpenSSL : Install OpenSSL-1.1.1 32-bit] ********************************************************************************************************************************** fatal: [jsvt288]: FAILED! => {"changed": true, "cmd": "set PATH=C:\\Strawberry\\perl\\bin;C:\\openjdk\\nasm-2.13.03;%PATH% && .\\vsvars32.bat && cd C:\\temp\\openssl-1.1.1 && perl C:\\temp\\openssl-1.1.1\\Configure VC-WIN32 --prefix=C:\\openjdk\\OpenSSL-1.1.1-x86_32 && nmake install > C:\\temp\\openssl32.log && nmake -f makefile clean", "delta": "0:00:04.374979", "end": "2019-10-22 07:43:05.908558", "msg": "non-zero return code", "rc": 1, "start": "2019-10-22 07:43:01.533578", "stderr": "\r\nFailure!  build file wasn't produced.\r\nPlease read INSTALL and associated NOTES files.  You may also have to look over\r\nyour available compiler tool chain or change your configuration.\r\n\r\nNASM not found - make sure it's installed and available on %PATH%\r\n", "stderr_lines": ["", "Failure!  build file wasn't produced.", "Please read INSTALL and associated NOTES files.  You may also have to look over", "your available compiler tool chain or change your configuration.", "", "NASM not found - make sure it's installed and available on %PATH%"], "stdout": "Configuring OpenSSL version 1.1.1 (0x1010100fL) for VC-WIN32\r\nUsing os-specific seed configuration\r\n", "stdout_lines": ["Configuring OpenSSL version 1.1.1 (0x1010100fL) for VC-WIN32", "Using os-specific seed configuration"]}

Interestingly, I haven't come into contact with that error message when I was fixing the Windows playbook using Vagrant -  however, looking at the content of the OpenSSL role, I have no idea why it wasn't failing.
I've tested the change (with a Vagrant VM) though and it doesn't cause anything to start failing :-)

Thanks @Willsparker - merging now


Removes root dependency management

This removes root dependency management for same rationale as
https://github.com/openzipkin/zipkin/pull/3284
Notably, to prevent interference or confusion in downstream projects.
This also forcibly prevents benchmarks from the deploy phase as the
prior config resulted in orphaned staging repos. See
https://github.com/openzipkin/zipkin/pull/3286

cc @jeqo @reta @kojilin this hopefully makes peer projects conflict less
I hope this also lets vertx-web tests pass, but this is just a hope. If not I will disable it again

no luck on vertx-web, but at least we know authoritatively that dependency management has nothing to do with it :D


integrate new storage lib into SDK

https://github.com/Azure/azure-storage-blob-go is the first of several new libs for Storage. We proposed vendoring in that lib and exposing it under services/storage for consistency with other libs.
We also will need to gradually deprecate the existing storage/ package.

cc @JeffreyRichter @seguler

Closing as we're not going to do this.