docs(performance): Add performance config documentation

Fixes #435

Any other looks? Want to get this in since Perf is in now.

Any other objections? Going to merge this.

Yeah, merge. More tweaks can be done in separate PRs.


Map Background Not Displaying In NeoDash Desktop

NeoDash version: 2.1.2
Neo4j Database version: 4.4.5 (Enterprise).

When putting a map in NeoDash Desktop, the map background doesn't display. For example:

But, in the web version of NeoDash, using the same exact Cypher Query, the Map loads properly. As such:

Please find out why the map is not appearing in the NeoDash Desktop Application
Thanks!

Hi @GQ16 ,
Thanks for reporting this bug. I've tried reproducing this locally but I'm not having the same issues.
Which version of Neo4j Desktop are you using? And which operating system are you on?
I know that the embedded web-app in Neo4j Desktop might work differently based on the OS, so that could be something to look into for us.
Thank you,
Niels

Hey @nielsdejong ,
I'm working on Neo4j Desktop 1.4.15 and my OS is Windows 10 Pro, below are more details about the PC:
Edition	Windows 10 Pro
Version	21H2
Installed on	4/4/2022
OS build	19044.1706
Experience	Windows Feature Experience Pack 120.2212.4170.0
Processor	Intel(R) Core(TM) i5-8350U CPU @ 1.70GHz   1.90 GHz
Installed RAM	16.0 GB (15.8 GB usable)
System type	64-bit operating system, x64-based processor
Hope this helps!
GQ

Hi @GQ16 - small update, I've been off for the past days so haven't had much time to look into this.
I'll get my hands on a Windows machine this friday and I'll try reproducing the bug, will post updates here.
Thank you for your patience,
Niels

hi @GQ16 - I regret informing you that I did not get the chance to look at this on Friday on a Windows machine. I'll see if I can get one through another means this week. In the meantime, if anyone else has a chance to check this, please let us know by posting here.

Tried reproducing this on a machine with Windows 10 Pro, and the latest versions of Neo4j, NeoDash, and Neo4j Desktop.
I'm able to see the map in both the browser and Neo4j Desktop versions, so difficult to tell what the issue is.
The only explanation I can think of is your connection to the online map provider (OpenStreetMap) is blocked by your network when using the Desktop app. Are you on a corporate network?

I am on a corporate network.

Can you access https://www.openstreetmap.org/?

Yes I can access the website from my corporate network.

@nielsdejong The latest NeoDash Desktop update solved the issue. Thanks for your correspondence!


Pull picture from gitlab info when creating portfolio

In portfolio creation script, pull the user's image from their gitlab info and push it to the repo.

Done!


Misleading error message when Selector filtering results in an empty set

What is your Test Scenario?

Query some existing elements in a DOM tree using Selector
Filter that selector by passing a callback always returning false and store the resulted selector in a separate variable
Try clicking on the filtered selector by using the variable from the previous step

What is the Current behavior?
An error message says:

The specified selector does not match any element in the DOM tree.

What is the Expected behavior?
The error message should be rephrased to reflect the fact that the filtered results are empty.
Upon the first look, one might think the issue is in the original selector, however, that's not the case because there are plenty of links on the TestCafe's main page.
What is your web application and your TestCafe test code?

Your complete test code (or attach your test files):

 import {Selector} from 'testcafe';

fixture('Demo for filter issue')
.beforeEach(t => t.navigateTo('https://devexpress.github.io/testcafe/'));

test('should provide correct failure message', async t => {
  const links = Selector('a');
  await t.expect(links.exists).ok();
  const filteredLinks = links.filter(() => false);
  await t.click(filteredLinks);
});

Your complete test report:

Running tests in:
 - Chrome 88.0.4324.192 / macOS 10.15.7
Demo for filter issue
✖ should provide correct failure message

The specified selector does not match any element in the DOM tree.

         | Selector('a')
       > |   .filter([function])

  Browser: Chrome 88.0.4324.192 / macOS 10.15.7

          5 |
          6 |test('should provide correct failure message', async t => {
          7 |  const links = Selector('a');
          8 |  await t.expect(links.exists).ok();
          9 |  const filteredLinks = links.filter(() => false);
       > 10 |  await t.click(filteredLinks);
         11 |});
         12 |

     at <anonymous>
  (/Users/vmoskalenko/Projects/test_cafe_filter_issue/test.spec.ts:10:11)
     at fulfilled
  (/Users/vmoskalenko/Projects/test_cafe_filter_issue/test.spec.ts:5:58)

1/1 failed (15s)

Steps to Reproduce:

Go to my website ...
Execute this command...
See the error...

Your Environment details:

testcafe version:  1.11.0
node.js version: v14.15.5
command-line arguments:  npx testcafe chrome test.spec.ts
browser name and version: Chrome 88.0.4324.192
platform and version: macOS 10.15.7

I agree that the error text seems to be misleading since it says "elements matching the selector are not found" instead of "the filter action returns an empty list". But it is correct for the entire selector chain because there are no elements that match all conditions on the page.
Also, please note that TestCafe's error output contains information about the failed part of the selector chain:

The '>' mark shows the place where the error occurs. In this case, it should be clear that no matching elements were found while the filter step of the selector's chain was executed.
Perhaps, the cause of the problem is that this mark is not very noticeable and we should emphasize it somehow.

For the short-term solution, the > arrow  could be definitely improved to something more noticeable.
For the long-term approach, though, it'd be great to have a handy message, precisely illustrating the filtering issue.
The aim is to spend as less time as possible when it comes to debugging a test, figuring out the real reason of the error.
At the moment, it might not be immediately obvious and might take a while to realize the filter (s), producing a Selector matching no elements in a DOM, is the actual source of the issue.

I registered a separate issue (https://github.com/DevExpress/testcafe/issues/6007) to collect all requests on this topic.


Data only

Institution/School Name
National Infotech College
Instiution/School Website
https://nationalinfotechcollege.edu.np/
Email Users
<EMAIL_ADDRESS>Please place an x between the square brackets if yes

[yes ] Students
[ ] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[ yes] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

I have added my college domain name.  please Verified it.


Won't open .m4v file

Helium is having issues opening .m4v files. Is this a bug? or is it just not supported?

Closing because t's a duplicate.

Duplicate of #129


Add LitElementLight

Splits the LitElement class into an abstract LitElementBase class, which is implemented by LitElement (renders to shadow root) and LitElementLight (renders to light dom). See https://github.com/Polymer/lit-element/issues/42 for use cases for LitElementLight.
I'm not sure about the name 'LitElementLight', but I could not think of anything else.
I think this results in a clearer API than overriding methods. It clearly explains the intent to developers, and leaves the project flexible for different strategies in the future. For example when/if https://github.com/w3c/webcomponents/issues/468 becomes a thing, LitElementLight will have a use case within shared components as well.
Fixes https://github.com/Polymer/lit-element/issues/42

I think this could be split up into separate files, but it would mean exposing multiple files and it seems like there is quite a lot in one file already. I'm not sure about the approach that should be taken, for instance which files should contain classString and styleString and which files should export html and svg from lit-html.

Sorry for the delay and thanks for the PR. We've elected to maintain the current way of customizing the rendering element (now by implementing createRenderRoot) as noted here.


[4.0.0] Migrated Prototype APIs with no GW environment configured give errors

Description
description
When there are prototype APIs with no gateway environment selected were there in the 2.6.0 setup, During 3.2.0 APIM revision related migration, environment name picked from registry artiafct has 'none' and it is used to create the APIRevisionDeployment object. Thus when deploying the revision, DB is updated with gateway as 'none' but no real API deployment takes place
suggested fix
In order to fix this in the migration client, can we log and skip revision deployment for Prototyped APIs with no gateways
Steps to Reproduce

In APIM 2.6.0 create a prototype api and deselect all gateway environments before saving it.
Migrate to APIM 4.0.0
Go to publisher, explore the prototype API and try to deploy it.

ERROR - ApisApiServiceImpl Error while adding retrieving API Revision for api id : 153ea023-1ac6-4e2c-b1e1-0f31f32f1a6b - Configured read only environment not found: non

Affected Component
APIM
Version
4.0.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

Need to fix in 4.1 and 4.x.


Does not complain about invalid arguments

cargo-supply-chain crates --nosucharg does not complain that the argument --nosucharg does not exist. The underlying pico-args does that, but apparently some logic we had to put on top to handle -- broke it.
This worked correctly prior to the migration to pico-args in #27

Fixed in #31


autocomplete.html contains

Bug, feature request, or proposal:
Change request
What is the expected behavior?
Angular 4 compliant code without warnings
What is the current behavior?
The following warning is logged on ng build --prod
The <template> element is deprecated. Use <ng-template> instead ("[WARNING ->]<template><div class="mat-autocomplete-panel" role="listbox" [id]="id" [ngClass]="_getClassList()" #p"): ng:///D:/personal/dev/projects/kidora/code/client/main-app/node_modules/@angular/material/autocomplete/autocomplete.html@0:0
Which versions of Angular, Material, OS, browsers are affected?
Angular 4
Material 2.0.0-beta.2

This has been fixed with SHA https://github.com/angular/material2/commit/cd55082a488e58a3f4948bba8e5d0a6771a57233. The warning will disappear with the next Material release.


[pulsarbot] Bot should not comment on closed/merged PRs

I noticed the Pulsar Bot comment on old PRs that have either been merged or closed. I don't think we should have the bot updating those old PRs since it just creates unnecessary notifications for contributors.
I'm not able to help add this feature. This issue is just meant to start a conversation.

This is a good catch! Currently, the docbot adds a doc label based on the PR description, when the PR description is incorrect, we still add a comment.
Note: For the doc label, we can only change the PR description, and then the docbot will check this and do the labeling.
I have an idea, if the PR description is incorrect and the state is merged or closed, we will no longer add a comment.

I have an idea, if the PR description is incorrect and the state is merged or closed, we will no longer add a comment.

Yes. Check the PR state in the bot or filter out in the workflow is OK.


Load a csx file from relative path into Jupyter Notebook, fixes #478

Makes this possible: https://github.com/strabu/dotnet-interactive-issue-repro
Would be great if we could have relative path-imports in VSCode-Notebooks!
greetings,
Alex

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: strabu sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

I broke the Linux-Build with my first commit and fixed/tested it on my MacBook later with https://github.com/dotnet/interactive/pull/1176/commits/5a5a652d6f67e1ed1e05eed28061c4216a188b0d

@colombod sorry, clicked the "re-request-button" by accident

To add one more thing to the WorkingDirectory-discussion, this (see screenshot) is what I can do with the change at the moment in VSCode.
Actually as a Notebook-User I would probably not want that behaviour.
I'd prefer my "magic" #load-Directive to give me the same result relative to my .ipynb-File, no matter what the CurrentDirectory is.
When I open VSCode one folder-level higher my "#load"-directives don't work any more with the current implementation.
I would have to change it and put the missing folder after "#load".
What I want as a user is: open my notebook with VSCode on any level (maybe it's in a monorepo some folders below)
and I want my Notebook to work. It should not be necessary for me as a consumer of the notebook, to know on which level I have to "Open Folder..." to make it work.
In my opinion these relative Paths have to be based on the location where the current Notebook-file is.
Is there a way to make it relative to .ipynb?

@strabu Could you rebase this onto main please?

@strabu Could you rebase this onto main please?

have done that, please check

Thanks. There are still a couple of outstanding comments. If you have any questions or want clarification on anything, please let us know. We'd love to get this merged.

Thanks. There are still a couple of outstanding comments. If you have any questions or want clarification on anything, please let us know. We'd love to get this merged.

@jonsequitur I've changed the small issues that were left from the tests and rebased onto main again.
In the 2 sample-Notebooks for C# and F# at https://github.com/strabu/dotnet-interactive-issue-repro
there is now a line on top:
Directory.SetCurrentDirectory(".."); //this should not mess up the following imports
...that demonstrates that a user cannot mess up the following #load-statements with relative paths.
In VSCode I can now Open Folder and point it to: C:\Users\Alex\source\repos\dotnet-interactive-issue-repro
and I can also do Open Folder and select: C:\Users\Alex\source\repos\dotnet-interactive-issue-repro\Notebook
in both cases the C# and F# Sample-Notebooks can import their scripts based on their relative paths.
So I'm now personally getting what I wanted initially as a VSCode-.NET-Interactive-Notebook-user.
To my understanding there are 2 things left from the feedback above that can be discussed further:

If we should make the change suggested by @brettfo related to:
var workingDirectory = _workspaceDirectory ?? Directory.GetCurrentDirectory();
If yes, the command ChangeWorkingDirectory needs to be called by the VSCode-Plugin which is not happening currently as far as I see. I did not wanted to make additional changes in Typescript-Code because I could accomplish my goal without doing so.

The discussion about changing the WorkingDirectory for F# and all it's implications and whether it's necessary now.

If we should make the change suggested by @brettfo related to:
var workingDirectory = _workspaceDirectory ?? Directory.GetCurrentDirectory();
If yes, the command ChangeWorkingDirectory needs to be called by the VSCode-Plugin which is not happening currently as far as I see. I did not wanted to make additional changes in Typescript-Code because I could accomplish my goal without doing so.

I'm not sure I understand the reasoning for one approach versus the other. @brettfo or @colombod?

The discussion about changing the WorkingDirectory for F# and all it's implications and whether it's necessary now.

I'd say this is out of scope for this change.

@strabu Sorry for the slow response here. Could you rebase this please?

@strabu Sorry for the slow response here. Could you rebase this please?

OK, have done that.


Drone/Slack: Corporate Proxy

Hi!
We currently use Drone in our company and it works just awesome. We also use Slack in our team and would like to notify a slack channel of the build status.
Currently, nothing happens with slack.
We have set it up this way:
notify:
  slack:
    username: $$SLACK_USER
    token: $$SLACK_TOKEN
    team: $$SLACK_TEAM
    channel: $$SLACK_CHANNEL
    on_started: true
    on_success: true
    on_failure: true

And the environment variables are set in the repository settings on Drone.
There is no output of the slack plugin. How can we have a more verbose logging output of Drone or the Plugin?
Does Drone/The Plugin System/The Slack plugin support a corporate proxy?
Our proxy is set via HTTP_PROXY/HTTPS_PROXYon the machine.

I can confirm the fix mentioned by @bradrydzewski worked for me. I've raised a PR to update the docs https://github.com/drone/docs/pull/31

Confirmed working. Closing issue


 Component

http://semantic-ui.com/elements/reveal.html

hey i would like to try tackling this component

Start with the CONTRIBUTING.md, you can hit me up on slack if you have any issues!

Are there any activity or I can take it in work?

Bump @Jlassos

Still working as time permits


Translations setup

The first setup for translations we had was quirky and custom built as Flutter had no proper or straightforward way to do translations.
Times have hopefully changed. Investigate approaches that allow crowd-sourcing the translations and implement.

https://proandroiddev.com/flutter-localization-step-by-step-30f95d06018d?gi=7cdae8d58ea9

The proof of concept must use .arb files which we can upload to translation platforms :-)

Done by @uiboy n @jozsefsallai


fix(select): unneeded top padding

mdc-select--no-label

I saw that solution in textarea. Anyway using the class is way better!
Changes as you suggested

seems stable now. wanna add to UI tests as well?

I've added ui-tests in this branch:
https://github.com/Vonage/vivid/pull/1133
Maybe we can merge the fix and I'll add the test there to prevent conflicts?


[Settings] FZ MDL2 icon fix

Summary of the Pull Request
Addressing #7258. The MDL2 icon in settings was not representing the Fluent icon. This PR updates the SVG code.

Old:

New:

PR Checklist

[X] Applies to #7258
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

Wouldn't this be a more accurate version of the Fluent Icon done in a monoline

@mdtauk That's a fair point. @crutkas agree?

Fair point, fine with swapping later too.

Updated to this one. I stuck with no corner radius, since MDL2 doesn't do this at the moment. Looks nicer! There are now no lines starting at a .5 position, so it doesn't look blurry anymore but crisp and sharp :)

Updated to this one. I stuck with no corner radius, since MDL2 doesn't do this at the moment. Looks nicer! There are now no lines starting at a .5 position, so it doesn't look blurry anymore but crisp and sharp :)

@niels9001 I beg to differ.

Updated to this one. I stuck with no corner radius, since MDL2 doesn't do this at the moment. Looks nicer! There are now no lines starting at a .5 position, so it doesn't look blurry anymore but crisp and sharp :)

@niels9001 I beg to differ.

@Jay-o-Way Please open a new issue. This PR has been merged for a while.

@Jay-o-Way Please open a new issue. This PR has been merged for a while.


Restore Purchase

In my app, I am managing the availability of products outside of the store, which means I display the item to the user if it is not listed as purchased in my app database.  Note: the product id in my app database and the store do match.  As a result, if the app is deleted or the user installs it on another device, I want to restore purchases when they click 'buy'.
To do so, I have wrapped the PurchaseAsync inside the GetPurchasesAsync if statement as follows.
//validate purchase has not already been made
var purchases = await CrossInAppBilling.Current.GetPurchasesAsync(ItemType.InAppPurchase);
if (purchases?.Any(p => p.ProductId == productId) ?? false)
{
	//Purchase restored
	await Download(file);  //restores purchase outside of the store
}
else
{
	//no purchases found.  try to purchase item
	var purchase = await CrossInAppBilling.Current.PurchaseAsync(productId, ItemType.InAppPurchase, "apppayload");
	if (purchase == null)
	{
		//Not purchased
		await DisplayAlert("Error", "Failed to make the purchase", "OK");
	}
	else
	{
		//Purchased!
		await Download(file);
	}
}

When I run tests with via sandbox test user, it does not recognize the previous purchase, so it tries to purchase it but then prompts say "This In-App Purchase has already been bought" and the product is not restored (or in my case downloaded).
Steps:

previously purchased the product
deleted the app on my phone and redeployed it
click on 'buy' for previously purchased product
product is not restored

Any thoughts on how to resolve this issue?  Thanks!
Version Number of Plugin: 1.1.03-beta
Device Tested On: iPhone 6 (iOS 10.2.1)

I believe that Sandbox users do not have previous purchases, as you can purchase them over and over and over again. I would try in TestFlight to validate.
This is the same logic that I use in my app and it worked in TestFlight, but not in sandbox.
Test that out and let me know.

Moved the app to TestFlight and experiencing the following:

consumable app purchased fine the first time, but does not allow for second purchase instead it says "This In-App Purchase has already been bought".
non-consumable app failed to download properly upon first purchase.  Press buy again and get "This In-App Purchase has already been bought".

So for iOS the Consumable and Non-Consumable are the exact same API, there is no actual difference in the API that I could tell so should be similar results
My test was to:

Open Test Flight App, purchase to non-consumable
Delete App
Reinstall from Test Flight
Call GetPurchases and it was found

I must have something screwed up in my code, because it is failing to restore (non-consumable) or purchase another (consumable) for me.

Installed app via Test Flight, purchased consumable and non-consumable product
Deleted App, Reinstalled from Test Flight
Clicked 'buy' on each product, which triggered code (as outlined in original issue message) and received notice "This In-App Purchase has already been bought".
The download that is triggered with successful purchase was not triggered through the above process

Do you know if it is outputting or throwing any errors?
I am simply calling: SKPaymentQueue.DefaultQueue.RestoreCompletedTransactions(); which should return all of them back.
I have had it ask me to log in a few times.
If you see that it was already bought for a non-consumable then it should restore correctly. I assume that trying to buy a consumable should buy one as you should be able to buy as many as you need.

A good trial here to debug the code is to download the source code and add the .csproj to your projects instead of installing the NuGet. This helped me a lot when creating the library.

Great suggestion.  It will probably be a few days before I get to it, as I am trying to debug a new issue (shown below) that is causing my app to crash, when I call OnAppearing() to refresh the view model.
Unhandled Exception:
System.NullReferenceException: Object reference not set to an instance of an object
at Xamarin.Forms.VisualElement.Finalize () [0x00040] in C:\BuildAgent3\work\ca3766cfc22354a1\Xamarin.Forms.Core\VisualElement.cs:794

Doing a clean and rebuild normally fixes the "System.NullReferenceException: Object reference not set to an instance of an object" error.

So I resolved my prior problem.  I didn't realize that Xamarin.Forms package updated to the -pre1 because the 'show pre-release versions' was checked when I installed InApp-Billing.
Anyway, with that out of the way, I've gone back to working on the original problem with no success.  I was about to install the .csproj as you suggested, but then I wasn't certain how to debug if my only means to test is via TestFlight.  Any feedback on that would be appreciated.
For reference, here is my code:
`		// Purchase expansion deck
async void buyClicked(object sender, EventArgs e)
{
// determine which deck was selected
Button btn = (Button)sender;
string file = (string)btn.CommandParameter;
var productId = "whatchasayin.deck." + file.ToLower();
		bool productCheck = await CheckPurchase(productId, file); //first check if already purchased

		if (!productCheck)
		{
			try
			{
				var connected = await CrossInAppBilling.Current.ConnectAsync();

				if (!connected)
				{
					//Couldn't connect
					await DisplayAlert("Connectivity Error", "Unable to connect to the internet.", "OK");
					return;
				}

				//determine if consumable or non-consumable product
				if (file == "Custom") // custom is a consumable product
				{
					var purchase = await CrossInAppBilling.Current.PurchaseAsync(productId, ItemType.InAppPurchase, "whatchasayin2017");

					if (purchase == null)
					{
						//Not purchased
						await DisplayAlert("Error", "Failed to make the purchase", "OK");
					}
					else
					{
						//Purchased!
						for (int c = 0; c < 10; c++)
						{
							Phrases addPhrase = new Phrases();
							addPhrase.ID = 0;
							addPhrase.PhraseGroup = file + " Deck";
							addPhrase.PhraseStatement = "";
							addPhrase.Active = false;
							addPhrase.Used = false;
							App.Database.SavePhrase(addPhrase);
						}
						// after updating database, refresh screen and show success message
						OnAppearing();
						await DisplayAlert("New Custom Cards Added", "Successfully added 10 new cards to the '" + file + " Deck'.\n\n" +
										   "Click 'Edit' for Custom Deck to add/edit your phrases and activate/deactivate cards.", "OK");
					}

					//If iOS we are done, else try to consume
					if (Device.OS == TargetPlatform.iOS)
						return;

					var consumedItem = await CrossInAppBilling.Current.ConsumePurchaseAsync(purchase.ProductId, purchase.PurchaseToken);

					if (consumedItem != null)
					{
						//Consumed!!
						for (int c = 0; c < 10; c++)
						{
							Phrases addPhrase = new Phrases();
							addPhrase.ID = 0;
							addPhrase.PhraseGroup = file + " Deck";
							addPhrase.PhraseStatement = "";
							addPhrase.Active = false;
							addPhrase.Used = false;
							App.Database.SavePhrase(addPhrase);
						}
						// after updating database, refresh screen and show success message
						OnAppearing();
						await DisplayAlert("New Custom Cards Added", "Successfully added 10 new cards to the '" + file + " Deck'.\n\n" +
										   "Click 'Edit' for Custom Deck to add/edit your phrases and activate/deactivate cards.", "OK");
					}
				}
				else {

					var purchase = await CrossInAppBilling.Current.PurchaseAsync(productId, ItemType.InAppPurchase, "whatchasayin2017");

					if (purchase == null)
					{
						//Not purchased
						await DisplayAlert("Error", "Failed to make the purchase", "OK");
					}
					else
					{
						//Purchased!
						await Download(file); // download non-consumable product
					}
				}
			}

			catch (Exception ex)
			{
				await DisplayAlert("Error", ex.Message, "OK");
			}

			finally
			{
				//busy = false;
				await CrossInAppBilling.Current.DisconnectAsync();
			}
		}
	}

	async Task<bool> CheckPurchase(string productID, string file)
	{
		try
		{
			var productId = productID;
			var connected = await CrossInAppBilling.Current.ConnectAsync();

			if (!connected)
			{
				//Couldn't connect
				await DisplayAlert("Connectivity Error", "Unable to connect to the internet.", "OK");
				return true;
			}

			//check purchases
			var purchases = await CrossInAppBilling.Current.GetPurchasesAsync(ItemType.InAppPurchase);

			if (purchases?.Any(p => p.ProductId == productId) ?? false)
			{
				//Purchase restored
				await Download(file);
				return true;
			}
			else
			{
				//no purchases found
				return false;
			}
		}
		catch (Exception ex)
		{
			await DisplayAlert("Error", ex.Message, "OK");
			return true;
		}
		finally
		{
			await CrossInAppBilling.Current.DisconnectAsync();
		}
	}`

Step-by-step test:

click on product to purchase and/or validate if already purchased so it is restored
Apple Store prompt for password
Apple Store "Confirm Your In-App Purchase" (provides details of the product and price) - I click BUY
Apple Store "This In-App Purchase has already been bought.  It will be restored for free." - I click OK.
Nothing happens on the app.  This is the same for consumable and non-consumable.
The non-consumable product is not re-downloaded
The consumable product is not loaded or noted as purchasable again.

I would try to debug on your device in debug mode. Just sign with signing certs for development and make sure version numbers match up.
Essentially you want to test out if the check purchase is working or now as that will restore any purchases.
What you need to try to debug through is this here:
https://github.com/jamesmontemagno/InAppBillingPlugin/blob/master/src/Plugin.InAppBilling.iOS/InAppBillingImplementation.cs#L104
You need to see if transactions are restored here or what is returned.
Then check the parsing:
https://github.com/jamesmontemagno/InAppBillingPlugin/blob/master/src/Plugin.InAppBilling.iOS/InAppBillingImplementation.cs#L302

From reading: http://stackoverflow.com/questions/5628710/restorecompletedtransactions-broken
Perhaps something with the test user is bad.
Also reading:

Products Consumable cannot be restored.
Products Non-Consumable can be restored.

Which makes sense as you are supposed to track consumables somewhere else like iCloud.

Okay, realized my problem with the non-consumable.  I wasn't persisting the receipt.  I found the code in the programming guide.  What isn't clear is where the code should be added.  Does this get added to FinishedLaunching() in AppDelegate.cs or within the PCL code after purchase is confirmed?

@jamesmontemagno did some digging around and I am not finding some of the lines of code in the iOS version of the plugin as recommended on the Xamarin site.  So wondering if that is why previous purchases are not recognized.
This line is supposed to register the purchase
PhotoFilterManager.Purchase(productId);
This line is required to close out the purchase with Apple
SKPaymentQueue.DefaultQueue.FinishTransaction(transaction);
I will do some adding to the code and testing to see if it makes a difference tomorrow.
`public void CompleteTransaction (SKPaymentTransaction transaction)
{
var productId = transaction.Payment.ProductIdentifier;
// Register the purchase, so it is remembered for next time
PhotoFilterManager.Purchase(productId);
FinishTransaction(transaction, true);
}
public void FinishTransaction(SKPaymentTransaction transaction, bool wasSuccessful)
{
// remove the transaction from the payment queue.
SKPaymentQueue.DefaultQueue.FinishTransaction(transaction);  // THIS IS IMPORTANT - LET'S APPLE KNOW WE'RE DONE !!!!
using (var pool = new NSAutoreleasePool()) {
NSDictionary userInfo = NSDictionary.FromObjectsAndKeys(new NSObject[] {transaction},new NSObject[] {new NSString("transaction")});
if (wasSuccessful) {
// send out a notification that we've finished the transaction
NSNotificationCenter.DefaultCenter.PostNotificationName (InAppPurchaseManagerTransactionSucceededNotification, this, userInfo);
} else {
// send out a notification for the failed transaction
NSNotificationCenter.DefaultCenter.PostNotificationName (InAppPurchaseManagerTransactionFailedNotification, this, userInfo);
}
}
}`

I've had not luck in identifying the issue.  After reviewing the code and documentation on Xamarin and Apple, it appears that things should be working.  I created a new tester in TestFlight and still same issue.  Initial InApp non-consumable purchase works fine.  Delete the app, reinstall, click the item to validate if purchased and the result is nothing is restored (or I am not alerted via code to trigger the restore in my code) but yet Apple prompts with messages that purchase will be restored.  I fear I am missing something very simple.

It could be that Finish call... Odd. I will add it in and test though and confirm on Monday.

Thank you for looking at it.
Some additional information:
I added Console.WriteLine() within the foreach loop for UpdatedTransactions() to show (1) TransactionState and (2) Payment.ProductIdentifier.  I found UpdatedTransactions() is called during GetProductInfoAsync, which I am using to get LocalizedPrice.  My Console shows transaction state = Purchased for my InApp product (that is the product I  want to restore) when it is getting the ProductInfo.
I also added Console.WriteLine() to GetPurchaesAsync to show (A) total purchases and to RestoreAsync() to show (B) the list of transactions if transactions != null.
When I run GetPurchasesAsync() to check for prior purchases, I get (A) purchases = 0 and (B) restore transactions = StoreKit.SKPaymentTransaction[].
So for some reason the resulting GetPurchases validation is empty.

Found some more interesting details.  I added the following to RestoreCompletedTransactionsFinished() just before TransactionsRestored?.Invoke(rt);
Console.WriteLine(" ** RestoreCompletedTransactionsFinished: run = " + queue.Transactions.Count()); 			for (int i = 0; i < queue.Transactions.Count(); i++) 				Console.WriteLine(" ** RestoreCompletedTransactionsFinished: queue transaction = " + queue.Transactions[i].TransactionState + " => " + queue.Transactions[i].Payment.ProductIdentifier);
The output showed my product in the payment queue.  Matter of fact, just like the GetProductInfoAsync() my InApp products were listed 33 times (I only have two InApp products) in both Failed and Purchased states.  I am assuming it is because I tried repurchasing when the restore failed.  Do you think the payment queue needs to be flushed out.

@jamesmontemagno figured it out for non-consumables.

I had to clear out the payment queue, since it was built up from past attempts.  So added the following code to RestoreCompletedTransactionsFinished():

foreach (SKPaymentTransaction transaction in queue.Transactions)
       SKPaymentQueue.DefaultQueue.FinishTransaction(transaction);  // THIS IS IMPORTANT - LET'S APPLE KNOW WE'RE DONE !!!!

I removed this code once the payment queue was cleaned up.

Added new method FinishTransaction() within public class InAppBillingImplementation : IInAppBilling.

public void FinishTransaction(SKPaymentTransaction transaction, bool wasSuccessful)
{
	// remove the transaction from the payment queue.
	SKPaymentQueue.DefaultQueue.FinishTransaction(transaction);  // THIS IS IMPORTANT - LET'S APPLE KNOW WE'RE DONE !!!!

        using (var pool = new NSAutoreleasePool())
	{
		NSDictionary userInfo = NSDictionary.FromObjectsAndKeys(new NSObject[] { transaction }, new NSObject[] { new NSString("transaction") });
		if (wasSuccessful)
		{
			// send out a notification that we've finished the transaction
			NSNotificationCenter.DefaultCenter.PostNotificationName(InAppPurchaseManagerTransactionSucceededNotification, succeededObserver, userInfo);
		}
		else {
			// send out a notification for the failed transaction
			NSNotificationCenter.DefaultCenter.PostNotificationName(InAppPurchaseManagerTransactionFailedNotification, failedObserver, userInfo);
		}
	}
}

Added a call to  FinishTransaction() within RestoreAsync() when transactions != null.

foreach (var t in transactions) FinishTransaction(t, true);

Added a call to  FinishTransaction() within PurchaseAsync() for success test:

if (!success)
{
	FinishTransaction(tran, false);
	tcsTransaction.TrySetException(new Exception(tran?.Error.LocalizedDescription));
}
else {
	FinishTransaction(tran, true);
	tcsTransaction.TrySetResult(tran);
}

Next step is to test consumable products.

Consumable products is now working correctly on iOS as well.

Are you able to send a PR down? I can take a look and integrate. Nice work on this. Odd that it worked for me most the time. I only had 1 IAP though

checkout: <IP_ADDRESS>-beta

Saw your note, sorry for the indentation variances.  I will try the revised update and let you know.

I implemented all the changes. A lot of the code that was in there for notifications isn't necessary

I can validate things are working perfect on my test app here.

Uninstalled .csproj and reinstalled the latest beta and everything is working perfectly on iOS.

Thanks for your help on this one.

Absolutely!  I learned a lot.  Especially with this being my first time using Xamarin, developing for iOS, and developing in C#.


chore(main): release 0.1.0

:robot: I have created a release beep boop
0.1.0 (2022-11-12)
Miscellaneous Chores

main: release 0.1.0 (c2b6d6c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/PascaleBeier/filament-pages/releases/tag/v0.1.0 :sunflower:


Fails to build with latest nightly

Rucene also fails to build with the stable channel (due to the use of #[feature]), so I tried the nightly release:
$ cargo build
   Compiling rucene v0.1.0
error[E0432]: unresolved import `std::boxed::FnBox`
  --> /Users/alex/.cargo/registry/src/github.com-1ecc6299db9ec823/rucene-0.1.0/src/core/util/thread_pool.rs:27:5
   |
27 | use std::boxed::FnBox;
   |     ^^^^^^^^^^^^^^^^^ no `FnBox` in `boxed`

$ rustc --version
rustc 1.41.0-nightly (59947fcae 2019-12-08)

Hi Alexander,
Thank you for reporting this issue. Could you please try building rucene with rust nightly-2019-10-28 which is specified in rust-toolchain? FnBox was removed recently and somehow your cargo is not using rust version specified in rust-toolchain.
Please let me know if you still have issues with it after switching to nightly-2019-10-28. And here are the commit and issue related to removing of FnBox.
https://github.com/rust-lang/rust/pull/62043
https://github.com/rust-lang/rust/issues/28796

Sorry, wasn't aware of the rust-toolchain file. Installing that nightly doesn't resolve anything, unfortunately:
$ rustup run nightly-2019-10-28 cargo build
   Compiling rucene v0.1.0
error[E0432]: unresolved import `std::boxed::FnBox`
  --> /Users/alex/.cargo/registry/src/github.com-1ecc6299db9ec823/rucene-0.1.0/src/core/util/thread_pool.rs:27:5
   |
27 | use std::boxed::FnBox;
   |     ^^^^^^^^^^^^^^^^^ no `FnBox` in `boxed`

error: aborting due to previous error

Can you try rustc --version to check what's the actual version it is using?

Sure:
$ rustc --version
rustc 1.40.0-nightly (95f437b3c 2019-10-27)
$ rustup which rustc
/Users/alex/.rustup/toolchains/nightly-2019-10-28-x86_64-apple-darwin/bin/rustc

Not sure why the discrepancy (2019-10-27 vs. 2019-10-28) here.

Hm, that's weird. BTW: 2019-10-27 is the actual build date which is correct.
It works here and we are both using Mac so there shouldn't be anything related to specific platform.

Sorry that I'm kind of very busy right now and will come back later with this issue soon. Thank you.

Thanks!

Hi Alexander,
I finally know what's happening now. I overlooked that you were using rucene in your project and the compiler is complaining about a very old version which is still using FnBox.
The root cause was that we haven't published any update to crates.io for quite some time. In order to fix this issue, I just published a new version with latest updates. Please run cargo update to update it, Sorry for the inconvenience, hopefully we can put some more resources to maintain a release cycle later on.

Sorry, should have been explicit; I am indeed trying to import Rucene into an app. Works now, thanks for the fix!

Sorry, should have been explicit; I am indeed trying to import Rucene into an app. Works now, thanks for the fix!

Thank you for trying rucene. If you have any questions about using it, please don't hesitate to let me know.


Delay of dynamic notebook toolbar action label visibility change

While rerunning cells, the actions on the notebook toolbar only render icons even when there are space in the toolbar. After the execution finishes, it re-renders properly.

I don't ever remember seeing this but I definitely don't see it now anyway


Example / works with Restful?

hey @juliomalegria ! I'm familiar with the JQuery uploads for which this is intended (from a form POST) and am wondering if you have examples of using the module for restful POST (and if this is supported?) I'm hoping to add chunked upload to a POST from a command line client (not a form) and am hoping this could do the trick :) Thanks!

ping @juliomalegria , really could use some help!


Switch to Pydantic 2 and Annotated

Pydantic 2 uses annotated types to annotate arguments instead of the gross magical default values.
Would look something like...
def predict(input: Annotated[File, Input(...)] -> File:
    # ...

... or something.
It also means we can annotate outputs.

Bumping this, also running into this issue since I wrote my configuration validation with the pydantic 2 syntax -- need to refactor to be compatible with cog

bumping this. we need to use an underlaying package on our image that depends on pydantic v2.


Push back finalizing deprecations for 0.12

0.12 will already have a big change in dropping Python 2.7 support. I'd rather
wait a bit longer to finalize these deprecations to minimize the impact on
users.
xref https://github.com/pydata/xarray/issues/2776

Agreed.
This part needs to be extended to somewhat 0.13
https://github.com/pydata/xarray/blob/81cedd6e6d111c3ac58655585ffa6945fcc0e39b/xarray/tests/test_dataarray.py#L2040-L2041

This part needs to be extended to somewhat 0.13

Yes, I already did this, in my original commit :)


feat: always display level and gear score in the company character table

:tada: This PR is included in version 2.1.0-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Integration with S3 as Storage Backend

Feature Request
How do we add storage to S3 instead of saving it to local disk with minimal memory footprint (Using Stream probably).
I found a similar implementation in GO Language at https://tus.io/blog/2016/03/07/tus-s3-backend.html and implementation in https://github.com/tus/tusd/tree/bbf706622c7c5234844fbf2b8c5efdbcda1e3861/s3store but it was quite difficult converting it to c# since i have no knowledge of GO Language.

Hi,
I think this was resolved on Twitter, no?
For others reading this issue. tusdotnet supports custom data stores that can use any backend storage they want.
If you have written a store for tusdotnet please feel free to reach out and I'll add a link to it in the readme.

Closing due to inactivity.


[BEAM-4201][SQL] Add integration test for PubsubIO JSON -> SQL

Integration test for PubsubIO->SQL
Follow this checklist to help us incorporate your contribution quickly and easily:

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

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

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

run java precommit

run java precommit

run java precommit

run java precommit

(btw this is awesome framework buliding)

fixed typos, added global windowing

run python precommit

run python precommit

run python precommit

Python precommit is on its own for this one. If we had fancier infrastructure it wouldn't even have run.

I may be out of date - previously ITs were run only as postcommit. Gradle makes it more fluid.

This one is probably not wired up yet, I only added the targets to SQL build

@alanmyrvold @jasonkuster : I'm wiring up the Beam SQL Pubsub integration test to the main javaPostCommit target. I am not sure whether Jenkins job which runs postCommit has all permissions needed (create/delete Pubsub topics+subscriptions). Can you check or point me to the account being used?

run java postcommit

run java postcommit

The service account being used should be an editor on the project; access shouldn't be a problem.

green


proposal: syntax fix for := 'shadow variable' problem

Problem
The := syntax is error-prone when dealing with multiple targets and variables of enclosing scope:
// often a mistake:
var a int
{
  a, b := foo()    // creates a new 'a' in this scope instead of using 'a' of outer scope
}

// the fix
var a int
{
  var b *Bar
  a, b = foo()
}

Not only is the fix verbose, it defeats a nicety of inferred typing that it often spares us from having to remember precise return type(s), e.g. whether a returned type is a pointer or not:
a := bar()       // whether bar() returns an *X or just an X, we often don't need to care

Normally, Go disallows redeclarations, but it makes an exception for the := syntax, such that we can't look at an := assignment and know for sure which of the target variables are being declared:
x, y, z := foo()    // only one of x, y, and z must be new, but just looking here doesn't tell us which variables are new

Another problem is that we can't mix 'name' targets and 'non-name' targets:
a, b[0] := foo()          // compile error: b[0] is a non-name target

Proposal
A syntax that marks individual targets of assignment as new variables. Let's say for now we'll use ' as a suffix marker:
x', y, z' = foo()         // assign to new variable 'x', existing variable 'y', and new variable 'z'

We wouldn't remove the existing :=  syntax, of course, but in time its use would be considered non-standard style.
With this marking syntax, I don't see a good reason to disallow mixing in non-name targets:
a', b[0] = foo()          // ok

It would also be nice if we could optionally specify types of one or more of the newly declared target variables to make them different from the corresponding return type:
a' Fruit, b = bar()       // first return type of bar() is Banana, but we store it in a Fruit interface variable

As a last little bonus, perhaps type casts could be allowed on assignment targets:
a' int64(), b' = foo()     // the int64() indicates a cast of the return value assigned to 'a'

//spares us from having to create an additional variable of the type we don't want:
notA', b' = foo()
a' = int64(notA)

I think these syntax choices are compatible with the existing grammar, but regardless, I'm not strongly attached to this exact syntax.

I like this proposal a lot. I understand you're not massively attached to that syntax, what about appending or prepending +? It indicates the addition of a new variable and I'm not aware of any clashes with existing grammar.
a, +b = foo()

a, b+ = foo()

Similar to this? https://github.com/golang/go/issues/377#issuecomment-66049402
BTW, maybe it is not a big problem if the rules are defined clearly.
But sometimes, it may hurt the readability.
func f() {
	var a = []int{1, 2, 3}
	{
		a', a[1] = map[int]int{}, 999 // Which variable the second a represents?
		                                           // It would be better to represent the old one,
		                                           // but may bring a little confusions.

		fmt.Println(a) // map[] or map[1:999]
	}
	fmt.Println(a) // [1 2 3] or [1 999 3]
}

@sam3d I lean towards ' because it's easy to type, isn't used for anything else except character literals (which are generally distinctly highlighted), and stands out within code. Also x+ = 3 looks too much like x += 3, so +x = 3 would probably be the better option of the two. Again though, I think + right next a letter looks too much like a lowercase letter and so doesn't stand out enough. A colon prefix might be the next best candidate: :x = 3
@go101 I'd say that's a niche enough scenario that simultaneous declaration and use in different targets of the same assignment should just be a compilation error. Can you think of any idioms where lack of some allowance would be missed?

A compilation error? So do you think the second a in the assignment should be the new declared a?
I think this is a bad design. It means the scope of the new declared a starts from the comma following the a identifier.
Could clarify which ones of the following as in the assignment line are the old one, and which ones are the new one?
var a = []int{1, 2, 3}
{
    a[1], a', a[1] = a[0], []int{7. 8. 9}, a[0]
}

An alternative fix for the example problem in the proposal which fixes the typing problem in some circumstances but not the verbosity issue is to use a third variable on purpose:
var a int
{
  tmp, b := example()
  a = tmp
}

Please don't do that, don't bring Rust's crap syntax into go.

@OneOfOne Please stay polite.  Thanks.
The ideas here seem very similar to ideas already expressed in #377, so closing this issue in favor of that one.

This proposal is just another variation of those in issue #377 (some of them are 8 years old), and the discussion should probably take place there.
And yes, I'm totally in favor of a per-variable short declaration syntax, I already commented here.
Multivariable := is the single thing about Go that drives me crazy, in fact I wrote a simple code rewriter so that I can do without := altogether.
@go101 my code rewriter turns each left hand side element into its own statement in order, so:
var a = []int{1, 2, 3}
var b = []int{1, 2, 3}
{
	:a, a[1] = map[int]int{}, 999
    fmt.Println(a) // map[1:999]

    b[1], :b = 999, map[int]int{}
    fmt.Println(b) // map[]
}
fmt.Println(a) // [1 2 3]
fmt.Println(b) // [1 999 3]

@DeedleFake I used the temporary variable solution for years (with subsequent assignments on the same line for clarity, but gofmt breaks it up :(, and there are other problems too).
var a int
{
    A, b := example(); a = A
}

@OneOfOne I don't see how my suggestion is like Rust
@go101 I'm saying you just have to pick a different name for the new variable:
var a = []int{1, 2, 3}
{
     // a[1], a', a[1] = a[0], []int{7. 8. 9}, a[0]           // compile error
    a[1], b', a[1] = a[0], []int{7. 8. 9}, a[0]               // ok
    a' = b
}

Come to think of it, I'm not happy that Go allows redeclaration of a variable from outer scope that has already been used in this scope:
a := 3
{
    foo(a)    // shouldn't be able to use 'a' of outer scope here
    a := 5    // because we redeclare it later in this scope
    foo(a)
}

This seems error-prone to me as well, but it's too late to change this.


Running .institutional_holders give the following error - ValueError: time data "..." doesn't match format "%B %d, %Y", at position 1

Describe bug
I was scraping the ticker ACST and getting institutional holders when I ran into this issue.
The issue can be replicated like this:
import yfinance
stock = yfinance.Ticker('ACST')
stock.institutional_holders # Gives error

The full error is:
ValueError: time data "Jun 29, 2023" doesn't match format "%B %d, %Y", at position 1. You might want to try:
    - passing `format` if your strings have a consistent format;
    - passing `format='ISO8601'` if your strings are all ISO8601 but not necessarily in exactly the same format;
    - passing `format='mixed'`, and the format will be inferred for each element individually. You might want to use `dayfirst` alongside this.

I'm using Python 3.11, and yfinance v0.2.28
Simple code that reproduces your problem
import yfinance
stock = yfinance.Ticker('ACST')
stock.institutional_holders # Gives error
Debug log
/Users/jack/Projects/ais/aisenv/lib/python3.11/site-packages/yfinance/scrapers/holders.py:39: FutureWarning: Passing literal html to 'read_html' is deprecated and will be removed in a future version. To read from a literal string, wrap it in a 'StringIO' object.
holders = pd.read_html(resp.text)
Traceback (most recent call last):
File "", line 1, in
File "/Users/jack/Projects/ais/aisenv/lib/python3.11/site-packages/yfinance/ticker.py", line 114, in institutional_holders
return self.get_institutional_holders()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/jack/Projects/ais/aisenv/lib/python3.11/site-packages/yfinance/base.py", line 1662, in get_institutional_holders
data = self._holders.institutional
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/jack/Projects/ais/aisenv/lib/python3.11/site-packages/yfinance/scrapers/holders.py", line 26, in institutional
self._scrape(self.proxy)
File "/Users/jack/Projects/ais/aisenv/lib/python3.11/site-packages/yfinance/scrapers/holders.py", line 63, in _scrape
self._mutualfund['Date Reported'] = pd.to_datetime(
^^^^^^^^^^^^^^^
File "/Users/jack/Projects/ais/aisenv/lib/python3.11/site-packages/pandas/core/tools/datetimes.py", line 1112, in to_datetime
values = convert_listlike(arg._values, format)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/jack/Projects/ais/aisenv/lib/python3.11/site-packages/pandas/core/tools/datetimes.py", line 488, in _convert_listlike_datetimes
return _array_strptime_with_fallback(arg, name, utc, format, exact, errors)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/jack/Projects/ais/aisenv/lib/python3.11/site-packages/pandas/core/tools/datetimes.py", line 519, in _array_strptime_with_fallback
result, timezones = array_strptime(arg, fmt, exact=exact, errors=errors, utc=utc)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "strptime.pyx", line 534, in pandas._libs.tslibs.strptime.array_strptime
File "strptime.pyx", line 355, in pandas._libs.tslibs.strptime.array_strptime
ValueError: time data "Jun 29, 2023" doesn't match format "%B %d, %Y", at position 1. You might want to try:
- passing format if your strings have a consistent format;
- passing format='ISO8601' if your strings are all ISO8601 but not necessarily in exactly the same format;
- passing format='mixed', and the format will be inferred for each element individually. You might want to use dayfirst alongside this.
Bad data proof
No response
yfinance version
0.2.28
Python version
3.11
Operating system
M2 Macbook Air; Ventura 13.1

I am unable to reproduct this on my machine. Are you still facing issues with this?

I had the same error:
FutureWarning: Passing literal html to 'read_html' is deprecated and will be removed in a future version. To read from a literal string, wrap it in a 'StringIO' object.

I am getting the same error. I am using python 3.9 version on windows 10.

@ValueRaider Close, please. Implemented at #1772


PERF: thread local context

[x] Closes #1133
[x] Closes #1418
[x] Tests updated
[x] Fully documented, including history.rst for all changes and api/*.rst for new API

Considering merging this for this release. I haven't seen any issues so far. If you have time to test it with other repos that use pyproj, that would be helpful.

#1316 may be helpful for testing...

https://anaconda.org/scientific-python-nightly-wheels/pyproj


Fail Quietly if Model Slot is Empty

… empty

[ ] Remove a hand model from a HandPool slot and play any scene.  Verify that no errors are thrown.

Looks great to me!


how to show centroid?

i'm a newbie in programming. I try to use this library. it's very useful for me.
I want to show centroid in K-means clustering. how to show it? thank u so much..

Currently centroid are calculated in Cluster and are not returned in any public method. You can make a PR if you find it useful. Phpml\Clustering\KMeans\Cluster is a good point to start :)


merge migrations to fix migration error, fixes #336

Issue Reference
Fixes #336
Summarize
Please, describe what the PR does, the changes you have made.
Ran python manage.py makemigrations --merge and committed the result, to fix the conflicting migrations. After pulling this, one should be able to run python manage.py migrate without errors.

https://github.com/IEEEKeralaSection/rescuekerala/commit/c2b099ef5b0ad21d83637238c205ea4755a56a4a
fixed


pbs with some citations .

I am stuck due to some citations that are no longer recognized in the pdflatx workflow .

This was a syntax error with your \verbatim command. This is fixed in the upcoming version.

On Fri, Jul 15, 2022 at 06:48:55AM -0700, Mireille LOUYS wrote:

I am stuck due to some citations that are no longer recognized in
the pdflatx workflow .

Ummm -- which ones?  In the master branch?  I don't see any
trouble...
By the way, in such cases it almost always helps to give the error
message that stumps you (not the whole log).


VLAZ: Watch breen.com.ua

VLAZ requests the watch of the watch_keyword breen\.com\.ua. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

breen\.com\.ua has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by cigien in Charcoal HQ


Tweak build help to clarify role of --bin

From user feedback the --bin's description "Build only the specified binary" could be understood as "Don't build lib" rather than "Build this bin (with lib) and not other bins".
I don't know if a better wording explaining subtelty of lib+bin crates would fit in the small space of CLI help. However, reordering the args to show --bin after --bins rather than after --lib gives it a more accurate context. So I've merely put the --bin (and test/bench/example) after their plural counterpart.
I've also noticed an issue with clap inserting global args between subcommand's args, and --locked definitely doesn't belong right between the two bin args. I've added a workaround for that issue. The workaround is otherwise harmless, and shouldn't cause problems if clap decides to update sort order of subcommands.
Changes this:
      --lib                     Build only this package's library
      --bin [<NAME>]            Build only the specified binary
      --locked                  Require Cargo.lock is up to date
      --bins                    Build all binaries
      --offline                 Run without accessing the network
      --config <KEY=VALUE>      Override a configuration value
      --example [<NAME>]        Build only the specified example
      --examples                Build all examples
  -Z <FLAG>                     Unstable (nightly-only) flags to Cargo, see 'cargo -Z help' for details
      --test [<NAME>]           Build only the specified test target
      --tests                   Build all tests
      --bench [<NAME>]          Build only the specified bench target
      --benches                 Build all benches

to this:
      --lib                     Build only this package's library
      --bins                    Build all binaries
      --bin [<NAME>]            Build only the specified binary
      --examples                Build all examples
      --example [<NAME>]        Build only the specified example
      --tests                   Build all tests
      --test [<NAME>]           Build only the specified test target
      --benches                 Build all benches
      --bench [<NAME>]          Build only the specified bench target
[…]
      --locked                  Require Cargo.lock is up to date
      --offline                 Run without accessing the network
      --config <KEY=VALUE>      Override a configuration value
  -Z <FLAG>                     Unstable (nightly-only) flags to Cargo, see 'cargo -Z help' for details

This is another data point that #11912 might be useful.
I'll take a deeper look of it tomorrow.

Will merge this and move on. We can always reverse the change easily.
@bors r+

:pushpin: Commit 75a1dd03075b325fc4ba0ec977fe7b641a63ace1 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 75a1dd03075b325fc4ba0ec977fe7b641a63ace1 with merge a6c0d43787269d48510c0a4e8acf2207790a6daa...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing a6c0d43787269d48510c0a4e8acf2207790a6daa to master...


Update intro paragraph on get started designing

Updated last sentence in the Intro paragraph to get rid of duplicate "slack channel channels".
Related Ticket(s)
{{Provide url(s) to the related ticket(s) that this pull request addresses}}
Description
{{Add a human-readable description / detail summary of what the PR is changing and any details around how and why}}
{{If applicable, include a screenshot indicating an example or examples of what the PR is changing in the application}}
There are duplicate "channel(s)" in the intro paragraph. Updating to remove double word.
Changelog
Changed
Before
If you have any questions please reach out on our slack channels channel.
After
If you have any questions please reach out on our slack channel.

@JennySanchez I think you have to commit the suggestion I made above before it will update to remove "our"


chore: Use baptiste0928/cargo-install@v1 to install cargo-hack

In every check ci jobs, it takes about 3 (in ubuntu-latest), 4 (in windows-latest), 5 (in macOS-latest) minutes to install cargo-hack. This pull request tries to speed up the jobs by introducing baptiste0928/cargo-install action using cache.

Once we installed cargo-hack, in the next job, we can use cached cargo-hack. So we'll be able to spare time to install cargo-hack taking up to about 5 minutes every time.

makes sense to me!


Add new bookmark and update other

Add "Qwik Heroicons" bookmark.
Add description to "Qwik Feather Icons" bookmark.
What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests

Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

I notice I'm able to merge my pull request, but I think is better to wait for approval :)


Move WSUS script from SCCM pages to WSUS pages

Can we please get this script moved over to the WSUS product pages? Currently it's listed under the Endpoint Manager pages, which doesn't make any sense.
https://learn.microsoft.com/en-us/troubleshoot/mem/configmgr/update-management/decline-superseded-updates

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

ID: 2e9e9c15-b10c-4a86-e02f-2292f3755f48
Version Independent ID: f343b9a3-b7ef-dd1c-158b-a2a112b16fd2
Content: The Server cleanup Wizard
Content Source: WindowsServerDocs/administration/windows-server-update-services/manage/the-server-cleanup-wizard.md
Product: windows-server
Technology: wsus
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Snapshot

Hello.
Is it possible to trigger a capture of a snapshot by API, or through MQTT?
If so, I coulnd't find any kind of documentation.
Thank you.

Hello.
I was wondering, if this is possible with use of additional packages by Kerberos.io?

It will be added in our new version (published end of week), we will have a API endpoint to request a snapshot. Hope this helps.

It will be added in our new version (published end of week), we will have a API endpoint to request a snapshot. Hope this helps.

Hello @cedricve .
Thank you, very much for you support and quick replies.
I just spun up latest image of the Agent. I tried to lookup snapshot in the swagger API documentation.
If it is already implemented, could you please point me to a specific endpoint or write a quick and dirty url how to get a
a snapshot? Otherwise, I'm looking forward to the update.
Kerberos.io is awesome system.
Thank you, very much.
Have a nice holidays.

@unichargepay feature added to separate branch, will be merged soon, and made available.

hello @unichargepay  made available in latest release: https://github.com/kerberos-io/agent/releases/tag/f739d52
Closing this request.


Droplet accepting files and passing onto python script

Hi,
I'm really stuck up with this for a long time. I'd really appreciate your kind help!
I'd like to drop ".csv" file into droplet and then a python script will read the .csv file and do the typical pandas calculation and output the new .csv into the same directory.
Python script calculation is not an issue. But i don't know how to accept the file and then pass this as dataframe into python. I believe it has got to do with args but i'm really puzzled. If you could share a very simple app that accepts csv(A1 cell is 5, A2 cell is 4) and then adds two number from rows n throws csv output(A1 cell answer as 9) ,that would be great!!!
P.S. Existing example for droplet is a bit puzzling. A very simple use case of csv would be of great great help.
This will propell mass usage and adoption! Thanks a ton for your time!!
Really look forward to help.

The path of dropped files are passed as arguments to your script. If you're using Python, you can access them via sys.argv, e.g. (some pseudocode there, fill in the blanks):
import sys
csv_file_path = sys.argv[1]
csv_data = readcsvfile(csv_file_path)
new_csv_data = do_calculations(csv_data)
... write to new path

Parsing CSV is something that you do in the script itself and has nothing to do with the Platypus wrapper as such.


Add font detection for iTerm2 on macOS

Description
This PR adds font detection for iTerm2.
Issues
There is a somewhat noticeable lag, possibly due to the regex. Is there any way we could speed this up a bit more?
Also this assumes that the user only has one iTerm2 profile. Unfortunately there is no way to determine which iTerm2 profile (and therefore which font) is currently being used. If the user has more than one profile, this may output the wrong font.

Closing in favor of #670.


Question about feature_group_count and batch_group_count in lax conv_general_dilated

Could anyone point me the relevant knowledge point or publication (or brief explanation) on these two parameters of conv_general_dilated? Thanks.

Thanks for the question!
See this section of the XLA Operation Semantics page. Does that cover what you have in mind?

Thanks, Matthew!


F12 does not find the AL file that contains the definition

1. Describe the bug
F12 for "Go to Definition" does not work. VSCode does not find the file if the AL file that contains the definition contains spaces in the name. This is happening on either file with spaces. Going to the definition by first listing all references through Shift+F12 and then selecting the function definition works as expected.
Files that do not contain spaces in the name do not have this problem.
Possibly some kind of URL-encoding issue (%20 = space)?
2. To Reproduce
This should be easily reproduceable on any AL project:

Open any AL object that contains a call to a second AL object. This second AL object must be in a AL file with a space in the name.
Go to the function call, where the other object is called.
F12
The file is not found.

3. Expected behavior
The file should be found.
4. Actual behavior
The AL language extension does not find the file. And error is thrown, similar to this:
Unable to open 'Codeunit%20xxxx%20Some%20Filename.al': File not found.
5. Versions:

AL Language: 9.0.611415
Visual Studio Code: 1.66.2
Business Central: /

I am unable to reproduce this issue in the latest release of the AL Language extension. Please try re-installing the extension and report back with further information if the problem persists.
For reference, I successfully navigated to the following symbols:
Page "Countries/Regions"
Codeunit "Activities Mgt."
Codeunit "Sales-Post Prepayment (Yes/No)"

I incorrectly presumed that DAL files would be the same as AL files in your own project; I am indeed able to repro this. Changing labels.

I also have the same problem:

Versions:
AL Language: 9.0.611415
Visual Studio Code: 1.66.2

On it.

Some additional context on Twitter
https://twitter.com/eHougaard/status/1517554573199564801?t=vvwRXCytZF2S1Nemgzi5Xw&s=19

Some additional context on Twitter https://twitter.com/eHougaard/status/1517554573199564801?t=vvwRXCytZF2S1Nemgzi5Xw&s=19
It also only happens on windows 11 for me. Vs code and Al version on windows 10 works fine. I am not sure that w11 is culprit, but possibly.

I made that comment on Twitter. More than happy to test if needed.
VSCode: 1.66.2
AL:9.0.611415
OS W11: 21H2 (22000.613)

I also have the same problem:
GoToDefinition

Versions:
SO: Win 11 Pro, 22H2, 22598.200
AL Language: v9.0.611415
Visual Studio Code: 1.66.2

AL VSIX version 9.0.615906  should have a fix for it.

I have updated to 9.0.615906 and this has not resolved my issue. I am still experiencing the same result. When I am in a file and I select a function and hit F12, it does not go to definition. When I view the errors/warning in the problems section, and select a warning/problem, it does not go to that section of code. It states it is unable to open that file.

I have updated to 9.0.615906 and this has not resolved my issue. I am still experiencing the same result. When I am in a file and I select a function and hit F12, it does not go to definition. When I view the errors/warning in the problems section, and select a warning/problem, it does not go to that section of code. It states it is unable to open that file.

Cannot confirm. It works just fine for me now.

I have updated to 9.0.615906 and this has not resolved my issue. I am still experiencing the same result. When I am in a file and I select a function and hit F12, it does not go to definition. When I view the errors/warning in the problems section, and select a warning/problem, it does not go to that section of code. It states it is unable to open that file.

After I recloned repo - it started to work. Thanks!


Multiple pub sockets

Hey.. I am having trouble connecting multiple pubs to the same socket. I am using an inproc socket and the idea is to have multiple nodes that can broadcast messages on the same socket to all/any node. However, after binding a publisher socket, even though other publisher sockets can connect they cant send messages..

yup I think there might be a bug with inproc on osx. What system?

I was testing this on Windows 8.1 ... havent tested on linux yet..

wanna paste your sample code?

Well I cant publish all the code, but this is a bit of what I am trying to do. I want many nodejs scripts to run concurrently and to communicate in a mesh network. When a script is run, it tries to bind the inproc socket, but if the socket is already bound then it connects, afterwards I will subscribe to the same channel to recieve messages from all channels. This would be the connection initialization code.
publisher.bind('inproc://shtest', function(err) {
if(err){
console.log(err);
publisher.connect('inproc://shtest',function(err){
if(err)
console.log(err);
});
}
else
console.log("Listening on 8688...");
});
return publisher;

it seems like there might be an issue with your code.
Are you connecting and binding the same publisher?

Well I am binding, on a socket but if the socket has already been established it connects to the socket using connect.. It works perfectly with the sockets but with intricate the messages are not being sent..

Sorry autocorrect made me sound crazy.. So my script tries to connect to a pub socket using bind  if that fails it tries to connect using the connect function... I'm trying to implement a mesh network    and I need each script to publish and subscribe to the same socket

The code works well with tcp but not inproc

Hey. Still have the same issue, I dnt know if its a constraint of zeromq itself or if its just the nodejs bindings, I have multiple scripts, lets call them nodes... each of them look like this

The idea is to implement a mesh network where each node can independently connect to and communicate with each other without depending on a central node. The code however acts weird, only the node that establishes the publisher socket has messages successfuly transmitted to the others while other publishers that just connect to the socket cannot transmit their information. To rephrase the pub socket that binds sends messages, the other ones that connect seem not to be able to transmit anything, even though they dnt show any error message. I will try the same infrastructure python but I think the binding has a problem.

Hey... I sorted it out.. I will use an xpub/xsub broker that will be integrated into my code. I have done basic testing and it seems to work, but I dnt know if it will work exactly how I want it to.. anyway.. Ill keep you posted.


[BUG]: onClick is not working

Is there an existing issue for this?

[X] I have searched the existing issues.

🐞 Describe the Bug
For some reason the createOrder is being called even when actions.reject was made before. This makes no senes at all.
        createOrder={(data, actions) => {
          return createOrder(data, actions);
        }}
        onClick={(data, actions) => {
          actions.reject();
        }}
        onInit={(data, actions) => {
          canProceed()
              .then((s) => {
                enableButton(s);
              })
              .catch(() => {
                enableButton(false)
              });
        }}

😕 Current Behavior
actions.reject(); does not block further execution
🤔 Expected Behavior
actions.reject(); should prevent and discard current flow.
🔬 Minimal Reproduction
Try this configuration:

        createOrder={(data, actions) => {
          return createOrder(data, actions);
        }}
        onClick={(data, actions) => {
          actions.reject();
        }}

### 🌍 Environment

```markdown
| Software         | Version(s) |
| ---------------- | ---------- |
| react-paypal-js  |            |
| Browser          |            |
| Operating System |            |

Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

➕ Anything else?
No response

Hey @AndrewEastwood, can you try using the disabled prop? We added this to make validation easier with the <PayPalButtons /> component.
Sorry for the confusion around validation. This disabled prop was originally documented in Storybook but it was removed.

@AndrewEastwood, also try returning actions.reject().Sent from my iPhone 12 Pro MaXOn Dec 26, 2022, at 3:23 PM, Greg Jopa @.***> wrote:
Hey @AndrewEastwood, can you try using the disabled prop? We added this to make validation easier with the  component.
Sorry for the confusion around validation. This disabled prop was originally documented in Storybook but it was removed.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

@gregjopa @mnicpt could you explain how do I use actions.reject(); from the onClick attribute? What exactly this rejection is suppose to do?

@AndyEastwood This is what reject does: https://github.com/paypal/paypal-smart-payment-buttons/blob/300798e412f1018192616b0bab4a23d99f7954de/src/props/onClick.js#L30 . That’s why I mentioned to return it. The onClick callback is used to allow client-side validation prior to rendering the checkout flow. You could also just return true or false and will work. Hope this helps.Sent from my iPhone 12 Pro MaXOn Dec 26, 2022, at 3:44 PM, Andy Eastwood @.***> wrote:
@gregjopa @mnicpt could you explain how do I use actions.reject(); from the onClick attribute? What exactly this rejection is suppose to do?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Seems it does not allow to do that because of onClick expects nothing to be returned from it.

...alright.. I think I get this suff working when I put return kwd there, like this:
onClick={(data, actions) => {
  return actions.reject();
}}

however, I'm seeing this side-effect:
https://user-images.githubusercontent.com/1483306/209677010-078c40b9-7455-47be-801a-4a317e857ffe.mov


Enhance LaTeX rendering in Markdown with support for environments

This PR improves the KatexMarkdownJS function to provide better LaTeX rendering capabilities within Markdown content. Key enhancements include:

Support for LaTeX environments (e.g., equation, align, gather, multline)
Separate handling for inline and display math
Customizable delimiters for inline and display math
Configurable list of LaTeX environments to be treated as display math

Changes:

Updated KatexMarkdownJS function in [file path, e.g., fasthtml/js.py]
Added support for processing LaTeX environments
Improved regex patterns for more accurate inline and display math detection
Made math delimiters and environment list customizable

Breaking Change:
The katex_tags parameter has been replaced with separate inline_delim and display_delim parameters. Users who previously specified katex_tags will need to update their code.
Migration guide:

Old: KatexMarkdownJS(katex_tags='$')
New: KatexMarkdownJS(inline_delim='$', display_delim='$$')

Minimum working example:
This example demonstrates some environments which failed before this PR.
from fasthtml.common import *

hdrs = (KatexMarkdownJS(),)
app, rt = fast_app(hdrs=hdrs)

@app.get("/")
def test():
    content = """
Here is some math $x+y=3$.

Examples of environments:
\\begin{align}
x &= y + z \\\\
a &= b + c + d + e
\\end{align}
and the equation environment:
\\begin{equation}
e = mc^2
\\end{equation}
Some more math using the display style deliminators:
$$
1 \\times 2 = 2
$$
final example
$$1 \\times 2 = 2$$
"""
    return Div(content, cls="marked")

serve()

cc: yym68686 as this is related to a previous PR

Attached are screenshots of before and after for the minimum example provided.

@jph00 I think those are all addressed!

speedy!


Add handling of overlapping regions in Regions2D.mask

In #87 we said we'll handle overlapping regions when it arises. I just ran into one case!
In this particular, one region is entirely within another one, and in fact the smaller one is supposed to be "hollow" within the larger one. See the brown region at ~17:00 within the large purple region.

I think we have a few choices to make here:

for regions that partially overlap:

automatically create a 3D mask

for the scenario when one region is nested within another one:

prune all the nested regions: this means that we won't have regions with holes
create "cuts" to connect the brown and purple region, so that we can get a combined mask that ends up having a hole in the middle. On the other hand, for predictions that will give us this case, we already have the mask to begin with, and the contours are extracted from that, so technically we don't need this urgently.

This is definitely not a v0.1.0 issue. I think these are more of backburners that we can pick up after v0.2.0. I think we can take care of the partially overlapping scenario first, and then see if we still need to handle the nested case.

This is addressed in #139 by disallowing overlapping regions for 2D masks and allowing those for 3D masks.


Update squarespace-server logging calls

Updated some logging calls missed in the last update about 5 days ago.

Ha, man. Good find. Looks good :D

Merged and published - +<EMAIL_ADDRESS>


AuthorizedUsersUrl overwrites KeyName keys in authorized_keys file

Hi, while testing the CI Stack, I ran into a potential issue with your cloudformation stack template for EC2..
If you include both a path to AuthorizedUsersUrl and a KeyName, it seems to overwrite the authorized_key file that the KeyName creates and replaces it with the copy of authorized_users specified by the AuthorizedUsersUrl.
This prevents us from using the AWS Key Pairs in AWS to SSH in. We had to add our own public keys to the file specified by AuthorizedUsersUrl.
Seems that these two should be merged if that's possible?

Hi @skoonin I took a look at this and you’re right the cron job that updates the /home/ec2-user/.ssh/authorized_keys file does replace the keys file with the contents of the URL.
I think it would be reasonable to blend this with the public keys listed by instance metadata service to preserve access for the key given in the CloudFormation template’s KeyName parameter.

We had to add our own public keys to the file specified by AuthorizedUsersUrl.

I’m glad you’ve found a workaround for the time being :+1:
I can’t say when someone will get to implementing this, but it does look like a reasonable thing to do :bow:


replace Reflector in kube-proxy

Replace servicesReflector and endpointsReflector with client.cache.Reflector.  #10463

@erictune

@k8s-bot ok to test

Thanks for the PR! This is much cleaner.

Thanks for your help. Will  repair it soon.

@lavalamp, presumably this is post-1.0?

This is a very good refactor to do. I think we should postpone it to after 1.0, though.

@lavalamp I see you and erictune has done the same work in /pkg/kubelet/config/apiserver.go. So I think it's better to keep kube-proxy sync with it. What do you think?

@jiangyaoguo Yes, it is good. But we're gearing up for a release and refactoring changes are out of scope (unless they fix a bug in the process). We will definitely take this PR or something like it after we cut 1.0. Sorry, thank you for your patience.

ok. I see. thanks. Do we have somewhere that we can see timetable or status of kubernetes？

Thank you very much for the pull request - we sincerely appreciate the work that goes into sending us a change.
We are currently in a feature-freeze and general code slush as we work toward our 1.0 release. Our initial triage indicates that this is a non-critical change at this time, so we'd like to leave it open and revisit it after 1.0. We expect the freeze to wrap up in early to mid July.
If you feel that this triage is incorrect, please respond and let us know why. We're currently accepting changes that meet any of the following criteria:
Changes directly related to v1.0 issues: https://github.com/GoogleCloudPlatform/kubernetes/issues?q=is%3Aopen+is%3Aissue+milestone%3Av1.0
Changes addressing P0 bugs: https://github.com/GoogleCloudPlatform/kubernetes/issues?q=is%3Aopen+is%3Aissue+label%3Apriority%2FP0
Changes to supported platforms to bring them up to par with the primary development platforms
Docs, user experience, and test related fixes
Thanks again for your contribution! The community is really what has made this project so exciting.

LGTM

@k8s-bot test this please

@jiangyaoguo thanks for the change and your patience :)


Update README.md

forgot to tell them to cd into bin directory

you dont need to cd into bin directory

ah waiit u was talking about systemctl


Artisan publish-permission changes

It now does only the thing automatically what the command says.
For migration you need to add --migration and --seed.

Hi mate 👋 @Houbsi
Can you fix conflicts?

Hi mate 👋 @Houbsi Can you fix conflicts?

I just fixed it

Ahh it's because i based both branches not on each other. Sorry.

Hello @Houbsi,
After committing, please run the following command to check for code style issues:
./vendor/bin/pint --dirty

Thanks!

Pint is not in the package composer.json dev part. Or do you have it integrated into a laravel application and there is pint installed?

Pint is not in the package composer.json dev part. Or do you have it integrated into a laravel application and there is pint installed?

Laravel Pint is the default, you can use it

Ah  okay, i coded in the raw package.. not in a laravel installation. Found the issue

Ah okay, i coded in the raw package.. not in a laravel installation. Found the issue

Are you fixing style issues?
Or should I?

Could you please? Having trouble with the setup atm.


Allow mipmaps with KHR_texture_transform

Resolves #10787 by allowing mipmaps in the sampler for textures using KHR_texture_transform.

Thanks for the pull request @jjhembd!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.

@sweco-sekrsv, could you test this branch and let us know if it works for your datasets?

@lilleyse Could you please give this change a look?

Looks good to me. Feel free to merge after resolving the conflicts.

Thanks all!


[SR SQL Planner Summer Camp] support export related-statements

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #7952
Problem Summary(Required) ：

Now we support export without broker https://github.com/StarRocks/starrocks/pull/9171, new export related-statements should this.
1.  Export with broker
EXPORT TABLE table_name
[PARTITION (partition_name[, ...])]
[(column_name[, ...])]
TO export_path
[opt_properties]
with
[broker_properties]

2. Export without broker
EXPORT TABLE table_name
[PARTITION (partition_name[, ...])]
[(column_name[, ...])]
TO export_path
[opt_properties]
with broker_name
[broker_properties]


Feature/delete user

Delete User and add a security that you cannot delete a user that does not exist (or does not exist anymore)

LGTM


Translate docs/tasks/configmap-secret/managing-secret-using-config-file/ into Japanese

This is a Feature Request

What would you like to be added

Translate docs/tasks/configmap-secret/managing-secret-using-config-file/ into Japanese
Why is this needed
This page has not been translated.
Comments

/language ja
/assign
/triage accepted

/close


GA apis for hanaonazure rp

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

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

What's the purpose of the update?

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

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

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

Contribution checklist:

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

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

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

Change to data plane APIs
Adding new properties
All removals

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

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

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

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

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

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

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

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

Hi @Yogi1994, Per ARM review requirement, you need to resubmit your PR via OpenAPI Hub which will help you copy existing base version as your first commit in PR.

For new API vesion, To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.
I am closing your PR now, pls re-submit again with OpenAPI Hub


fix(demo) Quick fix demo google geocoding example

Quick fix for the demo page.
Fix for : #1664

Closing this one as it will be in #1767


Some fixes for ruby 2.4.1 compatibility

Hi there!
On ruby-2.4.1 it raises an error:
     NoMethodError:
       undefined method `<<' for {:read_timeout=>60, :continue_timeout=>nil, :debug_output=>nil}:Hash
       Did you mean?  <
     # .rvm/gems/ruby-2.4.1/gems/fakeweb-1.3.0/lib/fake_web/ext/net_http.rb:50:in `request_with_fakeweb'

I made some little fixes in order to use the gem with ruby-2.4.1. May be someone would find it helpful.
# Gemfile

group :test do
  gem "fakeweb", github: "SamMolokanov/fakeweb", branch: "ruby-2-4-1-support"
end

#57

Same problem here, thank you for the workaround :) - please merge!

Closing this since @chrisk continued working. If anything could be useful from this PR, please let me know. Cheers!

Hey Sam, thanks for posting this! If it's okay, I'd like to leave it open for a little while longer until I've had time to review everything you did and make sure nothing falls through the cracks... sorry to pollute your PR list. I should have everything reviewed and merged soon.

Hey @SamMolokanov, would you mind switching to this repo's master branch and seeing if everything works for your project? I have versions of all these fixes merged now (along with many others), but would like to get some users to test while I update the docs, etc. before releasing.
# Gemfile

gem "fakeweb", github: "chrisk/fakeweb", branch: "master"

Hi @chrisk thank you for the update!
Your latest build works perfect 👍
PS. I have added a post-install message for people who use temporary solution.

Hi @chrisk! can I close this PR? :)

Hi Sam, thanks for testing! Good idea about the post-install message. Yes, we can close this now. Thanks for all your help.


[VNext] System.Text.Json as default serializer

[VNext] System.Text.Json as default serializer

Does this feature request also include System.Text.Json as default serializer to generate SQL from LINQ?

LINQ is still in a design phase so there are not any guarantees, but it should use the System.Text.Json

Just lost several hours to "The input content is invalid because the required properties – ‘id; ‘ – are missing” errors
Only to discover that [JsonPropertyName("id")] was being ignored
It's not like the System.Text.Json changes are very new :(

Did you change your using to System.Text.Json https://docs.microsoft.com/en-us/dotnet/api/system.text.json.serialization.jsonpropertynameattribute?view=net-5.0 or do you used the newtonsoft namespace?

@benc-uk the .NET v3 SDK was released before System.Text.Json was GA. Switching the entire SDK over from newtonsoft to System.Text.Json is a breaking change which would require a v4 SDK. Please look at the following sample on how to make the v3 SDK use the new System.Text.Json: https://github.com/Azure/azure-cosmos-dotnet-v3/tree/master/Microsoft.Azure.Cosmos.Samples/Usage/SystemTextJson

+1

+1

+1

Just lost several hours to "The input content is invalid because the required properties – ‘id; ‘ – are missing” errors Only to discover that [JsonPropertyName("id")] was being ignored
It's not like the System.Text.Json changes are very new :(

Maybe
https://github.com/Azure/azure-cosmos-dotnet-v3/pull/3282

Oh btw any plans to update the Newtonsoft dependency to the latest version to avoid possibly getting a version of that in the v3 that is prone to security issues? I think v10 is prone but v13 is not (that we know of so far).

Hello, any progress on this? We want to avoid Newtonsoft JSON and move to System.Text.Json ...

@hkusulja while the dependency is not yet removed, have you considered adding a customer serializer like shown in https://github.com/Azure/azure-cosmos-dotnet-v3/blob/master/Microsoft.Azure.Cosmos.Samples/Usage/SystemTextJson/CosmosSystemTextJsonSerializer.cs?

@hkusulja while the dependency is not yet removed, have you considered adding a customer serializer like shown in https://github.com/Azure/azure-cosmos-dotnet-v3/blob/master/Microsoft.Azure.Cosmos.Samples/Usage/SystemTextJson/CosmosSystemTextJsonSerializer.cs?

Does this solve the Linq property name casing issue?

It also doesn't solve the issue of Enums as strings in linq.

Please update status and information about statement:

Official support for System.Text.Json is also planned for v4 of the client.

Getting this through would be extremely appreciated.
Using cosmosdb means introducing another third party dependency of newtonsoft for serialization, especially with the security risks it can potentially impose.

Any updates on this issue?

Next preview will have the bits for LINQ to work with it
I have a proposal for how this can be resolved in https://github.com/Azure/azure-cosmos-dotnet-v3/issues/4325
@kirankumarkolli you could close this in favor of it and we'd not need a vnext :D

@kirankumarkolli Can you add a status summary covering the current state of play (what is the shortest one liner OOTB right now) and close this please?

@bartelink updated the description, this issue is to track the default change for vNext so will still be open.

And the main package won't have a dependency on JSON.net in vNext, right?
https://github.com/Azure/azure-cosmos-dotnet-v3/issues/37

Yes that's the goal and this issues is to track it.


Add a core test creating a maximum size inscription

Actually I don't think we really need to do this, if we have the max weight test.


Stopped working

I get this error every time I use this.

can confirm

can confirm

+1

Any work arounds

Hi! Yeah, unfortunately, google has changed the way you need to send and process the request, therefore it wasn't working. Luckily, the creator of @vitalets/google-translate-api was able to find a workaround for this, so I have updated the repository and it should work fine now.


Linking Modals to Buttons

wip

I am also noticing that the icons are always corresponding to developer and not designer.

ready for review, fixed icon bug and also refactored some modal code


pipenv editable install not detected

I have an editable install of a python module in a virtual environment managed by pipenv. The package was installed directly from git.
The package is not recognized by pylance unless it is explicitly added to the python.analysis.extraPaths list, even though it is located in the src (vs the lib/.../site-packages) directory. It is included in site-packages.
 Environment data

Language Server version: 2023.5.50
OS and version: darwin arm64
Python version (and distribution if applicable, e.g. Anaconda): 3.9.16
python.analysis.indexing: true
python.analysis.typeCheckingMode: basic

 Code Snippet

import great_expectations as gx # great_expectations has been installed directly from github using pipenv

 Repro Steps
Install an editable module using pipenv:
pipenv install -e git+https://github.com/great-expectations/great_expectations.git@develop#egg=great-expectations
 Expected behavior
Module is correctly analyzed.
 Actual behavior
Module must be included in analysis.extraPaths
 Logs
Here are what I think are the relevant logs...
2023-06-08 16:05:46.403 [info] [Info  - 4:05:46 PM] (88455)   Looking in extraPath '/Users/james/.local/share/virtualenvs/data-validation-3VeP54W-/src/great-expectations'
2023-06-08 16:05:46.403 [info] [Info  - 4:05:46 PM] (88455)   Attempting to resolve stub package using root path '/Users/james/.local/share/virtualenvs/data-validation-3VeP54W-/src/great-expectations'
2023-06-08 16:05:46.404 [info] [Info  - 4:05:46 PM] (88455)   Attempting to resolve using root path '/Users/james/.local/share/virtualenvs/data-validation-3VeP54W-/src/great-expectations'

Hard to know what are the right missing logs.

Please see our documentation on resolving imports from editable installs: https://microsoft.github.io/pyright/#/import-resolution?id=editable-installs

Thanks for pointing me to the docs--I was able to get it to work as expected by using a pip install instead of pipenv install; if it's an issue at all, then, it would seem to be with having pipenv allow that flag. Thank you!