Remote webdriver not working

I have a docker container running selenium-chrome (the "standalone-chrome" official container) and I'm trying to get scrapy-selenium to work with it. I have this in settings.py:
DOWNLOADER_MIDDLEWARES = {
      'scrapy_selenium.SeleniumMiddleware': 800
}
SELENIUM_DRIVER_NAME = 'chrome'
SELENIUM_COMMAND_EXECUTOR='http://localhost:4444/wd/hub'
SELENIUM_DRIVER_ARGUMENTS=['--headless']

and I tried different combinations, but I keep getting
[scrapy.middleware] WARNING: Disabled SeleniumMiddleware: SELENIUM_DRIVER_NAME and SELENIUM_DRIVER_EXECUTABLE_PATH must be set

I don't think SeleniumRequest is working, I get further down a KeyError for 'screenshot' on response.meta (I did set screenshot=True in SeleniumRequest).
The selenium container is indeed working, it works if I test it by passing it to webdriver.Remote. Any suggestions?

No. I hadn't noticed the project is not maintained anymore. I am now trying to use scrapy-splash but having other problems. It's a pity because in my limited experience Selenium works better than Splash
Maybe I'll come back to this and not use the remote webdriver, but this makes it harder to dockerize the scraper.

Having the same issue.

Remote selenium driver doesn't to work in version 0.7. For testing, replace middleware.py with the one from latest commit. Also, use SELENIUM_DRIVER_ARGUMENTS=['-headless'] for remote chrome driver.

I have the same issue so I am using selenium directly and there's no problem at all that is.. it doesn't make sense of using this package.

Are there any fixes to this issue? I am running into this issue when using 0.7


Unify the Cudnn Execution Plan Runner

This PR tries to remove the CudnnConvRunner and CudnnFusedConvRunner, since both the runners (for unfused and fused conv respectively) actually share the same execution path: build op graph -> get execution plans -> allocate workspace -> prepare input/output data -> execute the plan. This execution path is also true for other Cudnn supported ops. So, this PR tries to move all these through the improved CudnnExecutionPlanRunner.
This PR can also help the integration of the new ops from Cudnn in the future. That way, for newly added ops, we just need to (1) prepare a GetXXXGraph() to build the op graph (2) add a XXXSignature to mark the new inputs/outputs.
cc. @nluehr

@awpr, can you review?

Thanks, I like this direction of simplifying cuDNN frontend runners into one template -- just a few small style things to comment on.

Thanks for the comments. I have committed the new change. PTAL.  @awpr

It seems the PR has been hanging for a while. I see some tests fail but they seem unrelated to the PR. Can you help check? @awpr @jurahul

Looks like it needs some changes to internal code that uses the runners.  Should be pretty minimal changes; I can probably get to it tomorrow.


Added Harry Potter house emojis

Added Harry Potter house emojis

Gryffindor
Slytherin
Hufflepuff
Ravenclaw
Hogwarts Crest

Nice! Looks good, merging.


Feature/input pre processing

Adds pre-processing for the input (to Upper, remove counties, tokenize, replace synonyms)
Moves different collections of values to a separate files (asked by Tech Lead)
Replaces type aliases with simplier ones (asked by Tech Lead) in the parts of code that I worked on

In what file can I view the preprocessing?

In Tokens.scala and TokensTest.scala


Added settings for publishing to bintray

Basic support for publishing to bintray that can handle publishing to jcenter, maven-central and snapshots to oss.jfrog.org.
Bintray is configured in dryRun mode and that needs to changed before being able to publish.
User and Key needs to be passed to the gradle script via an environment variable or as a gradle property:
http://mrhaki.blogspot.se/2016/05/gradle-goodness-get-property-or-default.html
@rogeralsing a bintray account needs to be setup @rogeralsing and linked to jcenter.
Versioning is not covered by this PR so the version need to be bumped manually.
Regarding publishing to maven central there are a couple of thing that needs to be done but can be deferred:

Signing, this can be handled by bintray
Documentation need to published, at the moment we're only publishing the jar and sources. This can be done with https://github.com/Kotlin/dokka
It's recommended to not require alternative repositories like

        maven {
            url 'http://dl.bintray.com/kotlin/kotlin-eap-1.2'
        }

https://maven.apache.org/guides/mini/guide-central-repository-upload.html

Codecov Report

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

@@            Coverage Diff            @@
##             master      #11   +/-   ##
=========================================
  Coverage     23.12%   23.12%
+ Complexity      137      132    -5
=========================================
  Files            51       51
  Lines          1747     1747
  Branches        294      294
=========================================
  Hits            404      404
  Misses         1258     1258
  Partials         85       85

Continue to review full report at Codecov.

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

Parameters can be passed like this:
./gradlew bintrayUpload -PprotoBintrayUser=foo -PprotoBintrayKey=bar

or as environment variables
export ORG_GRADLE_PROJECT_protoBintrayUser=foo
export ORG_GRADLE_PROJECT_protoBintrayKey=bar

./gradlew bintrayUpload


MeshModel Registry Count Inaccurate

Current Behavior
Meshery Server's MeshModel registry can contain many hundreds of components, but the summary offered in the "Settings" page does not reflect this count accurately.
Screenshots/Logs

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

I would like to work on this!

@Revolyssup @theBeginner86 @Abhishek-kumar09 @MUzairS15

@pkbhowmick oh, I see. Given your findings, I suppose that this isn't a bug but more of a feature request to count the number of components per model in addition to the number of versions. So, a new column would be added to this table.

Thanks @leecalcote for the info, so there will be one more column containing the number of component running for each model. I am on it.

Good deal. Go, go, go.

There will be a few other enhancements needed in this capabilities browser (MeshModel Summary) after this.

Hey there! If this issue is up for grabs can I take a shot at fixing it?

@sid-008 I missed your message earlier. It'd be good to have you jump in, although, we've come to find that a count is dependent upon an enhancement in the /api/meshmodel APIs. // @UsmanDev09 @abdullah1308

Sure thing, let me know how I can help

Hello. I am looking for issues that I can work on. I see that this is an old issue, so I just wanted to double check whether this was fixed. When I look
here, it seems like you can look at the models and see how many components each model has. Is this Github issue describing a separate issue? or was this the issue and it was solved?

@leecalcote

@RipulHandoo, @alphaX86 ^^

@Yashsharma1911


removed attempt id when running in yarn mode, as yarn has a seperate …

…history server for failed attempts, and current attempt is always the running attempt

some issues, this is not really needed, i jumped the gun it seems.


fix: RollbarAUL cocoapod

Description of the change

fix: cocoapods build of RollbarAUL

Type of change

[x] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

n/a

Checklists
Development

[x] Lint rules pass locally
[ ] The code changed/added as part of this pull request has been covered with tests
[x] All tests related to the changed code pass in development

Code review

[ ]  This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] "Ready for review" label attached to the PR and reviewers mentioned in a comment
[ ] Changes have been reviewed by at least one other engineer
[ ] Issue from task tracker has a link to this pull request

@pawelsz-rb , thank you!


Break Resource page into separate pages for each core concept

Even after breaking the Programming Model content down, this Resources page still contained a lot of core content.  This made a lot of important features feel very "hidden", like resource options, components, dynamic providers, and more.
It also led to suboptimal search results, both in Google and in our own internal site search, because searches for specific concepts (like "pulumi aliases") would return the "Resources" page, which didn't even rank particularly well as it doesn't "feel" like definitive content for this specific topic.
This change breaks this page into 18 separate pages.  It keeps the "Custom Resource" content in the main Resources page (since that content is nearly entirely actually not specific to Custom Resources anyway), pulls the Component Resource, Resource Providers and Dynamic Providers content into separate top level items, and pulls the Resource Options content into it's own index with a separate page per resource option.
Some of the resulting pages are fairly light in content, but this actually offers room to flesh these out more fully now that they have room to breathe.
Reading through this content as part of the refactor, there is quite a bit of cleanup we could still do here.  I took a few liberties to clean up some things as part of this, but mostly focused on the refactor instead of changes to prose.
This also introduces a set of client-side redirects to the resources page, so that each of the previous anchors which now lives elsewhere will get redirected in the client, following the same strategy used in the Programming Model page refactor.

Any reason to stop at leaving the few topics still in the top-level resources doc? If we do keep the few topics there (creation, names, args, getters), it would be worth advertising that the other topics also exist. As far as I can tell, you'd need to pay close attention to the nav on the left hand side to know that other content exists. A section at the top and/or bottom, maybe?

should the children under resources be in alphabetical order?

I wasn't sure about this at first, but after another look, I think this ends up with a natural order that is also nicely alphabetical, so have pushed a change that updates this to be alphabetical for now.

the dynamic providers page is still quite long, would it make sense to break it up even more?

Although long, this one is more clearly logically "one topic".  I'll leave this as one page for now.

Any reason to stop at leaving the few topics still in the top-level resources doc?

I agree, I'm going to try one of these two options - splitting most of the remaining content out and/or adding more of an index of the sub-pages to the main resources page.

Gentle prod, @lukehoban
I've had to link to these pages a lot lately, and I would love love love for this change to be there.
By the way, will prior links "just work" after this change, including anchored links like https://www.pulumi.com/docs/intro/concepts/resources/#dependson?

Yep - still planning to land this. I agree with your feedback above about doing a little more to split the core custom resource out of the overview page - so want to finish that up before merging. But expect to land this this week.

By the way, will prior links "just work" after this change, including anchored links like https://www.pulumi.com/docs/intro/concepts/resources/#dependson

Yes. This uses the same client-side redirect trick we have used in the past to ensure that every anchor that was on the old page will redirect to its new location. It’s slightly more “jagged” than a server side redirect, but from my testing is still a good experience. You can try it yourself in the preview link above.

@lukehoban there is a ref issue over in the docs pr for this..
ERROR 2022/01/18 02:30:21 [en] REF_NOT_FOUND: Ref "additional-secret-outputs": "/tmp/hugo_cache/modules/filecache/modules/pkg/mod/github.com/pulumi/pulumi-hugo/themes/default@v0.0.0-20220118014907-135686cccef6/content/docs/intro/concepts/resources/options/_index.md:13:29": page not found


Work hours licensing inconsistency (CS Pro)

On the Dynamics Licensing Guide, it says Customer Service Professional does not support work hours but it also says it is supported. Which is the correct information?
I have tried and CS Pro users are able to create new work schedules so there is no apparent technical enforcement on any restrictions.
If work hours are available on CS Pro: It should be clarified on the Licensing Guide
If they are not: It should be clarified on the docs that CS Enterprise is a requirement for this setup.

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

ID: 96ee872c-0e8f-b9d4-197e-4007d7fbd156
Version Independent ID: fed16b86-49d4-94e5-e175-c9198052676e
Content: Create customer service schedule and define work hours (Dynamics 365 Customer Service)
Content Source: ce/customer-service/create-customer-service-schedule-define-work-hours.md
Service: dynamics-365-customerservice
GitHub Login: @lalexms
Microsoft Alias: laalexan

@JMCA2 : The Licensing Guide states that for Tables: Create, Update, Delete is available to Pro and Ent, however for Tables: Actions; Work hours: manage, it is restricted to Ent.
If you have additional questions/feedback, please use Dynamics 365 Licensing Guide feedback form.


When I get access token from /{tenant}/oauth2/v2.0/token, I get "expires_in" as string

If I use this token endpoint:  /{tenant}/oauth2/token
I get "expires_in" as a string:  "expires_in": "3599". I'm using my own tenant and app id in this case.
However, if I use this endpoint: /common/oauth2/v2.0/token
I get "expires_in" as an integer: "expires_in": 3599. I'm using the Postman  Samples provided in the document in this case.
Why?

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

ID: fef99ca3-72bb-2c48-a62b-9ff9c314d72b
Version Independent ID: e58fd3a1-0af8-6e16-3d78-3dc748992593
Content: Microsoft identity platform and OAuth 2.0 authorization code flow - Microsoft identity platform
Content Source: articles/active-directory/develop/v2-oauth2-auth-code-flow.md
Service: active-directory
Sub-service: develop
GitHub Login: @hpsin
Microsoft Alias: hirsin

@pablolaurin Thanks for the comment. We are actively investigating and will get back to you soon.

I found my mistake. It works as documented.
I'm using the V1 endpoint in the first case and the V2 endpoint in the second. The documentation for V1 shows examples with expires_in as string. For V2, expires-in is an integer.

phew Thanks for double checking @pablolaurin!


Turn on curl -v when --debug is specified

when genesis init returns a github error in a complicate network environment, we would like to see traffic details when it curls the release/pre-release tar ball

We should probably call this --trace, and also control it via the GENESIS_TRACE environment variable.

Note: current genesis --trace shows less output compare with --debug when running genesis init. It supposes to "Even more debugging, including debugging inside called tools (like spruce and bosh" from genesis -h

hmm, the code just has DEBUG AND TRACE, not GENESIS_DEBUG AND GENESIS_TRACE -- a minor but noteworthy difference.  Regarding the --trace option: Because you can specify both --trace and --debug, I think its better to put any "forwarding" debugging (ie turns on debugging output in subsequently called executables) into --trace as James suggests, and fix the description.  Alternatively, turning on --trace also turns on debug, but that's much messier leading to obscurity/confusion.

TRACE, GENESIS_TRACE, whatever the variable name, curl debugging is a 'second stage' debugging effort, something I'd rather not see with just --debug

@dennisjbell do you have commits in-flight for this one?

I've got commits in-flight on this, waiting for a merge to avoid conflicts.


Unable to communicate with fork server (OOM?)

Hi.
Using the last version of jqf after successful build and start (with afl++ afl --branch 2.25) i stuck into:
user@user-VirtualBox:~/work/Java/target$ ../jqf/bin/jqf-afl-fuzz  -m 30000000 -t 1000000  -i ../test/in -o ../test/out test1 testRead
Performing pilot run....  Pilot run success! Launching AFL now...
afl-fuzz++2.52c by<EMAIL_ADDRESS>schedules by<EMAIL_ADDRESS>[+] Using exploration-based constant power schedule (EXPLORE)
[+] You have 2 CPU cores and 2 runnable tasks (utilization: 100%).
[!] WARNING: Not binding to a CPU core (AFL_NO_AFFINITY set).
[] Checking core_pattern...
[] Setting up output directories...
[+] Output directory exists but deemed OK to reuse.
[] Deleting old session data...
[+] Output dir cleanup successful.
[] Scanning '../test/in'...
[+] No auto-generated dictionary tokens to reuse.
[] Creating hard links for all input files...
[] Validating target binary...
[] Attempting dry run with 'id:000000,orig:1'...
[] Spinning up the fork server...
[+] All right - fork server is up.
[-] PROGRAM ABORT : Unable to communicate with fork server (OOM?)
Location : run_target(), afl-fuzz.c:2441
When i change jqf to the March commit (git checkout d41d87d6899ee0b1ece05763c376bfcebc8f095a for example) and rerun setup.sh everything works fine.
Memory and timeout could be set to none - it does not mater.
Attaching my simple test1.java for reproduction.
test1.java.txt

Thank you! I confirm that I can reproduce this and also that the issue did not exist in the March commit. I appreciate the detailed report.
On further bisection, it seems the regression was introduced in commit 65a9f09ed due to the relaxing of entry point matching in ThreadTracer. The AFL error goes away if the environment variable JVM_OPTS is set to include the option -Djqf.tracing.MATCH_CALLEE_NAMES=true. While this is a workaround, it should not be necessary (and will likely reduce fuzzing speed). I will try to find a permanent fix today that allows the JQF-AFL integration to work by default.

Pushed a fix. Please reopen if this issue persists even after commit 8608101.


Unhandled rejection TypeError: Cannot read property 'nextSibling' .

Unhandled rejection TypeError: Cannot read property 'nextSibling' of null
at View.removeNodes (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-rc.1.0.0/aurelia-templating.js:1504:23)

I move to new route by using
this.router.navigateToRoute('index');

Then I press "back" button , route changes but app catches this error.
Also I catch this error trying to set route (for example 'localhost:9000/#/welcome") directly in address line

Please update to the latest version of the templating library.

Thx


Utilizing new client and overhauling Accumulator interface

Fixes #280
Fixes #281

@jipperinbham this is merged, so work away. If you could also be on the lookout for any issues you may see in the datadog output with these changes I would appreciate it a lot, thanks!

:+1:


enable cross-publishing for both sbt 0.13 and 1.x

This enables use of the sbt-scapegoat plugin in sbt 0.13 projects without having to do
(scalaBinaryVersion in ThisBuild) := "2.12" which causes lots of problems with projects doing cross-compiling.

@sksamuel would love if you could have a look and let me know if you make a new release


Merge Network to current dev branch and make it hidden for most users for now

QA passed.

Please recheck it in 1.5.5 and test how RemoteConfig will enable Network without any user actions

Now Network feature appeared. QA passed.


pip install carla failing

.

failing for me too
pip3 install carla
Collecting carla
Using cached carla-0.9.5.tar.gz (3.9 kB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [8 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "C:\Users\sreen\AppData\Local\Temp\pip-install-5g70b6jq\carla_b0fc1119f986485a9f6f7e424b78af99\setup.py", line 114, in
ext_modules=get_libcarla_extensions(),
File "C:\Users\sreen\AppData\Local\Temp\pip-install-5g70b6jq\carla_b0fc1119f986485a9f6f7e424b78af99\setup.py", line 75, in get_libcarla_extensions
libs = [x for x in os.listdir('dependencies/lib') if any(d in x for d in required_libs)]
FileNotFoundError: [WinError 3] The system cannot find the path specified: 'dependencies/lib'
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Me too :(
Any solution?

I was able to get it running :)
from what I understand CARLA can only work with python3.7, I was running on python 3.10.
Downgrading python to 3.7 fixed my issues.

Thank you!
It is solved!

Downgrading to 3.7 gave me the same error, any ideas?

If carla cannot be installed, it is recommended to use the whl installation solution:

Download the wheel from here: https://pypi.org/project/carla/#files
Unzip and download as shown in Figure ① below
Move to ${HOME}$/.local/lib/python3.8/site-packages as shown below ②
Finally, when grep carla below, as shown in figure ③ below

Same problem.
Defaulting to user installation because normal site-packages is not writeable
Collecting carla
Using cached carla-0.9.5.tar.gz (3.9 kB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [8 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "/tmp/pip-install-_w7w01nj/carla_09d577e291614b99b70f02f9b3c3d74c/setup.py", line 114, in
ext_modules=get_libcarla_extensions(),
File "/tmp/pip-install-_w7w01nj/carla_09d577e291614b99b70f02f9b3c3d74c/setup.py", line 31, in get_libcarla_extensions
linux_distro = platform.dist()[0]  # pylint: disable=W1505
AttributeError: module 'platform' has no attribute 'dist'
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Same problem.
Defaulting to user installation because normal site-packages is not writeable Collecting carla Using cached carla-0.9.5.tar.gz (3.9 kB) Preparing metadata (setup.py) ... error error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully. │ exit code: 1 ╰─> [8 lines of output] Traceback (most recent call last): File "", line 2, in File "", line 34, in File "/tmp/pip-install-_w7w01nj/carla_09d577e291614b99b70f02f9b3c3d74c/setup.py", line 114, in ext_modules=get_libcarla_extensions(), File "/tmp/pip-install-_w7w01nj/carla_09d577e291614b99b70f02f9b3c3d74c/setup.py", line 31, in get_libcarla_extensions linux_distro = platform.dist()[0] # pylint: disable=W1505 AttributeError: module 'platform' has no attribute 'dist' [end of output]
note: This error originates from a subprocess, and is likely not a problem with pip. error: metadata-generation-failed
× Encountered error while generating package metadata. ╰─> See above for output.
note: This is an issue with the package mentioned above, not pip. hint: See above for details.

Same error


Moving common role to a separate repo and galaxy. Travis CI integration

fixes #25 and adds travis fun at https://travis-ci.org/rhtconsulting/ansible-middleware-playbooks. Right now travis checks galaxy integration and syntax. Need the docker travis solution working to check playbooks themselves

A couple minor gremlins in the README leftover from the reshuffling of playbooks, but nothing that should hold back this PR. Will open issues/PR's  for those issues
merging


revup modifies the git commit in ways that doesn't take commit.cleanup into mind

Describe the bug
My gitconfig has the following part
[commit]
  template = ~/.gitcommit.template
  cleanup = scissors

The template is a markdown file that is used to format the commit message. The code here https://github.com/Skydio/revup/blob/main/revup/amend.py#L47 strips all # prefixed lines.
Expected behavior

revup should use gitconfig settings or should not modify the git commit message at all.

To Reproduce

Set git config commit.cleanup=scissors
Add a git commit message with markdown (add a heading with # in the start) git commit -a
Confirm that the markdown is present as entered by the user git log
Run revup amend and close the editor without any changes
Check git message git log

# prefixed lines are removed.
Logs
Terminal output from the command. If possible, rerun the command with revup -v to get more verbose logs.

thanks for the report. i've added some changes to respect the cleanup and commentChar options in config


Initial build 0.5.1

Upstream repo (v0.4.6)
Jira
whylabs-client -> whylogs

No license file present, and I can't find a url to reference in the documentation, but PKG-INFO states that the license is Apache 2.0. PKG-INFO is present in the package source from Pypi, but not the Github repository.
Added a patch to fix a reference to a non-existent file in setup.py.

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_ee6dkfz1gz/clone/recipe/meta.yaml:24: patch_must_be_in_build: patch must be in build when source/patches is set.
INFO: /tmp/abs_ee6dkfz1gz/clone/recipe/meta.yaml:1129: invalid_url: https://docs.whylabs.ai : URL domain redirect docs.whylabs.ai ->
WARNING: /tmp/abs_ee6dkfz1gz/clone/recipe/meta.yaml:1129: missing_license_file: The recipe is missing the about/license_file or about/license_url key.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_806xe8cqb2/clone/recipe/meta.yaml:24: patch_must_be_in_build: patch must be in build when source/patches is set.
INFO: /tmp/abs_806xe8cqb2/clone/recipe/meta.yaml:1134: invalid_url: https://docs.whylabs.ai/docs : URL domain redirect docs.whylabs.ai ->
WARNING: /tmp/abs_806xe8cqb2/clone/recipe/meta.yaml:1134: missing_license_file: The recipe is missing the about/license_file or about/license_url key.
Errors were found

Linter check found the following problems:
The following problems have been found:
INFO: /tmp/abs_9d6e2unuop/clone/recipe/meta.yaml:1138: invalid_url: https://docs.whylabs.ai/docs : URL domain redirect docs.whylabs.ai ->
WARNING: /tmp/abs_9d6e2unuop/clone/recipe/meta.yaml:1138: missing_license_file: The recipe is missing the about/license_file or about/license_url key.
Warnings were found

I've requested the license file https://github.com/whylabs/whylabs-client-python/issues/2

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_d006_it80o/clone/recipe/meta.yaml:1138: missing_license_file: The recipe is missing the about/license_file or about/license_url key.
INFO: /tmp/abs_d006_it80o/clone/recipe/meta.yaml:1138: invalid_url: https://docs.whylabs.ai/docs : URL domain redirect docs.whylabs.ai ->
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_acb9choq2s/clone/recipe/meta.yaml:1138: missing_license_file: The recipe is missing the about/license_file or about/license_url key.
INFO: /tmp/abs_acb9choq2s/clone/recipe/meta.yaml:1138: invalid_url: https://docs.whylabs.ai/docs : URL domain redirect docs.whylabs.ai ->
Warnings were found


[Testing] Updating test framework for karmor client

Currently we only have tests for karmor recommend , but other features might also need testing. Here's a list that can be tested:

[ ]  karmor logs (?)
[ ] karmor probe

Hi @PrimalPimmy I want to work on this :)

Hello @Parthiba-Hazra , sure! You can start with looking into how we can test Karmor probe here, to ensure any changes don't make the CI fail.

Hi @PrimalPimmy I think you want to provide some link on creating test cases for karmor probe ?

@Parthiba-Hazra Sure, you can check out how probe works internally, and how we can use it's external functions to test.
https://github.com/kubearmor/kubearmor-client/blob/main/probe/probe.go
We use Ginkgo tests here: https://github.com/kubearmor/kubearmor-client/tree/main/tests
need a new folder named probe and do the necessary testing. But ofcourse, you will have to try out the karmor cli first to understand how it works.


i don't know

error logs
https://gist.github.com/mat41997/ef68fcb4d9590b10244c5c71988db71b

Looks fine to me. The double log entries are fixed in the latest builds.
https://ci.lucko.me/job/LuckPerms/


On windows handle the long paths in realpathSync.native

Fixes #49470

@typescript-bot pack this

Gross
LGTM

@DanielRosenwasser @amcasey do we need to port this to 4.8 ? 4.7 potentially if we are going to have patch for 4.7 at all?

@sheetalkamat The issue linked in the description doesn't have a lot of upvotes.  Have we been hearing about a lot of people hitting this issue?  I'm not sure I'd backport to 4.7 based on the information I have now.

@amcasey its not even the issue linked.. My bad that i assumed the debugging information we received from @fwienber was for that issue.
@typescript-bot cherry pick this to release-4.8

@typescript-bot cherry-pick this to release-4.8


Visualscript editor sometimes selects multiple nodes as if pressing ctrl

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Godot 3.0 master https://github.com/godotengine/godot/commit/4f39ce32b9195405f934445b20059e86632b47f9
Issue description:

The editor doesn't blur the selected node when selecting another node as if you were selecting it with ctrl pressed. The only way to deselect all the nodes is clicking an empty space.
Steps to reproduce:
I don't know how to reproduce it. If I find how to I'll update this issue.

I am not sure this issue even exists anymore.. Still worth keeping open?


Use the official Dart image

fixes: https://github.com/dart-lang/dart-pad/issues/1902

Perhaps want to update to the 2.13.1 images while you're at it?

Blocked on: https://github.com/dart-lang/protobuf/issues/509

The official Dart image trails

Perhaps want to update to the 2.13.1 images while you're at it?

I'm pretty sure 2.13.1 didn't exist when I crafted this PR +)


parameterToString() incorrectly encodes time.Time

In client/client.go the function parameterToString() returns incorrectly encoded query parameter values for time.Time interfaces. Before the latest update:
if reflect.TypeOf(obj).Kind() == reflect.Slice {
	return strings.Trim(strings.Replace(fmt.Sprint(obj), " ", delimiter, -1), "[]")
} else if t, ok := obj.(time.Time); ok {
	return t.Format(time.RFC3339)
}

This worked but the function now reads
if reflect.TypeOf(obj).Kind() == reflect.Slice {
	return strings.Trim(strings.Replace(fmt.Sprint(obj), " ", delimiter, -1), "[]")
}

and returns 400 bad request status code

Sorry about that. It's been fixed now.
I was patching it locally and had contributed a PR to the OpenAPI Generator project which should have included it in 3.1.0:
https://github.com/OpenAPITools/openapi-generator/pull/502


fixes #16 - message for 'Could not resolve host' failure

if r10k deploy fails with a 'Could not resolve host' error, append a message explaining how to skip provisioning

 Great just fixed 4 issues,  2 to go.
 Your branch must be test infected. lib/vagrant-r10k/action/deploy.rb:1
 Your branch must be test infected. spec/acceptance/vagrant-r10k/vagrant-r10k_spec.rb:1
 Your branch must be test infected. spec/unit/action_deploy_spec.rb:1
 We ignored double quote rule for this review spec/unit/action_deploy_spec.rb#raises an error

Acceptance tests passed locally


146 Create Discussion API Stack

This PR creates the completed discussion API stack
New routes:
Discussions
GET http://localhost:4200/api/v1/auth/discussions/goal/:id
GET http://localhost:4200/api/v1/auth/discussions/topic/:id

PATCH http://localhost:4200/api/v1/auth/discussions/goal/:id
PATCH http://localhost:4200/api/v1/auth/discussions/topic/:id

POST http://localhost:4200/api/v1/auth/discussions/goal/:id
POST http://localhost:4200/api/v1/auth/discussions/topic/:id

Comments
POST http://localhost:4200/api/v1/auth/discussions/comment
PATCH http://localhost:4200/api/v1/auth/discussions/comment/:commentId
DELETE http://localhost:4200/api/v1/auth/discussions/comment/:commentId

Ratings
POST http://localhost:4200/api/v1/auth/discussions/rating/:commentId
DELETE http://localhost:4200/api/v1/auth/discussions/rating/:commentId

Intentionally not requesting review just yet as there are still the Swagger API definitions that need to go in before this PR is ready

@briangormanly ready when you are for the merge!


Invariant Violation

I have found a way to replicate at least one of the "Invariant Violation" errors we keep seeing in Sentry:

Be logged out
Click Quick Submit, get login prompt
Cancel login prompt without logging in

So we should be handling this better, but if this is why we're getting so many errors, why are so many people cancelling the login prompt? Will keep digging.
Here is the relevant Sentry issue:
https://sentry.io/participedia/participedia/issues/332110229/

This doesn't appear to be effecting users, just generating spurious messages to devs when the user's login has timed out (not completely spurious, we should handle login timeouts better). Closing for now.


Check forbidden branches in pre-commit hook

Changes
Since I have an admin role, I can push to main without a problem. This is leading me to push without realizing I'm working in the default branch.
I'm including this small pre-commit hook to prevent committing to main branch so I can move changes to the corresponding branch to follow the PR workflow. It can be bypassed with -n option if I need to push directly to main for whatever reason.
Context

There wasn't specific ticket for this.

The proof this works:

I checked out this PR branch locally, and I tried to commit on top of this branch, and Husky complains:
.husky/pre-commit: line 8: .: forbidden-branches.sh: file not found
husky - pre-commit hook exited with code 1 (error)

I'm using Git for Windows, you use ZSH, so maybe there's an incompatibility here?

I checked out this PR branch locally, and I tried to commit on top of this branch, and Husky complains:
.husky/pre-commit: line 8: .: forbidden-branches.sh: file not found
husky - pre-commit hook exited with code 1 (error)

I'm using Git for Windows, you use ZSH, so maybe there's an incompatibility here?

I've moved the script within pre-commit file, can you give it a try again? Thanks for double-checking!

I've moved the script within pre-commit file, can you give it a try again? Thanks for double-checking!

That seems to have done the trick, I can commit on this PR branch now.
Not sure if the script will actually work to prevent me committing to main, I think the only way to find out is to merge this PR... 😄

Not sure if the script will actually work to prevent me committing to main, I think the only way to find out is to merge this PR... 😄

I checked it before pushing, it's still working!

I confirm this works properly for me now as well. I cannot commit to main without Husky warning me about it.


move minting logic out of components

minting logic and associated functions (submitRaffle, allocateRaffle for example) are repeated between Generative, PaidGenerative and holderOfGenerative
extract as much as possible to reusable utils / composable and re use across components

I will be interested in working on this issue once it is ready to be worked on.

I would prefer someone from @kodadot/internal-dev as they're well familiar with codebase

@yangwao  Okay, alright. Thanks for the information.

@daiagi, do you plan to start working on it? As I see it a blocker for

https://github.com/kodadot/nft-gallery/pull/9711

I would like to see it done or someone else can take over

Mmm I don't think it's a blocker
But yes. I'll take care of it

Mmm I don't think it's a blocker

ah I thought this needs to be done against branch
https://github.com/kodadot/nft-gallery/tree/drops-refactor and then needs to be merged main.
If it can be done parallel, there is no rush.

@yangwao
i suggest merging drops-refactor into main after #9709
majority of refactor is done, rest, including this, can continue on main

@daiagi let's finish, it won't go for the upcoming release, so you have a free runway for this

Can close this
@hassnian already extracted logic out in massmint pr and leftovers are handled in #9750


libp2p migration: cannot rename peerstore dir

Describe the bug
Got this error after we migrate libp2p
Mar 11 03:12:26 contabo-20 systemd[1]: Started Lodestar beacon.
Mar 11 03:12:27 contabo-20 beacon_run.sh[2539444]: Debugger listening on ws://<IP_ADDRESS>:9229/4334cbb4-080c-4877-912a-896fb09d0b70
Mar 11 03:12:27 contabo-20 beacon_run.sh[2539444]: For help, see: https://nodejs.org/en/docs/inspector
Mar 11 03:12:29 contabo-20 beacon_run.sh[2539444]: Mar-11 03:12:29.347[]                 info: Lodestar version=v0.36.0/dapplion/network-globals//5d37d6f5 (git), network=prater
Mar 11 03:12:29 contabo-20 beacon_run.sh[2539444]: Mar-11 03:12:29.356[]                 info: Connected to LevelDB database name=/home/devops/beacon/peerstore
Mar 11 03:12:29 contabo-20 beacon_run.sh[2539444]:  ✖ Error: ENOTEMPTY: directory not empty, rename '/home/devops/beacon/peerstore' -> '/home/devops/beacon/peerstore.pre-0.36.0.bak'
Mar 11 03:12:29 contabo-20 beacon_run.sh[2539444]:     at Object.renameSync (node:fs:980:3)
Mar 11 03:12:29 contabo-20 beacon_run.sh[2539444]:     at deleteOldPeerstorePreV036 (/usr/src/lodestar/packages/cli/src/migrations/deleteOldPeerstore.ts:35:8)
Mar 11 03:12:29 contabo-20 beacon_run.sh[2539444]:     at Object.beaconHandler [as handler] (/usr/src/lodestar/packages/cli/src/cmds/beacon/handler.ts:60:3)
Mar 11 03:12:29 contabo-20 systemd[1]: beacon.service: Main process exited, code=exited, status=1/FAILURE
Mar 11 03:12:29 contabo-20 systemd[1]: beacon.service: Failed with result 'exit-code'.
Mar 11 03:12:29 contabo-20 systemd[1]: beacon.service: Scheduled restart job, restart counter is at 79.
Mar 11 03:12:29 contabo-20 systemd[1]: Stopped Lodestar beacon.

I run it in contabo-20

What about attempting to rename, else remove completely and continue?

Should be resolved with #3915


Support to install package which has a directory

Such as tar -C /usr/local -xzf go1.19.linux-amd64.tar.gz
export PATH=$PATH:/usr/local/go/bin


Testdrive fixes

Fixes for cloud test drive:

sudo -related fixes for make all to run on VMs when the user is not part of the docker group
git clone --recursive to clone submodules #32

fixes #32


Fix compiler warnings and add CODEOWNERS

Replace var to <type>, because the former is deprecated.
Add CODEOWNERS

The buildkite failure looks indeed like it is the case -- Badge::meta is not properly deleted so this might be a low-priority security issue.

@sorpaas Looks like (at least at the moment) we're not relying on any meta values anywhere in the code.
This means the in case of "mapping hijacking" (creating the new badge with the same id) the meta from the old record will be inherited by the new one.
This might lead to the "read meta after deletion" bug — but is not a security concern, as long as:

We only add badges to the array with push, thus effectively never reusing the same id twice.
(a little bit more far-fetched) We don't rely on anything in those meta fields in the code, so it's not a subject for "mapping hijacking", when one legitimate cell owner sets the mapping, and then after the deletion of the record the next user is inheriting all the mappings somehow, adding them extended permissions or whatnot.

Still, maybe the "safe-deletion" (with a boolean flag deleted on the record) will be a safer alternative to the delete we have now? Like @andresilva did for SimpleOperations.sol.

Tried a version using the safe deletion method:

Changed all current if return modifier to use require. This is both more clearer for the source code and contract user -- we're already throwing errors for out of bound indexes.
Added deleted flag in Badge, and change badgeCount to be a variable to store the actual count.

I'll write a test spec for this.


could not invoke backgroundgeolocation.start

I faced this issue when I import the background geolocation library and RN Google Maps library. Both work fine separtely
here is my build.gradle
`// Top-level build file where you can add configuration options common to all sub-projects/modules.
buildscript {
ext {
buildToolsVersion = "30.0.2"
minSdkVersion = 21
compileSdkVersion = 30
targetSdkVersion = 30
ndkVersion = "21.4.7075529"
googlePlayServicesVersion = "11.8.0"
gradle3EXPERIMENTAL = "yes"
}
repositories {
google()
mavenCentral()
}
dependencies {
classpath("com.android.tools.build:gradle:4.2.2")
// NOTE: Do not place your application dependencies here; they belong
// in the individual module build.gradle files
}
}
allprojects {
repositories {
mavenCentral()
mavenLocal()
maven {
// All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
url("$rootDir/../node_modules/react-native/android")
}
maven {
// Android JSC is installed from npm
url("$rootDir/../node_modules/jsc-android/dist")
}
    google()
    maven { url 'https://maven.google.com' }
    maven { url 'https://jcenter.bintray.com' }
    maven { url 'https://www.jitpack.io' }
}

}
AndroidManifest.xml
<uses-permission android:name="android.permission.INTERNET" />

<application
  android:usesCleartextTraffic="true"
  android:name=".MainApplication"
  android:label="@string/app_name"
  android:icon="@mipmap/ic_launcher"
  android:roundIcon="@mipmap/ic_launcher_round"
  android:allowBackup="false"
  android:theme="@style/AppTheme">
  <uses-library android:name="org.apache.http.legacy" android:required="false"/>
  <activity
    android:name=".MainActivity"
    android:label="@string/app_name"
    android:configChanges="keyboard|keyboardHidden|orientation|screenSize|uiMode"
    android:launchMode="singleTask"
    android:windowSoftInputMode="adjustResize">
    <intent-filter>
        <action android:name="android.intent.action.MAIN" />
        <category android:name="android.intent.category.LAUNCHER" />
    </intent-filter>
  </activity>
</application>

`
app/build.gradle
`apply plugin: "com.android.application"
import com.android.build.OutputFile
/**

The react.gradle file registers a task for each build variant (e.g. bundleDebugJsAndAssets
and bundleReleaseJsAndAssets).
These basically call react-native bundle with the correct arguments during the Android build
cycle. By default, bundleDebugJsAndAssets is skipped, as in debug/dev mode we prefer to load the
bundle directly from the development server. Below you can see all the possible configurations
and their defaults. If you decide to add a configuration block, make sure to add it before the
apply from: "../../node_modules/react-native/react.gradle" line.

project.ext.react = [
// the name of the generated asset file containing your JS bundle
bundleAssetName: "index.android.bundle",

// the entry file for bundle generation. If none specified and
// "index.android.js" exists, it will be used. Otherwise "index.js" is
// default. Can be overridden with ENTRY_FILE environment variable.
entryFile: "index.android.js",

// https://reactnative.dev/docs/performance#enable-the-ram-format
bundleCommand: "ram-bundle",

// whether to bundle JS and assets in debug mode
bundleInDebug: false,

// whether to bundle JS and assets in release mode
bundleInRelease: true,

// whether to bundle JS and assets in another build variant (if configured).
// See http://tools.android.com/tech-docs/new-build-system/user-guide#TOC-Build-Variants
// The configuration property can be in the following formats
//         'bundleIn${productFlavor}${buildType}'
//         'bundleIn${buildType}'
// bundleInFreeDebug: true,
// bundleInPaidRelease: true,
// bundleInBeta: true,

// whether to disable dev mode in custom build variants (by default only disabled in release)
// for example: to disable dev mode in the staging build type (if configured)
devDisabledInStaging: true,
// The configuration property can be in the following formats
//         'devDisabledIn${productFlavor}${buildType}'
//         'devDisabledIn${buildType}'

// the root of your project, i.e. where "package.json" lives
root: "../../",

// where to put the JS bundle asset in debug mode
jsBundleDirDebug: "$buildDir/intermediates/assets/debug",

// where to put the JS bundle asset in release mode
jsBundleDirRelease: "$buildDir/intermediates/assets/release",

// where to put drawable resources / React Native assets, e.g. the ones you use via
// require('./image.png')), in debug mode
resourcesDirDebug: "$buildDir/intermediates/res/merged/debug",

// where to put drawable resources / React Native assets, e.g. the ones you use via
// require('./image.png')), in release mode
resourcesDirRelease: "$buildDir/intermediates/res/merged/release",

// by default the gradle tasks are skipped if none of the JS files or assets change; this means
// that we don't look at files in android/ or ios/ to determine whether the tasks are up to
// date; if you have any other folders that you want to ignore for performance reasons (gradle
// indexes the entire tree), add them here. Alternatively, if you have JS files in android/
// for example, you might want to remove it from here.
inputExcludes: ["android/", "ios/"],

// override which node gets called and with what additional arguments
nodeExecutableAndArgs: ["node"],

// supply additional arguments to the packager
extraPackagerArgs: []
]
*/

project.ext.react = [
enableHermes: false,  // clean and rebuild if changing
]
apply from: "../../node_modules/react-native/react.gradle"
/**

Set this to true to create two separate APKs instead of one:

An APK that only works on ARM devices

An APK that only works on x86 devices

The advantage is the size of the APK is reduced by about 4MB.
Upload all the APKs to the Play Store and people will download
the correct one based on the CPU architecture of their device.
*/
def enableSeparateBuildPerCPUArchitecture = false

/**

Run Proguard to shrink the Java bytecode in release builds.
*/
def enableProguardInReleaseBuilds = false

/**

The preferred build flavor of JavaScriptCore.

For example, to use the international variant, you can use:
def jscFlavor = 'org.webkit:android-jsc-intl:+'

The international variant includes ICU i18n library and necessary data
allowing to use e.g. Date.toLocaleString and String.localeCompare that
give correct results when using with locales other than en-US.  Note that
this variant is about 6MiB larger per architecture than default.
*/
def jscFlavor = 'org.webkit:android-jsc:+'

/**

Whether to enable the Hermes VM.

This should be set on project.ext.react and mirrored here.  If it is not set
on project.ext.react, JavaScript will not be compiled to Hermes Bytecode
and the benefits of using Hermes will therefore be sharply reduced.
*/
def enableHermes = project.ext.react.get("enableHermes", false);

/**

Architectures to build native code for in debug.
*/
def nativeArchitectures = project.getProperties().get("reactNativeDebugArchitectures")

android {
ndkVersion rootProject.ext.ndkVersion
compileSdkVersion rootProject.ext.compileSdkVersion

defaultConfig {
    applicationId "com.testing"
    minSdkVersion rootProject.ext.minSdkVersion
    targetSdkVersion rootProject.ext.targetSdkVersion
    versionCode 1
    versionName "1.0"
}
splits {
    abi {
        reset()
        enable enableSeparateBuildPerCPUArchitecture
        universalApk false  // If true, also generate a universal APK
        include "armeabi-v7a", "x86", "arm64-v8a", "x86_64"
    }
}
signingConfigs {
    debug {
        storeFile file('debug.keystore')
        storePassword 'android'
        keyAlias 'androiddebugkey'
        keyPassword 'android'
    }
}
buildTypes {
    debug {
        signingConfig signingConfigs.debug
        if (nativeArchitectures) {
            ndk {
                abiFilters nativeArchitectures.split(',')
            }
        }
    }
    release {
        // Caution! In production, you need to generate your own keystore file.
        // see https://reactnative.dev/docs/signed-apk-android.
        signingConfig signingConfigs.debug
        minifyEnabled enableProguardInReleaseBuilds
        proguardFiles getDefaultProguardFile("proguard-android.txt"), "proguard-rules.pro"
    }
}

// applicationVariants are e.g. debug, release
applicationVariants.all { variant ->
    variant.outputs.each { output ->
        // For each separate APK per architecture, set a unique version code as described here:
        // https://developer.android.com/studio/build/configure-apk-splits.html
        // Example: versionCode 1 will generate 1001 for armeabi-v7a, 1002 for x86, etc.
        def versionCodes = ["armeabi-v7a": 1, "x86": 2, "arm64-v8a": 3, "x86_64": 4]
        def abi = output.getFilter(OutputFile.ABI)
        if (abi != null) {  // null for the universal-debug, universal-release variants
            output.versionCodeOverride =
                    defaultConfig.versionCode * 1000 + versionCodes.get(abi)
        }

    }
}

}
dependencies {
implementation fileTree(dir: "libs", include: ["*.jar"])
//noinspection GradleDynamicVersion
implementation "com.facebook.react:react-native:+"  // From node_modules
implementation "androidx.swiperefreshlayout:swiperefreshlayout:1.0.0"

debugImplementation("com.facebook.flipper:flipper:${FLIPPER_VERSION}") {
    exclude group:'com.facebook.fbjni'
}
compile project(':@mauron85_react-native-background-geolocation')

debugImplementation("com.facebook.flipper:flipper-network-plugin:${FLIPPER_VERSION}") {
    exclude group:'com.facebook.flipper'
    exclude group:'com.squareup.okhttp3', module:'okhttp'
}

debugImplementation("com.facebook.flipper:flipper-fresco-plugin:${FLIPPER_VERSION}") {
    exclude group:'com.facebook.flipper'
}

if (enableHermes) {
    def hermesPath = "../../node_modules/hermes-engine/android/";
    debugImplementation files(hermesPath + "hermes-debug.aar")
    releaseImplementation files(hermesPath + "hermes-release.aar")
} else {
    implementation jscFlavor
}

}
// Run this once to be able to run the application with BUCK
// puts all compile dependencies into folder libs for BUCK to use
task copyDownloadableDepsToLibs(type: Copy) {
from configurations.implementation
into 'libs'
}
apply from: file("../../node_modules/@react-native-community/cli-platform-android/native_modules.gradle"); applyNativeModulesAppBuildGradle(project)
`

@mauron85 kindly help me

Did you find any solution?

I'm having the same issue.
Any solution?

Follow this article
https://github.com/doublesymmetry/react-native-track-player/issues/1132


Add flex to make checkbox inline with label

Fixes #211

Do we want to add padding, or flip the ordering of checkboxes, or leave as-is?

@pierreTklein I think padding is the best look

@pierreTklein done


Examples aren't working - API rate limit exceeded in CodeSandbox

Describe the bug (required)
CodeSandbox is showing this on every example:
API rate limit exceeded for app ID c07a89833b557afc7be2.
Provide an example via Codesandbox! (required)
https://codesandbox.io/s/github/tannerlinsley/react-table/tree/master/examples/column-resizing
Steps To Reproduce (required)
Open the link above
Expected behavior (Recommended)
CodeSandbox examples should work :)
Screenshots

It is working again! Thanks :)

It is working again! Thanks :)


add de translations for email verification

WHY
Due to Backpack 6 supporting email verification out of the box I discovered that there are translations missing for the German language.
BEFORE - What was wrong? What was happening before this PR?
If the language is German you'll get the English texts related to email verification.
AFTER - What is happening after this PR?
If the language is German you'll get the German texts related to email verification.
HOW
How did you achieve that, in technical terms?
I translated the corresponding language keys.
Is it a breaking change?
No
How can we test the before & after?
Configure "de" as language and enable email verification. Then register a new user and check the texts on the verify view.

Thanks @hagealex 🙏


Use archiver.run function in runner

Description
archiver.archive() isn't actually a function, we use run() instead which was implemented here: https://github.com/cmu-delphi/covidcast-indicators/commit/ead53c8af78c833e578967a1a11c53f87d55cfaf#diff-817ffd086856eae7a95f784aee9b38cb94e4ef48e9d9307e5dc6930f93729225
Also fixed tests
Changelog
Itemize code/test/documentation changes and files added/removed.

runner.py
archiver.py

Fixes

changed all calls of archiver.archive() to archiver.run()

Additional context for future travelers: this means we've been running quidel_covidtest and google_symptoms all this time without any archive diffing

Also worth noting - at this point in time there aren't archive params in ansible for quidel_covidtest or google_symptoms, so there hasn't been archiver running anyway - see this.


CausalBegan demo for README

I failed to use the dataset of download link in download.py, could you provide a new download link of your dataset, which is trained by your code.

Hello, expert, may I ask you some questions about the causalgan code


changelog for KaTeX - complete w/ links

@Kully thanks for the PR.

@Kully thanks for the PR.

 My pleasure. 🎉

Policy question: do we want to update CHANGELOG (for "master" version, say) as we add features going forward, PR by PR?  (Perhaps we should open a new discussion thread.)

Let's discuss in a new thread.


Create Api Specification

Users
/Users
/Users/:id
/Users/:id/Datasets
/Users/:id/Datasets/:id/Statistics
/Users/:id/Datasets/:id/rows
/Users/:id/Datasets/:id/rows/:id/Statistics
/Users/:id/Datasets/:id/columns
/Users/:id/Datasets/:id/columns/:id/Statistics
/Users/:id/Tranformations
/Users/:id/Tranformations/:id
/Users/:id/Macros
/Users/:id/Macros/:id
DataSets
/DataSets
/DataSets/:id
/Datasets/:id/rows
/Datasets/:id/rows/:id
/Datasets/:id/columns
/Datasets/:id/columns/:id
Macros
/Macros
/Macros/:id
Transformations
/Tranformations
/Tranformations/:id

Users

/Users

GET - N/A (List all Users - If auth).
POST - Create User.
PUT - N/A
DELETE - N/A

/Users/:id

GET - Get Info for user with the id.
POST - N/A
PUT - Update User info.
DELETE - N/A

/Users/:id/Datasets (Doubt - ASK PAG)

GET - Get User DataSets
POST - Create DataSet
PUT - N/A
DELETE -  N/A

/Users/:id/Transformations  (Doubt - ASK PAG)

GET - List all user transformations
POST - Create a new transformation
PUT -  N/A
DELETE -  N/A

DataSets

/DataSets

GET - Lists all user datasets
POST - Creates new dataset
PUT - N/A
DELETE -  N/A

/DataSets/:id

GET - Get DataSet data
POST - Update DataSet
PUT - Overwrite DataSet
DELETE -  Delete DataSet

/Datasets/:id/Statistics (NEW)
Missing(Define filters)

GET - Get Statistic from DataSet from entire set
POST - N/A
PUT - N/A
DELETE - N/A

/Datasets/:id/rows

GET - N/A
POST - Creates new row (when creating rows, adds after the end of dataset)
PUT - N/A
DELETE - N/A

/Datasets/:id/rows/:n

GET - Gets Row Data
POST - Update  Row
PUT - Create or overwrite  Row
DELETE - Delete  Row

/Datasets/:id/rows/:n/sum

GET - Gets Row Sum
POST - N/A
PUT - N/A
DELETE - N/A

/Datasets/:id/rows/:n/count

GET - Gets Number of Elements of Row
POST - N/A
PUT - N/A
DELETE - N/A

/Datasets/:id/rows/:n/Statistics (NEW)
Missing(Define filters)

GET - Gets statistics from entire row
POST - N/A
PUT - Update Row
DELETE - Delete Row

/Datasets/:id/columns

GET -  N/A
POST - Create column (when creating columns, adds after the end of dataset)
PUT -  N/A
DELETE - N/A

/Datasets/:id/columns/:n

GET - Get column
POST - Update column
PUT - Create or overwrite column
DELETE - Delete column

/Datasets/:id/columns/:n/sum

GET - Gets Column Sum
POST - N/A
PUT - N/A
DELETE - N/A

/Datasets/:id/columns/:id/count

GET - Gets Number of Elements of Column
POST - N/A
PUT - N/A
DELETE - N/A

/Datasets/:id/column/:n/Statistics (NEW)
(Missing filters)

GET - Gets statistics from entire column
POST - N/A
PUT - N/A
DELETE - N/A

Transformations
/Transformations

GET - List all user transformations
POST - Create a new transformation
PUT -  N/A
DELETE -  N/A

/Transformations/:id

GET - List transformation
POST - N/A
PUT -  N/A
DELETE -  Delete Transformation

Macros

/Macros

GET - get all
POST - create
PUT - N/A
DELETE - N/A

/Macros/:id

GET - get macro by id
POST - update
PUT - override
DELETE -  delete

/Macros/:id/run

GET - N/A
POST - exec
PUT - N/A
DELETE -  N/A


Pipeline step for agentless node provisioning

this PR adds a openstack step, callable from within a pipeline.
With this syntax:
def IP = openstack serverName:"OpenStackDev", template:"CentOS7"
A node will be provisioned at the cloud named OpenStackDev, from the already defined template CentOS7. The step would return the node IP. The provisioned node won't be included as a Jenkins agent (and so, the JCloudsCleanupThread wont get rid of it)
The snipplet generator allows to select both parameters from the globally defined ones.
As a hacky solution, an empty/unlimited Scope is allowed so that JCloudsCleanupThread doesn't delete the provisioned node

What is the motivation for this change? What use case does it solve? I do not really like this opts out of cleanup...

The idea is to provision a node at a openstack cluster, allowing it to be alive out of the scope of a Jenkins job (and of course, being invoked from a pipeline)
A use case is: prepare a host so that a continuous deployment of a product is performed on a clean node. The job responsible of the new installation should boot a node and that node should be left powered on after the job completes.

That is a bit odd use-case, if you ask me :). But let's look at this...

The tools provided by the plugin should be safe by default with, if that is necessary, opt-in to unsafe. So the step should be explicitly told not to clean it up once the build is done.
The word openstack is a bit too general and does not suggests it booths the instance. I am not sure we want to follow the docker pattern with docker.boot or just rename the step to openstack.boot.
From long term perspective it would be better to return custom wrapper of (openstack4j) Server instance as IP might not be enough for everybody and we might need to extend that in the future.

Let me answer in the same order:

If anyone wishes to use a scoped node, a node(){} dsl is available for pipeline projects, where the scope would follow the already existant default configuration. This step would add the "not to clean it up once the build is done"
True that, its way too general. I didnt follow a compound pattern (plugin.action) as no action besides boot comes to my mind. Maybe a openstackBoot step will suffice?
Returning a Server object might be feasible and as its a Serializable class .However, as the booted node configuration is the one already present at the template settings, I believe the only interesting return types are IP addresses. Another point is that returning a complex object will require to handle whitelisting tons of internal methods. What about returning a Map <String, String>, with values such as ['floating':'<IP_ADDRESS>', 'fixed':'<IP_ADDRESS>']? This way, the map could be extended with further elements

If anyone wishes to use a scoped node, a node(){} dsl is available for pipeline projects, where the scope would follow the already existant default configuration. This step would add the "not to clean it up once the build is done"

There are two separate aspects, connecting the machine as a node and cleaning up the resource (whether it is a node or a machine). JCloudsBuildWrapper does the machine provisioning and clean up the resources. I would really like this to be the default here.

Maybe a openstackBoot step will suffice?

Sure

Returning a Server object might be feasible and as its a Serializable class .However, as the booted node configuration is the one already present at the template settings, I believe the only interesting return types are IP addresses. Another point is that returning a complex object will require to handle whitelisting tons of internal methods. What about returning a Map <String, String>, with values such as ['floating':'<IP_ADDRESS>', 'fixed':'<IP_ADDRESS>']? This way, the map could be extended with further elements

I would prefer not to expose 3rd-party classes this way as we would not be able to change the implementation in the future and we can not add custom methods for pipeline (extracting the FIPs is not trivial). Also, it likely holds lot more data to serialize then what we need to expose. People might as well be interested in instance ID so they can delete it themselves, etc. I really believe that final POJO with handful of data is optimal. This will enable to implement actions on such object (destroy, snapshot, etc.).

This last commits offer a docker-alike functionality (most code has been copied), allowing a global openstack object creation, with a Node element as a org.openstack4j.model.compute.Server one (might rename it) and adding some basic utility methods. The Node element is meant just to avoid imports at pipelines.
@olivergondza, if you like this approach, I will add a complete help.jelly for the OpenStackDSL

@olivergondza the help.jelly is complete as of the current implementation and OpenStack.groovy could be further improved to adress new functionality.
The only hiccup is the findbugs warning detected at JCloudsSlaveTemplate:309. Im unable to reproduce locally (on OpenJDK 8) and doesnt seem obvious to find where the null value is being detected. As it is a method used only for testing, I shamefully resorted to SuppressWarnings

Please propose unit-tests for the new functionality.

As most of the new functionality is on the Openstack.groovy pipeline script, the only testeable code is the nested class and the ServerScope parse method. Added those.

most of the new functionality is on the Openstack.groovy pipeline script

That is the main meat to test actually. Running a pipeline job executing the booting and destroying. The plugin has the openstack subbing figured out so it will provide you with local agents instead of openstack ones - see existing tests for inspiration.

Added tests for the pipeline dsl implementation. Findbugs still throws errors on an unmodified class

@olivergondza, WDYT about the last changes? I believe every comment has been addressed

Thanks, it looks good to me in general.

Will try to find spare time to reimplement as a Step

Third attemp:

As per @jglick suggestion, implemented a simple Step returning a POJO wrapping the Server object (and whitelisting some useful methods).
As per @olivergondza, the default provision method (withot definind a scope) will be current run scoped.

There is an ignored test as some Mockito internal classes are not Serializable (and the point of that test is to ckeck the SimplifiedServer is). Manual runs of that test after hpi:run throws no errors.

@jglick, @olivergondza the funcionality has been implemented as an Step. Could you review it? Furthermore, the ignored test went off


[PMEM-SPILL-34][POAE7-1119]Port RDD cache to Spark 3.1.1 as separate module

What changes were proposed in this pull request?
Port RDD cache to Spark 3.1.1 as separate module
How was this patch tested?
Not tested.

@winningsix @yeyuqiang Please take a review, thanks.


Contact between identical agentid?

Hey covasim team, thanks again for opensourcing this simulator. I'm asking this question on the public github so other people get to benefit as well from your answers. Best regards, Rob
Describe the bug
Some layers seem to define contacts between two identical agents. For example, contacts[layer]['p1'][i] == contacts[layer]['p2'][i] for some i. Is this expected or is this a bug?
To reproduce

Run a simulation like in the tutorial5.ipynb
Use as subtarget function for test_num the following:

  def subtarget_func(sim):
    contacts = sim.people.contacts

    for layerkey in contacts.keys():
      num_selfcontact = np.sum(
        contacts[layerkey]['p1'] == contacts[layerkey]['p2'])
      if num_selfcontact > 0:
        print(f"Found {num_selfcontact} contacts in layer {layerkey}")

    # Baseline intervention
    vals = np.ones(len(sim.people))  # Create the array
    vals[cv.true(sim.people.exposed)] = 100  # Probability for testing
    output = dict(inds=sim.people.uid, vals=vals)
    return output

Expected behavior
I would expect that a simulator wouldn't simulate a contact between the identical agent, unless this contact has a special meaning?
Screenshots or outputs
The above subtarget function will print an output like the following. The num_selfcontact differs randomly between runs, but is usually around 5 to 50.
All layers: ['h', 's', 'w', 'c']
Found 8 contacts in layer s
Found 13 contacts in layer w
Found 7 contacts in layer c

Platform (please complete the following information):

Covasim version: 3.1.4
Python 3.9 on Ubuntu 22

Hi @RobRomijnders -- well spotted, you're right! In Covasim's default "hybrid" network, we don't bother to remove self-connections, because they have (virtually) no effect on the network, and are computationally costly to find and remove. However, other network options -- such as SynthPops -- do have self-connections removed. We'll add this to the FAQ, because you probably aren't the only person who was confused by this~

Ok, good to know. Thanks!


Links should not have unclear purpose when containing an img with alt text

The following element should not fail on linkWithUnclearPurpose since alternative text is present.

I think this is a dupe of #156


Some thoughts on the structure of file system. And 1 Question for location of conf file

I'm trying various tools. In Windows 10, cmd and gitBash. slightly difference.

Also, the activate on Windows is in Scripts dir, on Mac, its in bin. Is this the intentional design? I think they should be merged in 1 dir, like bin.

I sometimes change between Windows and Mac and use the same set of config. I have to do it manually. Copying from 1 computer to another because I cannot remember what changed.
I tried to specify the location of the config file, but didn't succeed on both machines. What did I do wrong?

config="../../../Google\ Drive/Game/config.json"
on Mac doesn't work

I got this in the log
[IncubateEggs] [INFO] [next_egg_incubates] Eggs incubating: [...
Where can I turn this info off? Or set the intervals when it can be displayed?

Well, this is an important one. I like to watch the behavior of the bot and change the conf often.
If I restart the bot after changing. How will the daily limit be calculated?
Is it stored in a database somewhere or is it written to a file or is it rest every time I restart the bot?
If the last case, then it is not good.

@avexus what do you want done with this? is it a FR or a bug?


darwin.iproute2mac: init at 1.2.1

Motivation for this change
This adds iproute2mac, a CLI wrapper for basic network utilites on Mac OS X inspired with iproute2 on Linux.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

cc @LnL7

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: darwin.iproute2mac
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-linux (full log)
The following builds were skipped because they don't evaluate on x86_64-linux: darwin.iproute2mac
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-darwin (full log)
Attempted: darwin.iproute2mac
Partial log (click to expand)
configuring
no configure script, doing nothing
building
no Makefile, doing nothing
installing
post-installation fixup
strip is /nix/store/yyak5sjv68n9vdgnkd2cb5djk1l9sqj5-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/60jvgba0j6x9hq2q3m8q30c407v87jzd-iproute2mac-1.2.1/bin
patching script interpreter paths in /nix/store/60jvgba0j6x9hq2q3m8q30c407v87jzd-iproute2mac-1.2.1
/nix/store/60jvgba0j6x9hq2q3m8q30c407v87jzd-iproute2mac-1.2.1

Thanks!


How to expand a spiderfied a cluster markers group.

Hello,
I'm building a website where I need to expand a spiderified a cluster markers group on a specific action from the user. I understand that the cluster group will be expanded when the user clicks on it.
But I couldn't perform that programmatically.

Any ideas or advice?

The method is myMarkerCluster.spiderfy(), but first you need to get the cluster. (Will add this to the docs now...)
If you listen to the clusterclick event you get passed the cluster, or you can use getVisibleParent
https://github.com/Leaflet/Leaflet.markercluster#getting-the-visible-parent-of-a-marker


This extension is not mentioned in the official docs.

Is this the official vscode meson extension?
If so, it's not listed in the official documentation.
https://mesonbuild.com/IDE-integration.html#existing-integrations

Maybe with a PR, not a report ;)
And yes, that's on the documentation side. I have larger pains with this exension not being the top one in the store. But there's not much we can do against that outside gaining more users.


Include AML version as macro and runtime variable

In GitLab by @perarnau on Mar 20, 2019, 09:59

Add #define AML_VERSION 0.1.0 in aml.h
Add extern aml_version in aml.h
Define aml version in configure.ac
Link .o conatining aml_version definition according to configure.ac version.
Check version at runtime: AML_VERSION == aml_version.
If False, header may not be the same and code may fail awkwardly at runtime.

In GitLab by @perarnau on Mar 21, 2019, 09:53
I am taking the issue. Creating a branch aml-vresion rn.
By Nicolas Denoyelle on 2019-03-21T14:53:46 (imported from GitLab project)

In GitLab by @perarnau on Mar 21, 2019, 08:55
changed the description
By Swann Perarnau on 2019-03-21T13:55:15 (imported from GitLab project)

In GitLab by @perarnau on Mar 21, 2019, 09:53
assigned to @NicolasDenoyelle
By Nicolas Denoyelle on 2019-03-21T14:53:15 (imported from GitLab project)

In GitLab by @perarnau on Mar 22, 2019, 16:17
mentioned in merge request !36
By Swann Perarnau on 2019-03-22T21:17:43 (imported from GitLab project)

In GitLab by @perarnau on Mar 22, 2019, 16:18
closed
By Swann Perarnau on 2019-03-22T21:18:36 (imported from GitLab project)

In GitLab by @perarnau on Mar 21, 2019, 08:56
Let's try to aim for something close to semver.
In particular, I would split the version info into MAJOR/MINOR/PATCH and provide both a full string for the version and independent variables.
By Swann Perarnau on 2019-03-21T13:56:40 (imported from GitLab project)


Add my GPG key to the list of Gitian signing keys.

I added my key at the top of the list, where I think we should put Namecoin developers' keys.  I left the Bitcoin developers in there for now (see also https://github.com/namecoin/namecoin-core/issues/81).

Wrong repository.


Change the unreacheable return value of shape_find_owner()

The shape owner id can take the value of 0, which should be an illegal value, the fixes mean either
-change the indexes of both shapes and shapeOwners (for consistencies sake ,a previous PR) or
-the illegal value,
I opted to change the illegal value.

Hope i did it right

@Kemikal1 No worries, I know certain commands in git can be difficult to handle when you're not used to them.
Now it looks like you're back with 4 commits. You can just run git rebase -i upstream/master again, and in the selection list, set the proper title to the first commit in the list, then set the 3 other ones to fixup and it should do the trick.

This can't be cherry-picked for 3.x, because one of the file has a different name. So @Kemikal1 if you're up for it, feel free to open a new PR with the same changes on the 3.x branch.

It's ok, I can figure out the file renaming while cherrypicking in this case (usually, git can figure it out itself if the code is similar).


Adding gmcl to documentation index for distro

I'd like gmcl to be indexed and documented.

Sorry, now it's failing due to a know issue with our CI checker. We'll have to merge master into this pull request to fix it.

When I merged this PR overriding the tests yesterday I did not realize that the git diff error was masking a second one and since merging CI has been failing due to the trailing whitespace after status: developed
 E...--- file:///home/runner/work/rosdistro/rosdistro/noetic/distribution.yaml
+++ loaded-and-saved
@@ -2470,3 +2470,3 @@
       version: master
-    status: developed
+    status: developed

@nuclearsandwich . I didn't know whitespaces could make such an error, sorry for that and thanks for your help. should I do any further steps ?

I didn't know whitespaces could make such an error, sorry for that and thanks for your help. should I do any further steps ?

Nope! It's taken care of and hopefully #30011 will surface yaml formatting issues faster and with less noise in the future.


Allow using both define and module.exports.

If a script is loaded and uses require.js, module.exports will not be set. This means another script which uses commonjs will not be able to require punycode. This fixes that issue, allowing for both define + module.exports definitions.

tests/index.html tests require.js by default, and it works for me.

I'm not sure how that is related. I described the situation in which this doesn't work above. Also, you can easily see the scenario where it would not setup module.exports in the code.

If a script is loaded and uses require.js, module.exports will not be set. This means another script which uses commonjs will not be able to require punycode.

The tests don't fall under this scenario.

if ( 		typeof define == 'function' && 		typeof define.amd == 'object' && 		define.amd 	) { 		define('punycode', function() { 			return punycode; 		}); 	} else if (freeExports && freeModule) { 		if (module.exports == freeExports) { 			// in Node.js, io.js, or RingoJS v0.8.0+ 			freeModule.exports = punycode; 		} else { 			// in Narwhal or RingoJS v0.7.0- 			for (key in punycode) { 				punycode.hasOwnProperty(key) && (freeExports[key] = punycode[key]); 			} 		} 	} else { 		// in Rhino or a web browser 		root.punycode = punycode; 	}
The issue is the "else if". If the check for amd passes, module.exports will not be set.

cc @jdalton

@ganemone Can you clarify the scenario (baby step me through it with first, second, third bits).

I'm not sure when this will be an issue. Is it for code that's bundled with browserify/webpack and then loaded as an AMD module?

So the current issue is this:
// This script sets up amd
<script src="code/using/require.js"></script>
// This script tries to require('punycode')
<script src="code/using/commonjs"></script>

The second script is browserified. Since the amd check passes, module.exports will not be set. I'm realizing now however that maybe the solution is to use a browserify transform such as https://www.npmjs.com/package/amdclean.

I'm realizing now however that maybe the solution is to use a browserify transform such as https://www.npmjs.com/package/amdclean.

Ah nice!

I donno if this would get around this but in other projectsI export the lib to the global if run in the browser. Regardless and in addition to other AMD/CommonJS exports that may be in play.

Yea actually that is not what I thought it was at first glance, haha. But I basically need a transform that removes the amd check. There is probably one out there. Closing for now

Cool. Your scenario is an interesting one. I dig all the mix-and-mashups of it all.

So turns out it doesn't look like there is an existing transform that removes the amd check. I'm wondering, could we simply change the else if into an if when checking for module.exports? Or if that will screw up the amd check, could we add a separate check at the end that simply checks for module.exports?

Punycode.js is now Node.js v6+ only.


Separate dtables out into a Library component

Closes #140.
Introduces a Library that contains information about all dispatch tables (builtin and user-defined) as well as builtin traits.

Actions seems to be stuck? I'll close and reopen.


amfora: remove unnecessary postInstall phase for darwin

Motivation for this change
I forgot a part of https://github.com/NixOS/nixpkgs/pull/107884#issuecomment-758553109 before it was merged
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@sikmir sorry about that

@sikmir sorry about that

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109142 run on x86_64-darwin 1

1 package built:

amfora

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109142 run on x86_64-darwin 1

1 package built:

amfora


Carthage and OS X 10.10

I installed 0.9.2 via Carthage, get the following error:
.../MainWindowController.swift:11:8: Module file's minimum deployment target is OS X v10.11: .../Carthage/Build/Mac/SQLite.framework/Modules/SQLite.swiftmodule/x86_64.swiftmodule

Which means that it cannot run on OS X older than 10.11. I suggest to also support 10.10 (and probably 10.9).

Thanks Tyler. I'll take a look. I know we have an explicit OS X setting for CocoaPods but I forgot to double check that it was the same in the Xcode scheme used by Carthage. I'll check it out later today with a bit of luck. Or PRs are always welcome!

On 21 Jan 2016, at 5:21 AM, Tyler Long<EMAIL_ADDRESS>wrote:
I installed 0.9.2 via Carthage, get the following error:
.../MainWindowController.swift:11:8: Module file's minimum deployment target is OS X v10.11: .../Carthage/Build/Mac/SQLite.framework/Modules/SQLite.swiftmodule/x86_64.swiftmodule
Which means that it cannot run on OS X older than 10.11. I suggest to also support 10.10 (and probably 10.9).
—
Reply to this email directly or view it on GitHub.

Hi @tylerlong - I've created PR #336 and you should be able to test by changing your Cartfile line to be:
github "stephencelis/SQLite.swift" "36f311b"
If you have a chance, it would be nice to confirm before merging, thanks!

github "stephencelis/SQLite.swift" "36f311b"
I just tested and it works! I think it is safe to merge now.


[Feature] Fluentd Logging Sidecar

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Ray head and worker pod logs are not very useful. The application specific log is either streamed to drivers (which are ephemeral) or buried in /tmp/ray/session_latest/logs directory in each pod.
In production deployment, the directory content in the logs directory is typically scrapped by log agent and send to a log aggregation service. For example, we have recommended loki for Ray Serve user. However, this approach require specific setup for logging stacks.
Typically production K8s cluster already have pod log aggregation setup for all pods. This means, if there's a way to print out the logs to stdout of a pod, then the log will be automatically picked up and shipped to whatever logging solution K8s cluster adopted.
One approach is to just tail -f /tmp/ray/session_latest/logs directory as the pod cmd instead of sleep. This might work but lost all the metadata.
I want to propose an alternative to add a new sidecar container to each head and worker pod. The container will

run fluentd agent
read the logs directory (via volume mount)
read a config map
print the log and annotate it with file path, output jsonl by default

In this way, the fluentd container will be able to read and annotate logs properly, and the configured logging agent (Logstash, Loki, Splunk) can pick it up from pod stdout.
Please let me know what do you think! @DmitriGekhtman @Jeffwan
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Thanks for opening this issue! (Logging is one of many todo items swimming around in the back of my head :) )
Ray supports redirecting logs to stderr with a component prefix (at least in recent Ray versions).
cc @clarkzinzow @rkooo567 who are most familiar with this functionality
Could that remove the need for fluentd to scrape from the log files?

No I don't think the RAY_log_to_stderr flag is enough unfortunately. It doesn't work with ray start --head for example.

(Is it a fixable issue that it doesn't work with "ray start --head"?)
Ok, then using fluentd sounds reasonable.
My question then is if the operator should automatically insert the fluentd sidecar and logging volume mount.
Or do we document that setup and leave it to users? (We could update all of the example configs to demonstrate this setup.)
Or (intermediate option) do we add a toggle that allows users to enable/disable the logging side cars. (This is effectively a feature flag.)
For automatic configuration options -- I've seen that some users cannot pull directly from dockerhub, so making the sidecar image configurable would be important.

Based on the description, I feel like this is something we should support kuberay. However, I also wonder how other systems handle this (it seems to be a pretty common problem in all systems iiuc)?

Ray is special here because we run multiple processes in the same container and don't output logs to pod stdout. Other kubernetes native system runs one process per pod and log to pod output; which is automatically aggregated.

In Bytedance, we mount ray logs folder /tmp/ray to a fixed hostPath``/opt/tiger/containers and we do configure filebeat rules to collect all Ray logs. In this case, our log daemon is at the host level.
Using streaming sidecar container also works, I think we need to make it optional and pluaggable and do not fail if external logging system is not there.

Sounds good, definitely we're narrowing it to

document the setup and leave it to users

or

add a toggle that allows users to enable/disable automatic configuration of logging side cars.

at Microsoft we use Azure Kubernetes Service, with enabled AKS container log collection to Log Analytics through Azure Monitor for telemetry and logs.
Using FluentD is a great idea for users that are not already leveraging a cloud solution to handle logs. However it seems to be outside the scope of KubeRay, and more of a best practice design pattern when using KubeRay.
@DmitriGekhtman it seems we have a few design patterns of users using KubeRay, does it make sense to have a directory in the repo that we reserve for documenting best practices?

@akanso that is a great idea.
In general, detailed documentation with lots of examples is what we want!
I'll spend some time in the coming weeks fleshing out the docs.

:+1: for this.
@simon-mo

One approach is to just tail -f /tmp/ray/session_latest/logs directory as the pod cmd instead of sleep. This might work but lost all the metadata.

This doesn't work. tail and similar off-the-shelf programs have limitations, they don't detect new files in the path, like promtail does.
:question: Can this sidecar container already be added using the RayClusters spec? At the very least, it seems like it is possible to create a new worker type and hack the helm spec to simulate the sidecar pattern with Pod affinity rules (to the head).
Since this is exact effort is on my team's plate as well, what is the timeframe we are looking at for this feature?

It can already be added using the RayCluster spec.
I'm working on a PR which documents how to do this
https://github.com/ray-project/ray/pull/27607
To make this possible to do in Helm, one option is to expose sidecar container specs in values.yaml.
As discussed further up in this thread, it's also possible to extract logs by running a log-processing daemonset and mounting the ray containers' /tmp/ray director to the relevant hostPath. (This kind of approach has the advantage that you don't need to modify the CR and the disadvantage that you have to set up some node-level logging infra.)
To avoid imposing any particular logging solution, we are documenting strategies rather than modifying the operator to set up logging automatically.

Satisfactorily resolved by https://docs.ray.io/en/latest/cluster/kubernetes/user-guides/logging.html


Build in Android Studio occur exception

Error:(17, 0) Could not find property 'VERSION_NAME' on root project 'ExpandableLayout'.
Open File

Error:(17, 0) Could not find property 'VERSION_NAME' on root project 'ExpandableLayout'.
Open File

Please try (for version 1.3) :
Step 1: build.gradle (project)
version = VERSION_NAME
group = GROUP
Change To :
version = 1.3
group = 'com.github.traex.expandablelayout'
Step 2: build.gradle (library)
Remove this line :
apply from: 'https://raw.github.com/chrisbanes/gradle-mvn-push/master/gradle-mvn-push.gradle'


[Test] Disabled rdar64672291.swift when appropriate.

The test requires execution and to be run with the just-built stdlib.  These fixes were already made on master.

@swift-ci please test

Merging to fix CI.

Build failed
Swift Test OS X Platform
Git Sha - ea75ffa4fd13ce331085b3c3e93dda8cecb78c7c


ERROR: Remote host key changed, the connection will be terminated!

I have a problem with server and cli not connecting on my ubuntu, what should I do to fix it, I first did a reinstall of sysrepo and netopeer2 (make, make install), but it still doesn't work!
I tried to re-execute both merge_hostkey.sh and merge_config.sh files, and after successful execution it still didn't solve the problem of not connecting. I read the content of the two files, it mainly uses sysrepocfg to configure the environment, so I also executed it separately, here it is：
Command line in merge_config.sh
root@ubuntu:/usr/local/share/netopeer2# sysrepocfg -X -x "/ietf-netconf-server:netconf-server/listen/endpoint[1]/name | /ietf-netconf-server:netconf-server/call-home/netconf-client[1]/name"
<netconf-server xmlns="urn:ietf:params:xml:ns:yang:ietf-netconf-server">   <listen>     <endpoint>       <name>default-ssh</name>     </endpoint>   </listen> </netconf-server>
Command line in merge_hostkey.sh
root@ubuntu:/usr/local/share/netopeer2# sysrepocfg -X -x "/ietf-keystore:keystore/asymmetric-keys/asymmetric-key[name='genkey']/name"
<keystore xmlns="urn:ietf:params:xml:ns:yang:ietf-keystore">   <asymmetric-keys>     <asymmetric-key>       <name>genkey</name>     </asymmetric-key>   </asymmetric-keys> </keystore>
netopeer2-cli log：
root@ubuntu:/usr/local/share/netopeer2# netopeer2-cli

connect
nc ERROR: Remote host key changed, the connection will be terminated!
nc ERROR: Checking the host key failed.
cmd_connect: Connecting to the localhost:830 as user "root" failed.

netopeer2-server log：
root@ubuntu:/home/karl/work/netopeer2/build# netopeer2-server -d -v3
[INF]: SR: Connection 18 created.
[INF]: SR: Session 7 (user "root", CID 18) created.
[INF]: SR: Triggering "ietf-subscribed-notifications" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[ERR]: LN: Could not bind "<IP_ADDRESS>" port 830 (Address already in use).
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-server" "done" event on enabled data.
[INF]: SR: Triggering "ietf-keystore" "done" event on enabled data.
[INF]: SR: Triggering "ietf-truststore" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-acm" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-acm" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-acm" "done" event on enabled data.
[INF]: SR: Triggering "ietf-netconf-acm" "done" event on enabled data.

netopeer2-cli

You need to remove the previously stored host key from known_hosts but the CLI can assist you. Run knownhosts to print all the host keys and then knownhosts --del <index> to remove specific ones. If connecting locally, it will be a host key for localhost.

netopeer2-server

You have another application running that is bound on the same port 830. Most likely another instance of netopeer2-server is running in background so terminate it.


Nonce implementation for style src

Hello,
I recently tried implementing nonce based content-security-policy from the documentation using the config listed below. The script tags in my head all have the nonce attribute and work fine this way. However, my style tags do not. In my app.vue I did the following to try to get it to work as well, however my stylesheets cannot be loaded into the DOM. Is this a bug or am I doing something wrong?
onMounted(() => {
  // set nonce to all style tags
  const styleTags = document.querySelectorAll('style')
  for (const styleTag of styleTags) {
    styleTag.setAttribute('nonce', useNonce())
 }
})

security: {
    nonce: true,
    headers: {
      crossOriginEmbedderPolicy: 'unsafe-none',
      contentSecurityPolicy: {
        'img-src': ['*', 'data:', 'https:'],
        'style-src': [
          "'self'", // fallback value for older browsers, automatically removed if `strict-dynamic` is supported.
          "'nonce-{{nonce}}'",
        ],
        'script-src': [
          "'self'", // fallback value for older browsers, automatically removed if `strict-dynamic` is supported.
          "'nonce-{{nonce}}'",
          "'strict-dynamic'",
        ],
        'script-src-attr': [
          "'self'", // fallback value for older browsers, automatically removed if `strict-dynamic` is supported.
          "'nonce-{{nonce}}'",
          "'strict-dynamic'",
        ],
      },
    },
  },

Hey, thanks for rainsing a question.
I would love to get some feedback from @trijpstra-fourlights who is an expert of nonce ;)

I wouldn't call myself an expert per se ;), but I'm happy to try and help!
The nonce attribute cannot be read at runtime by the browser, nor set retroactively which is essentially what you're doing in the onMounted() snipplet.

This is by design (as in, by the browser) because allowing that would defeat the whole purpose of the nonce: "hello browser, this specific resource is absolutely trusted by me, the server. You can check this because the nonce attribute is set by me and it matches the CSP header."

To get nonce working, you'll have to provide the nonce attribute when the actual

Thank you very much! I believe I understand what you mean by letting the client generate the nonce after the

Thank you very much! I believe I understand what you mean by letting the client generate the nonce after the

That would be really useful! @trijpstra-fourlights could you create a PR to branch 1.0.0-rc.1? :)

Thanks, this works out great.  I have one more question, if my component framework (like Vuetify) uses inline-styles i'd have to use unsafe inline right?

Thanks, this works out great. I have one more question, if my component framework (like Vuetify) uses inline-styles i'd have to use unsafe inline right?

If you cannot control how these inline styles get injected, then indeed your only option to use unsafe-inline.
While I haven't tested this, it seems that vuetify supports supplying a nonce (although they use a static one in their example, which defeats the purpose).
Also found this issue which provides some insight on how to approach this:

We need to add the cspNonce entry in the theme, you can try generating the cspNonce, add a Nuxt Plugin, register vuetify:before-create hook and apply the cspNonce to the theme.

So if you hook into the vuetify:before-create hook, you could try to get the current nonce value const nonce = useNonce() and apply that to the cspNonce parameter on theme.
I haven't tested that though, so YMMV. Let us know if you are able to figure it out!
If not, I can probably help you out on getting it to work, but I can't promise on when I'll have time to take a proper stab at it..., so if it's blocking you, your best bet is to go for unsafe-inline for now.

Thanks, you are a saint. I tried to incorporate this but alongside

Thanks for extensive research @trijpstra-fourlights and for additional details @erikknorren 💚
After reading your conversion, in my honest opinion, I dont think that this should be added to Nuxt Security.
In general, it would be useful, but it is a bit of an edge case right now that requires a functionality that as you mentioned will create some overhead for all module users (not only those who are using it) which can be troublesome for performance for example.
This case of Vuetify inline styles shouldnt be a reason to add a feature that will add some dead code for other users. And considering that this issue can be easily mitigated with setting less strict CSP, I would recommend to follow this path now.
But lets keep this idea in mind. Maybe in the future versions, we will be able to implement similar functionality without causing overhead.
Thanks once again for everything!


DropdownMenu is showing black background when first opened on Linux (only on latest stable, in master it has been fixed it seems)

Steps to reproduce

Open flutter project on linux
Click on DropdownMenu
Black square appears where you should see the options to choose from, it goes away once you hover over it

Expected results
No black background
Actual results
black background appears
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Here is a photo, so you can see the problem:
https://github.com/user-attachments/assets/178c27a9-21a4-4fed-9912-bc2e091ecbcc
Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

@katronic-applications
Can you provide a runnable code sample that shows the black background ?
I tried with official example running on desktop but it didn't replicate.
Also, if it doesn't replicate in master, then it is probably fixed and will be available in next stable release.

I tried to reproduce it, but couldn't. I was thinking it had maybe something to do with the splash screen add on, or the dark theme.
In the process of testing this, I upgraded flutter to 3.22.3 from 3.22.2 on the stable channel and this fixed it!

Closing as fixed.


cassava-<IP_ADDRESS> broke GHC 7.0/7.2

Fyi, I had to edit in a base >= 4.5 lower bound, due to
Data/Csv/Builder.hs:56:5:
    Not in scope: `<>'
    Perhaps you meant one of these:
      `>>' (imported from Prelude), `<' (imported from Prelude),
      `<=' (imported from Prelude)

Data/Csv/Builder.hs:64:24:
    Not in scope: `<>'
    Perhaps you meant one of these:
      `>>' (imported from Prelude), `<' (imported from Prelude),
      `<=' (imported from Prelude)
xcabal: Error: some packages failed to install:
cassava-<IP_ADDRESS> failed during the building phase. The exception was:

On purpose as I only support the latest 3 major releases. Putting the bound in there is good however. Will add it upstream. Thanks!

adding that bound in future rlses is what I wanted to hint at... thx :)


How an I register a callback on leaf node selection ?

I want to dynamically load content of a file corresponding to the selected leaf node. From the documentation I am not able to figure out how can I register a callback for leaf node selection.

Would you mind asking this on stack overflow? We're trying to move all implementation questions there. Thanks!

You can feel free to tag me on the question there (jtrussell).