Could not create SSL/TLS secure

I am getting this error when project updated on azure server for testing. Its working on localserver.
Getting error on this code
connection.connect();
Error message:
An error occurred while sending the request. ---> System.Net.WebException: The request was aborted: Could not create SSL/TLS secure channel. at System.Net.HttpWebRequest.EndGetRequestStream(

My guess without looking into specifics is perhaps you’re using a deprecated TLS version? Such as 1.0 versus 1.2 versus 2.0.
Sent from my iPhone
On Jul 19, 2018, at 9:40 AM, amitabgwebs <notifications<EMAIL_ADDRESS>wrote:
I am getting this error when project updated on azure server for testing. Its working on localserver.
Getting error on this code
connection.connect();
Error message:
An error occurred while sending the request. ---> System.Net.WebException: The request was aborted: Could not create SSL/TLS secure channel. at System.Net.HttpWebRequest.EndGetRequestStream(
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/adplabs/adp-connection-NET/issues/13, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AH-yErYGx4cDAytC5N-aU1zdzCtZSZp5ks5uIIwxgaJpZM4VWaqW.

I checked and could't find TLS2.0 I am using TLS 1.2 in my code.
Not sure if I am correct,but I am using below code before connection.
                    ServicePointManager.Expect100Continue = true;
                    ServicePointManager.SecurityProtocol = SecurityProtocolType.Tls12;
                   connection.connect();

Any help is appreciated, thanks.

Haven't changed anything. Am getting this error suddenly on Tuesday
afternoon using the same code that's been working for a month.
We have 3 instances that are not working, while, if I use Postman to
manually call the API, it does work. Using the same certs in all places.
Even reduced the code to just a basic workers api call and get a 401 using
the python code (from adp) that we've been using all along.
Am trying to determine what it could be when the python connect lib stops
working but postman works (and even taking the code that python is using
for requests and that is the exact code I'm using in the python script. The
authentication using certs and api keys, etc.. IS working because I am
getting a successful connection in python, and getting a valid Bearer
token, after connecting.
The 401 happens when I call the workers API.
Thanks for any suggestions. If we could hop on a call.. would love anyone
to look over my shoulder.. that sometimes helps.
Thanks!
Russ
On Thu, Jul 19, 2018 at 10:14 AM Greg Kujawa<EMAIL_ADDRESS>wrote:

My guess without looking into specifics is perhaps you’re using a
deprecated TLS version? Such as 1.0 versus 1.2 versus 2.0.
Sent from my iPhone
On Jul 19, 2018, at 9:40 AM, amitabgwebs<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
I am getting this error when project updated on azure server for testing.
Its working on localserver.
Getting error on this code
connection.connect();
Error message:
An error occurred while sending the request. ---> System.Net.WebException:
The request was aborted: Could not create SSL/TLS secure channel. at
System.Net.HttpWebRequest.EndGetRequestStream(
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<
https://github.com/adplabs/adp-connection-NET/issues/13>, or mute the
thread<
https://github.com/notifications/unsubscribe-auth/AH-yErYGx4cDAytC5N-aU1zdzCtZSZp5ks5uIIwxgaJpZM4VWaqW>.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-406290982,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af-LOdf0QMibAGjOUhtJusLt_jY0Mhu-ks5uIJRDgaJpZM4VWaqW
.

All I recall is that this summer I had to explicitly set the correct TLS version for a good number of larger third party .NET API’s. Since they deprecated TLS to require 2.0. Postman worked while troubleshooting since it invoked the latest version by default. Look at Fiddler traces and you can see which TLS version your code is referencing.
Worked for me!
Sent from my iPhone
On Jul 19, 2018, at 10:41 AM, rhf <notifications<EMAIL_ADDRESS>wrote:
Haven't changed anything. Am getting this error suddenly on Tuesday
afternoon using the same code that's been working for a month.
We have 3 instances that are not working, while, if I use Postman to
manually call the API, it does work. Using the same certs in all places.
Even reduced the code to just a basic workers api call and get a 401 using
the python code (from adp) that we've been using all along.
Am trying to determine what it could be when the python connect lib stops
working but postman works (and even taking the code that python is using
for requests and that is the exact code I'm using in the python script. The
authentication using certs and api keys, etc.. IS working because I am
getting a successful connection in python, and getting a valid Bearer
token, after connecting.
The 401 happens when I call the workers API.
Thanks for any suggestions. If we could hop on a call.. would love anyone
to look over my shoulder.. that sometimes helps.
Thanks!
Russ
On Thu, Jul 19, 2018 at 10:14 AM Greg Kujawa <notifications<EMAIL_ADDRESS>wrote:

My guess without looking into specifics is perhaps you’re using a
deprecated TLS version? Such as 1.0 versus 1.2 versus 2.0.
Sent from my iPhone
On Jul 19, 2018, at 9:40 AM, amitabgwebs <notifications<EMAIL_ADDRESS>notifications<EMAIL_ADDRESS>wrote:
I am getting this error when project updated on azure server for testing.
Its working on localserver.
Getting error on this code
connection.connect();
Error message:
An error occurred while sending the request. ---> System.Net.WebException:
The request was aborted: Could not create SSL/TLS secure channel. at
System.Net.HttpWebRequest.EndGetRequestStream(
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<
https://github.com/adplabs/adp-connection-NET/issues/13>, or mute the
thread<
https://github.com/notifications/unsubscribe-auth/AH-yErYGx4cDAytC5N-aU1zdzCtZSZp5ks5uIIwxgaJpZM4VWaqW>.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-406290982,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af-LOdf0QMibAGjOUhtJusLt_jY0Mhu-ks5uIJRDgaJpZM4VWaqW
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-406300451, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AH-yEgxax0tstapYOEaHjbGzAc4JH2hsks5uIJqMgaJpZM4VWaqW.

Thank you so much!! I will try that! Very greatful for your advice. I will
let you know!
On Thu, Jul 19, 2018 at 10:44 AM Greg Kujawa<EMAIL_ADDRESS>wrote:

All I recall is that this summer I had to explicitly set the correct TLS
version for a good number of larger third party .NET API’s. Since they
deprecated TLS to require 2.0. Postman worked while troubleshooting since
it invoked the latest version by default. Look at Fiddler traces and you
can see which TLS version your code is referencing.
Worked for me!
Sent from my iPhone
On Jul 19, 2018, at 10:41 AM, rhf<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Haven't changed anything. Am getting this error suddenly on Tuesday
afternoon using the same code that's been working for a month.
We have 3 instances that are not working, while, if I use Postman to
manually call the API, it does work. Using the same certs in all places.
Even reduced the code to just a basic workers api call and get a 401 using
the python code (from adp) that we've been using all along.
Am trying to determine what it could be when the python connect lib stops
working but postman works (and even taking the code that python is using
for requests and that is the exact code I'm using in the python script.
The
authentication using certs and api keys, etc.. IS working because I am
getting a successful connection in python, and getting a valid Bearer
token, after connecting.
The 401 happens when I call the workers API.
Thanks for any suggestions. If we could hop on a call.. would love anyone
to look over my shoulder.. that sometimes helps.
Thanks!
Russ
On Thu, Jul 19, 2018 at 10:14 AM Greg Kujawa<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

My guess without looking into specifics is perhaps you’re using a
deprecated TLS version? Such as 1.0 versus 1.2 versus 2.0.
Sent from my iPhone
On Jul 19, 2018, at 9:40 AM, amitabgwebs<EMAIL_ADDRESS><EMAIL_ADDRESS>notifications<EMAIL_ADDRESS>wrote:
I am getting this error when project updated on azure server for
testing.
Its working on localserver.
Getting error on this code
connection.connect();
Error message:
An error occurred while sending the request. --->
System.Net.WebException:
The request was aborted: Could not create SSL/TLS secure channel. at
System.Net.HttpWebRequest.EndGetRequestStream(
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<
https://github.com/adplabs/adp-connection-NET/issues/13>, or mute the
thread<

https://github.com/notifications/unsubscribe-auth/AH-yErYGx4cDAytC5N-aU1zdzCtZSZp5ks5uIIwxgaJpZM4VWaqW>.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<
https://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-406290982>,

or mute the thread
<
https://github.com/notifications/unsubscribe-auth/Af-LOdf0QMibAGjOUhtJusLt_jY0Mhu-ks5uIJRDgaJpZM4VWaqW>

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<
https://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-406300451>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/AH-yEgxax0tstapYOEaHjbGzAc4JH2hsks5uIJqMgaJpZM4VWaqW>.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-406301552,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af-LOUe7FdohIiBOJ31yfLwPpHsJNotUks5uIJs_gaJpZM4VWaqW
.

not sure if I mentioned.. I am using python
On Thu, Jul 19, 2018 at 10:46 AM rh fleming<EMAIL_ADDRESS>wrote:

Thank you so much!! I will try that! Very greatful for your advice. I will
let you know!
On Thu, Jul 19, 2018 at 10:44 AM Greg Kujawa<EMAIL_ADDRESS>wrote:

All I recall is that this summer I had to explicitly set the correct TLS
version for a good number of larger third party .NET API’s. Since they
deprecated TLS to require 2.0. Postman worked while troubleshooting since
it invoked the latest version by default. Look at Fiddler traces and you
can see which TLS version your code is referencing.
Worked for me!
Sent from my iPhone
On Jul 19, 2018, at 10:41 AM, rhf<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Haven't changed anything. Am getting this error suddenly on Tuesday
afternoon using the same code that's been working for a month.
We have 3 instances that are not working, while, if I use Postman to
manually call the API, it does work. Using the same certs in all places.
Even reduced the code to just a basic workers api call and get a 401
using
the python code (from adp) that we've been using all along.
Am trying to determine what it could be when the python connect lib stops
working but postman works (and even taking the code that python is using
for requests and that is the exact code I'm using in the python script.
The
authentication using certs and api keys, etc.. IS working because I am
getting a successful connection in python, and getting a valid Bearer
token, after connecting.
The 401 happens when I call the workers API.
Thanks for any suggestions. If we could hop on a call.. would love anyone
to look over my shoulder.. that sometimes helps.
Thanks!
Russ
On Thu, Jul 19, 2018 at 10:14 AM Greg Kujawa<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

My guess without looking into specifics is perhaps you’re using a
deprecated TLS version? Such as 1.0 versus 1.2 versus 2.0.
Sent from my iPhone
On Jul 19, 2018, at 9:40 AM, amitabgwebs<EMAIL_ADDRESS><EMAIL_ADDRESS>notifications<EMAIL_ADDRESS>wrote:
I am getting this error when project updated on azure server for
testing.
Its working on localserver.
Getting error on this code
connection.connect();
Error message:
An error occurred while sending the request. --->
System.Net.WebException:
The request was aborted: Could not create SSL/TLS secure channel. at
System.Net.HttpWebRequest.EndGetRequestStream(
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<
https://github.com/adplabs/adp-connection-NET/issues/13>, or mute the
thread<

https://github.com/notifications/unsubscribe-auth/AH-yErYGx4cDAytC5N-aU1zdzCtZSZp5ks5uIIwxgaJpZM4VWaqW>.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<
https://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-406290982>,

or mute the thread
<
https://github.com/notifications/unsubscribe-auth/Af-LOdf0QMibAGjOUhtJusLt_jY0Mhu-ks5uIJRDgaJpZM4VWaqW>

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub<
https://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-406300451>,
or mute the thread<
https://github.com/notifications/unsubscribe-auth/AH-yEgxax0tstapYOEaHjbGzAc4JH2hsks5uIJqMgaJpZM4VWaqW>.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-406301552,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af-LOUe7FdohIiBOJ31yfLwPpHsJNotUks5uIJs_gaJpZM4VWaqW
.

Have you checked the expiration date of your counter signed ADP SSL Certificate?
FYI TLS 2.0 doesn't exist.
I suspect he meant to make sure you aren't using one of the deprecated versions which I suspect are SSL 1.0, SSL 2.0, SSL 3.0, TLS 1.0 and TLS 1.1.
TLS 1.2 is the most recent widely accepted version.
TLS 1.3 is still a proposed specification at this time I believe.

Correct! Sorry about that. Was mixing up SSL versions with TLS versions. Various third party API web services I connect to for my production apps required TLS 1.2, as TLS 1.0 and 1.1 were retired due to widespread security initiatives most companies have been taking this summer. By default my Visual Studio and .NET references weren’t invoking TLS 1.2 by default, so I had to explicitly specify it. Not sure if Python might be doing something similar under the hood…
From: sloppypointless<EMAIL_ADDRESS>Sent: Thursday, July 19, 2018 4:14 PM
To: adplabs/adp-connection-NET<EMAIL_ADDRESS>Cc: Kujawa, Greg<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [adplabs/adp-connection-NET] Could not create SSL/TLS secure (#13)
Have you checked the expiration date of your counter signed ADP SSL Certificate?
FYI TLS 2.0 doesn't exist.
I suspect he meant to make sure you aren't using one of the deprecated versions which I suspect are SSL 1.0, SSL 2.0, SSL 3.0, TLS 1.0 and TLS 1.1.
TLS 1.2 is the most recent widely accepted version.
TLS 1.3 is still a proposed specification at this time I believe.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-406400119, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AH-yEhFuGEcOWkd-Gs0DLoEmrkVCiW1jks5uIOifgaJpZM4VWaqW.

Hi,
We tested the code which is there in GitHub and it's working. Could you please check if you're passing the correct certificates while calling ADP API. Please download the python-code once again and test it and let us know if you still get the same issue.
Thanks,
Mahesh

I resolved the issue
On Wed, Jul 25, 2018, 7:58 AM ADPMarketplace<EMAIL_ADDRESS>wrote:

Hi,
We tested the code which is there in GitHub and it's working. Could you
please check if you're passing the correct certificates while calling ADP
API. Please download the python-code once again and test it and let us know
if you still get the same issue.
Thanks,
Mahesh
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-407729285,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af-LOYB3OlG5h6m45gN5dlO9E6dG07iJks5uKF13gaJpZM4VWaqW
.

So that we all have better closure as to the issue, what did you do that resolved it?
Greg Kujawa
IT Manager
DIAMOND CELLAR Holdings, LLC
6280 Sawmill Road
Dublin, OH 43017
614-923-9515 phone
614-336-4555 fax
www.diamondcellar.comhttp://www.diamondcellar.com
Office Hours: M Tu W Th F
[cid:image002.png@01CF8981.56A564A0]https://www.facebook.com/DiamondCellar[cid:image003.png@01CF8981.56A564A0]https://twitter.com/DiamondCellarCo[cid:image004.png@01CF8981.56A564A0]http://www.pinterest.com/diamondcellar/[cid:image005.png@01CF8981.56A564A0]http://www.yelp.com/biz/diamond-cellar-dublin[cid:image006.png@01CF8981.56A564A0]http://instagram.com/diamondcellar[cid:image007.png@01CF8981.56A564A0]https://plus.google.com/+DiamondCellarDublin/about
CONFIDENTIALITY NOTICE
UNAUTHORIZED INTERCEPTION IS PROHIBITED BY FEDERAL LAW
[Electronic Communications Privacy Act of 1986, 18 U.S.C. 2701(a) and 2702(a)] This message is protected by Constitutional Rights and applicable legal privileges.  It is strictly confidential.
From: rhf<EMAIL_ADDRESS>Sent: Wednesday, July 25, 2018 8:19 AM
To: adplabs/adp-connection-NET<EMAIL_ADDRESS>Cc: Kujawa, Greg<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [adplabs/adp-connection-NET] Could not create SSL/TLS secure (#13)
I resolved the issue
On Wed, Jul 25, 2018, 7:58 AM ADPMarketplace <notifications<EMAIL_ADDRESS>wrote:

Hi,
We tested the code which is there in GitHub and it's working. Could you
please check if you're passing the correct certificates while calling ADP
API. Please download the python-code once again and test it and let us know
if you still get the same issue.
Thanks,
Mahesh
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-407729285,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af-LOYB3OlG5h6m45gN5dlO9E6dG07iJks5uKF13gaJpZM4VWaqW
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-407735224, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AH-yEqQjCb129yjwVfl1B-oD3iGVfCsUks5uKGJNgaJpZM4VWaqW.

Hi Kujawa,
No change we did at our end. Not sure, may be at your end, you're not passing the certificates properly earlier.
As the issue is resolved now. Shall I close this ticket?
Thanks,
Mahesh

My app is running fine now. There was some communication that went out that
I did not receive that the API would start requiring the cert with every
API call, not just the token request.
On Wed, Jul 25, 2018, 9:10 AM ADPMarketplace<EMAIL_ADDRESS>wrote:

Hi Kujawa,
No change we did at our end. Not sure, may be at your end, you're not
passing the certificates properly earlier.
As the issue is resolved now. Shall I close this ticket?
Thanks,
Mahesh
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/adplabs/adp-connection-NET/issues/13#issuecomment-407748681,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af-LOaR2ETRCxW1sBmL7Qr_I6FQK0Gaxks5uKG5OgaJpZM4VWaqW
.

Hi,
We tested the code which is there in GitHub and it's working. Could you please check if you're passing the correct certificates while calling ADP API. Please download the python-code once again and test it and let us know if you still get the same issue.
Thanks,
Mahesh

Where in the github code can you see an example of this? I'm getting the same exact error in my .net application, and i cannot figure out why.
It works perfect locally, but when it's published to the server. tells me error with TLS. Can you please help?

Hi,
We tested the code which is there in GitHub and it's working. Could you please check if you're passing the correct certificates while calling ADP API. Please download the python-code once again and test it and let us know if you still get the same issue.
Thanks,
Mahesh

Where in the github code can you see an example of this? I'm getting the same exact error in my .net application, and i cannot figure out why.
It works perfect locally, but when it's published to the server. tells me error with TLS. Can you please help?

At this point this is a pretty old thread. Most providers retiring TLS 1.0 and 11. is old news. The two things that need to be done involve:

Ensuring that your code has explicitly specified System.Net.ServicePointManager.SecurityProtocol = System.Net.SecurityProtocolType.Tls12; in the main application startup.
Ensuring that the host you are running the code on is configured for TLS 1.2 at the lowest.

At this point this is a pretty old thread. Most providers retiring TLS 1.0 and 11. is old news. The two things that need to be done involve:

Ensuring that your code has explicitly specified System.Net.ServicePointManager.SecurityProtocol = System.Net.SecurityProtocolType.Tls12; in the main application startup.
Ensuring that the host you are running the code on is configured for TLS 1.2 at the lowest.

Thanks. I have been on that thread.. I can't find the exact place you need to tell the app to use your certificate.
It was working on server for us, then one day it stopped.

Thanks. I have been on that thread.. I can't find the exact place you need to tell the app to use your certificate.
It was working on server for us, then one day it stopped.


Tibanna instance type error with snakemake

Hi,
I am trying to use tibanna to launch snakemake workflows on AWS. However, I am constantly running into an error (as I can see on the cloud watch)
{ "error": "ClientError", "cause": { "errorMessage": "An error occurred (InvalidInstanceType) when calling the DescribeInstanceTypes operation: The following supplied instance types do not exist: [m1.medium, m3.medium]", "errorType": "ClientError", "stackTrace": [ "  File \"/var/task/service.py\", line 20, in handler\n    return run_task(event)\n", "  File \"/var/task/tibanna/run_task.py\", line 63, in run_task\n    execution = Execution(input_json)\n", "  File \"/var/task/tibanna/ec2_utils.py\", line 374, in __init__\n    self.create_instance_type_list()\n", "  File \"/var/task/tibanna/ec2_utils.py\", line 426, in create_instance_type_list\n    results = ec2.describe_instance_types(\n", "  File \"/var/task/botocore/client.py\", line 530, in _api_call\n    return self._make_api_call(operation_name, kwargs)\n", "  File \"/var/task/botocore/client.py\", line 960, in _make_api_call\n    raise error_class(parsed_response, operation_name)\n" ] } }

Here are my versions:
snakemake version: 7.20.0
tibanna version: 3.1.0
Python version: 3.7.12
Below is the command used:
snakemake --tibanna --tibanna-config spot_instance=true behavior_on_capacity_limit=retry_without_spot availability_zone=ap-south-1 --default-remote-prefix=<bucketname> -s test.yaml  --jobs 1

I have tried changing the instance type from the command
snakemake --tibanna --tibanna-config spot_instance=true behavior_on_capacity_limit=retry_without_spot availability_zone=ap-south-1  instance_type=t3.micro --default-remote-prefix=<bucketname> -s test.yaml  --jobs 1

However the instance name just gets appended to the list  [m1.medium, m3.medium] and runs into the same error
The following supplied instance types do not exist: [t3.micro, m1.medium, m3.medium]

I have tried using the api and there I am running into a different error:
Digest: sha256:6e33aee97fbca79e99a9603ba7502ef732c466bf96b912da9c390189431f04ea
Status: Downloaded newer image for snakemake/snakemake:latest
Error: you need to specify the maximum number of CPU cores to be used at the same time. If you want to use N cores, say --cores N or -cN. For all cores on your system (be sure that this is appropriate) use --cores all. For no parallelization use --cores 1 or -c1. <_io.TextIOWrapper name='<stderr>' mode='w' encoding='utf-8'>

Could you please let me know if I'm missing something? Looks like the instance type is hard-coded somewhere?
Thank you.

I'm running into the same issue. It seems like snakemake is looking up what kind of ec2 instance to use at some point and is producing junk results.

Seems like tibanna is the one doing instance type lookups, and it has the error.
https://github.com/4dn-dcic/tibanna/issues/382
I'll update this issue again when tibanna is updated.

This issue was resolved and this dead issue can probably be closed.


Disable/suppress merge-conflict extension when merge editor is used

We have the merge-conflict extension which highlights git merge markers (using codelens and decorations) and we adds conflict navigation commands. This conflicts with the new merge editor and we should find ways to suppress it when merge is being used, e.g hide commands based on the isMergeEditor context key

fyi @chrmarti - we did push some pragmatic changes to the merge-conflict extension to disable itself when the user enables the experimental git merge editor...


shippingPreference: "NO_SHIPPING"

Does passing shippingPreference: "NO_SHIPPING" not work? This functionality is in third party libs but doesn't seem to be here...
<PayPalScriptProvider options={{ "client-id": clientId, currency: "GBP", "disable-funding": "sofort", shippingPreference: "NO_SHIPPING" }}>
Doesn't seem to have any effect at the checkout page as it still shows the delivery address

After digging through the source code you can supply many options.. I was able to set no shipping address like this:
```
const createOrder = (data, actions) => {
return actions.order
.create({
purchase_units: [
{
amount: {
value: donationAmount,
},
},
],
application_context: {
shipping_preference: "NO_SHIPPING"
}
})
.then((orderID) => {
setOrderID(orderID);
return orderID;
});
}

i've used this code but no works...


[BUG]: Fault add Document to workspace

How are you running AnythingLLM?
AnythingLLM desktop app
What happened?
In the Anything LLM Desktop application on macOS version 15, when attempting to import documents into the workspace and encountering a failure during the "Save and Embed" operation, the error message you're seeing translates to:
"Error: 1 document failed to add, fetch failed."

Are there known steps to reproduce?
No response

@kuyinya Hello, how did you solve this problem?

@TianT1209 I had this same issue on MacOS, and I think the fetch failed message means that it failed to download the embedder model. This could be due to company firewall, for example.
What I did to fix the problem was to change my embedder model to Ollama Llama 3.1 in the AnythingLLM settings.


Fix shadow dom handling

Since https://github.com/ccampbell/mousetrap/pull/438 wasn’t merged upstream, I’m trying to get this into Discourse’s fork as per Mousetrap.js doesn’t properly stop callbacks for events originating from a shadow DOM (comment).
Prerequisites:

Get upstream changes (because https://github.com/ccampbell/mousetrap/pull/437 was merged).

(Original notes)
This pull request:

adds tests for open and closed shadow trees.
adds a sixth options parameter to KeyEvent.simulate. This was required to re-target the event target with closed shadow trees. The related code contains further explanation.

Important note: The tests pass when opening tests/mousetrap.html in a browser, but not when running npm test. This is due to the test environment not understanding Element.attachShadow; hence, it is throwing on the lines with the call to attachShadow in the test file. I’m not quite sure how to make these tests work with mocha.

I would really prefer if this was merged upstream. The problem was lack of review from the project?

Hi there. As you have pointed out it is not an ideal situation. The truth is I just don’t have a lot of time/resources to devote to working on Mousetrap.
That said, I’m happy to merge whatever changes are necessary into master (provided they are minor). I know from experience that it’s quite annoying to have to maintain your own fork because the upstream project won’t merge changes.
I commented on the other PR. If you decide to go with the micro library approach there are no hard feelings here.
Wishing you all a happy holidays!

@ccampbell wishing you a happy holiday. I totally understand your predicament.
@kleinfreund we are going to take our time here deciding what the best path is forward, ping us in 14 days if we do not reply here or on meta, it is going to take us time key decision makers are on a break at the moment.


[DRAFT] SA20: Add compatibility adapter for psycopg and asyncpg dialects

About
The idea is to unlock the CrateDB dialect with the psycopg3 and asyncpg drivers. The patch introduces the crate+psycopg://, crate+asyncpg://, and crate+urllib3:// dialect identifiers. The asynchronous variant of psycopg is also supported.
References

GH-392
GH-423
GH-425

Backlog
There is one flaw with the psycopg driver in asynchronous mode when using result streaming based on server-side cursors. It can be exercised using the async_streaming.py example program.
python examples/async_streaming.py psycopg

The same program works fine when using the asyncpg driver.
python examples/async_streaming.py asyncpg

Introduction
https://github.com/crate/crate-python/blob/387b19f9664ac8e78e898a37aa81dd18a38ee542/examples/async_streaming.py#L2-L15
Bug
https://github.com/crate/crate-python/blob/387b19f9664ac8e78e898a37aa81dd18a38ee542/examples/async_streaming.py#L42-L47
References

https://docs.sqlalchemy.org/en/20/orm/extensions/asyncio.html#synopsis-core
https://docs.sqlalchemy.org/en/20/_modules/examples/asyncio/basic.html

This patch has been replaced by two others.

https://github.com/crate-workbench/sqlalchemy-cratedb/pull/11
https://github.com/crate/cratedb-examples/pull/201

/cc @surister


[Feature Request] Webhook updates on encrypted imports

I would like to receive updates from the discord webhook but keep my import password protected.
Currently the webhook only work on public imports. Maybe versioning could be tracked without accessing the encrypted import data?

Nothing that the webhook uses is encrypted so that isn't a problem. However webhooks aren't sent for hidden/private/encrypted/restricted imports since the webhook option applies to all imports, so stuff that isn't public aren't sent that way.
Adding a separate webhook override option attached to the individual import could work though, I'll look into that this week.

thanks that was fast!


Typo in Get Azure Active Directory Token content

Describe the bug
On this page, https://docs.microsoft.com/en-us/rest/api/eventhub/get-azure-active-directory-token, there is a typo in the
"Use Postman to get the Azure AD token" section. For Step 5.e, the value should be https://eventhubs.azure.net not https://servicebus.azure.net.
To Reproduce
Steps to reproduce the behavior:
Go to https://docs.microsoft.com/en-us/rest/api/eventhub/get-azure-active-directory-token
Scroll down to the "Use Postman to get the Azure AD token"
Go to Step 5.e
The value that is listed as https://servicebus.azure.net should be be https://eventhubs.azure.net
Expected behavior
This article is about getting a token for Event Hubs, the resource value should be https://eventhubs.azure.net
Screenshots
N/A
Desktop (please complete the following information):
Windows 10 1909
Microsoft Edge
Smartphone (please complete the following information):
N/A
Additional context
N/A

@subspacefield Please move this issue to https://github.com/Azure/azure-rest-api-specs/issues


After the first child, it stops and I can't make any more requests.

i tried it on vs code-server
Even though 8GB of memory is allocated, continuous use is not possible.

I've just tried it with vs code-server using remote tunneling and it was able to make multiple turns. I am unsure if this issue is memory related because I don't think the extension is that memory intensive.
Have you provided your openai api key to the extension? It's a possibility it could stuck on that step.
You could check that by clicking on "Settings" from the top right corner.

In the settings panel, ensure you are looking at the "User" tab.

From there, you can scroll down the settings to see if the "Openai Key" config has your key.
If this is not the case, let me know and I can help figure it out together!


signal-cli-rest-api example works but cannot register signalbot device

I was able to run these steps on a Lightsail Ubuntu AWS machine: https://github.com/bbernhard/signal-cli-rest-api?tab=readme-ov-file#getting-started
To test, I sent a few test messages using the example:
$ curl -X POST -H "Content-Type: application/json" 'http://localhost:8080/v2/send' \
     -d '{"message": "Test via Signal API!", "number": "+4412345", "recipients": [ "+44987654" ]}'

However, when I continue along this repo's readme, and run step 1:
docker run -p 8080:8080 \
    -v $(PWD)/signal-cli-config:/home/.local/share/signal-cli \
    -e 'MODE=normal' bbernhard/signal-cli-rest-api:0.57

This succeeds.
But when I proceed to step 2, I get this error:
{"error":"Couldn't create QR code: no data to encode"}

Screenshot:

The full log:
$ sudo docker run -p 8080:8080 \
    -v $(pwd)/signal-cli-config:/home/.local/share/signal-cli \
    -e 'MODE=normal' bbernhard/signal-cli-rest-api:0.57
+ set -e
+ [ -z /home/.local/share/signal-cli ]
+ usermod -u 1000 signal-api
usermod: no changes
+ groupmod -g 1000 signal-api
+ chown 1000:1000 -R /home/.local/share/signal-cli
+ cat
+ cap_prefix=-cap_
+ cat /proc/sys/kernel/cap_last_cap
+ seq -s ,-cap_ 0 40
+ caps=-cap_0,-cap_1,-cap_2,-cap_3,-cap_4,-cap_5,-cap_6,-cap_7,-cap_8,-cap_9,-cap_10,-cap_11,-cap_12,-cap_13,-cap_14,-cap_15,-cap_16,-cap_17,-cap_18,-cap_19,-cap_20,-cap_21,-cap_22,-cap_23,-cap_24,-cap_25,-cap_26,-cap_27,-cap_28,-cap_29,-cap_30,-cap_31,-cap_32,-cap_33,-cap_34,-cap_35,-cap_36,-cap_37,-cap_38,-cap_39,-cap_40
+ [ normal = json-rpc ]
+ hostname -i
+ export HOST_IP=xxx
+ exec setpriv --reuid=1000 --regid=1000 --init-groups --inh-caps=-cap_0,-cap_1,-cap_2,-cap_3,-cap_4,-cap_5,-cap_6,-cap_7,-cap_8,-cap_9,-cap_10,-cap_11,-cap_12,-cap_13,-cap_14,-cap_15,-cap_16,-cap_17,-cap_18,-cap_19,-cap_20,-cap_21,-cap_22,-cap_23,-cap_24,-cap_25,-cap_26,-cap_27,-cap_28,-cap_29,-cap_30,-cap_31,-cap_32,-cap_33,-cap_34,-cap_35,-cap_36,-cap_37,-cap_38,-cap_39,-cap_40 signal-cli-rest-api -signal-cli-config=/home/.local/share/signal-cli
time="2024-04-21T15:19:39Z" level=info msg="Started Signal Messenger REST API"
[GIN] 2024/04/21 - 15:19:57 | 400 |  2.608821556s |      <IP_ADDRESS> | GET      "/v1/qrcodelink?device_name=local"

Has anyone else run into this? Any ideas on how to debug?
The goal is to integrate with the Claude API (https://docs.anthropic.com/claude/reference/getting-started-with-the-api) and my Open Interpreter (https://github.com/OpenInterpreter/open-interpreter).
Really excited to test this out!!

Maybe I need to have this service running in the background? https://github.com/bbernhard/signal-cli-rest-api - and I see in the readme it says "To run the the bot you need to start the signal-cli-rest-api service and link your device with it.". I linked my device, and then I stopped the service.

Maybe I need to have this service running in the background?

Yes, the python package acts as a handy wrapper around the signal-cli-rest-api service which is responsible for interacting with the signal servers

{"error":"Couldn't create QR code: no data to encode"}

I'm not sure what the issue is but you could head to signal-cli-rest-api since its coming from that project

Thanks @filipre ! I confirmed that the issue does not happen with https://github.com/bbernhard/signal-cli-rest-api.
Specifically, this command in the readme does not work:
docker run -p 8080:8080 \
    -v $(PWD)/signal-cli-config:/home/.local/share/signal-cli \
    -e 'MODE=normal' bbernhard/signal-cli-rest-api:0.57

It first yields an error about the $(PWD) being incorrect (on Ubuntu 22.04.1 LTS on AWS Lightsail).
Next, the QR code does not work or generate (says error, cannot generate).
This command fixes it:
docker run -p 8080:8080 \
    -v $HOME/signal-cli-config:/home/.local/share/signal-cli \
    -e 'MODE=normal' bbernhard/signal-cli-rest-api

Then I'm able to complete step 1 of the README.
Is this helpful? Do you need any further details on these errors to reproduce? Thanks so much!!

Closing due to inactivity, please re-open if you are still having issues.


<EMAIL_ADDRESS>

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: _______
[ ] ACS AEM Commons Version: _____
[ ] Reproducible on Latest? yes/no

Expected Behavior
Please describe briefly the expected behavior, i.e. when I do X, Y should happen.
Actual Behavior
Please describe briefly the actual behavior, i.e. when I do X, Z happens.
Steps to Reproduce
Starting from a vanilla instance of AEM at the above version, describe in explicit steps how to reproduce. Link to
any supporting assets, e.g. content packages, below
Links
Links to related assets, e.g. content packages containing test components

@nanibos80 Your description was probably lost?

@nanibos80 Please re-open when you provide a issue description - thanks!


SAML authentication - Teams tab SSO

If the application supports SAML SSO, then we could not able to use tab SSO generated JWT token. As it is not supported, Can you add the note for the same.

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

ID: 6d5b8b0f-3da3-8642-6edf-dc6608a830cf
Version Independent ID: f93cdabd-f6ab-29d0-f3c9-78deb21ca521
Content: Authentication flow for tabs - Teams
Content Source: msteams-platform/tabs/how-to/authentication/auth-flow-tab.md
Product: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

@v-shalinirobinson - Could you please help here to add the note in below doc
Authentication flow for tabs - Teams
Content for addition - If the application supports SAML SSO, then we could not able to use tab SSO generated JWT token. As it is not supported


Add critical error when webwork isn't built correctly

Oops, this was meant for pretext-cli, where it is already fixed.


Wrong ruby version invoked to run ruby script from Rails controller

Admittedly stabbing the dark trying to determine whether rbenv, passenger, or bundler is responsible (and therefor not configured correctly by me).
Rails is running as 2.7.4, but calling a ruby script in a controller is causing 2.7.0 to be invoked.
Error:
/home/rcm/.rbenv/versions/2.7.4/lib/ruby/2.7.0/bundler/definition.rb:495:in `validate_ruby!':
Your Ruby version is 2.7.0, but your Gemfile specified 2.7.4 (Bundler::RubyVersionMisMatch)

Environment:
Ubuntu 20 > Apache > Passenger > rbenv > Rails > Controller > ruby script invoked with backticks
I have a collection of ruby scripts which run as CLI commands to talk to hardware external to the host. These are shared by multiple applications written with different front ends. All work fine called directly, and work fine when called by a Rails controller in dev mode, and even rails server -e production mode.
Specifically when running under Apache > Passenger, I get the mismatch error.
rbenv global is set to 2.7.4
There are .ruby-version files at multiple levels.
Gemfile has ruby '2.7.4'
Apache conf has PassengerRuby /home/rcm/.rbenv/versions/2.7.4/bin/ruby
From inside the controller
ENV['PATH'] = home/rcm/.rbenv/versions/2.7.4/lib/ruby/gems/2.7.0/bin:/usr/local/sbin:... etc
which ruby = /usr/bin/ruby <-- why not the shimmed version?
If I eliminate the ruby line from the Gemfile, this eliminates the Mismatch error since there's no match to enforce, and the script runs. As a workaround for now that's fine (2.7.0 vs 2.7.4 not a big deal in the immediate term), but I'd still like to know what's happening here.
Is there any way that rbenv's version shimming is involved in this?
Sorry for the length. Thanks for reading.

This sounds tricky. I'll try my best to offer some advice here, but in the end it will be up to you to untangle what's happening here.

which ruby = /usr/bin/ruby <-- why not the shimmed version?

I can't tell since you haven't shown us the contents of your full PATH variable, or at least the relevant entries. The which utility is very simple: it prints the first executable found when searching in each of the directories listed in PATH. If you are confused at the output of which ruby, then please check your PATH; specifically whether the /home/<user>/.rbenv/shims directory has precedence over other Rubies installed on the system.
It sounds like your app is shelling out to a Ruby script that is not part of the same app, i.e. that it's not meant to use the same gem bundle. If so, you need to reset Bundler-related environment variables when shelling out, such as BUNDLE_GEMFILE. Bundler offers a Ruby method to do this for you: https://bundler.io/man/bundle-exec.1.html#Shelling-out
Finally, keep in mind that rbenv doesn't seamlessly support shelling out to a different Ruby version than the one that's currently activated by the app: https://github.com/rbenv/rbenv/issues/121. When shelling out to a different Ruby version, you may need to edit your PATH to take out the entry /home/<user>/.rbenv/versions/<version>/bin that was prepended to PATH by rbenv when launching your Ruby app in the first place.


Unit tests fail due to strict mode error

Link to PDF file (or attach file here):
N/A
Configuration:

Web browser and its version: Firefox 45.4.0
Operating system and its version: Debian Jessie
PDF.js version: Latest from git
Is an extension: No

Steps to reproduce the problem:

Run gulp unittest

What is the expected behavior? (add screenshot)
Tests run to completion
What went wrong? (add screenshot)
The Jasmine test runner does not load properly in Firefox, and the following error shows up in the browser console.
SyntaxError: in strict mode code, functions may be declared only at top level or immediately within another function worker.js:93:9
(this line is function IPDFStream() {}, which is inside if (typeof PDFJSDev === 'undefined' || !PDFJSDev.test('PRODUCTION')) {...})
This regression was introduced in 0576c9c, from #7322. The error is similar to #2172.
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):
N/A

The unit tests run fine on the Windows/Linux bots that we use. Could you test with the most recent version of Firefox?

Yes, the problem went away with a newer Firefox. I had used 45.4.0 ESR at first since it's the default for Debian.

Closing as fixed by upstream in that case.


iPad popover not appear in 1.7.8 with 3.5.0.GA

var Social = require('dk.napp.social');
        var params = {
            removeIcons: "weibo,print,contact,camera,copy,twitter"
        };
//            params.text = 'share this thing';

        params.view = myBtn;
        Social.activityPopover(params);

This is my code. The popover does not show up in 1.7.8 AND 1.7.7.
Tested from 1.7.4 to 1.7.6. All are working and showing the popover. But I can't tap beside the popover to hide it.
Maybe it is based on, that a popover needs a contentview? No error is shown in the logs.

I have compiled the module with SDK 3.5.0.GA and get the same issue. On iPad no popover appears.

I have the same issue with 1.7.7 on 3.4.0.GA

Great! Thanks


Add tests to increase code coverage

Codecov Report

Merging #5 (503545f) into master (23434f6) will increase coverage by 2.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   96.93%   98.97%   +2.04%
==========================================
  Files           3        3
  Lines          98       98
==========================================
+ Hits           95       97       +2
+ Misses          3        1       -2

Impacted Files
Coverage Δ

src/atomics.jl
100.00% <0.00%> (+1.75%)
:arrow_up:

src/threadtasks.jl
96.55% <0.00%> (+3.44%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 23434f6...106b51c. Read the comment docs.

Codecov Report

Merging #5 (503545f) into master (23434f6) will increase coverage by 2.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   96.93%   98.97%   +2.04%
==========================================
  Files           3        3
  Lines          98       98
==========================================
+ Hits           95       97       +2
+ Misses          3        1       -2

Impacted Files
Coverage Δ

src/atomics.jl
100.00% <0.00%> (+1.75%)
:arrow_up:

src/threadtasks.jl
96.55% <0.00%> (+3.44%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 23434f6...106b51c. Read the comment docs.

Thanks, looks good.

Thanks, looks good.


laser_filters: 1.8.12-1 in 'melodic/distribution.yaml'

Please add the following dependency to the rosdep database.
Package name:
TODO
Package Upstream Source:
TODO link to source repository
Purpose of using this:
TODO Replace. This dependency is being used for this reason. This is why I think it's valuable to be added to the rosdep database.
Distro packaging links:
Links to Distribution Packages

Debian: https://packages.debian.org/

REQUIRED

Ubuntu: https://packages.ubuntu.com/

REQUIRED

Fedora: https://src.fedoraproject.org/browse/projects/

IF AVAILABLE

Arch: https://www.archlinux.org/packages/

IF AVAILABLE

Gentoo: https://packages.gentoo.org/

IF AVAILABLE

macOS: https://formulae.brew.sh/

IF AVAILABLE

Alpine: https://pkgs.alpinelinux.org/packages

IF AVAILABLE

NixOS/nixpkgs: https://search.nixos.org/packages

OPTIONAL

Please Add This Package to be indexed in the rosdistro.
ROSDISTRO NAME
The source is here:
http://sourcecode_url
Checks

[ ] All packages have a declared license in the package.xml
[ ] This repository has a LICENSE file
[ ] This package is expected to build on the submitted rosdistro

I created this PR manually because bloom-release wasn't letting me update the source and doc branches. For some reason the PR defaulted to the "adding a rosdep rule" text in the description; this is actually just a release of the laser_filters package.


UNION ALL not supported

I saw that there was a previous task created for this but I think that Verdict has been reworked a lot and support for unions was probably lost somewhere along the way.
Here are some tests I ran.. Excuse the formatting it's from a proxy that returns JSON.
I currently have a workaround implemented for our use case but this would be awesome to have in the core. :smile:
-- "SELECT count(distinct mycolumn) FROM table_b_orc"
{
    "results": [
        {
            "cd2": "4.0"
        }
    ]
}

-- "SELECT count(distinct mycolumn) FROM table_a_orc"
{
    "results": [
        {
            "cd2": "6.0"
        }
    ]
}

-- "SELECT count(distinct mycolumn) FROM table_b_orc UNION ALL SELECT count(distinct mycolumn) FROM table_a_orc"
{
    "results": [
        {
            "cd2": "4.0"
        }
    ]
}

-- "BYPASS SELECT count(distinct mycolumn) FROM table_a_orc UNION ALL SELECT count(distinct mycolumn) FROM table_b_orc"
{
    "results": [
        {
            "_col0": "4"
        },
        {
            "_col0": "6"
        }
    ]
}

@Beastjoe Could you take a look at this issue (add relevant test cases too)? I think this should be a simple logical issue.

@pyongjoo Previously, I think we haven't considered the case of set operation inside user query. You can check visitQuery_expression() method in RelationGen class. It may be a bit complex to fix this. One solution is to treat the SetOperationRelation as a SelectQuery, but I am not sure what to do when scramble tables are involved. Do you have any suggestions? Thanks.

SetOperationRelation must not be SelectQuery simply because it is not. SetOperationRelation should just be a subclass of AbstractRelation.
Doesn't the antlr grammar already cover the case that a relation can include UNION? Otherwise, we can simply extend it.

Yes, the grammer has already covered the case. QueryNode only handles SelectQuery, I think some similar classes need to be created to handle the set operation inside user query.
Besides, I am not sure what should we do when both relations in set operation involves scramble tables, such as
SELECT count(distinct l_suppkey) FROM lineitem UNION ALL SELECT count(distinct s_suppkey) FROM supplier

or
SELECT count(distinct l_suppkey) FROM lineitem UNION ALL SELECT count(distinct l_suppkey) FROM lineitem

Hmm. I thought the query is like
select count(distinct col) from
( select * from tableA UNION ALL select * from tableB) t
which was the reason I said the union must not be treated as a select query. Let me double-check the OP’s another query example and let you know.
In this case, however, I agree that the union query should also be a select query.

@commercial-hippie I just noticed that the example query you posted here is different from the one I saw before. The one I saw before was:
select count(distinct some_id)
from (
  (select some_id
   from table1)
  UNION ALL
  (select some_id
   from table2)
)

Can you confirm which one we should focus on? The query you posted here is nothing but two select queries.

@pyongjoo I think the query you posted is probably the way to go..

@Beastjoe Can you quickly test if the query as follows runs without errors?
select count(distinct some_id)
from (
  (select some_id
   from table1)
  UNION ALL
  (select some_id
   from table2)
)

No. VerdictDB currently neglects the right side of the union all operation and only takes the leftmost query. I will start to fix this.

Thanks!

@Beastjoe Can you comment on the current status? No rush, but just wondering. Thanks.

I have finished standardizing query containing the set operations. The next step is to add support of creating AsyncAggExecutionPlan for set operations.
For instance, if a query is
select count(distinct some_id)
from (
  (select some_id
   from table1)
  UNION ALL
  (select some_id
   from table2)
)

my implementation will change the query to
select count(distinct some_id)
from (
  select some_id
  from (
    (select some_id
     from table1)
    UNION ALL
    (select some_id
     from table2)
))

If the set operation contain some scramble tables, we can then add the constraint in the inner query.

Instead of adding an extra depth for the inner query, could we simply make UNION as an instance of SelectQuery?

I think it is feasible.
For instance,
select count(distinct some_id)
from (
  (select some_id
   from table1)
  UNION ALL
  (select some_id
   from table2)
)

If table1 and table2 are scramble tables, and we wish to add the filter later, such as verdictdbblock, is it correct to add them directly in the set operation queries?

@Beastjoe I believe these are the todos we discussed. Please pursue these items as separate pull requests.

No automatic table name replacements; simply require users to explicitly specify scrambles in place of the original tables.
Ensure that all unioned tables have the same sampling probabilities.
Implement async aggregation plan for unioned tables.


Export visuals from Blender as .GLB

This change will export visuals as glTF models. I have some issues with loading materials from .obj atm (nothing serious, but it will take some time to fix it). Gazebo won't be able to load glTF, but afaik, we currently don't use the models in Gazebo.

Just rebased from the main feature branch this PR relates to


feat(Table): allow dynamically render checkbox

🔗 Linked issue
Resolves https://github.com/nuxt/ui/issues/2377, resolves https://github.com/nuxt/ui/issues/2222, resolves https://github.com/nuxt/ui/issues/2224, resolves https://github.com/nuxt/ui/issues/1358

❓ Type of change

[x] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
New Features

Introduced singleSelect prop (default: false) to control how the select function allows only one row to be selected at a time.
Added slot select-header allow user to customize column header checkbox interface
Added slot select-data allow user to customize row checkbox interface

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

I should update docs for new slot and give example to hide selected column in header.

@rdjanuar When using a v-model, it seems the first column is erased by the checkbox due to this PR. Could you look into it?

Okay, i look into it @benjamincanac

Thanks!


Replace awesome-slugify with python-slugify

This is to resolve #1618.

Thanks for your contribution @hongquan!


Pluggable encryption

I came here after looking at pass and seeing gpg tossed everywhere.  I have a Trezor which has offloaded encryption and decryption functions exposed via protobuf.  Would this be doable without major refactoring in gopass?  95% of the work I wouldn't want to write myself is git management, bash completion, etc.

You could take a look at this PR: https://github.com/justwatchcom/gopass/pull/645
It's adding another crypto backend so gpg wouldn't have to be used. This should give you an idea of the work involved if you want to work on it.

@rendaw To do this does actually require a major refactoring, as you can see in the PR @dewey mentioned.
Good things is that it's already being done and should be ready in a few short weeks. After #645 is merged it should be a lot easier (if not trivial) to add a different crypto backend. I have not access to this device but I'd be happy to assist you if you want to contribute a backend for that.

Awesome, I'll watch that.  Thanks @dewey @dominikschulz !
And this is off the cuff so I haven't thought about it much, but if you were interested and had it on an Amazon wishlist I might get it for you.

This should be resolved by #645 .


Blank screen

I downloaded the sample project and I see blank screen with buttons after clicking "Add Camera".
Sample project is not working.

I really want this library to work asap. Is there any quick fix or solution?


Menu icon is placed on top of first tab's icon

Describe the bug:
Same as #112 and the clock is not working too
Config YAML:
<!-- Paste your lovelace code for the card after this line -->
clock_am_pm: false
clock_date: true
clock_format: '24'
exceptions: []
main_config: true
type: 'custom:compact-custom-header'
disable: false
header: true
voice: clock
notifications: overflow
chevrons: false

HA and/or Browser Error:

Still functional, but i cant click the icon that is overlaped.
HA, Card, and Browser Versions:

Hassio: 0.93.0
compact-custom-header: 1.0.4
Chrome/74.0.3729.131

There are no errors in HA logs or Chrome Dev Tools console (F12)?
This might be a caching issue.

If you aren't using custom_updater make sure you're adding to the version number in resources
Be sure to clear your cache and refresh the page.

You found my mistake, i wasn't putting the version of the custom cards, in the raw config. I was thinking that when custom_updater was putting automatically when i updated the card and components.
Sorry to bother and thanks for helping me. :)


doc: update docs for flags

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


Support for multiple game windows

I personally cannot find a use case for it, but some online research suggests it's a feature worth looking into

meh. changed my mind


Add dummy.go to licenses dir

This allows users of https://github.com/google/go-licenses to run the tool in a go
module compliant way by depending on this package, and running the tool. Without this
that invariably fails with 'licenses.db cannot be found'.
With this, users can add
github.com/google/go-licenses
and
github.com/google/licenseclassifier/licenses
to their hack/tools.go file, and then the run of the license classifier is hermetic (uses only the versions known by go modules, which prevents weird inconsistencies due to mismatched tool versions).

/assign @wcn3

Hm maybe that doesn't work here :) Welp, anyone who can review, it'd be appreciated!

Thanks for assigning Robert :)


F# using keyword is not highlighted

From https://github.com/Microsoft/vscode/issues/4408:
Steps to Reproduce:
1.Open an F# source/script file in VS Code
open System.IO

let writetofile2 filename obj =
    using (System.IO.File.CreateText(filename)) ( fun file1 ->
        file1.WriteLine("{0}", obj.ToString() )
    )

writetofile2 "abc2.txt" "The quick sly fox jumped over the lazy brown dog."

2.Set the language to  F#
3.The  using  keyword is not syntax-highlighted
Problem is that using (System.IO.File.CreateText(filename)) ( is all one single token.

using is a function not a keyword
Syntax highlighting issues should be reported at https://github.com/ionide/ionide-fsgrammar/issues


Configurable default response_type

Feature Description

Currently the response_type is defaulted to "token" and this is the parameter that's used in email links. Some libraries like pow-assent only support code response types, so the current solution is to manually redefine all of these templates.
Describe the solution you'd like

A DEFAULT_RESPONSE_TYPE environment variable which allows us to override this. We could maintain backwards compatibility by defaulting to "token" when this is missing.
Describe alternatives you've considered

Additional context

@feynmanliang thank you for sharing this,
I will add this env over the weekend 👍

@feynmanliang added support for DEFAULT_AUTHORIZE_RESPONSE_TYPE and DEFAULT_AUTHORIZE_RESPONSE_MODE in #342 will be released as part of 1.1.43

Thank you!! 🥳 🎈


Fixes for Clover reporter

After switching from gotwarlost/instanbul, we noticed two major problems in Clover reports:

The unclosed <metrics /> tag prevents Jenkins' plugin from showing package details
The number of reported statements differs from the HTML report. It turned out that's because Clover reportee used lines instead of statements.

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:

@yahoocla CLI is now signed.


Executed a statement , but no operation was performed

Hi, I executed a statement, but no operation was performed. The log is as follows. Please help me check what the problem is.
PS D:\company\UFO-main> python -m ufo --task safetest
Welcome to use UFO??, A UI-focused Agent for Windows OS Interaction.

| | | ||  | / _
| | | || |   | | | |
| || ||  |  | || |
_/ |_|     __/
Please enter your request to be completed??:
open chrome and access the google search page
Round 1, Step 1, HostAgent: Analyzing the user intent and decomposing the request...
Observations??: I observe that the current screenshot shows the Chrome browser is visible, and the Google search page is opened.
Thoughts??: The user request can be solely complete on the Chrome browser. I need to open the Chrome browser to access the Google search page.Plans??: (1) Open the Chrome browser to access the Google search page.
Next Selected application??: Chrome
Messages to AppAgent??: (1) You need to open the Chrome browser to access the Google search page.
(2) You can use the 'Start' menu to open the Chrome browser.
(3) Alternatively, you can press the 'Ctrl + Shift + J' key combination to open the Chrome browser directly.
Status??: CONTINUE
Comment??: I plan to open the Chrome browser to access the Google search page.
Creating an experience indexer...
No sentence-transformers model found with name sentence-transformers/all-mpnet-base-v2. Creating a new one with MEAN pooling.
Warning: Failed to load experience indexer from vectordb/experience/experience_db.
Creating an demonstration indexer...
No sentence-transformers model found with name sentence-transformers/all-mpnet-base-v2. Creating a new one with MEAN pooling.
Warning: Failed to load demonstration indexer from vectordb/demonstration/demonstration_db.
Round 1, Step 2, AppAgent: Completing the subtask [Open the Chrome browser to access the Google search page.] on application [Chrome].
Error Occurs at AppAgentProcessor
Traceback (most recent call last):
File "D:\company\UFO-main\ufo\agents\processors\app_agent_processor.py", line 264, in parse_response
self.response_json = self.app_agent.response_to_dict(self.response)
File "D:\company\UFO-main\ufo\agents\agent\basic.py", line 162, in response_to_dict
return utils.json_parser(response)
File "D:\company\UFO-main\ufo\utils_init.py", line 76, in json_parser
return json.loads(json_string)
File "D:\Programs\Python\Python310\lib\json_init.py", line 346, in loads
return _default_decoder.decode(s)
File "D:\Programs\Python\Python310\lib\json\decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "D:\Programs\Python\Python310\lib\json\decoder.py", line 353, in raw_decode
obj, end = self.scan_once(s, idx)
json.decoder.JSONDecodeError: Expecting ',' delimiter: line 1 column 765 (char 764)
{"Observation": "The screenshot shows that I am on the Chrome browser. The Google search page is visible in the screenshot. The last action took effect by opening the Chrome browser to access the Google search page.", "Thought": "Based on the screenshots and the control item list, I need to use the 'search_box' API to open the Google search page. This API is suitable for searching for information on the internet.", "ControlLabel": "search_box", "ControlText": "Google Search", "Function": "search_box", "Args": {}, "Status": "CONTINUE", "Plan": ["(1) Call the 'search_box' API to open the Google search page.", "(2) The Google search page is now open.", "(3) The user can now search for information on the internet.", "(4) The task is now complete.", "Comment": "By using the 'search_box' API, I have opened the Google search page, allowing the user to search for information on the internet.", "SaveScreenshot": {"save": false, "reason": ""}]}
{"Observation": "The screenshot shows that I am on the Chrome browser. The Google search page is visible in the screenshot. The last action took effect by opening the Chrome browser to access the Google search page.", "Thought": "Based on the screenshots and the control item list, I need to use the 'search_box' API to open the Google search page. This API is suitable for searching for information on the internet.", "ControlLabel": "search_box", "ControlText": "Google Search", "Function": "search_box", "Args": {}, "Status": "CONTINUE", "Plan": ["(1) Call the 'search_box' API to open the Google search page.", "(2) The Google search page is now open.", "(3) The user can now search for information on the internet.", "(4) The task is now complete.", "Comment": "By using the 'search_box' API, I have opened the Google search page, allowing the user to search for information on the internet.", "SaveScreenshot": {"save": false, "reason": ""}], "Result": [], "UserConfirm": null}
{"Observation": "The screenshot shows that I am on the Chrome browser. The Google search page is visible in the screenshot. The last action took effect by opening the Chrome browser to access the Google search page.", "Thought": "Based on the screenshots and the control item list, I need to use the 'search_box' API to open the Google search page. This API is suitable for searching for information on the internet.", "ControlLabel": "search_box", "ControlText": "Google Search", "Function": "search_box", "Args": {}, "Status": "CONTINUE", "Plan": ["(1) Call the 'search_box' API to open the Google search page.", "(2) The Google search page is now open.", "(3) The user can now search for information on the internet.", "(4) The task is now complete.", "Comment": "By using the 'search_box' API, I have opened the Google search page, allowing the user to search for information on the internet.", "SaveScreenshot": {"save": false, "reason": ""}], "Result": [], "UserConfirm": null}
{"Observation": "The screenshot shows that I am on the Chrome browser. The Google search page is visible in the screenshot. The last action took effect by opening the Chrome browser to access the Google search page.", "Thought": "Based on the screenshots and the control item list, I need to use the 'search_box' API to open the Google search page. This API is suitable for searching for information on the internet.", "ControlLabel": "search_box", "ControlText": "Google Search", "Function": "search_box", "Args": {}, "Status": "CONTINUE", "Plan": ["(1) Call the 'search_box' API to open the Google search page.", "(2) The Google search page is now open.", "(3) The user can now search for information on the internet.", "(4) The task is now complete.", "Comment": "By using the 'search_box' API, I have opened the Google search page, allowing the user to search for information on the internet.", "SaveScreenshot": {"save": false, "reason": ""}], "Result": [], "UserConfirm": null}
{"Observation": "The screenshot shows that I am on the Chrome browser. The Google search page is visible in the screenshot. The last action took effect by opening the Chrome browser to access the Google search page.", "Thought": "Based on the screenshots and the control item list, I need to use the 'search_box' API to open the Google search page. This API is suitable for searching for information on the internet.", "ControlLabel": "search_box", "ControlText": "Google Search", "Function": "search_box", "Args": {}, "Status": "CONTINUE", "Plan": ["(1) Call the 'search_box' API to open the Google search page.", "(2) The Google search page is now open.", "(3) The user can now search for information on the internet.", "(4) The task is now complete.", "Comment": "By using the 'search_box' API, I have opened the Google search page, allowing the user to search for information on the internet.", "SaveScreenshot": {"save": false, "reason": ""}], "Result": [], "UserConfirm": null}
{"Observation": "The screenshot shows that I am on the Chrome browser. The Google search page is visible in the screenshot. The last action took effect by opening the Chrome browser to access the Google search page.", "Thought": "Based on the screenshots and the control item list, I need to use the 'search_box' API to open the Google search page. This API is suitable for searching for information on the internet.", "ControlLabel": "search_box", "ControlText": "Google Search", "Function": "search_box", "Args": {}, "Status": "CONTINUE", "Plan": ["(1) Call the 'search_box' API to open the Google search page.", "(2) The Google search page is now open.", "(3) The user can now search for information on the internet.", "(4) The task is now complete.", "Comment": "By using the 'search_box' API, I have opened the Google search page, allowing the user to search for information on the internet.", "SaveScreenshot": {"save": false, "reason": ""}], "Result": [], "UserConfirm": null}
{"Observation": "The screenshot shows that I am on the Chrome browser. The Google search page is visible in the screenshot. The last action took effect by opening the Chrome browser to access the Google search page.", "Thought": "Based on the screenshots and the control item list, I need to use the 'search_box' API to open the Google search page. This API is suitable for searching for information on the internet.", "ControlLabel": "search_box", "ControlText": "Google Search", "Function": "search_box", "Args": {}, "Status": "CONTINUE", "Plan": ["(1) Call the 'search_box' API to open the Google search page.", "(2) The Google search page is now open.", "(3) The user can now search for information on the internet.", "(4) The task is now complete.", "Comment": "By using the 'search_box' API, I have opened the Google search page, allowing the user to search for information on the internet.", "SaveScreenshot": {"save": false, "reason": ""}], "Result": [], "UserConfirm": null}
{"Observation": "The screenshot shows that I am on the Chrome browser. The Google search page is visible in the screenshot. The last action took effect by opening the Chrome browser to access the Google search page.", "Thought": "Based on the screenshots and the control item list, I need to use the 'search_box' API to open the Google search page. This API is suitable for searching for information on the internet.", "ControlLabel": "search_box", "ControlText": "Google Search", "Function": "search_box", "Args": {}, "Status": "CONTINUE", "Plan": ["(1) Call the 'search_box' API to open the Google search page.", "(2) The Google search page is now open.", "(3) The user can now search for information on the internet.", "(4) The task is now complete.", "Comment": "By using the 'search_box' API, I have opened the Google search page, allowing the user to search for information on the internet.", "SaveScreenshot": {"save": false, "reason": ""}], "Result": [], "UserConfirm": null}
{"Observation": "The screenshot shows that I am on the Chrome browser. The Google search page is visible in the screenshot. The last action took effect by opening the Chrome browser to access the Google search page.", "Thought": "Based on the screenshots and the control item list, I need to use the 'search_box' API to open the Google search page. This API is suitable for searching for information on the internet.", "ControlLabel": "search_box", "ControlText": "Google Search", "Function": "search_box", "Args": {}, "Status": "CONTINUE", "Plan": ["(1) Call the 'search_box' API to open the Google search page.", "(2) The Google search page is now open.", "(3) The user can now search for
Observations??: None
Thoughts??: None
Selected item???: [No control selected.], Label: [No control label sel
ected.]
Action applied??: None()
Status??: None
Next Plan??:
Comment??: None
Please enter your new request. Enter 'N' for exit.

Which model do you used? It seems that the model failed to produce a json format output, so its action cannot be parsed


always use $HOME/data/userdata

fixes https://github.com/openshift/postgresql/issues/76

@praiskup @rhcarvalho @mnagy ptal.  logically this seems safe, but maybe i'm missing something?

The existing scheme of using one dir or another is already problematic, we could in consecutive runs end up with data in /data and next in /data/userdata.
Now, why do we make up the dir name "userdata"?

The only potential problem I see with this change is breaking people that fell on the first branch of the conditional and have data stored under /data.

@rhcarvalho good point about compatibility, i've checked in an attempt to include backwards compatibility in the logic and fixed your other comments, ptal.

I'm not sure about the ${HOME}/data/PG_VERSION. Assuming you tested that, LGTM.
BTW -- what I've run into in the past with misconfigured permissions is that data/userdata might exist and NOT be owned by the user...
As an improvement, we might want to test that the directory we are setting up as PG_DATA is writable by the user, and if not provide an error message instead of failing later with cryptic messages.

I'm not sure about the ${HOME}/data/PG_VERSION. Assuming you tested that, LGTM.

i started up an image and confirmed that file gets created in that dir, that was the basis for the logic.

Looks OK, though I would probably rather return non-zero value (or exit
immediately with "fatal" error) after 'mkdir -p' failure.

@rhcarvalho wrote

Now, why do we make up the dir name "userdata"?

That is good question, btw.

i thought we are not using $HOME but absolute path as $HOME is something that might change and depend on the user environment.

In case of 'postgres' user, it is created by RPM and $HOME directory should
be properly pre-defined (and I would really discourage to change the default).
From this POV it looks fine, but I do not object if you plan to change this
to absolute path.

@praiskup

Looks OK, though I would probably rather return non-zero value (or exit
immediately with "fatal" error) after 'mkdir -p' failure.

the start scripts that invoke this run with -e, so this should exit on error.
@rhcarvalho

Now, why do we make up the dir name "userdata"?

because it's the data dir and its owned by the user.


Fixed issue with markdown conversion

Fixed issue with markdown conversion. Previously used Pandoc to create pdf's from markdown, started to get an error with the mat.write_pdf(file) in the unit_tests.py. This change implements md2pdf which completes the conversion however still has issues with $math$ symbol rendering on the final pdf.

@hespuck19, does the new converter work for SS316L?

Yep. Still works with the change to the file path part. Did you want two test cases one with AlCu and one with SS316L?
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Stephen DeWitt @.>
Sent: Friday, June 14, 2024 3:29:56 PM
To: ORNL-MDF/mist @.>
Cc: Spuck, Hope @.>; Mention @.>
Subject: Re: [ORNL-MDF/mist] Fixed issue with markdown conversion (PR #6)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
@hespuck19https://github.com/hespuck19, does the new converter work for SS316L?
—
Reply to this email directly, view it on GitHubhttps://github.com/ORNL-MDF/mist/pull/6#issuecomment-2168636049, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BGSKMTLNES6ELWZPYXUXUR3ZHNADJAVCNFSM6AAAAABJJBAEK2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRYGYZTMMBUHE.
You are receiving this because you were mentioned.Message ID: @.***>

@hespuck19, yes, please include both AlCu and SS316L in the test.


Report button

Several others have said there report button doesn’t work. This is bad because it means people can’t report sexual posts that are harassing others. Please, please try to fix this. Thank you.

It does work, but only temporarily. It's good that it only works temporarily, or malicious users could force others to stop posting, or even create an automated system that reports every post as soon as it is posted. Still, there needs to be more moderation and a profanity filter. If the IP addresses of malicious posts and the accounts of malicious posts were banned, there wouldn't be as much of a problem.

I see that this has been implemented.


Compilation error for fenv_t while building for aarch64-linux-gnu.

While building ortools for aarch64-linux-gnu getting below error:
In file included from ./ortools/util/fp_utils.cc:14:0:
./ortools/util/fp_utils.h: In member function 'void operations_research::ScopedFloatingPointEnv::EnableExceptions(int)':
./ortools/util/fp_utils.h:87:11: error: '**struct fenv_t**' has no member named '**__control_word**'
     fenv_.__control_word &= ~excepts;
           ^
./ortools/util/fp_utils.h:89:11: error: 'struct fenv_t' has no member named '**__mxcsr**'
     fenv_.__mxcsr &= ~(excepts << 7);
           ^
make: *** [objs/util/fp_utils.o] Error 1

This works with g++ compiler, i am not understanding why?
Please help me if there are any clue.

hi,
According to https://android.googlesource.com/platform/bionic/+/afe835d/libc/arch-arm64/include/machine/fenv.h#78 i would say fenv struct is like on Apple
so in https://github.com/google/or-tools/blob/master/ortools/util/fp_utils.h#L85
We should use something like
#if defined(__APPLE__) || defined(__arm__)
    fenv_.__control &= ~excepts;
#else  // Linux
    fenv_.__control_word &= ~excepts;
#endif

Will be better to merge this, no ?


detectron2: ImportError: cannot import name '_C'

It raise import error when I tried to run demo file

@ @vanchung1995   You didn't install the conda environment and build our TextFuseNet-detectron2 correctly, and we will upload a step-by-step installation as soon as possible. please pay attention to our update.

Thanks a lot. But how can I install it on pip? Can you create guideline for pip install ?

The reason is detection was not installed correctly. Do the follow link:
https://colab.research.google.com/drive/16jcaJoc6bCFAQ96jDe2HwtXj7BMD_-m5#scrollTo=b-i4hmGYk1dL

update torch==1.6 and torchvision==0.7


make flash problem

Hi,
I am trying to install esp-idf in Windows 10 but "make flash" has this problem :
mainuser@mainpc MINGW32 ~/esp/hello_world
$ make flash
Traceback (most recent call last):
File "C:/msys32/home/mainuser/esp/esp-idf/components/esptool_py/esptool/esptool.py", line 21, in
import argparse
File "C:\Python27\Lib/argparse.py", line 85, in
import collections as _collections
File "C:\Python27\Lib/collections.py", line 20, in
from _collections import deque, defaultdict
ImportError: No module named _collections
make[1]: *** [/home/mainuser/esp/esp-idf/components/esptool_py/Makefile.projbuild:51: /home/mainuser/esp/hello_world/build/bootloader/bootloader.bin] Error 1
make: *** [/home/mainuser/esp/esp-idf/components/bootloader/Makefile.projbuild:40: /home/mainuser/esp/hello_world/build/bootloader/bootloader.bin] Error 2
Any help ?.
Thanks in advance and regards.

Hi @JPMJPM ,
It looks like IDF is pulling in a different Python install, an external one at C:\Python27. Then this is failing to import a Python built-in module, I don't know why that would be - maybe that external Python install is broken somehow?
You run "make menuconfig" and try changing "Python 2 Interpreter" under "SDK Tools Configuration" to "/mingw32/bin/python", does it then work?

Thanks projectgus,
With "/mingw32/bin/python" in "SDK Tools Configuration" it does not work but with
"C:/Python27/python" everything work fine.
Regards.

Glad you found a solution. I'm guessing this has something to do with the PATH environment variable. To help us see if we can work around this for other users, could you please run the command  env | grep -i python in the MINGW32 terminal and paste the output here?

mainuser@mainpc MINGW32 ~/esp
$ env | grep -i python
PYTHONHOME=C:\Python27
PYTHONPATH=C:\Python27\Lib
mainuser@mainpc MINGW32 ~/esp
$

How did start the shell? Run mingw32.exe or run the bat with -mingw32 argument should work.

I start the shell running a desktop shortcut with target argument "C:\msys32\mingw32.exe".

echo $PATH
env | grep MSYS
env | grep MINGW

/mingw32/bin/python works great on my device.

@fanthos It's the external Python install and (I think) PYTHONPATH environment variable which makes the difference here.
@JPMJPM If you want to fix this problem without needing to change sdkconfig for each project then you can profile add a line "unexport PYTHONPATH" to /etc/profile or ~/.profile inside the MSYS2 environment. Then it will use the bundled Python for everything and not call out to C:\Python27.
However, if you're happy with the solution you already have then that should be fine too.

Hi projectgus,

unexport PYTHONPATH in /etc/profile ---> unexport, command not found
unexport PYTHONPATH in ~/.profile ---> the known error ImportError: No module named _collections

Thanks @JPMJPM for the update. Sorry for the slow reply.
If you open the shell and run:
echo $PATH
echo $PYTHONPATH

... what is printed?

Thanks @JPMJPM  for the update. Sorry for the slow reply.
To be honest, I don't fully understand how Windows environment varialbes are exported into MSYS2. Maybe something like:
export PYTHONPATH=""

will work better?

I think the command should be unset PYTHONPATH or something.


fix: problems deleting gameobjects with children

When creating the scene all gameobjects go to vector sceneCamera provoking that when we search for objects inside camera we take those objects as cameras.
#172 must be approved in order to try this changes out correctly.

Task linked: CU-865bubdex Problem with GetComponent

This is the addition I did, in case you want to try it in you machine:

For easy copy-pase:
if (ImGui::MenuItem("REMOVE: Add camera"))
{
	App->scene->GetSelectedGameObject()->CreateComponent(ComponentType::CAMERA);
	auto debug = App->scene->GetLoadedScene()->GetSceneCameras();
	debug.push_back(App->scene->GetSelectedGameObject());
	App->scene->GetLoadedScene()->SetSceneCameras(debug);
}

This has nothing to do with memory leaks, this is an issue of the timing of GameObjects and Components deletition, which ends up causing code to run on garbage memory. Also, please report whatever bug you found so we can all be up to date. And this MR is perfectly fine as is, you fixed an error with scene deserialization, so no need to close it.

I insist, the problem with the memory leak affects also this scope what you are talking (GameObjects and Components deletition) i dont want to push something that solves half of the problem, so this pr will be closed until i found a solution that could be solf the whole problem. Thanks you for your comprehension
P.S.: later on i will write about the memory leak problem

See issue #169 for what is solved

Wait, nevermind the first error that I reported, forgot that we fixed it in #172.

If this were the cause of this issue it would crash everytime we delete something, as it stands it seems to crash randomly on develop. We certainly need to clear the various collections in the engine when deleting, but again, I seriously doubt this is what's causing that. Regardless, I think a better way of doing this would be to exploit the deleters of smart pointers. What you did here would need to be replicated in the Quadtree, and also do a similar thing for Components and Resources. Hopefully you see why I don't consider this the best approach. I talked a bit about this in #166

Good catch, but while i was implementing it i saw that if we delete the component Camera we no longer can track by this solution, because the camera will remain inside cameraScene but we can not ensure that the gameobject is a camera

Wait, ComponentCamera should not have the option to be removed, its canBeRemoved variable is set to false, so the Remove Comp. button should not appear in the ImGui.

Seems that something happens, because the button is available to delete the component

Okay, looks like when the component windows were decoupled from the Component::Draw() the canBeRemoved functionality was not taken into account, let me open an issue for this.

Done, addressed in #176

If this were the cause of this issue it would crash everytime we delete something, as it stands it seems to crash randomly on develop. We certainly need to clear the various collections in the engine when deleting, but again, I seriously doubt this is what's causing that. Regardless, I think a better way of doing this would be to exploit the deleters of smart pointers. What you did here would need to be replicated in the Quadtree, and also do a similar thing for Components and Resources. Hopefully you see why I don't consider this the best approach. I talked a bit about this in #166

it's true that crash randomly but only in the last scene, if you load the first one it happens on every father that you want to delete (forget floor it crashes always but for different reasons). I don't see why using the deleters of smart pointers would improve the solutions, having in mind that the problem was that we not clear the vectors, what is the solution that you propose?

Thank you for pointing out another way to force the crash, the more information we have the better. And sorry if I didn't explain myself properly, I'll try to lay out my idea. I'll try to keep it short.
The definition of a smart pointer like std::unique_ptr<A>() is actually hiding another template argument that the class has. If you hover over it, you will notice that it's setting an argument with type std::default_delete<A>. In the documentation, you can see that this is basically a class that calls delete a internally.

What I would like to do is to actually set the deleter that we want to use, and use a struct that we define. That is were we would call the removal from the collections. By doing it this way, our solution would be more modular - as I said, what you did here is perfectly correct, but would need to be replicated in Quadtree, and potentially other places, and the same thing will need to happen with other classes like Resource, which will remain in the map inside ModuleResources. Another advantatge is that we would de-couple the objects from the classes that contain them: a GameObject won't need to know in which Quadtrees it's present, or whether it's in ModuleRender's renderList, etc., which is related to the fact that it would be more modular.
To give you an idea of what I have in mind, this is a small prototype that I came up with a few weeks ago:
template<typename P>
class CollectionAwareDeleter
{
public:
	CollectionAwareDeleter(std::vector<P*>& collectionToManage) : collectionToManage(collectionToManage)
	{
	}
	~CollectionAwareDeleter() = default;

	void operator() (P* p)
	{
		collectionToManage.erase(std::remove(std::begin(collectionToManage), std::end(collectionToManage), p));
		delete p;
	}

private:
	std::vector<P*>& collectionToManage;
};

And this is how it could be used:
unique_ptr<A, CollectionAwareDeleter<A>> ptrA = unique_ptr<A, CollectionAwareDeleter<A>>(new A(), CollectionAwareDeleter<A>(rawPtrVector));

This sort of thing is already used in ModuleScene and ModuleInput, where we have unique pointers to SDL structs that need to be deleted in a particular way.
Hopefully I explained myself properly. Again, it's not that your solution is bad by any means, but hopefully you see why I think this would be easier to use in the long term.


Freeze and a lot lag since last update !!!

hi since the last update, i have a lot of lag and freeze , when a jump line in vsc !
befor was very fast rendering color bracket, and some texture element, but now its very slow.
When i jump line, its freeze about (4 to 6 second !)
plz how i can come back to last stable version?, do you have a link ?
i desactivate all plugin and its not a plugin..
edit: Fixed, remove all Breakpoints.
You have a mistake in Breakpoints rendering thats freeze !

Related to https://github.com/Microsoft/vscode/issues/37880?

Yeah, duplicate of #37880
@djmisterjon please update to latest stable and the problem should be fixed


Best way to setup dev machine for LuceneNet website edits

@Shazwazza
Hi Shannon,
I'd like to make the following small addition to the home page of the Lucene.net website give developers more information about the Lucene 4.8 version which will hopefully give them the confidence to give it a try.  Specifically I'd like to add something similar to:
Latest Version - Lucene 4.8 Beta

The beta version is extremely stable
Has more than 7000+ passing unit tests
Integrates well with .Net 5.0
Supports NetStandard 2.0 and NetStandard 2.1
Supports with .Net Framework 4.5.1+
Some developers already use it in production environments

It's a pretty easy add but I need to first configure my dev machine for it.  Initially I just downloaded the a zip file of the master at https://github.com/apache/lucenenet-site/ to check it out.  Looks like a static website, IIS not required, and I didn't see any Visual Studio project files in the folder structure.  So I just double clicked the index.html file to view it in the browser but it doesn't quite render properly.  On my machine it looks like this below (Notice the missing blue banner):

Whereas the actual site looks like this:

So I may be mistaken and it may not be designed to run from the desktop like this.  Any pointers on how I can setup a dev environment locally to do some changes to the index.html file?

@rclabo
The documentation for building the website and API docs sites gives some info about how all of this works, but it would be great for someone to poke holes in that documentation to see how well it holds up to scrutiny.

Updates to the website happen in the pre-build stage in this repo. See apidocs and site.
The documentation is then "assembled" using the DocFx build engine.
The main/API sites are then updated by copying and pasting the files into https://github.com/apache/lucenenet-site/, which is setup to serve the site live.

The build process serves the site up on your localhost so you can preview it as you make changes.
@Shazwazza has been working on getting more of this process automated (like setting it up so accepting a PR automatically builds/deploys the site), but we are not quite there yet.

NOTE: It would be great if we could come up with a way to be able to "preview" pages more quickly than to go through the drawn out build process, if it is possible. Consider that a feature request.

NOTE: I am currently updating that document because we have switched over to using mini-sites for the API docs, which each have their own DocFx configuration file and the document still only mentions the common configuration file that we used when there was one site.

@rclabo The should still be reasonably accurate.  To push this live is done manually now but happy to do that if you just start with a PR for updating the website files.
As @NightOwl888 mentioned I'm still working on automating the deployment of this which is now relying on Infra to get this done https://issues.apache.org/jira/browse/INFRA-21453?filter=-2 . @NightOwl888 do you know any tricks to get them to respond? If that task can be completed, deploying the website is very easy.

@NightOwl888 do you know any tricks to get them to respond? If that task can be completed, deploying the website is very easy.

The only "trick" I know is to keep nagging :unamused:. Crude but effective.

Awesome that all sounds great and was just what I needed to know.  Especially

"The build process serves the sites up on your localhost so you can preview them as you make changes."

and

"With the normal Lucene.Net git source cloned to your machine you can just run this via powershell .\websites\site\site.ps1 -ServeDocs. To push this live is done manually now but happy to do that if you just start with a PR for updating the website files."

Awesome.  It'll be a couple days till I can do this. but I now feel equiped to get make the change, preview it on my machine and submit a PR.  Awesome.

As @NightOwl888 mentioned I'm still working on automating the deployment of this which is now relying on Infra to get this done https://issues.apache.org/jira/browse/INFRA-21453?filter=-2 . @NightOwl888 do you know any tricks to get them to respond? If that task can be completed, deploying the website is very easy.

I replied to the JIRA issue, but given the fact that they wouldn't even generate a temporary PAT for us to migrate our issues from JIRA to GitHub, I don't think it is likely they will allow this. We should put our heads together, do research, and come up with alternatives in case this doesn't happen.

Thanks, at least you got a reply :) They have set this up for other projects (i.e. Apache Arrow) though, they've even slightly documented it: https://infra.apache.org/github-actions-secrets.html

Well, that looks more promising. I guess they are getting tons of requests for these so they are allowing in certain scenarios.

I was able to edit the LuceneNet website home page to include some new content and preview it on my machine and submitted a PR for a few changes.  Progress.  One thing I couldn't do was modify the css because when I made changes to the main.css file my local git didn't notice the changes.
I see the file is located locally at lucenenet/websites/site/_site/styles/main.css  I wonder if it's the leading underscore on _site that is causing the git to not see the changes to the file?  Not sure. Any suggestions on how I can edit the style sheet so that I can affect visual changes that I can see locally and include in a future PR?

Also:
I ran into one other issue.  My local branch I'm working in is 'Documentation_work'.  When I build and serve the website by running the powershell scripst
./websites/site/site.ps1 -ServeDocs
The command appears to be working and serves the website at http://localhost:8081/   but errors are showing up in powershell window  that say things like

[21-03-26 07:38:02.951]Info:BuildCommand.BuildCore.Build Document.Load.ConceptualDocumentProcessorFor git repo <C:/Users/Ron/source/repos/lucenenet>, can't find remote branch in this repo and fallback to use local branch [Documentation_work]: fatal: no upstream configured for branch 'Documentation_work'

I have a remote for my master but it appears to want me to setup a remote for my current dev branch but I'm not sure what it wants the remote set to for that branch.  How do I fix that?

Hi, the _site is the build output of the website so is ignored by git. The source files for the website are here: https://github.com/apache/lucenenet/tree/master/websites/site as for the CSS and layout files, docfx works based on templates and overrides. By default it has it's own internal template including CSS and then we have a custom template to override certain aspects of that. Our custom template is here https://github.com/apache/lucenenet/tree/master/websites/site/lucenetemplate and our custom styles are here https://github.com/apache/lucenenet/tree/master/websites/site/lucenetemplate/styles

DocFx requires an upstream git repository because it it queries for git information and current branch information to determine how things like "Improve this doc" buttons work (among other things especially when building source code docs)

Thank you, that help a lot.  So I guess that means that DocFx is looking at my current git branch.  If my current git branch is one that I'm working in to make DocFx template or layout changes, lets say it called Working_Branch what should I set at the upstream repository to.  Should the upstream for that branch be  https://github.com/apache/lucenenet.git just like for master?

Yeah normally you'd have the upstream git repo set to  https://github.com/apache/lucenenet.git and then your branches would be tracked to push/merge with the same corresponding name. Depending on what name you've configured your upstream repo to be (most times its origin) you can see how these branches are tracked by: git remote show origin. Guess it depends on how you use and configure git. I just use Git Extensions UI which takes the guess work out of all of this :)

Thanks.  I'm still a newbie with git and have a lot to learn.

My local branch is  'Documentation_work' and when I had it checked out I tried to run the same command I do for my master which is
git remote add upstream  https://github.com/apache/lucenenet.git
It came back with this error "error: remote upstream already exists."  So I guess I'm confused, can you shed any light on why this error?

Remotes are local references to remote repositories. They exist in the local configuration of your copy of the repository.
You can view a list of all of the remotes by using the command
git remote

Or, even better, you can see the URLs of the remotes
git remote -v

Do note it is possible to set up a remote to pull from one repo and push to another (hence the 2 entries for each remote name).
But the bottom line is, if a remote already exists in your local repository that maps it to a remote one, there is no need to add it again. From that point, you can use its name.

Do note using remotes is only a shortcut to make commands simpler - you could just as well use URLs to specify which repository to push to, fetch from, or pull from.
Also, a remote exists at the top level of the repository, so you can use it for any number of branches.

That all makes good sense.  So I wonder if the reason the powershell script is complaining is because I named mine upstream rather then origin.  I wonder that because the powershell script error said

can't find remote branch in this repo

and Shannon @Shazwazza  mentioned that the the remote is often named origin.
Or maybe the error is saying that I literally need a branch on the remote that is the same as my branch name.  But if that's the case I don't see how I can solve error since there is no way for me to create a branch on the remote repository at https://github.com/apache/lucenenet.git

[21-03-26 07:38:02.951]Info:BuildCommand.BuildCore.Build Document.Load.ConceptualDocumentProcessorFor git repo <C:/Users/Ron/source/repos/lucenenet>, can't find remote branch in this repo and fallback to use local branch [Documentation_work]: fatal: no upstream configured for branch 'Documentation_work'

I wonder if the reason the powershell script is complaining is because I named mine upstream rather then origin. I wonder that because the powershell script error said

can't find remote branch in this repo

Okay, it sounds like you haven't configured a remote for your specific branch. Assuming you have Documentation_work checked out and you want to use the remote named upstream:
git branch --set-upstream-to=upstream/foo

Reference: https://stackoverflow.com/a/2286030
Note it is possible to have a different remote and local name, but I have never done it because of the confusion it would cause me.
I have no idea why the script would trigger that message, though. It doesn't seem to be doing anything with Git at all.

It seems the original question has been answered and the original task has been completed in #454, so I am closing this.