Haste module overrides for react-native-windows resolution failures

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
Run react-native info in your terminal and paste its contents here.
  React Native Environment Info:
    System:
      OS: macOS High Sierra 10.13.3
      CPU: x64 Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz
      Memory: 90.15 MB / 16.00 GB
      Shell: 3.2.57 - /bin/bash
    Binaries:
      Node: 8.9.4 - /usr/local/bin/node
      Yarn: 1.7.0 - /usr/local/bin/yarn
      npm: 6.1.0 - /usr/local/bin/npm
    SDKs:
      iOS SDK:
        Platforms: iOS 11.4, macOS 10.13, tvOS 11.4, watchOS 4.3
      Android SDK:
        Build Tools: 23.0.1, 25.0.3, 26.0.2, 27.0.3
        API Levels: 23, 25, 27
    IDEs:
      Android Studio: 3.0 AI-171.4443003
      Xcode: 9.4.1/9F2000 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.3.2 => 16.3.2
      react-native: 0.56.0-rc.1 => 0.56.0-rc.1
    npmGlobalPackages:
      react-native-cli: 2.0.1

Description
Describe your issue in detail. Include screenshots if needed. If this is a regression, let us know.
react-native-windows overrides haste modules in react-native, currently using the @providesModule attribute. For example, the ScrollView module in react-native has behavior that limits it to iOS and Android, we we override the module in react-native-windows here.
Prior to react-native 0.56.0 RC (which moved to metro 0.38), this behavior worked as expected. Now, the haste module overrides for windows no longer seem to get resolved. Specifically, I get the following error:
Unable to resolve module `AccessibilityInfo` from `/Users/rozele/code/sandbox/v56/node_modules/react-native/Libraries/react-native/react-native-implementation.js`: Module `AccessibilityInfo` does not exist in the Haste module map

This might be related to https://github.com/facebook/react-native/issues/4968
To resolve try the following:
  1. Clear watchman watches: `watchman watch-del-all`.
  2. Delete the `node_modules` folder: `rm -rf node_modules && npm install`.
  3. Reset Metro Bundler cache: `rm -rf /tmp/metro-bundler-cache-*` or `npm start -- --reset-cache`.  4. Remove haste cache: `rm -rf /tmp/haste-map-react-native-packager-*`.

Please note - the AccessibiltyInfo module just happens to be the first module in the dependency graph that we provide a *.windows.js override for, so it has nothing to do with that module specifically.
Reproducible Demo
Let us know how to reproduce the issue. Include a code sample, share a project, or     share an app that reproduces the issue using https://snack.expo.io/. Please follow the guidelines for providing a MCVE: https://stackoverflow.com/help/mcve
A very simple test that repos (on both Windows and Mac):

Generate a new react-native project with 0.56.0-rc.4:

react-native init test --version 0.56.0-rc.4

Generate a react-native-windows project:

yarn add rnpm-plugin-windows
react-native windows --windowsVersion 0.55.0-rc.0

Try to generate a windows platform bundle

react-native bundle --platform windows --entry-file index.js --bundle-output test.bundle

Thanks for reporting this. In order to prevent having to override the hasteImplModulePath configuration in end-user code, we're going to add an exception for react-native-windows in react-native. I'm working in a fix right now.
Meanwhile, we're going to work in a long-term solution for overriding core components from plugins, as the current approach is pretty hacky.

@rubennorte - definitely agree that a less hacky long term solution is ideal. I've never liked the "special case" of react-native-windows in the metro / react-native repos.
That being said, my parameters for an ideal solution would be one that allows me to install react-native-windows and have it "just work" without changes to the rn-cli.config.js. We have a precedent for plugging into the react-native link command via a bit of config in the package.json file here. If we could have something declarative like that for metro / haste modules, that would be ideal.

@rubennorte I'm getting the same 'unable to resolve module' error for 'react-native-elements'  😕

Hey @rozele is this still an issue or 0.56.0 fixed it?

this => Unable to resolve module AccessibilityInfo is still an issue for 0.56.0, compiling for android on windows platform .... what is going wrong ? this is a major issue....

I got the same issue on Windows 10 :(

Same issue here! Trying to compile RN 0.56.0 for Android.
Here it's the react-native info output:
React Native Environment Info:
    System:
      OS: Windows 10
      CPU: x64 Intel(R) Core(TM) i5-6200U CPU @ 2.30GHz
      Memory: 1.49 GB / 3.88 GB
    Binaries:
      Yarn: 1.5.1 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
      npm: 5.6.0 - C:\Program Files\nodejs\npm.CMD
    IDEs:
      Android Studio: Version  <IP_ADDRESS> AI-173.4819257

I followed the same steps in this issue and then yarn upgrade<EMAIL_ADDRESS>The error I'm getting after react-native bundle --platform windows --entry-file index.js --bundle-output test.bundle is:
Unable to resolve module `ViewContext` from `.../ReactNativeWindowsTestNew/node_modules/react-native-windows/Libraries/Image/Image.windows.js`: Module `ViewContext` does not exist in the Haste module map

Which is normal because ViewContext doesn't exist anymore in React Native.
This isn't related to this issue, as the hasteImpl/providesModule thing has been solved for react-native-windows.

@rubennorte according to the other issue I linked (#20015) the error Unable to resolve module AccessibilityInfo is present in projects that are not strictly using react-native-windows (and there is also a related metro issue open atm -> https://github.com/facebook/metro/issues/181 )

React Native Environment Info:
System:
OS: Windows 7
CPU: x64 Intel(R) Core(TM) i3-3210 CPU @ 3.20GHz
Memory: 3.06 GB / 7.71 GB
Binaries:
Yarn: 1.7.0 - C:\Users\sbsgroup\AppData\Roaming\npm\yarn.CMD
npm: 5.6.0 - C:\Program Files\nodejs\npm.CMD
IDEs:
Android Studio: Version  <IP_ADDRESS> AI-173.4819257

react-native-cli: 2.0.1
react-native: 0.56.0

We need 0.56 release work without module not found issue on android emulator in windows .

I would like to suggest that the issue detail be updated to reflect the fact that the same issue is happening on Windows 10 when compiling for Android. I spent two days troubleshooting as this happened during my very first react native project setup :(

@rubennorte Please, this is still an issue and it's becoming really frustrating. I had to downgrade to 0.55.4, it worked but there are vulnerabilities that npm suggests npm install react-native @0.56.0 will fix. Going back to that version will return the same "AccessibilityInfo" bug.
What can be done?

Very frustrating, a stable version with a bug and no attention and explanation for a fix 👎

I was able to resolve this by downgrading to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

downgrading isnt really resolving a issue, the originale issue stated that everything worked prior to 0.56

Everything working with 0.55 upgraded to 0.56 and babel it fails :/ I need 0.56 for some bug fixes :/

A drop grinds the stone. They are working on it. React-native-windows/skype devs are also strugling. I saw one complaining in a related issue. Facebook can't betray Microsoft 👌

Same question, I'll almost give up React-native

0.56 is broken on windows. Hopefully, this gets fixed soon!

Same error here, it's okay to have bugs sometimes, but what can we do about it ?
Isn't there some lines we can just add to the config files so that it can work for now ?

I'm locking this since the comments are all but useful in helping with solving this issue.
Aside from that, a commit landed in master with a fix: https://github.com/facebook/react-native/commit/3ac86c366c91f8d62f0128057019b94a783b4249 and we'll make sure to cherry pick it and release a 0.56.1 release asap.

Update:
the TLDR is that the original issue causing this was nested into a dependency of Babel. After having tested various workarounds in the past weeks, in collaboration with the Metro team we agreed that (sadly) it's really complex to backport the "true fix" to 0.56.
We are cutting a 0.57 rc0 in these hours in order to make a new version available to everyone asap, and test that in fact 0.57 will solve this (related: https://github.com/facebook/react-native/commit/583cb137b8e7a63486766f6ef242d7214bf24f2e ).
I'll try to update everyone as soon as we have better info related to this.
We are also trying to avoid situations like this in the future by integrating a some Windows tests in the CI, you can follow along and help and provide feedback about that here: https://github.com/facebook/react-native/issues/20260


Ivis Transformer Implemented

As requested in the #35 issue, I added Ivis transformer.
There are two arguments (k and batch_size) which depend on the size of the data. That's why I decided to fill them separately.
If there are any complaints, I'm ready to modify the code.

@koaning ,
I believe all errors are from another files. I can fix them if you want.

Thanks for submitting a pull request 🚀 @tttthomasssss will take a look at it as soon as possible ✨

Mhmm ... that's strange. Feel free to fix, but more-over: we need to add some tests.

Another thing: the ivis dependency needs to be added to the setup.py file as a dependency.

Mhmm ... that's strange. Have you been forking from a recent master?
More-over: we need to add some tests. It'd be preferable if you could copy some of the tests from other transformers to also work for this one.
I recall having some issues in real life with this transformer in terms of installation, so I'd be good to confirm this.

About style errors, this file really has some long lines (line 11 and 19)

I tried to run tests but I have some problems. Am I doing it right?

There's a discussion to be had on fill_arguments.

I get what you're saying. I just thought that if the size of the embset would be less than default values of k or batch_size (128 and 150), users would have to specify these values themselves. But it's okay I guess.

Mhmm ... that's strange. Have you been forking from a recent master?
More-over: we need to add some tests. It'd be preferable if you could copy some of the tests from other transformers to also work for this one.
I recall having some issues in real life with this transformer in terms of installation, so I'd be good to confirm this.

About style errors, this file really has some long lines (line 11 and 19)

I tried to run tests but I have some problems. Am I doing it right?

Oh I get it now, I was doing it wrong

If the tests are having a hard time locally: have you run make develop locally? You can check the actions in the Makefile. The idea is that we prepare the tests such that we don't have to download Gigabytes of embeddings.

@sprintyaf it seems like you're not running black locally. This is probably due to the fact that you didn't start the pre-commit hooks.
If you're unfamiliar with either of them, you may appreciate these tutorials on my blog on pre-commit hooks and the black formatting tool.

@sprintyaf it seems like you're not running black locally. This is probably due to the fact that you didn't start the pre-commit hooks.
If you're unfamiliar with either of them, you may appreciate these tutorials on my blog on pre-commit hooks and the black formatting tool.

will do! thanks for the tutorials

due to some problems, I created a new pull request #106
I'll close this one.


ModuleNotFoundError: No module named 'carto.auth'; 'carto' is not a package

Hello,
I followed the instructions and did run for the script but I keep getting the following error
ModuleNotFoundError: No module named 'carto.auth'; 'carto' is not a package
What I m missing?
Please advice.

Issue fixed.
Renamed the file from carto.py to another name

I don't understand the "fix". We're seeing this same error:
    from carto.auth import APIKeyAuthClient
ModuleNotFoundError: No module named 'carto'


Remove Python 2.6 (follow up to 87880158)

Coverage remained the same at 100.0% when pulling 48fd4a1f7e6e91f68a306c3054ec239ed82f79f0 on techtonik:patch-5 into 878801586aea075f70c9ebca31da825ad4505915 on WoLpH:develop.

Coverage remained the same at 100.0% when pulling 48fd4a1f7e6e91f68a306c3054ec239ed82f79f0 on techtonik:patch-5 into 878801586aea075f70c9ebca31da825ad4505915 on WoLpH:develop.

This has completely slipped my attention... apologies @techtonik
Thanks for the help!


Database: includes and excludes for types

Allow to configure set of include and exclude path patterns which are applied to DatabaseConfiguration.getDbObjectConfiguration().
Example:
PERSISTENCE/excludeType[0]=xri://@openmdx*org.opencrx.kernel.depot1/provider/($..)/segment/($..)/booking/($..)/($..)/($..)/($...)
PERSISTENCE/excludeType[1]=xri://@openmdx*($..)/provider/($..)/segment/($..)/indexEntry/($..)/($..)/($..)/($...)

fixed with opencrx-4.2.0+


featureadd url functions: domain and protocol

Proposed changes
Issue Number: close #13661
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[x] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[x] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 37.36 seconds
load time: 567 seconds
storage size:<PHONE_NUMBER>6 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221025232912_clickbench_pr_33979.html

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.35 seconds
load time: 567 seconds
storage size:<PHONE_NUMBER>2 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221027232704_clickbench_pr_34892.html

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.28 seconds
load time: 588 seconds
storage size:<PHONE_NUMBER>3 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221028121709_clickbench_pr_35095.html

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.59 seconds
load time: 574 seconds
storage size:<PHONE_NUMBER>8 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221029040802_clickbench_pr_35540.html


Search term is not trimmed

Hi!
It would be great if we can get the ability to trim() the searched text from whitespaces.
Or Is it possible now? If yes, how?

@frikinside I made the changes for this to work. If you add me to the collaborators I will create a branch and I will open a Merge Request. Thank you!


adapt to newer hspec-wai WaiSession type

Not sure if this is useful or interesting in general, but I have "upgraded" types to use newer hspec-wai WaiSession type that takes a state as a parameter.

We'll probably wanna bump this package's version and put a bound on hspec-wai to capture that cutover.

The code looks good to me aside from that though!

I am investigating a different avenue whereby one would use a different (non IO based) monad than ClientM to run servant endpoints against a WaiSession.


Request to merge

Please merge

Peter, your commit includes 6 files. Please split these changes into different branches and open a separate pull request per change.
For instance, a branch adding-name for adding your name. Another branch modifying-readme for making changes to the readme, etc. 🙏

Your file changes should just be one file names.txt and it should contain a single change with just your name.

Please can you close this PR and create another one?

Please close and create new one. The file changes here is too much. If you don't know how kindly read the README.

@KingCephas2022 the changes in this PR is too much and may not be intentional, pls kindly close this PR and create a new one (editing only the name.txt) as advised. NB the README can guide you on how to.
If you let me, I can equally assist you in closing this PR maybe if you can't do that from your end.


[lab] Use the public API for module augmentation

A follow-up on https://github.com/mui-org/material-ui-x/pull/2307#discussion_r684743848. I assume that the author of this PR has copyed our source.

Branches are located on this remote not your fork

Oops sorry, it has happened in #27728 too. It happened because my .git env was locally not working. I had rm -rf the whole folder but forgot to reconfigure it correctly. Regarding why my .git env was locally not working, it was related to filename case sensitivity. I pushed f715fa6ddc9f94cd8f14916963c2cc1f2bf9b7d5 on HEAD to fix it.


Rename/refactor JSNativeApi class

The JSNativeApi class consists almost entirely of extension methods for JSValue. Either it should be renamed to JSValueExtensions, or the methods should be moved into the JSValue class itself.

Fixed by #187


Allow recording "application/x-www-form-urlencoded" as text

Currently is does not match in this method: https://github.com/betamaxteam/betamax/blob/master/betamax-core/src/main/java/software/betamax/tape/MemoryTape.java#L290
I have tested message.getBodyAsText() and it gives correct string -- name/value pairs are separated by the ampersand (&)

Next week I will try to make a pull request


copy.deepcopy(model), why?

Hello, your project has helped me a lot. Thank you very much. But I have a question: why do I need copy.deepcopy(model) when I am trying to implement a federated learning  model, it seems that without copy.deepcopy all models will have the same weight. It's only when you use it that the model is different. So why is that?

Hi,
Can you provide the filename together with the corresponding line number? y = copy.deepcopy(x) returns y which is a copy of variable x. Any future operations on x will not affect y.


Compatibility With Other Cookie Managers

In fact, I've tried a lot of extensions for importing and exporting cookies. Milk is one of the best. But if we talk about custom research, then I, as a product manager, immediately saw the pain. A problem that, according to my theory, many users have many extensions  and not all is compatibility.
Yeap, is import and export compatibility between different plugins.
Every developer declares - I don't want to hear anything, my extension only works with my files.
Ha, why not break this law?
But this is one of the serious problems.
Now importing milk is one of its strengths, it imports many other cookies from other extensions. Not all. But a lot.
But the export is not very good. Many extensions do not understand milk cookie files.
I think that just the support of other popular extensions would be the feature that solves this problem. Many people encounter it, but of course it is better to confirm my hypothesis with custdev.
Example, also good to know learning MILK understood error in cookies like onlyhttp (only Http)  (as capital letters) and fix/ still import files with this errors too.

Thank you! I really appreciate the fact that you like MILK and want to make it better! Importing cookies got even better in the upcoming release, you can preview it in the next branch.

Yeap, is import and export compatibility between different plugins.
Every developer declares - I don't want to hear anything, my extension only works with my files.
Ha, why not break this law?

MILK exports the cookie in the EXACT format that Chrome returns via their API. See the chrome.cookies API specification for details about the structure of the cookie data. This is the only format that makes sense to me to use to export. Why would anyone change it? I understand that other cookie managers decided to export it in another format, but why should I go against Chrome's specification?
I would like to know the use case of using multiple cookie managers, why not use just one? Why not just use MILK :)
I suspect that the difference is that other cookie managers include the "url" property in their exported data, which is redundant and can be constructed on the fly with the other information that is included in the cookie data.
I want to help you. How about you give me a list of cookie managers that you want it to work with and I will look into it for you and see what I can do. No promises because I do not really want to change the format because it will probably make the exported file larger but I am willing to take a look for you. Maybe I can make it into an option in the options page.
Anyways, thanks for reaching out and I look forward to hearing back from you soon!

I would like to know the use case of using multiple cookie managers, why not use just one? Why not just use MILK :)
its easy. Like i say before (in another topic) someone import cookie to chrome, someone to firefox. Example incognito mode not work with import cookie in chrome for ANy plugin, just firefox works.

also you forget that someone can use another plugin and export there and transfer the file to me. I'm using milk and can't import it.
Why? I dont know. Some of the plugins do the wrong export, some minify the version, some encode cookies into their own format so that no one can look inside. Someone for other reasons. Here you have to look at each separately. At least 3-5 of the most popular apps.
Example yestoday my friend ask me - mmm Alex (my name) but how i can import cookies to my mobile brower? I know every apps save cookies to DB/storage/memory but what about mobile browser? Chrome or Firefox?
And he is right. Iv googling this and not found nothing for chrome and someone for firefox but not works for lastes firefox version.

How about you give me a list of cookie managers that you want it to work with
sure, iv tested it again in next days and post here.

ohh i remember. Another season for Compatibility. Please tell me here is no any standards for cookie files? I mean for opera, firefox, chrome? What is happened if iv import cookie files generated by MILK chrome to Firefox Cookie-Editor?

ohh i remember. Another season for Compatibility. Please tell me here is no any standards for cookie files? Not just chrome API but univeral meta...like HTML5 or CSS3

Not sure what you mean by HTML5 or CSS3, but there is no official standard for this unfortunately. That being said the format that Firefox and Chromium based browsers like Chrome and Opera use are the same:

https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/cookies/Cookie
https://developer.chrome.com/docs/extensions/reference/cookies/#type-Cookie

Let me know when you have the list of extensions and I can see what I can do for this issue.

Hey @miraserver, any progress on compiling a list of cookie managers for me? Please let me know, thanks!

Hey @miraserver, any progress on compiling a list of cookie managers for me? Please let me know, thanks!

Closing due to inactivity.

Closing due to inactivity.


core dumped while running recon-all

We're getting a core dump and not sure what to do about it.
We start with:
export SINGULARITYENV_TEMPLATEFLOW_HOME=/home/fmriprep/.cache/templateflow
/cm/shared/singularity/bin/singularity run \
    -B /data:/data -B /data1:/data1 -B /data2:/data2 -B /data3:/data3 -B /cm/shared:/cm/shared \
    -B /home/ajanes/.cache/fmriprep:/home/fmriprep/.cache/fmriprep \
    --cleanenv \
    /cm/shared/singularity/images/fmriprep-1.5.0.simg \
    /data1/ajanes/Axial \
    /data1/ajanes/Axial/derivatives \
    participant \
    --fs-license-file /cm/shared/freesurfer-6.0.1/license.txt \
    --participant_label 005 \
    --output-spaces "MNI152NLin2009cAsym:res-2 anat func fsaverage" \
    --n_cpus 4 \
    --mem-mb 8192 \
    --notrack \
    --use-aroma --ignore-aroma-denoising-errors \
    --use-syn-sdc \
    -w  /data1/ajanes/fmriprep-working

and it runs until:
CORRECTING DEFECT 37 (vertices=24, convex hull=24, v0=172721)
After retessellation of defect 37 (v0=172721), euler #=244 (46784,129300,82760) : difference with theory (0) = -244

CORRECTING DEFECT 38 (vertices=188655, convex hull=16433, v0=177560)
Segmentation fault (core dumped)
Linux node4 3.10.0-693.el7.x86_64 poldracklab/fmriprep#1 SMP Thu Jul 6 19:56:57 EDT 2017 x86_64 x86_64 x86_64 GNU/Linux

recon-all -s sub-005 exited with ERRORS at Thu Oct 10 15:22:20 EDT 2019

For more details, see the log file /data1/ajanes/Axial/derivatives/freesurfer/sub-005/scripts/recon-all-rh.log
To report a problem, see http://surfer.nmr.mgh.harvard.edu/fswiki/BugReporting

Standard error:

Return code: 1

191010-15:22:24,104 nipype.workflow ERROR:
	 could not run node: fmriprep_wf.single_subject_005_wf.anat_preproc_wf.surface_recon_wf.autorecon_resume_wf.autorecon_surfs
191010-15:22:24,105 nipype.workflow ERROR:
	 could not run node: fmriprep_wf.single_subject_005_wf.anat_preproc_wf.surface_recon_wf.autorecon_resume_wf.autorecon_surfs

recon-all-rh.log
fmriprep.o433134.txt

We can close this, this definitely was (a) a freesurfer issue and (b) a data issue.

How did you solve the error?


Add junotest package

Problem: It's tedious to add test cases all the time, which leads to poor testing practices (e.g., duplicating the same test case in many places in the codebase).
Proposed solution: add a junotest package that exports test fixtures and helper functions. These can be called from tests in other packages. If this solution is implemented, we need to modify the Makefile to ensure the junotest package is not included in the final binary: it should exclusively be used for testing.

A little different than the proposed solution, but the problem is now solved by #532


Drop on address bar should not be allowed

Hi there and thanks for making this great directive.
I noticed that I can drop items on the browser's address bar and in Chrome this redirects the user away from your page.
Firefox and IE behave a little better - you can still drop on the address bar, but they won't redirect the user.
I had a look at angular-ui-tree and it does not allow you to drop on the address bar. In fact it looks like you can't drag items out of the viewport at all.
It would be great if the drop event was cancelled if the user (accidentally) drops on the address bar.
Thanks

Nothing we can do here with native drag & drop, unless IE starts supporting other mime types than text/plain. If you don't care about native drag & drop, you can use angular-ui-tree, it's a great directive as well.

Ah - gotcha. Thanks for the explanation.


OIC C API: server not able to define path/href

The OIC server C API call sol_oic_server_add_resource(sol_oic_resource_type *, ...) is used to define the resources the OIC server has. However, the API does not allow the app/user to define what the path (or href) of the resource is. Instead, the implementation auto generates the path (/sol/0, /sol/1, etc).
The OIC server should be able to define what the path of it's resources are, like /a/light, /room/door. The current API does not allow this to be possible.

We have a pull request adding an API for that. take a look at #1410

The PR seems to address this case. Once landed we can close this.


feat: add separate status and help filtering

Amazing as always! Thank you very much 💪💪👍

You're welcome! Hopefully this results in more attraction


(JS-0242) Use const declarations for variables that are never reassigned

Description
Variables that are not re-assigned are referred to as constant variables. So, we recommend using const for such variables instead of let to define because the value of a variable declared with const remains the same within its scope. It cannot be updated or re-declared and also reduces cognitive load …
Occurrences
There are 19 occurrences of this issue in the repository.
See all occurrences on DeepSource → deepsource.io/gh/mybot-rblx/FunBlox/issue/JS-0242/occurrences/

For pull request #6


Kfdef v1

Related: #73
Adding KfDef v1 - mostly copied from v1beta1.
Please let me know if you think we need to update the definitions!

This change is

Shall we take care this: https://github.com/kubeflow/kfctl/blob/master/pkg/apis/apps/kfconfig/types.go#L53

/test all

Shall we take care this: https://github.com/kubeflow/kfctl/blob/master/pkg/apis/apps/kfconfig/types.go#L53

that's KfConfig.  but yea we should do that as internal code improvement.

Please let me know if you think we need to update the definitions!

What does update definitions mean?
Are there any changes to the KFDef spec relative to v1beta1.
/lgtm
/approve
/hold
Assuming there are no API changes relative to v1beta1 you can go ahead and remove the hold.

/hold cancel


Downloader GUI: Replacing black font color with blue to make the links readable

The change will ensure that links are visible in both dark and light mode for macOS.
Issue
The downloader when used on macOS's dark mode has the text with foreground color as black and the background as dark grey, which makes it difficult to read and makes the text quite unreadable. Also difficult for color blind people. As can be seen in the image below entry boxes for URL & download_dir have quite an unreadable text (highlighted by red arrows).

Not a problem when using the light mode though

Solution
Replace the existing black value with (blue)#007aff for the disabledforeground attribute
Advantages:

Add support for color blind people.
The new color- blue (#00faff) is compliant with Apple's human interface guide; and is named by Apple as systemBlueColor.

Proposed change

On Windows and Ubuntu
These two operating systems have downloader application in light mode only.

Windows existing
Windows proposed

Ubuntu existing
Ubuntu proposed

Hello!
This looks great. Thank you for the change and the detailed PR.
However, I do have one issue: selecting the text turns it black again, as you can see here:

This is because the foreground value is not changed from the default of black in your PR. See here for some more info on Entry. If we also update foreground, then I believe it should work perfectly.

I had some issues pushing my 0e2d61129982fa1cf0d53f4b8beafd8f3e87bca9 to your branch, and I made a mistake with the push here. My bad. That said, it's all merged now (despite this saying "closed" instead). You can see your commit in https://github.com/nltk/nltk/commits/develop, for example.
Thank you for this work!

Thanks a lot @tomaarsen. Glad that it eventually got a go ahead.🤝


fix paths in update_infos_to_v2 on Windows by using pathlib

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Fix update_infos_to_v2 functions to work with Windows paths.
Modification
Use pathlib instead of relying on split('/') which does not work for Windows paths.
BC-breaking (Optional)
Nothing should break since it is only contained inside the functions.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

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

@vavanade  Thanks for your first PR~ Please sign the CLA.

Dear vavanade,
First of all, we want to express our gratitude for your significant PR in the MMDet3D project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
If you are Chinese or have WeChat，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. We look forward to seeing you there! Join us ：https://discord.gg/raweFPmdzG
Thank you again for your contribution❤
Best regards！ @vavanade


Getting Error while trying to install laraadmin

Class 'Yajra\Datatables\DatatablesServiceProvider' not found
Laravel version : 5.8.16

do you have installed yajra datatables ??
I am not contributor of the project as you can see the
Latest commit b8557a2  on Jan 19, 2018
Which says that the version of package is no longer maitained


Only the "maxReplicase" property shows in the opened Scaling.json file

OS: All
Build Version: 20220405.6
Regression: Not a regression
Repro Steps:

Expand one container app -> Right click "Scaling" node -> Select "View Properties".
Check whether all properties show in the opened Scaling.json file or not.

Expect:
All properties show in the opened Scaling.json file.

Actual:
Only the "maxReplicase" property shows in the opened Scaling.json file.

Thanks for paying attention to this, but this is intended behavior. When the minReplica isn't specified, we just display it at the minimum value possible which is 0.
I'll close this as by-design.


Update onboard-windows-10-multi-session-device.md

Changed old name to Defender ATP as Microsoft Defender for Endpoint was stated as old and new name.

Thank you, that note should be removed completely (already doing that).


python3 compatibility fix - print as a keyword replaced with print() …

python3 compatibility fix

Will this be compatible with Python 2.x as well?  We could import print_function from future to work with both perhaps.  I'm not able to test this unfortunately.

For python versions that don't support the print as a function, it'll still work fine.
print(x) would be interpreted as print (x), so the parenthesis don't make any difference.
print(x, y) would be interpreted as print (x, y), so it'll print a tuple, which although slightly different is close enough for lots of purposes.


Atoms

implement different atoms in store ui.
An atom is considered implemented when:

It has a .test.ts file alongside the component implementation
It has a .stories.ts file alongside the component implementation containing
a. How to style it
b. Basic doc on the purpose

A good inspiration on the atoms to create: storefrontui

To styling the components we want to use data-attributes like the Reach UI components.

Here's a list of atoms I think we should have, beyond those already implemented:
Atoms
Bullets
Bullets should indicate where the user is located on a list, like in a slider, for example.
Page View
Should arrange items side-by-side and allow for flexibility on content, size, and item dimensions. Every item is a page. We could use slider-layout as inspiration, but we want it less opinionated, to have only the slide logic (no buttons or bullets), one page per item, and less IO bloated.
I like Flutter's API a lot.
Price
Should display prices with different attributes. strikedthrough or installments, for example.
I like Flutter's API a lot.
TextArea
We should offer a solution for TextArea similar to our current Input one.
Label
I think we should offer a really simple label component just to help users with SEO and accessibility, but that's debatable.
Ratings
Should display a list of user-chosen icons for positive and negative ratings. The amount of icons is customizable. Should we support fraction ratings?
Product Specification
Not sure how to provide this, but I feel we should. I think we should discuss this. How to atomize a table in a sensible way?


CLASSIC minting: Adding NFT to collection without listing it for sale (not working)

Is your feature request related to a problem?
Classic minting: Adding NFT to your collection that already exists but not listing it for sale is not working.
I wanted to mint it with no price (so it would not be for sale). Not possible because:

or

I could not proceed.
Describe the solution you would like
If the price is 0 or nothing we should still be able to mint that NFT. In this case, it will just not be listed for sale, but you will be able to see it minted in the collection.
Describe alternatives you have considered
Nope
Screenshots
No response
Are there opened related issues?
No response

✌


Problems with responsive elements not snapping in place after browser resize

Hi!
I posted this in the at the Jeet.gs repo: https://github.com/mojotech/jeet/issues/461
Anyone know if this can be a sass-mq issue?
Thanks :)

I'm sure this is not a problem with sass-mq and is more likely to be user error.

Hi @coffeeneed, thanks for using sass-mq!
Not sure this is related to sass-mq directly but can you please share a reduced demo of the bug so we can have a look at it?

Closing this issue as it hasn't been updated in about a month. Feel free to reopen if you are still encountering this problem.


Parse tagged newick strings

Tags no longer show up in node names. They are now added to node elements as a key with value true, similar to how branch selection on the main application works. Main application can parse and visualize trees with a single tag called "Foreground".

See the following movie, which was created with this newick string:
(((EELA:0.150276,CONGERA:0.213019):0.230956,(EELB:0.263487,CONGERB:0.202633):0.246917):0.094785,((CAVEFISH{Foreground}:0.451027,(GOLDFISH{Foreground}:0.340495,ZEBRAFISH{Foreground}:0.390163){Foreground}:0.220565):0.067778,((((((NSAM:0.008113,NARG:0.014065):0.052991,SPUN:0.061003,(SMIC:0.027806,SDIA:0.015298,SXAN:0.046873):0.046977):0.009822,(NAUR:0.081298,(SSPI:0.023876,STIE:0.013652):0.058179):0.091775):0.073346,(MVIO:0.012271,MBER:0.039798):0.178835):0.147992,((BFNKILLIFISH:0.317455,(ONIL:0.029217,XCAU:0.084388):0.201166):0.055908,THORNYHEAD:0.252481):0.061905):0.157214,LAMPFISH:0.717196,((SCABBARDA:0.189684,SCABBARDB:0.362015):0.282263,((VIPERFISH:0.318217,BLACKDRAGON:0.109912):0.123642,LOOSEJAW:0.3971):0.287152):0.140663):0.206729):0.222485,(COELACANTH{Foreground}:0.558103,((CLAWEDFROG{Foreground}:0.441842,SALAMANDER{Foreground}:0.299607){Foreground}:0.135307,((CHAMELEON{Foreground}:0.771665,((PIGEON{Foreground}:0.150909,CHICKEN{Foreground}:0.172733){Foreground}:0.082163,ZEBRAFINCH{Foreground}:0.099172){Foreground}:0.272338){Foreground}:0.014055,((BOVINE{Foreground}:0.167569,DOLPHIN{Foreground}:0.15745){Foreground}:0.104783,ELEPHANT{Foreground}:0.166557){Foreground}:0.367205){Foreground}:0.050892){Foreground}:0.114731):0.295021)

@stevenweaver The main viewer now handles multiple annotations. The following files should be useful for verification.
single-tag.txt
two_tags.txt
three-tags.txt

Hi Stephen,
It looks like this commit causes phylotree to fail when it previously wasn't. The tree successfully parses with ETE toolkit and throws an exception at line 860.
broken.txt
Best,

Hello @stevenweaver ,
I am having a hard time reproducing the error... when I check out the last commit and try to load your tree in the main application, my experience seems as it should... please see the attached image. Can you provide more information on exactly how your failure occurs?
Thanks,


Fixed #18 (will improve code later)

The fix is a bit hacky. To improve the code I'll need to change the recursive function and remove finalPath from the callback, but that breaks del when deleting array indices. Not sure what to do about that. Maybe it can check whether the last element in path is a number and handle that separately.
It passes the tests; I changed the tests where it was "returns the original object if passed an empty object".
There are also a few changes in test.js

Thanks for this. It looks good. I'll make some minor improvements and create a new major release.


opt print current path when encountering error

Proposed changes
Like this:
mysql> select * from text_deflate;
ERROR 1105 (HY000): errCode = 2, detailMessage = (<IP_ADDRESS>)[CANCELLED][INTERNAL_ERROR]cur path: hdfs://ns/usr/hive/warehouse/yytest.db/text_deflate/test2.snappy. Compressed file has been truncated, which is not allowed

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


ISIS-1616 - support for "As-is" HTML rendering

introducing a new value type: Markup
Markup holds immutable HTML
Markup is applicable as action result and as property type

Demo snipped:
@javax.jdo.annotations.Column(allowsNull = "true")
@Property(editing = Editing.ENABLED)
@Getter @Setter
private String html;

public Markup getMarkup() {
	return new Markup(html);
}

@Action(semantics = SemanticsOf.IDEMPOTENT)
public Markup calcMarkup() {
	return getMarkup();
}

Fix for ISIS-1715 menuOrder not honored.
https://issues.apache.org/jira/browse/ISIS-1715

Thanks for this, Andi, will try to review and get merged soon.
Quick request: in future, could you raise separate PRs for separate features, rather than adding commits to an existing branch?  I know it's more effort your side, but it helps keep the git history clean.
Thx again
Dan

Thanks Dan, and sorry for the single-branch / multi-feature mix-up ... still learning git. I'll try to do better next time!
BR Andi


Update json-simple

This project uses old version of com.googlecode.json-simple:json-simple library.
This version has a bunch of issues, including unnecessary JUnit dependency: fangyidong/json-simple/issues/91.
I propose you to use more recent fork: https://github.com/cliftonlabs/json-simple. Which fixed this and a lot of other issues.

@grv87 Thanks for pointing this out.  Moving to the cliftonlabs version seems reasonable.  We could also address the issue on Junit by adding an exclusion to the POM file.  @yevster  - any opinions on the approach?

@grv87 In my dev environment, I changed to the latest published version of the cliftonlabs json-simple.  I ran into a number of compile failures requiring changes beyond just updating the import statements.  It's going to require a bit of time to analyze, change the names of some of the classes / methods and any other required changes.
I won't be able to spend much time on this over the next month, but if you want to make the changes and create a PR, I can review and merge.

@goneall,
Only 2.* version of cliftonlabs's fork is claimed to be compatible.
Latest (3+) is not, there were breaking changes.
You really want to update to latest? Why not just use last version from 2.+ series?

@grv87 I'll try using the latest 2. version

@grv87 Version 2.3.1 worked.  The changes are merged into the master branch.  Let me know if you need a release spun for this change.  Otherwise, I'll probably do another release in 2-3 months.

I just pushed out a release 2.1.15 which resolves this issue


Wrong bitly metric

Wrong metric used in Advanced Page. The number represents the total clicks all time.

fixed in pre-release


Add ElasticSearch exporter

This commit adds the ElasticSearch stats exporter.

I'm surprised this wasn't already listed.


intellij error-prone plugin is out of date

The last release of the Intellij error-prone plugin was 28.06.2016 and is missing many of the checks that the latest version from Maven has.

Synchronizing the IntelliJ updates with the normal error-prone releases would be really nice, if possible.

We don't maintain the IntelliJ plugin, but it is has been receiving regular updates lately: https://plugins.jetbrains.com/plugin/7349-error-prone-compiler/versions

We don't maintain the IntelliJ plugin, but it is has been receiving regular updates lately: https://plugins.jetbrains.com/plugin/7349-error-prone-compiler/versions


Fix Atlassian Confluence API parameter

Required items, please complete
Change(s):

Fix Atlassian Confluence API parameter

Reason for Change(s):

The audit API of confluence takes startDate and endData as a parameter. These two paramters should be in utc timestampe format but the current implementation passed a datetime string. And thus the function app is not retrieving any log from the API. See: https://developer.atlassian.com/cloud/confluence/rest/api-group-audit/#api-wiki-rest-api-audit-get

Version Updated:

N/A

Testing Completed:

Only tested in my personal environment

Checked that the validations are passing and have addressed any issues that are present:

N/A

@nevermoe Please update the changes in zip file. thanks!!!

@nevermoe Please update the changes in zip file. thanks!!!

@vmanojreddy Hi, I just updated the init.py and host.json in the zip file. I don't know if I should update the .python_packages directory in the zip file so I just leave it unchanged.
Thanks!

Tested the data connector by deploying the function app. we see error for function. Please check and update. Thanks.

@vmanojreddy Hi, sorry I didn't test that case. I think I fixed this bug. Would you please test deploy again? Thanks!

@nevermoe Tested the data connector by deploying the function app. Still we are getting error for function. Please check and update. Thanks!!!

@vmanojreddy
Hi, thank you for testing. I checked the zip file and the init.py is successfully updated. However, from the error message, it indicates that it's using an old version of init.py. Is the init.py cached somewhere?

@vmanojreddy Hi, Any updates here?

@vmanojreddy Hi, The error code is 404 which indicates there is something wrong with the confluence URL. Are you sure the tested confluence domain is correct? Because I made no change in this function, it's probably some configuration problem. If you can provide more information of the http request/response log, it would be more easy for me to locate the problem.

@vmanojreddy Hi, do we have any progress here?

Hi @nevermoe sorry for late, I deployed the function app again with proper Confluence URL. Still we are getting error in function. Please check and update. Thanks!!!

@vmanojreddy Hi, the time format written to the funcstatemarkerfile in storage account is in old format. I am doubting that some old version of function apps is running and writing strings to this file. Would you please have a check that all the function apps and the storage accounts deployed previously are deleted and deploy this new version again?

@vmanojreddy Hi, how's everything going

Hi @nevermoe  I am working and trying to deploy i am facing issues. I will check and update you. thanks!!!

@vmanojreddy Thank you very much for your help! Looking forward to hearing good news!

@vmanojreddy Hi, just to check, how's everything going?

Hi @nevermoe
I have tried deleting and redeploying the function app, storage account and dependent resources. As you have applied condition on the past_time in the file init.py line:38, that issue was fixed. But we are still getting other errors, please add not null or empty check on the result variable on line 84 and also it would be great if you can add more custom exceptions wherever you feel like breaking, so that it will be easier to identify the issue.

Hi @nevermoe
Could you please resolve the above comments and conflicts as well. Thanks!!

Hi @nevermoe
Do we have any update on the changes requested. Please let us know if you need any clarification
Thanks

Hi @nevermoe
Could you please resolve the comments, so that we can verify and merge your changes.
Thanks

Hi @nevermoe
Sorry for multiple follow ups, can you please resolve the comments.
Thanks

@v-mchatla @v-spadarthi Hi, thank GOD you guys finally come back to review this issue. You got a 404 error so it's likely you entered a wrong confluence url when using the ARM. E.g. for site like https://xxxxxxxxx.atlassian.net/, you should only enter the xxxxxxxxx part in ConfluenceHomeSiteName.

@v-mchatla @v-spadarthi Hi, thank GOD you guys finally come back to review this issue. You got a 404 error so it's likely you entered a wrong confluence url when using the ARM. E.g. for site like https://xxxxxxxxx.atlassian.net/, you should only enter the xxxxxxxxx part in ConfluenceHomeSiteName.

Hi @nevermoe,
We don't have any test data for this to test. Can you please provide some test data if you have, otherwise I will check with team on how to proceed in this case and update you.
Thanks.

@v-mchatla I don't have any test data. The test should be run with a JIRA service and I think you have tested the JIRA connection successfully before according to this thread.

Hi @nevermoe
Please resolve the conflicts. I will try to test once again and let you know the result.
Thanks

@v-mchatla Hi, I think the conflicts are resolved.

Hi @nevermoe
Thanks for resolving the conflicts. I will check with my team on how to proceed in this case as we don't have test data to do thorough testing and will update you soon.
Thanks

Hi @nevermoe
We are still working on it, will update you soon.
Thanks

@v-mchatla : Please have a look and update

Hi @nevermoe
Sorry for the delay. I will be working on it today and provide you the update.
Thanks

Hi @nevermoe,
I tried retesting the function app by redeploying but still getting the same error. I don't have test Confluence URL. If this function app is working fine in your environment, then we are good. Please confirm on the same so that we can proceed further.

Thanks

@v-mchatla Hi, the errors shows HOMESITENAME.atlassian.net.atlassian.net is NOT a valid URL and apparently you didn't input a valid HOMESITENAME. I think in some previous comments some guy in your team managed to test it with your own Confluence URL. Anyway, these codes run good in my environment but if you can't test it, I can't help with that.

Hi @nevermoe,
Thanks for the confirmation. I will just validate all the files and approve this PR in sometime.
Thanks

Hi @nevermoe,
I could see 2 host.json files. Do we need both or just keep which one is required?
Thanks

@v-mchatla Hi, I removed the unnecessary host.json.

Hi @nevermoe,
Thanks for the quick response. Checks are running at my end. Once all the checks are passed, I will approve and merge the PR


[Event Hubs] Disable CI logging

API change check
API changes are not detected in this pull request.


Using MRQ with Python 3

I'm trying to update some code from Python 2 to  Python 3, but I'm experiencing lots of incompatibilities. What versions of mrq and its dependencies (greenlet and gevent are the most problematic) should I be running?

It looks like this repository has changes compared to the last release. Adding it to requirements.txt as:
mrq @ git+https://github.com/pricingassistant/mrq.git@master

does the trick and runs with:
gevent              23.9.0.post1
greenlet            2.0.2

This project looks as if it's not maintained anymore, but a 0.9.11 release including these changes would be cool.


In the json fields two-dimentional array key is not properly working.... more then 2 values not saved in to the DB..

Laravel Version: #.#.#
PHP Version:
Laravel-admin: #.#.#

Description:
my controller  code is
$form->table('matching_profile','Matching Profile Details', function ($table) {
            $table->text('Matching Profile Id')->rules('required');
            $table->text('Matching Profile Name')->rules('required');
            $table->datetime('data_date_time')->default(today()->format('d-m-Y H:m a'))
->format('DD-MM-YYYY h:mm:ss a');
             $table->select('Matching Status')->options(['1'=>'Pending',2=> 'Interest',3=> 'Not Interest',4=> 'Declaine']);
        });

model
protected $casts = [
     'matching_profile' =>'json',

];
public function setColumnNameAttribute($value)
{
}
please help me  fast

not saved the more than 4 set values
we want many values can add and save

Does it actually named setColumnNameAttribute in your model? That is an example. See how mutators work in Laravel's models.
It should be named setMatchingProfileAttribute in your case.

but this can not working.. always this same error showing

key is $this->attributes['matching_profile'] = json_encode(array_values($value));

thank u sir... its fixed
On Wed, Jan 10, 2024 at 8:57 AM Daniel @.***> wrote:

key is $this->attributes['matching_profile'] = json_encode(array_values
($value));
—
Reply to this email directly, view it on GitHub
https://github.com/z-song/laravel-admin/issues/5825#issuecomment-1884132893,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5OU22LN4WG46K5CPZ4DWITYNYDDBAVCNFSM6AAAAAA5HHYW7KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBUGEZTEOBZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add android-common-ktx library

Why
We published the first patch version of android-common-ktx with Android's component extensions for usage. By adding this library, we will maintain the extensions outside android-templates scope.
Who Benefits?
Android developers

If I'm not mistaken, we can close this issue? 😁


I embed code and html in md，but useless

http://ddtalk.github.io/blog/2015/12/29/navbar/
this is the website，what can i do?

You can use some inline HTML and code samples as outlined here:
https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet


:bug: Fix linting issues (2 of n)

Please check if the PR fulfills these requirements

[ n/a] Tests for the changes have been added (for bug fixes / features)
[x] PR title follows the guidelines defined in  https://github.com/ossf/scorecard/blob/main/CONTRIBUTING.md#pr-process

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Code quality improvement

What is the current behavior? (You can also link to an open issue here)
This is my 2nd batch of commits for fixing existing linting violations. This PR fixes issues from 7 linters:

golint (various style rules)
godot (top level declaration comments end in a period)
lll (lines are not longer than 120 characters)
gofumpt (various style rules)
ineffassign (checks for assignments that have no effect)
nestif (checks for high levels of nesting)
noctx (makes sure context is passed to HTTP requests)

Documentation for these linters can be found on https://golangci-lint.run/usage/linters/. I will make future PRs to fix more linters.

What is the new behavior (if this is a feature change)?
n/a

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No

Other information:
Each commit tackles the issues from a single linter. I would recommend reviewing this PR commit by commit.

Sure, no problem!


feat(backup): use s3 based backup in staging and local

Ticket https://phabricator.wikimedia.org/T343112
Requires #1080 https://github.com/wbstack/charts/pull/125

I tested using a keypair created like this using my local mc installation which seemed to work as intended.


FIX: skip dc simulation unittest due to edb api bug

This changes were already part of https://github.com/ansys/pyedb/pull/791/files
@ring630 Please, next time you do changes I'm working on, let me know before merging them.


Consolidate toward name SeaweedFS

This updates README.md to consistently use the name "SeaweedFS", per #94.

I've rewritten the patch to only address the name in text. The link changes were premature, and can be handled in a separate patch later.


TreeError [SettingsTree] Tree element not found: [object Object]

Issue Id: a5d39d35-f2ec-cebb-30f8-bf7b129c1200Versions - 1.42.0-  be0aca7188ec6a76e7c2379758c0fbc1e9c21f7b-  705f01577dd37d6f17eaa683ab89b0c392fa1a7cStack Error: TreeError [SettingsTree] Tree element not found: [object Object]/vs/base/browser/ui/tree/objectTreeModel.ts#270:24 (this)/vs/base/browser/ui/tree/objectTreeModel.ts#194:19/vs/base/browser/ui/tree/abstractTree.ts#771:25/vs/workbench/contrib/preferences/browser/settingsTree.ts#1512:22 (fgWithOpacity)/vs/workbench/contrib/preferences/browser/settingsTree.ts#1520:144

@JacksonKearl This is SettingsTree.

Stale and I don't know why the stack is pointing at theming code


update version to 2024.4.0

/intelci: run

@napetrov can we merge it now, so that I can use newly added result_options from sklearnex side?


Was the README clear?

I found your Readme file not clear enough, please add a few steps on how to use your program. Thanks in Advance.

I added some more explanations. I hope it's more clear.


SLB via IMDS windows example doesn't bypass proxies

As the document correctly points out, bypassing proxies is a requirement when interacting with IMDS. However the -NoProxy flag is not actually present:

This is surprising because the .md correctly specifies the flag:

There appears to be an issue when the rich text formatting of the code blob is generated, and for some reason the flag is being left out. This needs to be addressed as it is misleading in that the example people often copy paste without understanding is wrong and that there's a label saying it does something it doesn't actually do.

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

ID: dad6b33b-2c86-ed40-5d7a-61bf0fff6299
Version Independent ID: 431d4e4f-121d-55b1-82e2-0f30d7424108
Content: Retrieve load balancer metadata using the Azure Instance Metadata Service (IMDS) - Azure Load Balancer
Content Source: articles/load-balancer/howto-load-balancer-imds.md
Service: load-balancer
GitHub Login: @asudbring
Microsoft Alias: allensu

Hi @LittleBoxOfSunshine
We currently have a PR in to fix this.  Unfortunately we are in a content freeze until next week because of Ignite.  The fix will be on the live site when merges commence after the freeze.
Thanks!
#reassign:@asudbring

#reassign:@asudbring

Issue resolved
#please-close


Test the paged attention vs the original candle implementations

Provide a few examples, to compare the Llama generation relying on the paged attention algorithms vs the original candle Llama implementation.

We suggest following:

[ ] Once you have resolved #3, please create a script that can run the original candle Llama implementation, as well as yours.
[ ] Provide a few prompt examples.
[ ] Run these prompt examples on each of the implementations, using your script.
[ ] Record the outputs of each inference and compare them.
[ ] You should check if the outputs are exactly the same, and if not provide a hint of which outputs you think are more accurate as an answer to the input prompts.
[ ] Open a PR, explaining each of the steps above.

Available? Please assign

@jorgeantonio21 Yo, let me tackle this issue. I got @WillNovus on my team

@Josh-121 @jedstroke assigned you both ! Good luck with it (want to highlight this issue depends on the previous ones #1, #2 and #3).

Thanks @jorgeantonio21


Refactor/uipackage

To make the org.xbmc.kore.ui package more manageable I want to subdivide it into multiple packages.
I came up with the following division:

generic: fragments/activities used by multiple sections
sections: these are the different screens we show (I was also thinking of naming this screens)

addon
audio
file
hosts
remote
settings
video

viewgroups: our custom viewgroups
views: our custom views

All abstract and base classes will remain under org.xbmc.kore.ui.
I'm not sure what to do with the OnBackPressedListener. Should this be under generic or move it outside  org.xbmc.kore.ui and place it under utils?
There are more issues to resolve but I think we first need to come up with a good grouping of all the classes under org.xbmc.kore.ui.

I agree, but i think that it might be cleaner if you would get rid of the sections and just keep addons, music, etc at the base level.
Regarding OnBackPressedListener, why not leave it at the base level, like the abstract classes?


Comma power is connected to the vehicle, it automatically switches to ACC mode

Describe the bug
To reiterate the ignition issue, when the car is completely off, connecting the comma power (OBD) and RJ45 cable to the harness box and then automatically powers the car and does not turn off when the ignition button is pressed.
I'll post a video about this phenomenon later.
Which car does this affect?
AZERA Hybrid 6TH Generation
Provide a route where the issue occurs
66189dd8ec7b50e6|2023-09-20--07-02-12
openpilot version
0.9.5
Additional info
No response

It's hard to tell exactly from your video, but it appears that you are using non official hardware (specifically the car harness). Can you confirm?

We can't really provide support for non official hardware. If I had to guess, you (or whoever you bought this from) may have wired the camera ignition line into the 12v line on the harness. This would feed 12v constant from the OBDII port into the car and camera ignition line. Ensure that you are following the diagrams here and have the proper wiring. https://github.com/commaai/neo/blob/master/car_harness/black_panda_harness.jpg

Would be interesting to know what the smallest set of hardware causes this. Does just the comma power (no t
Does it happen with:

comma power (no cable)
comma power (with cable)

harness box

harness connector

3X/device


FIX: Attempt to fix the sporadic defocusing problem

Remove the .blur() immediately before .focus()
Move the _focusTextArea() call from didRender to didInsertElement
Remove the _messageIsEmpty check from this call. IMO we still want to focus, even if there's a draft message in the textarea.

let's give it a shot! Not sure about the blur.. can't remember.


Interactions failing with 404 sometimes

I've been noticing some strange behaviour on our bot we're using with slash-create.
After initially launching the bot the commands would work fine, but if it has been running for a couple minutes all interactions would fail with the code 10062 unless I add a await ctx.defer() in front of every command.
More importantly if I use the new MessageComponents the interaction will also fail after 2-3 minutes or so after not interacting with it, even though the interaction is supposed to last up to 15 minutes.
I'm not sure if it's just a timing issue, intended or if it's actually a bug.
here is the full stack of the error that occurs (for the gserv command in this case):
(node:3523014) UnhandledPromiseRejectionWarning: DiscordRESTError [404]: Unknown interaction
Error:
    at RequestHandler.request (/home/metaconcord/node-metaconcord/node_modules/slash-create/lib/util/requestHandler.js:62:15)
    at SlashCreatorAPI.interactionCallback (/home/metaconcord/node-metaconcord/node_modules/slash-create/lib/api.js:79:45)
    at ComponentContext._respond (/home/metaconcord/node-metaconcord/node_modules/slash-create/lib/creator.js:585:28)
    at ComponentContext.acknowledge (/home/metaconcord/node-metaconcord/node_modules/slash-create/lib/structures/interfaces/componentContext.js:35:24)
    at Timeout.<anonymous> (/home/metaconcord/node-metaconcord/node_modules/slash-create/lib/structures/interfaces/componentContext.js:24:47)
    at listOnTimeout (internal/timers.js:549:17)
    at processTimers (internal/timers.js:492:7)

The code for the bot is public and the commands are located here:
https://github.com/Metastruct/node-metaconcord/tree/master/app/services/discord/commands

Might be some sort of timing issue with using setTimeout to auto-defer. The reason that all interactions would fail without deferring is that there needs to be an initial response for the interaction to be valid. This should already be solved since a timeout is created when a command context is made. The components are even weirder since they are just reading off of Date.now() and checking to see if its been 15 minutes. Is there anything messing with dates internally?

I've done some testing and came to the conclusion that this only happens if you call a longer running synchronous function via ctx.send or just returning in the run function.

That's odd. Seems like a timing issue, since the error is coming from the timeout of the component context. You shouldn't need to manually defer.

did some more testing, here is what I used to test:
async run(): Promise<any> {
	const is = Date.now();
	console.log(is);
	while (Date.now() - is < 5000) {}
	return "a";
}

this always fails for me, I printed the time on each step and it delays here
https://github.com/Snazzah/slash-create/blob/master/src/creator.ts#L781
I also added some extra printing to the defer function
https://github.com/Snazzah/slash-create/blob/master/src/structures/interfaces/messageInteraction.ts#L274
and it's not being run at all, as if the timeout never happens, unless I manually defer before explicitly.
maybe using something like Promise.race in _runCommand could work?
hope this helps.

Try debugging from here:

https://github.com/Snazzah/slash-create/blob/master/src/structures/interfaces/context.ts#L91

This is where the timeout should be created for command contexts.

I already did this yesterday, the function does exist and all but it doesn't seem run while the promise is being awaited.
>>> setting this._setTimeout func in context<PHONE_NUMBER>839
>>> ctx created, timeout func: Timeout {
  _idleTimeout: 2000,
  _idlePrev: [TimersList],
  _idleNext: [TimersList],
  _idleStart: 8828,
  _onTimeout: [Function],
  _timerArgs: undefined,
  _repeat: null,
  _destroyed: false,
  [Symbol(refed)]: true,
  [Symbol(asyncId)]: 993,
  [Symbol(triggerId)]: 119
}<PHONE_NUMBER>839
>>> Command run() in SlashCommand<PHONE_NUMBER>840
>>> pre await promise _runCommand<PHONE_NUMBER>840
(node:2102474) UnhandledPromiseRejectionWarning: DiscordRESTError [404]: Unknown interaction
Error:
    at RequestHandler.request (/home/metaconcord/node-metaconcord/node_modules/slash-create/lib/util/requestHandler.js:62:15)
    at SlashCreatorAPI.interactionCallback (/home/metaconcord/node-metaconcord/node_modules/slash-create/lib/api.js:79:45)
    at CommandContext._respond (/home/metaconcord/node-metaconcord/node_modules/slash-create/lib/creator.js:588:28)
    at CommandContext.send (/home/metaconcord/node-metaconcord/node_modules/slash-create/lib/structures/interfaces/messageInteraction.js:79:24)
    at SlashTestCommand.finalize (/home/metaconcord/node-metaconcord/node_modules/slash-create/lib/command.js:164:24)
    at SlashCreator._runCommand (/home/metaconcord/node-metaconcord/node_modules/slash-create/lib/creator.js:573:32)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)

Weird bug, but it is at least fixable with manual deferring. Not sure how to restructure to avoid this, but in most cases this won't be an issue. Thanks for being in-depth with this though.


Automatic pipeline

Hi, pipeline is ready

when you want to push new version in nuget
all that you must do is to create new tag with next version

This is example

The tag should be with bigger version than last and scheme is vX.Y.Z (v1.0.9) x is MAJOR version, y is MINOR and z is PATCH

and after you push this tag to origin version will be ready automatic

you can see the progress in actions tab in your repo

If you have any questions ask me :)

awesome, thank you


Use terminus to invoke drush to pull DB from Pantheon

The Problem/Issue/Bug:
With the existing comman to pull db from Pantheon, I got an error:
[error]  Could not locate a site your user may access identified by @foo.bar
How this PR Solves The Problem:
Trying to move to use terminus drush instead

Actually, I was too quick on the submit. It recognizes the site, but I got a different error
 [notice] Command: foo.bar -- drush sql-dump [Exit: 0]
Database downloaded to: /home/amitaibu/Sites/foo/drupal/.ddev/.downloads/db.sql.gz
Importing database...
Pull failed: failed to extract provided archive: gzip: invalid header

Ok, some understanding I have about current implementation:
Right now in master upon authentication it does terminus aliases. However when using drush @${project} it would expect pantheon.foo.bar and not foo.bar as the current example suggests. So I do think that removing the terminus aliases and switching to terminus drush might be a better option.
I'm not clear about the gzip: invalid header error though.

Usually gzip invalid header means that the data isn't gzipped. And you're doing a sql-dump without --gzip.

btw, another advantage of using terminus drush is that it recognizes sites that I belong to through "Organization", but not a member of specifically.

And you're doing a sql-dump without --gzip.

It's actually there, but for some reason it's not working. -- sql-dump --gzip >...

You can do the gzipping on the local side then...

Haven't circled back to it yet. I'll close it for now, until I find a working solution.

Thanks, sure appreciate you working on it. I'm not having any trouble with the existing one, so if I can help with the problem you're having there, let me know. I'm definitely not a terminus or pantheon expert.

There was a version of drush that didn't properly set the site alias location, so you might want to add --alias-path=~/.drush to the drush command...

Would love to have you try and review the new strategy (backup-based) in https://github.com/drud/ddev/pull/2957 which is far, far better for a large site.  Thanks @amitaibu .

Sure, I'll check it (Probably in a couple of days).
Using terminus looks like a great approach. Should I install ddev with that new branch? Is there a README on how to do that (I currently use brew for installation)

All that changes here is the example file, so you can just take the "pull" lines from the pantheon.yaml.example and put them in your pantheon.yaml. You don't have to change ddev at all unless you don't have v1.17+, which I'm sure you do.


[Makefile] Upgrade the template, reverting the user content

Internal functionality, thus is not covered with tests.

From my point of view, functionality of self-update should not be a part of Makefile itself, even if non-exposed to the user, as it will add more complexity to the Template upgrade problem. However, If solving this problem from the other point, via a specific tool, we better save user's choice to config.yaml (project name, project slug, etc.) and re-use it. Note, in current version "Project Name" is not specified in cookiecutter command (this is minor and super-rare case, but since this target exists, this is a bug).
Since, I hope, we are moving towards minimizing the size of auto-generated code, this target is temporary and is to be used only by @mariyadavydova , so if its presence solves the problem, I'm OK with it.


Close all pubsub channels when connection is closed

Without this, Channel#wait_message() doesn't return nor raise any exceptions when the connection is lost.

thanks

@popravich please update CHANGES.txt also

I'm doing this before each release

Well, that's up to you. I prefer update CHANGES after every merge to don't forget something important.

True, but don't worry, I do git log <old_tag>..HEAD every time to check everything)


No access to the site

Describe the issue
I cannot access to the site it shows me Error 401 or 521 can you help me please
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

For access issues, instead of filing a bug please send email to<EMAIL_ADDRESS>with details.


Update axios to latest

@ronjouch
Axios at this version is throwing a warning for me downstream in nativefier:
(node:46090) ExperimentalWarning: The Fetch API is an experimental feature. This feature could change at any time
    at emitExperimentalWarning (node:internal/util:224:11)
    at lazyUndici (node:internal/bootstrap/pre_execution:188:5)
    at get (node:internal/bootstrap/pre_execution:204:16)
    at Object.isFormData (/Users/me/Dev/personal/nativefier/node_modules/gitcloud/node_modules/axios/lib/utils.js:57:3)
    at Object.transformRequest (/Users/me/Dev/personal/nativefier/node_modules/gitcloud/node_modules/axios/lib/defaults.js:58:15)
    at transform (/Users/me/Dev/personal/nativefier/node_modules/gitcloud/node_modules/axios/lib/core/transformData.js:18:15)
    at Object.forEach (/Users/me/Dev/personal/nativefier/node_modules/gitcloud/node_modules/axios/lib/utils.js:245:10)
    at Object.transformData (/Users/me/Dev/personal/nativefier/node_modules/gitcloud/node_modules/axios/lib/core/transformData.js:17:9)
    at dispatchRequest (/Users/me/Dev/personal/nativefier/node_modules/gitcloud/node_modules/axios/lib/core/dispatchRequest.js:30:31)
    at Axios.request (/Users/me/Dev/personal/nativefier/node_modules/gitcloud/node_modules/axios/lib/core/Axios.js:108:15)
    at Axios.<computed> [as get] (/Users/me/Dev/personal/nativefier/node_modules/gitcloud/node_modules/axios/lib/core/Axios.js:129:17)
    at Function.wrap [as get] (/Users/me/Dev/personal/nativefier/node_modules/gitcloud/node_modules/axios/lib/helpers/bind.js:9:15)
    at gitCloud (/Users/me/Dev/personal/nativefier/node_modules/gitcloud/gitcloud.js:5:16)
    at inferIconFromStore (/Users/me/Dev/personal/nativefier/src/infer/inferIcon.ts:77:28)
    at inferIcon (/Users/me/Dev/personal/nativefier/src/infer/inferIcon.ts:109:11)
    at icon (/Users/me/Dev/personal/nativefier/src/options/fields/icon.ts:25:27)

Uh? Okay, didn't know Axios would sometimes call fetch behind the scenes. And I guess they recently improved since then how they handle it, to no longer show these warnings. Okay, merging, will cut a release and bump in Nativefier soon.


Free up disk space in CI

Delete unneeded files from the CI runner on Ubuntu (+ re-install some packages that are needed for unit tests).
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.
[x] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

@fa9r working so far. The GH Action I added is pretty simple, so as part of next week's rehaul potentially we might just want to directly add the raw bash commands it's calling so as not to have magical stuff happening that's hard(er) to debug down the road.


Implement Tree component

Resolves https://github.com/arturbien/React95/issues/209 (add Tree component)

@alpavlove thanks for PR! I will try to review it over the weekend 🚀

Hi @arturbien. I'm really looking forward to receiving feedback and getting the this merged

@alpavlove really sorry for the delay. been super busy lately. hopefully this weekend I'll find the time to go through. this

@alpavlove first of all - great job 🔥 I've picked up some things that should be changed:

Folder items are not focused when tabbing throught the tree (In below example latte and espresso is skipped when tabbing throught the items. Or is it because there's no callback specified for those items?)

https://user-images.githubusercontent.com/28541613/168461291-6a2a63d4-5821-4bae-9b1c-48f3b7b39199.mov

Make the dotted tree path 2px wide instead of 1px wide (React95 uses a bit wider borders/lines compared to OG win 95)

Make the expand/collapse icon box to have 2px border width

A subtree should be expanded right below the icon, not below the 'expand/collapse' icon

Focus styles should be different. Instead of using 1px dotted border around entire item, use focus styles as shown below (navy background with yellow dotted outline) around item label. You can find the same styles used in <Select /> component once user selects a new value in the dropdown

For reference you can run this awesome Windows 95 emulator and have the OG Tree for reference there: https://www.pcjs.org/software/pcx86/sys/windows/win95/4.00.950/

@arturbien thanks for the detailed feedback. All comments should be resolved now. The emulator is really awesome :D

@alpavlove sheeesh now it looks way better!
I'll still bother you some more because I've found couple minor issues. I'll give you an update during the weekend. 🤓

@arturbien thanks. I'm really looking forward to getting this done

Hey @arturbien. That's a friendly reminder to give an update 🙂 Thanks in advance

@alpavlove sorry for being so slow with responses. I got deadlines in two projects that I am working on rn 🥵. Will do before the end of the week

@alpavlove

The expand/collapse (+ / -) placement is wrong. It should be placed on the vertical line, not next to it:

Navy background stays on the item even when it's not focused (it should be only highlighted when it's in focus)

Thanks for taking your time @arturbien

Navy background stays on the item even when it's not focused (it should be only highlighted when it's in focus)

I assume you're talking about the state when an item is selected. So the navy outline indicates that and it doesn't seem I should remove that. Any thoughts?

@alpavlove the background and outline should be visible only when an item is focused. You can use Win 95 I linked you before for reference. If user expands tree item but then clicks a button, or anywhere else on the page for that matter, the navy background+outline should be removed from tree item (since it's no longer in focus)

@arturbien it didn't dispel my concern regarding the selected state. How do we indicate that?

@arturbien expand/collapse icon has been relocated and the selected item state has been removed

@alpavlove awesome job. we'll finalize this PR this weekend.

@alpavlove PR approved but before we merge please squash all commits into one commit with a message "feat: add Tree component" 🚀

@arturbien sure thing. It's done. Thanks for your time!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Example script hanging without any output or any error hint.

I have completed the installation with Conda (using the install.sh script), but could not successfully run the example script.
When I run the pcqv2.sh script, it just hangs without any output or any error message.
I'm not sure anybody else has faced the same problem. Can you give me some advice to resolve the issue?
For more information, I'm using the GCP with NVIDIA V100 GPUs and CUDA11.1.
Within the same environment, I have checked running the fairseq NMT example codes and the Graphormer v1. codes.
Both of them did not produce any errors.

Thanks for using Graphormer.
If there is no error message, would you please provide a minial docker image and code/script/instruction which could reproduce your problem?

@zhengsx, Thanks for the quick reply.
I have been using the Conda environment without Docker.
I tried to use the following script to create a Docker image and still faced the same problem.
(Btw, I'm sorry for the messy Dockerfile script, I'm not really used to using Docker.)
FROM nvidia/cuda:11.1.1-cudnn8-devel-ubuntu18.04
ENV DEBIAN_FRONTEND=noninteractive
RUN apt-get update && apt-get install git sudo -y

RUN apt-get install -y software-properties-common
RUN add-apt-repository -y ppa:deadsnakes/ppa
RUN apt-get update && apt-get install git sudo -y

RUN apt-get install --no-install-recommends -y python3.9 python3.9-dev python3-pip python3-dev python-dev python-setuptools python3-setuptools
RUN apt-get install -y python3.9-distutils
RUN apt-get install -y gfortran libopenblas-dev liblapack-dev
RUN apt-get install -y gcc g++

RUN update-alternatives --install /usr/bin/pip pip /usr/bin/pip3 1
RUN update-alternatives --install /usr/bin/python python /usr/bin/python3.9 1

RUN python -m pip install --upgrade wheel setuptools pip distlib

Within the Docker image, I simply followed the instructions to install Graphormer.
Then, I used the example script pcqv2.sh as follows.
sudo docker run -it --gpus all --ipc=host --name=graphormer graphormer:latest bash
git clone --recursive https://github.com/microsoft/Graphormer.git
cd Graphormer
bash install.sh
cd examples/property_prediction
bash pcqv2.sh

Still, it just hangs without any output or any error message.

@mswzeus @zhengsx I meet the same problem. Have you solved the problem yet?
Sometimes there is no response for half an hour after running the command bash zinc.sh:

Sometimes errors are reported:
    from torch_geometric.data import InMemoryDataset
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch_geometric/__init__.py", line 5, in <module>
    import torch_geometric.data
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch_geometric/data/__init__.py", line 1, in <module>
    from .data import Data
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch_geometric/data/data.py", line 8, in <module>
    from torch_sparse import coalesce, SparseTensor
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch_sparse/__init__.py", line 41, in <module>
    from .tensor import SparseTensor  # noqa
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch_sparse/tensor.py", line 13, in <module>
    class SparseTensor(object):
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch/jit/_script.py", line 1128, in script
    _compile_and_register_class(obj, _rcb, qualified_name)
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch/jit/_script.py", line 138, in _compile_and_register_class
    script_class = torch._C._jit_script_class_compile(qualified_name, ast, defaults, rcb)
RuntimeError:
object has no attribute sparse_csr_tensor:
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch_sparse/tensor.py", line 511
            value = torch.ones(self.nnz(), dtype=dtype, device=self.device())

        return torch.sparse_csr_tensor(rowptr, col, value, self.sizes())
               ~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE

@zhengsx @skye95git  Fortunately, a friend told me that it seems like a setuptools issue.
After the installation, I removed the setuptools package pip uninstall setuptools.
I'm not sure why it was the problem but it seems like the example script now runs without raising any errors.

@zhengsx @skye95git Fortunately, a friend told me that it seems like a setuptools issue. After the installation, I removed the setuptools package pip uninstall setuptools. I'm not sure why it was the problem but it seems like the example script now runs without raising any errors.

Thanks very much! I have try it and find it works. After removing the setuptools package pip uninstall setuptools, there is en error No module named '_distutils_hack'. I update the setuptools pip install --upgrade setuptools, it still works.
But I can't download molecules and zinc datasets directly. Can I download and load manually? If so, where should I modify it?
File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch_geometric/datasets/zinc.py", line 86, in download
    path = download_url(self.url, self.root)
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch_geometric/data/download.py", line 34, in download_url
    data = urllib.request.urlopen(url, context=context)
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/urllib/request.py", line 214, in urlopen
    return opener.open(url, data, timeout)
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/urllib/request.py", line 517, in open
    response = self._open(req, data)
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/urllib/request.py", line 534, in _open
    result = self._call_chain(self.handle_open, protocol, protocol +
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/urllib/request.py", line 494, in _call_chain
    result = func(*args)
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/urllib/request.py", line 1389, in https_open
    return self.do_open(http.client.HTTPSConnection, req,
  File "/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/urllib/request.py", line 1349, in do_open
    raise URLError(err)
urllib.error.URLError: <urlopen error [Errno 101] Network is unreachable>

@zhengsx @skye95git Fortunately, a friend told me that it seems like a setuptools issue. After the installation, I removed the setuptools package pip uninstall setuptools. I'm not sure why it was the problem but it seems like the example script now runs without raising any errors.

Hi, Now pcqv1.sh can actually run without any errors, but it doesn't seem to work anymore after it prints out some information. It stops in the following position:
/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch/cuda/__init__.py:106: UserWarning:
A100-SXM4-40GB with CUDA capability sm_80 is not compatible with the current PyTorch installation.
The current PyTorch install supports CUDA capabilities sm_37 sm_50 sm_60 sm_61 sm_70 sm_75 compute_37.
If you want to use the A100-SXM4-40GB GPU with PyTorch, please check the instructions at https://pytorch.org/get-started/locally/

  warnings.warn(incompatible_device_warn.format(device_name, capability, " ".join(arch_list), device_name))
2022-03-30 10:30:27 | INFO | fairseq.distributed.utils | distributed init (rank 0): tcp://localhost:16416
/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch/cuda/__init__.py:106: UserWarning:
A100-SXM4-40GB with CUDA capability sm_80 is not compatible with the current PyTorch installation.
The current PyTorch install supports CUDA capabilities sm_37 sm_50 sm_60 sm_61 sm_70 sm_75 compute_37.
If you want to use the A100-SXM4-40GB GPU with PyTorch, please check the instructions at https://pytorch.org/get-started/locally/

  warnings.warn(incompatible_device_warn.format(device_name, capability, " ".join(arch_list), device_name))
2022-03-30 10:30:27 | INFO | fairseq.distributed.utils | distributed init (rank 2): tcp://localhost:16416
2022-03-30 10:30:27 | INFO | torch.distributed.distributed_c10d | Added key: store_based_barrier_key:1 to store for rank: 2
/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch/cuda/__init__.py:106: UserWarning:
A100-SXM4-40GB with CUDA capability sm_80 is not compatible with the current PyTorch installation.
The current PyTorch install supports CUDA capabilities sm_37 sm_50 sm_60 sm_61 sm_70 sm_75 compute_37.
If you want to use the A100-SXM4-40GB GPU with PyTorch, please check the instructions at https://pytorch.org/get-started/locally/

  warnings.warn(incompatible_device_warn.format(device_name, capability, " ".join(arch_list), device_name))
/home/linjiayi/anaconda3/envs/graphormer/lib/python3.9/site-packages/torch/cuda/__init__.py:106: UserWarning:
A100-SXM4-40GB with CUDA capability sm_80 is not compatible with the current PyTorch installation.
The current PyTorch install supports CUDA capabilities sm_37 sm_50 sm_60 sm_61 sm_70 sm_75 compute_37.
If you want to use the A100-SXM4-40GB GPU with PyTorch, please check the instructions at https://pytorch.org/get-started/locally/

  warnings.warn(incompatible_device_warn.format(device_name, capability, " ".join(arch_list), device_name))
2022-03-30 10:30:27 | INFO | fairseq.distributed.utils | distributed init (rank 3): tcp://localhost:16416
2022-03-30 10:30:27 | INFO | fairseq.distributed.utils | distributed init (rank 1): tcp://localhost:16416
2022-03-30 10:30:27 | INFO | torch.distributed.distributed_c10d | Added key: store_based_barrier_key:1 to store for rank: 1
2022-03-30 10:30:27 | INFO | torch.distributed.distributed_c10d | Added key: store_based_barrier_key:1 to store for rank: 3
2022-03-30 10:30:27 | INFO | torch.distributed.distributed_c10d | Added key: store_based_barrier_key:1 to store for rank: 0
2022-03-30 10:30:27 | INFO | torch.distributed.distributed_c10d | Rank 0: Completed store-based barrier for 4 nodes.
2022-03-30 10:30:27 | INFO | fairseq.distributed.utils | initialized host dgx079.scc.idea as rank 0
2022-03-30 10:30:27 | INFO | torch.distributed.distributed_c10d | Rank 2: Completed store-based barrier for 4 nodes.
2022-03-30 10:30:27 | INFO | fairseq.distributed.utils | initialized host dgx079.scc.idea as rank 2
2022-03-30 10:30:27 | INFO | torch.distributed.distributed_c10d | Rank 1: Completed store-based barrier for 4 nodes.
2022-03-30 10:30:27 | INFO | torch.distributed.distributed_c10d | Rank 3: Completed store-based barrier for 4 nodes.
2022-03-30 10:30:27 | INFO | fairseq.distributed.utils | initialized host dgx079.scc.idea as rank 1
2022-03-30 10:30:27 | INFO | fairseq.distributed.utils | initialized host dgx079.scc.idea as rank 3

Have you finished running pcqv1.sh and obtained any results?

Hi @skye95git , I just have a galance of your error message, it seems like some usage problems of incompatible pytorch and your GPU card. Thus I initialize the Discussion forum, and you may ask all of your usage problems about how to use Graphormer and anything else you want to discuss at there, and discuss with the guys in Graphormer community who met similar problem with you.
Let's try to leave the issue channel for real bugs and feature request of Graphormer, what do you think about it?

Hi @skye95git , I just have a galance of your error message, it seems like some usage problems of incompatible pytorch and your GPU card. Thus I initialize the Discussion forum, and you may ask all of your usage problems about how to use Graphormer and anything else you want to discuss at there, and discuss with the guys in Graphormer community who met similar problem with you.
Let's just leave the issue channel for real bugs and feature request of Graphormer, what do you think about it?

Ok. Thanks.