wait-for-postgres.sh

I made some modifications:
— Added a timeout variable and a start_time variable to keep track of the elapsed time. We also added an if statement that checks if the elapsed time has exceeded the timeout value. If it has, the script exits with an error message.
— Replaced the hardcoded database username and password with environment variables.
— Added the set -o pipefail command to ensure that the script fails if any command in a pipeline fails. We also added some error messages to make it easier to diagnose problems.

Thanks for the updates to the Postgres script! I’ll test these shortly.
Can you remove the codeql files? Those can end up producing many errors that will go unfixed as I don’t have a linter in this project

Thanks for the updates to the Postgres script! I’ll test these shortly.
Can you remove the codeql files? Those can end up producing many errors that will go unfixed as I don’t have a linter in this project

Hi @cbaker6. It's done!

I made some changes so this doesn't break the docker compose files


Sharing files between modules

I am hoping that this is just a question, and I have overlooked the answer, otherwise I guess it is an enhancement suggestion =)
Wonder if it is possible to somehow share a single file so that it can be used in multiple modules?
Assuming one is using the pattern of breaking out all modules into a separate repo, they get cloned somewhere under /tmp, so relative paths seems to be out of the question.
The docs about interpolation says:

Terragrunt built-in functions only work within a terragrunt = { ... } block. Terraform does NOT process interpolations in .tfvars files

So it seems there is no way to pass a runtime-calculated absolute path as a variable value to a module? Since variables must be placed outside the terragrunt block?
Is there some other way to achieve this?
My specific use case is that I have written a generic user_data.sh script that I am using like this:
data "template_file" "user_data" {
  template = "${file("user_data.sh")}"
  vars = {
    playbook = "rabbitmq"
  }
}

But as of now, the only way I can see is to copy this file into each module where I want to use it.

Why do you need to copy the file? Couldn't each module reference the file?
data "template_file" "user_data" {
  template = "${file("${path.module}/../shared-files/user-data.sh")}"
}

See also the note about using double-slashes in the remote config docs.

Thanks for the reply. I guess your suggestion is based on having the shared-files dir in the modules-repo, not in the live-repo?
I think I was trying to have in the live-repo. But I guess this works just as well, thanks!
Not sure how the double-slashes relates to this? I thought they only applied to the source in the terraform block.

Yes, put the shared file in the modules repo.
If you're using that modules repo with the remote configurations (with a source URL in the Terragrunt config), the double-slash allows you to use relative paths. Without it, the relative paths won't work correctly.


Keystore unit tests are disabled and fail when they are enabled

Keystore unit tests currently have the annotation @Disabled, these should be enabled.
When the unit tests are enabled, they result in failure:
com.hedera.hashgraph.sdk.KeystoreTest > Keystore.toStream produces decodable value FAILED
    java.lang.Error: platform does not support AES-CTR ciphers
        at com.hedera.hashgraph.sdk.Crypto.initAesCtr128(Crypto.java:47)
        at com.hedera.hashgraph.sdk.Crypto.encryptAesCtr128(Crypto.java:101)
        at com.hedera.hashgraph.sdk.Keystore.exportJson(Keystore.java:99)
        at com.hedera.hashgraph.sdk.Keystore.export(Keystore.java:81)
        at com.hedera.hashgraph.sdk.KeystoreTest.keystoreToStream(KeystoreTest.java:36)

        Caused by:
        java.security.NoSuchAlgorithmException: Cannot find any provider supporting AES/CTR/PKCS5Padding
...

This exception seems to be thrown by the following code block in Crypto.java on line 44:
        try {
            aesCipher = Cipher.getInstance("AES/CTR/PKCS5Padding");
        } catch (NoSuchAlgorithmException | NoSuchPaddingException e) {
            throw new Error("platform does not support AES-CTR ciphers", e);
        }

This has been fixed, apparently you can't use PKCS5Padding with CTR.


[1.11] Move networkd setup to mesos unit

@mellenburg  is this required for 1.11.0 GA?

Also, please fill the PR template to provide more context on this PR.

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@orsenthil There is already a fix for this in the prereq installer so this is fine to leave OUT of 1.11.0 GA I think


extension-module should link libpython for Android and Cygwin

When browsing documentation earlier this morning I came across a note suggesting that libpython should be linked on both android and cygwin targets (see last paragraph here).
At the moment this is not the case.
We should change build.rs so that even with extension-module set (or whatever mechanism might replace extension-module) we always link to libpython on these targets.

Not really sure what target cygwin would correspond to in the target platforms, x86_64-pc-windows-gnu?

Not really sure what target cygwin would correspond to in the target platforms, x86_64-pc-windows-gnu?

Good question. I can't see evidence that Rust provides a target for cygwin. I think if it existed it might be x64_64-pc-cygwin, which is based on briefly trawling the internet and finding this stackoverflow post.
IMO cygwin is increasingly redundant with WSL anyway, so I think let's ignore support for it for now until somebody comes along asking.

fair enough, makes dealing with one less exception :smile:

#1095 solved this for android, and at the moment there is no Rust cygwin target (afaik), so closing this issue for now.


IMDB parser needs updating

Hi,
Since today I'm receiving this error while executing my tasks with imdb plugin:
ERROR    utils.imdb    task     Possible IMDB parser needs updating, Please report on Github.
In the plugin i have the following conditions:
imdb:
  min_score: 6.5
  min_votes: 5000
  min_year: 2010
  reject_genres:
    - horror
    - documentary
    - musical
    - music

Can you help me?
Thanks in advance,
Luís Tomás

I've already reinstalled pip (because of ImportError: cannot import name IncompleteRead) and updated flexget to the last version (1.2.438)

I am getting the same error.
Example 1:
imdb_lookup    MOVIES    Parsing imdb for Holland Road 2015 720p WEBRIP x264 AC3-NoHaTE
utils.imdb    MOVIES          Possible IMDB parser needs updating, Please report on Github.
imdb_lookup   MOVIES          Unable to set imdb_name for tt2369740 from http://www.imdb.com/title/tt2369740/
Example:2
imdb_lookup MOVIES Parsing imdb for Freaks Of Nature 2015 LIMITED 720p BluRay x264-SNOW
utils.imdb    MOVIES    Possible IMDB parser needs updating, Please report on Github.
imdb_lookup MOVIES Unable to set imdb_name for tt1817771 from http://www.imdb.com/title/tt1817771/
pip 8.0.2 from /usr/local/lib/python2.7/dist-packages (python 2.7)
Flexget Version: 1.2.438

Same here.  Crash log :
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Imdb lookup failed for tom and jerry mouse trouble 2014 dvdrip xvid ac3 acab
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Imdb lookup failed for mickey mouse clubhouse: minnie rella 2014 dvdrip x264 ac3 titan
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Imdb lookup failed for buddy hutchins - falling down again 2015 bdrip x264-noscreens
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Imdb lookup failed for buddy.hutchins-falling.down.again.2015.720p.brrip.x264.aac-etrg
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Imdb lookup failed for buddy hutchins - falling down again 2015 bdrip x264-noscreens
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Imdb lookup failed for buddy.hutchins-falling.down.again.2015.1080p.brrip.x264.aac-etrg
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Imdb lookup failed for natures.microworlds.series.2.1of6.canadas.coastal.forests.576p.hdtv.x264.aac .mp4
2016-01-27 11:00 VERBOSE  tmdb_lookup   movies          TMDB lookup failed for Natures.Microworlds.Series.2.1of6.Canadas.Coastal.Forests.576p.HDTV.x264.AAC .mp4
2016-01-27 11:00 VERBOSE  movie_queue   movies          IMDB and TMDB lookups failed for Natures.Microworlds.Series.2.1of6.Canadas.Coastal.Forests.576p.HDTV.x264.AAC .mp4.
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Imdb lookup failed for wild.australia.4of4.koala.forest.720p.hdtv.x264.aac .mp4
2016-01-27 11:00 VERBOSE  tmdb_lookup   movies          TMDB lookup failed for Wild.Australia.4of4.Koala.Forest.720p.HDTV.x264.AAC .mp4
2016-01-27 11:00 VERBOSE  movie_queue   movies          IMDB and TMDB lookups failed for Wild.Australia.4of4.Koala.Forest.720p.HDTV.x264.AAC .mp4.
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Imdb lookup failed for the shuttered room - [thriller] - (1967) - dvdrip -  [=faith=]
2016-01-27 11:00 VERBOSE  tmdb_lookup   movies          TMDB lookup failed for The Shuttered Room - [Thriller] - (1967) - DVDRip -  [=Faith=]
2016-01-27 11:00 VERBOSE  movie_queue   movies          IMDB and TMDB lookups failed for The Shuttered Room - [Thriller] - (1967) - DVDRip -  [=Faith=].
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Imdb lookup failed for shut.your.facebook.720p.hdtv.x264-barge
2016-01-27 11:00 VERBOSE  tmdb_lookup   movies          TMDB lookup failed for Shut.Your.Facebook.720p.HDTV.x264-BARGE
2016-01-27 11:00 VERBOSE  movie_queue   movies          IMDB and TMDB lookups failed for Shut.Your.Facebook.720p.HDTV.x264-BARGE.
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Parsing imdb for The Forsaken (2001) DvdRip Xvid {1337x}-Noir
2016-01-27 11:00 ERROR    lazy_lookup   movies          Unhandled error in lazy lookup plugin
2016-01-27 11:00 CRITICAL manager       movies          An unexpected crash has occurred. Writing crash report to /home/pi/.flexget/crash_report.2<IP_ADDRESS>044487345.log
2016-01-27 11:00 INFO     imdb_lookup   movies          Imdb lookup failed for prophecy forsaken2005 dvdrip xvid ac35 1eng-hlawoun
2016-01-27 11:00 VERBOSE  tmdb_lookup   movies          TMDB lookup failed for Prophecy Forsaken2005 Dvdrip Xvid AC35 1Eng-hlawoun
2016-01-27 11:00 VERBOSE  movie_queue   movies          IMDB and TMDB lookups failed for Prophecy Forsaken2005 Dvdrip Xvid AC35 1Eng-hlawoun.
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Searching from imdb By The Sea 2015 HC WEBRip XviD-FGT
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Found http://www.imdb.com/title/tt3707106/?ref_=fn_ft_tt_1
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Parsing imdb for By The Sea 2015 HC WEBRip XviD-FGT
2016-01-27 11:00 ERROR    lazy_lookup   movies          Unhandled error in lazy lookup plugin
2016-01-27 11:00 CRITICAL manager       movies          An unexpected crash has occurred. Writing crash report to /home/pi/.flexget/crash_report.2<IP_ADDRESS>051744482.log
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Searching from imdb By.the.Sea.2015.HC.HDRip.XviD.AC3-EVO
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Found http://www.imdb.com/title/tt3707106/?ref_=fn_ft_tt_1
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Parsing imdb for By.the.Sea.2015.HC.HDRip.XviD.AC3-EVO
2016-01-27 11:00 ERROR    lazy_lookup   movies          Unhandled error in lazy lookup plugin
2016-01-27 11:00 CRITICAL manager       movies          An unexpected crash has occurred. Writing crash report to /home/pi/.flexget/crash_report.2<IP_ADDRESS>059123788.log
2016-01-27 11:00 VERBOSE  imdb_lookup   movies          Searching from imdb By the Sea (2015) 720p HC HDRip 900MB - MkvCage
2016-01-27 11:01 VERBOSE  imdb_lookup   movies          Found http://www.imdb.com/title/tt3707106/?ref_=fn_ft_tt_1
2016-01-27 11:01 VERBOSE  imdb_lookup   movies          Parsing imdb for By the Sea (2015) 720p HC HDRip 900MB - MkvCage
2016-01-27 11:01 ERROR    lazy_lookup   movies          Unhandled error in lazy lookup plugin
2016-01-27 11:01 CRITICAL manager       movies          An unexpected crash has occurred. Writing crash report to /home/pi/.flexget/crash_report.2<IP_ADDRESS>106125266.log
2016-01-27 11:01 VERBOSE  imdb_lookup   movies          Parsing imdb for By the Sea 2015 HDRip XviD AC3-iFT
2016-01-27 11:01 ERROR    lazy_lookup   movies          Unhandled error in lazy lookup plugin
2016-01-27 11:01 CRITICAL manager       movies          An unexpected crash has occurred. Writing crash report to /home/pi/.flexget/crash_report.2<IP_ADDRESS>109878944.log
2016-01-27 11:01 VERBOSE  imdb_lookup   movies          Parsing imdb for By the Sea 2015 HDRip XviD AC3-EVO
2016-01-27 11:01 ERROR    lazy_lookup   movies          Unhandled error in lazy lookup plugin
2016-01-27 11:01 CRITICAL manager       movies          An unexpected crash has occurred. Writing crash report to /home/pi/.flexget/crash_report.2<IP_ADDRESS>114219925.log
2016-01-27 11:01 VERBOSE  imdb_lookup   movies          Parsing imdb for By the Sea 2015 HDRip XviD AC3-EVO
2016-01-27 11:01 ERROR    lazy_lookup   movies          Unhandled error in lazy lookup plugin
2016-01-27 11:01 CRITICAL manager       movies          An unexpected crash has occurred. Writing crash report to /home/pi/.flexget/crash_report.2<IP_ADDRESS>118034975.log
2016-01-27 11:01 VERBOSE  imdb_lookup   movies          Parsing imdb for By the Sea (2015) HC - 720p WEBRiP - 1GB - ShAaNiG
2016-01-27 11:01 ERROR    lazy_lookup   movies          Unhandled error in lazy lookup plugin
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/flexget/utils/lazy_dict.py", line 36, in getitem
func(self.store)
File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/metainfo/imdb_lookup.py", line 235, in lazy_loader
self.lookup(entry)
File "/usr/local/lib/python2.7/dist-packages/flexget/plugin.py", line 124, in wrapped_func
return func(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/flexget/utils/database.py", line 30, in wrapper
return func(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/metainfo/imdb_lookup.py", line 374, in lookup
movie = self._parse_new_movie(entry['imdb_url'], session)
File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/metainfo/imdb_lookup.py", line 405, in _parse_new_movie
parser.parse(imdb_url)
File "/usr/local/lib/python2.7/dist-packages/flexget/utils/imdb.py", line 245, in parse
log.error('Possible IMDB parser needs updating, Please report on Github.')
AttributeError: 'NoneType' object has no attribute 'find'

I guess this will be fixed very soon. ;-)

Got same issue.. Guess its a general thing
Possible IMDB parser needs updating, Please report on Github.

Same here, waiting for update.  :)


File loading

Provide option to open custom file

Added in https://github.com/0legg/bodylookin/commit/9ceec01d77314399d1082f84363bbdb35c6f21f7


[skip-ci] Packit: constrain koji and bodhi to the fedora package

This helps to avoid dup downstream jobs. Doesn't affect upstream.

ah this one's a 400 error

/packit build


ENH: Use sphinx-gallery, add API page

This PR:

Switches to sphinx-gallery
Adds API page for class/function doc (I've always wanted this)
Cleans up some docstrings
Adds intersphinx linking
Fixes some OSX DYLD_LIBRARY_PATH silliness (proper fix for #171)

These were necessary to get things looking good on OSX.

Coverage decreased (-0.1%) to 78.283% when pulling aab6dbd6dc8ac878987c82e4587958b5acefc549 on Eric89GXL:api into 9fa66d3f4313b4234a63adbd9ddaf51da3d76d50 on nipy:master.

@Eric89GXL can you share a rendered site to see?

Coverage decreased (-0.1%) to 78.894% when pulling b9238ee9dd1e5419f008c0109fa4520057d0a860 on Eric89GXL:api into 432c762e23bd373877fbbfb1305c7c58ca22ed88 on nipy:master.

@agramfort finally here:
https://ci.appveyor.com/api/buildjobs/o8v5r82fmmgbjhw0/artifacts/doc%2F_build%2Fhtml.zip

looks great

Thanks for taking the initiative here, Eric. The new "modern" css looks mostly good. Here are some thoughts:

The box on the index page with the logo/name/tagline (I think web devs call this a "hero" element?) feels too large for its relatively uninteresting visual content.
The tagline "Python library for visualizing cortical surface representations of neuroimaging data" feels a little dull, I guess because it starts with "Python library" which is obvious from the name, and is a little wordy. Elsewhere we have used "Cortical neuroimaging visualization in Python" which is a little snappier and emphasizes the specific application of the library a little better. But it can be improved.
The bootswatch theme in general is clean and nice, but the lack of any color on pages other than the homepage feels very bland.
By moving the first sentence of the intro description into the tagline, we start with "PySurfer is primarily intended for use with Freesurfer, " which does not grab my attention! This paragraph needs some improvement.
I don't like how we've lost the links to other parts of the site from the main text on the homepage.
I just noticed that our install page still says "does not work on Python 3", which is no longer True. Also do we have a hard dependency on IPython still?
"Documentation" in the header should probably be "Tutorial" or something more specific, since everything is really "documentation". Although the stuff in there isn't a very good tutorial.... Also "Examples" should maybe be "Gallery".
Images in the example gallery are not working (at least in the static Appveyor download).

The box on the index page with the logo/name/tagline (I think web devs call this a "hero" element?) feels too large for its relatively uninteresting visual content.

The bootswatch theme in general is clean and nice, but the lack of any color on pages other than the homepage feels very bland.

By moving the first sentence of the intro description into the tagline, we start with "PySurfer is primarily intended for use with Freesurfer, " which does not grab my attention! This paragraph needs some improvement.

Jumbotron :)
With these aesthetic choices we risk getting into a back-and-forth bikeshed-type argument because there is no clear right answer. So I'll take a stab at addressing these comments, but I'd like to stop at whatever is the minimal acceptable documentation improvement, and iterate over subsequent PRs (probably not by me) to tweak such aesthetic / layout / wordings further. Okay?

I don't like how we've lost the links to other parts of the site from the main text on the homepage.

I disagree on this point. They are redundant with the links at the top. If we put them on the main page, they should really have identical text / naming as the top links. So unless the links at the top need some additional explanation that you could put to the right of such main-body links (and I don't think they do), I don't see a reason to also have them on the main page.

I just noticed that our install page still says "does not work on Python 3", which is no longer True. Also do we have a hard dependency on IPython still?

Technically Python 3 is possible if you use non-standard Anaconda repos. I won't add that for now (can be a separate PR because the wording might be tricky), but I'll remove the bit about IPython.

"Documentation" in the header should probably be "Tutorial" or something more specific, since everything is really "documentation". Although the stuff in there isn't a very good tutorial.... Also "Examples" should maybe be "Gallery".

I'll change these.

Images in the example gallery are not working (at least in the static Appveyor download).

Yes AppVeyor only does the "noplot" build.

So I'll take a stab at addressing these comments, but I'd like to stop at whatever is the minimal acceptable documentation improvement, and iterate over subsequent PRs (probably not by me) to tweak such aesthetic / layout / wordings further. Okay?

Well ... sure ... but I don't see the point of merging a PR that is aesthetically/textually a regression from the current docs. Unless I'm missing something, adding automatically generated API docs and moving the example gallery to sphinx-gallery is orthogonal to using sphinx-bootswatch and altering the homepage text. So I think it's fine for a PR about the former not to get bogged down in a discussion of the latter, but maybe they should be separate PRs/discussions.

I disagree on this point. They are redundant with the links at the top. If we put them on the main page, they should really have identical text / naming as the top links. So unless the links at the top need some additional explanation that you could put to the right of such main-body links (and I don't think they do), I don't see a reason to also have them on the main page.

They are not redundant because they serve different purposes. The links at the top are navigational elements for people who know what they are looking for. The main text on the homepage should tell people what there is to look for. I agree that the latter goal is better served by a bit of narration about the different elements of the website than just a list of links. You can look at the seaborn homepage to get an idea of what my preference would be.

I don't see the point of merging a PR that is aesthetically/textually a regression from the current docs.

I agree, but I didn't realize you considered these (and I don't consider these) aesthetic changes as regressions. If you think of them the changes way, they should be separated so I'll revert.

If we end up sticking with the sphinx_bootstrap/bootswatch themes, one thing I hate about the CSS is the giant text in the API parameter descriptions; in the seaborn docs I make it smaller.

I agree, but I didn't realize you considered these (and I don't consider these) aesthetic changes as regressions. If you think of them the changes way, they should be separated so I'll revert.

Perhaps its just my familiarity (and authorship) bias, but the new site with the bland color palette and reduce text feels very unfinished and unwelcoming. I think it can be made better than the original docs with some judicious changes, even simple things like including a small version of the log in the header. I recognize that bikeshedding about aesthetics can be annoying, but you are kind of inviting it when you propose a whole-sale restyling of the docs...

I recognize that bikeshedding about aesthetics can be annoying, but you are kind of inviting it when you propose a whole-sale restyling of the docs...

Yeah this is true. I wrongly / naively assumed that folks would agree with me main thrust of the changes. I have now reverted them in any case so we can deal with them separately, so this should now hopefully just be functional stuff switching to sphinx-gallery etc. now.

New build:
https://ci.appveyor.com/api/buildjobs/g7a95ykjrgaqunaj/artifacts/doc%2F_build%2Fhtml.zip
Travis is just waiting for OSX to build, but should come back green.

CIs are happy, ready for review/merge from my end.
FWIW talking to @agramfort he didn't like the modern look either!

I tried building the docs (make html_stable) but had some problems. A few examples complained about not finding the fsaverage BA labels -- I think this is because Freesurfer 6.0 added "exvivo" to their names. The second issue is crashing out of quite a few examples because it's looking for the SUBJECTS_DIR in /Applications/freesurfer/subjects. But that's not where my SUBJECTS_DIR is, which the $SUBJECTS_DIR variable in my terminal confirms. Is there some reason that the changes make pysurfer think it's running on a mac? git grep Applications is empty...

Here are the exceptions:

Exception occurred:
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/sphinx_gallery/gen_gallery.py", line 280, in sumarize_failing_examples
    "\n" + "-" * 79)
ValueError: Here is a summary of the problems encountered when running the examples

Unexpected failing examples:
/home/mwaskom/code/pysurfer/examples/plot_resting_correlations.py failed leaving traceback:
Traceback (most recent call last):
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/sphinx_gallery/gen_rst.py", line 472, in execute_code_block
    exec(code_block, example_globals)
  File "<string>", line 8, in <module>
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/surfer/viz.py", line 416, in __init__
    subjects_dir = _get_subjects_dir(subjects_dir=subjects_dir)
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/surfer/utils.py", line 659, in _get_subjects_dir
    % subjects_dir)
ValueError: The subjects directory /Applications/freesurfer/subjects does not exist.

/home/mwaskom/code/pysurfer/examples/plot_transparent_brain.py failed leaving traceback:
Traceback (most recent call last):
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/sphinx_gallery/gen_rst.py", line 472, in execute_code_block
    exec(code_block, example_globals)
  File "<string>", line 16, in <module>
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/surfer/viz.py", line 416, in __init__
    subjects_dir = _get_subjects_dir(subjects_dir=subjects_dir)
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/surfer/utils.py", line 659, in _get_subjects_dir
    % subjects_dir)
ValueError: The subjects directory /Applications/freesurfer/subjects does not exist.

/home/mwaskom/code/pysurfer/examples/plot_label.py failed leaving traceback:
Traceback (most recent call last):
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/sphinx_gallery/gen_rst.py", line 472, in execute_code_block
    exec(code_block, example_globals)
  File "<string>", line 13, in <module>
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/surfer/viz.py", line 1289, in add_label
    % filepath)
ValueError: Label file /usr/local/freesurfer/current/subjects/fsaverage/label/lh.BA1.label does not exist

/home/mwaskom/code/pysurfer/examples/plot_topographic_contours.py failed leaving traceback:
Traceback (most recent call last):
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/sphinx_gallery/gen_rst.py", line 472, in execute_code_block
    exec(code_block, example_globals)
  File "<string>", line 13, in <module>
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/surfer/viz.py", line 416, in __init__
    subjects_dir = _get_subjects_dir(subjects_dir=subjects_dir)
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/surfer/utils.py", line 659, in _get_subjects_dir
    % subjects_dir)
ValueError: The subjects directory /Applications/freesurfer/subjects does not exist.

/home/mwaskom/code/pysurfer/examples/plot_probabilistic_label.py failed leaving traceback:
Traceback (most recent call last):
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/sphinx_gallery/gen_rst.py", line 472, in execute_code_block
    exec(code_block, example_globals)
  File "<string>", line 10, in <module>
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/surfer/viz.py", line 416, in __init__
    subjects_dir = _get_subjects_dir(subjects_dir=subjects_dir)
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/surfer/utils.py", line 659, in _get_subjects_dir
    % subjects_dir)
ValueError: The subjects directory /Applications/freesurfer/subjects does not exist.

-------------------------------------------------------------------------------
The full traceback has been saved in /tmp/sphinx-err-oK1cAm.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at <https://github.com/sphinx-doc/sphinx/issues>. Thanks!
Makefile:42: recipe for target 'html_stable' failed
make: *** [html_stable] Error 1

can you revert the doc/_static/banner.png changes?

I can revert the banner but it seemed useful to leave it transparent ?

initial banner looks nicer. I seemed to see visual artifacts on circles.

Ahh I must have done a bad job of making it transparent. I'll revert it.

Is there some reason that the changes make pysurfer think it's running on a mac? git grep Applications is empty...

Shouldn't be, but I can check. Are you on Linux or Windows?

Comments addressed. @mwaskom build problems should be fixed, feel free to try again.

I still get one error on Freesurfer 6.0
Exception occurred:
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/sphinx_gallery/gen_gallery.py", line 280, in sumarize_failing_examples
    "\n" + "-" * 79)
ValueError: Here is a summary of the problems encountered when running the examples

Unexpected failing examples:
/home/mwaskom/code/pysurfer/examples/plot_label.py failed leaving traceback:
Traceback (most recent call last):
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/sphinx_gallery/gen_rst.py", line 472, in execute_code_block
    exec(code_block, example_globals)
  File "<string>", line 40, in <module>
  File "/home/mwaskom/anaconda2/lib/python2.7/site-packages/surfer/viz.py", line 1289, in add_label
    % filepath)
ValueError: Label file /usr/local/freesurfer/current/subjects/fsaverage/label/lh.entorhinal.label does not exist

In the header there is a link to the "modules" page, which isn't that useful for PySurfer. Could that be changed to link directly to the python_reference.html page?

I bet this is not related to the doc changes, but I just noticed in the gallery that the negative side of Overlays has a backwards colorbar. @christianbrodbeck is this related to your recent PR do you think?

I did modify some overlay code, I'll check, which example?

It's apparent in the gallery in the plot_fmri_volume example which shows the negative side of the overlay colormap, and you can see it partway through running the plot_resting_correlations example.

Yes I can confirm, I'll work on a fix

In the header there is a link to the "modules" page, which isn't that useful for PySurfer. Could that be changed to link directly to the python_reference.html page?

Maybe, but I don't have time to look,let's do it in another PR. For now I've reverted the change that added those links.
I also fixed the label / build problem.

Maybe, but I don't have time to look,let's do it in another PR. For now I've reverted the change that added those links.

Sounds good — our API is a little weird in that pretty much everything you'd want to know is handled by methods on Brain (sigh) so it's worth thinking about how to best organize the API docs so people can get that information with minimal clicking. Totally fine with punting on those details, though.

CIs are happy, let me know if there's anything else needed for this one.

Looks good for now, thanks!


fix: simplify .gitattributes and remove .runsettings* and .gdlintrc

Why
I can't tell for sure but it seems like .runsettings* and .gdlintrc paths were included in the release by mistake since the .gitattributes is an excluded list.
The current .gitattributes structure looks difficult to maintain. If .runsettings* and/or .gdlintrc should be kept in the final release, this PR can easily be updated to include them explicitly.
What
I flipped the .gitattributes around to be an includes list by excluding everything first. This approach should keep it more maintainable as you may add other files to the project later that are not needed in the final release.
This is testable by running this command against this branch:
git archive --format=tar improve-gitattribute | gzip > <replace-this-with-a-path>

That's great, thank you yes, that makes it much easier 👍


how to generate the ‘sam3d_Plansv2.1_plans_3D.pkl’ file

Thank you for your excellent work, and this work inspire me a lot!  I have a quetions in your open-sourced code that how to generate the ‘sam3d_Plansv2.1_plans_3D.pkl’ file？

I will apreciate it if you could tell me the process.

Sorry for the late reply. Our repo is currently under revision. Frankly, we
re-used the source code from UNETR++ so it is a little bit dirty, not easy
to be used. We will upload the .pkl file soon.
Thank you for your concern.
On Mon, 18 Sep 2023 at 11:11 se16wangmeiwei @.***>
wrote:

Thank you for your work! I have a quetions in your open-sourced code that
how to generate the ‘sam3d_Plansv2.1_plans_3D.pkl’ file？
I will apreciate it if you can tell me.
—
Reply to this email directly, view it on GitHub
https://github.com/DinhHieuHoang/SAM3D/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJZKHC4WJ3RK3YQL3KWQZDLX27CV5ANCNFSM6AAAAAA44DQNNM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Sorry for the late reply. Our repo is currently under revision. Frankly, we re-used the source code from UNETR++ so it is a little bit dirty, not easy to be used. We will upload the .pkl file soon. Thank you for your concern.
…
On Mon, 18 Sep 2023 at 11:11 se16wangmeiwei @.> wrote: Thank you for your work! I have a quetions in your open-sourced code that how to generate the ‘sam3d_Plansv2.1_plans_3D.pkl’ file？ I will apreciate it if you can tell me. — Reply to this email directly, view it on GitHub <#1>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJZKHC4WJ3RK3YQL3KWQZDLX27CV5ANCNFSM6AAAAAA44DQNNM . You are receiving this because you are subscribed to this thread.Message ID: @.>

Thank you for your reply, I will be waiting for you. ^^


Question about document chunks

On the left hand side, if a document has been added to the database, I see for example:

Collection: UserData
Docs: =1
Chunks: 353

while in expert settings (default) it's set up:

Number of document chunks (query) or pages/parts (summarize) (-1 = auto fill model context, all pages/docs for summarize)
3

Does this mean that only 3 chunks are used? Or are these different kind of chunks?

Yes, top_k_docs=3 by default.  For highest quality I recommend top_k_docs=-1, but it will be slower.  A good balance might be 10.  We will manage things in any case to reduce the count to not exceed the context of the LLM.

b5996103ce8f1fda8d2612587da68f82302d3923


[Feature Request] Can we have a timeline of infected people on the website?

so that the users would be able to read the latest numbers as well as the past figures on https://coronavirus.zone/

Hey, I made it using Google Datastudio. So anyone can use and share, I will share my Datastudio Connector and you will be able create yours own dashboards.
https://datastudio.google.com/u/1/reporting/078c25fe-d721-4697-91d6-ddc94372ed0e/page/DQBDB

@olegpopovskiy It looks good!!!

Calvin, the CRON process add new data each 6 hours, I don't know any source which gives a new data more often.
World Health Organization updates post a new "pdf" rapport just once a day, so I think 6 hours is relevant.
How it works: Datastudio dashboard updates itself automatically through Datastudio Connector (just a Google Script) that grab data from db. I made 2 Datastudio Connectors, one gives only the last values, the second gives all available history.
Actual state: https://datastudio.google.com/datasources/cda8ed00-4a50-45ee-8fe7-e77e9ca35dd9
Timeline https://datastudio.google.com/datasources/2a9eb533-1a4d-4154-9445-63933c6e01e0
So you can create your own dashboard, share and embed it on website.


[#132312331] Update paas-rds-broker submodule

#132312331 Update dependencies of rds-broker
What
This change bumps the submodule for paas-rds-broker to include the changes from https://github.com/alphagov/paas-rds-broker/pull/18
How to review
Check the commit shas look good.
Can be tested by running the branch for https://github.com/alphagov/paas-cf/pull/574
Who can review
Not @richardc

Works fine and passes all the tests


XIVY-3565 adjust pipeline for release/8.0

consume 8.0 product

simulated this pipeline run here: https://jenkins.ivyteam.io/job/ivy-project-demos/job/release%252F8.0/2/

seems that you want to merge this branch into master.
change looks good, if you merge it to releaes/8.0 branch.


Fix projects page images for mobile

Currently, images in the projects page break the viewing experience. This pull request changes how they are displayed with grid magic to make them automatically wrap to look neat.
Before:

After:

yay

@FireIsGood real quick since I'm busy atm, does this retain the desktop appearance?

Yes, it uses grid but has essentially auto 1fr which shows up the exact same as flex from before. I kept the same max-height so the sizing should also match exactly. I can grab a picture in a minute.


Update openssl to version 1.1.1r for OpenJ9

See also https://github.com/eclipse-openj9/openj9/pull/16086.

@keithc-ca - I read an article yesterday saying that this version is being pulled by the OpenSSL consortium?

https://mta.openssl.org/pipermail/openssl-announce/2022-October/000237.html
We should probably just wait for 1.1.1s.


cmake: extend target_ld_options() to support grouping of flags

Fixes #28456
This commit extends target_ld_options() with a NO_SPLIT flag.
Specifying NO_SPLIT will ensure that all linker flags will be applied
together when testing the compiler and linker.
This allows a caller to ensure that flags are tested together.
This fixes the RISC-V case where the flags -mabi and -march were
omitted because they were tested individually.
Note, the update of target_ld_options() will allow the same flag on
zephyr_ld_options()
Signed-off-by: Torsten Rasmussen<EMAIL_ADDRESS>

@nashif have time to approve ?


rename operation menu

Update Operation to Operations
Environment

Walrus version: v0.5.0-rc3

Environment

Walrus version: main a803edf588fc3dc8024ebdb4745b184b3b40bba7
UI version: dev-186c2e0

Test Result: pass


Feature/mattermost sink

Great work!
I left few comments. It's mostly same comment, in few different places. Can you also add a screenshot of how the message looks to the docs?

Yeah, I've updated the docs with new screenshot, so it includes the file attachments now


Define PropTypes to avoid "unknown props on  tag" React warning message

React 15.1.0 added the following warning
Warning: Unknown props live, minPeriod, maxPeriod on <time> tag. Remove these props from the element. For details, see https://fb.me/react-unknown-prop
You should avoid passing down those props to the rendered <time> tag.

The just released version 0.7.1 silences the warning for Ago's own properties (date, tooltipFormat and autoUpdate). Regarding the properties you pass to Ago (e.g. live, minPeriod, maxPeriod), https://fb.me/react-unknown-prop states that "When transferring props to a child component, you should ensure that you are not accidentally forwarding props that were intended to be interpreted by the parent component."
I consider this issue resolved. Feel free to re-open if need be.


[Error] Attempted to assign to readonly property.

Hi!
I'm getting the following error when importing the module in my project:

OS: macOS Sierra 10.12.6
RN Versión: 0.48.3

I closed it because it was a linking issue.


Issue with my bot in discord

The discord bot is working absolutely amazing. I made my own bot using the same code from the github repo and followed every single thing in order to run my own bot, But I do have some issues with my bot though,
ISSUE: Whenever i have to interact with the bot i have to run 'npm start' and keep the terminal & chrome open (with ChatGPT logged in) or else the bot goes offline.
I want my bot to continuously work even if i stop the terminal
I would be more than happy if anyone can help me out.

I think there is nothing you can do atm. Thats how the system works right now. Until the official API launches, I dont think there is anything the auther can change.


Tabs for external components

Doc: added tabs for external components

Please rebase this PR and take care about file name case. .dxy.in files are expected to use identical name as the component folder.


There is something wrong when crx gate is controlled in parameterized quantum circuit.

Environment

Qiskit Terra version:  0.23.0
Python version: 3.9
Operating system: Windows 11

What is happening?
Running the circuit will result in an error when a CRX gate is controlled in the parameterized circuit.
How can we reproduce the issue?
Input:
beta = Parameter('θ')
circ = QuantumCircuit(3)
circ.crx(beta, 2, 0)
c1 = circ.control(1)
c1.measure_all()
backend = Aer.get_backend('aer_simulator')
c = transpile(c1, backend)
circuits=c.bind_parameters({c.parameters[0]: 2})
job = backend.run(circuits)
counts = job.result().get_counts()
print(counts)

Output:
Simulation failed and returned the following error message: ERROR: Failed to load qobj: Unable to cast Python instance to C++ type (#define PYBIND11_DETAILED_ERROR_MESSAGES or compile in debug mode for details)
But if we use cry gate, the result is correct:
Input:
beta = Parameter('θ')
circ = QuantumCircuit(3)
circ.cry(beta, 2, 0)
c1 = circ.control(1)
c1.measure_all()
backend = Aer.get_backend('aer_simulator')
c = transpile(c1, backend)
circuits=c.bind_parameters({c.parameters[0]: 2})
job = backend.run(circuits)
counts = job.result().get_counts()
print(counts)

Output:
{'0000': 1024}
If we replace c1 = circ.control(1) with c1 = circ.control(2), the result is also correct:
Input:
beta = Parameter('θ') circ = QuantumCircuit(3) circ.cry(beta, 2, 0) c1 = circ.control(2) c1.measure_all() backend = Aer.get_backend('aer_simulator') c = transpile(c1, backend) circuits=c.bind_parameters({c.parameters[0]: 2}) job = backend.run(circuits) counts = job.result().get_counts() print(counts)
Output:
{'0000': 1024}
What should happen?
Expected answer
{'0000': 1024}
Any suggestions?
No response

The same error when mcrx gate and mcry gate are used in PQC.

This seems to be caused by #8723, as the transpiled circuit contains CUGates where the parameter is still unbound. As a workaround until that is fixed you could use the RXGate with two controls:
from qiskit.circuit import Parameter, QuantumCircuit
from qiskit.circuit.library import RXGate
from qiskit import Aer, transpile

beta = Parameter('θ')
gate = RXGate(beta).control(2)
circ = QuantumCircuit(3)
circ.append(gate, [0, 1, 2])
circ.measure_all()

backend = Aer.get_backend('aer_simulator')
bound = circ.bind_parameters([2])
tqc = transpile(bound, backend)
job = backend.run(tqc)
counts = job.result().get_counts()
print(counts)

This should have been fixed by #11032.  Feel free to re-open if that's not the case.


ifs test: skip reason when non linux

Provide a skip reason when the tests: ifs, ifs_array_bist are run in an OS that is not linux.

@alexjch, I think I'm not following, why is needed to add a skip message?


simplify TachyonFile

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2548/
Test PASSed.

+1 LGTM

Thanks!


inverse for Isometry

Fixes https://github.com/Qiskit/qiskit-terra/issues/5099
In #4181, we are deprecating the root class Gate to accept matrices as parameters. However, Isometry creates a generic Gate with a matrix as parameter:
iso = np.array([[1, 0], [0, -1]])

isogate = Isometry(iso, 0, 0)
isogatedg = isogate.inverse()

type(isogatedg)

qiskit.circuit.gate.Gate

This results in several warning messages when a unitary is controlled, see https://github.com/Qiskit/qiskit-terra/issues/5099
This PR adds inverse method to Isometry.

I'm not sure if this is a general enough fix. I still see a bunch of deprecation warnings in the output from running the tests. For example: https://dev.azure.com/qiskit-ci/qiskit-terra/_build/results?buildId=19203&view=logs&j=8eacdd59-c2e8-5617-75a5-8ed7c854a78f&t=5df005f4-d5f4-5d2d-0307-5b24b15488fc&l=1190

Hamiltonian gate was not having validation. Now it does.


Update the Windows Documentation

I am currently trying to get my webIDL changes done, but cannot get GitAhead to work with my 2FA, so I am reverting to windows to the Github Desktop because they don't have a Linux version.

Thanks for the PR! I don't think this is a Windows-only thing, though. Perhaps this is a bug in Cargo?

Could I put a disclaimer that if you encounter an issue (I used Linux first with no problems), that you could try this alternate command?

Er sorry, my point is that something weird happened here. If this doesn't work for you that may mean this was an accidental breaking change in Cargo or some other Rust tooling. The unix/windows behavior should not be different here. If this needs a fix there's nothing Windows-specific about the fix, it just needs to be fixed. Ideally there'd be a little bit of digging to figure out what changed as well to see if this is a bug in Cargo.


ResponseInterface points to a callable

In the release 3.0.0alpha9 more precisely src/ConfigProvider.php we found that definition of ResponseInterface::class in the line 54 is:
ResponseInterface::class => Container\ResponseFactoryFactory::class,

The problem with that is that the component https://github.com/zendframework/zend-expressive-authentication when retrieving a ResponseInterface from the container, the new definition returns a callable, but the return type hinting of \Zend\Expressive\Authentication\ResponsePrototypeTrait::getResponsePrototype expects a ResponseInterface

Fixed in zendframework/zend-expressive-authentication#18.
Merged into zend-expressive-authentication 1.0.0alpha3.


Add missing import length in PySpark Serving Docco

Adds the missing import which completes hello world example for pyspark serving

Thanks for finding this!


Issue handling response back

Here is the error i am getting after updating open ai library 0.28.1 and also latest code is breaking:
Traceback (most recent call last):
File "C:\Code\chatapp\sample-app-aoai-chatGPT\app.py", line 237, in stream_with_data
lineJson = formatApiResponseStreaming(rawResponse)
File "C:\Code\chatapp\sample-app-aoai-chatGPT\app.py", line 332, in formatApiResponseStreaming
"content": rawResponse["choices"][0]["delta"]["content"],
KeyError: 'delta'
used to work before.

I try to correct the output schema and test still the page goes blank when we get streaming response.

Hi @balakreshnan thanks for raising this issue, we will look into it. In the meantime you might try to downgrade the openai package to 0.27.7 or earlier.


Disappearing unread count

On macOS Big Sur, with the pre-release NNW 6.0d4 and Feedly account, every refresh removes the badge of unread items, everywhere.
I didn’t check iCloud account because I’ll use it  as main account when the iOS test build is available.
Another thing that happens every time is that if NNW receive the command to quit takes a long time to exit complete from the app with the spinning wheel rotating.

@kevinm6 Could you please upload a recording of the problem you are seeing with disappearing unread item badges?  I just want to make sure we are talking about the same thing.

@kevinm6 Could you please upload a recording of the problem you are seeing with disappearing unread item badges?  I just want to make sure we are talking about the same thing.

@vincode-io sure! Here we are… as you can see, with the iCloud account there’s no similar bug.
If you need some other test to do, ask me.
https://user-images.githubusercontent.com/72861758/105008535-a9014300-5a39-11eb-828e-1ba6a07bc2dd.mov

@vincode-io sure! Here we are… as you can see, with the iCloud account there’s no similar bug.
If you need some other test to do, ask me.
https://user-images.githubusercontent.com/72861758/105008535-a9014300-5a39-11eb-828e-1ba6a07bc2dd.mov

Sorry, is almost the same of #2727, but for me happens only with the Feedly account where I have linked even NNW 5.0.5 (Build 56) on the iPhone! Instead the iCloud one works fine for now, with only one device synced.

Sorry, is almost the same of #2727, but for me happens only with the Feedly account where I have linked even NNW 5.0.5 (Build 56) on the iPhone! Instead the iCloud one works fine for now, with only one device synced.

I don't know if this can help:
I've removed the Feedly account, close NNW (it's taking a lifetime to close), then open it again and sign in back to Feedly account and wait for the complete sync of all the feeds.
Now I don't have this bug for the unread badges on both Feedly and iCloud account.

I don't know if this can help:
I've removed the Feedly account, close NNW (it's taking a lifetime to close), then open it again and sign in back to Feedly account and wait for the complete sync of all the feeds.
Now I don't have this bug for the unread badges on both Feedly and iCloud account.

@kevinm6 When the issue with you Feedly unread count badges was happening, did it persist after restarting NNW, but before you deleted your Feedly account?  I'm trying to determine if we got memory or the database into a weird state.

@kevinm6 When the issue with you Feedly unread count badges was happening, did it persist after restarting NNW, but before you deleted your Feedly account?  I'm trying to determine if we got memory or the database into a weird state.

Yes, exactly!
Was happening even after restarting NNW.
I've tried after restart it and I've tried also deactivating all the accounts and enable them after the restart.

Yes, exactly!
Was happening even after restarting NNW.
I've tried after restart it and I've tried also deactivating all the accounts and enable them after the restart.

@vincode-io I think you are in the right direction, because now I remember it kept happening also after I tried to move all the feeds to the iCloud account, refresh the app and copy them back.
Maybe it could be some bug or conflict  that generated by the update.

@vincode-io I think you are in the right direction, because now I remember it kept happening also after I tried to move all the feeds to the iCloud account, refresh the app and copy them back.
Maybe it could be some bug or conflict  that generated by the update.

What I think happened is that the FetchAllUnreadCountsOperation was failing or getting back zero results, but that selecting the articles themselves worked fine.  That is why when you click on a Feed the unread count would appear.  When articles are fetched, they update the Feed with the unread count from the fetched articles.
Why that is happening, is a mystery to me.  In retrospect I should have had you send me the database for your Feedly account.  It was probably corrupted in some way, but now we don't have a way to figure out how or if it was.

What I think happened is that the FetchAllUnreadCountsOperation was failing or getting back zero results, but that selecting the articles themselves worked fine.  That is why when you click on a Feed the unread count would appear.  When articles are fetched, they update the Feed with the unread count from the fetched articles.
Why that is happening, is a mystery to me.  In retrospect I should have had you send me the database for your Feedly account.  It was probably corrupted in some way, but now we don't have a way to figure out how or if it was.

Could it be possible that maybe the exactly same name of the folders feeds on different accounts (same database) helps to create this problem?
Sorry, but I don't know how you developed the code about counting the unread badges

Could it be possible that maybe the exactly same name of the folders feeds on different accounts (same database) helps to create this problem?
Sorry, but I don't know how you developed the code about counting the unread badges

Different accounts are stored in separate databases.

Different accounts are stored in separate databases.

Got it.
I’m trying the new version @brentsimmons (6.0d5)! It seems much faster and all for now is working fine.
The only thing is that I didn’t found any updates from the previous one, in NNW > Settings > Advanced > Check For Updates (and the radio button is set to "Test Builds”), I don’t know if is intended.

Got it.
I’m trying the new version @brentsimmons (6.0d5)! It seems much faster and all for now is working fine.
The only thing is that I didn’t found any updates from the previous one, in NNW > Settings > Advanced > Check For Updates (and the radio button is set to "Test Builds”), I don’t know if is intended.

I just haven’t actually released it yet. :)

I just haven’t actually released it yet. :)

We can revisit this if it comes up again.


Registry errors

I'm seeing a lot of errors like Registry error Request to https://registry.npmjs.org/jshint-stylish/ failed when running npm-check.
The modules for which the request fails change every time I run npm-check.

Interesting, sorry this is happening for you.
Can you give me more details?

Has this always been happening?
Is it happening today?
Does it happen with other projects?
Can you share your package.json?
How's your internet connection? Can you share the output for https://www.npmjs.com/package/speed-test

In the future I'll try to provide more details with the registry errors. I've never seen them before so I didn't think it would be necessary.

Hi @dylang, thanks for your quick reply.

Has this always been happening?
On my work desktop, yes. It seemed to be working just fine at home last night.

Is it happening today?
Yes, I just reproduced it.

Does it happen with other projects?
I just tried to reproduce it with 4 other projects and was not able to.

Can you share your package.json?
I've sent you a copy to the email address listed on your GitHub profile.

How's your internet connection?

      Ping  49 ms
  Download  65.3 Mbps
    Upload  12.2 Mbps

I'm on x64 Windows 7 with Node 4.0.0 in case that has any relevance.

I ran it with your package.json and using Node 4.0.0 and had no issues. There was just one package it couldn't get a version for, and it looks like it's a privately scoped one.
I don't have a Windows VM to try this on, but the network code is pretty generic, just http requests using common modules.

On my work desktop, yes. It seemed to be working just fine at home last night.

Could it be an issue with your network at work?
Do you have co-workers that could try it? I'm especially curious if it is only happening on Windows.
Have you tried other versions of Node?
Does anyone else have this issue? I could use a hand to get something I can repro locally.

Just FYI, the complete error is:
{ [GotError: Request to https://registry.npmjs.org/chai/ failed]
  nested:
   { [Error: connect ETIMEDOUT <IP_ADDRESS>:443]
     code: 'ETIMEDOUT',
     errno: 'ETIMEDOUT',
     syscall: 'connect',
     address: '<IP_ADDRESS>',
     port: 443 },
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 443 }

I'll see if I can find out anything else.

Co-worker has the same issue. He's on Windows 8.1 as well. I'll check OSX later as well.

Seems to work fine on OSX. I'm on Windows 10 at home. So either this is specific to our network in combination with Windows, or it's something with Windows 8, I guess :\

<IP_ADDRESS> is not the ip address I get for registry.npmjs.org. Maybe that's some kind of proxy or load balancer which is not as reliable as the npm registry everyone else is using?
 nslookup registry.npmjs.org
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

Non-authoritative answer:
registry.npmjs.org	canonical name = a.sni.fastly.net.
Name:	a.sni.fastly.net
Address: <IP_ADDRESS>

I'm going to close this as it does not seem related to npm-check, but if you find a solution or reason why this happening I'd love to know.

I have the same problem, with the same ip address. See #144.


Case-insensitive update prompt reply

LGTM

@LalatenduMohanty Fancy using the github review approvals?  Then I'll enable that as a mandatory status check before merge.

@jimmidyson Yup, I think thats a good idea.


Platform: extend WinSDK further

This extends the WinSDK module definition further to better modularise
the headers.  This should improve the header organisation as well as
setup additional autolink rules.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test Windows platform

@swift-ci please smoke test


Style guide template closing tags mismatch

Summary
The style guide template at wagtail/contrib/styleguide/templates/wagtailstyleguide/base.html is not correctly formatted. There are td and p tags that are not closed. That is only the beginning: w3c validator reports 31 errors.
I'd also like to rewrite <img src="{% gravatar_url<EMAIL_ADDRESS>%}" /> =>  <img src="{% gravatar_url<EMAIL_ADDRESS>%}" />. Single and double quotes makes these tags more legible.
Steps to reproduce

Follow instructions on adding the styleguide http://docs.wagtail.io/en/v2.0/contributing/styleguide.html
Lookup the source copy paste in https://validator.w3.org/
See errors

👍 there should be no validation errors on any part of Wagtail, including this. For the img tag rewrite – agreed.

Closed by #4400.


Improve benefits summary

Improve handling of benefits choice

route properly to summary if savings threshold exceeded
Improve the summary page to properly display messages
add a deleted view for when DWP result is deleted
improve last_benefit_check to get most recently generated!

Looks fine


Master broken with bash

Prompt is really messed up on master with bash

Yes, I have a hard time to figure out how to put the bash version non printing chars in the output. #17 Bash may have bugs when v1 is released. I am happy to fully support at least 1 shell.

Those non printing chars you see %{ and %} are for ZSH and BASH uses fecked up \[ and \] where the biggest problem is to get that escaping backslash right.

I decided to leave BASH support out from the v1 release, I just can't yet figure out how to properly handle those invisible characters down to BASH.

Axing bash support for now.


Selectively Remove Direct Message and Invite to Stream Capability from Zulip Users

We will most likely move to Zulip form Slack for three reasons

Self hosted - no issues with sensitive data on external servers
Threaded Conversations - WOW
Ability to modify to suit our needs

On Slack, our DM/Channel Messag ratio was about 90%, we had no way to enforce people to use Channels (Streams) so that members of the team could benefit from the exchanges collectively
We want to implement functionality to do the following
Globally Switch Direct Messaging to Default either Off or ON
Allow override of this global setting on a per user basis.
Our implementation scenario would be to allow managers to Dm, but Team members to not be able to do DM, they must use a Stream
Additionally, we would like to be able to be able to apply the same logic to invitations to streams, have a Global Switch with a per user override
Depending on what is involved in this, I may have one of my Django Devs take a look, but before this, would appreciate any tips on the best way to implement this functionality from those who know the product better than us.
I was just thinking about hiding the css that displayed the DM Create Message Button or showed it as the case may be, or is that too simplistic ..

So there are a few features discussed here:

Being able to restrict who can invite other users to streams is a totally reasonable feature to support.  The simplest implementation would be to add a flag on the Realm object with the effect of enforcing that only administrators can add other users to streams; would that work for your purpose?  If so, that's a feature that's of general interest so I'd be happy to review a pull request to add it (check out the new feature tutorial on readthedocs for how to add a new Realm flag; the rest of the feature should be pretty straightforward).
I'd like to understand a bit better the goals around preventing users from sending private messages (e.g. is this more a discouragement thing or a security requirement?); there's a few ways that one could do it and I'm not sure which is best for what you're trying to do.

@MarkCupitt If you'd like to talk with us about this in chat sometime, "diversity of user needs" in the Zulip developer realm would be a great place to talk about it. (Calendar of Zulip developer events where a lot of people are in that realm simultaneously -- let me know if you'd like something better for your timezone.)

Both of these features were added at various times in the last few years.  Closing.


Update .pyup.yml

Switch pyup to master branch

Coverage remained the same at 76.426% when pulling e14fcf48e1b948c4bca85ed051b649603fbb4278 on SillyInventor-patch-1 into 6582d826c46935162e296e955508487a8046e1c6 on master.


GEO key

Can you add GEO key?
Or can I add custom  key=value pair?

Could setProperty be set to public so that custom properties like GEO could be added easily?


fix for working with compressed data. Fixes #6746.

Using correct API that gives length of uncompressed data so correct amount of memory is allocated. Derived from https://github.com/grpc/grpc/pull/6719.

LGTM other than my suggestion to expand the comment.

LGTM

The C++ build failure and ObjC "opt" test failure are unrelated. Marking ready to merge.

Please wait for the Travis tests to complete.

Oh, they were rerun because you pushed a new commit; I see.

test this please

test this please

test this please


Old image? Agent / Notebook on kubernetes not starting (memory-limit not set)

memory / cpu may not be null
memory_limit / cpu_limit Unknown field

Same issue for cowait agent and cowait notebook.
Looks to me like it's running on an old image.
Traceback (most recent call last):
  File "/opt/conda/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/opt/conda/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/var/cowait/cowait/exec.py", line 45, in <module>
    asyncio.run(main())
  File "/opt/conda/lib/python3.7/asyncio/runners.py", line 43, in run
    return loop.run_until_complete(main)
  File "/opt/conda/lib/python3.7/asyncio/base_events.py", line 583, in run_until_complete
    return future.result()
  File "/var/cowait/cowait/exec.py", line 27, in main
    taskdef = env_get_task_definition()
  File "/var/cowait/cowait/worker/env.py", line 24, in env_get_task_definition
    return TaskDefinition.deserialize(taskdef)
  File "/var/cowait/cowait/tasks/definition.py", line 107, in deserialize
    return TaskDefinitionSchema().load(taskdef)
  File "/opt/conda/lib/python3.7/site-packages/marshmallow/schema.py", line 724, in load
    data, many=many, partial=partial, unknown=unknown, postprocess=True
  File "/opt/conda/lib/python3.7/site-packages/marshmallow/schema.py", line 905, in _do_load
    raise exc
marshmallow.exceptions.ValidationError: {'memory': ['Field may not be null.'], 'cpu': ['Field may not be null.'], 'memory_limit': ['Unknown field.'], 'cpu_limit': ['Unknown field.']}

I think we need to publish a new image, the latest cowait/task is tracking 0.3.2, not the master branch.
Building the base image locally should fix this problem


Styles customization

Hello, I need some help with new approach in components customization.
How can I override the font size in tooltip?
How can I change the hover background color on letter avatar?
How can I change the font family of menu items
Thanks

How can I override the font size in tooltip?

Are you talking about tooltips for IconButtons? There is a tooltipStyles prop that you can use to override the font size.

How can I change the hover background color on letter avatar?

Try using the style prop of the Avatar component? For that, you would have to apply a CSS class to the Avatar component using the className prop and then define a rule for the hover of that class name in a stylesheet. Or, you could also try the onMouseOver and onMouseOut events of your Avatar component to achieve the desired effect.

How can I change the font family of menu items

Again, either try overriding the style prop or you could also change attributes in the ThemeManager for menu / menuItem / menuSubheader.


Bot randomly restarting, and locking up

The bot randomly restarts, and the log throws an exception, something to do with deleting the music cache.

:+1: Having the same issue:
/home/ubuntu/MusicBot/musicbot/bot.py:467: DeprecationWarning: change_status is
deprecated, use change_presence instead.
  await self.change_status(game)
/home/ubuntu/MusicBot/musicbot/bot.py:467: DeprecationWarning: change_status is
deprecated, use change_presence instead.
  await self.change_status(game)
Unclosed client session
client_session: <aiohttp.client.ClientSession object at 0x7f6417809ba8>
Cleaning up... Done.
Restarting in 6 seconds...
Connected!  Musicbot v1.9.5_2

Bot:<PHONE_NUMBER>00361216/Tenno DJ#9082
Owner:<PHONE_NUMBER>92208128/Jroot#8988

agraubert, looks fairly easy to fix.. from what i can see from you, simply open up the bot.py file in a text editor(notepad ++ would be preffered, for the line numbers, and syntax highlighting) and go to line 467 and change the "change_status" to "change_presence" instead, see if that helps.
however justandras, not sure what your problem could be.. try running the bot, and posting the logs if it still locks up/restarts, and we can take it from there.

I've enabled DebugMode right now, next time the bot locks up I will be posting the logs. I totally forgot that there is a debug mode, thanks for reminding.

@Pesoen The bot went offline tomorrow again, and here is the log.
https://imgur.com/a/GM5mJ

Hmm, it keeps complaining that it cannot delete the audio cache. That could be the problem. Try removing the audio cache folder and see if that helps, will look over the other errors when I get home.

If I restart the bot I don't get the error message, so It's kinda strange

Odd. So it only comes after it stops working, and then starts working again?
also what version are you running? Stable dev or review?

How can I check the version? I have installed it in the summer, so it's fom that time period, but this problem existed when I installed it. Windows says that the files have been modified @ 2016.09.28.

Then i guess you are using the master version.. But maybe one that is outdated. Try moving your current bot to another location on your pc, and follow the instructions on installing it again, could help you find a sollution. Simply copy the configuration folder into the new bot folder to keep current settings.

You're using an outdated version of the bot. Try and update and see if it helps.


Unconditionally disable switch when input is empty

this can results in empty user input being passed to git cli where a non-empty string was needed.
regarding tests i haven't any tests related to toggle switches i would be like to guided on that.

lol https://github.com/NeogitOrg/neogit/pull/1015
lesson learned next time i will crawl deeply in repo before making a PR

No worries :)


Remote write: Make HTTP 429 errors recoverable only when the user opts-in

We have to make the 429-retry mechanism in remote-write opt-in, I'd prefer as a remote write configuration flag (per remote-write url).

This is indeed a breaking change. From what I can we see, we never drop data here? This means Prometheus will just keep falling behind if it is being constantly rate-limited? This assumes that Prometheus doesn't send data to a remote system that basically limits it constantly.
This is not true, we limit people in GrafanaCloud pretty regularly and they're completely fine with it. Further, in Cortex, we return 429 not just on a samples per second limit and do it for other cases too. For example, we have a limit for active series and if a user sends more than their active series limit, we return a 429. In this case, Prometheus would never be able to proceed because it would constantly try to send the sample that creates a new series and it would fail forever.
Now I am not sure if Cortex has the right behavior with 429 for other limits, it seemed pretty right when we built it ;) I still think it has the right behavior, but given Cortex has a major chunk of remote-write use-cases, we shouldn't roll out this change in Prometheus until we fix it in Cortex. WDYT @roidelapluie @csmarchbanks?

Originally posted by @gouthamve in https://github.com/prometheus/prometheus/issues/8237#issuecomment-777369196

@gouthamve 429 is not the right behaviour, it should be 400
The HTTP 429 Too Many Requests response status code indicates the user has sent too many requests in a given amount of time ("rate limiting").

It should in general not be about the content of the requests being wrong, but just too many request being sent.

429 is not the right behaviour for cortex IMO

I agree Cortex shouldn't use 429 for the max series/metadata limits and we should (will) change it. However, having the new behaviour not the default would allow Cortex to gain some time to let Cortex operators to upgrade.

I will take up this and submit a fix.

I am happy with a configuration flag (probably defaulted to off for now).
Another future solution would be to limit the number of retries for 429s, e.g. only retry once or twice then drop them to avoid backing up too much

Another future solution would be to limit the number of retries for 429s, e.g. only retry once or twice then drop them to avoid backing up too much

May be non trivial, but an alternative approach could be retrying 429 until Prometheus is not lagging behind a threshold. After that drops 429 and move forward with the remote write.

https://github.com/prometheus/prometheus/issues/7912 is an issue to do that for anything retryable.

Another future solution would be to limit the number of retries for 429s, e.g. only retry once or twice then drop them to avoid backing up too much

That's what I meant with Max-Retries header in the PR.


Introduce Debug Task

We should introduce some VS Code tasks for debugging Piral instances or pilets.
It is yet unclear if we can use the same for piral-cli-parcel and piral-cli-webpack. Maybe distinguish for now?

A potential Parcel config
A primer for Webpack
Task provider in a VS Code extension

https://code.visualstudio.com/api/extension-guides/debugger-extension

I'd take this one.

launch.json:
{
  "version": "0.2.0",
  "configurations": [
    {
      "type": "pwa-msedge",
      "request": "launch",
      "name": "webdebug",
      "url": "http://localhost:1234",
      "webRoot": "${workspaceFolder}",
      "runtimeExecutable": "stable",
      "runtimeArgs": ["--headless"]
    }
  ],
  "compounds": [
    {
      "name": "Launch Piral Instance",
      "configurations": ["webdebug"],
      "stopAll": true,
      "preLaunchTask": "npm start"
    }
  ]
}

and the tasks.json:
{
  "version": "2.0.0",
  "tasks": [
    {
      "type": "npm",
      "script": "start",
      "label": "npm start",
      "isBackground": true,
      "problemMatcher": {
        "owner": "webpack",
        "severity": "error",
        "fileLocation": "absolute",
        "pattern": [
          {
            "regexp": "ERROR in (.*)",
            "file": 1
          },
          {
            "regexp": "\\((\\d+),(\\d+)\\):(.*)",
            "line": 1,
            "column": 2,
            "message": 3
          }
        ],
        "background": {
          "activeOnStart": true,
          "beginsPattern": "Running",
          "endsPattern": "compiled successfully"
        }
      },
      "options": {
        "cwd": "${workspaceFolder}"
      },
      "dependsOn": []
    }
  ]
}

Part of 0.3.


Can you test against c-json

Hi, I wrote a simple JSON parser and would love for you to include the results in your project.
You can get to it at https://github.com/amwales-888/c-json
On linux to build, type 'make' and copy the 'json' binary to the parse location, the format to run is.
'json filepath' it will return 0 on success and 1 on failure.

I found this issue while looking for documentation on adding my own json parser to the list. I could add this parser too if I can get some guidance.


feat: scope detection

Summary of Changes

implemented a feature to detect the scope of classes, functions, calls and variables
works with arbitrarily complicated nested modules
it generates a three, where each node consists of its astroid node, its children and its parents

An example can be seen below:

Testing instructions

run  test_get_scope

🦙 MegaLinter status: ❌ ERROR

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ PYTHON
black
3
2
0
1.98s

❌ PYTHON
mypy
3

16
2.56s

❌ PYTHON
ruff
3
3
89
0.08s

✅ REPOSITORY
git_diff
yes

no
0.04s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by


Performance Issues due to Debug Mode

Section 3.4 states that the game is running slowly due to unbatched draw calls, but the reality is that rust's debug mode is very slow. Compiling in release mode or turning on optimizations when compiling in debug mode will make the game much faster. I added the following to make rust optimize in debug mode:
[profile.dev]
opt-level = 2

Here's a table with the performance I got with the various techniques:

Optimizations
Spritebatching
FPS

No
No
7 fps

No
Yes
47 fps

Yes
No
144 fps

Yes
Yes
144 fps

I know of this technique from following (and occasionally contributing to) veloren, which is a good example of how specs can be used to make a larger game.

@desttinghim this is super interesting, thanks for the thorough analysis. it's pretty insane the effect the debug optimizations have. this was all with cargo run or cargo run --release? I would be curious to see how batching does in release mode, I think I was using release throughout the whole book. The optimisations flag would not do anything in release right?

This was all with cargo run. The optimizations flag won't do anything in cargo run --release, since only [profile.dev] is changed. You can read more about profiles here: https://doc.rust-lang.org/cargo/reference/profiles.html
Something I should probably note is that my screen refreshes at 144hz, which is why I got 144 fps instead of 60.


[DOC] Add CXX and Python API Doc

CXX API Doc refer to
https://paddle-inference-demo-qili93.readthedocs.io/en/add_api/api_reference/cxx_api_index.html
Python API Doc refer to
https://paddle-inference-demo-qili93.readthedocs.io/en/add_api/api_reference/python_api_index.html

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: qili93:x: liqi27liqi27 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


rkt: support overlay filesystem

This mounts stage1 and the application images as an overlay filesystem
using each ACI's cached tree as the lower filesystem.
Also, the mounts are done in a separate mount namespace so they will be
unmounted when the container exits and they're not visible by the rest
of the system.
Systems that don't support overlay fall back to plain copying.
Addresses #387

cc @jonboulle @philips @vcaputo

Addressed @alban's comments and rebased to master

please rebase now that #332 finally landed!

Yay! Rebased

No red flags from me! One thing I noticed is we have 4 paths with "if useOverlay; else". We may consider refactoring this in the future but LGTM.

Let's change the 0776 to 0755.

:+1:


Pin version of mdBook

Hi!
We would like to issue a new release of mdBook containing some changes that could potentially break current books, the config file changed a bit for example. However we found out that a lot of projects automatically deploy using the latest available version and so any breaking update could result in books not being deployed anymore..
We would like to avoid that and recommend pinning the version of mdBook in your deployment script. Cargo supports semver ranges on cargo install (right now it issues a deprecation warning, but in the future this warning should go away thanks to rust-lang/cargo#4229)
You can thus simply change cargo install mdbook to
cargo install mdbook --vers "^0.0.22"

Next release will be 0.1.0 and we will try to adhere to semver as much as possible.
For more info, azerupi/mdBook#327 is the tracking issue on our repository.

Thank you very much for taking the time to open this issue, we'll do this change soon.


Make screenreader commentary follow HTML5 workflow

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #111
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Changes include:

Setting the aria-hidden attribute on auro-menu to be true in order to stop all menu options from being read when auro-select gains focus
Adding a new method that forces the screenreader to read the placeholder text like a label when auro-select gains focus

Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:shipit:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fixed from_embeddings documentation

Description: Updated FAISS documentation that throws an error,

This doesn't seem like a necessary fix, and if you give me some code to reproduce, I'm happy to reopen this!


Add simple marshalling for graveler.Value

As discussed f2f this allows separating graveler from the underlying trees wrapper, making each
unaware of the others.  Use our own trivial serialization format that allows high-speed
operation with no copying: in particular there is no need to copy data, or to deserialize more
than Identity.

This change is

Codecov Report

Merging #1067 (61ae2ea) into master (4bc2256) will increase coverage by 0.06%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master    #1067      +/-   ##
==========================================
+ Coverage   46.65%   46.72%   +0.06%
==========================================
  Files         160      161       +1
  Lines       12806    12841      +35
==========================================
+ Hits         5975     6000      +25
- Misses       6070     6075       +5
- Partials      761      766       +5

Impacted Files
Coverage Δ

graveler/value.go
71.42% <71.42%> (ø)

catalog/mvcc/cataloger_commit.go
77.14% <0.00%> (-2.86%)
:arrow_down:

catalog/mvcc/cataloger_create_branch.go
84.61% <0.00%> (+3.07%)
:arrow_up:

Continue to review full report at Codecov.

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


Cannot install it for visual studio 2022

I am using visual studio 2022 and godot 3.4.4 both x64. I downloaded the extension and hit install. After some time I got this error:
How to make it work?
Package 'Xamarin.VisualStudio,version=<IP_ADDRESS>' failed to install. System.IO.IOException: The file 'C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Xamarin\Microsoft.VisualStudio.RemoteControl.dll' already exists.
at Microsoft.VisualStudio.Setup.Installer.VsixInstaller.g__install|31_1(PackagePart part, Boolean saveExtensionManifest, <>c__DisplayClass31_0& , <>c__DisplayClass31_1& , <>c__DisplayClass31_2& , <>c__DisplayClass31_3& , <>c__DisplayClass31_4& )
at Microsoft.VisualStudio.Setup.Installer.VsixInstaller.InstallCoreInternal(String localPath, String installDir, InstallablePackage pkg)
at Microsoft.VisualStudio.Setup.Installer.FileInstaller.<>c__DisplayClass7_0.b__0()
at Microsoft.VisualStudio.Setup.Installer.FileInstaller.DoAction(Func`1 action)
at Microsoft.VisualStudio.Setup.Installer.FileInstaller.InstallCore(String localPath, String installDir, InstallablePackage pkg)
at Microsoft.VisualStudio.Setup.Installer.InstallerBase.InstallHelper(InstallData installData)

Ok, I seem to made it install. Not sure what went wrong. I had Desktop development with C++ selected altogether with c# .net development. So I removed the C++ one and used modify for visual studio. Then installed the plugin successfully.

Thanks for posting your findings. I removed all my C++ components
Desktop development with C++
Mobile development with C++
Game development with C++
And that fixed it for me as well.


usenativedriver for better animation performance

Is your feature request related to a problem? Please describe.

The UseNativeDriver flag allows the animations to happen completely on the native side.
https://facebook.github.io/react-native/docs/animations.html#using-the-native-driver

Describe the solution you'd like
A clear and concise description of what you want to happen. Add any considered drawbacks.

When using native driver some properties are no longer animatable. It should be possible to use transforms scale and translate to achieve the desired results (for the animated mask).

Just wondering if this has been considered before. Thanks!

I know I tried using the native driver but there were some issues which I don't quite remember. I'll try to check this once again. Thanks!


Convert to javascript please

Hello I want to add this script to my website like Printer.js

prntr.cjs.js:3 Uncaught ReferenceError: process is not defined
    at prntr.cjs.js:3:5
(anonymous) @ prntr.cjs.js:3
prntr.esm.js:1 Uncaught SyntaxError: Cannot use import statement outside a module

Help me :(

I am adding umd build which you should use:
<script type="module"<EMAIL_ADDRESS>


How does this implementation differ from the one by Chengdazhi?

The implementation by Chengdazhi is Old (2 years ago) and contains build requirements. However, your implementation is pretty simple, easy, and straightforward! (Thanks for that! :) But I just want to know what is the difference between these two implementations, and have PyTorch included a few operators in their latest version so that we don't require to build anything from scratch for Deformable Convs??

@rushirajsherlocked
Hi, Parmar!
You said But I just want to know what is the difference between these two implementations, and have PyTorch included a few operators in their latest version so that we don't require to build anything from scratch for Deformable Convs??

what is the difference between these two implementations

You can compare the difference between Chengdazhi's implementation and torchvision's implementation  to check the difference.

and have PyTorch included a few operators in their latest version so that we don't require to build anything from scratch for Deformable Convs

Yes, you are right!


Allow looping over ranges

Make foreach (var i in 0..10) possible.
See https://youtu.be/jmmz1cInNow

I would like to tackle this one, if possible.

Go for it


Parse Server loses MongoDB connection when MongoDB server is restarted

I am running a parse-server on a Digital Ocean droplet in addition to a MongoDB server on a separate Digital Ocean droplet. When i reboot or restart the MongoDB server the connection between parse-server and MongoDB is lost and requires a stop and restart of the parse-server to get things back up and running. Not sure why thats happening and is there anything to prevent it from happening? I was hoping that if the MongoDB server was ever rebooted that once it was finished parse-server would just reconnect automatically. Thanks for all the awesome work you guys are doing!
Make sure these boxes are checked before submitting your issue -- thanks for reporting issues back to Parse Server!

[x ] You've met the prerequisites.

[x ] You're running the latest version of Parse Server.

[x ] You've searched through existing issues. Chances are that your issue has been reported or resolved before.

Environment Setup
2 Digital Ocean Droplets
1 = Parse-Server running on Digital Ocean
2 = MongoDB running on Digital Ocean
Steps to reproduce
Reboot MongoDB server or Restart MongoDB from command line
Logs/Trace

maybe you are not making your MongoDB replica set, so when the time you restart your MongoDB, the connection is dropped.

Thank you! That was the problem!

reopening because still having issues maintaining database connection after setting up a replication set. Heres what I am doing.
Step 1. Replication set up and running
Step 2. Kill the current Primary member
Step 3. Test connectivity and all seems to work.
Step 4. Restart the killed previous Primary
Step 5. Kill the current Primary member
Step 6. Test connectivity and connection is now lost with Parse Server and requires a reboot.
Here is my sample format of the MongoDB connection URI:
mongodb://test:1234567@mongo0,mongo1,mongo2:27017/sample?replicaSet=sample

@iOSDev33 does it still occur with 2.2.4?

No response in many days, closing.

Same problem here. The version of parse-server is 2.2.10


Trivial: regarding CMAKE_CURRENT_LIST_DIR

Hi, just wondering why cmake has this:
< string(REGEX REPLACE "cmake$" "" dlib_path ${CMAKE_CURRENT_LIST_FILE})

Just could simply be replaced by:
> set(dlib_path ${CMAKE_CURRENT_LIST_DIR})

I'm sorry but it's too trivial to make a pull request, just sharing what I noticed.
Anyway thanks for lib, it's super great!

I'm glad you like dlib :)
It's because CMAKE_CURRENT_LIST_FILE didn't exist in older versions of
cmake.  It's been so long no one uses those versions of cmake anymore so it
would be safe to update the cmake scripts though.  I'm going to organize
the cmake files a little today and I"ll update this as well.

Thanks for sharing history.
Though it's your decision but let me express that I don't want wasting your time, please feel free to leave as it is, as long as you'd feel so.
This is last comment to bother you before closing this.
I will close after some time waiting for something to happen; you might want to leave change info.
Thanks again!

No worries.  I really was going to clean up the cmake files.  I'm
prepairing the next dlib release right now.  Once that's finished I'll
clean up some things about the current cmake files, including this.

OK I'm very much looking forward to the new release. Thank you for your effort!

New release is out.  I also just updated all the cmake files.
Cheers,
Davis

Thanks for sharing new version! :)


Unify paths

Q
A

Bug fix?
No

New feature?
Yes

BC breaks?
No

Deprecations?
No

Fixed tickets
#922 #921 #920 #879

I'm trying to unify all paths into a variables (similar to Capistrano). What do you think about this PR? I know it's not complete, and will require to change some things and some documentation maybe.
With this PR, one should be able to modify the structure of deployer using the new variables.
We can include new paths like: current_release or previous_release to simplify and make simple to use release_path outside a release.

Ping @elfet. Does this have any change to get merged? I have another PR depending on this one.

This is cool release. But what about this?

I know it's not complete, and will require to change some things and some documentation maybe.

Also what PR you want to?

I have another PR depending on this one.

Also i'm wondering is this functionality is really needed? I mean how many people will be using it? Because this PR bring a lot of config option which lead to complicate thinks and refactoring.
For example releases_path and release_path differ only in one s letter which will lid to potential  bugs. I think good naming will solve problems.

I have another PR which focus on moving all if on the code to the test() function. It was already there, but not used consistently on the code. But since it touched a lot of code too, didn't want to rebase a lot of times.
Capistrano variables structure: https://www.admon.org/scripts/capistrano-variables-list/
Capistrano code: https://github.com/capistrano/capistrano/blob/0965373a4b239665360e15e3ee409f130b6c094b/lib/capistrano/dsl/paths.rb


Extensions do not detect when csharpier is installed or upgraded and in some cases prevent upgrading.

This probably applies to VSCode, VS and Rider.
The extensions only determine the version of csharpier when they start. If someone does not have csharpier installed, then installs it. They need to restart the IDE.
If 0.12.0+ is being used, then the extension keeps csharpier running. Which prevents someone from updating it without closing the IDE.

Modify the keep csharpier running code

If someone is not active in a document, the csharpier process should be killed
When someone becomes active in a document, determine the csharpier version if it is 12+ warm up the process

This may be affected by the logic for using the solution version of csharpier instead of a global version. #493

#563 does a better job of ensuring the IDE does not keep the process running. But we still want to either determine when csharpier was updated and reload the processes in the IDE, or give the user the ability to reload the processes.
VSCode has the timeout implemented, but it should come back out.
Another way to detect when the version changes is to watch the tool manifest file and the global install location.
Or to recheck we are using the versions on occasion. Without killing the running versions.

#569 covers this


Implement User Profiles and Contribution Tracking

Description: Create detailed user profile pages showcasing the user's contributions, such as submitted snippets, bookmarked snippets, and rated snippets.

Functionality:

Display all snippets submitted by the user.
Show snippets the user has bookmarked and rated.
Include a leaderboard highlighting top contributors.

Additional Features: Allow users to see their activity (e.g., ratings, comments, bookmarks) and show their total contributions and ranking on the leaderboard.
Implementation: Create a user profile page, integrate it with Clerk authentication, and pull data from snippets, bookmarks, and ratings.

May I work on this issue @notsoocool

Hi @priyanshuverma-dev ,
Thanks for your interest in contributing! 🙌 You are welcome to work on this issue. Feel free to start by creating a branch and share updates or questions as you progress.
Looking forward to seeing your contributions! 🚀

May I Integrate Prisma ORM also as using raw mongodb will increase complexity.
Should create a new issue related to this and assign me if you (please 😁).
I will solve that first then this issue.
Let me know if it sounds good to you.
Thanks
Priyanshu Verma

@priyanshuverma-dev comment on the new issue and i will assign it to you

@notsoocool I have pulled PR for prisma migration #29 .
Approved and merge so that I can work on this issue

@notsoocool Which branch should I clone for this Issue. I will use Prisma ??

@priyanshuverma-dev  Please pull the develop branch only and continue working with MongoDB for now. I have kept your Prisma ORM integration under consideration, and if it aligns well with the project, I will merge it later. For the time being, let's proceed with MongoDB.

@notsoocool Sounds good to me but it will conflict because use native mongodb that is not done in prisma. Let me work on this and update you.

@notsoocool Sounds good to me but it will conflict because use native mongodb that is not done in prisma. Let me work on this and update you.

no it wont conflict because i haven't merge you pr in develop branch, it is stored in diff branch

@notsoocool How do we fetch user submitted snippets as we don't store submitted by in snippet model??

And profile requires these fields
const user = {
  name: "Alice Johnson",
  username: "alice_j",
  avatar: "/placeholder.svg?height=100&width=100",
  bio: "Passionate coder and open-source contributor",
  joinDate: "January 2022",
  contributions: 127,
  rank: 5,
}

Current have only email.
Profile Page can be only user centric mean logged user can see his own profile only.
Profile I have created till know

For mobile

Its looking great, so for contributions add points for adding snippets, for bookmarks, editing comments etc etc

@notsoocool How do we fetch user submitted snippets as we don't store submitted by in snippet model??

using snippetRequest Model


Convert generated id value to its PHP representation.

According to the conversion rules of a specific DBAL mapping type.

Spotted some failures on tests, will fix them first.

Re-created at #6020


Pip 24.1 mismatch with gym 0.19.0

closes #138

Is the problem here that the version is too high for gymnasium?

This is due to incompatibilities between the older gym and pip. The real fix would be releasing v1.0.0 with gymnasium, but this should serve as a temporary fix

I think the problem might be solved by trying to install with python3 setup.py install instead

This still seems to fail.


Prioritize shields by road classification

The north–south freeway seen here in Northern Kentucky is Interstate 75, but you’d be forgiven for thinking it’s Kentucky Route 1292 or 2366. Both those routes run very close to I-75 along frontage roads, their shields colliding out the I-75 shields. It would be nice if there were some way of prioritizing the shields of higher-classed roads over those of lower-classed roads in the event of a symbol collision. This is not to say that the state routes should’ve necessarily gone unlabeled at this zoom level, just that that shields should’ve avoided the higher-classed road. If that would mean hiding all the shields for these lower-classed roads until a higher zoom level, so be it.

The transportation_name layer has a class property that, in principle, would allow us to have a separate shield layer for each road classification the way we used to for road names before #518. But multiple layers using the same shield rendering technique might pose a performance penalty.


[Flaky Test] tests/python/unittest/test_custom_datatypes.py::test_myfloat

These tests were found to be flaky (intermittently failing on main or failed in a PR with unrelated changes). See the docs for details.
Tests(s)

tests/python/unittest/test_custom_datatypes.py::test_myfloat

Jenkins Links

https://ci.tlcpack.ai/job/tvm/job/main/3612/display/redirect
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/3625/pipeline
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/main/3640/pipeline

cc @gussmith23 @jroesch @hypercubestart

A dup with https://github.com/apache/tvm/issues/11580, feel free to send a PR to disable this test.


about std buffer

I have a question, I haven't looked into the iteration over ASCII strings and UTF-8 strings in std, but I noticed that it's faster than fiter for a 1k buffer. How large is the buffer in std, or is it implemented in a different way?

The iterator using the std lib is the following:
read_to_string(&file).unwrap().chars()

The read_to_string() reads the entire file content into a string and allocates as much memory as needed. So if the file is 1MB big the function allocates 1MB on the heap for that.


Template autoformatting at buildtime

I hoped to add a jte section to Spotless.
But it looks like the formatter is tied pretty intimately to IntelliJ

https://github.com/casid/jte-intellij/tree/master/src/main/java/org/jusecase/jte/intellij/language/format

It seems like it would be possible to make a pretty-printer by implementing a TemplateParserVisitor which implements all the callbacks by appending to a big String.
Any thoughts?

@nedtwigg in theorie yes, you could give it a try. For jte parts it should work, but I'm unsure how to format the underlying language (for instance HTML/JS/CSS). The TemplateParser does omit some stuff if configured, so I think the gg.jte.TemplateConfig would need to be set to htmlCommentsPreserved = true and trimControlStructures = false.

If the template is being used in plain text mode it could be rendering anything. You can't generalise format in that case.

True, but HTML is common enough that TemplateConfig has fields like htmlCommentsPreserved. Something can be useful without being universal.

I don't have time to work on it, but will leave it open as @nedtwigg suggested.


Add to the usage doc: how to control a UR robot via the moveit2 Cpp interface

I'm sorry, but this is kind of out of scope for the driver, as this would be a MoveIt! tutorial. https://github.com/UniversalRobots/Universal_Robots_ROS_Tutorials might be a place for such a thing, but we currently don't plan to write that, so any contribution there would be welcome.
I'm going to close this here, since it is not a direct driver's issue.


Create JoolianDiscovery.netkan

Reference issue:  Joolian Discovery #3825
Edit by @plague006: Closes #3825

Looks good to me, merging with my thanks.


Provide a way to customize cluster-init scripts

When working in enterprise environment you have sometimes to customize the actions at node startup, actually you can do this by updating the provided cluster-init scripts. Being able to isolate these changes in a custom script will make upgrade path easier to apply and maintain.

This would be useful for us as well

available in version 1.0.28+


Custom CA

Hi we are using our own self made certs and wondering if there is a environment variable I can call to add in the CA certificate.
I'm able to mount the cert but unable to run update-ca-certificates in the container. I really didn't want to rebuild docker image with CA copied in. Is there anyway around this.
defectdojo@f95abdab0c66:/etc/ssl/certs$ update-ca-certificates
Updating certificates in /etc/ssl/certs...
rm: cannot remove 'ca-certificates.crt': Permission denied

I went ahead and ran the container as root, and update-ca-certificates , which work from within django-defectdojo-uwsgi-1.
Double checked this by install curl and hitting my Keycloak instance.
When trying in the browser I get the sign-in then the following message after filling out my Keycloak creds.
Authentication failed: HTTPSConnectionPool(host='keycloak.sdm', port=443): Max retries exceeded with url: /auth/realms/sdm/protocol/openid-connect/token (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1002)')))

Take a look at the NOTE section  of the https://defectdojo.github.io/django-DefectDojo/integrations/social-authentication/#saml-20

@downright-ux does it help?

Yes sorry that worked

Yes this works fine thank you

Yes this works fine thank you

Hi! Can you explain me, what I need to do to fix this issue?