Discussion: Hierarchical approach as currently implemented poses problems for sub-types

The AlkEthOH test set reveals some "sampling problems" which we believe need to be resolved by some design modifications.
Specifically, the hydrogen element contains multiple sub-types in AlkEthOH - most obviously, HC and HO which are hydrogens with attached carbons and hydrogens with attached oxygens. However, there are additional sub-types H1, H2, and H3 that are sub-types of HC relating to what other type of atom is connected. The tree looks something like this:
         hydrogen
        /         \
      HC           HO
       |
      H1
       |
      H2
       |
      H3

Currently, as we understand it, the scoring approach dictates smarty can only discover the H1, H2, and H3 types if it first discovers HC (and we haven't seen them discovered yet regardless). This is because if smarty matches HO first, then, since all remaining types are covered by HC, discovering HC leads to no improvements to the score AND, worse, would lead to no atoms being typed by the generic hydrogen (so this automatically gets rejected since each pattern is required to match at least some types).
It seems clear that in the limit of infinite sampling at finite temperature, one ought to be able to explore both branches and find that only the "discover HC first" branch yields to achieving the best possible score. However, we think that:
a) this is an inefficient way of exploring
b) we can easily make the "greedy" algorithm work better, and
c) this will likely result in other problems in more complicated trees - i.e. if both branches extended downwards several levels, we would only be able to explore one or the other but not both (since the generic type is required to match at least some atoms).
We believe that at least two changes should be made to resolve this; I'll lay them out briefly here and then can follow up by creating issues on the individual items once we agree on the concept:

Base atom types should be allowed to match no atoms and there should be no benefit to the score for them to match atoms; these base atom types should get special treatment.

Indeed, Christopher would argue that base atom types OUGHT to end up matching no atoms HERE; these out to be basically a catch-all for anything we can't recognize or don't know anything about yet
This will help because then developing new specialized types HC and HO in the example above will result in score increases

Postpone discussion of this point following, need to clarify something with Bayly.
2) "Reward exploration of trees which contain more derivative atom types" (more details on scoring function proposal in a subsequent issue): There should be a score reward for creating a new sub-type which encompasses several derivatives of an existing type, i.e. we should prefer creation of the HC sub-type over the HO sub-type somewhat because HC removes several atom types at once from the generic hydrogen and thus is a more promising avenue for discovery of further sub-types.

Can you elaborate on the definition of H1, H2, and H3 types?

Ideally with SMARTS definitions?

@cbayly13 @bannanc - can you provide examples?

H1 corresponds to a hydrogen attached to a carbon attached to 1 electron withdrawing group (it's radius is smaller due to the electron withdrawing group). H2 and H3 are similar, but it is a carbon with 2 or 3 electron withdrawing groups respectively.
@cbayly13 made SMARTS for all of the parameter types in the AlkEthOH set, here are the strings for the H 1,2,3 hydrogens.
H1: [$([#1]-C-[#7,#8,F,#16,Cl,Br])]
H2: [$([#1]-C(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]
H3: [$([#1]-C(-[#7,#8,F,#16,Cl,Br])(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]

To clarify - we think there are really two problems with respect to these SPECIFIC types, one being that we can't explore the HC branch of the tree at all if we go to HO first, and secondarily we don't think we're creating the right types of descriptors yet (which @cbayly13 is working on -- see #12 )

Ah!  OK.
It's important to note that there isn't a single unique hierarchical typing ruleset that will give these atom types. This could be represented many different ways.
A big question is whether, after defining
[#1]-C         HC
[#1]-O         HO
[$([#1]-C-[#7,#8,F,#16,Cl,Br])]                                                                            H1
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]                                               H2
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]               H3

the parent type (hydrogen) still types any atoms.  If it doesn't, the tree could be represented in many ways:
[#1]             HC
[#1]-O         HO
[$([#1]-C                                                                            H1
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]                                               H2
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]               H3

or
[#1]             HO
[#1]-C         HC
[$([#1]-C-[#7,#8,F,#16,Cl,Br])]                                                                            H1
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]                                               H2
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]               H3

or
[#1]             H1
[#1]-C         HC
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]                                               H2
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]               H3
[#1]-O         HO

or
[#1]             H3
[#1]-C         HC
[$([#1]-C-[#7,#8,F,#16,Cl,Br])]                                                                            H1
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]                                               H2
[#1]-O         HO

or many, many other schemes that correctly type all the atoms in a last-one-wins manner.
In light of this, I don't think the specific concern that you can only reach the desired typing along a single route is correct, but you may be right that we might be overly constraining exploration by requiring both child and parent to match some atom types.
If we relax this requirement, however, how do we prevent ridiculous elaboration of atom types that do not match atoms?  Should we allow parents to not match types but require children do?  That might provide more "evolutionary fodder" for child atom types while not increasing model complexity.

If we relax this requirement, however, how do we prevent ridiculous elaboration of atom types that do not match atoms? Should we allow parents to not match types but require children do? That might provide more "evolutionary fodder" for child atom types while not increasing model complexity.

Yes, this is exactly what we want.

(I'm trying to translate two hours of whiteboard scribblings amongst four or five of us into GitHub issues and it doesn't always go well.)

That should be easy. Do you want me to implement that?  (Will take two minutes.)

Here is my output for the rejection, this can be helpful to understand why it has been rejected.
The AtomTyper classifies all the H bound to Carbon as hydrogen carbon adjacent, so hydrogen base type (hydrogen) will not match with anything.
It's printing the first molecule from AlkEthOH, just to have an example.
Attempting to create new subtype: '[#1]' (hydrogen) + '$(*~[#6])' (carbon-adjacent) -> '[#1&$(*~[#6])]' (hydrogen carbon-adjacent)
Computing type statistics
carbon
carbon
carbon
oxygen
oxygen total-h-count-1
oxygen total-h-count-1
hydrogen carbon-adjacent
hydrogen carbon-adjacent
hydrogen carbon-adjacent
hydrogen carbon-adjacent
hydrogen oxygen-adjacent
hydrogen oxygen-adjacent
Parent type '[#1]' (hydrogen) now unused in dataset; rejecting.
Rejected.
Computing type statistics
carbon
carbon
carbon
oxygen
oxygen total-h-count-1
oxygen total-h-count-1
hydrogen
hydrogen
hydrogen
hydrogen
hydrogen oxygen-adjacent
hydrogen oxygen-adjacent

That should be easy. Do you want me to implement that? (Will take two minutes.)

@jchodera - if you want to do that, we'd be delighted. Though note this is something we can do, so we'd rather have the SMIRFF XML if given the choice. ;)

How about you take a stab at it and ask if you run into trouble!

How about you take a stab at it and ask if you run into trouble!

Perfect.

Just to clarify a question I left unanswered above:

A big question is whether, after defining

[#1]-C         HC
[#1]-O         HO
[$([#1]-C-[#7,#8,F,#16,Cl,Br])]                                                   H1
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]                             H2
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]       H3
>the parent type (hydrogen) still types any atoms. If it doesn't, the tree could be represented in many ways

The answer is "no". The parent type hydrogen does NOT still match anything in that case, so the tree as I drew it is correct.

I'll see if I can resolve this issue myself.


Restrict Provisioning Developer Environment #1693

[Enter feedback here]
Kindly review the steps to Restrict Provisioning Developer Environment #1693
https://github.com/MicrosoftDocs/powerapps-docs/issues/1693

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

ID: 4deb237b-2aaa-88d5-ae53-dc8dff4c1410
Version Independent ID: a440d8b3-aa78-7546-8785-5b8cd4c78bf7
Content: Environments overview - Power Platform
Content Source: power-platform/admin/environments-overview.md
Service: power-platform
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

@DavidPaf
Handling via internal mail.


Apply Provisioning Template failing at copy Lists

Which PnP repository to report the issue?

PnP Sites Core - https://github.com/OfficeDev/PnP-Sites-Core

Category
[x] Bug
[ ] Enhancement
Environment
[x] Office 365 / SharePoint Online
[ ] SharePoint 2016
[ ] SharePoint 2013
Expected or Desired Behavior
Applying a Template to a new Site Collection works.
Observed Behavior
An Error is throw in version 2.7.1609.3:
Information: 0 : 2016-09-20 12:53:55.6156      [List instances]        [9]     [Debug] Creating list {res:List_Dokumente_Title} failed: Passing in an empty or null value of parameter 'strTitle' :    at Microsoft.SharePoint.Client.ClientRequest.ProcessResponseStream(Stream responseStream)
at Microsoft.SharePoint.Client.ClientRequest.ProcessResponse()
at Microsoft.SharePoint.Client.ClientRequest.ExecuteQueryToServer(ChunkStringBuilder sb)
at Microsoft.SharePoint.Client.ClientRequest.ExecuteQuery()
at Microsoft.SharePoint.Client.ClientRuntimeContext.ExecuteQuery()
at Microsoft.SharePoint.Client.ClientContext.ExecuteQuery()
at Microsoft.SharePoint.Client.ClientContextExtensions.ExecuteQueryImplementation(ClientRuntimeContext clientContext, Int32 retryCount, Int32 delay)
at Microsoft.SharePoint.Client.ClientContextExtensions.ExecuteQueryRetry(ClientRuntimeContext clientContext, Int32 retryCount, Int32 delay)
at OfficeDevPnP.Core.Framework.Provisioning.ObjectHandlers.ObjectListInstance.CreateList(Web web, ListInstance list, TokenParser parser, PnPMonitoredScope scope)
at OfficeDevPnP.Core.Framework.Provisioning.ObjectHandlers.ObjectListInstance.ProvisionObjects(Web web, ProvisioningTemplate template, TokenParser parser, ProvisioningTemplateApplyingInformation ap
Steps to Reproduce
Getting Template from a completely new Site Collection and applying it to a new one with ProvisioningTemplateCreationInformation and ProvisioningTemplateApplyingInformation,  web.ApplyProvisioningTemplate using all Features. The Template SC is a new Team Site with just a few additional Site Groups. It fails at copying the default Documents Library.
PnPProvisioningDemo.zip

This issue was moved to OfficeDev/PnP-Sites-Core#766

This issue was moved to OfficeDev/PnP-Sites-Core#767


Video-recorder container crashes almost right away

Dear developers,
We run into a problem with trying to record a video.
Our setup:

Windows VM with Docker Desktop
We’ve setup Selenoid with the Configuration tool: .\cm selenoid start –port 4445

Test scripts are running fine without video recording.
When we enable video, we can see that a video-recorder container is started, but crashes almost immediately. Test script finishes fine.

This is the inspect of the Selenoid container:
==============================================
Environment
OVERRIDE_VIDEO_OUTPUT_DIR
/c/Users/vkosminskyi/.aerokube/selenoid/video
DOCKER_API_VERSION
1.41
PATH
/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
Mounts
/etc/selenoid
/c/Users/vkosminskyi/.aerokube/selenoid
/opt/selenoid/video
/c/Users/vkosminskyi/.aerokube/selenoid/video
/opt/selenoid/logs
/c/Users/vkosminskyi/.aerokube/selenoid/logs
/var/run/docker.sock
//var/run/docker.sock
Port
4444/tcp
<IP_ADDRESS>:4445
We tried writing a file from within the container itself to /opt/selenoid/video and it does appear in the Windows folder, so writing permissions seem fine.
Any idea what the problem can be?

@evertsc try to get video-recorder container logs while it's running (it is alive for a few seconds).

We've already tried that a few times, but we were too late. Will try again.


Code for loading Spatial Attention weights

In the paper, you mentioned that "We randomly initialize weights relevant to temporal attention while spatial attention weights are initialized using a ViT model trained in a self-supervised manner over the ImageNet-1k dataset". May I ask if you have open sourced this code in the repo? If not, when will you release it?

This is already released; the code should load these by default in this manner.

Thank you for your quick response. Can you point me to where does the loading happen? I may have missed it, but I failed to find where that happens. Thanks again!

https://github.com/kahnchana/svt/blob/master/models/timesformer.py#L28
https://github.com/kahnchana/svt/blob/master/models/timesformer.py#L604

Hi! First of all, thank you for providing the code for this fantastic paper.
I've been trying to run some experiments using the ImageNet pre-trained weights from DINO, to compare it to training from scratch on video data. I am not completely sure that these are loaded by default as it is now. The default values on default_cfgs (timesformer.py#L28) are only used in timesformer.py#L391 within class vit_base_patch16_224 and in timesformer.py#L416 within class TimeSformer (but I haven't been able to find anywhere in the code where these classes are used), as well as within functions get_vit_base_patch16_224 and get_aux_token_vit (both in timesformer.py #L592 and #L612 respectively).
The line #L604 you mention (within get_aux_token_vit) does indeed pre-load the weights, but as far as I understand it this function is only called if MODEL.TWO_TOKEN is True (see train_ssl.py#L204). As MODEL.TWO_TOKEN is set to False by default in scripts/train.sh, get_vit_base_patch16_224 (#L592) is called instead, which only pre-loads if cfg.TIMESFORMER.PRETRAINED_MODEL evaluates to True, which it doesn't, as it defaults to an empty string (see utils/defaults.py#L259).
I think that I can get to modify the code so it actually pre-loads by modifying get_vit_base_patch16_224 to pre-load weights similarly to get_aux_token_vit, but wanted to make sure I am not mistaken. I see that the only difference between these two functions (aside from the weight loading) is one extra token... So what does TWO_TOKEN stand for? What does it change aside from this? I have not been able to find any comment documenting this in the code.
Thanks a lot in advance!


Check boxes on template are doing something weird

Lost our blue check somewhere along the line.. We could go for the same highlight color we're going to use in versions and device preview, or we could bring back the check box if that's easier. Regardless something funky going on right now

This probably just a styling issue – something about the panel styling not jiving with the dark background. Should be an easy fix with some styles with the forms.less


Automatic definition of resource tables in thrift-server

We should add an option to the controller app to create SQL tables at the end of each pipeline run. For example, we can pass the port number of the thriftserver (e.g., in compose-controller-spark-sql.yaml) to the controller app and it issues the SQL create table queries through JDBC.

User story:
As a user I can set a configuration within the pipelines so that the SQL tables for each of the Resource types is created
Steps:

Define configuration setting in the pipeline config
If set, then when the pipelines completes, execute SQL queries for creating the Tables from the parquet files for each Resource type - e.g. CREATE TABLE IF NOT EXISTS default.Patient USING PARQUET LOCATION '/dwh//Patient';

Test criteria:
User should be able to connect via jdbc and query table e.g. SELECT COUNT(*) Patients from default.Patient

More clarification: We should probably create dated tables, i.e., instead of just default.Patient it should be default.Patient-TIMESTAMP (or something like that) where TIMESTAMP matched the timestamp of the DWH directory.

I believe this is fixed with #485.


Connection pooling support

Currently, every request results in a new TCP connection to the origin, which is very expensive due to the multiple round trips required to establish a connection to the origin.
Normally, a crate like this would store the hyper client used by a given bucket within the bucket object, allowing for a connection pool to be established by hyper to the origin. I assume that surf works similarly. For origins which support h2, this would lead to only one outbound TCP connection being needed for communicating with the origin, keeping latencies minimal.
Depending on the details of the operation and the properties of the network path between the client and the server, this may lead to substantial reductions in operation latency.
This is currently one my major blockers for using this crate, as requests I make via this crate are very slow due to the unnecessary round trips.

Theoretically, it would actually be best to just allow the user to pass a tower::Service<Request> to the bucket in the constructor. This would allow a user to submit any arbitrary HTTP client to the bucket and be able to directly tune all of its parameters (e.g h2 params).

Why not just using a reqwest::Client per Bucket? If the client needs some specific HTTP/2 parameters (which I don't think) then add it to the bucket configuration. I could write a PR if moving to reqwest is accepted. I don't see why a library like this needs hyper.

Why not just using a reqwest::Client per Bucket? If the client needs some specific HTTP/2 parameters (which I don't think) then add it to the bucket configuration. I could write a PR if moving to reqwest is accepted. I don't see why a library like this needs hyper.

This feels like the easiest approach, I just brought up the other approach because I saw that this crate used to use reqwest but dropped it because it was felt to be too heavyweight.

IMO there's also a reasonable question about whether this crate needs to support anything other than hyper. I can see a potential argument to support the sync clients, but the async-std ecosystem is basically dead at this point and it may not be worth the added complexity and maintenance burden to continue supporting it.

Will be easy to add such thing into https://github.com/ZitaneLabs/rust-s3-async

@stalkerg are you affiliated with that fork, or the fork it forks? It would be nice for the one from aalekhpatel07 to not be classed as a fork on GitHub, so that issues can be filed and discussions can happen.

@Noah-Kennedy no :( but you can make a new fork and enable the issue tracker. It's an MIT license.

This patch works for me:
diff --git a/s3/src/bucket.rs b/s3/src/bucket.rs
index eb2c1a1..61dbf6f 100644
--- a/s3/src/bucket.rs
+++ b/s3/src/bucket.rs
@@ -47,6 +47,10 @@ use crate::utils::{error_from_response_data, PutStreamResponse};
 use http::header::HeaderName;
 use http::HeaderMap;

+use hyper::Client;
+use hyper::client::HttpConnector;
+use hyper_tls::HttpsConnector;
+
 pub const CHUNK_SIZE: usize = 8_388_608; // 8 Mebibytes, min is 5 (5_242_880);

 const DEFAULT_REQUEST_TIMEOUT: Option<Duration> = Some(Duration::from_secs(60));
@@ -89,6 +93,7 @@ pub struct Bucket {
     pub extra_headers: HeaderMap,
     pub extra_query: Query,
     pub request_timeout: Option<Duration>,
+    pub client: Client<HttpsConnector<HttpConnector>>,
     path_style: bool,
     listobjects_v2: bool,
 }
@@ -433,6 +438,7 @@ impl Bucket {
             extra_headers: HeaderMap::new(),
             extra_query: HashMap::new(),
             request_timeout: DEFAULT_REQUEST_TIMEOUT,
+            client: Self::make_client(DEFAULT_REQUEST_TIMEOUT),
             path_style: false,
             listobjects_v2: true,
         })
@@ -457,6 +463,7 @@ impl Bucket {
             extra_headers: HeaderMap::new(),
             extra_query: HashMap::new(),
             request_timeout: DEFAULT_REQUEST_TIMEOUT,
+            client: Self::make_client(DEFAULT_REQUEST_TIMEOUT),
             path_style: false,
             listobjects_v2: true,
         })
@@ -470,6 +477,7 @@ impl Bucket {
             extra_headers: self.extra_headers.clone(),
             extra_query: self.extra_query.clone(),
             request_timeout: self.request_timeout,
+            client: self.client.clone(),
             path_style: true,
             listobjects_v2: self.listobjects_v2,
         }
@@ -483,6 +491,7 @@ impl Bucket {
             extra_headers,
             extra_query: self.extra_query.clone(),
             request_timeout: self.request_timeout,
+            client: self.client.clone(),
             path_style: self.path_style,
             listobjects_v2: self.listobjects_v2,
         }
@@ -496,6 +505,7 @@ impl Bucket {
             extra_headers: self.extra_headers.clone(),
             extra_query,
             request_timeout: self.request_timeout,
+            client: self.client.clone(),
             path_style: self.path_style,
             listobjects_v2: self.listobjects_v2,
         }
@@ -509,6 +519,7 @@ impl Bucket {
             extra_headers: self.extra_headers.clone(),
             extra_query: self.extra_query.clone(),
             request_timeout: Some(request_timeout),
+            client: Self::make_client(Some(request_timeout)),
             path_style: self.path_style,
             listobjects_v2: self.listobjects_v2,
         }
@@ -522,6 +533,7 @@ impl Bucket {
             extra_headers: self.extra_headers.clone(),
             extra_query: self.extra_query.clone(),
             request_timeout: self.request_timeout,
+            client: self.client.clone(),
             path_style: self.path_style,
             listobjects_v2: false,
         }
@@ -2066,6 +2078,7 @@ impl Bucket {
     /// async code may instead await with a timeout.
     pub fn set_request_timeout(&mut self, timeout: Option<Duration>) {
         self.request_timeout = timeout;
+        self.client = Self::make_client(timeout);
     }

     /// Configure bucket to use the older ListObjects API
@@ -2231,6 +2244,13 @@ impl Bucket {
     pub fn request_timeout(&self) -> Option<Duration> {
         self.request_timeout
     }
+
+    fn make_client(request_timeout: Option<Duration>) -> Client<HttpsConnector<HttpConnector>> {
+        let mut http_connector = HttpConnector::new();
+        http_connector.set_connect_timeout(request_timeout);
+        let https_connector = HttpsConnector::new();
+        Client::builder().build::<_, hyper::Body>(https_connector)
+    }
 }

 #[cfg(test)]
diff --git a/s3/src/request/tokio_backend.rs b/s3/src/request/tokio_backend.rs
index 75cb211..f13f2dd 100644
--- a/s3/src/request/tokio_backend.rs
+++ b/s3/src/request/tokio_backend.rs
@@ -3,9 +3,7 @@ extern crate md5;

 use bytes::Bytes;
 use futures::TryStreamExt;
-use hyper::client::HttpConnector;
-use hyper::{Body, Client};
-use hyper_tls::HttpsConnector;
+use hyper::Body;
 use maybe_async::maybe_async;
 use std::collections::HashMap;
 use time::OffsetDateTime;
@@ -39,39 +37,6 @@ impl<'a> Request for HyperRequest<'a> {
             Err(e) => return Err(e),
         };

-        #[cfg(any(feature = "use-tokio-native-tls", feature = "tokio-rustls-tls"))]
-        let mut tls_connector_builder = native_tls::TlsConnector::builder();
-
-        #[cfg(not(any(feature = "use-tokio-native-tls", feature = "tokio-rustls-tls")))]
-        let tls_connector_builder = native_tls::TlsConnector::builder();
-
-        if cfg!(feature = "no-verify-ssl") {
-            cfg_if::cfg_if! {
-                if #[cfg(feature = "use-tokio-native-tls")]
-                {
-                    tls_connector_builder.danger_accept_invalid_hostnames(true);
-                }
-
-            }
-
-            cfg_if::cfg_if! {
-                if #[cfg(any(feature = "use-tokio-native-tls", feature = "tokio-rustls-tls"))]
-                {
-                    tls_connector_builder.danger_accept_invalid_certs(true);
-                }
-
-            }
-        }
-        let tls_connector = tokio_native_tls::TlsConnector::from(tls_connector_builder.build()?);
-
-        let mut http_connector = HttpConnector::new();
-        http_connector.set_connect_timeout(self.bucket.request_timeout);
-        // let https_connector = HttpsConnector::from((http_connector, tls_connector));
-
-        let https_connector = HttpsConnector::new();
-
-        let client = Client::builder().build::<_, hyper::Body>(https_connector);
-
         let method = match self.command.http_verb() {
             HttpMethod::Delete => http::Method::DELETE,
             HttpMethod::Get => http::Method::GET,
@@ -91,7 +56,7 @@ impl<'a> Request for HyperRequest<'a> {

             request.body(Body::from(self.request_body()))?
         };
-        let response = client.request(request).await?;
+        let response = self.bucket.client.request(request).await?;

         if cfg!(feature = "fail-on-err") && !response.status().is_success() {
             let status = response.status().as_u16();

This moves creation of the Hyper client into Bucket constructor. Previously, 100 tasks running in parallel would blow up CPU usage on my laptop, now 1000 tasks in parallel work fast and good.
I would also recommend to apply patch from #343 to eliminate random bucket errors.

implemented with https://github.com/durch/rust-s3/commit/7d7c57b5cc967299ebf338337c7fb3b68e57c03c for with-tokio feature


Appstudio update test-component-pac-wjkc

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-wjkc-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
17 seconds
clone-repository

✅ Succeeded
31 seconds
sast-snyk-check

✅ Succeeded
1 minute
build-container

✅ Succeeded
13 seconds
inspect-image

✅ Succeeded
9 seconds
deprecated-base-image-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
4 minutes
clair-scan

✅ Succeeded
25 seconds
sbom-json-check

✅ Succeeded
11 seconds
show-sbom

✅ Succeeded
9 seconds
show-summary

Pipelines as Code CI/test-component-pac-wjkc-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
17 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
31 seconds
sast-snyk-check

✅ Succeeded
13 seconds
inspect-image

✅ Succeeded
9 seconds
deprecated-base-image-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
4 minutes
clair-scan

✅ Succeeded
25 seconds
sbom-json-check

✅ Succeeded
11 seconds
show-sbom

✅ Succeeded
9 seconds
show-summary

Pipelines as Code CI/test-component-pac-wjkc-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
23 seconds
clone-repository

✅ Succeeded
9 seconds
sast-snyk-check

✅ Succeeded
24 seconds
build-container

✅ Succeeded
11 seconds
inspect-image

✅ Succeeded
9 seconds
deprecated-base-image-check

✅ Succeeded
55 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
8 seconds
show-summary


Cloud Firestore samples use older library version; SnapshotAsync is now GetSnapshoAsync

Include="Google.Cloud.Firestore" Version="1.0.0-beta02"
Latest version is:
Include="Google.Cloud.Firestore" Version="1.0.0-beta11"
However, updating to the newer version causes samples to fail.
For example, in firestore/api/Quickstart:
dotnet run retrieve-all-documents
Program.cs(82,53): error CS1061: 'CollectionReference' does not contain a definition for 'SnapshotAsync' and no accessible extension method 'SnapshotAsync' accepting a first argument of type 'CollectionReference' could be found
This error is fixed by changing SnapshotAsync to GetSnapshotAsync.

Looks like this has already been fixed.


Add HLSCustomOp for AccPool

Add a HLSCustomOp type for the finn-hlslib function AccPool_Batch.

Fixed by PR #105


added the ability to trigger a workflow manually

Hi Keisuke,
This only changes the github-action so I'll merge myself.
Thanks,
Scott


neod.exe: Right-click, Select All doesn't work in the code window

Nothing is highlighted in the code window.

No longer applies in new version


z=1.8 and 5.5 redshift pileups on tile 7009; likely bad sky residuals

This tile got a ton of EFFTIME in pretty poor conditions.  Redshift distribution shows clear pileups at z=1.8 and 5.5, as well as SKYCHI2PDF warnings.
https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/7009/20211215/tile-qa-7009-thru20211215.png
I suspect that the sky variation is just too large and things are being misfit as high redshifts.  We either need to improve the sky modeling or mark the component exposures as bad so that the tile may be reobserved.
Possibly the dark time equivalent of https://github.com/desihub/desisurveyops/issues/4 .

TILEID 2295 is another example.

Related sky subtraction issue looks to be able to lead to pileups at z=0.5, putting the 4000 A break at the B/R camera boundary.  I've marked 5805 as unsure for now, though this doesn't look like it causes any LyA issues.  But maybe we could mark it as bad.

Addressed as part of sky subtraction updates.


Document core.allow_broker=true and the AAD Broker plugin is not supported for non-commercial clouds

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command
az cloud set --name AzureUSGovernment
az config set core.allow_broker=true
az login
Describe the bug
If the cloud is set to any other cloud besides the default commercial cloud the AAD Broker plugin will fail to authenticate with the message below during login. The error is not intuitive indicating that the issue is usage of the AAD broker plugin.
Request Id: xyz
Correlation Id: xyz
Timestamp: 2023-03-28T15:14:12Z
Message: AADSTS50011: The redirect URI 'ms-appx-web://Microsoft.AAD.BrokerPlugin/04b07795-8ddb-461a-bbee-02f9e1bf7b46' specified in the request does not match the redirect URIs configured for the application '04b07795-8ddb-461a-bbee-02f9e1bf7b46'. Make sure the redirect URI sent in the request matches one added to your application in the Azure portal. Navigate to https://aka.ms/redirectUriMismatchError to learn more about how to fix this.

To Reproduce
Use the following commands to try and login from a GCC High tenant.
az cloud set --name AzureUSGovernment
az config set core.allow_broker=true
az login
To mitigate, you need to disable the Broker using az config set core.allow_broker=false

@jiasli for awareness

any update about this issue?
I use
"az config set core.allow_broker=false"
as a workaround


Doc: Extra top padding after having clicked on anchor links

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
An extra padding top is created for each section after having clicked on the anchor links in the documentation.
Basic rendering

Rendering after the manipulation (with the extra top padding)

I haven't looked at the issue precisely but this is probably related from near and far to:
.bd-content>:target {
 padding-top:5rem;
 margin-top:-5rem
}

It has changed in https://github.com/twbs/bootstrap/commit/195440f2fb1e94c014a9cf08f3eae40f3d224620.
Nothing has been tested yet on my side but there was another system in place before apparently based on:
// Offset for the sticky header
@include media-breakpoint-up(md) {
  :root {
    scroll-padding-top: 4rem;

Note: It happens with all breakpoints.
Reduced test cases
1st test case
After having clicked on "Importing", the padding-top stays on top until the page is refreshed.

2nd test case
It is maybe more obvious in this video. We can directly see the extra padding on top of "CSPs and embedded SVGs" that's added even if the content is not near the header.

What operating system(s) are you seeing the problem on?
Linux
What browser(s) are you seeing the problem on?
Firefox
What version of Bootstrap are you using?
v5.2.0 (current main branch)

I think I am have found a solution to this.
Please assign it to me.

Hey @Catalyst497 . We don't affect issues but you can create a PR to provide a solution :)


Issues with JSON fields, z.any() and relations

When a model has a JSON field, its relationship declaration will throw a type error.
// Example schema

model User {
  id    Int    @id @default(autoincrement())
  meta  Json
  posts Post[]
}

model Post {
  id       Int  @id @default(autoincrement())
  authorId Int
  author   User @relation(fields: [authorId], references: [id])
}

The issue comes from the Json type resolving to z.any(), and the extended schema not mapping correctly to the extended interface due to some part being optional and some other being required.
Rather than the builtin any, the package could include some runtime helpers to resolve JSON types. See Zod's documentation for implementation suggestions, though this may cause some issues when trying to resolve the schema types using z.typeOf<typeof modelSchema>.

Note: it seems that it's not related to JSON, I also have this issue with a model with an optional Date.

Hey @CarterGrimmeisen, the fix you added in b0b4152 (injecting a recursive JSON schema) seems to work, do you have an ETA on a release ?


Design change: require filename (or -)

The tool currently lets you do this:
cat data.csv | openai-to-sqlite embeddings embeddings.db

But this is a bit weird, it means if you run openai-to-sqlite embeddings embeddings.db it will hang waiting for input.
If you want to specify a file you have to use the -i option:
openai-to-sqlite embeddings embeddings.db -i data.csv

I'm going to change it to require the filename as a positional argument, and support - to mean standard input.

Actually no it has to still be optional because of --sql - you should get an error if you omit it without including --sql though.


create event of inventory that sync with Devices

add unit test as well
realted to story THEEDGE-2727
Signed-off-by: mgold1234<EMAIL_ADDRESS>Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
What is it?

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] I run make pre-commit to check fmt/vet/lint/test-no-fdo

Codecov Report
Base: 38.12% // Head: 38.02% // Decreases project coverage by -0.09% :warning:

Coverage data is based on head (40d6735) compared to base (4fa2039).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1691      +/-   ##
==========================================
- Coverage   38.12%   38.02%   -0.10%
==========================================
  Files          68       69       +1
  Lines        9509     9532      +23
==========================================
  Hits         3625     3625
- Misses       5446     5469      +23
  Partials      438      438

Impacted Files
Coverage Δ

pkg/services/device/event_inventory_sync.go
0.00% <0.00%> (ø)

pkg/services/devices.go
50.43% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

/retest

Please take a look at this fix https://github.com/RedHatInsights/edge-api/pull/1709  to understand the problem.
as in this PR mockDeviceService.EXPECT().SyncInventoryWithDevices(event.RedHatOrgID) has no function , it can be added or removed and the test will pass


Use exit codes of 2 and 3

so that we can distinguish between the two different ways that
pycobertura can explicitly fail and even more importantly distinguish
these types of failures from exit code 1, which is what seems to be
returned if an exception occurs.
Currently, any kind of failure returns an exit code 1 so the caller
cannot distinguish between various cases.
Cc: @aconrad, @sudarkoff

Thanks @msabramo!
So we'd have:

0: all is good.
1: some exception occurred (likely a bug)
2: coverage worsened (implies 3)
3: not all changes are covered

Looks great. Can you update the CHANGES file as well as the tests?

Will do...

@aconrad: Updated tests and CHANGES.md and also did some refactoring that hopefully you like.

Thanks! Mind also testing the class ExitCodes since you've started using it for tests? Right now if I were to change the exit codes it wouldn't fail.

Thanks! Mind also testing the class ExitCodes since you've started using it for tests? Right now if I were to change the exit codes it wouldn't fail.

Done.

I wonder in retrospect if I should've not used exit code 1 -- that way exit code 1 would signify that we're running an older version of pycobertura and hence, don't know what it means.
So we'd have:

0: all is good.
1: old version of pycobertura before it used different exit codes; we don't know why it failed unless we inspect output.
2: some exception occurred (likely a bug)
3: coverage worsened (implies 3)
4: not all changes are covered

Released v0.8.0.
I just saw your note after releasing. But since we're not 1.0 just yet, I don't really want to worry about old versions. I updates the CHANGES file to mention that it's a backwards incompatible change for those who care.


manage_default_security_group with empty eg/ingress doesn't remove rules

Description
If you have existing VPCs with default SGs and want to bring them under management in order to remove the default SG's rules, you cannot. When the VPC is created, its default SG has default ingress & egress rules. I would expect that the following would remove them (and maintain that state).
  manage_default_security_group  = true
  default_security_group_egress  = []
  default_security_group_ingress = []
  default_security_group_name    = "DEFAULT: Do Not Use"

Instead, existing rules remain. I also tested creating a new VPC with the default SG managed by the module as shown above, and then went and added an ingress rule manually. On the next TF apply, it did not remove the rule, as would be expected.
We need to manage, and maintain, the default SG to be empty, and I can't seem to find a way to accomplish that with the way it currently works.
Versions

Terraform: v1.0.6 on darwin_arm64

Provider(s):

provider registry.terraform.io/fluxcd/flux v0.8.0
provider registry.terraform.io/gavinbunney/kubectl v1.13.1
provider registry.terraform.io/hashicorp/aws v3.74.3
provider registry.terraform.io/hashicorp/cloudinit v2.2.0
provider registry.terraform.io/hashicorp/kubernetes v2.8.0
provider registry.terraform.io/hashicorp/local v2.1.0
provider registry.terraform.io/hashicorp/null v3.1.0
provider registry.terraform.io/hashicorp/random v3.1.0
provider registry.terraform.io/hashicorp/tls v3.1.0
provider registry.terraform.io/integrations/github v4.9.2
provider registry.terraform.io/terraform-aws-modules/http v2.4.1

Module:

Reproduction
Create a new VPC with this module, with the manage_default options set.
Apply
Manually add a rule to the default SG
Apply
The new rule is not removed.
Code Snippet to Reproduce
module "vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "2.77.0"

  cidr                           = "<IP_ADDRESS>/16"
  default_security_group_egress  = []
  default_security_group_ingress = []
  default_security_group_name    = "DEFAULT: Do Not Use"
  manage_default_security_group  = true
  name                           = "test"
  private_subnets                = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]
  public_subnets                 = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]
}

Expected behavior
When a VPC's default SG comes under "management", it should respect the defined rules, even if the rules are empty: []
Actual behavior
The empty rules defined the default security group are not respected. Manually added rules presist.
Additional context
This is becoming a more and more common request from cloud vulnerability scanners (the actual validity of this can be argued elsewhere), so it's important to be able to maintain a default SG with no ingress or egress rules.
Updating to the latest version: 3.12.0 did not seem to change the behavior.
Also upgraded TF to 1.1.6 with no change.

I think what you are referring to is https://github.com/hashicorp/terraform-provider-aws/issues/3234
rules that are added manually, outside of Terraform's control, are not detected for deletion
There are also some other issues that might be related, but they are constrained to the provider, not this module:

https://github.com/hashicorp/terraform-provider-aws/issues/1555
https://github.com/hashicorp/terraform-provider-aws/issues/20697

@bryantbiggs Is this not this module?
module "vpc" {
  source  = "terraform-aws-modules/vpc/aws"

I'm checking right now to see if I get anything different using:
resource "aws_default_security_group" "default" {
  vpc_id = module.vpc.vpc_id
}

OK, it looks like from doing more reading that the behavior I desire is simply not possible, whether it be with the VPC module or the aws_default_security_group resource. I guess we'll just have to manually manage the rules.

@spanktar AFAIK, using aws_default_security_group works.  If I do:
resource "aws_vpc" "test" {
  cidr_block = "<IP_ADDRESS>/22"
}

resource "aws_default_security_group" "default" {
  vpc_id  = aws_vpc.test.id
  ingress = []
  egress  = []
}

output "test_default_security_group_id" {
  value = aws_default_security_group.default.id
}

And then do:
aws ec2 describe-security-groups --group-ids <output group id>
I get:
{
    "SecurityGroups": [
        {
            "Description": "default VPC security group",
            "GroupName": "default",
            "IpPermissions": [],
            "OwnerId": "057199631211",
            "GroupId": "sg-032183a78fa089a89",
            "IpPermissionsEgress": [],
            "VpcId": "vpc-00eb285da1e9706fb"
        }
    ]
}


Fix Manage Data Usage when no option is selected

In Manage Data Usage, if you select Download EML or Download Data and then click Select without picking an option from the list, ezEML throws an exception.

Closed in release 2024.01.03.


Where is device-init on v1.7.1?

There is no device-init at /usr/local/bin, and device-init.yaml file at /boot has no effect. Hostname is still black-pearl and wifi did not start.
Did cloud-init replace device-init?

Exactly. Cloud-init is more flexible and better known and there are more docs out there.
Have a look at the latest blog post and the /boot/user-data file.

Was this stated some where? It's not clear from the release notes.

Oh you‘re right, we should mention it in https://github.com/hypriot/image-builder-rpi/releases/tag/v1.7.1 intoduced with 1.7.0 and some housekeeping in the download page.
I‘ll reopen this issue for these tasks.

I've added
## Breaking change
- No more device-init, please convert your `/boot/device-init.yaml` to `/boot/user-data`

to the 1.7.0 and 1.7.1 releases.

Thanks.


Feature request: capacity to drop elements from the legend directly

The "proper" way to achieve this is to muck with the factor levels of the variable in question directly, which is at best annoying and time-consuming.
The ability to drop legend elements directly with functionality similar to the current rename.groups implementation in DBBarPlot would be extremely convenient, particularly when subsetting.

dittoBarPlot() is up!  It should fix this issue for you.
Before plotting but after cells.use subsetting, the updated function converts both the x.data and y.data to a character and then (back) to a factor.  Doing so eliminates unused levels, which I believe was the root cause of this issue.


between kiwi and flutter_test version solving failed. issue

i try to use kiwi 2.0.1 in my flutter project. but pubspec.yaml gives error as

Flutter_test is not using the latest meta version. Kiwi is. So you have to add this to specifically define what version should be used
dependency_overrides:
  meta: 1.2.3

thanks a lot  @vanlooverenkoen .
it's working...
thank you


Add dockerfile

I often struggle to get python working correctly on my Mac.
python --version
zsh: command not found: python

python3 --version
/opt/homebrew/bin/python3: can't open file

This PR adds a Dockerfile which should make this code more portable for those with broken pythons (or python not installed at all).
If a user has docker/docker-compose installed they can get your script working with just 1 command
docker compose run labelgenerator

Note: I chose to install ttf-mscorefonts-installer and not mention --roboto based on comments you made in another issue.
If a user wants to use roboto they can modify the entrypoint in docker-compse.yaml
    entrypoint: python3 /resistorLabels/LabelGenerator.py --roboto

Understandable.
I'm not really a python developer, and even though I have years of software experience, I'm still often hung up by pip vs pip3 virtualenv vs pyenv vs virtual-env. If I can't figure it out, then I expect a significant portion of other users to experience the same problems.
Docker seemed like the most ubiquitous abstraction layer.

For reference, if this problem comes up again:
Installing python should be as easy as going to https://www.python.org/downloads/, downloading it and installing it.
pip/pip3 should both work, as long as you are sure that there isn't a lingering python 2 installation on your system which the pip command refers to.
No virtual environments are required, it shouldn't be a problem to install the single dependency of this script globally.
There are a lot of python tutorials, and the first chapter should usually contain everything required to execute this script.

I'll revisit this if a second person runs into the same problems.
An alternative would be to re-write it in a compiled language so that people can download it as an executable. Then indeed a yaml/json based config would be required instead of adjusting the values in the source file.
But that would make it harder for people like you to create pull requests, so I'm unsure about the proper way to go.

https://opensource.com/article/19/5/python-3-default-mac
It sounds to me like pyenv is pretty reliable on MacOS. Your biggest hurdle will be to clean up your broken installation to start fresh.


Docs

Finally, a conceptual overview in the form of text.

Coverage remained the same at 26.33% when pulling b727d900610089d611a23478dff2e2e8fc5e3622 on docs into 429b83c371f2b69d10dbc1c5af59849bc6e22e2e on master.


winget install with --include-unknown

Brief description of your issue
--include-unknown should be a valid command to be used with install.
If already installed it should upgrade.
Steps to reproduce
winget install --ID <*> --include-unknown
Expected behavior
If there is a package which is installed with unknow version upgrade it.
Actual behavior
Does not work
Environment
winget --info
Windows Package Manager v1.5.2201
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.2215
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.20.2201.0

Winget Directories
-------------------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Enabled
LocalArchiveMalwareScanOverride           Disabled

My independent opinion is that the Install and upgrade commands should be idempotent and error on fail, such as if there's no valid action, instead of performing another command on fail. As the latter would be less predictable.
But @denelon would have better advice here.

@stephengillie - This should probably be Issue-Bug.
The install command switches to the upgrade flow if it detects the package is already installed, and when the version is unknown gives the message shown below. But, as @sirinath pointed out, the install command doesn’t accept the --include-unknown argument.
PS D:\Git\winget-pkgs> winget list windirstat
Name             Id                    Version Available
--------------------------------------------------------
WinDirStat 1.1.2 WinDirStat.WinDirStat Unknown 1.1.2
PS D:\Git\winget-pkgs> winget install windirstat
Found an existing package already installed. Trying to upgrade the installed package...
This package's version number cannot be determined. To upgrade it anyway, add the argument --include-unknown to your previous command.

Using --force will cause the command to stay in the install flow and not switch to the upgrade flow, and may work as a workaround for this issue, but doesn’t address the errant behavior. And using --no-upgrade simply causes the install flow to terminate if the package is already installed


set 'proxy' with incorrect parameter make nvm always crash

My Environment

[X] Windows 10

I'm using NVM4W version:
Run nvm version if you don't know.
I have already...

[X] read the README to be aware of npm gotchas & antivirus issues.
[X] reviewed the wiki to make sure my issue hasn't already been resolved.
[X] verified I'm using an account with administrative privileges.
[X] searched the issues (open and closed) to make sure this isn't a duplicate.
[X] made sure this isn't a question about how to use NVM for Windows, since gitter is used for questions and comments.

My issue is related to (check only those which apply):

[X] settings.txt
[X] A standard shell environment (terminal/powershell)
[X] A non-standard shell environment (Cmder, Hyper, Cygwin, git)

Actual Behavior
C:\Users\Himself65>nvm version
1.1.7

C:\Users\Himself65>nvm proxy .

C:\Users\Himself65>nvm
panic: runtime error: slice bounds out of range

goroutine 1 [running]:
main.Setup()
        C:/Users/corey/Documents/workspace/nvm-windows/src/nvm.go:685 +0x8ed
main.main()
        C:/Users/corey/Documents/workspace/nvm-windows/src/nvm.go:60 +0x61

I think you didn't check if it's the valid parameter when setting.

related code:
https://github.com/coreybutler/nvm-windows/blob/88353cfcea140e02e2b13584fd89454adc9f52b3/src/nvm.go#L728-L751

Hi, just passing by and wanted to check if this PR is going to be merged, as as of v1.1.7 the issue is still present

And also, since nvm off is not working now, how do I uninstall it?

Hi, just passing by and wanted to check if this PR is going to be merged, as as of v1.1.7 the issue is still present

I think no, this project has no longer maintained

@randomactions the project is still maintained, but I don't have a Windows machine to test on at the moment.

Hi, is there any workaround for current version to have nvm working again?

The issue here is . is not a valid proxy. A proxy needs to be a URL. If you're receiving this error, manually reset the proxy in the settings.txt file (in the local nvm4w installation root).

Hi, for anyone who's trying to get back nvm working again, removing line with proxy in settings.txt worked for me:
root: C:\Users\USER\AppData\Roaming\nvm
arch: 64
- proxy: -h
originalpath:
originalversion:
node_mirror:
npm_mirror:

nvm version: 1.1.5

I just installed nvm-windows, and my settings.txt looks like this:
root: C:\Users\wagge\AppData\Roaming\nvm
path: C:\Program Files\nodejs

No "proxy" in there, but I get the same error.

Is this projet maintained?

I just installed nvm-windows, and my settings.txt looks like this:
root: C:\Users\wagge\AppData\Roaming\nvm
path: C:\Program Files\nodejs

No "proxy" in there, but I get the same error.

Is this projet maintained?

same issue with me, no proxy in settings

I have the same problem, cannot install nodejs ...

same problem here.

Nothing special in settings.txt

@bozzgo13 try nvm install 16.18.1 (without the version part)

@bozzgo13 try nvm install 16.18.1 (without the version part)

this helped me. Tnx a lot

@bozzgo13 try nvm install 16.18.1 (without the version part)
thaaannnnkkk youuuuu jesus lol

@bozzgo13 try nvm install 16.18.1 (without the version part)

You can also use 'nvm install latest' if you want to install the latest version of Node.
What I don't understand is why the panic appears to be looking for files on the developers OneDrive. Is there a way to edit this?

@bozzgo13 try nvm install 16.18.1 (without the version part)

Oh wow! Thanks

For those of you having trouble, do as follows (WINDOWS 10)

nvm install latest

go to file explorer and browse to %AppData%, make sure you're in the Roaming folder
Find the folder titles v##.#.# (in my case v19.7.0)
back into cmd/powershell/whatever

nvm use 19.7.0

do not include the v

node -v

Verify your node version.

@bozzgo13 try nvm install 16.18.1 (without the version part)

Thank you. Its worked for me

@bozzgo13 try nvm install 16.18.1 (without the version part)

Thank you!!!!

I have the same problem, I did not set proxy.
C:\Users\Liu.D.H>nvm install 20.0.0
panic: runtime error: index out of range [3] with length 3

goroutine 1 [running]:
main.checkVersionExceedsLatest({0x118160c8, 0x6})
        C:/Users/corey/OneDrive/Documents/workspace/libraries/oss/coreybutler/nvm-windows/src/nvm.go:875 +0x220
main.install({0x118160c8, 0x6}, {0x6e9d26, 0x2})
        C:/Users/corey/OneDrive/Documents/workspace/libraries/oss/coreybutler/nvm-windows/src/nvm.go:308 +0x337
main.main()
        C:/Users/corey/OneDrive/Documents/workspace/libraries/oss/coreybutler/nvm-windows/src/nvm.go:87 +0xaea

C:\Users\Liu.D.H>nvm version
1.1.10

C:\Users\Liu.D.H>node -v
v18.14.2

C:\Users\Liu.D.H>
C:\Users\Liu.D.H>where nvm
C:\Users\Liu.D.H\AppData\Roaming\nvm\nvm.exe

C:\Users\Liu.D.H>cat C:\Users\Liu.D.H\AppData\Roaming\nvm\settings.txt
root: C:\Users\Liu.D.H\AppData\Roaming\nvm
path: C:\Program Files\nodejs

C:\Users\Liu.D.H>

After I update nvm to 1.1.11, it seems works again.


Adds missing li, dt, dd, caption elements

About
Adds missing li, dt, dd, caption elements to ensure font overrides in list items, definition list items, and table captions.
Checklist

[ x] Wrote documentation on this page
[x ] Formated code

Impacted Areas in Application
CSS rendering for lists, definition lists, table captions
Screenshots

Hey @philgooch ,
Thank you for the PR, I have this in the new version coming out, their was a few two many conflicts.
Will message when the version is coming out.
Thank you
Robert James


Auto import for types like Ref

Environment

Operating System: Darwin
Node Version:     v14.16.1
Nuxt Version:     3.0.0-27277498.850ef69
Package Manager<EMAIL_ADDRESS>Bundler:          Vite
User Config:      env, alias, srcDir, dev, build, publicRuntimeConfig, buildModules, css, vite
Runtime Modules:  -
Build Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>./app/t5-ui/nuxt

Reproduction

Describe the bug
I want to use Ref to type assertions, But:

Additional context
No response
Logs
No response

Hey, due to the TS limitation being something upstream, is still still relevant, or can it be closed?

@ennioVisco True thing, https://devblogs.microsoft.com/typescript/announcing-typescript-5-0-beta/#support-for-export-type
Let's keep it open then 😊

Upstream label removed - maybe we can tinker with an implementation for TS5 🤔


Error: gyp failed with exit code: 1 at ChildProcess.onCpExit

Description:
I tried to install several packages and I got the same error.  System Windows_NT 10.0.15063
gyp ERR! command "C:\nodejs\node.exe" "C:\USER\AppData\Roaming\npm\node_modules\node-gyp\bin\
node-gyp.js" "configure" "--msvs_version=2015"
gyp ERR! cwd C:\WINDOWS\system32
gyp ERR! node -v v6.9.4
gyp ERR! node-gyp -v v3.6.1
gyp ERR! not ok

gyp info spawn C:\USER.windows-build-tools\python27\pythonw.exe

I'm not in front of a Windows system right now to check but does gyp work with pythonw.exe? If not then the question is whether this is a misconfiguration or fault in the python finding code.

@richardlau well, change to the next addres:  C:\USER\.windows-build-tools\python27\python.exe, but, it still doesn't work. Try setting node-gyp --python /path/to/python2.7, but do not grab this configuration. I have already compiled packages with node-gyp, and they work fine, but now this error appeared with some packages that I need, and I still do not solve!

@LizanLycan can you try removing the spaces around = in your command?
node-gyp configure --msvs_version=2015

If that doesn't work, add --verbose or even --silly to get more output.

@joaocgreis Still does not work. I tried again with: node-gyp --python /path/to/python2.7 and the same error, because its a global configuration

Same problem after update to nodejs 8 !

Closing this out because it's been dormant for over a month.  If it's still an issue, please provide the requested logs.

pkg-config' is not recognized as an internal or external command,
operable program or batch file.
gyp: Call to 'pkg-config --libs  cairo poppler-qt5' returned exit status 1 while in binding.gyp. while trying to load binding.gyp

@sanelib pkg-config is an external tool and not normally available on Windows.  Try reaching out to the authors of the package you are trying to install, perhaps Windows isn't supported.

After I input command: node-gyp rebuild,  I find following error:
gyp: binding.gyp not found (cwd: C:\Users\leon\AppData\Roaming\npm\node_modules\node-gyp) while trying to load binding.g
yp
gyp ERR! configure error
gyp ERR! stack Error: gyp failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (C:\Users\leon\AppData\Roaming\npm\node_modules\node-gyp\lib\configure.js:33
6:16)
gyp ERR! stack     at emitTwo (events.js:125:13)
gyp ERR! stack     at ChildProcess.emit (events.js:213:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:197:12)
gyp ERR! System Windows_NT 10.0.15063
gyp ERR! command "D:\Program Files\nodejs\node.exe" "C:\Users\leon\AppData\Roaming\npm\node_modules\node-gyp\
bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\leon\AppData\Roaming\npm\node_modules\node-gyp
gyp ERR! node -v v8.0.0
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok

@jeanru you need to run node-gyp rebuild in a directory that contains a binding.gyp.

@gibfahn  Thanks, I have fixed this issue by create binding.gyp file manually, and then run node-gyp rebuild

@gibfahn Thanks, I have fixed this issue by creating binding.gyp file manually, and then run node-gyp rebuild

I am getting this error every time I run a node-gyp command and when I try to install enmap and quick.db
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (C:\Users\ADRIÁN\AppData\Roaming\npm\node_modules\node-gyp\lib\configure.js:351:16)
gyp ERR! stack     at ChildProcess.emit (events.js:210:5)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:272:12)
gyp ERR! System Windows_NT 10.0.18362
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\ADRIÁN\\AppData\\Roaming\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "configure" "--msvs_version=2015"
gyp ERR! cwd C:\Users\matt\Music\Discord-All\DiscordiumBots\Customers-Bots\SCP Assistant
gyp ERR! node -v v12.13.0
gyp ERR! node-gyp -v v6.0.1
gyp ERR! not ok```

I have `Node.js v12.13.0`, `npm v6.12.0` and `Python 2.7.16`
In my directory, there is a `binding.gyp` file I created myself, it is empty as I don't know if I have to add anything.

I get the same error as above. Detailed: https://github.com/nodejs/node-gyp/issues/1969
binding.gyp should contain something like this:
{   "targets": [     {       "target_name": "addon",       "sources": [ "addon.cc" ]     }   ] }

Alright, thank you so much!

But how should I fix the error I am getting? Would adding that to binding.gyp fix the error?

In my case the error was because of a folder name. The fix was to rename this folder C:\Users[username] to English, without spaces.


where is tensorboard log dir

Where is the log directory in tensorboard after fine tuning internvl-chat?  Can not find the log directory to view training logs in tensorboard. Thanks!

You can find it in the ./work_dirs/${project_name}
For example, if you run this script to finetune our model, you can find the tensorboard log in the ./work_dirs/internvl_chat_v1_5_phi3_3_8b_dynamic_res_finetune

This issue has been inactive for two months, so I am closing it. If you have any further questions or encounter any problems, please feel free to reopen it. Thank you!


Xml dumping fixes

Attempt to fix comments from https://github.com/futzu/SCTE35_threefive/discussions/75#discussioncomment-10765232 plus a bunch of other stuff where I will continue the conversation.
Highlights:

fixed up some validation issues around number types
removed some (strictly) invalid attributes
removed explicit scte35: namespace in node name. this is now inserted as an attribute on the SpliceInfoSection, which I don't really like but still prefer to prefixing everything
add support for more descriptors
fixed DTMFDescriptor in general (was missing tag)
improved SegmentationDescriptor parsing and dumping (but still a fair way to go)

The XML now validates against the latest schema and can be parsed by a XML parser.

Looks great man, really.
It seems like you were able to follow it fairly well,
which is super cool.
Thanks man, it's nice to have some fresh eyes on it.


Handle different response types from Curity

When the subrequest is made to Curity, the following status codes should be handled:

Status Code from Curity
Status Code from NGINX

200 with valid JWT
response from back-end

204 without valid JWT
401

1XX, 201-400, 402, 405-499
500

401, 403, 404
400

500-502, 504-599
502

503
503

When Curity is updated to accept application/jwt at the introspection endpoint, it will return a 204, no content, when the JWT is expired. In this case, the NGINX proxy should reply with a 401. After that point, Curity won't reply with a 200 except when the access token is valid.


gmp: Add ptest

Purpose of pull request
This PR adds ptest of gmp.
This ptest executes make check-TESTS in tests/ directory. There are some sub-directories under tests/ directory, so it executes make check-TESTS there as well.
Test
How to test

Enable ptest and install gmp package

$ . ./repos/poky/oe-init-build-env build
$ bitbake-layers add-layer ../repos/meta-debian/
$ cat << EOS >> conf/local.conf
DISTRO = "deby"
MACHINE = "qemuarm64"
PACKAGE_CLASSES = "package_deb"
DISTRO_FEATURES_append = " ptest"
EXTRA_IMAGE_FEATURES += "ptest-pkgs"
IMAGE_INSTALL_append = " gmp"
EOS

Build core-image-minimal image

$ bitbake core-image-minimal

Run qemu and execute ptest of gmp

$ runqemu nographic slirp
...(snip)...
# ptest-runner -l
...(snip)...
# ptest-runner -t 3600 gmp

Also, I confirmed that SDK builds succeed with the following settings:

Set DISTRO=deby and run bitbake core-image-minimal -c populate_sdk with meta-debian and poky
Set DISTRO=emlinux and run bitbake core-image-minimal-sdk -c populate_sdk with meta-debian, meta-debian-extended, meta-emlinux and poky

Test result
# ptest-runner -l
Available ptests:
busybox /usr/lib/busybox/ptest/run-ptest
gmp     /usr/lib/gmp/ptest/run-ptest
util-linux      /usr/lib/util-linux/ptest/run-ptest
zlib    /usr/lib/zlib/ptest/run-ptest
# ptest-runner -t 3600 gmp
START: ptest-runner
2024-04-09T05:01
BEGIN: /usr/lib/gmp/ptest
make: Entering directory '/usr/lib/gmp/ptest/tests'
make[1]: Entering directory '/usr/lib/gmp/ptest/tests'
PASS: t-bswap
PASS: t-constants
PASS: t-count_zeros
...(snip)...
PASS: t-urbui
PASS: t-urmui
PASS: t-urndmm
============================================================================
Testsuite summary for GNU MP 6.1.2
============================================================================
# TOTAL: 7
# PASS:  7
# SKIP:  0
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0
============================================================================
make[1]: Leaving directory '/usr/lib/gmp/ptest/tests/rand'
make: Leaving directory '/usr/lib/gmp/ptest/tests/rand'
=== Test Summary ===
TOTAL: 190
PASS: 190
SKIP: 0
XFAIL: 0
FAIL: 0
XPASS: 0
ERROR: 0
DURATION: 718
END: /usr/lib/gmp/ptest
2024-04-09T05:13
STOP: ptest-runner

ptest-gmp.log
Test summary

TOTAL: 190

PASS: 190
FAIL: 0

I run this ptest 3 times and obtained the same results.

merged in upstream


Adding a bookmark removes the top folder/bookmark from the list

Also, check if scrolling of bookmark works prev and now


How to auto connect

I can't understand what is the correct URL syntax in order to establish an auto connection, if it is possible.

Yes, it's possible; syntax documentation can be found here : https://github.com/cedrozor/myrtille/blob/master/DOCUMENTATION.md#start-remote-application-from-url
If you want to run a specific application upon auto-connection, you have to set the "&program=" param with the executable file path and name.
As the querystring must be url encoded, you have to use a tool like http://www.url-encode-decode.com/ for all params, then copy & paste the result into the url.
For example, to run

C:\Program Files\MyApp\AppName.exe -with -some -params

the url param is

&program=C%3A%5CProgram+Files%5CMyApp%5CAppName.exe+-with+-some+-params

If your user password is

Hello!

the url param is

&password=Hello%21

You can also hash the password for more security if you wish (https://github.com/cedrozor/myrtille/blob/master/DOCUMENTATION.md#password-hash), but it's not mandatory.
Related issues :
https://github.com/cedrozor/myrtille/issues/41
https://github.com/cedrozor/myrtille/issues/53

Hello @cedrozor, thanks for your reply and this great project, well I've found out that if you don't set width and height, client avoids auto connect, or auto login in other words, even if every credential is set, and after other checks, I have also noticed that there is no window self-resizing feature because my use case applies to an iframe that should show the client's content within a web page.
We can state that for the time being, it is always necessary to set width and height within the url to avoid visualization and authentication issues.
But in fact this also applies to browser use, and it is really strange that I must to set width and height even if I want to simply load that url in the browser with auto login, and I must click the connect button in order to open the session. I don't know if it is an expected behaviour or a bug.

Strange indeed, it's not the expected behavior. If you dont set width and height, it should be auto-detected from the browser window.
I didn't tested myrtille within an iframe, it may require some tweaking:

make sure the parent window (browser page) is on the same domain to avoid cross domain restriction
the javascript code which auto-detects the resolution, when starting a program from url, applies to the current window (the iframe), not the parent (the browser page).

About clicking the connect button, it's not normal; did you added it to the url? (&connect=Connect%21). It's required.

Thanks to an user feedback, and after doing some additional testing with version 1.5.0, I noticed that the "&program=" param must be added to the to the url for the auto-connect feature to work.
This is a regression and is tagged for fix. I will update the documentation which flag this param as optional.
If you don't want a program to be run on session start, leave the param empty ("&program="); otherwise , set its path\name (i.e.: "&program=notepad.exe").

Any progress on this?

Fixed into myrtille 1.6.0 (to be released soon)


Move the 'Permissions required for the DSA' down further.

Move the 'Permissions required for the DSA' down further. It doesn't make sense to lead off the article with this section since the user can't complete it since they haven't created the DSA yet. Further, after having read the doc and created the account, they may forget to come back to the top and perform that section.
[Enter feedback here]

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

ID: 12a9a5e8-afc3-7554-6a0b-248fae64a53f
Version Independent ID: 12a9a5e8-afc3-7554-6a0b-248fae64a53f
Content: Directory Service account recommendations
Content Source: ATPDocs/directory-service-accounts.md
Service: microsoft-defender-for-identity
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

Thank you for your comment. We'll investigate and get back to you.

@ianc911  - Thanks for your feedback. We've moved the section lower down on the page (the change should go live soon).


Does Youki support Linux kernel version 4. x? What is the minimum version of the kernel that can be supported?

I would like to use youki in prod environment~, however, the Linux kernel version I need to consider .

@wxmayifei 👋 Hi, youki requires the kernel version >= 5.3, and we don't have a plan to support the old kernel 😭
From README:

Requires
Rust(See here), edition 2021
linux kernel ≥ 5.3

em. .......，


Kusto query error on summarizing pod status

The below two lines doesn't work for the phase count query:
        | summarize PodStatus=any(PodStatus) by TimeGenerated, PodUid, ClusterId
        | summarize TotalCount = count(),
                    PendingCount = sumif(1, PodStatus =~ 'Pending'),
                    RunningCount = sumif(1, PodStatus =~ 'Running'),
                    SucceededCount = sumif(1, PodStatus =~ 'Succeeded'),
                    FailedCount = sumif(1, PodStatus =~ 'Failed')
                by ClusterName, bin(TimeGenerated, trendBinSize)

The first line should be using ClusterName to match the query parameters, otherwise the 2nd summarize doesn't recognize ClusterName field.

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

ID: 4873260a-eaeb-2ff9-ee38-36f3243b697f
Version Independent ID: a17aaa80-c0db-74ee-7e09-d71410bc7a62
Content: Log alerts from Azure Monitor for containers - Azure Monitor
Content Source: articles/azure-monitor/insights/container-insights-log-alerts.md
Service: azure-monitor
GitHub Login: @bwren
Microsoft Alias: magoedte

@tdihp Thanks for the feedback. We're investigating this and will get back to you shortly.

@tdihp Thanks for the feedback. We're investigating this and will get back to you shortly.

@tdihp Thank you for catching that error in the docs! I have submitted a PR to fix the issue, the change is merged internally and will go-live in the public docs shortly.

@tdihp Thank you for catching that error in the docs! I have submitted a PR to fix the issue, the change is merged internally and will go-live in the public docs shortly.


Can't add google account to device.

I am unable to add any google accounts to the device.
I am running LineageOS 14.1 on a OnePlus3
I have confirmed that I have the latest microG versions (updated through F-Droid) and that GmsCore is in /system/priv-app and has android permissions granted.
I can go through the login process, but it hangs on the "Just a Sec" screen. I took a logcat and redacted all sensitive information (It does look like the login succeeded):
03-18 12:23:28.765  9208  9208 D GmsAuthLoginBrowser: pageFinished: https://accounts.google.com/EmbeddedSetup/signinconsent?[REDACTED]
03-18 12:23:28.771  9208 10733 D GmsHttpFormClient: -- Request --
03-18 12:23:28.771  9208 10733 D GmsHttpFormClient: accountType=HOSTED_OR_GOOGLE&add_account=1&[REDACTED]
03-18 12:23:28.775 11120 11120 I cnss-daemon: RTM_NEWROUTE Indication
03-18 12:23:28.775 11120 11120 I cnss-daemon: ip type is ipv6
03-18 12:23:28.854 18298 18298 D wpa_supplicant: wlan0: Control interface command 'SIGNAL_POLL'
03-18 12:23:28.858 18298 18298 D wpa_supplicant: CTRL-DEBUG: global_ctrl_sock-sendto: sock=9 sndbuf=212992 outq=0 send_len=61
03-18 12:23:29.226  9208 10733 D GmsHttpFormClient: -- Response --
03-18 12:23:29.226  9208 10733 D GmsHttpFormClient: Token=[REDACTED]
03-18 12:23:29.226  9208 10733 D GmsHttpFormClient: Auth=[REDACTED]
03-18 12:23:29.226  9208 10733 D GmsHttpFormClient: SID=[REDACTED]
03-18 12:23:29.226  9208 10733 D GmsHttpFormClient: LSID=[REDACTED]
03-18 12:23:29.226  9208 10733 D GmsHttpFormClient: services=mail,talk,multilogin,omaha,android,cl,friendview,chromiumsync,writely,youtube[REDACTED]
03-18 12:23:29.226  9208 10733 D GmsHttpFormClient: Email=[REDACTED]
03-18 12:23:29.226  9208 10733 D GmsHttpFormClient: GooglePlusUpdate=0
03-18 12:23:29.226  9208 10733 D GmsHttpFormClient: firstName=[REDACTED]
03-18 12:23:29.226  9208 10733 D GmsHttpFormClient: lastName=[REDACTED]
03-18 12:23:29.264 17854 18488 W AccountManagerService: insertAccountIntoDatabase: Account<EMAIL_ADDRESS>type=com.google}, skipping since the account already exists
03-18 12:23:29.266  9208 10733 W GmsAuthLoginBrowser: onException: android.view.ViewRootImpl$CalledFromWrongThreadException: Only the original thread that created a view hierarchy can touch its views.
03-18 12:23:31.873 18298 18298 D wpa_supplicant: wlan0: Control interface command 'SIGNAL_POLL'
03-18 12:23:31.881 18298 18298 D wpa_supplicant: CTRL-DEBUG: global_ctrl_sock-sendto: sock=9 sndbuf=212992 outq=0 send_len=61
03-18 12:23:34.891 18298 18298 D wpa_supplicant: wlan0: Control interface command 'SIGNAL_POLL'
03-18 12:23:34.896 18298 18298 D wpa_supplicant: CTRL-DEBUG: global_ctrl_sock-sendto: sock=9 sndbuf=212992 outq=0 send_len=61

I do see these lines:
03-18 12:23:29.264 17854 18488 W AccountManagerService: insertAccountIntoDatabase: Account<EMAIL_ADDRESS>type=com.google}, skipping since the account already exists

It seems to think that the account already exists, though it isn't in Settings>Accounts.

I would try removing the device from security center on a computer, then attempting to sign in again.

This could also be a manifestation of #510.

Yes, now I get a specific error about being unable to add the account.

The fix does seem to be the one here: https://github.com/microg/android_packages_apps_GmsCore/pull/478/
However, I have verified that the account is NOT already added to the device (in device settings > Accounts), and I have tried revoking the device's access via myaccount on google, but I can't add the account.
I'm pretty sure this is the key: (from logcat)
03-18 12:23:29.264 17854 18488 W AccountManagerService: insertAccountIntoDatabase: Account<EMAIL_ADDRESS>type=com.google}, skipping since the account already exists

There seems to be some sort of "ghost" account in the database that isn't actually in the device accounts.

What is the content of /system/data/sync/accounts.xml?

Fixed it!
I had to delete these files to properly remove the Google account and be able to relogin:
/data/system_de/0/accounts_de.db
/data/system_ce/0/accounts_ce.db
/data/system/sync/accounts.xml
I found a similar bug at the nextcloud project and they solved it like this:
https://github.com/nextcloud/android/pull/949
Maybe a similar fix can be incorporated into microG.

Are these accounts databases recreated by Android after you delete them?

Yes the files are recreated after a reboot on LOS 14.1.

The problem should be fixed in recent versions of microG GmsCore, please try it and report back.


5.0.0 Release Tasks

Working on the 5.0.0 release...

[ ] Update CHANGELOG
[ ] Cut a rug tag
[ ] Add tag release notes
[ ] Submit PR to update BLT
[ ] Blog post on Planet (highlight new things)
[ ] Make sure https://www.drupalvm.com is updated.

See milestone: https://github.com/geerlingguy/drupal-vm/milestone/11

Almost finished, just need to do https://github.com/geerlingguy/drupal-vm/issues/1881 — plan on doing that either over this weekend or on first day or two of DrupalCon so I have enough time to put it through its paces (since 16.04->18.04 is not a super-simple change for downstream users, thus the major version bump).

Done with #1881. On to this ticket's punchlist!

I have a blog post in the can, unpublished...

One of the most important tasks: https://github.com/geerlingguy/drupal-vm/issues/1919

www.drupalvm.com updated: https://github.com/geerlingguy/drupalvm.github.io/commit/883e8a6b1e5601cadc8cbf5f55a9897b7a0a9a82

BLT issue opened: https://github.com/acquia/blt/issues/3536
Closing this issue! 🎉


[fix] Run correctly when used alongside @vue/compat

🚀 What's changed?

Don't break when run in a project using @vue/compat

🐛 Fixes

I didn't file a bug report because I'm lazy. However, when running VueFlow in a Vue 3 project that has @vue/compat installed, the package crashes when rendering edges. This is because of the change in the way functional components work in Vue 3 (simple functions are now functional components, but unless the ASYNC_COMPONENT flag in @vue/compat is disabled, it will attempt to process them as legacy-style async components which just... totally does not work).
I did test this in my project by running pnpm build and then using npm link to link @vue-flow/core into my project. Before this change, big crash. After this change, no crash and I'm able to render a basic chart without errors.
🪴 To-Dos

No todos to my knowledge :tada: I believe I got all the components across both TS and Vue files across all packages but please let me know if I missed any. I'm no stranger to these types of PRs - I've made similar ones for @vue/test-utils and vue-router some time back (also, VueDraggable - but that package appears to not be maintained anymore). This package looks great and I'm excited to try it out in my project once this gets merged!

Thanks for the PR, I'll check it out 👍

appreciate it @bcakmakoglu! I don't want to rush you but it would be super helpful if I could get this through quickly (want to avoid getting blocked on deploying with this library after finishing my prototype)

I’ll include the PR in tomorrows release
Is that ok? There’s a couple details I want to include in the release but it’ll be done by tomorrow
Also I’ve been wondering, Is the compat mode necessary for non-functional components?
probably doesn’t hurt but just wanted to know if it’s necessary for regular components as well

yes, tomorrow would be perfect! Thank you!
to answer your second question - it's definitely necessary for the functional components, but I'm not sure if it's strictly necessary for the non-functional ones. There are a lot of different flags that sometimes subtly change functionality (like how watch() works with arrays, for example) and it's hard to check for all of them. Obviously components written for Vue 3 aren't using things like $scopedSlots so there's some flags that won't be relevant but I think it's generally a pretty good practice to explicitly define all components with mode 3 in public projects, just in case someone (like me) who has a partially-migrated project needs to use it. (I actually have a export const FULLY_COMPATIBLE = { MODE: 3 } in my project to make this read even more nicely, but I don't know how necessary that is in most projects).

@snoozbuster May I ask if you could update your commit message to adhere to conventional commits?
Something like fix: add compat config to components?
Other than that, this looks good to me 👍
I can add a changeset afterwards to patch the package versions

@bcakmakoglu yep, done! thanks again!


add navigation bar in construction page

Added navigation bar but the issue cards disappear

issue card appeared, seems like its an issue with throttle from GitHub


reference serial from sequences

User gave negative docs feedback on search results for sequence. That keyword shows up in the SERIAL docs, but on our SQL Feature Support list, we don't reference SERIAL in the Sequences entry. This PR fixes that.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/72466e2054b9858870f3ff66db66c6bc4508ecc3/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/1cbb46d6a33c95831a596877b574ff77b0b7398c/


Getting construct references for resources

Hey,
This library seems very nice and the api seems very clear. I'm trying to use it and I don't understand how I'm supposed to get internal resources for example load balancer. I need load balancer to add it to cloudfront origin, but the HttpLoadBalancerExtension property loadBalancer is private, I could override the class and extract the load balancer, but that seems an overkill.
What is your suggested way?
Cheers,
Arturs

Best way currently is to grab the children of your stack, like so:
declare const yourService: Service;

const elb = stack.node.findChild("`{yourService.id}-load-balancer`);

Does that solve your problem? This is a typical escape-hatch way of grabbing child resources. Feel free to reopen if you have other questions!


Add order_siblings_by queryset method

Add an order_siblings_by queryset method to allow the tree sibling ordering to be specified in a queryset.
Add sibling ordering test to the testapp to verify the function of order_siblings_by

Thanks! That's great.


License Issue

Due to you including grub which has a GPLv3 license, you need to license orbital under GPLv3.

@bitcynth Orbital's MIT license only covers the code inside the main repository, not subrepositories.
Each subrepository, i.e. orbital-bios, orbital-grub, orbital-qemu, are licensed exactly as the parent repositories they were forked from. Refer to the LICENSE files in each of them for more information.

Well that is not how the GPLv3 works, your project still needs to be GPLv3... it is contagious

my FreeBSD build runs on grub, go tell FreeBSD to change their license as well.

FreeBSD has not grub included in their repository, does it? You can solve this with problem with makefile.

neither is it here, it's merely linked to by this repo.

For all intents and purposes, each submodule is an independent project with absolutely orthogonal goals. They are all aggregated under this repository for mere organizational reasons. I'm not including any of their sources in this repository, only an URL/hash, pointing to a revision which is licensed exactly as the repository it was forked from.
Seriously, I seriously don't see what's the issue here. There's essentially no legal material on the matter, so discussing this any further is pointless and a waste of time for everyone. Plus, it wouldn't be the first case of repositories with permissive software licenses linking to GPL ones [1].
[1] https://softwareengineering.stackexchange.com/a/315858

Just to be clear I see no licensing issues either, there are license files included where they should be, I just had to check twice, my mistake. Long time ago I contacted FSF because of same issue and they told me that it isn't a violation of GPLv3 license. Adding this as reference comment here.

To add a little bit to this, I once skipped Stallman an email about a repository that accepts no commits, but is under a free license; he very much approves such a scenairo. And, if you read the GPL fine print, you'll see this issue is compatible.


Fix tarball install

Updated PR #97. Thanks @gbarceloPIB
What type of PR is this?

[X] bug
[ ] cleanup
[ ] documentation
[ ] feature

What this PR does / why we need it:
Fixes tarball install when using custom install path different from /usr/local. For example, when /usr/local is read only or mounted as btrfs subvol. Behavior is similar to https://get.rke2.io/ script.
Which issue(s) this PR fixes:
Fixes #96
Release Notes
Fixed broken tarball install when tarball_dir is not default `/usr/local`

Reviewed with @dgvigil and @mddamato. Approved


streamlined menu.py

Streamlined menu.py

Note to myself: Don't forget to change version number in space_game.py and README.md


Command: open_url

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/1197099-command-open_url?utm_campaign=plugin&utm_content=tracker%2F282001&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282001&utm_medium=issues&utm_source=github).

Migrated to limetext/lime-backend#19.


Expose HTTP header key names

Feature Request
Packages versions used:
github.com/Azure/azure-sdk-for-go/sdk/azcore v1.7.0-beta.2
github.com/Azure/azure-sdk-for-go/sdk/azidentity v1.4.0-beta.1
github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/compute/armcompute/v5 v5.1.0-beta.1
github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/marketplaceordering/armmarketplaceordering v1.2.0-beta.1
github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/resourcegraph/armresourcegraph v0.8.0
github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/resources/armresources v1.2.0-beta.1

NOTE: Beta versions are used in order to leverage the fake packages introduced.

For traceability we extract the following headers:

X-Ms-Correlation-Request-Id
X-Ms-Request-Id
x-ms-error-code
x-ms-client-request-id

We saw that for the above constants have been defined in internal/shared package.
Since these constants are in an internal package having a dependency on that package is not good for any consumer. Is it possible to move these constants to a non-internal package which consumers of the SDK can easily use instead of hard coding these header keys? This will also ensure that if the the name of the header changes the consumers do not have to adapt as long as well defined constant names are used which are kept the same.

NOTE: We noticed that x-ms-correlation-request-id is currently missing from the constants. Is this a miss or is this header no longer set?

@raych1, could you help to take a look?

Hi @jhendrixMSFT , do you think if these constants can be exported to be consumed by the end user?

Also add @JeffreyRichter to help to evaluate. For azure specific constants, shall we export them?

I'm opposed to exposing these and moving them to a non-internal package.
There is nothing special about these constants and, in fact, we could get rid of these constants entirely and just hard code the strings in our code where we need them. These headers are all well-documented and our "x-ms-error-code" string is no more special than anyone else's "x-ms-error-code" string.

But, by exposing these constants, we open ourselves to having customer dependencies that we would not have otherwise.
We have to document these and maintain them in perpetuity which might cut off some opportunities for us in the future.
By documenting them, we implicitly raise their importance and customers might take a dependency that we do not want them to take.
Also, by documenting them we pollute our public surface area which is already bigger than we'd like it to be for a minimal-value scenario.

Making these public has no upside to the SDK and does not enable any scenario for a customer that is not already trivially possible. In addition, a customer who wants to use these header values knows that they want to use them and knows what the values are already -- they'd have to somehow discover these constants in the SDK which is very unlikely and so even if we did this work, it is unlikely to be useful.

It is not uncommon to make certain request/response headers as part of the public API. See here. Since these are part of the response therefore one can argue that its part of the contract as these are available in the response for consumers of the API to consume.

Making these public has no upside to the SDK and does not enable any scenario for a customer that is not already trivially possible

X-Ms-Correlation-Request-Id, X-Ms-Request-Id  - these are typically used by consumers when reporting issues to Azure as this gives a handle to the azure teams to trace a particular request.
x-ms-error-code - this is used by the consumer of the API to extract the azure error code in addition to the HTTP response code and print it in the logs for better diagnostics.

These headers are all well-documented and our "x-ms-error-code" string is no more special than anyone else's "x-ms-error-code" string.

Sure, request-ID, correlation-ID and error-code are very generic concepts but the name of these properties are MS specific and therefore one can argue that they be part of the contact. If this is part of internal package then can any consumer consume these reliably? The answer is NO because it is internal and can be changed which then mandates all the consumers to also always adapt.

But, by exposing these constants, we open ourselves to having customer dependencies that we would not have otherwise.

Since these are so generic and common properties why would you even have the need to change it?

These headers are part of the service's REST contract and not part of the Go SDK contract. The only reason the SDK would change these values is because the REST API contract changed. And, I can say with 100% certainty that these header values will never change as I also am on the Azure Breaking Change Board. Changing them would break all the old SDKs too - it will just never happen.
You are correct: We would never need to change these values (unless we spelled them wrong in the first place).
We have a goal of minimizing our SDK surface area to keep it simple, easy to maintain, reduce documentation, etc. I just don't see how making these public enables any scenario that isn't trivially solved today by other means.

@unmarshall , I close this issue as won't fix. Feel free to re-open this issue if needed.


Add installation guide for NixOS

See also https://github.com/netbirdio/dashboard/issues/67

@misuzu looks nice!


[Question] [ws-sbom-generator] Cannot run in windows

Hi,
I tried to run ws-sbom-generator on Windows powershell with command which is successfully executed in linux, but i only got two line:
INFO 2022-10-21 18:17:39,630 25936 sbom_generator: Creating SBOM Document from Mend 'ACC_Python'
ERROR 2022-10-21 18:17:41,878 10612 sbom_generator: Error running SBOM Generator
Is there any suggestion what is going wrong?
Environment Details

OS: windows 10 enterprise
Browser :chrome, edge
Version: v22

HI @Tom-HY-Tsai !  Unfortunately, I can't reproduce your error still. Can you send me (in private) your JSON file which you are using in --extra parameter? What version of Sbom-generator did you use?

Hi, DimarrWS
Greeting and Very thanks for your reply!
I just follow command to install sbom-generator, and I found no parameter
to print version i installed (ex: -v), so the hash value of sbom-generator
is provided here for your reference :e968fcbbfa9c0741a0d54c4cd7525149 if
that OK to you .
and I also attach JSON file, I thought, might be wrong, it need a "empty"
JSON as a skeleton for generator, so I copy content from readme and the
file is attached.
please let me know if there is any information you need
Best regards
Tom
DimarrWS @.***> 於 2022年10月23日 週日 下午3:54寫道：

HI @Tom-HY-Tsai https://github.com/Tom-HY-Tsai ! Unfortunately, I can't
reproduce your error still. Can you send me (in private) your JSON file
which you are using in --extra parameter? What version of Sbom-generator
did you use?
—
Reply to this email directly, view it on GitHub
https://github.com/whitesource-ps/ws-sbom-generator/issues/138#issuecomment-1288043691,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOKYZXN2OLLSMOY3KJT72Z3WETVMPANCNFSM6AAAAAARLA7AWA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @Tom-HY-Tsai ! Can you try to run the tool without -e parameter? Also I hope you have folder Output , which you provide as -o parameter?

Hi, DimarrWS
Thanks for your reply,
I just ran the tool without -e, and it still fails, and yes, I have output
directory
please check screenshot as attached
Thanks for your help
Best Regards
Tom
DimarrWS @.***> 於 2022年10月24日 週一 下午2:30寫道：

Hi @Tom-HY-Tsai https://github.com/Tom-HY-Tsai ! Can you try to run the
tool without -e parameter? Also I hope you have folder Output , which you
provide as -o parameter?
—
Reply to this email directly, view it on GitHub
https://github.com/whitesource-ps/ws-sbom-generator/issues/138#issuecomment-1288484414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOKYZXP57AO6PXLJKZCMIXLWEYUH3ANCNFSM6AAAAAARLA7AWA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hmmm, I do not see any screenshots here. Looks like nothing attached...
Tom, what is the exact name of your organization on saas environment?

Hi, DimarrWS
That's kinda weird, I attached it again(screenshot-2.png), and my
organization is ASUS
Thanks
Best Regards
Tom
DimarrWS @.***> 於 2022年10月24日 週一 下午3:16寫道：

Hmmm, I do not see any screenshots here. Looks like nothing attached...
Tom, what is the exact name of your organization on saas environment?
—
Reply to this email directly, view it on GitHub
https://github.com/whitesource-ps/ws-sbom-generator/issues/138#issuecomment-1288527390,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOKYZXPB55W5Y66QM4G4IWTWEYZVBANCNFSM6AAAAAARLA7AWA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi, DimarrWS
I copy my screen as text, please check if you still not receive the
screenshot, i'm sorry that  i need to remove some parameters
PS C:\Users\TomHY_Tsai> ws_sbom_generator.exe -u  -k  -y
organization -a saas -s  -o ./output  -t jsonWARNING 2022-10-24
15:01:20,860 20860 sbom_generator:
C:\Users\TomHY_Tsai\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.10_qbz5n2kfra8p0\LocalCache\local-packages\Python310\site-packages\ws_sbom_generator\resources\sbom_extra.json
configuration file was not found. Be sure to create a file in the following
structure (-e/--extra):            {                "namespace":
"http://CreatorWebsite/pathToSpdx/DocumentName-UUID
http://CreatorWebsite/pathToSpdx/DocumentName-UUID",
"org_name": "Organization Name",                "org_email":
@.",                "person": "person name",
"person_email": @."            }INFO 2022-10-24
15:01:21,892 5220 sbom_generator: Creating SBOM Document from Mend
'ACC_Python'ERROR 2022-10-24 15:01:23,478 20860 sbom_generator: Error
running SBOM GeneratorPS C:\Users\TomHY_Tsai> dir .\output\PS
C:\Users\TomHY_Tsai>
Thanks
Best regards,
Tom
Her-Yann Tsai @.***> 於 2022年10月24日 週一 下午3:21寫道：

Hi, DimarrWS
That's kinda weird, I attached it again(screenshot-2.png), and my
organization is ASUS
Thanks
Best Regards
Tom
DimarrWS @.***> 於 2022年10月24日 週一 下午3:16寫道：

Hmmm, I do not see any screenshots here. Looks like nothing attached...
Tom, what is the exact name of your organization on saas environment?
—
Reply to this email directly, view it on GitHub
https://github.com/whitesource-ps/ws-sbom-generator/issues/138#issuecomment-1288527390,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOKYZXPB55W5Y66QM4G4IWTWEYZVBANCNFSM6AAAAAARLA7AWA
.
You are receiving this because you were mentioned.Message ID:
@.***>

If I understand correct you ran tool successfully on Linux env., right?
Has your Windows computer same access to your environment ? (I mean that IP is in White List)

Hi, DimarrWS
We found the workaround to fix this issue, it seem the problem is related
to locale,
the tools work when i change system administrative language to us-en
FYI,
Thanks for your help
Best regards,
Tom
DimarrWS @.***> 於 2022年10月24日 週一 下午5:03寫道：

If I understand correct you ran tool successfully on Linux env., right?
Has your Windows computer same access to your environment ? (I mean that
IP is in White List)
—
Reply to this email directly, view it on GitHub
https://github.com/whitesource-ps/ws-sbom-generator/issues/138#issuecomment-1288689518,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOKYZXKBSMEM7BBXCEMZJ43WEZGHTANCNFSM6AAAAAARLA7AWA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I see, Tom. Thank you for the update.
Please, contact me anytime you will have problems with the tool. I will be glad to assist.
BR,
Dima


Collection of minor performance optimizations

See individual commits. Most of these are related to using unordered maps/sets instead of ordered ones and are the result of multiple profiling sessions. All these small optimizations add up to about 10% of CPU usage on the message handler thread.
This PR also contains a backport of bitcoin#13176 as I've also noticed unexpected high CPU use for CNode::AddInventoryKnown.

mapArgs was not that bad, I just did it while I was in optimization mode :)
The sporks is a different thing. We now perform a lot of IsNewInstantSendEnabled calls, which summed up to 1% of total time spent. I assume the actual reason is the temporary mapValueCounts map, which we should try to avoid in an upcoming refactoring. Replacing the maps with unordered maps just looked like the fastest/cheapest way to optimize it a tiny little bit.

Added a few commits to fix Travis and also ensure that future backports don't go wrong.