getParticipantFromUser Not working

Hi,getParticipantFromUser() creates the following query and cause any method that uses this method to not work
select * from participants where participants.deleted_at is null and participants.thread_id is null and participants.thread_id is not null and user_id = 2 limit 1

Can you share the code that you're working with? Is the thread that you are trying to access available in the DB?


Bug: Classification finds exactly 100 more items than it should

Background
Using the private dataset mentioned in #1355 and #1356 the following pattern is established:

Import n items
Run a classification
Repeat above steps i items until n*i == total

where n=400 and total~=20000
Expected Behavior
Each classification finds exacty 400 items
Actual Behavior
After around 10-12 batches, suddenly the count is reported is 500 items. This pattern repeats in subsequent batches.

This ticket was originally written with a filter problem in mind. I.e. "we find more than we should". However, after a very thorough investigation we have been able to find out that the filter is working as intended. Instead the save functionality dropped batches which weren't saved. So the correct title should be: "Classification sometimes doesn't save the last 100 items".

This ticket was originally written with a filter problem in mind. I.e. "we find more than we should". However, after a very thorough investigation we have been able to find out that the filter is working as intended. Instead the save functionality dropped batches which weren't saved. So the correct title should be: "Classification sometimes doesn't save the last 100 items".


changing svg color with spring

is there a way to animate svg fill to change colors with spring?
i tried to do this but the color is only changing at the end of the spring animation and not in a smooth way. just suddenly changing from color A to color B.

Without seeing the code I'm guessing you're trying to transition between to string representations of color?
What you want to do is transition between numeric values and interpolate that into a string rgba(255,255,${your_value})

<linearGradient id={"grad"+id}x1="0%" y1="0%" x2="0%" y2="100%" >                              <stop offset="0%" style={{stopColor:"rgba(255,"+g+",0,1)",stopOpacity:1}} />                              <stop offset="100%" style={{stopColor:"rgba(255,0,0,1)",stopOpacity:1}} />  </linearGradient>
where g is the the variable bound to the spring.

See also #85 and #153 (particularly @herrstucki's http://bl.ocks.org/herrstucki/27dc76b6f8411b4725bb)

rgba(255, ${ parseInt(g) }, 0, 1) should work

Please look at https://github.com/chenglou/react-motion/issues/304#issuecomment-331750756 comment.

Also https://github.com/chenglou/react-motion/issues/394#issuecomment-262631976

Added generic issue to cover this case in README: https://github.com/chenglou/react-motion/issues/494


[WORK IN PROGRESS] CI

Ignore for now.

I am going to close this for now.
I want to focus first on cleaning up our tests and race conditions so that we can paralelize, then we can come back to this.


Added Dropbox remote uploader

Made changes to frontend and added code for dropbox remote loader
Used https://python.langchain.com/docs/integrations/document_loaders/dropbox/ as reference.
fixes #1179

Removed recursive in the frontend

I tried testing it it seems like its not working, I created an app with access token and chose a folder path.
It does not seem to trigger worker to start training
Thank you!

I'll try fixing it and make a revision

Hey @dartpain Can you check it now?

It seems that I keep getting error 400. Can you provide some instructions please?
Maybe an example would be great.


dotnet: Support Aarch64

It just worked :shrug: so why not.

Tested profiling of tests/containers/dotnet/Dockerfile app with gProfiler exe & binary from this branch - both work.

The same manifest hash 749439ff7a431ab4bc38d43cea453dff9ae1ed89a707c318b5082f9b2b25fa22 has an image for both x86 and aarch64.


Error with double carets in listings code block

Loving this template so far but ran into this issue with inline double carets ^^ - took me quite a while to figure out this was the problem.  Was able to work around it by separating into two separate in-line blocks, but I'm guessing that this isn't intended behaviour.
Running pandoc test.md -o test.pdf --from markdown --template eisvogel.tex --listings with
test.md:
`^^`

produces:
Error producing PDF.
! File ended while scanning use of \lst@temp.
<inserted text>
                \par
<*> /tmp/tex2pdf.-123e5503f3b3824b/input.tex

Here is how much of TeX's memory you used:
 28660 strings out of 494923
 487490 string characters out of 6180742
 718748 words of memory out of 5000000
 31684 multiletter control sequences out of 15000+600000
 552952 words of font info for 32 fonts, out of 8000000 for 9000
 14 hyphenation exceptions out of 8191
 49i,4n,95p,10561b,1220s stack positions out of 5000i,500n,10000p,200000b,80000s

!  ==> Fatal error occurred, no output PDF file produced!

I'm assuming it's to do with the syntax highlighting, because it ran fine without --listings.

Fixed upstream in commit https://github.com/jgm/pandoc/commit/9d0506e4047f797a0672d7767830b59f1710d191 which is available with pandoc 2.10.


CORS request was denied

Hi,
We are trying to use elasticsearch-tableau-connector on local desktop and getting "CORS request was denied" error while connecting to elasticsearch. Our elasticsearch url is https://xxxx:9200
I have already added following section in elasticsearch.yml file of all elasticsearch servers, still we are getting this error.
Please provide us some pointers to troubleshoot it.
#Cors enable for Tableau
http.cors.enabled : true
script.inline : true
script.indexed : true
http.cors.allow-origin : "*"
http.cors.allow-methods : OPTIONS, HEAD, GET, POST, PUT, DELETE
http.cors.allow-headers : X-Requested-With,X-Auth-Token,Content-Type,Content-Length
http.jsonp.enable : true
cors.enabled : true
http.cors.allow-credentials : true
http.detailed_errors.enabled : true
Tested connectivity and I am able to reach to elasticsearch from local desktop. Output below:
"name" : "node335M",
"cluster_name" : "cd_elk_cluster-dev",
"version" : {
"number" : "2.2.0",
"build_hash" : "8ff36d139e16f8720f2947ef62c8167a888992fe",
"build_timestamp" : "2016-01-27T13:32:39Z",
"build_snapshot" : false,
"lucene_version" : "5.4.1"
},
"tagline" : "You Know, for Search"
}
Thanks,
Rakesh

I believe you need the configuration:
http.cors.enabled : true

it looks like you have:
cors.enabled : true

Closing due to inactivity, please open a new issue if this persists.


open_manipulator_p_simulations: 1.0.0-2 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository open_manipulator_p_simulations to 1.0.0-2:

upstream repository: https://github.com/ROBOTIS-GIT/open_manipulator_p_simulations.git
release repository: https://github.com/ROBOTIS-GIT-release/open_manipulator_p_simulations-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.10.0
previous version for package: null

open_manipulator_p_gazebo
* First release of the open_manipulator_p_simulations stack
* Contributors: Ryan Shim, YongHo-Na, HyeJong KIM

open_manipulator_p_simulations
* First release of the open_manipulator_p_simulations stack
* Contributors: Ryan Shim, YongHo-Na, HyeJong KIM

@JaehyunShim friendly ping


Support better multi-tenancy with @tenant transformer

Is your feature request related to a problem? Please describe.
Multi-tenancy (MT) implementation support using Amplify is very limited. The only MT strategy that is possible is to have a tenantId column in tables. With 100+ tenants, table sizes might explode. Making lookups slow and deleting a tenant data complicated.
A better alternative is to have tenant specific tables using prefixing: tenantid1_posts, tenantid_2_posts ... . This can possibly be achieved with a @tenant transformer with something like:
type Blog @model {
  id: ID!
  name: String!
  posts: [Post] @connection(name: "BlogPosts")
}
type Post @model @tenant {
  id: ID!
  title: String!
  blog: Blog @connection(name: "BlogPosts")
  tenantid: String
}

such as when a tenantid is provided, the entry will be created in a separate tenantid prefixed table/collection.
About MT strategies
https://d0.awsstatic.com/whitepapers/Multi_Tenant_SaaS_Storage_Strategies.pdf

@mdegrees We are looking at more multi-tenancy support this year, however rather than look at a specific implementation could you uplevel a bit and go into detail around your use cases and scenarios? This would allow us to look at if there is a point feature causing problems here or a larger design that is cross cutting categories which needs implementation. Thanks.

@mdegrees We are looking at more multi-tenancy support this year, however rather than look at a specific implementation could you uplevel a bit and go into detail around your use cases and scenarios? This would allow us to look at if there is a point feature causing problems here or a larger design that is cross cutting categories which needs implementation. Thanks.

Great to know this is on your radar, @undefobj . Do you have a sense of when this may be tackled by the team, or suggestions for strategies related to multi-tenancy in the meantime?

@undefobj my use case is largely similar to Slack. Users create spaces. Some spaces are private some are public. In each space, users belong to privilege groups: admins, moderators, regular users. Admins get to decide what's authorized and what's not on their space. For e.g they can enable/disable moderators to delete posts.
The first wall I hit was: configurable auth. Amplify allows you to statically define the rules, but what if auth is configurable at the user level.
The second wall I hit was isolating data. In an sql database I would create a database per space.
For security reasons but also for ease of manipulation. Deleting a tenant for e.g is as simple as deleting a database. An acceptable compromise is isolation at the table level: adding a prefix to tenant tables (the proposal).
In a perfect world where amplify has an API and deployment of a new amplify instance takes less than 5 seconds, this problem would've not existed. I would just interact with the amplify API , create or update a whole amplify instance upon user actions.
Thank you

@undefobj my use case is largely similar to Slack. Users create spaces. Some spaces are private some are public. In each space, users belong to privilege groups: admins, moderators, regular users. Admins get to decide what's authorized and what's not on their space. For e.g they can enable/disable moderators to delete posts.
The first wall I hit was: configurable auth. Amplify allows you to statically define the rules, but what if auth is configurable at the user level.

What does this mean specifically? You want to define an owner on a database record? Amplify supports this. I'm not really following what "configurable at the user level" means.

The second wall I hit was isolating data. In an sql database I would create a database per space.
For security reasons but also for ease of manipulation. Deleting a tenant for e.g is as simple as deleting a database. An acceptable compromise is isolation at the table level: adding a prefix to tenant tables (the proposal).
In a perfect world where amplify has an API and deployment of a new amplify instance takes less than 5 seconds, this problem would've not existed. I would just interact with the amplify API , create or update a whole amplify instance upon user actions (on behalf of users).

If you're going  to separate each tenant by database, then why not just duplicate Amplify projects? Separate databases is a way to go but I think this is opposite from what many people define multi tenancy to be. Usually it's shared resources be that User Directories, Databases, or even Compute instances (the lowest level). Separate databases will have more operational overhead and ultimately costs which is why shared tenancy gives you business advantages.
I don't think a specific directive solves these problems. Perhaps this article talking about different techniques might help you here: https://medium.com/@dantasfiles/multi-tenant-aws-amplify-cc3252c4def4
Note that the OR rules for @auth are on our roadmap and that might be a way to help your design further.

@undefobj Do you have anything to share about the design of your AND rule implementation? Are there any plans to develop it in the open? AND rules were critical for me to create a multi-tenant system.

@undefobj Do you have anything to share about the design of your AND rule implementation? Are there any plans to develop it in the open? AND rules were critical for me to create a multi-tenant system.

Right now just matching proposal 5 is planned: https://github.com/aws-amplify/amplify-cli/issues/1043
If there's more feedback needed it will be public.

What does this mean specifically? You want to define an owner on a database record? Amplify supports this. I'm not really following what "configurable at the user level" means.

Imagine we're creating AMAs as a service. It's a service where customers create dedicated ask me anything pages. AMAs can be private or public. It's the customer who decides. (1) There is a switch in the admin panel to go private or public. (2) There is also a setting option that enables or disables moderators to edit the incoming questions.
In this case auth rules are dynamic and managed at the user level. I can default the schema to some auth rules. But the tenant can go and update them. I've learned about dynamic group access in the docs. If I've well understood to address feature number 2 for e.g, you would go with something like
type Question @model
  @auth(rules: [
    { allow: groups, groupsField: "groupsCanUpdate", operations: [update] }
  ]) {
  id: ID!
  title: String!
  content: String
  owner: String
  editors: [String]!
  groupsCanUpdate: [String]!
}

Then resolve groupsCanUpdate array from a settings table?
Also if auth is defined per record. Does it mean if groupsCanAccess is set to  ['admin', 'moderators'] then the tenant decides to change this value, I have to update all the previous records with the new [string] value? What if there was thousands of them?

Yes if you're setting group ACLs on an individual record, then you need to go back and update the records if information changes. This is why ideally you would separate groups by tenant and then update the groups rather than the records. So it's really up to you on what type of control and partitioning you'd like to give but the controls are in the system.

@undefobj is there any example i can look at? What would it look like for the above example? thank you

No I don't have an example of updating records handy, but you should be able to do this using Lambda and the DynamoDB document client.

@undefobj ok, thank you!
This reddit discussion revolves around the same pain points. Referencing it here for further readers and contributors.
https://www.reddit.com/r/aws/comments/fccx6k/multitenant_app_using_aws_amplify/


WIP: Add improved CSS border style props to View

@nick-thompson, Would you mind giving this a quick pass to let me know if you think it's going roughly in the direction before I do any refactoring and quirky behaviour fixes. All the basics are working just fine with this approach. Bit of repetition that can be cleaned up but over all I think the approach came out pretty nicely and offers a decent level of customisation without going too mad.

Yep, this looks like a good direction. I think we can probably refactor some of the js to clean up a bit, but "make it run, make it right, make it fast" and all that.
Before we go too hard this way though: I wonder, do you notice any performance implications when rendering, e.g. long lists of things that each have a border? We're definitely doing a bunch of small string concatenation here, which I think is one of duktape's performance warnings.
I'm also going to poke at some browser implementations of these border renderings just to make sure there's not some obvious alternative way of doing this that I'm totally missing. Will try to get to that this weekend

Before we go too hard this way though: I wonder, do you notice any performance implications when rendering, e.g. long lists of things that each have a border? We're definitely doing a bunch of small string concatenation here, which I think is one of duktape's performance warnings.

Yeah I'm not sure. Totally possible. I originally started working on this in C++ as I wasn't too sure on how the performance  with a pure SVG or Canvas approach would be. I can try this today and take a look at how things are performing. It's certainly the cleanest approach as far as I can tell ..... might not be the fastest though.

Before we go too hard this way though: I wonder, do you notice any performance implications when rendering, e.g. long lists of things that each have a border? We're definitely doing a bunch of small string concatenation here, which I think is one of duktape's performance warnings.

Yup. I tackled this in C++ originally as I figured if this is happening at the View level I should probably go for the quickest thing possible. The SVG approach is arguably the cleanest.....not sure if it will be the fastest. I can try to get some perf tests on this today.

@nick-thompson , Based on our recent discussions to move away from an SVG based approach for CSS border props I'm going to close this and will open up a different MR in due course which takes a more performant approach.

@nick-thompson , Based on our recent discussions to move away from an SVG based approach for CSS border props I'm going to close this and will open up a different MR in due course which takes a more performant approach.


Fixed

Fixed issue #664

@RahulScripted Please see the conflicts . Resolve the conflicts . Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Adding extra commands

Is it possible to execute additional commands like --write-subs for youtube-dl and --embed-subs for ffmpeg?

Not possible at the moment. I don't have that much experience with how embed subtitles work in different container formats so i will need to do some research to figure how and if it would work.

Did some research. It look promising but probably require quite a bit of changes, can't promise when i have time and energy to work on it.

There is a experimental branch now with subtitles support if you want to try https://github.com/wader/ydls/pull/6
Seems to work kind of ok but the code is a bit messy


Make error with benchmark/small SGEMM

I get following error when do the following instruction:
cd benchmark/small_SGEMM && make

error output:
../../NN_LIB/SMM_NT_thread.c: In function ‘void SGEMM_NT_mp(float*, float*, float*, long int, long int, long int)’:
../../NN_LIB/SMM_NT_thread.c:6362:19: warning: ignoring return value of ‘int posix_memalign(void**, size_t, size_t)’, declared with attribute warn_unused_result [-Wunused-result]
     posix_memalign(&ptr, 64, T * GEMM_K * 17  *sizeof( float ));
     ~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
g++ -c -O3 ../../NN_LIB/DGEMM_NN.c -o DGEMM_NN.o
g++ LibShalom_sgemm.o test_temp_L1.o test_temp.o NN_SMM.o test_SMM_NT.o SMM_thread.o SMM_NT_thread.o DGEMM_NN.o -O2 -fopenmp  -o main
/usr/bin/ld: SMM_thread.o: in function `LibShalom_sgemm_mp(int, int, float*, float*, float*, long, long, long)':
SMM_thread.c:(.text+0x5a0): undefined reference to `Small_MGN_NN_SGEMM(float*, float*, float*, long, long, long, long)'
/usr/bin/ld: SMM_thread.c:(.text+0x620): undefined reference to `Small_NGM_NN_SGEMM(float*, float*, float*, long, long
, long, long)'
/usr/bin/ld: SMM_NT_thread.o: in function `SGEMM_NT_mp(float*, float*, float*, long, long, long) [clone ._omp_fn.0]':
SMM_NT_thread.c:(.text+0x36b0): undefined reference to `SGEMM_NT_kernel_exist_1(float*, float*, float*, long, long, long, long, long, float*, long)'
/usr/bin/ld: SMM_NT_thread.c:(.text+0x3744): undefined reference to `SGEMM_NT_kernel_exist_1(float*, float*, float*, long, long, long, long, long, float*, long)'
collect2: error: ld returned 1 exit status
make: *** [Makefile:4：main] error 1

Hello, submit the following command line in this path (https://github.com/AnonymousYWL/LibShalom/tree/main/evaluate): " sh run.sh ".
Thank you for your attention to this work.


Add integration services

Added Travis and CI badge
Added Code Climate badge

Covers issues #3 and #4

Yes!


Page Linking (target attribute) not working with special characters

If you have a external link like:
[link](http://www.xyz.com/folder/#page?target=_blank)
then the target attribute isn't working because of the  #
A link like:
[Link](https://www.landkreis-bayreuth.de/KulturBildung/Weiterbildung/VHS-Programm.aspx?view=~/custom/vhs-kalender/details&orgid=330f1b0f-d22b-429a-a90f-679f76967836)
gets processed as:
https://www.landkreis-bayreuth.de/KulturBildung/Weiterbildung/VHS-Programm.aspx?view=~%2Fcustom%2Fvhs-kalender%2Fdetails&orgid=330f1b0f-d22b-429a-a90f-679f76967836
Guess it's because of the ~ the / gets processed as %2F
and is only working if you put the link in html code in your markdown file.

First URL issue: the fragment needs to be last. So write it like this:
[link](http://www.xyz.com/folder/?target=_blank#page)
Second URL issue: in the query part of the URL, the one after the ?, special characters need to be encoded. So it's a problem in the destination site, rather than in Grav. Grav processes the URL as it should be. You can write it as a normal HTML link, as you said.


Update admin.php to latest version

This PR updates admin.php to the latest deployed version scrubbed of any sensitive data or environment-specific details (e.g. where to save PHP sessions or PHP include paths). The changes have been made by other maintainers over the last 10 years without updating the copy here. The intent is to bring this repository up-to-date before making any further updates.
Note that this PR makes it look like there are a lot of changes, but that's mostly due to whitespace differences.  The actual changes after controlling for whitespace are listed below. To generate the diff, I ran git diff --ignore-all-space or git diff -w for short.
Differences after ignoring changes in whitespace:
diff --git a/scripts/admin.php b/scripts/admin.php
old mode 100755
new mode 100644
index 711687b..97d844b
--- a/scripts/admin.php
+++ b/scripts/admin.php
@@ -1,9 +1,13 @@
 <?php @session_start();
+
+	require_once("../lib/domxml-php4-to-php5.php");
+
 	//clearBrowserCache();
 	header('Content-Type: text/html; charset=UTF-8');

 	// Logout: End Session and reload page
-	if(isset($_GET['logout']) && $_GET['logout'] == '1'){
+	if($_GET['logout'] == '1'){
+		error_log('Logging out...');
 		logout();
 	}

@@ -18,11 +22,23 @@
 	// $_SESSION['passwords']['turkish'] = "";

 	$_SESSION['current'] = '';
-	$current_directory = basename(getcwd());
-	if($current_directory !== FALSE) {
-		$_SESSION['current'] = strtolower($current_directory);
-	}

+	$current = '';
+	$cwd = strtoupper(getcwd());
+	if (strstr($cwd, "HEBREW"))
+		$current = 'hebrew';
+	else if (strstr($cwd, "SPANISH"))
+		$current = 'spanish';
+	else if (strstr($cwd, "FRENCH"))
+		$current = 'french';
+	else if (strstr($cwd, "TIBET"))
+		$current = 'tibet';
+	else if (strstr($cwd, "TEST"))
+		$current = 'test';
+	else
+		$current = 'test';
+
+	$_SESSION['current'] = $current;

 	// Checks if user is logged in, ask to login
 	function login(){
@@ -36,13 +52,12 @@
 				$_SESSION['pword'] = $_SESSION['passwords'][$_SESSION['current']];
 			}
 		}
-		$logged_in = isset($_SESSION['pword']) && $_SESSION['pword'] == $_SESSION['passwords'][$_SESSION['current']];
-		if(!$logged_in) {
+		if ($_SESSION['pword'] != $_SESSION['passwords'][$_SESSION['current']]){
 			echo('<form method="post">Password: <input type="password" name="password" />' .
 					'<button type="submit">Submit</button></form>

Flashcards');
 		}
 		// User is logged in:
-		else {
+		else if ($_SESSION['pword'] == $_SESSION['passwords'][$_SESSION['current']]){
 			/**********************************************
 			 * SERVER TASKS
 			 *********************************************/
@@ -105,9 +120,6 @@
  * General Setup
  **************************************/
 	function initializePHP(){
-		if (PHP_VERSION >= '5') {
-			require_once('../lib/domxml-php4-to-php5.php');
-		}
 		// Acknowledge Current Session and Open Map File
 		$_SESSION['map'] = domxml_open_file("xml/map.xml");
 		$_SESSION['prototype'] = domxml_open_file("xml/prototype.xml");
@@ -507,7 +519,7 @@
 		unset($_POST['password']);
         unset($_SESSION['pword']);
 		session_destroy();
-        header("Location: admin.php");
+        header("Location: admin.cgi");
 	}

 	// takes in a file path ('test/xml/Week1.xml')

PHP Session Save Path and Include Path:
I should note that the PHP session save path and include path were previously declared at the top of admin.php and were stripped out after removing the hard-coded secrets. These should be configured separately in the PHP environment, and were likely artifacts of the code being hosted elsewhere temporarily, but I'm making a note of them here in case they are needed again.
<?php
ini_set("include_path", '/home/USER/php:' . ini_get("include_path") );
session_save_path("/path/to/sessions/");
@session_start()
// ...

FYI @Tanvez @jaguillette @kevinguiney

@arthurian fyi, you can view whitespace-less diffs directly in github diffs (including PR diffs), e.g.:
https://github.com/Harvard-ATG/HewbrewFlashcards/pull/1/files?diff=unified&w=1
For more, see https://github.blog/2018-05-01-ignore-white-space-in-code-review/.

@elliottyates Very cool! Didn't know you could do that :)

I'm not familiar with these particular changes, but they look okay to me.


Cannot import sql from databricks

Description
After pip installing the databricks-sql-connector in a notebook as described in https://learn.microsoft.com/en-us/azure/databricks/dev-tools/python-sql-connector we get a dependency issue between certain packages

After this we have a ModuleNotFoundError,

but it can autocomplete the desired module

Reproduction
pip install databricks-sql-connect
Expected behavior
3 months ago I used the same code and it worked fine to install the package and use the sql module, while now it can't import it.
Other Information

Databricks cluster runtime 13.2
Databricks-sdk Version: 0.1.6 & 0.5.0(both failed)

Sorry about this! The default version of the SDK installed on DBR 13.2 is 0.1.6. I was able to reproduce this according to your instructions.
To mitigate, you should be able to run
%pip install databricks-sdk -U

followed by
%pip install databricks-sql-connector

and then
dbutils.library.restartPython()

to upgrade the SDK and install the SQL connector.
Please let me know if this solves your problem.

Thanks for the swift response and great explanation, works like a charm!


Tendril types for Send and concurrency.

This satisfies two of the items in “plans for the future” from the README. By setting NonAtomic as the default for the new type parameter, backwards compatibility (and convenience) is mostly maintained as Tendril<F> will still be non-atomic; Tendril<F, Atomic> opts in to the concurrency features.
The design and implementation of Atomic has not been vetted and should definitely not be merged without such vetting, because I’m not this tall.

r? @pcwalton, particularly the usage of atomics and transmute.

I think this is all good now except for needing @pcwalton’s review of the concurrency aspects.

The RC manipulation looks OK to me.

v0.1.4 is on crates.io


Send local file to Kodi

Currently, I use DroidUPnP to send music on my phone straight to Kodi to play using UPnP.
However, it would be nice if Kore could do this by itself, because it is a lot nicer to only use a single app for all your Kodi-related needs instead of having to switch between several apps.

I understand and see the merit of your suggestion, but turning Kore into a UPnP server isn't really in my plans right now. I just don't see this being done in the short/medium term, so i don't think this issue should be kept open.


Issue getting container working following these steps

Following these steps, we are unable to get this to work.  The Dockerfile appears to be missing the last two lines with the CMD to start the services.   We were able to get it working if we use pip requirements instead of Conda.

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

ID: a5111706-3d4f-f436-5f72-49b879c8d3fa
Version Independent ID: a34c079f-a4e6-219d-1fb5-56ee900c2a58
Content: Deploy ml models to Azure Functions Apps (preview) - Azure Machine Learning
Content Source: articles/machine-learning/how-to-deploy-functions.md
Service: machine-learning
Sub-service: core
GitHub Login: @vaidyas
Microsoft Alias: vaidyas

@bigdatamoore we will review your feedback and get back to you shortly. Thanks.

@vaidya-s can you please review the above feedback or reassign to the appropriate content author? Thanks.

Adding @Bozhong<EMAIL_ADDRESS>From: GiftA-MSFT<EMAIL_ADDRESS>Sent: Wednesday, May 13, 2020 11:24 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Vaidyaraman Sambasivam<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Issue getting container working following these steps (#54455)
@vaidya-shttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fvaidya-s&data=02|01|vaidyas%40microsoft.com|9cc6d09d7e7d427e93aa08d7f76ad563|72f988bf86f141af91ab2d7cd011db47|1|0|637249910536746732&sdata=FyEHc6IE9GEV13kPX%2BhL1oABLEnkZ%2B%2BkiDe%2BVVz2KQQ%3D&reserved=0 can you please review the above feedback and/or reassign to the appropriate content author? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F54455%23issuecomment-628165529&data=02|01|vaidyas%40microsoft.com|9cc6d09d7e7d427e93aa08d7f76ad563|72f988bf86f141af91ab2d7cd011db47|1|0|637249910536756725&sdata=KRHfTTmeQzbd6fOlPMsGWEsXSmneXKHSaR2e27vpSpw%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAK5XKTVBXUAWWPEIKORD3B3RRLQUXANCNFSM4M4KRE7A&data=02|01|vaidyas%40microsoft.com|9cc6d09d7e7d427e93aa08d7f76ad563|72f988bf86f141af91ab2d7cd011db47|1|0|637249910536756725&sdata=NP%2BZFER8tnkLbHJXL47Moddb4r7BxJaCvcEH4P5SRzE%3D&reserved=0.

Hi @bigdatamoore
Which step in the documentation were you unable to repro?

Hi @GiftA-MSFT
This functionality is in preview, it lacks fit-n-finish
We are working towards getting our serverless alternative running seamlessly.


Create chatters list

[ ] fetch chatter list from twitch api
[ ] listen to JOIN and PART messages from irc
[ ] create a popover that shows all users by group
[ ] optional: make it searchable
[ ] optional: add actions for selected users like timeout and ban

add a filter "active in last x minutes"


Feature request: Tab item close handling could be improved to be more mvvm friendly

Is your feature request related to a problem? Please describe.
I ran into the same issue as the OP in #2302
While i did find a workaround to link up TabItems in a mvvm friendly way, getting a solution which is async+await friendly is more of an issue.
Describe the solution you'd like
Ideally https://github.com/MahApps/MahApps.Metro/blob/71d455826f77d37ad0aa33a922e2b5394406fa52/src/MahApps.Metro/Controls/MetroTabControl.cs#L88 would be public delegate Task ...
... so resulting event handlers can be executed asynchronously. That way a viewModel could contain method signatures like Task TryCloseTabAsync and do non blocking calls and set the close args onec the async call is done.
The current state is very annoying if you try to delegate close permission handling to the viewModel, since you have to do a blocking call on the viewModel of a tab in the event handler before setting the args to Canceled.
Closed Issues
#2302

Hi @taori
As long as this feature is not integrated here,  you may try using Dragablz. I think they support what you want and they integrate with MahApps. https://github.com/ButchersBoy/Dragablz
Happy coding
Tim


Wait till amount is availble

#123 Wait until items are availble
Changed Put, Take and Craft to wait for at least one item when using X = ALL
Changed Put, Take and Craft to wait for X if X is != ALL
Changed drop to wait for an item
Changed drop to remove the dropped item from the inventory
I haven't tested drop, as it was broken in c++ part. Craft, take and put have been verified.

I changed it to warnings, which shouldn't spam quite as much

Sorry, but I don't want warnings either. Unless the character stops running there is no problem in waiting for some ticks before executing a step - nothing is lost by that, given that the ticks would just be "wasted" afterwards as the character is running without any steps to execute.
As a developer I would though like to get a warning telling me how many ticks was wasted waiting to executed a step, if the character stops with steps still pending for that run sequence.

That seems substantially different from the original description.

We currently don't have the ability to choose between debug and warnings messages. And a message of waiting for a number items before proceeding, seems perfect for debug level msg.
We don't currently have the ability to create delayed messages or measure the tick between starting a task and ending it. It does sound useful for a number of other things.

I believe it is important to give the developer a heads up, when the TAS does something different than what they might expect.

I can see the case, where waiting 3 ticks for an item might not warrant a warning, a verbose msg would fit here.
Waiting 120 ticks or forever, should give a warning.
If the character is walking, it might walk past the item source, which could be problematic and should give a debug msg.

I think it is out of scope for this PR to solve 1&2, we could wait for a future where they are solved or work with what we have now. In the latter case, we have to either print a warning message that could be annoying or not print a message which could be confusing.

I am pretty sure that I didn't say that I want the user to be spammed with messages when the feature is used. :) It is the same behavior as we have with building something. If the buildings isn't available to build the character will keep on running until it either has the building available and it is build or the characters stops which then triggers a warning - because that most likely wasn't the intend to have the character just stand around waiting.
It isn't possible for the character to run past something, given that the tool will ensure that the character won't be out of reach to perform a step. I don't think that waiting for 120 ticks should give a warning as long as the character is running - until the character stands still I expect that the developer have chosen to do it - and it isn't a waste of ticks (unless the character is standing still).


[Event Hubs] Improve EventPosition class

As per the API proposal in #2718 (comment), Improve EventPosition class.
This PR updates the following:

Move getExpression() out of EventPosition class.
change startOfStream -> firstAvailableEvent and endOfStream -> newEventsOnly
firstAvailableEvent and endOfStream  return as a static properties on the class.

This fixes #3174


Update trunk with upstream version 2.1.3

This PR updates trunk with upstream version 2.1.3 (commit: https://github.com/software-mansion/react-native-gesture-handler/commit/1c9198d0321a2b0d41da283e301a1d3014a4a1c3).

Closing this PR as version 2.2.0 was already merged into trunk in https://github.com/wordpress-mobile/react-native-gesture-handler/pull/14.


Insertrow modal placeholders have commas when dynamically generating the table.

This is probably because the  gets its child through a map operation.
this.props.keys.map( (key) =>  <TableHeaderColumn key={key} width={key==="sno"?"50px":"150px"} dataField={key}> {key.toUpperCase()} </TableHeaderColumn> )

ok, I'll check this out, but I'm pretty busy so maybe need some time.

Also, I found another bug on mobile. I'll attach the screenshot. The new button overlaps on the insert modal.

@Arvin6 You can use the headerText prop to specify your placeholders so that they won't have commas.
this.props.keys.map( (key) =>
    <TableHeaderColumn
        key={key}
        width={key==="sno"?"50px":"150px"}
        dataField={key}
        headerText={key.toUpperCase()}>
        {key.toUpperCase()}
    </TableHeaderColumn>
)

@rfinck thank you so much. It worked. :)

Fixed on v3.1.6.


Rename app in logs

Could you extend this PR to replace the package name as well, related to issue #119

will merge first, rest can be done in 1.4


KeyError: 'is_user_system_message'

Describe the bug
I was getting a crash part way through the run
Writing Markdown 📄 files:   3%|██                                                                           | 12/452 [00:00<00:00, 833.60it/s] Traceback (most recent call last):   File "/home/me/tools/code/public/chatgpt-history-export-to-md/main.py", line 227, in <module>     main()   File "/home/me/tools/code/public/chatgpt-history-export-to-md/main.py", line 103, in main     conversation.save_to_dir(markdown_folder)   File "/home/me/tools/code/public/chatgpt-history-export-to-md/models/conversation.py", line 329, in save_to_dir     file.write(self.markdown)   File "/home/me/tools/code/public/chatgpt-history-export-to-md/models/conversation.py", line 286, in markdown     markdown = self.yaml_header   File "/home/me/tools/code/public/chatgpt-history-export-to-md/models/conversation.py", line 256, in yaml_header     "custom_instructions": self.custom_instructions,   File "/home/ivan/tools/code/public/chatgpt-history-export-to-md/models/conversation.py", line 239, in custom_instructions     if context_message and context_message.metadata["is_user_system_message"]: KeyError: 'is_user_system_message'
To Reproduce
I think some of my data is missing a "is_user_system_message" key?
Additional context
i updated
line 239 in conversation.py to use the get method so it doesnt fail is that key is not there.
if context_message and context_message.metadata.get("is_user_system_message", None):

Thanks for pointing that out. These data inconsistencies are kinda odd ngl.
I implemented your fix ! Lemme know if you have further issues.


docs: :building_construction: internal flow diagram for write_package_properties()

Description
Closes #638
Reviewer Focus

This PR needs a quick review.
It's pretty simple :)

Very pretty, and sensible.


Implement Dependency RFC 0004 for rust

Implement Dependency Management RFC Phase 1 for rust. Check out the RFC for more details and background.
1. Determine dependency source strategy

Background
When possible, dependencies should be used directly from upstream, rather than undergoing any additional compilation or modifications performed by Paketo-maintained code. For each dependency, the corresponding buildpack maintainer group will decide if the dependency can be used directly from upstream, and must identify the location from which the dependency will be pulled from. Some of the Paketo Java buildpacks perform directory stripping during the buildpack build process itself. This could be a viable alternative to performing directory modifications during the dependency management process for maintainers to consider.

[ ] Determine whether the dependency can be used directly from its upstream, rather than undergoing additional compilation/modification.

If using directly from upstream, note where dependencies will come from
If compiled or modified, note the decision for this

[ ] Document decision in a language-family level RFC. This decision can be combined in the RFC with the decision made for other buildpacks in the language family.

2. Version retrieval and metadata generation code
Please refer to the retrieval RFC section on version retrieval for details around the API and background.

[ ] Create retrieval code for the dependency

Will live in the buildpack under: dependency/retrieval/
Per the RFC, this will be a combination of (1) discovering new dependency versions based on the buildpack.toml, and (2) generating metadata for each new version.
Note : Per the RFC caveat - if the dependency is to be compiled, the SHA256 and URI field from the metadata should be omitted in this step.

3. Compilation code (if needed)
If the dependency will be compiled/modified, then refer to the compilation RFC section for API details and background.

[ ] Create compilation action

Will live in the buildpack under: dependency/actions/compile/

4. Dependency testing (optional)
It's up to maintainer discretion if the dependency will be tested. It's recommended to test the dependency if it's been compiled. Check out the testing RFC section for details.

[ ] Add tests for dependency

Will live in the buildpack under: dependency/test.

5. Makefile Setup
When using the generalized workflows for dependency management down the line, version retrieval and dependency testing will be executed via a Makefile in order to provide the workflow a standardized way to run the code, regardless of what language it was written in. Check out the RFC section for what this should look like.

[ ] Add Makefile

Will live in the buildpack at: dependency/Makefile

6. Leveraging the new code
This issue serves to set up all the main logic for the dependency management. The work to actually leverage this code and migrate off of the dep-server will be completed in a separate issue once workflows and infrastructure is set up.

I have submitted #200 which changes where paketo-community/rust-dist pulls the Rust dependency. That PR will change it to pull from upstream/official Rust. That should work just fine, it'll just be a slightly larger download as I believe the previous Paketo builds were stripping out some unnecessary stuff. Probably better to pull from upstream and not remove things though, for completeness. If someone complains we could look at doing something more customized.

As a user of rust, I think it makes sense to default to what is provided upstream, and defer any decisions about stripping out components until we have feedback that it would be valuable.


Missing required property 'azkaban.native.lib'

I see this issue was fixed but error still there after fresh download.
(Download git clone as of 7:00pm mtn time)
Does someone have a detailed way to fix ?
azkaban.utils.UndefinedPropertyException: Missing required property 'azkaban.native.lib'
at azkaban.utils.Props.getString(Props.java:478)
at azkaban.jobExecutor.ProcessJob.run(ProcessJob.java:120)
at azkaban.execapp.JobRunner.runJob(JobRunner.java:595)
at azkaban.execapp.JobRunner.run(JobRunner.java:446)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
22-11-2016 18:56:58 MST test ERROR - Missing required property 'azkaban.native.lib' cause: null
22-11-2016 18:56:58 MST test INFO - Finishing job test attempt: 0 at<PHONE_NUMBER>014 with status FAILED

see this pull request. pull/827

Im not using the solo server download. I am using tar from the build off the directory azkaban-master/azkaban-exec-server/build/distributions
which I copied to another host to run as my executor.

Correct that fixed it.


Do not display output when assigned to var/val

this especially applies to eager DataFrame evaluation, which by definition are lazy.. so either:

when assigned to val/var the output shouldn't be created...
make DataFrame display functions to be called explicitly only

regarding output, there is a "simple" way but not perfect, just terminate the cell with a unit () and nothing will be printed -- it's poor I know

fixed in #552


hyperkube/coreos versions compatibly

The hyperkube entrypoint is /hyperkube in version 1.2
Seems to cause issues with kube-wrapper for our version of coreos

Closing this as it's /hyperkube for a reason because the CoreOS guys have deliberately moved that there and installed some relative dependencies in their container. They've since released a 1.2 container we just need to track https://quay.io/repository/coreos/hyperkube?tag=latest&tab=tags


Add ccache to the pipeline to speed it up

In Pull Requests, this uses the cache from the parent commit from main the first time, and re-uses its own cache in case of a rebuild of the same commit.
In main this uses the previous commit as a starting point as well. There's a 200MB ccache limit so old objects are evicted before we use up too much space in the per-repo GitHub cache, so multiple build targets can have space for their caches.

I don't know how to review this but I do welcome this improvement!


Make sure that any rust code in the documentation compiles

We have lots of documentation for wasm-bindgen in markdown. There are a fair amount of code snippets in rust. As wasm-bindgen changes, we want to make sure that these code snippets stay up to date.
The best way to do that seems to be with something like the following crate https://github.com/budziq/rust-skeptic

Due to this issue, compiling the start example failed.

@huangjj27 ah ok thanks! What you'll need to do to update that is:

Run rustup update nightly
Run cargo update
Change #![feature(proc_macro, wasm_custom_section, wasm_import_module)] to #![feature(use_extern_macros)]

and you should be good to go!

Ah, and also likely run cargo install -f wasm-bindgen-cli

Thx @alexcrichton,
I had the exact same problem as @huangjj27 and your comment resolved my issue.

@alexcrichton I'm sorry that I forgot to give you the feedback that your advice solved my issue that day, And thank you very much!

@alexcrichton ,
When I run cargo install -f wasm-bindgen-cli, some errors has occurred.
I already run rustup update nightly and cargo update.How should I do resolve this errors?
   .....
   Compiling filetime v0.2.1
   Compiling log v0.4.5
   Compiling unicode-bidi v0.3.4
   Compiling num_cpus v1.8.0
error[E0658]: non-reference pattern used to match a reference (see issue #42640)
   --> C:\Users\XXXXX\.cargo\registry\src\github.com-1ecc6299db9ec823\parity-wasm-0.32.0\src\builder\module.rs:233:86
    |
233 |                 if let Some(existing_entry) = self.module.types.types().iter().enumerate().find(|(_idx, t)| {
    |                                                                                ^^^^^^^^^ help: consider using a reference: `&(_idx, t)`

error[E0658]: non-reference pattern used to match a reference (see issue #42640)
   --> C:\Users\XXXXX\.cargo\registry\src\github.com-1ecc6299db9ec823\parity-wasm-0.32.0\src\builder\module.rs:234:10
    |
234 |                     let elements::Type::Function(ref existing) = t;
    |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using a reference: `&elements::Type::Function(ref existing)`

error: aborting due to 2 previous errors

error: Could not compile `parity-wasm`.
warning: build failed, waiting for other jobs to finish...
error: failed to compile `wasm-bindgen-cli v0.2.22`, intermediate artifacts can be found at `C:\Users\XXXXX\AppData\Local\Temp\cargo-install.XzPFE7jGIHmS`

Caused by:
  build failed

I'm gonna close this because I think that we're doing a pretty good job on this front wrt all of the examples, and otherwise I think it's largely infeasible to do for all the other small snippets here and there. That being said PRs are of course always welcome to fix any bugs we have!


Table selector won't work if table name is null

https://github.com/apache/pinot/blob/d1d2ddbc116f6dddbefdf1e69dd409434eaae0e4/pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/Connection.java#L86-L94

Thanks for reporting the issue.
If table name is not provided, we should parse the query and extract the table name from the query. This requires pulling in the CalciteSqlParser class, which is currently in pinot-common module.
We can either:

Make pinot-client depend on pinot-common
Pull CalciteSqlParser into pinot-spi

cc @xiangfu0

Assuming we are okay with these dependencies in pinot-client @xiangfu0 , labeling this as beginner as it looks straightforward enough to fix. Lmk if it is not.

I am planning to pick up this task.

I am planning to pick this up making the pinot-client depend on pinot-common.

Hey @npawar @Jackie-Jiang 👋 , can you please assign someone to review this PR 🙏

The PR is on the right track.
One thing worth mentioning is that the get table name is the best effort.
And if the tableName is null, the brokerSelector should return any broker. E.g. select 1 is usually used for testing the db connection by many external systems. This kind of query has no table name, but also a valid query.

Solved with #9902


⚠ Bofrost.si has degraded performance

In d8e5d4e, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 3801 ms

Resolved: Bofrost.si performance has improved in 851ca04 after 7 minutes.


MST Test Suite integration

The tests take ~30 seconds to run on my M1 pro macbook (with failing asserts commented out) - not too bad, but I anticipate adding a lot more tests to the suite.
Due to the slowness, I deliberately named the test file such that python -m unittest discover won't find it. It can be manually invoked via python -m unittest arroba.tests.mst_test_suite.
The test suite itself is still in a state of flux, hence this is a draft PR for now.
Currently, it only tests basic MST diffing (detection of created, updated, deleted records, and new and deleted MST blocks)
Unfortunately the tests currently don't pass - Diff.new_cids is flakey (which you note in one of your existing diff tests). Everything else seems good though! (due to the symmetry of diffing, you could in theory compute the reverse-diff and take Diff.removed_cids to get a correct result - but presumably you want to fix it properly heh)

Note to self, I should probably add something to the README about how to init the submodule

Oh and I like separating it out from unittest discover, but I'd happily run it in CI, 30s is totally fine. Example past runs: https://app.circleci.com/pipelines/github/snarfed/arroba

ah yes, subTest is what I was looking for. Thanks for the feedback!
Current stats:
Ran 2 tests in 79.933s

FAILED (failures=30596)

(some of these are plausibly bugs in the test cases themselves, since I generated them with my own code which has low test coverage itself!)

Thanks! I'm happy to merge this whenever. Lots more to do, sure, but it's contained and harmless and a great start as is. Up to you though!

Discord thread: https://discord.com/channels/1097580399187738645/1316355962633982015

Cool, feel free to merge it now. When I add new tests it should just be a case of updating the submodule, and if I add new test types I can PR  those in too.
I'll think about making it a pip-installable package in the future, but submodule definitely works for now.
I just bumped the submodule version, no changes to the tests themselves but it includes my visualisation script.

Thanks again, this is so great!

[cracks knuckles] time to get to work


[Err] [Plugin.hh:212] Failed to load plugin libgazebo_ros_control.so: libgazebo_ros_control.so: cannot open shared object file: No suchfile or directory

I am following this guide from articulated robotics https://articulatedrobotics.xyz/mobile-robot-12-ros2-control/.
i've already installed.
sudo apt install ros-humble-ros2-control ros-humble-ros2-controllers ros-humble-gazebo-ros2-control.
but during launch i've got this error.
[Err] [Plugin.hh:212] Failed to load plugin libgazebo_ros_control.so: libgazebo_ros_control.so: cannot open shared object file: No suchfile or directory.
and
[Err] [Model.cc:1160] Exception occured in the Load function of plugin with name[gazebo_ros2_control] and filename[libgazebo_ros2_control.so]. This plugin will not run.
i've checked the file using the command
ls /usr/lib/x86_64-linux-gnu/gazebo-11/plugins | grep libgazebo_ros2_control.so
but that file not found in plugin directory.
Is this not available for ros2 humble?.
how do i solve it.
where can i found this file to install it manually?.

log.
Warning [parser_urdf.cc:3027] urdf2sdf: revolute joint [Tip_Gripper_Idol_Joint] with limits: lowStop[3.14] > highStop[-3.14], switching the two.
[INFO] [1679715236.387070533] [gazebo_ros2_control]: Loading gazebo_ros2_control plugin
[INFO] [1679715236.404079925] [gazebo_ros2_control]: Starting gazebo_ros2_control plugin in namespace: /
[INFO] [1679715236.405457275] [gazebo_ros2_control]: Starting gazebo_ros2_control plugin in ros 2 node: gazebo_ros2_control
[INFO] [1679715236.405979906] [gazebo_ros2_control]: Loading parameter file /home/user/ros2_moveo/install/moveo_moveit_config/share/moveo_moveit_config/config/ros2_controllers.yaml

[INFO] [1679715236.416286275] [gazebo_ros2_control]: connected to service!! robot_state_publisher
[INFO] [1679715296.511643774] [gazebo_ros2_control]: Recieved urdf from param server, parsing...
[INFO] [1679715296.593671949] [gazebo_ros2_control]: Loading joint: Joint_1
[INFO] [1679715296.593759480] [gazebo_ros2_control]:    State:
[INFO] [1679715296.594361962] [gazebo_ros2_control]:             position
[INFO] [1679715296.594404703] [gazebo_ros2_control]:                     found initial value: 0.000000
[INFO] [1679715296.594470460] [gazebo_ros2_control]:             velocity
[INFO] [1679715296.594500998] [gazebo_ros2_control]:    Command:
[INFO] [1679715296.594520993] [gazebo_ros2_control]:             position
[INFO] [1679715296.594601784] [gazebo_ros2_control]: Loading joint: Joint_2
[INFO] [1679715296.594626381] [gazebo_ros2_control]:    State:
[INFO] [1679715296.594638847] [gazebo_ros2_control]:             position
[INFO] [1679715296.594650680] [gazebo_ros2_control]:                     found initial value: 0.000000
[INFO] [1679715296.594664901] [gazebo_ros2_control]:             velocity
[INFO] [1679715296.594676526] [gazebo_ros2_control]:    Command:
[INFO] [1679715296.594692974] [gazebo_ros2_control]:             position
[INFO] [1679715296.594747487] [gazebo_ros2_control]: Loading joint: Joint_3
[INFO] [1679715296.594783566] [gazebo_ros2_control]:    State:
[INFO] [1679715296.594816336] [gazebo_ros2_control]:             position
[INFO] [1679715296.594849622] [gazebo_ros2_control]:                     found initial value: 0.000000
[INFO] [1679715296.594886816] [gazebo_ros2_control]:             velocity
[INFO] [1679715296.594921471] [gazebo_ros2_control]:    Command:
[INFO] [1679715296.594955570] [gazebo_ros2_control]:             position
[INFO] [1679715296.594996854] [gazebo_ros2_control]: Loading joint: Joint_4
[INFO] [1679715296.595030232] [gazebo_ros2_control]:    State:
[INFO] [1679715296.595057229] [gazebo_ros2_control]:             position
[INFO] [1679715296.595357483] [gazebo_ros2_control]:                     found initial value: 0.000000
[INFO] [1679715296.595393695] [gazebo_ros2_control]:             velocity
[INFO] [1679715296.595415972] [gazebo_ros2_control]:    Command:
[INFO] [1679715296.595433839] [gazebo_ros2_control]:             position
[INFO] [1679715296.595468058] [gazebo_ros2_control]: Loading joint: Joint_5
[INFO] [1679715296.595491367] [gazebo_ros2_control]:    State:
[INFO] [1679715296.595505004] [gazebo_ros2_control]:             position
[INFO] [1679715296.595551399] [gazebo_ros2_control]:                     found initial value: 0.000000
[INFO] [1679715296.595609503] [gazebo_ros2_control]:             velocity
[INFO] [1679715296.595657636] [gazebo_ros2_control]:    Command:
[INFO] [1679715296.595756073] [gazebo_ros2_control]:             position
[INFO] [1679715296.595945970] [resource_manager]: Initialize hardware 'GazeboSystem'
[INFO] [1679715296.596234585] [resource_manager]: Successful initialization of hardware 'GazeboSystem'
[INFO] [1679715296.596544645] [resource_manager]: 'configure' hardware 'GazeboSystem'
[INFO] [1679715296.596634922] [resource_manager]: Successful 'configure' of hardware 'GazeboSystem'
[INFO] [1679715296.596668183] [resource_manager]: 'activate' hardware 'GazeboSystem'
[INFO] [1679715296.596691468] [resource_manager]: Successful 'activate' of hardware 'GazeboSystem'
[Err] [Model.cc:1160] Exception occured in the Load function of plugin with name[gazebo_ros2_control] and filename[libgazebo_ros2_control.so]. This plugin will not run.
[Wrn] [Publisher.cc:135] Queue limit reached for topic /gazebo/default/user_camera/pose, deleting message. This warning is printed only once.

Tried to install from source but got error.
/home/user/ros3_ws/src/ /gazebo_ros2_control/src/gazebo_ros2_control_plugin.cpp: In member function 'virtual void gazebo_ros2_control::GazeboRosControlPlugin::Load(gazebo::physics::ModelPtr, sdf::v9::ElementPtr)':
/home/user/ros3_ws/src/ /gazebo_ros2_control/src/gazebo_ros2_control_plugin.cpp:290:67: error: '__gnu_cxx::__alloc_traits<std::allocator<hardware_interface::HardwareInfo>, hardware_interface::HardwareInfo>::value_type' {aka 'struct hardware_interface::HardwareInfo'} has no member named 'hardware_plugin_name'
  290 |     std::string robot_hw_sim_type_str_ = control_hardware_info[i].hardware_plugin_name;
      |                                                                   ^~~~~~~~~~~~~~~~~~~~
gmake[2]: *** [CMakeFiles/gazebo_ros2_control.dir/build.make:76: CMakeFiles/gazebo_ros2_control.dir/src/gazebo_ros2_control_plugin.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:139: CMakeFiles/gazebo_ros2_control.dir/all] Error 2
gmake: *** [Makefile:146: all] Error 2
---
Failed   <<< gazebo_ros2_control [2min 0s, exited with code 2]

Well something is definitely off as you should not be seeing "[Err] [Plugin.hh:212] Failed to load plugin libgazebo_ros_control.so:" in this context at all.
I suggest talking to whoever wrote that tutorial.


handle serviceentry delete

Fixes https://github.com/istio/istio/issues/19961
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@howardjohn would it be possible to test this in your env with your test case? If it works, I will add tests.

@ramaraochavali: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
cf2dc24c9790d143c57ce7b6ee608e66e2aee1a0
link
/test unit-tests_istio

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

@ramaraochavali: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
657a6981ce6fdd9058f6e59049683479479197e6
link
/test unit-tests_istio

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

@ramaraochavali: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
39e650ab67671751cbad2101e94cdc925a19cb9e
link
/test unit-tests_istio

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

/test unit-tests_istio

@howardjohn added tests as well. PTAL

@ramaraochavali: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
fa00e5f173e60d59bcd99481914bda44905737fc
link
/test unit-tests_istio

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

@ramaraochavali: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
fa00e5f173e60d59bcd99481914bda44905737fc
link
/test unit-tests_istio

e2e-bookInfoTests-envoyv2-v1alpha3_istio
fa00e5f173e60d59bcd99481914bda44905737fc
link
/test e2e-bookInfoTests-envoyv2-v1alpha3_istio

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

/retest

I have one thought in mind: is it feasible we make both ServceEntry and kube Service/Endpoints handler consistent to prevent diverge?

I have one thought in mind: is it feasible we make both ServceEntry and kube Service/Endpoints handler consistent to prevent diverge?

I agree this would be useful.


StringComparison

Related Issue
(#1105)
Proposed Changes
Added a file comparing two strings .
Most string comparison algorithms compare the strings character by character. C++ provides us some built in operators like == and != to compare strings along with strcmp and compare functions.

Additional Info

Any additional information or context

Checklist

[x] ✅ My code follows the code style of this project.
[x] 📝 My change requires a change to the README.
[x] 🎀 I have updated the README accordingly.
[x] 👀 I have read the CONTRIBUTING document.
[x] ✨ I have added tests to cover my changes.
[x] 🚩 All new and existing tests passed.
[x] 🌟 ed the repo

Algorithm:

Declare two strings.
take the input of two strings.
comparing the two strings we can use any of the three ways mentioned below:

i) strcmp() inbuilt function
ii) compare() inbuilt function,
iii) C++ relational operators ( ‘==’, ‘!=’)

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Output Screenshots

when strings are equal

when strings are not equal

@Lakhankumawat sir you told me  to change the file name , but i am not able to make that change on the same PR so i have created new PR and made the changes.
https://github.com/Lakhankumawat/LearnCPP/pull/1112#discussion_r851043110

@all-contributors please add @aparnamittal code ,


Mycobank link

Creates links to MycoBank from Names and Observations
Addresses Pivotal story #61044172
Caveats:
(1) I updated the app_banner_box and observer/news in en.txt.  Please feel free to modify.  More important, I don't know how to make localizations update.  (They should initially be updated to be === English.)
(2) The links are to a search in MycoBank.  If there's an official MycoBank translation for the MO user's language, the results are in that translation.
The MO method uses a custom array to translate from MO language to MycoBank.  Another approach is to internationalize and localize this using I18n.  I didn'tt do that because (1) MO users should not be able to change the localizations, and I don't know how to avoid that; (2) the custom array is more flexible in that it will automatically deal with any new languages added to MO.
(3) I changed travis.yml to make Travis builds pass.  See [MO Developers] strange Travis build failures in image processing
Some issues with the MycoBank search results.
(a) The hits are all taxa whose name includes the text of the searched taxa.  The biggest annoyance is the search for genus, as the hits include at least all subordinate taxa of that genus.
(b) I do not see how to search for a rank between genus and species (e.g., subgenus).  So for those ranks I just search for genus.  This is doubly annoying because of (a) above.
All-in-all, less than ideal, but it's a big improvement, at least for my use case.  I hope it also will save MO users the trouble of trying to paste MycoBank links into descriptions, comments, etc.

Please hold up on merging this.  There is something very strange going on:  when I pull this branch back down to my local machine, then run the tests on the latest VM, I get 11 failures.  I will post a longer message in the MO-Developers forum.

This PR is now ready to go.
Sorry about all the duplicate commits.  (I'm doing something wrong in my Git GUI when I rebase.)

Thanks for all the Rubocop cleanup in addition to the fix!


[Feature] Expand testCaseIDMatcher to work with annotations

Hello,
I just seen that #76 was resolved, thanks for the improvement !
As I realised that it was very close to the initial purpose of #55, except it would focus on annotations rather than tags, would it be possible to add the support for it please ?
It would be much less visible on the main page of the test report, but would still be visible when reviewing a specific test, with the possibility to directly add a clickable link to the test plan.
Example:
Using the new matcher
testCaseIdMatcher: azureTestID,

If needed, a parameter to specify where to search ID between tag, titles or annotations could be added
testCaseIdZone: ['title', 'annotation'],

Specifying test ID using annotation
test("Title", {
  tag: ['@tag1', '@tag2'],
  annotation: [{ type: 'azureTestID', description: '42' }]
  }, async ({ page }) => {
  //perform actions
});

@Az8th
I have published beta version, can you check it?
<EMAIL_ADDRESS>
also you can read README  about version

Waow ! Nice work, will test it today ;)

@alexneo2003 Works like a charm, thanks for your work !
Just wondering if testCaseIdZone should have separate title and tags values.
I initially thought that due to the old way of declaring them (in the title with an @), they were not separated, but it seems that the API now include a separate testInfo.tags property !
There is only one thing that I can't recover from my prior setup though, in order to have the Tests tab in the pipeline run, I use the following task in my pipeline file:
 - task: PublishTestResults@2
    displayName: "Tests reports publication on Azure"
    condition: eq(variables['initDone'], 'true')
    inputs:
      searchFolder: "Reports"
      testResultsFiles: "e2e-junit-results.xml"
      testResultsFormat: "JUnit"
      mergeTestResults: true
      failTaskOnFailedTests: false
      testRunTitle: "${{parameters.project}} Playwright tests"

And use this reporter in the Playwright config file : ['junit', { outputFile: 'Reports/e2e-junit-results.xml' }]
Nevetheless, as this task also creates a test run, I removed it to avoid duplicates, but the tab is no longer visible on my pipeline run. What is strange is that both Test results tab acts the same when comparing Test run reports. Am I missing something to make this appear, or is it not supported yet, and shall open a dedicated issue ?

Thanks for your answers, I understand the backward compatibility importance, feel free to close the issue when it is officialy released ;)
About the test run duplicate, I have an idea on how we could be able to fix this, I will open a separate issue to avoid polluting this one


fix: multiple husky print & prettier path

This is only fix issue for windows environment.

For husky is print multipe task. Ref: https://github.com/typicode/husky/issues/968#issuecomment-1176848345
For prettier is need to wrap with "..." if the path contain a space.

@all-contributors please add @hendraaagil for code


Configurable logging when no response is returned

We have an app that was relying on the behavior prior to #1875 / #1879 to pick and choose which HttpService were logged and which were ignored. (For this app, the routes corresponding to load balancer health checks are ignored, because they're pretty much just noise in the logs.)
val loggedServices: HttpService[F] = ???
val healthCheck: HttpService[F] = ???

BlazeBuilder[F]
  .bindHttp(8080, "<IP_ADDRESS>")
  .mountService(server.middleware.Logger(logHeaders = true, logBody = true)(loggedServices) <+> healthCheck, "/")
  .serve

Unfortunately, with 0.18.12, this no longer works—now all the noisy requests are logged, along with a corresponding “service returned None.” Do you have any thoughts on supporting the old behavior via a flag on Logger? I think the intent of the changes was good, but maybe this use case was overlooked.

Sorry, this one slipped through the cracks.
This is addressed in master by only logging on HttpApp.  Any solution would be 0.18 only.  Adding a flag is not likely to be binary compatible.  Is there anything we can do, short of a copy and paste?

Don't worry about the timing, if it was a higher priority I knew where to find you. 🙂
I'm not sure what you mean by a copy and paste. Do you mean duplicating the entire Logging middleware wholesale? If so, maybe I can just do that locally and not worry about it in 0.18. If not, could you elaborate?
I haven't really looked at the changes on master yet and couldn't figure out exactly what you meant about HttpApp from a brief browse-thru of the documentation. Could you elaborate a bit on that too, please?

Yes, I meant duplicating wholesale, if we couldn't figure out how to add the flag without breaking compatibility.
Does this help for HttpApp?

I think this is obsolete since the middleware was parameterized?  If not, please reopen.


Adding docker login to makefile

Signed-off-by: Lidor Ettinger<EMAIL_ADDRESS>
Description

This PR fixes #21
Notes to reviewers
Checklist

[] Signed commits.
[] PR title includes [<name of area>]:<description>

LGTM!


PHOENIX-1071 Get the phoenix-spark integration tests running.

Uses the BaseHBaseManagedTimeIT framework now for creating the
test cluster and setup/teardown.
Tested with Java 7u75 i386 on Ubuntu, and 7u40 x64 on OS X.

Good work @jmahonin . I have merged the files onto master and 4.x-HBase-0.98 branches.


Running a driver test on Fuchsia fails with NoSuchMethod exception

Repro instructions:

Apply the patch: https://github.com/flutter/flutter/pull/45623
In the folder: flutter/examples/stocks, run flutter drive -v -d $FUCHSIA_DEVICE_ID --target=test_driver/scroll_perf.dart

You should see the following errors:
[+1219 ms] Unhandled exception:
[   +2 ms] NoSuchMethodError: The method 'group' was called on null.
[        ] Receiver: null
[        ] Tried calling: group("scrolling performance test", Closure: () => Null, onPlatform: null, retry: null, skip: null, solo: false, tags: null, testOn: null, timeout: null)
[        ] #0      Object.noSuchMethod (dart:core-patch/object_patch.dart:53:5)
[        ] #1      group (package:test_api/test_api.dart:183:13)
[        ] #2      main (file:///usr/local/google/home/kaushikiska/Code/flutter/flutter/examples/stocks/test_driver/scroll_perf_test.dart:12:3)
[        ] #3      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:307:19)
[        ] #4      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)
[  +32 ms] Stopping application instance.
[        ] Stopping application.
[        ] executing: ssh -F /usr/local/google/home/kaushikiska/Code/fuchsia/out/default/ssh-keys/ssh_config <IP_ADDRESS> tiles_ctl list
[  +85 ms] Found 1 tiles:
           Tile key 2 url fuchsia-pkg://flutter_tool/stocks#meta/stocks.cmx size 1333.3x882.2x1000.0 (unfocusable)
[   +2 ms] executing: ssh -F /usr/local/google/home/kaushikiska/Code/fuchsia/out/default/ssh-keys/ssh_config <IP_ADDRESS> tiles_ctl remove 2
[  +55 ms] "flutter drive" took 18,235ms.
Driver tests failed: 255

#0      throwToolExit (package:flutter_tools/src/base/common.dart:28:3)
#1      _runTests (package:flutter_tools/src/commands/drive.dart:316:5)
<asynchronous suspension>
#2      DriveCommand.runCommand (package:flutter_tools/src/commands/drive.dart:137:23)
<asynchronous suspension>
#3      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:615:18)
<asynchronous suspension>
#4      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:515:33)
<asynchronous suspension>
#5      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:146:29)
#6      _rootRun (dart:async/zone.dart:1126:13)
#7      _CustomZone.run (dart:async/zone.dart:1023:19)
#8      _runZoned (dart:async/zone.dart:1518:10)
#9      runZoned (dart:async/zone.dart:1465:12)
#10     AppContext.run (package:flutter_tools/src/base/context.dart:145:18)
#11     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:505:20)
#12     CommandRunner.runCommand (package:args/command_runner.dart:197:27)
#13     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:413:21)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:146:29)
#15     _rootRun (dart:async/zone.dart:1126:13)
#16     _CustomZone.run (dart:async/zone.dart:1023:19)
#17     _runZoned (dart:async/zone.dart:1518:10)
#18     runZoned (dart:async/zone.dart:1465:12)
#19     AppContext.run (package:flutter_tools/src/base/context.dart:145:18)
#20     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:364:19)
#21     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:112:25)
#22     new Future.sync (dart:async/future.dart:224:31)
#23     CommandRunner.run (package:args/command_runner.dart:112:14)
#24     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:250:18)
#25     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:63:22)
#26     _rootRun (dart:async/zone.dart:1126:13)
#27     _CustomZone.run (dart:async/zone.dart:1023:19)
#28     _runZoned (dart:async/zone.dart:1518:10)
#29     runZoned (dart:async/zone.dart:1502:12)
#30     run.<anonymous closure> (package:flutter_tools/runner.dart:61:18)
<asynchronous suspension>
#31     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:146:29)
#32     _rootRun (dart:async/zone.dart:1126:13)
#33     _CustomZone.run (dart:async/zone.dart:1023:19)
#34     _runZoned (dart:async/zone.dart:1518:10)
#35     runZoned (dart:async/zone.dart:1465:12)
#36     AppContext.run (package:flutter_tools/src/base/context.dart:145:18)
#37     runInContext (package:flutter_tools/src/context_runner.dart:64:24)
#38     run (package:flutter_tools/runner.dart:50:10)
#39     main (package:flutter_tools/executable.dart:65:9)
#40     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:305:32)
#41     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)

cc: @zanderso , @blasten , @gw280

Seems like I hit https://github.com/flutter/flutter/issues/35940

Did a fix similar to the one for https://github.com/flutter/flutter/issues/35940 not work?

I only realized that this was not Fuchsia specific after I found the error. I fixed it in the same PR.


Nested MultiLineComment detection too loose

input:
#cs
# #cs
#
# #ce
#ce

output:
Line 5, column 4: Expected "#", any character, or end of line but end of input found.

Nested #cs tags must only be proceeded by white-space on the same line.

Giving the comment end tag rule a name should also be considered outside of this bug, to better clarify the error messages


🚀 Possibility to delete folder from browser

Feature Description
I wish i had the possibility to delete a folder from the Folder & Files section
to have a new section in the manage filesfolder.php side
to choose a existing folder/file/playlist and click on delete.
It would be also be easy to have all existing audiofolders with a checkbox so user can delete 2 or more folder att the same time
Now the only way to delete folders are to remote in with ftp and delete:
the playlist in:
/home/pi/RPi-Jukebox-RFID/playlist
and the folder in:
/home/pi/RPi-Jukebox-RFID/shared/audiofolders

Unless you can do it yourself, there is no active development for version 2.x.
If you wish you to see this feature in 3.x, just label it with future3.


Adding the "Design Add and Search Words Data Structure" .

what to add the Trie implementation .
could you please assign me ?

@ishika1501 please describe properly what you wish to do.


[TASK] Backup and restore benchmarking

What's the task? Please describe.
After changes made during v1.2.0, we want to benchmark the backup restore performance. Compute the time taken to create backup and restore different size of data.
Describe the items of the task (DoD, definition of done) you'd like.

[x] Backup creation time benchmarking
[x] Restore from backup time benchmarking

Hey @shuo-wu ,
Here is a benchmark result between node/pod and s3 bucket. By leveraging a neat script by dvassallo/s3-benchmark. With following parameters:
./s3-benchmark --bucket-name for-s3-benchmark --region ap-southeast-1 --payloads-max 12 -payloads-min 12, test results:
--- SETUP --------------------------------------------------------------------------------------------------------------------

Uploading 2 MB objects
 100% |████████████████████████████████████████|  [1s:0s]

--- BENCHMARK ----------------------------------------------------------------------------------------------------------------

Download performance with 2 MB objects (m5.2xlarge)
                           +-------------------------------------------------------------------------------------------------+
                           |            Time to First Byte (ms)             |            Time to Last Byte (ms)              |
+---------+----------------+------------------------------------------------+------------------------------------------------+
| Threads |     Throughput |  avg   min   p25   p50   p75   p90   p99   max |  avg   min   p25   p50   p75   p90   p99   max |
+---------+----------------+------------------------------------------------+------------------------------------------------+
|       8 |      69.9 MB/s |  118   105   107   109   110   113   430   448 |  228   197   200   202   203   207   999  1074 |
|       9 |      83.1 MB/s |  113   102   106   108   109   111   346   408 |  215   194   199   201   203   205   759  1043 |
|      10 |      91.6 MB/s |  113   103   107   108   110   111   399   408 |  217   195   200   202   204   206   944  1049 |
|      11 |      95.6 MB/s |  114   102   106   108   110   113   401   408 |  227   194   199   201   203   209   954  1042 |
|      12 |     108.4 MB/s |  115   104   107   108   109   111   387   408 |  219   196   199   201   203   205   959  1041 |
|      13 |     116.6 MB/s |  114   101   105   107   109   111   399   410 |  221   193   198   201   203   206   971  1044 |
|      14 |     123.3 MB/s |  116   103   107   109   110   112   408   412 |  224   195   200   203   205   207  1012  1063 |
|      15 |     131.7 MB/s |  117   104   107   109   110   113   404   431 |  225   196   200   203   205   208   958  1026 |
|      16 |     133.0 MB/s |  117   102   106   107   109   114   403   408 |  237   195   199   201   204   211   990  1048 |
+---------+----------------+------------------------------------------------+------------------------------------------------+

--- CLEANUP ------------------------------------------------------------------------------------------------------------------

Deleting any objects uploaded from i-0e5ac7e62666b4a1f
 100% |████████████████████████████████████████|  [2m11s:0s]

The bandwidth is 1Gi hence transferring 10Gi data would take roughly 1min30s ~ 2min. Since the data is generated by dev/urandom hence the compression does not contribute to reducing the transferring data size here but just slow down the backup progress.
Based on the compression investigation in https://github.com/longhorn/longhorn/issues/1409#issuecomment-948693046:
The total backup time (that includes the compression time) = the data transferring time * 3.
Hence the expected time for backing up 10Gi random data is around 6min. This matches the above test result.
Conclusion: Longhorn backup works as expected. We can improve the performance by reconsidering the compression in the future.

As for the restore, it's still unclear why the restore time varies. It can be affected by the network since the DO does not provide a guaranteed 1Gi bandwidth. Or there may be something wrong with the code. Compared with backup creation that is handled by one replica, restore always means that all replicas of a restore volume will download data from the server simultaneously from the server. Then both bandwidth and backup server type would affect the final result.
Here, there are the important factors we can be careful about anyway:

The backup server type should be changed when testing backup and restore.
The cluster & the backup server (bandwidth, CPU) for the restore should be the same as that for the backup.
The data are all from /dev/urandom.
For the restore, we can use 1-replica restore volume first. Then check if multiple-replica restore volume would have worse performance.

cc @innobead

As for the restore, it's still unclear why the restore time varies. It can be affected by the network since the DO does not provide a guaranteed 1Gi bandwidth. Or there may be something wrong with the code. Compared with backup creation that is handled by one replica, restore always means that all replicas of a restore volume will download data from the server simultaneously from the server. Then both bandwidth and backup server type would affect the final result.
Here, there are the important factors we can be careful about anyway:

The backup server type should be changed when testing backup and restore.
The cluster & the backup server (bandwidth, CPU) for the restore should be the same as that for the backup.
The data are all from /dev/urandom.
For the restore, we can use 1-replica restore volume first. Then check if multiple-replica restore volume would have worse performance.

The test would require more effort. Not sure if we need to do it now. cc @innobead

I think the concern is fair, but the priority can be low and @kaxing or QA can plan in the future.

@innobead Sure. And it’s better to rename this ticket then open another ticket for the restore test.

@innobead Sure. And it’s better to rename this ticket then open another ticket for the restore test.

@khushboo-rancher Please help create another one for the benchmark of backup & restore, but actually the part for backup was done already. Thanks.

Closing and let's track #3208


Issue/oak 10573 azure write timeout

https://issues.apache.org/jira/browse/OAK-10573

Now I see that AzureSegmentArchiveWriter, also has retry parameters. Need to align it with parameters introduced in this PR.
https://github.com/apache/jackrabbit-oak/blob/trunk/oak-segment-azure/src/main/java/org/apache/jackrabbit/oak/segment/azure/AzureSegmentArchiveWriter.java#L45

Now I see that AzureSegmentArchiveWriter, also has retry parameters. Need to align it with parameters introduced in this PR. https://github.com/apache/jackrabbit-oak/blob/trunk/oak-segment-azure/src/main/java/org/apache/jackrabbit/oak/segment/azure/AzureSegmentArchiveWriter.java#L45

The code in this PR does not need to be changed, since already existing parameters, azure.segment.archive.writer.retries.max  and azure.segment.archive.writer.retries.intervalMs are configuring Retrier, and in execute method there is always writeAccessController.checkWritingAllowed(); check.


Incompatibility with flutter_localizations because of intl dependency

After Flutter update to 3.10.0 (with Dart 3.0) I can't use dash_chat_2 with flutter_localizations or easy_localisations packages..
Because dash_chat_2 >=0.0.16 depends on intl ^0.18.1 and dash_chat_2 <0.0.16 depends on intl ^0.17.0, every version of dash_chat_2 requires intl ^0.17.0 or ^0.18.1.
And because every version of flutter_localizations from sdk depends on intl 0.18.0, flutter_localizations from sdk is incompatible with dash_chat_2.
So, because eschool_mobile depends on both flutter_localizations from sdk and dash_chat_2 ^0.0.15, version solving failed.
The strange thing is that in pubspec.yaml of dash_chat_2 the "intl: ^0.18.0" is written... Seems like one of the internal dash_chat_2 packages sets additional dependency to  intl ^0.18.1

See https://github.com/SebastienBtr/Dash-Chat-2/pull/49


ContinuousScalar is not supported in the SDK

As a user I would like to be able to create a ContinuousScalar and have it initialize the VariableData correctly (at the moment it is forced to creating a Pose2-format structure, which causes a BoundsError when it is loaded into the cloud).
TAC

Add ContinousScalar to the supported VariableTypes
Parametrize the VariableNodeData so that it initializes correctly for each type
Add unit tests to validate operation

This is in IIF right? If so, should work as long as you can produce the PackedVariable payload.

No this requires significant refactoring, I'll make these changes in a bit.


ENH: simplify minimum norm TFR code and imrpove readability

Only a small diff here, but red lines ;-)
In preparation for future extensions.
cc @Eric89GXL @mainakjas @agramfort ...

Hooray DRY! LGTM

thanks @dengemann


AdEventListener is defined multiple times.

Type com.google.ads.interactivemedia.v3.api.AdEvent$AdEventListener is defined multiple times.
Can see the AdEventListener interface in google ads library as well on react-native-video.
package com.google.ads.interactivemedia.v3.api;
import androidx.annotation.InspectableProperty;

public abstract class AdEvent {
    public abstract InspectableProperty getType();

    public interface AdEventListener {
        public void onAdEvent(AdEvent adEvent);
    }
}

google ads:
/Users/foolish/.gradle/caches/transforms-3/48958b293b3dc79594bf867ed7e1a7b5/transformed/jetified-interactivemedia-3.24.0.jar:com/google/ads/interactivemedia/v3/api/AdEvent$AdEventListener.class
react-native-video
/Users/foolish/.gradle/caches/transforms-3/48958b293b3dc79594bf867ed7e1a7b5/transformed/jetified-interactivemedia-3.24.0.jar:com/google/ads/interactivemedia/v3/api/AdEvent$AdEventListener.class
It is creating issue on release build with minify enabled. How to handle it?

Did you try a gradlew clean ?

Yep. No luck.

Do you enable ads ? Did you try rm -rf node_modules ?

We have enabled ads using another library which utilities the following SDK,
implementation 'com.google.ads.interactivemedia.v3:interactivemedia:3.24.0'
and ads aren't enabled from react-native-video
Also yes we have cleared the node modules and tried building.

@subhechhu As conditional compilation is not 'natural' in java, There is a stub defined in react native video to allow using ima sdk or not. If you enable ads insertion, the stub but be build instead of ima sdk.
So you have 2 choices I think:

either enable ads insertion in react native video (it will not be used if ou don't specify adsTagUrl)
do a patch to remove the stub (see files in : android/src/main/java/com/ )


Custom Leaflet image finder regex

this method for finding images fails when leaflet.js is not called leaflet.js. In mapbox.js land, we've settled on hardcoding this, but that approach breaks as soon as we want to distribute non-hosted, like as a zip download, bower, or etc.

We might want to consider using the error stack as the source for the filename of the leaflet library:
https://gist.github.com/plepe/3891980
https://gist.github.com/BR0kEN-/8620537

@IvanSanchez does it work on old IE?

See #3559 for an alternate approach using CSS properties.

Hi,
I have the same issue than @branonbarrett when minifying leaflet.js : "Couldn't autodetect L.Icon.Default.imagePath, set it manually.".
The only way is to define L.Icon.Default.imagePath manually, or do you think that it's going to be fixed on next release ?
Thanks

Closing; not seeing any actions out of this issue.


Java updates

@nadvolod first off, don't freak out; the removal of enums / strict method setters is temporary, we can pull that code back in as we need it. :)
This brings Java in parity with Ruby & PR #89 for Python. I think this PR is sufficient for beta for Java. Which isn't to say that we can't continue to put in the extra features before a release. I'm just trying to get us on equal footing first.
There was a fair bit in here that we were duplicating from Selenium and some overhead that I think is useful but not required in the short term. The big thing with respect to the Options class is that we are not doing what Selenium Options classes do. They maintain a Map of key value pairs and create custom getters/setters that read and write to that Map. This implementation is using lombok to set native Java field values, which means that it does not allow any non-known capabilities, but also, it means nothing has to be hard coded. The heavy lifting is managed with reflection in the toCapabilities method. Also, the setCapability method is currently private. I think it should stay private, but I'm open to differing opinions here, I'll discuss it more below.
What's Next (for ruby/python as well, but specifically looking at it from a Java standpoint):

Make it easy for users to know and choose acceptable values

Enums for everything with a known and limited set of options (discussion below)
Static methods: I don't understand the purpose of these if we do the enums correctly.

Is this effectively an alternative to enum? (I'm struggling with enums, so considering this as an alternative might be useful)
Is this mostly to support a Fluent pattern? (I don't love it, but if we want it we just need to make sure it is organized)
Does the implementation of this PR with all of its getters and setters affect how we think about these? (e.g. mixing withEdge and setBrowserVersion would give different results depending on order called)

Be smarter about Mac/Safari and IE/Windows combination validations (@joshmgrant this is the safari conversation we had on the Python PR; the default right now is to require users to be more explicit first before we do convenience things that might result in more maintenance costs)

Mocking the tests (I figured out ENV mocks, but didn't get far enough along with mockito to figure out how to ensure that the driver is getting sent the right things). It's not close enough to what I'm used to in Ruby for it to be trivial, so I just made sure the tests actually passed on my Sauce account. Not ideal; but honestly the session stuff right now is the least concerning bit of this code to me.

ok, enums.
I've tried several different ways to get this working and can't figure out a way to do it that doesn't break the current specs...
The main use case I'm interested in is being able to pull in serialized information (YAML, JSON, Property, XML) as a lookup rather than needing to create a large conditional for everything. In Ruby & Python we can initialize the options object with a Hash obtained from one of these sources. I don't think we should do that in Java. In Java I implemented a setCapabilities method that takes a Map instance. With the current code, what I need to do works. Using an enum makes it impossible to set a String value, which means we either have to maintain the Map that I said above I don't want to do, or we need to be able to translate things. We can hard code in these translations, but that is a huge maintenance cost. We can try to use reflection to do the translations, but after a couple different tries I couldn't get it to work.
For reference, I think the implementation we want would allow this:
getCapability("platformName")              // => null

setPlatformName(PLATFORM.MAC_SIERRA)
getPlatformName                            // => PLATFORM.MAC_SIERRA
getCapability("platformName")              // => "macOS 10.12"

setCapability("platformName", "Windows 10")
getCapability("platformName")              // => "Windows 10"
getPlatformName                            // => PLATFORM.WIN_10

Anyway, please provide feedback, I'm going to keep thinking the thoughts on the next step here.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 7

Complexity increasing per file
==============================
- java/src/main/java/com/saucelabs/simplesauce/SauceOptions.java  6
- java/src/main/java/com/saucelabs/simplesauce/SauceDataCenter.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1
- java/src/main/java/com/saucelabs/simplesauce/TestResult.java  1

Complexity decreasing per file
==============================
+ java/src/main/java/com/saucelabs/simplesauce/SauceSession.java  -2

Clones added
============
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceOptionsTest.java  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- java/src/main/java/com/saucelabs/simplesauce/SauceDataCenter.java  1
- java/src/main/java/com/saucelabs/simplesauce/SauceOptions.java  6
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1
- java/src/main/java/com/saucelabs/simplesauce/TestResult.java  1

Complexity decreasing per file
==============================
+ java/src/main/java/com/saucelabs/simplesauce/SauceSession.java  -2

Clones added
============
- java/src/test/java/com/saucelabs/simplesauce/SauceOptionsTest.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1

See the complete overview on Codacy

ironically my unit tests are failing when run in paralel.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- java/src/main/java/com/saucelabs/simplesauce/SauceDataCenter.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1
- java/src/main/java/com/saucelabs/simplesauce/TestResult.java  1
- java/src/main/java/com/saucelabs/simplesauce/SauceOptions.java  8

Complexity decreasing per file
==============================
+ java/src/main/java/com/saucelabs/simplesauce/SauceSession.java  -2

Clones added
============
- java/src/test/java/com/saucelabs/simplesauce/SauceOptionsTest.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- java/src/main/java/com/saucelabs/simplesauce/SauceOptions.java  8
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1
- java/src/main/java/com/saucelabs/simplesauce/TestResult.java  1
- java/src/main/java/com/saucelabs/simplesauce/SauceDataCenter.java  1

Complexity decreasing per file
==============================
+ java/src/main/java/com/saucelabs/simplesauce/SauceSession.java  -2

Clones added
============
- java/src/test/java/com/saucelabs/simplesauce/SauceOptionsTest.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Complexity increasing per file
==============================
- java/src/main/java/com/saucelabs/simplesauce/TestResult.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1
- java/src/main/java/com/saucelabs/simplesauce/SauceOptions.java  8
- java/src/main/java/com/saucelabs/simplesauce/SauceDataCenter.java  1

Complexity decreasing per file
==============================
+ java/src/main/java/com/saucelabs/simplesauce/SauceSession.java  -2

Clones added
============
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceOptionsTest.java  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Complexity increasing per file
==============================
- java/src/main/java/com/saucelabs/simplesauce/SauceDataCenter.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1
- java/src/main/java/com/saucelabs/simplesauce/TestResult.java  1
- java/src/main/java/com/saucelabs/simplesauce/SauceOptions.java  8

Complexity decreasing per file
==============================
+ java/src/main/java/com/saucelabs/simplesauce/SauceSession.java  -2

Clones added
============
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceOptionsTest.java  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 2

Complexity increasing per file
==============================
- java/src/main/java/com/saucelabs/simplesauce/TestResult.java  1
- java/src/main/java/com/saucelabs/simplesauce/SauceOptions.java  8
- java/src/main/java/com/saucelabs/simplesauce/SauceDataCenter.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1

Complexity decreasing per file
==============================
+ java/src/main/java/com/saucelabs/simplesauce/SauceSession.java  -2

Clones added
============
- java/src/test/java/com/saucelabs/simplesauce/SauceOptionsTest.java  1
- java/src/test/java/com/saucelabs/simplesauce/SauceSessionTest.java  1

See the complete overview on Codacy

Code from this PR is going to be broken into smaller chunks for separate consideration.


⚠ Faculté génie website (prod) has degraded performance

In 856b533, Faculté génie website (prod) (https://www.usherbrooke.ca/genie/) experienced degraded performance:

HTTP code: 200
Response time: 912 ms

Resolved: Faculté génie website (prod) performance has improved in dda1018.


Get price returns success : False

Hello, if i want to view the price of this item i get success : false, some other items give false too, not only this item. Any1 knows a fix?
http://steamcommunity.com/market/priceoverview/?currency=1&appid=730&market_hash_name=XM1014 | Blue Spruce

You must use item quality for example: http://steamcommunity.com/market/priceoverview/?currency=1&appid=730&market_hash_name=XM1014 | Blue Spruce (Minimal Wear)


[CSHARP-303] [CSHARP-304] Integration test expand

Expanding integration tests for CSHARP-303 and CSHARP-304

:+1: Merged!


Move TempRValueOpt from CopyForwarding.cpp -> TempRValueElimination.cpp.

The pass is standalone from CopyForwarding and is big enough to stand on its
own.
This changes is NFC beyond clang-format + xcode rename.

@swift-ci smoke test

Fixed a typo.

@swift-ci smoke test

@swift-ci smoke test

thanks... been meaning to do this