Flutter sprites test

Add a test for the flutter_sprites package (redux of https://github.com/flutter/flutter/pull/909); this lets the flutter analyze --flutter-repo command analyze the flutter_sprites package.

I do see exceptions from the analyzer package locally when trying to run the full flutter analyze --flutter-repo command - don't know how the travis bot will be effected.

Here are the current analysis issues for that package (also tracked here: https://github.com/flutter/flutter/issues/1013). We may want to wait to land this PR until after those issues are fixed, to keep the bot green.
[error] Type check failed: _effectPlayer.play(effect, false, (volume + evt.volumeVariance * randomSignedDouble()).clamp(0.0, 2.0), (pitch + evt.pitchVariance * randomSignedDouble()).clamp(0.5, 2.0), (pan + evt.panVariance * randomSignedDouble()).clamp(-1.0, 1.0), (SoundEffectStream s) {playingList.remove(newPlaying);}) (Future<SoundEffectStream>) is not of type SoundEffectStream (/home/travis/build/flutter/flutter/packages/flutter_sprites/lib/src/sound_manager.dart, line 120, col 25)
[warning] The method 'sharedInstance' is not defined for the class 'SoundEffectPlayer' (/home/travis/build/flutter/flutter/packages/flutter_sprites/lib/src/sound_manager.dart, line 67, col 55)
[warning] The method 'stop' is not defined for the class 'SoundEffectPlayer' (/home/travis/build/flutter/flutter/packages/flutter_sprites/lib/src/sound_manager.dart, line 88, col 23)
[warning] A value of type 'Future<SoundEffectStream>' cannot be assigned to a variable of type 'SoundEffectStream' (/home/travis/build/flutter/flutter/packages/flutter_sprites/lib/src/sound_manager.dart, line 120, col 25)
[warning] 1 positional arguments expected, but 6 found (/home/travis/build/flutter/flutter/packages/flutter_sprites/lib/src/sound_manager.dart, line 120, col 43)
[warning] The setter 'volume' is not defined for the class 'SoundEffectStream' (/home/travis/build/flutter/flutter/packages/flutter_sprites/lib/src/sound_manager.dart, line 138, col 71)
[warning] The getter 'volume' is not defined for the class 'SoundEffectStream' (/home/travis/build/flutter/flutter/packages/flutter_sprites/lib/src/sound_manager.dart, line 138, col 98)
[warning] The method 'stop' is not defined for the class 'SoundEffectPlayer' (/home/travis/build/flutter/flutter/packages/flutter_sprites/lib/src/sound_manager.dart, line 139, col 79)
[warning] The method 'stop' is not defined for the class 'SoundEffectPlayer' (/home/travis/build/flutter/flutter/packages/flutter_sprites/lib/src/sound_manager.dart, line 145, col 25)
[warning] 1 positional arguments expected, but 3 found (/home/travis/build/flutter/flutter/packages/flutter_sprites/lib/src/sound_manager.dart, line 204, col 24)

LGTM modulo #1013. Thanks for doing this!
This will fix #759.

cc @vlidholt can you help land this?

We'll likely want to get the analysis issues fixed before this lands.

Looks like adding this package to analysis uncovers a bug in the command-line analyzer (https://travis-ci.org/flutter/flutter/builds/102399407#L287). I believe this is one that exists on the stable sdk and is fixed in the dev sdk. Landing this PR while we're still testing against 1.13.0 would turn the bot red :(

I don't really understand the travis failure.

The command line analyzer is throwing an exception when analyzing these files. It only shows up w/ this file set and the cli options we're passing in. The bug is fixed in the dev sdk - I think we have to wait until 1.14.0 is stable before we can land this -

Do you know what the bug actually is?

No, not specifically. This was one exception: https://github.com/dart-lang/sdk/issues/25305. Note that flutter analyze --flutter-repo does not have the exception when run w/ Dart VM version: 1.14.0-dev.7.0.

Hmm, I wonder if there a type annotation comment in there somewhere... Will
keep looking.
On Sat, Jan 16, 2016, 16:03 Devon Carew<EMAIL_ADDRESS>wrote:

No, not specifically. This was one exception: dart-lang/sdk#25305
https://github.com/dart-lang/sdk/issues/25305. Note that flutter
analyze --flutter-repo does not have the exception when run w/ Dart VM
version: 1.14.0-dev.7.0.
—
Reply to this email directly or view it on GitHub
https://github.com/flutter/flutter/pull/1043#issuecomment-172276582.

Wow, if I just inline the files instead of using "part", that fixes it.

I have it narrowed down to:
import 'package:box2d/box2d.dart' as box2d;

import 'src/action.dart';
import 'src/action_spline.dart';
import 'src/color_sequence.dart';
import 'src/constraint.dart';

...where the four src/*.dart files are 0 bytes long.

Removing the box2d package import, with no other changes, also fixes the analyzer crash (at the cost of breaking all the code, obviously).

I'm checking this in since the weather demo means that we're analyzing flutter_sprites now anyway.


📖 [release-0.23.0] Some doc file cleanup

Remove some obsolete files.
Add some non-obsolete files into the navigation tree, so that the tree appears in their rendering.

Summary
This PR does some doc cleanup in branch release-0.23.0, primarily removing mkdocs error messages about files that are not referenced from the navigation tree. Some are obsolete files that this PR deletes. This PR adds some others into the navigation tree. In one case, a file was merged into the user-guide.md file.
Related issue(s)
This addresses parts of #2204 and #2221 in the release-0.23.0 branch.

/cc @KPRoche
/cc @francostellari
/cc @ezrasilvera

/cc @clubanderson

/lgtm
/approve


Engine crashing on multiple stop commands.

Using windows v3 binary.
Steps to reproduce:
position fen rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1
go infinite
stop
stop
I don't know if sending multiple stop commands is UCI compliant. However, I saw it in the log and tested it, and the engine crashes.

Thank you for reaching out. This is a bug I became aware of a couple days ago, it occurs because I force join on stop so when you send stop again it tries to join a thread that's already joined and crashes. A fix for it (along with other bugs) will be forthcoming next commit.


DBZ-2592 Add script to list contributors of a release

https://issues.redhat.com/browse/DBZ-2592

This might be a little noob approach, especially when multiple repos were a concern. :sweat_smile:
Any suggestions @gunnarmorling?

Hi @ani-sha, I think the approach looks good but what I believe is still missing is taking the output you've gathered from the git history and converting that into markdown syntax.  I assume that's still a todo?

@Naros could you give an example of how the names should be aligned in the markdown file?

@Naros, I'm going to leave this to discuss and merge to you and @ani-sha, if that's ok? In the end just let me know how to run the thing for the next announcement post :)

@Naros, @ani-sha, any news here? Would love to use this script for the next announcement, if possible :)

@gunnarmorling, waiting for @Naros to have a quick look and confirm how the names would be laid in the MD file :)

My apologies @gunnarmorling and @ani-sha, I must have missed the GH notification for this one somehow.  The format you're wanting to try and mimic is the following:
https://github.com/<github-username>[Real name]

I would suggest depending on what you can resolve from the GH commit, you could use the username on the commit and resolve it against the rows in jenkins-jobs/scripts/config/Aliases.txt.  If you get a hit, then you should use the real name defined there.  If you get no hit, you could for now just use the account name as the real name.  It's at least a good starting point and we can then take that as an opportunity that if we have some that don't resolve, to update the Aliases file accordingly.

Hey, @Naros Thanks for the suggestion. If I am not wrong the commit-msg provides the Author Name/Real Name (and some users have their username as author name), I was actually struggling to get the username.

Yep, in that case I think Aliases.txt will come in handy for you :).

@gunnarmorling @Naros And it's done! :)  I used GitHub API for the same.
For running the script you have to set three vars ie

TAG1 (from which release) eg v1.5.0.Alpha1
TAG2 (to which release) eg v1.5.0.Beta1
AUTH_TOKEN (Presonal Access Token)

Ah, this is really cool! A few suggestions:

Auth token shouldn't be needed actually for this operation; I just could run the curl command without the header
Let's make tag1 and tag2 parameters for the script instead of env vars, and validate their presence (i.e. abort if they are not both given)
Can we then have a GH Actions workflow, which makes this invokeable from the GH Actions UI, requiring the two tags as input parameters (see [here|https://stackoverflow.com/questions/62736315/invoke-github-actions-workflow-manually-and-pass-parameters])

I'm also getting this error locally, I reckon it's due to some difference between Linux (which I suppose you're using) and macOS:
parse error: Invalid numeric literal at line 1, column 3
sort: No such file or directory

parse error: Invalid numeric literal at line 1, column 3
sort: No such file or directory

@gunnarmorling If you reverse the tags that would throw this error, I am not sure why exactly.

@ani-sha, yeah, so in my case it was caused by the fact that &>> isn't supported by zsh apparently (default shell on macOS). Could you use the alternative syntax described [here|https://unix.stackexchange.com/questions/170572/what-is-in-a-shell-script] instead? Then it runs 🎉 .

That said, here's the output for v1.5.0.Beta1...v1.5.0.Beta2:
[Andrew Garrett],martper2
[Andrew Garrett],mengqiu
https://github.com/AndreyIg[Andrew Garrett],AndreyIg
https://github.com/AndreyIg[Andrey Ignatenko],
https://github.com/Naros[Andrew Garrett],Chris Cranford
https://github.com/Naros[Chris Cranford],
https://github.com/ani-sha[Andrew Garrett],Anisha Mohanty
https://github.com/ani-sha[Andrew Garrett],ani-sha
https://github.com/ani-sha[Anisha Mohanty],
https://github.com/ant0nk[Andrew Garrett],Anton Kondratev
https://github.com/ant0nk[Anton Kondratev],
https://github.com/bingqinzhou[Andrew Garrett],Bingqin Zhou
https://github.com/bingqinzhou[Bingqin Zhou],
https://github.com/ddseapy[Andrew Garrett],David Seapy
https://github.com/ddseapy[David Seapy],
https://github.com/debezium-builder[Andrew Garrett],Debezium Builder
https://github.com/denisprog[Andrew Garrett],DLiseichykau
https://github.com/denisprog[Victar Malinouski],
https://github.com/fuxiao224[Andrew Garrett],fuxiao224
https://github.com/fuxiao224[fuxiao224],
https://github.com/gunnarmorling[Andrew Garrett],Gunnar Morling
https://github.com/gunnarmorling[Gunnar Morling],
https://github.com/jpechane[Andrew Garrett],Jiri Pechanec
https://github.com/jpechane[Jiri Pechanec],
https://github.com/keweishang[Andrew Garrett],Kewei Shang
https://github.com/keweishang[Kewei Shang],
https://github.com/lga-zurich[Andrew Garrett],Luis Garcés-Erice
https://github.com/lga-zurich[Luis Garcés-Erice],
https://github.com/mpermar[Andrew Garrett],mpermar
https://github.com/mpermar[Martín Pérez],
https://github.com/roldanbob[Andrew Garrett],Bob Roldan
https://github.com/roldanbob[Andrew Garrett],roldanbob
https://github.com/roldanbob[Bob Roldan],
https://github.com/roldanbob[Robert Roldan],
https://github.com/vanhoale[Andrew Garrett],Hoa Le
https://github.com/vanhoale[Hoa Le],
https://github.com/vaosinbi[Andrew Garrett],Vladimir Osin
https://github.com/vaosinbi[Vladimir Osin],
https://github.com/zrlurb[Andrew Garrett],Peter Urbanetz
https://github.com/zrlurb[Peter Urbanetz],

A few observations:

Duplications should be removed (e.g. via piping through uniq)
Extra names (besides the actual links) should be removed
Some names which need resolving against the alias file it seems: martper2, mengqiu
Icing on the cake would be to remove the trailing comma and replace the last one with ", and"; I know, I'm greedy :-)

Btw. the parse error was caused by this:

Something wrong with the API invocation (e.g. tags reversed, empty auth header as in my case, etc.)
the jq in the loop will write an error message to the file
the second jq cannot parse that, as it's not JSON

That's why it's a best practice to aboard a script early on in case of any error.

@gunnarmorling yes exactly, that's strange, I am looking into that.
What are the results for v1.5.0.Alpha1...v1.5.0.Beta1 ? Can you share it?

Yes, that's what I'm getting for Beta1:
https://github.com/Cyril-Engels[Yilong Chang],
https://github.com/Naros[Chris Cranford],
https://github.com/adhaamehab[Adham],
https://github.com/ahmedjami[ahmed],
https://github.com/ani-sha[Anisha Mohanty],
https://github.com/bingqinzhou[Bingqin Zhou],
https://github.com/fahimfarookme[Fahim Farook],
https://github.com/gunnarmorling[Gunnar Morling],
https://github.com/harveyyue[Harvey Yue],
https://github.com/jcechace[Jakub Cechacek],
https://github.com/jpechane[Jiri Pechanec],
https://github.com/keweishang[Kewei Shang],
https://github.com/kppullin[Kevin Pullin],
https://github.com/michaelcizmar[Michael Cizmar],
https://github.com/mrshanepaul[Shane Paul],
https://github.com/nitin1891[Nitin Agarwal],
https://github.com/nitin456[Nitin Agarwal],
https://github.com/r-ballard[unknown],
https://github.com/rgannu[Ganesh Ramasubramanian],
https://github.com/rk3rn3r[René Kerner],
https://github.com/roldanbob[Bob Roldan],
https://github.com/tprelle[Thomas Prelle],
https://github.com/twthorn[Thomas Thornton],

Wow, this worked fine, something messed up for Beta2 :astonished:, scripts need extra care :)

I suppose this is caused some missing alias entries, e.g. for "martper2".
@jpechane, e.g. we have this commit by Martín Pérez, but using a different author name than usual ("martper2"): https://github.com/debezium/debezium/commit/3727273d9e5fa167fe7e8a7e00570cffb6a2e531#diff-15cdf214087b7f2f5dfff4133cbd436157df20e599e6cfc24f4d62088c553e1b. How comes the script that checks for alias completeness didn't catch this (of course the commit should not have been merged to begin with, we don't want any "Octocat" commits, // CC @Naros)?
Also, ideally, this script here actually shouldn't yield that contribution for Beta2, as it it was done before. It's caused though by the recent Oracle connector merge, it seems the tag1...tag2 syntax brings in all the commits which got merged from the Oracle branch. Not a problem though, as it's a one-time issue, and we quickly can remove those additional names which actually didn't commit during the Beta2 frame manually.

@ani-sha, I've pushed a commit which adds the missing alias. With this one, the Beta2 output looks much better now.

@ani-sha, so the problem are the commits by mengqiu and martper2. They return null for the GitHub URL (that's what I'm referring to by "Octocat commits", these commits are done with an e-mail that's not linked to any GH account, thus no profile image is shown next to them in the history; as said, we shouldn't have those to begin with, but these slipped through). So the script needs to be tolerant to that, and in this case just show the name (resolved against the alias file still), but without link.

How comes the script that checks for alias completeness didn't catch this (of course the commit should not have been merged to begin with, we don't want any "Octocat" commits,

I'll look into this @gunnarmorling, I'm surprised it didn't.
@ani-sha, here are my suggestions:
First, change the jq line where you find the name/github_url to the following:
jq -r 'if .github_url then (.github_url + " " + .name) else ("https://github.com/" + " " + .name) end' $CONTRIBUTORS_LIST_JSON > $CONTRIBUTORS_NAMES

What this does is if there is no .github_url for the user, it'll be written as:
https://github.com/ person's name

This should allow the formatting to work as desired and we'll simply need to look for these in the markdown output formatted as:
https://github.com/[Person's name]

and update these lines accordingly.
One last nit-pick, add --silent to the curl call to suppress the progress bar text so the only output should then be what comes from the sort/uniq at the very end.

@gunnarmorling I looked into the martper2 alias issue, it seems we simply added martper2 to the COPYRIGHT.txt directly and that is what allowed the check-contributors script to not inform us about the commit.  With adding the alias as you've done in this PR, what we should then do is add Martín Pérez to the COPYRIGHT.txt.  Could you do that @ani-sha in a follow-up commit in this PR once the script logic is complete?

Ah, great catch, Thx! "martper2" should then also be removed from
COPYRIGHT.txt.

@gunnarmorling Few minor things remaining to address. Will try to wrap it by tomorrow.

Sounds good. But no need to work overtime, Monday is fine :)

Ah, I noticed one more thing: for Beta1, there's one entry in the result list where the name is null (GH URL is present, though).

Ah, I noticed one more thing: for Beta1, there's one entry in the result list where the name is null (GH URL is present, though).

@gunnarmorling Are you referring to the username fuxiao224 ?

Applied, thanks a lot, @ani-sha. I've captured a list with a few small follow-up improvements at https://issues.redhat.com/browse/DBZ-2592.


Add optional parameter to xDSCWebService

The new optional parameter will allow for fully automated deployments including SSL when combined with xCertReq.

Thank you, Michael.
Please, fix encoding, here are instructions about our CI system and common tests.

@mgreenegit please

make common tests pass
Merge from dev branch and resolve merge conflicts
Send pull request to dev branch.

Closing reoppening to trigger the CLA bot

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

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

TTYL, MSBOT;

@mgreenegit Can you please resend it to the dev branch?

@mgreenegit, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

I resolved the merge conflicts and pushed the branch here, but the tests failed

If there is no update of the pull request within 7 days, we will close the pull request.  After that, if you have an update, feel free to reopen the pull request or a new pull request.

Closing because the pull request has not been updated for feedback given for 21 days.  Once you update the code, please feel free to reopen the pull request.  Thanks, Travis.


should links fields be removed?

We like the links fields and believe it will be helpful.  Why would you like them to be removed?

Closing this issue.


InstructorsController.cs throws warning in the Index() function after changes

In the InstructorsController.cs, where the Index() function definition is to be changed, the following warning comes up.

btw @kunaldas1997 ... It's best if you open issues using the This page feedback button and form at the bottom of the English-US topic. Use of the This page feedback form adds metadata to your GitHub issue that cross-links the topic and automatically pings the author.

btw @kunaldas1997 ... It's best if you open issues using the This page feedback button and form at the bottom of the English-US topic. Use of the This page feedback form adds metadata to your GitHub issue that cross-links the topic and automatically pings the author.

forgot to report from there. Sorry about that.


Overview/state-scripts

New Overview/State-scripts section

@mirzak Ready for a second round of slaughter

havent we agree that Drew reviews all?


cf html5-push guid removes all other apps from repository

Description
cf html5-push guid removes all other apps from repository
What is the expected behavior?
Only one app should be pushed and all others should remain intact.
What is the actual behavior?
The new app is pushed but all others are deleted.
Environment

OS - Windows
SAP CF Landscape Canary
IaaS provider - AWS

CLI Output
Set environment variable DEBUG=1, run the flow and paste output here.

@rahuldeepattri , thanks for your feedback!
Unfortunately, HTML5 Applications Repository service currently does not support partial upload of content associated with app-host service instance. This means that to update a single application, all applications of the same app-host service instance should be redeployed. This is a consistent behaviour across all deployment options, including GACD, deployer application and CF CLI plugin.
Once HTML5 Applications Repository expose API that allows to deploy/redeploy single application, we plan to add this functionality to cf html5-push as well. I'll forward your feature request to HTML5 Applications Repository product owner.
So far you can do the following:

Create an empty directory and change your current working directory to it
mkdir html5 && cd html5
Run cf html5-list to see app-host-id of the application you want to redeploy
Run cf html5-get --all <app-host-id> to download all applications deployed with same service instance (same app-host-id) to current working directory
Replace or make changes to application you want to redeploy
Redeploy all applications to the same app-host service instance
cf html5-push -r

@micellius Thanks for giving the work around as well. Will be useful to update github docs for the same.

Pls move it to do again.

Hi, is it possible to have multiple app-host in the same space? And how to consume it from one approuter?

Hi @mdejavue ,
Yes, it is possible to consume multiple app-host instances in one Approuter.

If you have multiple app-host instances in the same space, they are all automatically consumable with app-runtime service instance (of the same space).
If you'd like to consume app-host instances from different spaces, this is also possible, while more complex to setup. You'll need to bind services that expose their app-host IDs as part of binding information to Approuter as well as add manifetst.json > sap.cloud > public:true to HTML5 apps that intend to be shared cross-space.
To see, which HTML5 apps from which app-host instances and other business services are available in your Approuter, you can use cf html5-list -a <approuter_app_name>. If you'd like to see the URLs of how to consume these applications via this specific Approuter, you can add -u at the end of command above.

Hi @micellius,
would also be interrested in this feature. That would allow quicker cycles when a deployed version is needed to test specific features.
Best Regards
Gregor


Amended NavballUpDefault due to 1.1 changes

KSP 1.1 changes the way the navball is accessed, so any version older than NavballUpDefault-v0.2 (the latest release) will not work on KSP v1.1

LGTM, thanks. @BrianErikson since you're updating your own metadata would you be interested in self-hosting it so you don't have to submit PRs to us?

@dbent, thanks for the merge. As for self-hosting, I'm not quite sure what you mean? It seems all of the other mod authors are updating their NETKAN files this way as well unless I'm mistaken.

@BrianErikson See PlaneMode.netkan as an example. It's just a reference to a .netkan file I host in my own repository.
It's not wide spread yet but is currently used by a few mods like RealismOverhaul or the USI mods.


Standardize item system type format

https://github.com/foundryvtt/dnd5e/issues/2168

Thanks for doing this work. Do you think it would be possible to omit the compendium data changes at the moment as they add a lot of noise to the PR? It looks like it could do with a rebase as well. Please let me know if you're able to do that, otherwise I can probably handle it.


Generate metrics from tracker in JSON format

Create a config file specific to a Jetson nano's location and other specific data. So that we can insert the data to the json file using Kafka.
i.e. Location


Seems to time out with SUD ver 4.0.4

Tested on Raspberry Pi 1 B with latest Raspbian distro, fully upgraded.  Seneye SUD v 4.0.4@14200000.
After adding manual udev rule in /etc/udev/rules.d/10-usb.rules I got past USB access denied message:
ACTION=="add" \ , SUBSYSTEMS=="usb" \ , ENV{DEVTYPE}=="usb_device" \ , ATTRS{idProduct}=="2204" \ , ATTRS{idVendor}=="24f7" \ , GROUP="plugdev" \ , MODE="0777"
Then reloading udev config as root:
udevadm control --reload-rules && udevadm trigger
I finally was able to use pyseneye interactively and get through these commands:
from pyseneye.sud import SUDevice, Action
d = SUDevice()
d.action(Action.ENTER_INTERACTIVE_MODE)
Timing out at:
s = d.action(Action.SENSOR_READING)
File "/home/pi/.local/lib/python3.7/site-packages/pyseneye/sud.py", line 608, in action
My foo is not strong enough to find the problem, but I wonder if there may have been some changes to how you should interpret the seneye from the version you made pyseneye for, and the version I am using?

Using the Seneye example code for C++ on Mac OS X with hidapi, I get:
"  Device: F50020C75B6DF2D6AF3620E813006001 v.4.0.4  Type:  Home
This device need to be connected to SCA or SWS"
Thus it seems that the computer can at least talk to the seneye, get its version string. I do not have the slide at hand to test it with that; but it may be that this is part of the problem. I will give the slide a try when I get to it, and try the C++ example again.

I have heard reports that Seneye have introduced something into the
firmware that cause the Seneye to stop working unless it has a new slide
activated in the device online. That is something I haven't managed to work
out how to replicate in this integration, as it looks like Seneye have
encrypted these calls over the SUD interface.
I haven't verified this change myself though.
On Tue, Mar 30, 2021 at 10:58 PM Joakim Skjefstad @.***>
wrote:

Using the Seneye example code for C++ on Mac OS X with hidapi, I get:
" Device: F50020C75B6DF2D6AF3620E813006001 v.4.0.4 Type: Home
This device need to be connected to SCA or SWS"
Thus it seems that the computer can at least talk to the seneye, get its
version string. I do not have the slide at hand to test it with that; but
it may be that this is part of the problem. I will give the slide a try
when I get to it, and try the C++ example again.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mcclown/pyseneye/issues/15#issuecomment-810161609,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI6SUEB7REHLCRWVVQYMLDTGG4GJANCNFSM4ZRWLFFQ
.

--
Stephen Mc Gowan.

The official example code in C++ from OS X works very well as long as I have a slide in the Seneye. I can imagine this being the issue at hand. I will proceed to test your python package on a raspberry pi, with the slide in.

No problems at all, without a slide in the device, it times out. Everything works fine with a slide in it, after it has been activated, so this issue is closed. Activation of slide must be done through standard seneye software.


Handle multiple genres in play queue

I'm still playing with https://github.com/CDrummond/lms-material/pull/993 but along the way I found something to fix: if a track is tagged with multiple genres, the value returned in genre will be randomly one of them. So get and use genres instead.

Some more places to look at:

browse-resp.js:1400:                let useComposerTag = i.composer && lmsOptions.showComposer && useComposer(i.genre);
browse-resp.js:1784:                let useComposerTag = i.composer && lmsOptions.showComposer && useComposer(i.genre);
nowplaying-functions.js:173:    let useComposerTag = playerStatus.current.composer && lmsOptions.showComposer && useComposer(playerStatus.current.genre);
nowplaying-functions.js:255:    let maiComposer = lmsOptions.maiComposer && view.playerStatus.current.composer && undefined!=view.playerStatus.current.composer_id && useComposer(view.playerStatus.current.genre);
nowplaying-functions.js:350:        if (lmsOptions.showComposer && view.playerStatus.current.composer && view.playerStatus.current.composer_id && useComposer(view.playerStatus.current.genre)) {
queue-page.js:58:        let useComposerTag = i.composer && lmsOptions.showComposer && useComposer(i.genres);
itemlinks.js:136:    if (undefined!=useComposerTag ? useComposerTag : (i.composer && lmsOptions.showComposer && useComposer(i.genre))) {

Perhaps the real fix is for LMS to return the first genre when requested, not just picking a random one. That at least would make sense. Not sure I really want to be checking a set of genres for every track in the queue - there could be thousands!

I think thousands is unlikely, but there could be a few. (Or do you mean thousands of tracks, rather than genres?)
The behaviour of LMS is a "feature" of sqlite: unlike some databases it allows selection of a column that's not in the "GROUP BY" but the value that column gets is "undefined" according to the docs. This is noted as a bug in the LMS source, and I think at a later stage a subquery was added to populate genres with the full list.
If it was me, I'd have removed the old single genre at that point, but backwards compatibility, I suppose.
Technically, it would be easy enough to replace the random value of the returned genre with a chosen single value, but the problem is, which one to choose? We don't have a hierarchy of genres defined anywhere, and I don't think it's something that could ever be agreed upon to everyone's satisfaction.
Realistically, how many genres is a track going to have? 3? 4? I think more than that would be pretty rare. So IMO the extra processing overhead would not be noticeable.
The alternative would be for the user to add all their more specific genres to the lists in Material's server-side prefs, I see that you have quite a few already there by default (but sods law, not the one I was using). But I'm always in favour of things "just working" without the user having to think too much and manipulate prefs, as far as possible.

Yes I meant thousands of tracks.
The point still stands that if LMS has a flag to return a single genre it should be consistent. Either the first, or the more prominent (however that is decided). Just returning any random genre sounds like a bug to me.

As I said, it is a bug, noted as one in the source. I think the fix was adding the subquery to return all the tracks genres. I assume the old code is still there for backward compatibility with hardware squeeze devices for example.

I've created branch 997-multiple-genres which should implement these changes. Can you checkout and confirm this is OK? Thanks.

Changes form my branch merged into master, so closing.

Sorry, missed your previous message, and haven't tried it out yet, but does passing i instead of i.genres to the utils function get around what I found, that i.genres could sometimes be an object, which is why I had to do the toString?

Not sure how it could be an object? Never seen that. But my code to split the field does cast to a string first:
function splitStringArray(val, plainSep) {
    return undefined==val || Array.isArray(val) ? val : (""+val).split(plainSep ? "," : MULTI_SPLIT_REGEX).map(function(itm) { return itm.trim() });
}

...this is used split all multi-string fields. (If only LMS would return an array!  (Which the above code should handle)).
Now I'm wondering if i should change ""+val to val.toString() ???

I recreated it:
with:
const genreArray = genre.split(/\s*,\s*/);
everything seems ok in the UI, but you get this repeatedly while a track is playing:
vue.min.js?r=2.7.15:11 TypeError: genre.split is not a function
    at useArtistTagType (utils.js?r=GIT-ccc790a0a:1044:30)
    at useComposer (utils.js?r=GIT-ccc790a0a:1049:12)
    at buildArtistAlbumLines (queue-page.js?r=GIT-ccc790a0a:58:71)
    at a.<anonymous> (queue-page.js?r=GIT-ccc790a0a:530:82)
    at pn (vue.min.js?r=2.7.15:11:22017)
    at t.$emit (vue.min.js?r=2.7.15:11:44399)
    at a.handlePlayerStatus (server.js?r=GIT-ccc790a0a:629:17)
    at server.js?r=GIT-ccc790a0a:880:26

Is that error with the current code? Or before? As I use (""+genre).split(...

That is with my PR (this one), but taking out the toString in my version of useArtistTagType. I'll have to pull your latest changes and try it out.
BTW, we tried returning an array from LMS in https://github.com/LMS-Community/slimserver/pull/1195 but it didn't get past Michael!

All seems OK with your latest master. No console errors.


Feature request : compatibility with objecthash (java, go, c, python, ruby) libs

There are two projects which perform almost exactly the same function as this one:
@benlaurie
https://github.com/benlaurie/objecthash
@cryptosphere
https://github.com/cryptosphere/ruby-objecthash
Between them, they support C, Go, Python, Java, and Ruby. All versions output the same SHA256 hashes for the same input and provide tests to ensure they stay that way.
It would be awesome if this library provided a compatible implementation for Javascript for e.g. verifying hashes made on the server side.
Any chance? 🎱

Huh, cool stuff! I’ll take a look at it, and there’s a good chance something like it is implementable, although I can say right now that these libraries have different objectives.
This library aims to support hashing of any kind of input, including circular object structures, DOM XML objects, Node.js Buffers, etc.
Also, benlaurie’s implementation seems to perform a lot more hash operations per object, namely one per encountered value, whereas this implementation currently only hashes what is basically a string representation of the entire input.
That being said… I like the idea. I won’t make any promises, but I’ll look into it!

Great. Thanks for taking a look.

You could take a look at 'objecthash' which is a js implementation of the libraries you mention.
https://github.com/emschwartz/objecthash-js

I think the goals here are to divergent to make this compatible with object hashing from other programming languages. As @addaleax mentioned, this library attempts to support all the types of objects in the runtime, which is a different approach.
Closing this issue, as I don't think this is something that will be implemented.


python310Packages.bond-async: 0.1.22 -> 0.1.23

Diff: https://github.com/bondhome/bond-async/compare/refs/tags/v0.1.22...v0.1.23
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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.

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

3 packages built:

home-assistant-component-tests.bond
python310Packages.bond-async
python311Packages.bond-async


Copyedit Correlation ID plugin documentation

Where is the problem?
https://docs.konghq.com/hub/kong-inc/correlation-id/#how-it-works
What happened?
Some current language is not in alignment with our style guide and contains needless complexity. Additionally, there are some grammatical errors.
Note: Only edit content starting at the How it Works section, as the content above is generated.
This is the file to edit: https://github.com/Kong/docs.konghq.com/blob/c47c8e0dc1f45c3fc37b60a2e41bd859b1712c5f/app/_hub/kong-inc/correlation-id/index.md
What did you expect to happen?
Clear, simple language in accordance with our style guide.
Code of Conduct and Community Expectations

[X] I agree to follow this project's Code of Conduct
[X] I agree to abide by the Community Expectations

Can you assign this to me? @falondarville

Hi @Sayanta66, yes. I'm going to hold off on assigning the other Issues all to you right now as others may want to contribute! But I can assign them one at a time as you get through them! Thanks so much for your contributions. 🎉

@lena-larionova @falondarville I am done with my previous tickets. I would love to work on this now.

Closing as the PR has been merged. 🚀


Can't find a prize for the player opening the crate

Describe the bug
Players cant receive items from my items i added in my crate.
Versions (please complete the following information):

Minecraft Server Version: 1.8.8
Plugin Version: 1.10

*Do NOT say Latest use /version crazycrates in the console.
Pastebin any errors in console
Link: https://pastebin.com/KLNCt9WY
Issue
I have created a custom crate but when I try to win a prize from it, it says that there is not a prize available to win even though there is prizes within the crate. I have also tried to remove my items that i added and put the default items back in the crate and that works and i'm able to open the crate, but with my items i cant open the crate.
Screenshots

More images:

Got it fixed.


feat: implement Copy for all types that already implement Clone

Hey @mkroening,
Thanks so much for all the contributions! I should've got everything merged now, and will release a new version to crates.
The only change I've reverted is failing on styling on CI - it's personal preference but I'm not a fan of failing CI jobs for it :)

:rocket: Published as pci_types v0.8.0

Thank you so much! :)


added a new concept page for Sidecar containers

Fixes #42762

Is this mostly a move of the existing content to the new concept page?

@shannonxtreme

Is this mostly a move of the existing content to the new concept page?

Yes, with some addon points.

I think below to changes are required

Add label [Sidecar containers] and make a link with [sidecar-containers.md] page in the left side menu below the [Init Containers].
Add [Sidecar containers] link from [whatsnext] in [Init Containers].

Hi @tamilselvan1102,

Add label [Sidecar containers] and make a link with [sidecar-containers.md] page in the left side menu below the [Init Containers].

Is this what you are talking about? If yes, it is already added to the PR.

@tamilselvan1102

Add [Sidecar containers] link from [whatsnext] in [Init Containers].

Updated, Thanks for remembering.

/sig node

/label tide/merge-method-squash

Relevant to https://github.com/kubernetes/website/issues/43610

(Overall, this is a really good and useful PR - I've added feedback because the concept of a sidecar container touches on lots of parts of Kubernetes, and it's good to capture all of those places).

/cc

@T-Lakshmi
please make it as single commit.

Page previews
https://deploy-preview-43346--kubernetes-io-main-staging.netlify.app/docs/concepts/workloads/pods/pod-lifecycle/
https://deploy-preview-43346--kubernetes-io-main-staging.netlify.app/docs/concepts/workloads/pods/init-containers/
https://deploy-preview-43346--kubernetes-io-main-staging.netlify.app/docs/concepts/workloads/pods/sidecar-containers/

/remove-label tide/merge-method-squash

LGTM for docs
@kubernetes/sig-node-pr-reviews does this look right, now?

LGTM for docs
@kubernetes/sig-node-pr-reviews does this look right, now?

Let me have a look today...

LGTM, however with 1.29 sidecars are going to beta and will be enabled by default.
@sftim do we want to already acknowledge that?

with 1.29 sidecars are going to beta and will be enabled by default.

I'll check with release docs. Personally, I think this is better merged than not.

/approve
/hold
See https://github.com/kubernetes/website/pull/43346#issuecomment-1850445456

I am fine with merging this into main as-is, then opening a PR against the dev-1.29 branch once I have frozen k/website.
/hold cancel

LGTM from https://github.com/kubernetes/website/pull/43346#issuecomment-1850405731
/lgtm

opening a PR against the dev-1.29 branch once I have frozen k/website.
with 1.29 sidecars are going to beta and will be enabled by default.

@sftim, @matthyx, @katcosgrove,
Thanks for merging the PR, I will open a PR to fix this once 1.29 release.

https://github.com/kubernetes/website/pull/44305 adapts that change for v1.29

#44305 adapts that change for v1.29

@sftim,
Yes, I saw the PR and merge changes in the dev1.29 branch docs.
Meanwhile, as sidecars are moving to beta state, I believe we should remove the text below from sidecar containers page.

By default, this feature is not available in Kubernetes. To avail this feature, you need to enable the feature gate named SidecarContainers.

WDYT?

Does that text appear on the dev-1.29 branch? See https://deploy-preview-43082--kubernetes-io-vnext-staging.netlify.app/docs/ for the preview of that branch.

Yes, It's there in dev-1.29 branch.
I could see this text at the end of the section in the preview page.

OK. Would you be willing to send in a PR to fix that bug? You can branch from main shortly after the release that is scheduled for later today.

Sure, I will work on it.
Thanks.

fd224a167fd41ef23303de20516813f7e4e77185


Input data validation for RectSphereBivariateSpline

I am trying to interpolate some 2D data on a sphere using scipy.interpolate.RectSphereBivariateSpline. Despite following the documentation, I get a data validation error. I've uploaded a sample netcdf file with data to trigger the error. As far as I can see, I've satisfied the requirements on the input latitude and longitude variables.
Reproducing code example:
from netCDF4 import Dataset
from scipy import interpolate

with Dataset('spl_debug.nc', 'r') as fid:
    u = fid.variables['colatitude'][:]
    v = fid.variables['longitude'][:]
    r = fid.variables['input_var'][:]

spl = interpolate.RectSphereBivariateSpline(u, v, r, s=0, pole_continuity=(True,True))

Error message:
ValueError                                Traceback (most recent call last)
/gpfsm/dhome/sbasu1/Code/GEOS/mass_flux/spl_bug.py in <module>
      7     r = fid.variables['input_var'][:]
      8
----> 9 spl = interpolate.RectSphereBivariateSpline(u, v, r, s=0, pole_continuity=(True,True))

/discover/nobackup/projects/tm5var4d/packages/python/3.8.2/lib/python3.8/site-packages/scipy-1.4.1-py3.8-linux-x86_64.egg/scipy/interpolate/fitpack2.py in __init__(self, u, v, r, s, pole_continuity, pole_values, pole_exact, pole_flat)
   1721         if ier not in [0, -1, -2]:
   1722             msg = _spfit_messages.get(ier, 'ier=%s' % (ier))
-> 1723             raise ValueError(msg)
   1724
   1725         self.fp = fp

ValueError:
ERROR: on entry, the input data are controlled on validity
       the following restrictions must be satisfied.
          -1<=iopt(1)<=1, 0<=iopt(2)<=1, 0<=iopt(3)<=1,
          -1<=ider(1)<=1, 0<=ider(2)<=1, ider(2)=0 if iopt(2)=0.
          -1<=ider(3)<=1, 0<=ider(4)<=1, ider(4)=0 if iopt(3)=0.
          mu >= mumin (see above), mv >= 4, nuest >=8, nvest >= 8,
          kwrk>=5+mu+mv+nuest+nvest,
          lwrk >= 12+nuest*(mv+nvest+3)+nvest*24+4*mu+8*mv+max(nuest,mv+nvest)
          0< u(i-1)<u(i)< pi,i=2,..,mu,
          -pi<=v(1)< pi, v(1)<v(i-1)<v(i)<v(1)+2*pi, i=3,...,mv
          if iopt(1)=-1: 8<=nu<=min(nuest,mu+6+iopt(2)+iopt(3))
                         0<tu(5)<tu(6)<...<tu(nu-4)< pi
                         8<=nv<=min(nvest,mv+7)
                         v(1)<tv(5)<tv(6)<...<tv(nv-4)<v(1)+2*pi
                         the schoenberg-whitney conditions, i.e. there must be
                         subset of grid co-ordinates uu(p) and vv(q) such that
                            tu(p) < uu(p) < tu(p+4) ,p=1,...,nu-4
                            (iopt(2)=1 and iopt(3)=1 also count for a uu-value
                            tv(q) < vv(q) < tv(q+4) ,q=1,...,nv-4
                            (vv(q) is either a value v(j) or v(j)+2*pi)
          if iopt(1)>=0: s>=0
          if s=0: nuest>=mu+6+iopt(2)+iopt(3), nvest>=mv+7
       if one of these conditions is found to be violated,control is
       immediately repassed to the calling program. in that case there is no
       approximation returned.

Scipy/Numpy/Python version information:
import sys, scipy, numpy; print(scipy.__version__, numpy.__version__, sys.version_info)
1.4.1 1.18.4 sys.version_info(major=3, minor=8, micro=2, releaselevel='final', serial=0)

I suppose we may want to check with a more recent version of SciPy, if there is indeed a genuine concern of a bug here.

If someone already has a more recent version of scipy installed, can they run the above code snippet to verify the presence/absence of the bug? Would be more efficient than me changing my scipy stack, only to find that the bug is still there.

I can confirm I can reproduce this on latest.

Indeed, adding +2 to nuest = mu+6+2 in the f2py wrappers fixes it, cf gh-16872.
The minimal reproducer is
    u = np.arange(1, 10) *np.pi / 10
    v = np.arange(1, 10) *np.pi / 10
    r = np.zeros((9, 9))
    RectSphereBivariateSpline(u, v, r, s=0, pole_continuity=(True, True))

and I checked locally that gh-16872 fixes the error with the OP dataset too. The latter is a 850k netCDF file with 361x507 size masked arrays --- a bit too much to add to the test suite unless really needed.


Fix typo on OBTAINING_MAC_ADDRESS

Requires https://github.com/SlimeVR/SolarXR-Protocol/pull/150

yeah, i fixed that in the solarxr side, it was because the step was added in the middle of the enum instead of appended


Music playback - stop - crash observed with error!

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.

Should run normally.
What behavior are you observing?
Application crashes.
There are errors:
AlexaPresentation:processExecuteCommandsResultExecutorFailed:reason=asked to process missing directive,messageId=RenderPlayerInfo
[2020-12-10 19:14:39.261] 2020-12-10 13:44:38.454 [ 1b] E CapabilityAgent:sendExceptionEncounteredAndReportFailed:reason=infoNotFound
[2020-12-10 19:14:39.261] 2020-12-10 13:44:38.464 [ 1b] E AlexaPresentation:processExecuteCommandsResultExecutorFailed:reason=asked to process missing directive,messageId=RenderPlayerInfo
SIGABORT:
SIGABRT: software abort
Backtrace and logs:
PR_TO_SSSDK.txt
backtrace.txt
Provide the steps to reproduce the issue, if applicable:
Play Songs from Amazon Music . Pause it .
Tell us about your environment:
Modified the sampleApp...
What version of the AVS Device SDK are you using?
AVS SDK version 1.20.1, APL version 1.3.3 and SS SDK version 2.2.1
Tell us what hardware you're using:
Crosscompiled
Tell us about your OS (Type & version):

[ ] Linux

Have you tried the same use case with AVS Device SDK SampleApp?

[ ] The issue is not reproducible on the AVS Device SDK SampleApp.

Hi @sakthisg,
Thanks a lot for reaching out and providing the backtrace, it seems like GUI layer might be sending incorrect value with updateCursorPosition message. From the backtrace looking at frame 6 i.e. APLClient::AplCoreConnectionManager::handleUpdateCursorPosition, it is attempting to parse payload and rapidjson could be causing this error.
When you hit the crash, can you go to the frame and try to fetch the payload by printing them to logs (the symbols are missing currently), that will give more information. Just as a reference, rapidjson does not implicitly assumes integer value to be a subset of double causing some asserts.

Closing this issue due to inactivity. Feel free to reopen it if you're still experiencing the issue.

Closing this issue due to inactivity. Feel free to reopen it if you're still experiencing the issue.

-DRAPIDJSON_MEM_OPTIMIZATION=OFF is only used for AVS SDK build.
In latest version of smart screen sdk, we merged the rapidjson libraries. Right now we are referring avs sdk's rapidjson and apl-core's rapidjson: https://developer.amazon.com/en-US/docs/alexa/alexa-smart-screen-sdk/mac-os.html
We don't have a rapidjson library under smart-screen-sdk anymore.
I would suggest you to update the smart screen sdk to the latest version if possible. If not, please share the details of the crash (at which scenario it crashes, detailed logs) so we can investigate further.

-DRAPIDJSON_MEM_OPTIMIZATION=OFF is only used for AVS SDK build.
In latest version of smart screen sdk, we merged the rapidjson libraries. Right now we are referring avs sdk's rapidjson and apl-core's rapidjson: https://developer.amazon.com/en-US/docs/alexa/alexa-smart-screen-sdk/mac-os.html
We don't have a rapidjson library under smart-screen-sdk anymore.
I would suggest you to update the smart screen sdk to the latest version if possible. If not, please share the details of the crash (at which scenario it crashes, detailed logs) so we can investigate further.

Thanks for the inputs.  Most of the times the core is not getting generated.
In our device we  are using:
AVS SDK version 1.20.1, APL version 1.3.3 and SS SDK version 2.2.1
For the above version used can you please confirm the rapid json flags to be used?
Command Issued:
Ask Alexa for the Wikipedia entry on Grace Hopper.
After that if we press back key in our device.
/home/kumar.jagadish/commit_userid/alexa/prebuilts/mvoiceapp/avs/prebuilts/include/rapidjson/document.h:1692: double rapidjson::GenericValue<Encoding, Allocator>::GetDouble() const [with Encoding = rapidjson::UTF8<>; Allocator = rapidjson::MemoryPoolAllocatorrapidjson::CrtAllocator]: Assertion `IsNumber()' failed.

Thanks for the inputs.  Most of the times the core is not getting generated.
In our device we  are using:
AVS SDK version 1.20.1, APL version 1.3.3 and SS SDK version 2.2.1
For the above version used can you please confirm the rapid json flags to be used?
Command Issued:
Ask Alexa for the Wikipedia entry on Grace Hopper.
After that if we press back key in our device.
/home/kumar.jagadish/commit_userid/alexa/prebuilts/mvoiceapp/avs/prebuilts/include/rapidjson/document.h:1692: double rapidjson::GenericValue<Encoding, Allocator>::GetDouble() const [with Encoding = rapidjson::UTF8<>; Allocator = rapidjson::MemoryPoolAllocatorrapidjson::CrtAllocator]: Assertion `IsNumber()' failed.

Hello @sakthisg,
The rapidjson flag to use on AVS is: -DRAPIDJSON_MEM_OPTIMIZATION and the flag to be used on Alexa smart screen sdk is: -DAPLCORE_RAPIDJSON_INCLUDE_DIR.
As for the crash, have you been able to fetch the payload through the backtrace as @VaruPan suggested? This will help us debug further

Hello @sakthisg,
The rapidjson flag to use on AVS is: -DRAPIDJSON_MEM_OPTIMIZATION and the flag to be used on Alexa smart screen sdk is: -DAPLCORE_RAPIDJSON_INCLUDE_DIR.
As for the crash, have you been able to fetch the payload through the backtrace as @VaruPan suggested? This will help us debug further

Hi @VaruPan/@karpuv
We fetch the payload from APLClient::AplCoreConnectionManager::handleUpdateCursorPosition. The paylaod is as follows,
Payload and crash
[2021-01-11 22:50:42.007] [smartscreenSDK - AplCoreConnectionManager] handleUpdateCursorPosition payload {"x":null,"y":null}
[2021-01-11 22:50:42.007] TpvSmartScreenAlexaApp: /home/kumar.jagadish/add_prints_updateCursorPosition/alexa/prebuilts/mvoiceapp/avs/prebuilts/include/rapidjson/document.h:1692: double rapidjson::GenericValue<Encoding, Al
[2021-01-11 22:50:42.023] locator>::GetDouble() const [with Encoding = rapidjson::UTF8<>; Allocator = rapidjson::MemoryPoolAllocatorrapidjson::CrtAllocator]: Assertion `IsNumber()' failed.
scenario:
while GUI page loaded, pressed the back key and home key on our device.
Observation:
seen that x and y co-ordinates in payload is null. since in the code, GetFloat() API used to read the co-ordinates from payload.  it is eventually causing the crash.
can we block to send this payload to APL. if yes, is there any regression or side effect with that.
if not, can you give any suggestion to avoid this issue.
BTW, may i know how this cursorPosition payload is used in APL..?

Hi @VaruPan/@karpuv
We fetch the payload from APLClient::AplCoreConnectionManager::handleUpdateCursorPosition. The paylaod is as follows,
Payload and crash
[2021-01-11 22:50:42.007] [smartscreenSDK - AplCoreConnectionManager] handleUpdateCursorPosition payload {"x":null,"y":null}
[2021-01-11 22:50:42.007] TpvSmartScreenAlexaApp: /home/kumar.jagadish/add_prints_updateCursorPosition/alexa/prebuilts/mvoiceapp/avs/prebuilts/include/rapidjson/document.h:1692: double rapidjson::GenericValue<Encoding, Al
[2021-01-11 22:50:42.023] locator>::GetDouble() const [with Encoding = rapidjson::UTF8<>; Allocator = rapidjson::MemoryPoolAllocatorrapidjson::CrtAllocator]: Assertion `IsNumber()' failed.
scenario:
while GUI page loaded, pressed the back key and home key on our device.
Observation:
seen that x and y co-ordinates in payload is null. since in the code, GetFloat() API used to read the co-ordinates from payload.  it is eventually causing the crash.
can we block to send this payload to APL. if yes, is there any regression or side effect with that.
if not, can you give any suggestion to avoid this issue.
BTW, may i know how this cursorPosition payload is used in APL..?

Hi @Jagadish-Bommisetty,
The rendering engine needs to know the cursor position to respond correctly to clicks/hover/swiping events. I suspect blocking these messages may result in missed events. I encourage you to post this question on rendering engine project for further clarifications.
You can also investigate on - why the browser is sending null cursor positions? I can confirm that we do not see null values in payload on reference platforms.
Thank you for your contribution to the Alexa Smart Screen SDK!

Hi @Jagadish-Bommisetty,
The rendering engine needs to know the cursor position to respond correctly to clicks/hover/swiping events. I suspect blocking these messages may result in missed events. I encourage you to post this question on rendering engine project for further clarifications.
You can also investigate on - why the browser is sending null cursor positions? I can confirm that we do not see null values in payload on reference platforms.
Thank you for your contribution to the Alexa Smart Screen SDK!


tests: add tests for message bodies in the p2p protocol

I'm opening early a work in progress to get better feedback from unit tests when the p2p message change, inspired by spending countless hours running valgrind to debug p2p communications while working on #3149 and making really slow progress.
I am currently focussing on construction only, but I think it would make sense to also test parsing of messages that come in over p2p.

Currently, no unit tests cover the contents of p2p message bodies (headers are less interesting because they're static). Message body regressions are extremely hard to spot because of the generic message creation we use inside net_processing, but as this is the core specification of the protocol, these are extremely important. The most complete tests I've seen for this are the tests written for btcd/wire and I think it would be good to match or, preferably, exceed that level of detail in our own unit tests.
A potential followup PR could make calls to CNetMsgMaker explicit in the net_processing code too, to further guard against cases where for example a PR changes the arguments to the call. I figured the best start would be to introduce the tests, to also have a control mechanism when reworking the message creation calls.
This PR aims to introduce those tests over 2 suites:

net: all p2p networking & control messages

[ ] addr
[ ] feefilter
[ ] getaddr
[ ] ping
[ ] pong
[ ] verack
[ ] version

blockchain: all chain & transaction messages

[ ] block
[ ] blocktxn
[ ] cmpctblock
[ ] filteradd
[ ] filterclear
[ ] filterload
[ ] getblocks
[ ] getblocktxn
[ ] getdata
[ ] getheaders
[ ] headers
[ ] inv
[ ] mempool
[ ] merkleblock
[ ] notfound
[ ] reject
[ ] sendcmpct
[ ] sendheaders
[ ] tx

For now, I'm extremely interested in opinions re: concept.

ConceptACK. This is awesome! Will take a deeper look over the next few days and try to come up with some useful feedback but am in agreement we could use some more robustness re logs/debugging net proc and having this in as a more refined control would be super useful for experimentation purposes. :+1:

Concept ACK for me as well. I want to keep an eye on this to make sure this grows in a direction that's as easy to maintain as it is to find and debug failed assertions. Looks good so far.


Verify existence of messages, entities and filters before starting registration

Before starting the registration of any plugins, verify that the following attributes exist in the target environment for all steps:
<Step>
    <Message>update</Message>
    <PrimaryEntity>contact</PrimaryEntity>
</Step>

Required for the primaryEntityType, the sdkMessage, and sdkMessageFilter

This validation will likely necessitate several extra API hits.  So, for flexibility:

save the metadata query results for downstream use during registration
have a flag preventing validation (e.g. validate once,  not every time)
have the validation as an individual method/action

This validation will likely necessitate several extra API hits.  So, for flexibility:

save the metadata query results for downstream use during registration
have a flag preventing validation (e.g. validate once,  not every time)
have the validation as an individual method/action


Allow setting max_token and temperature for OpenAI chat client

Is your feature request related to a problem? Please describe.
I was surprised to see that max_tokens and temperature have been hardcoded for the OpenAI_Chat class implementation. This is not optimal, as I would think that at least increasing the max_tokens would be relevant for some scenarios and lowering the temperature might be necessary to improve robustness.
https://github.com/vanna-ai/vanna/blob/fb384d43a0fb50a7cbe366cd6c242d5d37b16569/src/vanna/openai/openai_chat.py#L71
Describe the solution you'd like
The max_tokens and temperature can be set through the config argument to the class.
Additional context
I also saw that there were some strange linting-artifacts likely due to not looking too careful at the code after running the pre-commit hooks.
I can make a PR to address these concerns.

Made a PR:
https://github.com/vanna-ai/vanna/pull/263


[CI] Configure Meshery for Codespaces

Current Behavior
Later this year, GitHub Codespaces will be made available to the general public. In the meantime, select open source organizations may engage with Codespaces early to both benefit from the tool and offer feedback on its experience and functionality.
Desired Behavior
The ability to open Meshery and it's components in Codespaces via a button like this:

Implementation
The instructions to make Meshery's repos work with Codespaces looks something like this: customize your Codespace with a development container (an example).
Acceptance Tests
A single click to get Meshery and its components up and running in Codespaces.

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

👀 this sounds pretty interesting. Can I take this up @leecalcote?

Are we (as GitHub members & layer5/Meshery) automatically members to this "hidden" CodeSpace?

I already worked on a similar issue. Love to work on this.

Great to see the interest! I have enabled Codespaces for the meshery and layer5 GitHub Organizations which means that organization members will have access when working on repos in those organizations.
A few key documents when getting started:

The video and intro are good places to start https://docs.github.com/en/codespaces
Customizing the Codespaces https://docs.github.com/en/codespaces/setting-up-your-project-for-codespaces/introduction-to-dev-containers
Example devcontainers https://github.com/microsoft/vscode-dev-containers/tree/main/containers
The newly open development containers specs https://containers.dev/

@cpepper96 @sivanaikk, two brains are better than one on this. I anticipate some collaboration will help drive the effort forward and help you overcome potential hurdles. Thanks for raising your hands. 🙌

@cpepper96 Let's pair and work on the issue. I read the documents shared before by @craiglpeters

Meshery UI is not coming up over HTTPS from codespaces #5973

Propose plugins and suggested settings that align with the projects’ lint-enforced coding conventions to add in codespace #6056


ActionMap Modifier Passthrough

If there exists ActionMap bindings for a Key and for a modifier + the same Key, the modifier version will always be called.
Example
Binding function_1 to "w" and function_2 to "shift w".
Pressing "w" or "shift w" will both call function_2.

Fixed by #343.


Install problem from composer

`composer require edofre/yii2-fullcalendar-scheduler "1.1.x-dev"
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for edofre/yii2-fullcalendar-scheduler 1.1.x-dev -> satisfiable by edofre/yii2-fullcalendar-scheduler[1.1.x-dev].
- edofre/yii2-fullcalendar-scheduler 1.1.x-dev requires npm-asset/fullcalendar-scheduler 1.7.1 -> no matching package found.
`

Installation works without problems for me.


update test and bug fix

Coverage increased (+14.2%) to 28.589% when pulling ba7c73c04814ca701efac700d7d809386d7a3442 on dev into da859cab00122efb54a61cc12f335939bcd94a91 on master.


Minor python style tweaks

This implements the following minor style tweaks that I'd rather have in before the freeze:

Binary operators that are across multiple lines lead with the operator on the new line
Consistency in how long function arguments are broken up

lint error, otherwise looks good


Frontend Prettier in one shot

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/781
Originally created by @derek-knox on 2019-07-11 13:46:24

[ ]  @derek-knox Update the FE codebase formatting in one shot as to not conflate this update with other feature work (easier to review and confirm nothing got borked)

View 3 previous comments from the original issue on GitLab


Update to pixi.js 6.3 once it updates

Description
In the new 6.3 version there will be support for server side apps, not giving errors when the library is loaded server side, i'd hope you could update to that version so that it will be easier to use this module in apps that use server side rendering
https://github.com/pixijs/pixijs/pull/8050

Thanks for the suggestion, although I'm not planning to implement this on short term.
However feel free to collaborate and create a PR with the implementation and tests. Thanks!


SCSS Compilation runs very slowly in debug

I'm using VS 2013 Premium with the 64bit version of libsass-net.
When running outside the debugger the compilation of an scss file takes < 1 second, running within the debugger results in a compilation of around 10-20 seconds.
I don't suppose you have any thoughts on what the problem might be?
Kind regards
Ian

FYI The problem seemed to go away after deleting both the .suo file and all the temporary asp.net files.


Cannot find

The header of this article states:  "You can also see the version information of Outlook apps that users are using, which will allow you to follow up with those who are using unsupported versions [...]"
Reviewing the document, I can find the report that shows the total for users using Office 2010 or 2013, but there is no way to find who the users are.  If there is a way, the documentation doesn't clearly show, and I haven't found it.

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

ID: 535888ea-6011-1467-bf9a-1de66cb8c402
Version Independent ID: d72f6ddc-15a0-5398-251e-92496afdf309
Content: Office 365 Reports in the admin center - Email apps usage
Content Source: Office365-Admin/activity-reports/email-apps-usage.md
Service: o365-administration
GitHub Login: @KwekuA
Microsoft Alias: kwekua

@jcrowland Thank you for your feedback. You may not be able to see the username section unless you add it. You can do that using the steps provided in Step8 https://github.com/MicrosoftDocs/OfficeDocs-O365ITPro/pull/879
Hope this helps!
Thanks
Sri

@jcrowland  - From our understanding, the issue you raised has been answered by msbemba so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!


Better handling of differential uploads for PDFs

Currently, upload_to_zenodo will try to lob whatever the specified PDF is at Zenodo. Zenodo itself is idempotent, e.g. won't change the upload if the MD5 checksum matches, but this (a) is slow because each paper is 1-4MB, which is roughly 100MB per conference and over 1GB in aggregate, and (b) requires that the PDFs are accessible if local, or suffers both a download and upload if the electronic edition (ee) is a URL on the web.
There are few ways around this (and maybe others):

track MD5 checksums from Zenodo in the proceedings database
before uploading, ask Zenodo what the latest MD5 checksum is
toggle PDF uploading as a global arg

(3) is certainly the easiest to implement, but also the easiest to misuse / create drift. That said, perhaps we start there and revisit if it becomes problematic?

realizing I left it out, the issue with (3) is if we want to only update some of the PDFs ... (3) is an all or nothing option. If we wanted a partial bypass, we'd have to do something else (side-chain bypass info?)


One-drive links are broken

None of the one-drive links to checkpoints and/or datasets seem to be working.

Hi,
I have re-check the one-drive links and they can work normally. I can email them to you if you need.

Sorry for the issue, apparently the was (is) something wrong with my VPN. My bad. Thanks @jpWang !


Cannot find pretrained weight for DETR

Under mmdetection/configs/detr, if I click on the 'model' inside results and models, it redirects me back to the same page and does not start downloading the weight file. Usually, when I have done this for other models, the weights start being downloaded.
Could you provide me the link to the DETR pretrained weights? Thanks in advance.

@ZwwWayne @hellock

Hi @TheDarkKnight300 ,
I have converted DETR pretrained weights (R50) from https://github.com/facebookresearch/detr to mmdet format(configs/detr/detr_r50_8x4_150e_coco.py). Please check it out.
https://drive.google.com/file/d/1qQwVqF_SkncL663ig7GpGauFYSYeQErU/view?usp=sharing

It works perfectly, thank you. @Ngoc-Nguyen-NIS

DETR checkpoints will be released soon in the next week.


Msgpack

It looks like you didn't end up changing the getItem endpoint from responding with text/plain to msgpacked item in the format {"key": "someKey", "value": "someValue"}.
I made this change on the client. Did you just forget the getItem endpoint? Or did I misunderstand?
Looks great otherwise, thanks for doing this!

You're right. I made that change, then pulled it out so this branch would only be about the msgpack changeover.
In the meantime there shouldn't be any client compat issues since it never actually calls the single-get endpoint.

Ok, I implemented the last few changes. I've tested with the chore/.. ldbrest_client branch. Works well!
I'll wait for you to merge @teepark


Please Can I Join This Organisation? 🙏

I want to join this organization.

Our Website 🌐 | Repositories 📚 | LinkFree Profile 🔗
You have received an email to join, or press this link - https://github.com/App-Choreography - , and you'll see a banner, for you to join the organisation! 🎉
You can also show that you are part of this org, by making yourself public. This will show our profile org on your profile:
Follow This:

After you join the organisation, search your name in the list of people. 🔍
Follow this below: ⬇

When you join our org, you will be automatically be part of our community team, where you will get updates, and news about our org!
We are so happy you have joined! Welcome to App Choreography! 🥳


Fix TXT records

This is basically a hack since route53 doesn't really support multiple TXT records.
TODO:

[x] Delete TXT records
[x] Set multiple TXT records

Fix: Unquote TXT records
Fix: Unquote/Quote SPF records
Fix: Correct returned records name
TXT records Catch:

record TTL will be set to the last TXT record TTL
TXT records will always be upserted i.e. it won't fail if TXT record already exists

Fixes: https://github.com/libdns/route53/issues/17
Fixes: https://github.com/libdns/route53/issues/16

Is there any ETA on when this might get merged? This seems like it would resolve an existing bug in https://github.com/caddy-dns/route53 as well if it can get merged and incorporated into a new release of that project.

A very hacky approach, but needed. Merging this...


DB::raw where IN only returns first row (or imposes first ID on all rows)

Laravel Version: 5.1.46
PHP Version: 5.6.25
Database Driver & Version: MySQL 5.7.14

Description:
When using DB::raw together with IN and a bound parameter Laravel only returns the first row (or, when not using a unique ID, returns all rows with the ID set to the first ID from the IN).
Steps To Reproduce:
    DB::select(
        DB::raw("
            select * from `entity` where `id` in (:ids)"
        ), array(
            'ids'   => '1,2,3,4,5',
        )
    );

Expected result: rows 1, 2, 3, 4 and 5.
Actual result: only row 1 is returned.
Without the bound parameter it works fine:
    DB::select(
        DB::raw("
            select * from `entity` where `id` in (1,2,3,4,5)"
        )
    );

Check https://stackoverflow.com/questions/1586587/pdo-binding-values-for-mysql-in-statement


Only war enhanced importer

Changes / Comments
Added proper calculations to the Scatter button
Hit location now reverses the attack roll
Added Move Mod, an AGI modifier that only affects movement
Added the smelly corpse of what should have been an import/export function
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

howdy I have never done anything like this before But this is the enhanced character sheet update right? if I could get some help It would me much appreciated! : )


Update to Django 3.2

#2680 is the base PR for this one. First, we need to merge that, then we can move to it.

Codecov Report

Merging #2684 (7e8b28e) into main (591f2c3) will increase coverage by 0.14%.
The diff coverage is 93.24%.

@@            Coverage Diff             @@
##             main    #2684      +/-   ##
==========================================
+ Coverage   80.63%   80.78%   +0.14%
==========================================
  Files         651      655       +4
  Lines       23189    23222      +33
==========================================
+ Hits        18698    18759      +61
+ Misses       4491     4463      -28

Impacted Files
Coverage Δ

hypha/apply/activity/models.py
96.39% <ø> (ø)

hypha/apply/flags/views.py
56.52% <0.00%> (ø)

hypha/apply/users/admin_views.py
30.76% <0.00%> (ø)

hypha/apply/users/utils.py
71.05% <ø> (ø)

hypha/apply/users/views.py
53.43% <ø> (ø)

hypha/settings/base.py
68.72% <0.00%> (ø)

hypha/apply/funds/utils.py
70.00% <84.61%> (+7.03%)
:arrow_up:

hypha/apply/categories/admin.py
88.46% <100.00%> (ø)

hypha/apply/categories/apps.py
100.00% <100.00%> (+100.00%)
:arrow_up:

hypha/apply/dashboard/apps.py
100.00% <100.00%> (+100.00%)
:arrow_up:

... and 38 more

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 591f2c3...7e8b28e. Read the comment docs.


Clean up and bump cdk-pgstac to latest eoapi-cdk

removed .env and deploy.sh, which were deployment tools living in the root of the repository that are not used anymore since we have a functional deployment github action.
updated README, which was still describing a manual deployment. It's now describing the github action deployment.
bumped cdk-pgstac to eoapi-cdk 5.0.0 (the name of the library was changed)

@jjfrench could you quickly check this and approve so that I can merge ? It's pretty minor, the cdk-pgstac bump is 4.2.2 -> 5.0.0, and the two changes with this bump are (1) the name changed to eoapi-cdk, (2) I improved one of the cloud formation outputs.

Do we want to import from eoapi-cdk as well?

Thanks @jjfrench. Anything else ?

hm I am not changing the package-lock file and that's what is used I think when dependencies are installed in the workflow...

Looks good, do we want to merge and then create another PR for the titiler fix?

I'd say :

let's move the bump to https://github.com/MAAP-Project/maap-eoapi/pull/15 since it depends on yet another update to eoapi-cdk
let's keep here only the README change and deploy.sh removal and merge rn


Add a way to build a Docker image for running scripts

We've been running scripts locally and on Jenkins using virtualenv. This has been working mostly fine so far but has a few downsides:

We're not explicitly managing the python version. This lead us to use pyenv on Jenkins because Ubuntu 14.04 shipped with an old Python 2 version without proper SSL certificate verification support. This also creates problem for python 3 since in order to switch individual scripts to run with 3.6 instead of 2.7 we'd need to pin the python version in each Jenkins job explicitly. Switching the python version also requires deleting existing venv folders from the job workspace, which would have to be done manually.
Some of the scripts rely on additional tools / packages (like wkhtmltopdf) which have to be separately installed locally and on Jenkins. Their version is managed differently in different environments and there's no way to express that dependency in this repo.

Building a docker image with the scripts allows us to specify a Python version, upgrade existing Jenkins jobs to Python 3 without managing a python interpreter with pyenv and potentially makes it much easier to deploy and run scripts on PaaS (since we already have a deployment pipeline for docker images).
This adds Dockerfile, .dockerignore and make *-docker targets. Since scripts requirements are quite different from the application ones the Docker image is based on the python one, not our own base image.
We're only building a Python 3 image since we've agreed that we want to make scripts python3-only in the future and jobs that aren't compatible yet can use the existing pyenv setup for now.
This is also only setting up the python packages it doesn't install any script-specific dependencies (like wkhtmltopdf). Those will have to be added later once the scripts have been tested under python3.
Add freeze-requirements
At the moment, scripts running in Jenkins are using the local venv created by the Jenkins job, which means they won't update any packages unless a version is changed explicitly.
This means that recreating that venv could end up with a different set of package versions than what's currently installed.
We need to freeze our nested package requirements to make sure we're not accidentally updating packages between builds.
Similar to the application changes, this renames requirements.txt to requirements-app.txt. Updates Makefile to add freeze-requirements.
requirements-app.txt is not a great name for the scripts repo, but it's consistent with what we call it in other repos.
This also switches Travis to use make test target instead of running py.test directly. make requirements is setting up dependencies in the virtualenv, but running py.test directly isn't activating that virtualenv explicitly. The only reason this worked on Travis is because it's activating a clean virtualenv before running any steps for the python builds.
NOTE: This is upgrading dmutils to bring a newer version of the cryptography package in Python 3. There's a major change related to S3 in the new utils version, so scripts using it might need to be updated.

The Dockerfile is all a bit dark arts to me, but everything looks sensible as far as I can tell.
Were we going to add some warnings to scripts that we suspect might not work properly after this update as part of the PR? If not then :+1:


NOTICE - Splitting of debugging cli into a seperate repo

During a couple of conversation on the route to release 5.0.0 it became apparent that the debugging cli should be partitioned away from the main core-dump collection. This is due to the shared configuration between the client and server not being ideal and the release cadence of the cli being very different than the server.
The likely location for the cli will be https://github.com/IBM/core-dump-containers/ which itself will be renamed
This will happen as part of the 6.0.0 release due in Novemeber

Removed in https://github.com/IBM/core-dump-handler/commit/92b2b9d6678252fb01780139900440e522f390d3

Completed as part of https://github.com/IBM/core-dump-handler/pull/43


[sort-imports] update storage-blob with respect to sort-imports rule

This PR is working in conjunction with other PRs to solve #9252.
As per my previous PR #18598, I will be making multiple PRs to fix individual sections of the azure codebase with respect to the new sort-imports rule.
In this PR, I have fixed all files under sdk/storage/storage-blob.

It seems I'm getting a lot of failed checks, which is likely due to some conflicts when I called git cherry-pick. I will revert back to a stable version and manually fix linter errors to avoid this.

There seems to still be tests failing, which could have occurred after I ran rushx lint --fix. I think I'll close this PR and restart :)


cascading Action.ShowCard shows as onclick in Teams

cascading Action.ShowCard shows as onclick in Teams (in dark color) before clicking on them---pls check the screenshot.

OK in https://adaptivecards.io/designer/, but changed in Teams. Could you pls help confirm and share any insight? Thanks.

URL: https://adaptivecards.io/explorer/Action.ShowCard.html

This issue has been copied over to Microsoft Teams repo here: https://github.com/MicrosoftDocs/msteams-docs/issues/8246


Flaky test cmd/agent/app/reporter/grpc/TestBuilderWithCollectors/with_collector_connection_status_ready

https://github.com/jaegertracing/jaeger/pull/2657/checks?check_run_id=1486634709#step:5:346
=== RUN   TestBuilderWithCollectors/with_collector_connection_status_ready
    builder_test.go:182:
        	Error Trace:	builder_test.go:182
        	Error:      	Not equal:
        	            	expected: int(1)
        	            	actual  : int64(0)
        	Test:       	TestBuilderWithCollectors/with_collector_connection_status_ready
    builder_test.go:183:
        	Error Trace:	builder_test.go:183
        	Error:      	Not equal:
        	            	expected: int(1)
        	            	actual  : int64(0)
        	Test:       	TestBuilderWithCollectors/with_collector_connection_status_ready
    builder_test.go:184:
        	Error Trace:	builder_test.go:184
        	Error:      	Not equal:
        	            	expected: "<IP_ADDRESS>:41669"
        	            	actual  : ""

        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-<IP_ADDRESS>:41669
        	            	+
        	Test:       	TestBuilderWithCollectors/with_collector_connection_status_ready

Probably introduced in #2423 - cc @BernardTolosajr @jpkrohling

But could've been dormant even earlier.

I will check the issue. thanks

I will check the issue. thanks

Hi!
How big of an issue is this flaky test?
I made some changes to the code and tested with:

go test github.com/jaegertracing/jaeger/cmd/agent/app/reporter/grpc -run TestBuilderWithCollectors -count=10000 -failfast

What I saw so far:
The test case "TestBuilderWithCollectors/with_collector_connection_status_ready" is the only one where a real TCP connection is established. And it usually failed on my machine when the vast amount of test repetition caused some temporary resource issues in the systems TCP stack.

hasn't come up for a while, closing


mbed dm init not working - cannot import python_2_unicode_compatible

Hi,
I want to try a firmware update, using mbed-cli version 1.10.2 and python 2.7.15+ in Ubuntu18.
The error I am getting is:
george@ubuntu:~/repo/mbed-os-example-pelion$ mbed dm init -d "Arm" --model-name "product_name"
[mbed] Working path "/home/george/repo/mbed-os-example-pelion" (program)
Traceback (most recent call last):
  File "/home/george/repo/mbed-os-example-pelion/mbed-os/tools/device_management.py", line 27, in <module>
    from manifesttool import create, parse, verify, cert, init, update
  File "/home/george/.local/lib/python2.7/site-packages/manifesttool/create.py", line 19, in <module>
    from manifesttool.v1.create import main as create_v1
  File "/home/george/.local/lib/python2.7/site-packages/manifesttool/v1/create.py", line 20, in <module>
    import codecs, hashlib, sys, json, os, base64, binascii, logging, ecdsa, uuid
  File "/home/george/.local/lib/python2.7/site-packages/ecdsa/__init__.py", line 1, in <module>
    from .keys import SigningKey, VerifyingKey, BadSignatureError, BadDigestError,\
  File "/home/george/.local/lib/python2.7/site-packages/ecdsa/keys.py", line 72, in <module>
    from . import ecdsa
  File "/home/george/.local/lib/python2.7/site-packages/ecdsa/ecdsa.py", line 57, in <module>
    from . import ellipticcurve
  File "/home/george/.local/lib/python2.7/site-packages/ecdsa/ellipticcurve.py", line 49, in <module>
    from six import python_2_unicode_compatible
ImportError: cannot import name python_2_unicode_compatible
[mbed] ERROR: "/usr/bin/python" returned error.
       Code: 1
       Path: "/home/george/repo/mbed-os-example-pelion"
       Command: "/usr/bin/python -u /home/george/repo/mbed-os-example-pelion/mbed-os/tools/device_management.py init -d Arm --model-name product_name"
       Tip: You could retry the last command with "-v" flag for verbose output
---

How can this be fixed? pip install -U mbed-cli didn't help.

Installing the mbed-os requirements results in a segmentation fault:
george@ubuntu:~/repo/mbed-os-example-pelion$ pip install -r mbed-os/requirements.txt
Ignoring hidapi: markers 'platform_system != "Linux"' don't match your environment
Ignoring pywin32: markers 'platform_system == "Windows"' don't match your environment
Ignoring wmi: markers 'platform_system == "Windows"' don't match your environment
Collecting colorama==0.3.9 (from -r mbed-os/requirements.txt (line 1))
  Using cached https://files.pythonhosted.org/packages/db/c8/7dcf9dbcb22429512708fe3a547f8b6101c0d02137acbd892505aee57adf/colorama-0.3.9-py2.py3-none-any.whl
Collecting urllib3[secure]==1.24.2 (from -r mbed-os/requirements.txt (line 2))
  Using cached https://files.pythonhosted.org/packages/df/1c/59cca3abf96f991f2ec3131a4ffe72ae3d9ea1f5894abe8a9c5e3c77cfee/urllib3-1.24.2-py2.py3-none-any.whl
Collecting prettytable==0.7.2 (from -r mbed-os/requirements.txt (line 3))
Collecting junit-xml==1.8 (from -r mbed-os/requirements.txt (line 4))
Collecting pyyaml==4.2b1 (from -r mbed-os/requirements.txt (line 5))
Collecting jsonschema==2.6.0 (from -r mbed-os/requirements.txt (line 6))
  Using cached https://files.pythonhosted.org/packages/77/de/47e35a97b2b05c2fadbec67d44cfcdcd09b8086951b331d82de90d2912da/jsonschema-2.6.0-py2.py3-none-any.whl
Collecting future==0.16.0 (from -r mbed-os/requirements.txt (line 7))
Collecting six==1.12.0 (from -r mbed-os/requirements.txt (line 8))
  Using cached https://files.pythonhosted.org/packages/73/fb/00a976f728d0d1fecfe898238ce23f502a721c0ac0ecfedb80e0d88c64e9/six-1.12.0-py2.py3-none-any.whl
Collecting mbed-cloud-sdk<2.1,>=2.0.6 (from -r mbed-os/requirements.txt (line 9))
  Using cached https://files.pythonhosted.org/packages/50/34/4bc3b0efe5d4cb56fba71535ef16c607f7ce125d80841a3e2b2e9962df44/mbed_cloud_sdk-2.0.8-py2.py3-none-any.whl
Collecting requests<2.21,>=2.20 (from -r mbed-os/requirements.txt (line 10))
  Using cached https://files.pythonhosted.org/packages/ff/17/5cbb026005115301a8fb2f9b0e3e8d32313142fe8b617070e7baad20554f/requests-2.20.1-py2.py3-none-any.whl
Collecting idna<2.8,>=2 (from -r mbed-os/requirements.txt (line 11))
  Using cached https://files.pythonhosted.org/packages/4b/2a/0276479a4b3caeb8a8c1af2f8e4355746a97fab05a372e4a2c6a6b876165/idna-2.7-py2.py3-none-any.whl
Collecting pyserial<=3.4,>=3 (from -r mbed-os/requirements.txt (line 12))
  Using cached https://files.pythonhosted.org/packages/0d/e4/2a744dd9e3be04a0c0907414e2a01a7c88bb3915cbe3c8cc06e209f59c30/pyserial-3.4-py2.py3-none-any.whl
Collecting Jinja2<2.11,>=2.10.1 (from -r mbed-os/requirements.txt (line 13))
  Using cached https://files.pythonhosted.org/packages/65/e0/eb35e762802015cab1ccee04e8a277b03f1d8e53da3ec3106882ec42558b/Jinja2-2.10.3-py2.py3-none-any.whl
Collecting intelhex<=2.2.1,>=1.3 (from -r mbed-os/requirements.txt (line 14))
  Using cached https://files.pythonhosted.org/packages/bf/77/bf670318b3db325c71e2ac6a90b7bcfdf9fc739b7cf6aebb31715721623e/intelhex-2.2.1-py2.py3-none-any.whl
Collecting mbed-ls<1.8,>=1.5.1 (from -r mbed-os/requirements.txt (line 15))
  Using cached https://files.pythonhosted.org/packages/8e/2c/72855a353999cad5ff6ef13120ef60eb1cbc3112ecdd5a02fdd022e22236/mbed_ls-1.7.10-py2.py3-none-any.whl
Collecting mbed-host-tests<1.6,>=1.4.4 (from -r mbed-os/requirements.txt (line 16))
  Using cached https://files.pythonhosted.org/packages/7f/94/1e727eb5955341a23ac892326ef3d8c1544202b13453f57c09d48e78b307/mbed_host_tests-1.5.10-py2.py3-none-any.whl
Collecting mbed-greentea<1.8,>=0.2.24 (from -r mbed-os/requirements.txt (line 17))
  Using cached https://files.pythonhosted.org/packages/85/9f/8336a68476298eabdd405123ebbfec802f71a89610f52e92d52fcfaf30ea/mbed_greentea-1.7.4-py2.py3-none-any.whl
Collecting beautifulsoup4<=4.6.3,>=4 (from -r mbed-os/requirements.txt (line 18))
  Using cached https://files.pythonhosted.org/packages/f8/c7/741c97d7366f4779ca73d244904978b43a81fd37d85fcf05ad19d472c1ce/beautifulsoup4-4.6.3-py2-none-any.whl
Collecting pyelftools<=0.25,>=0.24 (from -r mbed-os/requirements.txt (line 19))
  Using cached https://files.pythonhosted.org/packages/b8/45/00776dc25dc2ad0573d48c2d86f2d4a554216d00bf4575423557f5a398ca/pyelftools-0.25-py2.py3-none-any.whl
Collecting manifest-tool==1.5.2 (from -r mbed-os/requirements.txt (line 20))
Collecting icetea<1.3,>=1.2.1 (from -r mbed-os/requirements.txt (line 21))
Collecting pycryptodome<=3.7.3,>=3.7.2 (from -r mbed-os/requirements.txt (line 22))
  Using cached https://files.pythonhosted.org/packages/29/68/0ee43d649a2f748f805c427558e3ec8d832faa34ca1f369195b274266e1d/pycryptodome-3.7.3-cp27-cp27mu-manylinux1_x86_64.whl
Collecting pyusb<2.0.0,>=1.0.0 (from -r mbed-os/requirements.txt (line 23))
Collecting cmsis-pack-manager<0.3.0,>=0.2.3 (from -r mbed-os/requirements.txt (line 25))
  Using cached https://files.pythonhosted.org/packages/01/2d/631ba8f2eaffff4862fef21aa7399ed1c1f976b75089502fad26629dfa54/cmsis_pack_manager-0.2.10-py2.py3-none-manylinux1_x86_64.whl
Collecting psutil==5.6.2 (from -r mbed-os/requirements.txt (line 28))
Collecting cryptography<2.5,>=2.4.x (from -r mbed-os/requirements.txt (line 29))
  Using cached https://files.pythonhosted.org/packages/7f/ba/383b51cc26e3141c689ce988814385c7659f5ba01c4b5f2de38233010b5f/cryptography-2.4.2-cp27-cp27mu-manylinux1_x86_64.whl
Collecting Click<7.1,>=7.0 (from -r mbed-os/requirements.txt (line 30))
  Using cached https://files.pythonhosted.org/packages/fa/37/45185cb5abbc30d7257104c434fe0b07e5a195a6847506c074527aa599ec/Click-7.0-py2.py3-none-any.whl
Collecting pyOpenSSL>=0.14; python_version == "2.7" and extra == "secure" (from urllib3[secure]==1.24.2->-r mbed-os/requirements.txt (line 2))
  Using cached https://files.pythonhosted.org/packages/9e/de/f8342b68fa9e981d348039954657bdf681b2ab93de27443be51865ffa310/pyOpenSSL-19.1.0-py2.py3-none-any.whl
Collecting ipaddress; python_version == "2.7" and extra == "secure" (from urllib3[secure]==1.24.2->-r mbed-os/requirements.txt (line 2))
  Using cached https://files.pythonhosted.org/packages/c2/f8/49697181b1651d8347d24c095ce46c7346c37335ddc7d255833e7cde674d/ipaddress-1.0.23-py2.py3-none-any.whl
Collecting certifi; extra == "secure" (from urllib3[secure]==1.24.2->-r mbed-os/requirements.txt (line 2))
  Using cached https://files.pythonhosted.org/packages/57/2b/26e37a4b034800c960a00c4e1b3d9ca5d7014e983e6e729e33ea2f36426c/certifi-2<IP_ADDRESS>-py2.py3-none-any.whl
Collecting functools32; python_version == "2.7" (from jsonschema==2.6.0->-r mbed-os/requirements.txt (line 6))
Collecting python-dotenv>=0.8.2 (from mbed-cloud-sdk<2.1,>=2.0.6->-r mbed-os/requirements.txt (line 9))
  Using cached https://files.pythonhosted.org/packages/cb/2a/07f87440444fdf2c5870a710b6770d766a1c7df9c827b0c90e807f1fb4c5/python_dotenv-0.13.0-py2.py3-none-any.whl
Collecting python-dateutil>=2 (from mbed-cloud-sdk<2.1,>=2.0.6->-r mbed-os/requirements.txt (line 9))
  Using cached https://files.pythonhosted.org/packages/d4/70/d60450c3dd48ef87586924207ae8907090de0b306af2bce5d134d78615cb/python_dateutil-2.8.1-py2.py3-none-any.whl
Collecting chardet<3.1.0,>=3.0.2 (from requests<2.21,>=2.20->-r mbed-os/requirements.txt (line 10))
  Using cached https://files.pythonhosted.org/packages/bc/a9/01ffebfb562e4274b6487b4bb1ddec7ca55ec7510b22e4c51f14098443b8/chardet-3.0.4-py2.py3-none-any.whl
Collecting MarkupSafe>=0.23 (from Jinja2<2.11,>=2.10.1->-r mbed-os/requirements.txt (line 13))
  Using cached https://files.pythonhosted.org/packages/fb/40/f3adb7cf24a8012813c5edb20329eb22d5d8e2a0ecf73d21d6b85865da11/MarkupSafe-1.1.1-cp27-cp27mu-manylinux1_x86_64.whl
Collecting mbed-os-tools<0.1.0,>=0.0.9 (from mbed-ls<1.8,>=1.5.1->-r mbed-os/requirements.txt (line 15))
  Using cached https://files.pythonhosted.org/packages/0f/9d/c2cf2b747a3a14f9433bbeefa47118a7cc27e066ac897d7025f66a8d143f/mbed_os_tools-0.0.13-py2.py3-none-any.whl
Collecting ecdsa>=0.13 (from manifest-tool==1.5.2->-r mbed-os/requirements.txt (line 20))
  Using cached https://files.pythonhosted.org/packages/b8/11/4b4d30e4746584684c758d8f1ddc1fa5ab1470b6bf70bce4d9b235965e99/ecdsa-0.15-py2.py3-none-any.whl
Collecting protobuf<3.6.0,>=3.5.0 (from manifest-tool==1.5.2->-r mbed-os/requirements.txt (line 20))
  Using cached https://files.pythonhosted.org/packages/9d/61/54c3a9cfde6ffe0ca6a1786ddb8874263f4ca32e7693ad383bd8cf935015/protobuf-3.5.2.post1-cp27-cp27mu-manylinux1_x86_64.whl
Collecting pyasn1<0.3.0,>=0.2.1 (from manifest-tool==1.5.2->-r mbed-os/requirements.txt (line 20))
  Using cached https://files.pythonhosted.org/packages/a5/ae/6b4c4cb9420edddd7401782f55504130d1269f2e5ae3ba3c986da167dd6c/pyasn1-0.2.3-py2.py3-none-any.whl
Collecting pyparsing>=2.1.0 (from manifest-tool==1.5.2->-r mbed-os/requirements.txt (line 20))
  Using cached https://files.pythonhosted.org/packages/8a/bb/488841f56197b13700afd5658fc279a2025a39e22449b7cf29864669b15d/pyparsing-2.4.7-py2.py3-none-any.whl
Collecting asn1ate>=0.5 (from manifest-tool==1.5.2->-r mbed-os/requirements.txt (line 20))
Collecting mbed-flasher<0.11,>=0.10.1 (from icetea<1.3,>=1.2.1->-r mbed-os/requirements.txt (line 21))
Collecting semver>=2.0 (from icetea<1.3,>=1.2.1->-r mbed-os/requirements.txt (line 21))
  Using cached https://files.pythonhosted.org/packages/cb/48/009529f08dd0f71262a229f0a7948dd5d27eb0e0a9edc74e6e6860ff2925/semver-2.9.1-py2.py3-none-any.whl
Collecting jsonmerge>=1.4 (from icetea<1.3,>=1.2.1->-r mbed-os/requirements.txt (line 21))
Collecting yattag>=1.0 (from icetea<1.3,>=1.2.1->-r mbed-os/requirements.txt (line 21))
Collecting appdirs>=1.4 (from cmsis-pack-manager<0.3.0,>=0.2.3->-r mbed-os/requirements.txt (line 25))
  Using cached https://files.pythonhosted.org/packages/56/eb/810e700ed1349edde4cbdc1b2a21e28cdf115f9faf263f6bbf8447c1abf3/appdirs-1.4.3-py2.py3-none-any.whl
Collecting milksnake>=0.1.2 (from cmsis-pack-manager<0.3.0,>=0.2.3->-r mbed-os/requirements.txt (line 25))
  Using cached https://files.pythonhosted.org/packages/27/be/e10e73f857ac98ef43587fa8db37a3ef6de56e728037a7b9728de26711c7/milksnake-0.1.5-py2.py3-none-any.whl
Collecting enum34; python_version < "3" (from cryptography<2.5,>=2.4.x->-r mbed-os/requirements.txt (line 29))
  Using cached https://files.pythonhosted.org/packages/6f/2c/a9386903ece2ea85e9807e0e062174dc26fdce8b05f216d00491be29fad5/enum34-1.1.10-py2-none-any.whl
Collecting asn1crypto>=0.21.0 (from cryptography<2.5,>=2.4.x->-r mbed-os/requirements.txt (line 29))
  Using cached https://files.pythonhosted.org/packages/e9/51/1db4a60049fb7390959be586b6eb743098e6cea3f6b2d3ed9e17fec62ba2/asn1crypto-1.3.0-py2.py3-none-any.whl
Collecting cffi!=1.11.3,>=1.7 (from cryptography<2.5,>=2.4.x->-r mbed-os/requirements.txt (line 29))
  Using cached https://files.pythonhosted.org/packages/08/29/8001b940ef40e7a25ffe8f3188bc9b118934b513d64f769dbf461e46f4ed/cffi-1.14.0-cp27-cp27mu-manylinux1_x86_64.whl
Collecting typing; python_version < "3.5" (from python-dotenv>=0.8.2->mbed-cloud-sdk<2.1,>=2.0.6->-r mbed-os/requirements.txt (line 9))
  Using cached https://files.pythonhosted.org/packages/22/30/64ca29543375759dc589ade14a6cd36382abf2bec17d67de8481bc9814d7/typing-<IP_ADDRESS>-py2-none-any.whl
Collecting fasteners (from mbed-os-tools<0.1.0,>=0.0.9->mbed-ls<1.8,>=1.5.1->-r mbed-os/requirements.txt (line 15))
  Using cached https://files.pythonhosted.org/packages/18/bd/55eb2d6397b9c0e263af9d091ebdb756b15756029b3cededf6461481bc63/fasteners-0.15-py2.py3-none-any.whl
Collecting lockfile (from mbed-os-tools<0.1.0,>=0.0.9->mbed-ls<1.8,>=1.5.1->-r mbed-os/requirements.txt (line 15))
  Using cached https://files.pythonhosted.org/packages/c8/22/9460e311f340cb62d26a38c419b1381b8593b0bb6b5d1f056938b086d362/lockfile-0.12.2-py2.py3-none-any.whl
Collecting setuptools (from protobuf<3.6.0,>=3.5.0->manifest-tool==1.5.2->-r mbed-os/requirements.txt (line 20))
  Using cached https://files.pythonhosted.org/packages/ab/b5/3679d7c98be5b65fa5522671ef437b792d909cf3908ba54fe9eca5d2a766/setuptools-44.1.0-py2.py3-none-any.whl
Collecting pycparser (from cffi!=1.11.3,>=1.7->cryptography<2.5,>=2.4.x->-r mbed-os/requirements.txt (line 29))
  Using cached https://files.pythonhosted.org/packages/ae/e7/d9c3a176ca4b02024debf82342dab36efadfc5776f9c8db077e8f6e71821/pycparser-2.20-py2.py3-none-any.whl
Collecting monotonic>=0.1 (from fasteners->mbed-os-tools<0.1.0,>=0.0.9->mbed-ls<1.8,>=1.5.1->-r mbed-os/requirements.txt (line 15))
  Using cached https://files.pythonhosted.org/packages/ac/aa/063eca6a416f397bd99552c534c6d11d57f58f2e94c14780f3bbf818c4cf/monotonic-1.5-py2.py3-none-any.whl
Installing collected packages: colorama, enum34, idna, asn1crypto, pycparser, cffi, six, ipaddress, cryptography, pyOpenSSL, certifi, urllib3, prettytable, junit-xml, pyyaml, functools32, jsonschema, future, typing, python-dotenv, chardet, requests, python-dateutil, mbed-cloud-sdk, pyserial, MarkupSafe, Jinja2, intelhex, beautifulsoup4, appdirs, monotonic, fasteners, lockfile, mbed-os-tools, mbed-ls, mbed-host-tests, mbed-greentea, pyelftools, ecdsa, setuptools, protobuf, pyasn1, pyparsing, asn1ate, manifest-tool, mbed-flasher, semver, jsonmerge, yattag, icetea, pycryptodome, pyusb, milksnake, cmsis-pack-manager, psutil, Click
Successfully installed Click-7.0 Jinja2-2.10.3 MarkupSafe-1.1.1 appdirs-1.4.3 asn1ate-0.6.0 asn1crypto-1.3.0 beautifulsoup4-4.9.0 certifi-2<IP_ADDRESS> cffi-1.14.0 chardet-3.0.4 cmsis-pack-manager-0.2.10 colorama-0.4.3 cryptography-2.7 ecdsa-0.15 enum34-1.1.10 fasteners-0.15 functools32-3.2.3.post2 future-0.18.2 icetea-1.2.4 idna-2.9 intelhex-2.2.1 ipaddress-1.0.23 jsonmerge-1.7.0 jsonschema-2.6.0 junit-xml-1.9 lockfile-0.12.2 manifest-tool-1.5.2 mbed-cloud-sdk-2.0.8 mbed-flasher-0.10.1 mbed-greentea-1.7.4 mbed-host-tests-1.5.10 mbed-ls-1.7.10 mbed-os-tools-0.0.13 milksnake-0.1.5 monotonic-1.5 prettytable-0.7.2 protobuf-3.5.2.post1 psutil-5.6.2 pyOpenSSL-19.1.0 pyasn1-0.2.3 pycparser-2.20 pycryptodome-3.7.3 pyelftools-0.25 pyparsing-2.4.7 pyserial-3.4 python-dateutil-2.8.1 python-dotenv-0.13.0 pyusb-1.0.2 pyyaml-4.2b1 requests-2.23.0 semver-2.9.1 setuptools-44.1.0 six-1.14.0 typing-<IP_ADDRESS> urllib3-1.25.9 yattag-1.13.2
Segmentation fault (core dumped)

Note that I am doing this from a freshly set up Virtual Machine. I expected it would work.

Thank you for raising this detailed GitHub issue. I am now notifying our internal issue triagers.
Internal Jira reference: https://jira.arm.com/browse/IOTBTOOL-671

Managed to solve the issue by using python3.


Refactor: integration tests

Summary
Details and comments

The good thing @korgan00 is that for me, locally, all the tests are working. So as we were assuming probably something related with your computer.
There are some problems with a couple of tests I think due to permissions. I will try to take a look these days and let you know but awesome work here ❤