Add site branding and colors from visual work done for CopticScriptorium.org

(kept CopticScritorium.org palette, just desaturated a bit for reading)

Will have to do a hard refresh (or visit http://coptic.aweb.io/static/img/favicon.ico) to see favicon

Use favicon with manuscript background, use asul font, use original colors without being desaturated, add full logo instead of partial

Add footer content and sponsor logos

Okay, updated colors, branding and layout.  Will have to fix up responsive styles, but this should get the branding moving in the right direction regarding our most recent Skype call.

P.S. May need to do a refresh to see updates, esp. for favicon.

Oh here is the ticket -- see my comments here.  https://github.com/CopticScriptorium/cts/issues/39
Thanks for the efforts!  Some  of it looks really good, but some of it ended up not quite right.  I have some suggestions in the other thread.

Also see my email re the footer: The footer content is not quite the same (namely, the license). Can you make the content the same?  I have added an issue to github about licensing of your code.  Everything else is cc-by.  We neglected to add it to our footer until today.  Your copyright notice reminded me!  We are using open culture/open source licenses as much as possible.
If there is some way for your footer script to call our footer content, that would be great -- then we don't have to redo yours when we redo ours.  (this is in email)

Thanks for all the specific feedback!  I'm making sure I don't lose any of it by relisting items from the email here:
-please add our floating nav bar to the top or put a link to the main
page-- we will add a link to the urn resolver on our nav bar, as well,
when we are done
-put the URN box back on the right where it was before [or on the far
right of your menu bar]. that had a very clean look
-you can keep our banner on the top with the manuscript image in the
background or you can just use the banner logo w/ white background but
please get rid of the manuscript image for the rest of the page.  If you
want to make it look like our other pages, that background image is
brightened substantially -- see our css
https://github.com/CopticScriptorium/CopticScriptorium.github.io/blob/master/scriptorium.css
with     background-color: hsla(40,53%,100%,0.80);  The ms image has an
80% brightness on our homepage for the content box.  Given the format
you have here with the rows I recommend eliminating it or fading it out
dramatically in the background of the content area.
-For now keep the font and the brown background for the menu-- that
looks good
-footer:  I am making a ticket about licensing your code.  In the
meantime, could you please make the footer look exactly like our main
site footer (including the licensing instead of copyright); yours looks
good, and I think the main thing is needing to change the copyright. We
neglected to add our cc-by license it to our footer until today.  Your
copyright notice reminded me!  We are using open culture/open source
licenses as much as possible.  if there is some way for your footer
script to call our footer content, that would be great -- then we don't
have to redo yours when we redo ours.

Ok. Sounds good. Please also check the mobile layout. I notice copticscriptorium.org resizes the banner, etc, for mobile layout. The one in this site does not resize and so takes up a ton of screen space. Thank you!
<EMAIL_ADDRESS>Sent from my iPhone

On Mar 20, 2015, at 5:50 PM, Luke Hollis<EMAIL_ADDRESS>wrote:
Thanks for all the specific feedback! I'm making sure I don't lose any of it by relisting items from the email here:
-please add our floating nav bar to the top or put a link to the main
page-- we will add a link to the urn resolver on our nav bar, as well,
when we are done
-put the URN box back on the right where it was before [or on the far
right of your menu bar]. that had a very clean look
-you can keep our banner on the top with the manuscript image in the
background or you can just use the banner logo w/ white background but
please get rid of the manuscript image for the rest of the page. If you
want to make it look like our other pages, that background image is
brightened substantially -- see our css
https://github.com/CopticScriptorium/CopticScriptorium.github.io/blob/master/scriptorium.css
with background-color: hsla(40,53%,100%,0.80); The ms image has an
80% brightness on our homepage for the content box. Given the format
you have here with the rows I recommend eliminating it or fading it out
dramatically in the background of the content area.
-For now keep the font and the brown background for the menu-- that
looks good
-footer: I am making a ticket about licensing your code. In the
meantime, could you please make the footer look exactly like our main
site footer (including the licensing instead of copyright); yours looks
good, and I think the main thing is needing to change the copyright. We
neglected to add our cc-by license it to our footer until today. Your
copyright notice reminded me! We are using open culture/open source
licenses as much as possible. if there is some way for your footer
script to call our footer content, that would be great -- then we don't
have to redo yours when we redo ours.
—
Reply to this email directly or view it on GitHub.

Okay, I've updated these items:

[x] Added floating nav bar
[x] Put urn box back to the right
[x] Removed parchment background for body; parchment color in header is the same color as the parchment color on the copticscriptorium.org site--the background-color with an 80% opacity is being set to the #content element on the copticscriptorium site.  Where possible, I used the copticscriptorium styles directly to help things look consistent.
[x] Updated footer content--currently, there's not a great way to pull the copticscriptorium.org footer content directly, but I'll keep thinking about this.

Let me know your thoughts, and if everything looks good, I'll update the responsive styles so that the site looks good on tablets/mobile devices.
Thanks!

Much better, thanks.  Still not quite right with the header on mobile but much better, thanks!  See screen shots below for examples:
What it looks like:
What our site looks like and this should look like solely in terms of the banner:
Sent from my iPad

On Mar 20, 2015, at 7:20 PM, Luke Hollis<EMAIL_ADDRESS>wrote:
Okay, I've updated these items:
Added floating nav bar
Put urn box back to the right
Removed parchment background for body; parchment color in header is the same color as the parchment color on the copticscriptorium.org site--the background-color with an 80% opacity is being set to the #content element on the copticscriptorium site. Where possible, I used the copticscriptorium styles directly to help things look consistent.
Updated footer content--currently, there's not a great way to pull the copticscriptorium.org footer content directly, but I'll keep thinking about this.
Let me know your thoughts, and if everything looks good, I'll update the responsive styles so that the site looks good on tablets/mobile devices.
Thanks!
—
Reply to this email directly or view it on GitHub.

Okay, I've updated the responsive styles to make everything look good on smaller browsers.  This ticket should be ready to close.

Yay! Last two things: the visited links in the footer are the wrong color, and we need to add the Apache license somewhere, probably in the footer. Tell me if I should add it to our main footer or if you will add it here.
<EMAIL_ADDRESS>Sent from my iPhone

On Mar 20, 2015, at 7:56 PM, Luke Hollis<EMAIL_ADDRESS>wrote:
Okay, I've updated the responsive styles to make everything look good on smaller browsers. This ticket should be ready to close.
—
Reply to this email directly or view it on GitHub.

Okay, sounds good--update the visited color and added the apache license link and text.  Let me know your thoughts.

Looks good--please close!  Hurray!
<EMAIL_ADDRESS>Sent from my iPhone

On Mar 20, 2015, at 8:12 PM, Luke Hollis<EMAIL_ADDRESS>wrote:
Okay, sounds good--update the visited color and added the apache license link and text. Let me know your thoughts.
—
Reply to this email directly or view it on GitHub.

Sounds good! Closing.


Fix badges in README

The README badges for the build status and docker pulls need to be updated. The GitHub workflow has changed names, and the image is now hosted on GHCR instead of Docker Hub.

This was fixed by commit dc84c6a03ad4525e97b273ded8bb9354686e5f81.


elixir 1.17

update test matrix
update credo to 1.7.7

Pull Request Test Coverage Report for Build bdb9c90768958f1f84f849c8bf62f26e78ab9ada
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.233%

Totals

Change from base Build 58755aa00d891b574f480564a36b21b6b08fb1ba:
0.0%

Covered Lines:
906

Relevant Lines:
913

💛  - Coveralls


chore(GroupTheory/GroupAction): split off Defs file

The goal of this PR is to have a file with the definitions for the orbit and stabilizer of a group, and enough theory to define quotient groups.

[x] depends on: #18340

bors merge


[Bug]:  Error: failed to find allowed user domains, err = object does not exist

Terraform CLI Version
1.9.2
Terraform Provider Version
0.93
Terraform Configuration
Upgraded the provider from 0.92 to 0.93

Category
category:resource
Object type(s)
No response
Expected Behavior
After align resources with new 0.93 standards, the plan should be successful.
Actual Behavior
The resource snowflake_saml2_integration fails with the below message message.
Reverting to snowflake_saml_integration doesn't help. I get the same error.
│ Error: failed to find allowed user domains, err = object does not exist
│
│   with snowflake_saml2_integration.saml_integration,
│   on snowflake_aad_provisioner.tf line 31, in resource "snowflake_saml2_integration" "saml_integration":
│   31: resource "snowflake_saml2_integration" "saml_integration" {

Steps to Reproduce
Upgrade provider to 0.93 with a deployment containing snowflake_saml2_integration.
How much impact is this issue causing?
High
Logs
No response
Additional Information
No response
Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

Hey @nicolamarangoni. Thanks for reaching out to us.
Please share the config and the DEBUG logs.

This is our config:
resource "snowflake_scim_integration" "aad_provisioning" {
  name        = "AAD_PROVISIONING"
  run_as_role = "AAD_PROVISIONER"
  scim_client = "AZURE"
  enabled     = true
}

resource "snowflake_saml2_integration" "saml_integration" {
  name                      = "AAD_SAML"
  enabled                   = true
  saml2_enable_sp_initiated = true
  saml2_provider            = "ADFS"
  saml2_issuer              = "https://sts.windows.net/3825a6f3-24cb-47d4-8aa2-35d3e5891324/"
  saml2_sso_url             = "https://login.microsoftonline.com/3825a6f3-24cb-47d4-8aa2-35d3e5891324/saml2"
  saml2_x509_cert           = jsondecode(data.aws_secretsmanager_secret_version.azure_app.secret_string)["saml2_x509_cert"]

terragrunt.log

Adding something like allowed_user_domains = ["example.com"] doesn't fix it

Thanks! cc: @sfc-gh-jmichalak

Hi @nicolamarangoni 👋 We're assuming that we get user domains from Snowflake describe, but for some reason it's probably missing here. Can you check that ALLOWED_USER_DOMAINS is listed in DESCRIBE INTEGRATION AAD_SAML?

Same here.

@gbatiz @nicolamarangoni Thanks for your feedback, this is undocumented Snowflake behavior and we're investigating it internally.

@gbatiz Until we know if this is a Snowflake issue or intentional undocumented behavior, you can use provider alias for the same resource (so that only saml resource works on the previous one but you use the newest for the databases and all other objects).

@gbatiz Until we know if this is a Snowflake issue or intentional undocumented behavior, you can use provider alias for the saml resource (so that only saml resource works on the previous version but you use the newest for the databases and all other objects).

Reading the docs, I am not sure how to do this, if possible. Version is not something the providers aliases can differ on. Do you have a working example of this by any chance?

I don't have a working solution share, but this is a bit tricky solution, so after giving it a second, maybe we shouldn't have recommended it.
Still, the alternative would be to split into different deployments but this is a more time-consuming solution.
We are investigating internally the encountered behavior; as a last resort, we will provide a workaround in v0.94.0.

Thanks! For now we commented out the resource in and removed it from terraform state. Once the issue is resolved, we import it back.

@gbatiz @nicolamarangoni This has been fixed in v0.94. Please follow our migration guide. Also, we've added a note about this to the migration guide for v0.93.

Closing as completed. Please open a new one if the error persists in the newest provider version.


Fix CI environment and version related issues

Fixes #148 #105

I'll probably have to address the netcdf issue using micromamba like what I did for coupledmodeldriver: https://github.com/noaa-ocs-modeling/CoupledModelDriver/pull/165

Codecov Report
Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 20.81%. Comparing base (71804ad) to head (05d10d7).

Files
Patch %
Lines

ensembleperturbation/client/perturb_tracks.py
0.00%
1 Missing :warning:

ensembleperturbation/perturbation/atcf.py
50.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #149      +/-   ##
==========================================
+ Coverage   20.79%   20.81%   +0.02%
==========================================
  Files          28       28
  Lines        3852     3848       -4
==========================================
  Hits          801      801
+ Misses       3051     3047       -4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Pre-plan meal

The pull isn't finished might have to change the classes of lunch, breakfast, and dinner as it isn't in the api

Got the API mostly set however for the preplan it fails to print out the food for the week still saying 502 error code


Update readme

Define clearly what is:

state
command: Condition, Permission, Side-Effects and Callbacks
Transition
Process
ProcessManager and why it should work as a mixin

It needs to provide more information on these questions:

What does this project do?
Why is this project useful?
How do I get started?
Where can I get more help, if I need it?

Also:

Code of Conduct


sota_sanity.bbclass: introduce sota_check_boolean_variable

The current sanity check are too strict for some boolean variables,
introduce sota_check_boolean_variable to allow a boolean value to be
set like yes/y/true/t/1 or no/n/false/f/0.
Signed-off-by: Ming Liu<EMAIL_ADDRESS>

Changes in V2:
1 Also change the checking code to use oe.types.boolean.


Extending PointsAnnotation and CircleAnnotation with object_id

I have added object_id in PointsAnnotation and CircleAnnotation.
Description
I need this change because i want to have  possibility to send object id through protobuf message, in order  to be able to display it on ImageView. This is only the first part, the second is contained in foxglove/studio repo.
I will post here link of PR (the second part from foxglove/studio repo), for better understanding why i need this changes.
https://github.com/foxglove/studio/pull/5921

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi,
What is the intended use case for the object id? I noticed that your other PR #109 references the object id being a simple label like "car" or "truck". Is it just a visual label, or do you want to tie the identifier to a particular instance of data?

To an extent the answer is Yes on bot counts. We wanted to implement the ability to "render" the object ID on screen, and also have it share object ID with other representations of the data in other views. So that the user can identify that the rectangle in Image view is the same object as a cube in 3D view.

For rendering the object ID on screen, have you tried TextAnnotation?
For displaying additional data, would adding a metadata field (similar to SceneEntity) work for your use case?

We believe that TextAnnotations are the best way to achieve this feature.


Update rancher/telemetry version to non-rc v0.5.19

Update rancher/telemetry version to non-rc v0.5.19.
Signed-off-by: Guilherme Macedo<EMAIL_ADDRESS>

Restarted CI

Replaced by: https://github.com/rancher/rancher/pull/38629


copr: introduce try_opt

We deal with None a lot in coprocessor, hence I introduce try_opt to reduce the duplicate code.

LGTM


Cannot Pull Docker Image 2023.4.0

Describe the problem
The tag for the latest image, 2023.4.0, does not seem to exist. I see all the other docker tags with the different java versions, as well as latest. However, the tag for just the version and nothing else does not appear to exist on Docker Hub or GitHub Container Registry. Both fail with a manifest unknown error:
barrelmaker@pollux:~$ docker pull docker.io/itzg/minecraft-server:2023.4.0
Error response from daemon: manifest for itzg/minecraft-server:2023.4.0 not found: manifest unknown: manifest unknown

barrelmaker@pollux:~$ docker pull ghcr.io/itzg/minecraft-server:2023.4.0
Error response from daemon: manifest unknown

Has this tag type been accidentally left off, or is this change intentional?
Container definition
No response
Container logs
No response

If it's missing, then not intentional. Thanks for letting me know.

Latest build now pushes the "primary" tag of 2023.4.1 correctly
https://hub.docker.com/repository/docker/itzg/minecraft-server/tags?page=1&ordering=last_updated&name=2023.4.1

Thank you! Image pulling properly now 👍🏻


Citations in notebooks

As it has been requested during the seminar, we should include citations in the notebooks. There is a nice (and working) extension, which has support for bibtex.
It works by having a secret cell (by notebook) or a bibliography.ipynb (shared) containing the references. Inserting citations is super easy and the references will be updated by clicking one button.
https://github.com/Calysto/notebook-extensions/blob/master/Screenshot from 2017-08-09 19-48-12.png
If we want to use it, I can make this extension available on conda-forge.
ATM: it can be installed via conda install calysto-nb-extensions blas -c defusco

Let's test it first.
We have a bibtex file (in the manuscript repository) where I would like to collect all citations. Can we import this file somehow into the notebooks?

On 12.07.2018 10:53, Christoph Wehmeyer wrote:

Let's test it first.
We have a bibtex file (in the manuscript repository) where I would like to collect all citations. Can we import this file somehow into the notebooks?

I guess we can import it with a manual step (citations do not change all
the time, so this should be fine) - or provide a script which generates
the Bibliography.ipynb
The only difference to the input bib file is the wrapping with the
 tags

Since the output of this extension is static text, we do not need the user to install anything!


build: add its own include path to each library

What type of PR is this?
/kind cleanup
/kind feature
Any specific area of the project related to this PR?
/area build
Does this PR require a change in the driver versions?
What this PR does / why we need it:
Libraries of falcosecurity/libs do not bring their own include path with each CMake target. This forces clients to set this include path on their own, possibly getting it wrong or failing to track it when it changes. This PR fills this gap, thus simplifying usage of the libraries.
Also, the list of libraries linked by many targets was missing the PRIVATE/PUBLIC property. This has been fixed by using PRIVATE.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Using libraries `sinsp` or `scap` from a client CMake project does not require to specify any specific include path. The two targets will bring their own include path once they are referenced in the `target_link_libraries` CMake statement.

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

Needs approval from an approver in each of these files:

OWNERS

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

/milestone 0.15.0

LGTM label has been added.  Git tree hash: 0c96ba83e9e37302fba6bd9c5bc7a7517d85ff04

This seems almost ready I would try this for 0.14.0

/hold
I will rerun the CI just to be sure it's ok

Needs to be rebased on top of latest master to fix ci.

New changes are detected. LGTM label has been removed.

LGTM label has been added.  Git tree hash: 8b3457ab0be7bef3c2fcb65953847ecf6a340ee3

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,incertum]

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

ohh we need another rebase, fun!

New changes are detected. LGTM label has been removed.

LGTM label has been added.  Git tree hash: 1ccea71a9f98cf35abb1f23e07992297cbd7c959

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,incertum]

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

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

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,incertum]

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

/unhold


WIP feat: Add support for replicas and cross region replicas

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[ ] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Support fallbacking from replicas that go down and support multiple replica options. See this ticket's comment for full details.
How did you test this code?

I hard coded a single default database. Extensive testing, but not ideal, so I'm opening this PR as a work in progress to get some advice on how to deal with it.

Codecov Report
Attention: 38 lines in your changes are missing coverage. Please review.

Comparison is base (f88f334) 95.94% compared to head (637092f) 89.08%.
 Consider uploading reports for the commit 65ba9d6 to get more accurate results

Files
Patch %
Lines

api/app/routers.py
13.95%
37 Missing :warning:

api/app/settings/common.py
85.71%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3300      +/-   ##
==========================================
- Coverage   95.94%   89.08%   -6.86%
==========================================
  Files        1067     1052      -15
  Lines       32613    32189     -424
==========================================
- Hits        31291    28677    -2614
- Misses       1322     3512    +2190

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

CI failure is related to this: #3361

Looks like you've actually got some legit failures.


Ritual cooldowns

This is mostly functional, just need to address the TODO in BaseRitual.gd to send the appropriate Ritual to start the cooldown on after hitting back.
RitualCooldownManager.start_cooldown(RitualCooldownManager.BOUNCER) # TODO - ...

Everything is good with the location data on the Ritual objects!


[crypto] add RSA 3072 pub-key encryption

During provisioning, we need to generate an RMA token on-device and encrypt it with an HSM generated pubkey before exporting it from the device (see #17393). The current crypto lib only supports doing a signature verification, not just a simple encryption operation. This adds a function that can simply encrypt a 3072 bit value with a pubkey.
@jadephilipoom lmk what you prefer here, if you had a different API in mind, etc. Thanks!

Is there a specific reason to use RSA here instead of a different algorithm -- for instance, an ECDH key exchange followed by symmetric encryption with AES? I generally like to steer away from RSA if I can, because it's not nearly as easy to use securely as ECC (see this blog post for a very opinionated but concise explanation). The keys are also huge, which could be annoying for long-term storage (384 bytes for RSA-3072 as opposed to 32 bytes for the similarly-secure ECDH-P256).
An additional complication: RSA encryption is currently not included in the top-level cryptolib API at all because originally we didn't think it was needed. However, I don't strongly oppose adding it because after looking at some higher-level specs recently I think we do in fact need it for compatibility. But I need to get some feedback on that conclusion before I add another major algorithm.

#18127 has been updated to use ECC as suggested, so closing this. We can reopen later if we decide RSA is needed for RMA token export.


Don't compatible with Django 1.10

When I upgrade django 1.8 to 1.10, it gives this error:

Django Version: 	1.10.3
Exception Type: 	TypeError
Exception Value: 	sequence index must be integer, not 'slice'
Exception Location: 	/usr/local/lib/python2.7/dist-packages/paginator/paginators.py in paginate, line 43

This problem comes from that "page_range" returned an iterator instead of a list after Django version of 1.9. So the solution is quite simple: modify line 35 of /usr/local/lib/python2.7/dist-packages/paginator/paginators.py

page_range = paginator.page_range

to

page_range = list(paginator.page_range)


Initargs

Add possibility to change escape char + public var can be change by calling the constructor

Does this PR supercede #1?

Yes if you will, by having two PRs I let you the choice to integrate just one new functionnality at the time

Gotcha. It makes sense to expose this option to the CLI if it's available for use in the library.

I don't understand what you are asking ?

Scratch that last comment, cc7d78c exposes the options via the constructor.
Would you mind changing the following?
From:
self.input = inFile
self.output = outFile

To:
self.input = input
self.output = output

I, prefer to keep the API consistent wherever possible.

The problem is that "input" is a builtin function name and I'm not at ease giving this name to a parameter, it may cause problem.
Like
input = 'toto'
parser=preprocessor(input=input)

May prevent following call to the built-in python function.

You're right.
I'd prefer to rename the method members too but that would constitute a breaking change to the API. I'll merge this for now and clean up the rest later.
Ref https://github.com/evanplaice/pypreprocessor/issues/3
Thanks for the contribution.


fixes on not nullable properties

primaryColor and defaultLabelStyle property has a nullable atribution, which is not supported

This will fix my issue with build. Thanks

Can you kindly share the progress on this?

Is there any update on this? it's a blocker for us!

@szainab
It is fixed in https://pub.dev/packages/flutter_awesome_select/versions/6.1.2-beta

Sorry, I was very busy in the last few month))) 🙏


Custom camera and microphone permission support

Hi, My client wants to buy Natcorder asset from Asset store but we want to confirm first if we can force Camera and Microphone permission at app start (basically whenever we want) and not when the recording starts. Is that possible?
Thanks,
Dan.

Hey @dhanrajsinh24 , NatCorder doesn't handle any permissions. Instead, you can use the method that Unity provides, Application.RequestUserAuthorization.


SNOW-1119855: Push base unimplemented methods to query compiler when possible (1/?)

Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

Fixes SNOW-1119855 (first of several PRs)

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code

[ ] If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing

[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency
[ ] If this is a new feature/behavior, I'm adding the Local Testing parity changes.

Please describe how your code solves the related issue.

This PR prepares for the imminent removal of base.py from the Snowpark pandas codebase by pushing as many unimplemented methods down to the query compiler as possible. Some unimplemented methods must still be overridden because Modin uses other implemented query compiler/frontend methods in their implementation; these methods have been moved to src/snowflake/snowpark/modin/plugin/extensions/base_overrides.py.
The following methods are affected:
The below BasePandasDataset methods either have existing corresponding query compiler stubs that can be used directly, or default to pandas at the query compiler layer, which Snowpark pandas specifies to raise a NotImplementedError.

Pushed to the query compiler

align
at_time (via between_time)
between_time
clip
combine (via binary_op)
combine_first (via binary_op)
explode
infer_objects
kurt
kurtosis (via kurt)
mode
sem
tz_convert (via tz_convert)
tz_localize (via tz_localize)
__sizeof__

The below methods do not have corresponding query compiler methods, and either (1) would work in Snowpark pandas but haven't been tested/need too many queries, (2) use frontend/other query compiler methods that we cannot override, or (3) manipulates index/columns in ways that we may not support.

Moved to base_overrides.py
asof: modin uses dropna + reindex
bool: method is deprecated in pandas but pretty trivial
droplevel: modin does manipulations on column/index objects
ewm: modin defaults at frontend, would materialize data
filter: modin uses loc + manipulations on column/index
pipe: just kind of weird, probably would materialize data
pop: modin uses loc + del
reorder_levels: modin uses properties on index object
set_flags: modin defaults at frontend, would materialize data
swapaxes: modin uses transpose + index/column manipulations
swaplevel: modin uses native index swaplevel
transform: modin uses QC agg method
truncate: modin manipulates index objects
__finalize__: modin defaults at forntend, would materialize data

These methods are also moved to base_overrides.py.

I/O methods that default on the frontend and would materialize data
to_clipboard
to_excel
to_hdf
to_json
to_latex
to_markdown
to_pickle
to_string
to_sql
to_timestamp
to_xarray

Other
__array_wrap__: this was removed in pandas 2.0
reindex_like: at some point, this method was moved from base.py into dataframe.py/series.py upstream; we continue to mark these as unimplemented in our vendored copies of dataframe/series for now
map: this was also split into dataframe.py/series.py; since we partially support the method for series (but not currently DF) we mark this method as unimplemented only in dataframe.py
transform: we partially support the method for DF (but not for series), so we add an override in dataframe_overrides.py with an implementation and an unimplemented stub in series_overrides.py

You can move base.py out of the code coverage check given it will be removed soon.

You can move base.py out of the code coverage check given it will be removed soon.

If I modify tox.ini (which seems to be where files get excluded) it'll trigger a review request on the Python team, so I'll just add manual # pragma: no covers for now.


[GFX-1868] Fix W3 level MSVC warnings in public headers

Jira ticket URL (Why?)
GFX-1868
Short description (What? How?) 📖
We already fixed some MSVC warnings in #75 which was further optimized in #83. This got us so far that we could make an internal build with Vis. on Windows. But Filament contained many more public headers which we may not be using right now, but might need anytime in the future.
This PR fixes all W3 level MSVC warnings in the remaining public headers, regardless if it is used in Shapr or not.
Upstreaming scope
GFX-1891
Testing
Design review 🎨
n/a
Affected areas 🧭
n/a
Special use-cases to test 🧷
n/a
How did you test it? 🤔
Manual 💁♂
I tested on VS2019 and V2022 with a .cpp file that includes all public headers of Filament (except for Apple/Linux only). You can test it too with https://github.com/shapr3d/shapr3d/tree/palver123/GFX-1868_filament_warnings_test (just have to change the path of your Filament git repo in Libraries.cmake)
Automated 💻

@marcisolti you are right I forgot that we disable a bunch of warnings in Shapr3D codebase, so I tested with the wrong compiler flags. I update palver123/GFX-1868_filament_warnings_test and fixed 2 more warnings in 5ff9ee92687e04f0e000267a4d4df022668fe181 and 3f00a9ea3125c5a934bb2ce74c5cafc91b6124cf


Click on sql_text column in sp_whoisactive does not open new document

Hi all!
If we execute sp_whosactive and click on sql_text column there should be opened new document with data from sql_text column. It often happens that document is not opened though document counter on a left bar increments counter. After Azure studio is reopened all those documents appears.

Azure Data Studio Version:
Version: 1.10.0 (user setup)
Commit: 5fdb967ffcea41df0335ec17cfa396a694bb174e

Steps to Reproduce:

run sp_whoisactive with default options
click on sql_text column which does not open new document. In left bar we can see 10 open documents while none is shown in Azure Data Studio.

restart Azure Data Studio and all documents are now shown

I have the same problem.


Feature/kafka sink

LGTM.
Looks like the same code we have internally.
Do you think it would be worth it to make it a library and a separate project under the github umbrella?
The advantages is you get to use the same lib for both projects, and only one meta folder? via git submodules?
Maybe we can even depend on that library internally?
I'm fine releasing this to the public and then later on making it a library and the merging that internally.

I think we need git submodules, however it is not exactly  the same code. I modified the producer so we don't have to depend on folly.


Only merge obviously record-like objects

Closes #15

Good catch, lgtm ✅! Just added some unit tests.

thankyou! I'm happy for you to merge whenever :)


New package: PowerLaws v1.0.0

Registering package: PowerLaws
Repository: https://github.com/kmarray98/PowerLawSamplers.jl
Created by: @kmarray98
Version: v1.0.0
Commit:<PHONE_NUMBER>260a7fb8cc9a153ff6225279d048fd
Reviewed by: @kmarray98
Reference: https://github.com/kmarray98/PowerLawSamplers.jl/issues/1

Closed in favor of #101884.


SBT 0.13.7 does not react on Ctrl-C or Ctrl-D hotkeys

Steps to reproduce:
Win 8.1 ruRU, chcp 1251 (866, 60001 and 1252 don't work either)
build.sbt https://gist.github.com/flirtomania/e1273618ab706a39eaba
Main class: https://gist.github.com/flirtomania/aee23feca65126db6c61

run sbt in project directory (now it does not react on ctrl-c, still terminates on ctrl-d)
call "run" task, (application runs ok)

Problems:
sbt run task does not react on ctrl-c or ctrl-d hotkeys
Expectations:
pressing ctrl-c / ctrl-d should cause run task to stop or quit sbt

Hi, could you see #1442 and see if
cancelable in Global := true

fixes it? Also is this an Akka specific issue?

Hi,
cancelable in Global := true
did not worked.
The issue reproduces also on Windows 7 running with -Dinput.encoding=Cp1252 and without it (this is related to up arrow behaviour on russian versions of windows).
This is how I tried without Akka: https://gist.github.com/flirtomania/a2c291551c5b53c7e1f4
It still did not react on Ctrl-C or Ctrl-D
Both OS versions have Russian language.

Ctrl-C doesn't work for me either, with or without "cancleable" set. Ctrl-D does work however (Windows 8.1).

Same problem running a Spring Boot application in Windows with "sbt run". "cancelable in Global := true" does not help.

I noticed now that it is not stoppable when running "run" from within the sbt console, but when running "sbt run" directly from the windows terminal/cygwin, ctrl-c works fine.

I also experience this issue on windows 10 with sbt 0.13.9 and it hinders the workflow when a task, such as run, cannot be interrupted without killing the whole cmd/powershell session.
Additionally, when forked is set to true, the forked task will run even after sbt has been killed.

Still a problem on Windows 10 with sbt ~run

Same story with Windows 7 and SBT 0.13.8. Since I started working with sbt in 2012 I can't remember that Ctrl+C works as expected. It's one of many small things that makes work with SBT harassing :weary:

Same thing with sbt 0.13.8 -- Ctrl-C doesn't work either

Pretty annoying, any updates on this?

This seems to be a bug with jline, because using version 2.14.3 fixes this issue for me. The relevant issue jline/jline2#127 mentions as much.
Since the currently stable release does not use jline version 2.14.3 (or above) I just replaced the jline.jar in my .sbt folder (C:\Users\<USER>\.sbt\boot\scala-2.10.6\lib\jline.jar) with a newer version from the maven repository. This is just a dirty hack and since the original file was likely jline version 2.13 this might cause severe bugs due to incompatibilies.
Also, the jline version seems to be a dependency of scala-2.10.6 and might not be easily updated with just an sbt release.
I hope this clears things up as to why ctrl+c is ignored completely by sbt on windows.

TL;DR: We should update JLine. At least 2.14.3. The Scala Team has already.
Thanks @Araeos for the detail and cross-linking.

This issue will now be expedited via the Lightbend subscription.

The latest advertised release linked on the Download Scala states it's for Scala v2.12.3, but it downloads the the SBT v1.0.2 which still installs Scala v2.10.6 (with the bad jline.jar).
Specifically, this is occurring for the Windows MSI installer. Any idea when the link will be updated so new people don't have to deal with this error?


First implementation

First implementation with net6 targeting.

Please also add the link to ReadFile README to https://github.com/FrendsPlatform/Frends.SFTP/blob/main/README.md


System.Exception: Failed to locate a compiler for 'CompileBasicCondition'

Summary
hi, i'm using SqlKata on Unity game engine (Unity-2020.3.40f1)
i launched app on editor and it was fine, but app crashed after build.
can i get some help here?
Error logs
System.Exception: Failed to locate a compiler for 'CompileBasicCondition'.
  at SqlKata.Compilers.ConditionsCompilerProvider.FindMethodInfo (System.Type clauseType, System.String methodName) [0x00000] in <00000000000000000000000000000000>:0
  at SqlKata.Compilers.ConditionsCompilerProvider.GetMethodInfo (System.Type clauseType, System.String methodName) [0x00000] in <00000000000000000000000000000000>:0
  at SqlKata.Compilers.Compiler.CompileCondition (SqlKata.SqlResult ctx, SqlKata.AbstractCondition clause) [0x00000] in <00000000000000000000000000000000>:0
  at SqlKata.Compilers.Compiler.CompileConditions (SqlKata.SqlResult ctx, System.Collections.Generic.List`1[T] conditions) [0x00000] in <00000000000000000000000000000000>:0
  at SqlKata.Compilers.Compiler.CompileWheres (SqlKata.SqlResult ctx) [0x00000] in <00000000000000000000000000000000>:0
  at SqlKata.Compilers.Compiler.CompileSelectQuery (SqlKata.Query query) [0x00000] in <00000000000000000000000000000000>:0
  at SqlKata.Compilers.Compiler.Compil

How to reproduce

Create new project with unity 2020.3.40f1
Import package NuGetForUnity
Install package SqlKata 2.4.0
Create script and write any new Query and SqliteCompiler.Compile
Save and build application to android

Scripting Backend: IL2CPP
Api Compatibility Level: .NET 4.x (Unity 2020 dotnet support documentation)
Target Architectures: ARMv7, ARM64

Others
i test some logics in ConditionsCompilerProvider.FindMethodInfo, but it was not working for expected.
here the script i tested:
var query = new Query("ANY_TABLE_NAME");
var compiler = new SqliteCompiler();
var queryString = compiler.Compile(query);

Debug.Log($"SqliteCompiler methods: {string.Join(",", compiler.GetType().GetRuntimeMethods().Select(x => x.Name))}");

/*
Play on Editor says:
SqliteCompiler methods: get_EngineCode, get_parameterPlaceholder, set_parameterPlaceholder, get_parameterPrefix, set_parameterPrefix, get_OpeningIdentifier, set_OpeningIdentifier, get_ClosingIdentifier, set_ClosingIdentifier, get_LastId, set_LastId, CompileTrue, CompileFalse, CompileLimit, CompileBasicDateCondition, FindCompilerMethodInfo, CompileCondition, CompileConditions, CompileRawCondition, CompileQueryCondition, CompileSubQueryCondition, CompileBasicCondition, CompileBasicStringCondition, CompileNestedCondition, CompileTwoColumnsCondition, CompileBetweenCondition, CompileInCondition, CompileInQueryCondition, CompileNullCondition, CompileBooleanCondition, CompileExistsCondition, get_ColumnAsKeyword, set_ColumnAsKeyword, get_TableAsKeyword, set_TableAsKeyword, get_EscapeCharacter, set_EscapeCharacter, get_SingleRowDummyTableName, generateNamedBindings, PrepareResult, CompileRaw, Whitelist, Compile, Compile, CompileSelectQuery, CompileAdHocQuery, CompileDeleteQuery, CompileUpdateQuery, CompileInsertQuery, CompileCteQuery, CompileColumn, CompileCte, OnBeforeSelect, CompileColumns, CompileUnion, CompileTableExpression, CompileFrom, CompileJoins, CompileJoin, CompileWheres, CompileGroups, CompileOrders, CompileHaving, CompileRandom, CompileLower, CompileUpper, checkOperator, Wrap, WrapValue, Resolve, Parameter, Parameterize, WrapArray, WrapIdentifiers, Equals, Finalize, GetHashCode, GetType, MemberwiseClone, ToString

Play on After Build with Android says:
SqliteCompiler methods: get_EngineCode,get_parameterPlaceholder,get_parameterPrefix,get_OpeningIdentifier,get_ClosingIdentifier,get_LastId,CompileLimit,FindCompilerMethodInfo,CompileCondition,CompileConditions,get_ColumnAsKeyword,get_TableAsKeyword,get_EscapeCharacter,get_SingleRowDummyTableName,generateNamedBindings,PrepareResult,CompileRaw,Compile,CompileSelectQuery,CompileAdHocQuery,CompileDeleteQuery,CompileUpdateQuery,CompileInsertQuery,CompileCteQuery,CompileColumn,CompileCte,CompileColumns,CompileUnion,CompileTableExpression,CompileFrom,CompileJoins,CompileJoin,CompileWheres,CompileGroups,CompileOrders,CompileHaving,Wrap,WrapValue,Parameter,Parameterize,WrapArray,WrapIdentifiers,Equals,Finalize,GetHashCode,GetType,MemberwiseClone,ToString
*/

I am not familiar with Unity, but what comes to my mind, that what's happening is something similar to JavaScript tree shaking, where the compiler remove methods with no reference, and since CompileBasicCondition is called via reflection so there is no direct reference.
I am not sure what would be the solution here, but I am pretty sure that directly referencing the CompileBasicCondition method in your code, will do the trick

From their doc, I found this

Enable Strip Engine Code in the Unity Editor Player settings (go to Edit > Project Settings
, then select the Player category). This option statically analyzes the managed code in the Project, and removes any unused code.

https://docs.unity3d.com/2020.3/Documentation/Manual/dotnetProfileLimitations.html
so maybe disabling this option will help in your case

thanks for quick response!
i solved this problem just writing link.xml
<!-- link.xml -->
<linker>
    <assembly fullname="SqlKata" preserve="all"/>
</linker>

unity build with il2cpp will always stripping unused codes, so it should be preserved manually.


Refactor/controller runtime

@AlexNPavel We forgot to remove the unused files and folders from the older project before the refactor like tmp/build and deploy/sa.yaml.
https://github.com/operator-framework/operator-sdk-samples/tree/master/memcached-operator/tmp/build
We can remove those in a follow up PR.

@AlexNPavel We forgot to remove the unused files and folders from the older project before the refactor like tmp/build and deploy/sa.yaml.
https://github.com/operator-framework/operator-sdk-samples/tree/master/memcached-operator/tmp/build
We can remove those in a follow up PR.

@AlexNPavel We forgot to remove the unused files and folders from the older project before the refactor like tmp/build and deploy/sa.yaml.
https://github.com/operator-framework/operator-sdk-samples/tree/master/memcached-operator/tmp/build
We can remove those in a follow up PR.


USHIFT-2105: cert rotation test wait for greenboot

This test changes the time immediately after starting. If the test is executed when the system is running greenboot (from a reboot, for example), the time warp makes it fail with fake timeouts, as time jumped to the future. Depending on the test execution order this might happen randomly.
Force waiting for greenboot before starting to avoid glitches.

Which issue(s) this PR addresses:

Closes #

/lgtm


Version 2.8.1

@noteflakes Could you release a new version, please?

Hey, just released version 2.8.1. Thanks @gschlager!


Retry fetch parcels a couple of times on error

All error pages have lint errors.

The retry itself works :)

Fixed the to-> href;
Fixed lints
The a instead of Link is because Link is not meant for external links


fix: ensure special-characters in passwords (like backslashes)

Using the interpolator to inject passwords/secrets was incorrect in specific cases when the secret contained special characters.. by quoting them we ensure the secrets integrity.
I tested the behavior with my setup and the maven tests succesfully completed.
Best regards
Lukas

@pantovad please have a look and provide feedback!

@LUMERIIX thank you very much for the contribution. Merging.


Expand tests, dynamically update CASSANDRA_VERSION var

Expand tests, dynamically update CASSANDRA_VERSION var. Also went back and adjusted retries in cassandra test.
This image (mgmt-api-for-cassandra) cannot trigger cassandra tests, as there are other cassandra tests which are not compatible with this image, hence why copying.

As part of a Chainguard effort to reduce code duplication and provide higher quality images, we have stopped accepting pull requests for changes to images in this repository. Due to this, we are closing all pull requests today in order to start fresh.
If you think this pull request was closed in error, please feel free to re-open this pull request or open an issue and we will try to help you out. Thank you!


Change to use split-cmd

Closes #28

Cool! Thanks :)

u're welcome! btw, I'd like to suggest accept contributions with the all-contributors-cli, and maybe referencing it in CONTRIBUTING.md

Yea.. hela-config-tunnckocore v2 there is such task.
It's just not documented, but you can do hela ac to run all-contributors-cli.
https://github.com/tunnckoCore/hela-config-tunnckocore/blob/c033f3a3f9c844eaf099d6f8d54baaeefe3c2b2f/src/index.js#L32-L44

Cool!

Hey @thiagodp. Actually looking on the coverage, i realized that it may fail when shell is used. Would you mind adding a test for this and moving
const parts = splitToObject(cmdLine)

above the if statement. Which leads to one suggestion for your library - a stringify or something like that.

@olstenlarck , currently the code is only using splitToObject when type is not shell:
 const mapper = (cmdLine) => {
      const run = cmd[type]

      if (type === 'shell') {
        return run(cmdLine, options)
      }

      const parts = splitToObject(cmdLine)
      return run(parts.command, parts.args || [], options)
    }

Exactly. It just runs execas shell, and that's why i'm thinking it still may fail when cmdLine is like git commit -m "foo bar".

Isn't exec's shell suppose to run the command directly? It doesn't have an argument parameters, but only command and options, so it would not be necessary to split the command or parsing it.

No problem 👍


[Asset] Add quick start code for ExportAssets API

All checks have passed. Please take a look. Thanks!

Sorry for the checkstyle issue. I'd have fixed them but I just couldn't run those scripts on my local machine and it says it passed Kokoro on github. Thanks for pointing them out.

@dpebot merge when green


Remove separate make target check for CNI presubmit

Defaulted to "build" make target check for CNI-presubmit. This will unblock https://github.com/aws/eks-distro-prow-jobs/pull/87.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks


Fused MHA Support in XLA GPU: Stream Executor changes

cuDNN currently supports the following patterns for Multi-headed attention:
BMM1 - BMM2
BMM1 - Scale - Bias - Mask - Softmax - BMM2
BMM1 - Scale - Bias - Mask - Softmax - Dropout - BMM2
BMM1 - Scale - Mask - Softmax - BMM2
BMM1 - Scale - Mask - Softmax - Dropout - BMM2
BMM1 - Softmax - Dropout - BMM2
This PR adds support for the stream executor for these patterns.

I think @akuegel is also fixing this internally.

I think @akuegel is also fixing this internally.

I just checked, and there are differences in the fixes. Let's import again and see whether additional fixes will be needed, and if not, we can take it as is.

This has been merged. I had to also apply some fixes suggested by ClangTidy as it was also blocking the submit.

pushed a commit to google/tsl that referenced this pull request

@akuegel what are the issues? We can work on fixing them and re-merge.

Can we re-open this PR to work on fixing the issues? I don't see a way to re-open this.

It fails to build under CUDA 11.4 /CuDNN 8.2:
external/xla/xla/stream_executor/cuda/cuda_dnn.cc: At global scope:
[external/xla/xla/stream_executor/cuda/cuda_dnn.cc:3500](https://cs.corp.google.com/piper///depot/google3/external/xla/xla/stream_executor/cuda/cuda_dnn.cc?l=3500):5: error: 'cudnnBackendTensorReordering_t' has not been declared
 3500 |     cudnnBackendTensorReordering_t cudnn_tensor_order_type =
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[external/xla/xla/stream_executor/cuda/cuda_dnn.cc:3501](https://cs.corp.google.com/piper///depot/google3/external/xla/xla/stream_executor/cuda/cuda_dnn.cc?l=3501):9: error: 'CUDNN_TENSOR_REORDERING_NONE' was not declared in this scope; did you mean 'cudnn_frontend::cudnnBackendTensorReordering_t::CUDNN_TENSOR_REORDERING_NONE'?
 3501 |         CUDNN_TENSOR_REORDERING_NONE,
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |         cudnn_frontend::cudnnBackendTensorReordering_t::CUDNN_TENSOR_REORDERING_NONE
In file included from [bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_ConvDesc.h:35](https://cs.corp.google.com/piper///depot/google3/bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_ConvDesc.h?l=35),
                 from [bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend.h:100](https://cs.corp.google.com/piper///depot/google3/bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend.h?l=100),
                 from [external/xla/xla/stream_executor/cuda/cuda_dnn.cc:56](https://cs.corp.google.com/piper///depot/google3/external/xla/xla/stream_executor/cuda/cuda_dnn.cc?l=56):
[bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_utils.h:376](https://cs.corp.google.com/piper///depot/google3/bazel-out/k8-opt/bin/external/cudnn_frontend_archive/_virtual_includes/cudnn_frontend/third_party/cudnn_frontend/include/cudnn_frontend_utils.h?l=376):5: note: 'cudnn_frontend::cudnnBackendTensorReordering_t::CUDNN_TENSOR_REORDERING_NONE' declared here
  376 |     CUDNN_TENSOR_REORDERING_NONE,
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~

etc.

@akuegel After reopening the PR, why does it now have only one file? Was this partially merged?

It looks like only the last commit got reverted. I don't think the issue (of build failing with CUDA 11.4 and CuDNN 8.2) will have resolved with reverting the last commit only.

Actually all of it was reverted:
https://github.com/tensorflow/tensorflow/commit/7a4d2e8b1a503a45ff86d40b0387fc832302379f
Why it doesn't show properly merged in this PR, I don't know :-(
Maybe it is easier to create a new PR?

Actually all of it was reverted:
7a4d2e8
Why it doesn't show properly merged in this PR, I don't know :-( Maybe it is easier to create a new PR?

Yeah..I guess. Let me do that.

@akuegel @cheshire opened a new PR https://github.com/tensorflow/tensorflow/pull/60139


failed to spawn remote thread:GetLastError=0

failed to spawn remote thread:GetLastError=0

This is likely due to using a non-ascii file path, if this occurs in future please try to provide more information.


MTCNN model files Conversion?

I think MTCNN has 3 nets and the facenet model file is in tensrflow format?
How did you convert the MTCNN model file to one tflite file?
Did you face any issue in converting facenet model into tflite?

Please refer to the project.


A problem occured when I set the content of a subcomponent to vnode

Version
3.1.1
Reproduction link
sfc.vuejs
Steps to reproduce

Call the setVNode method to set up the content of the subcomponent in the onMounted of the parent component.
Click the show target button to display the sub components
It is found that setvnode has no effect
Click the set vnode content button, and the set vnode effect will come out
Click the hide target button to hide the subcomponent, and then click the show target button to show the subcomponent again
Find that the content of the subcomponent becomes blank
The corresponding operation of the set html content button does not have this problem

What is expected?
The parent component can dynamically switch the content of the child component between HTML fragment and vnode
What is actually happening?
When the subcomponent is hidden, setting content to vnode is invalid; When the subcomponent is in the display state, setting the content to vnode is effective, but it becomes blank after it is hidden and displayed again

Subcomponent can display content through content property or default slot settings. Content property supports HTML fragment strings and vnode
Code supporting vnode

watch(() => props.content , (newVal, oldVal) => {
  if(!internalInstance) return;
  if (isVNode(newVal)) {
    internalInstance.slots.default = () => [newVal];
    innerContent.value = '';
  } else {
    if(isVNode(oldVal) && !isVNode(newVal)){
      delete internalInstance.slots.default;
    }
    innerContent.value = newVal;
  }
});

You cannot change the slots programatically, only declaratevely. Regarding your code, you can do this

Remember to use the forum or the Discord chat to ask questions!


Can pass options to Rufus::Scheduler.new

I want to pass in some :lockfile options to the scheduler. This allows for any options to be set.
As part of the PR, I also updated to rufus-scheduler 3.0+, which has better options other than subclassing Rufus::Scheduler to add different locking semantics.

@adrian-gomez would be fantastic to get this merged. It's very useful for multihost locking.

@adrian-gomez ping. Just let me know what needs to happen to get this merged.

Sorry for the delay...

@elpic can you cut a new version of them?

@jaredjenkins ready.

Thanks @elpic. I think you'll want to bump this number: https://github.com/Moove-it/sidekiq-scheduler/blob/master/lib/sidekiq-scheduler/version.rb#L2


fix 'va' param documentation for mortgage insurance API

the params table for mortgage insurance API has the wrong info for 'required' condition, I updated to loan_type

good catch! thanks!


Update RHEL_7/installDb.sh to add a new package before installing postgres.

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

Completed.


Hopefully, the ensemble action of Baldur's Gate 3 can be added in the future

I was absolutely pleasantly surprised by the new musical ensemble action added in version 2.0.0!It would be great to be able to join Baldur's Gate 3 for the bards!

Are you asking for a Baldur's Gate 3 version of it? If so, I do not know how to make mods for BG3, and I also don't have enough time to learn how :(

Oh, are you talking about the ensemble action? What is that exactly?

Oh, are you talking about the ensemble action? What is that exactly?

https://www.youtube.com/watch?v=Cty_fb3dBXU
Just like the TGA's 2023 ensemble, Baldur's Gate 3 alone may be more singular, and it may be very epic if you can use it in this video

Maybe it's because I'm using translation software, and some of the meanings aren't clear


Simpler co2 model

Purpose
This PR completes the Soil Biogeochemistry module for prescribed soil drivers in a simpler soil co2 model. A tutorial using the module will come in a future PR. See issue #113.
To-do
Content
Adds the right hand side function and update aux function for the biogeochemistry model, using a simpler parameterization.
Removes previous parameterizations which required antecedent conditions from soil.

[x] I have read and checked the items on the review checklist.

bors r+


Fix documentation code examples

See failing examples by cargo test --doc
Fix example codes to work with current API.

Fixed in 0.3.4


What do scripts run as on Linux

States 'Scripts run as System on Windows'. So what do they run as on Linux?

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

ID: b3ff94e6-3c1f-b9e1-1a46-b75fe2ffca1b
Version Independent ID: 8f3ca735-ddd4-f4a9-4ee0-189604018784
Content: Run PowerShell scripts in an Windows VM in Azure
Content Source: articles/virtual-machines/windows/run-command.md
Service: automation
GitHub Login: @georgewallace
Microsoft Alias: gwallace

@jtuliani Thanks for reaching out, this doc is specific to Windows, Linux has it's own version of Run Command for Linux. You can see it here. https://docs.microsoft.com/en-us/azure/virtual-machines/linux/run-command

Closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Camera product type publishing fails

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior:
Publishing camera product type is not working in the latest develop
Expected Behavior:
It should be published correctly as expected.
Version
1.0.0
What platform you are running on?
Windows
Steps To Reproduce:

Create Camera Product Type
Publish

Are there any labels you wish to add?

[X] I have added the relevant labels to the bug report.

Relevant log output:
No response
Additional context:
No response


There's 3 ops are not supported yet

hi,
i am using ch_ppocr_mobile_v1.1_rec_train.tar model and exported to fluid model and wanted to convert to onnx model. but failes with below error:
paddle.version = 1.8.5
Now, paddle2onnx support convert paddle model to onnx opset_verison [9, 10, 11], opset_verison be seted as 9, automatically treated as opset_version: 9.
Translating PaddlePaddle to ONNX...
Total:198, Current:1 : feed image
Total:198, Current:176 : im2sequence add
"==========Importance Notice===========
Since im2sequence operator is used in your paddlepaddle model, the translated onnx model only support input data with batch_size=1.
======================================"
Total:198, Current:198 : fetch x n e_add
There's 3 ops are not supported yet
=========== ctc_align ===========
=========== top_k ===========
=========== lstm ===========
please let me know  if i need to do any woraround.
Thanks
Vijay
skypeid: vijayky88

I have met the same question.

@SWHL any alternative available ?

Now, there is a big difference between  Paddle::lstm and ONNX:LSTM. We are upgrading the interface of LSTM,  At the end of the year we will try to add the conversion of lstm.

@vijayky88  The LSTM op of the PyTorch can convert to the onnx. Trouble Converting LSTM Pytorch Model to ONNX

This is good news if we will be able to export this model to onnx by the
end of this year!
Thanks
On Tue, Oct 27, 2020 at 7:53 PM SWHL<EMAIL_ADDRESS>wrote:

@vijayky88 https://github.com/vijayky88 The LSTM op of the PyTorch can
convert to the onnx. Trouble Converting LSTM Pytorch Model to ONNX
https://stackoverflow.com/questions/57299674/trouble-converting-lstm-pytorch-model-to-onnx
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PaddlePaddle/paddle2onnx/issues/135#issuecomment-717279761,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAL4TT3RSQ4TBMECR4OLBDSM3JXPANCNFSM4S5SHUPQ
.

I have met the same question with opset 13. Has the problem been solved？

I have met the same question with opset 13. Has the problem been solved？

PaddleOCR model is supported now, the problem you met is Paddle2ONNX cannot convert PaddleOCR model to onnx with opset 13?

我在转换自己使用PaddleOCR训练的识别模型时也会出现相同的问题
paddle2onnx --model_dir ./pretrained_models/crnn --model_filename model --params_filename params --save_file convert_model --opset_version 10 --enable_onnx_checker True
Traceback (most recent call last):
File "d:\anaconda3\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "d:\anaconda3\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "D:\Anaconda3\Scripts\paddle2onnx.exe_main.py", line 7, in
File "d:\anaconda3\lib\site-packages\paddle2onnx\command.py", line 142, in main
enable_onnx_checker=args.enable_onnx_checker)
File "d:\anaconda3\lib\site-packages\paddle2onnx\command.py", line 114, in program2onnx
enable_onnx_checker=enable_onnx_checker)
File "d:\anaconda3\lib\site-packages\paddle2onnx\convert.py", line 77, in program2onnx
export_onnx(paddle_graph, save_file, opset_version, enable_onnx_checker)
File "d:\anaconda3\lib\site-packages\paddle2onnx\convert.py", line 32, in export_onnx
onnx_graph = ONNXGraph.build(paddle_graph, opset_version, verbose)
File "d:\anaconda3\lib\site-packages\paddle2onnx\graph\onnx_graph.py", line 229, in build
onnx_graph.build_op_nodes(paddle_graph.node_map)
File "d:\anaconda3\lib\site-packages\paddle2onnx\graph\onnx_graph.py", line 183, in build_op_nodes
OpMapper.check_support_status(node_map, self.opset_version)
File "d:\anaconda3\lib\site-packages\paddle2onnx\op_mapper\op_mapper.py", line 144, in check_support_status
raise NotImplementedError(error_info)
NotImplementedError:
There's 3 ops are not supported yet
=========== lstm ===========
=========== top_k ===========
=========== ctc_align ===========

目前PaddleOCR下相关的模型转换都已经没有问题了
感兴趣的话，可以参考RapidOCR和PaddleOCRModelConverter

@kano201 看起来你的模型是之前版本的OCR模型，在PaddleOCR最新代码中的模型可以支持转成ONNX

Thanks，I switched to PaddleOCR2, which converted to onnx successfully。

pp-tsm模型推理，paddle转onnx，
There's 1 ops are not supported yet
=========== temporal_shift ===========
temporal_shift也不支持，怎么办？


Need to know when there is no data vs. officer did not try to get data

Sometimes there just is no data for a field - like when a crew member does not have a license number. Because of these cases the office would like to be able to check a 'none' checkbox so that the app knows there is no data for that. Data marked as 'none' should validate and not warn that it is empty. See basic design as per https://docs.google.com/document/d/1rDXmdlB3lfqAb7uAhR1u96LbyNaUyl3_2t2O_C0kI_E/edit#bookmark=id.ua1q0ri03hpz and the document at https://docs.google.com/document/d/1rDXmdlB3lfqAb7uAhR1u96LbyNaUyl3_2t2O_C0kI_E/edit# has all the screens and what fields should have checkboxes vs. not.

Given the complexity of this functionality along with drafts, we are going to stop work on this functionality and leave it out for now.
Thanks everyone for the work you've already done on this.


[sshd] add @Tjzabel as authorized user to irc-lug.rit.edu

This is an old action item I've had on my plate, but this PR adds @Tjzabel as an authorized user to the irc-lug.rit.edu host. This change is tested and also deployed to production already.

Needs testing
@Tjzabel: are you able to SSH into the machine now?
Details
Sort of interesting how I did this. Originally, I had a global target user defined for this entire repository, but I am realizing this doesn't scale. I tried out something new and added some variables to the sshd role with a list of authorized users and their SSH public keys. Then I loop through the list in the role's tasks, and ta-da! It works.
Something I am wondering: does it make more sense to define these users globally or only in this specific role? Probably yes, since @Tjzabel isn't a member of the teleirc or matterbridge groups, which grant read/write privileges to private files in the deployment directories. I'll try and get that in a future PR.

@jwflory Thanks for doing this! I will test this out over the weekend, and let you know the results.

@Tjzabel Ping – were you able to SSH into the machine?

I have a new change coming in that is based off of this. I have good reason to believe this change is in place. I'm merging this PR. :clapper:


lefthook: 1.6.14 -> 1.6.15

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/evilmartians/lefthook/releases.
meta.description for lefthook is: Fast and powerful Git hooks manager for any type of projects
meta.homepage for lefthook is: https://github.com/evilmartians/lefthook
meta.changelog for lefthook is: https://github.com/evilmartians/lefthook/raw/v1.6.15/CHANGELOG.md
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.6.15 with grep in /nix/store/pjrjxiva6crsbxinwbjczvyldrg6y8l5-lefthook-1.6.15
found 1.6.15 in filename of file in /nix/store/pjrjxiva6crsbxinwbjczvyldrg6y8l5-lefthook-1.6.15

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

lefthook

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pjrjxiva6crsbxinwbjczvyldrg6y8l5-lefthook-1.6.15 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A lefthook https://github.com/r-ryantm/nixpkgs/archive/ba02c4b4ac960667c73c036a968bf277df0552ed.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ba02c4b4ac960667c73c036a968bf277df0552ed#lefthook

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pjrjxiva6crsbxinwbjczvyldrg6y8l5-lefthook-1.6.15
ls -la /nix/store/pjrjxiva6crsbxinwbjczvyldrg6y8l5-lefthook-1.6.15/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

lefthook

Maintainer pings
cc @AndersonTorres for testing.

@NixOS/nixpkgs-merge-bot merge


Make available as a package in Nix

i don't think that the maintainers of repren need to do anything but it seems weird not to mention it here; i submitted repren as a potential package available for NixOS:
Package request: repren · Issue #361538 · NixOS/nixpkgs
repren is amazing and i am so excited to see the Python 3 update today!

… NixOS is also amazing, it's already in https://github.com/NixOS/nixpkgs/pull/361562

Awesome, thanks for doing that! I've got to try nix sometime soon.


Inconsistent test result of '+inf'

The +inf returns
{
    "type": "float",
    "value": "inf"
}

in the valid/spec/float-2.json file
https://github.com/toml-lang/toml-test/blob/8bdabb239e349833ee60c793cd720c44b1fc5b8f/tests/valid/spec/float-2.toml#L3
https://github.com/toml-lang/toml-test/blob/8bdabb239e349833ee60c793cd720c44b1fc5b8f/tests/valid/spec/float-2.json#L6-L9
However, it returns
{
    "type": "float",
-    "value": "inf"
+    "value": "+inf"
}

in the valid/float/inf-and-nan.json file
https://github.com/toml-lang/toml-test/blob/8bdabb239e349833ee60c793cd720c44b1fc5b8f/tests/valid/float/inf-and-nan.toml#L8
https://github.com/toml-lang/toml-test/blob/8bdabb239e349833ee60c793cd720c44b1fc5b8f/tests/valid/float/inf-and-nan.json#L10-L13

Is this causing practical issues for you?

I just wrote a parser for TOML for fun and found that it cannot pass these test suites.

That's actually a different issue @glguy, with the "I" in "Inf" being capitalized sometimes. That was fixed last year. I'm not 100% sure if that's in the latest tagged release (need to look that up), but it's definitely in the latest master.
Verified with:
% git log -n1
commit 4bcf07d (HEAD -> main, tag: toml-parser-<IP_ADDRESS>, origin/main, origin/HEAD)
Author: Eric Mertens<EMAIL_ADDRESS>Date:   Sat Feb 24 08:27:42 2024 -0800

    Use NonEmpty in lexer context stack

% git diff
diff --git i/test-drivers/encoder/Main.hs w/test-drivers/encoder/Main.hs
index 73998b2..e322d23 100644
--- i/test-drivers/encoder/Main.hs
+++ w/test-drivers/encoder/Main.hs
@@ -56,6 +56,6 @@ decodeValue "date-local"     (lexValue -> Right (TokLocalDate      x)) = pure (T
 decodeValue "float"          (lexValue -> Right (TokFloat          x)) = pure (Toml.Double    x)
 decodeValue "float"          (lexValue -> Right (TokInteger        x)) = pure (Toml.Double    (fromInteger x))
 -- extra infinities as toml-tests are inconsistent
-decodeValue "float"          "+Inf"                                    = pure (Toml.Double    (1/0))
-decodeValue "float"          "-Inf"                                    = pure (Toml.Double    (-1/0))
+-- decodeValue "float"          "+Inf"                                    = pure (Toml.Double    (1/0))
+-- decodeValue "float"          "-Inf"                                    = pure (Toml.Double    (-1/0))
 decodeValue _                _                                         = empty
diff --git i/toml-parser.cabal w/toml-parser.cabal
index 9faf2c0..eb1ca76 100644
--- i/toml-parser.cabal
+++ w/toml-parser.cabal
@@ -134,7 +134,7 @@ test-suite readme
         markdown-unlit:markdown-unlit ^>= {0.5.1, 0.6.0},

% toml-test ./TomlDecoder
toml-test v2023-12-07 [./TomlDecoder]: using embedded tests
  valid tests: 179 passed,  0 failed
invalid tests: 368 passed,  0 failed

% toml-test -encoder ./TomlEncoder
toml-test v2023-12-07 [./TomlEncoder]: using embedded tests
encoder tests: 179 passed,  0 failed

(I made sure to rebuild the TomlDecoder/TomlEncoder with the changes).


⚠ SceneTime has degraded performance

In 21ca343, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1274 ms

Resolved: SceneTime performance has improved in fb1b634 after 8 minutes.


open multiple instances of R in the background

Hello, I have installed the R package and its plugins for the interconnect. I am intrigued by the excess consumption of ram.

Is there any way to avoid this?

I have the same problem with the languageserve process staring more than one, which causes a lot of memory usage. I don't know under what circumstances  the second or more languageserver process are started.

The task process handles the parsing and diagnostics. If you think the number of sessions is too many, please try writing the following in your ~/.Renviron and see if it works smoothly when you are editing code?
R_LANGSVR_POOL_SIZE=0

Thanks for your reply, I found a problem when I hit Ctrl+S to save, there was a constant languageserve process being created. These processes cannot be shut down automatically. I know there are processes for code diagnostics, but why don't these processes end automatically. This makes a lot of memory.

Would you like to go to output pane and choose "R Language Server" and paste the log of process start and exit here?

If the languge client is constantly creating language server processes, there should be many lines like
Language Server (12345) started.

Hi, This is the output of vscode. vscode version   is 1.58.2.
R Language Server (7641) started
R Language Server (8230) started
R Language Server (8230) exited with exit code 0
R Language Server (8327) started
R Language Server (10022) started
R Language Server (10038) started
R Language Server (10299) started
R Language Server (10299) exited with exit code 0
R Language Server (10388) started
R Language Server (10701) started

R version 4.1.0 (2021-05-18)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 18.04.5 LTS

Matrix products: default
BLAS:   /usr/lib/x86_64-linux-gnu/blas/libblas.so.3.7.1
LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.7.1

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C
 [3] LC_TIME=zh_CN.UTF-8        LC_COLLATE=en_US.UTF-8
 [5] LC_MONETARY=zh_CN.UTF-8    LC_MESSAGES=en_US.UTF-8
 [7] LC_PAPER=zh_CN.UTF-8       LC_NAME=C
 [9] LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=zh_CN.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
 [1] forcats_0.5.1       stringr_1.4.0       dplyr_1.0.7
 [4] purrr_0.3.4         readr_1.4.0         tidyr_1.1.3
 [7] tibble_3.1.2        ggplot2_3.3.5       tidyverse_1.3.1
[10] BioinfoR_<IP_ADDRESS>00

loaded via a namespace (and not attached):
 [1] Rcpp_1.0.7       cellranger_1.1.0 pillar_1.6.1     compiler_4.1.0
 [5] dbplyr_2.1.1     tools_4.1.0      jsonlite_1.7.2   lubridate_1.7.10
 [9] lifecycle_1.0.0  gtable_0.3.0     pkgconfig_2.0.3  rlang_0.4.11
[13] reprex_2.0.0     cli_3.0.1        rstudioapi_0.13  DBI_1.1.1
[17] curl_4.3.2       haven_2.4.1      xml2_1.3.2       withr_2.4.2
[21] httr_1.4.2       fs_1.5.0         generics_0.1.0   vctrs_0.3.8
[25] hms_1.1.0        grid_4.1.0       tidyselect_1.1.1 glue_1.4.2
[29] R6_2.5.0         fansi_0.5.0      readxl_1.3.1     modelr_0.1.8
[33] magrittr_2.0.1   backports_1.2.1  scales_1.1.1     ellipsis_0.3.2
[37] rvest_1.0.0      assertthat_0.2.1 colorspace_2.0-2 utf8_1.2.1
[41] stringi_1.7.3    munsell_0.5.0    broom_0.7.8      crayon_1.4.1

Hello, when I use the vscode R plug-in for a while, the system process is as shown in the figure below, and the memory usage is very high.

When I manually kill these processes, the memory is significantly reduced.

R Language Server (4766) started

R Language Server (5102) started
R Language Server (6215) started
[Error - 8:35:08 AM] Request textDocument/documentSymbol failed.
Error: Connection got disposed.
	at Object.dispose (/home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-jsonrpc/lib/common/connection.js:1044:27)
	at Object.dispose (/home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-languageclient/lib/common/client.js:73:35)
	at /home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-languageclient/lib/common/client.js:2320:28
[Error - 8:35:08 AM] Request textDocument/documentLink failed.
Error: Connection got disposed.
	at Object.dispose (/home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-jsonrpc/lib/common/connection.js:1044:27)
	at Object.dispose (/home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-languageclient/lib/common/client.js:73:35)
	at /home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-languageclient/lib/common/client.js:2320:28
[Error - 8:35:08 AM] Request textDocument/colorPresentation failed.
Error: Connection got disposed.
	at Object.dispose (/home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-jsonrpc/lib/common/connection.js:1044:27)
	at Object.dispose (/home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-languageclient/lib/common/client.js:73:35)
	at /home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-languageclient/lib/common/client.js:2320:28
[Error - 8:35:08 AM] Request textDocument/documentSymbol failed.
Error: Connection got disposed.
	at Object.dispose (/home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-jsonrpc/lib/common/connection.js:1044:27)
	at Object.dispose (/home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-languageclient/lib/common/client.js:73:35)
	at /home/wangyang/.vscode/extensions/reditorsupport.r-lsp-0.1.16/node_modules/vscode-languageclient/lib/common/client.js:2320:28
R Language Server (6215) exited with exit code 0
R Language Server (6384) started
R Language Server (6462) started
R Language Server (6475) started
R Language Server (7126) started
R Language Server (13890) started
R Language Server (14036) started
R Language Server (16111) started
R Language Server (16840) started
R Language Server (17082) started
R Language Server (17082) exited with exit code 0
R Language Server (21154) started
R Language Server (21154) exited with exit code 0
R Language Server (21758) started
R Language Server (22184) started
R Language Server (22197) started
R Language Server (22184) exited with exit code 0
R Language Server (22283) started
R Language Server (22197) exited with exit code 0
R Language Server (22427) started
R Language Server (22429) started
R Language Server (22429) exited from signal SIGKILL
R Language Server (22427) exited from signal SIGKILL
R Language Server (22283) exited from signal SIGKILL
[Error - 10:15:41 AM] Connection to server is erroring. Shutting down server.
[Error - 10:15:41 AM] Connection to server is erroring. Shutting down server.
[Error - 10:15:41 AM] Connection to server is erroring. Shutting down server.
[Error - 10:15:46 AM] Connection to server got closed. Server will not be restarted.
R Language Server (21758) exited from signal SIGKILL
R Language Server (16840) exited from signal SIGKILL
[Error - 10:15:49 AM] Connection to server is erroring. Shutting down server.
[Error - 10:15:49 AM] Connection to server is erroring. Shutting down server.
[Error - 10:15:49 AM] Connection to server is erroring. Shutting down server.
R Language Server (16111) exited from signal SIGKILL
[Error - 10:15:52 AM] Connection to server is erroring. Shutting down server.
[Error - 10:15:52 AM] Connection to server is erroring. Shutting down server.
[Error - 10:15:52 AM] Connection to server is erroring. Shutting down server.
[Error - 10:15:56 AM] Connection to server got closed. Server will not be restarted.
R Language Server (14036) exited from signal SIGKILL
R Language Server (13890) exited from signal SIGKILL
[Error - 10:15:59 AM] Connection to server is erroring. Shutting down server.
[Error - 10:15:59 AM] Connection to server is erroring. Shutting down server.
[Error - 10:15:59 AM] Connection to server is erroring. Shutting down server.
[Error - 10:16:02 AM] Connection to server got closed. Server will not be restarted.
R Language Server (7126) exited from signal SIGKILL
[Error - 10:16:06 AM] Connection to server got closed. Server will not be restarted.
R Language Server (6475) exited from signal SIGKILL
R Language Server (6462) exited from signal SIGKILL
[Error - 10:16:09 AM] Connection to server is erroring. Shutting down server.
[Error - 10:16:09 AM] Connection to server is erroring. Shutting down server.
[Error - 10:16:09 AM] Connection to server is erroring. Shutting down server.
[Error - 10:16:12 AM] Connection to server got closed. Server will not be restarted.
R Language Server (6384) exited from signal SIGKILL
R Language Server (5102) exited from signal SIGKILL
[Error - 10:16:15 AM] Connection to server is erroring. Shutting down server.
[Error - 10:16:15 AM] Connection to server is erroring. Shutting down server.
[Error - 10:16:15 AM] Connection to server is erroring. Shutting down server.

@wangyang1749 Looks like you are still using vscode-r-lsp which is already integrated into vscode-R and should be uninstalled. Please use the latest vscode-R.
Are you working in multi-root workspace? Or there should not be many language server processes started.

Just to add again that I am also seeing this issue (running vscode inside of wsl2 on windows inside a docker container):

After some testing I think its relating to windows sleep/hidernation.  I can't say for certain as I haven't found a repeatable way of forcing the issue but it seems correlated with when the laptop goes to sleep. At least I've been running a background job making http get requests from the same container that sometimes errors if the laptop goes to sleep for an extended time which seems to be correlated with the additional spawning process. That being said its not caused by forcing the laptop to sleep but only after leaving the machine alone for an extend period of time. I will run some tests and update accordingly

Ok scrap that, without putting the laptop to sleep one process failed and 3 started:

Thanks for providing the info. Looks like the language client is spawning language servers in an unexpected way.

There have been a lot of issues with spawn on Windows, such as not passing PATHEXT. So much so that there are npm packages for cross-platform spawn compatibility.
Whilst it looks like this is an issue on both Windows and Linux (and Mac?), this might be something to note (I would still prefer spawn over exec).

Hi @renkun-ken  and team, Is there anything I can do / provide to help diagnose & resolve this issue as its becoming a blocker on my current project due to RAM exhaustion :(

@gowerc Are you using multi-root workspace, editing untitled files, or files outside your workspace root?
Would you like to "toggle developer tools" and share the logging of "Start language server for ..."?

This might be a separate issue but I am also seeing this in the "output" panel:

It is caused by #468 and fixed via #469. Please install  the latest development version of languageserver.

And the console log as requested:

Thanks for posting the image here. It clearly exposes the cause of starting many langaugeserver processes here. When an R file that is outside any workspace folder open in vscode, a standalone languageserver is started for each file. It should be optimized somehow, e.g. all such files share the same languageserver. I'll take a closer look at this.

I guess to be clear I havent done anything explicit to open these files so I'm assuming this is all happening behind the scenes ?

I guess to be clear I havent done anything explicit to open these files so I'm assuming this is all happening behind the scenes ?

Yes. The current language server support for multi-root workspace, untitled files, and external files via multi-server approach is too resource consuming. Let me come up with a better approach.

Currently on my work Mac if I set R_LANGSVR_POOL_SIZE=1 in my ~/.Renviron file then upon opening vscode in a directory with an open R file I see:
~/Desktop/Work/test_streams
❯ ps -u gowerc | grep R.framework
  503 73175 ??         0:00.24 /Library/Frameworks/R.framework/Resources/bin/exec/R --silent --slave --no-save --no-restore -f /Users/gowerc/.vscode/extensions/ikuyadeu.r-2.3.1/R/help/helpServer.R
  503 73184 ??         0:01.32 /Library/Frameworks/R.framework/Resources/bin/exec/R --quiet --slave -e languageserver::run(port=52670)
  503 73209 ??         0:00.28 /Library/Frameworks/R.framework/Resources/bin/exec/R --no-readline --slave --no-save --no-restore
  503 73211 ??         0:00.49 /Library/Frameworks/R.framework/Resources/bin/exec/R --no-readline --slave --no-save --no-restore

So I guess its behaving as expected :)
Will test on my windows machine this evening.
What are the consequences / negatives then of using R_LANGSVR_POOL_SIZE=1 ?

My context:

Windows 10 - Version 21H2
VSCode 1.63.2
R extension 2.3.4
R 4.1.2
languageserver 0.3.12
Are you using multi-root workspace - No
editing untitled files - No
files outside your workspace root - No
R_LANGSVR_POOL_SIZE=0 - in .Renviron
"terminal.integrated.enablePersistentSessions": false - in VSCode settings
"terminal.integrated.persistentSessionReviveProcess": "never" - in VSCode settings

Use cases:

Open VSCode, open a folder, open a R file, open a Terminal (NOT R Terminal)

In Process Explorer (under Visual Studio Code process): there are a few "R for Windows terminal front-end" and "R for Windows front-end" processes (I think thatnumber depends about the configuration above and cpus).
Closing VSCode, those processes are gone
OK

Open VSCode, open a folder, open a R file, open a Terminal (NOT R Terminal)

Execute "R: Create R Terminal"
Closing VSCode, those processes are gone
OK

Then I tried to open other R files and a few different combinations of user actions, trying to get orphan processes when I close VSCode. I cannot repro it, even if, in the past, I saw sometimes a few ones.
Last thing. Generally speaking, one of the processes related to R consumes about 2GB of RAM in my context, just only with the file editor opened or the R Terminal just created.

Likewise, it seems vscode-r never terminate the R front-end, so when we restart vscode multiple times, the R for Windows front-end accumulates gradually.
My running environment:

OS: Windows 10 20H2-19042.1415
vscode
Version: 1.63.2 (user setup)
Commit: 899d46d82c4c95423fb7e10e68eba52050e30ba3
Date: 2021-12-15T09:40:02.816Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19042
radian version: 0.5.13
R version: 4.1.1
languageserver: 0.3.12
Are you using multi-root workspace? - No
files outside your workspace root - No
"terminal.integrated.enablePersistentSessions": false - in VSCode settings
"terminal.integrated.persistentSessionReviveProcess": "never" - in VSCode settings

when I seted R_LANGSVR_POOL_SIZE = 0 in my ~/.Renviron and opened a vscode instance (never opened any files), then quit vscode, these process remained runing:

when I seted R_LANGSVR_POOL_SIZE = 1 in my ~/.Renviron and opened a vscode instance (never opened any files), then quit vscode, these process remained runing:

when I reopen vscode several times, the process R for Windows front-end in vscode will accumulate:

This shouldn't be caused by radian, I have run radian alone with R in windows cmd.exe, which didn't present above results.

Resolved with https://github.com/REditorSupport/vscode-R/pull/941

Thanks @renkun-ken and @albertosantini for tracking down and fixing this bug :) will make a big difference for windows users !


Refactor structure in GroupBy to dispatch to SeriesGroupy earlier

ref #525
the initial method on the DataFrame groupby goes back to missing series groupby support, this cleans that up

merging, others are building on top of this one


CRAM integration tests

Finishes #677 and #673

Review complete -- back to @edwardkw

After some revision, this still successfully handles the lack of a reference for #673, but a runtime null pointer is encountered with a wrong reference (previously, a sequence dictionary error was being caught), so this doesn't quite do it for #677. The other issues outlined were also fixed.

Second-pass review complete -- back to @edwardkw

In general, you should wait for an official "LGTM" or :+1: before merging :)

Looks good though, so this gets a post-merge :+1:  :)

I accidentally clicked it too soon, and figured I might as well delete the branch. My bad.


What environment variables are required?

What needs to be in .profile/.bash_profile for GVM to work? e.g. environment variables

Nothing. The installer modifies .bashrc.


Fix change:song event not firing

Fixes #45
This creates a wrapper function that safely fetches the textContent of a possibly null or empty element.
Also handles the albumArt changing without the metadata changing
/cc @gmusic-utils/gmusic-js

High level content looks good, left some feedback to improve maintenance in the future

Updated PR

Looks good to me, :shipit:


[5.5] Dynamically compose template methods (setUp/tearDown) from TestCase traits

The current TestCase:: setUpTraits method has a hard-coded list of method calls. This uses class_uses_recursive similarly to the way Model::bootTraits behaves, calling setUp<TraitName> and tearDown<TraitName> if they're set.
This means that if you had a test case & trait like:
<?php

trait FooBar
{
    protected $foo;

    public function setUpFooBar()
    {
        $this->foo = 'bar';
    }
}

class MyTest extends TestCase
{
    use FooBar;

    public function testFooBar()
    {
        $this->assertEquals('bar', $this->foo);
    }
}

The $foo property would automatically set to "bar" before each test case.

This would be a breaking change.


when installing docker with juju getting error as:hook failed: "config-changed"

i am installing docker in juju,while installing i am getting error as below:
root@juju:~# juju status
Model    Controller  Cloud/Region         Version  SLA
default  lxd-test    localhost/localhost  2.3.4    unsupported
App              Version  Status   Scale  Charm      Store       Rev  OS      Notes
big-db           5.7.21   error        1  mysql      jujucharms   58  ubuntu
big-wiki         1.19.14  blocked      1  mediawiki  jujucharms   19  ubuntu
central-monitor           active       1  nagios     jujucharms   23  ubuntu  exposed
docker                    error        1  docker     jujucharms    8  ubuntu
Unit                Workload  Agent  Machine  Public address  Ports           Message
big-db/0*           error     idle   1        <IP_ADDRESS>     3306/tcp        hook failed: "monitors-relation-joined" for central-monitor:monitors
big-wiki/0*         blocked   idle   2        <IP_ADDRESS>                    Database required
central-monitor/0*  active    idle   0        <IP_ADDRESS>    80/tcp,443/tcp  ready
docker/0           error     idle   3        <IP_ADDRESS>                     hook failed: "config-changed"*
Machine  State    DNS           Inst id        Series  AZ  Message
0        started  <IP_ADDRESS>  juju-e3695e-0  xenial      Running
1        started  <IP_ADDRESS>   juju-e3695e-1  xenial      Running
2        started  <IP_ADDRESS>  juju-e3695e-2  trusty      Running
3        started  <IP_ADDRESS>   juju-e3695e-3  trusty      Running
can any one help me out in this issue...
root@juju:~# juju version
2.3.4-xenial-amd64
root@juju:~#
root@juju:~# snap version
snap    2.31.2
snapd   2.31.2
series  16
ubuntu  16.04
kernel  4.13.0-36-generic
root@juju:~#

To make the docker charm work on lxd (the localhost provider), you'll need to apply a custom lxd profile. See https://github.com/juju-solutions/bundle-canonical-kubernetes/wiki/Deploying-on-LXD

can i know the exact configuration file?i am totally new to juju

export JUJU_MODEL=default # (the name of your juju model)

# download the default lxd profile
wget https://raw.githubusercontent.com/conjure-up/spells/master/canonical-kubernetes/steps/00_process-providertype/lxd-profile.yaml

# update lxd profile yaml and apply it
sed "s/##MODEL##/$JUJU_MODEL/" lxd-profile.yaml | lxc profile edit "juju-$JUJU_MODEL"

# destroy and recreate juju model
juju destroy-model -y $JUJU_MODEL
juju add-model $JUJU_MODEL

# now, redeploy your applications...

Hi tvansteenburgh,
still no use,getting same error.
root@juju:~# juju status
Model    Controller  Cloud/Region         Version  SLA
default  lxd-test    localhost/localhost  2.3.4    unsupported
App     Version  Status  Scale  Charm   Store       Rev  OS      Notes
docker           error       1  docker  jujucharms    8  ubuntu
Unit       Workload  Agent  Machine  Public address  Ports  Message
docker/0*  error     idle   0        <IP_ADDRESS>           hook failed: "config-changed"
Machine  State    DNS           Inst id        Series  AZ  Message
0        started  <IP_ADDRESS>  juju-c6c97d-0  trusty      Running
root@juju:~#

Hi tvansteenburgh,
still i am getting same error.can u plz  help me on this issue.

Please paste the log from the docker charm (juju debug-log -i unit-docker-0 --replay --no-tail).

Hi,
Please find the some logs:
unit-docker-0: 16:26:08 DEBUG unit.docker/0.install Collecting urllib3<1.23,>=1.21.1 (from requests!=2.11.0,>=2.5.2->docker-py->-r hooks/python-pkgs.txt (line 5))
unit-docker-0: 16:26:08 DEBUG unit.docker/0.install   Using cached urllib3-1.22-py2.py3-none-any.whl
unit-docker-0: 16:26:08 DEBUG unit.docker/0.install Collecting chardet<3.1.0,>=3.0.2 (from requests!=2.11.0,>=2.5.2->docker-py->-r hooks/python-pkgs.txt (line 5))
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install   Using cached chardet-3.0.4-py2.py3-none-any.whl
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install Collecting certifi>=2017.4.17 (from requests!=2.11.0,>=2.5.2->docker-py->-r hooks/python-pkgs.txt (line 5))
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install   Using cached certifi-2018.1.18-py2.py3-none-any.whl
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install Requirement already satisfied: pycparser in /usr/local/lib/python2.7/dist-packages (from cffi>=1.7; platform_python_implementation != "PyPy"->cryptography->ansible->-r hooks/python-pkgs.txt (line 3)) (2.18)
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install cloud-init 0.7.5 requires argparse, which is not installed.
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install cheetah 2.4.4 requires Markdown>=2.0.1, which is not installed.
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install requestbuilder 0.1.0b1 requires argparse, which is not installed.
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install ssh-import-id 3.21 requires argparse, which is not installed.
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install euca2ools 3.0.1 has requirement requestbuilder, but you'll have requestbuilder 0.1.0b1 which is incompatible.
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install Installing collected packages: AnsibleCharm, urllib3, chardet, certifi, requests, docker-py
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install   Found existing installation: AnsibleCharm 0.1
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install     Uninstalling AnsibleCharm-0.1:
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install       Successfully uninstalled AnsibleCharm-0.1
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install   Running setup.py develop for AnsibleCharm
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install   Found existing installation: urllib3 1.7.1
unit-docker-0: 16:26:09 DEBUG unit.docker/0.install Cannot uninstall 'urllib3'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall.
unit-docker-0: 16:26:09 ERROR juju.worker.uniter.operation hook "install" failed: exit status 1
unit-docker-0: 16:26:09 INFO juju.worker.uniter awaiting error resolution for "install" hook
unit-docker-0: 16:27:01 INFO juju.worker.uniter awaiting error resolution for "install" hook
unit-docker-0: 16:31:09 INFO juju.worker.uniter awaiting error resolution for "install" hook
root@juju:~#


Expander and Do & Don't icons not displaying properly on Internet Explorer 11

Bug Report
What is the issue?
The plus and minus icons on the Expander (Details variant) component does not display properly on Internet Explorer 11. A similar problem is happening with the Do & Don't list ticks and crosses, but this only seems to be an intermittent problem (if the icons don't initially display, scroll up or down the page seems to make them appear).
The components are still functional - this is a visual inconsistency.
IE 11 is currently 3.2% of total visits to www.nhs.uk
What steps are required to reproduce the issue?
Use Internet Explorer 11 and visit https://nhsuk.github.io/nhsuk-frontend/components/details/expander.html
Use Internet Explorer 11 and visit https://nhsuk.github.io/nhsuk-frontend/components/do-dont-list/index.html
NHS website page: https://www.nhs.uk/conditions/stress-anxiety-depression/understanding-stress/
What was the environment where this issue occurred?

Browser: Internet Explorer
Browser version: 11
NHS.UK frontend package version: Latest

Screenshots

Expander: I think the background-size of the plus/minus icons needs setting for both the width and height. Internet Explorer 11 requires both of them to be set rather than just 1.
background-size: 32px 32px;

Do & Don't list: I think the implementation of the Do & Don't component is different on the NHS website as it is on the frontend library. This may be specific to the NHS website.