Run-TestsInNavContainer - test failing, when running through client - succeeded

Hi,
I have now situation, where tests, triggered through Run-TestsInNavContainer are failing (3 of 12). If I run same test suite through web client, all are green.
Two errors are about "record... already exists"...
It leads me to one thing: how are the tests started? Isn't there some parallelism? Seems like it run both codeunits in the testsuite in parallel, not in serial like when started directly from UI.

Run-TestsInNavContainer is running the tests through the UI - it is emulating a client and pressing the button one by one. In fact, if you run it with -detailed you will see the result for every test before the next is started.
What BC version is this?

I am running it with -detailed...
Version: Platform 14.0.29530.29573 + Application 29654 (W1 14.0)
But thanks, the detail I needed is "pressing the button one by one". This is the difference. Running one by one is doing separate transaction per test, but running from UI with "run all" (which is default I am using when doing it manually) is doing transaction per codeunit (based on the isolation level of the test codeunit). There was dependency in my test making one to fail when running separately...

Does "pressing the button" mean that each test function (not test codeunit) is executed separately, in a separate transaction?

Thanks, sounds like I will have to refactor the method to run the codeunits unless you specify a specific testfunction.

Looking at the code - that is how it should work. If you do not specify testFunction - it should run selected on each codeunit.

@kine - do you specify testFunction?

That's great, then I have no complaints at all :)

This image:

Shows that normally - the codeunits are run (-detailed just displays the functions as well) - if you specify -testFunction - the individual functions are run.
@kine - should we do a screen share session to investigate your issue?

During the screen share session we found out that the problem was in my case that the powershell function is connecting to first company available, not the one set for the user, thus the tests were failing because existing data in demo company, but manually tested in empty company.
There is no CompanyName parameter to run the powershell script for specific company or it should take the one set for the user, like when opening the client in standard way. Or both (if not specified, use the default one).

This is actually caused by Windows Authentication - with NavUserPassword it worked exactly as you expected.
The problem with Windows Authentication is, that the powershell session is running inside the container and in order to make that work, I have to create another Windows Auth user (containeradministrator) and use that for the connection - I haven't found other ways of doing that.
I have fixed this by adding the CompanyName parameter and grabbing the company from the original Windows User if you didn't specify CompanyName.
If you specify CompanyName - it always takes precedence.
I do recommend using NavUserPassword for CI/CD scenarios:-)

Included in <IP_ADDRESS>


InterfaceRequest

Currently, we basically make all the interfaces look a bit like a WSGI request, then http.Request can use .environ and stuff to find information like IP address and host.
I think it would be better to have an InterfaceRequest wrapper that would take the raw request and then the interface could implement the supported methods, and then http.Request would be instantiated like this:
request = Request(InterfaceRequest(raw_request))

I think the class would be something like:
class InterfaceRequest(object):
    def __init__(self, raw_request):
        self.raw = raw_request

    def get_headers(self):
        raise NotImplementedError()

    ...

This was my original note:

create an interface request class that can handle bridging between the interface (uwsgi, asgi) and the request object. So basically, the interface would create an InterfaceRequest class and pass that to Request, then Request would use InterfaceRequest for things like .ip and .scheme

So the big question is, why would we do this over just having these things set directly on http.request in the interface's create_request method like we kind of do now? This solution would make it so the code is run on demand instead of all ran in the create_request method. Likewise, it would provide a standard template for what needs to be implemented, right now to add a new interface I usually look at what a previous interface's create_request method is doing.
Biggest annoyance would be all the tests that just create a request object, I'd need to change all those to take the InterfaceRequest instance, and there are a lot of tests.

I think the new ASGI code renders this idea obsolete


Problem Searching for Items in ES

I'm having trouble searching for items posted to ES.  Let me give you the lay of the land first.
Here's the static logging class.
public static class Log {
            ...
	public static Task Indeed(IndeedLog wse) {
		wse.Timestamp = DateTime.Now;

		return IndexWebservice(wse);
	}

            private static Task IndexWebservice<T>(T webservice) where T : class {
		var indexName = "webservices-" + String.Format("{0:yyyy.MM.dd}", DateTime.Now);

		// Daily webservice index must be created if it hasn't been already
		if (!Elasticsearch.Cluster.Indices.Contains(indexName)) {
			Elasticsearch.Client.CreateIndex(c => c.Index(indexName)
				.InitializeUsing(new IndexSettings { NumberOfReplicas = 1, NumberOfShards = 5 })
				.AddMapping<WebserviceLog>(m => m.MapFromAttributes())
				.AddMapping<IndeedLog>(m => m.MapFromAttributes())
				.AddMapping<BackgroundCheck>(m => m.MapFromAttributes()));

			Elasticsearch.Cluster.UpdateIndex(indexName);
		}

		return Elasticsearch.Client.IndexAsync(webservice, i => i.Index(indexName).Refresh());
	}
            ...
 }

Here's our logged POCO.
    [ElasticType(Name = "indeed")]
public class IndeedLog {
	[ElasticProperty(Type = FieldType.String, Index = FieldIndexOption.NotAnalyzed)]
	public string Id { get; set; }
	[ElasticProperty(Name = "@timestamp")]
	public DateTime Timestamp { get; set; }
	public string RawJson { get; set; }
}

And finally a singleton class to setup our connection to ES
   public class Elasticsearch {
	// Static holder for instance
	private static readonly Lazy<Elasticsearch> Connection = new Lazy<Elasticsearch>(() => new Elasticsearch());
	private static readonly StaticConnectionPool Pool = new StaticConnectionPool(new[] { new Uri("http://log1.ourserver.com:9200"), new Uri("http://log2.ourserver.com:9200") });

	/// <summary>
	/// Elasticsearch client
	/// </summary>
	private ElasticClient ElasticClient { get; set; }

	// Private constructor to prevent direct instantiation
	private Elasticsearch() {
		var settings = new ConnectionSettings(Pool);

		// Default map type is required because there's no default index
		settings.MapDefaultTypeIndices(m => m.Add(typeof(EmailLog), "emails-*"));
		settings.MapDefaultTypeIndices(m => m.Add(typeof(Session), "sessions-*"));
		settings.MapDefaultTypeIndices(m => m.Add(typeof(SessionHit), "sessions-*"));
		settings.MapDefaultTypeIndices(m => m.Add(typeof(IndeedLog), "webservices-*"));
		settings.MapDefaultTypeIndices(m => m.Add(typeof(BackgroundCheck), "webservices-*"));

		ElasticClient = new ElasticClient(settings);

		Indices = new List<string>();
	}

	// Public accessors
	public static Elasticsearch Cluster {
		get { return Connection.Value; }
	}

	public static ElasticClient Client {
		get { return Connection.Value.ElasticClient; }
	}

	// Current indices
	public List<string> Indices { get; private set; }

	public void UpdateIndex(string indexName) {
		var index = indexName.Split('-')[0];

		if (!Indices.Any(x => x.Contains(index))) {
			Indices.Add(indexName);
			return;
		}

		for (var i = 0; i < Indices.Count; i++) {
			if (Indices[i].Contains(index)) {
				Indices[i] = indexName;
				return;
			}
		}
	}
}

So I have an action in our system that posts to ES by simply doing something like
Log.Indeed(new IndeedLog{ Id = GUID.ToString(), RawJson = "{}"})
And up until recently, I was able to pull that item back out with the following. GUID is the same guid used to post it (it's being passed around with a message queue).
Elasticsearch.Client.Search(il => il.Query(q => q.Term(t => t.Id, GUID))).Documents
Myself and my teammate that setup the system spent over half the day trying to figure out why we can't pull items back out.  We've run out of ideas on what it could be.  So we come humbly to this forum looking for ideas/solutions.
From the singleton, you might notice how we have a few other POCOs we post.  Session, for instance, has no problems being retrieved.  I've compared both processes and as far as I can tell, they are posted and retrieved the exact same way.
TIA,
Eric

Hey @aluink sorry for the slow reply.  Have you figured this out yet?  If not, mind posting the contents of ConnectionStatus from the response?

Hey Greg,
We actually got it working.  Now that I read it, that code up there isn't exactly what we had when it was having issues.  What is posted is quite close to what we have now that works.  The original code had the POCO inheriting from another one.
    public static Task Webservice(WebserviceLog wse) {
		wse.Timestamp = DateTime.Now;

		return IndexWebservice(wse);
}

    private static Task IndexWebservice<T>(T webservice) where T : WebserviceLog {
		var indexName = "webservices-" + String.Format("{0:yyyy.MM.dd}", DateTime.Now);

		// Daily webservice index must be created if it hasn't been already
		if (!Elasticsearch.Cluster.Indices.Contains(indexName)) {
			Elasticsearch.Client.CreateIndex(c => c.Index(indexName)
				.InitializeUsing(new IndexSettings { NumberOfReplicas = 1, NumberOfShards = 5 })
				.AddMapping<WebserviceLog>(m => m.MapFromAttributes())
				.AddMapping<IndeedLog>(m => m.MapFromAttributes())
				.AddMapping<BackgroundCheck>(m => m.MapFromAttributes()));

			Elasticsearch.Cluster.UpdateIndex(indexName);
		}

		return Elasticsearch.Client.IndexAsync(webservice, i => i.Index(indexName).Refresh());
	}

[ElasticType(Name = "webservice")]
public abstract class WebserviceLog {
	[ElasticProperty(Type = FieldType.String, Index = FieldIndexOption.NotAnalyzed)]
	public string Id { get; set; }
	[ElasticProperty(Name = "@timestamp")]
	public DateTime Timestamp { get; set; }
}

[ElasticType(Name = "indeed")]
public class IndeedLog : WebserviceLog {
	public string RawJson { get; set; }
}

[ElasticType(Name = "backgroundCheck")]
public class BackgroundCheck : WebserviceLog {
	public string Provider { get; set; }
	public string Data { get; set; }
}

Our current working solution is to have it all in a single class like this:
[ElasticType(Name = "webservice")]
public class WebserviceLog {
	[ElasticProperty(Type = FieldType.String, Index = FieldIndexOption.NotAnalyzed)]
	public string Id { get; set; }
	[ElasticProperty(Name = "@timestamp")]
	public DateTime Timestamp { get; set; }
	public string Provider { get; set; }
	public string Data { get; set; }
}

The rest of the code is the same as my first post.  We really liked the benefits of the inherited model.  Do you happen to know why it would give us problems?  Are inherited POCOs supported by the API?

Yes, inheritance is supported.
Shooting from the hip here since I don't know the full query you're executing, but since you don't have a default index mapped to WebserviceLog, if you call Search<WebserviceLog>(...) and pass it an instance of BackgroundCheck or IndeedLog, NEST is going to use WebserviceLog to infer the index and not the latter two types which you are mapping to webservices-*.
Try adding:
settings.MapDefaultTypeIndices(m => m.Add(typeof(WebserviceLog), "webservices-*"));

Also, unrelated but you may also want to use a term filter here instead of a query:
il => il.Query(q => q.Term(t => t.Id, GUID))

to take advantage of the bitset caching in Elasticsearch.  More on that here.

Alright, great. Knowing that inheritance is supported gives me the hope of knowing it can work.  When I get some breathing room in my project cycle, I'll try to work it back in.
Thanks for the help.  I'll close this for now, but I may comment again when I find time to work on it some more.


Face_Orientation_Predictor

New Feature: Face Orientation Prediction
We are excited to announce the addition of a new feature to our project: Face Orientation Prediction using a neural network. This enhancement allows users to predict the orientation of grayscale face images, categorizing them into one of four possible orientations: straight, up, left, and right.
Feature Highlights
Data Collection: Automatically download face images from a specified URL and save them locally.
Preprocessing: Resize images to 32x30 pixels and normalize pixel values for improved model performance.
Neural Network Training: Train a neural network to classify face orientations with high accuracy.
Model Persistence: Save the trained model to a file for easy reuse and deployment.
Prediction and Visualization: Load the saved model and predict orientations for new images, with an option to visualize predictions.
How to Use
Download the Dataset:
Update the script with the URL containing face images.
Run the script to download and save the images.
Preprocess the Images:
Utilize the provided functions to resize and normalize the images.
Train the Model:
Train the neural network using the preprocessed dataset.
Save and Load the Model:
Save the trained model for future predictions.
Load the saved model to predict orientations for new images.
Visualize Predictions:
Display images with their predicted labels using the visualization functions.
We believe this feature will be incredibly useful for projects involving facial recognition, image processing, and machine learning.

Please mention issue number u have solved

#1761

Hi @akshitagupta15june mam  I noticed my pull request #1789  hasn't been assigned any labels yet. Could you please review and label it? I believe this might help me appear on the Girlscript Summer of Code leaderboard. Thanks!

@akshitagupta15june yes mam I have added the implementation inside the awesome face operation folder. Please review and merge this pull request at your earliest convenience mam as the GSSOC'24 leaderboard requires merged pull requests for participants to appear on the leaderboard. Thank you!


Fix CVE-2024-24786 in cf-cli and terraform for 2.0

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ *] The toolchain has been rebuilt successfully (or no changes were made to it)
[ *] The toolchain/worker package manifests are up-to-date
[ *] Any updated packages successfully build (or no packages were changed)
[ *] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ *] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ *] All package sources are available
[ *] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ *] LICENSE-MAP files are up-to-date (./LICENSES-AND-NOTICES/SPECS/data/licenses.json, ./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md, ./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)
[ *] All source files have up-to-date hashes in the *.signatures.json files
[ *] sudo make go-tidy-all and sudo make go-test-coverage pass
[ *] Documentation has been updated to match any changes to the build system
[ *] Ready to merge

Summary

Fix CVE-2024-24786 in cf-cli and terraform for 2.0
Change Log

Patch cf-cli to fix CVE-2024-24786
Patch terraform to fix CVE-2024-24786

Does this affect the toolchain?

NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-2024-24786

Test Methodology

Local build successful
Buddy build - https://dev.azure.com/mariner-org/mariner/_build/results?buildId=691319&view=results

Auto cherry-pick results:

main :white_check_mark: -> https://github.com/microsoft/azurelinux/pull/11469

Auto cherry-pick pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=695404&view=results


NH-29380 Remove unused enable_sanitize_sql config

Similar to https://github.com/solarwindscloud/solarwinds-apm-python/pull/163
This removes enable_sanitize_sql from APM Python config. It is not used for anything at the moment. Sanitization of sql statements used for any span attributes is partially handled by the Python query libraries at the moment (prepared statements). Further sanitization (e.g. hardcoded statements) should be handled upstream by the Otel instrumentation libraries and an Otel configuration. Issues are out there for this already:

https://github.com/open-telemetry/opentelemetry-python-contrib/issues/1376
https://github.com/open-telemetry/opentelemetry-python-contrib/issues/1543

Oh man peanut brain move with the merge conflicts today 🙈


Add benchmark test for Postgres Accelerator

Close by: https://github.com/spiceai/spiceai/pull/2652


KeyInfo support in metadata

I was asked by an IdP to insert our certificate name in KeyDescriptor -> KeyInfo -> KeyName. I don't see where the KeyDescriptor could be set and I can't find much information doing a google search. Is that something that will be supported in the future? Or is there a reason why it cannot be supported that I am not getting? I appreciate any feedback. Thanks

KeyDescriptor -> KeyInfo -> KeyName is not supported instead you can use KeyDescriptor -> KeyInfo -> X509Data.

Thanks for confirming.


Don't report an error when accessing a name of a global block-scoped variable on globalThis outside of noImplicitAny

fixes https://github.com/microsoft/TypeScript/issues/58345

Even though the bug is labelled Help Wanted, I didn't see a consensus on the change we should make here. So that's probably the first task.
Also, will this fix the bug for the person who filed it? I'm not sure whether restricting this change to noImplicitAny: false makes sense -- since it really is a circularity error. I personally think it should remain one, but you can make the case that removing it here is useful. (and in JS there is special-case code to ensure that this pattern does work).

It’s not a circularity. Block-scoped top-level variables define global variables but not globalThis properties (this is different from var)


Ability to hide title

It would be great to be able to hide the title. Having no title on schema and form should result in no fieldset title.
// Schema
name: {
  type: 'object',
  properties: {
    first: {
      title: 'First Name',
      type: 'string'
    },
    last: {
      title: 'Last Name',
      type: 'string'
    }
  }
}

// Form
{
  key: 'name',
  items: [
    {
      title: 'FIRST_NAME',
      key: 'name.first',
      type: 'text',
      placeholder: 'FIRST_NAME',
      validationMessage: {}
    }, {
      title: 'LAST_NAME',
      key: 'name.last',
      type: 'text',
      placeholder: 'LAST_NAME',
      validationMessage: {}
    }
  ]
}

Does, {"notitle": true} in the Standard Options not do this? How should it be different?

Maybe "supressPropertyTitles" in Global Options would work for you?

@Anthropic @davidlgj I will check this today, thank you for the suggestions! Also, I think having the styling options available in the Standard Options will help as well.

Fieldsets do not seem to be respecting the notitle property.
https://github.com/Textalk/angular-schema-form/blob/master/src/directives/decorators/bootstrap/fieldset.html#L2

"Fieldsets do not seem to be respecting the notitle property. ". I found this is true? Is there any solution to workaround this?

@ricklxm  Basically its just a ng-hide="form.notitle" that is needed on the legend tag. It just needs to be done :)

@ricklxm @davidlgj Please take a look at my PR #414 if you get a chance. I was just too lazy to do it when I originally posted lol

Thanks @stramel! Your PR is now in the development branch.


User variable not resolved

I have defined a user variable PROJECT_HOME in the "User Defined Variables" element.
The PROJECT_HOME variable contains this code:
${__BeanShell(import org.apache.jmeter.services.FileServer; FileServer.getFileServer().getBaseDir();)}

After that, I reuse it in the "CSV Data Set Config" element, in the filename as follow:
${__env(DATA_CONFIG,,${__eval(${PROJECT_HOME})})}/Credentials.csv

With the "HTTP(S) Test Script Recorder" the resolution of the variable PROJETC_HOME is resolved correctly, as follow:
2021-12-07 10:20:39,175 INFO o.a.j.s.FileServer: Close: D:\Projets\Load-Tests\Auto-Correlation/Credentials.csv
But with this plugin I have the following issue:
2021-12-07 10:09:08,562 INFO o.a.j.s.FileServer: Stored: ${PROJECT_HOME}/Credentials.csv
Environment
Windows 10

JMeter version 5.4.1
Correlation Recorder: latest (1.2.1)
Plugin Manager (if applicable)
Protocol version (if apply, eg: Siebel CRM IP17 & IP18)

Step to Reproduce

Define the PROJECT_HOME variable as
${__BeanShell(import org.apache.jmeter.services.FileServer; FileServer.getFileServer().getBaseDir();)}  in the config element "User Defined Variable".
In the config element "CSV Data Set Config" set the "Filename" as ${__env(DATA_CONFIG,,${__eval(${PROJECT_HOME})})}/Credentials.csv
Define a simple scenario
Click on Start

Expected behaviour
I expected that the ${PROJECT_HOME} is resolved as D:\Projets\Load-Tests\Auto-Correlation instead of remaining as ${PROJECT_HOME}
Complementary information
none
Additional context
none

Hello @plo750,
This is pretty strange, I reproduce it and just added it to our backlog.
We will be tackling this in the future and, whenever a release is ready with the fixes, I'll let you know.
Thanks for taking the time to report this behaviour, we really appreciate it.
Happy Hacking!


Highlight confusable Unicode characters

There could be a setting e.g. "editor.highlightUnicode": "none" | "non-ASCII" | "confusables".
Links:

http://www.unicode.org/reports/tr39/
https://www.unicode.org/Public/security/14.0.0/IdentifierStatus.txt
https://www.unicode.org/Public/security/14.0.0/confusables.txt
https://www.unicode.org/Public/security/14.0.0/confusablesSummary.txt
https://www.unicode.org/Public/security/14.0.0/intentional.txt
https://en.wikipedia.org/wiki/Unicode_control_characters

Thoughts:

Characters that are not present in IdentifierStatus.txt have the status Restricted and they could be obsolete, deprecated, limited use, or not a character, etc. They should not be used.
Control characters could also be highlighted
(maybe) link to e.g. https://util.unicode.org/UnicodeJsps/character.jsp?a=202E

https://github.com/microsoft/vscode/tree/alex/unicode-characters

Semi-related; I asked for a not-too-dissimilar feature some time ago: https://github.com/microsoft/vscode/issues/1727

Semi-related; I asked for a not-too-dissimilar feature some time ago: https://github.com/microsoft/vscode/issues/1727

@Kroc I apologize for missing or not answering to your later comments from #1727. I don't exactly remember what happened back then, but I can speculate that perhaps the issue started out as a discussion about rendering all characters as monospace (like e.g. in a terminal) and I considered that to be a feature request when I saw that Chromium does the same rendering that we do.
Later the discussion mentioned that there might be security implications, but what was mentioned was watermarked content using zwj, which is not a security problem per se, it is a privacy problem (AFAIK watermarked content might be used to track down a specific person that might have copy-pasted content).
While this is a known problem since more than 10 years, I think the merit of https://www.trojansource.codes/ is that they have shown a way to weaponize it.
Also, AFAIK no editors today try to tackle confusables, like e.g.:
( а → a ) CYRILLIC SMALL LETTER A → LATIN SMALL LETTER A

@hediet A first implementation could focus just on https://www.unicode.org/Public/security/14.0.0/IdentifierStatus.txt and would highlight Restricted code points.


Install Node Exporter by Ansible

Tutorial: https://grafana.com/docs/grafana-cloud/monitor-infrastructure/metrics/prometheus-config-examples/noagent_linuxnode/
node_exporter: https://github.com/prometheus-community/ansible/tree/main/roles/node_exporter
prometheus: https://github.com/prometheus-community/ansible/tree/main/roles/prometheus

prometheus_remote_write
https://grafana.com/orgs/hongbomiao/hosted-metrics/55773

Close to in favor of #10467


[FancyZones] Turn off window transparency when no zones are present

Holding shift and dragging the PowerToys Window caused it to become transparent, no zones were showing
Environment
Windows build number: 10.0.18363.720
PowerToys version: Release v0.16.0
PowerToy module for which you are reporting the bug (if applicable):
Fancy Zones
Steps to reproduce
Navigate to settings, fancy zones, make sure that the second option to drag using shift is enabled. Try shift and drag the settings Window, observe the opacity of the Window.
Expected behavior
Zones should pop up
Actual behavior
No zones pop up, and Window becomes transparent.
Screenshots

Hi @salmanmkc
did you first select a zone layout for the current monitor?
If you don't like the transparency, there is an option in the FancyZones settings to turn it off.

Ah thank you, that fixes it. Although the issue I guess would be that this happens when no zones are set up. A check to see if zones are set up should be done before. I may look into implementing this and submit a PR.

Any change to the current behavior needs to support multi monitor scenarios (and in the future it may also have to support dragging the window to a virtual desktop).
Even if I understand the reasons for the change, the change has a significant impact that needs to be tested in different scenarios and has limited benefits for the common case.
At present we cannot consider this a high priority fix, so it would be on you to test it thoroughly for the multi monitor scenarios.

Will try to do that, I can't seem to build the project, I'm getting the cannot open source file "altstr.h" error

Make sure all the nuget packages have been downloaded.

All nuget packages installed and updated, am I running the wrong solution? PowerToys solution?

That is the correct one, make sure that the active platform in the VS toolbar is x64 (it should be the only available)

Yep, debug, x64, runner, Local Windows Debugger

I think we should consider some type of feedback to inform the user that there are no active zonesets on the current monitor, rather than simply not doing anything.
For example a small overlay on the top-left of the screen that says "No layout selected" or something similar.

Tracking in https://github.com/microsoft/PowerToys/issues/6611

Tracking in https://github.com/microsoft/PowerToys/issues/6611

I don't think we'll fix this.  As the end user, you're opt'ing for a mix mode behavior


Dynamic objects do not receive SDFGI reflections when LightmapGI is used.

Tested versions
v4.2.1.stable.official [b09f793f5]
System information
Godot v4.2.1.stable - Windows 10.0.22621 - Vulkan (Forward+) - dedicated AMD Radeon RX 6700 XT (Advanced Micro Devices, Inc.; 31.0.23013.1023) - AMD Ryzen 7 5700X 8-Core Processor (16 Threads)
Issue description
Exactly as the title said. Disabling probe generation does not fix the issue.
As a side note, because of an already existing bug, toggling the visibility of the lightmap causes SDFGI reflections to become visible until you reload the scene or move a dynamic object. This behavior is almost desirable if not for the fact that the object no longer receive indirect lighting from LightmapGI.
Steps to reproduce

Open MRP
Reflections are not initially visible on the sphere
Disable LightmapGI
Reflections now appear properly on the sphere

Minimal reproduction project (MRP)
Lightmap.zip

If I'm not mistaken, I think https://github.com/godotengine/godot/pull/86135 would fix this.

Checked the PR out with the MRP and no luck. The PR only allowed static objects to receive reflections from dynamic GIs.

cc @guerro323

It should work now in the latest https://github.com/godotengine/godot/pull/86135 commit

I can confirm it works for both SDFGI and VoxelGI with that PR :slightly_smiling_face:


ballerina shell script is ignoring BALLERINA_HOME var if its set

The code does this:
# set BALLERINA_HOME
BALLERINA_HOME=`cd "$PRGDIR/.." ; pwd`

It needs to check whether it exists first.

That check was removed because of other ballerina servers in same machine also refer the same BALLERINA_HOME location instead of their home location. Then all refer the common location for their logs, etc and sometime server failed to start as well.

That is correct. We need to ignore the value of the existing value.


spokes to spokes connection without NVA

I wasn't ware the NVA option when I setup onprem to hub to spokes network, but I found a workaround to make sopkes to spokes connection work. I think it's much easier and simpler to setup. the trick is just add a bigger address space into the S2S VPN's local network gateway.
Lets say the onprem network has address <IP_ADDRESS>/16,   hub vnet has address <IP_ADDRESS>/24, I can configure spoke vnet has address <IP_ADDRESS>/24, <IP_ADDRESS>/24, <IP_ADDRESS>/24, etc.  then I can add <IP_ADDRESS>/19 into local network gateway as VPN remote address. This will trigger hub gateway to advertise <IP_ADDRESS>/19 to all the spokes network. and it seems doesn't cause any problem, since hub and spokes network has more accurate route for individual 10.1.xx.0/24 subnet.

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

ID: 104eba32-c77e-b013-e161-ff616d16a0f7
Version Independent ID: 3bc27d0a-ef32-f5f6-4c85-fe8e2e45155e
Content: Implementing a hub-spoke network topology in Azure
Content Source: docs/reference-architectures/hybrid-networking/hub-spoke.md
Service: guidance
GitHub Login: @telmosampaio
Microsoft Alias: pnp

Thank you for sharing, @jackchenquest!
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


sync Query filter expression using "gt" instead of "ge"

Currently, in datasync, we are using the following as our filter expression for delta queries (sync queries):
https://github.com/aerogear/graphback/blob/c0056b4bd3250df8a3e39131302dcd3138b312b9/packages/graphback-datasync/src/providers/DatasyncMongoDBDataProvider.ts#L108-L113
Would it be better if we had ge instead of gt ?
It seems to me (as far as I can understand) that gt might miss some updates that may be happening at the same time as the sync query, therefore, would it be better if we did ge(greater than or equal) instead of gt(strict inequality)?

/cc @craicoverflow, @machi1990, @wtrocki

Yep. Looks like oversight

@machi1990 ROLF - now I will never know if that was you doing all nighter or you confired bot that has AI and applies labels :D

@machi1990 ROLF - now I will never know if that was you doing all nighter or you confired bot that has AI and applies labels :D

It's indeed confusing. We should have a dedicated account soon.
It's me not a bot here ;-)

No way. We need to keep it as it fits perfectly to the joke of yourself having 24 hours coverage... I would extend the boot to do more actions like:
/Matchi approve please
Imagine your contribution graph after that

@wtrocki did you mean to close this?

Yep.. That was discussion issue. @ssd71 will apply this in single line on the existing PR.


The in operator returns True after cache expiration

After a cached expires (with finite ttl) the in operator still returns True (that the item is in the cache) until the get() method is called and corrects the internal state. It should be consistent with get() and return False right after expiration happens. Observed in the latest cachout==0.11.2.
It seems that the __contains__() method is not checking self.expired(key), in contrast to _get() which does. That would be the cause.
In addition there's a method has() which internally calls get() and checks expiration and distinguishes non-presence from None value. Does it mean that not checking expiration in __contains__ is intentional?
import cacheout
# works for all implementations (descendants) the Cache class
cache cacheout.Cache(ttl=1)

Only calls to the in operator. After expiration it returns the wrong value.
cache.set('foo', 'bar')
for i in range(3):
    print('foo' in cache)
    time.sleep(0.2)

# True
# True # <-- wrong
# True # <-- wrong

Call the get() modifies the state and the in operator returns correct value afterwards.
cache.set('foo', 'bar')
for i in range(3):
    print('foo' in cache, cache.get('foo'))
    time.sleep(1.5)

# True bar
# True None # <-- wrong
# False None

cache.set('foo', 'bar')
for i in range(3):
    print(cache.get('foo'), 'foo' in cache)
    time.sleep(1.5)

# bar True
# None False # OK
# None False

Thanks for reporting! Definitely a bug.
Fixed in 0.12.1.

Thank you for a very quick bug fix!


React-Native on iOS canvas view does not show any drawings

Hello there,
Just created an app using this guide: https://alibaba.github.io/GCanvas/docs/Integrate GCanvas on ReactNative.html
Done automatic and manual installation.
The example code does not show any drawn images, it just shows a clear background.
Everything renders without any issues. However, I get "var canvas_tag = 3" (not "2" as shown in the example).
Tried calling draw function in the constructor and ComponentDidMount.
Is this a bug or are there any issues in the guide?

Solved by calling the draw() function from TouchableOpacity event.


IME Problem in WebView

Hello, I am Korean and made some desktop application with atom-shell.
I found a problem for IME in WebView on OSX

Charaters did not composite each other.
I had seen it when using some applications on OSX before, anyway..
Did I do something wrong?
I just coded like below with atom-shell-v0.20.7-darwin-x64.
<!DOCTYPE html>

  <title>SmartNamer</title>

  <input type="text">
  <webview id="guest"
  httpreferrer="http://www.mabook.com/smartnamer/#app"
  src="http://www.mabook.com/smartnamer/index.php"></webview>

Additional Info.

I just tested more condition.

In Main Browser, no problem.
In IFRAME, no problem.
In WebView, problem!!

main.html
<!DOCTYPE html>

main
    <input type="text">

iframe

webview
    <webview
    httpreferrer="http://www.mabook.com/smartnamer/#app"
    src="test.html"></webview>

test.html

		<input type="text">

Please check it out.

I can reproduce this bug in the <webview> of Chrome extensions, I suggest reporting it to upstream Chromium because it can not be fixed on atom-shell side.

OK. Thanks for your suggestion.
I am going to report this to upstream Chromium.
When it has been solved,  I will close this issue.

@zcbenz
I reported this problem to upstream Chromium.
Thanks.

It has been fixed in recent Chrome versions.


Provide a list of running notebook clusters

I opened notebooks in a few different billing projects.
I closed those windows.
I later realized that I had left notebook clusters running.
I needed to remember which notebooks I had open in which projects.
Would be great to get a list and an easy way to shut them all down.

Hi Matt! Our PO, @bradtaylor, is out this week, so I'm going to try to give you an answer on this one!
We're trying to take care of clusters as much as possible for users so they don't have to think about them. Saturn only uses clusters that it automatically creates itself, and all Saturn clusters are created with a 30 minute auto-pause timeout. This means after 30 minutes of not being in use (either by a user in a notebook or running a computation), they will automatically suspend themselves. When you go to open a notebook, Saturn will automatically resume the cluster from its paused state.
When a cluster is paused, you only pay for storage, something on the order of dollars per month. Is this sufficient for what you're looking for?

Specifically, I'm informed that it runs about $20 for a cluster with a 500GB disk, which is our default. One is created for each Firecloud billing project that you have access to.

If the auto-pause feature works then as a user I probably don't need a list.
But that is putting a lot of faith in the auto-pause mechanism.
What happens when it doesn't work?
As an end user, will I just see a large bill at the end of the month?
What is my recourse?
Is this mechanism in place today? I have a cluster that is running that I was not using for a couple of days.

Having turned off autopause, we are returning to this. We are currently implementing a page to manage billing projects per #1181. It might make sense to show the cluster as part of the billing projects list, since cluster:project is 1:1 for a given user. Users could pause/delete/start/create their cluster there for each project.


Fixed .smalltalk.json to run test just once

Coverage remained the same at 100.0% when pulling a6da429524d7e659acea3515ee996654fc23ee35 on fixed-test-suite into ace3a73037b9c6434353bbf34f52a7ca3e5e572e on master.


Add hover styles to the whole item in the sync interval selection

The text should have a style, not only the radio buttons.

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


Path

Which path should I put this file in？

The folder literally tells you to extract into the Content Warning folder

Ze mode is not working with patch 1.7.a


Problem of Synchronous Transfer?

How does Lattigo solve the problem of synchronous transfer? Isn't this required to symmetrically exchange decryption data? etc?
Is synchronous transfer required for secure key-exchange?

By synchronous transfer, are you referring to the need of parties to establish confidential and authenticated channels ?


Handle Dynamic Connection Strings

We need to handle dynamic connection strings. The idea is that different clients of our Web API need to have different databases. We want that when client A makes a request to the Web API, database X is used; & when a client B does the same, then database Z is used

In above mentioned the issues. comes under the multitenancy while they are give the request capture the which type client through middleware or pipeline then make an object that particular repository.


Log cause of failed attempt to open control stream

Motivation
I encountered the error Unable to open control stream due to the limit on the number of streams on the client side. I want to clarify the reason for the inability to open a control stream in logs.
Modification
Add cause of failed attempt to log message.
Result
Error looks like:
io.netty.incubator.codec.http3.Http3Exception: Unable to open control stream, cause: io.netty.incubator.codec.quic.QuicException: QUICHE_ERR_STREAM_LIMIT
	at io.netty.incubator.codec.http3.Http3ConnectionHandler.lambda$createControlStreamIfNeeded$0(Http3ConnectionHandler.java:91)
...
io.netty.incubator.codec.http3.Http3ConnectionHandler.createControlStreamIfNeeded(Http3ConnectionHandler.java:88)
	at io.netty.incubator.codec.http3.Http3ConnectionHandler.channelActive(Http3ConnectionHandler.java:152)

Thanks a lot @abezhovets


Adding underlay BGP testcase

This test covers underlay bgp established verification tested with snappi

TODO: add this in sonic-mgmt


Fix tflint - switch from reviewdog/action-tflint to terraform_tflint

Support for monorepo is not excellent in the reviewdog design. We need to set the working directory and when there is change to multiple env, have to rewrite to support matrix build.
https://github.com/reviewdog/action-tflint/issues/25
To Do:

[ ] check on automatic init when run with --modules flag ( https://github.com/antonbabenko/pre-commit-terraform/issues/87 )
[ ] add options to skip warning
[ ] clean code

When --modules disabled,
https://github.com/SPHTech/bt-drupal-infra/actions/runs/3145865576/jobs/5113661015

When --modules enabled.
https://github.com/SPHTech/bt-drupal-infra/actions/runs/3145560001/jobs/5113420619

Should we keep the warnings enabled? I feel addressing them helps keep the code concise.
Also if we were to switch to this version of tflint, repositories having their checks ok'ed before will be blocked if tflint issues starts popping up. Will cause disruption if there's a huge list of errors to fix. Suggest we have a on.workflow_dispatch.inputs to allow manual skips of tflint checks. But set default to not skip. We enable this switch for a set period of time allowing code owners time to fix any issues blocking their checks before we remove the option.

@niroz89 let's get this in?

Im good with the changes, has been tested on awedio-infra repo. @niroz89 take one last pass at the changes?

Verified working on the following cases:
No errors - passes - https://github.com/SPHTech/awedio-infra/actions/runs/3936747293
Errors with skipped true - passes https://github.com/SPHTech/bt-drupal-infra/actions/runs/3936944181
Errors with default skipped false - fails https://github.com/SPHTech/bt-drupal-infra/actions/runs/3936993043

I'm good with the changes, I can't approve this PR as the initial PR creator.. @lawliet89 over to you.


Return array to parameters.

I'm declaring a TargetType extension. I'm providing parameters for all the items in the enum, but for one of the items, I need the parameters to be return as an array, not a dictionary.
This might explain better:
var parameters: [String: Any]? {
    switch self {
    case .doSomething(let ArrayOfStrings):
        return ArrayOfStrings
    }
}

I'm using JSONEncoding and I need to be able to return an array.

Hello I am not sure I am understanding your question completely but are you asking how to convert a dictionary to an array?
If so, for example:
let dictionary = [ "a" : "hello", "b": "how are you"]
let arrayOfValues = Array( dictionary.values.map { $0 })

That should work.
If I misunderstood can you please clarify? Maybe with an example of your data?

@gmoalvarez No, I need to return an array, but the variable will only let me return a dictionary object. For example, look at the input that the GitHub API needs: https://developer.github.com/v3/issues/labels/#add-labels-to-an-issue

But those are not parameters, right? That's just the URL.

@BasThomas Well, if I use JSONEncoding for ParameterEncoding, wouldn't the parameters be sent as JSON? Maybe, I'm incorrect.

O I think I understand. You will need to modify the request then. Looking at the example in the Moya docs for Request Mapping:
https://github.com/Moya/Moya/blob/master/docs/Endpoints.md#request-mapping
let requestClosure = { (endpoint: Endpoint<GitHub>, done: MoyaProvider.RequestResultClosure) in
    var request = endpoint.urlRequest

    // Modify the request however you like.
    request.httpBody = Array( endpoint.parameters.values.map { $0 }) //convert to array here!
    done(.success(request))
}
let provider = MoyaProvider<GitHub>(requestClosure: requestClosure)

@gmoalvarez Awesome, but just to make sure, if I'm just using dictionary objects rather than an array, can I use JSONEncoding?

I think you can use JSONEncoding for your parameters in that case, yep!

If not, you can follow this tutorial:
https://github.com/Moya/Moya/blob/master/docs/Examples/ArrayAsRootContainer.md
I haven't used array as a root container before but it sounds straightforward since you are customizing the request anyway.


CameraDevice.front not working on Android

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Hello,
I have encountered an issue related to using CameraDevice.front on the Android platform, and I would like to report this problem. When I attempt to access CameraDevice.front, I am facing difficulties and unable to utilize the front camera on Android devices. This issue seems to be affecting the functionality of the CameraDevice.front feature, which is crucial for certain applications and functionalities.
I have tested this problem on multiple Android devices running different versions of the operating system, and the issue persists across all of them. I have also checked the necessary permissions and ensured that the camera hardware is functioning properly on these devices.
Expected results
The CameraDevice.front should successfully provide access to the front camera of the Android device.
Actual results
The CameraDevice.front fails to function correctly and does not provide access to the front camera. This issue prevents me from utilizing the front camera in my application.
This problem severely impacts the functionality and user experience of applications that rely on the front camera. It would be greatly appreciated if the development team could investigate and resolve this issue as soon as possible.
Thank you for your attention to this matter.
Best regards,
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

Hi @LynkKid, this is an android limitation not an issue with the image_picker plugin.
See this section from the plugin documentation.
https://github.com/flutter/packages/blob/d94759e4600cb39ba9cdacd1d8f9a96ebead2049/packages/image_picker/image_picker/lib/image_picker.dart#L47-L51
Closing from here as this is an android platform limitation and not an issue with the image_picker plugin.
Thank you


Will fail when using two watch-livereload tasks

When using grunt-concurrent with two watch-tasks with livereload-option set to true, this task will fail.

No, livereload will fail as you're trying to use multiple livereload instances using the same port. You need to set unique ports. In the future please use StackOverflow for support questions.

If so a error message would be helpful instead of just "Failed".


groupby+map performance regression on MultiIndex dataset

What happened?
We have upgraded to 2022.12.0 version, and noticed a significant performance regression (orders of magnitude) in a code that involves a groupby+map. This seems to be the issue since the 2022.6.0 release, which I understand had a number of changes (including to the groupby code paths) (release notes).
What did you expect to happen?
Fix the performance regression.
Minimal Complete Verifiable Example
import contextlib
import os
import time
from collections.abc import Iterator

import numpy as np
import pandas as pd
import xarray as xr

@contextlib.contextmanager
def log_time(label: str) -> Iterator[None]:
    """Logs execution time of the context block"""
    t_0 = time.time()
    yield
    print(f"{label} took {time.time() - t_0} seconds")

def main() -> None:
    m = 100_000
    with log_time("creating df"):
        df = pd.DataFrame(
            {
                "i1": [1] * m + [2] * m + [3] * m + [4] * m,
                "i2": list(range(m)) * 4,
                "d3": np.random.randint(0, 2, 4 * m).astype(bool),
            }
        )

        ds = df.to_xarray().set_coords(["i1", "i2"]).set_index(index=["i1", "i2"])

    with log_time("groupby"):

        def per_grp(da: xr.DataArray) -> xr.DataArray:
            return da

        (ds.assign(x=lambda ds: ds["d3"].groupby("i1").map(per_grp)))

if __name__ == "__main__":
    main()

MVCE confirmation

[x] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[x] Complete example — the example is self-contained, including all data and the text of any traceback.
[X] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[x] New issue — a search of GitHub Issues suggests this is not a duplicate.

Relevant log output
xarray current main `2022.12.1.dev7+g021c73e1`, but affects all version since 2022.6.0 (inclusive).

> creating df took 0.10657930374145508 seconds
> groupby took<PHONE_NUMBER>635315 seconds

<hr>

xarray 2022.3.0:

> creating df took 0.09968900680541992 seconds
> groupby took 0.19161295890808105 seconds

Anything else we need to know?

And just want to point out that the stacktraces/profile look very different between 2022.3.0 and main/latest. Looks like https://github.com/pydata/xarray/blob/021c73e12cccb06c017ce6420dd043a0cfbf9f08/xarray/core/indexes.py#L185 might be fairly expensive operation. Separately the seem to be quite a bit of time spend in copy -> copy_indexes path.

Thanks for the report @ravwojdyla.
Since #5692, multi-indexes level have each their own coordinate variable so copying takes a bit more time as we need to create more variables. Not sure what's happening with _maybe_cast_to_cftimeindex, though.
The real issue here, however, is the same than in #6836. In your example, .groupby("i1") creates 400 000 groups whereas it should create only 4 groups.

👋 @benbovy thanks for the update. Looking at https://github.com/pydata/xarray/pull/5692, it must have been a huge effort, thank you for your work on that! Coming back to this issue, in the example above the version 2022.6.0 is about 600x slower, in our internal code, the code would not finish in a reason time, so that forced us to downgrade to 2022.3.0. Are you aware of any workarounds for this issue with current code (assuming I would like to preserve MultiIndex).

Are you aware of any workarounds for this issue with the current code (assuming I would like to preserve MultiIndex).

Unfortunately I don't know about any workaround that would preserve the MultiIndex. Depending on how you use the multi-index, you could instead set two single indexes for "i1" and "i2" respectively (it is supported now, use set_xindex()). I think that groupby will work well in that case. If you really need a multi-index, you could still build afterwards from the groupby result.

Thanks @benbovy! Are you also aware of the issue with plain assign being slower on MultiIndex (comment above: https://github.com/pydata/xarray/issues/7376#issuecomment-1350446546)? Do you know what could be the issue there by any chance?

Thanks @benbovy! Are you also aware of the issue with plain assign being slower on MultiIndex (comment above: https://github.com/pydata/xarray/issues/7376#issuecomment-1350446546)? Do you know what could be the issue there by any chance?

I see that in ds.assign(foo=~ds["d3"]), the coordinates of ~ds["d3"] are dropped (#2087), which triggers re-indexing of the multi-index when aligning ds with ~ds["d3"]. This is a quite expensive operation.
It is not clear to me what would be a clean fix (see, e.g., #2180), but we could probably optimize the alignment logic so that when all unindexed dimension sizes match with indexed dimension sizes (like your example) no re-indexing is performed.

@benbovy thanks for the context and the PR #7382, exciting to see the improvement!

Closed in #7382 and #7830


fix: #856 directory stack may be empty when window is closed

The issue was that cd - may not always be able to resolved if the working directory has never truly changed. Decided to keep the beviour of overriding it so that the status buffer does not suddenly get a different cwd through some other plugin kicking in, like project.nvim


Revive nodejs version

This PR introduces cross-compilation of revive to WebAssembly (WASM) using Emscripten.
The enhancements made allow the compiler to be executed in a Node.js environment, facilitating broader application and integration possibilities.

Mostly LGTM!
However I'd like to have a build job on the CI for the emscripten target and at least a very basic test case executed.

A simple test is in second PR. I think we could add the compilation with emscripten target to this PR. WDYT?

Mostly LGTM!
However I'd like to have a build job on the CI for the emscripten target and at least a very basic test case executed.

A simple test is in the second PR. I think we could add the compilation with emscripten target to this PR. WDYT?

Fine with me to add any tests later but I'd appreciate a basic CI job ensuring that at least the build process of the compiler itself works with this PR :)

Are the emscripten LLVM build artifacts to be found anywhere? If not we should release them on our own terms, can't justify building them with each run.


problem with restore-backups

Hello,

I Create Push Backup S3 Secret .
After Create Backup Sites Job
Then Backups to Cloud
and then i try Restore Backups from Cloud I have next error -
 Traceback (most recent call last): File "/home/frappe/frappe-bench/commands/restore_backup.py", line 187, in <module>  main() File "/home/frappe/frappe-bench/commands/restore_backup.py", line 148, in main  pull_backup_from_s3() File "/home/frappe/frappe-bench/commands/restore_backup.py", line 134, in pull_backup_from_s3  for obj in bucket.objects.filter(Prefix=bucket_dir): File "/home/frappe/frappe-bench/env/lib/python3.7/site-packages/boto3/resources/collection.py", line 83, in __iter__  for page in self.pages(): File "/home/frappe/frappe-bench/env/lib/python3.7/site-packages/boto3/resources/collection.py", line 166, in pages  for page in pages: File "/home/frappe/frappe-bench/env/lib/python3.7/site-packages/botocore/paginate.py", line 255, in __iter__  response = self._make_request(current_kwargs) File "/home/frappe/frappe-bench/env/lib/python3.7/site-packages/botocore/paginate.py", line 332, in _make_request  return self._method(**current_kwargs) File "/home/frappe/frappe-bench/env/lib/python3.7/site-packages/botocore/client.py", line 316, in _api_call  return self._make_api_call(operation_name, kwargs) File "/home/frappe/frappe-bench/env/lib/python3.7/site-packages/botocore/client.py", line 626, in _make_api_call  raise error_class(parsed_response, operation_name) botocore.errorfactory.NoSuchKey: An error occurred (NoSuchKey) when calling the ListObjects operation: The specified key does not exist.

Can you help me what I do wrong?
Thanks!
I see my backup on S3 like
Amazon S3/frappe/frappe/backups/test.example.ru/20200520_005432

Can you share your yamls?

I've faced an error like this once, when I manually tried to
create backup directory structure on s3 and tried to restore.
Creating backup folder structure first
Then uploading files one by one in bucket caused error during restore.
Push backup directly creates file locations.
Manually adding folders creates folder structure along with files.
Example,
Manual case created following structure/objects,
bucket-dir/
bucket-dir/site.com/
bucket-dir/site.com/timestamp/
bucket-dir/site.com/timestamp/db.sql.gz
bucket-dir/site.com/timestamp/private-files.tar
bucket-dir/site.com/timestamp/public-files.tar
push-backup command creates following structure/objects,
bucket-dir/site.com/timestamp/db.sql.gz
bucket-dir/site.com/timestamp/private-files.tar
bucket-dir/site.com/timestamp/public-files.tar
restore-backup command doesn't handle blank directory structure. Loop through first three items of manual case throws error.

Can you share your yamls?

s3-secret.txt
backup-sites.txt
push-backups.txt
restore-backups.txt
I don't use manually created backup yet.
When i do job backup-sites and push-backups i see follow directory on S3

and after restore-backups job get error
I use image - frappe/erpnext-worker:v11, may be problem in it

Why is there double nesting?
[bucket-name] > [bucket-name] > [bucket-dir] > [site-name] > [timestamp]

It should be
[bucket-name] > [bucket-dir] > [site-name] > [timestamp]

try changing endpoint url to? "http://s3.amazonaws.com"
https://docs.aws.amazon.com/general/latest/gr/s3.html
Note: Not related, I think it is recommended to use https

It's my mistake. I use wrong ENDPOINT_URL. Thanks for url!
I use firstly)
And yet. How do better restore from my local site (for example, site1.local) to new site test.stteam.ru
I try now to upload renamed archive of site site1.local to test.stteam.ru. I think it'not good practise and have follow message now:

What's next step. Sorry if i ask many...

First try clearing cookies / application cache for the site on browser
If it doesn't work, try running migration job using v11 image.
rename > backup > push > restore should work for restoring local site.

rename > backup > push > restore should work for restoring local site.
Explain me, pls...

I do backup site1.local  , after rename archive and upload to s3 and try restore from S3. It's right way?
Now try migrate my old site...

I do backup site1.local , after rename archive and upload to s3 and try restore from S3. It's right way?
Now try migrate my old site...

ssh into vps
git clone frappe_docker

cd frappe-bench

# symlink commands which run in docker-bench in native frappe-bench
ln -s ~/frappe_docker/build/common/commands ~/frappe-bench/commands

# activate frappe-bench python environment
. ./env/bin/activate

# change to sites directory
cd sites

# rename site
mv site1.local test.example.ru

# run backup command
WITH_FILES=1 python ../commands/backup.py

# set required env variables
export BUCKET_NAME=bucket-name
export BUCKET_DIR=my-backups
export ACCESS_KEY_ID=access_key
export SECRET_ACCESS_KEY=secret_key
export ENDPOINT_URL=https://cloud.url
export REGION=s3-region

# run push backup command
python ../commands/push_backup.py

It's great instruction!
Thanks! It's work perfectly...

There was a fix related to push and restore https://github.com/frappe/frappe_docker/pull/363

re-open if needed


tf2_ros static_transform_publisher for Lidar

Hello, I have a problem with the static transformation topics. The lidar needs the gnss driver for the compensation which is based on the transformation of tf coordinates.
When I launch the lidar topic the monitor does not recognize it as in image 1, however the static transformation nodes are activated.

But when I run the following command:
rosrun tf2_ros static_transform_publisher 0 0 0 0 0 0 0 world novatel

there the monitor recognizes the lidar and I retrieve the data on the lidar topic:

Here is the procedure I use:
$ bash docker/scripts/dev_start.sh
$ bash docker/scripts/dev_into.sh
$ bash apollo.sh build_gpu
$ bash scripts/bootstrap.sh start
$ roslaunch gnss_driver gnss_driver.launch
$ ./scripts/gps.sh

then I start the lidar module from dreamview.
Can you help me understand the cause?

Please provide the Apollo version that you are using.

it's ok, this solved thanks for your feedback.


Add new apiserver

There are four main Concepts in go-restful:

Container: the global http.ServeMux, holds a collection of WebServices
WebService: a collection of Route
Route: HTTP Method, path and function call
Filters: dynamically intercepts requests and responses, can be used
as middleware

Current coverage is 44.71% (diff: 0.00%)

Merging #101 into master will increase coverage by 0.54%

@@             master       #101   diff @@
==========================================
  Files            16         16
  Lines          7051       7061    +10
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3114       3157    +43
+ Misses         3160       3113    -47
- Partials        777        791    +14

Powered by Codecov. Last update 144aa3c...aa9672b

+1

LGTM


[SPARK-8955][SQL] Replace a duplicated initialize() in HiveGenericUDTF with new one

HiveGenericUDTF#initialize(ObjectInspector[] argOIs) in v0.13.1 is duplicated, so it needs to be replaced with new one.

Merged build triggered.

Merged build started.

Test build #36939 has started for   PR 7324 at commit db3277a.

Test build #36939 has finished for   PR 7324 at commit db3277a.

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

Merged build finished. Test PASSed.

Merged build triggered.

@marmbrus Could you check and merge this?

Merged build triggered.

Merged build started.

Test build #37165 has started for   PR 7324 at commit 79cc9fe.

Test build #37165 has finished for   PR 7324 at commit 79cc9fe.

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

Merged build finished. Test PASSed.

@marmbrus This is just a reminder.

Merged build triggered.

Merged build started.

Test build #37358 has started for   PR 7324 at commit 989d05e.

Why should we add a comment to tell us something the compiler already tells
us?
On Jul 15, 2015 6:04 AM, "Apache Spark QA"<EMAIL_ADDRESS>wrote:

Test build #37358 has started
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37358/consoleFull
for PR 7324 at commit 989d05e
https://github.com/apache/spark/commit/989d05eb54b2cbe0d7924ea31df643d66315c5b5
.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/7324#issuecomment-121608387.

Test build #37358 has finished for   PR 7324 at commit 989d05e.

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

Merged build finished. Test PASSed.

Ok, I'll close this.

HiveGenericUDTF  deprecated initialize(ObjectInspector[] argOIs)
HiveGenericUDTF  latest initialize(StructObjectInspector argOIs)

I face the same issue.Has this problem solved ？

Please use the new one. Any issue there?


Transaction isolation

Does Diesel allow control of the isolation level of a transaction? It doesn't appear that it does right now, in which case consider this a feature request!
Specifically, for my use case, I want to use the "serializable" transaction isolation level in Postgres for an operation where a transaction should fail if a value it read in a query has changed by the time the transaction is committed. In other words, it queries a table, and if the data returned matches the expected condition, adds a new row to another table, but that insert should fail if the row previously queried from the first table has changed. (If there's a better way to do that, I'm all ears!) In other other words, defending against a time of check to time of use error.

So the main question that comes to mind with this feature is what the behavior should be when inside of a nested transaction. I guess the simplest answer would be to not support specifying the isolation level when starting the transaction, and instead just add support for SET TRANSACTION instead.

Any idea of the timetable for this? Is this a post-1.0 feature? I've got a few things I need it for already.

Also, it looks like I'm going to need PG's advisory locks. Should I open a separate issue for that, or shall we repurpose this one to cover support for all the various concurrency control mechanisms in PG?

Seems like advisory locks can just be done with sql_function!. I'll try to get SET TRANSACTION support in the next release

Taking this off the milestone. It's a relatively minor feature to add, but it's not super critical at the moment, as we don't offer a ton over `execute("SET TRANSACTION WHATEVER")


FileNotFoundError: [WinError 2] The system cannot find the file specified

Issue.
I am getting below error everytime, for normal simple files and projects its working fine.
How To Reproduce
Steps to reproduce the behavior (example):

I asked devika to create a react app.
As simple search engine that will fetch data from xml files hosted on remote server ( like sitemap)
Publish that website on Vercel

Expected behavior
It should work correctly.
Screenshots and logs
Log posted below
Configuration
- OS: Windows
- Python version: 3.10
- Node version: 20
- bun version: 0.1.0
- search engine:  google
- Model: gemini

Additional context
[Add any other context about the problem here.](error: Exception in thread Thread-49 (): Traceback (most recent call last): File "C:\Users\Amjad.pyenv\pyenv-win\versions\3.10.11\lib\threading.py", line 1016, in _bootstrap_inner self.run() File "C:\Users\Amjad.pyenv\pyenv-win\versions\3.10.11\lib\threading.py", line 953, in run self._target(*self._args, **self._kwargs) File "C:\devika\devika.py", line 90, in thread = Thread(target=lambda: agent.subsequent_execute(message, project_name)) File "C:\devika\src\agents\agent.py", line 207, in subsequent_execute self.runner.execute( File "C:\devika\src\agents\runner\runner.py", line 247, in execute self.run_code( File "C:\devika\src\agents\runner\runner.py", line 105, in run_code process = subprocess.run( File "C:\Users\Amjad.pyenv\pyenv-win\versions\3.10.11\lib\site-packages\gevent\subprocess.py", line 2001, in run with Popen(*popenargs, **kwargs) as process: File "C:\Users\Amjad.pyenv\pyenv-win\versions\3.10.11\lib\site-packages\gevent\subprocess.py", line 808, in init self._execute_child(args, executable, preexec_fn, close_fds, File "C:\Users\Amjad.pyenv\pyenv-win\versions\3.10.11\lib\site-packages\gevent\subprocess.py", line 1295, in _execute_child hp, ht, pid, tid = CreateProcess(executable, args, FileNotFoundError: [WinError 2] The system cannot find the file specified)

i have the same with ollama3


Do not merge: CI breakage revert

Revert "policy: remove CIDRPolicy structure"
This reverts commit c96dd9d01d7fa2b88c9d903cd47f4322c41926d5.

test-me-please

First pass failed: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/13622/

test-me-please

second pass passed

test-me-please


1.20.2 Support

Add preliminary support for 1.20.2.
I have not tested extensively yet so maybe wait to merge. Will use over the next few days.
ALSO: note that I did not update the mod dependencies in props/1.20.2
EDIT: Very late but it worked fine when I used it for multiple days.

Looks like there aren't any differences between 1.20.1 and 1.20.2 in terms of the code. If you need 1.20.2 specifically, you can compile and use it yourself (the only downside is, if you are making a modpack, you will have to include it as a file instead of as a download link). Also due to changes to support NeoForge, a lot of the build logic was changed, notably on 1.20.4+ Mixins are part of the NeoForge toml file, so I would need to duplicate the previous Forge logic and add checks everywhere for 1.20.2 and below compared to 1.20.4 and above, which would complicate things even more than they already are. So for now, I am closing this, but again feel free to compile and use it yourself if it works fine.


Swapped links in SLB doc

"Internal" pointed to the ELB docs, while "External" pointed to the ILB docs. Swapped them.

@ryanborMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Update Composer dependencies (2020-06-20-00-09)

Visual regression test passed!
View the visual regression test report


GEOMESA-885 Adding flush to accumulo feature writer

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>

Merged as e30f4e52d0bb2f0171e9192976dbd990cc5f371e on master.


Add copy button to supported networks pages

Apply Address and CopyText to supported networks pages throughout the rest of the docs (other than CCIP)

TODO: Apply to Automation - supported networks page isn't as straightforward


Combine murderer messages

When listing weapon holders on a key press, if the murderer has been detected holding a bow, there should only be one message detailing who the murderer is and that they have a bow.

Fixed in da342fdbe95aa6c86c602934b7c68e32b52c0478.


Allow users to delegate their own vote

Is your feature request related to a problem? Please describe.
In my particular use case, several committees nominate Delegates to carry their vote in a larger assembly. At the moment, entry of the correct vote delegation for 21 separate committees puts considerable burden on the meeting chairs as there is no way to allow the chairs of individual committees to delegate the vote by themselves without user management permission.
Describe the solution you'd like
The addition of a “Allow delegation of vote” permission to a user group, potentially with a constraint on the users the vote can be delegated to.
Delegations of votes should be logged and a history of vote delegations should be visible to admin users.
Describe alternatives you've considered
Manually setting delegations of votes as an admin works, but places an additional burden on admin users.

what is the status of this issue?

It's on our roadmap, we'll have to see when we have time to implement it and how to do it.

As inferred from the discussion, a setting is the preferred way to go.


Remove deduplication in configs

Create a single common config and extend from it.
See for more details https://devblogs.microsoft.com/typescript/announcing-typescript-5-0/#supporting-multiple-configuration-files-in-extends.

Duplicated by #8


path/filepath: fatal error: unexpected signal during runtime execution

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
my go version is 1.9.3
Does this issue reproduce with the latest release?
yes.
What operating system and processor architecture are you using (go env)?
darwin(macOS Sierra 10.12.5)/amd64
What did you do?
I run commond "./create_android_update.h nothing 20 10", and came the errors.
tool2.zip
If possible, provide a recipe for reproducing the error.
I commit it. Please have a look at it. Thank U very much.
What did you expect to see?
I hope it won't show the golang's runtime error . tips: it will runthe luajit commend after runtime error.
What did you see instead?
`guxiaoliangdeMacBook-Pro:tool guxiaoliang$ ./create_android_update.sh nothing 20 10
11970
fatal error: unexpected signal during runtime execution
[signal 0xa code=0x2 addr=0xcd66bee pc=0xd7bb]
runtime stack:
runtime.gothrow(0xe57c8, 0x2a)
/usr/local/go/src/runtime/panic.go:503 +0x67
runtime.sigpanic()
/usr/local/go/src/runtime/sigpanic_unix.go:14 +0x4d
runtime.unlock(0x1438e0)
/usr/local/go/src/runtime/lock_sema.go:110 +0x10b
incidlelocked(0xffffffff)
/usr/local/go/src/runtime/proc.c:2783 +0x44
retake(0x368dd20a, 0x150eda0e, 0x150eda0e)
/usr/local/go/src/runtime/proc.c:3004 +0xe8
sysmon()
/usr/local/go/src/runtime/proc.c:2930 +0x22f
mstart()
/usr/local/go/src/runtime/proc.c:859 +0xa5
runtime.mstart()
/usr/local/go/src/runtime/proc.c:836 +0x80
goroutine 1 [syscall]:
syscall.Syscall(0x3, 0x10664800, 0x11d9, 0x11d9, 0x1050f630, 0x4b, 0x0)
/usr/local/go/src/syscall/asm_darwin_386.s:20 +0x5
syscall.read(0x3, 0x10664800, 0x11d9, 0x11d9, 0x4b, 0x0, 0x0)
/usr/local/go/src/syscall/zsyscall_darwin_386.go:970 +0x54
syscall.Read(0x3, 0x10664800, 0x11d9, 0x11d9, 0xee8a, 0x0, 0x0)
/usr/local/go/src/syscall/syscall_unix.go:136 +0x4a
os.(*File).read(0x10362338, 0x10664800, 0x11d9, 0x11d9, 0xe1ad, 0x0, 0x0)
/usr/local/go/src/os/file_unix.go:191 +0x70
os.(*File).Read(0x10362338, 0x10664800, 0x11d9, 0x11d9, 0x10664800, 0x0, 0x0)
/usr/local/go/src/os/file.go:95 +0x70
bytes.(*Buffer).ReadFrom(0x20, 0x0, 0x105f5ae0, 0x0, 0x105f5ae0, 0x17a0fc, 0x0)
/usr/local/go/src/bytes/buffer.go:169 +0x1d1`

Hello there @mowenchuxin, thank you for the bug report and for the reproduction!
So:
a) So you are dereferencing a nil *os.FileInfo

where if we add a print statement
--- old.go	2018-02-01 07:49:22.000000000 -0700
+++ new.go	2018-02-01 07:49:16.000000000 -0700
@@ -215,6 +215,7 @@
 	filepath.Walk(root, func(p string, info os.FileInfo, err error) error {
 		wg.Add(1)
 		defer wg.Done()
+		fmt.Printf("p: %s\ninfo: %v\nerr: %v\n", p, info, err)
 		if info.IsDir() {
 			return nil
 		}

in there we'd get
p: /Users/emmanuelodeke/Desktop/openSrc/bugs/golang/23650/tool2/android_update/update_10_to_20/temp/scripts
info: <nil>
err: lstat /Users/emmanuelodeke/Desktop/openSrc/bugs/golang/23650/tool2/android_update/update_10_to_20/temp/scripts: no such file or directory
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x10a6ada]

goroutine 1 [running]:
main.md5_code.func1(0xc42001e3f0, 0x68, 0x0, 0x0, 0x10f0300, 0xc420082270, 0x0, 0x0)
	/Users/emmanuelodeke/Desktop/openSrc/bugs/golang/23650/tool2/src/md5_res/md5_res.go:219 +0x1da
path/filepath.Walk(0xc42001e3f0, 0x68, 0xc420062240, 0x0, 0x0)
	/Users/emmanuelodeke/go/src/go.googlesource.com/go/src/path/filepath/path.go:401 +0x6c
main.md5_code(0xc42001e3f0, 0x68, 0xc42001e460, 0x68, 0x8)
	/Users/emmanuelodeke/Desktop/openSrc/bugs/golang/23650/tool2/src/md5_res/md5_res.go:215 +0x14b
main.main()
	/Users/emmanuelodeke/Desktop/openSrc/bugs/golang/23650/tool2/src/md5_res/md5_res.go:35 +0x3cc

Please check your errors as per https://golang.org/pkg/path/filepath/#Walk

b) How come your utilities were built with Go1.4.2?
$ goversion tools/md5_res
tools/md5_res go1.4.2

so your reported Go version should actually be go1.4.2, and I mention this because in your report, your stacktrace doesn't have a disambiguated signal number i.e it is 0xb or 11 which is SIGSEGV
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x14 pc=0x461d]

goroutine 1 [running]:
main.func·003(0x1033c460, 0x68, 0x0, 0x0, 0x184ae8, 0x10338360, 0x0, 0x0)

but that's a bug that @ianlancetaylor and I fixed in 2016 for Go1.7 with commit https://github.com/golang/go/commit/1a7fc2357b1c26dcdf4fa57dee67a1172696801f aka CL https://go-review.googlesource.com/#/c/22753/. And I also noticed that the paths in the stacktrace are radically different
--- before	2018-02-01 07:58:25.000000000 -0700
+++ after	2018-02-01 07:58:12.000000000 -0700
@@ -1,12 +1,12 @@
 panic: runtime error: invalid memory address or nil pointer dereference
-[signal 0xb code=0x1 addr=0x14 pc=0x461d]
+[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x10a690a]

 goroutine 1 [running]:
-main.func·003(0x1033c460, 0x68, 0x0, 0x0, 0x184ae8, 0x10338360, 0x0, 0x0)
-	/Volumes/hd2/k007/tool/src/md5_res/md5_res.go:218 +0x8d
-path/filepath.Walk(0x1033c460, 0x68, 0x10354f04, 0x0, 0x0)
-	/usr/local/go/src/path/filepath/path.go:392 +0x91
-main.md5_code(0x1033c460, 0x68, 0x1033c4d0, 0x68, 0x1033c4d0)
-	/Volumes/hd2/k007/tool/src/md5_res/md5_res.go:237 +0xfb
+main.md5_code.func1(0xc42001e3f0, 0x68, 0x0, 0x0, 0x10f00e0, 0xc420082270, 0x0, 0x0)
+	/Users/emmanuelodeke/Desktop/openSrc/bugs/golang/23650/tool2/src/md5_res/md5_res.go:218 +0xba
+path/filepath.Walk(0xc42001e3f0, 0x68, 0xc420062240, 0x0, 0x0)
+	/Users/emmanuelodeke/go/src/go.googlesource.com/go/src/path/filepath/path.go:401 +0x6c
+main.md5_code(0xc42001e3f0, 0x68, 0xc42001e460, 0x68, 0x8)
+	/Users/emmanuelodeke/Desktop/openSrc/bugs/golang/23650/tool2/src/md5_res/md5_res.go:215 +0x14b
 main.main()
-	/Volumes/hd2/k007/tool/src/md5_res/md5_res.go:35 +0x2af
+	/Users/emmanuelodeke/Desktop/openSrc/bugs/golang/23650/tool2/src/md5_res/md5_res.go:35 +0x3cc

In summary:
a) Please check your errors in the Walk routine and avoid that nil dereference
--- old.go	2018-02-01 08:25:50.000000000 -0700
+++ new.go	2018-02-01 08:26:48.000000000 -0700
@@ -215,6 +215,10 @@
 	filepath.Walk(root, func(p string, info os.FileInfo, err error) error {
 		wg.Add(1)
 		defer wg.Done()
+		if err != nil {
+			// Check and handle the error appropriately
+			return err
+		}
 		if info.IsDir() {
 			return nil
 		}

b) Please try out the latest Go release Go1.10rc1 https://golang.org/dl/#unstable, 1.10 is due to be released pretty soon, 1.4.2 is no longer supported and make sure to run make in order to regenerate the binaries and even make them executable. If perhaps you are getting these from a vendor, let them know that their utilities could use a refresh in order to be on par with the supported. You can check what Go versions your Go binaries are built with using @rsc's https://github.com/rsc/goversion
go get rsc.io/goversion

I'll close this bug as the problem here is that err is not being checked yet info is being dereferenced when nil(the normal state). Please feel free to reopen if the problem is something else.

Thank u very much and Sorry for reply late. I am a newer to golang and this is a tool from an old program, so I do not know the version is Go1.4.2 , I fixed it with your suggest, now it can work. Thank u again.


Cluster name should be required and unique

Version: master 4/25
Although technically not required, we should make Cluster Name required and unique. It's a better user experience for:

Users to be warned when accidentally creating a cluster with the same name (this can also happen with multiple users/admins)
Users to not be confused when multiple clusters with the same name exists (2 Users creating cluster named A and unknown which is which, apart from specific settings maybe)
Users to find their cluster by name instead of the ID (which they may forget/close browser/end of day)
Using automation, you get an error when creating the "same" cluster again (same being the same name, with the drawbacks mentioned above)

This really should be enforced in the API if we're going to do it.  The UI isn't going to be involved in "automation".

API needs to take care of this.  We have put in a fix in UI in meantime.

Reverted the UI change. Please add the validation in backend.

Tested with latest build from master.
When trying to create cluster with a cluster name that is already associated with another cluster, API fails with error message - "Validation failed in API: Cluster name is not unique"
{"actionLinks":{},"baseType":"error","code":"NotUnique","fieldName":"Cluster name","links":{},"message":"","status":422,"type":"error"}

When trying to create cluster with no name, API fails with error message - "Validation failed in API: Cluster name is required"
{"actionLinks":{},"baseType":"error","code":"MissingRequired","fieldName":"Cluster name","links":{},"message":"","status":422,"type":"error"}


Fix #1452, #1569, SB/MSG/SBR documentation cleanup and remove unused CFE_SB_NO_SUBSCRIBERS

Describe the contribution
Fix #1452 - various documentation cleanup
Fix #1569 - remove CFE_SB_NO_SUBSCRIBERS (combined due to doc dependence)
Testing performed
Build and ran unit tests, passed
Expected behavior changes
None
System(s) tested on

Hardware: Intel i5/Docker
OS: Ubuntu 18.04
Versions: Bundle main + these commits

Additional context
None
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

Resolved conflicts in d6c0091

CCB:2021-05-26 APPROVED


Metrics.

@trallard told me we might be able to add a plausible.io tracking link.

I tried adding https://closember.org/ to Plausible, but I got the following error - do you know anything about this @Carreau?
This domain has already been taken. Perhaps one of your team members registered it? If that's not the case, please contact<EMAIL_ADDRESS>

@trallard Oops, that would be me. You should be able to add it now!
(I was looking into this issue. I thought I had deleted it after I noticed you self-assigned, but I've removed the site on my end now.)


stages(kickstart): run ksvalidator as part of the tests

Run ksvalidator as part of the test_kickstart.py tests. This ensures that the file we write is valid.
This addresses the suggestion from @bcl in https://github.com/osbuild/osbuild/pull/1424#issuecomment-1796054555 (thank for that!).
One thing that I noticed while adding this is that the "ostree.url" requires a valid file/http/https url so I wonder if we should make our schema stricter?
Alternatively (if we want to avoid the extra test dependency) something like https://github.com/osbuild/osbuild/compare/main...mvo5:kickstart-integration?expand=1#diff-e1602f2a3af5d3fe5bd7f3cbbc64aa2ac0a61fea3b9847668f0c59ff11825ee4R645 could be done and we install/run ksvalidaor inside the stage and test there. However it takes longer and is less flexible so I opted for this approach. But happy to switch if you feel the other approach is preferable (or even do both).

A 3rd solution to installing the pykickstart dependency is that we could install it in the containers that CI is ran in by default.
I prefer what's currently here (adding it to the dependencies in tox.ini) but I could see an argument made to use the version shipped-in-fedora-repos instead; in that case I'd prefer installing it in the container :)

Thanks! That's also an interesting idea. If we go this way I assume the test that runs ksvalidator would just skip if no ksvalidator is installed(?) (i.e. it would be two tests now with the same test intputs, one vaidating the excepted lines and one that runs ksvalidator if it's available).

A 3rd solution to installing the pykickstart dependency is that we could install it in the containers that CI is ran in by default.
I prefer what's currently here (adding it to the dependencies in tox.ini) but I could see an argument made to use the version shipped-in-fedora-repos instead; in that case I'd prefer installing it in the container :)

Thanks! That's also an interesting idea. If we go this way I assume the test that runs ksvalidator would just skip if no ksvalidator is installed(?) (i.e. it would be two tests now with the same test intputs, one vaidating the excepted lines and one that runs ksvalidator if it's available).

Yea. Never entirely happy about doing it with a skipif but it makes sense to split them out and share the parametrization.


When game host loses focus, game glitches for all players

When the host of a game moves off the game tab or browser, the game becomes very laggy and glitchy for other players.
We could think about implementing a "Your Host has Paused this Game" message or something to avoid this seeming like a bug.

Related: #5247

Fixed via https://github.com/microsoft/pxt/pull/9186


Remove googleapis-common-protos pin

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

For some reason this pin seems to be causing the following in our environment:
The following packages are incompatible
├─ dagster-gcp >=0.21.7,<1  is installable and it requires
│  └─ googleapis-common-protos !=1.57.0 *_1, which can be installed;
└─ googleapis-common-protos >=1.61.0  is not installable because it conflicts with any installable versions previously reported.

and the referenced fix has been merged a long time ago anyway.

@conda-forge-admin please rerender

@conda-forge-admin please rerender

Often a yak shave with the big FAANG packages as deps, buh: thanks for digging into this.


how to convert a numpy string array into the correct numpy array encodings for face recognition comparison ?

face_recognition version: 1.0.0
Python version: Python 3.6.3
Operating System: MacOS High Sierra

Description
I have an API that communicates with the mobile. The mobile will send the face_encoding in a string formed numpy array. How can I convert in the numpy array encoding so that can use the face_recognition.compare_faces method to do the face comparison ?
Currently, I use np.array(unknown_face_features_string.split(','), dtype=float) but it returns ValueError: could not convert string to float:
What I Did
whereby request.POST['face_features'] =

"-0.0561027,0.148504,0.0766109,0.0123607,-0.00353531,-0.0240078,-0.0836124,-0.03222,0.181277,-0.0111063,0.270465,0.0105144,-0.139278,0.00535686,-0.0059066,0.0873017,-0.189508,-0.00254752,-0.0657484,-0.0390603,0.0615764,0.0155173,0.0793468,0.0375025,-0.216556,-0.316628,-0.0929957,-0.110074,0.106954,-0.157345,0.00164702,0.00836864,-0.165602,-0.0543957,0.0028924,0.0108578,-0.0867328,-0.0828893,0.223985,0.0599059,-0.136025,0.0992446,-0.110337,0.331373,0.210697,0.0160679,0.07047,-0.0986981,0.0780619,-0.197312,0.0552125,0.153301,0.00372774,0.0625982,0.0752206,-0.122434,0.0230314,0.114629,-0.197551,0.0808682,0.0767851,-0.0750638,0.0292691,-0.0697488,0.123947,0.0862854,-0.0810472,-0.125884,0.085837,-0.0862188,0.027633,0.0968399,-0.129141,-0.251257,-0.271362,0.0819597,0.39041,0.184892,-0.155702,0.0615741,-0.0261523,-0.0274833,0.0997764,0.114738,-0.0624778,-0.00650277,-0.0234835,0.114332,0.132601,0.00269146,-0.0860263,0.171939,-0.0583306,-0.00452455,0.0361284,-0.0114907,-0.130202,0.0340496,-0.0721987,0.0212415,0.0228887,-0.060828,0.0252066,0.0889736,-0.143087,0.189118,0.00960924,-0.0408581,-0.00351588,0.0323516,-0.112198,0.025282,0.176388,-0.303327,0.253976,0.188248,0.0578906,0.145686,0.0537029,0.0609984,-0.0295396,0.087931,-0.14044,-0.0119921,0.00319132,0.00276587,0.116404,-0.0297028,"

        unknown_face_features_string = request.POST['face_features']
        unknown_encoding = np.array(unknown_face_features_string.split(','), dtype=float)

This returns ValueError: could not convert string to float:

The intention is to run this command after that
        results = face_recognition.compare_faces([known_encoding], unknown_encoding)

You can try with the fromstring method.
numpy.fromstring(request.POST['face_features'], dtype=float, sep=',')
https://docs.scipy.org/doc/numpy-1.14.0/reference/generated/numpy.fromstring.html

The method @brunosps suggested should work perfectly for you.


Link to this issue tracker in settings

Good idea, will add in next update.

Might add a link to the repo in the settings.


Longitude/Latitude to X/Z coord calculation

Do you have a Longitude/Latitude to X/Z coordinate calculation so I can turn a real world location into a location in Minecraft Earth?

There is a small frame embedded in the website. You can use it here.