Add tax calculation and display tax amount in payment process

This pull request adds tax calculation functionality to the payment process. It includes updates to the Stripe.js file to handle tax calculation and logging, as well as updates to the stripeGenerate.vue file to display the tax amount in the payment button. The stripePaymentIntent method in the PaymentController class now retrieves the user's country and checks if it matches the country provided in the request body. If not, it updates the user's country. The tax amount is calculated using the stripe.calculateTax method and stored in the taxAmountExclusive variable. The stripePaymentIntent method also creates a stripe payment intent and returns the paymentIntent along with the tax amount. Additionally, the stripePaymentIntent schema and the calculateTax method in Stripe.js have been updated to include the customer's country for accurate tax calculation. The users.js file has been modified to include the user's name in the GraphQL query for user details.

Summary by Sourcery
Add tax calculation to the payment process, updating the Stripe.js and stripeGenerate.vue files to handle and display tax amounts. Enhance the PaymentController to retrieve and update user country information for accurate tax calculation.
New Features:

Introduce tax calculation functionality in the payment process, allowing tax amounts to be calculated and displayed during payment.

Enhancements:

Update the Stripe.js file to handle tax calculation and logging, including the addition of constants for tax codes and behavior.
Modify the stripeGenerate.vue component to display the tax amount in the payment button and require country selection for tax purposes.

@sourcery-ai review

@sourcery-ai review


update(build): add back PR #373

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind cleanup

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

What this PR does / why we need it:
"Wax on, wax off". As per previous description: https://github.com/falcosecurity/plugins/pull/400
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: LucaGuerra
Once this PR has been reviewed and has the lgtm label, please assign jasondellaluce for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: e8ca12b20685e2d887233cdf0154df631d2720d5

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Hidden roads and paths that are not able to be deleted

Hello,
There is this a false road and pedestrian path in Ulaanbaatar, Mongolia. You can see it by in Edit mode and zoom out till where you cannot edit, you will be able to see a road from Sukhbaatar square to south over the train road, like a triangular. But when you zoom in to edit, it disappears. This is really annoying because this false road shows on Facebook maps, a road does not even existed.
Screenshots below:
https://imgur.com/a/5U8DTcv
Thanks

https://i.imgur.com/sD9fbS1_d.jpg?maxwidth=640&shape=thumb&fidelity=medium
https://i.imgur.com/GqMmwnO_d.jpg?maxwidth=640&shape=thumb&fidelity=medium
https://i.imgur.com/OKWtMGE_d.jpg?maxwidth=640&shape=thumb&fidelity=medium

I removed some spurious points where they converged in the south but I can’t see anything at the northern end of the lines
Points had no tags – may or may not fix the issue
https://www.openstreetmap.org/#map=16/47.9065/106.9217
Cheers - Phil
From: afropoker<EMAIL_ADDRESS>Sent: Thursday, 12 July 2018 10:31 PM
To: openstreetmap/iD
Cc: Phil Wyatt; Comment
Subject: Re: [openstreetmap/iD] Hidden roads and paths that are not able to be deleted (#5135)
https://i.imgur.com/sD9fbS1_d.jpg?maxwidth=640 https://i.imgur.com/sD9fbS1_d.jpg?maxwidth=640&shape=thumb&fidelity=medium &shape=thumb&fidelity=medium
https://i.imgur.com/GqMmwnO_d.jpg?maxwidth=640 https://i.imgur.com/GqMmwnO_d.jpg?maxwidth=640&shape=thumb&fidelity=medium &shape=thumb&fidelity=medium
https://i.imgur.com/OKWtMGE_d.jpg?maxwidth=640 https://i.imgur.com/OKWtMGE_d.jpg?maxwidth=640&shape=thumb&fidelity=medium &shape=thumb&fidelity=medium
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/openstreetmap/iD/issues/5135#issuecomment-404495817 , or mute the thread https://github.com/notifications/unsubscribe-auth/AIXm4-yocRG3onDeRmAEOhNMsvlF6bi8ks5uF0GOgaJpZM4VMsjH .  https://github.com/notifications/beacon/AIXm4wo1DEh5nwa5oDucHWLVIo-TU69Yks5uF0GOgaJpZM4VMsjH.gif

It may be something to do with the duplicate of the park boundary that looks to intersect some pathways
https://www.openstreetmap.org/edit?editor=id#map=18/47.91711/106.91830
From: Phil Wyatt<EMAIL_ADDRESS>Sent: Thursday, 12 July 2018 10:51 PM
To: 'openstreetmap/iD'
Subject: RE: [openstreetmap/iD] Hidden roads and paths that are not able to be deleted (#5135)
I removed some spurious points where they converged in the south but I can’t see anything at the northern end of the lines
Points had no tags – may or may not fix the issue
https://www.openstreetmap.org/#map=16/47.9065/106.9217
Cheers - Phil
From: afropoker<EMAIL_ADDRESS>Sent: Thursday, 12 July 2018 10:31 PM
To: openstreetmap/iD
Cc: Phil Wyatt; Comment
Subject: Re: [openstreetmap/iD] Hidden roads and paths that are not able to be deleted (#5135)
https://i.imgur.com/sD9fbS1_d.jpg?maxwidth=640 https://i.imgur.com/sD9fbS1_d.jpg?maxwidth=640&shape=thumb&fidelity=medium &shape=thumb&fidelity=medium
https://i.imgur.com/GqMmwnO_d.jpg?maxwidth=640 https://i.imgur.com/GqMmwnO_d.jpg?maxwidth=640&shape=thumb&fidelity=medium &shape=thumb&fidelity=medium
https://i.imgur.com/OKWtMGE_d.jpg?maxwidth=640 https://i.imgur.com/OKWtMGE_d.jpg?maxwidth=640&shape=thumb&fidelity=medium &shape=thumb&fidelity=medium
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/openstreetmap/iD/issues/5135#issuecomment-404495817 , or mute the thread https://github.com/notifications/unsubscribe-auth/AIXm4-yocRG3onDeRmAEOhNMsvlF6bi8ks5uF0GOgaJpZM4VMsjH .  https://github.com/notifications/beacon/AIXm4wo1DEh5nwa5oDucHWLVIo-TU69Yks5uF0GOgaJpZM4VMsjH.gif

I cant seem to find a way to delete. Seems like a bug to me. Any one can help?

Thanks for fixing @tastrax  -  I don't see any issue here now.
@afropoker, iD doesn't allow users to edit very large features (where you can't see the entire feature).  As a workaround, you can delete individual points until the feature is small enough to work with, or use another editor like level0 or JOSM.

Excuse me. The issue is not fixed!

Excuse me. The issue is not fixed! Please reverse

Can you link to the osm object that you aren't able to delete?  Or share the way id?  I looked at the map around https://www.openstreetmap.org/#map=16/47.9065/106.9217 and nothing really stood out as broken.
Your screenshots were from rendered maps - not actual OSM data as it appears in iD.

https://www.openstreetmap.org/edit?editor=id#map=15/47.9118/106.9230 It seems to be the spurious lines converging into a point. When you zoom in to edit level they disappear
Cheers - Phil
From: Bryan Housel<EMAIL_ADDRESS>Sent: Friday, 13 July 2018 6:40 AM
To: openstreetmap/iD
Cc: Phil Wyatt; Mention
Subject: Re: [openstreetmap/iD] Hidden roads and paths that are not able to be deleted (#5135)
Excuse me. The issue is not fixed! Please reverse
Can you link to the osm object that you aren't able to delete? Or share the way id? I looked at the map around https://www.openstreetmap.org/#map=16/47.9065/106.9217 and nothing really stood out as broken.
Your screenshots were from rendered maps - not actual OSM data as it appears in iD.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/openstreetmap/iD/issues/5135#issuecomment-404643314 , or mute the thread https://github.com/notifications/unsubscribe-auth/AIXm4y5RHzR2SUd5UB-gIDwGOvBqirOWks5uF7QVgaJpZM4VMsjH .  https://github.com/notifications/beacon/AIXm49WMKj3fWQfkcXIIrbwdTiRcBZQbks5uF7QVgaJpZM4VMsjH.gif

Thanks for clarification @tastrax  - The spurious lines are in the raster "locator overlay" layer.
That locator overlay layer that displays at low zooms is not part of the iD project - it is an external layer managed by Mapbox cartography team. It is updated sometimes, but often contains older data.  I don't know how often it is updated, but the important thing is that the real data in OSM has been fixed already.

Is there a way to contact Mapbox team? That line has been there for over 2 year as I always see it on facebook map
Thanks a lot for your help.

https://www.mapbox.com/contact/
From: afropoker<EMAIL_ADDRESS>Sent: Friday, 13 July 2018 3:30 PM
To: openstreetmap/iD
Cc: Phil Wyatt; Mention
Subject: Re: [openstreetmap/iD] Hidden roads and paths that are not able to be deleted (#5135)
Is there a way to contact Mapbox team? That line has been there for over 2 year as I always see it on facebook map
Thanks a lot for your help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/openstreetmap/iD/issues/5135#issuecomment-404729748 , or mute the thread https://github.com/notifications/unsubscribe-auth/AIXm4zNJtX0kWXJtKvBfRkYT2MEqODvbks5uGDBSgaJpZM4VMsjH .  https://github.com/notifications/beacon/AIXm41D65UmpXCG4ux8MFwvGDzsuSSh-ks5uGDBSgaJpZM4VMsjH.gif

Is there a way to contact Mapbox team? That line has been there for over 2 year as I always see it on facebook map
Thanks a lot for your help.

It sounds like @tastrax only fixed this artifact 1-2 days ago.  It takes some time for other rendered maps to catch up.
see https://www.reddit.com/r/openstreetmap/comments/6ynboz/how_long_does_it_take_for_a_edit_to_get_published/
I think @ericfischer may be the person who knows most about how frequently the Locator Overlay map is updated.


New Idea

This app is so good bro please Can u Upload same this type of projects because i love app development

Thank You so much bro for support. I will try my level best to upload Application's Project
👍👍👍


adding managed service account CRD inspection

Signed-off-by: Nathan Weatherly<EMAIL_ADDRESS>https://github.com/stolostron/backlog/issues/19265

/assign @zkayyali812 @JakobGray @cameronmwall

/lgtm


youtube API and watching video

Probably has been asked many times: does the "youtube API" provides the data in order to fetch and play a video file (from the popular channel for instance)?
I could not find anything about this, except a "iframe API" for a fat browser embeded player on google dev support.
Am I wrong or "video streaming" www sites are using the inherent complexity of "open source" fat browsers as a kind of DRM system?

youtube-dl does not use any kind of YouTube API.
Closing as offtopic.


Add a http file reference as project reference

I have added a dll library downloaded as an http reference within a group. The paket.dependencies  file contains:
group Files
http file:///absolute/path/to/library.dll target/path/library.dll

Then in the paket.references file I use:
group Files
	File:library.dll

This is adding a content node in the references ItemGroup in the .csproj, such as:
    <Content Include="..\..\paket-files\files\localhost\target\path\library.dll">
      <Paket>True</Paket>
      <Link>paket-files/library.dll</Link>
    </Content>
    ...
    <Reference Include="System" />
    ...

This is just a link to the source file which I can not use to create a project reference.
Is there any way to add this file as a project reference?

not yet. but I think it's not that hard to do.
let me see if I can hack something together

Thanks. If you give me some clues I might try to make the enhancement.

it would be usefule if you could generate a minimal repro that uses a sample dll (doesn't need to compile - dll might be 0 bytes)

As a couple of tests or a folder with all required files (source file, solution file, project file)?

yes a folder with all required files would be nice. I would then put this into our integration test suite

Here you have. http path must be changed!.
i001341.zip

please test the latest version.
I added your sample repo as integration test and implemented a fix. but Im not completely sure if it works. so please see it as an alpha version that needs further testing.

Starting with a greenfield folder, it succesfully creates a project reference to the dll. But there are some undesirable behaviors:

It copies the dll to a new local folder below the project folder: paket-files/library.dll. This is not needed because the library is already downloaded below the paket-files directory. Only the project reference should be created.
When the reference is deleted in paket.references, it succesfully deletes the reference, but the mentioned new local file paket-files/library.dll is not deleted.
When paket-files/library.dll is found below the project directory, the installation fails. So forced updates of the library will fail.
If we already had the library referenced as a link in a content item, this is not deleted when the new reference is added (minor issue to be documented).

just fixed this. paket-files/library.dll is no longer copied.
regarding docs - could you please take a look at https://github.com/fsprojects/Paket/blob/master/docs/content/http-dependencies.md and send a PR with some docs improvements that you find useful? would help a lot


Do not parse nil usage_percentage

Attempts to fix issue https://github.com/recurly/recurly-client-net/issues/168

+1


built-in SFTP upload

@thanethomson You mentioned built-in SFTP upload in #35 and I also think it would be a cool feature to add. Do you have any plans to implement it?

Yup:

I'd say SFTP would be first priority, and then FTP(S) later (I'd consider them legacy technologies, but some people are forced to use them for various reasons).
Implementing SSH keys and passwords as features are both pretty easy to implement.
Yes - Statik throws an exception if something goes wrong.
No - this is probably going to be the biggest challenge. Might be a good idea to implement this later once the overall SFTP functionality is working.

I submitted a pull request for adding basic SFTP upload functionality: #73

@konstruktiv , this is already completed and can be closed?

Built-in SFTP implementation with added key support: https://github.com/thanethomson/statik/commit/e0f0c5f51ad678c0c214da501f790aec66351e09


remove ignore override from pyproject.toml

          to not block new pull requests, we ignore `override` for now via #911

Originally posted by @SiQube in https://github.com/aeye-lab/pymovements/issues/905#issuecomment-2564840818

the issue is pretty straight-forward. the base class DatasetDefiintion has for example type: experiment: Experiment | None = None. our implementation of the public datasets, e.g. BSC, then uses experiment: Experiment = Experiment(...) which leads to the override error.
this can be resolved via:

#908 (favorite)
specifying empty Experiment() etc. in DatasetDefinition
assigning baseclass types to datasets, e.g. experiment: Experiment | None = Experiment(...)(to make mypy happy but least favorite)


Show Context Menu

I felt this deserves it's own thread especially for those who might want to contribute to it in future. It's a perfect solution and as usual you have done a very good job. Thank you for helping other developers.
I was able to set the "iconBarWidth" of ShowContextMenuA to 0 just to see how it works and it was good. I tried to adjust the size of the "rowHeight" from 44 to something like 15 or 20 but it doesn't seem to be working. I tried expanding that too for ShowContexMenu without any luck. I know this is still under development hence the reason why.

Hi and thank you for your valuable time. RowHeight is fixed now. WIll look into menuWidth later on.
Thanks again


refactor(examples): Cart

Related issue

Closes #
Scope of work

fix font variable name in SfSidebar
adjusted styles to Cart

Screenshots of visual changes

Checklist

[x] No commented blocks of code left
[x] Run tests and docs
[x] Self code-reviewed

@martaradziszewska, please check padding's IMO we should use padding: 24px for all

can we move total items to top and decrease spacing?


Make Settings Specific

Hello!
The GOOGLE_OAUTH_CLIENT_CONFIG and GOOGLE_PICKER_API_KEY settings don't mention this project at all...and in a larger project, we might (and do) have a number of Google keys for various integrations.
I'm wondering if we can name these settings in such a way as to make it clear they are used for this project?
WAGTAIL_CONTENT_IMPORT_GOOGLE_OAUTH_CLIENT_CONFIG is long, but might be the most clear. :)  Same with WAGTAIL_CONTENT_IMPORT_GOOGLE_PICKER_API_KEY.
Thank you!

Hello! This has been changed to the WAGTAIL_CONTENT_IMPORT_ prefix in version 0.3.0, which should hopefully fix this issue.


[BUG] After stopping CRC the Kube context is left in inconsistent state causing timeouts

General information

OS: macOS
Hypervisor: hyperkit
Did you run crc setup before starting it (Yes/No)? Yes
Running CRC on: Laptop

CRC version
CodeReady Containers version: 1.15.0+e317bed
OpenShift version: 4.5.7 (embedded in binary)
CRC status
DEBU CodeReady Containers version: 1.15.0+e317bed
DEBU OpenShift version: 4.5.7 (embedded in binary)
CRC VM:          Stopped
OpenShift:       Stopped
Disk Usage:      0B of 0B (Inside the CRC VM)
Cache Usage:     12.8GB
Cache Directory: /Users/deboer/.crc/cache
CRC config
no output
Host Operating System
ProductName:	Mac OS X
ProductVersion:	10.15.6
BuildVersion:	19G2021
Steps to reproduce

crc start
crc stop
kubectl get pods, odo push, or basically anything that uses the kube context

Expected
If I connect to a remote OpenShift cluster or use other local Kube tools and then disconnect/stop, the Kube context is left pointing to a cluster that I can't connect to anymore, but it 'fails fast': tools that try to connect fail immediately.
e.g. after stopping minikube and running 'kubectl get pods' it immediately responds with:
The connection to the server localhost:8080 was refused - did you specify the right host or port?
I expect CRC to have the same behaviour.
Actual
After stopping CRC the Kube context is left pointing to a cluster (api-crc-testing or api.crc.testing) on a bridge network (192.168.*). For some reason clients can't tell this host doesn't exist anymore and connections to it don't fail fast, which eventually causes timeouts on the client side. This is bad enough with kubectl (20s timeout?), but odo has an even longer timeout (4min?) which makes it unusable and appear to hang.
When stopping CRC please remove the kube context, remove the bridge network, remove the host resolution, or do something similar so that clients can tell it doesn't exist or will fail immediately trying to connect.

When stopping CRC please remove the kube context, remove the bridge network, remove the host resolution, or do something similar so that clients can tell it doesn't exist or will fail immediately trying to connect.

@praveenkumar any idea what causes the response not to reply 'Host unreachable' or 'Connection refused'?
Also, would removing the context be possible?

I tested this on linux, will check on the mac also but I didn't get that much waiting time as described in the issue.
$ oc whoami
kube:admin

$ crc stop
INFO Stopping the OpenShift cluster, this may take a few minutes...
Stopped the OpenShift cluster

$ time oc whoami -v=10
I1007 14:03:42.797261  693344 loader.go:375] Config loaded from file:  /home/prkumar/.kube/config
I1007 14:03:42.798023  693344 round_trippers.go:423] curl -k -v -XGET  -H "Accept: application/json, */*" -H "User-Agent: oc/openshift (linux/amd64) kubernetes/d7f3ccf" -H "Authorization: Bearer oUurQFo7e5xjPoz1h3QPFUGVBLL8tEaXBquoz9oaans" 'https://api.crc.testing:6443/apis/user.openshift.io/v1/users/~'
I1007 14:03:45.905233  693344 round_trippers.go:443] GET https://api.crc.testing:6443/apis/user.openshift.io/v1/users/~  in 3107 milliseconds
I1007 14:03:45.905329  693344 round_trippers.go:449] Response Headers:
I1007 14:03:45.905665  693344 helpers.go:234] Connection error: Get https://api.crc.testing:6443/apis/user.openshift.io/v1/users/~: dial tcp <IP_ADDRESS>:6443: connect: no route to host
F1007 14:03:45.905769  693344 helpers.go:115] Unable to connect to the server: dial tcp <IP_ADDRESS>:6443: connect: no route to host

real	0m3.233s
user	0m0.152s
sys	0m0.038s

$ time odo version -v=9
I1007 14:05:06.924601  693547 preference.go:165] The path for preference file is /home/prkumar/.odo/preference.yaml
I1007 14:05:06.924638  693547 occlient.go:448] Trying to connect to server api.crc.testing:6443
I1007 14:05:07.925073  693547 occlient.go:451] unable to connect to server: dial tcp <IP_ADDRESS>:6443: i/o timeout
odo v1.1.3 (44440eeac)

real	0m1.106s
user	0m0.138s
sys	0m0.038s

What I see is below - when context is to stopped docker-desktop (or any other context) it fails fast. CRC contexts are fine while using it, but timeouts after I stop CRC.
Interestingly enough, if I switch context to Minikube immediately after running CRC I see the same problem - but if I start Minikube and stop it the problem goes away. This leads me to think there is some hyperkit/network cleanup that Minikube is doing but CRC is not.
`deboer-mac:crc-macos-1.15.0-amd64 deboer$ kubectl config use-context docker-desktop
Switched to context "docker-desktop".
deboer-mac:crc-macos-1.15.0-amd64 deboer$ time kubectl get pods
The connection to the server kubernetes.docker.internal:6443 was refused - did you specify the right host or port?
real	0m0.062s
user	0m0.057s
sys	0m0.017s
deboer-mac:crc-macos-1.15.0-amd64 deboer$ ./crc start
...
Started the OpenShift cluster
WARN The cluster might report a degraded or error state. This is expected since several operators have been disabled to lower the resource usage. For more information, please consult the documentation
deboer-mac:crc-macos-1.15.0-amd64 deboer$ kubectl config use-context crc-admin
Switched to context "crc-admin".
deboer-mac:crc-macos-1.15.0-amd64 deboer$ time kubectl get pods
No resources found in default namespace.
real	0m2.165s
user	0m0.145s
sys	0m0.175s
deboer-mac:crc-macos-1.15.0-amd64 deboer$ ./crc stop
Stopping the OpenShift cluster, this may take a few minutes...
Stopped the OpenShift cluster
deboer-mac:crc-macos-1.15.0-amd64 deboer$ time kubectl get pods
Unable to connect to the server: dial tcp <IP_ADDRESS>:6443: i/o timeout
real	0m30.209s
user	0m0.101s
sys	0m0.063s`

Found this bug entry after running into the same issue on my Mac with CRC 1.20.0. Running "kubectl get pods" failed with "Unable to connect to the server: dial tcp <IP_ADDRESS>:6443: i/o timeout" after stopping CRC and logging into another k8s cluster. Thanks to @deboer-tim 's comment above I found I could fix the issue as follows:

Determine current context: kubectl config current-context
This was "sample-app/api-crc-testing:6443/kube:admin" for me.

Get list of current contexts and take note of the one you want to use: kubectl config get-contexts

Switch to that context: kubectl config use-context
Yup, use-context, not set-context which does something different.

After this kubectl get pods again worked as expected.

Found this bug entry after running into the same issue on my Mac with CRC 1.20.0. Running "kubectl get pods" failed with "Unable to connect to the server: dial tcp <IP_ADDRESS>:6443: i/o timeout" after stopping CRC and logging into another k8s cluster. Thanks to @deboer-tim 's comment above I found I could fix the issue as follows:

Determine current context: kubectl config current-context
This was "sample-app/api-crc-testing:6443/kube:admin" for me.

Get list of current contexts and take note of the one you want to use: kubectl config get-contexts

Switch to that context: kubectl config use-context
Yup, use-context, not set-context which does something different.

After this kubectl get pods again worked as expected.

I would like to look into this issue. Could someone please assign it to me?

I would like to look into this issue. Could someone please assign it to me?

Done

I can reproduce this issue. When I also do crc stop and try to access pods using kubectl get pods I get these errors after some wait:
E1010 21:50:00.401494  159438 memcache.go:265] couldn't get current server API group list: Get "https://api.crc.testing:6443/api?timeout=32s": net/http: TLS handshake timeout
E1010 21:50:32.402863  159438 memcache.go:265] couldn't get current server API group list: Get "https://api.crc.testing:6443/api?timeout=32s": context deadline exceeded - error from a previous attempt: read tcp <IP_ADDRESS>:35508-><IP_ADDRESS>:6443: read: connection reset by peer
E1010 21:51:04.403878  159438 memcache.go:265] couldn't get current server API group list: Get "https://api.crc.testing:6443/api?timeout=32s": context deadline exceeded - error from a previous attempt: read tcp <IP_ADDRESS>:54090-><IP_ADDRESS>:6443: read: connection reset by peer
E1010 21:51:36.405070  159438 memcache.go:265] couldn't get current server API group list: Get "https://api.crc.testing:6443/api?timeout=32s": context deadline exceeded - error from a previous attempt: read tcp <IP_ADDRESS>:34104-><IP_ADDRESS>:6443: read: connection reset by peer
E1010 21:52:08.406982  159438 memcache.go:265] couldn't get current server API group list: Get "https://api.crc.testing:6443/api?timeout=32s": context deadline exceeded - error from a previous attempt: read tcp <IP_ADDRESS>:58892-><IP_ADDRESS>:6443: read: connection reset by peer
error: Get "https://api.crc.testing:6443/api?timeout=32s": context deadline exceeded - error from a previous attempt: read tcp <IP_ADDRESS>:58892-><IP_ADDRESS>:6443: read: connection reset by peer

I think this issue is happening because crc is not cleaning up current-context field in ~/.kube/config. Here is my observation for behavior of crc and minikube start/stop commands with kubeconfig:
CRC

current context in kubeconfig after crc startcurrent-context: default/api-crc-testing:6443/kubeadmin

current context in kubeconfig after crc stopcurrent-context: default/api-crc-testing:6443/kubeadmin

Minikube

current context in kubeconfig after minikube startcurrent-context: minikube

current context in kubeconfig after minikube stopcurrent-context: ""

It seems crc does not perform clean up in kubeconfig during crc stop command. I do see code for cleaning up kubeconfig :
https://github.com/crc-org/crc/blob/5611baa4fc9614f838da088fe72f80a369a4fe9d/pkg/crc/machine/kubeconfig.go#L230
It gets invoked in crc delete command here:
https://github.com/crc-org/crc/blob/5611baa4fc9614f838da088fe72f80a369a4fe9d/pkg/crc/machine/delete.go#L38
When I compare it with minikube, minikube seems to be cleaning up kubeconfig in case of both stop and delete commands:

<EMAIL_ADDRESS><EMAIL_ADDRESS>
I see these two ways to solve this issue:

Make the behavior of crc consistent with minikube, also invoke cleanKubeconfig method while stopping cluster.
While stopping the cluster, only set current-context field in kubeconfig to "". Keep Clusters, AuthInfos and Contexts inside the kubeconfig.

I see these two ways to solve this issue:
* Make the behavior of `crc` consistent with `minikube`, also invoke `cleanKubeconfig` method while stopping cluster.

* While stopping the cluster, only set `current-context` field in kubeconfig to `""`. Keep `Clusters`, `AuthInfos` and `Contexts` inside the kubeconfig.

If it's easy to regenerate Clusters, AuthInfos and  Contexts on cluster start, we can go with the first option and remove everything, especially if the code for that already exists.

I have made the changes (https://github.com/rohankanojia-forks/crc/commit/473485b47f94262cac9e1004e65a54ec163a0633) but I'm seeing a strange behavior (not sure if it's due to my code changes or whether I'm testing it incorrectly)
When I do crc start after crc stop (that has cleaned up kubeconfig), I get this error:
# Stop CRC cluster
/home/rokumar/go/src/github.com/crc-org/crc/out/linux-amd64/crc stop
INFO Stopping the instance, this may take a few minutes...
Stopped the instance

# Check kube config
~ : $ cat .kube/config
apiVersion: v1
clusters: null
contexts: null
current-context: ""
kind: Config
preferences: {}
users: null
# Start cluster again
~ : $ /home/rokumar/go/src/github.com/crc-org/crc/out/linux-amd64/crc start
WARN A new version (2.42.0) has been published on https://developers.redhat.com/content-gateway/file/pub/openshift-v4/clients/crc/2.42.0/crc-linux-amd64.tar.xz
INFO Using bundle path /home/rokumar/.crc/cache/crc_okd_libvirt_4.15.0-0.okd-2024-02-23-163410_amd64.crcbundle
INFO Checking if running as non-root
INFO Checking if running inside WSL2
INFO Checking if crc-admin-helper executable is cached
INFO Checking if running on a supported CPU architecture
INFO Checking if crc executable symlink exists
INFO Checking minimum RAM requirements
INFO Check if Podman binary exists in: /home/rokumar/.crc/bin/oc
INFO Checking if Virtualization is enabled
INFO Checking if KVM is enabled
INFO Checking if libvirt is installed
INFO Checking if user is part of libvirt group
INFO Checking if active user/process is currently part of the libvirt group
INFO Checking if libvirt daemon is running
INFO Checking if a supported libvirt version is installed
INFO Checking if crc-driver-libvirt is installed
INFO Checking crc daemon systemd socket units
INFO Checking if vsock is correctly configured
WARN Preflight checks failed during `crc start`, please try to run `crc setup` first in case you haven't done so yet
capabilities are not correct for /home/rokumar/go/src/github.com/crc-org/crc/out/linux-amd64/crc

When using regular crc I'm able to start cluster successfully .


Older snapshots/minor versions not listed (+how to find them)

Right now, it seems like version_manifest.json is used to get a list of all versions.  However, that only includes the latest versions, the latest revisions of some older versions, and the latest snapshots.
It can be pretty useful to compare the first snapshot of a version with the version before it, so I think those versions should be listed.
Now, the hard part is finding the versions.  The good news is that you can use the Minecraft.Download urls to get them; the bad news is that there isn't any good index of those URLs.  Additionally, I believe mojang has said that they prefer that people download jars from launchermeta URLs since the s3 ones are more expensive or something.
While I don't have any good recommendations for the index (other than maybe storing a list of all snapshots from the wiki, but that'd need to be updated), I can say that there's a way to get the snapshots from launchermeta if you've got their name.

Request the JSON from Minecraft.Download, such as http://s3.amazonaws.com/Minecraft.Download/versions/13w16a/13w16a.json.
Use the downloads block in that version JSON (which should be the same as the launchermeta json) to download the actual client jar (like you would with a normal version).
Work off of that jar.

I think that the JSONs, since they're comparatively small files, are probably fine to download from Minecraft.Download if s3 is used to actually get the jars.

The reason that we doesn't support < 1.8 is because Burger seems to break for those versions.
We thought it'd get too over-complicated to download an older compatible version of Burger for every old version.
Do you mean having all the snapshots between every version or only the ones in development?
Depending on a wiki wouldn't really be an option, no. The aim of this application is to find changes out before it's released on any wiki.
If we support all the snapshots (above 1.8), it'd probably be better to create another service that indexes all the Minecraft versions (detects new versions from the version_manifest.json) and provide an API for it.

Do you mean having all the snapshots between every version or only the ones in development?

Yea, that's mainly what I mean, but it would be hard to implement.  I wish mojang had some kind of full index; it would make things much easier.

We thought it'd get too over-complicated to download an older compatible version of Burger for every old version.

At some point I hope to make burger handle compatibility for older versions as well as newer versions automatically; it just doesn't do so right now.


[RELEASE-v1.16] Update golang.org/x/crypto to 0.31.0

What this PR does / why we need it:
See discussion here.
Which issue(s) this PR fixes:
JIRA:
Does this PR needs for other branches:

Does this PR (patch) needs to update/drop in the future?:

JIRA:

/lgtm
/approve


Error when SpecialOnlyFunction is given argument annotations

This means that patterns like the following:
loc:
goto loc

// -----

label "hello"

...are now forbidden, due to causing potential wreckage and UB-on-runtime with Tickompiler's argument annotation system.
Affected commands: goto, label, random, case, unrest
https://github.com/rhmodding/Tickompiler/blob/398d0985795753d333954c97079aeef9e9dc1b8d/src/main/kotlin/rhmodding/tickompiler/Functions.kt#L76-L80
As a side, makes ./gradlew executable for convenience of every *nix-based dev here

just gonna drop your technical explanation here for context/future proofing
Tickompiler has a special definition, SpecialOnlyFunction, for commands that
take only an arg0 (special argument, like in 0x100<X>). It also uses something
that the source code calls "argument annotations" to indicate which values in
the code will turn into string/sub pointers when packed. However, these
argument annotations are only implemented for regular arguments, not arg0,
since the arg0 can't take pointers at all. So what happens is, Tickompiler
loads the file-relative offset into the loc and places an argument annotation
indicating a Tickflow pointer... for the first argument. Which doesn't exist,
since it's only arg0. This results in a crash when decompiling (since it's
trying to get the first argument of a list of 0) and what most likely turns
into bugs and undefined behavior when packing, since it adds at least 0xC000000
to the operation (which would add 0x3000 to the arg0).

This kind of error can also happen with commands such as rest or random,
however, since it makes no sense to put a sub there, this situation hasn't
happened yet.

Tickompiler has no special handling code for goto sub, so any kind of
functioning behavior relies on undocumented Megamix code and pure luck.


Jupyter/Binder URL is broken in "Introduction to C#" tutorials page

Hello, everyone.
Jupyter/Bind tutorial's URL is broken due to GitHub's change in branch naming convention, from "master" to "main".
Description
Current broken URL: https://mybinder.org/v2/gh/dotnet/try-samples/master?filepath=hello-csharp%2Fhello-world.ipynb
Solution
Actual functioning URL: https://mybinder.org/v2/gh/dotnet/try-samples/main?filepath=hello-csharp%2Fhello-world.ipynb

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

ID: b8cdfbba-c2bd-f9b0-52a9-d82cc9b2dcf9
Version Independent ID: 287733f4-7593-ba5e-f26c-531fde328319
Content: Introduction to C# - interactive tutorials
Content Source: docs/csharp/tour-of-csharp/tutorials/index.md
Product: dotnet-csharp
Technology: csharp-get-started
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for pointing this out @vmussa
I see that @Youssef1313 has already submitted a fix. I'll review that and merge it today.


[HOLD for payment 2024-07-10] [HOLD for payment 2024-07-02] LOW: [$500] Stop using reportAction.originalMessage or reportAction.message.text

Problem:
Each comment in the server database is stored in reportAction.message in HTML form.  However, for some reason lost to time, when we send that same comment out as an Onyx update, we:

Rename the original message in the database to reportAction.originalMessage in the update (which is confusing)
Create a new message object that contains a copy of the html message, as well as a stripped text version.
Put the message object inside a totally unnecessary one-element array

That means every comment is actually sent out three times (four, if you include the update to the report object, but that's out of scope for now).  This is clearly wasteful in a number of ways:

It requires postprocessing on the server to do this extraneous operation
It increases network bandwidth by 3x
It increases RAM use 3x
It increases on-device storage 3x
It's confusing AF

Additionally, it's particularly problematic when sending Onyx updates out via UrbanAirship, which is extremely limited in the payload sizes allowed -- with anything over a certain limit just dropped quietly and never delivered, causing "gaps" in our update stream (which require more network calls to "backfill" the Onyx data on app open, which makes things slow).  Basically, this was just a mistake introduced for reasons we can't remember, but that we want to undo.
Solution:
To solve this, please:

Phase 1 (External / clientside): Stop using originalMessage, start using array-message

Update every instance of originalMessage.html to use message.html || message[0].html
Update every instance of message.text to dynamically strip the text from message.html || message[0].html
Update every key within originalMessage to use the original message

Phase 2 (Internal / serverside): Once we've confirmed the client no longer uses originalMessage or message.text or any other key within originalMessage:

Stop sending originalMessage
Start sending message without an array

Phase 3 (External / clientside): Remove checking for "array message"

This issue is for phase 1 and phase 3.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01856d1399295ef760
Upwork Job ID:<PHONE_NUMBER>881405440
Last Price Increase: 2024-07-12
Automatic offers:

cubuspl42 | Reviewer | 0

Issue OwnerCurrent Issue Owner: @puneetlath

Hey I’m going to investigate why the payload is so big in Onyx storage. Possibly track what change exactly caused the problem. Then either revert that piece of change or use one of suggestions by @thienlnam

Hey I’m going to investigate why the payload is so big in Onyx storage. Possibly track what change exactly caused the problem. Then either revert that piece of change or use one of suggestions by @thienlnam
Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01fb7b172c01252917

@ShridharGoel

Similarly, check all usages and update them

Are you saying to literally repeat this code snippet in all places? Couldn't we refactor the code to avoid this repetition?
As the scope of the issue is well-defined and rather narrow, I believe this part of the plan can be a part of the proposal.
@nkdengineer
Aren't you quoting browser-specific local notifications code? 🤔 As I understand it, the ultimate goal is to optimize the payload size of remote push notifications.

Aren't you quoting browser-specific local notifications code? 🤔 As I understand it, the ultimate goal is to optimize the payload size of remote push notifications.

@cubuspl42 In the OP it's mentioned

@thienlnam thinks we added text for notifications

So I'm quoting the place that we're showing notification based on text.
It's only 1 part of the proposal though, what do you think about the rest of my proposal?

Are you saying to literally repeat this code snippet in all places? Couldn't we refactor the code to avoid this repetition?

@cubuspl42 My alternative solution achieves this

@quinthar
Would you summarize the Expensify notifications architecture in a few sentences so that we can be more confident with proposal preparation and review?
As I understand that most (if not all?) notification processing on mobile Native is server-side (push-based), and we're talking about optimizing the pushed payload size. But I'm not 100% confident how that relates to the Onyx schema.
Also, there were some doubts about the scope...

I'm assuming here that we're only omitting the text from reportActions in urban airship native updates, if we're going to remove text in Pusher update too, then we need to populate the text in the Pusher report actions updates by htmlToText.

Would you clarify this too? My intuition was that we wanted to nuke all usage of the text property client-side, also removing it from the typing. But it would be good to be sure about it.

Updated the proposal again (sorry!!!) in light of message being in this weird one-element array.  Thanks for your understanding and help in cleaning up this mess!

Proposal updated after latest requirements update

@nkdengineer I think your proposal shows more preparation. How is your capacity, though? Would you be able to start working on this within 24-48 hours?

Would you be able to start working on this within 24-48 hours?

@cubuspl42 Yes, I definitely can

@thienlnam Hi! I see this is a HIGH priority issue so it should start at $500 (from the announcement here)

While bugs/feature requests will now start at $250, any that are deemed Critical or High to our roadmap (noted via the label High Priority) will automatically be updated to $500. This is to ensure we get the necessary attention and urgency applied to these issues.

Perhaps the automation did not work well in this case.
If that's right, could you help to adjust the bounty here, thanks 🙏

Hm, it's likely because the issue is missing the "High priority" label. But I'm not sure if the "HIGH:" prefix and the "High priority" label represent the same concept 🤔

We already have a draft PR here to fix things in phase 1, @thienlnam please let me know when phase 2 is completed on the server side, I will then update phase 3 accordingly.
Cc @cubuspl42

Great work!  What's your ETA for submitting the PR for review?

I will open PR tomorrow

@nkdengineer Just so we're on the same page, phase 1 must be deployed to production first. Also, some time has to pass, so Native users have the time to update. So we're not blocked by phase 2 in any way.

@cubuspl42 The PR is here.

Great!  @cubuspl42 what is your ETA for review?

I'm on it 🙂

Great, what is your ETA for review?

It was yesterday. I raised concerns about whether we actually covered the whole scope of the Phase 1. I even pinged you 🙂 We can continue the dialogue on the PR.

@parasharrajat reassigning, please take over as C+. If you don't have bandwidth, unassign yourself. Thanks.  PR is

https://github.com/Expensify/App/pull/40168

Thanks @parasharrajat , 👋 @c3024
@c3024 commented on the PR earlier today

@thienlnam @quinthar
The issue talks about only text and html

Phase 1 (External / clientside): Stop using originalMessage, start using array-message
Update every instance of originalMessage.html to use message.html || message[0].html
Update every instance of message.text to dynamically strip the text from message.html || message[0].html

but originalMessage has many more fields like reason, resolution, IOUTransactionID, events, oldName, newName, participantAccountIDs, etc. which are not present in message. So this phase 1 should be to update every instance of originalMessage (not just fields of html or text) with array message
Also in phase 2

Phase 2 (Internal / serverside): Once we've confirmed the client no longer uses originalMessage or message.text:
Stop sending originalMessage
Start sending message without an array

message should be sent including all the above mentioned other fields to completely deprecate originalMessage.
Can you please update phase 1 and 2 of the issue description to include these?

Added Bug to assign a BZ. @quinthar @thienlnam , can you address @c3024 's request above when you have a min
https://github.com/Expensify/App/issues/39797#issuecomment-2130753109

I've updated the description

I did some exploratory work on this topic in https://github.com/Expensify/App/pull/41872 - maybe useful for you @nkdengineer @c3024 @thienlnam

self-assigning as I've been pulled into the PR for review

@nkdengineer @c3024 how is this going? Are we almost done?

Is this being worked on?

Still in review - pending a review from @roryabraham. Dropping my assignment since I'm not adding much value as a reviewer here

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[x]  [@c3024] The PR that introduced the bug has been identified. Link to the PR: This is an improvement and not a bug.
[x]  [@c3024] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: NA
[x]  [@c3024] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: No since this is not a bug.
[x]  [@c3024] Determine if we should create a regression test for this bug. No, this touches several parts of the repo. There are individual regression tests for individual cases.
[x]  [@c3024] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again. NA

bump on the checklist so we can pay you.

Done

Can you also link me your Upwork profile so I can send you a job?

Here it is - https://www.upwork.com/freelancers/~0105555e2f227dbf47
Can the pay be bumped up here? This is a large PR touching many files and there were some big revisions as well. Thanks!

@nkdengineer please accept offer here: https://www.upwork.com/nx/wm/offer/103096930
@c3024 please accept offer here: https://www.upwork.com/nx/wm/offer/103096934
Both of you, please ping me on this issue when you've accepted.

@puneetlath thanks, accepted!

@c3024 has been paid. Just waiting for you now @nkdengineer.

@puneetlath Accepted thanks 🙇

All paid. Thanks everyone!


chore: configure CI

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@9ee5cac). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main        #4   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?        10
  Lines           ?       509
  Branches        ?       104
========================================
  Hits            ?       509
  Misses          ?         0
  Partials        ?         0

Flag
Coverage Δ

unittest
100.00% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Changed initialization of AppPrefs to not be so lazy

Looks good! approved.  Eric will have to merge

Thanks for the feedback guys.  I added back the original method and marked as deprecated.  Also removed synchronized from the new method.


This overload of add_ is deprecated

D:\Bert-Chinese-Text-Classification-Pytorch\pytorch_pretrained\optimization.py:275: UserWarning: This overload of add_ is deprecated:
add_(Number alpha, Tensor other)
Consider using one of the following signatures instead:
add_(Tensor other, *, Number alpha) (Triggered internally at  ..\torch\csrc\utils\python_arg_parser.cpp:1025.)
next_m.mul_(beta1).add_(1 - beta1, grad)

pytorch版本不对


yasm segment selectors

segment selectors don't round-trip edit/assemble through yasm currently. Need to adjust asmx86 output to match yasm input.

Fixed in 1.1.1063


sql: enable clearing user password

PR: https://github.com/cockroachdb/cockroach/pull/43892
From release notes:

CREATE USER and ALTER USER now accept the parameter [WITH] PASSWORD NULL to indicate the user's password should be removed, thus preventing them from using password authentication. This is compatible with PostgreSQL. [#43892][#43892] {% comment %}doc{% endcomment %}
Previously, when setting a user's password to enable password authentication for the user, it was not possible to revert this choice. The only way to disable password authentication was to either drop the user or add a specific per-user HBA rule. This has been fixed and the PostgreSQL-compatible ALTER USER WITH PASSWORD NULL statement can now be used to clear the user's password. [#43892][#43892] {% comment %}doc{% endcomment %}

As discussed with Raphael and Richard on Slack, there isn't any use case for documenting CREATE USER WITH PASSWORD NULL since it is already the default value when a password is not specified. We decided to document the ALTER USER use-case only.

Addressed in https://github.com/cockroachdb/docs/pull/7061


"Add as Contact" buttons don't work

P1, virtual
On the List Checkins page https://crisischeckin-d.azurewebsites.net/Volunteer/ListbyDisaster, after a list of checkins has been displayed, when an “Add as Contact” button is clicked, the page is cleared and the Disaster and “Filter by Date” fields are redisplayed.  Seems to have nothing to do with adding a contact.
Note:  I realize it could simply not be ready, but I didn't find documentation which lets me know that.  Hopefully I didn't overlook it!  :-)

If I log in as Administrator, I actually get a Null Reference exception when I click this button.
To reproduce, go to "View Checkins," select "Hurricane" and click Go, under Volunteer Name click on Add as Contact button under ", Bob."

What is it supposed to do ?

I'm not certain.  I was invited to do some testing as part of a QA
conference I was attending, and haven't seen detailed requirements or
acceptance criteria.  I only remember that nothing happened when I used the
buttons.
Mike Tull
Sr. Quality Assurance Technician, Classy Llama
www.classyllama.com
On Wed, Jun 15, 2016 at 3:16 AM, kevin tshimanga<EMAIL_ADDRESS>wrote:

What is it supposed to do ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/HTBox/crisischeckin/issues/631#issuecomment-226119143,
or mute the thread
https://github.com/notifications/unsubscribe/APir0kSd50WBQrSB7BARqDNTjtzfk1qXks5qL7TRgaJpZM4IYZiS
.

@NaveenP94 has details but in affect it will add that user as a contact to your organization.  The work is in flux so I am guessing two things happen:
for the null error, it is because the administrator is not part of an organization so code is not checking for that
for the refresh, this is a UX issue as it actually adds it as a contact to the organization but the current page loses the search context on 'refresh' and resets its view instead of showing the list of contacts with your organization.
I believe there is an issue to create separate tabs/views for contacts, resources, etc. and what this should do is redirect to the contacts view where the new one would then be seen


Fixed URL for FilterLists under Security APIs table

Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[X] Your submissions are formatted according to the guidelines in the contributing guide
[X] Your additions are ordered alphabetically
[X] Your submission has a useful description
[X] The description does not end with punctuation
[X] Each table column should be padded with one space on either side
[X] You have searched the repository for any relevant issues or pull requests
[X] Any category you are creating has the minimum requirement of 3 items
[X] All changes have been squashed into a single commit

Thanks for contribution, but this URL is broken:
https://filterlists.com/api/v1/lists


setuptools error on upgrading to 1.4.1

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): NO
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS Sierra
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.4.0
Python version: 3.6 (normal install, NO virtualenv, NO anaconda, ...)
Exact command to reproduce: sudo pip3 install --upgrade tensorflow

When upgrading TF 1.4.0 to 1.4.1 using the command above, I get the error message:
Could not import setuptools which is required to install from a source distribution.
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/pip/req/req_install.py", line 387, in setup_py
    import setuptools  # noqa
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/setuptools/__init__.py", line 10, in <module>
    from setuptools.extern.six.moves import filter, map
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/setuptools/extern/__init__.py", line 1, in <module>
    from pkg_resources.extern import VendorImporter
ModuleNotFoundError: No module named 'pkg_resources.extern'

I have been able to solve the problem with the following command:
pip3 install -U setuptools

After this, running pip3 install --upgrade Tensorflow runs successfully.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Bazel version
CUDA/cuDNN version
GPU model and memory

Bazel version: irrelevant, not building from source
CUDA/cuDNN version: irrelevant, not using GPU
GPU model and memory: irrelevant, not using

The error message seems to be during the installation of tensorflow?
Unless @yifeif sees something else in the error, I think this is a pip/setuptools bug, not a tensorflow one as the command the error is seen is a pip command.

It sounds as if it may make sense to report this to pip/setuptools. Closing here, but please reopen if you think this is a TensorFlow bug.


command "CUDA_VISIBLE_DEVICE" no avaliable

The command CUDA_VISIBLE_DEVICE is simply not available at my prompt, it is probably missing to install a package now which one?
Someone can help me?

It is not command it is environment variable. You may control which gpu to use with --devide_ids , so in principle you don't need this.


feat: add table of modules for a libraries-bom version to README

Follow up from https://github.com/googleapis/java-cloud-bom/pull/6069
Publishing the table of included artifacts for a libraries-bom to the README

@alicejli Have you tried using the Maven flatten plugin?
Add the following configuration to libraries-bom/pom.xml:
      <plugin>
        <groupId>org.codehaus.mojo</groupId>
        <artifactId>flatten-maven-plugin</artifactId>
        <configuration>
          <updatePomFile>true</updatePomFile>
          <pomElements>
            <dependencyManagement>expand</dependencyManagement>
          </pomElements>
        </configuration>
      </plugin>

Then run mvn flatten:flatten. You should get the .flattened-pom.xml that should list all of the artifacts and versions contained in the lbraries-bom.

@alicejli Have you tried using the Maven flatten plugin? Add the following configuration to libraries-bom/pom.xml:
      <plugin>
        <groupId>org.codehaus.mojo</groupId>
        <artifactId>flatten-maven-plugin</artifactId>
        <configuration>
          <updatePomFile>true</updatePomFile>
          <pomElements>
            <dependencyManagement>expand</dependencyManagement>
          </pomElements>
        </configuration>
      </plugin>

Then run mvn flatten:flatten. You should get the .flattened-pom.xml that should list all of the artifacts and versions contained in the lbraries-bom.

Ah that's very handy! Thanks for showing. That is much simpler.
Are there any concerns with adding this configuration to the libraries-bom/pom.xml? I don't think I need the  <updatePomFile>true</updatePomFile> line for it to work.
It seems like if I have:

A github actions workflow that runs mvn flatten:flatten
a script that parses the .flattened-pom.xml and updates the README, and discards the created .flattened-pom.xml files.

that would be a neat way to create the table.

@meltsufin I've updated the scripts to use the maven flatten plugin which reduces a lot of the complexity - thanks for the idea!
cc @burkedavison as we chatted about this a bit this morning
The main challenge that I'm curious to get your thoughts on how to solve is getting the latest/updated client documentation/product reference links.
Those links are contained within each module's .repo-metadata.json file, and the previous iteration of this PR was grabbing those links when checking out each repo.
The .repo-metadata.json files are not updated frequently (if ever), so it seems a bit much to re-introduce checking out the repos for just those links. The main thing that would get out of date is whenever a new client library is generated.
The most ongoing toilsome method (which I don't think would be too toilsome given that we don't generate new client libraries that frequently) would be to do nothing and have it as a documented step as part of new client library generation to manually update the links for the product/client reference in this repo.
Do you have any thoughts on the best approach to keeping those links updated?

have it as a documented step as part of new client library generation to manually update the links for the product/client reference in this repo

Despite updates to .repo-metadata.json being rare, they do happen; and solving this situation by manual-process would set up a second source of truth for that data with inevitable desyncing issues that must be discovered each time before we can fix them. (One example source of concern: We don't review changes made to handwritten repos, so we'd be trusting that every Java SDK developer knows about this table's implementation needs.)
This may be a departure from the current direction we've been discussing; but I'd rather see logic that fetches the source of truth, and regenerates the client-library portion of the table from scratch each time rather than in-place updating. In the end, I think the goal should be that this table doesn't need any special handling when adding, removing, or updating client libraries.

The main challenge that I'm curious to get your thoughts on how to solve is getting the latest/updated client documentation/product reference links. Those links are contained within each module's .repo-metadata.json file, and the previous iteration of this PR was grabbing those links when checking out each repo. The .repo-metadata.json files are not updated frequently (if ever), so it seems a bit much to re-introduce checking out the repos for just those links. The main thing that would get out of date is whenever a new client library is generated.

We certainly should not create another manually maintained source of truth for library metadata. However, I don't like getting this information from the .repo-metadata.json either. It's seems error-prone. I think the best place to get this information is from the pom.xml metadata. It already supports things like project name and URL. If we are currently not providing this information in the POM, let's update our generation scripts to do that. In the meantime the table can just be missing this information.

The main challenge that I'm curious to get your thoughts on how to solve is getting the latest/updated client documentation/product reference links. Those links are contained within each module's .repo-metadata.json file, and the previous iteration of this PR was grabbing those links when checking out each repo. The .repo-metadata.json files are not updated frequently (if ever), so it seems a bit much to re-introduce checking out the repos for just those links. The main thing that would get out of date is whenever a new client library is generated.

We certainly should not create another manually maintained source of truth for library metadata. However, I don't like getting this information from the .repo-metadata.json either. It's seems error-prone. I think the best place to get this information is from the pom.xml metadata. It already supports things like project name and URL. If we are currently not providing this information in the POM, let's update our generation scripts to do that. In the meantime the table can just be missing this information.

Cool. We don't currently provide the links to the client/product documentation in the POM, but I can create an issue to do so (with a follow-up issue to update this table with the links from the POM once they exist).
I did some testing and I don't believe there's a way to use the maven flatten plugin to get the metadata for the dependencies as well as it only includes the info here: https://maven.apache.org/ref/2.2.1/maven-model/maven.html#class_dependency. So to get the links from the pom.xml metadata, it would involve a script to ping Maven Central. This is all doable, but just want to point out that I don't think we can create the full table from just using the flatten plugin.
One question before I update this PR - @meltsufin would it be okay to keep the current links? I feel like the table is much more useful with them and even if they get stale/new library links won't exist, I think it's better to have them than not. I can add a footnote to the table saying links may be out of date in the meantime.

@alicejli Correct, the flatten plugin is not going to give us that metadata from the POMs and we would have to fetch them from Maven Central, which I still think is more robust than the .repo-metadata.json.
Regarding the links, I assume you're talking about the product links, I'm OK with keeping them temporarily.

@alicejli Correct, the flatten plugin is not going to give us that metadata from the POMs and we would have to fetch them from Maven Central, which I still think is more robust than the .repo-metadata.json.
Regarding the links, I assume you're talking about the product links, I'm OK with keeping them temporarily.

Yep, I agree on the point about getting the metadata from Maven Central. And yes, I'm talking about the links to the product documentation as well as the Cloud RAD client reference info. I will update the PR accordingly and then re-request review.

Yep, I agree on the point about getting the metadata from Maven Central. And yes, I'm talking about the links to the product documentation as well as the Cloud RAD client reference info. I will update the PR accordingly and then re-request review.

Can Cloud RAD links be automatically calculated on the fly?

Yep, I agree on the point about getting the metadata from Maven Central. And yes, I'm talking about the links to the product documentation as well as the Cloud RAD client reference info. I will update the PR accordingly and then re-request review.

Can Cloud RAD links be automatically calculated on the fly?

Good point. Will auto-generate those.


lib9: no support for non-ascii chars in path on Windows

multibyte characters in Windows path is not encoded in utf8, but all our command
line programs implicitly assume utf8 encoding of path by using lib9/fmt
(even smprint("%s", path) can't correctly handle multibyte paths).

the worst case is: if your user name contains multibyte characters, then you won't be
able to use cmd/go at all, because your os.TempDir() contains multibyte characters;
and you don't have any easy workarounds (os.TempDir() uses syscall.GetTempPath,
and you can't override result of GetTempPath easily). Could we make os.TempDir()
look for %TEMP% or %TMP% first before calling GetTempPath() so that the user at least
could change the temporary directory to workaround the aforementioned problem?

Fixed by the transition of toolchain components to Go in Go 1.5.


Language Server uses 100% CPU and hangs indefinitely (12+ hours) on large project

Environment data

VS Code version: 1.41.1
Extension version (available under the Extensions sidebar): 2020.1.58038
OS and version: Reproduced on both Ubuntu 18.04.3 LTS and macOS 10.15.3
Python version (& distribution if applicable, e.g. Anaconda): 3.6.9, 3.7.6 and 2.7.17
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): none, venv and virtualenv
Relevant/affected Python packages and their versions: any
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info microsoft/vscode-python#3977): Language Server

Expected behaviour
When opening a Python file, the python language server should start, analyze files, and provide intellisense, navigation, autocomplete and so on.
Actual behaviour
When opening a python file, the language server starts, but does not appear to analyze any files (output log prints a list of search paths, and then nothing else) and sits using 100% CPU usage indefinitely (left running overnight so at least 12 hours of sitting at 100% with no output)
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Open a python file. The project is a mixed-language project which I suspect may be causing the issue, it contains a src directory with several standalone bash and python scripts and then several directories containing C code with the majority of actual python source inside src/python/<package name>. The python extension appears to detect src as the search path (from the language server output panel) so it might be getting stuck looking over a bunch of non-python files? There appears to be no way to change this behavior, adding src/python to python.autocomplete.extraPaths or PYTHONPATH in .env adds src/python to the search paths, but does not remove src.

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
Python
User belongs to experiment group 'ShowPlayIcon - start'
User belongs to experiment group 'ShowExtensionSurveyPrompt - control'
User belongs to experiment group 'DebugAdapterFactory - experiment'
User belongs to experiment group 'UseTerminalToGetActivatedEnvVars - control'
User belongs to experiment group 'WebHostNotebook - control'
> conda --version
> pyenv root
> python3.7 -c "import sys;print(sys.executable)"
> python3.6 -c "import sys;print(sys.executable)"
> python3 -c "import sys;print(sys.executable)"
> python2 -c "import sys;print(sys.executable)"
> python -c "import sys;print(sys.executable)"
> /usr/local/bin/python -c "import sys;print(sys.executable)"
> conda info --json
> ~/.pyenv/shims/python3.7 ~/.vscode-server/extensions/ms-python.python-2020.1.58038/pythonFiles/interpreterInfo.py
> ~/.pyenv/shims/python3.7 ~/.vscode-server/extensions/ms-python.python-2020.1.58038/pythonFiles/interpreterInfo.py
> ~/.local/bin/flake8 --format=%(row)d,%(col)d,%(code).1s,%(code)s:%(text)s ~/appliance/src/python/<package>/__init__.py
cwd: ~/appliance
Starting Microsoft Python language server.

Python Language Server
[Info  - 12:18:07 PM] Analysis cache path: /home/<user>/.cache/Microsoft/Python Language Server
[Info  - 12:18:07 PM] Microsoft Python Language Server version <IP_ADDRESS>
[Info  - 12:18:07 PM] Workspace root: /home/<user>/appliance
[Info  - 12:18:08 PM] GetCurrentSearchPaths /usr/local/bin/python
[Info  - 12:18:08 PM] Interpreter search paths:
[Info  - 12:18:08 PM]     /usr/lib/python3.6
[Info  - 12:18:08 PM]     /usr/lib/python3.6/lib-dynload
[Info  - 12:18:08 PM]     /home/<user>/.local/lib/python3.6/site-packages
[Info  - 12:18:08 PM]     /usr/local/lib/python3.6/dist-packages
[Info  - 12:18:08 PM]     /usr/lib/python3/dist-packages
[Info  - 12:18:08 PM] User search paths:
[Info  - 12:18:08 PM]     /home/<user>/appliance/src/python
[Info  - 12:18:08 PM]     /home/<user>/appliance/src

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
https://gist.github.com/rainyday/f1602f84b4a87d9de6ebcfff21d31a0c

Unlikely, LS does not look into non-Python files. Most probably circular import references are causing trouble or simply there are a lot of imports. Try getting LS log per https://github.com/microsoft/python-language-server/blob/master/TROUBLESHOOTING.md#filing-an-issue.

Setting python.analysis.logLevel to Trace does not provide any additional output from the python language server.

I see the same problem using a late-2020 version VS Code on a mac, all the deets are below.  I use a virtual environment based on Py 3.6.  The 100% cpu problem does NOT happen on all folders. Here are my observations.
VS code works as expected (no 100%) CPU when I open a VS code window on a small folder with all good scripts then open a tab with a Python file.  "Good" here means all have valid imports -- the packages are in my virtual env.
VS code fails (activity monitor shows Microsoft.Python.LanguageServer process at 100% ) if I open a VS code window on a large folder with some bad scripts, then open a tab with a Python file.  "Bad" means some scripts have at least one invalid import -- the packages are not in my virtual env. It does not matter what python file I open first - good or broken.  Unfortunately this is the source folder for an internal company project so I cannot just zip it up and post it here :(
I don't know if broken files and/or missing imports are the sole cause.  Probably not, because I tried creating a bogus-import file in an otherwise pristine folder and that did not trigger the problem.
To get the LanguageServer CPU to stop I have to close the folder with the bad scripts; of course quitting the app entirely works also.
Version: 1.52.1
Commit: ea3859d4ba2f3e577a159bc91e3074c5d85c0523
Date: 2020-12-16T16:30:02.420Z (1 mo ago)
Electron: 9.3.5
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0

I see the same problem using a late-2020 version VS Code on a mac, all the deets are below.  I use a virtual environment based on Py 3.6.  The 100% cpu problem does NOT happen on all folders. Here are my observations.
VS code works as expected (no 100%) CPU when I open a VS code window on a small folder with all good scripts then open a tab with a Python file.  "Good" here means all have valid imports -- the packages are in my virtual env.
VS code fails (activity monitor shows Microsoft.Python.LanguageServer process at 100% ) if I open a VS code window on a large folder with some bad scripts, then open a tab with a Python file.  "Bad" means some scripts have at least one invalid import -- the packages are not in my virtual env. It does not matter what python file I open first - good or broken.  Unfortunately this is the source folder for an internal company project so I cannot just zip it up and post it here :(
I don't know if broken files and/or missing imports are the sole cause.  Probably not, because I tried creating a bogus-import file in an otherwise pristine folder and that did not trigger the problem.
To get the LanguageServer CPU to stop I have to close the folder with the bad scripts; of course quitting the app entirely works also.
Version: 1.52.1
Commit: ea3859d4ba2f3e577a159bc91e3074c5d85c0523
Date: 2020-12-16T16:30:02.420Z (1 mo ago)
Electron: 9.3.5
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0

This project is not being actively maintained. You may want to try out Pylance: https://github.com/microsoft/pylance-release

This project is not being actively maintained. You may want to try out Pylance: https://github.com/microsoft/pylance-release

Thanks @jakebailey but please help me understand.  Did I post to the wrong GitHub project about this problem?  Which VS Code extension is responsible for the process Microsoft.Python.LanguageServer?  In my VS Code the only Python extension I have installed is called just "Python" and it's version 2020.12.424452561.

Thanks @jakebailey but please help me understand.  Did I post to the wrong GitHub project about this problem?  Which VS Code extension is responsible for the process Microsoft.Python.LanguageServer?  In my VS Code the only Python extension I have installed is called just "Python" and it's version 2020.12.424452561.

This is the repo for MPLS, which is that process, yes. The Python extension only runs it (but the code does not live there). MPLS still works for many people and the Python extension can still run it, but we aren't actively advertising it as the best option.

This is the repo for MPLS, which is that process, yes. The Python extension only runs it (but the code does not live there). MPLS still works for many people and the Python extension can still run it, but we aren't actively advertising it as the best option.


Add 'commit-starts-with' optional parameter

An optional parameter has been added to validate that the commit being evaluated starts with a specific string.
If this optional parameter is set, we will check the subject line of the current commit to ensure it matches what was expected. We will replace the string "[version]" in this parameter with the current release version.
The README has been updated to explain how this works.

I am not sure that a check based on a magic string in a commit message or the name of the release branch would be a sufficient solution to the problem presented in https://github.com/MetaMask/metamask-module-template/pull/153. Anyone, contributors included, could create such a commit and create a PR for it, and we could accidentally merge that PR thinking it was innocuous.

It's not a perfect solution. But it seems pretty good for repositories where we're using "Squash & Merge", and creating release branches with create-release-branch. The PR title "Release [version]" is kinda hard to miss.

I tested this, and it seems to work. Here is a negative example: https://github.com/Gudahtt/controllers-fork/actions/runs/3518613740/jobs/5897785814
And here is a positive example: https://github.com/Gudahtt/controllers-fork/actions/runs/3518683757/jobs/5897923800


feat: dividing an element of a euclidean domain by its radical

For a in a EuclideanDomain, define divRadical a as a / radical a

Prove some related theorems.
This is needed in the proof of Mason-Stothers theorem, and also a part of PR #15706.

bors r+


Add loop option to Sampler

It would be nice to be able to have the sources of the sampler looping. That would allow the sampler to be used for instruments like pads, organs etc.
It's a simple change allowing a loop property to be passed in the constructor options and then set that property for each source when triggering notes.

Just came across this limitation as well, thanks for suggesting it. Figured out a good workaround for your pads - if you set the sampler attack and release to a slow enough value and then send it through enough reverb and/or delay then you can trigger the synth with a setInterval function and it sounds endless, here's my code that works well:
const reverb2 = new Tone.Reverb({
    decay: 1,
    wet: 1,
  }).toDestination();

  const filter = new Tone.Filter({
    type: "bandpass",
    frequency: params.cutoff,
    rolloff: -12,
  });

  const reverb = new Tone.Reverb({
    decay: 1,
    wet: 1,
  });

  const sampler = new Tone.Sampler({
    urls: {
      C3: "pad.wav",
    },
    baseUrl: "/",
    volume: 0,
    attack: 4,
    release: 4,

    onload: () => {
      setInterval(() => {
        sampler.triggerAttackRelease(padKeys, Infinity);
      }, 2000);
    },
  });

  sampler.chain(reverb, filter, reverb2);

Don't know how this would sound with an organ so let me know if you figure it out :)


feat(SkipToContent): SkipToContent + SkipToContent target visual identity + emotion

Overview
Consolidates and standardizes usage of SkipToContent and SkipToContent target (Fixes on going z-index issues for elements that must be at specific places in the dom).

Monolith PR: https://github.com/codecademy-engineering/Codecademy/pull/22733

Adds SkipToContentTarget
Adds Arrow Down Mini Bold Icon
Converts SkipToContent to new identity and adds a transition.
Fixes issue with top 100vh in some instances.
Ensures zIndex is above the headers.

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-icons@4.1.1-alpha.4cb0a1.0@codecademy/gamut-kit@0.2.4-alpha.4cb0a1.0@codecademy/gamut-labs@10.5.3-alpha.4cb0a1.0@codecademy/gamut@26.0.2-alpha.4cb0a1.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-icons@4.1.1-alpha.7acac9.0@codecademy/gamut-kit@0.2.4-alpha.7acac9.0@codecademy/gamut-labs@10.5.3-alpha.7acac9.0@codecademy/gamut@26.0.2-alpha.7acac9.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60443b46a5d98f050ca2bead--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@4.1.1-alpha.97ffb3.0@codecademy/gamut-kit@0.2.4-alpha.97ffb3.0@codecademy/gamut-labs@10.5.3-alpha.97ffb3.0@codecademy/gamut@26.0.2-alpha.97ffb3.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60444a73f64f701549737598--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://60469571d5e86500a1930348--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@4.1.1-alpha.895b3e.0@codecademy/gamut-kit@0.2.6-alpha.895b3e.0@codecademy/gamut-labs@11.0.2-alpha.895b3e.0@codecademy/gamut@26.1.1-alpha.895b3e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60469868362236097f8ace7e--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-icons@4.1.1-alpha.3899b5.0@codecademy/gamut-kit@0.2.6-alpha.3899b5.0@codecademy/gamut-labs@11.0.2-alpha.3899b5.0@codecademy/gamut@26.1.1-alpha.3899b5.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6046a28fb8af50103e5f07b2--gamut-preview.netlify.appDeploy Logs


Make scala plugin support the configuration cache

This issue is about making the scala core Gradle plugin support the configuration cache.
Upon completion, all configuration and tasks provided by the scala plugin are compatible with the configuration cache.

See https://github.com/gradle/gradle/pull/17403 and https://github.com/gradle/gradle/pull/17782


reorganize backup, restore, upgrade and migration documents

** Background **
Currently, our documentation has two sections relating to backup, restore, upgrading and migration.  These two documents are confusing, overlapping and not task specific.

https://github.com/pachyderm/pachyderm/blob/master/doc/managing_pachyderm/backup_restore_and_migrate.md
https://github.com/pachyderm/pachyderm/blob/master/doc/deployment/upgrading.md

** Reorganization Proposal **:
Break these documents into four discrete operations documents

Backing up Pachyderm
Restoring Pachyderm from backup
Pachyderm minor version upgrades (upgrades)
Pachyderm major version upgrades (migrations)

I believe this is done. @pappasilenus please reopen if not


Fetch node repo from s3 and untar it in /tmp/node. Use this to install docker on admiral swarm worker, if successfully able to fetch node tarball from s3.

https://github.com/Shippable/pm/issues/10048

this is done, closing

this is done, closing


Changes to GO:0099115 chromosome, subtelomeric region

Discussing with @ValWood @RLovering @colinlog

GO:0099115 chromosome, subtelomeric region-> change definition
OLD A heterochromatic region of the chromosome, adjacent to the telomere (on the centromeric side) that contains repetitive DNA and sometimes genes.
NEW A region of the chromosome, adjacent to the telomere (on the centromeric side) that contains repetitive DNA and sometimes genes. This region is usually heterochromatin.

Merge
'subtelomeric heterochromatin'
'nuclear subtelomeric heterochromatin'

add parent: GO:0099115 chromosome, subtelomeric region

Note comment from Telomere expert:
Yes I would define telomeric region/telomeric DNA as the region of uninterrupted telomeric DNA repeats (ie. TTAGGG in vertebrates).
Basically I agree with almost all highlighted text below, with possible exception of including the phrase ‘heterochromatic region’ in the definition of subtelomeric region – as this could imply that expression of all genes in these regions is permanently supressed?

Need to obsolete:

GO:0005724  nuclear telomeric heterochromatin  - 27 EXP
GO:0031933  telomeric heterochromatin - 5 EXP

Has been inconsistently used. Should annotate to either:

GO:0000781 chromosome, telomeric region
GO:1990421  subtelomeric heterochromatin

I will open an annotation review ticket.
Thanks, Pascale

Dear all,
The proposal has been made to obsolete

GO:0005724 nuclear telomeric heterochromatin - 27 EXP
GO:0031933 telomeric heterochromatin - 5 EXP

The reason for obsoletion is that there is no heterochromatin at the telomere. Annotations should be reviewed from this ticket: https://github.com/geneontology/go-annotation/issues/2883
There are no mappings to those terms; those terms are not present in any subsets.
You can add any comments to the ticket:
https://github.com/geneontology/go-ontology/issues/19086
We are opening a comment period for this proposed obsoletion. Since there are some annotations we’d like to propose to obsolete this term on March 31st, 2020. Unless objections are received by March 31st, 2020, we will assume that you agree to this change.
Thanks, Pascale

See discussion on the annotation ticket https://github.com/geneontology/go-annotation/issues/2883
Call with @colinlog @ValWood @RLovering
Decision is NOT to obsolete, and change the structure on the ontology
Now:
GO:0000781 chromosome, telomeric region and
GO:1990421 subtelomeric heterochromatin
are siblings

We want to move GO:1990421 subtelomeric heterochromatin as a child of
GO:0000781 chromosome, telomeric region
and add a new child, 'chromosome, telomeric repeat region'

narrow synonym: chromosome, telomeric loop region

Added subclasses:

[x] GO:0099115 chromosome, subtelomeric region
Added -> is_a: GO:0000781 ! chromosome, telomeric region

[x] GO:1990707 nuclear subtelomeric heterochromatin
Added ->  is_a: GO:0000784 ! nuclear chromosome, telomeric region

[x]  Changed definition
GO:0000784  name: nuclear chromosome, telomeric region

OLD def: "The terminal region of a linear nuclear chromosome that includes the telomeric DNA repeats and associated proteins." [GOC:elh]
NEW def: "The end of a linear chromosome, required for the integrity and maintenance of the end. A chromosome telomere usually includes a region of telomerase-encoded repeats the length of which rarely exceeds 20 bp each and that permits the formation of a telomeric loop (T-loop). The telomeric repeat region is usually preceded by a sub-telomeric region that is gene-poor but rich in repetitive elements. Some telomeres only consist of the latter part (for eg. D. melanogaster telomeres)." [GOC:elh]

NEW TERM
+id: GO:0140445
+name: chromosome, telomeric repeat region
+namespace: cellular_component
+def: "A complex of DNA and protein that seals the end of a chromosome. The telomeric repeat DNA consists of simple tandemly repeated sequences specific for each species. Typically one strand is G-rich and the other C-rich. The G-rich strand forms a 3'-terminal overhang, the length of which varies with species. The single strand overhang is bound by a variety of proteins, including telomere capping proteins that bind to the single-stranded DNA and seal the telomeric loop." [PMID:11352055, PMID:30208292]
+is_a: GO:0000781 ! chromosome, telomeric region
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/19086 xsd:anyURI
+created_by: pg
+creation_date: 2020-03-27T13:29:16Z

Hi @pgaudet
If you did not already, could you rehouse
telomere cap complex GO:0000782
shelterin complex GO:0070187
CST complex GO:1990879
under the new term GO:0140445  chromosome, telomeric repeat region
tks

If you did not already, could you rehouse
telomere cap complex GO:0000782
shelterin complex GO:0070187
CST complex GO:1990879
under the new term GO:0140445 chromosome, telomeric repeat region

Now done, thanks


System Freezes

My virtual studio box keeps freezing. I am not able to effectively utilize it.

/needsMoreInfo

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Create sql-data-warehouse-security-threat-detection.md

Creating Threat Detection article

Hi @lodipalm, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (lodipalm). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Will complete this in -pr


Establish Code of Conduct

Official Code of Conduct for 1KC

Hello sir, I would like to make Code Of Conduct, plz assign this to me

Hi @rauthanshubham1, thanks for the interest. For now, we are using a generic code of conduct and will revisit in the near future.


Update data-lake-store-data-scenarios.md

I am PM on Azure Data Lake team.

Hi @sachincsheth, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@sachincsheth : Thanks for your contribution to the Azure documentation! The author, @nitinme, has been notified to review your proposed change.

@sachincsheth . Thanks for your contribution. Could you please fix the formatting of the value you added in the table. In the final view, it is being rendered outside the table.

There were issues with this pull request. Created a new one.

Hi @sachincsheth, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


[stable/prometheus-operator] Decouple Grafana dashboards from prometheus-operator chart.

Problem:
Since I am using Grafana as a central place to aggregate all kinds of different datasources, I find deploying Grafana as a sidekick of Prometheus too restrictive. I would, however, like to use the nice Dashboards this chart comes with (https://github.com/helm/charts/tree/master/stable/prometheus-operator/templates/grafana/dashboards) in my own instance of Grafana.
Solution 1:
Add dashboards defined under prometheus-operator chart to Grafana community at: https://grafana.com/grafana/dashboards, this way the dashboards have their own revision history and "gnetId" and are decoupled from this chart. This will alloow me to reference "gnetId" in my Grafana values file.
Solution 2:
Remove the ".Values.grafana.enabled" filter from dashboard ConfigMap templates, and allow ".Values.grafana.defaultDashboardsEnabled" to be enabled without installing THIS Grafana. This will allow me to use the ConfigMap elsewhere.
Alternatives:

Hold my own copy of the json files defined in ConfigMaps, which I can then reference from my Grafana values file.
Add the dashboards to Grafana community myself, which I can then reference with "gnetId".

NB! I would be willing to do this feature myself, if that is appropriate.

Hey; this would be great. Currently the chart generates 35000 lines of code when run against a k8s cluster; so it was pretty hard to do much anything at all. I created some tooling to work with the chart:
https://gist.github.com/haf-afa/575ed9f546037ea99c5b9e97f75b9c83
Some stats from using this:
The template.yaml file that Helm generates contains 129 non-empty files, 31 empty ones.
You can remove almost all things it generates with grafana_templates in the name; and keep all grafana_dashboards.
Then you can use the Grafana Helm chart instead of this to deploy Grafana independently. Have a look at the linked gist.
Hey stale, open up this issue.


SMACC Viewer does not show the currently active state

I'm running SMACC on Ubuntu 20.04 using ROS noetic. I was trying to run the examples, and even though the state machine itself seems to be working fine, I cannot see the currently active state on the viewer.

I have installed SMACC Viewer using apt (sudo apt-get install ros-noetic-smacc-viewer) but I also had to install wx manually using sudo pip install -U wxPython. Please let me know if I can assist with something else.

I encountered the same issue, have a look at issue #41, there a solution is given. Specifically in this comment: https://github.com/reelrbtx/SMACC/issues/41#issuecomment-896988990

Noetic debian files for smacc_viewer are updated now in package-cloud with the proposed fix.
in #41.
Thanks.


Debugging

Is there a way to debug each step in the docs for using keyvault flex volume?
I'm trying to connect keyvault from a different resource group than the AKS cluster. I followed all the steps for AAD Pod Identity, but they're very confusing, especially if you're not super familiar with all the azure concepts. I have a suggestion: specify a command to get every missing piece (clientId, clientSecrets, service principal ids etc...) I spend a whole 15 minutes until I found that directoryId and tenantId are the same thing, so please, please clarify as much as possible where to get each missing piece. On top of that, there's no way for me to know at which step did I mess up. Did all the permission changes go through properly? How can I check?
Lastly, even after following all the steps, I wasn't able to get everything to work: when i run kubectl get events i get the following error:
Applying binding id-binding node aks-agentpool-28658367-1 for pod nginx-default-aks-kv-identity resulted in error compute.VirtualMachinesClient#CreateOrUpdate: Failure sending request: StatusCode=403 -- Original Error: Code="LinkedAuthorizationFailed" Message="The client '*****' with object id '*****' has permission to perform action 'Microsoft.Compute/virtualMachines/write' on scope '/subscriptions/******/resourceGroups/MC_****_*****_centralus/providers/Microsoft.Compute/virtualMachines/aks-agentpool-28658367-1'; however, it does not have permission to perform action 'Microsoft.ManagedIdentity/userAssignedIdentities/assign/action' on the linked scope(s) '/subscriptions/*********/resourcegroups/********/providers/Microsoft.ManagedIdentity/userAssignedIdentities/aks-kv-identity'."

@oxhagolli Thanks for the feedback! Will definitely try to make the instructions more clear.
Regarding the error you got, it seems the service principal used by the AKS cluster does not have permission to assign the new identity to the VMs. Please try the following command:
az role assignment create --role "Managed Identity Operator" --assignee <sp id> --scope <full id of the managed identity>

This step is here for future reference: https://github.com/Azure/kubernetes-keyvault-flexvol#option-2-pod-identity


Extension - Add validate account in case sign transaction with Ledger account

🚀 Deployed on https://pr-3278--sw-web-runner.netlify.app


Specflow NUnit "no test matches the given testcase filter"

Hi,
After the changes in 89db567e4dea2fab75c8cf0803a5ee4200660c0f (next commit after 1.2.0 label) the auto-generated specflow tests stop working.
The issue seems to be that the test class name gets duplicated in "FullyQualifiedName" search.
I have tried to pinpoint actual problem so I could come up with an solution but I was not able to do that.
My guess is that the treesitter query in DotnetNeotestAdapter.discover_positions is the one that double class name.
I have attached an small nunit test project where I have this issue.
Expected FullyQualifiedName: dummy_test.Features.DummyTestFeature.DummyScenario
But it is: dummy_test.Features.DummyTestFeature.DummyTestFeature.DummyScenario
dummy_test.zip

I did some more tests.
it seems that the problem is with the added treesitter query in nunit-queries.lua:
    ;; Matches test classes (no TestFixture attribute)     (class_declaration       name: (identifier) @namespace.name     ) @namespace.definition
I created this small test class:
[TestFixture] public class UnitTester {     [Test]     public void DoTest()     {         Assert.IsTrue(true);     } }
If I remove [TestFixture] attribute on the class I will get the correct "fullyQualifiedName" but with it I get the same issue as mention before (class name is doubled)
I'm not sure if it is ok to just remove the treefilter above for nunit specific or if that would break something else?

Thanks for raising the issue @teast , and for your proposed solution. I've noticed a couple of things wrong with the NUnit specflow files that need sorting out also.
There is one hiccup with your solution, given that the TestFixture attribute isn't required in later version of NUnit, so I'll have a play around and see if there is another solution that will catch both Nunit cases correctly, with or without the TestFixture in place.

@teast - Hopefully this fixes the issue. Let me know if it hasn't.

@Issafalcon It seems to have solved it. I tested it out in a solution that has both specflow and unit tests with nunit and all tests seems to be picked up and marked accordingly to their result.
I also tested to run a single speclfow and unit test and there was no problem there either.
Great work on the plugin and thank you for a fast reply and fix! :)

You're welcome @teast . It makes it easier when people come with decent reproduction steps, and especially when they come with a proposed solution like yourself!

I am experiencing this issue as well and have tried modifying the treesitter query but don't quite have a handle on the syntax yet. I am using the following sample test:
using NUnit.Framework;

[TestFixture]
public class CalculatorTests
{
    [Test]
    [TestCase(1, 2, 3)]  // Test case 1: 1 + 2 = 3
    [TestCase(-1, -2, -3)]  // Test case 2: -1 + (-2) = -3
    [TestCase(0, 0, 0)]  // Test case 3: 0 + 0 = 0
    public void Add_TwoNumbers_ReturnsSum(int a, int b, int expectedSum)
    {
        // Arrange
        Calculator calculator = new Calculator();

        // Act
        int result = calculator.Add(a, b);

        // Assert
        Assert.That(expectedSum, Is.EqualTo(result));
    }
}

public class Calculator
{
    public int Add(int a, int b)
    {
        return a + b;
    }
}

Which creates duplicate test names as can seen here:

Hi @marvindore ,
Thanks for raising the issue!
This is actually an issue that is unrelated to the issue that this ticket was raised for.
Unfortunately, there are limitations in what treesitter queries can do, and this scenario, is one which cannot be solved (at least not with a decent amount of effort and more treesitter expertise than I have) by treesitter queries alone. The only way to avoid this duplication is to remove the [Test] attribute from your test methods that also have [TestCase] attributes. This won't affect how the functionality of your tests, and will allow it to continue working using neotest-dotnet. I will add something to the documentation to outline this limitation with NUnit.

Totally understand, thank you so much for developing this plugin and your continued support.


feature: Discord server

We can have a discord server for discussion

Great idea! I will be setting it up.


Add basic govspeak editor and contextual guidance

This PR:

Sets up a basic markdown editor using GitHub's toolbar web component. For the moment the editor doesn't contain the govspeak syntax and will be extended in a separate PR
Add contextual guidance component with dummy content for now. It will be updated when the content is ready and the content will be separated from the view.

Trello card

@benthorner Updated addressing all suggestions except for two for which I added comments.