Getting Cannot execute /home/user/.poetry/bin/poetry error when configuring the poetry environment

Describe the bug
When I setup my poetry environment it correctly auto detects the poetry executable but it says it is not executable. The file is there and it is "executable".
To Reproduce
Steps to reproduce the behavior:

Add environment
Select poetry environment
set correct poetry executable
See error

Expected behavior
That the poetry executable gets correctly set.
Environments (please complete the following information):

IDE: PyCharm Community 2021.1 EAP (same issue with 2020.3)
OS: ubuntu 20.10
Poetry Version [e.g. 1.0.8 ]
Plugin version 1.1.5

Additional context
This is the relevant method AFAICT
fun validatePoetryExecutable(poetryExecutable: @SystemDependent String?): ValidationInfo? {
    val message = if (poetryExecutable.isNullOrBlank()) {
        "Poetry executable is not found"
    }
    else {
        val file = File(poetryExecutable)
        when {
            !file.exists() -> "File ${file.absolutePath} is not found"
            !file.canExecute() || !file.isFile -> "Cannot execute ${file.absolutePath}"
            else -> null
        }
    }

    return message?.let { ValidationInfo(it) }
}

@michaelwiles
Thank you for creating this issue.
I will investigate the problem.

@koxudaxi I know its a little off topic but...
When I run with ./gradlew clean runIde it works. I can use the plugin successfully.
But when I do a ./gradlew clean buildPlugin after having commented out the code that causes this problem and then go to pycharm and manually install from file system the "old code" is still present - install from build/distribution.
the other question is how do I log so that I can do trouble shooting?
Currently I've removed the rogue code and am running pycharm and running with the first option so I can work. Thank you for this plugin.

@michaelwiles

But when I do a ./gradlew clean buildPlugin after having commented out the code that causes this problem and then go to pycharm and manually install from file system the "old code" is still present - install from build/distribution.

I don't know why the behavior. I often run ./grealdew clean  without buildPlugin. And delete the build directory 😅
How did you install poetry?
And, Would you please tell me the permission on the executable?

I tested the plugin on macOS and Windows10.
I may test it on Ubuntu or other distros.
Also, I guess that the condition is valid.
https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/io/File.html#canExecute()
https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/io/File.html#isFile()

@michaelwiles
Also, This code came from the pipenv integration of PyCharm.
https://github.com/JetBrains/intellij-community/blob/1e7fb15e147145e9ac0526f0bfdb7f028c11fe96/python/src/com/jetbrains/python/sdk/pipenv/pipenv.kt#L124-L135
fun validatePipEnvExecutable(pipEnvExecutable: @SystemDependent String?): ValidationInfo? {
  val message = if (pipEnvExecutable.isNullOrBlank()) {
    PyBundle.message("python.sdk.pipenv.executable.not.found")
  }
  else {
    val file = File(pipEnvExecutable)
    when {
      !file.exists() -> PyBundle.message("python.sdk.file.not.found", file.absolutePath)
      !file.canExecute() || !file.isFile -> PyBundle.message("python.sdk.cannot.execute", file.absolutePath)
      else -> null
    }
  }

I'm also getting this error.
Ubuntu 20.10.
Installed poetry with curl -sSL https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py | python - as recommended in the official docs
pyenv global is 3.9.0
I can run /home/<myusername>/.poetry/bin/poetry just fine from a shell.

Thank you for giving me the detail.
I'm installing Ubuntu on my Virtualbox.
But, I can't reproduce the problem.
I will continue working on resolving the problem tomorrow.

I might have better ignore the check that the file is executable on Linux 🤔

Hi, same problem here:

Ubuntu 20.04

Poetry 1.1.5, installed poetry with curl -sSL https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py | python - as recommended in the official docs

pyenv global is 3.9.2

PyCharm professional is version 2020.3.3, installed via snap with classic confinement

@erols
Thank you for the version information.
I'm installing Ubuntu 20.10 into VirtualBox. (I installed Ubuntu 20.04 last week)
I don't use snap to install PyCharm. I will try it.

Corrected Pycharm version to 2020.3.4

Great!! I just reproduced the problem in Ubuntu 20.10.
The problem doesn't happen in Ubuntu 20.04 🤔
I will check the reason tomorrow.

I just understood why the plugin show Cannot execute .../.poetry/bin/poetry
The plugin executes poetry --version to check executable poetry.
But,  The execution is used the system python interpreter. it means pyenv is not used.
Also, I can't run poetry with a system Python interpreter.
Would you please test this command?
$ /usr/bin/python3 `which poetry`

I guess Poetry should be called by an interpreter which is what you used when installing poetry.
Otherwise, the interpreter may not execute poetry.
Unfortunately, the system interpreter of Ubuntu 20.10 can't execute poetry without additional installing.
pyenv is set in shell settings like .bashrc
But, the plugin creates a process to execute poetry without shell settings.
I'm thinking about how do we execute the poetry. 🤔

I just understood why the plugin show Cannot execute .../.poetry/bin/poetry
The plugin executes poetry --version to check executable poetry.
But, The execution is used the system python interpreter. it means pyenv is not used.
Also, I can't run poetry with a system Python interpreter.
Would you please test this command?
$ /usr/bin/python3 `which poetry`

Hi,
I ran this command and got this error:
rols@erols-tux:~$ /usr/bin/python3 `which poetry`
Traceback (most recent call last):
  File "/home/erols/.poetry/bin/poetry", line 17, in <module>
    from poetry.console import main
  File "/home/erols/.poetry/lib/poetry/console/__init__.py", line 1, in <module>
    from .application import Application
  File "/home/erols/.poetry/lib/poetry/console/application.py", line 7, in <module>
    from .commands.about import AboutCommand
  File "/home/erols/.poetry/lib/poetry/console/commands/__init__.py", line 4, in <module>
    from .check import CheckCommand
  File "/home/erols/.poetry/lib/poetry/console/commands/check.py", line 2, in <module>
    from poetry.factory import Factory
  File "/home/erols/.poetry/lib/poetry/factory.py", line 18, in <module>
    from .repositories.pypi_repository import PyPiRepository
  File "/home/erols/.poetry/lib/poetry/repositories/pypi_repository.py", line 33, in <module>
    from ..inspection.info import PackageInfo
  File "/home/erols/.poetry/lib/poetry/inspection/info.py", line 25, in <module>
    from poetry.utils.env import EnvCommandError
  File "/home/erols/.poetry/lib/poetry/utils/env.py", line 20, in <module>
    import packaging.tags
  File "/home/erols/.poetry/lib/poetry/_vendor/py3.8/poetry/core/_vendor/packaging/tags.py", line 7, in <module>
    import distutils.util
ModuleNotFoundError: No module named 'distutils.util'
erols@erols-tux:~$

I guess Poetry should be called by an interpreter which is what you used when installing poetry.
Otherwise, the interpreter may not execute poetry.
Unfortunately, the system interpreter of Ubuntu 20.10 can't execute poetry without additional installing.
pyenv is set in shell settings like .bashrc
But, the plugin creates a process to execute poetry without shell settings.
I'm thinking about how do we execute the poetry. thinking

Actually I can't run poetry with system python on Ubuntu 20.04. I get this error:
pyenv: python: command not found

The `python' command exists in these Python versions:
  3.9.2

Note: See 'pyenv help global' for tips on allowing both
      python2 and python3 to be found.

sorry, I though I had already posted this
erols@erols-tux:~$ /usr/bin/python3 `which poetry`
Traceback (most recent call last):
  File "/home/erols/.poetry/bin/poetry", line 17, in <module>
    from poetry.console import main
  File "/home/erols/.poetry/lib/poetry/console/__init__.py", line 1, in <module>
    from .application import Application
  File "/home/erols/.poetry/lib/poetry/console/application.py", line 7, in <module>
    from .commands.about import AboutCommand
  File "/home/erols/.poetry/lib/poetry/console/commands/__init__.py", line 4, in <module>
    from .check import CheckCommand
  File "/home/erols/.poetry/lib/poetry/console/commands/check.py", line 2, in <module>
    from poetry.factory import Factory
  File "/home/erols/.poetry/lib/poetry/factory.py", line 18, in <module>
    from .repositories.pypi_repository import PyPiRepository
  File "/home/erols/.poetry/lib/poetry/repositories/pypi_repository.py", line 33, in <module>
    from ..inspection.info import PackageInfo
  File "/home/erols/.poetry/lib/poetry/inspection/info.py", line 25, in <module>
    from poetry.utils.env import EnvCommandError
  File "/home/erols/.poetry/lib/poetry/utils/env.py", line 20, in <module>
    import packaging.tags
  File "/home/erols/.poetry/lib/poetry/_vendor/py3.8/poetry/core/_vendor/packaging/tags.py", line 7, in <module>
    import distutils.util
ModuleNotFoundError: No module named 'distutils.util'

@erols
Thank you for your report!!
Use System Python to execute Poetry
If you don't use Python2.7 then you can make a symbolic link for python
I found a good package https://packages.ubuntu.com/en/groovy/python-is-python3
Also, you need a pip package to execute poetry with the system's Python3.
https://packages.ubuntu.com/en/groovy/python3-pip
Use pyenv to execute Poetry
I don't know the best way.
The plugin doesn't know what shell is used by a user.
In the first place, the plugin can't know users use pyenv mainly. :(

Hi! I did the following:

added a symbolic link from python to python3
disabled pyenv

Executing python correctly runs the system python 3.8.5
However, executing poetry still fails in PyCharm and in bash

Hi. I got it working finally. Steps from freah ubuntu 20.04:

If installed, disable pyenv by commenting out pyenv init in .bashrc
Start new shell
install pip3 sudo apt install python3-pip
pip3 install --upgrade keyrings.alt -> This is the fix
Now install poetry curl -sSL https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py | python3 -
Now poetry works in the command line
Re-enable pyenv
Check in Pycharm. Using poetry in Add Interpreter now working

Thank you @koxudaxi - very helpful.
So in a nutshell poetry must be executable as a "system" command - without the local shell being fully hydrated - for this to happen, python must be executable as a "system" command.
For me this means adjusting my poetry script to actively reference the system python shell - which is python3 in my case. I want to always control the local python executable so I don't want python to resolve on the system.
So what I've done is create a new poetry executable (poetry-pycharm) which uses this python3 executable: #!/usr/bin/env python3 is now the first line.
I think what might be helpful is to differentiate between the poetry file not being executable and the poetry file executing but failing to return successfully from 'poetry --version'. In this regard if we can log the output of this call we could send the user to the log messages in pycharm mayble for them to troubleshoot.
You want poetry to run in a small a shell as possible and requiring it to have a virtual environment causes a bit of a chicken and egg problem as poetry is a virtual environment tool...
So in other words, I don't think the plugin can fix this issue. All I think it needs to do is provide better feedback to the user to indicate that although it found the poetry executable it failed to execute it and it needs a working poetry executable in order to run.

Hi Michael,
I missed that I had to edit .poetry/bin/poetry. I have updated my steps.
I agree that this cannot be fixed in the Pycharm plugin. Unfortunately the poetry team should make this work out of the box on latest ubuntu flavours, or Ubuntu should come with a python executable and pip installed.

@erols @michaelwiles
Thank you for your comments.
I agree.

So in other words, I don't think the plugin can fix this issue. All I think it needs to do is provide better feedback to the user to indicate that although it found the poetry executable it failed to execute it and it needs a working poetry executable in order to run.

Users use flavors of different Python interpreters for each user.
The plugin can show the error message of the poetry command.
But, the message may not indicate the problem's direct reason to the user.
I could know the reason by your feedbacks.
Hi, @abn
Would you tell me what your advice on this problem?
We have problems poetry doesn't run with an interpreter of Ubuntu20.10. But, some users usually use pyenv for poetry.
PyCharm plugin can't detect whether users usually use pyenv for poetry. Because the plugin doesn't read shell settings.
Unfortunately, Poetry can't run on the system's python interpreter of Ubuntu20.10. We need to install few packages into a system .https://github.com/koxudaxi/poetry-pycharm-plugin/issues/196#issuecomment-805277273
How should we solve the problem?
Thank you.


What Dart formatter does Flutter team use?

Hello, I noticed that Flutter team does not seem to adhere to Dart formatting rules. I was wondering if you could explain what tool or formatter settings you are using, so that I may also use them.
Thank you
Luke

Hi @lukepighetti, we have a style guide here: https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo.  It is generally similar to dartfmt, with some exceptions though.

@jonahwilliams Thanks for the notes – do you guys edit your code manually after running it through dartfmt or is there a version that we can get access to?

100% artisan all natural hand formatted  😄

Cold pressed too, I bet. Alright, thanks for the heads up. I will stop looking for the holy grail of flutter formatters. Maybe someday someone will make it happen.


build(git): remove antares-launcher submodule

Since version v2.12, Antares Launcher is used as a Python library (it is a dependency). The purpose of this PR is to remove the antares-launcher Git submodule that is no longer used to build the application.

A Git submodule is a feature that allows a Git repository to include another Git repository as a subdirectory. This allows you to include one Git repository as a part of another repository, making it easier to manage and track changes across multiple projects.

resolves #1335

Le build sur Jenkins fonctionne bien : https://devin-ic.rte-france.com/job/antares/job/IC_DC/524/


How to respond on slash command asynchronous way?

I use slack-bolt:1.12.0 and kotlin:1.5.30.
I have long-time computations before I'm able to send a response on a slash command. Due to Slack requirements of responding time I have to execute app.ack() first and only after that do the other stuff. It is not only about slash command but views, attachements and everything else.
Actual result:
Right now I have to use CompletableFutures in the registration of the command to prevent Slack errors:
app.command("/issue") { req, ctx ->
    val payload = req.payload.text.trim()
    CompletableFuture.runAsync {
        ctx.respond( produceResponse(req) )
    }
    ctx.ack()
}

In every examples I managed to find a response is sent just like this:
app.command("/command") { req, ctx ->
    ctx.respond("Response example")
    ctx.ack("Command acked")
}

Was it designed on purpose? These scheme could lead to Slack timeout errors and show ugly "dispatch_failed" error to the users if something goes wrong even if the computations aren't as long as I have. Moreover, it doesn't work properly with Kotlin Coroutines because coroutines are expected to be finished before we return the ack() value (or it could be just my misunderstanding of how coroutines work)
Expected result:
I expected its usage more simple, for instance:
app.command("/issue") { req, ctx ->
    ctx.ack("Command acked")  // Telling Slack a hook was consumed
    ctx.respond( produceResponse(req) ) // Sending a proper response which takes times to be built.
    // other actions connected to the command
}

Hi @yoyojin-dev, thanks for asking the question! Perhaps, my comment at https://github.com/slackapi/java-slack-sdk/issues/825#issuecomment-906409841 can be an answer for your question. In the Java world, managing thread pools is not so simple. That's why bolt-java does not encapsulate the layer from developers.
I hope this helps! If you have further questions, let us know.

Hi, @seratch ! Thank you for the very quick answering!
Perhaps, it would be great to include that code to the Java/Kotlin examples to prevent further questions like this:)


balance transfer: fix register response

It responses empty secret value after successfully register and enroll, and the value of "enrollmentSecret" in persistence is also empty string. The added code set the "_enrollmentSecret" value of the User object and update the user context in persistence again.

Hello @zhayujie,
First, thanks for the contribution!
This is a read-only mirror. to see how you can help us please read the contribution guideline
https://hyperledger-fabric.readthedocs.io/en/release-1.1/CONTRIBUTING.html or hit us up on https://chat.hyperledger.org if you have questions...
Cynthia

Thanks for the contribution! this is a read-only mirror. please read the contribution guide.


read enum value as string

This PR changes enum value reading so that the symbol string is returned rather than its index, consistently with the original API.

Merged. Thanks for the patch!


How to use it without Qml

Hello, thanks for your sharing.
Our product is old and cannot use qml,  can you kindly show some c++ sample code for testing?
Thanks

Hi, thank you for your interest in this library.
Unfortunately I don't have any sample code that uses this library without QML. Also I suspect that some tweaking will be necessary to use this in pure Qt6/C++. Should be pretty straight forward though. Happy coding!


Bump pion/ice

Use new version of pion/turnt that fixes deadlock and panic

Codecov Report

Merging #809 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master   #809      +/-   ##
========================================
+ Coverage      76%    76%   +<.01%
========================================
  Files          59     59
  Lines        4250   4272      +22
========================================
+ Hits         3230   3247      +17
- Misses        748    749       +1
- Partials      272    276       +4

Impacted Files
Coverage Δ

sctptransport.go
76.87% <0%> (+1.25%)
:arrow_up:

pkg/media/ivfwriter/ivfwriter.go
61.11% <0%> (+2.28%)
:arrow_up:

Continue to review full report at Codecov.

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


NIFI-12825: implemented ListHBaseRegions processor

Summary
NIFI-12825
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

+1 LGTM, verified with ListHBaseRegions and ScanHBase, table had 3 regions and I got 3 FlowFiles out with the correct rowkeys. Thanks for the new feature! Merging to main (this may be able to be backported but I'm only merging to main for now)


support webpack treeshaking

This would build the package in a way that webpack is able to tree-shake the code. This reduces the bundle size from the whole 60kb (minified + gzipped) to only the size of the icons imported into the application code.

Thanks @barnabasJ, this looks great! 👍 Have you tested it? Would you mind doing the same to the 'iconoir-react-native' package?

@paescuj I published the changes to @barnabasj/iconoir-react for testing purposes, I was able to add this package and see the size difference in my project. If you'd like to test it yourself you can use https://www.npmjs.com/package/size-limit to check the bundle size. I will do the same for the react-native package as soon as I find time.
Btw. I plan on removing the package from npm once this is merged.

@barnabasJ Nice, looks good! Thanks!


Restrict notifications to current curators

Coverage increased (+0.02%) to 92.995% when pulling d06f1959bfc4cc46c71342742ad77a91635258bc on johngian:fix-nda-email into ab7d22ee71a07af4c2a47ad8459494df1eff9498 on mozilla:master.

Coverage increased (+0.02%) to 92.998% when pulling 69002edeea43e7a60b0fd19f73564276c0834dc2 on johngian:fix-nda-email into 77c0dc17efd33d8429aa7a327c981387c4331aa8 on mozilla:master.


Integrating two-up

Ready for review!

Pixelated bugs:
Chrome: https://bugs.chromium.org/p/chromium/issues/detail?id=843615.
Safari: https://bugs.webkit.org/show_bug.cgi?id=185683.
I need to add image-rendering: optimizespeed; for Firefox.
I need to fix my test to work in Edge 😢.

(aside: ugh, I made some comments here but never submitted the review)

@developit will it not let you submit it now?


[SPARK-25248.1][PYSPARK] update barrier Python API

What changes were proposed in this pull request?
I made one pass over the Python APIs for barrier mode and updated them to match the Scala doc in #22240 . Major changes:

export the public classes
expand the docs
add doc for BarrierTaskInfo.addresss

cc: @jiangxb1987

Test build #95399 has started for PR 22261 at commit afb50ee.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/2657/
Test PASSed.

Merged build finished. Test PASSed.

Test build #95399 has finished for PR 22261 at commit afb50ee.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95399/
Test PASSed.

What is .1 in the title [SPARK-25248.1]?

There are two PRs from that JIRA, one for Scala APIs and one for Python APIs


:memo: Chore(evmts): Add jsdoc to actions

Description
Add jsdoc comments
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #640

PR #641   👈

This stack of pull requests is managed by Graphite.


How to get coordinates from postal code

I am involved in a project, I need to open a Google Map in the application of a postal code. Is there a way for me to switch from the postal code to coordinates?
Ex: postal code: 500000

Hi @hautran90, converting address information into coordinates is called (reversed) geocoding. Geocoding services are provided by several companies (e.g. Google, Apple and Here).
If you want this functionality on an Android or iOS platform there are (limit) free SDKs available. We made a Flutter plugin called geocoding that provide access to these native SDK via Dart. But note that there are restrictions (mostly rate limits) on the use of this API.
There are also other plugins available if you want to make use of some of the paid services which are not restricted to the limitations that come with the free services.
Hope this helps. Feel free to leave another comment if you have more questions.


Irys tx_id as the mint#

I recently changed the Mint# of the NFT to be selected by the user.
It should instead be derived from the arweave tx_id, this way people can't mint duplicate NFTs. If it already exists, than it already exists.
There's a few ways to do this, and problems to be aware of like avoiding collisions and ideally keeping the database smaller by not needing the actual tx_id in metadata.
One idea is to repeatably convert different non-int chars of the tx_id to ints, and decode it at query time. But to ensure this is a long-term solution we have to learn how tx_ids are generated, hopefully with sufficient randomness.
Ultimately, if we do it this way, nfts wont need metadata, and users won't have to think about tx id. The problem is if nft_ids/mint# cap has less information than the tx_ids on arweave, we'll need another solution.

This is solved, all in the backend, for both the OG and Scion collection.


Add HTTPCode to APIError

APIError was intended to be transport-agnostic, so it currently doesn't have any fields for HTTP or gRPC response codes. However, will lead to some clunky error handling code for users once we use APIError on apiary library calls. To extract the most important fields, users will have to call errors.As twice with two separate error types:
if err := bucket.Create(ctx, "bad-project-id", nil); err != nil {
        var aErr *googleapi.Error
        if errors.As(err, &aErr) {
                fmt.Printf("Status: %d\n", aErr.Code)  // pulls HTTP response code
        }
        var apiErr *apierror.APIError
        if errors.As(err, &apiErr) {
                fmt.Printf("Reason: %s\n", apiErr.Reason())  // pulls error reason
                fmt.Printf("Details: %sn", apiErr.Details())
        }
        // Handle error
}

Code is in fact the only field from googleapi.Error that we are currently missing out on in APIError. For manual clients such as storage where most users will be HTTP-only, it would be cleaner and easier to document if we could add a method to retrieve this into APIError as well.
It may also be nice to pull the gRPC status into APIError as well; however, I think it's preferable to keep these as separate methods (HTTPCode and GRPCCode say) in order to avoid having to map between HTTP and gRPC statuses.

It seems like the json errors should include a grpc status code too, but the code is currently ignoring it:
https://pkg.go.dev/github.com/googleapis/gax-go/v2/apierror/internal/proto#Error_Status
It seems like the apierror should return that, or unknown if it's not included. Instead APIError.GRPCStatus() is returning nil for http errors, which means "OK".


Upgrade Bevy to 0.7

N.B. Please update bevy_tiled_camera version on crates.io, for now the terminal uses its git main branch as dev-dependency

Done and done, thanks again!


run with macvlan failed

docker network create -d macvlan --subnet <IP_ADDRESS>/24 --gateway <IP_ADDRESS> -o parent=eth0 vlan4
docker run -it --net=vlan4 centos:latest
ocker: Error response from daemon: OCI runtime create failed: Could not get veth interface: eth0: Incorrect link type macvlan, expecting veth: unknown.

/cc @egernst @mcastelino for input

Hi @gogolxdong - please could you:

Enable full debug:$ sudo sed -i -e 's/^#\(enable_debug\).*=.*$/\1 = true/g' /usr/share/defaults/clear-containers/configuration.toml

Re-run the docker command.
Paste the output of sudo cc-collect-data.sh into this issue (after reviewing it to make sure it doesn't contain any confidential/private details).

@gogolxdong -- thanks for opening the issue -- i can reproduce it locally and will look into this today.

Thanks @gogolxdong - once we've got a handle on this and fixed it, can we create a test either in this repo or https://github.com/clearcontainers/tests I wonder to avoid a regression?

Yes.  That was my first step on this -- see if we had that in our CI yet.  Will open an issue for that now.

@gogolxdong this is a limitation of the current implementation. I have a PR pending to add support for macvlan. Once this merges we will have macvlan support.
https://github.com/containers/virtcontainers/pull/447


The "polyfills.io" service is unavailable

What
Refer to Title
Details
The domain "polyfills.io" was for resolving the problem of "polyfill" can not be accessed in Malaysia. Refer to:  https://github.com/JakeChampion/polyfill-service/issues/2638#issuecomment-913573567
I have checked the DNS, it can't be resolved

Checks from Dig Web Interface

Hi, yes, I do not own the polyfills.io domain, if it no longer resolves, that is out of my control. Is polyfill.io still blocked in Malaysia?

u can self host

What
Refer to Title
Details
The domain "polyfills.io" was for resolving the problem of "polyfill" can not be accessed in Malaysia. Refer to: #2638 (comment)
I have checked the DNS, it can't be resolved  Checks from Dig Web Interface

Self Host it!

Hi, yes, I do not own the polyfills.io domain, if it no longer resolves, that is out of my control. Is polyfill.io still blocked in Malaysia?

Not Sure for now, we have replaced the domain with the polyfill.io, Keep monitoring the stability

Hi, yes, I do not own the polyfills.io domain, if it no longer resolves, that is out of my control. Is polyfill.io still blocked in Malaysia?

Not Sure for now, we have replaced the domain with the polyfill.io, Keep monitoring the stability

Looks like the stability of the polyfill.io domain is fine, but for safety's sake, we will create a CDN that points o the polyfill.io domain as a backup.


script/build --install  tries to install things elsewhere

Prerequisites

[ ] Put an X between the brackets on this line if you have done all of the following:
N/A

Description
I'm trying to integrate a local Atom with a packaging approach that installs to a temporary location (destroot/DESTDIR), from which it would then be bundled and, ultimately, installed. I'm running into a few issues doing that, including the one below:
If I understand the build instructions correctly, this should be possible using script/build --install=/path/to/destroot. Initial results suggest that would inde indeed be the case if it weren't for certain files that are installed elsewhere (not even in the location provided by getInstallDirectory() in command-installer.js).
Steps to Reproduce

script/build
script/build --install=/path/to/destroot

Expected behavior: [What you expect to happen]
The entire install tree should appear under /path/to/destroot/$prefix
Actual behavior: [What actually happens]
Moving generated startup blob into "/path/to/atom-1.28.0/out/atom-1.28.0-amd64/snapshot_blob.bin"
Skipping creating debian package. Specify the --create-debian-package option to create it.
Skipping creating rpm package. Specify the --create-rpm-package option to create it.
Skipping artifacts compression. Specify the --compress-artifacts option to compress Atom binaries (and symbols on macOS)
Installing "atom-1.28.0-amd64" at "/path/to/destroot/opt/local/share/atom"
>>> Installing icons at "/usr/share/icons/hicolor"
>>> Error: EACCES: permission denied, open '/usr/share/icons/hicolor/1024x1024/apps/atom.png'
  at Object.fs.openSync (fs.js:646:18)
  at copyFileSync (/path/to/atom-1.28.0/script/node_modules/fs-extra/lib/copy-sync/copy-file-sync.js:21:16)
  at Object.copySync (/path/to/atom-1.28.0/script/node_modules/fs-extra/lib/copy-sync/copy-sync.js:32:7)
  at fs.readdirSync.forEach.imageName (/path/to/atom-1.28.0/script/lib/install-application.js:103:14)
  at Array.forEach (<anonymous>:null:null)
  at module.exports (/path/to/atom-1.28.0/script/lib/install-application.js:99:36)
  at binariesPromise.then.then.then.then.packagedAppPath /path/to/atom-1.28.0/script/build:132:7)
  at <anonymous>:null:null

Reproduces how often: [What percentage of the time does it reproduce?]
100%
Versions
Atom 1.28.0 built from source (using npm 6.1.0 and nodejs 8.11.3), pulls in apm 1.19.0
Happens on Linux.

Thanks for the report!

Error: EACCES: permission denied, open '/usr/share/icons/hicolor/1024x1024/apps/atom.png'

This particular error is a duplicate of https://github.com/atom/atom/issues/16853 so you can subscribe there for updates if you'd like.

If I understand the build instructions correctly, this should be possible using script/build --install=/path/to/destroot. Initial results suggest that would inde indeed be the case if it weren't for certain files that are installed elsewhere (not even in the location provided by getInstallDirectory() in command-installer.js).

Would portable mode be an option for you?
https://flight-manual.atom.io/getting-started/sections/installing-atom/#portable-mode

Would portable mode be an option for you?

Not really, no.
The underlying idea was to build Atom from source and have it use the shared libraries already installed through the same package manager (or on the host), including nodejs and FFmpeg.
Apparently that's not what happens when you build from source...

The --install option unfortunately does not work great on non-macOS systems. Personally I always use the --create-debian-package instead, and install the package using dpkg instead. This gives me more control over the entire installation process. I would the same flexibility with the RPM package.
None of this solves the real problem, but hopefully it's a reasonable workaround for the time being.

Actually, correcting the install path in that icon base dir function is the only thing required. That has to be a path-specific patch of course as I haven't found any way yet to control the install location (prefix), so it's not a generic fix either. But it works with a packaging system that supports patching the code automatically.

The bigger problem here is the fact that nodejs applications apparently always fetch their own dependencies even if you build them from source. That really makes it a lot less interesting to include them in a packaging system that has its own dependency management (like one finds in almost any distribution).


Disable scope pattern in Codacy

Currently, codacy rates default scope as bad and fails the check on simple View declarations.
Also, it spams unneeded comments about it on the PR, can we please turn the scope pattern check in Codacy?
@the-dagger @mananwason

@the-dagger @mananwason These fail comments have spammed my entire PR with unnecessary comments. Do look into this.

yeah please do look into this. it does comment a lot

@niranjan94 Please take a look

Sure, looking into it.

Done


Config context causes timeout on devices API endpoint

Environment

Python version: 3.6.15
Nautobot version: 1.2.7

We have 6300 devices in Nautobot and no config context defined. If I try to get all devices with no filters, it times out. I have to run the request several times to get the data to cache and than I am able to get the data in 37 seconds. If I exclude config context, I get the data right away in 25 seconds. I'm using pynautobot with threading, it uses 1000 limit for single request. The timeout on server is 60 seconds.
There seems to be some intensive computing done on getting config context even if there is none defined. This is the load on the server without using the filter, the peak is when I tried to get the data. First is from test server, second from production (a bit more powerful VM). It took 7 tries without threading to get the data from production server.


Output directory on network path not working

I am trying to run rattler-build with an output directory on network and a git repo as source:
rattler-build build --output-dir //server/share/my-dir

but it fails with the following error:
fatal: could not create leading directories of '\\?\UNC\server\share\my_dir\src_cache\my_project.git': Invalid argument

It seems rattler is adding the ?\UNC\ part to the adress, which is making git clone fail. I tried running git clone with the  ?\UNC\ part and it gave me the same error.

Ah, interesting! I think we could use the dunce crate to get rid of these, possibly.


[OpenAPI 3] Spurious error on op with same path and different verb from a shared route

Playground
import "@typespec/http";

using TypeSpec.Http;
@service({
  title: "Widget Service",
  version: "1.0.0",
})
namespace DemoService;

@sharedRoute
@post
op createUrlAttachment(
  @header contentType: "application/json",
  @path itemId: string,
  contents: { x: string },
): OkResponse | NotFoundResponse;

@sharedRoute
@post
op createFileAttachment(
  @header contentType: "multipart/form-data",
  @path itemId: string,
  contents: bytes,
): OkResponse | NotFoundResponse;

@delete op delete(@path itemId: string): OkResponse | NotFoundResponse;

The delete op gets the error "All shared routes must agree on the value of the shared parameter." but I believe they do agree.

fix the bug


Grafana intigration with InfluxDB

About this you could read statistics like players, network load, cpu load etc

#61 related


.dotfiles to apple arm

This repo is compatible with ARM architecture, BUT, You need first install brew yourself.
In this moment you need to do a step before that begin:

Install brew, yourself.

Sorry, ;)

problem with brew
(echo; echo 'eval "$(/opt/homebrew/bin/brew shellenv)"') >> /Users/test/.zprofile
eval "$(/opt/homebrew/bin/brew shellenv)"
brew bundle --verbose


Question: Should I use this lib for making a ratio improving seeder?

I want to make a little program to help me maintain a good ratio on some private trackers.  I want to programmatically add all new torrents to my client, have that client only download a fraction of the pieces then start seeding the pieces as soon as possible and as fast as possible.  Your library has the api I need.  You allow me to select arbitrary pieces and you allow me to monitor traffic to figure out if this torrent is helping or hurting my ratio.
However, I know so little about networking or bittorrent.  Will it upload efficiently if I have say 100 torrents running at the same time?  As in if I create 100 instances of the engine with they behave nicely together or will they all try to upload at the same time and create congestion?
Also is this designed well to do what I'm looking to do or is it so optimized for streaming that it performs relatively poorly as a seeder.

I'm interested in this question as well. Any help would be appreciated.


Fix kademlia unit tests

Some Kademlia unit tests fail randomly(?):

test_256_keys_rev -- assertion error is triggered because entry is added into a full bucket
test_closest_peers_256 -- find_node can give some nodes that are farther from the key then others. This happens because nodes are returned in the same order they are in buckets, and when the closest bucket contains < 20 items, first items from the next one are used, and others in the same bucket might be closer. We need to sort the list of nodes from a bucket before returning.

Re test_closest_peers_256.
This is a case that causes an error in the test.
Local node's key is 000..000.
Key for the find_node query: 00b459a9caf351363317d5f90002d08c44c4ef2a2708a84b4a7d32cc91e04875, bin prefix is 0000_0000_1011.... (K)
The key of the last (a.k.a. furthest) entry returned by find_node: 007f6662549fa595b59ee80dfa94a6ff94663be13f8ef65f8913bd1c6fbf33a5, bin prefix 0000_0000_0111 (F)
The key of an entry that is not in find_node but is closer to the entry: 0035087f4318e42467ab4c189b2e6fc7ede223afd910cea55a8f1d1ee11faab5, bin prefix 0000_0000_0011 (C)
Indeed, dist(K, F)=0000_0000_1100, and dist(K,C)=0000_0000_1000.

Our implementation misses proper iteration over K-buckets:
https://pub.tik.ee.ethz.ch/students/2006-So/SA-2006-19.pdf#subsection.3.3.1


Replace f32 usage with Decimal

Decimal is now used to retain decimal accuracy. As per the requirements of the OCPP specification.
Fixes #109

Fixes #109

Thanks for reviewing and merging!

Thanks for reviewing and merging!

Released v1.0.5 now, thank you too @huntc


7792 backend integration

Issue #:
bcgov/entity#7792
Description of changes:

Integration with backend

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

WIP progress. another PR for test cases :)


turn off histogram by default for int key columns

We should also turn off the default int formatting (no thousand separator).


feat(udf): support multiprocess pool for CPU-bound Python UDFs

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
partially-resolve #13744
This PR adds a multiprocess pool to run CPU bound Python functions. Users can add the workers option in @udf or @udtf to enable parallel execution:
# a function that takes a long time to execute
@udf(input_types=["BIGINT"], result_type="BIGINT", workers=10)
def square(x: int) -> int:
    sum = 0
    for _ in range(x):
        sum += x
    return sum

Timing results:
dev=> select square(x::int8) from generate_series(10000, 11000) t(x);
time：245.066 ms (single)
time：294.942 ms (workers=10)

dev=> select square(x::int8) from generate_series(100000, 101000) t(x);
time：2377.646 ms (single)
time： 377.491 ms (workers=10)

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[x] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[x] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
Python UDFs now support multiple workers to run CPU-bound functions.

The overhead of data exchange between processes looks too large. I'm looking for some other ways, like spawning separate Arrow Flight servers on each process and enabling the SO_REUSEADDR option to allow them listening on the same port.

Closed this PR as the overhead of multiprocessing is too large. We would like to recommend another approach to scale Python UDF server: start a server on each CPU core and use a proxy such as Nginx to do load balance.


mail attribute is missing from this document

signInNames.emailAddress ([Identities] and othermails is listed, but mail attribute is not listed.
It is possible to add the AAD "email" mail attribute in a B2C ief.
With a NOTE: The mail attribute is the AAD eMail attribute, not the signInNames.emailAddress ([Identities]
You must add the mail claim to the user read step and then as an output claim on the token.

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

ID: b30bc6cd-5d1a-6966-747e-ec03607d8967
Version Independent ID: 5e6fd653-e232-3a19-3739-7144ea587c51
Content: User profile attributes in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/user-profile-attributes.md
Service: active-directory
Sub-service: b2c
GitHub Login: @kengaderdus
Microsoft Alias: kengaderdus

@debaxtermsft
Thanks for your feedback! We will investigate and update as appropriate.

@debaxtermsft
I'm going to assign this to the document author so they can take a look at it accordingly
@kengaderdus
Could you Please review this and update as appropriate.

@debaxtermsft
Thanks for your feedback and patience.
Yes, email is available as a user property, but we don't recommend updating this property for Azure AD B2C user profiles. Use the otherMails property instead #please-close.


Add activity feed API

#2  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jrschumacher and the rest of your teammates on  Graphite


Port #426 to 0.4 line

This is a squash/port of #426.
$ git merge --squash add_opencl_option
$ fix stuff
$ git commit

This "works" for me locally in the sense that it allows me to spin up a local Dask "cluster" without access to NVIDIA hardware.


Tooltip on tabs could show actual file

Or show it briefly in the statusbar when switching tabs.

With setTabToolTip that can be achieved conveniently. Just need to get notified whenever the file changes.


Content changes in the boilers and insulation partial templates

Trello Card: https://trello.com/c/4v0WFqqv
Parent Trello Card: https://trello.com/c/fwdVopYW
Factcheck
Energy grants calculator
Expected changes
URL on GOV.UK

Change content in the boilers and insulation section

Before

After

@ikennaokpala @leenagupte:

If it has passed codereview, could you please remove the label Ready for code review
Do we need fact-check?

@pmanrubia Yes, it needs factcheck.

This work has been abandoned. The focus will be on meeting the need of this calculator elsewhere on GOV.UK.


understanding the registry

Hi everyone!
First up, well done on typings; it has the look of something really interesting, forward thinking and useful :+1:
I've been reading the docs and realised that I was completely unclear about what the typings registry actually is.  If I'm in that boat, perhaps others are too.  It would be great to get some clarity and try and improve the docs as a result.
So here goes.  The registry takes the form of this repo: https://github.com/typings/registry
Every package (correct term?) in the registry takes the form of a package-name.json file which looks roughly like this:
{
  "versions": {
    "1.0.0": "github:typings/typed-chalk#a7e422c5455e70292e5675a727d43a7b05fc3e58"
  }
}

i.e. it's a pointer to some location.  From the docs

Possible Locations

http://<domain>/<path>
file:<path>
github:<org>/<repo>/<path>#<commit>
bitbucket:<org>/<repo>/<path>#<commit>
npm:<package>/<path>
bower:<package>/<path>

All cool.  Now comes the speculation / assumptions / questions.  Entering this:
typings search react --ambient

Will lookup the react typings in DefinitelyTyped and in typings/registry/ambient?  So the --ambient essentially means "lookup in DefinitelyTyped/DefinitelyTyped as well as in typings/registry/ambient"?  Or does it just mean one of these?
Conversely, does this:
typings search react

Just lookup typings in typings/registry? And it would lookup both in typings/registry/ambient and in typings/registry/npm?
Are there plans for other folders as well as typings/registry/ambient and typings/registry/npm?  If so, what is their purpose / how does typing make use of them.  Is typings/registry/bower a likely addition, for instance?
I hope these questions don't seem too daft; I just want to remove ambiguity.  I'd be happy to submit a docs PR once I'm a little clearer myself.
All the best and well done on the great work!

Will lookup the react typings in DefinitelyTyped and in typings/registry/ambient?

Yes. The --ambient flag means install/search/remove/etc an ambient definition, not related to "ambient" packages. This is a good reason to reconsider the naming of "ambient" in the registry, perhaps to "global" and/or "environment".

Just lookup typings in typings/registry? And it would lookup both in typings/registry/ambient and in typings/registry/npm?

Looks up in all non-ambient sources - npm, bower, github, common. More can be added as we go.
On structure, these are the current defined and tested locations: https://github.com/typings/registry#structure. More can be added simply enough, such as the tweak mentioned above.
No questions are daft, just means there needs to be a lot more documentation :+1: I would love PRs on this, that's definitely the place I need the most help of anything. I do need to design a website (unless someone else wants to give it a shot) and docs will be included there - until then, feel free to add docs directly to this repo.

Thanks for the response @blakeembrey.  I'm still a little confused as to when DefinitelyTyped/DefinitelyTyped is included in install/search/remove/etc operations.  Is it always queried or queried only with the addition of the --ambient flag?

Yes. The --ambient flag means install/search/remove/etc an ambient definition, not related to "ambient" packages. This is a good reason to reconsider the naming of "ambient" in the registry, perhaps to "global" and/or "environment".

Could you clarify what the difference is between an "ambient definition" and "ambient" packages please?   Does the former refer to DT and the latter to typings/registry/ambient?

Looks up in all non-ambient sources - npm, bower, github, common. More can be added as we go.

Given things are dual published to npm and to bower it would be expected to have entries for in the registry for each, both pointing back to the same typings?

Only queries with the --ambient flag, and only on search operations (E.g. typings search or typings install <package>).

Could you clarify what the difference is between an "ambient definition" and "ambient" packages please? Does the former refer to DT and the latter to typings/registry/ambient?

The former refers to definitions that are "global". E.g. have no imports or exports at the top level and usually declare module "...". The ambient folder in the registry is just a collection of these definition types.

Given things are dual published to npm and to bower it would be expected to have entries for in the registry for each, both pointing back to the same typings?

Absolutely, that's perfectly doable here. I suspect dual publishing is on a decline though.

I suspect dual publishing is on a decline though.

Definitely. My last project was npm only.

Only queries with the --ambient flag, and only on search operations (E.g. typings search or typings install <package>).

Cool.  I might submit a docs PR just to clarify this.  Is it a little weird that DT is only queried for ambient typings given that it often exposes definition files that are both ambient and modular? Given that they're always ambient I guess it's not so strange but it's curious from the angle that people also use DT for definitions that are non-ambient.  (eg on a previous project I was using tsd to install the React definitions which I used with Webpack; so I was all about the import statements.)
I might submit a docs PR around this just now as I think it's worth clarifying.

By the differing ambient modes of DefinitelyTyped, I assume you're talking about how some just do an ambient definition and then write another ambient definition that includes that but also exposes it as a module? Anyway, it's been on my roadmap to better support this use-case, but I need to extend the API to start resolving files and get better context to do that - just a matter of time.

I'm going to assume we have sufficient understanding now :smile: Closing for housecleaning, but feel free to continue discussion. @johnnyreilly Did you want to be an member here? Your feedback has been valuable.

Yes please @blakeembrey. Sign me up!


DOC: update docs for read_csv().na_values and keep_default_na

[x] closes #13967
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[ ] whatsnew entry

Current coverage is 85.26% (diff: 100%)

No coverage report found for master at 1919e26.

Powered by Codecov. Last update 1919e26...e67209f

Do I need add what's new for this? And if so, in which what's new file?
Also I am not sure about this error:
2016-08-18 18:50:05 (7.84 MB/s) - 'miniconda.sh' saved [21310143/21310143]
ERROR: File or directory already exists: /Users/travis/miniconda

whatsnew is not needed. And don't worry about the travis error, that is something unrelated.

@jorisvandenbossche Hey I made some changes. Is there anyway I can preview it?

You just look at the docstring in the console. (with ipython: pd.read_excel?)

I squashed and removed the backticks.
I did test:
In [12]: _NA_VALUES = set([
   ....:     '-1.#IND', '1.#QNAN', '1.#IND', '-1.#QNAN', '#N/A N/A', '#N/A',
   ....:     'N/A', 'NA', '#NA', 'NULL', 'NaN', '-NaN', 'nan', '-nan', ''
   ....: ])

In [13]: s = """as NaN: '""" + "', '".join(sorted(_NA_VALUES)) + """'."""

In [14]: print s
as NaN: '', '#N/A', '#N/A N/A', '#NA', '-1.#IND', '-1.#QNAN', '-NaN', '-nan', '1.#IND', '1.#QNAN', 'N/A', 'NA', 'NULL', 'NaN', 'nan'.

I didn't check the doc with installation and pd.read_excel? because I got a weird error out of nowhere. I am trying to fix it but I have no clue how this is happening yet. So should I post the problem in anywhere if I have no luck hacking it? Like google group or irc channel? Thanks!

Okay I fixed it. Something's wrong with Cython.
Ready to merge!

thanks!
Can you do a followup where we make this wrap on the NaNs (they generate an extra long line)
na_values : str or list-like or dict, default None
    Additional strings to recognize as NA/NaN. If dict passed, specific
    per-column NA values. By default the following values are interpreted
    as NaN: '', '#N/A', '#N/A N/A', '#NA', '-1.#IND', '-1.#QNAN', '-NaN', '-nan', '1.#IND', '1.#QNAN', 'N/A', 'NA', 'NULL', 'NaN', 'nan'.
thousands : str, default None
    Thousands separator for parsing string columns to numeric.  Note that
    this parameter is only necessary for columns stored as TEXT in Excel,
    any numeric columns will automatically be parsed, regardless of display
    format.

Sure I'll try!


Make width and height variables

This helps make it more obvious whats going on.
Note, could wrap lines here - but didn't since the file is already not strictly 99 width max.

Thanks, it is indeed much clearer like that! 😄


Release Training Code of Real3D-Portrait.

how long will you open-source the Train Code?I am very interested .

I am very interested too.


[WIP] Negotiated Versioning

I'm opening this up for feedback...
Still missing some tests and some XML docs etc., but the code is working.

Thanks a lot. This is a cool approach. I think down the road the versioning strategies introduced here could be reused for an alternative way too - instead of forcing you to use the mapper, we could allow you to direct user to different actions based on the version.

XML docs

:shipit:

I think down the road the versioning strategies introduced here could be reused for an alternative way too - instead of forcing you to use the mapper, we could allow you to direct user to different actions based on the version.

Yeah, I've thought about that, and seen others do something similar, but I figured (for my needs, anyway) that this would be the best approach. I usually just return the same resource, but only need different representations for different versions :smile:

One thing that I haven't started thinking about is version negotiation for input models. That would definitely need different actions.

let's pull this in and consider the different action support as the next step. Thanks a lot, this is a great feature.


[EBPF] attacher: Remove procfs update wait

What does this PR do?
There is some code to retry procfs reads for up to 10ms after receiving exec events "because procfs could be slow to update".  This retry loop makes handling of short-lived processes much slower and leads to the callback channels getting full and missing events.
The code has been present from the initial version[0] of the process monitor.  In that initial version the retry loop was waiting for procutil's NewProcess() to succeed after receiving an exec event from netlink.  It's hard to see why it was needed even then, since:

the netlink event is emitted in the kernel after exec is completed
for exec to even be called, the process has to already have been forked and be running
if the process is running the /proc/PID will exist, there is no race in the kernel here.  Such a race would for example imply that /proc/self is unreliable.

In the current code, the delay is waiting for /exe and /comm and not the /proc/PID directory itself, and the exec is emitted by netlink or by the event stream.  Even in this case it's hard to see what race is being addressed since the exec event is emitted after the binary has been determined and the argument page is set up.
Furthermore, recently metrics were added to check if the "temporary failure followed by success on retry" case ever happens. This has been running on nightly clusters for some days now and that case has not been seen[1] even once.
Remove it since it's apparently not needed.
[0] https://github.com/DataDog/datadog-agent/pull/14706/files#diff-2945b330cb2d393879b702461e6be88cc1de711796c00acb8bcbe83694e90afbR143-R159
[1]
https://ddstaging.datadoghq.com/metric/explorer?fromUser=true&state=H4sIAAAAAAAAA12RXYrDMAyE76LHxZSmbbbUV1mKMbGcCvyTWk4hDbn7oqTtwr5Jw-fRSJ7hgYUpJ9Bw2B9OpmnMvgEFfbHDjUH_zODQU6K6QjNUqgFBA6itNExP6Zvvj2ID9WIY0FcRp0GAShEZCyGDgoL3EbluAwrykBOj8blEW_-z93GrBE02ipdIk8R0tlrDeSydyBFroe79ZgIN9tFrnrhi3HXDuBsZy_y1wHJVIMPGYDffV_NnvVyX6yIYD4GqcRQxyZ0E_6hdTp761SBQpAq62SvgXKpcKheHBTQ45A5WL1nLl3WFGbha4Zrz8dCc28uxbc8nBZjcW7u0L62gL8g3E7OTLTmQo9SDgsGOjA60t4FxUeApVCyviJ_I5kFP8_6EPATiCssvbpvrpvkBAAA&start=1731574771705&end=1732179571705&paused=true#N4Ig7glgJg5gpgFxALlAGwIYE8D2BXJVEADxQEYAaELcqyKBAC1pEbghkcLIF8qo4AMwgA7CAgg4RKUAiwAHOChASAtnADOcAE4RNIKtrgBHPJoQaUAbVBGN8qVoD6gnNtUZCKiOq279VKY6epbINiAiGOrKQdpYZAYgUJ4YThr42gDGSsgg6gi6mZaBZnHKGniqyMkIGFA4MAB0GlgaCHCqTvLaOABGcI1wvfKCjZ61mWzajd04mYIajUZ1wABUPAAEvVgbwHZ4aAg8YxpOmfgiCAAUAJQgPAC6VK7uBxih4S+qbzGl8fdPEBtLBoHKgeQYUEIdrKeowM6YDQaCCZRJoURwJxyRTlHDoqBojFOehMZQiNweND3fgQeyYLBYhQ5EDokRKR48PhA+TohAAYSkwhgKBEBzQPCAA
Motivation
https://datadoghq.atlassian.net/browse/USMON-694
Describe how to test/QA your changes
Automated tests cover the functionality of watching for and attaching to processes.
Possible Drawbacks / Trade-offs
Additional Notes

/merge

/merge


'Test migrated servers.' is not clickable

Hi - the docs state:
'Do a test migration as follows:
In Migration goals > Servers > Azure Migrate: Server Migration, click Test migrated servers.'
However, there's nothing to click on there. Maybe that section needs to be reviewed a bit. Thanks !

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

ID: 0f6cf488-07d3-1b2b-69f2-f04c7890bd55
Version Independent ID: fd658be4-17c5-19c7-b6c9-630090880c3c
Content: Migrate on-premises Hyper-V VMs to Azure with Azure Migrate Server Migration
Content Source: articles/migrate/tutorial-migrate-hyper-v.md
Service: azure-migrate
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@jochenvanwylick Thanks for the comment. We will review your feedback and update you soon.

@jochenvanwylick Check the status of replication by clicking on Replicating servers in Azure Migrate: Server Migration. If replication job ran successfully and if you're unable to click test migrated servers do a refresh and see if it allows you to do test migration by clicking the tab 'test migrated server'. Appreciate if you could share the screenshot of the portal showing 'Test migrated servers' tab not clickable.

Hi @SadiqhAhmed-MSFT , so I was able to work around it no problem, it's just that the instructions say that the line should be clickable, but it's not.
I'm a couple of steps ahead now, and migrated the servers. Now you can see the 'replicating servers' are 0 and that removes the hyperlink to that overview. As soon as the number is higher than 1, the line is clickable, but at 0 it's not.
And so right after replication, the number of 'test migrated servers' is 0 and thus the line is not clickable. Only after testing the first server, you can use the line to go to the VM overview.
Does that make sense ?

@jochenvw Your understanding about "As soon as the number is higher than 1, the line is clickable, but at 0 it's not." is correct. I hope we are good to close this issue. Let me know if you've additional questions I can help with.

@jochenvanwylick Closing this thread for now. Feel free to reopen the issue if you need any help further on this.


Issue 486 unit testing 01

Please use the following format for the title:
nsc-events-nextjs _ unit-testing _ 486 _Unit Testing
Please fill out this description with the following:
Resolves #486
This pull request adds more unit testing to the admin page. I figured
out to mock the nextjs link and router.

Checkout my branch
Run npm install for any missing dependencies
In console run: "npm run test:coverage"

Could someone point me in the right direction for the type issue I am having with the usage of children and href?


VideoSourceFactory destructor does not free all devices

Currently only the Epiphan devices are freed in the destructor.
However the API promises to free all of them.
This might be related to #14

[ ] consider running a memory leak checker

[ ] why does the memory checker still show definitely lost bytes?


HHH-11816 - JoinProcessor doesn't handle certain table names correctly

https://hibernate.atlassian.net/browse/HHH-11816

I created a new Pull Request which adds the following improvements:

We consider the Dialect-specific quoting character
The parsing logic is reduced to a single Regular Expression.


Fix for newest rust nightly

Thanks, but this should basically naturally happen regardless. A specification of 0.3.0 is actually "0.3.0 compatible" as defined by semver, and in this case 0.3.3 is compatible so this crate should automatically use the newer version of rustc-serialize with a cargo update. Thanks for the PR though!


add minitest to development group in Gemfile

Was getting this:
fx-mbp-13:bluemonk_ipaddress fluong$ rake test
/Users/fluong/.rvm/rubies/ruby-2.2.1/bin/ruby --verbose -w -I"lib:lib:test" -I"/Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib" "/Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib/rake/rake_test_loader.rb" "test/**/*_test.rb"
/Users/fluong/github/bluemonk_ipaddress/test/test_helper.rb:2:in `require': cannot load such file -- minitest/autorun (LoadError)
        from /Users/fluong/github/bluemonk_ipaddress/test/test_helper.rb:2:in `<top (required)>'
        from /Users/fluong/github/bluemonk_ipaddress/test/ipaddress/ipv4_test.rb:1:in `require'
        from /Users/fluong/github/bluemonk_ipaddress/test/ipaddress/ipv4_test.rb:1:in `<top (required)>'
        from /Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib/rake/rake_test_loader.rb:10:in `require'
        from /Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib/rake/rake_test_loader.rb:10:in `block (2 levels) in <main>'
        from /Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib/rake/rake_test_loader.rb:9:in `each'
        from /Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib/rake/rake_test_loader.rb:9:in `block in <main>'
        from /Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib/rake/rake_test_loader.rb:4:in `select'
        from /Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib/rake/rake_test_loader.rb:4:in `<main>'
rake aborted!
Command failed with status (1): [ruby --verbose -w -I"lib:lib:test" -I"/Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib" "/Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib/rake/rake_test_loader.rb" "test/**/*_test.rb" ]
/Users/fluong/.rvm/gems/ruby-2.2.1/bin/ruby_executable_hooks:15:in `eval'
/Users/fluong/.rvm/gems/ruby-2.2.1/bin/ruby_executable_hooks:15:in `<main>'
Tasks: TOP => test
(See full trace by running task with --trace)

But now it works.

fx-mbp-13:bluemonk_ipaddress fluong$ rake test
/Users/fluong/.rvm/rubies/ruby-2.2.1/bin/ruby --verbose -w -I"lib:lib:test" -I"/Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib" "/Users/fluong/.rvm/gems/ruby-2.2.1/gems/rake-11.1.0/lib/rake/rake_test_loader.rb" "test/**/*_test.rb"
/Users/fluong/github/bluemonk_ipaddress/lib/ipaddress.rb:72: warning: mismatched indentations at 'end' with 'unless' at 70
/Users/fluong/github/bluemonk_ipaddress/lib/ipaddress/prefix.rb:81: warning: mismatched indentations at 'end' with 'class' at 21
/Users/fluong/github/bluemonk_ipaddress/test/ipaddress/ipv6_test.rb:18: warning: duplicated key at line 30 ignored: "1080:0:0:0:8:800:200C:417A"
/Users/fluong/github/bluemonk_ipaddress/test/ipaddress/ipv6_test.rb:18: warning: unused literal ignored
/Users/fluong/github/bluemonk_ipaddress/test/ipaddress/ipv6_test.rb:20: warning: duplicated key at line 31 ignored: "1080::8:800:200C:417A"
/Users/fluong/github/bluemonk_ipaddress/test/ipaddress/ipv6_test.rb:20: warning: unused literal ignored
Run options: --seed 33632

# Running:

......................................................................................................................

Finished in 0.477777s, 246.9769 runs/s, 1052.7914 assertions/s.

118 runs, 503 assertions, 0 failures, 0 errors, 0 skips


Greek translation

Yup, thanks, i just edited it out.  I had a conflict and it must have slipped my attention.

Thank you very much @gemantzu, much appreciated as always 😀


Failing to detect some mobile devices

In version 0.4.3, this user-agent:
Mozilla/5.0 (Linux; Android 5.0.2; SM-G9250 Build/LRX22G; wv) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/43.0.2357.121 Mobile Safari/537.36 [FB_IAB/FB4A;FBAV/<IP_ADDRESS>.273;]

Will give 'UnknownMobile' with the .phone() method call.
In version 1.2.0 the same method returns null.

Yes, this is true. Version 1.0.0 had breaking changes:
v1.0.0:
  date: 2015-03-22
  changes:
    - use<EMAIL_ADDRESS>    - breaking change: phone() and tablet() won't return 'UnknownMobile' -> have to check mobile() as well (issue #20)
    - breaking change: fallback detection phone/tablet is now based on smaller screen side (issue #23)
    - expose internal implementation in case somebody wants to quick fix or monkey-patch the behaviour

I'm sorry that your code broke, but this behaviour makes a bit more sense and was demanded by active users.

No worries, thanks for the info!

Btw, added SM-G9250 to Samsung list in 2.8.14. Thanks!

Thanks @serbanghita
I'll release a new version in a few days...


adding changelog link to plugin summary.

It would be great if we could add a link to Changelog in the plugin summary.

Author: Team Grav -<EMAIL_ADDRESS>Homepage: https://github.com/getgrav/grav-plugin-admin
Bug Tracker: https://github.com/getgrav/grav-plugin-admin/issues
Keywords: admin, plugin, manager, panel
License: MIT
Description: Adds an advanced administration panel to manage your site
Readme:   https://github.com/getgrav/grav-plugin-admin/blob/develop/README.md
Changelog: https://github.com/getgrav/grav-plugin-admin/blob/develop/CHANGELOG.md

What's the reason why you want to specify it? Grav picks the CHANGELOG.md file by convention.

Before updating a plugin it's mandatory to read the changelog to be aware of the upcoming changes. So right now you have to navigate to the correct changelog file, but it would help probably those out who read it before updating a plugin. This is just a suggestion to improve usability :).

I think this would be useful to see a changelog in the summary.

Something we're working in the near future...


Various Bug Fixes. Make 'Active' Default On Scan Import Forms

BugFix 1. I believe from tagulous, a notion of a ‘CastTaggedQuerySet' is introduced. When Googling, I only saw this type with reference to tagulous. Our report generator does not know how to handle this. QuerySets are only referenced in two locations in the generate report method and only come from a few sources. There maybe a more eloquent fix, but to keep consistent with other if else logic.  I simply added an 'or' in the same style which seems to have corrected the issues. I believe we want to treat CastTaggedQuerySet in the same fashion as QuerySets.
BugFix 2. Function not updated to reflect removal of args parameter. Line 21 helper/notifications still has args even though it appears all other functions were rewritten to use kwargs. create_notification causes closing engagements / tests / findings / to throw an error. I've removed the args to make the methods consistent, which seems to resolve my issue.
Enhancement 1. Setting default for scan import status to active. This seems to be consistent with our current usage. 2. Not having by default makes it easy to create many inactive / mitigated findings from reupload usage.

Two additional bug fixes.

Certain metrics pages are broken for non-staff users. A key error was created from: https://github.com/DefectDojo/django-DefectDojo/blob/master/dojo/filters.py#L1085. Upon examining the form fields, the field is actually 'duplicate' on that specific form.
Reupload was broken and caused all findings to always be mitigated on reupload. Status is decided by diffing sets here: https://github.com/DefectDojo/django-DefectDojo/blob/dev/dojo/test/views.py#L841                 to_mitigate = set(original_items) - set(reactivated_items) - set(unchanged_items)
The issue is that original_items is not of the same type and thus the intersection can't be calculated. Currently, the type of 'original_items' is 'CastTaggedQuerySet' which surprised me. I understand transitional cast classes, but this queryset isn't even related to tags, and I believe taguluous is the only addition using such cast classes. Even before tagulous, I'm not sure how going to flat object created a successful comparison, or if it was bugged previously. My suspicion is that because ID is our unique id, it somehow functioned as intended. I think the larger problem I'm not addressing may be the CastTaggedQuerySet class. I'm investigating how we could re-cast or type in a better fashion, but there isn't much documentation so it will require more digging on my part to come up with a more complete solution. I'd like to go ahead and address the issues while I continue to research the casting, as this may create or have created other undiscovered bugs.

After further review on 2. the loop expects finding ids. This could be addressed by going to objects or sticking with ids. Flattening the other sets may solve the issue. My solution will be to flatten all lists as I'm assuming there is a reason for working with id's rather than objects.

In my local testing I am not able to replicate when a finding_id is a finding. Trying some debug lines.

Removing debugging. Unsure why it is working now.

I think there are unit tests for that reimport so I'm interested what is not working with opening/closing of findings on reimport.
I also think the same logic is in place in the API, so we should keep those aligned.

look good now, i'll leave the merge up to you.


Support anonymous login/signup interaction

ref #1373
ref for testing:

to signup:

generate JWK (can use https://mkjwk.org/)
obtain challenge by posting to http://localhost:3000/_auth/challenge with body {"purpose": "anonymous_request"}
self-sign JWT (can use https://jwt.io/), e.g.: eyJhbGciOiJSUzI1NiIsInR5cCI6InZuZC5za3lnZWFyLmF1dGguYW5vbnltb3VzLXJlcXVlc3QiLCJqd2siOnsia3R5IjoiUlNBIiwiZSI6IkFRQUIiLCJ1c2UiOiJzaWciLCJraWQiOiJGODgyM0Q5OS0wQzUyLTRGQTgtODE3MC1FQUNCNzBENkZFNDMiLCJhbGciOiJSUzI1NiIsIm4iOiJreFNoSGJ4WEdwcWxxOXFpeWFVdmlrT3RQVFpwS3hNLVZsTS1FTWtJa053ZnducGFCamFWamh2RFlJdVoyLXF1SXpEbVRfWlM3RHR4ZXJoV3VsRzJ6USJ9fQ.eyJpYXQiOjE1ODg3NTQ0MjEsImV4cCI6MTU4ODg1NDcyMSwiYWN0aW9uIjoiYXV0aCJ9.LiwJxQ6sWxfTNSGnINj3-SO4xeBanNB4kn_1F2Q8hYSFpjeY-zsFvDM1279aAwtOforDCdLrRRAg6CHZ4tV1jg
post to http://localhost:3000/oauth2/token?client_id=<client id>&grant_type=urn:skygear-auth:params:oauth:grant-type:anonymous-request&jwt=<JWT>

to login

obtain challenge by posting to http://localhost:3000/_auth/challenge with body {"purpose": "anonymous_request"}
self-sign JWT, e.g.: eyJhbGciOiJSUzI1NiIsInR5cCI6InZuZC5za3lnZWFyLmF1dGguYW5vbnltb3VzLXJlcXVlc3QiLCJraWQiOiJGODgyM0Q5OS0wQzUyLTRGQTgtODE3MC1FQUNCNzBENkZFNDMifQ.eyJpYXQiOjE1ODg3NTQ0MjEsImV4cCI6MTU4ODg1NDcyMSwiY2hhbGxlbmdlIjoiRlg3SDZTNlM1VzY1VlpGRVE4VEhDM1dRREZOUUpRWEgiLCJhY3Rpb24iOiJhdXRoIn0.R0EciMp8Dt_E74PU1SizX_JLN55nFhuR4laIdh5gSh51ADSth8RM5YBmOg166TiVzn1kBz9YWHwijolOUwzCaQ
post to http://localhost:3000/oauth2/token?client_id=<client id>&grant_type=urn:skygear-auth:params:oauth:grant-type:anonymous-request&jwt=<JWT>

Personally I'd prefer to support only third-party apps first. First-party app can be web-app only, which we don't have a concrete use-case on hand to design appropriately.

Note on RN SDK implementation:

RN don't have built-in webcrypto (we've polyfilled RNG before)
We need two things: generate asymmetric JWK and signing JWT
Found this library: https://github.com/kevlened/jose, which uses https://github.com/kevlened/isomorphic-webcrypto

RSA is not supported, so ECDSA has to be used instead. (Not sure about difference in security properties)
Replace our RNG with one provided by this?

I prefer we use the crypto functions provided by the platform without using third party library.
For JWK and JWT, we can use a pure javascript library

Then I suppose we'll implement a subset of JOSE & JWT ourselves, using platform crypto functions as primitive:

RS256 JWK generation
RS256 JWT signature

OAuth protocol convention is POST form and return JSON. Should be align with this convention in /oauth2/challenge?


feat(queue): frontend should not to add time manually

Overview
Backend was getting full access to control a new queue creation. Default date of queue creation is today, managed by time package.
Testings

[x] Add new queue with no daily_queue_date

Codecov Report

Merging #69 (677563e) into master (ff1ec09) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #69   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           11        11
  Lines          204       204
=========================================
  Hits           204       204

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 ff1ec09...677563e. Read the comment docs.


TOMEE-2943 Added .asf.yaml

Thanks for the feedback @Daniel-Dos .
The CI failure is not related to .asf.yaml  file. I'll proceed with the merge.


[SPARK-14154] [MLlib] Simplify the implementation for Kolmogorov–Smirnov test

What changes were proposed in this pull request?
jira: https://issues.apache.org/jira/browse/SPARK-14154
I just read the code for KolmogorovSmirnovTest and find it could be much simplified following the original definition.
Send a PR for discussion
How was this patch tested?
unit test

Test build #54161 has started for PR 11954 at commit c7dcb97.

Test build #54161 has finished for PR 11954 at commit c7dcb97.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54161/
Test PASSed.

Merged build finished. Test PASSed.

Test build #54284 has started for PR 11954 at commit fa94568.

Test build #54284 has finished for PR 11954 at commit fa94568.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54284/
Test PASSed.

Test build #54297 has started for PR 11954 at commit fa94568.

Test build #54297 has finished for PR 11954 at commit fa94568.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54297/
Test PASSed.

Merged to master


Update go-tuf dependency to latest

Signed-off-by: Priya Wadhwa<EMAIL_ADDRESS>
Summary

Release Note

Documentation

Codecov Report

Merging #2662 (ceb2afa) into main (5990090) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main    #2662   +/-   ##
=======================================
  Coverage   30.00%   30.00%
=======================================
  Files         146      146
  Lines        9299     9299
=======================================
  Hits         2790     2790
  Misses       6077     6077
  Partials      432      432

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


Updated with a link to the new emulator version

Docs Build status updates of commit de10903:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/install-the-tools.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Refactor Notification Telemetry to Remove Code Duplication

Refactor the two existing NotificationTelemetryProvider set of files to remove code duplication before introducing additional providers.

/azp run TransportPackage-Foundation-PR

Only minor nit is maybe change m_helper->m_telemetryHelper. At this point we have so many helpers it'd be great to clarify which ones we are using.

/azp run TransportPackage-Foundation-PR

/azp run TransportPackage-Foundation-PR

Only minor nit is maybe change m_helper->m_telemetryHelper. At this point we have so many helpers it'd be great to clarify which ones we are using.

Addressed, thanks

/azp run TransportPackage-Foundation-PR

/azp run TransportPackage-Foundation-PR

/azp run TransportPackage-Foundation-PR