Add type "Local gateway" to virtualNetworkGateway to 2020-07-01

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

###Reference:
https://github.com/Azure/azure-rest-api-specs/pull/10802
Contribution checklist:

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

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

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

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

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

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

This PR is the same as #10802 but in 2020-07-01 version

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk


Use newer tensorflow image to be compatible with torch version

This should fix #92, which appears to be caused by a mismatch between versions of CUDA (in the tensorflow -gpu image) and torch.
The Dockerfile is using a specific version of tensorflow (2.7.0 in this case) that contains an older version of CUDA, whereas torch is being installed without specifying a version – so it's installing the latest, which relies on a newer version of CUDA.
As a workaround for the currently-published snowzach/doods2:amd64-gpu image, you can downgrade torch in the image with something like:
pip install torch==1.12.1+cu113 torchvision==0.13.1+cu113 --extra-index-url https://download.pytorch.org/whl/cu113

It might be a good idea to specify the torch version (in the Dockerfile) in the future, but I guess that's up to the project maintainer.

Thanks!


Deprecated features used

With chef-12 the following warnings are reported because /etc/sysconfig/nfs is
listed in server and client templates
Deprecated features used!
  Cloning resource attributes for template[/etc/sysconfig/nfs] from prior resource (CHEF-3694)
Previous template[/etc/sysconfig/nfs]: /var/chef/cache/cookbooks/nfs/recipes/_common.rb:36:in `block in from_file'
Current  template[/etc/sysconfig/nfs]: /var/chef/cache/cookbooks/nfs/recipes/server.rb:26:in `from_file' at 1 location:
    - /var/chef/cache/cookbooks/nfs/recipes/server.rb:26:in `from_file'

Which version of Chef 12.x and which version of the cookbook are you using?

This is with chef 12.12.15 and cookbook version 2.2.8

I've reworked the pull somewhat to make it more reasonable.
Still not sure if this is the best solution, but it does solve the issue.

@joerocklin: Any chance to get #85 and #86 merged?

Merged


linker command failed

Just after I added react-native-icons into my project, I've got this issue:

sorrry, I've made mistake during installation

@Dharmoslap How did you solve this? Do you remember? I'm stuck at the same error. It would be helpful if you could give me an advice.

I also was able to solve this. Mistakenly it seemed I added the library twice during setting up.


feat: new generate-schema command to generate a JSON schema for the configuration

New generate-schema command
Updated the documentation
Added the first iteration of the schema to then create a PR on SchemaStore once merged into main
New CI check that there are no uncommitted changes to the schema in the PR
Resolves #724
Configurations now require the following decorators to output a title and description in the schema:

    /// # Publish Timeout
    /// Timeout for the publishing process
    pub publish_timeout: Option<String>,

Alright, finally got the CI down! Ready for review/merge.

I made the changes, ready for round two!
I did end up scrapping the code used in one of the three comments, as I found out about the Path struct.
It plays nicer with changing of OSes versus manually built paths.

🎉
Thanks for the review, now I know about the anyhow::context!
I will work on getting a PR on ShemaStore to add this, probably over the weekend.


Incorrect information about the behaviour in a Warned state

The information about the subscription being in a warned state is incorrect.
It is not true that the subscription can be used normally or that all operations are available in this state.
Take a look at https://github.com/Azure/azure-resource-manager-rpc/blob/master/v1.0/subscription-lifecycle-api-reference.md which states

Warned
The subscription has been warned (generally due to forthcoming suspension resulting from fraud or non-payment). Resources must be offline but in running (or quickly recoverable state). Do not deallocate resources. GET/DELETE management APIs must function; PUT/PATCH/POST must not. Don't emit any usage. Any emitted usage will be ignored.

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

ID: 489bc2af-1095-0bfb-0f44-713c7c528388
Version Independent ID: 493e514b-4b39-091f-f25a-0cc9f87ed1bb
Content: Azure subscription states
Content Source: articles/cost-management-billing/manage/subscription-states.md
Service: cost-management-billing
Sub-service: billing
GitHub Login: @anuragdalmia
Microsoft Alias: andalmia

@bmcder Thanks for your comment. We will review the issue and get back to you shortly.

Assigning to content author @anuragdalmia for review and updating document if necessary

@bmcder Article updated. Thanks for letting us know about the issue.
#please-close


devops: Add a GitHub issue template for bug reports

Merging so we can see it — post any feedback at all here


[App Service]: Updating websites to new SDK version 4.0.0

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

App Service


Can't import Bestiary 3 - Cannon Golem

Can't import statblock:
Cannon Golem      CR 15
XP 51,200
Male cannon golem (Pathfinder RPG Bestiary 3 135)
N Large construct
Init +7; Senses darkvision 60 ft., low-light vision; Perception +2
Defense
AC 31, touch 16, flat-footed 24 (+7 Dex, +15 natural, -1 size)
hp 140 (20d10+30)
Fort +6, Ref +13, Will +8
DR 15/adamantine; Immune construct traits, immunity to magic
Offense
Speed 30 ft.
Melee 2 slams +29 (2d10+10)
Ranged cannon +26 (6d6+7/19-20/×4)
Space 10 ft.; Reach 10 ft.
Special Attacks cannon
Statistics
Str 30, Dex 24, Con —, Int —, Wis 15, Cha 2
Base Atk +20; CMB +31; CMD 48
Feats Improved Critical (cannon)[B]
SQ alloyed, blasting critical, gun training
Other Gear cannon[UC]
Special Abilities
Alloyed (Ex) Slam and Cannon attacks overcome DR as adamantine, cold iron, and silver.
Blasting Critical (Ex) On a critical hit with a slam, immediately make a cannon attack vs. that foe.
Cannon (Ex) Magazine holds 20 balls, can reload with a swift action and fire twice per rd.
Construct Traits (+30 HP) Constructs have many immunities.
Damage Reduction (15/adamantine) You have Damage Reduction against all except Adamantine attacks.
Darkvision (60 feet) You can see in the dark (black and white only).
Gun Training (Ex) A cannon golem adds its Dex modifier to the damage dealt by its cannon.
Immunity to Ability Damage Immunity to ability damage
Immunity to Ability Drain Immunity to ability drain
Immunity to Bleed You are immune to bleed.

Immunity to Disease You are immune to diseases.
Immunity to Energy Drain Immune to energy drain
Immunity to Exhausted You are immune to the exhausted condition.
Immunity to Fatigue You are immune to the fatigued condition.
Immunity to Magic (Ex) Immune except vs. water descriptor spells and heat metal.
Immunity to Mind-Affecting effects You are immune to Mind-Affecting effects.

Immunity to Poison You are immune to poison.

Immunity to Stunning You are immune to being stunned.
Low-Light Vision See twice as far as a human in dim light, distinguishing color and detail.

Used your latest module that was available as of this afternoon at about 2:30pm.  I used Hero Lab to produce this statblock. For greater accuracy, you might want to use one of the online resources. Hero Lab can truncate some of the text. All the other statblocks from Bestiary 3 imported -- a total of more than 86% imported!


HTTP links not working in Video Stream

Environment
Docker
Version
0.15.3
Describe the problem
I'm trying to add a video feed from my camera. It's connected to Frigate via go2rtc. The URL to it the feed is http://<server>:<port>/stream.html?src=<name> but when I put that in the Video Feed widget, it doesn't work. Checking the network traffic, it looks like its trying to access https://<server>:<port>/stream.html?src=<name> instead. I have set my site settings for my Homarr URL to allow insecure content. If I add the stream URL via an iFrame, it shows.
Logs
Nothing of note, just standard informational stuff:
ℹ Saving updated configuration of 'default' config.
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Saving updated configuration of 'default' config.
ℹ Local icons repository directory exists and contains 0 icons
ℹ Saving updated configuration of 'default' config.
ℹ Local icons repository directory exists and contains 0 icons
ℹ Saving updated configuration of 'default' config.
ℹ Local icons repository directory exists and contains 0 icons
ℹ Saving updated configuration of 'default' config.
ℹ Local icons repository directory exists and contains 0 icons
ℹ Saving updated configuration of 'default' config.
ℹ Saving updated configuration of 'default' config.
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
ℹ Saving updated configuration of 'default' config.
ℹ Local icons repository directory exists and contains 0 icons
ℹ Local icons repository directory exists and contains 0 icons
Context
Settings for the widget, showing HTTP:

Network traffic, showing HTTPS:

Please tick the boxes

[X] I confirm that I attached the proper logs
[X] I've read the docs
[X] I've checked for duplicate issues
[X] I've tried to debug myself

Do you use a certificate for Homarr? It might be that your browser is attempting to upgrade the protocol to https

Closing due to inactivity.


Use pointer receivers instead of value receivers.

Fixes #48
Also implement tags from @bradleyfalk as well as a member function for Permanently delete members.

Looks good, I'll review this weekend! Thanks for your commits!

Oh, I should probably have made a separate branch for this pull request.
I had a need to get the tags functionality in as well so I merged @bradleyfalk changes into my master branch. And I also needed the Delete Permanent function, so I implemented it.
If you want I could probably create a separate branch for this.

I think we just need to test it -- make sure there are no issues switching to pointer receivers.


Modifying columnDef.enableFiltering has no effect after the grid has rendered

Even with calling notifyDataChange, disabling enableFiltering on a columnDef does not update the column header to remove the filters.  Changing something like displayName and then calling notifyDataChange works, but modifying enableFiltering does not.  I really hope I missed something, but it looks like a bug.
Example:
http://plnkr.co/edit/ILZh6kosopnHqGtNjZ3c?p=preview
More details:  In the 'ui-grid/uiGridHeaderCell' template, filterable is still resolving to true
Workaround:  Create a custom headerCellTemplate, and replace filterable with grid.options.enableFiltering && col.enableFiltering

This is a duplicate of #2067


install-autoshutdown-server-extension/on-jupyter-server-start.sh failing

I've created a lifecycleconfig at the domain level. The script is install-autoshutdown-server-extension)/on-jupyter-server-start.sh.
However, the script fails immediately when trying to cd into /home/sagemaker-user/:
+ export TIMEOUT_IN_MINS=120
--
+ TIMEOUT_IN_MINS=120
+ cd /home/sagemaker-user
/opt/ml/lifecycleconfig/lifecycle_script.sh: line 17: cd: /home/sagemaker-user: No such file or directory
+ export TIMEOUT_IN_MINS=120
+ TIMEOUT_IN_MINS=120
+ cd /home/sagemaker-user
/opt/ml/lifecycleconfig/lifecycle_script.sh: line 17: cd: /home/sagemaker-user: No such file or directory

Is there something I'm missing? I'm on jupyterlab v1, and the following is what I see when starting up:

I have created the lifecycle script as a JupyterServer type, and I'm running into the same issue when I try to start a image terminal.  I have the lifecycle script applied to my user profile only.

My KernelGateway app still persist despite having configured everything accordingly. The difference between my various tests is that more and more terminals get  provisioned in the Studio notebook. Also, the logs explained here show no indication of errors/failures.

This doesn't work as kernel gateway app, i'm not sure i understand what on jupyterserver start means, i have to have the studio running to attach this lifecycle configuration, i'm able to attach this to the Studio, what i don't understand is when will this be applied, do i have to restart the server ?

@JMarsz-EA - can you share more details? The Image terminal runs on the KernelGateway app, not the JupyterServer app, so there is some mismatch in the configuration.
@nate-benton90 - are you seeing any logs in your Cloudwatch? See the Troubleshooting section in this blog for the log stream to look for. You should see logs like "comparing idle time.." etc. happen every 10 seconds. Please raise a support case otherwise, and our team will look into it.
@GavinatorK This blog explains SageMaker Studio architecture, along with the different app types. To answer your question, once you attach this LCC, you will need to restart Studio

Hello, I have tried all the above: attaching as KernelGateway, as JupyterServer, deleting default and restarting Studio, but nothing works because it fails with /opt/ml/lifecycleconfig/lifecycle_script.sh: line 32: cd: /home/sagemaker-user: No such file or directory

@mrbrandt92 It has to be attached as a JupyterServer LCC. If you're running it in KGW, it will fail with the error you mentioned, since there's no /home/sagemaker-user in the KGW app. What's the error you get when you run it as the JS LCC?

that was the same error I got after I ran:

LCC_CONTENT=openssl base64 -A -in SCRIPT_NAME.sh

aws --profile profile_name sagemaker create-studio-lifecycle-config
--studio-lifecycle-config-name my-config-name
--studio-lifecycle-config-content $LCC_CONTENT
--studio-lifecycle-config-app-type JupyterServer

aws --profile profile_name sagemaker update-domain --domain-id d-MY_DOMAIN_ID
--default-user-settings '{
"JupyterServerAppSettings": {
"DefaultResourceSpec": {
"LifecycleConfigArn": "arn:aws:sagemaker:us-east-1:ACCOUNT_ID:studio-lifecycle-config/my-config-name",
"InstanceType": "system"
},
"LifecycleConfigArns": [
"arn:aws:sagemaker:us-east-1:ACCOUNT_ID:studio-lifecycle-config/install-autoshutdown-extension"
]
}}'

So it attaches and I can create and/or update a user to attach that lifecycle config, but when I go to create a new notebook and attach that start-up script, it fails. I'm realizing now that I didn't try creating a new notebook without attaching the script and seeing if my custom packages are already available to install, let me try that.
UPDATE: we had custom packages we needed installed on the kernel, but we had both the custom packages and the auto shutdown script in the same script trying to apply to EITHER JupyterServer or KernelGateway. We needed to separate them out in order to work; thank you for the guidance!

Hey,
When I attach the script as default lifecycle configuration for my domain I get:
stdbuf: failed to run command ‘/opt/ml/lifecycleconfig/lifecycle_script.sh’: No such file or directory
Any idea how to fix it? It's set to JupyterServer app

Had the same problem.
It seems that I have a wrong line ending format in the shell script.
Fixed it with:
Notepad++
Replace \r\n with \n

Had the same problem. It seems that I have a wrong line ending format in the shell script.
Fixed it with:
Notepad++ Replace \r\n with \n

This fix is real!!!!


Expose weapon firing cycle for enemy units?

Right now there isn't any way to know when an enemy unit has attacked, aside from tracking damage on your own units and guessing.  It would be extremely useful (and match what a human could infer from animations) to expose more information about the firing cycle of enemy units, such as weapon_cooldown (which is currently only set for player-owned units).

Lack of such kind of data is the only thing that stops me from diving into programming my own micro-oriented bot.


Why does the CosmosEntity impl for serde_json::Value return a clone of the whole Value?

I was trying to use the serde json!() macro for tests with different document structures.  create_document relies on the partition_key impl from cosmos_entity.rs and takes the whole Value as the partition key.

What was the purpose behind implementing the trait this way?
Is there a work around?

I'm not sure if this is an issue, but I don't understand the purpose of the current behavior.

I'm still curious about the specific behavior choice, but I got it working with a new type around Value.


Terraform refusing to run due to sensitive outputs.

Hey Google team, it looks like at least as of Terraform v1.0.4, some of these module's outputs are considered sensitive and the module refuses to apply as-is:
╷
│ Error: Output refers to sensitive values
│
│   on outputs.tf line 101:
│  101: output "primary" {
│
│ To reduce the risk of accidentally exporting sensitive data that was
│ intended to be only internal, Terraform requires that any root module
│ output containing sensitive data be explicitly marked as sensitive, to
│ confirm your intent.
│
│ If you do intend to export this data, annotate the output value as
│ sensitive by adding the following argument:
│     sensitive = true
╵
╷
│ Error: Output refers to sensitive values
│
│   on outputs.tf line 111:
│  111: output "instances" {
│
│ To reduce the risk of accidentally exporting sensitive data that was
│ intended to be only internal, Terraform requires that any root module
│ output containing sensitive data be explicitly marked as sensitive, to
│ confirm your intent.
│
│ If you do intend to export this data, annotate the output value as
│ sensitive by adding the following argument:
│     sensitive = true
╵
ERRO[0004] 1 error occurred:
	* exit status 1

I can PR a fix with sensitive = true added to that output, if that's the approach you'd like to take to resolve the issue.

How are you trying to use the module?
 You should not be running terraform apply directly from the repo, but instead reference the module from your own configuration (as shown in the README).

This is a bug in Terragrunt then. Please file an issue with them, as their behavior is inconsistent with Terraform core.
More context: https://github.com/terraform-google-modules/terraform-google-sql-db/issues/229#issuecomment-866945524


RuntimeError: DataLoader worker (pid(s) 28643) exited unexpectedly

Hi, thanks for your work!
When i train the BAT model in kitti dataset(class Car/Pedestrian), it exited unexpectedly. But when i change the tracking class to Van/Cyclist, it works well. I don't know how to solve it, so looking forward to your reply!

my environment:
python 3.7
cuda 10.1
pytorch 1.4.0

This is probably related to your running systems. Can you run it with "--workers 0"?


Update scan documentation

All minor, just noting so these don't get forgotten:

[x] DeltaListScan fails if the input is a list; it requires a numpy array
[x] The API documentation for OuterProductAbsScan and probably others renders badly
[x] The API documentation for InnerProduct*Scan seems to be missing.
[ ] It might be worth implementing *Product*ListScan. One could just use ScanND and cyclers, but it's easy enough to provide a convenience class for this.

cc @vivekthampy

Punting on the last point. I think it is a good "gateway" to the more powerful ScanND way of doing things. @vivekthampy has been using ScanND successfully. See #258 for the docs fixes.


Improve "Stop remote recording" action

Closes https://github.com/getappmap/appmap-intellij-plugin/issues/146
Best reviewed commit by commit.
You can test this by doing a remote recording, e.g. via Tools > AppLand > .... The stop action should pick up the right location the first time it's invoked in a project.

Cleanup of the code and fix a few minor issues of "stop remote recording". The code needed some care...
Refactoring to allow fetching the default storage location in the background
Don't terminate early in "Stop remote recording" if the update of the AppMap name failed (e.g. due to an unknown or invalid encoding of the retrieved AppMap)
Update the default location of recorded AppMap

The location is now calculated like this:

The last used location of "Stop AppMap recording". I retained the old logic, but this can be removed, if necessary
The location configured in the project's appmap.yml file. If there's more than one appmap.yml then the config files are ignores. There's no context to decide which config file to use. Append remote to the configured path.
Use projectBaseDir/temp/appmap/remote as fallback

Questions:

Should the stop action remember and restore the last used location or should it always pick up the location from appmap.yml first?

:tada: This PR is included in version 0.26.0 :tada:
The release is available on:

v0.26.0
GitHub release

Your semantic-release bot :package::rocket:


elm install 404 Not Found

Hi,
It appears that the installation of this library is broken as the source code for version 2.0.29 is not available in the expected location. For reference, here is the output of elm install (elm version is 0.19.1).
$ elm install Skinney/elm-phone-numbers
Here is my plan:

  Add:
    Skinney/elm-phone-numbers    2.0.29

Would you like me to update your elm.json accordingly? [Y/n]: Y
-- PROBLEM DOWNLOADING PACKAGE -------------------------------------------------

I was trying to download the source code for Skinney/elm-phone-numbers 2.0.29,
so I tried to fetch:

    https://github.com/Skinney/elm-phone-numbers/zipball/2.0.29/

But it came back as 404 Not Found

This may mean some online endpoint changed in an unexpected way, so if does not
seem like something on your side is causing this (e.g. firewall) please report
this to https://github.com/elm/compiler/issues with your operating system, Elm
version, the command you ran, the terminal output, and any additional
information that can help others reproduce the error!

If you could fix this, it would be greatly appreciated. Thanks!

Nevermind, after searching the elm package repository I realized the package was in a different location. This worked:
elm install  robinheghan/elm-phone-numbers


[SF] Service Fabric integration should support more than one endpoint/listener per service

What should we add or change to make your life better?
The Service Fabric integration for dynamic discovery of routes PR #257 would help us a lot. But it should support more than 1 endpoint / listener per service because  a Service Fabric service can provide more than one endpoint.
Why is this important to you?
A SF service could provide different public api's on different ports e.g. an additional management or administration api (especially 3rd-party guest apps which can't be easily decomposed to different services).
See discussion: https://github.com/microsoft/reverse-proxy/pull/257#issuecomment-726649119

Perhaps it makes sense to align Yarp Clusters with a combination of SF.Service and SF.Endpoint?
So endpoints would end up more in line with Yarp Cluster, instead of just a given service. And like you said, it means we'd have to move routes under a given endpoint.
Yarp Cluster -> fabric:/App1/Svc2/WebInterface
                    -> https://<IP_ADDRESS>:30001
                    -> https://<IP_ADDRESS>:30002
                    -> https://<IP_ADDRESS>:30003

Yarp Cluster -> fabric:/App1/Svc2/AdminInterface
                    -> https://<IP_ADDRESS>:30004
                    -> https://<IP_ADDRESS>:30005
                    -> https://<IP_ADDRESS>:30006

I think this makes sense from what I understand.
About using XML to define that structure. There are alot of similarities between orchestrators. Most of the orchestrators support at least some sort of string label/tag. I'm working on a Docker auto-discovery right now for Yarp and I was able to largely re-use the LabelsParser from #257. We should likely refactor that out into a common namespace for folks who intend to implement configuration providers.
With that in mind, I tend to lean towards using Dictionary<string, string> with Labels instead of XML.

Triage: We should timebox investigation into a design to see if it changes current design significantly. Otherwise Backlog.

SF work has moved to https://github.com/microsoft/service-fabric-yarp


grpc_cli does not work if the proto file contains imports

What version of gRPC and what language are you using?
1.22.2
What operating system (Linux, Windows,...) and version?
Linux
What runtime / compiler are you using (e.g. python version or version of gcc)
grpc_cli on client
What did you do?
make a grpc_cli call to a user.proto with imports (https://github.com/restorecommerce/protos/blob/master/io/restorecommerce/user.proto)
./grpc_cli type localhost:50051 io.restorecommerce.user.LoginRequest

getting below error:
[libprotobuf ERROR google/protobuf/descriptor.cc:3561] Invalid proto descriptor for file "":
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   io/restorecommerce/resource_base.proto: Import "io/restorecommerce/resource_base.proto" was not found or had errors.
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   google/protobuf/empty.proto: Import "google/protobuf/empty.proto" was not found or had errors.
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   io/restorecommerce/meta.proto: Import "io/restorecommerce/meta.proto" was not found or had errors.
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   io/restorecommerce/address.proto: Import "io/restorecommerce/address.proto" was not found or had errors.
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   io/restorecommerce/contact_point.proto: Import "io/restorecommerce/contact_point.proto" was not found or had errors.
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   io/restorecommerce/image.proto: Import "io/restorecommerce/image.proto" was not found or had errors.
Type io.restorecommerce.user.LoginRequest not found.

I did try even with proto_path and protofiles and does not seem to work - still get same error as above:
./grpc_cli type --proto_path=$PROTO_ROOT --protofiles=io/restorecommerce/user.proto localhost:50051 io.restorecommerce.user.LoginRequest

What did you expect to see?
grpc_cli should resolve the returned proto containing import statements and display the requested Type.
What did you see instead?
[libprotobuf ERROR google/protobuf/descriptor.cc:3561] Invalid proto descriptor for file "":
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   io/restorecommerce/resource_base.proto: Import "io/restorecommerce/resource_base.proto" was not found or had errors.
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   google/protobuf/empty.proto: Import "google/protobuf/empty.proto" was not found or had errors.
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   io/restorecommerce/meta.proto: Import "io/restorecommerce/meta.proto" was not found or had errors.
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   io/restorecommerce/address.proto: Import "io/restorecommerce/address.proto" was not found or had errors.
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   io/restorecommerce/contact_point.proto: Import "io/restorecommerce/contact_point.proto" was not found or had errors.
[libprotobuf ERROR google/protobuf/descriptor.cc:3564]   io/restorecommerce/image.proto: Import "io/restorecommerce/image.proto" was not found or had errors.
Type io.restorecommerce.user.LoginRequest not found.

any updates on this ?

Unfortunately, I haven't been able to spend time on this. There are other higher priority things on my plate right now :(


fix roadPlanner coverage%

Pull request summary
existing roadCoverage calculations takes into account all roadCoverages[] rooms which are being populated by directives.
those rooms can be very far and many of them are not remote/SK rooms.
by doing so, the road convergence % will mostly result in < 75%, which is the threshold for 2:1 transporter body.
the fix will consider rooms in roadLookup instead (which include rm/sk/self-colony rooms only.
use the below in console to check existing roadCoverage:
_.forEach(Memory.colonies,room =>{
console.log(room.roadPlanner.roadCoverage);
})
Description:

the below shows the impact on coverage before and after

Room#
BEFORE
AFTER

1
0.65
0.96

2
0.29
0.96

3
0.19
0.96

4
0.38
0.96

5
0.41
0.96

6
0.46
0.93

7
0.50
0.96

8
0.40
0.96

9
0.33
0.96

10
0.52
0.96

11
0.51
0.96

12
0.50
0.96

13
0.65
0.96

14
0.50
0.96

15
0.54
0.96

16
0.92
0.96

17
0.82
0.96

18
0.45
0.96

19
0.48
0.96

20
0.60
0.88

21
0.98
0.96

22
0.96
0.96

23
0.41
0.96

24
0.66
0.96

25
0.92
0.96

26
0.00
0.96

Added:

None

Changed:

None

Removed:

None

Fixed:

None

Testing checklist:

[x] Changes are backward-compatible OR version migration code is included
[x] Codebase compiles with current tsconfig configuration
[x] Tested changes on {choose PUBLIC/PRIVATE} server OR changes are trivial (e.g. typos)

I haven't bene able to reproduce the problem of incorrect road coverages. If you're getting incorrect road coverage calculations it seems as though something is incorrectly modifying this.colony.destinations. We should try to identify what is actually causing the issue for you rather than slapping a band-aid on the problem. Can you create an issue with some more detailed data on what this.colony.destinations is to try to identify the offending process?

looking closely at the code, this.colony.destinations is being populated by the constructors of:
extract/harvest/upgradeSite.
I believe the issue happens when a harvest site is being claimed
there is no cleanup mechanism to remove it from destinations.
the newly claimed room will remain in destinations as if it is still a harvest site.
roads will not be build towards the claimed room. this will affect the roadCoverage percentage


Possible bug or need to improve documentation

After looking at the documentation for the import/no-named-as-default eslint rule, I'm still confused about what exactly I'm doing wrong.
I have the following file structure
.
├── ButtonBack.css
├── ButtonBack.jsx
├── __tests__
│   └── ButtonBack.test.jsx
└── index.js

The ButtonBack.jsx contains the following code
import React from 'react';
import PropTypes from 'prop-types';

export default class ButtonBack extends React.Component {
  ... code removed to keep example short ...
}

_tests_/ButtonBack.test.jsx contains the following code
import React from 'react';
import { shallow } from 'enzyme';
import ButtonBack from '../ButtonBack'; // <== this line has an eslint warning

... code removed to keep example short ...

The problem is, my linter says that import ButtonBack from '../ButtonBack violates the following lint rules:

import/no-named-as-default
import/no-named-as-default-member

I can't figure out why my import statement violates the lint rule.

Removing the name of the class in ButtonBack.jsx (export default class extends React.Component) does not solve the issue.

there is only one export in ButtonBack.jsx

I wonder if the ButtonBack.css is causing an issue; what's your eslint config?


feat: include current idx when rendering header or footer

Implemented passing current idx to renderHeader and renderFooter methods.
It might be needed in situations where one would like to display current picture data in header or footer (in my example I need picture name displayed in footer)

Great, thank you


When i analyze the project use command + i in Xcode, it cause memory leak

the memory leak is in AFHTTPSessionManager.m,code is:

(instancetype)manager {
return [[[self class] alloc] initWithBaseURL:nil];
}

i just want to know , if i can change it like this to solve the problem:

(instancetype)manager {
static AFHTTPSessionManager *manager = nil;
static dispatch_once_t onceToken;
dispatch_once(&onceToken, ^{
manager = [[self alloc] initWithBaseURL:nil];
});
return manager;
}

No you can not change that. That is not meant to be a singleton accessor method.

thank you


Fix issue with empty args and workers

when workers are enabled, it seems empty lines are removed from the arguments list. This PR makes sure to always have 1 line per arg (by padding with a leading _ which is discarded).
This is actually currently a blocker for us. I would love to have better tests around this (that exercise everything with and without workers, but I don't want to try to shave that yak before we get this build issue unblocked).

added #246

Sounds good. Will update and add links to the issue I filed on bazel.
On Wed, Jul 5, 2017 at 18:08 Ittai Zeidman<EMAIL_ADDRESS>wrote:

@ittaiz requested changes on this pull request.
Hey, two really small comments
In twitter_scrooge/twitter_scrooge.bzl
https://github.com/bazelbuild/rules_scala/pull/245#discussion_r125807549
:

@@ -140,7 +140,7 @@ def _gen_scrooge_srcjar_impl(ctx):

in order to generate code) have targets which will compile them.
_assert_set_is_subset(only_transitive_thrift_srcs, transitive_owned_srcs)

path_content = "\n".join([_colon_paths(ps) for ps in [immediate_thrift_srcs, only_transitive_thrift_srcs, remote_jars, external_jars]])

path_content = "\n".join(["_" + _colon_paths(ps) for ps in [immediate_thrift_srcs, only_transitive_thrift_srcs, remote_jars, external_jars]])

Can you extract this into a constant with a clear name? Something like
PaddingToPreserverAruent or PaddingToSeparateArgumentToOwnLine? Names are
different once I'm really not sure I got the point well enough...
In src/scala/scripts/TwitterScroogeGenerator.scala
https://github.com/bazelbuild/rules_scala/pull/245#discussion_r125807690
:

@@ -41,7 +41,7 @@ class ScroogeGenerator extends Processor {

def processRequest(args: java.util.List[String]) {
def getIdx(i: Int): List[String] =

 if (args.size > i) args.get(i).split(':').toList.filter(_.nonEmpty)

 if (args.size > i) args.get(i).drop(1).split(':').toList.filter(_.nonEmpty)

Can we extract 1 into a constant as well to express why we're dropping it?
Something like .drop(PaddingCharacterUsedForArgumentPreservation)
I know the names are long but honestly I fear this will cryptic code
really fast
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/245#pullrequestreview-48230700,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEJdq1MNQaW6h5hoQeh5tU6L5W9OLEkks5sLF24gaJpZM4OPFCs
.

LGTM. I'd wait for green just to be on the safe side. Going to a company party so you can merge when you want :)

@ittaiz thanks for the review, sir!


Exploring new opportunities with the recent rebase attempt

Upon utilizing the AlSaqr branch of the repository, it appears there may have been complications likely stemming from a recent rebasing attempt. Undertaking a review to ascertain if similar discrepancies persist and address them accordingly would be of great help!

Thanks for reporting the unresolved conflict!
Addressed in 0f2c854


RFR: Start st2auth as part of launchdev

+1ed https://github.com/StackStorm/st2/pull/1470

:ship: :+1:


Add option to specify custom trained model for brain extraction

This is incorporated in the two checkpoints to feed the node with (in_ckpt_loc for localization and in_ckpt_seg for segmentation)


Epinephrine can now stabilize animals in critical condition

About the PR
This is in response to a feedback thread made on discord, however there is no equivalent issue here. This PR adds the Reagent Condition RelativeDamage which uses a percentage of the maximum damage an entity can take before dying, and changes epinephrine so that this is used instead of Total Damage. As a result, anything that goes into crit can now be stabilized by epinephrine.
Why should this PR exist: Animals are affected by other reagents, and a lot of players expect epinephrine to affect them too. However since most do not have over 100 health, epinephrine does not get activated. Also this condition could probably be used in other future reagents.
Potential Problems: I have written the condition in a bit of a strange way because I could not figure out any way to use a certain helper method without it leading to endless Null Reference Exceptions, also this assumes that everything will go into crit on half of its health. However epinephrine currently just assumes that something will go into crit on 100 health and can theoretically heal anything that can endure more than 100 health before going into crit which can metabilize reagents so I am not sure this is a problem. Alternatively, a new condition can be created that just checks if the state of an entity is crit but I believe this would not really see any use case beyond epinephrine.
I am not completely sure on the changelog message, and a suggestion on a better one would be appreciated.
Media

https://user-images.githubusercontent.com/32041239/210184779-76c4e15c-e414-4a0b-86c0-939f780a149d.mp4

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: Epinephrine will now be able to stabilize the smallest crew members of the station such as Ian, and other animals.

good concept but the "in crit" condition breaks for entities who either don't have crit states or whose crit states aren't at exactly 50% health. You'd be better off basing the effect on mobstate.

good concept but the "in crit" condition breaks for entities who either don't have crit states or whose crit states aren't at exactly 50% health. You'd be better off basing the effect on mobstate.

I am not sure why it now fails a test, I ran tests locally as well and it says it failed a test regarding cargo due to an entity not being valid. Also even more strangely, running them with the previous commit gave me a test failure but when it was checked here it came up with no errors.

Changes to MobState have broken this, please do not merge for now.

Ok now it should work again.


[Feature request] Set schema for request

Hi,
I'm working in a team on a project using Postgres. We need to set a Schema for a Request, using search path won't work cause setting the path withing one thread will screw any other request from other threads. Unfortunately we don't know how to implement that in a clean way. I'm pretty new to Go so please forgive that noobish request.
Kind regards

Would love that feature, too. Currently doing it via set scema 'name', but it's not a very clean way and I already ran into some problems because of this.
Would really appreciate such a feature.
Greetz Chaos

nvm, added it myself. Was easy to do. Will make a pull request after more tests
greetz


sticker file haven't create file

hi!
so I have a problem with the sticker generator. where is done?
then look like don't have a sticker file has created. but not there?
please fix this?
also, sorry  I use Grammarly

Hi - when you use the tool it should output PNG files into a directory called Stickers in the root of your project.

Closed as you haven't replied. The stickers get outputted to a directory called "stickers". I have tested this and it works for me:


define DNDLongPress. allowableMovement

mimics UILongPress to allow for song unintentional wiggle

Hey deeje, thanks for your contribution. There are some minor code style adjustments, I would like to see, but then I'll gladly merge this in.


Immediate fail build when instance is unavailable

... when running any task that requires instances available (like aemDeploy, aemSatisfy)
this will make build even more smart :)
in a same way as checking -Paem.force and tasks on graph
checking for available instances is already done in await action;

Implemented in 8.x


Feature todolist

in this branch the following changes have been made according to this lesson

setup environment for reacting
developed react app for the todo list
add new todos
check the todos
delete the todos
generate a unique id
Add style to the  component to have better UI using
inline object
external style
module style ( recommended to avoid selection problems)
SASS for the to-do item

Hello @beteland123 Nice job you have done! However, there is a small detail you can fix. Please, kindly correct the spelling...

ok @JuanDBta , Thanks for your suggestion! I'll fix the issue based on your feedback. 👍


Fix MinGW vsnprintf compile errors and warnings

This PR:

Adds Appveyor CI config file.
Fixes compile errors on MinGW about missing declaration of vsnprintf. The attempted fix in https://github.com/boostorg/test/pull/195, although correct, did not fix the error.
Silences gcc warnings about unused global variables.

There is already a PR about AppVeyor, see #185 . For some reason it is not active yet. It would be good if you can split the PR on those orthogonal issues. Thanks!

That PR sets up MinGW differently than my config, I'm not sure it is equivalent. Also, it doesn't enable CI for topic branches. I've included the config because I had to ensure the problem reproduces and test the change in Appveyor since I couldn't reproduce it locally.
The config is in a separate commit, you can skip merging it. If you want, I can remove this commit from this PR. Please, confirm.

For some reason it is not active yet. It would be good if you can split the PR on those orthogonal issues.

You have to add the GitHub project to your Appveyor project list in order to get it running. You should have admin rights for the GitHub project though.

I split the PR in 3 branches:
Concerning the vsnprintf issue (that is the purpose of the PR), it is currently running the tests in the branch topic/PR-205-Fix-MinGW-vsnprintf-compile-errors-warnings. I'll merge that once the tests have cycled.
The GCC warnings is on branch topic/silence-gcc-warnings and the AppVeyor one is in the branch topic/CI-appveyor. I need to merge the Appveyor work with the other running PR on CI. Concerning the gcc warning, I am not sure: I can see some of the warnings, but all those symbols are used at some point. Since I do not know what BOOST_ATTRIBUTE_UNUSED does and since this is just about warnings (lesser priority), I suggest we wait a bit. The other option for silencing the warnings would be to add the proper pragmas in the suppress_warnings.hpp file.

Since I do not know what BOOST_ATTRIBUTE_UNUSED does...

This is __attribute__((unused)) for gcc, which explicitly indicates that the variable may appear unused in the program. Disabling the warning with a pragma may work, but it may also disable the warning for genuinely leftover unused variables, so I prefer the attribute.

Changes merged to next except for the AppVeyor (tried to enable the thing from Appveyor but still waiting for some approval by boost.org).
Closing, thanks!


Create power_of_two.cpp

This Pull Request is for HacktoberFest 2020
Description of Change

Checklist

[ ] Added description of change
[ ] Added file name matches File name guidelines
[ ] Added tests and example, test must pass
[ ] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[ ] Relevant documentation/comments is changed or added
[ ] PR title follows semantic commit guidelines
[ ] Search previous suggestions before making a new one, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.

Notes:

@Panquesito7 please review this as well.

@Panquesito7 sir, it would be kind of you please merge it.

please merge it

Will do until @kvedala reviews the PR. 🙂

changes to the file pancake_sort.cpp

It seems those changes were made by the CI (https://github.com/TheAlgorithms/C-Plus-Plus/pull/1315/commits/caf2ecc54e0abc07f479d5eeb390f0614f9484eb) automatically.

changes to the file pancake_sort.cpp

It seems those changes were made by the CI (caf2ecc) automatically.

ah, ok


When using clear() method, it crashed

corba = ansys.mapdl.core.launch_mapdl(override=True,
loglevel='INFO',
jobname=job_name,
run_location=path,
nproc=core_num,
mode='grpc',
cleanup_on_exit=True,
remove_temp_files=True)
print(corba.clear())
None
Exception in thread Thread-51:
Traceback (most recent call last):
File "C:\CodeRuntime\Python38\lib\threading.py", line 932, in _bootstrap_inner
self.run()
File "C:\CodeRuntime\Python38\lib\threading.py", line 870, in run
self._target(*self._args, **self._kwargs)
File "C:\CodeRuntime\Python38\Lib\site-packages\ansys\mapdl\core\mapdl_grpc.py", line 331, in _consume_responses
raise MapdlExitedError('Lost connection with MAPDL server') from None
ansys.mapdl.core.errors.MapdlExitedError: Lost connection with MAPDL server

Python version is 3.8.8
Ansys version is 2021R1

Sorry about that.  That absolutely shouldn't be happening.  Could you try the following?

corba.clear('NOSTART')
`corba.run('/CLEAR')
corba.prep7(); corba.clear()

When I run corba.run('/CLEAR')
Out7: ''
Exception in thread Thread-105:
Traceback (most recent call last):
File "C:\CodeRuntime\Python38\lib\threading.py", line 932, in _bootstrap_inner
self.run()
File "C:\CodeRuntime\Python38\lib\threading.py", line 870, in run
self._target(*self._args, **self._kwargs)
File "C:\CodeRuntime\Python38\Lib\site-packages\ansys\mapdl\core\mapdl_grpc.py", line 331, in _consume_responses
raise MapdlExitedError('Lost connection with MAPDL server') from None
ansys.mapdl.core.errors.MapdlExitedError: Lost connection with MAPDL server
It seems to get a right feedback. But after about one second, it crashed.
others have the same situation above.

Can you run other commands?  Also, does setting it with the following help?
corba = ansys.mapdl.core.launch_mapdl(override=True,
  loglevel='INFO',
  jobname=job_name,
  run_location=path,
  nproc=core_num,
  mode='grpc',
  cleanup_on_exit=True,
  additional_switches='-smp',
  remove_temp_files=True)

When I use the commands you put above, sometimes but very seldom it didn't crash.
But most time it crashed whatever corba.clear(), corba.run('/CLEAR') or corba.prep7(); corba.clear().

I have another problem i found last night. Sometimes chain_command() can't run all command in the loop in it. I don't know why, but when i don't use chain_command() it works well.

We've tested ansys-mapdl-corba on a variety of platforms and environments and found it to be quite stable, but there might be something particular to your environment that's making it so unstable.  Can you please run and send me back the response to:
import ansys.mapdl.core as pymapdl
pymapdl.Report()

For the time being, I'd set the mode to 'corba' if you wish to continue using pymapdl due to the stability issues when using the grpc mode.

PyMAPDL Software and Environment Report
Date: Sun Feb 28 11:19:06 2021 中国标准时间
OS : Windows
CPU(s) : 12
Machine : AMD64
Architecture : 64bit
Environment : IPython
NVIDIA Corporation : GPU Vendor
GeForce RTX 2060/PCIe/SSE2 : GPU Renderer
4.5.0 NVIDIA 456.71 : GPU Version
Python 3.8.8 (tags/v3.8.8:024d805, Feb 19 2021, 13:18:16) [MSC v.1928 64 bit
(AMD64)]
pyvista : 0.28.1
vtk : 9.0.1
numpy : 1.20.1
scipy : 1.6.1
appdirs : 1.4.4
pexpect : Could not import
ansys.mapdl.core : 0.58.0
ansys.mapdl.reader : 0.50.6
matplotlib : 3.3.4
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Alex<EMAIL_ADDRESS>Sent: 2021年2月27日 23:46
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [pyansys/pymapdl] When using clear() method, it crashed (#367)
We've tested ansys-mapdl-corba on a variety of platforms and environments and found it to be quite stable, but there might be something particular to your environment that's making it so unstable. Can you please run and send me back the response to:
import ansys.mapdl.core as pymapdl
pymapdl.Report()
For the time being, I'd set the mode to 'corba' if you wish to continue using pymapdl due to the stability issues when using the grpc mode.
―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/pyansys/pymapdl/issues/367#issuecomment-787092486, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABGUIH5VDNPZKFT4FWLIX2DTBEHV3ANCNFSM4YJQQTLQ.

Thanks!  I doubt that it's the case, but it's possible it's due to AMD vs Intel.  We're going to look into this.

Hi @PatchouliHina,
Can you checkout the solution I proposed in #385 and let me know if that works for you?  It's possible that there's a hardware configuration on your end that we didn't account for (i.e. SSD vs HDD).

Hi @PatchouliHina,
Could you please take a look at the solution proposed at #385?  It's possible the work around will work in your case as well.  I think there's a hardware case we didn't account for (i.e. HDD vs SSD).

Hi @PatchouliHina did this solution fix your issue?

I have update Python to 3.9.5 and ansys-mapdl-corba to 0.59.3, it doesn't crash now. Thank you.

I have update Python to 3.9.5 and ansys-mapdl-core to 0.59.3, it doesn't crash now. Thank you.

Awesome!  We just started supporting 3.9.5, so glad to see this worked.


How can I use it in a normal flutter app？

For example, if I have the following codes:
Future<void> main(List<String> args) async {
  WidgetsFlutterBinding.ensureInitialized();

  // if I put it here, the UI won't start, I guess
  /*
  runApp(GetServer(
    getPages: [
      GetPage(name: '/', page:()=> Home()),
    ],
  ));
  */

  runApp(MultiProvider(
    providers: [
      ChangeNotifierProvider(create: (context) => AppModel()),
    ],
    child: MaterialApp(
      title: TITLE,
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyApp(),
    ),
  ));
}

class MyApp extends StatefulWidget {
  @override
  _MyAppState createState() => _MyAppState();
}

class _MyAppState extends State<MyApp> {
  @override
  Widget build(BuildContext context) {
    return Container();
  }
}

How can I run the server in the background while still have the UI working properly?

You are missing somethings, the server and the app are two different applications, two different projects.
Think about your server it will be some API that you will be consuming on your app.

OK, the dart is a programming language.
Not a frontend framework.

OK, the dart is a programming language.
Not a frontend framework.


Rumble when Switch on rg351p

Is it possible to add the choice to make the rg351p rumble or not when switched on ?
Thank You.

I wish.  It's currently not possible unless Anbernic provides a solution for this which I'm not holding my breath for.  :(

I wish.  It's currently not possible unless Anbernic provides a solution for this which I'm not holding my breath for.  :(


Add requirements.txt

Add list of project dependencies in requirements.txt.

We should use pip compile instead of generating requirements.txt manually.

We should use pip compile instead of generating requirements.txt manually.

How does this work? Do you have link?

We should use pip compile instead of generating requirements.txt manually.

How does this work? Do you have link?

https://github.com/jazzband/pip-tools

We should use pip compile instead of generating requirements.txt manually.

I've updated the requirements.txt using pip-compile.

Moving this PR to #3.


notes: 2.2.0 -> 2.2.1

Description of changes
https://github.com/nuttyartist/notes/compare/v2.2.0...v2.2.1
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

No need set QT_STYLE because : https://github.com/NixOS/nixpkgs/pull/210077


Cursor cannot be placed after whitespace at end of wrapped line when using 'contenteditable'.

Issue
When using mode contenteditable, you cannot place a cursor at the end of a wrapped line if it ends in whitespace.
In browser textarea, browser contenteditable and CodeMirror textarea, when you click the white space at the end of a line, the cursor appears on the same line, after the whitespace. In CodeMirror contenteditable, it appears on the next line.
This issue is also apparent when using the "End" key to jump to the end of the line, as the cursor will be drawn on the next line down.
It can also be apparent when using "Up" and "Down" keys, as the cursor will appear to jump from the start of a line to a later point in the line, below the last character of the previous line. This looks very wrong.
Unfortunately my use case for CodeMirror requires contenteditable and I cannot use textarea.
Demo
Here is a demo, showing examples with CodeMirror using contenteditable and textarea, as well as  examples using browser versions.
https://jsbin.com/maharafaca/1/edit?js,output
Possibly related issues:
#5415
#4840
#4841
CodeMirror: 5.53.2
Browsers: Chrome 81 on Mac, Firefox 72 Mac

The editor uses the browser's native cursor in contentEditable mode. As your demo shows, Chrome will always show a cursor at a wrap point on the next line. Firefox does have support for something like this, but it works differently than what CodeMirror does by default—it'll require an additional arrow key press to move across to the next line.
It might be possible that this can be fixed in Firefox. I think with Chrome, that is unlikely to be possible.

I'm confused -- surely my demo shows that Chrome does not show a cursor on the next line?
If CodeMirror uses the browser's native cursor, why is the behaviour different to the browser contenteditable div?

That might be a Linux/macOS difference—Chrome linux doesn't allow me to place the cursor after the whitespace on the wrap point at all.
CodeMirror handles cursor motion itself, and then updates the DOM selection to match it. Its internal model does distinguish between these cases, as do browser's selection models, apparently, but the JavaScript DOM selection interface is very simplistic, and doesn't provide a direct way to control this. It's possible that there's a kludge that could make this work, though. I haven't looked into it yet.

I don't think it's an OS difference -- I have tried the same thing on Ubuntu 20.04 with both Chrome and Firefox and I see the same behaviour, so I'm not sure why you are unable to place the cursor after the whitespace, unless it's down to whatever flavor of Linux you are running.
Just to make sure we are talking about the same thing, here is a screencast of the issue (Chrome, Ubuntu):

If that is not the behaviour you are seeing locally (for all 4 textual areas), that is also strange.

Oh, right, it seems I can place the cursor there with the mouse, just not with the arrow keys. That's good news, I guess, since it means Chrome is capable of drawing such cursors, and we just need to find a way to create them with a script.

Well, it's good news that there may be a potential fix!
For anyone experiencing the same issue - as a workaround for my case, I capture the mousedown event and any keydown events which could cause this effect (Home, End, Cmd-Right, Up, Down, etc) and tell CodeMirror to ignore them by setting a truthy codemirrorIgnore property on the event.
Unfortunately this means that some CodeMirror features I was using like the matchBrackets plugin and multiple cursors no longer work, but it's more important for me that the navigation keys work as they do when handled by the browser. I'm not sure what other potential consequences this might have.
@marijnh
It doesn't seem possible to tell CodeMirror to ignore an event from a key that is bound in the extraKeys option? I think it would be a nicer solution to ignore a key in there, as it would be in a position to take advantage of all the cross-platform compatibility things that CodeMirror does.


2284 break up external compaction it

Closes #2284

I confirmed all the ITs pass < 10 min. I will take a deeper look into the code this evening.

@Manno15 - confirming that that tests pass for you in the allotted time is enough for me. I didnt really change the tests too much, just reorganized them and modified them to extend AccumuloClusterHarness instead of ShardMiniAccumuloCluster. Up to you if you want to delve into the code...

I tried to group the methods that had the same dependencies to reduce the amount of duplicated code. I figured that the method names were descriptive enough.


View and Edit pages will wait for their VM/Pool to be loaded

Issue #574 was caused when a user would enter the app at a view or edit page with a VM or Pool that would be on a fetch page >1. The VM/Pool would be requested but often would either fail the request or would not arrive prior the initial data load flag being set true. The concurrency issues would leave the user either on a blank page or redirected to the card view page.
A number of changes, have been made to eliminate this issue:

App component will wait for login to complete before rendering routes. This will allow the route Pages to assume all relevant login data needed is available so additional API calls may be made.

Route titles (visible in the breadcrumbs) will show the requested VM or Pool UUID until the VM or Pool name is available from the store.

Each detail/edit route page will fetch the VM or Pool requested if it is not already available in the redux store.

Each detail/edit route page will sit with only the app header and breadcrumbs rendered until the VM or Pool is available in the redux store. The "Loading..." indicator should be present while fetching data.

Since the refresh scheduler fetches data from the REST API, wait until the login saga is successful before starting. This prevents erroneous refreshes if the initial load fails or takes longer then the scheduler delay.

The single VM fetch REST call can now follow all the data paths like the fetch by page does. This reduces the number of REST calls required from 6 to 1. It now also supports both shallow and deep fetches.

The fetchSingleVm saga is upgraded to tag the VM IDs to the redux state that it has successfully fetched.

The fetchSinglePool saga was upgraded in the same way.

The redux store state.vms reducer can now keep track of the VM/Pool IDs that are loaded independent of those loaded by the paged loader. Actions and constants were added in support of the reducer changes.

Selectors were added to fetch those 2 lists from the store.

The refreshData saga was upgraded so it refreshes both the individually fetched and page fetched VMs and Pools without dropping any of them from the redux store. The removal of 'stale' VMs and Pools from the store was moved to this saga so it can be done once and preserve ALL VMs and Pools regardless of how they were fetched.

Additionally:

The 'selecting VM details' unit test needed to be fixed by initializing a mock store in the test. Specifically the selectors needed to be initialized or they throw an exception. The initial state of reducers used by selectors are now exported to be used in the unit test.

Login saga refactored to unwind the nested if blocks and improve readability.

Removed isPool prop from VmDetail since it can be derived from other props.

This change is

Rebase is pending.  I'll push a new rebase/merge commit so the existing comments don't go wonky.

The Reviewable tool keeps history.

Actually I think, this fix could be more simple. @jniederm PR was just merged, in which he monitoring requests. With some changes, we could monitoring certain single VM fetching and find out if it finished successful or if it fail. @sjd78 @mareklibra @jniederm what do you think?

I would not extend the low-level HTTP request monitoring for this kind of functionality, I'm afraid of introducing spaghetti this way.
Anyway, I agree this can be done simpler way. Copy paste of my former inscribed comment as a proposal:

Do I understand correctly, that this state/actions around *FetchedById are just because the regular refresh reads recently the paginated data only? Means the VM requested when enetering the application via URL with VM ID would not be regularly refreshed?

If so, I would like to avoid this extra level of state information.
It might be enough for the refreshData to check the state for list of recent VMs (those already downloaded for whatever reason) and compute difference between those already stored and reloaded with pagination.

I believe this would be much simpler and less error prone in the future since we will maintain just a single list.

Two main points to deal with after the first set of reviews:

refreshData can probably be done without keeping a separate list of VMs fetched by id.  It wasn't obvious (to me at least) until I moved the "remove missing" bits up to the refreshData() function itself.
For an item view or edit Page, the item needs to be fully loaded to the redux store (a deep fetch is needed and if the VM/Pool was fetched by the page, it will only have a shallow fetch).  As @jniederm suggests, there is a better way to handle the fetch in the Pages so as not to anger react.

Additionally, and although not explicitly stated, I now understand that there is a dual use of store.vms.vms.  First is to drive the card view and left nav.  The store only needs minimal VM info to support that functionality.  Second is to drive the VM detail view and edit pages.  The store needs to fetch additional data to support those views.
Makes sense to me to break those 2 uses apart.  Keep the data for the lists in one place, and keep a full VM in another place.  The second place can keep multiple VMs just in case a user opens multiple tabs (or some such other silliness).  If you guys agree that change would make sense (@mareklibra , @jniederm , @bond95), I can create an issue and work on it in future.

These last 2 commits should address almost all of the changes requested.  The one I know I haven't addressed yet is @jniederm's suggestion:

Hence I suggest to use fresh new shiny getDerivedStateFromProps() callback (which is called both on mount and props change and allows firing redux actions) that would force-fetch the VM whenever props.match.params.id is changed. (State initialization would be necessary.)

I'll work that change out next.

Makes sense to me to break those 2 uses apart. Keep the data for the lists in one place, and keep a full VM in another place. The second place can keep multiple VMs just in case a user opens multiple tabs (or some such other silliness).

@sjd78 , not sure if I understand, so please correct me. I would like to avoid serving redux store hierarchy these purely UI needs. The React component composition can be changed very quickly, i.e. there's redesign effort in progress. The redux store hierarchy should reflect actual data set of the application and let's React derive representation for it. Duplication of data in the store does not sound good to me.

Makes sense to me to break those 2 uses apart. Keep the data for the lists in one place, and keep a full VM in another place. The second place can keep multiple VMs just in case a user opens multiple tabs (or some such other silliness).

@sjd78 , not sure if I understand, so please correct me. I would like to avoid serving redux store hierarchy these purely UI needs. The React component composition can be changed very quickly, i.e. there's redesign effort in progress. The redux store hierarchy should reflect actual data set of the application and let's React derive representation for it. Duplication of data in the store does not sound good to me.

@mareklibra - It's just finding a balance between keeping the data in the store more like the REST data (think DataAccessObject) or transforming it to be more like the view (think ViewObject).  I'm ok with leaving the store, reducers and sagas as-is.

Hence I suggest to use fresh new shiny getDerivedStateFromProps() callback (which is called both on mount and props change and allows firing redux actions) that would force-fetch the VM whenever props.match.params.id is changed. (State initialization would be necessary.)

Done!  Works great.
Now anytime the selected VM or Pool id changes for a view or edit page, the VM or Pool will be refreshed by the Page itself.  Note: The selected id changes when the page is reloaded (nothing to something) or react-router changes the route match props (link navigation).

Also this PR fixes https://github.com/oVirt/ovirt-web-ui/issues/563

git pull merge removed :smile:
Commits have been squashed down to 5 commits.  I can squash b2f0d1d into cce4073 if the PR needs to be fewer commits.

@bond95, @mareklibra - You guys want to take another look at this PR?  I'd like to get it merged sooner then later if possible.  Thanks.

rebased

LGTM
I'm about to pre-release 1.4.0 so let's see this live.


Lotsa javadoc fixes, other changes to resolve warnings

Looks good, approved!

I can't merge, eric will have to.


Option to delete notifications

Is your feature request related to a problem? Please describe.
Currently there is no option from app/at client to delete notifications
Describe the solution you'd like

change notification syntax and verb handler to delete notification

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@murali-shris should we move this up so we can work on in PR43/44?

This is more a question for Kevin / Colin as it is a prioritization call


Load PulseAudio's module-device-manager only for KDE

See http://lists.freedesktop.org/archives/pulseaudio-discuss/2016-February/025385.html. tl;dr: on non-KDE systems it causes weird behavior with streams not respecting fallback devices.

We may want to backport this to stable, too.

Should break nothing, I'll merge this in several days unless somebody objects...


Develop a Fiori Launchpad App with Your Own Dev Tools

Mission URL: https://developers.sap.com/mission.sapui5-cf-launchpad.html
Well, this tutorial have a lot o issues:
In Tutorial 7 - Display Data from the Northwind Service Step 5 Redirect traffic to data source

I think that "welcomeFile" : "/cp.portal" is wrong, at least it doesn't work for me because I get "SiteNotFound" when I do.

This ui5.yaml is TOO sensitive, you must to respect the identation or it will not work.

The favorite button, try to choose one product as favorite and try to navigate to other products, for now on all products are favorites !!!

In Tutorial 8 Step 6 Add hierarchy information it's state to put the code what I think is only PART of it after the model section, it's doesn't work for me, I tried many combinations and the only one that worked for me is to add this is code right after "sap.ui5" declaration (too sensitive too).

I can't deploy this project more than once, if try i get a lot of issues.

Step 1: Build the projetct - I can build the MTA file but there is no log to see errors and there are errors.

Step 2: Deploy the built archive - I was able to a sucessful deployment only once and never again, I try to redo this tutorial a lot of times and is always the same, I get errors with _deployer. I try to change the memory/disc parameters and I try to a lot of stuff I read on internet but nothing !!!

I'm sorry that you faced so many issues during the tutorial. This shouldn't happen! I'm not sure if I can solve all the points you mentioned but I can definitely comment on all points :)

I think that "welcomeFile" : "/cp.portal" is wrong, at least it doesn't work for me because I get "SiteNotFound" when I do.

This is not wrong, cp.portal is the right value for this property. I assume it didn't work because the "Portal deployed" app failed in your case. The link to this location won't work if that task is not completed successfully.

This ui5.yaml is TOO sensitive

I feel your pain. YAML is quite nice to read but writing can be horrible. If you do edit this file often, I definitely recommend using VS Code plugins to make your life easier like https://marketplace.visualstudio.com/items?itemName=redhat.vscode-yaml or https://sap.github.io/ui5-tooling/pages/Configuration/#validation-ide-support
I didn't want to include too many non-ui5 things into the tutorial but maybe I should add these tools.

The favorite button...

The button is not supposed to do anything but being a toggle button (as there is no real controller logic associated with the control).  I'll add this remark to the tutorial to avoid confusion, thanks.

Add hierarchy information

You can add this new section at any "child position" of the sap.ui5 node. I'm not sure why it didn't work after the "models" property. Maybe the ShellUIService node was attached to the wrong parent node? JSON is the other industry-standard config-approach next to YAML. In this case, writing it is earlier but reading is harder.
If you do these tasks often, you can leverage another IDE extension to be supported during editing http://json.schemastore.org/ui5-manifest

Step 1: Build the project

So you're saying the build works fine, right?

Step 2: Deploy the built archive

This problem has already been addressed in #5108 and #5115
I hope I could help

Thank you for the enlightment !!

Awesome! Does this mean the issue is resolved for you? If so, please go ahead and close the issue.


[extractor/pornhub] Fix extractor

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

closedCaptionsFile field on the page is not absolute URL (anymore?) but relative one.
This change makes extracting subtitles working again.
I understand this change doesn't add/update tests. I'm not sure it this type of changes could be tested with the current _TESTS section. If you could give me some hints I'll try to update this PR though.
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

python test/test_download.py TestDownload.test_PornHub_3
I'm not sure how to fix that one.

        'info_dict': {
            'id': 'ph601dc30bae19a',
+           'ext': 'mp4',

I thought keys in that dict were supposed to be returned by the extractor. That's not the case and it is actually used as input?

was this intentional? 9ecbbcd

No, it wasn't. My bad. I have reverted that one.

in the future, try to avoid force-pushing during review

Noted, I'll only add new commits to PRs. Are all commits in PR going to be squashed before merge?

Just a side note for a context why I did force push: in #5617 I was advised to not create separate PR for update of tests. I thought that I would start with making tests pass before fixing bug in the extractor related to subtitles. I thought it was cleaner to reorder commits (which is why I made that force push) rather than creating revert of the single commit with subtitles and to add it back after fixing tests.

Are all commits in PR going to be squashed before merge?

Yes. And as I just commented in #5636#issuecomment-1326747242, I think this PR (#5618) should absorb the changes being made/discussed in #5636 and #5624 and supersede those PRs. Keeping it all in 1 PR will be cleaner/simpler/easier.
Some relevant discussion points from the other PR:
https://github.com/yt-dlp/yt-dlp/pull/5624#issuecomment-1325623562
https://github.com/yt-dlp/yt-dlp/pull/5624#issuecomment-1326532013
From this information I found a way to make a successful JSON request. The platform cookie needs to be set as mac and a Safari/MacOS UA header needs to be passed, like this:
         self._set_cookie(host, 'age_verified', '1')

-        def dl_webpage(platform):
+        def dl_webpage(platform, headers={}):
             self._set_cookie(host, 'platform', platform)
             return self._download_webpage(
                 'https://www.%s/view_video.php?viewkey=%s' % (host, video_id),
-                video_id, 'Downloading %s webpage' % platform)
+                video_id, 'Downloading %s webpage' % platform, headers=headers)

-        webpage = dl_webpage('pc')
+        headers = {'User-Agent': 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.5 Safari/605.1.15'}
+        webpage = dl_webpage('mac', headers=headers)

and
             if '/video/get_media' in video_url:
-                medias = self._download_json(video_url, video_id, fatal=False)
-                if isinstance(medias, list):
+                medias = self._download_json(
+                    video_url, video_id, fatal=False, errnote=False, headers=headers)
+                if medias is None:
+                    self.write_debug(f'{video_id}: get_media endpoint did not return valid JSON')
+                elif isinstance(medias, list):
                     for media in medias:
                         if not isinstance(media, dict):
                             continue

But I don't think we should hardcode a single Safari UA. Maybe we could somewhat randomize the version numbers, or have a list of UA strings. And even then, it's probably only a matter of time until PH puts a stop to this method too.

The extractor has two levels of abstraction over the default web access APIs: the base _download_webpage_handle() that is meant to detect the need for JS captchas and the local function dl().
Should the UA be applied in all requests by modifying the base _download_webpage_handle() instead?

The suggested UA/cookie combo doesn't help yt_dlp to avoid cornhub instead of JSON with Py3.9, nor yt-dl to avoid cornhub instead of the webpage with Py3.9. I would have fingered yt-dlp's 3.11 security back-port if the yt-dlp case worked.
cornhub is returned without a redirect, masquerading as the original URL, but the pattern r'(?i)>Tons\s+of\s+Hard\s+Cobs\s+getting\s+Grabbed' gives it away.

I want to add some information for issue https://github.com/yt-dlp/yt-dlp/issues/4298
for https protocol sometimes it available to download and sometimes its not available. for example a few months the https protocol is still available and then its gone. then a few weeks ago suddenly it showed up again and then gone again.
usually the file size of https protocol is larger than its hls counterpart and have a significance difference.
whats your opinion about this?


use yaml.safe_load instead of load

Fix #103 by replacing all calls to load by safe_load. This exists
for both ruamel and PyYAML.

Coverage remained the same at 87.2% when pulling 41c2d5ae2b962389e5298324cf0f9226ad051bba on jarondl:yaml-safeload into e7fac6a5f5d27a472b206b896f684ee83650b5be on Grokzen:unstable.

Manually merged it in commit 74afb81cde8175a0dc4d35cd69bea808bf36fc78


[sokol_gfx] Any plans to support base vertex location?

When using a single big vertex+index buffer to store multiple meshes, it would be useful if sg.draw also had a base vertex location parameter. This way it wouldn't be necessary to pre-process the index buffers and add the offsets manually. So essentially the GPU would load vertex from index_buffer[i] + vertex_offset.
This feature is supported in D3D11 and OpenGL 3.2+, however I'm not sure if all other APIs sokol targets support it. Maybe each backend could report an error if the base_vertex parameter is used and not supported. There could also be a way to query the support.
Another issue is that adding a new parameter to sg_draw would break existing code, which isn't very good considering how important this function is...

I just noticed vertex_buffer_offsets from Bindings struct, I guess that does something similar? It would still be nice to have that info on a per-draw basis, but this could also be useful...

Right, that's cool. However I'm mostly concerned with D3D11, which seems to bind everything all the time at the moment... (unless the runtime/driver can skip it?)
I think I'll just do apply_bindings every time for now, the renderer uses instancing as much as possible so the number of draw calls should be low anyway. If I run into perf issues I could just write the hot inner loop with d3d11 directly by bypassing sokol

However I'm mostly concerned with D3D11

...yeah I didn't specifically benchmark yet how much filtering redundant bindings would actually help in D3D11, but I really don't think it's a performance issue. First, D3D11 already uses 'batched' bindings calls (e.g. all bindings of one type and shader stage are applied with a single D3D11 call), and second, IME D3D11 is really well optimized internally, so if it helps I would expect that they do the redundant-bindings-filtering internally.

PS: what might make sense in D3D11 though is skipping the D3D11 binding calls by resource types... e.g. if you call sg_apply_bindings() with only the vertex offsets changes, don't do the other binding calls (IASetIndexBuffer, VSSetShaderResources, PSSetShaderResources, VSSetSamplers, PSSetSamplers).
...e.g. if you find that writing the hot inner loop directly in D3D11 is significantly faster because of those redundant bind calls, just holler and we can figure something out.

Also I just found out the vertex_buffer_offsets don't seem to work with indexed meshes (at least on D3D11), which is unfortunate...

Hmm, it should. This sample is using indices:
https://github.com/floooh/sokol-samples/blob/master/sapp/bufferoffsets-sapp.c
Also the sokol_imgui.h header is using both vertex_buffer_offsets and index_buffer_offset:
https://github.com/floooh/sokol/blob/c2bb83f0b35e09d97a354b5f4cf4c3df783c4193/util/sokol_imgui.h#L2682-L2684
Your index ranges for a 'submesh' need to be zero-based though. E.g. an index 0 points to the vertex at the vertex_buffer_offset.

PS:

So essentially the GPU would load vertex from index_buffer[i] + vertex_offset.

That's what's happening, except that the vertex_offset counted in bytes, not in 'number of vertices'. Maybe that's the problem?

Right, I understand the offset is in bytes and the vertex indices are zero based, so it looks like I was just doing something wrong. Thanks for the examples!

If you still discover a bug with the buffer offsets, let me know!


GDScript

https://monnef.gitlab.io/golden-gadget/

Fits with #21.

https://monnef.gitlab.io/golden-gadget/

This is very similar to https://github.com/Dr-Dan/godot-fakeo-db.

List all the editor plugins:
VSCode:
https://github.com/godotengine/godot-vscode-plugin
Idea, PyCharm and other Jetbrains IDE's:
https://plugins.jetbrains.com/plugin/13107-gdscript
Emacs:
https://github.com/godotengine/emacs-gdscript-mode
Sublime:
https://github.com/beefsack/GDScript-sublime
Vim:
https://github.com/habamax/vim-godot


Docker installation

Hi..any budy have a document of Docker installation on Windows server 2019/windows10 VM's. I tried the installation, but getting an error while switching from windows containers to Linux containers. Is it possible to install Docker on Windows VM's without enabling virtualization on VM?
Because we can't enter into BIOS on VM.The Same error it throwing on Win10Professional & Winserver2019 EE

I think you should definitely have a virtualistion (VM ware ) for installing docker

But in VM's we can't enable virtualization ..how to resolve this issue?

Hi Rao,
Can you please go through below link hope it may help you
https://4sysops.com/archives/install-docker-on-windows-server-2019/

try to take linux vm

Thanks for ur support


Data peer review panel for Voyager Uranus Parkes data restoration

Potential reviewers: Dick French and Marzia Parisi.  Also Richard Chen and Dustin Buccino for PDS review.

Richard submitted his review 10/12/24

Dustin submitted his review 18/12/24


Update sbt-coveralls to 1.3.9

About this PR
📦 Updates org.scoverage:sbt-coveralls from 1.3.1 to 1.3.9
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-coveralls" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scoverage", artifactId = "sbt-coveralls" }
}]

labels: sbt-plugin-update, early-semver-patch, semver-spec-patch, commit-count:1

coverage: 33.621% (-0.3%) from 33.898% when pulling 90a82bac1c815420279d86c607c0e7b20fa4ea23 on scala-steward:update/sbt-coveralls-1.3.9 into d082b04ae38161e522ab7991a62ceff130832f4e on evolution-gaming:master.

Superseded by #108.


[FEATURE] buffered channel drain next

Summary
This is a version of calyptia/plugin#34 which removes one of the intermediate buffers and also makes the size of the buffer channel configurable via the go.MaxBufferedMessages configuration option.
I have also added tests to make sure that this new model works and does not run into deadlocks.

I sent my comment's diff as a PR: https://github.com/fluent/fluent-bit/pull/7922


Possible to obtain Gamertag's email?

I'm trying to figure out a way to verify someone is in fact the owner of a gamertag before I allow them to attach it to their local account on my site. But unless I can get a Gamertag's email address for their Microsoft LIVE account I can not do this.
I have OAuth implemented for Microsoft already so I can have them link that to get their Microsoft account's email, but unless the XboxAPI can return a Gamester's account's email address I can not confirm them.
If I'm not able to confirm them I'll have to setup some sort of "Report this Gamertag as mine" system for when someone adds someone else's Gamertag to their account.

Unfortunately the API does not return the email address of a gamertag.
If your looking to validate a profile, then do it like the other sites and ask the user to change (or add something to) the bio. Then you can check this unique code is present etc.

Ah thank you. I will implement that verification method as well as the reporting methods.


Visualization doesn't get video？

I'm generating 6 png images instead of videos, how can I fix it

@MRYUjg I'm having the same problem, may I ask how you solved it?


About Image Size

Hi~
I want to know what the image size is recommended for single pair matching.
Thanks.

There is no limitation on the image resolution if using the recurrent zoom-in strategy. All cropped patches will be processed at 256x256.

I got it. Thanks.


[Logistic] Meeting notes 201023

prepare for advertising content
submit for review today and hope it can get through by early next week.

Obsolete, close.


can't load version 0.6.1

Hi,
I've switched to the latest version (from a working 0.5.x), and I got:
ERROR in ./~/vue-material/dist/vue-material.js
Module not found: Error: Cannot resolve module 'Vue' in /home/da/ibt2/node_modules/vue-material/dist

@ ./~/vue-material/dist/vue-material.js 6:83-97
I include vue-material with:
import Vue from 'vue';
import VueMaterial from 'vue-material';
import 'vue-material/dist/vue-material.css';
Am I doing something wrong, or is it a bug?
I've also tried removing the whole node_modules directory and rebuilding it with npm install, to no avail.
Thanks.

Hi @alberanid
Have you installed Vue properly?

@marcosmoura I can confirm that I get the same message. If I import from vue-material/src/index.js and build vue-material into the bundle myself it works fine, but if I bundle the built js (normal and debug) from node_modules I'm getting the same message.

Just chiming in here. I seem to be not having any issues. I have updated to 0.6.1, using the new components fine. Very strange.

@korylprince @alberanid
Running on which build system, OS, Vue version? Webpack, Browserify or Rollup? With babel or another transpiler?

@marcosmoura
Arch Linux,
Vue 2.1.8
Webpack 1.14.0
Babel (core) 6.21.0
I'm wondering if it's a case sensitive thing. I'm digging now to see if I can identify what is going on.

I'm running Arch Linux here too. With the same packages. I've tested in two different projects and created another based on Webpack 2. No problems :(

Okay, then it's almost got to be a webpack thing. Can you post what webpack config you're using for the projects?

Basically the Vue CLI template:
var path = require('path')
var webpack = require('webpack')

module.exports = {
  entry: './src/main.js',
  output: {
    path: path.resolve(__dirname, './dist'),
    publicPath: '/dist/',
    filename: 'build.js'
  },
  resolveLoader: {
    root: path.join(__dirname, 'node_modules'),
  },
  module: {
    loaders: [
      {
        test: /\.vue$/,
        loader: 'vue'
      },
      {
        test: /\.js$/,
        loader: 'babel',
        exclude: /node_modules/
      }
    ]
  },
  devServer: {
    historyApiFallback: true,
    noInfo: true
  },
  devtool: '#inline-source-map'
}

if (process.env.NODE_ENV === 'production') {
  module.exports.devtool = '#source-map'
  module.exports.plugins = (module.exports.plugins || []).concat([
    new webpack.DefinePlugin({
      'process.env': {
        NODE_ENV: '"production"'
      }
    }),
    new webpack.optimize.UglifyJsPlugin({
      compress: {
        warnings: false
      }
    })
  ])
}

@marcosmoura : Ubuntu 16.04, npm 3.5.2, webpack 1.14.0.  If fails just tying to build the project, which. It's a standard config created by vue init webpack. Moving back to 0.5.2 make it works again.
Thanks for your help.

I've found something...
Here's my current webpack resolve section:
resolve: {
    alias: {
        "vue$": "vue/dist/vue.common.js"
    },
    extensions: ["", ".js", ".vue"]
}

If I add "Vue$": "vue/dist/vue.common.js" (notice capital V) to the alias section I can compile and run everything. So there does appear to be some sort of case issue.. Still digging...

Why do you need this alias?

@marcosmoura : I can confirm @korylprince finding.  But no idea why it fix the issue.

@marcosmoura It's from the Vue docs:
https://vuejs.org/v2/guide/installation.html#Standalone-vs-Runtime-only-Build

Okay, so I found what's causing the error message:
https://github.com/marcosmoura/vue-material/blob/aef7021375f0c55522b0942caca409a56192f56a/build/webpack/prod-lib.js#L50
Is there a reason why you're using Vue as the external instead of vue? Changing it to vue fixes the issue for me.

Releasing another update. Im so sorry about so many updates :(

@alberanid @korylprince The new version is working?

It works now. Thank you.

@marcosmoura : 0.6.2 works for me!  Thanks for the fix.


Shard Connections.json into files to make by-hand changes cleaner

Discussed more offline - our guidance here should be just to delete connections.json. That's akin to "log out" or clear browser cookies.


package: improved adding pre-commit during postinstall

postinstall "script" will check if .git/hooks is existing before
trying to create a file in a non-existing directory

Context: during packaging .git directory is removed, npm install is done afterwards


Include Numpy 1.20 in CI

Let's test Numpy 1.14, 1.18, and 1.20 in CI.

Do you want to add an entry to the changelog and then we can merge?

@lgarrison Is the branch safe to delete?


Added --min argument.

This is something I use for filtering outliers, I hope you find it useful!

Thanks so much!


Toast accessibility issues

Bug report
Axe devTools extension has identified 2 accessibility issues on Toast code example
<ol> is not allowed children of role="status".

role="status" is not valid on <li> elements.

Expected behavior
No accessibility issues. Toasts should not be list items.
Reproducible example

Install Axe devTools
Open Radix Toast code example https://codesandbox.io/s/z688kv?module=App.js&file=/App.js
Ensure toast is open by default
Scan page using Axe devTools

Suggested solution
Replace <ol> and <li>'s with alternative elements.
Your environment
Production

Software
Name(s)
Version

Radix Package(s)
Toast
2.0.1

React
n/a

Browser

Chrome

Assistive tech

Node
n/a

npm/yarn

Operating System

I am seeing a couple more issues (along with the ones you listed) with the latest tooltip (repro here)

Focusable content should have tabindex='-1' or be removed from the DOM

So I agree with all of the above and would love to see the Toast component have no accessibility issues out of the box. For those with stringent accessibility requirements who need a workaround for the time being, this reduced my number of issues from an AxeDevtools scan, but did not fix them completely:
import React from 'react';
import * as Toast from '@radix-ui/react-toast';

const AccessibleRoot = React.forwardRef<any, any>((props, forwardedRef) => {
    return (
        <div {...props} ref={forwardedRef}>
            <Toast.Title>Toast Title</Toast.Title>
            <Toast.Description>This is the toast description</Toast.Description>
        </div>
    );
});

export const MoreAccessibleToast = () => {
    return (
        <Toast.Provider>
            <Toast.Root
                open
                asChild
            >
                <AccessibleRoot />
            </Toast.Root>
            <Toast.Viewport asChild>
                <div />
            </Toast.Viewport>
        </Toast.Provider>
    );
};

(Thank you Radix for making the asChild functionality. It's being put to good use!). This allowed me to replace the <ol> and <li> elements with divs. This is obviously a quick / minimal example. Please forgive the anys. I hope this saves someone else some time! It did not solve all of my issues. I haven't found a workaround for the hidden but focusable spans that @willwill96 mentioned.


Adding SRV support for MongoDB

This will enable MongoDB Atlas mongodb+srv:// URLs to be used without specifying the single servers, therefore being able to change and scale any MongoDB cluster without affecting any client.

Closing as there is a url option.


Release azure-cli 2.0.6

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Codecov Report

Merging #3280 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #3280   +/-   ##
=======================================
  Coverage   70.27%   70.27%
=======================================
  Files         391      391
  Lines       25084    25084
  Branches     3813     3813
=======================================
  Hits        17629    17629
  Misses       6344     6344
  Partials     1111     1111

Impacted Files
Coverage Δ

src/azure-cli/azure/cli/__init__.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Started releasing azure-cli 2.0.6...

Release of 'azure-cli' with version '2.0.6' successful. View at https://pypi.python.org/pypi/azure-cli/2.0.6.

GitHub release created. https://github.com/Azure/azure-cli/releases.


Color error messages generated from pyrpipe

Color error messages generated from pyrpipe.
e.g. message generated by execute_command when the command fails

fixed in 970f57247eb00640de07a8c4ed938dec6c04833f


WIP: migrate alpine images to this repo

What's left to undraft this?

What's left to undraft this?

Nothing!


Add test for large batches in DeformConv2d

Follow-up for https://github.com/pytorch/vision/pull/2027

Codecov Report

Merging #2040 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #2040   +/-   ##
======================================
  Coverage    0.48%   0.48%
======================================
  Files          92      92
  Lines        7411    7411
  Branches     1128    1128
======================================
  Hits           36      36
  Misses       7362    7362
  Partials       13      13

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 f2f085b...b5aa349. Read the comment docs.

Thanks for the review Christian!


Cannot add multiple file comments

Type: Bug

Open a file in a PR that doesn't have any file comments yet.
Click Add File Comment button
Add a comment
Click the Add File Comment button again

Expected behaviour: another comment box is shown to add a comment.
Actual behaviour: the current comment is hidden.
Extension version: 0.96.0
VS Code version: Code 1.93.0 (4849ca9bdf9666755eb463db297b69e5385090e3, 2024-09-04T13:02:38.431Z)
OS version: Linux x64 6.8.0-41-generic
Modes:

Duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/5786


update na_cci to use new NA_MEM flag conventions

This is the na_cci change that bridges the gap between the na.h flag changes from @soumagne in Mercury and the verbs flag changes from @scottatchley in CCI.

Merged in f77db0d8f1acd69d3595372e94cc58c19391de37


Refactor affinity module

Description of changes
Adding this kind of functionality complicated things quite a bit, so a pretty major refactor was necessary to keep things manageable. Notable changes include:

build_knn_index no longer computes the neighbors, but only returns the correct index. This lets us pass in already existing KNNIndex objects to affinities.
All Affinity classes can now be called in two ways. 1) with a data matrix, as is standard, or 2) using an existing KNNIndex object. This is in preparation for scanpy integration, where we will have to use existing KNNGs.
KNNIndex objects now have the default parameter `metric="euclidean". This just makes it easier to debug or instantiate manually.
Add nearest_neighbors.PrecomputedNeighbors which allows us to wrap the standard KNNG representations.

Because there are now a whole bunch of possible ways to call this API, I added a test module which runs over the different ways to create an iris embedding. See tests/test_different_usages.py.
Includes

[X] Code changes
[X] Tests
[ ] Documentation

Wow, this precomputed thing escalated pretty quickly :)

Haha, yeah. Things got pretty complicated when I just patched things up. So I tried to simplify it, but all the different ways we can call things now and string them together have made things a bit messy and necessarily complicated.
I'm trusting that my tests are thorough enough that nothing broke here. Most of the major things are tested pretty well though.