[UI bug] Overlap of return annotation with [source]

What's happening?
When using linkcode to attach the [source] for each method, and when using autodoc-process-signature to manually generate a custom return annotation, there is an UI overlap of both those elements.

My html_theme_options (unrelated to above but still):
html_theme_options = {
    # 'style_external_links': True,  # Use after https://github.com/pradyunsg/furo/discussions/348
    'navigation_with_keys': True,
    'dark_css_variables': {'admonition-title-font-size': '0.95rem',
                           'admonition-font-size': '0.92rem'},
    'light_css_variables': {'admonition-title-font-size': '0.95rem',
                            'admonition-font-size': '0.92rem'}
}

autodoc_type_hints is set to none.
Furo version: 2022.1.2
Sphinx version: 4.3.2
Reproducer
Will commit changes to remote soon, so I'll edit this then
Expectation
Furo should recognize that there will be an overlap and put it in the next line, like it did below:

Code of Conduct

[X] I agree to follow the Code of Conduct.

IISc with autodoc_type_hints=None, the return type shouldn't show at all, right?


add support for internal and external node ip addresses

[ ] update openstack module

Closing. It is now included in #4.


Django Defender is not working on my userlogin view

in my admin it's work correctly
in my user login view it's not working, it didn't show any access attempts and not blocking any IP and username

Are you using the built in django LoginView or do you have something custom for handling your logins?

I had my own login function in my views

Ok, since you have a custom view you will need to wrap your view with the decorator, or add all the different logic into your login view manually.
Did you try that already?

Sorry, I can't understand

Is it working now or do you still have an issue?

I still have if I use Django built-in login system it works correctly but if I use my login function it doesn't work

hey

@Hamza-Lachi You will need to either add Django defender directly to our  custom login function like it shows in the docs here: https://django-defender.readthedocs.io/en/latest/#adapting-to-other-authentication-methods
Or you can use the watch_login decorator directly on your login view in the URLs file like you see in the stack overflow answer. https://stackoverflow.com/a/5797020/356788
Hope that helps

@Hamza-Lachi can we close this issue?

how  Django defender will help in OTP based login app?

@kencochrane thank you! I ran in the same issue and your comment from May 27, 2020 resolved it for me. Would it make sense to mention this in the docs (unless it is already there and I have not seen it)?

@TTorture feel free to submit a PR to add it to the docs, it probably isn't there and if a few people are asking about it, there will probably be more in the future. Thank you.


APO cache is not cleared properly.

Confirmation
My issue isn't already found on the issue tracker.
I have replicated my issue using the latest version of the plugin and it is still present.
WordPress version
5.7
Cloudflare-WordPress version
4.4.0
PHP version
7.4.0
Expected result
that the APO cache after a change / publish / etc really clears
Actual result
Only the desktop version cache is cleared
Steps to reproduce
Activate the plugin
Activate APO
Activate "Cache by device type"
Create content.
View the page with mobile and desktop (the order does not matter).
Publish new content.
Top up on mobile (nothing has changed: no new content)
Reload on desktop (new content appears)
Additional factoids
I investigated the problem at length and here are my conclusions.
When the "zonePurgeFiles" method is executed with an array of 30 lines, it clears the cache well but not in mobile (tablet I have not checked).
I modified the request by adding the headers "CF-Device-Type" but nothing changes.
If I delete the url in the Cloudflare interface directly it clears all the caches on that page.
If I modify the request not to send an array but just 1 url like this (imagining that the first line $files[0] is my test url 😉):
$request = new Request('DELETE', 'zones/'.$zoneId.'/purge_cache', array(), array('files' => $files[0]));
$response = $this->callAPI($request);

the cache clears well.
Looks like the request by array does not clear the devicetype cache
References
No response

similary #394

here is some additional feedback.
I need to use mobile / desktop cache unbundling despite my site being responsive, there is an ad script that can't work properly.
I also use before activating APO the plugin WP-ROCKET with options:

[x] Enable caching for mobile devices
[x] Separate cache files for mobile devices

I did a test:

Clear all WP-Rocket cache
Clear all Cloudflare cache
Load the home on the desktop (WP-ROCKET OK / Cloudflare OK file)
Load the home on the mobile (WP-ROCKET OK / Cloudflare OK file)
Creation of a new article
WP Rocket erases index-https.html (with gzip too) and index-mobile-https.html (with gzip too)
Loading the page on desktop: OK Cloudlfare updated with the 'age' header
Loading of the page on mobile: KO Cloudflare does not give a new version
Verification of the WP-Rocket cache folder: the index-https.html file has been successfully recreated but not the mobile one.

The problem really comes from the side of Cloudflare which does not delete the "mobile" file with the API

I'm also facing the same problem . Pls.fix it

Is anyone working on this issue? A month has crossed.

"9. Verification of the WP-Rocket cache folder: the index-https.html file has been successfully recreated but not the mobile one."
If I understand this correctly, if WP-rocket cache doesn't update the mobile file then Cloudflare is going keep caching that old file provided by WP-rocket.

No, Rocket does not generate a mobile file because Cloudflare's cache "responds before reaching the server".
The problem comes from the Cloudflare purge which does not take place.
Look at the headers in particular the "Age" and you will see that the mobile version is older than the computer version.
The problem seems to really come from the Cloudflare API

@rvola If Cloudflare API key is the issue, do we need to update / change the key?
If yes how do we update the new API key in the Wordpress plugin?

sorry I made myself misunderstood.
This is not CloudFlare's key to the problem.
It is the API command which does not clear ALL the cache of the page

OK. Can we do anything from our side or only Cloudflare can help?

I do not know.
I put a hypothesis, tried to redo my test protocol to know if it comes from the plugin.

Thank You.
Pls update here if you got the solution.

for my part I have no solution.
If the authors of the plugin can look, it would appeal to everyone

It's really annoying! Crawlers are not even considering new content because of this issue. Earlier I was using WP Rocket but now I shifted to LITESPEED Cache the problem is same and that means it is directly from Cloudflare APO

Hello,
Could a moderator escalate the problem to the Cloudflare API Team?

I have explained in their forum too https://community.cloudflare.com/t/apo-cache-is-not-clearing-properly/274355

thanks @girirajnet

Exact same issue here with LSCache + Cloudflare APO.
Cloudflare Cache By Device Type = enabled
LSCache Mobile Cache = enabled
Expected result
that the APO cache after a change / publish / etc clears & updates for all types of devices
Actual result
Only the desktop version cache is cleared

I had a look at APO purge issue when "Cache By Device Type" is enabled. I can't reproduce the issue without more details ( purging of all variants works fine on our test setup).
There is a Cloudflare Purge API quirk for  "Cache By Device Type": when the purge happens it only purges for 1 particular device type variant not all of them. So internally APO team supports additional infrastructure to listen to Purge requests, detects "Cache By Device Type" and purges the remaining variations. I hope this explains why there is no special treatment of "Cache By Device Type" in the WordPress plugin.

OMG, I thought I was the only one waiting for two months to get this fixed. Having the exact same issue. We even removeed LSCache and the issue is still there.

OH, BTW Cloudflare is aware of the issue and they are looking into it.

Thanks for following up.
I fully understand the frustration that this is taking longer than expected to get addressed. As this issue is now escalated, I expect to have product engineering team investigating potential root cause of your report and similar issues shortly.
When we have a new update for you, we will let you know straight away. We appreciate your patience whilst we are working on this.
Sincerely,

Cloudflare support team is giving the same answer for past 2 months :-) . Hope they will solve it soon.

Actually for the past 2 months I was receiving the "we are looking into it" answer. They have now acknowledged it and working on a fix. That's what I have been told.

Thank You for the update. We have been waiting so long.

Tell me about it...

And this was the first ticket where they couldn't find the problem.

I released a changed that could have fixed the issue, please test it out.

Not resolved after testing, still Only the desktop version cache is cleared

Are you working at CF? This is probably a server side issue. Nothing to do with plugin, though a cron job may be applied.

I had a look at APO purge issue when "Cache By Device Type" is enabled. I can't reproduce the issue without more details ( purging of all variants works fine on our test setup).
There is a Cloudflare Purge API quirk for "Cache By Device Type": when the purge happens it only purges for 1 particular device type variant not all of them. So internally APO team supports additional infrastructure to listen to Purge requests, detects "Cache By Device Type" and purges the remaining variations. I hope this explains why there is no special treatment of "Cache By Device Type" in the WordPress plugin.

Earlier mobile and desktop version VIEW were conflicting, so we were getting Desktop view in mobile and mobile view in desktop.
then Cloudflare has released Cache By Device Type, and it has solved the VIEW conflicting, but after that if we made any changes in the existing article or created an article, the cache is updating only in desktop version but not mobile.
This is the problem everyone is facing / complaining now. Hope CF will take serious action on this.
Thank You
Giri

Unfortunately the change was reverted due to increased load on Cache purge infrastructure. We will engage with Cache and API team to find a solution for this issue.

If you would like to beta test the potential fix on your site please DM me at https://community.cloudflare.com/u/yevgen with the details. I will be able to enable the change specifically for the selected domains.

I logged in but could not find a button to start a conversation...

If you would like to beta test the potential fix on your site please DM me at https://community.cloudflare.com/u/yevgen with the details. I will be able to enable the change specifically for the selected domains.

I haven't renewed the APO subscription again due to cache issue. Hope @coolrecep is working with you.
If you need help let us know.. will renew again and check.

I'm a PRO custome but I can not PM him. Go figure...

I'm a PRO customer but I can not PM him. Go figure...

Me too don't know how to DM.
@sejoker can you explain to us how to do? We would like to fix the issue ASAP.

FYI @coolrecep I've started a thread at https://community.cloudflare.com.

I could not find it either. Link to the thread please.

Where is the thread? I've crawled through your posts and found none. Be specific please!

My bad, I created a new private conversation here: https://community.cloudflare.com/t/apo-cache-purge-issues/291140

Your fix does not seem to be working @sejoker

@coolrecep answered in https://community.cloudflare.com/t/apo-cache-purge-issues/291140.

@coolrecep if you get update and if its worth Pls share here. Thank You

@coolrecep if you get update and if its worth Pls share here. Thank You

Not working for me right now but I'm sure we'll figure it out.

@coolrecep if you get update and if its worth Pls share here. Thank You

Not working for me right now but I'm sure we'll figure it out.

Thank You @coolrecep for the update. If it's working, Pls don't forget to update here :-)

Any update please @sejoker
Is there anything to beta test? My sites have 50% mobile views so it's sort hurting not having the mobile cache not working as I am sure many others are in the same boat.
Thanks for reading.

It's working fine now. Informed @sejoker as well.

It's working fine now. Informed @sejoker as well.

Awesome to hear, do we need to wait for a plugin update or is the fix live now on the backend with cloudflare?
Thanks for sharing

It's working fine now. Informed @sejoker as well.

Thank You @coolrecep for your update.
Will check

I'm sorry to inform you that I came back to Istanbul and it seems that the issue persists. I tried on my mobile right now and contents are 9 days old!
When I was on another province I tried with Chrome incognito mode with mobile profile, it was working Fine. Back to square one.
I'll keep communicating with @sejoker

I'm sorry to inform you that I came back to Istanbul and it seems that the issue persists. I tried on my mobile right now and contents are 9 days old!
When I was on another province I tried with Chrome incognito mode with mobile profile, it was working Fine. Back to square one.
I'll keep communicating with @sejoker

Yes . Its not working 🙁.
Pls update once you got the solution from @sejoker

@sejoker do we have any updates please ? The problem with Purging Cache when "Cache by device type" is enabled still persists

@sejoker please any news?

Disabling Cache By Device Type solves the problem but this time, mobile version of the site is served even in desktop mode.
Cloudflare rep sent me an email last week, I told them about this problem and no reply yet...
It has been a year god damn it! I'm being helpful here but you are not providing any fix. I have provided all the information you need.

Disabling Cache By Device Type solves the problem but this time, mobile version of the site is served even in desktop mode.
Cloudflare rep sent me an email last week, I told them about this problem and no reply yet...
It has been a year god damn it! I'm being helpful here but you are not providing any fix. I have provided all the information you need.

Yeah the cache by device is what I did but it aint ideal, if we been waiting a year already then I am stopping all my subs next month which I got several, finding a better solution its a shame but this ain't good enough, cloudflare too big don't care, it's what happens sadly.

Disabling Cache By Device Type solves the problem but this time, mobile version of the site is served even in desktop mode.
Cloudflare rep sent me an email last week, I told them about this problem and no reply yet...
It has been a year god damn it! I'm being helpful here but you are not providing any fix. I have provided all the information you need.

@coolrecep When APO introduced the desktop and mobile version conflict issue was there and many complained then only they introduced "Cache by device type" which is solved the cache (desktop version in mobile and mobile version in desktop) conflict problem, but after that mobile version cache is not updating which we are all facing the issue now and complaining.
If we logged in to our account it's showing the updated cache version, but if we are checking the site without logged in then it's not showing the updated version.
Even If anyone is commented in the article, it's not showing the comment, so we have to clear the particular article cache then only it will be visible to all.
I'm not sure why CF is taking so much time to fix the issue. If it's a free service means we may not force them to do, but it's a paid service but still they don't care about it.
Even I have raised two tickets and explained everything and many like you explained and supported but no response.
It's really disappointed and frustrating.

Can't sign up B/C it's a part of APO.
https://blog.cloudflare.com/automatic-signed-exchanges/

I hear the frustration (well earned) from you folks. At the moment our team is resource-limited and fully preoccupied with launching Cloudflare Images product. We are actively hiring to expand APO team so hopefully in September/October we be in the position to address purging issues for "Cache By Device Type" and extend APO functionality further (better caching rate, support for cdn-cache-control etc.)

Hello @sejoker Hope the Cloudflare Images has been implemented.
Now the October first week. We are expecting to solve the cache issue. Its difficult to clear the cache every time and proceed.
Can you Pls look in to the issue?
Thanks
Giri

@girirajnet thanks for flagging this. The fix will work out of box for new customers who will activate APO "Cache by Device Type". For existing customers the additional step is necessary:

after the upgrade to 4.6.0 please navigate APO card in Cloudflare plugin and make sure APO "Cache by Device Type" is activated.
after that any new content change should send purge requests both on desktop and mobile devices.

I recommend purge all Cache once to make sure you content is in sync for desktop and mobile variants.
@girirajnet please tell more:

Do you have "Purge cache on mobile" enabled?
Have you navigated to APO card in the plugin?
Have you purged all cache (won't be necessary in all cases, but probably worth it to do once)?
Do you experience a different use case as @rvola originally described? It could be an additional use case we need to look into.

Hello @sejoker
Here are my answers
"Do you have "Purge cache on mobile" enabled?"
I don't have this option. I have options only "Purge Everything and Purge Individual files"
"Have you navigated to APO card in the plugin?"
Yes.
"Have you purged all cache (won't be necessary in all cases, but probably worth it to do once)?"
Yes. I have done in from APO plugin and web interface too.
"Do you experience a different use case as @rvola originally described? It could be an additional use case we need to look into"
The changes are updating properly in the desktop but not in mobile. This is the issue.
After complete purge, I made changes in the article but the changes reflecting in desktop but not in mobile. If I purge a particular article link then it's showing mobile too.
@coolrecep Happy to hear that the issue is resolved for you but am not sure why its not working for me.
Thanks
Giri

You may need to post an article or clear the cache if there is any. That was the solution for us at least.

Do you have "Purge cache on mobile" enabled?

@girirajnet  I meant APO's "Cache By Device Type" option enabled?

If I purge a particular article link then it's showing mobile too.

This is suspicious, by default the purge only applies to desktop variant. It could be unrelated to the issues discussed here.
What mobile device you use for testing?

Cache By Device Type

Yes @sejoker Cache By Device Type is enabled.
Particular article means its an example link only.
I have tested a few other articles as well. The changes are reflecting properly in mobile for the first time, but if I update the article again, it's not showing the updated version.
it works only for first attempt. I'm not sure should I change anything the settings?
Browser Cache TTL I set as 2 minutes.
I'm using OnePlus 8 and I checked in my friend mobile too. Tried different browsers and with private mode. Looks same.

could you send an User-Agent header you have in OnePlus 8?

There are plenty of sites that can show it, for example: https://whatmyuseragent.com/

I can confirm this user-agent should be using Mobile variant in cache. It's a mystery to me why cache purging doesn't work from your side.

STILL NOT WORKING.  VERY FRUSTRATING!!!!!!!!


would you move the phantomjs dependency to devDependencies?

would you move the phantomjs dependency to devDependencies?

@iambumblehead Duplicate of #100, pls see there for details.

@jkphl thanks for the response --understood :)


Readme, gitignore, and /data/

This PR:

adds some text to the readme file (including the directory structure diagram)
adds a gitignore
sends example figs from make_dataset and train_resunet to  /data instead of the top level directory and examples
i did not spend time to do any mkdir commands to further route these files into new directories... they are dumped into data...

I think we need to test this from a clean git clone... ... w/ the gitignore, i think this might remove folders that are necessary for routing files... so some well placed  os.mkdir might be needed...


[flutter_tools] make verbose macOS builds actually verbose

Description
The VERBOSE_SCRIPT_LOGGING config was never set, causing verbose mode to not forward logs from re-entrant flutter or to place re-entrant flutter in verbose mode.
Related to https://github.com/flutter/flutter/issues/33582

Ahh good point.


Could not get your pretrained weights on google drive disk!

Could you share the weights you trained on BaiduyunDisk or change the access permission of the file shared on google disk?

sorry! I get it, thank you for your nice work!!


Test CheckForServiceAvailability

Will close #23
Tag the repo with in increased path version after merging this.

Yes this was already there, I just cleaned the code and changed paths to Github.
Where would you store the yml file?

Maybe a separate branch?
Let's merge this as is and discuss next week.

I have the feeling as well this is added complexity for no real reason. A better way to do this, IMHO, would be to make the BE return the unaivalabiltity and log it here (or maybe just simply notify the users). TBH I would get rid of this logic, but indeed we can do in another PR.
Something else though. I think this requires the file ...available.txt which is missing from the release. So this will always fail right? should we merge the one with it first?

think this requires the file ...available.txt which is missing from the release.

Sorry I don't see what file you're talking about. It checks the service availability from datasetIngestorServiceAvailability.yml.

yes, that one. Ah ok I see it now on github. I got confused by this and was looking for it in the release

Ok then I'll merge this one and later we can talk about hosting the yml file on a different branch.


HDDS-8739. Snapdiff should return complete absolute path in Diff Entry

What changes were proposed in this pull request?
https://docs.google.com/document/d/15eLimSG0Tre4qfe1D5PeSWMeZ9Si0uENUt1YcF21LeQ/edit?usp=sharing
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-8739
How was this patch tested?
Unit tests & integration tests

@swamirishi
missing license: https://github.com/apache/ozone/actions/runs/5295547422/jobs/9586397706
checkstyle: https://github.com/apache/ozone/actions/runs/5295547422/jobs/9586397810


ImportError: cannot import name '_C' from partially initialized module 'torchsdf' (most likely due to a circular import)

Traceback (most recent call last):
File "setup.py", line 176, in
ext_modules=get_extensions(),
File "setup.py", line 135, in get_extensions
include_dirs = get_include_dirs()
File "setup.py", line 160, in get_include_dirs
_, bare_metal_major, _ = get_cuda_bare_metal_version(CUDA_HOME)
File "setup.py", line 59, in get_cuda_bare_metal_version
raw_output = subprocess.check_output([cuda_dir + "/bin/nvcc", "-V"], universal_newlines=True)
TypeError: unsupported operand type(s) for +: 'NoneType' and 'str'
Traceback (most recent call last):
File "", line 1, in
File "/home/ustc/PycharmProjects/DexGraspNet/TorchSDF/torchsdf/init.py", line 1, in
from torchsdf.sdf import *
File "/home/ustc/PycharmProjects/DexGraspNet/TorchSDF/torchsdf/sdf.py", line 2, in
from torchsdf import _C
ImportError: cannot import name '_C' from partially initialized module 'torchsdf' (most likely due to a circular import) (/home/ustc/PycharmProjects/DexGraspNet/TorchSDF/torchsdf/init.py)
when I tried bash install.sh,it occured

Sorry for replying so late. I tried to install torchsdf on three machines (two Ubuntu servers and one docker container on my Windows laptop) with conda, and didn't encounter such circular import, failing to repeat this error. Can you tell me more about your operations? Or you can try bash clean.sh before installation. Or you can try pip install -e .. Or you can manually modify csdf/__init__.py. Hope this can help you a little.

Sorry for replying so late. I tried to install torchsdf on three machines (two Ubuntu servers and one docker container on my Windows laptop) with conda, and didn't encounter such circular import, failing to repeat this error. Can you tell me more about your operations? Or you can try bash clean.sh before installation. Or you can try pip install -e .. Or you can manually modify csdf/__init__.py. Hope this can help you a little.

conda create -n graspnet python=3.8
conda activate graspnet
install pytorch and pytorch3d：
conda install pytorch==1.11.0 torchvision==0.12.0 torchaudio==0.11.0 cudatoolkit=11.3 -c pytorch
conda install -c fvcore -c iopath -c conda-forge fvcore iopath
pip install --no-index --no-cache-dir pytorch3d -f https://dl.fbaipublicfiles.com/pytorch3d/packaging/wheels/py38_cu113_pyt1110/download.html
Package：
conda install transforms3d -c conda-forge
conda install plotly -c conda-forge
conda install trimesh -c conda-forge
pip install urdf_parser_py
pip install scipy
pip install networkx
conda install rtree
cd Projects/cd DexGraspNet/
sent documents to my server：
scp -P 25422 -r /home/ustc/PycharmProjects/DexGraspNet/TorchSDF chenchangmao@<IP_ADDRESS>:/home/chenchangmao/Projects/DexGraspNet
do：
cd TorchSDF/
git checkout 0.1.0
bash install.sh
error:
RuntimeError:
The detected CUDA version (9.1) mismatches the version that was used to compile
PyTorch (11.3). Please make sure to use the same CUDA versions
ImportError: cannot import name '_C' from partially initialized module 'torchsdf'
That's all that I have done.

Sorry for replying so late. I tried to install torchsdf on three machines (two Ubuntu servers and one docker container on my Windows laptop) with conda, and didn't encounter such circular import, failing to repeat this error. Can you tell me more about your operations? Or you can try bash clean.sh before installation. Or you can try pip install -e .. Or you can manually modify csdf/__init__.py. Hope this can help you a little.

conda create -n graspnet python=3.8 conda activate graspnet install pytorch and pytorch3d： conda install pytorch==1.11.0 torchvision==0.12.0 torchaudio==0.11.0 cudatoolkit=11.3 -c pytorch conda install -c fvcore -c iopath -c conda-forge fvcore iopath pip install --no-index --no-cache-dir pytorch3d -f https://dl.fbaipublicfiles.com/pytorch3d/packaging/wheels/py38_cu113_pyt1110/download.html
Package： conda install transforms3d -c conda-forge conda install plotly -c conda-forge conda install trimesh -c conda-forge pip install urdf_parser_py pip install scipy pip install networkx conda install rtree cd Projects/cd DexGraspNet/
sent documents to my server： scp -P 25422 -r /home/ustc/PycharmProjects/DexGraspNet/TorchSDF chenchangmao@<IP_ADDRESS>:/home/chenchangmao/Projects/DexGraspNet
do： cd TorchSDF/ git checkout 0.1.0 bash install.sh
error: RuntimeError: The detected CUDA version (9.1) mismatches the version that was used to compile PyTorch (11.3). Please make sure to use the same CUDA versions
ImportError: cannot import name '_C' from partially initialized module 'torchsdf' That's all that I have done.
I also tried bash clean.sh and pip install -e.,it didn't work.

I notice that "The detected CUDA version (9.1) mismatches the version that was used to compile PyTorch (11.3). Please make sure to use the same CUDA versions".
Maybe you can check your cuda complier version by nvcc -V. For my two tested Ubuntu servers, they are "Cuda compilation tools, release 11.7, V11.7.64" and "Cuda compilation tools, release 11.3, V11.3.58". Note that "cuda compilation tools" is not "cudatoolkit" installed by conda, but installed on your native system. So I guess the error is due to the lower version of your cuda compilation tools.
Maybe you can have a try to install torchsdf (just pytorch and torchsdf) on a machine with higher cuda version to see if the same error would happen. Or you can try to install a higher version cuda on your working machine. (For exmaple, https://developer.nvidia.com/cuda-11.3.0-download-archive) By "runfile(local)" installation, you can either ask sudoer to update cuda (though I do not recommand), or install cuda into your user environment.
Hope these can help you.

I notice that "The detected CUDA version (9.1) mismatches the version that was used to compile PyTorch (11.3). Please make sure to use the same CUDA versions".
Maybe you can check your cuda complier version by nvcc -V. For my two tested Ubuntu servers, they are "Cuda compilation tools, release 11.7, V11.7.64" and "Cuda compilation tools, release 11.3, V11.3.58". Note that "cuda compilation tools" is not "cudatoolkit" installed by conda, but installed on your native system. So I guess the error is due to the lower version of your cuda compilation tools.
Maybe you can have a try to install torchsdf (just pytorch and torchsdf) on a machine with higher cuda version to see if the same error would happen. Or you can try to install a higher version cuda on your working machine. (For exmaple, https://developer.nvidia.com/cuda-11.3.0-download-archive) By "runfile(local)" installation, you can either ask sudoer to update cuda (though I do not recommand), or install cuda into your user environment.
Hope these can help you.

Thank you very much! I replace cuda complier 9.1 with version 11.3, it worded.


Update Grafana image version to 7.5.15

Many thanks for submitting your Pull Request :heart:!
Grafana operator has update ganfana image version so we need to update it as well.
Please make sure your PR meets the following requirements:

[ ] You have read the contributors' guide
[ ] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[ ] Pull Request contains a link to the JIRA issue
[ ] Pull Request contains a description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've ran make before-pr and everything is working accordingly
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

How to retest this PR or trigger a specific build:

Run operator BDD testing
Please add comment: /jenkins test

Run RHPAM operator BDD testing
Please add comment: /jenkins RHPAM test

/jenkins test


Dynamic Event Registration

Dynamic Event Registration
This PR enables any event to itself append to existing events or even overwriting existing events.
To accomplish this, this PR does the following:

mf (the same object as is used during register(mf) to the list of possible arguments that are automatically determined and passed to an event upon execution
mf.overwrite_event to redefine previous definitions completely
mf.register_event clears the cache of the event-object automatically
adds a simple test case for event registration during an event

correct mf is an optional, automatically passed argument as state and event
is un-registration a desired feature? if so, it would be easy to make the api public. The function is already present in this PR.

I think un-registration might come in handy.

I think un-registration might come in handy.

Done.

I think un-registration might come in handy.

Done.


checkov: 3.2.74 -> 3.2.79

Diff: https://github.com/bridgecrewio/checkov/compare/refs/tags/3.2.74...3.2.79
Changelog: https://github.com/bridgecrewio/checkov/releases/tag/3.2.79
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 309013 run on x86_64-linux 1

2 packages built:

checkov
checkov.dist


Port to C#, collision example between convex bodies

Hi can it be used in C# application (Nuget) ? Is there an example for collision check between the convex bodies after decomposition ?

Collision checking is usually done by running a physics simulation. There is no C# support


Dev/normalization/win event sync

Fixes #
Proposed Changes
-adding network for windowEvent deployment

@oshezaf

@v-rucdu : those are intentional. The files block cloning the repo. Did you see @YaronFruchtmann e-mail on the topic?

@v-rucdu : those are intentional. The files block cloning the repo. Did you see @YaronFruchtmann e-mail on the topic?

@oshezaf : Yes, Yaron has fixed that issue and merged it in master... If we merge these changes as is these files will also be deleted... Thanks!!!

@oshezaf now the PR is clean


Update eni-max-pods.txt

Issue #, if available:
Description of changes:
This adds support for i4i instance types (among others).
Steps:

Clone https://github.com/aws/amazon-vpc-cni-k8s
Using an AWS account that is opted-in to all regions:

go run scripts/gen_vpc_ip_limits.go

Results in misc/eni-max-pods.txt.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I built a 1.22 AMI (make 1.22) from this branch and verified that I could create a single-node group with an i4i.large on said AMI.


APPSRE-7135: Add support for read replicas waiver to GABI schema

Add support for the read replicas requirement waiver to GABI schema through a new optional string type attribute called readReplicasWaiverPledge that will be used to store a free-form waiver pledge text.
Related:

APPSRE-7135
!78208

Signed-off-by: Krzysztof Wilczyński<EMAIL_ADDRESS>

Please add the attribute to graphql-schemas/schema.yml as well 🙇

Please add the attribute to graphql-schemas/schema.yml as well 🙇

@chassing, done. Thank you!


Added optional flag for ignoring ssh error "no supported methods remain"

In some cases, while terraform provisioning machine, ssh daemon starting before applying working configuration.
We had real case with Ubuntu automated installer, ssh starting before installation compleated.

LGTM 🚀


normalize pv and gsp coordiantes

Pull Request
Description
normalize pv and gsp coordinates
need PR - https://github.com/openclimatefix/nowcasting_dataset/pull/538
Fixes #106
How Has This Been Tested?
add unittests

[ ] No
[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

we should tell Jacob about this change when he gets back... his code might be expecting absolute (un-normalised) spatial coordinates... I'm not sure...   (the code I'm writing at the moment certainly expects un-normalised spatial coordinates, which then get converted into relative spatial encodings on a per-example basis)

Definitely - I couldnt see anything that would make ti go wrong - but yea, will have to let him know. I just quickyl wanted to put them in cnn model and see if they made any difference
https://app.neptune.ai/o/OpenClimateFix/org/predict-pv-yield/e/PRED-649/charts


Remove 'bigchaindb load' command, tests & docs

The bigchaindb load command is a tool we've used when testing the performance of BigchainDB Server. It writes transactions straight into the backlog table using the RethinkDB/MongoDB Python driver (not using the HTTP API).
bigchaindb load doesn't need to be in the core BigchainDB Server functionality / code base, because it's not something a normal BigchainDB Server operator needs. In the future, we might make a separate, stand-alone tool for bulk-loading the backlog table directly (for use in testing). If we do, we can build that tool based on the bigchaindb load code in the Git history.
Also, bigchaindb load is currently broken, since BigchainDB transactions no longer include a timestamp or random asset ID. See issue #1255.
See also: pull request #1261, which removes the corresponding code in the AWS deployment scripts.

Codecov Report

Merging #1273 into master will increase coverage by 0.77%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1273      +/-   ##
==========================================
+ Coverage   97.18%   97.96%   +0.77%
==========================================
  Files          51       51
  Lines        2380     2353      -27
==========================================
- Hits         2313     2305       -8
+ Misses         67       48      -19

Continue to review full report at Codecov.

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


segfault in unpack_fused.unpack_one  or unpack_fused.unpack_two_int doctests

Happens on amd64 arch across a good range of ubuntu releases and debian 8 (jessie)
cython_0.26~b0-1~nd14.04+1_amd64.build-unpack_one (unpack_fused)
cython_0.26~b0-1~nd14.04+1_amd64.build:Doctest: unpack_fused.unpack_one ... Segmentation fault (core dumped)
cython_0.26~b0-1~nd14.04+1_amd64.build-make[1]: *** [override_dh_auto_test] Error 139
--
cython_0.26~b0-1~nd16.04+1_amd64.build-unpack_two_int (unpack_fused)
cython_0.26~b0-1~nd16.04+1_amd64.build:Doctest: unpack_fused.unpack_two_int ... Segmentation fault (core dumped)
cython_0.26~b0-1~nd16.04+1_amd64.build-debian/rules:112: recipe for target 'override_dh_auto_test' failed
--
cython_0.26~b0-1~nd16.10+1_amd64.build-unpack_two_int (unpack_fused)
cython_0.26~b0-1~nd16.10+1_amd64.build:Doctest: unpack_fused.unpack_two_int ... Segmentation fault (core dumped)
cython_0.26~b0-1~nd16.10+1_amd64.build-debian/rules:112: recipe for target 'override_dh_auto_test' failed
--
cython_0.26~b0-1~nd17.04+1_amd64.build-unpack_two_int (unpack_fused)
cython_0.26~b0-1~nd17.04+1_amd64.build:Doctest: unpack_fused.unpack_two_int ... Segmentation fault (core dumped)
cython_0.26~b0-1~nd17.04+1_amd64.build-debian/rules:112: recipe for target 'override_dh_auto_test' failed
--
cython_0.26~b0-1~nd80+1_amd64.build-unpack_one (unpack_fused)
cython_0.26~b0-1~nd80+1_amd64.build:Doctest: unpack_fused.unpack_one ... Segmentation fault (core dumped)
cython_0.26~b0-1~nd80+1_amd64.build-debian/rules:112: recipe for target 'override_dh_auto_test' failed

it is not the case on debian stretch or unstable.
Full build logs are at http://neuro.debian.net/_files/_buildlogs/cython/0.26~b0

Is this new in 0.26?

I think so

Could be a duplicate of #1764.

Ok, I will try to rebuild updated one later today

I think it is gone now . uploaded  0.26~b0+git23-ga4fb856ac  version to debian experimental BTW


[DevOps] Test AirGap VLANs in Lab

Description
We have had some AirGap VLANs that should be blocking outbound traffic.
How To Test

Have a harvester cluster set up (any nodes)
Have a VM running in each of these VLANs:
a. 2011
b. 2012
c. 2015
d.  2016
e. 2017
f. 2021
Use a pre-built QCOW2 Image, with something like Packer as we need a VM with (qemu-guest-agent) and additionally any other packages aready set up, as this VM will be deployed into AirGap Networks, so outbound communication should not succeed:
a. 2013
b. 2014
Ensure the VMs in 2013 & 2014 can not make outbound traffic eg: (shouldn't ping <IP_ADDRESS>, shouldn't curl/wget an ubuntu cloud image, shouldn't communicate to an ntp server)
Ensure the VMs in 2013 & 2014, can ping / download a file from 2011, 2012, 2015, 2016, 2017, 2021 VMs
Update Confluence Page & Close Out: https://github.com/rancherlabs/harvester-access-lab/issues/4

It should be in QA sprint not QA task. :D
I'll remove the QA tasks project to eliminate the confuse.

@khushboo-rancher thanks for correcting that :smile:

2021, is operating as expected :smile: - I'll go ahead and close this out :+1:
Thanks @ibrokethecloud :smile:


Fixed test for machines using comma as decimal separator

Some cultures e. g. German cultrue use a comma and not a period as decimal separator. Thus 12.5 becomes 12,5 and the test fails. Using the invariant culture avoids the problem here.

@pplcc ty for the nice contribution. i like it.


make std::pair arrays roundtrip

std::pair is used to write out key value pairs and concatenate when in arrays to make objects. This allows dynamic keys without a normal map structure.
I think that these arrays of std::pair should roundtrip. The hashing algorithm should then use a search algorithm like binary searching. This eliminates the memory overhead of a hash table.
This is an API breaking change.

Binary searching is not possible because the keys are not guaranteed to be sorted. A flat map would be better. Instead in these cases we should just clear the vector and replace it with the input.


Tuple sketch

This is an implementation of generic tuple sketch, specialized array of doubles sketch (with serialization compatible with Java equivalent) and experimental theta sketch on top of the same base layer (it should eventually replace the existing theta sketch). Perhaps some reorganization might help to separate this large set of files into several groups (for instance, moving the common base to a separate directory).

The last few edits look good. Merge away!


Run failing is some OA11 from Wales and England

This issue has been seen in some Wales regions: e.g. W06000004 and W06000019 (there might be others, these are the ones that i've seen fail.
Failing in the household_microsynth step, with error:
...............Traceback (most recent call last):   File "/mnt/batch/tasks/workitems/SPCJobTest_England10/job-1/Task5_W06000019/wd/household_microsynth/scripts/run_microsynth.py", line 100, in do_hh     msynth.run()   File "/mnt/batch/tasks/workitems/SPCJobTest_England10/job-1/Task5_W06000019/wd/miniconda/envs/spc_env/lib/python3.9/site-packages/household_microsynth-0.0.1-py3.9.egg/household_microsynth/household.py", line 71, in run     self.__add_households(area, constraints)   File "/mnt/batch/tasks/workitems/SPCJobTest_England10/job-1/Task5_W06000019/wd/miniconda/envs/spc_env/lib/python3.9/site-packages/household_microsynth-0.0.1-py3.9.egg/household_microsynth/household.py", line 143, in __add_households     p0 = humanleague.qisi(constraints, [np.array([0, 1, 2]), np.array([0, 3, 2]), m4408dim], [m4404, m4405, m4408]) RuntimeError: pick failed: 0.000000 from [0.000000, 0.000000, 0.000000, 0.000000, 0.000000]
More details in running logs from Azure:
stderr_W06000019.txt
stdout_W06000019.txt

Update with list of LADs where this is an issue:

E07000238

After some investigations, it turns out that the problem occurs in line 143 of the household.py script.
If we look more in detail, the next lines in the script (line 147) deals with convergence issues from line 143. I'm not sure if what we observe from line 143 is a conversion error but looks kind of like it (maybe is failing instead of returning the right error message).
If this was the case, the fix is simple, replacing line 143 with the following:
try:
    p0 = humanleague.qisi(constraints, [np.array([0, 1, 2]), np.array([0, 3, 2]), m4408dim], [m4404, m4405, m4408])
except RuntimeError:
    p0 = None

this solves the problem because it makes it a convergence issue, and line 149 runs without problems.
I have tested this locally with some of the failing LADs (W06000004, E07000238) and runs successfully.

the library which this depends on has an invalid C function which fails to return anything in certain conditions, related to the previous comment: https://github.com/virgesmith/humanleague/blob/7ba7203ed79ff1f3ab33cda812bd45335a3a5876/src/QISI.cpp#L23 - As Camila has mentioned, the easiest fix is to update household.py rather than dig into 2nd order libraries.


Ensure all SQL writes are in try/catch

Traceback (most recent call last):
 File "/Users/gfitzp/Sites/fitzflix/venv/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 1203, in _execute_context
   conn = self._revalidate_connection()
 File "/Users/gfitzp/Sites/fitzflix/venv/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 472, in _revalidate_connection
   raise exc.InvalidRequestError(
sqlalchemy.exc.InvalidRequestError: Can't reconnect until invalid transaction is rolled back

2023-07-09 10:14:36 13450 [Warning] Aborted connection 13450 to db: 'fitzflix_db' user: 'fitzflix_user' host: 'localhost' (Got an error reading communication packets)

https://severalnines.com/blog/common-mysql-error-got-error-reading-communication-packet/
Use mysqld --help --verbose for checking configuration settings

Removed DB connection pooling in 9eb2665

Even without connection pooling got error:
sqlalchemy.exc.OperationalError: (pymysql.err.OperationalError) (2006, "MySQL server has gone away (BrokenPipeError(32, 'Broken pipe'))")
[SQL: SELECT [file.id](http://file.id/) AS file_id, file.untouched_basename AS file_untouched_basename, file.media_library AS file_media_library, file.file_path AS file_file_path, file.dirname AS file_dirname, file.basename AS file_basename, file.plex_title AS file_plex_title, file.movie_id AS file_movie_id, file.feature_type_id AS file_feature_type_id, file.series_id AS file_series_id, file.season AS file_season, file.episode AS file_episode, file.last_episode AS file_last_episode, file.version AS file_version, file.quality_id AS file_quality_id, file.fullscreen AS file_fullscreen, file.crop AS file_crop, file.container AS file_container, file.format AS file_format, file.codec AS file_codec, file.video_bitrate_kbps AS file_video_bitrate_kbps, file.date_added AS file_date_added, file.date_updated AS file_date_updated, file.date_localized AS file_date_localized, file.date_transcoded AS file_date_transcoded, file.date_archived AS file_date_archived, file.aws_untouched_key AS file_aws_untouched_key, file.aws_untouched_date_uploaded AS file_aws_untouched_date_uploaded, file.aws_untouched_date_deleted AS file_aws_untouched_date_deleted
FROM file]

sqlalchemy.exc.PendingRollbackError: Can't reconnect until invalid transaction is rolled back. (Background on this error at: https://sqlalche.me/e/14/8s2b)

Might be fixed in 2449306 ?

Issue hasn't occurred in a while, closing ticket 🤞🏻


Parse error from pipenv lock --requirements

Say, if requirements.txt is generated from pipenv lock --requirements, it would contain a like which looks like flags:
-i https://pypi.org/simple/

this results parse error:
pkg_resources.extern.packaging.requirements.InvalidRequirement: Parse error at "'-i https'": Expected W:(abcd...)

Input file in question:

https://github.com/Taxel/PlexTraktSync/blob/9f1806e2cf67ceb6e9309e97e6ecbae6c29113c7/requirements.txt

Full trace included of the failure:

➔ python -m build
* Creating venv isolated environment...
* Installing packages in isolated environment... (setuptools-declarative-requirements, setuptools>=42, wheel)
* Getting dependencies for sdist...
Traceback (most recent call last):
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/packaging/requirements.py", line 98, in __init__
    req = REQUIREMENT.parseString(requirement_string)
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/pyparsing.py", line 1654, in parseString
    raise exc
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/pyparsing.py", line 1644, in parseString
    loc, tokens = self._parse( instring, 0 )
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/pyparsing.py", line 1402, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/pyparsing.py", line 3417, in parseImpl
    loc, exprtokens = e._parse( instring, loc, doActions )
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/pyparsing.py", line 1402, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/pyparsing.py", line 3739, in parseImpl
    return self.expr._parse( instring, loc, doActions, callPreParse=False )
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/pyparsing.py", line 1402, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/pyparsing.py", line 3400, in parseImpl
    loc, resultlist = self.exprs[0]._parse( instring, loc, doActions, callPreParse=False )
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/pyparsing.py", line 1406, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/pyparsing.py", line 2711, in parseImpl
    raise ParseException(instring, loc, self.errmsg, self)
pkg_resources._vendor.pyparsing.ParseException: Expected W:(abcd...) (at char 0), (line:1, col:1)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/glen/.cache/direnv/PlexTraktSync/python-3.10.0/lib/python3.10/site-packages/pep517/in_process/_in_process.py", line 349, in <module>
    main()
  File "/Users/glen/.cache/direnv/PlexTraktSync/python-3.10.0/lib/python3.10/site-packages/pep517/in_process/_in_process.py", line 331, in main
    json_out['return_val'] = hook(**hook_input['kwargs'])
  File "/Users/glen/.cache/direnv/PlexTraktSync/python-3.10.0/lib/python3.10/site-packages/pep517/in_process/_in_process.py", line 284, in get_requires_for_build_sdist
    return hook(config_settings)
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/setuptools/build_meta.py", line 159, in get_requires_for_build_sdist
    return self._get_build_requires(config_settings, requirements=[])
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/setuptools/build_meta.py", line 135, in _get_build_requires
    self.run_setup()
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/setuptools/build_meta.py", line 150, in run_setup
    exec(compile(code, __file__, 'exec'), locals())
  File "setup.py", line 1, in <module>
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/setuptools/__init__.py", line 153, in setup
    return distutils.core.setup(**attrs)
  File<EMAIL_ADDRESS>line 121, in setup
    dist.parse_config_files()
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/declarative_requirements/integration.py", line 104, in wrapper
    dist._finalize_requires()
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/setuptools/dist.py", line 528, in _finalize_requires
    self._move_install_requirements_markers()
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/setuptools/dist.py", line 567, in _move_install_requirements_markers
    inst_reqs = list(pkg_resources.parse_requirements(spec_inst_reqs))
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3084, in parse_requirements
    yield Requirement(line)
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3094, in __init__
    super(Requirement, self).__init__(requirement_string)
  File "/private/var/folders/jm/_gwj2_bj37g_qc4pj1xvk6fw0000gp/T/build-env-j87mq569/lib/python3.10/site-packages/pkg_resources/_vendor/packaging/requirements.py", line 100, in __init__
    raise InvalidRequirement(
pkg_resources.extern.packaging.requirements.InvalidRequirement: Parse error at "'-i https'": Expected W:(abcd...)

ERROR Backend subproccess exited when trying to invoke get_requires_for_build_sdist

@s0undt3ch: is this project dead? what alternative route I should use?
while releasing the package, I added a workaround for CI, but now it's blocking installing from git repo directly:

https://github.com/Taxel/PlexTraktSync/pull/838#issuecomment-1066712376

No, the project is not dead and it's being actively used in several projects.
Time to look at these issues is however scarce.
Those lines should be skipped, should be simple to add that and submit a PR?

https://github.com/s0undt3ch/setuptools-declarative-requirements/pull/7, but I don't know how to test.


there is a bug with swackets package

when I install the swackets package this automatically deletes all my data in the index.html file.
my index.html file is not responding even my close atom and I reopen it.
when I uninstall swackets package after all is fine my index.html is responding well, so I think there is a problem with the swackets package.

Should be fixed now update swackets and atom! Let me know please thanks!


Missing text in multiple places

In multiple places on this page, when it gives a code snippet example and then says "Where:" then has a list of bullets that describe what each tag in angled brackets is for, most of the tags are missing in the bullets.

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

ID: c2f60077-848f-40d7-8ac9-b0b6bcd8201b
Version Independent ID: 07bdb9d0-cec1-e4b5-bd41-a5f5867a8bf8
Content: Configure recommendations and usage event types in SharePoint Server
Content Source: SharePoint/SharePointServer/administration/configure-recommendations-and-usage-event-types.md
Product: sharepoint-server-itpro
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

@AndreaBarr please assign to me.

@ttplaya Thank you for submitting this feedback.
From our understanding the issue has been resolved. If you feel it hasn't been resolved please re-open this issue.
@officedocsbot close


[FIX] clarify that scans.json is allowed and recommended

closes #450
Explicitly say that scans.json is allowed and recommended if any arbitrary column beyond filename and acq_time are included.
As discussed in #450 this seems already to be controlled in the validator (to some extend) and reading the remaining spec, this particular style of handling "arbitrary columns" is not new --> thus, I see this rather as a clarification than an enhancement.

@yarikoptic please check once more, I addressed your comments.


Annotate widgets with a category and subcategory to autogenerate widget catalog

If our widget classes had annotations for category and subcategory, we could:

extract a widget catalog, and allow our users to slice and dice based on what they are looking for
explore organization options for docs.flutter.io

Ideally, these annotations are extractable by a tool. For example, dart annotations or very-easy-to-extract doc comments.

cc @Hixie if you can design the annotation for category and subcategory, we can look to use that design for the new canonical categorization. Any thoughts?

I'm happy to add the API that would enable the annotation, but I'm not sure what the categories should be and I'd rather not be the one who has to go through every class adding them. :-)

Right, so good news:

we know the categories and subcategories (or, at least we have a first attempt). See: https://github.com/flutter/website/blob/master/_data/catalog/widgets.json

you don't have to add them all.

If you can add a few, we can then follow your pattern. What we need from you is "how to add a category/subcategory to a widget class"
Thanks!

will do

We wish to specify one or more hierarchy paths per widget.
For example, for RaisedButton, we would say: "belongs in Basic/Buttons and Material Design/Buttons"
Up to you for exactly how to specify this, but we would like to be able to assign each widget to one or more category/sub-category.
Once you do one or two, we'll take care of the rest :)
Thank you!

More details: we prefer open-ended Strings.

How's this?
/// A material design "raised button".
///
/// A raised button consists of a rectangular piece of material that hovers over
/// the interface.
///
/// Use raised buttons to add dimension to otherwise mostly flat layouts, e.g.
/// in long busy lists of content, or in wide spaces. Avoid using raised buttons
/// on already-raised content such as dialogs or cards.
///
/// If the [onPressed] callback is null, then the button will be disabled and by
/// default will appear like a flat button in the [disabledColor]. If you are
/// trying to change the button's [color] and it is not having any effect, check
/// that you are passing a non-null [onPressed] handler.
///
/// Requires one of its ancestors to be a [Material] widget.
///
/// If you want an ink-splash effect for taps, but don't want to use a button,
/// consider using [InkWell] directly.
///
/// See also:
///
///  * [FlatButton], a material design button without a shadow.
///  * [DropdownButton], a button that shows options to select from.
///  * [FloatingActionButton], the round button in material applications.
///  * [IconButton], to create buttons that just contain icons.
///  * [InkWell], which implements the ink splash part of a flat button.
///  * <https://material.google.com/components/buttons.html>
@Category(const <String>['Basic', 'Buttons'])
@Category(const <String>['Material Design', 'Buttons'])
@DocumentationIcon('https://material-design.storage.googleapis.com/publish/material_v_9/0B7WCemMG6e0VbDh6YmNiYVc3SHM/components_buttons_usage2.png')
@Summary('A material design raised button. A raised button consists of a rectangular piece of material that hovers over the interface.')
class RaisedButton extends StatelessWidget {
  /// Creates a raised button.
  ///
  /// The [child] argument is required and is typically a [Text] widget in all
  /// caps.
  const RaisedButton({
    Key key,
    @required this.onPressed,
...

yeah! pretty cool, thank you.
cc @InMatrix wdyt ? Will this format have the expressivity required to make the Widgets the canonical source of this categorization?

Yes, it looks great and can potentially be consumed by dartdoc someday. One thing I'm not sure is whether this will make changing or adding categories harder since the categorization information lives in the framework source instead of the website source for either administrative or technical reasons (e.g., testing). If you tell me "no," then I'm happy with switching to annotations in source for widget categorization. Thanks!

I'm not sure what @Summary would be used for in dartdoc since we already have the one-line-doc concept.  (In this case, @Summary would equal dartdoc's one line doc if the first two sentences were in the same paragraph).
Otherwise, @InMatrix is correct, dartdoc could do something with this eventually.

@Summary would be what we show here: https://flutter.io/widgets/material/#Buttons
Notice that this is not quite what we show here: https://docs.flutter.io/flutter/material/material-library.html
I'd also be fine with making those the same and dropping @Summary, I included it just to show that if we wanted to keep them separate we could do so.
My understanding is that the primary purpose of these annotations is to autogenerate the files that are then used to autogenerate the https://flutter.io/widgets pages, and that this would not have any impact on the API docs themselves. I suppose once the data is there though we could find ways to use it in the API docs as well, though (e.g. automatically including the image in the top right, or providing some sort of category-based index in the API docs themselves).
Alternatively I suppose we could move the https://flutter.io/widgets pages entirely into something generated by dartdoc and hosted on docs.flutter.io. I have no opinions on that.

Chatted with @InMatrix .

If you tell me "no," then I'm happy with switching to annotations in source for widget categorization.

API docs will continually be improved and tweaks, don't think there is a concern about locking down the API docs and their metadata.
We can look for the @Summary, if that exists, use it. If it doesn't, use the first line.

https://flutter.io/widgets pages entirely into something generated by dartdoc and hosted on docs.flutter.io.

Convergence is something we'd explore.
I think we're good to go to land this and experiment. Thanks all!

The annotations have landed, so have at it!

With dart-lang/dartdoc#1610 well in hand, there's enough infrastructure that building this support into dartdoc isn't much of a reach. It seems that not much has happened since @Hixie landed the annotation classes (they seem to be unused?), so we're not constrained by that.  Therefore I picked an implementation strategy that works a bit better for dartdoc.
I now have a prototype that can generate something that looks very much like your existing widgets.json.
Given some code and some magic comments:
/// Category information should not follow inheritance.
///
/// {@category Excellent}
/// {@category More Excellence}
/// {@subCategory Things and Such}
/// {@image https://flutter.io/images/catalog-widget-placeholder.png}
/// {@samples https://flutter.io}
class BaseForDocComments {}

Dartdoc will generate the following json:
  {
    "name": "BaseForDocComments",
    "qualifiedName": "fake.BaseForDocComments",
    "href": "fake/BaseForDocComments-class.html",
    "type": "class",
    "categories": [
      "Excellent",
      "More Excellence"
    ],
    "subcategories": [
      "Things and Such"
    ],
    "image": "https://flutter.io/images/catalog-widget-placeholder.png",
    "samples": "https://flutter.io"
  },

I should also be able to fairly easily add links out from the API docs for individual symbols to categories and subcategories by specifying URIs in dartdoc_options.yaml, and include the image given somewhere on the API doc page for that object.  I can also expand the href to a fully-specified URI via configuration entries as well, moving this even closer to an exact replica of widgets.json, and find a way to handle the values built out of HTML too.
If this looks like something we'd be able to use, I'll refine the prototype and get it to something we can ship with.

Yes, this is great! I have some additional questions:

Will this enable a landing page of a widget category within the generated API doc? For example, will DartDoc be able to generate something like https://flutter.io/widgets/material/?
Will the left navigation of the API doc be able to show the categories?

Thanks!

@InMatrix

If you want a landing page (wasn't completely clear from the discussion earlier), one could be generated fairly easily once this structure is in place.  Some options:

I could give you the option of creating your own mustache template so you can load the data out of dartdoc directly into it.
I could also create a default landing page that takes a markdown README specifically for each category -- that would look more like the rest of dartdoc's API docs.
I can make dartdoc link to something external for categories that you build yourself using the json data.
I can make dartdoc do any combination of those things based on options in dartdoc_options.yaml or the command line.

Yes to showing them in navigation, though I haven't figured out yet what a good way to show the categories is when they're mixed in with uncategorized entries.  Ideas welcome on that, but as I get closer to getting the plumbing for this done I'll do some research/experimenting on my own too.

Tagging @kwalrath as I know Angular has maintained pages somewhat similar to the widget catalog and I'd like whatever we build for Flutter to be usable for that too.

Thanks, @jcollins-g.
Tagging @nshahan, who is in charge of https://github.com/dart-lang/angular_components and https://github.com/dart-lang/angular_components_example.

Hi, I just stumbled upon DocumentationIcon, and after lots of Googling this is all that turned up. Can someone point to an example on how to use it?
Also, what is the recommended style for categories? I only just started documentation, and it took me a while to find it, but I'm using
/// {@category <NAME>}
/// {@category <OTHER NAMES>}

Should I be using
/// documentation
@Category (["name", "other names"])
class ...

instead? That's what the Category documentation seems to suggest

Is there a chance we'll be working on this? There is a concrete use case for the Inspector. We're trying to color-code widgets in the treeview based on their category. Some mocks are available at https://github.com/flutter/devtools/issues/3195.


Discord button

Added Discord invite link to website
Signed-off-by: Sprtcrnbry<EMAIL_ADDRESS>

I am going to update the timeline history, will merge together with this PR after finishing it. So I will commit to your branch soon.

@ammarfaizi2 Alright, thanks

Any further review?


A http-middleware implementation

Hello.
Just want to drop here a middleware dispatcher implementation of this interface that I've created here: https://github.com/middleland/dispatcher
As you can see, it's a minimal implementation allowing to use the dispatcher as a middleware, as showed in the examples.
I really like the interface and love to see something similar in the final psr version.

You have a dedicated ClientDispatcher.
I'm afraid this illustrates the problem I've been been trying to explain - because our interface relationships aren't symmetric with those of PSR-7: ServerMiddlewareInterface is not a specialization of MiddlewareInterface, but ServerRequestInterface is a specialization of RequestInterface.
It's not a natural fit for the concepts defined by PSR-7 at all: there's no such thing as a "client request" - the two types are not supposed to be mutually exclusive, and a "server request" is supposed to be interchangible with a "generic request".
I remain convinced that the way we're modeling this is wrong.

Yes, but there's a big difference: In PSR-7 ServerRequestInterface extends RequestInterface adding more methods, not overriding them.
Php does not allow to extend an interface to change the signature of a method defined previously. So, if you have the method process(RequestInterface $request, DelegateInterface $next) and override it with process(ServerRequestInterface $request, DelegateInterface $next) returns a fatal error. This is why two different interfaces are needed. I remember some different approaches discussed to handle this:

Creating the empty interface MiddlewareInterface that is extended by ClientMiddlewareInterface and ServerClientInterface (I'm not a fan of empty interfaces)
Creating just one interface that accepts RequestInterface and throw an exception if the middleware need a ServerRequestInterface but RequestInterface is provided.
Creating two different interfaces (the solution currently implemented)
Extend the MiddlewareInterface adding a new method, instead override it. For example: processRequest() and processServerRequest().

My favorite solution is the second, but I know that is not full typed as the others.

Php does not allow to extend an interface to change the signature of a method defined previously

Well aware of that, yes. The problem is, the type relationship is generic, and there is no way to describe a generic type relationship in PHP at present, not even with a php-doc block. (at least not officially, although etsy/phan is attempting something experimental in that direction.)
The bottom line is that this type relationship cannot be described by the PHP type system.
Attempting to achieve type safety by creating an incorrect type relationship (and by "incorrect", I mean "directly conflicting with PSR-7") does not make for a sound proposal, and doesn't actually achieve full type-safety anyhow, static or run-time.

My favorite solution is the second, but I know that is not full typed as the others

I agree, and I think we have to accept reality for what it is.
In my opinion, the only other acceptable model, is the one interface plus a second identical interface, essentially a marker/tag interface, e.g.:
interface MiddlewareInterface {
    public function process(RequestInterface $request, DelegateInterface $delegate);
}

interface ServerMiddlewareInterface extends MiddlewareInterface {
}

While this does not provide type-safety for server-middleware, at least it correctly mirrors the type-relationships in PSR-7, creating symmetry, such that:
var_dump($server_middleware instanceof ServerMiddlewareInterface); // TRUE
var_dump($server_middleware instanceof MiddlewareInterface);       // TRUE

Contrast that with what we're proposing currently, which means that:
var_dump($server_middleware instanceof ServerMiddlewareInterface); // TRUE
var_dump($server_middleware instanceof MiddlewareInterface);       // FALSE (!)

This makes no sense in the light of PSR-7, where:
var_dump($server_request instanceof ServerRequestInterface); // TRUE
var_dump($server_request instanceof RequestInterface);       // TRUE

At least in this case, the type-relationship is symmetrical with PSR-7 - it just isn't strict enough, but that's impossible given the constraints of the type-system.
At least, the existence of ServerRequestInterface in this case serves to identify middleware, to developers, as requiring a server request, and, assuming developers use it correctly, does provide the benefit in middleware stacks of being able to distinguish middleware tagged as requiring a server request.
Given that this provides no type safety, I would also settle for a single interface - while a single interface doesn't really create symmetry, at least it doesn't directly conflict with the PSR-7 type-model.

I like this model. Is true that does not provide true type safety but the middleware could throw a InvalidArgumentException. The single interface model (my favorite solution) was discarted early in the fig mailing list, and this one suggested by you is similar but a bit better typed. This solution reduce my implementation in just one dispatcher class, and that would be amazing.

I am fine with ServerMiddlewareInterface extends MiddlewareInterface and no type check. To use native type checking, it does require that:
class FooMiddleware extends ServerMiddlewareInterface
{
    public function process(RequestInterface $request, DelegateInterface $delegate)
    {
        $action = $this->getDispatchedAction($request);
        // ...
    }

    private function getDispatchedAction(ServerRequestInterface $request)
    {
        return $request->getAttribute('dispatch.action');
    }
}

This is what we have been doing without formal interfaces and it seems to work out fine. Can trip users up though.


Customize Type for multi choice Fields

I have a document list with a metadatafield, which is configured as Multi choise field. In this case, it is important to set the correct type.
I have to set the type like this:
{"ContractpartnerId":{"results":[2]},"__metadata":{"type":"SP.Data.MyContractItem"}}
If it is set to SP.ListItem, I received the error message:
{   "error": {     "code": "-1, Microsoft.SharePoint.Client.InvalidClientQueryException",     "message": {       "lang": "en-US",       "value": "An open collection property 'ContractpartnerId' was found. In OData, open collection properties are not supported."     }   } }
Is it possible, to set the type optionally to a custom type if required?
Currently, it seems to be hard coded.
UploadFile Function
Maybe just change it to:
	 if(!jsonMetadata.has("__metadata")){    JSONObject submeta = new JSONObject();    submeta.put("type", "SP.ListItem");    jsonMetadata.put("__metadata", submeta); }

Thanks for your feedback. I'll review the api to provide a way to overwrite
the default element type.
El mié, 3 feb 2021 a las 14:42, Ulrich Wohlfeil<EMAIL_ADDRESS>escribió:

I have a document list with a metadatafield, which is configured as Multi
choise field. In this case, it is important to set the correct type.
I have to set the type like this:
{"ContractpartnerId":{"results":[2]},"__metadata":{"type":"SP.Data.MyContractItem"}}
If it is set to SP.ListItem, I received the error message:
{ "error": { "code": "-1,
Microsoft.SharePoint.Client.InvalidClientQueryException", "message": {
"lang": "en-US", "value": "An open collection property 'ContractpartnerId'
was found. In OData, open collection properties are not supported." } } }
Is it possible, to set the type optionally to a custom type if required?
Currently, it seems to be hard coded.
UploadFile Function
https://github.com/kikovalle/PLGSharepointRestAPI-java/blob/efc9baaa3af094d0b7dc938d3017600f5f174706/src/main/java/com/panxoloto/sharepoint/rest/PLGSharepointOnPremisesClient.java#L429
Maybe just change it to:
if(!jsonMetadata.has("__metadata")){ JSONObject submeta = new
JSONObject(); submeta.put("type", "SP.ListItem");
jsonMetadata.put("__metadata", submeta); }
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kikovalle/PLGSharepointRestAPI-java/issues/17, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEXV7KEY4SV6XROCORXRYCLS5FHGHANCNFSM4XA37BQA
.

Fixed in commit 6db5f734af857b33f6fa3e08f2114a067a257f8c . If metadata payload passes to method contains type then that type will be sent to sharepoint.


Optimize sqlite reporter

Goes after #318.
Metrics before/after this PR:
FMT:
Saving result: .................... 115ms
Saving result: .................... 10ms

LLVM:
Saving result: .................... 1188ms
Saving result: .................... 10ms

Based on hints from this SO thread:
https://stackoverflow.com/questions/1711631/improve-insert-per-second-performance-of-sqlite


fix(engine/importer): use correct field to import concrete implexprs

Close #834.
@W95Psp some of the existing insta tests are failing and I'm not knowledgeable enough about them to see if they need to be updated.

Hi, thanks for your PR! Sorry for the delay, the week is quite busy... will try to look into those tests tomorrow!

(forgot to push, doing that in a minute)

Ok, I'm in a rush and so I rushed the review as well... I wanted to merge too fast, and actually I think there is a problem with the PR. Let me comment on the issue.

@W95Psp
As discussed in the issue, I'm trying to import the trait goal alongside the information stored in ImplExprAtom.
However, this seems to break the uses of ppx_inline in many of the modules in lib/phases. I guess it's caused by incorrect inlining of the code in subtype.ml. And I'm not sure how to fix this.
Any suggestions would be greatly appreciated!

@W95Psp As discussed in the issue, I'm trying to import the trait goal alongside the information stored in ImplExprAtom.
However, this seems to break the uses of ppx_inline in many of the modules in lib/phases. I guess it's caused by incorrect inlining of the code in subtype.ml. And I'm not sure how to fix this.
Any suggestions would be greatly appreciated!

Forget what I said here, I think I just needed to run dune clean before building.

After some back and forth, I decided to only import the trait goal for the concrete case for now.
I'm not sure if it's actually needed for the other cases. If such a need arises in the future, we can reconsider how to import this information again.
All tests should pass now. The only change is that the traits test for f* added a line of implicit dependency.

Hi @paulmure, I'm reviewing this PR this morning, and I think I did something similar recently... https://github.com/hacspec/hax/pull/827
Sorry for the duplicate work... (the last month has been pretty work intensive toward a verification project...)
Wdyt, shall we close this one or is it still needed?

That looks like the information we need. Thank you!


ValueError: Mesh package://example-robot-data/robots/ ...

Hello. Error raises when I run the command bellow. Actually I couldn't launch any models.
ksyundo@ksyundo-MacBookPro:~$ python3 -m crocoddyl.examples.quadrupedal_gaits "display" "plot"

Traceback (most recent call last):
  File "/usr/lib/python3.5/runpy.py", line 184, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.5/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/opt/openrobots/lib/python3.5/site-packages/crocoddyl/examples/quadrupedal_gaits.py", line 17, in <module>
    anymal = example_robot_data.loadANYmal()
  File "/opt/openrobots/lib/python3.5/site-packages/example_robot_data/robots_loader.py", line 66, in loadANYmal
    robot = RobotWrapper.BuildFromURDF(modelPath + URDF_SUBPATH, [modelPath], pinocchio.JointModelFreeFlyer())
  File "/opt/openrobots/lib/python3.5/site-packages/pinocchio/robot_wrapper.py", line 17, in BuildFromURDF
    robot.initFromURDF(filename, package_dirs, root_joint, verbose, meshLoader)
  File "/opt/openrobots/lib/python3.5/site-packages/pinocchio/robot_wrapper.py", line 21, in initFromURDF
    model, collision_model, visual_model = buildModelsFromUrdf(filename, package_dirs, root_joint, verbose, meshLoader)
  File "/opt/openrobots/lib/python3.5/site-packages/pinocchio/shortcuts.py", line 46, in buildModelsFromUrdf
    geom_model = pin.buildGeomFromUrdf(model, filename, geometry_type, package_dirs)
ValueError: Mesh package://example-robot-data/robots/anymal_b_simple_description/meshes/base/anymal_base.dae could not be found.

Model files exists
ksyundo@ksyundo-MacBookPro:~$ ls /opt/openrobots/share/example-robot-data/robots
anymal_b_simple_description  hyq_description   kinova_description  talos_data
double_pendulum_description  icub_description  romeo_description   tiago_description
hector_description           iris_description  solo_description    ur_description

I'm using Ubuntu xenial and python 3.5.2
ksyundo@ksyundo-MacBookPro:~$ lsb_release -c
Codename:	xenial
ksyundo@ksyundo-MacBookPro:~$ python3 --version
Python 3.5.2

How can I fix the file path error?

You need to define the ROS_PACKAGE_PATH. For a more detailed instruction, see the README file of the example-robot-data repository.

it worked. Thank you!

@syundo0730 How did you fix it? I have the same problem

@syundo0730 How did you fix it? I have the same problem

@amirrezarazmjoo I did what cmastalli told https://github.com/loco-3d/crocoddyl/issues/776#issuecomment-629758146

@amirrezarazmjoo I did what cmastalli told https://github.com/loco-3d/crocoddyl/issues/776#issuecomment-629758146

it worked. Thank you!

Hi, can you tell me how did you define the ROS_PACKAGE_PATH? I don't find guidance in example-robot-data... Thank you very much.

Hi,@syundo0730, can you tell me how did you define the ROS_PACKAGE_PATH? I don't find guidance in example-robot-data... Thank you very much.

You need to include the folder where the mesh files are installed, i.e.
export ROS_PACKAGE_PATH=$ROS_PACKAGE_PATH:$INSTALLATION_PATH/share

where $INSTALLATION_PATH is /opt/openrobots/ if you've installed the binaries with robotpkg.

@cmastalli Hi, thank you for your reply. I add this into ~/.bashrc and resource it. And then I ran gepetto-gui, and tried to load robot like this official video, but there is no robot I can choose. What did I miss?

@cmastalli Hi, thank you for your reply. I add this into ~/.bashrc and resource it. And then I ran gepetto-gui, and tried to load robot like this official video, but there is no robot I can choose. What did I miss?

I am not aware of how to do it. Perhaps @nim65s (or @jmirabel) knows the answer.

The button is still here but the functionality should not be used. It was
meant to be used by plugins. It hasn't been updated when switching to
plugins.
Loading a robot shall be done using the python interface. If you want to
add buttons to the GUI, check the python python interface.


Issue/3951 replace instance UUID modals (#4060)

Pull request opened by the merge tool on behalf of #4060

Merged in f1301649780ef13ea6683bc664af6655b368c6c3


Fix robots.txt by removing the Disallow entries.

These are now in metadata.xlsx or as direct metadata on affected pages in the documents.
Disallow in robots.txt is not a way to keep a page out of a search engine. For more info see:
https://developers.google.com/search/docs/crawling-indexing/robots/intro
Fixes: #138
Test URLs:

Original: https://www.sunstar-foundation.org/
Before: https://main--sunstar-foundation--hlxsites.hlx.live/
After: https://fix-robots--sunstar-foundation--hlxsites.hlx.live/

@bosschaert , could you please create similar PRs for Sunstar.com and sunstar-engineering.com?


[Mini RFC] Rename ProgressBar components to *Gauge

https://github.com/spotify/backstage/blob/1812fd05e6cc27cb1c845020b422d6e82b330183/packages/core/src/components/ProgressBars/CircleProgress.tsx#L80 to Gauge
https://github.com/spotify/backstage/blob/1812fd05e6cc27cb1c845020b422d6e82b330183/packages/core/src/components/ProgressBars/HorizontalProgress.tsx#L31 to LinearGauge
https://github.com/spotify/backstage/blob/1812fd05e6cc27cb1c845020b422d6e82b330183/packages/core/src/components/ProgressBars/ProgressCard.tsx#L39 to GaugeCard
Always get them confused with the Progress component and don't remember what they're for until I actually look at them in storybook :p

Sure!

I'm +1 for renaming this component but -1 on *Gauge. It's not a word that is familiar to me.
Suggestion: *Loading or *LoadingIndicator

But it's not loading things in general right? It's more like "you are 20% done with this task" or "you are 70% cost efficient" - things that you gauge. The word itself I think is well fitting (and commonly used in this kind of scenario).

But it's never used as a loading indicator 😅 ?
Fwiw MUI want to add a Gauge component too: https://material-ui.com/discover-more/roadmap/#new-components

Ooooo I misread this. I (for some unclear reason) thought you wanted to rename the Progress component.
I'm 👍 on this 😃

@stefanalund That confusion is exactly why I want to rename 😁


Do not require individual lodash packages

https://github.com/apollostack/apollo-client/blob/b2d9a7c8c969e3b03f47e349e8b0702547e20e48/package.json#L42-L60
This is an anti-pattern. When you include a package this way, it cannot be de-duped across different projects. It results in this mess:

These packages are deprecated and will no longer be supported as of lodash v5.
https://github.com/lodash/lodash/wiki/Roadmap
Instead, you should be depending on the main library, but importing individual packages, e.g.
const find = require('lodash/find');

There is a Babel plugin that does this for you,
https://github.com/lodash/babel-plugin-lodash
However, seeing that you are using ts, unless there is a ts equivalent, you'd need to code all the references by hand.
If you are going to go this way, please add a suggestion for the consumers to also use https://github.com/lodash/lodash-webpack-plugin to further decrease bundle size.

@gajus We do want to move away from using the individual lodash pacakges, but ideally without introducing another build step.

We have dropped the entire Apollo stack from product more or less because of the above screenshot. A simple query mechanism with in-built state management should not weight half of the application. I'd say therefore it is an important issue (and the entire bundle size in general).
However, this was only one of the reasons. The other being poor integration with immutable.js and very hacky integration with isomorphic rendering tools.

I think that
import merge from 'lodash/merge';
instead of
import merge from 'lodash.merge';
is a good start

@seeden Yeah, I think that's a good star. I would definitely merge it if you made a PR. I can also do it myself, but I don't think I would get to it this week.

@helfer I have created PR #995

@helfer I created PR #997


E/19

This MR makes changes to the ImportData -> ManageData. It creates a side bar that displays the Projects that are added via the "upload/import" form on the ManageData page. As well as, the necessary and relative React Redux slices for Sprints and Projects.
How To Test:
npm start
Closes #18, #19

Mr does not have issue title.


Add API to allow third party ui nodes to pass data between server and client code

Description
Please provide methods for third party ui nodes that:

Allow code in the node's server .js file to send a message (subject to msg._client) to the node's attached clients.
Allow code in the node's client implementation to send a message to the server code so that, for example, the state store can be updated without sending a message to attached nodes, and without directly updating the datastore.

See this forum post for background information.
Have you provided an initial effort estimate for this issue?
I am no FlowFuse team member

I believe sending messages (only) to server node is already supported via custom events (just need to fix reconnection bug)
Let me add some further requirements to the server node API, to make it complete:

as requested by @colinl, send a message to a specified client (by _client) or all clients which include this node
Be able to drop an incoming msg, before it is broadcasted to the clients (can this be done today by setting the msg to null in onInput?)
Expose a live list of all connected clients (which contain this node), with their respective Ids. I assume such list is maintained in the framework, so just need to expose it to the server node (e.g., like the Datastore API).
Send notifications upon client add/remove. This is available through ui-event/ui-control nodes, but our custom ui nodes need to be self-contained

3. Expose a live list of all connected clients (which contain this node),

Clients which contain the node, or just those showing a page with the node active?
Out of interest, what do you want to use that list for?

Clients which contain the node, or just those showing a page with the node active?

With the node active.

Out of interest, what do you want to use that list for?

In general, that is good info for the server node to have.
Specifically for my ui-tabulator node, the tabulator package is instantiated and processing in the client. When in Shared (as opposed to Multi-user) mode, when I send a msg to the table node, I receive multiple identical replies (one per client), and need to collapse them by msg Id, which is very messy (or not possible in case of events).
If I had a client list, I could select any client as "spokesman" and ignore responses/events from others.

Allow code in the node's server .js file to send a message (subject to msg._client) to the node's attached clients.

base.emit() will do the trick. You just need to get reference to the UI Base in your code. If your node bind to a group via a config option, you can do group.getBase(), otherwise a RED.getConfigNode will be required

Allow code in the node's client implementation to send a message to the server code so that, for example, the state store can be updated without sending a message to attached nodes, and without directly updating the datastore.

As @omrid01 pointed out - the correct approach here is a custom socket handlers

base.emit('my-event:' + node.id, msg, node) in the server, along with
this.$socket.on('my-event:' + this.id, (msg) => {...} in the client, works to send a message to the client, thanks.
When using a custom handler as you suggest for sending data back to the server, if in the client I use
this.$socket.emit('my-custom-event', this.id, data)
and in the server
onSocket: {
  'my-custom-event': function (conn, id, msg) { ...}
}

then again that does work.  However, if I have multiple nodes then all server instances receive the message from any node in the clients.  Is there a way of including the node id in the event name as is done for sending to the clients?  I can't use
'my-custom-event' + msg.id: function (conn, id, msg) { ...}
as that is not valid javascript.
I see that I can test the passed in id and compare it with node.id if that is the only way.

Yeah, two options as you've detailed:
Option 1: handler per node instance
const events = {}
events['my-custom-event' + node.id] = function (conn, id, msg) { ...}
onSocket = events

Option 2: Check in the single handler
onSocket: {
  'my-custom-event': function (conn, id, msg) {
      if (id === node.id) {
         // do stuff
      }
   }
}

OK. Do I need to disconnect, as is done in the client code, or is that handled automatically in the server?

Do I need to disconnect, as is done in the client code, or is that handled automatically in the server?

All handled server-side. ui-base checks for any open handlers when you delete your last instance of your node and stops the listeners

That all seems to be working, thanks.

Hi @colinl,
I think this issue can be closed, since pull request 1123 has been implemented?

Yes.


not enabled at first when doing nvim <filename> with lazy.nvim

To reproduce:
{
  "m4xshen/hardtime.nvim",
  opts = {}
},

open nvim from the command-line like nvim <filename>. hardtime.nvim will not start until you open another buffer. Probably because of this:
https://github.com/m4xshen/hardtime.nvim/blob/main/lua/hardtime/init.lua#L153

I can't reproduce the issue. Are you using the latest version of hardtime.nvim?

because of this:
https://github.com/m4xshen/hardtime.nvim/blob/main/lua/hardtime/init.lua#L153

But I think the BufNew event is also triggered with nvim <filename>


add better android app launch

A small change: I've "improved" (removed the Actionbar) the look of the app while launching (example below).

So the app launch now looks somewhat more like this:

Some further improvements would be the ability to create an own theme which could be used for a splash screen or like above to color the whole screen red ;).
Also done some refactoring on the manifest (now it looks pretty!)

For anyone who want to make proper fix:
You should not use android:theme="@android:style/Theme.Holo.NoActionBar" inside AndroidManifest.xml.
Use you own custom theme instead - android:theme="@style/AppTheme" (AppTheme is just for example, actually you can use any other name).
Than create res/values/styles.xml (for pre-HONEYCOMB):
<?xml version="1.0" encoding="utf-8"?>
<resources xmlns:android="http://schemas.android.com/apk/res/android">
    <style name="AppTheme" parent="@android:style/Theme.NoTitleBar.Fullscreen" />
</resources>

And  res/values-v11/ (for HONEYCOMB and above):
<?xml version="1.0" encoding="utf-8"?>
<resources xmlns:android="http://schemas.android.com/apk/res/android">
    <style name="AppTheme" parent="@android:style/Theme.Holo.NoActionBar" />
</resources>

There is also other features that can be tweaked by custom app theme (use no background for speedup, for example), but it is out of scope of this PR.


Always use int64 and float64 whenever possible

This will make the code a lot easier to deal with.

This will cause issue with 32-bit systems when they have too much data but who cares.


how long did you run train.sh?

For each run I believe it took around one week, but it really depends on what GPU you're using. Please use this thread for further questions to answer your questions more easily.


K8s restructure 2

My main concern is about the roles. Instead of using tasks with import_role, why don't we just put all of the tasks into the top level playbook. There aren't too many of them and it makes it much easier to follow in a linear fashion (imo).

Because we need to do things on different nodes and on different times. eg. setting up the first master (in the bootstrap() method) is different to then joining worker nodes (in the join()) method. This is not fully implemented yet but with the current structure, it should be relatively easy.

Okay, I see your point. I do want to keep the ansible parts as easy to follow as possible, so looking for ways to keep it to a minimum. I'm concerned about the number of parts a test writer would have to follow, so we need to keep in mind what the python api looks like when actually writing the tests. That said, this looks good to me, so I'm happy to merge once my build finishes :-).

Seems to fail with libvirt (hangs during ceph fs status myfs in the toolbox container) but passed within ECP according to Josh. merging...