Unable to connect to Minio endpoint on PUT (GET works fine)

Which version of the SDK?
My aws-lambda-cpp submodule is at the commit 1c884fe
Plattform
Ubuntu 19.10
Minio is running inside a Docker container (FROM minio/minio)
My Problem:
I would to GET an object from Minio, which is emulating S3, and PUT it right back (this is for testing only). My GET requests works, but my PUT request fails with the error Unable to connect to endpoint. However, I am able to PUT an object to Minio using curl (compare this script).
An extensive description of my problem, including the relevant code, can be found in stackoverflow.

The version of the aws-lambda-cpp is v0.2.4-4-g1c884fe

Hi @CAJan93 ,
could you activate and provide a sanitized trace log?

The PUT request will be repeated since it fails. I also wondered if this is a permissions error with Minio, but I am giving everyone access to everything since this is a local test env only.

[DEBUG] 2019-11-27 09:08:48.309 AWSAuthV4Signer [140524469766208] Canonical Header String: content-length:1024
content-type:application/octet-stream
host:<IP_ADDRESS>:9000
x-amz-content-sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
x-amz-date:20191127T090848Z

[DEBUG] 2019-11-27 09:08:48.309 AWSAuthV4Signer [140524469766208] Signed Headers value:content-length;content-type;host;x-amz-content-sha256;x-amz-date
[DEBUG] 2019-11-27 09:08:48.309 AWSAuthV4Signer [140524469766208] Canonical Request String: PUT
/develop/file

content-length:1024
content-type:application/octet-stream
host:<IP_ADDRESS>:9000
x-amz-content-sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
x-amz-date:20191127T090848Z

content-length;content-type;host;x-amz-content-sha256;x-amz-date
e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
[DEBUG] 2019-11-27 09:08:48.309 AWSAuthV4Signer [140524469766208] Final String to sign: AWS4-HMAC-SHA256
20191127T090848Z
20191127/us-east-1/s3/aws4_request
5a208b6e2e6efd4b4ac7598fd59ff76db8f0919b1e71a1885873f0183174ccf3
[DEBUG] 2019-11-27 09:08:48.309 AWSAuthV4Signer [140524469766208] Final computed signing hash: 7e3b5da3cda553553b7e30aabd77ed6c459fd71cd111fc324c25332f508eacc8
[DEBUG] 2019-11-27 09:08:48.309 AWSAuthV4Signer [140524469766208] Signing request with: AWS4-HMAC-SHA256 Credential=minio/20191127/us-east-1/s3/aws4_request, SignedHeaders=content-length;content-type;host;x-amz-content-sha256;x-amz-date, Signature=7e3b5da3cda553553b7e30aabd77ed6c459fd71cd111fc324c25332f508eacc8
[DEBUG] 2019-11-27 09:08:48.309 AWSClient [140524469766208] Request Successfully signed
[TRACE] 2019-11-27 09:08:48.309 CurlHttpClient [140524469766208] Making request to http://<IP_ADDRESS>:9000/develop/file
[TRACE] 2019-11-27 09:08:48.309 CurlHttpClient [140524469766208] Including headers:
[TRACE] 2019-11-27 09:08:48.309 CurlHttpClient [140524469766208] authorization: AWS4-HMAC-SHA256 Credential=minio/20191127/us-east-1/s3/aws4_request, SignedHeaders=content-length;content-type;host;x-amz-content-sha256;x-amz-date, Signature=7e3b5da3cda553553b7e30aabd77ed6c459fd71cd111fc324c25332f508eacc8
[TRACE] 2019-11-27 09:08:48.309 CurlHttpClient [140524469766208] content-length: 1024
[TRACE] 2019-11-27 09:08:48.309 CurlHttpClient [140524469766208] content-type: application/octet-stream
[TRACE] 2019-11-27 09:08:48.309 CurlHttpClient [140524469766208] host: <IP_ADDRESS>:9000
[TRACE] 2019-11-27 09:08:48.309 CurlHttpClient [140524469766208] user-agent: aws-sdk-cpp/1.7.210 Linux/5.3.0-19-generic x86_64 GCC/7.3.1
[TRACE] 2019-11-27 09:08:48.309 CurlHttpClient [140524469766208] x-amz-content-sha256: e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
[TRACE] 2019-11-27 09:08:48.309 CurlHttpClient [140524469766208] x-amz-date: 20191127T090848Z
[DEBUG] 2019-11-27 09:08:48.309 CurlHandleContainer [140524469766208] Attempting to acquire curl connection.
[INFO] 2019-11-27 09:08:48.309 CurlHandleContainer [140524469766208] Connection has been released. Continuing.
[DEBUG] 2019-11-27 09:08:48.309 CurlHandleContainer [140524469766208] Returning connection handle 0x16934e0
[DEBUG] 2019-11-27 09:08:48.309 CurlHttpClient [140524469766208] Obtained connection handle 0x16934e0
[TRACE] 2019-11-27 09:08:48.312 CurlHttpClient [140524469766208] HTTP/1.1 100 Continue

-27 09:08:48.312 CurlHttpClient [140524469766208]

[ERROR] 2019-11-27 09:08:51.313 CurlHttpClient [140524469766208] Curl returned error code 28 - Timeout was reached
[DEBUG] 2019-11-27 09:08:51.313 CurlHandleContainer [140524469766208] Releasing curl handle 0x16934e0
[DEBUG] 2019-11-27 09:08:51.313 CurlHandleContainer [140524469766208] Notified waiting threads.
[DEBUG] 2019-11-27 09:08:51.313 AWSClient [140524469766208] Request returned error. Attempting to generate appropriate error codes from response
[ERROR] 2019-11-27 09:08:51.313 AWSClient [140524469766208] HTTP response code: -1
Exception name:
Error message: Unable to connect to endpoint
0 response headers:
[WARN] 2019-11-27 09:08:51.313 AWSClient [140524469766208] If the signature check failed. This could be because of a time skew. Attempting to adjust the signer.
[DEBUG] 2019-11-27 09:08:51.313 AWSClient [140524469766208] Date header was not found in the response, can't attempt to detect clock skew
[WARN] 2019-11-27 09:08:51.313 AWSClient [140524469766208] Request failed, now waiting 0 ms before attempting again.
[DEBUG] 2019-11-27 09:08:51.313 EnvironmentAWSCredentialsProvider [140524469766208] Found credential in environment with access key id minio
[INFO] 2019-11-27 09:08:51.313 EnvironmentAWSCredentialsProvider [140524469766208] Found secret key
[DEBUG] 2019-11-27 09:08:51.313 AWSAuthV4Signer [140524469766208] Calculated sha256 e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855 for payload.
[DEBUG] 2019-11-27 09:08:51.313 AWSAuthV4Signer [140524469766208] Canonical Header String: content-length:1024
content-type:application/octet-stream
host:<IP_ADDRESS>:9000
x-amz-content-sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
x-amz-date:20191127T090851Z

[DEBUG] 2019-11-27 09:08:51.313 AWSAuthV4Signer [140524469766208] Signed Headers value:content-length;content-type;host;x-amz-content-sha256;x-amz-date

Hmmm... It looks like it is timing out after 3 seconds, which I think is the default timeout.
Could you try adding this to your client config, see if that works?
clientConfig.connectTimeoutMs = 30000;


Little piece of advice of TEorthology

Dear @4ureliek,
Thank you for all the available and resourceful scripts. I have been using parseRM and now I am playing around with TEorthology and it is making my life really easier. I would like some piece of advice of TE orthology before I run a massive analysis with it. My intended goal is to identify how a bunch of TE elements already identified across a invertebrate lineage are evolutionary related to each other. My ultimate goal is to identify the core TE elements (TEs found in all analyses organisms), lineage-specific ones, and also some that are shared between between two or more animals. In short words, I would like to generate a pan-repeatome for my clade of interest. I think based on the description of your tool that it would be idea for this kinda of analysis. Then I would use this to reannotate the genomes.
I would like to know if I it is possible to see the output of the software to check if this would fit my goals before investing a lot of time into running the software. Since this is virtually a all-against-all comparison and the .out RM files are quite massive, I am assuming this will take a fair amount of time. Additionally, I have a question about the parameter tree used by the software. Is this tree used as a backbone to infer presence/absence in the species I am analysing? Running with and without it changes much the output of running time?
Best regards and thanks again for the scripts,
André

Hi,
For the easy answer: the tree is only used to order the columns in the output, which is a user convenience thing I remember adding so it was easier to look at the results, but won't affect the results themselves.
Regarding an output, unfortunately, this is old enough that I do not have outputs on hand. I would really recommend creating test files and running the code with them (will also allow you to see if everything runs right). For example I would use 100 lines from the repeat masker .out file you have, and provide 2 genomes (the same one as the RM .out file and another one), or run it with -filter set to one repeat only, to limit the number of repeats that are blasted. Nothing really replaces looking at the data on your own for a small sample size, with data you know, before running the whole thing!
The code will take copy+flanking sequences when the flanking are not a TE in the RM.out file (this is the first caveat: this requires well annotated genomes as input), then blast that.
The output will have columns for each provided genomes, with counts of extracted copies, then the number of total hits, and then hits with the flanking sequences present (when the flanking sequences are present, and you know the input was not nested in some repeats, then this is in favor of this copy being orthologous between the genomes). Note that the results are rarely clear cut; you'd expect false negatives and false positives.
The code is blast based. It uses blast defaults, and the cutoff evalue is set by the user with -e. Lowering it will speed things up but it depends what divergence you expect between your most diverged genomes.

Hello @4ureliek,
thanks for the swift answer. I will follow your instructions and try to narrow down the analyses to a couple of TE families which interest me the most. I am afraid computing all elements will be extremely time-consuming.
A short additional question. Is there any available scripts designed to plot the results originated by parseRM (including the repeat landscape). I have written a few Rscripts to parse the divsum files generated by calcDivergenceFromAlignment.pl and I guess I can adapt those for this purpose.
Cheers,
André

Yes you are correct, it will likely take a while even with parallelization, since blast is just not that fast, and the code likely not extremely optimized (e.g. if blast jobs are too large, splitting query and DBs into chunks speeds things up massively).
I have not written code to plot automatically the repeat landscapes as far as I can remember - I would need to go dig into my archives, but for these landscapes I made the paper figures with Illustrator, not R (https://nyaspubs.onlinelibrary.wiley.com/doi/abs/10.1111/nyas.13295)

I will take the "splitting" into consideration. Let's see what will come from my analyses, but at any case, thanks again for your help and scripts.

Oh, I meant inside the code - splitting the inputs will be annoying to do, unless it's with using the filter option. Let me know how it goes!

Hey @adeoliveira86 - I found an example of output; it was ran with cat, dog and human.
Columns:
#"Total_hits" means total of hits with evalue less than the one set through -e
#"ortho_hits" means POTENTIALLY orthologous elements = hits that contain at least 20% of the extracted flanking sequence
#Acess to individual copies through files in ./orthology.Cat.Dog.Cow.Human/DATA/*.extract/*.out.ortho.tab

For example:
| [Query_species] |   | Target_species#(with tree info): |   |   |   |   |
-- | -- | -- | -- | -- | -- | -- | -- | --
  |   |   | (hg38 | , | ((canFam3 | , | felCat5) | ,
  |   |   | hg38 | hg38 | canFam3 | canFam3 | felCat5 | felCat5
Rname | [name] | [Nb_extracted_seq] | Total_hits | Ortho_hits | Total_hits | Ortho_hits | Total_hits | Ortho_hits
Tigger1#DNA/TcMar-Tigger | canFam3 | 2607 | 3173 | 0 | 3348 | 2173 | 1649 | 35
In this case you can see that most of the dog copies of Tigger1#DNA/TcMar-Tigger are recovered as potentially orthologous (it is never 100% because of how blast will work); human has none, and cat has so few they are probably false positives.
I used this to check whether TEs looked like dog/cat ones or more ancient than that. You'd have to decide on your cutoffs depending on the 'recovery rate' and how some TEs you know are shared behave (depends on quality of assemblies, divergence between species, etc).

Hello @4ureliek,
thanks for sharing the output. This looks like something that could be extremely useful for the questions I am asking. Do you mind if I get in touch with you via email to discuss a little bit more what I am envisaging and have your feedback?

Sure thing - you can my gmail.com email, with prefix 4urelie.k


Pressing 'v' drops you into the editor mode

Pressing 'v' drops you into the editor mode.
Steps to reproduce:
Install ptpython and latest prompt_toolkit (1.0.10).
Launch ptpython and type the letter 'v'.
It drops you into the nano editor instead of inserting the letter v.
Platform:
OS X, iTerm2

Hi @amjith,
Thank you for pointing out this issue! See also https://github.com/jonathanslenders/ptpython/issues/170
I fixed it and pushed a new prompt_toolkit release.
Can you try to upgrade prompt_toolkit to the latest version, and see whether it works again?
Thank you,
Jonathan

Thanks! Fixed in 1.0.11.


epub is missing images ?

Just tried this sample and the epub file seems to have broken images. Tried it in two different reader. PDF output seems good.

Related to https://github.com/asciidoctor/asciidoctor-epub3/issues/22

@rabidgremlin thank you for noticing.
I just fixed the issue by moving book.adoc to the root of the project since asciidoctor-epub3 is not able to handle images that live outsude the source directory (where book.adoc previously was)


Consider removing all from package declaration

The __all__ keyword serves the following purpose:

if a package’s __init__.py code defines a list named __all__, it is taken to be the list of module names that should be imported when from package import * is encountered

However the two __all__ clauses in bruker2nifti explicitly declare all the modules currently. As such, they seem redundant to me.

Uhm... I was following this explanation, and it seems the all should be written manually.
Also the numpy.init.py seems to add it
Why did you say there are two of them? One is defined implicitly by default?

re "two instances":

https://github.com/SebastianoF/bruker2nifti/blob/master/bruker2nifti/__init__.py#L4
https://github.com/SebastianoF/bruker2nifti/blob/master/bruker2nifti/parsers/__init__.py#L1

re "this explanation": it doesn't say anything different from the tutorial on the Python website (the link in my post) -- if the two were different, I'd follow the Python docs as they are the official ones, and also up-to-date (the other one is from 2012).
In any case, __all__ makes a difference only when someone does an import of the form from bruker2nifti import *: only those modules listed explicitly under the __all__ clause get imported then.


MissingPluginException for plugins.flutter.io/firebase_livestream_ml_vision0 (Only iOS)

Hi,
I tried to use this example - but every time I receive this exception
The following MissingPluginException was thrown while activating platform stream on channel
plugins.flutter.io/firebase_livestream_ml_vision0:
MissingPluginException(No implementation found for method listen on channel
plugins.flutter.io/firebase_livestream_ml_vision0)
flutter:
When the exception was thrown, this was the stack:
#0      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:314:7)
<asynchronous suspension>
#1      EventChannel.receiveBroadcastStream.<anonymous closure> (package:flutter/src/services/platform_channel.dart:504:29)
<asynchronous suspension>
#6      FirebaseVision.initialize (package:firebase_livestream_ml_vision/src/firebase_vision.dart:235:10)
<asynchronous suspension>
#7      _MyHomePageState._initializeCamera (package:example/src/example.dart:25:13)
<asynchronous suspension>
#8      _MyHomePageState.initState (package:example/src/example.dart:19:5)

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, v1.9.1+hotfix.2, on Mac OS X 10.14.6 18G87, locale pl-PL)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 10.3)
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.35.0)

Yes, I have seen this issue - in fact I'm facing it too; I'm working on a fix but I'm not getting anywhere with it yet. You're welcome to open a PR if you can figure out a fix.

was this ever resolved


Profile picture displays incorrectly on Firefox

On Firefox, the avatar on a profile page displays full sized underneath the banner, rather than scaled down on top of it. Here's my profile on Firefox (left) and Chromium (right).

This is almost definitely because cqw units are still experimental on Firefox - enabling layout.css.container-queries.enabled in about:config fixes this issue. I'm not sure what the workaround for this would be.

Thank you for reporting this issue.
But not sure why the cqw isn't working – it should be supported by the latest Firefox without the flag according to caiuse. Is your Firefox up to date?

The version caniuse is testing on is still in beta from what I can tell. This shouldn't be much of an issue then, because it will be fixed on stable in a few weeks.


KSPROPERTY_TYPE_SET return value is set to succeed

I'm talking about this line. Why is it there? If I try to use a camera with more than one app at the same time, KSSTATE_ACQUIRE fails for the second app (as expected), but because of that line the app thinks that it succeeded and it gets confused.

It's one of those parts that are not really tested very well. It would be appreciated if you can submit a PR for a fix?

Here you go:
https://github.com/flowerinthenight/windows-camera-class-filter-driver/pull/3
I just removed that line.

Thanks very much. I just merged your PRs.

Thanks!


implement a drop-down/right-click menu to copy location

We should just have a drop-down menu (right-click) on map that makes it possible to copy the clicked location witht these formats:
lat,lon: 45.5,-73.6
lat/lon: 45.5/-73.6
lat lon: 45.5 -73.6
lon,lat: -73.6,45.5
lon lat: -73.6 45.5
which would work in almost all scenarios (geojson lon,lat, usual google lat,lon, openstreetmap url lat/lon, and poly with spaces instead of commas)

See the drop-down menu in QGIS with the lat-lon plugin:

@kaligrafy j'imagine que pour ça il faudrait attendre la migration à deck.gl?

oui c'est mieux!


esp32: More JTAG docs

Clean up & rework JTAG documentation for ESP-32 boards to provide full
commands and clarify gotchas.
Signed-off-by: Andy Ross<EMAIL_ADDRESS>

Think what I wrote makes sense for WROVER, but note I basically faked it as I don't have one yet.

merged on the command line by mistake, but was already fixed with feedback, closing


Add in basic cloudtrail mappings

Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.

[ ] Take a minute to review the code of conduct.
[ ] Make sure you are making a pull request against the provider branch (left side).
[ ] Check your code additions will fail neither code linting checks nor unit test.

Terms and conditions for use, reproduction and distribution
🚨 Apache License.
Description
Please describe your pull request.
💔Thank you from the Automated Cloud Governance Working Group at ONUG!

I got it done!


libreoffice-still: <IP_ADDRESS> -> <IP_ADDRESS>

Description of changes
Update to supported libreoffice release
Changelog
fixes #171907
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review run on x86_64-linux 1

11 packages built:

jpsxdec
libreoffice (libreoffice-still)
libreoffice-still-unwrapped
openlp
openlpFull
paperwork
python310Packages.paperwork-backend
python310Packages.paperwork-shell
python39Packages.paperwork-backend
python39Packages.paperwork-shell
unoconv

the provided script was used to update download.nix if that helps


Fixed identifier highlighting

I'm new to writing tree-sitter parsers, so please correct me if this is an incorrect change.
All identifiers were being tagged as @variable.builtin. Looking at some of the documentation provided by nvim, @variable.builtin should be reserved for variables provided by the language (so this. in dart)
Changes

All (identifier)s are now tagged with @variable instead of @variable.builtin
(this) is now tagged as @variable.builtin
added a basic CI file so tests are run on new changes, happy to revert this if you'd prefer not having it in the repo
re-ran tree-sitter generate and tree-sitter build-wasm with the tree-sitter cli v0.20.9. This resulted in changes on master, leading me to think these were outdated from previous PRs

[!NOTE]
Running tree-sitter generate on v0.20.9, now results in the following output, these conflicts can probably be cleaned up in a later PR

Looks good!


Update hdinsight-high-availability.md

Fixed a typo.

Hi @ShawnJackson, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (v-shawja). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Perf: Individual Learner dashboard issues

Tested with the 300M row dataset on ClickHouse Cloud I've found the following issues:
OOM:

Problem Results
Watched Video Segments
Partial and full views per video
Partial and full views per video (table)

Slow Queries (> 3 seconds):

Learner Summary (100 rows, 23 seconds)
Problems attempted per section/subsection (43 rows, 5 seconds)
Problems attempted per section (9 rows, 4 seconds)
Problems attempted per subsection (34 rows, 4 seconds)
Video views per section/subsection (24 rows, 4 seconds)
Video views per section (7 rows, 4 seconds)
Page views per section (7 rows, 4 seconds)
Video views per subsection (17 rows, 4 seconds)
Page views per subsection (30 rows, 3 seconds)
Page views per section/subsection (37 rows, 3 seconds)

Fixes for everything but problem interactions and problem results are here https://github.com/openedx/tutor-contrib-aspects/pull/812

@bmtcril - I see the linked issue was merged. Does this mean we should close/mark this as done? Or is any of this work still being done.

I think we can close these, we need to do another round of testing now that we have finalized the dashboards and can create new issues if need be.


CMD+Click support or go to correct key with go to declaration

Is your feature request related to a problem? Please describe.
This is pure feature, there no a bug. I really love the typesafe with this library given, but seems i miss the DX that find the correct key automatically when i click cmd+left_click.
Currently the response from my IDE (webstorm) is says: "Cannot find declaration to go to"
Describe the solution you'd like
I don't know how to make the solution
Describe alternatives you've considered
Currently i didn't have any alternative that working on, so i need to find it by myself.

You might want to check i18n Ally for JetBrains (see also the next-intl VSCode integration).


Initialize LSP documents earlier.

Prior to this we would start tracking LSP documents when an ITextView was associated with our ITextBuffer. The issue with this is when the ILanguageClient infrastructure initializes itself before we do and starts our LanguageServer it's we start retrieving requests that require access to our LSPDocumentManager. That's an issue because our LSPDocumentManager may not have been initialized yet resulting in failure to fulfill requests.

Changed how we initialize ITextBuffers. Before we were setting up all of the logic to change the content type of a text buffer and populate its properties at the EditorFactory layer; however, how we were doing it (waiting for the ITextBuffer to load) resulted in ITextDocumentListener events firing prior to our content type changes would occur. This is a problem becasue the ILanguageaClient infrastructure will start making its decision to turn on LSP features for your ITextBuffer at the point and time. Now we change the ITextBuffers content type (and set properties) during the ITextDocumentListener created pipeline.
This resulted in almost all of our logic in our editor factory to be split out into two classes.

LSPEditorFeatureDetector, we needed to be able to detect the "enabledness" of features in two locations. Once at the EditorFactory layer to ensure we know when to disallow other editor factories participation in the ITextBuffer creation and a second at the ITextDocumentListener layer when we inspect an ITextDocument and need to determine if we want to "initialize" it (change its content type etc.).
RazorLSPTextDocumentCreatedListener, this is our ITextDocumentListener i've been referring to. It now houses the logic on how to change the content type and populate the ITextBuffer's properties.

Changed our LSPDocumentManager to no longer depend on ITextViews for ref counting documents. Instead we just take an ITextBuffer and keep an internal count of how many times we've been asked to track a document. In practice this number never goes past 1 however it doesn't hurt to be defensive.

Added IVT from CodeAnalysis.Razor to our LanguageServerClient project in order to enable the retrieval of our FilePathComparison type.

Given the new refactoring of our feature detector and the ITextDocumentListener pieces I was able to add extensive testing to ensure all things work as expected.

Fixes dotnet/aspnetcore#19160
/cc @ToddGrun @alexgav @jimmylewis In this PR I changed how we "change" the content type of our buffers. It actually happens much earlier than IVsTextBufferDataEvents. Food for thought 😄

Oh and @ToddGrun / @jimmylewis to follow up on our convo from another PR. [Shared] isn't the default: https://github.com/dotnet/aspnetcore-tooling/pull/1642/commits/542ac7353851ce757e502bae5fddd1c83c0958fa
Had to re-add it to all of our types 😆


Fix SSL_get_shared_ciphers() (1.1.0)

The function SSL_get_shared_ciphers() is supposed to return ciphers shared
by the client and the server. However it only ever returned the client
ciphers.
Fixes #5317
This is the 1.1.0 version of #6113

Pushed. Thanks.


Allow negative numbers for cantor pairing

Cantor pairing only works for natural numbers. TileMap allows cells to have negative indices. This can actually break the existing logic. In the cantor pairing function we need to translate negative numbers to natural numbers. I chose to use similar overflow behavior you would find when wrapping an unsigned integer. When a number comes in that is negative like -1 it gets translated to<PHONE_NUMBER>412324865 and decreases from there.
I had to switch around order of operations in the math to divide earlier and prevent integer overflow. I spent a while debugging an issue related to this so hopefully this saves someone some effort.

Looks great! Good catch.


Support for ESP8266

i would love to see support for the ESP8266 wifi controller which i am really excited about because it really packs a punch for $2.5.
The toolchain can be obtained via https://github.com/pfalcon/esp-open-sdk
Thanks for this great project

Dear @makefu,
Thank a lot for your request. We have already received the requests from the users about ESP8266. Now I understand, that we should add this development platform to PlatformIO list.
A few questions:

Do you any board based on this chip?
Which operation system do you use?
Are you interested in help us with building toolchain for multiple host targets? (Mac OS X, Windows, Linux 32/64, Linux ARM). It should look quite easy according your link above (make bla bla bla).
How does look "uploading" process for this chip SoC?

Hi Ivan,
Regarding you questions

Yes, i do have a dozen or so of these chips laying around. Due to the fact that they can be sourced extremely cheaply from china (aliexpress: http://www.aliexpress.com/wholesale?catId=0&initiative_id=SB_20150312025300&SearchText=esp8266 ebay:http://www.ebay.com/sch/i.html?_nkw=esp8266 ) it should not be a problem to obtain one. If you wish i can donate a couple for the controllers and send them to a address of your choice :)
I am currenly developing on linux and am willing to support this project with my knowledge if i can
Unfortunately linux is my only weapon of choice
The uploading is done by booting the uC in firmware upload mode (pull a pin to GND) and using https://github.com/themadinventor/esptool to upload the firmware

Thanks. If we add the support for this chip, then I will ask you to test it :smile:
How do you develop this chip on Linux now? Which type of toolchain do you use? Where is located SDK? Can you share here/or in separate Gist some "Hello World" project?
Which type of Linux distributive?
Thanks for information. Looks good.

yay!

under linux/cygwin you will need https://github.com/pfalcon/esp-open-sdk which is a consolidation of everything you need (Xtensa lx106 architecture toolchain built via crosstool-ng and ESP8266 IoT SDK from Espressif Systems)

the detailed description of what you need to build the toolchain: https://github.com/esp8266/esp8266-wiki/wiki/Toolchain
a fast-forward method is to use https://github.com/pfalcon/esp-open-sdk which does all the heavy lifting
after that you can use the lx106 compiler with the SDK like this:

$ xtensa-lx106-elf-gcc -I$(THISDIR)/sdk/include -L$(THISDIR)/sdk/lib

this is a hello world with a makefile: https://github.com/leon-anavi/esp-hello-world
this is a transparent serial bridge for the esp to connect to other uCs https://github.com/beckdac/ESP8266-transparent-bridge
both are using the sdk and the xtensa compiler

i am running archlinux but i have access to ubuntu and debian unstable for testing stuff
!

just fyi, there hackaday published an article about the esp8266 which is very detailed and gives some background about the ESP8266: http://hackaday.com/2015/03/18/how-to-directly-program-an-inexpensive-esp8266-wifi-module/

Additional article: http://hackaday.com/2015/03/18/how-to-directly-program-an-inexpensive-esp8266-wifi-module/

Just wanted to chime in that I've been working with the ESP8266 using esptool.py and a docker buildbox and it would be really nice to have support for this instead of swapping back and forth as I'm currently doing.
If there is any way I could help I'd be glad to know! Thanks!

@chipolux Sure! Please repost/retweet our vote https://twitter.com/platformio_org/status/583353126494822402

Dear @makefu, @chipolux,
You can try the latest develop version of PlatformIO, where have been released the initial support of espressif platform. See here how to install development version of PlatformIO.
We know about a few issues with invalid firmware and we are working on it. Do you have any problems with your ESP8266 based projects?
See examples https://github.com/platformio/platformio/tree/develop/examples/espressif
P.S: Thanks a lot @valeros for the great job on this issue! :+1: :beers:

Awesome @ivankravets!
I'll take a look at it later today with a few projects on OSX and Linux!

Alright, so it pulls down and starts up okay but there's a problem with linking:
xtensa-lx106-elf-g++ -o .pioenvs/esp8266/firmware.elf -nostdlib -Wl,--no-check-section -u call_user_start -Wl,-static -T /Users/nakyle/.platformio/packages/ldscripts/esp8266.ld .pioenvs/esp8266/src/piomain.o -L/Users/nakyle/.platformio/packages/framework-arduinoespressif/sdk/lib -L.pioenvs/esp8266 -Wl,--start-group -lpp -lmain -lwpa -llwip -lnet80211 -lphy -lhal .pioenvs/esp8266/libFrameworkArduino.a .pioenvs/esp8266/libFrameworkArduinoVariant.a -lgcc -lc -Wl,--end-group
/Users/nakyle/.platformio/packages/toolchain-xtensa/bin/../lib/gcc/xtensa-lx106-elf/4.8.2/../../../../xtensa-lx106-elf/bin/ld: cannot open linker script file /Users/nakyle/.platformio/packages/ldscripts/esp8266.ld: No such file or directory
collect2: error: ld returned 1 exit status
scons: *** [.pioenvs/esp8266/firmware.elf] Error 1

I've completely wiped out my ~/.platformio directory and reinstalled platformio and scons but still no go.

Oh, I suppose you guys might want the platformio.ini I'm working with:
[env:esp8266]
platform = espressif
framework = arduino
board = esp01

Otherwise though, using the native libraries it seems to compile all my little projects and push them to the chip just fine.
Is there a reason you guys went with esptool over esptool.py? I've not ran into any issues with esptool.py so far and it has friendlier output when flashing the chip.

Otherwise though, using the native libraries it seems to compile all my little projects and push them to the chip just fine.

Great! :+1:  Could I ask you to try this example: https://github.com/platformio/platformio/tree/develop/examples/espressif/esp8266-webserver ?

Is there a reason you guys went with esptool over esptool.py? I've not ran into any issues with esptool.py so far and it has friendlier output when flashing the chip.

I agree with you.
@valeros Could I ask you to switch to https://github.com/themadinventor/esptool ? This is Python based utility and should work under all host OS.

@ivankravets tried with that example and still got the same error, esp266.ld doesn't exist.

@chipolux give me please an output of these commands:
$ platformio --version
$ platformio update

Thanks!

Here ya go!
nakyle at lnakyle in ~/Projects/embedded/esp8266/platformio/webserver (embedded)
$ platformio --version
PlatformIO, version 1.4.0.dev0

nakyle at lnakyle in ~/Projects/embedded/esp8266/platformio/webserver (embedded)
$ platformio update

Platform espressif
--------
Updating sdk-esp8266 package:
Versions: Current=1, Latest=1    [Up-to-date]
Updating tool-esptool package:
Versions: Current=1, Latest=1    [Up-to-date]
Updating framework-arduinoespressif package:
Versions: Current=1, Latest=1    [Up-to-date]
Updating toolchain-xtensa package:
Versions: Current=1, Latest=1    [Up-to-date]

@chipolux Thanks, I got it and I've just fixed. Please re-install develop version of PlatformIO. Do you have any problems?

Hi, i tried the example code at https://github.com/platformio/platformio/tree/develop/examples/espressif/esp8266-webserver and it seems to building a nice little firmware:
saved binimage file, total size is 42304 bytes, checksum byte is 0x5E
using ELF file ".pioenvs/esp01_8266/firmware.elf"
saved section ".irom0.text" to file ".pioenvs/esp01_8266/firmware_40000.bin"
"xtensa-lx106-elf-size" -B -d .pioenvs/esp01_8266/firmware.elf
text	   data	    bss	    dec	    hex	filename
192653	   4684	  36336	 233673	  390c9	.pioenvs/esp01_8266/firmware.elf
====================================================================== [SUCCESS] Took 3.62 seconds ======================================================================

Thanks for your efforts, really!

@makefu Could I ask to upload firmware to device? We have problem with invalid firmware. Like, "doesn't work properly". We are working on it.

Awesome, looks like the new version works! I'll don't have a chip handy right now but in a few hours I'll make sure it can push this and another project or two!
using ELF file ".pioenvs/esp01_8266/firmware.elf"
saved section ".irom0.text" to file ".pioenvs/esp01_8266/firmware_40000.bin"
"xtensa-lx106-elf-size" -B -d .pioenvs/esp01_8266/firmware.elf
text       data     bss     dec     hex filename
192653     4716   36336  233705   390e9 .pioenvs/esp01_8266/firmware.elf
====================== [SUCCESS] Took 106.00 seconds =======================

Webserver flashed perfectly. Trying my own couple of arduino framework projects now.
nakyle at lnakyle in ~/Projects/embedded/esp8266/platformio/webserver (embedded)
$ platformio run --target upload
[Mon Apr  6 17:03:48 2015] Processing esp01_8266 (platform: espressif, board: esp01, framework: arduino)
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
BeforeUpload(["upload"], [".pioenvs/esp01_8266/firmware_00000.bin", ".pioenvs/esp01_8266/firmware_40000.bin"])
Auto-detected UPLOAD_PORT: /dev/cu.SLAB_USBtoUART
/Users/nakyle/.platformio/packages/tool-esptool/esptool -vv -cd none -cb 115200 -cp /dev/cu.SLAB_USBtoUART -ca 0x00000 -cf .pioenvs/esp01_8266/firmware_00000.bin -ca 0x40000 -cf .pioenvs/esp01_8266/firmware_40000.bin
esptool v0.4.2 - (c) 2014 Ch. Klippel<EMAIL_ADDRESS>setting board to none
setting baudrate from 115200 to 115200
setting port from /dev/tty.usbserial to /dev/cu.SLAB_USBtoUART
setting address from 0x00000000 to 0x00000000
espcomm_upload_file
stat .pioenvs/esp01_8266/firmware_00000.bin success
opening port /dev/cu.SLAB_USBtoUART at 115200
tcgetattr
tcsetattr
serial open
opening bootloader
resetting board
trying to connect
setting character timeout 0
done
setting character timeout 1
done
espcomm_cmd: sending command header
espcomm_cmd: sending command payload
trying to connect
setting character timeout 0
done
setting character timeout 1
done
espcomm_cmd: sending command header
espcomm_cmd: sending command payload
espcomm cmd: receiving 2 bytes of data
setting character timeout 0
done
setting character timeout 1
done
espcomm_open
Uploading 42336 bytes from .pioenvs/esp01_8266/firmware_00000.bin to flash at 0x00000000
erasing flash
espcomm_cmd: sending command header
espcomm_cmd: sending command payload
setting timeout 5000
setting character timeout 50
done
setting timeout 1
setting character timeout 1
done
espcomm cmd: receiving 2 bytes of data
writing flash
..........................................
setting address from 0x00000000 to 0x00040000
espcomm_upload_file
stat .pioenvs/esp01_8266/firmware_40000.bin success
espcomm_open
Uploading 155080 bytes from .pioenvs/esp01_8266/firmware_40000.bin to flash at 0x00040000
erasing flash
espcomm_cmd: sending command header
espcomm_cmd: sending command payload
setting timeout 5000
setting character timeout 50
done
setting timeout 1
setting character timeout 1
done
espcomm cmd: receiving 2 bytes of data
writing flash
........................................................................................................................................................
starting app without reboot
espcomm_cmd: sending command header
espcomm_cmd: sending command payload
espcomm cmd: receiving 2 bytes of data
closing bootloader
=========================================================================== [SUCCESS] Took 21.76 seconds ===========================================================================

@chipolux, Does device response on GET request? Could I ask you to check it using browser?

Sure, I've flashed the webserver example back to the chip but I'm just getting junk from the serial port. I've tried adjusting the baud rate with platformio serialports monitor as well as adjusting it in the example but I've not got anything to work there.
Any ideas?

Weird, just testing with this bit of code works, but serialports monitor crashes with a device not configured error after the first line comes through. I can still cat /dev/cu.SLAB_USBtoUART and watch the messages flow though.
const int led = 2;

void setup(void)
{
  Serial.begin(9600);
  pinMode(led, OUTPUT);
  digitalWrite(led, LOW);
}

void loop(void)
{
    Serial.println("Just a test.");
    digitalWrite(led, HIGH);
    delay(500);
    digitalWrite(led, LOW);
    delay(500);
}

Well I isolated an issue with my usb-to-uart adapter that was causing some issues.
But I also figured out that something about ESP8266WebServer causes a fatal exception on the chip.
Fatal exception (0):
epc1=0xion (0):
epc1=0x40108ca4, epc2=0x00000000, epc3=0x00000000, excvaddr=0x00000000, depc=0x00000000
Fatal exception (0):
epc1=0xion (0):
epc1=0x40108ca4, epc2=0x00000000, epc3=0x00000000, excvaddr=0x00000000, depc=0x00000000

Looks like it must be a problem with the firmware the toolchain is generating as I've tried with the arduino ide and it's working from there.

Friends, please upgrade PlatformIO to the latest develop and update all packages using > platformio update.
Please check your version:
> platformio --version

Should be equal to 1.4.0.dev2.
Looks like we fixed the bug with "Fatal exception".
P.S: @gandy92 please re-test with your board. Thanks!

@ivankravets Everything seems to be working! I've tested out the wifi scan and the wifi server examples and both worked flawlessly!

@chipolux Thanks for report! We will replace "esptool" with "esptool.py" and I'm going to release 1.4.0 this tomorrow.

@ivankravets @valeros Awesome! You guys rock! This will make my workflow a million times easier!

@chipolux Thanks! :blush:
Are you interested in native ESP8266 programming using SDK (without Arduino framework)?

@ivankravets Sorry, didn't yet get my programming gear for the ESP8266 to work properly, not a problem with PlatformIO, though..

@gandy92  try connect/disconnect. We also have problems with this chip time by time.

@ivankravets I've played with the native SDK a bit when I first picked up the chips but I haven't had a need to use it instead of the Arduino libs yet..
Most people doing stuff with the chip are either using NodeMCU or the Arduino libs as far as I can tell so I doubt many people will really need the native support but I imagine it'd be good to have just in case.

@ivankravets are you working on integrating platform.io support within the Arduino IDE environment? That would be very useful.

@electronicsguy see http://docs.platformio.org/en/latest/ide/arduino.html

I'm working on bulding a truly cross-platform toolchain, since esp-open-sdk is not building well on windows: github.com/kireev.co/esp-alt-sdk.

@kireevco does https://github.com/esp8266/Arduino uses your toolchain?

@ivankravets no, not yet, I'm working with them on that matter.
https://github.com/sminghub/sming does work well already. By the way, it would be very nice to add sming to platform.io.

@kireevco please open a separate issue for switching to https://github.com/kireevco/esp-alt-sdk when you will agreed it with https://github.com/esp8266/Arduino. We currently switched to their toolchain for compatibility.
As for Sming - https://github.com/platformio/platformio/issues/398. Thanks!


Not compatible with Python 3.7?

Linux, CPython 3.7.3.
By just importing from iota import Iota I get:
Traceback (most recent call last):
  File "/home/path/main.py", line 1, in <module>
    from iota import Iota
  File "/home/path/venv/lib/python3.7/site-packages/iota/__init__.py", line 38, in <module>
    from .api import *
  File "/home/path/venv/lib/python3.7/site-packages/iota/api.py", line 12, in <module>
    from iota.commands import BaseCommand, CustomCommand, core, \
  File "/home/path/venv/lib/python3.7/site-packages/iota/commands/__init__.py", line 13, in <module>
    import filters as f
  File "/home/path/venv/lib/python3.7/site-packages/filters/__init__.py", line 27, in <module>
    from filters.extensions import FilterExtensionRegistry
  File "/home/path/venv/lib/python3.7/site-packages/filters/extensions.py", line 11, in <module>
    from class_registry import EntryPointClassRegistry
  File "/home/path/venv/lib/python3.7/site-packages/class_registry/__init__.py", line 5, in <module>
    from .registry import *
  File "/home/path/venv/lib/python3.7/site-packages/class_registry/registry.py", line 33, in <module>
    class BaseRegistry(with_metaclass(ABCMeta, Mapping)):
  File "/home/path/venv/lib/python3.7/site-packages/six.py", line 827, in __new__
    return meta(name, bases, d)
  File "/usr/lib/python3.7/abc.py", line 126, in __new__
    cls = super().__new__(mcls, name, bases, namespace, **kwargs)
TypeError: type() doesn't support MRO entry resolution; use types.new_class()

Process finished with exit code 1

Hi @Nitrooo thanks for reporting this!  Aye, the library needs a few tweaks for compatibility with Python 3.7.
Marking this issue as help wanted; the fix should be fairly simple; very similar to this one.

HI,
I do have the same issue in my side, import iota not working in my side due to :
TypeError: type() doesn't support MRO entry resolution; use types.new_class()
Will this issue be fixed soon, and when will it be published on Pypi? Otherwise, how can i resolve the issue within iota lib?
Thanks a lof in advance

Curious if any progress has been made on this?  I'm hoping to use pyota with Python 3.7.  Is it still not compatible?

The develop branch now supports Python 3.7.  @todofixthis would it be possible to release that so we can close this issue?


data missing in the report

Hi
I was trying to generate a classification dashboard. The "reference" dataframe has 128770 rows, however, the dashboard only shows that it has 122 rows (number of objects). And for "current" dataframe, it has 632337 rows, but the dashboard only shows it has 1079 rows. I am not sure what went wrong.
Yang

Hi @superyang713
Looks very interesting. My current guess is that it can be related to the nan values.
Could you please check the amount of the rows which have a least one nan value in? I assume that the amount of skipped rows will be equal to the amount of the rows with nans, but lets check first.
The following commands can be handy:
#column-based nans statistics
data.isna().sum(axis=0)

#row-based nans statistics
data.isna().sum(axis=1)

#total amount of the rows with nans
raw_data.isna().sum(axis=1).sum()

Thank you for the reply. You are right, the dataset has missing values.  Sorry I just started using the package, and did not know that I have to fill out the missing values before generating the dashboard.
My trained model is actually a sklearn pipeline that has an imputer  and other transformers in it.  So I only need to feed the raw data into the model. But for generating the Dashboard, I have to use fill out the missing values first then. Thank you for the response.

Hi @superyang713 ,
No worries it all! I wanted to thank you for bringing this up! To be honest, it is not exactly how we plan to handle missing values (currently all the rows with the missing values are just ignored). We are going to offer some more nan handling strategies there.

copy

Okay thanks @emeli-dral  I will check with new release and validate it. Thanks for the response.

Okay thanks @emeli-dral I will check with new release and validate it. Thanks for the response.


Setting password without escaping

For me on Debian testing setting the password only worked without the escaping of the single quotes.

I think this is a copy & paste error from a Python script.
Thanks.


Update smithy4s-aws-http4s, ... to 0.17.15

About this PR
📦 Updates

com.disneystreaming.smithy4s:smithy4s-aws-http4s
com.disneystreaming.smithy4s:smithy4s-sbt-codegen

from 0.17.13 to 0.17.15
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.disneystreaming.smithy4s" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.disneystreaming.smithy4s" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, version-scheme:early-semver, commit-count:1

Superseded by #63.


722 derive params exposure deprecation fix

Attempted updates for @thomas-neitmann's spots at #760

So I finally managed to make the "Check Templates" workflow working and sure enough it errored because some updates we've made where not incorporated into the template scripts. This is not specific to this PR, though. I'll adjust the templates in a separate PR.


[BUG] - Incorrect change output computation inside makeTransactionBodyAutoBalance

External
Area
Tx balancing.
Summary
makeTransactionBodyAutoBalance tries to compute change output by considering only inputs, minted value & outputs here but ada source could also come from stake/drep credential unregistration as well. This can cause the makeUnsignedTx function called shortly afterwards to fail as this computed change could be negative. This was not the case for cardano-api-<IP_ADDRESS> where such a conversion was not being performed at least before the ada value in change output was set to maximum word limit.
Also, the utxo parameter expected by makeTransactionBodyAutoBalance is suppose to include reference inputs, collateral inputs as well and not just spending inputs but totalValueAtSpendableUTxO seems to suggest that it's only spending inputs. This is not a big deal if our goal is just to overestimate but thought I'd also mention this misnomer.
Version
Using tag cardano-api-<IP_ADDRESS>.

Fwiw, I did this change upon cardano-api-<IP_ADDRESS> and it now again works with our tx building tool, Atlas.


test-commit

Description

tests derived fields when defined in index mappings
tests derived fields when defined in query
tests mappings and settings updates for derived fields
tests index and search analyzer applicability on derived fields
tests query string
tests fetch fields and highlight on derived field of type text. Currently highlighter of type unified is supported.
tests ignore_malformed, format, source_indexed_field and properties for explicit type definitions.

The diff will reduce once preceding PRs are merged - https://github.com/opensearch-project/OpenSearch/pull/13592, https://github.com/opensearch-project/OpenSearch/pull/13717 and https://github.com/opensearch-project/OpenSearch/pull/13720
Related Issues
Resolves
https://github.com/opensearch-project/OpenSearch/issues/13715

https://github.com/opensearch-project/OpenSearch/issues/13143
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] API changes companion pull request created.
[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[x] Public documentation issue/PR created

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

hello

hello

{"run-benchmark-test": 1}

{"run-benchmark-test": 1}

{"run-benchmark-test": 1}

{"run-benchmark-test": 1}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_1"}

{"run-benchmark-test": "id_6"}

{"run-benchmark-test": "id_6"}

{"run-benchmark-test": "id_5"}

{"run-benchmark-test": "id_5"}

{"run-benchmark-test": "id_2"}

{"run-benchmark-test": "id_5"}

{"run-benchmark-test": "id_2"}

{"run-benchmark-test": "id_2"}

{"run-benchmark-test": "id_2"}

{"run-benchmark-test": "id_2"}

{"run-benchmark-test": "id_2"}

{"run-benchmark-test": "id_9"}

{"run-benchmark-test": "id_9"}

{"run-benchmark-test": "id_9"}

{"run-benchmark-test": "id_9"}

{"run-benchmark-test": "id_9"}

{"run-benchmark-test": "id_9"}

{"run-benchmark-test": "id_9"}

{"run-benchmark-test": "id_9"}

{"run-benchmark-test": "id_9"}

{"run-benchmark-test": "id_9"}

{"run-benchmark-test": "id_9"}

{"run-benchmark-test": "id_9"}


[WFLY-4825] Add capability for data sources defined on the subsystem.

The first commit just cleans up the add handler for a data source to use the standard abstraction.
The second commit adds the org.wildfly.data-source capability to the subsystem. This currently only uses the capability on the resources added to the subsystem with an alias of the jboss.data-source.$JNDI. The deployment processors that add services still use the jboss.data-source.$JNDI name as the primary service name.
It could make sense to expand the capabilities to add something specific for XA data sources and non-XA data sources with the current capability the standard for them all. If this is needed I can add that to the PR.

Windows Build 1918 is now running using a merge of 0a6e140ae207bc9becdc475838df7c4b0d2eb8c0

Linux Build 6843 is now running using a merge of 0a6e140ae207bc9becdc475838df7c4b0d2eb8c0

Linux with security manager Build 1740 is now running using a merge of 0a6e140ae207bc9becdc475838df7c4b0d2eb8c0

Linux with security manager Build 1740 outcome was SUCCESS using a merge of 0a6e140ae207bc9becdc475838df7c4b0d2eb8c0
Summary: Tests passed: 855, ignored: 321 Build time: 0:05:42

Linux Build 6843 outcome was FAILURE using a merge of 0a6e140ae207bc9becdc475838df7c4b0d2eb8c0
Summary: Tests failed: 2 (2 new), passed: 2886, ignored: 377, muted: 1 Build time: 0:44:24
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.jca.datasource.remove.WebAppDsRemoveTestCase.testDatasourceRemove: <no details avaliable>

org.jboss.as.test.manualmode.security.SecuredDataSourceTestCase.closeServer: <no details avaliable>

Windows Build 1918 outcome was FAILURE using a merge of 0a6e140ae207bc9becdc475838df7c4b0d2eb8c0
Summary: Tests failed: 2 (2 new), passed: 2886, ignored: 377, muted: 1 Build time: 0:44:59
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.jca.datasource.remove.WebAppDsRemoveTestCase.testDatasourceRemove: <no details avaliable>

org.jboss.as.test.manualmode.security.SecuredDataSourceTestCase.closeServer: <no details avaliable>

Windows Build 1933 is now running using a merge of c39bb15f0193d093858f7306b0d746e9cab9f3b7

Linux with security manager Build 1755 is now running using a merge of c39bb15f0193d093858f7306b0d746e9cab9f3b7

Linux Build 6858 is now running using a merge of c39bb15f0193d093858f7306b0d746e9cab9f3b7

Linux with security manager Build 1755 outcome was SUCCESS using a merge of c39bb15f0193d093858f7306b0d746e9cab9f3b7
Summary: Tests passed: 855, ignored: 321 Build time: 0:05:41

Linux Build 6858 outcome was FAILURE using a merge of c39bb15f0193d093858f7306b0d746e9cab9f3b7
Summary: Tests failed: 2, passed: 2922, ignored: 371, muted: 1 Build time: 0:46:19
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.jca.datasource.remove.WebAppDsRemoveTestCase.testDatasourceRemove: <no details avaliable>

org.jboss.as.test.manualmode.security.SecuredDataSourceTestCase.closeServer: <no details avaliable>

Windows Build 1933 outcome was FAILURE using a merge of c39bb15f0193d093858f7306b0d746e9cab9f3b7
Summary: Tests failed: 2, passed: 2922, ignored: 371, muted: 1 Build time: 0:48:14
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.jca.datasource.remove.WebAppDsRemoveTestCase.testDatasourceRemove: <no details avaliable>

org.jboss.as.test.manualmode.security.SecuredDataSourceTestCase.closeServer: <no details avaliable>

Windows Build 1934 is now running using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218

Linux with security manager Build 1756 is now running using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218

Linux Build 6859 is now running using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218

Linux with security manager Build 1756 outcome was SUCCESS using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218
Summary: Tests passed: 855, ignored: 321 Build time: 0:05:35

Linux Build 6859 outcome was FAILURE using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218
Summary: Tests failed: 2 (1 new), passed: 2922, ignored: 371, muted: 1 Build time: 0:47:03
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.timerservice.mgmt.TimerManagementTestCase.testSuspendAndActivate: <no details avaliable>

org.jboss.as.test.integration.jca.datasource.remove.WebAppDsRemoveTestCase.testDatasourceRemove: <no details avaliable>

Windows Build 1934 outcome was FAILURE using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218
Summary: Tests failed: 1, passed: 2923, ignored: 371, muted: 1 Build time: 0:48:08
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.jca.datasource.remove.WebAppDsRemoveTestCase.testDatasourceRemove: <no details avaliable>

retest this please

Windows Build 1956 is now running using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218

Linux Build 6881 is now running using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218

Linux with security manager Build 1778 is now running using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218

Linux with security manager Build 1778 outcome was SUCCESS using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218
Summary: Tests passed: 864, ignored: 311 Build time: 0:06:10

Linux Build 6881 outcome was SUCCESS using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218
Summary: Tests passed: 2928, ignored: 367, muted: 1 Build time: 0:47:12

Windows Build 1956 outcome was SUCCESS using a merge of 4214d0dfc6917bacac089dac05546c04b6f20218
Summary: Tests passed: 2929, ignored: 367 Build time: 0:48:16

Capability documentation PR https://github.com/wildfly/wildfly-capabilities/pull/1

@jesperpedersen and/or @maeste is this okay?

Linux with security manager Build 1893 is now running using a merge of 49645f265e38456ce146db41ffca506e4a090111

Windows Build 2071 is now running using a merge of 49645f265e38456ce146db41ffca506e4a090111

Linux Build 7016 is now running using a merge of 49645f265e38456ce146db41ffca506e4a090111

Linux with security manager Build 1893 outcome was FAILURE using a merge of 49645f265e38456ce146db41ffca506e4a090111
Summary: Tests passed: 532, ignored: 185; exit code 1 (new) Build time: 0:02:32
Build problems:
Process exited with code 1

Linux Build 7016 outcome was FAILURE using a merge of 49645f265e38456ce146db41ffca506e4a090111
Summary: Tests passed: 532, ignored: 185; exit code 1 (new) Build time: 0:02:52
Build problems:
Process exited with code 1

Windows Build 2071 outcome was FAILURE using a merge of 49645f265e38456ce146db41ffca506e4a090111
Summary: Tests passed: 532, ignored: 185; exit code 1 (new) Build time: 0:03:47
Build problems:
Process exited with code 1

Linux with security manager Build 1894 is now running using a merge of f60b3ce948ae47e8a1a3afe2db6627e0deaec772

Linux Build 7017 is now running using a merge of f60b3ce948ae47e8a1a3afe2db6627e0deaec772

Windows Build 2072 is now running using a merge of f60b3ce948ae47e8a1a3afe2db6627e0deaec772

Linux with security manager Build 1894 outcome was SUCCESS using a merge of f60b3ce948ae47e8a1a3afe2db6627e0deaec772
Summary: Tests passed: 871, ignored: 280 Build time: 0:06:21

Windows Build 2072 outcome was SUCCESS using a merge of f60b3ce948ae47e8a1a3afe2db6627e0deaec772
Summary: Tests passed: 2944, ignored: 344 Build time: 0:48:01

Linux Build 7017 outcome was FAILURE using a merge of f60b3ce948ae47e8a1a3afe2db6627e0deaec772
Summary: Tests failed: 1 (1 new), passed: 2941, ignored: 344, muted: 2 Build time: 0:52:10
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ws.wsse.signencrypt.SignEncryptMultipleClientsTestCase.encryptedAndSignedRequestFromJohn: <no details avaliable>

@jamezp LGTM, squash though

oops just noticed the squash comment. In general I think its ok if there are multiple commits as long as

They can pass the test suite in isolation, such that git bisect works
They are discrete well contained units of work
There is a very small number of them
They are related


Audit Express Designs

Rough Visual Audit - break into separate tasks as needed.

[x] ActionButton (API Change) #1219 - progress
[x] Button (API Change) #1228  - progress
[x] #1241
[x] #1256
[x] #1255
[x] #1242
[x] #1243
[x] #1244
[x] #1245
[x] #1246
[x] #1247
[x] #1248
[x] #1249
[x] #1250
[x] #1251
[x] #1252
[x] #1253
[x] #1254

@GarthDB It looks like most of the related issues have been resolved, should I close this out?


Can't import history as google doesn't offer JSON

Trying to import history from youtube requires the data to be in json format, but for some reason that's no longer possible to request from google. They'll only offer watch-history.html and search-history.html in their takeout now.
Not sure if there's a reliable way to convert those into JSON, but none of the online tools I tried worked, either because the files were too big or just too convoluted.

But this makes it currently not possible, for me at least, to input history.

Are you sure about this? Could you maybe upload a clip of how you retrieved the files from Google Takeout?


HSM For Keystore

Use case
Android and iOS both have Hardware Security Module(HSM) that allows developers to securely store their encryption keys.
A pretty good feature would be the same in Flutter to have a more secure application.

Proposal
I'd like Flutter to create a plugin that could give me access to Android and iOS HSM Keystore so as to store important encryption keys.
This would allow all the developers to create a more secure application.
P. S.: I'd work on it myself but my knowledge of Android and iOS isn't good enough to implement it.

actually there is a equivalent of this in flutter you can check it out flutter_secure_storage

@preetshah21699
Please try above plugin and see if it serves your purpose.

@ahmeddigi, it is a good plugin. But I want to use my own cryptographic keys to store the data securely rather than use their mechanism.
Alng with that, I would like to securely store the randomly generated keys not in the database but in the KeyChain for iOS and KeyStore for Android.

I'd like Flutter to create a plugin that could give me access to Android and iOS HSM Keystore so as to store important encryption keys.

I am not sure if a first party plugin is feasible for this or not and more sounds like a community project for the plugin, but labeling it for further insights from team on this proposal.

Thank you @darshankawar I appreciate your consideration.
Please note that this kind of a plugin will help users to create more secure apps by leveraging the HSM module.

I'm not sure I understand the request here; can you elaborate? What native APIs are you looking for a wrapper of, specifically?

But I want to use my own cryptographic keys to store the data securely rather than use their mechanism.

What do you mean by "their mechanism". The OS-provided secure storage?

Alng with that, I would like to securely store the randomly generated keys not in the database but in the KeyChain for iOS and KeyStore for Android.

How is that different from what flutter_secure_storage—which wraps Keychain and KeyStore—is doing?

@stuartmorgan,
"AES encryption is used for Android. AES secret key is encrypted with RSA and RSA key is stored in KeyStore"
The above line is what is used in Flutter Secure Storage. So, it means they're using their own encryption to store the data and just storing their key in the KeyStore. I'd like to store my own keys into the KeyStore rather than have them encrypted and store that encrypted value in an unsecure storage.
I want a wrapper for KeyChain on iOS and KeyStore on Android.

So you specifically want a plugin that stores keys, rather than securely storing arbitrary data?
There's a significant amount of overlap between that and flutter_secure_storage in terms of implementation; have you considered filing that as a feature request for that plugin?

Hi @stuartmorgan. I think that is a possibility. I just didn't think about it as such.
I can open an issue there as well. This was helpful. Thanks.

It would be nice to have this. I'd like to have a way to import the same private key to share it in my app across devices and server side without requiring and compromising it first. The basic use case I can think of is using the RSA algorithm, i.e. how can you properly protect the keys to encrypt and decrypt data?

Based on the updated guidance in https://github.com/flutter/flutter/wiki/Issue-hygiene#closing-issues, I'm going to close this issue, better reflecting the expectation that this is not something that the Flutter team plans to implement.


When pasting code from vs code to OneNote, the formatting will be all messed up and code becomes unreadable

When pasting code from vs code to OneNote, the formatting will be all messed  up and code becomes unreadable

Hi,
I am guessig you are using a dark theme inside VSCode? I just tried a couple of things, using dark mode in VS Code.

Pasting code from VSCode into OneNote Desktop app on Windows.
Pasting code into the OneNote WebApp in a browser on Ubuntu.
Pasting code into my OneNote Snap package on Ubuntu.

Everything works fine on the Windows native app. But on Linux, in both Chrome and my Snap package, a dark theme causes all the text to have a dark highlight and makes some parts unreadable.
This seems to be a Microsoft issue, I unfortunately can't really control how the web app works. I just serve the app inside a container.
I have a suggestion though. If you need a notes app for coding, OneNote is probably not the best tool. You could try a Markdown based notes app. They support better code annotation, generally rely on local storage so switching between notebooks and pages is quicker. And once you get used to markdown, things will just go smoother.
I created this OneNote app a few years ago when I was transitioning from Windows to Linux. I rarely use it anymore, just update it ever so often. I personally use:

Joplin, which is open source. Supports Vim and Emacs key bindings. Uses local storage for notes, but you can set up sync using Dropbox, OneDrive, NextCloud etc. No plugin for drawing like OneNote unfortunately.

Obsidian.md. It is closed source, but free to use and has a lot of plugin options. Has an interesting graph-view thing (which i don't use personally). It also supports Vim bindings. Has a good plugin for drawing (Excelidraw). Sync could be an issue since the integrated sync is paid, but there is a plugin for external sync. Or you can manually set it up using dropbox or NextCloud or something.

They both come with apps for Windows, Mac, Linux, iOS and Android.
Boost Notes used to have a good local app, but I think they ended support for that and have transitioned to a cloud based service.


Use workspace size returned from getSolution() call for convolutions

https://rocmsoftwareplatform.github.io/MIOpen/doc/html/find_and_immediate.html
MIOpen calls should be using workspace  size returned from getSolution()

Removed checks for solution_id == 0.


Introducing new API version 2021-11-01 for Azure Site Recovery service

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

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

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

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

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

Contribution checklist:

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

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

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits.

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

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

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

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

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

    "x-ms-examples": {

Also specify x-ms-long-running-operation-options
Check details here: http://azure.github.io/autorest/extensions/#x-ms-long-running-operation-options

Refers to: specification/recoveryservicessiterecovery/resource-manager/Microsoft.RecoveryServices/stable/2021-11-01/service.json:3927 in 14e7c8c. [](commit_id = 14e7c8c3d2796babdf9fbb2fba918701425a6b75, deletion_comment = False)

@RamyasreeChakka fixed. The other lint failures are known issues.

  "get": {

In addition to GET collection, implement GET on individual resource i,e., implement below API
GET /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.RecoveryServices/vaults/{resourceName}/replicationAppliances/{applicaneName}

Refers to: specification/recoveryservicessiterecovery/resource-manager/Microsoft.RecoveryServices/stable/2021-11-01/service.json:209 in 5dbeb78. [](commit_id = 5dbeb78089d5d75c1924793b01f10d8ec80890ea, deletion_comment = False)

Hi @RamyasreeChakka
As discussed offline, we will implement the Get API in the future version. Tracking workitem for the same: https://msazure.visualstudio.com/One/_workitems/edit/12973033
Please approve if you find everything else fine.
Thanks.

  "get": {

In offline discussion, Vishnu Priya mentioned GET on individual resource would be implemented in next API versions.

In reply to:<PHONE_NUMBER>

Refers to: specification/recoveryservicessiterecovery/resource-manager/Microsoft.RecoveryServices/stable/2021-11-01/service.json:209 in 5dbeb78. [](commit_id = 5dbeb78089d5d75c1924793b01f10d8ec80890ea, deletion_comment = False)

@viananth - signed off from ARM side

@ArcturusZhang @msyyc could you please approve if it looks good?


Watching directory on shared folder events(add, change, ready) not triggered

I am using Chokidar package to watch folders from shared directive having issue but working on local folder. Please help me I am using Windows-10 using below code
The issue is event not triggered if any new files added, updated and deleted.  I am using version - 3.5.1
Thanks in advanced
const resultFolder = `\\\\${IP}\\results`
chokidar.watch(resultFolder, {
                ignored: /(^|[\/\\])\../, // ignore dotfiles
                persistent: true,
		depth: 0,
                ignoreInitial: false,
		ignorePermissionErrors: true,
		awaitWriteFinish: true,
		usePolling: true
            })
          .on('add', path => log(`File ${path} has been added`))
          .on('change', path => log(`File ${path} has been changed`))
          .on('unlink', path => log(`File ${path} has been removed`));

            ```

duplicate of existing issues


Basic: correct dependencies for linking on Windows

This corrects the link dependencies for Windows.  llbuildBasic is always
built as a static library.  This requires that the dependencies be
linked PUBLIC so that users of the library are able to have the
dependencies fulfilled.  llbuild now properly links in ShLwApi (the
dependency was previously silently fulfilled by the implicitly linked
libraries from CMake).

@swift-ci please smoke test


word2vec pre-trained embeddings

It would be nice to have w2v pre-trained embeddings

@leezu added these in #225


[Question] Setting up IntelliSense

Hi! I just have a quick question about workflow. I'm really new to shaders so please excuse me if this is a dumb question :)
I noticed someone else was setting up intellisense in #10 and wanted to try it out as well. I included glm in my workspace and set my file name to be helloworld.glsl.cpp. In my file I have
#ifdef _WIN32
#include <glm/glm.hpp>
using namespace glm;
#endif

This makes intellisense work, but it has some bad side effects like losing the color picker, and the cpp linter complaining that uniform is not a type name. I'm happy with my setup but it feels a tad hacky.
I'm just curious if this kind of setup is normal, or if maybe I'm missing something obvious.

@RobertAron unfortunately as of today the shader extensions are
"extensions": [
          ".vs",
          ".fs",
          ".gs",
          ".comp",
          ".vert",
          ".tesc",
          ".tese",
          ".frag",
          ".geom",
          ".glsl",
          ".glslv",
          ".glslf",
          ".glslg"
],

.cpp files are handled by the editor as c++ files only.


successfull message

Thanks for the contribution, but adding cheers to the end of the page is not exactly helpful. Did you have any further plans with this pull request?

I'm closing this for now. Feel free to reopen when you can explain the thinking behind this.


Update main nav

Summary
As per similar docs.mm.com one, this PR updates the main navigation. The main nav has gone through several iterations on mm.com and it's stabilized enough that we can sync everything with docs. Going forward we'll try to keep the components pretty synced between mm.com and docs sites. Some changes are pushed out as tests on mm.com, so the changes won't be synced to docs until decisions are made post-test.
Ticket Link
https://app.asana.com/0/1201972580778757/1203959122017590/f
Lemme know if anything, thanks!

Thank you @cwarnermm + @azigler !


Make the optional field conditional

I have a field called "status" that is a required field. I have two other fields, Location and Habitat that I want to require when "status" == "Living". How do I make this happen?
Thanks,
Steve

Closing old issues. Please comment if this is still an issue and should be reopened.


Create EventSys/Router passthrough route creator

Description
To reduce code, let's create a helper function that creates a route by re-exposing an EventSys event. Let's also look at other possible helper functions to reduce Controller and Router code.
Acceptance Criteria
Tasks
Use this section to keep track of work

QA Notes
Environments
Steps

Router is gone and engine events creators can be dispatched directly from the frontend, so this is no longer applicable.


Android - Attachment -Attachment appears with a long delay.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Disable the internet connection in the device
Navigate to a chat and upload an attachment
Enable the internet connection in the device

Expected Result:
The attachment appears without any delay
Actual Result:
Attachment appears with a long delay.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native

Version Number: <IP_ADDRESS>
**Reproducible in staging?:**Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/211698216-fe23dddd-4d22-49cb-851b-eb0ef2857751.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Struggling to reproduce this one given I'm using internet for the android browserstack version.

Going to find someone with an android to test in APAC

This was my experience following the reproduction steps. I'm not sure how instant it should be but the delay doesn't seem very long to me.
https://user-images.githubusercontent.com/59587260/212803931-77c4c33f-47ba-47df-8d3b-02e49b2d005c.mp4

Thanks Kadie! I'm going to close this one out then.


New user group membership management

Needs plone.restapi changes: Checkout plone.restapi master branch until released

Merged: https://github.com/plone/plone.restapi/pull/1325
Merged: https://github.com/plone/plone.restapi/pull/1439
Merged: For testing the many_users/many_groups case https://github.com/plone/plone.restapi/pull/1435 is needed.
Merged: Search for fullname: https://github.com/plone/plone.restapi/pull/1444

Fix https://github.com/plone/volto/issues/3018

The draft is ready for reviewing. Before I switch to 'ready':
These PRs need review:
@tisto @sneridagh please review https://github.com/plone/plone.restapi/pull/1325 and https://github.com/plone/plone.restapi/pull/1435, thanks!

I reviewed this and it's really an impressive work, congrats!
I suggest 2 small improvements, feel free to comment on them:

Somehow obey the "Many users" setting of Plone. This way all users will not be shown in the panel if the setting is enabled
Allow searching also in the fullname, right now only username based searches are allowed

I reviewed this and it's really an impressive work, congrats!
I suggest 2 small improvements, feel free to comment on them (or just discard them 👅 ):
* Somehow obey the "Many users" setting of Plone. This way all users will not be shown in the panel if the setting is enabled

* Allow searching also in the fullname, right now only username based searches are allowed

Thanks for your review and feedback @erral!
The options "many_user" and "many_groups" are obeyed. The requested changes for this PR to plone.restapi were split in three PRs. Two are merged but not released.
There is one left for @plone/volto-team :  https://github.com/plone/plone.restapi/pull/1325
Check out https://github.com/plone/plone.restapi/pull/1325 for switching options "many_user" and "many_groups".
<  searching for fullname
hm, i do want this. didn't i implemented it?

ah OK i tested just with one branch of plone.restapi, let me check it.
Regarding the fullname, i tested and I only could search for username, strange... The search is done in the front or backend?

I was testing with that exact branch, and I don't see any difference enabling or disabling the many_users option, am I missing something?

Ah... I see, the PR is not rebased, so I wasn't testing it with the latest PRs merged

It looks like the controlpanel serializer fails in my Plone site when calling the @usergroup endpoint and that's the reason why I wasn't seeing the many_users and many_groups stuff.

It looks like the controlpanel serializer fails in my Plone site when calling the @controlpanels/usergroup endpoint and that's the reason why I wasn't seeing the many_users and many_groups stuff.

We do have a eionet LDAP at EEA where we can test many_users flag. I will try to test this branch today.

FWIW I was testing with the buildout provided by plone.restapi itself which is based on Plone 5.2.x

@erral

It looks like the controlpanel serializer fails in my Plone site when calling the @controlpanels/usergroup endpoint and that's the reason why I wasn't seeing the many_users and many_groups stuff.
Specifically the usergroup controlpanel does not have a title and group attributes and it fails when rendering it

Sorry, I can't reproduce. In the meantime further fixes were done, so maybe this is already solved.
The settings "many users" and "many groups" are taken from @controlpanels/usergroup. And cause its schema IUserGroupsSettingsSchema is already in Plone 5, this is OK also for Plone 5, as long as you checkout for now plone.restapi master until released.
BTW I take care to update the description of this PR with every step. Multiple PRs on plone.restapi are involved.

Regarding the user search, the @users endpoint just searches based on id:
https://github.com/plone/plone.restapi/blob/bc42a349bb7740d7a61155d4f0c50c92a66c5c9f/src/plone/restapi/services/users/get.py#L55

This is pipelined. Please review: https://github.com/plone/plone.restapi/pull/1444

FWIW I was testing with the buildout provided by plone.restapi itself which is based on Plone 5.2.x

Hm, I checked on both Plone 6 and Plone 5 just now again. Please comment again, if you still have problems.
Thank you for the feedback!

I reworked the panel for many users and many groups

No requests to the backend, until you search or filter.
page size is the default of the backend: 25. To see more, a button at the end of the list requests one more batch per click. No old style pagination.

@nileshgulia1, @avoinea, please throw an eye on this.
Please review also the last of four dependencies in plone.restapi: https://github.com/plone/plone.restapi/pull/1444 to finish the user group membership control panel . This would also help the CI here.

@ksuess Also Let's wait for @sneridagh  and we can merge after cypress is green!

Please release plone.restapi. Tests are prepared. I would love to see them green!

Group membership control panel is a need in a project. @tisto, please release plone.restapi to get tests green here, thanks.

Wouldn't  it be cool to have a panel that supports adding users to groups? Here it is!
You're just a review step away! Tests are greener than green, cucumber green!


Only deploy on master

The travis deploy scripts is trying to deploy pull requests, then obviously failing. causing the pull request to look like it's red - when it's not!
This should hopefully fix that

closing due to inactivity


Bad regression fix and doesn't require node_modules dir anymore.

Wasn't building on run anymore.

So this was removed and we're adding it back why was it removed in the first place?

I have tested this and can confirm it makes the problem I reported go away (i.e. fn run will now do a build if needed)

@hibooboo that one gone not so long ago, and we need to bring that back.


Still showing as threats with VirusTotal

Based on latest compiled release: https://www.virustotal.com/gui/file/fc3500507d6bc50f5172f3e2ee336e00a088f84ecc24199651a32ee089990de3/detection

This is a fight that I can never truly win. I'll attempt to follow a guide and compile a custom version of pyinstaller to deal with this on windows. I'll take a look at this tomorrow.

The threats you are showing are indeed "false positives"
The only OS I'm worried about is Windows to be honest. Are you having a virus detected within windows?

Hello @jlw4049,
I apologise for missing the notifications containing your updates, and would like to thank you for resolving this issue in a structured, professional and timely manner (that is not meant to be patronising and I apologise if it comes across that way). Additionally, As I write this, I can confirm your hard work has removed false positive detections from all 59 active heuristic engines used by VirusTotal, as well as a fully update-to-date Windows Security suite provided with Windows 11 (this previously also reported a false positive).
Once again, thank you and I hope you're staying safe given the current global situation.
Cheers,
Piers

Hello @jlw4049, I apologise for missing the notifications containing your updates, and would like to thank you for resolving this issue in a structured, professional and timely manner (that is not meant to be patronising and I apologise if it comes across that way). Additionally, As I write this, I can confirm your hard work has removed false positive detections from all 59 active heuristic engines used by VirusTotal, as well as a fully update-to-date Windows Security suite provided with Windows 11 (this previously also reported a false positive).
Edit: When submitting this report, I was using Windows 10 on a 6700K @ 4.4 GHz. Since then, I've upgraded to a 5900X and a clean install of Windows 11. The reason I mention this is not to brag, but to convey the improved scalability provided with the latest version. I realise this is partly based on newer FFmpeg builds, but it's also down to your programming and optimisations. I have a couple of suggestions and concerns which I will report in a new ticket.
Once again, thank you and I hope you're staying safe given the current global situation.
Cheers, Piers

No problem, researching it for a while seemed like there was no way to fix it. Then I came across a thread where I was able to actually compile my own version of pyinstaller x64 bit and it virtually removed every false positive. I'm glad I was able to figure it out!


[SPARK-12396][Core] Modify the function scheduleAtFixedRate to schedule.

Instead of just cancel the registrationRetryTimer to avoid driver retry connect to master, change the function to schedule.
It is no need to register to master iteratively.

Can one of the admins verify this patch?

add to whitelist

Test build #48249 has started for PR 10447 at commit 34cab3a.

Changes LGTM. I'll merge this once tests pass

Test build #48249 has finished for PR 10447 at commit 34cab3a.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * class JavaWordBlacklist \n  * class JavaDroppedWordsCounter \n

Merged build finished. Test PASSed.

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

Merging to master

@echoTomei Thanks a lot!


[LIBCLOUD-944] Add support for private IP addresses in GCE instance creation

Add support for private IP addresses in GCE instance creation
Description
GCE driver doesn't currently allow private IPs to be specified when creating instances. Updated gce.py driver to allow the optional parameter internal_ip which can be used when creating GCE instances.
Status

done, ready for review

Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)
[x] Documentation
[x] Tests
[ ] ICLA (required for bigger changes)

Coverage decreased (-0.004%) to 88.966% when pulling 27d79ea671b1d502930719d00cacbd8337be3a8b on vongazman:LIBCLOUD-944_private_ip into f1bff48142bd942ce1e81abb9769be3dd09a3d76 on apache:trunk.

Thank you @vongazman for your contribution. Great first-time GitHub contribution! The changes make perfect sense! 👍
However, this pull requests lacks tests. I realize that the existing external_ip parameter is not tested, but I believe you should add tests at least for internal_ip. You can look at the DimensionData tests to see how you can update the fixtures and test code to do that.

Codecov Report

Merging #1107 into trunk will increase coverage by <.01%.
The diff coverage is 86.66%.

@@            Coverage Diff             @@
##            trunk    #1107      +/-   ##
==========================================
+ Coverage   85.48%   85.48%   +<.01%
==========================================
  Files         348      348
  Lines       66443    66458      +15
  Branches     5914     5917       +3
==========================================
+ Hits        56796    56809      +13
- Misses       7239     7240       +1
- Partials     2408     2409       +1

Impacted Files
Coverage Δ

libcloud/compute/drivers/gce.py
76.05% <100%> (+0.01%)
:arrow_up:

libcloud/test/compute/test_gce.py
97.49% <84.61%> (-0.08%)
:arrow_down:

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 5e1972e...df21efb. Read the comment docs.

Thanks @vongazman! ✨ I've included this in trunk.
(Reworked the commits to remove the intermediate ones that would have made bisecting difficult, and signed them off as part of the standard procedure.)


[SPARK-24732][SQL] Type coercion between MapTypes.

What changes were proposed in this pull request?
Currently we don't allow type coercion between maps.
We can support type coercion between MapTypes where both the key types and the value types are compatible.
How was this patch tested?
Added tests.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/635/
Test PASSed.

cc @gatorsmile @cloud-fan

Test build #92559 has started for PR 21703 at commit 928501a.

LGTM

Test build #92559 has finished for PR 21703 at commit 928501a.

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

Merged build finished. Test PASSed.

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

Merged to master.


typescriptFormatter not work in<EMAIL_ADDRESS>

Is this a bug report?
bug

message.getFile is undefined

Did you try recovering your dependencies?
not work in

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

in fork-ts-checker commit 5f5d967e07c7da0e860b6854857da145c33c7c0f remove getFile and add file

Attn @Timer

When i initially read this i forgot we have forked fork-ts-checker-webpack-plugin, as you can see in react-scripts package.json (references fork-ts-checker-webpack-plugin-alt).
It appears you're using a package that isn't part of create-react-app. If you've ejected and are updating the packages yourself, then i'm afraid you're on your own.
I suggest you file an issue over at https://github.com/Realytics/fork-ts-checker-webpack-plugin/releases if you think there is a problem with 0.5.0.

thanks


search people

ERROR SQL syntax at PeopleController line 52. to solving this bugs is change "ILIKE" word in the query builder to "LIKE" at line 19

ILIKE is not a bug in postgresql, but thank you for your open issue, i will update it to LIKE and use str lower to handle case-censitive

I have fix this issue with this PR, i change to use LIKE instead to use ILIKE, to generally used database type, thanks


Triggering Integration Flows

Add new content to [1].
[1] https://github.com/wso2/docs-ei/blob/master/en/streaming-integrator/docs/guides/triggering-integration-flows.md

Added [1] for the EI 7.0.0 release.
[1] https://ei.docs.wso2.com/en/latest/streaming-integrator/guides/triggering-integration-flows/

Added [1] for the EI 7.0.0 release.
[1] https://ei.docs.wso2.com/en/latest/streaming-integrator/guides/triggering-integration-flows/


fix(vehicle_cmd_gate): use real velocity for the lateral filter

Description
Use the real velocity for the lateral command filter.
Otherwise, if the velocity command oscillates, the filtered steering oscillates.
The drawback is that it won't guarantee the constraints are met in the series of the output command itself. (We need to see the actual velocity data to check it.) But this is much safer than the previous implementation.
Related links
TIERIV Internal ticket
Tests performed
Run in a real vehicle testing and it worked well.
Notes for reviewers
None
Interface changes
None
Effects on system behavior
Improve the filtering. No effect in the default parameter.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Need to fix test


[WIP] Added terraform plan for Openstack Docker and respective module

What is the problem that this PR is trying to fix?
Adding new plan for openstack infrastructure to create basic deployment of travis-worker with docker backend

There are several aspects which I am not able to figure out like or require more discussion:
a) Service is staring as root user not normal as travis user for now.
b) Worker is built using source due to absence of binary available for power, causing a delay of 2-3 minutes for cloud-init to complete.
c)  Not sure of how to pass sensitive data such as AMQP_URL or Openstack account details.
d) No image tagging done for docker images to be used during the build process.

What approach did you choose and why?
Tested over OSU setup
How can you test this?
This is WIP, and would require more insights and assistance to take it to completion.
What feedback would you like, if any?

Please let me know if the approach or direction taken is suitable or not

I'm sorry for my long delayed feedback.  I believe this is safe to merge in its current form, and modifications and improvements can come later :+1: :heart:


Cannot edit user pubkey

Chef Server Version
13.2.0
Platform Details
Ubuntu
Configuration
Standalone
Scenario:
Edit a user's information
Steps to Reproduce:

sudo chef-server-ctl user-edit USER
Make changes (for example to public_key)
Save and quit (example using vi)

Expected Result:
The User info is saved
Actual Result:
No changes saved and I get this error:
ERROR: ArgumentError: Please pass in the object class to hydrate or use #edit_hash

Maybe this isn't supported by design, but the error message is not straightforward. I was able to change user public keys with sudo chef-server-ctl add-user-key USER --public-key-path PATH --key-name default.