[Howto guides] Install wechaty

The starter.md file contains a step by step guide on how to run the wechaty bot with the most famous messaging apps- Wechat and Whatsaap

I saw quite a lot of files but did not see the starter.md. Could you screenshot where we should review? Thank you.

Could you screenshot where we should review? Thank you.

Yes pls add screenshots, ask for help if you need help in adding screenshots of the preview : )

Could you screenshot where we should review? Thank you.

Yes pls add screenshots, ask for help if you need help in adding screenshots of the preview : )

This is the starter.md file. This can be reviewed at  docusaurus/docs/howto/starter.md.

Hey @vasvi-sood and @abhishek-iiit, I would be glad if you may keep your commit history clean next time you raise a PR.
By clean commit history and a good screenshot, I mean something like this : #928
Consider squashing commits, and try keeping the commit history easy to review.

This PR neither have screenshot(s) nor clean commit history making it hard to review 😅
I am confused, from where should I start reviewing 51 commits ?

We'll add a related screenshot. We had some issue in pushing the branch to GitHub due to an error in package.json file. We had to make various commits for that. We'll take care of it from next time.

This is how the starter.md file looks. Kindly review it @proudofsimin  and @Rohitesh-Kumar-Jain

Hi,
I followed these steps but in the end maybe I am doing something wrong, I am unable to get dong for a ding

What is to be done after scanning the code

You need to scan the code with whatsapp web, then ask a friend to send you a ding message. Please ensure ding is lowercase.

You need to scan the code with whatsapp web

Do I need to keep WhatsApp web open while scanning on the phone?

Try to fix the issues yourself first, and if you aren't able to fix these issues I will demonstrate how to work on two features simultaneously at the next meeting which will be nothing but the implementation of these steps https://gist.github.com/Rohitesh-Kumar-Jain/54e9f93bb6fad32ea2385016d1e2858d
One way to fix this error is :

Switch to master

git switch master

make a new branch for this PR

git switch -c branch-name

copy paste work related to this PR (related to only this PR).

commit changes from that branch

push changes

 git push origin branch-name

make a new PR, and close this one with reference to this one

congrats one PR solved

now, switch to master

git switch master
repeat steps 1-8 for other faulty PRs (all of your PRs are faulty 😅)
Don't wait for mentors to close PRs, close yourself : )
PRs like this are unacceptable and are going to cause merge conflict as improperly rebased.

Hello @Rohitesh-Kumar-Jain, I did exactly the same way, as you suggested here as well as in your previous comment. But, all the commits that were one in the master branch show too in the created branch. PR #939 and #940 are done exactly in the same manner.

But, all the commits that were one in the master branch show too in the created branch.

 You just need to keep your master branch in sync with the upstream that's it.
No where in my comments I mentioned you to make commits in your master branch.

@abhishek-iiit pls first backup all your documentation, I am having a fear that you might lose all your progress while making changes.
I see your master is 41 commits ahead the upstream, that needs to be corrected first.

I am saying again backup all of your progress in some files first, before doing his manovuer
fix your current master branch
First of all, checkout to your master branch, and then checkout to a new PR branch based on that.
git switch master
git switch -c new-branch

Then push the new branch to your fork and create a new PR based on that branch (the PR will be identical to this one with the exception of the branch). When you've done that, we can reset your master branch.
git switch master
git fetch upstream master # fetch upstream/master without merging into local master
git reset --hard upstream/master # reset local master to upstream/master's state
git push --force origin master # force-update your fork's master to reflect upstream

@huan and @lijiarui
What do you think should be done in this situation ?

@abhishek-iiit pls the first backup all your documentation, I am having a fear that you might lose all your progress while making changes.
Took the backup of the documentation. Thank you, for the help. Now, will make PR with suggested changes :)

Hi @vasvi-sood thank you for your work, but I am quite confused that I remembered last time we said that we will remove the section of "create the bot" since we have the installation section in tutorials. Why are we doing it again?

We thought that instaling wechaty meant the six line code to get started with wechaty. We wanted to create a guide for using wechaty with Whatsapp and Wechat separately because those are the most popular messaging apps. So it helps if developers find a separate section for that.

You need to scan the code with whatsapp web

Do I need to keep WhatsApp web open while scanning on the phone?

No, you don't need to. Can you please share the error you are getting.

No, you don't need to. Can you please share the error you are getting.

I am not getting any error, after I scan the QR code, even my phone gets registered, but where to go from there?
I got on 'dong's for 'ding's send to my phone

I will suggest you to first focus on your existing PRs, correct them.
And don't forget to backup your work. (I am having a fear that you might lose your work while trying to fix git issues, and that will be a disaster, so pls the first backup and save everything you have documented so far, and then feel free to refactor your PRs)

@huan and @lijiarui

What do you think should be done in this situation ?

@Rohitesh-Kumar-Jain It seems this PR commit too many files unrelated to this PR. It @vasvi-sood found it's hard to use git to correct them, a better way maybe close this PR, create a new branch and create a new PR.

@vasvi-sood and @abhishek-iiit I have a really simple idea to fix your issues if you are finding git way hard to fix issues.
@abhishek-iiit create a fresh fork (don't delete this fork, as this fork anyways belongs to @vasvi-sood).
And then just follow the instructions I gave you to work on more than 1 features, and raise new PRS, and we will close these PRs.

Thank you for your guidance and support, we have done the same way to fork the project(doesn't delete the previous one) and will open PR with the new forked repo.

@abhishek-iiit  and @vasvi-sood  thanks for the PR it is actually helpful for installing Wechaty locally ,one thing I would like to suggest is ,as all the installation steps   mentioned are easy for Linux users but Windows users place difficulty to run some of the commands like sudo etc., can you please add a section for Windows user too? as it would be even more widely reachable

@abhishek-iiit and @vasvi-sood thanks for the PR it is actually helpful for installing Wechaty locally ,one thing I would like to suggest is ,as all the installation steps mentioned are easy for Linux users but Windows users place difficulty to run some of the commands like sudo etc., can you please add a section for Windows user too? as it would be even more widely reachable

Hi @shraddhavp,
Thank you for your great suggestion. This PR is accidentally overlap with #905 from the tutorial team, so it's preferred if you could notify in there.


Middle Click (or Scroll Wheel Click or Shift + Click) should open new window.

On Windows (and Ubuntu with Unity Desktop Manager), clicking on a taskbar item with the scroll wheel button, middle mouse button, or shift + click causes a new window to be opened instead of an existing instance. VS Code does not do this.
Repro:

Open VS Code and open a folder.
Click the VS Code icon using the scroll wheel click.

Expected: A new VS Code window will open
Actual: The existing VS Code window is shown.

Duplicate of https://github.com/Microsoft/vscode/issues/48


allow cookies to be set as secure cookie.

Make the secure property of cookies configurable.

rebased and merged, dank u :-)


Mesh Smoothing support

The Readme suggest, that the code implements mesh smoothing. Nevertheless, the current implementation just tries to read a different mesh with the _smoothed suffix and throws an error if that does not exist. Is this intentional?

Hello,  I left out this part of the code because of OpenMesh dependency. If you are interested in the original code with smoothing you can find it under simplify_mesh/src/smooth/taubin.hpp.


results of voc-15-5

Hi,
I run your code on voc-15-5, and get:
Final Mean IoU 69.35 Average Mean IoU 74.84 Mean IoU first 75.25 Mean IoU last 50.47
however, the paper stated is:
75.73 51.71 70.09 75.19
why the results are different and poor than the paper stated?

Paper vs your run:

75.73 vs 75.25
51.71 vs 50.47
69.35 vs 70.09
74.84 vs 75.19

The difference is always small (except maybe for old). It's probably due to the inherent randomness of the initialization (can change depending on cuda version, pytorch version, etc.).
I don't see that as a major difference. Have you look at 15-1 ?


Add nkut

add Nan Kai University of Technology

@wuda1989 I have to close this request without merging, because you requested to add a non-existent domain to this repository: ;; connection timed out; no servers could be reached Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Incorrect response shown in REST API for GHES

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
This document still says that the response will have a filled field for 'license', which is does not have on GHES (the field contains null). This support has been removed from the server it seems and the field is now empty. This lowers the usage of things like the dependency-review-action and should be clear in the docs that the field is empty. Same for the field source_repository_url.
Example response:
[
  {
    "change_type": "removed",
    "manifest": "package.json",
    "ecosystem": "npm",
    "name": "helmet",
    "version": "4.6.0",
    "package_url"<EMAIL_ADDRESS>    "license": "MIT",
    "source_repository_url": "https://github.com/helmetjs/helmet",
    "vulnerabilities": []
  },
  {
    "change_type": "added",
    "manifest": "package.json",
    "ecosystem": "npm",
    "name": "helmet",
    "version": "5.0.0",
    "package_url"<EMAIL_ADDRESS>    "license": null,
    "source_repository_url": null,
    "vulnerabilities": []
  },
  {
    "change_type": "added",
    "manifest": "Gemfile",
    "ecosystem": "rubygems",
    "name": "ruby-openid",
    "version": "2.7.0",
    "package_url"<EMAIL_ADDRESS>    "license": null,
    "source_repository_url": null,
    "vulnerabilities": [
      {
        "severity": "critical",
        "advisory_ghsa_id": "GHSA-fqfj-cmh6-hj49",
        "advisory_summary": "Ruby OpenID",
        "advisory_url": "https://github.com/advisories/GHSA-fqfj-cmh6-hj49"
      }
    ]
  }
]

What part(s) of the article would you like to see updated?
The field in the response should be empty for the GHES versions where it is empty (not sure how far back this goes)
Additional information
No response

Hey @rajbos 👋
Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:

Thanks for opening this issue! Looking at the response schema, it looks like this field is defined as string or null, indicating that the field value can be either. Are you expecting that the response schema (and therefore the corresponding example) to say that this value can only be null?

GitHub CLI api
https://cli.github.com/manual/gh_api
gh api
--method DELETE
-H "Accept: application/vnd.github+json"
-H "X-GitHub-Api-Version: 2022-11-28"
/repos/OWNER/REPO/actions/artifacts/ARTIFACT_ID

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
This document still says that the response will have a filled field for 'license', which is does not have on GHES (the field contains null). This support has been removed from the server it seems and the field is now empty. This lowers the usage of things like the dependency-review-action and should be clear in the docs that the field is empty. Same for the field source_repository_url.
Example response:

[

  {

    "change_type": "removed",

    "manifest": "package.json",

    "ecosystem": "npm",

    "name": "helmet",

    "version": "4.6.0",

    "package_url"<EMAIL_ADDRESS>
    "license": "MIT",

    "source_repository_url": "https://github.com/helmetjs/helmet",

    "vulnerabilities": []

  },

  {

    "change_type": "added",

    "manifest": "package.json",

    "ecosystem": "npm",

    "name": "helmet",

    "version": "5.0.0",

    "package_url"<EMAIL_ADDRESS>
    "license": null,

    "source_repository_url": null,

    "vulnerabilities": []

  },

  {

    "change_type": "added",

    "manifest": "Gemfile",

    "ecosystem": "rubygems",

    "name": "ruby-openid",

    "version": "2.7.0",

    "package_url"<EMAIL_ADDRESS>
    "license": null,

    "source_repository_url": null,

    "vulnerabilities": [

      {

        "severity": "critical",

        "advisory_ghsa_id": "GHSA-fqfj-cmh6-hj49",

        "advisory_summary": "Ruby OpenID",

        "advisory_url": "https://github.com/advisories/GHSA-fqfj-cmh6-hj49"

      }

    ]

  }

]

What part(s) of the article would you like to see updated?
The field in the response should be empty for the GHES versions where it is empty (not sure how far back this goes)
Additional information
No response


[QST] Initiate Communication Task

Is it possible to create a task (a node in the dask graph) that initiate communication asynchronously and then have a later task wait on the communication to finish?
Motivation
I am working on optimizing distributed merge of two dataframes. I would like to implement double buffering so that workers can communicate while computing local dataframe merges.
cc @mrocklin

Tasks should not explicitly communicate.  Instead they produce results, the scheduler decides subsequent tasks should be run, and then those workers initiate communications to gather the necessary results. Once all dependencies for a result have arrived on a worker then the worker feels free to start that task.
In this way overlapping communication and computation already happen in Dask, assuming that there are tasks and communications to overlap.
For merge computations in partifcular I would expect to see a bit of overlapping towards the beginning, but, due to the all-to-all nature of the computation, I expect that there will be a decent period where there is only communication without extra computation to fill in.

As demonstration of this if you look at a task stream plot for a non-trivial computation you'll often see that red blocks (transfers) overlap non-red blocks (computation)
https://matthewrocklin.com/raw-host/dask-report.html

Make sense, but it the case of merge this ad hoc scheduling is a problem.
I think it is possible to hide most of the computation of the local concat and merge behind communication but it requires a very specific scheduling.
I think I finally found a viable approach :)
Will continue working on it tomorrow and report back.

I would be curious to hear more about your plan.  Let me know if there is some way I can help.

My plan was to create a task graph that implements latency hiding by making by interleaving independent task chains.
I can create the graph, but the scheduler re-distribute the initial dataframe partitions between the workers so it doesn't work.
I guess, it is not possible to pin results to a specific worker?
Alternatively, we can do as in cuML where they manually execute operations on the workers without involvement of the scheduler.

I guess, it is not possible to pin results to a specific worker?

You can specify that certain tasks run on certain workers, yes, but this kind of manual control rarely works out well in practice.  It's not robust to new or dying workers, load balancing, or a variety of other systems.
For the UCX + Dask + cuDF work my sense is that accelerating this is possibly not the biggest bottleneck right now.  Personally I'm more curious about how to make full DGX-1 or multi-node work  effectively.  My sense is that we still have order-of-magnitude issues going on there that are, I hope, more straightforward to integrate into a long-term solution.

You can specify that certain tasks run on certain workers, yes, but this kind of manual control rarely works out well in practice. It's not robust to new or dying workers, load balancing, or a variety of other systems.

Do you have an example of this you can point me to? I would like to benchmark the performance impact even if it isn't a general solution :)

See
https://distributed.dask.org/en/latest/locality.html#specify-workers-with-compute-persist
On Thu, Dec 12, 2019 at 10:51 AM Mads R. B. Kristensen <
<EMAIL_ADDRESS>wrote:

You can specify that certain tasks run on certain workers, yes, but this
kind of manual control rarely works out well in practice. It's not robust
to new or dying workers, load balancing, or a variety of other systems.
Do you have an example of this you can point me to? I would like to
benchmark the performance impact even if it isn't a general solution :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/5699?email_source=notifications&email_token=AACKZTCZYZUYTTJ7MJFOSLLQYJ2YXA5CNFSM4JZCX3TKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGXPGDY#issuecomment-565113615,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACKZTHUYZVQOR5CO5HCY63QYJ2YXANCNFSM4JZCX3TA
.

Putting on my Dask maintainers hat, It's going to be very hard to convince folks that specifying workers directly in algorithms like merge is a good idea.  There are several things that make this janky for Dask, so it's unlikely to be accepted.

Experimentation is always good though, I just don't want you to be surprised by showing better numbers and then having your work turned away.


Hare Architecture document is private

Problem
The link in the README.md document to the Hare Architecture re-directs to a Seagate-private Confluence page. It looks like the document was once public, but now re-directs to a private one.
Offending link:
https://github.com/Seagate/cortx-hare/blob/bd943b6dd93e84f265dd34291fad7f94e8732c43/README.md?plain=1#L340
Clicking on that link re-directs you to a login page, if you're able to login, a page in the private CORTX space.
Expected behavior
The document should be public and not require a login if it's linked from a public GitHub page.
How to reproduce
Click on the link from while logged out of Seagate Confluence.
Deployment information
N/A
Additional information
No response

For the convenience of the Seagate development team, this issue has been mirrored in a private Seagate Jira Server: https://jts.seagate.com/browse/CORTX-34076. Note that community members will not be able to access that Jira server but that is not a problem since all activity in that Jira mirror will be copied into this GitHub issue.

Patrick Hession commented in Jira Server:

I tried to move it but it says I don't have the permissions to move this document

@mukul-seagate11 can you see if you have permissions to update this?

Closing as an obsolete


Unity Scene loaded but it throws the exception of EntryPointNotFoundException

Plugin Version or Commit ID
v0.11.0
Unity Version
2021.3.16f
Your Host OS
macOS Ventura 13.2
Target Platform
Android
[Windows Only] Visual Studio C++ and Windows SDK Version
No response
[Linux Only] GCC/G++ and GLIBC Version
No response
[Android Only] Android Build Tools and NDK Version
Build Tools SDK version 33
NDK Version 21.4.7075529
[iOS Only] XCode Version
No response
Command Sequences
Used the following command to build the .AAR file
python build.py build --android arm64 --android_ndk_api_level 21 -vv
Log
EntryPointNotFoundException: Unable to find an entry point named 'mp_SidePacket__' in 'mediapipe_jni'. Tried the following entry points: Unable to get function 'mp_SidePacket__' because of 'The requested function was not found (0x06000001) dlerror() = undefined symbol: mp_SidePacket__
at Mediapipe.UnsafeNativeMethods.mp_SidePacket__ (System.IntPtr& sidePacket) [0x00000] in <00000000000000000000000000000000>:0
at Mediapipe.SidePacket..ctor () [0x00000] in <00000000000000000000000000000000>:0
at Mediapipe.Unity.FaceMesh.FaceMeshGraph.BuildSidePacket (Mediapipe.Unity.ImageSource imageSource) [0x00000] in <00000000000000000000000000000000>:0
at Mediapipe.Unity.FaceMesh.FaceMeshGraph.StartRun (Mediapipe.Unity.ImageSource imageSource) [0x00000] in <00000000000000000000000000000000>:0
at Mediapipe.Unity.ImageSourceSolution`1+d__13[T].MoveNext () [0x00000] in <00000000000000000000000000000000>:0
at UnityEngine.SetupCoroutine.InvokeMoveNext (System.Collections.IEnumerator enumerator, System.IntP
Additional Context
So, i have imported the AAR file which was generated by the command mentioned in installation guide
python build.py build --android arm64 --android_ndk_api_level 21 -vv
The build was created successfully by running the command, then i imported the files inside the unity and after that I have exported the unity project as an android project. After that i created the .AAR file of the exported project. Further, I used the AAR file to an another android project also yes i did imported the mediapipe_android.aar file as well inside the new android project. But the thing is when i run the app it successfully loaded the unity scene but it doesn't detect the face. The below is the error which I am facing. Also for your information I even tried the libmediapipe_jni.so file loading manually by using the System.loadLibrary() and same i did for the libopencv_java3.so. But nothing works, these two libraries do loaded successfully but when launching the unity it throws the error given below. Can you please help me out in solving the error...?
EntryPointNotFoundException: Unable to find an entry point named 'mp_SidePacket__' in 'mediapipe_jni'. Tried the following entry points: Unable to get function 'mp_SidePacket__' because of 'The requested function was not found (0x06000001) dlerror() = undefined symbol: mp_SidePacket__                                                                                at Mediapipe.UnsafeNativeMethods.mp_SidePacket__ (System.IntPtr& sidePacket) [0x00000] in <00000000000000000000000000000000>:0                                                                                at Mediapipe.SidePacket..ctor () [0x00000] in <00000000000000000000000000000000>:0                                                                                at Mediapipe.Unity.FaceMesh.FaceMeshGraph.BuildSidePacket (Mediapipe.Unity.ImageSource imageSource) [0x00000] in <00000000000000000000000000000000>:0                                                                                at Mediapipe.Unity.FaceMesh.FaceMeshGraph.StartRun (Mediapipe.Unity.ImageSource imageSource) [0x00000] in <00000000000000000000000000000000>:0                                                                                at Mediapipe.Unity.ImageSourceSolution1+d__13[T].MoveNext () [0x00000] in <00000000000000000000000000000000>:0
at UnityEngine.SetupCoroutine.InvokeMoveNext (System.Collections.IEnumerator enumerator, System.IntP`

The build was created successfully by running the command, then i imported the files inside the unity

Since you don't need to do anything manually to import libraries other than running the command, I feel you might be doing something wrong.
Did you perhaps build the AAR with an older version of the source code and copy it to the latest repository?
mp_SidePacket__ has been renamed to mp_PacketMap__ (cf. https://github.com/homuler/MediaPipeUnityPlugin/pull/941) in the latest source code, so it won't work with the older libraries.
P.S. By the way, why are you building it yourself instead of using the released library?

I think I've mistakenly copied to the latest version of the code. Anyways, I will try this solution and get back to you. But I was doing this because the size of the app is around 400 MB and I want to compress the size of the app. Can you please mention some detailed solutions on that because when running this command python build.py build --android arm64 --android_ndk_api_level 21 -vv it gives me DLLNotFoundException.

Please download the package from the v0.12.0 release page.
The symbols of the libraries in the packages are already stripped (e.g. mediapipe_android.aar size is about 25M).
If you still want to build the libraries by yourself, please read the Wiki or run python build.py build --help.
In most cases, you may want to specify --linkopt=-s and --solution.

because when running this command python build.py build --android arm64 --android_ndk_api_level 21 -vv it gives me DLLNotFoundException.

As I've said, it is highly likely that this is happening due to a difference in the versions of the code and library.

So, I tried the mentioned solutions, and both of them worked for me. Thanks a lot for helping me out. But in my app why there's a lag, the detection is slower as compared to an earlier version.

But in my app why there's a lag, the detection is slower as compared to an earlier version.

Please create a new issue and provide detailed explanation there.


Satellite Commands default to switch

When creating a button in the Satellite Commands, the GUI says it's a button (ie, a Sleep Command). But when clicking save, it shows up as a Switch in MQTT.
Closing the Command window and re-opening then clicking Modify on the Command shows it as a switch.
Repeatedly fixing it does not solve the problem.
The regular user HASS.Agent Commands work fine as buttons.

Hey @JvdMaat,
That's even the case with the latest beta?


fix: Attempt to workaround the static-bin-macos failure

The current CI failure in master is related to the static-bin-macos job, see https://github.com/ocaml/opam-repository/issues/19676
On second thought, this issue is due to both a broken depext and a (hopefully partly) broken post-check of the presence of the brew package.
So, maybe this PR could workaround the failure…😅

CI is now green! 🙂 So, merging.


Expose openapi and graphql inside service config

This PR does three things:

rename the core config of Platformatic DB into db
bump the config format to v0.18.0-dev
adds service.openapi and service.graphql configuration option to Platformatic Service; those are disabled by default.

@rozzilla you asked in discord how to add OpenAPI & GraphQL to a Platformatic Service. Here it is.


Adapter.js

Hello Muaz,
any news about integration with adapter.js?
Tks,
Gabriel

@muaz-khan Is this really a good idea? Adapter.js is changing frequently, so there is no need to add it to this package. Users should be install & include it separately.
.

Adapterjs is downloaded and included in the DEV directory; which means that their recent updates will not affect v3. (until we manually replace it in the DEV directory & recompile v3)
BTW, we can make adapterjs optional. So we will use our own implementation if adapterjs is missing in the demo files.


Error discovering local network interfaces: Spyder 2.3.0, python 2.7.7

From<EMAIL_ADDRESS>on 2014-09-17T16:15:02Z
Spyder Version:  2.3.0
Python Version:  2.7.7
Qt Version    :  4.8.4, PyQt4 (API v2) 4.10.1 on Linux
pyflakes >=0.5.0:  0.6.1 (OK)
pep8 >=0.6      :  None (NOK)
IPython >=0.13  :  2.1.0 (OK)
pygments >=1.6  :  1.6 (OK)
sphinx >=0.6.6  :  1.2.2 (OK)
psutil >=0.3    :  0.6.1 (OK)
rope >=0.9.2    :  0.9.4 (OK)
matplotlib >=1.0:  1.4.0 (OK)
sympy >=0.7.0   :  None (NOK)
pylint >=0.25   :  0.27.0 (OK)
What steps will reproduce the problem?

Launch spyder from the command line
Watch as the error appears:
[durack1@oceanonly ~]$ WARNING: Unexpected error discovering local network interfaces: 'SysOutput' object has no attribute 'flush'

What is the expected output? What do you see instead?
No error
If additional information is required, I'm more than happy to provide this.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1971

From ccordoba12 on 2014-09-19T06:28:21Z
This error will be fixed in 2.3.2
Thanks for your patience
Status: Duplicate
Mergedinto: 1789

From contrebasse on 2014-09-19T02:18:58Z
This is a duplicate of the soon-to-be-solved Issue #1789


mqtt wont connect

mqtt_bambulab.py", line 107, in async_subscribe
    MQTT_CLIENT.tls_set_context(ssl_ctx)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'dict' object has no attribute 'tls_set_context'

It works when MQTT_CLIENT is not declared as a global variable.
def async_subscribe():
  MQTT_CLIENT = mqtt.Client()

First system macOS 15.1.1 Python 3.10.10
Second system Linux Debian 12.6 6.1.0-22-amd64 Python 3.11.2

moving
MQTT_CLIENT = {}  # Global variable storing MQTT Client

up to line 12 in mqtt_bambulab.py works

Thank you, it worked somehow on windows so I didn't notice right away.


tony endpoint

@CodiumAI-Agent /review

PR Analysis

🎯 Main theme: Addition of a new endpoint to the application
📝 PR summary: This PR introduces a new endpoint '/tony' to the application. When accessed, it returns the string "tony".
📌 Type of PR: Enhancement
🧪 Relevant tests added: No
🔒 Security concerns: No security concerns found

PR Feedback

💡 General suggestions: It's good to see new features being added. However, it would be beneficial to add some tests for this new endpoint to ensure it works as expected. Also, consider adding some functionality to the endpoint, as currently, it only returns a static string.

 🤖 Code feedback:

relevant file: app.py
suggestion: Consider adding some functionality to the '/tony' endpoint. Currently, it only returns a static string, which might not be very useful. You could, for example, make it return some dynamic data or perform some operation. [important]
relevant line: @app.get("/tony")

How to use

Tag me in a comment '@CodiumAI-Agent' and add one of the following commands:
/review [-i]: Request a review of your Pull Request. For an incremental review, which only considers changes since the last review, include the '-i' option.
/describe: Modify the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest improvements to the code in the PR. Extended mode employs several calls, and provides a more thorough feedback.
/ask <QUESTION>: Pose a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.

To edit any configuration parameter from configuration.toml, add --config_path=new_value
For example: /review --pr_reviewer.extra_instructions="focus on the file: ..."
To list the possible configuration parameters, use the /config command.

@CodiumAI-Agent /review

PR Analysis

🎯 Main theme: Adding a new endpoint and removing some error handling code
📝 PR summary: This PR introduces a new endpoint '/tony' that returns a static string. It also removes some error handling code in the 'predict' endpoint.
📌 Type of PR: Enhancement
🧪 Relevant tests added: No
🔒 Security concerns: No security concerns found

PR Feedback

💡 General suggestions: It's good practice to add some tests when introducing new functionality, like the '/tony' endpoint. Also, removing error handling code can lead to unexpected behavior. If the error handling code is not needed, it would be helpful to explain why in the PR description.

 🤖 Code feedback:

relevant file: app.py
suggestion: Consider adding a docstring to the new 'tony' endpoint to explain its purpose. [medium]
relevant line: @app.get("/tony")

relevant file: app.py
suggestion: The error handling code that was removed seems to be handling cases where the model output is repeated or missing. If these cases no longer need to be handled, it would be helpful to explain why in the PR description. If they do need to be handled, consider adding the error handling code back. [important]
relevant line: -                if model_output["repeats"][j] > 0:

How to use

Tag me in a comment '@CodiumAI-Agent' and add one of the following commands:
/review [-i]: Request a review of your Pull Request. For an incremental review, which only considers changes since the last review, include the '-i' option.
/describe: Modify the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest improvements to the code in the PR. Extended mode employs several calls, and provides a more thorough feedback.
/ask <QUESTION>: Pose a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.

To edit any configuration parameter from configuration.toml, add --config_path=new_value
For example: /review --pr_reviewer.extra_instructions="focus on the file: ..."
To list the possible configuration parameters, use the /config command.


tests: check exception policy stats counters - v4

Edit the existing exception policy tests to check for the new exception policy stats counters.
Add one more test, to showcase behavior for exception policy stats counters when set up to log zero-valued counters and counters for each app-proto error.
Previous PR: #1617
Changes from last PR:

do not set up deltas or threads for stats, as the JSON schema doesn't accept that yet

Ticket
Redmine ticket:
https://redmine.openinfosecfoundation.org/issues/5816

Rebased with https://github.com/OISF/suricata-verify/pull/1674


Add evm functionality to the worker

adds evm unit tests

adds evm call and getter wrappers

renames Trusted Call and Getter function from account to sender_account

adds additional unit test for signing verification (unrelated to evm, but I still found it important)

[ ] remove sgx-runtime branch specifiction from .tomls after merging https://github.com/integritee-network/sgx-runtime/pull/63 (if this should go into master)

related issue: #493

Closing this in favour of #921


#940 Transformation should be allowed if printable chars are used

Table row can have symbols/words/space
All printable chars should be allowed
while doing check for supported pattern

The failure seems to be happening because of https://github.com/Behat/Gherkin/commit/db8be8a11b49926fc52e3b4c53134b711b7ed487

Yup

Thank you very much for contribution!


runtime: send http response before unlocking failpoint

Otherwise a failpoint could crash the program before sending a
response to the enabling http request.
/cc @xiang90

lgtm


Individual Phenotype Representation

I would like to question the way phenotypes are currently embedded in the 'Individual' structure ( https://github.com/ga4gh/schemas/blob/master/src/main/resources/avro/metadata.avdl#L96 ).
The comments question 'Is this the right representation?', and I would point out that the Genotype2Phenotype group is currently working to create the 'Association' data structure ( https://github.com/kellrott/schemas/blob/g2p/src/main/resources/avro/genotypephenotype.avdl#L97 )
Under this schema, the phenotype would be linked to the Individual via an 'Association', which would provide the opportunity to provide evidence for the association. The same association data structure can also be used to link samples, phenotypes and genomic features to phenotypes.

Thanks for pointing this out. We'll have to look into this, but on a first glance e.g. the AssociationType is way too rigid/overspecified. Also, we're getting rid of ENUMS if possible ...

enum AssociationType {
VARIANT_PHENOTYPE,
GENOMICFEATURE_PHENOTYPE,
SAMPLE_PHENOTYPE,
INDIVIDUAL_PHENOTYPE,
SAMPLE_PHENOTYPE
}

The enum is an attempt to get a data structure that 'subclasses'. The association represent multiple possible associations, but rather then put each of the into different data structures (like a VariantPhenotypeAssocation and a SamplePhenotypeAssociation structure) a single 'super' structure is used. The enumeration is a quick way for the user to determine the association type, rather then having to scan each possible permutation of non-null connectors.
At the same time, only particular associations are valid/make sense. An association connecting a Sample and an Individual need evidence, a Sample should simply belong to an Individual

Samples don't have to be children of individuals; they can be pooled, environmental ... There is a larger scope, not only specific to human disease etc.
But my comment is more about the technical aspect: An enum is a very rigid structure, which only can be modified with schema updates. We just gut rid of the one for GeneticSex (not solving it, though ...).
I'm actually quite positive to have it now on the level to discuss & modify this here ;-)

Thank you for your comments. One of the questions about this data structure is how much should we try to 'protect' users from creating bad data. As it stands, they could fill in the wrong optional fields for the declared type. So there are three ways to go:

Force the enumeration, because it refers to components of the schema, and therefore shouldn't change unless the schema itself is changing
Use a string field that should have a predefined string in it, ie 'variant_to_phenotype', but also allows them to fill in complete nonsense
Completely remove the 'type' declaration, and force the end user to infer the association type by scanning which connector fields are non-null.

I'm curious about the thoughts about this from the GA4GH community.

@kellrott As one more example, you can take a look at the Matchmaker API's representation of phenotype terms (called Features): PR #258. They're basically OntologyTerms, but have some additional metadata (ageOfOnset and observed at the moment).

I am in favor of controlled vocabulary, since it will makes searches more robust.  This will allow for the possibility of find something similar to your phenotype search.   The only thing to maybe take into account - that I've seen in the past - is where there was a huge amount of choices under some categories that caused people to select the most general, which made later integrative searches of comparing datasets almost impossible.  Most folks prefer to keep filtering their searches with new searches on the search result, rather than writing one long, detailed search.  So keeping these categories as clean as possible will make them more user- and scientist-friendly.


Use modern.dev

https://github.com/tiddly-gittly/Modern.TiddlyDev
Do you need a PR?
Also I'd like to add some features to this, are you there to accept PRs? Or may I just fork this to https://github.com/tiddly-gittly and publish my releases?

I just want to dev using ts

Hi, it's probably best simply to fork the code and produce a typescript version?

Oh, I haven't see this. Since you are still updating it, I don't want to split the community by forking it.


add publish authorization logic

published page can only be accessed by publisher in default.
add column
alter table publish add viewers text;

There is no transformation in server-side but client-side will send commna separated viewr list log like bob,mark
audit log setting in application.yml
fluentd.publish.tag: applog.yanagishima.publish

sample fluentd log for auditing
2021-04-07 19:44:18.000000000 +0900 applog.yanagishima.publish: {"viewers":"bob,mark","query_id":"20210407_103117_01025_xsfyd","publish_id":"77dc3445a2ed3cf78f09d5cb3cf07df0","engine":"presto","datasource":"test","user":"alice"}

@ebyhr
I think limit in publishList is necessary like queryHistoryUser
So, I added in https://github.com/yanagishima/yanagishima/pull/305/commits/5c37168b581762b15fb57cdfa46f41ccd5dc34a6
Could you please review?

@wyukawa The change looks good to me.


Implement devices in the protocol

Implement real devices in the protcol and fixed how synthetic trackers where listed
Add support for feeder app trackers in the protocol (needed the devices for that)

Just some style changes (why do you guys make your code look so squished?)

I havent changed the indent of the original code. If you wat to change that i think i would have to be a separate pr.

Reduced my suggestions to code you've added.


Which python version?

I tried python3.5.4 & python3.6.2, but failed to install torchnet:
  Could not find a version that satisfies the requirement torchnet (from versions: )
No matching distribution found for torchnet

modify title

@MichaelLiang12
It seems not the python version issues, please check here: https://github.com/pytorch/tnt

Yes, torchnet is not on pypi yet. you have to install from source for now. Ref to https://github.com/pytorch/tnt/issues/9
pip install git+https://github.com/pytorch/tnt.git@master

Thank you.


Support starting a hotfix

Context
Following the git-flow workflow, when a hotfix is started, the hotfix branch is created with the next version as part of the branch name and two pull requests are created, one targeting main and one targeting develop
Change
Create a script that automates the creation of the hotfix branch and the pull requests
Acceptance criteria

A developer can start a hotfix with one command
Hotfixes are consistent

Works as expected


DestVI tensor nan error

Hi, Thank you for your great work. I meet the same error with #65 . I am trying to run DestVI on other datasets, and I've transformed all the data into h5ad format with all necessary values to train the model.
I'm following the DestVI_tutorial.ipynb to run the training in order to do cell type deconvolution
My single cell data:
AnnData object with n_obs × n_vars = 32534 × 979
obs: 'Area', 'AspectRatio', 'Width', 'Height', 'Mean.CD298', 'Max.CD298', 'Mean.PanCK', 'Max.PanCK', 'Mean.CD45', 'Max.CD45', 'Mean.CD20', 'Max.CD20', 'Mean.DAPI', 'Max.DAPI', 'dualfiles', 'Run_name', 'Slide_name', 'ISH.concentration', 'Dash', 'tissue', 'slide_ID_numeric', 'Run_Tissue_name', 'Panel', 'Diversity', 'totalcounts', 'log10totalcounts', 'background', 'remove_flagged_cells', 'IFcolor', 'nb_clus', 'leiden_clus', 'negmean', 'class', 'cell_type', 'cell_ID'
My spatial data:
AnnData object with n_obs × n_vars = 340 × 979
obs: 'fov', 'spot_id', 'x', 'y', 'cell_counts', 'Ascending.vasa.recta.endothelium', 'B-cell', 'Connecting.tubule', 'Descending.vasa.recta.endothelium', 'Distinct.proximal.tubule.1', 'Distinct.proximal.tubule.2', 'Epithelial.progenitor.cell', 'Fibroblast', 'Glomerular.endothelium', 'Indistinct.intercalated.cell', 'MNP.a.classical.monocyte.derived', 'MNP.b.non.classical.monocyte.derived', 'MNP.c.dendritic.cell', 'Myofibroblast', 'NK', 'Pelvic.epithelium', 'Peritubular.capillary.endothelium.1', 'Peritubular.capillary.endothelium.2', 'Podocyte', 'Principal.cell', 'Proliferating.Proximal.Tubule', 'Proximal.tubule', 'T CD4 memory', 'T CD4 naive', 'T CD8 memory', 'T CD8 naive', 'Thick.ascending.limb.of.Loop.of.Henle', 'Transitional.urothelium', 'Treg', 'Type.A.intercalated.cell', 'Type.B.intercalated.cell', 'mDC', 'macrophage', 'mast', 'monocyte', 'neutrophil', 'pDC', 'plasmablast'
obsm: 'location'
Single cell data has the same gene list with spatial data
However, when training the model: it works fine in CondSCVI, but I meet the ValueError in DestVI training:
Exception has occurred: ValueError Expected value argument (Parameter of shape (979,)) to be within the support (Real()) of the distribution Normal(loc: torch.Size([979]), scale: torch.Size([979])), but found invalid values: Parameter containing: tensor([nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan, nan,         nan, nan, nan, nan, nan, nan, nan, nan,...
I am try to use raw data input or use log-normalized data input, or change the learning rate, but I still meet the ValueError.
Have you faced with similar error before?

You use very few genes. I guess you are using Nanostring CosMx data as single cell data (judging by the other columns) and therefore end up with the limited amount of genes. Is the other data GeoMx (given the low number of spots). Can you display np.min(np.sum(st_adata, axis=1))? I guess you have spots with very few counts this can lead to instability. I would recommend for GeoMx some bulk deconvolution technology as it contains more cells per 'spot' and only a few spots. Cell2location will handle this combination better, I guess. The meaning of activation state loses interpretability when the number of cells in a spot increases.

Yeah, I use the CosMx data
and my single cell result:
np.max(sc1.X), np.min(sc1.X), np.all(np.isfinite(sc1.X)), np.min(sc1.X.sum(1))
(158.0, 0.0, True, 20.0)
my spatial data result:
np.max(st1.X), np.min(st1.X), np.all(np.isfinite(st1.X)), np.min(st1.X.sum(1))
(3330.0, 0.0, True, 0.0)
Thanks for your answer

You have to filter everything with a count of zero and for stability decide to use something like 10 for st1.

thanks for your advice, I will try it.

Hi, I have the same error for visium object. any advice?

Filtering min_counts was not performed


Update index.md

Added a fun fact about the biggest DDoS attack ever recorded

[ x] I have read freeCodeCamp's contribution guidelines.
[ x] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

You ignored a list of requirements including "My pull request has a descriptive title (not a vague title like Update index.md)". Descriptive titles and good explanations of changes are necessary for a well maintained codebase. Free Code Camp is choosing to relax some of its requirements to help people reach their Hacktoberfest goals, but this PR would normally be rejected. In the future, please help repository maintainers by providing useful titles and do not ignore contributing guidelines or requirements.

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding. ✈


Fix httproute filter creation logic

Purpose

Fixes: https://github.com/wso2/apk/issues/1473

Codecov Report
Patch coverage has no change and project coverage change: -1.76% :warning:

Comparison is base (3adef64) 25.10% compared to head (e095a1e) 23.35%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1480      +/-   ##
==========================================
- Coverage   25.10%   23.35%   -1.76%
==========================================
  Files         260       66     -194
  Lines       30755     4261   -26494
  Branches     4830     2165    -2665
==========================================
- Hits         7720      995    -6725
+ Misses      22769     3266   -19503
+ Partials      266        0     -266

Flag
Coverage Δ

adapter
?

common-controller
?

config-deployer-service
23.35% <ø> (-0.03%)
:arrow_down:

idp-domain-service
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 195 files with indirect coverage changes

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


can we use sticky header in header section?

i need make a fix header in a section

Also interested in this


Room app

Please be as critical as possible!

I believe I have fixed all that was requested. Mind taking another look?


Integration with nginx-proxy - Automatic Docker Virtual Host

Hi,
Can this integrate with: https://github.com/nginx-proxy/nginx-proxy
The above project is really helpful with deploying the proxy automatically upon deploying the application containers.
So, I'm hoping that this nginx-proxy-manager could integrate with an existing installation of nginx?
Thanks,

Someone has created a little script for this: https://github.com/jc21/nginx-proxy-manager/issues/506


[Request] Suggestion

Hi, just came accross your theme, he is realy nice, and well design, but could it be possible to add an option to remove the effect on "member list" that we can enable to stay always on,
And also can you bring back the scroll bar on in chat (my mouse wheel doesnt work verywell. ) ? it doesnt seem the be shown on the latest version
Thanks

Hello, I'll add an import for the member-list animation once I finish working on my new theme, sounds good?

Yup, don't worry take your time, thanks again


Restarting an event loop thread to deal with indefinitely blocking handlers.

Due to some reason (implementation problem, environment problem, etc), there's a case where a handler sometimes blocks indefinitely.  In this case, an administrator might want to create a new event loop thread and migrate all channels (except the blocked one) to the new thread to keep the server working.

@trustin can we just close this ?

And the reason is?

@trustin no-one seems to have time doing it and no-one asked for it except you ;) ? If you are still interested please assign it to you. Just trying to cleanup the issue tracker a bit atm

Closing, then. :-)


Symbolizers should not take any; should take duck-typed option bags

This would add helpful type and exhaustiveness checking for those writing rules in TypeScript.

Had to revert this as it is too restrictive for function attributes. It is important to let the user distinguish between a function that takes (z,f) vs just (z) - in the 2nd case certain Symbolizers can optimize by doing less context mutating. Getting this to work with the type system is quite a puzzle. Until then function attributes should generally be specified with a full signature like (z:number, f:Feature):string for type safety and exhaustiveness checks.

Possible approach by @nf-s : https://github.com/protomaps/protomaps.js/pull/68#issuecomment-956002443

The current solution is, when writing rules in TypeScript, any attribute function has signature (zoom: number, feature?: Feature).
In terms of library core values, I see two happy paths:

Via a "traditional" script-includes, with rules written in plain JavaScript. The distinction between (z,f?) and (z,f) is irrelevant here because the types are gone.
using a bundler, protomaps as an es6 module dependency with TypeScript. There is a small ergonomics issue - having to use ? everywhere f is referenced. This annoyance seems like an acceptable tradeoff in the case you care about typing.

closed by https://github.com/protomaps/protomaps-leaflet/commit/bbe8075aa741765b462668a6d9ec47cf111ab766


remember the last selected devices?

Is there any way that I can save the input and output midi devices so I don't have to select again?

You can save the name in the onSelect() handler and use it in the select() function later.

perfect! This orientation helped me a lot to be able to do it, THANK YOU!

 :)


Modeline and file-based options on http/github endpoints

This https://github.com/apache/camel-k/pull/1515 introduces new ways to run integrations from the cloud directly.
What about modeline options that can be declared in the remote file?
E.g. the file declares something like:
// camel-k: property-file=application.properties

from("").to("")

Currently file-based options are ignored on remote URLs. But can we do better? ("possibly" avoiding infinite recursion).

should they follow the same path? i.e.
// camel-k: property-file=github:user/repo/application.properties

I'd better allow relative paths, so they work both when checking out the repo or running integrations remotely.

It is a little complex as you don't know what is the relative path, assuming that you can run an integration with multiple sources

mh, wonder if we should add a working-dir option like:
kamel run --working-dir github:user/repo MyRoute.java --property-file=application.property

would be translated to:
kamel run github:user/repo/MyRoute.java --property-file=github:user/repo/application.property

The current approach, also for local files, is to ignore the current CWD dir in the shell. All file-based modeline options are always relative to the file that contains them, so it will be for remote files.
So the --working-dir github:user/repo is implicit if you run kamel run github:user/repo/MyRoute.java.

Yep but my point is that there's a different behaviour for the same option used inside a source and outside the source so property-file=application.properties may related to two different locations depending where the option is used, if that's acceptable I can work on it

Yep but my point is that there's a different behaviour for the same option used inside a source and outside the source so property-file=application.properties may related to two different locations depending where the option is used, if that's acceptable I can work on it

Oh, yes. I think this difference is already there.
Take a file like ./path/It.java and a property file ./application.properties.
To run the file and link it to the properties, you should run:
kamel run ./path/It.java --property-file application.properties

Because CLI options are not related to the file but to the current location.
But if you want to link it using modeline, you should write this line in the file:
// camel-k: property-file=../application.properties

I don't see it as a big problem, we just need to document it ;)

I guess we can do both so:

if the working-dir is set, all the non absolute paths are relative to the working dir
if not set, the non absolute paths are relative to the context (to the file for modeline, to the current cwd for the CLI)


Feature/confirm release configs

Update to variable names and arguments in confirmReleaseConfigs/Jenkinsfile to reflect release numbers coming from URL rather than environment. Also made some terms more explicit

It looks like there is no major functionality change, just some variable renaming. Is this correct? Also, what does "Jenkins URL" refer to? The URL to the project in Jenkins?

It looks like there is no major functionality change, just some variable renaming. Is this correct? Also, what does "Jenkins URL" refer to? The URL to the project in Jenkins?

Thats right, just variable renaming. And also correct, the URL to the project in Jenkins. Should I make the argument/variable names more explicit to reflect that better?

Well, when I read def currentReleaseFromURL = (pwd() =~ /Releases\/(\d+)\//)[0][1];, the pwd() part, it "prints" the current working directory which makes me think of paths and filesystems, not URLs. But I guess a path could be a form of URL, so it's probably OK. ;)

Well, when I read def currentReleaseFromURL = (pwd() =~ /Releases\/(\d+)\//)[0][1];, the pwd() part, it "prints" the current working directory which makes me think of paths and filesystems, not URLs. But I guess a path could be a form of URL, so it's probably OK. ;)

Oh, thats a really good point, I am going to change this to reflect that


Add jwt access credentials

Add ServiceAccountJwtAccessCredentials class
Change updateMetadata callback function signature to accept authUri parameter
if no scope is set in ServiceAccountCredentials, use ServiceAccountJwtAccessCredentials instead
to calculate the token for the ServiceAccountJwtAccessCredentials class, use the passed in authUri to encode the JWT. no outbound call needs to be made
add some unit tests

Fixes issue #43

Please review, thanks!

Yes, this looks already done in this case.


Get rid of unused CPI service

Bug Report
Currently, the CPI manifest contains a Service CR which is no longer used by upstream anymore.
https://github.com/vmware-tanzu/community-edition/blob/main/addons/packages/vsphere-cpi/1.22.3/bundle/config/upstream/vsphere-cpi/04-service.yaml
This could cause conflicts when other NodePort service use this port.
Expected Behavior
No service created for CPI
Steps to Reproduce the Bug
Create another NodePort service with port 43001 before creating CPI
Screenshots or additional information and context
Checking the pods under kube-system namespace, we could see that vsphere cloud provider is restarted for multi times.
root@harbor:~[56330]# kubectl --context workload-113-admin@workload-113 get pods -n kube-system | grep cloud
vsphere-cloud-controller-manager-6gl6t 0/1 CrashLoopBackOff 52 4h18m

Checking its details log, it is showing error of "F0721 05:07:22.260682 1 server.go:114] Server Listen() failed: listen tcp :43001: bind: address already in us". That is the reason of workload cluster hits createStalled.
0721 05:07:22.260342 1 clientconn.go:882] blockingPicker: the picked transport is not ready, loop back to repick
I0721 05:07:22.259953 1 reflector.go:175] Starting reflector *v1.Node (0s) from k8s.io/client-go@v0.0.0-20201201102650-cc926379659d/tools/cache/reflector.go:125
F0721 05:07:22.260682 1 server.go:114] Server Listen() failed: listen tcp :43001: bind: address already in use
goroutine 166 [running]:
k8s.io/klog/v2.stacks(0xc0000be001, 0xc00071a540, 0x75, 0xba)
 /go/pkg/mod/k8s.io/klog/v2@v2.4.0/klog.go:1026 +0xb8
k8s.io/klog/v2.(*loggingT).output(0x3ccea00, 0xc000000003, 0x0, 0x0, 0xc000896380, 0x3bed792, 0x9, 0x72, 0x0)
 /go/pkg/mod/k8s.io/klog/v2@v2.4.0/klog.go:975 +0x1a3
k8s.io/klog/v2.(*loggingT).printf(0x3ccea00, 0xc000000003, 0x0, 0x0, 0x0, 0x0, 0x24f740f, 0x1a, 0xc0003b7a70, 0x1, ...)
 /go/pkg/mod/k8s.io/klog/v2@v2.4.0/klog.go:750 +0x18c
k8s.io/klog/v2.Fatalf(...)
 /go/pkg/mod/k8s.io/klog/v2@v2.4.0/klog.go:1502
k8s.io/cloud-provider-vsphere/pkg/cloudprovider/vsphere/server.(*server).Start.func1(0xc000992870)
 /go/src/github.com/k8s.io/cloud-provider-vsphere/pkg/cloudprovider/vsphere/server/server.go:114 +0x13f
created by k8s.io/cloud-provider-vsphere/pkg/cloudprovider/vsphere/server.(*server).Start
 /go/src/github.com/k8s.io/cloud-provider-vsphere/pkg/cloudprovider/vsphere/server/server.go:111 +0x54

Environment Details

Build version (tanzu version): 1.5 and before
Deployment (Managed/Standalone cluster):
Infrastructure Provider (Docker/AWS/Azure/vSphere):
Operating System (client):

Diagnostics and log bundle

Hey @vmware-tanzu/pkg-vsphere-cpi-owners! Are you able to take a look at this and see if the service is no longer needed? And if so, prioritize its change?

Yeah I'm one of the owner, we'll fix this in 1.23

This can be closed


Fix: requeue when managedproxyconfiguration not found.

This PR is to fix the issue when the first reconcile failed because managedproxyconfiguration not found.

/assign @skeeey

/lgtm


Preventing ovmCREATE and ovmCREATE2 Collisions

Description
This little PR prevents ovmCREATE2 from succeeding if the given salt has previously been successfully used by the contract.
Questions
N/A
Metadata
Fixes

Fixes YAS 459

Contributing Agreement

[x] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

Whoops, realized this check will apply to CREATE as well with the addition of ovmSetNonce opcode.  Updating!

Update: since the StateTransitioner does not yet expose proveContractExclusion, we can't get this working in fraud proofs quite yet.  Holding off any merge till we resolve.


[BEAM-1251] Modernize Python 2 code to get ready for Python 3

Signed-off-by: cclauss<EMAIL_ADDRESS>Please add a meaningful description for your change here
Fix Python 3 syntax errors and undefined names.
@aaltay I am confused why it is necessary to keep making these PRs.  I thought that we had put in place automated testing on Python 3 that would use flake8 to flag syntax errors and undefined names so that these anachronisms would not be checked into the codebase.

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---
---

It seems that Lint and Docs failed. You can run them via tox -e py27-lint, tox -e py3-lint and tox -e docs from within sdks/python/

Note that this change conflicts with https://github.com/apache/beam/pull/5373, which fixes some of the same issues.

@charlesccychen It is not so much that these PRs are conflicting.  It is more like they are trying to make many of the same changes.  We have been trying for a while to make this repo Python 3 compatible by running futurize so let's not slow things down by waiting.  Full speed ahead on all efforts.

@superbobry @holdenk Could I please get your review on these changes?

?

Thanks @cclauss!


Fixes sizeForSupplementaryViewOfKind always at index=0

An NSIndexPath is being built from IGListAdapter+UICollectionView.m
before being passed to
-[IGListSupplementaryViewSource sizeForSupplementaryViewOfKind:atIndex:]
however this later is taking the item instead of the section -
resulting in the index always being equal to 0, even for section
indexes > 0.
Changes in this pull request
Issue fixed: #714
Checklist

[x] All tests pass. Demo project builds and runs.
[x] I added tests, an experiment, or detailed why my change isn't tested.
[x] I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
[x] I have reviewed the contributing guide

@gobetti, @rnystrom: This actually fixes the crash Im having.

@MarvinNazari what is the crash?

@rnystrom crash happens when used with ASDK
basically happens when i insert a new section that have UICollectionElementKindSectionHeader and update the adapter.
Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'no UICollectionViewLayoutAttributes instance for -layoutAttributesForSupplementaryElementOfKind: UICollectionElementKindSectionHeader at path <NSIndexPath: 0xc000000000000016> {length = 2, path = 0 - 0}'


change the namespace to lowercase

The camelcase namespace is making issues while listing the az feature, It's better to have a lowercase namespace, streamlined to the brownfield documentation

@vins2win : Thanks for your contribution! The author(s) have been notified to review your proposed change. @caya

@mscatyao - Could you review this proposed update, and change the the article's metadata author value to mscatyao to support automated notification? Thanks.

@KumudD Could you help change the metadata author value in all the author's articles to mscatyao to support automated notification? Usually, this is done by the content team in one pull request in the private repo. (Currently, notifications are going to a GitHub account that appears to be unaffiliated with Microsoft.) Thanks!

Hey @vins2win,
Sorry for the super long delay on this one.
Namespaces and features can be camel case and should not be case sensitive from an Azure CLI perspective.  See CLI implementation here: https://docs.microsoft.com/cli/azure/feature?view=azure-cli-latest
I see both microsoft.ContainerService and Microsoft.ContainerService returned back when making REST calls to the features endpoint of Microsoft.ContainerService.  This would need to be a change from the resource provider vs docs.
I will log some feedback regarding this.
Thank you for reporting!
Jack
#please-close


signup and login page fixes

Have

Inserted App logo in login and sign up page
Improved back to home button and its responsive ness
removed showing 'login' in login page and same case for signup page.

This fixes #479
Screenshots:

@vishanurag Greetings, Kindly review my PR and merge. Thanks.


[Documentation] Add Geo Replicated subscriptions documentation in website

In PIP-33, Geo Replicated subscriptions, feature is added, but documentation is not updated.
related PRs: #4396 #4299 #4340 #4354
This issue tries to track the documentation work.

@Jennifer88huang can you add this to the backlog? It is good to have this documentation added before 2.5.0 release.

Sure, got it.

After reading PIP-33, and discussing with @zymap, we come to the conclusion that snapshot belongs to internal implementation, and users need not know it. If so, I'll not document how snapshot works in the user docs. If there is any misunderstanding, feel free to tell me.
Thing to be confirmed (@jiazhai @tuteng @zymap @merlimat) :

"The snapshots are taken periodically. Proposed default is every 1 second." "The frequency of this snapshot will be configurable in broker.conf" Is the snapshot set by default? Should we tell users the limitations?
Is "replicated subscription" enabled by default? or Is the default value of .replicateSubscriptionState true or false?


Use last command's help/version flags.

Kind of resolves #219.
The issue was that if a command wants to use a parameter like --version or --help, the version/help flag system swallows it and gets priority. Previously, only the help/version flags of the root app were used; now the help/version flags of the resolved command are used (more intuitive/correct).
@orhayat with this PR, you can use version in your command as the following:
# notice the `version_flags`
@app.command(name="print-control-file", version_flags=[])
def print_control(version:Annotated[str|None,Parameter(help="control file version")]):
    x=f"""Package: <package name>
Version:{version}-release
Architecture: amd64
Section: admin
Priority: optional
Depends: python3,lsof
Recommends:
"""
    print(x)

I think I would like for the command to automatically have precedence over the help/version flags, but I don't think that will be possible until cyclopts v3.
Keeping this as a draft as I'm not totally satisfied with the solution; I want to think about it a little more.

alright, so I have decided to go forward with this. Worst case, it's more intuitive before and allows for the very reasonable use-case in #219.
In cyclopts v3 I will probably try to have the command's parameters get priority over version/help-flags.

there is a bit of ??? around which app's version_print and help_print should be used, but I think we can punt on that issue for now.


Error in Using this Graph Transformer Layer on random graph

Hi!
I am trying to use this Graph transformer layer on the random graph (see below). The error is occurring KeyError: 'wV'
import torch
import dgl
import networkx as nx
from model import GraphTransformerLayer (# this imports the layers/graph_transformer_layer.py )
torch.manual_seed(42)
def create_random_graph(num_nodes, node_feature_dim):
g = dgl.DGLGraph()
g.add_nodes(num_nodes)
node_features = torch.randn(num_nodes, node_feature_dim)
g.ndata['feat'] = node_features
return g
num_nodes = 10
node_feature_dim = 16
num_heads = 4
random_graph = create_random_graph(num_nodes, node_feature_dim)
model_layer = GraphTransformerLayer(in_dim=node_feature_dim, out_dim=node_feature_dim, num_heads=num_heads)
output_features = model_layer(random_graph, random_graph.ndata['feat'])


favicon partial

improve favicon markup and move into partial

maybe include this with gulp? https://github.com/haydenbleasel/favicons#gulp

oh yh that looks nice. could be a gulp task outside of the watch/default but part of build


Provide a choco package

For Windows users it would be very convenient to ship mvnd via choco.
jbang already has a choco package, so this might be a good inspiration.

Contributions are highly welcome!

FTR, this is what I found in jbang: https://github.com/jbangdev/jbang/tree/master/src/main/scripts/choco

The Chocolatey package is approved and generally available: https://chocolatey.org/packages/mvndaemon

It does not install a required DLL: #607

It indeed only installs the mvndaemon itself and adds it to the path. When I created and tested the package it worked and it has quite some downloads without any remarks.
Unfortunately at this point in time I don't have a Windows machine available to test the current behavior and if needed change the package. If the mvndaemon team say that the DLL should always be installed and someone can change and test the Chocolatey scripts, then I can release a new version based on that.

With mvnd 1.0.0 being released, this issue deserves some attention.
Maybe it's worth looking a JReleaser which supports building SDKMan, Homebrew and Chocolatey packages. From an external point of view, mvnd seems to be a good candidate for JReleaser.
/cc @aalmiray

Just realized that I pointed to JReleaser some time ago: https://github.com/apache/maven-mvnd/issues/884#issuecomment-1784710330
#623 by @gnodet is related, too.

FWIW I believe mvnd accepted a PR more than a year ago that configured JReleaser for its releases. However, additional auth and vetting from ASF was required for using GH 3rd party actions such as oracle/setup-graal and jreleaser/release-action IIRC.
Perhaps that is no longer an obstacle. Happy to help get this done.

My 2 cents: I no longer maintain the Chocolatey package. It's now maintained by @ajshastri  ajshastri.

I assume so, normally I just ping him like I did above. Otherwise you can contact Chocolatey maintainers via their package website: https://community.chocolatey.org/packages/mvndaemon

@OLibutzki I've updated the package. I've also updated the repo to have issues.

Sadly, mvnd 1.0.0 seems busted on Windows, at least the console output, see #1033 and related issue :disappointed:

FWIW I believe mvnd accepted a PR more than a year ago that configured JReleaser for its releases. However, additional auth and vetting from ASF was required for using GH 3rd party actions such as oracle/setup-graal and jreleaser/release-action IIRC.
Perhaps that is no longer an obstacle. Happy to help get this done.

This hasn't changed.  We still need vote. So we need the release manager to perform some tasks.  But if the packaging for third party installers can be done in a subsequent script, such as in https://github.com/apache/maven-mvnd/blob/master/build/release-publish.sh#L32-L33 (those 2 scripts could be rewritten to leverage JReleaser if it makes sense), that would be nice.


Reactivate appx DBSCAN impl with new disjoint set dependency

Hi everyone!
I saw the issues with the partitions dependency described in #295 and #299, and finally the removal of the appx DBSCAN in #298.
Coincidentally, I recently was looking for a disjoint set crate to use for myself, and when I wasn't satisfied with what I found on crates.io (all existing ones seem to be not maintained and/or not documented/not tested), I decided to publish my own: https://crates.io/crates/disjoint.
So it turned out there were just some small additional features necessary in my crate to enable it as a possible replacement for partitions for you. I implemented these now, and therefore can now provide this pull request to reactivate your appx DBSCAN.
I split the implementation in two commits:

Reverted #298. This just reintroduces the appx DBSCAN impl without any further changes.
Replaced dependencies: partitions -> disjoint.

If you just look at the second commit, you can see that it's really almost a drop-in replacement. Only a few lines change due to the slightly different interface.

Any chance you're willing to run/post some benchmarks? I believe part of the rationale for removing Appx. DBSCAN, beyond the dependency issue, was that it didn't really add any new capability since the implementation itself was actually slower than just standard DBSCAN, leaving little practical reason why someone would choose to use it. @YuhanLiin, any thoughts?

So, I benchmarked my disjoint set implementation itself, as I wanted it to be not slower than existing alternatives, and I wanted to choose implementation details to optimize for speed.
For that, I looked at how fast Kruskal's algorithm runs on an example graph with 1 million vertices and ~8.8 million edges, already sorted by weight, with measures including creating the disjoint set data structure and allocating memory for the result Vec on demand.
The imo most fair comparison to the partition crate is to use DisjointSetVec<()> for that, even though you could use DisjointSet directly (which turns out to not even be faster as Rust handles Vec with size_of() == 0 very efficiently), and to not use the Quick-Union variant of the algorithm, as that's not possible with partition.
With these restrictions, on my local  AMD Ryzen 5 3600, I get the following results:

partition: 290ms +/- 10ms
disjoint: 57ms +/- 1ms

That being said, of course this is a very specific benchmark, and I assume Appx. DBSCAN is a very different algorithm, so this doesn't necessarily mean anything for the performance of that. I don't know anything about what Appx. DBSCAN even is, to be honest (I just drop-in replaced the old dependency with a new one), but my gut feeling would be that the performance of the disjoint set data structure doesn't even matter that much here, as there is so much other stuff happening. But I really don't know and probably someone with more knowledge about clusting algorithms should do benchmarks for that, if you are interested in that.

For a quick comparison can you run the normal DBSCAN and Approx DBSCAN benchmarks and post the results? If Approx is slower than we should just get rid of it.

Codecov Report
Patch coverage: 72.72% and project coverage change: +1.41 :tada:

Comparison is base (4adece5) 37.08% compared to head (8091015) 38.50%.

: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
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #304      +/-   ##
==========================================
+ Coverage   37.08%   38.50%   +1.41%
==========================================
  Files          94       94
  Lines        6196     6218      +22
==========================================
+ Hits         2298     2394      +96
+ Misses       3898     3824      -74

Impacted Files
Coverage Δ

...infa-clustering/src/appx_dbscan/cells_grid/cell.rs
45.90% <ø> (+45.90%)
:arrow_up:

...linfa-clustering/src/appx_dbscan/cells_grid/mod.rs
55.35% <60.00%> (+55.35%)
:arrow_up:

...linfa-clustering/src/appx_dbscan/clustering/mod.rs
48.48% <83.33%> (+48.48%)
:arrow_up:

... and 50 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


WIP: Add molecular units and their behavior functions implementation.

[ ] Implement ϕ (molecule behavior) function.

Coverage increased (+10.0%) to 50.0% when pulling 9beb1c703dac164b1fe8c78f69597489bbceae9e on molecules into e6ef858188e06d1b7ea3e01ca8a5d9256a8178c1 on master.

Coverage increased (+10.0%) to 50.0% when pulling 550521214f1474eb5a4d6683b949cda8f37ee1fc on molecules into e6ef858188e06d1b7ea3e01ca8a5d9256a8178c1 on master.


Extract the Title from patterns that live in the /patterns

In https://github.com/WordPress/gutenberg/pull/36751 Gutenberg added the ability to automatically register patterns that live in the /patterns directory. These patterns provide metadata via headers in the PHP file as in:
<?php
/**
 * Title: My heading
 * Slug: my-theme/my-heading
 * Categories: text
 */

From the existing metadata, we need the Title to be translatable. This PR adds support for looking up those patterns and extracting the Title string into the pot file.
TODO

add test

From the existing metadata, we need the Title to be translatable

For this to be actually useful, Gutenberg should also actually translate the title. I don't see any $pattern_data['title'] = translate_with_gettext_context( $pattern_data['title'], 'Title of the pattern' ); call in https://github.com/WordPress/gutenberg/pull/36751

For this to be actually useful, Gutenberg should also actually translate the title. I don't see any $pattern_data['title'] = translate_with_gettext_context( $pattern_data['title'], 'Title of the pattern' );

Yeah, I need to follow-up with that as well :)

I've prepared the Gutenberg PR that uses this data at https://github.com/WordPress/gutenberg/pull/40047 but I'm having some trouble to get the translated strings.

Pending some tests this looks good to go! 👍

Pending some tests this looks good to go! +1

Tests would need to wait until tomorrow morning :)

@swissspidy pushed the test.


Built-in ProGuard rules (ver. 8.0.0) prevents all names from being obfuscated

It looks like exactly the same problem as described in https://github.com/JakeWharton/butterknife/issues/104
Probably because you are using
-keepnames class *
instead of
-keepclasseswithmembernames class *
Bye.

Send a PR?

Just tried it, doesn't match correctly.

ver. 8.0.0 ,has't it any Change Log ?

Do you mind if I send a PR?


I have the problem when "deeprank-gnn-esm-predict 1B6C.pdb "

2023-08-28 14:39:42,737 predict:218 INFO - Generating graph, using 95 processors
/mnt/data/anaconda/envs/deeprank-gnn-esm-gpu/lib/python3.9/site-packages/pdb2sql/interface.py:147: UserWarning: No contact atoms detected in pdb2sql
warnings.warn('No contact atoms detected in pdb2sql')
Issue encountered while computing graph  /tmp/tmpckjcd5ha/1B6C.pkl
list index out of range
Graphs added to the HDF5 file
Traceback (most recent call last):
File "/mnt/data/anaconda/envs/deeprank-gnn-esm-gpu/bin/deeprank-gnn-esm-predict", line 8, in
sys.exit(main())
File "/mnt/data/anaconda/envs/deeprank-gnn-esm-gpu/lib/python3.9/site-packages/deeprank_gnn/predict.py", line 336, in main
graph = create_graph(pdb_path=pdb_file.parent, workspace_path=workspace_path)
File "/mnt/data/anaconda/envs/deeprank-gnn-esm-gpu/lib/python3.9/site-packages/deeprank_gnn/predict.py", line 223, in create_graph
GraphHDF5(
File "/mnt/data/anaconda/envs/deeprank-gnn-esm-gpu/lib/python3.9/site-packages/deeprank_gnn/GraphGenMP.py", line 141, in init
self._add_embedding(outfile=outfile, pdbs=pdbs, embedding_path=embedding_path)
File "/mnt/data/anaconda/envs/deeprank-gnn-esm-gpu/lib/python3.9/site-packages/deeprank_gnn/GraphGenMP.py", line 247, in _add_embedding
assert not torch.all(torch.eq(embedding_tersor, 0))
AssertionError

Hi there,
It seems to be a problem with pdb2sql, no contacting atoms were found in the structure and it caused the following error. I can not reproduce this error with 1B6C.pdb. May I ask how you got your pdb file and what the version of your pdb2sql package is?

Hi, I get the same error when I just download 1b6c from pdb and my pdb2sql version is 0.5.3.

Hi, Thanks for pointing it out.
I apologize for mixing up the chainIDs in the example. ChainA and chainE are not interacting, which caused the error. I think it should work now.
I also updated the code to support custom input chain IDs. Let me know if there is any issue. @jnoske @liusfore


open file? (feature request)

Sorry if this is misplaced, but I am wondering if I can open a particular notebook by passing a filename at the command line?

I mean that it could just open in the system default browser at a default port

Those are two separate requests:
https://github.com/fonsp/Pluto.jl/issues/236 @Roger-luo this can be added to the CLI right?
https://github.com/fonsp/Pluto.jl/pull/190

Thanks! I tried https://github.com/fonsp/Pluto.jl/issues/236 and left a comment there about what went wrong for me

yes, this should be fairly easy to add to the CLI. I could do it in a PR.

@fonsp just wondering if it's possible to pass the environment path to the notebook process (when creating it) directly? e.g some internal APIs that I can call. I cannot find where Pluto create the process after reading the code, I thought there will be some call to addprocs somewhere...
now since we need to have a new command open in CLI, I will need to create some subprocess myself to set JULIA_PROJECT back (to fix the CLI's deps version, I need to run CLI in a read only julia environment, which is currently the package environment).

ah I think I found it, are notebook process created by this line? https://github.com/fonsp/Pluto.jl/blob/d4be9bfc927c3d7ac12824a31361f7456f6f3ab3/src/evaluation/WorkspaceManager.jl#L85

Yep! The next lines run a number of commands on the new process.

not sure if it should be working yet, but since those PRs on opening from command line merged, I tried it and got
 abradley@gink  ~/Dropbox/Julia/Pluto  pluto open aliasing.jl
ERROR: LoadError: UndefVarError: ServerConfiguration not defined
Stacktrace:
 [1] getproperty(x::Module, f::Symbol)
   @ Base ./Base.jl:26
 [2] open(file::String; host::String, port::Int64, launchbrowser::Bool, project::Nothing)
   @ PlutoUtils ~/.julia/packages/PlutoUtils/5FSCF/src/PlutoUtils.jl:26
 [3] open(file::String)
   @ PlutoUtils ~/.julia/packages/PlutoUtils/5FSCF/src/PlutoUtils.jl:24
 [4] command_main(ARGS::Vector{String})
   @ PlutoUtils ~/.julia/packages/Comonicon/R34G0/src/codegen/ast.jl:202
 [5] command_main()
   @ PlutoUtils ~/.julia/packages/Comonicon/R34G0/src/codegen/ast.jl:46
 [6] top-level scope
   @ ~/.julia/bin/pluto.jl:1

is there something more for me to do to setup? (just went through the readme)

This PR is not yet merged

@AshtonSBradley thanks for trying, and sorry for the inconvenience of the early stage, if you want to try it now, you can do
] add Pluto#master PlutoUtils#roger/cli

we need to wait for Pluto v0.12.0 to get tagged for this feature.

Just a heads up that 0.12 is now actually released :)


[es] #1775 Issue: Localized the software-as-a-service.md document

Describe your changes
Localized the content/en/software-as-a-service.md document into Spanish
Related issue number or link (ex: resolves #issue-number)
Tracking issue: #1775
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@electrocucaracha I am a newbie contributor so If I have done any mistake please give me a review
Thanks a lot for your consideration :)

Thanks for your contribution!
I just realized that this concept of "Software as a Service (SaaS)" is marked as Deprecated in the main branch.
Should we translate these concepts @electrocucaracha? I don't remember well.

Thanks for your contribution!
I just realized that this concept of "Software as a Service (SaaS)" is marked as Deprecated in the main branch.
Should we translate these concepts @electrocucaracha? I don't remember well.

Right, maybe these terms should have the lowest priority until we discuss them with other maintainers.

Thanks for your contribution!
I just realized that this concept of "Software as a Service (SaaS)" is marked as Deprecated in the main branch.
Should we translate these concepts @electrocucaracha? I don't remember well.

Okay no problem😊

@ai-naymul  you need to change it to the branch dev-es,  instructions [here](Contribution guide)

@ai-naymul you need to change it to the branch dev-es, instructions [here](Contribution guide)

I have changed the branch to dev-es :)

Also, add the word(s) missing to the spellcheck file, please.

Also, add the word(s) missing to the spellcheck file, please.

Please check the file now
Thank you :)

Hi @ai-naymul , What is your local language, do you speak Spanish? Regards

Hi @ai-naymul , What is your local language, do you speak Spanish? Regards

No I don't speak spanish, our local language is Bengali and English.


Signature previews alignment in 0D/1D

Sorry for being incredibly pedantic, but I've noticed that the 0D/1D previews in the signature take up ever so slightly less horizontal space than the 2D previews, so things are not aligned correctly (such as the invertible/oriented indicators).

I believe the signature previews should be made to fit inside a square box of a fixed size irregardless of the dimension, similar to the boundary previews that Yulong did.

Think that should do it (issue with max-width vs width).


Main -> Release Service Details Page Find

Merge main into release-service-details-page-find.
This contains the delete release.

Just for my own knowledge, why is there a intermediate main branch rather than going straight from main?

Just a preference thing, also means you can easily fix merge conflicts should there be any


ceph save spec initila config as artifact

These files should be saved for troubleshooting.
As a pull request owner and reviewers, I checked that:

[ ] Appropriate testing is done and actually running
[ ] Appropriate documentation exists and/or is up-to-date:

[ ] README in the role
[ ] Content of the docs/source is reflecting the changes

https://review.rdoproject.org/r/c/testproject/+/50641 not working

@afazekas is this still active and/or relevant?


fix(content-group-simple): multiple storybook fixes

Related Ticket(s)
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/5665
Description
Multiple fixes for content group simple both web component and react stories

@NSDrowned can you update the snapshot file?

@jeffchew working now! 👍


Hiding non significant spanEvent

Thanks for the excellent opensource APM. We started using this in our lower lifecycle env and so for our developers are happy with this. One feedback they give me is, why are you showing all the spanevent that are 0ms . Its not significant at all.
We have a huge codebase and our transaction call a lot of methods, most of them spring. I have a the below setting to track all the springbeans
profiler.spring.beans.1.scope=component-scan
profiler.spring.beans.1.base-packages=com.foo, com.bar
profiler.spring.beans.1.name.pattern=
profiler.spring.beans.1.class.pattern=
profiler.spring.beans.1.annotation=org.springframework.stereotype.Controller,org.springframework.stereotype.Service,org.springframework.stereotype.Component
The base packages are our root package, so it a lot of springbeans involved. Which this setup, what we get is a lot of springbeans with 0 execution time, adding a lot of noice to our transaction details. See the screenshot below (its jar a part of this)

If you see the screenshot above, out of the 12 events, 4 is having a non-zero. If you have a threshold of say 5ms, then we can get rid of all the 12 events reducing a lot of noice. Typically we dont really care about anything that is taking 10ms or less. But this is more of our usecase. Others might like to have other threshold values.
So what will be nice is to have a threshold in pinpoint-env.config which will only send spanEvents only if the goes above the threshold. Is this possible today?
If its not possible, can you give me some hint on how to do it and i can create a PR for this change.

If the Agent discards the SpanEvent, there is a problem that may cause the alignment to be broken.
e.g)
| methodA
\-- methodB sequence : 1  ms :100
\-- methodC sequence : 2  ms:0 <- Discard this
|     \--- methodC-1 sequcen : 3
\-- methodD sequence : 4  ms : 200

There are no SpanEvents 2 and 3, which makes it difficult to sort spanEvents.
| methodA
\-- methodB sequence : 1  ms :100

???

\-- methodD sequence : 4  ms : 200

I'm thinking of a virtual link concept, but it's still an idea.
| methodA
\-- methodB sequence : 1  ms :100
\-- vlink, 2, 3
\-- methodD sequence : 4  ms : 200

Just a suggestion.
I think for the case,
| methodA
-- methodB sequence : 1  ms :100
-- methodC sequence : 2  ms:0 <- Discard this
|     --- methodC-1 sequcen : 3
-- methodD sequence : 4  ms : 200
all should be displayed. Summary, discard spanevent if there is no child with ms that goes over certain amount. So in this case methodC and methodC-1 will be displayed.
If we apply the above criteria for another case in which methodC and methodC-1 will not be displayed.
| methodA
-- methodB sequence : 1  ms :100
-- methodC sequence : 2  ms:0
|     --- methodC-1 sequcen : 0
-- methodD sequence : 4  ms : 200
can't we just display it as below?
| methodA
-- methodB sequence : 1  ms :100
-- methodD sequence : 4  ms : 200
What kind of difficulties are expected with ordering SpanEvents 1 and 4 when there is no SpanEvents 2 and 3?
For a possible better way, taking care of it from FE. Displaying spanevents with a checkbox option also seems nice.

The above example is just an easy explanation.
This is not a simple problem.

Is it lost? Is it omitted?
Difficulty rendering because sequence and depth values are lost together.
What if there is asynchronous call or rpc call in the child node?

Thanks @emeroad  for explaining the challenges. I am confident that you will come with some solution.  Our problem is, our spanEvents for a given transaction is huge, usually around 1000 - 2000 spanevents. It brings in 2 problem. (1) loading it from hbase is very slow. The page load time is a lot. (2) We are are seeing instability in hbase because we dump too much data to it.  As a use who is trying to troubleshoot, which is the primary role of the APM, all these 0 ms entries are not very useful for me to get to the root cause of the problem.
Just a thought.
if methodC and methodC1 is part of a different class and if its not added to the spring.beans configuration, we dont do anything about it and just track it as a GAP in methodD right?
Can we get that behavior if the self execution time does not cross a threshold.

I recommend reducing the range of base-packages.
For now, this is an easy way.

We do our best to organize the packages, but sometimes it becomes hard. For example, we have a util call in our main application package. This util class does small work, but its called many many times. Think of it like StringUtils.isEmpty().. Our stack is filled with this util class. Do we have a way to just exclude 1 or more classes ?


[KYUUBI #6253] Support running JDBC engine on YARN AM

:mag: Description
Issue References 🔗

This pull request fixes #6253
Describe Your Solution 🔧
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Types of changes :bookmark:

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:

Related Unit Tests

Checklist 📝

[ ] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

Codecov Report
Attention: Patch coverage is 60.17699% with 45 lines in your changes are missing coverage. Please review.

Project coverage is 58.88%. Comparing base (a632edc) to head (c384049).
Report is 8 commits behind head on master.

Files
Patch %
Lines

...apache/kyuubi/engine/jdbc/JdbcProcessBuilder.scala
3.57%
27 Missing :warning:

...ain/scala/org/apache/kyuubi/engine/EngineRef.scala
0.00%
8 Missing :warning:

...yuubi/engine/jdbc/JdbcYarnModeProcessBuilder.scala
90.00%
5 Missing and 1 partial :warning:

...i/engine/deploy/yarn/EngineYarnModeSubmitter.scala
0.00%
3 Missing :warning:

...pache/kyuubi/engine/KyuubiApplicationManager.scala
50.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6275      +/-   ##
============================================
+ Coverage     58.48%   58.88%   +0.40%
  Complexity       24       24
============================================
  Files           651      652       +1
  Lines         39513    40163     +650
  Branches       5441     5579     +138
============================================
+ Hits          23109    23651     +542
- Misses        13937    14032      +95
- Partials       2467     2480      +13

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

Just minor comments, LGTM

Thanks, merged to master.


Only clang-tidy files whose content hash changed

clang-tidy takes almost 30min in the CI, so let's remember the
content hashes of all files we processed so that we only have to
look at changed files.
Signed-off-by: Henner Zeller<EMAIL_ADDRESS>

Here is an example where the 30min clang-tidy now only takes 15 seconds as there were no changed files:
https://github.com/chipsalliance/verible/runs/4048584990?check_suite_focus=true#step:7:16


Alfred4 not working

Alfred 4 support was added in v0.7.2. It works fine for me.
If you can't provide a better description of your problem (the debugger output at least), I can't help you.

@deanishe

I solved it

https://github.com/deanishe/alfred-safari-assistant#macos-mojave


Migrate from old url regex urls to new path urls

Note: for support questions, please use the cookiecutter-django tag on stackoverflow. This repository's issues are reserved for feature requests and bug reports. If you need quick professional paid support for your project, contact<EMAIL_ADDRESS>

**I'm submitting a ... **

[ ] bug report
[x] feature request
[ ] support request => Please do not submit support request here, see note at the top of this template.

Do you want to request a feature or report a bug?

Django versions pre 2 used django.conf.urls.url() instances within a projects urlpatterns. The "preferred" way of doing this is to use django.urls.path(), which does not require the use of regex. For any paths which require the use of regex, we can use django.urls.re_path().

What is the current behavior?

The old django.conf.urls.url() still works, but is mapped to django.urls.re_path(), without any warning as it is not yet been marked for depreciation

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

What is the expected behavior?

The new system should use django.urls.path(), or django.urls.re_path() when regex is required

What is the motivation / use case for changing the behavior?

The new path syntax is cleaner, simpler and more readable than the previous regex version.

Please tell us about your environment:

cookiecutter-django with django2

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

https://docs.djangoproject.com/en/2.0/releases/2.0/#simplified-url-routing-syntax
https://docs.djangoproject.com/en/2.0/ref/urls/

Is anyone working on this? I could do this in the next week and provide a pull request, I think.

@saschalalala that'd be of great value to everyone if you could!

Before starting to work on it, just to be clear: Cookiecutter is Django 2.0+-only, right? So there is no need to somehow provide backwards compatibility?

@saschalalala, yes, that's right: Python 3.6, Django 2.0.

Any news on this one, @saschalalala ?

@webyneter Just created a pull request. Sorry, I totally forgot this one m(


fix: the global thread pool data race in unit tests

What ❔
Another attempt to fix the data race in rayon by globally initializing the thread pool.
If it does not work, I'll put it into a thread-safe global variable.
Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[x] Tests for the changes have been added / updated.
[x] Documentation comments have been added / updated.
[x] Code has been formatted via cargo fmt and checked with cargo clippy.

Hey @antonbaliasnikov, I'm wondering if this sanitizer can produce false positives.
In this branch I've added a log before initializing a singleton thread pool, and it is printed only once. However, the sanitizer reports a data race in the thread pool initializer.
Do you agree it can be a false positive?

Hey @antonbaliasnikov, I'm wondering if this sanitizer can produce false positives. In this branch I've added a log before initializing a singleton thread pool, and it is printed only once. However, the sanitizer reports a data race in the thread pool initializer. Do you agree it can be a false positive?

@hedgar2017 , unfortunately, I still see the data race on this branch too :(
I need some time to dig deeper into that to understand if it can be a false positive.

Yes, the log is executed once, and a data race is reported anyway.
It should be a false positive by definition from another reason: there is only one test run, so one thread is executed in the pool.


CIP-1694 | Add acknowledgements for community workshops in french

Added the 2nd list of acknowledgements for the french translation.

From pull request (#578)
revision 430f64d

@KtorZ @Ryun1 @SebastienGllmt @Crypto2099 since we'll be getting more of these in the future, I've updated the description of the Correction label so it includes updates like this (Fixing minor issue or typo; updating translations)... to disambiguate it with the Update label... please feel free to dispute 😎


Twitter weird response get empty out put

Hello,
I get this terminal opt and an empty csv opt file.
GetOldTweets-python-master % python Exporter.py --querysearch '#cat' --since 2020-01-01 --until 2020-01-05
Searching...

Twitter weird response. Try to see on browser: https://twitter.com/search?q=%20%23cat%20since%3A2020-01-01%20until%3A2020-01-05&src=typd
Done. Output file generated "output_got.csv".

Appreciate your help!
Best,
Xu

Hi,
I am also facing with this problem. Is there any solution?