Error when attempting to download a video from Vice

Whily trying to download a video from Vice (Using youtube-dl 2017.02.16) I get confronted with the following error:
 youtube-dl https://video.vice.com/nl/video/vice-special-too-young-to-die/58a31e56d3b56ab07bae4187
 [Vice] vice-special-too-young-to-die: Downloading webpage
ERROR: Unable to download webpage: <urlopen error [Errno 1] _ssl.c:510: error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure> (caused by URLError(SSLError(1, '_ssl.c:510: error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure'),))

Please paste the output of
youtube-dl -v https://video.vice.com/nl/video/vice-special-too-young-to-die/58a31e56d3b56ab07bae4187

youtube-dl -v https://video.vice.com/nl/video/vice-special-too-young-to-die/58a31e56d3b56ab07bae4187
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://video.vice.com/nl/video/vice-special-too-young-to-die/58a31e56d3b56ab07bae4187']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.02.16
[debug] Python version 2.7.6 - Linux-3.13.0-37-generic-x86_64-with-LinuxMint-17.1-rebecca
[debug] exe versions: avconv 9.20-6, avprobe 9.20-6
[debug] Proxy map: {}
[Vice] vice-special-too-young-to-die: Downloading webpage
ERROR: Unable to download webpage: <urlopen error [Errno 1] _ssl.c:510: error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure> (caused by URLError(SSLError(1, '_ssl.c:510: error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure'),))
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 416, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2006, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 404, in open
    response = self._open(req, data)
  File "/usr/lib/python2.7/urllib2.py", line 422, in _open
    '_open', req)
  File "/usr/lib/python2.7/urllib2.py", line 382, in _call_chain
    result = func(*args)
  File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 1060, in https_open
    req, **kwargs)
  File "/usr/lib/python2.7/urllib2.py", line 1184, in do_open
    raise URLError(err)

[debug] Python version 2.7.6 - Linux-3.13.0-37-generic-x86_64-with-LinuxMint-17.1-rebecca

Sorry, but Python 2.7.6 is too old. video.vice.com now requires a feature called SNI (Server Name Indication), which is available in > 2.7.9 and 3.x only. Please upgrade your Python.

I have python3.5 installed now. How can I tell youtube-dl to use that version?

python3.5 /usr/local/bin/youtube-dl


Scaling tool to improve LP matrix

This branch contains the files and MESSAGE model updates to allow the model to run with an improved LP matrix.
A tutorial is provided in message_ix/tools/make_scaler directory, but the text still needs some edits to improve clarity.
Exclusion of unrelated changes, e.g., recursive dynamics and learning commits, are also required before integration with IIASA's main branch.

How to review

[ ] Please check out this branch and try to work with this tool
[ ] Write comment to report any bugs or improvement suggestions

PR checklist

[ ] Continuous integration checks all ✅

[ ] Add or expand tests; coverage checks both ✅

[ ] Add, expand, or update documentation.

[ ] Update release notes.

Thanks for this PR, @ywpratama  :)
One thing to note right away: please keep the existing file structure intact. We want all tutorials to be available under tutorial/wherever they belong, so please add your tutorial there, too, instead of in some random location.


[ANCHOR-341] Fix missing type field of the PutCustomerRequest in SEP-24 post transactions

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Fix missing type field of the PutCustomerRequest in SEP-24 post transactions.
Please also refer to: https://github.com/stellar/stellar-docs/pull/165#discussion_r1231301534
Why
Bug fixes.

Reference Server Preview is available here:https://anchor-ref-pr944.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr944.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr944.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr944.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr944.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr944.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr944.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr944.previews.kube001.services.stellar-ops.com/


llvmPackages_10: 10.0.0 -> 10.0.1

Motivation for this change
RC1 is out, but it only has a single tarball, so we have to do the partitioning ourselves
http://lists.llvm.org/pipermail/release-testers/2020-May/001226.html
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@DieGoldeneEnte the downloading part seems to work, cmake still has trouble. See last commit message. Feel free to iterate on this or suggest a better way :-)

I would suggest waiting a little, maybe the tarballs will be added later (I think the source-code archives are auto-generated). Someone already asked for the usual tarballs (http://lists.llvm.org/pipermail/llvm-dev/2020-May/141684.html).
Also I think it is better to not split the source-archive. If you split it up you will get 2 copies in your nix-store and since there are a lot of dependencies between the different tools you still get a good amount of rebuilds (I think everything depends on llvm and it gets updated multiple times a day).
I would need to take a closer look to get an overview of the differences when building with the mono-repo.

Well, having such a monorepo-style thing would help tracking master, but that probably has a very limited audience.

@ggreif I agree, the mono-repo is the right way (also to track other llvm-repositories like the one from amd).
I think the best way wouldbe to just use the same source (the complete one) for all the tools. It might be useful to separate smaller components not depending on llvm, like libunwind, but for the large components like clang this is not as useful (for development use one could still give a separate source for the component in question).

 Updating...

@GrahamcOfBorg build llvm_10

@GrahamcOfBorg build llvm_10 clang_10 lldb_10 lld_10 llvmPackages_10

The only problem I see is #87528. So this is ready for review.

LGTM. It builds and I didn't have any problems with small test compilations.
I checked the llvm-patch. The canges are still only in master and not included in this version.

Result of nixpkgs-review pr 88248 1

14 packages failed to build:
- ameba
- crystal (crystal_0_34)
- crystal2nix
- crystal_0_31
- crystal_0_32
- crystal_0_33
- gitAndTools.thicket
- google-cloud-cpp
- icr
- lucky-cli
- mint
- oq
- scry
- shards

15 packages built:
- bat-extras.prettybat
- chromium
- clang-tools
- clang_10
- embree2
- ispc
- lld_10
- lldb_10
- llvm_10
- luxcorerender
- python37Packages.tiledb
- python38Packages.tiledb
- tiledb
- tinygo
- zig

@Mic92 Thanks for doing this! Do you get different results with 10.0.0? (It is okay to say "don't know" then I'll try it locally.

This is the result I get with nixpkgs-review pr 88248 I only have 3 fails:
4 packages failed to build:
google-cloud-cpp python37Packages.tiledb python38Packages.tiledb scry

25 packages built:
ameba bat-extras.prettybat chromium clang-tools clang_10 crystal crystal2nix crystal_0_31 crystal_0_32 crystal_0_33 embree2 gitAndTools.thicket icr ispc lld_10 lldb_10 llvm_10 lucky-cli luxcorerender mint oq shards tiledb tinygo zig

on a x86_64 system.
The tiledb-fails are because of tests, previously these had nothing to do wit the compiler.
google-cloud-cpp fails on master also (with clang 10.0.0). and seems to be unrelated to llvm.

This is the result I get with nixpkgs-review pr 88248 I only have 3 fails:
4 packages failed to build:
google-cloud-cpp python37Packages.tiledb python38Packages.tiledb scry

25 packages built:
ameba bat-extras.prettybat chromium clang-tools clang_10 crystal crystal2nix crystal_0_31 crystal_0_32 crystal_0_33 embree2 gitAndTools.thicket icr ispc lld_10 lldb_10 llvm_10 lucky-cli luxcorerender mint oq shards tiledb tinygo zig

on a x86_64 system.
The tiledb-fails are because of tests, previously these had nothing to do wit the compiler.
google-cloud-cpp fails on master also (with clang 10.0.0). and seems to be unrelated to llvm.

I doubt I would trust this result more. I don't have a nix-daemon running on this machine, so there might be subtle bugs.

TODO:
patching sources
applying patch /nix/store/kavpkf622k44kw6j7hs40b4zchx116g4-llvm-extension-handling.patch
patching file CMakeLists.txt
Reversed (or previously applied) patch detected!  Assume -R? [n]
Apply anyway? [n]
Skipping patch.
2 out of 2 hunks ignored -- saving rejects to file CMakeLists.txt.rej
patching file cmake/modules/AddLLVM.cmake
Hunk #1 FAILED at 884.
1 out of 1 hunk FAILED -- saving rejects to file cmake/modules/AddLLVM.cmake.rej
patching file cmake/modules/CMakeLists.txt
Reversed (or previously applied) patch detected!  Assume -R? [n]
Apply anyway? [n]
Skipping patch.
1 out of 1 hunk ignored -- saving rejects to file cmake/modules/CMakeLists.txt.rej

error: build of '/nix/store/919m86sc9rf5m5mspnvcsdk48il3ccjv-llvm-manpages-10.0.1rc2.drv', '/nix/store/qmw96fj70maf93k0hpqa2f1a40zzdz02-lldb-manpages-10.0.1rc2.drv' failed

@DieGoldeneEnte we still have the manages problem. Any ideas what they could be?

Sorry, somehow forgot to mention it here.
The problem is with recommonmark and sphinx3 not working nicely together (see #93117).

See 2a5899b (and parent) for how to revert to candidates.

FYI recommonmark is marked as broken for now (on master). As a result llvmPackages 8-10 can't be build completely (manpages for llvm fail). Older versions are not affected, because they don't use recommonmark.
I think we can remove (comment out) the llvm manpages for 8 until 10 (and the lldb for 10) since it can't be viewed with man anyways.
What do you think @ggreif @matthewbauer ?

@DieGoldeneEnte I am okay with that, since you can read the manages on the internet anyway. Do you want to add a commit?

I'll open an other PR so I can disable them all in one commit.
They still merge automatically :)


[Bug]: Select widget when opened adds extra space on the right

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When adding a select widget to the rightmost end of the page, opening the popover causes whitespace to be added and the rest of UI shifting to left. Expectation is that the popover will open to the left than right, similar to how datepicker works.

Steps To Reproduce

Add a select widget in the right most part of the app
Open the select widget
Observe the change in UI

Public Sample App
No response
Version
Production - 1.6.24

Assuming 20% users face this
Stats

Stat
Values

Reach
73

Effort (months)
0.5


Can't create a pull request

Hi!
I am just newbie and can't create a pull request.
Will you guide me in details.... please...
For this reason, I am posting the matter as a issue here. Sorry for this...
Actually if I want to get output info of all formats available along with the downloadable links, I may try with '-F -g' option, but this arguments returns nothing to output window (cmd).
I am feeling uneasy to write a GUI of youtube-dl. The application uses youtube-dl to get infos and the links. End-user should have the choice to use youtube-dl or other download manager. I need to call youtube-dl twice, first for getting information and another call for getting link of the requested format as it will be requested from end-user end.
To make the matter more simple to realize,  I want to say that I need a table like below:
Format          Code           Resolution      Note              Links
a...........            a........           a.................      a...........            a...........
b...........            b........           b.................      b...........           b...........
c...........              c........             c................        c...........             c...........       and more .......
Actually I need to get a column of links.
But there is no provision to get it now, as far I know...
At this point, I am forced to have a look in the code of youtube-dl, have found that some changes of code may make the youtube-dl able to give this type of output and it will not consume much more time to produce.....
On Youtube-dl.py  at the 'def list_formats(self, info_dict):'
If I modify the code as below...
def list_formats(self, info_dict):
    formats = info_dict.get('formats', [info_dict])
    #Heres to check that user wants to get links....
    if self.params.get('forceurl', False):
        table = [
            [f['format_id'], f['ext'], self.format_resolution(f), self._format_note(f), f['url']]
            for f in formats
            if f.get('preference') is None or f['preference'] >= -1020]

        header_line = ['format code', 'extension', 'resolution', 'note', 'link']
        if len(formats) > 1:
            table[-1][-3] += (' ' if table[-1][-3] else '') + '(best)'
       #Now need to skip of checking 'quiet' option and let it to be printed on output
        self.to_screen(
            '[info] Available formats for %s:\n%s' %
            (info_dict['id'], render_table(header_line, table)))
    else:
        table = [
            [f['format_id'], f['ext'], self.format_resolution(f), self._format_note(f)]
            for f in formats
            if f.get('preference') is None or f['preference'] >= -1020]

        header_line = ['format code', 'extension', 'resolution', 'note']
        if len(formats) > 1:
            table[-1][-2] += (' ' if table[-1][-2] else '') + '(best)'

        self.to_stdout(
            '[info] Available formats for %s:\n%s' %
            (info_dict['id'], render_table(header_line, table)))

It will give Youtube-dl more ability....
What do you think?
Lets have a cheers, I think.
One more thing:  English is not my mother-tongue and I am not much educated, just 10+2. So you may have some problems to understand my words. Sorry for this inconvenience.
Thank you.

There is a clean step by step instruction on how to make PRs.
-F and -g are not supposed to be used together in the first place.
To get arbitrary data you must consume the output of --dump-json. You should not change the format of tables.


feat: update to rust 2021 edition

Hey there and happy new year 🥂
Just glimpsed over this crate and saw that it was using rust 2018, used the core lib but with no std, any reason for this?

Thanks for the PR but until there's a need to update past 2018 I think it's fine to leave this on 2018 for now, there's not really a burning need to change things. Otherwise the various other bits and pieces removed here are unfortunately breaking changes, so this also wouldn't be able to land due to those changes.


error cloning repo in Windows

The error is due to invalid path Payloads/File-names/traversal/..::..:::;

Yup!
You may need to find a different way to download then ;)
Could try the zip out.

The files wouldn't be able to exist anyway on a Windows system, and therefor contributing would be more complicated than it has to on Windows. I think it'd be better if those files were compressed into a zip with a valid filename on this repo.

Windows workaround:
After cloning the repo you have it locally, but git reset --hard fails, because of invalid filenames. So we need to commit removal of these files. They are already staged for removal, which can be done with git commit -mremoval. But this would commit removal of all. So:

git status to see all the files staged for removal
issue git restore --staged xxx for each file/directory that you want to retain (you may also prepare a batch using output of previous command)
git reset --hard

I suggest to remove also Web-Shells, or the builtin defender will go crazy :)


Opt to enable linkerd proxy shutdown from lifecycle preStop

What this PR does / why we need it
Added a new linkerd object to values & schema. If enabled=true & shutdownAfterCheckpoint=true, then the STS pod's lifecycle.preStop hook will be configured to issue a shutdown call to the local linkerd-proxy.
This is entirely optional. If a deployment is not using linkerd, then this option should be left in the default disabled state.
Which issue this PR fixes
N/A
Special notes for your reviewer
N/A
Checklist

[x] Chart Version bumped
[x] CLA signed

Ok, quick heads-up that I've reverted this to draft from, as I need to do some additional experimentation here.

Maybe also do set -euo pipeline in the script just to be safe?

Closing this for now as we will be exploring a better way to handle these conditions outside of the chart.


Particle System Sandcastle ES6 Update

Fixes #10038
The goal of this pull request is to re-work the Particle System Weather sandcastle demo such that it is functional when the prefer-const ES6 rule is enabled.
My initial commits have made the following changes:

Removed the line disabling one of our global ES6 rules: /* eslint-disable prefer-const */
Updated where snowGravityScratch and rainGravityScratch are declared.
Removed the snowSystem and rainSystem variables.
In the original implementation, both the snowSystem and the rainSystem objects were added to the scene as primitives. Their visibility was toggled using the show. I suspect that this may have caused performance issues. I added a commit that ensures that only the necessary primitives are kept in the scene.

I am now able to run the Particle System Weather demo locally without the line:
/* eslint-disable prefer-const */
It has been mentioned various times that the current code is not well structured. As seen above, I made various edits. The biggest thing that stood out to me was how primitives were handled. However, if there is anything specific that still needs to be reworked, I would be happy to take a look at it.

Thanks for the pull request @srothst1!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.

Hi @ggetz! Thank you very much for the feedback. I just added various commits and comments that address your suggestions.

Awesome, thanks @srothst1 !


Add missing sample metadata file

Description of the bug
In: https://github.com/genomic-medicine-sweden/gmsmetapost/blob/5dc8bd61abe4233342b08f228acd66c919e23da7/nextflow.config#L55
This should be pointing to: "$baseDir/assets/sample_results.csv" instead.
Command used and terminal output
No response
Relevant files
No response
System information
No response

This was resolved by https://github.com/genomic-medicine-sweden/gmsmetapost/pull/32
so I'm closing this issue as resolved.


Add TouchableScale Component

Hi, This is my first 'real' PR so I don't expect much. Any feedback would be amazing though.
Test Plan:
I wasn't sure if you tested individual components or not. So I did not write a test for this new component.
Release Notes:
[GENERAL] [FEATURE] [/Libraries/Components/Touchable/TouchableScale.js] - Created A new Touchable Component

Those failing CI tests are not related to my PR.

I thought about but didn't actually bother to do it after I have forked the repo.


Super Staff Bros.: Fix Taunt

Certain moves, such as Trolling Lobby, do not get properly disabled by Taunt (instead allowing the move to be "used," but displaying a message that the user can't use the base move due to the Taunt). This fixes that issue.

I was doing a rewrite for this but this will do for now.


clippy and some chores

mozilla#718

The update does a merge commit of master, not a rebase.. sigh


Make removing comments from config files work reliably

What this PR does
Reliably remove comments from Rails config files when generating Suspenders apps. With the parser gem, we're now removing comments by processing their surrounding code using AST (abstract syntax tree) processors.
We were using a regex to do that, which is text-based and thereby subject to edge cases that only a proper AST solution can handle.
First motivation
This pull request made changes to some config files to fix the Action Mailer asset host.
While working on merging that PR, the tests were not passing because they looked for an output that was missing in one of the changed config files:
config.action_mailer.asset_host = %q{"https://#{ENV\.fetch\("ASSET_HOST", ENV\.fetch\("APPLICATION_HOST"\)\)}}

The PR code changes are kind of straightforward and that config option should have been present in the config file but it was not.
The reason was that Suspenders was removing code comments with an overly permissive regex : (/^.*#.*$/), which removed every line containing a # character, no matter where they were in the line - so it was also removing the new Action Mailer config setting.
That regex was also silently messing up with other config lines, for example:
config.public_file_server.headers = {
  'Cache-Control' => "public, max-age=#{2.days.to_i}"
}

As a quick fix to merge that PR, I've changed the regex to /^\s*#.*$/ but it was not an ideal solution - which is what I'm striving for with this PR. With the old text-based regex solution, there were still nasty edge cases to deal with, and we may never know when they'd occur. Also, the previous regex did not work with inline comments.
Second motivation
The final aesthetics of the config files could be improved in my opinion. This was how they looked like:
Rails.application.configure do
  config.cache_classes = false
  config.eager_load = false
  config.consider_all_requests_local = true
  if Rails.root.join('tmp', 'caching-dev.txt').exist?
    config.action_controller.perform_caching = true
    config.action_controller.enable_fragment_cache_logging = true
    config.cache_store = :memory_store
    config.public_file_server.headers = {
      'Cache-Control' => "public, max-age=#{2.days.to_i}"
    }
  else
    config.action_controller.perform_caching = false
    config.cache_store = :null_store
  end
  config.active_storage.service = :local
  config.action_mailer.raise_delivery_errors = true
  config.after_initialize do
    Bullet.enable = true
    Bullet.bullet_logger = true
    Bullet.rails_logger = true
  end

  # ...
end

And with the newest solution they look like this:
Rails.application.configure do
  config.cache_classes = false

  config.eager_load = false

  config.consider_all_requests_local = true

  if Rails.root.join('tmp', 'caching-dev.txt').exist?
    config.action_controller.perform_caching = true
    config.action_controller.enable_fragment_cache_logging = true

    config.cache_store = :memory_store
    config.public_file_server.headers = {
      'Cache-Control' => "public, max-age=#{2.days.to_i}"
    }
  else
    config.action_controller.perform_caching = false

    config.cache_store = :null_store
  end

  config.action_mailer.raise_delivery_errors = true

  config.after_initialize do
    Bullet.enable = true
    Bullet.bullet_logger = true
    Bullet.rails_logger = true
  end

  # ...

Which preserves newlines where appropriate, so that's easier to read.
Third motivation
This pull request uses the parser gem, and I feel like introducing that gem in the project may be a good move over the long term.
For example, we need to finish up the gem action to sort gem declarations alphabetically when adding gems to the Gemfile, and the most reliable way to do that is with AST parsing.

+1 to using parser. Great idea. Love it.
What do we think of leaving the comments there instead?

@mike-burns In my opinion, removing comments is a nice feature so I would vote for keeping it, as most default comments are noise. Another option is to have a dedicated generator where the user would explicitly opt-in for this feature.
Personally, I would maybe extract this functionality to a gem in the future while making the necessary changes for it to be as generic as possible.

@mike-burns We were already chasing Ruby versions, so that should not be an issue :cry:
I wrote up the following ticket to deal with this Ruby version problem: https://github.com/thoughtbot/suspenders/issues/1073
Hopefully, it will address the concerns of "chaining Ruby versions".


Wrong version in warning banner

The version compared and displayed in the warning banner is not the "version_match" attribute, but the theme version, as a result of #1394:
https://github.com/pydata/pydata-sphinx-theme/blob/4a29125149102d06e397dae711af148e8350a6eb/src/pydata_sphinx_theme/assets/scripts/pydata-sphinx-theme.js#L432
This result in the following image (note that the version of the banner is not the package version):

It seems something went wrong with the fix: https://pydata-sphinx-theme.readthedocs.io/en/stable/index.html
I made a realease yesterday and the stable version is referenced as "unknown version" now

That is actually a different problem related to #1416.

This seems to be an issue at https://pydata-sphinx-theme.readthedocs.io/en/stable/ as well?

This seems to be an issue at https://pydata-sphinx-theme.readthedocs.io/en/stable/ as well?

How so?  The dropdown in the stable docs says "choose version" not "unknown version" nor an incorrect version string.  "Choose version" is the new fallback value in case the version match cannot be determined; again, that is a different problem than the OP reported and is tracked in #1416

https://pydata-sphinx-theme.readthedocs.io/en/stable/index.html has a banner saying this is an unknown version and switch to stable, which lands you on the same page.

ah!  my apologies, I was focused on the dropdown switcher and didn't even notice the banner.  Nonetheless I believe that the cause is still the problem described in #1416 --- it's difficult to debug as I haven't been able to replicate it on PR builds or local builds.

I'm using Python 3.12, PyData Sphinx Theme 0.15.1, and Sphinx 7.2.6. I'm able to replicate this issue.

From what I can tell, the value unknown version is getting inserted because the generated file documentation_options.js has an empty string for VERSION:
const DOCUMENTATION_OPTIONS = {
    VERSION: '',
    LANGUAGE: 'en',
    COLLAPSE_INDEX: false,
    BUILDER: 'html',
    FILE_SUFFIX: '.html',
    LINK_SUFFIX: '.html',
    HAS_SOURCE: true,
    SOURCELINK_SUFFIX: '.txt',
    NAVIGATION_WITH_KEYS: false,
    SHOW_SEARCH_SUMMARY: true,
    ENABLE_SEARCH_SHORTCUTS: true,
};

Manually changing the VERSION to '1.0.0' and refreshing the page provides the expected message banner:

As to why VERSION was empty, I tracked this back to conf.py. I set the Sphinx variable version but not release. VERSION in documentation_options.js gets set to the release value and not version.

As to why VERSION was empty, I tracked this back to conf.py. I set the Sphinx variable version but not release. VERSION in documentation_options.js gets set to the release value and not version.

Thanks for the report @xia-stan. Indeed we use the release value and not the version value from conf.py. We are assuming folks follow what is stated in the Sphinx docs for release:

If you don’t need the separation provided between version and release, just set them both to the same value.

I think this issue can be closed now, on the assumption that if release is properly set, the warning banner will work as intented.  But the fact that folks keep stumbling around the version switcher and the warning banner suggests that either (1) what we're doing is too fragile/magic, or (2) we're not documenting it clearly enough.  I'll open a new issue to focus our attention on those improvements.


@available doesn’t affect literal initialization

Describe the bug
When a literal initializer is marked with an availability annotation, it's seemingly ignored by the compiler at the point of use, unless the initializer is used explicitly.
Steps To Reproduce
struct Foo {
    let bar: String
}

@available(*, deprecated)
extension Foo: ExpressibleByStringLiteral {
    init(stringLiteral value: StringLiteralType) {
        self.bar = value
    }
}

let a: Foo = Foo(stringLiteral: "") // warning: 'init(stringLiteral:)' is deprecated
let b: Foo = "" // no warning

Expected behavior
Both assignments should cause a deprecation warning.
Note: every other availability status (e.g. unavailable) is also ignored.
Screenshots

What I understand is that you deprecated stringLiteral the way which initializes the bar.
But, why neither "a" nor "b" gives a warning and only "a" gives a warning?
If this is correct understanding, you need to read the documentation.
In the swift/Literals.md, you can find below.
The ExpressibleByStringLiteral Protocol
/// A type that can be initialized with a string literal.
///
/// The String and StaticString types conform to the
/// ExpressibleByStringLiteral protocol. You can initialize a variable or
/// constant of either of these types using a string literal of any length.
///
///     let picnicGuest = "Deserving porcupine"
///
So, Even if you deprecated it, It can be initialized with a string literal like "", "a", "b" and so on.
I hope it will help you.

Reference)
https://github.com/apple/swift/blob/main/docs/Literals.md

Thank you for such a quick turnaround @LucianoPAlmeida! 👏


Stone Tower Temple Adjustments

This updates the exception for Inverted Stone Tower's sceneId getting converted (hopefully to allow upside down chests to work). Looking at Play_SaveCycleSceneFlags(), only the collectible and clearedRoom flags are saved to Inverted, while the rest are saved to Normal Stone Tower Temple. I haven't retested every check, but the ones I checked in both scenes are still working.
Additionally, I made some changes to pots that I think resolve the issues I was seeing in STT. At some point we should probably go through and double check all of the pots in other scenes that convert with Play_GetOriginalSceneId() to ensure there aren't any other issues.

Build Artifacts

2ship-mac.zip
2ship-linux.zip
2ship-windows.zip

I just noticed I messed up my conditional. I'll fix it tomorrow

Actually, I think I fixed it. Won't be able to test again until tomorrow though.

This should be good to review now. I really need to stop opening PRs right before going to bed :Shrek:


Remove unused variables

buf and first is never used so I removed it. Additionally I added proper vars for regex variables.

Good catch!


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/eventing 536f2b9...8dd3598:
  > 8dd3598 Retry on any webhook EOF error (# 5895)
  > 1e9f9f1 Implement CE Subscriptions filters (# 5715)
  > 62219cc upgrade to latest dependencies (# 5894)
bumping knative.dev/eventing-kafka e696b90...e491a58:
  > e491a58 upgrade to latest dependencies (# 984)
  > 74f24de upgrade to latest dependencies (# 983)
  > 872a0ef fix vulnerable dependency packages  (# 981)
  > b6a55c2 Update actions (# 982)
bumping knative.dev/reconciler-test c5f938a...ae9e2e2:
  > ae9e2e2 isReady verifies resource generation and observed generation (# 259)
  > edafe72 Retry on webhook EOF errors (# 260)

/lgtm


No module.json update for 4.1.0

No module.json update for 4.1.0, this is breaking some detection of this new version (in particular on The Forge).

Thanks much!

Thanks much!


Probabilities contain NaN while using my nifti input

Hi, thank you so much for sharing your work and helping us out!
I have been able to run your model successfully on the initial set of images. However, I am not able to use my .nii data successfully. I get the following error:
*********************  STARTING NETWORK TRAINING *******************
*********************  STARTING TRAINING ************************
*************  Starting training model (Reading parameters) **************
--- Training model (Reading parameters...)
--- Do training in 3 epochs with 2 subEpochs each...
-------- Reading Images names used in training/validation -------------

================== Images for training ================
Image(0): MR_Img1.nii  |  GT: GT_Img1.nii
Image(1): MR_Img2.nii  |  GT: GT_Img2.nii
Image(2): MR_Img3.nii  |  GT: GT_Img3.nii
Image(3): MR_Img4.nii  |  GT: GT_Img4.nii
Image(4): MR_Img5.nii  |  GT: GT_Img5.nii

================== Images for validation ================
Image(0): MR_Img6.nii  |  GT: GT_Img6.nii

... Loading model from outputFiles/LiviaNet_Test/Networks/liviaTest_Epoch15
... Network architecture successfully loaded....

============== EPOCH: 16/3 =================
--- SubEPOCH: 1/2
... Get samples for subEpoch...
('len(randIdx) is ', 5)
... getting 200 samples per subject...
...Processing subject: 1. 20.0 % of the whole training set...
...Processing subject: 2. 40.0 % of the whole training set...
...Processing subject: 3. 60.0 % of the whole training set...
...Processing subject: 4. 80.0 % of the whole training set...

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
/mnt/d/JupyterNotebooks/LiviaNET/src/networkTrainingLarynx.py in <module>()
    80
    81 if __name__ == '__main__':
---> 82    networkTraining(sys.argv[1:])

/mnt/d/JupyterNotebooks/LiviaNET/src/networkTrainingLarynx.py in networkTraining(argv)
    75     # Training the network in model name
    76     print " ******************************************  STARTING NETWORK TRAINING ******************************************"
---> 77     startTraining(networkModelName,configIniName)
    78     print " ******************************************  DONE  ******************************************"
    79

/mnt/d/JupyterNotebooks/LiviaNET/src/LiviaNet/startTraining.pyc in startTraining(networkModelName, configIniName)
   137                                                 sampleSize_Train,
   138                                                 receptiveField,
--> 139                                                 applyPadding
   140                                                 )
   141

/mnt/d/JupyterNotebooks/LiviaNET/src/LiviaNet/Modules/IO/sampling.pyc in getSamplesSubepoch(numSamples, imageNames, groundTruthNames, roiNames, imageType, sampleSizes, receptiveField, applyPadding)
    87                                                      samplingDistribution,
    88                                                      sampleSizes,
---> 89                                                      receptiveField
    90                                                      )
    91

/mnt/d/JupyterNotebooks/LiviaNET/src/LiviaNet/Modules/IO/sampling.pyc in getSamplesSubject(imageIdx, imgSubject, gtLabelsImage, roiMask, samplingDistribution, sampleSizes, receptiveField)
   165                                                 size = numOfSamplesToGet,
   166                                                 replace=True,
--> 167                                                 p=maskCoordsFlattened)
   168
   169         centralVoxelsCoord = np.asarray(np.unravel_index(centralVoxelsIndexes, maskCoords.shape))

mtrand.pyx in mtrand.RandomState.choice()

ValueError: probabilities contain NaN

Dataset
|--MR     (7 CT nii images MR_Img1 to MR_Img7)
|--Labels (7 CT nii images GT_Img1 to GT_Img7)
|-ROI       (Empty)
I have nothing in the ROI folder.
My configuration file is :

############################################################################################################################################
#################################################        CREATION OF THE NETWORK       #####################################################
############################################################################################################################################

############## =================== General Options  =================  ################
[General]
networkName = liviaTest
# Saving Options
folderName = LiviaNet_Test

############## =================== CNN_Architecture  =================  ################
[CNN_Architecture]
numkernelsperlayer = [10,20,30,100]

# Kernels shapes:  (Note, if kernel size is equal to 1 on one layer means that this layer is fully connected)
# In this example there will be 3 conv layers and 1 fully connected layer (+ classification layer)
kernelshapes = [[3, 3, 3], [3, 3, 3], [3, 3, 3], [1]]

# Intermediate layers to connect to the last conv layer (just before the first fully connected layer)
intermediateConnectedLayers = []

# In the current implementation it does not support pooling (To be done...)
pooling_scales = [[1,1,1],[1,1,1],[1,1,1]]

# Array size should be equal to number of fully connected (FC) layers + classification layer
dropout_Rates = [0.25,0.5]

# Non-linear activations
# Type: 0: Linear
#       1: ReLU
#       2: PReLU
#       3: LeakyReLU
activationType = 2

# TODO. Include activation type for Softmax layer
# Number of classes: background + classes to segment
n_classes = 2

# ------- Weights initialization ----------- #
# There are some ways to initialize the weights. This is defined by the variable "weight_Initialization"
# Here, there is a list of supported methods
# 0, Classic
# 1: Delving (He, Kaiming, et al. "Delving deep into rectifiers: Surpassing human-level performance on imagenet classification." ICCV'15)
# 2: Load Pre-trained
#      ----------
# There is also the choice of which layers will be initialized with pre-trained weights. This is specified in the variable
# "load weight layers". This can be either empty (i.e. all layers will be initialized with pre-trained weights in case
# "weight_Initialization" is 1)
weight_Initialization_CNN = 1 #was 1
weight_Initialization_FCN = 1
#load weight layers = [] # Next release
# If using pre-trained models, specify the folder that contains the weights and the indexes of those weights to use
# To ease the transfer between different softwares (i.e matlab for instance), and between different architectures,
# the weights for each layer should be stored as a single file.
# Right now weights have to be in .npy format
weights folderName = /mnt/d/JupyterNotebooks/LiviaNET/trainedWeights
# Same length as conv layers
weights trained indexes = [0,1,2]
#weight_Initialization_Sec = 1

############## =================== Training Options  =================  ################
[Training Parameters]
#n_epochs=20
batch_size=10
number Of Epochs = 3
number Of SubEpochs = 2
number of samples at each SubEpoch Train = 1000
# TODO. To define some changes in the learning rate
learning Rate change Type = 0
# Subvolumes (i.e. samples) sizes.
# Validation equal to testing samples
sampleSize_Train = [25,25,25]
sampleSize_Test  = [45,45,45]

# Cost function values
# 0:
# 1:
costFunction = 0
SoftMax temperature = 1.0
#### ========= Learning rate ========== #####
L1 Regularization Constant = 1e-6
L2 Regularization Constant = 1e-4

# TO check
# The array size has to be equal to the total number of layers (i.e. CNNs + FCs + Classification layer)
#Leraning Rate = [0.0001, 0.0001, 0.0001, 0.0001,0.0001, 0.0001, 0.0001, 0.0001,0.0001, 0.0001, 0.0001, 0.0001,0.0001, 0.0001 ]
Leraning Rate = [0.001]
# First epoch to change learning rate
First Epoch Change LR = 1
# Each how many epochs change learning rate
Frequency Change LR = 2
# TODO. Add learning rate for each layer

#### ========= Momentum ========== #####
# Type of momentum
# 0: Classic
# 1: Nesterov
Momentum Type = 1
Momentum Value = 0.6
# Use momentum normalized?
momentumNormalized = 1

#### ======== Optimizer ===== ######
# Type: 0-> SGD
#       1-> RMSProp (TODO. Check why RMSProp complains....)
Optimizer Type = 1

#In case we chose RMSProp
Rho RMSProp = 0.9
Epsilon RMSProp = 1e-4

# Apply Batch normalization
# 0: False, 1: True
applyBatchNormalization = 1
BatchNormEpochs = 20

# Apply padding to images
# 0: False, 1: True
applyPadding = 1

############################################################################################################################################
#################################################            TRAINING VALUES           #####################################################
############################################################################################################################################

[Training Images]
#here
imagesFolder = /mnt/d/JupyterNotebooks/LiviaNET/Dataset2/MR/
GroundTruthFolder = /mnt/d/JupyterNotebooks/LiviaNET/Dataset2/Label/
# ROI folder will contain the ROI where to extract the pacthes and where to perform the segmentation.
# Values of the ROI should be 0 (non interest) and 1 (region of interest)
#ROIFolder = /mnt/d/JupyterNotebooks/LiviaNET2/Dataset/ROI/
# If you have no ROIs
ROIFolder = []
# Type of images in the dataset
#     0: nifti format
#     1: matlab format
# IMPORTANT: All the volumes should have been saved as 'vol'
imageTypes = 0

# Indexes for training/validation images. Note that indexes should correspond to the position = inex + 1 in the folder,
# since python starts indexing at 0
indexesForTraining = [0,1,2,3,4]
indexesForValidation = [5]

Can you please help me out with what is happening and how I could go about fixing this? Thanks.

Hi @divinediu
We have found that not having ROI images causes NaN values during the training due to the sampling strategy. Basically, if there is no ROI we sample from all the image, which may produce the NaN values for empty (black) sub patches. There are some easy fixes, but we didn't have time to modify it. A simple solution is to generate a ROI by yourself and include it in the training folder. A simple thresholding should be enough for this purpose.
Jose

Thanks a lot for your quick response, I am not sure how to generate a quick ROI. Could you refer me to something to look up, please?

I included also some code for that
https://github.com/josedolz/LiviaNET/blob/7a53f6f67c88cc6a9f79636834771bec9df3acf8/src/generateROI.py#L1

I cannot thank you enough! I will try this out! :)


Update AGP to 8.1.1

What does this PR do?
This PR updates AGP to version 8.1.1
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1656 (b163bd6) into release/2.2.0 (d3a3f83) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@                Coverage Diff                @@
##           release/2.2.0    #1656      +/-   ##
=================================================
+ Coverage          83.57%   83.67%   +0.10%
=================================================
  Files                458      458
  Lines              15765    15764       -1
  Branches            2353     2353
=================================================
+ Hits               13175    13190      +15
+ Misses              1962     1950      -12
+ Partials             628      624       -4

Files
Coverage Δ

...droid/core/internal/receiver/ThreadSafeReceiver.kt
100.00% <100.00%> (ø)

... and 13 files with indirect coverage changes


Translate readme.md in hindi

What would you like to be added
Translated the readme.md file to the Hindi language.
Why is this needed
Extending the community of Kubernestes by adding support for the native language of India.

/language hi


add Github Enterprise support

This PR adds basic support for Github Enterprise instances.
I briefly tested with the instance I have and the 'PR' and 'Issues" functions work as expected.
Gists do not, due to underlying bug in GH CLI, there is a relatively easy workaround available.
In fact, setting GH_HOST environment variable kinda renders this pull request useless because it makes the PR/issue commands work out of the box too so I would like to get your input/preferences on this.

Thanks for the contribution. I dont have a strong opinion whether we should rely on using GH_HOST externally or handling it internally. I think it does not hurt to handle it within Octo so Im ok merging this PR in.
However, this reminded me that there are a couple of places where we should account for the GHE hostname:

util.lua:508. Parsing URLs to extract repo name and issue number

function M.parse_url(url)
  local repo, kind, number = string.match(url, "https://github.com/([^/]+/[^/]+)/([^/]+)/(%d+)")
  if repo and number and kind == "issues" then
    return repo, number, "issue"
  elseif repo and number and kind == "pull" then
    return repo, number, kind
  end
end

After refactoring some code, the only URL handling code should be:
[constants.lua]:
M.URL_ISSUE_PATTERN = "[htps]+://.*github.com/([^/]+/[^/]+)/([pulisue]+)/(%d+)"

Thanks for that, I missed the parse_url stuff since all of my repos use SSH. Anyways, I've rebased on top of current master branch and updated URL_ISSUE_PATTERN to include hostname setting as the domain is usually in format of github.company.net and similar.

Thanks!


Add qBittorrent

Microsoft Reviewers: Open in CodeFlow

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


missing properties in Devel graph

There are properties in the Demo graph like kgo:averageLifespan and kgo:prefComparisons that are missing from the Devel graph. Please restore these to the Devel graph as soon as possible.

I do believe I just restored these properties for the taxa that have them.

Yes, this is fixed now. Thanks.


A question about training

Hi,
I have followed the 'Racoon' retraining example.
I can not find the code that control which layers are trainable and which are not (or there is no such code)?
Wouldn't you want to gradually open layers for training (like first epochs allowing only the last layers to change weights, then more, maybe up to the features finding layers, later maybe even before them)?
Anyway, I learned a lot. Very efficient implementation.
Thanks!

It's in the common.py and backbone.py.


Issue122 expertspage ux

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #122
Test URLs:

Before: https://main--24life--hlxsites.hlx.page/experts
After: https://issue122-expertspage-ux--24life--hlxsites.hlx.live/experts

This can be cancelled as the experts page is ready with the new block - author-list with 100% LHS.


File "main.py", line 1, in      from customtkintertitlebar import Tk ModuleNotFoundError: No module named 'customtkintertitlebar'

Not able to use this package at all :/

Issue solved... I installed the wrong version of the module


Signup, Login, Logout and Forgot password

Signup, Login, Logout and Forgot password.

PS: Também adicionei novos requisitos aos requirements.txt


Added option to smooth out shadows

Dark shadows look terrible when converted, so I've added an option to fix that. Changing the value of smoothshadows multiplies the lightmap brightness by that number, and makes every lightmapped texture darker by the same amount to compensate for the now brighter shadows.
Before

After

I wonder if there is an automatic way to properly convert the brightness. Maybe check for compile flags in the entity lump to see if there is something that modifies lightmap brightness/smoothness?

Looking a little more at the problem I think the faces are just too bright by default. mat_fullbright 1 is so much brighter than it should be on world brushes. The darker func_brush faces is how they are supposed to look I'm pretty sure, since those look normal with fullbright on. Hold off on this for now in case a simpler fix is possible in the face lump data.


Request ID

This is a possible approach to #2146 but I feel kinda queasy using an environment variable instead of an API but maybe that's just me

Would this be needed once we have wasi-observe and otel?

@karthik2804 on that I was guided by https://github.com/fermyon/spin/issues/2146#issuecomment-2000217985 but for sure I do not know enough either way

Could this (also?) be a Spin variable?

I think this may in fact be best served via wasi-observe. Distributed traces are typically keyed off of a trace ID which is either passed in via e.g. W3C Trace Context or generated by the server (if it is the first hop in the traced system). That trace ID could be emitted with logs and - if necessary - exposed to the guest via a wasi-observe import.


Replace icons set with branded business outlined icons

We are looking for an icon set, where we could change icons from current font awesome to new where they offer outlined icons for brands like twitter, medium, discord.

[ ] check if their licence allows using at OSS
[ ] check if they allow enough wide set of icons for the future use
[ ] their bundle is lightweight

Other library proposal

Feather icons ~286 icons
Hero icons ~230 icons
Ionicons ~1300 icons

All of them are MIT licensed.

I'm testing in a particular order, twitter, medium, discord, telegram (opt send)

Other library proposal

Feather icons ~286 icons

missing discord

Hero icons ~230 icons

no twitter

Ionicons ~1300 icons

don't have outlined brand icons :|

All of them are MIT licensed.

I think either we figure out something half-through solution or wait to keep searching for proper set 👀

I see [image: image.png] might be good!
On Thu, Oct 28, 2021 at 5:13 PM roiLeo @.***> wrote:

What about Boxicons https://github.com/atisawd/boxicons
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kodadot/nft-gallery/issues/796#issuecomment-953943308,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABM5POJJWNNSN5XPQGQQDR3UJFZA3ANCNFSM5EXS22AQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I've just hijacked Viki's notebook and found that FortAwesome will be implementing Vue3 with FA6 which probably is off table for now.
Found that https://www.npmjs.com/package/vue-material-design-icons might work just need to tackle which icons use for what.
Let's prepare PR and I can ping point icons where which use-- @yangwao
Found that Medium is missing
Discord is there,
right top arrow is there as well mdiArrowTopRight
Let's give it a shot?

Once we will have ability to use Vue3 #2692 we can check this one.

when vue3?

when vue3?

Recommend to join Discord

when vue3?

if you can help us with migration, that would be superior! :)

From #4788,

We might want to integrate iconify (or icones.js) for that purposes this will allow us to have easy access to multiple icon pack.

@yangwao Icones.js / Iconify offers this

let's go with Iconify

https://iconify.design/getting-started/
https://github.com/nuxt-modules/icon

Currently fontawesome kit was down

Sure cc @kodadot/internal-dev

any changes for me? are switching to a different icon set?

@preschian can we close this?

This one is not done yet


Hyphens and underscores in hostname

Are they allowed? I used --hostname host_0_1 and it showed as host01 on boot. Trying hyphens now.
Edit: Or is that just the output of the prompt setting?

I confirmed the underscores were preserved, so I guess the prompt does not display them.

Double checked and I guess underscores are not allowed, but hyphens are.

RFC 952 specifies legal characters for hostnames:
A "name" (Net, Host, Gateway, or Domain name) is a text string up
   to 24 characters drawn from the alphabet (A-Z), digits (0-9), minus
   sign (-), and period (.).  Note that periods are only allowed when
   they serve to delimit components of "domain style names".

Yep. Oddly, the first hit on google said underscores are allowed. Don't
believe everything (or the first thing) you read on the internet!
On Tue, Jul 9, 2024 at 6:51 PM Benn @.***> wrote:

RFC 952 https://www.rfc-editor.org/rfc/rfc952 specifies legal
characters for hostnames:
A "name" (Net, Host, Gateway, or Domain name) is a text string up
to 24 characters drawn from the alphabet (A-Z), digits (0-9), minus
sign (-), and period (.).  Note that periods are only allowed when
they serve to delimit components of "domain style names".
—
Reply to this email directly, view it on GitHub
https://github.com/gitbls/sdm/issues/233#issuecomment-2219039059, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEQXSN77F7VDPUJQO5JLGTZLRZPHAVCNFSM6AAAAABKTWRY3GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJZGAZTSMBVHE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

--
Timothy H. Keitt
www keittlab org


Crontab format not running

Hey guys,
I don't know if I'm missing something here but this config is not running at the desired time:
config :quantum, cron: [
 "30 11 * * *  MyModule.SendSms.perform"
]

It should run at 11:30 AM right? I tried several other combinations and waited to see if it triggers locally but the only one I see working was each 1 min * * * * *
quantum: 1.8.0
timex: 3.1.5
I'd appreciate any help,
Cheers

I have tested here using a similar node.js lib
It runs at 14:15 PM:
const schedule = require('node-schedule')

schedule.scheduleJob('15 14 * * *', () =>
  console.log('----------'))```

I used the same expression on quantum to run at 14:20 PM:
config :quantum, cron: [
 "20 14 * * *": fn -> IO.puts("------") end
],
timezone: :local

But the quantum version is not working as well

@rafaeljesus two questions:

Which time zone are you using? Quantum uses UTC by default (yes, I have seen that you are setting timezone to :local in your second comment :)).
Have you started quantum by adding it to the list of applications?

Thanks,
Constantin

Hi @c-rack thank you for your quick reply!
I am using my default mac timezone which is Brasilia Summer Time (Brazil). As you can see I tested with :local and the default one :utc
Yes I added quantum on application list, also tried adding as the first element of the list and as the last just to check if it would be conflicting with some other lib.
I just tested here again on other mac and still not working, I'll create a simple mix project and test quantum as a unique dependency

It seems that I can not pass a string to timezone: key
# should run at 10:30 AM
config :quantum,
  cron: [
    "30 10 * * *": fn -> IO.puts("Hey it works!!") end
  ],
  timezone: "America/Sao_Paulo"

It throws ** (RuntimeError) Unsupported timezone: America/Sao_Paulo so it is using the UTC timezone even if I pass :local to it,
So to work out, I must to take the difference out using UTC:
# It runs at 10:30 AM but I must to +2
config :quantum,
  cron: [
    "30 12 * * *": fn -> IO.puts("Hey it works!!") end
  ]

@rafaeljesus have you tried latest master branch instead of hex.pm published version? it should support passing the timezone as America/Sao_Paulo. If that works for you, I could publish a new version to hex.pm over the weekend.

@renews Thanks! I'll publish a new version shortly.
@rafaeljesus Have you tried master branch as well?

Sorry mates, I'm going to test right now

@c-rack Thank you for working on this on the weekend mate, it works now 🎉

As version 1.8.1 is released to hex.pm, I close this issue.


OCPBUGS-43313: Delete CAPI Cluster CR on HC deletion early

Currently the CAPI Cluster is deleted by deleting the namespace and we block HC deletion on the HCP resource. We should signal CAPI Cluster resource for deletion early so child objects make decisions based on this. E.g. exlude drain and wait for volumes. This would also prevent edge chicken-egg scenarios

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/retest

/jira refresh

We are deleting the CAPI cluster much earlier than that already
https://github.com/openshift/hypershift/blob/960357ff58ecf72c3c0c7f43a7540ac1c4acda7d/hypershift-operator/controllers/hostedcluster/hostedcluster_controller.go#L3835-L3840


2 pages missing when decoding a 4 page pdf

not sure where to start digging through this but any help would be great on where I should start digging.

@euske - i was wondering if you could shed light on what
flush()

it's used here:
if context:
    continue
else:
    self.flush()

at psparser.py, line 605. Flush basically just does a return.

@euske - i was wondering if the trailer dictionary is located in multiple locations in the pdf - have you looked into what happens in that case?


feat: Cortex CLI

Problem
Users need a way to interface with model management (nitro) from command line, i.e. jan download XXX
Customer request example (anonymized for their privacy):

Success Criteria
A clear and concise description of what you want to happen.

NPM Client library that makes model and inference calls (through server)
A CLI that mirrors client library capabilities (there are packages that can bundle the two)

Sub Issues

Map/design a CLI interface to our current server/implementations, i.e. jan CMD PARAM
Wrap model ext
Wrap inference ext
(optional) Wrap monitoring ext

Additional context
Add any other context or screenshots about the epic request here.

Closing this issue as done. Thank you team!


Fix: Build on 32-bit arch would raise int overflow

Description
Fixes https://github.com/Consensys/gnark/issues/1192
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How has this been tested?

[x] Test on Ubuntu 22.04

GOOS=linux GOARCH=arm go build
GOOS=linux GOARCH=386 go build

[x] Test on MacBook M1 Pro.

Before this PR and after this PR, go build always success.
This PR would fix gomobile Android binding https://github.com/Consensys/gnark/issues/1192#issuecomment-2221872640

Btw, some unit tests would also raise int overflow under 32-bit Arch
➜  gnark git:(master) GOOS=linux GOARCH=arm /usr/local/go/bin/go test
# github.com/consensys/gnark/internal/backend/circuits [github.com/consensys/gnark.test]
internal/backend/circuits/div.go:17:24: cannot use<PHONE_NUMBER> (untyped int constant) as int value in argument to api.DivUnchecked (overflows)
internal/backend/circuits/inv.go:17:19: cannot use<PHONE_NUMBER> (untyped int constant) as int value in argument to api.Inverse (overflows)
internal/backend/circuits/mul.go:30:11: cannot use<PHONE_NUMBER>4 (untyped int constant) as int value in assignment (overflows)
internal/backend/circuits/sub.go:14:15: cannot use<PHONE_NUMBER>3 (untyped int constant) as int value in argument to api.Sub (overflows)
internal/backend/circuits/sub.go:31:11: cannot use<PHONE_NUMBER>3 (untyped int constant) as int value in assignment (overflows)
internal/backend/circuits/sub.go:32:13: cannot use<PHONE_NUMBER>5 (untyped int constant) as int value in assignment (overflows)
internal/backend/circuits/sub.go:38:10: cannot use<PHONE_NUMBER>3 (untyped int constant) as int value in assignment (overflows)
FAIL    github.com/consensys/gnark [build failed]

How has this been benchmarked?
No
Checklist:

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I did not modify files generated from templates
[ ] golangci-lint does not output errors locally
[ ] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

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

Thanks for the contrib -- however, I doubt this is the only place where the code is not 32-bit friendly.

Yes, at least in my repo, after fixing it, the gomobile bind run successfully.


New esob

Adding a new option for energy SOB to match csky treatment(although I think the csky is one wrong)
Adding a util function to trim the simulation.

Just adding new functionality, I will merge it.


cc-wrapper: Always export environment variables for binutils

Motivation for this change
Before, this only happened when cross compiling. This is less of a hack (and soon the entire preWrap script will go), uniformity is good.
Things done
Built Linux and Darwin stdenvs.

CC @ali-abrar This narrows down the bugs we've seen to the infixSalt-ing, basically.

Well, this one is uncharacteristically small an readable :).

IMHO we went in the wrong direction, we should have stopped (and should stop) exporting AR, AS, LD etc. [1] (everything but CC [2]) for cross rather than start exporting them for native, and provide the cross utilities under unprefixed names in the PATH instead. This would have avoided (and will avoid) much debugging and unnecessary workarounds against upstream build scripts, both for native and for cross packaging [3].
[1] This does not apply to BUILD_{var} and {var}_FOR_BUILD.
[2] CC is used consistently upstream and it smooths the difference between gcc and clang.
[3] For example ed will just build, without #32489.

@orivej my original goal was to end up always prefixing all tools with their target platforms (wrapper script only for multi-platform tools). I think the current design makes more sense with that objective.
Yours probably would be epitomized with instead prefixing the binaries with build- and target-, which would also work, and avoid a lot of the of the add-flags.sh and role craziness. But it would mean that buildPackages.buidlPackges.my_compiler (e.g. buildPackages.stdenv.cc) would not work.


update builtin targets to Rust 1.54.0

Checklist

[x] I have read the Contributor Guide
[x] I have read and agree to the Code of Conduct
[x] I have added a description of my changes and why I'd like them included in the section below

Description of Changes
This includes several changes:

Rust 1.54 introduces the new WASM target family. More target families
might be added in the future, so make this enum open-ended like Arch, Os
and others.
Use builtin constants for target-lexicon matching.
Update the builtin targets and target-lexicon, and handle the new bpf
architectures and wasm family.

I believe that with this change a new version of cfg-expr can be released.


[SPARK-18080][ML][PySpark] Locality Sensitive Hashing (LSH) Python API.

What changes were proposed in this pull request?
Add MinHash and RandomProjection Python API.
How was this patch tested?
Add doc tests.

Test build #68135 has started for PR 15768 at commit 85d22c3.

Test build #68135 has finished for PR 15768 at commit 85d22c3.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class LSHParams(Params):
class LSHModel():
class MinHash(JavaEstimator, LSHParams, HasInputCol, HasOutputCol, HasSeed,
class MinHashModel(JavaModel, LSHModel, JavaMLReadable, JavaMLWritable):
class RandomProjection(JavaEstimator, LSHParams, HasInputCol, HasOutputCol, HasSeed,
class RandomProjectionModel(JavaModel, LSHModel, JavaMLReadable, JavaMLWritable):

Merged build finished. Test FAILed.

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

Test build #68136 has started for PR 15768 at commit cdeca1c.

Test build #68136 has finished for PR 15768 at commit cdeca1c.

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

Merged build finished. Test PASSed.

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

cc @jkbradley @sethah

I began to review this, but got sidetracked with a lot of the details we are currently discussing on the original LSH PR.

This can now proceed since [https://github.com/apache/spark/pull/15874] has been merged.  Sorry for the delay!  This will need to slip to 2.2

Pinging on this: What's a reasonable ETA for updating the PR?  Thanks @yanboliang !

@jkbradley I just came back from vacation, and will update this PR before the weekend. Thanks.

Test build #3550 has started for PR 15768 at commit cdeca1c.

Test build #3550 has finished for PR 15768 at commit cdeca1c.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

Btw, @yanboliang and @Yunni  did you sync?  I'm fine with the takeover, but don't want to stomp on toes.  Both can be listed as authors when this gets merged.  Should we close this issue with the other taking its place?

I'm OK to close this one and glad to help to review #16715 , but I think I will have time until next week.


Indexing project fails because rebar3 is not included in bundled Elixir

Fresh install of 0.21.1 using elixir-tools.nvim, I don't get any completions or symbols for my project because indexing fails.
The log shows the following:
23:25:05.192 [debug] sent notification server -> client window/logMessage

23:25:05.192 [info] Shall I install rebar3? (if running non-interactively, use "mix local.rebar --force") [Yn]

23:25:05.197 [debug] sent notification server -> client $/progress

23:25:05.197 [debug] sent notification server -> client window/logMessage

23:25:05.197 [warning] Unexpected compiler response: {:error, %{message: "Could not find \"rebar3\" to compile dependency :poolboy, please ensure \"rebar3\" is available", __struct__: Mix.Error, __exception__: true, mix: 1}}

Since the runtime is now bundled with next-ls, I don't know how I can install rebar myself. All dependencies work locally using Elixir 1.16.2, OTP 26.2.4 both installed using mise.

Can you delete this directory ~/.cache/elixir-tools/nextls/elixir and then restart?
I think maybe something went wrong where it unarchived the bundle but failed before installing rebar and hex.

Actually I think #452 will fix it.
if you can, can you try and build it locally and try (you'll need to first do ./burrito_out/next_ls_<os>_<arch> maintenance uninstall and then set the cmd field in the setup in elixir-tools.nvim to point to the binary for your OS
you'll need zig installed tho, but mise should install it
BURRITO_TARGET=linux_amd64 MIX_ENV=prod mix release --overwrite
you can see the valid burrito targets for your machine in mix.exs (or else it will build all of them)
or i can release the linked patch and you update and see if it works

OK, update when you can and tell me if it solves your problem

Hey, sorry I couldn’t test it before you cut a release, time zones and stuff meant I was snoring away.
Can confirm that 0.21.2 fixes the issue and my project gets indexed now. Thanks so much!

It looks like the issue is still present in 0.23.2
[Next LS] NextLS v0.23.2 has initialized!
[Next LS] [extension] Credo initializing with options %NextLS.InitOpts.Extensions.Credo{enable: true, cli_options: []}
[Next LS] Booting runtimes...
[Next LS] Beginning DB migration...
[Next LS] Database is on the latest version: 7
[Next LS] Finished DB migration...
[Next LS] Booting runtime for .

...

Environment:

LSP=nextls
NEXTLS_PARENT_PID=g1h3K25leHQtbHMtMTcyNDkxODQ2NzI4ODMxMzI1MEBNYWNCb29rLVByby1UaW0AAAEHAAAAAGbQKsQ=
MIX_ENV=dev
MIX_TARGET=host
MIX_BUILD_ROOT=.elixir-tools/_build
ROOTDIR=false
BINDIR=false
RELEASE_ROOT=false
RELEASE_SYS_CONFIG=false
PATH=<...>
[Next LS] Connected to node nextls-runtime-1724918468870896042@MacBook-Pro-Tim
[Next LS] Runtime for folder  is ready...
[Next LS] Could not find "rebar3", which is needed to build dependency :unicode_util_compat
[Next LS] Shall I install rebar3? (if running non-interactively, use "mix local.rebar --force") [Yn]
[Next LS] Unexpected compiler response: {:error, %{message: "Could not find \"rebar3\" to compile dependency :unicode_util_compat, please ensure \"rebar3\" is available", __struct__: Mix.Error, __exception__: true, mix: 1}}

Removing the .elixir-tools and _build folders and restarting the server/editor has fixed the issue


URL Rewrite Check fails

Hi,
I have followed steps as closely as possible to homeinstall script except I can't run it directly because while I do have Debian 9 on the system, I have an existing Apache Server running and will be using virtual hosts and I use certbot for getting Let's Encrypt certificates and enfocing https which is where I think my issue is coming from....
As can be seen on attached screenshot, I am getting the error that suggests URL Rewrite check is failing:

I have tried the following without success:
I have removed htaccess, added it back, modified the rules from script on the apache conf file while keeping and excluding the rules added by Let's Encrypt and this error just wont go away.
Any advice on what I can do to get past this will be hugely appreciated.

Check for "AllowOverride All" in the site Apache configuration file and Apache mod_rewrite installed and restart Apache after changing anything.
If all of those things have been done, we have occasionally seen false positives for this check because of

SSL configuration
DNS/firewall restrictions which prevent a site talking to itself.

From a command line on the server, try 'curl https://...../test/rewrite' and it should pinpoint what's going on. You should get a text 'ok' response iirc. The browser may not be a sufficient test in this case but curl should exercise it in the same way as the server code.

Hi Thanks for the response. I added the following in both site.conf and site-le-ssl.conf:
                AllowOverride All
        </Directory>

then I ran the curl https://.../test/rewrite and it worked


Paste Not Working

Hi –
I am trying to paste items from the clipboard into an MS Word document. I've tried using hotkeys and all of the following, and nothing seems to work.
To choose the history item and paste, press OPTION (⌥) + ENTER, or OPTION (⌥) + CLICK the item, or use OPTION (⌥) + n shortcut.
To choose the history item and paste without formatting, press OPTION (⌥) + SHIFT (⇧) + ENTER, or OPTION (⌥) + SHIFT (⇧) + CLICK the item, or use OPTION (⌥) + SHIFT (⇧) + n shortcut.
Please advise. Thank you.

Does it work in other apps or only Word?
When Maccy tries pasting, it should show a dialog asking for "Accessibility" permissions to perform the paste. Do you remember seeing and accepting it?
Can you double-check that Maccy is present in System Preferences / Security & Privacy / Privacy / Accessibility? If it's not there, please add it by clicking on the "+" button and selecting Maccy.
If nothing works after the previous step, please share the screenshot of the "Preferences" window of Maccy.

Just as a heads up, when the original dialog asking for "Accessibility" permissions popped up, I accepted and it took me to System Preferences, but Maccy was not automatically populated into the existing list. Clicking on the "+" button and selecting Maccy.app in my applications folder did add it though, and it works fine after that.
For reference, I'm using Monterey 12.0.12 (21A559) on a 2020 M1 MacBook Air. Thanks for the great utility, p0deje!


chore(webpack): report errors when build fails

Context

Changes proposed in this pull request

Whenever a build occurred, Webpack was not reporting the errors.

Extension builds preview

Name
Link

Latest commit
575049850881b837ec28fbc40242e144f516ad80

Latest job logs
Run #9090813208

DownloadDownload


inline scripts/styles in head.html gets wrongly minified

Description
The only place where inline scripts/styles are allowed is in the head.html. the pipeline minifies it wrongly.
eg:
<script>
// test
a=1;
</script>

becomes:
<script>// test a=1; </script>

see https://github.com/rehypejs/rehype-minify/issues/44


Removed PickleCorpusView

This will remove PickleCorpusView feature in view of pickle-less NLTK.
This will also invalidate  #2331 #3124 #3230
Related: #3266

Merging this first to remove pickle from codebase.


Assertion failed

What happened?
I was playing with the "Sunflower" DartPad and got the following "Assertion failed" in the console.
Steps to reproduce problem
I don't know
Additional info
Browser
Firefox 78.0.2
Machine
Operating system: Windows 10
DartPad's output
Error: Assertion failed: file:///Users/brettmorgan/Documents/GitHub/dart-services/flutter/packages/flutter/lib/src/foundation/stack_frame.dart:113:12
match != null
"Expected DartError@https://storage.googleapis.com/compilation_artifacts/2.9.0-14.1.beta/dart_sdk.js:6079:7 to match RegExp/^(.+) (\\d+):(\\d+)\\s+(.+)$/."Error: Error: Assertion failed: file:///Users/brettmorgan/Documents/GitHub/dart-services/flutter/packages/flutter/lib/src/foundation/stack_frame.dart:113:12
match != null
"Expected DartError@https://storage.googleapis.com/compilation_artifacts/2.9.0-14.1.beta/dart_sdk.js:6079:7 to match RegExp/^(.+) (\\d+):(\\d+)\\s+(.+)$/."

@RedBrogdon is this the result of the name format change in Dart releases tripping up the masking regex?

Yes, this is the "failure to report error messages correctly" issue we've already got on the books (which still needs to be fixed).
See #1574, #1580, and a half-dozen others. Since we don't have the exact code here, I'm going to close this issue as a duplicate.


Visit option not available while pin using PinterestSDK ios

I have tried with 2.3 and latest version(PDKClient) of the PinterestSDK, I Pin it using ` [PDKPin pinWithImageURL:[NSURL URLWithString:[NSString stringWithFormat:@"%@",imageURL?:[NSNull null]]] link:[NSURL URLWithString:[NSString stringWithFormat:@"%@",sourceURL]]
suggestedBoardName:@"Test" note:@"Test Pin" withSuccess:^
{
NSLog(@"successfully pinned pin");
}
andFailure:^(NSError *error)
{
NSLog(@"pin it failed Error:%@",error);
                  }];`

pin posted successfully , but visit option is missing in iOS alone. Android and Html it shows the visit button and it redirect to respective webpage of the client website

Yes. I have a same issue #118  If you are using older version pinterest app, it will fine. I guess there's something wrong for the latest pinterest app.

@djbalakrishnan @jay-zhu - Can you verify this is still occurring in the latest version of the Pinterest app (6.15)?

@rahul-malik ,no, still cannot visit the link.

@rahul-malik & @jay-zhu  , I have find out the issue, we are encoding share url and PDK again encode the same leads to the problem, now I have identified and corrected by remove encode in my side, it is working as expected now


Update S3 backend version to v0.1.4 to fix trailing / bug in URLs

@turetske, I'll cut a v7.9 release when this is passing tests and gets merged. It fixes a bug in the S3 plugin that mishandles S3 service URLs with a trailing slash. Contents of the bugfix can be found here

Bypassing branch protections to merge this since tests are passing and this is an internal container version upgrade.


[SPARK-47973][CORE] Log call site in SparkContext.stop() and later in SparkContext.assertNotStopped()

What changes were proposed in this pull request?
This change logs and stores the call site when SparkContext.stop() is called, and later when SparkContext.assertNotStopped() is called.
Why are the changes needed?
This is for better debuggability.
Does this PR introduce any user-facing change?
Yes, the error message shown
How was this patch tested?
Updated unit tests.
Was this patch authored or co-authored using generative AI tooling?
No.

Merged to master.


Valkyrien Skies 2 compatibility

Forgero crashes at world generation menu when launching with Valkyrien Skies.

without log no fix

Whoops, sorry.
latest.log

ServerInteractionManagerReachMixin is the problem here, we are already looking to change this as it also breaks our mod with sinytra so hopefully in the next update

Awesome, thank you.

This was closed, but the crash still happens. A compatibility would be much appreciate, I love this mod.
crash-2024-08-26_22.26.20-client.txt

The patch that fixes this issue has not yet been published to curseforge or modrinth. I expect to release the latest version in a couple of days!

Oh, great. Looking forward to it!

Fixed in v0.12.5.


Internal error on calling /mine [GET]

I followed the instruction. Tried http://localhost:80/mine in postman.
> docker run --rm -p 80:5000 blockchain
 * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)

[2020-03-04 19:22:19,375] ERROR in app: Exception on /mine [GET]
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "blockchain.py", line 229, in mine
    return jsonify(response), 200
  File "/usr/local/lib/python3.6/site-packages/flask/json.py", line 251, in jsonify
    if current_app.config['JSONIFY_PRETTYPRINT_REGULAR'] and not request.is_xhr:
  File "/usr/local/lib/python3.6/site-packages/werkzeug/local.py", line 347, in __getattr__
    return getattr(self._get_current_object(), name)
AttributeError: 'Request' object has no attribute 'is_xhr'
<IP_ADDRESS> - - [04/Mar/2020 19:22:19] "GET /mine HTTP/1.1" 500 -

System Windows 10, Docker version <IP_ADDRESS>.
Btw. The strange thing is that it shows it's running on <IP_ADDRESS> instead of <IP_ADDRESS>.

It's no problem that it's running on <IP_ADDRESS>, you can hit the endpoint at http://localhost:5000.
As for the error, Flask has a dependency that received a major version update somewhat recently. The easiest fix is to upgrade your version of Flask by running pip install --upgrade Flask.
Check out this stackoverflow answer for more details and a different solution: https://stackoverflow.com/a/60137731

Is it resolved?

My problem is similar to yours:


validate build & run on Windows

Someone told me they had trouble building on Windows. That should be possible, so take the time to try it and work around any problems with it.

I've installed the latest Go 1.16 64-bit on Windows 10 Pro and the build went fine.
otel-cli span background fails but that's expected with the Unix socket. Will need to come up with an alternative for Windows if folks want it.

With releases available this should be good for now.


Using images or materials for shape fill

I am wanting to use an image as a fill for polygons and paths. I can't seem to pin down how I would apply this to results from GeometryBuilder::build_as or using ShapeBundle {} struct
    let shape = shapes::Polygon {
        points: options.points.to_vec(),
        closed: true,
    };

    commands
        .spawn_bundle(GeometryBuilder::build_as(
            &shape,
            DrawMode::Outlined {
                // Would expect I could use a FillMode::image(Handle<Image>) for the fill mode
                // or FillMode::material(
                //     materials.add(CustomMaterial {
                //       color: Color::RED,
                //       color_texture: asset_server.load("some_image.png"),
                //     })
                // )
                fill_mode: FillMode::color(Color::CYAN),
                outline_mode: StrokeMode::new(Color::BLACK, 0.0),
            },
            Transform::default(),
        ))

Is this possible currently?
Thank you!

It's one of those features that I wanted to implement, but currently I don't want to commit much time into developing new features.
If anybody wants to tackle this, feel free to do it. This issue can also be used to discuss a potential solution.

Thank you Nilirad!  Is this something that lyon can do in its library, or is it something specific that will need to be added to bevy_prototype_lyon to have an image fill? Or is this something with adding the image to the geometry inside bevy_prototype_lyon? The GeometryBuilder makes me think it creates a mesh, so we can apply the material to the mesh?
I wouldn't mind looking into it, but I'm newer to rust/bevy and this library. Need some pointing to areas of interest, if possible.
For instance, I think this is the closest example of a struct that has a material and a mesh2d, MaterialMesh2dBundle. So do we need to add a material to this struct? I can look into this option later.
Thank you!

Hello I am also interested in this. I'm curious if in the meantime if it would be possible to use the existing mesh2d handle from the created shape in the materialmesh2d bundle. I'm gonna do some testing on how to do this without modifying the library cause I am not at the skill level to do that but I will update here if i find a temporary solutions.

So after messing around for a bit it seems one of the limiting things is the lack of vertices having uv's but I may be wrong i'm very tired. I'm mildly curious why you use a custom render pipeline? Out of wanting to legitimately learn why wouldn't bevy's default pipeline work? It would probably make it easier to do stuff like materials as well. Just curious honestly same with the custom shader. Either way awesome library!

I don't exactly remember why I initially chose to use a custom render pipeline. Also, a lot of the new rendering code was made by @rparrett after the initial release.
A lot of things have changed, both on bevy and lyon, since I created this crate, so it may be possible that a simpler solution would work.

Maybe I will mess around trying to use the standard bevy pipeline in a fork.

I have been messing for a few hours and I don't think I'm familiar enough with bevys render pipeline yet to do this myself but I may tackle it again in the future.

I don't exactly remember why I initially chose to use a custom render pipeline. Also, a lot of the new rendering code was made by rparrett after the initial release.

I think it may have been necessary at some point in the past in order to use vertex colors. I'd guess that it's not necessary anymore and you could just use a MaterialMesh2d.
Another obstacle might be supporting DrawMode::Outlined properly, since both the fill and stroke share the same mesh/entity.

Any chance of this happening? It would solve my current problem very nicely.


print

From<EMAIL_ADDRESS>on 2012-06-12T11:37:12Z
Spyder Version:  2.1.9
Python Version:  2.7.2
Qt Version:      4.7.4, PyQt4 (API v1) 4.8.6 on Windows
What steps will reproduce the problem?

What is the expected output? What do you see instead?

Please provide any additional information below
.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1090

From ccordoba12 on 2012-06-13T05:10:25Z
We don't know what you were trying to do. If you have problems, try to open an issue with a detailed report.
Status: Invalid


add theme pandollo for hexo

Thank you so much for your contribution.


Keeping single/multile focused input in visible area while user is typing on iOS/Android

The main goal of this feature is to enhance the behavior for users who are accustomed to automatically focused inputs in their native application. Since this library is the best cross-platform implementation for keyboard handling, it would be great to have such functionality out of the box.
After some research, we found out that iOS native implementation doesn't have this feature, so you have to play with native TextFields manually for achieving it. On the other hand, Android supports it without any extra manipulation.
In the current code examples, the most suitable place where we can implement this is within the KeyboardAwareScrollView. One way to achieve this autoscroll behavior is to calculate somehow if the current target is under the keyboard and then scroll to the focused input. We can extend this method using the React useImperativeHandle hook and then connect it to KeyboardAwareScrollView, but I'm not sure if it will work well without any performance loss.

Hello @VladyslavMartynov10 👋
Good question!
Actually autoscroll happens on iOS (when you type first letter, because iOS is emitting keyboardWillShow/keyboardDidShow events and we call maybeScroll function): https://github.com/kirillzyusko/react-native-keyboard-controller/blob/5c587ca331eedb4b906e21d9774226b6294b0aad/example/src/components/AwareScrollView/KeyboardAwareScrollView.tsx#L87-L102
If you again scroll your focused input to invisible area and type another symbol - input will remain invisible (because no new keyboardWillShow/keyboardDidShow events will be emitted).
I was thinkning on how to create a cross-platform solution and right now I'm thinking about enhancing useReanimatedFocusedInput hook. I want to change a signature to be:
const {input} = useReanimatedFocusedInput({
  onChangeText: (text) => {
    // we can call our internal `maybeScroll` on every text change
    // in native code before this event we need to assure that `layout` has up-to-date value
    // so we can call `maybeScroll` and it'll assure, that focused TextInput remains visible even
    // if user intentionally moved it out of visible area
  },
});

Also I think that such behavior can be activated on KeyboardAwareScrollView level via prop like keepVisibleWhileTyping.
What do you think about this approach?

Yeah, I just forgot that now library is using  useReanimatedFocusedInput hook which can give us all necessary layout/focus information. I think the approach you proposed is the most appropriate here and easily extendable in the future.

@VladyslavMartynov10 I think the reason why it doesn't work now is because this condition:
if (focusWasChanged && !keyboardWillAppear) {

is evaluated as
if (false && true) {

And maybeScroll is not getting called. Which is kind of good, because we have cross-platform code that works in identical way on both platforms iOS/Android - and if we add additional scroll-on-text-change logic, then we can be sure there is no other conflicts with remaining code 😎
I'll try to enhance the functionality of useReanimatedFocusedInput to achieve the behavior you described 👍

Thank you @VladyslavMartynov10 for raising that problem and for your sponsoring! I appreciate that!
I've added the requested functionality and updated KeyboardAwareScrollView example.
I'm going to roll out a new release 1.10.0 soon (hopefully before a New Year 😅) and include KeyboardAwareScrollView as a pre-built component available out-of-box from this library.
But if you need these changes ASAP then you can use a version from main branch :)

Thanks @kirillzyusko, great work! I'll write back some feedback after several testing.


prow/cmd/hook: add push bazel rules

Not sure if this creates the correct image but since we just need the hook image for testing the approval plugin, I'm hoping this would work.
With this change,

to build the image -> bazel run //prow/cmd/hook:image
to push the image -> bazel run //prow/cmd/hook:push

(both at root of test-infra repo)
Dockerhub repo is at - https://hub.docker.com/repository/docker/nikhita/hook
cc @ykakarap

Btw I don't think you need to merge this, just created the PR so we have a log of what changes went in for testing, etc.
Alternatively, we can use phony for testing the approval plugin too but it might be cumbersome - https://github.com/kubernetes/test-infra/tree/master/prow/cmd/phony#phony


Fix Join Ride Functionality

Join Ride Functionality not working

This feature needs to be built from the ground up.
The button is just there on the FE. No calls are going to the BE for this

Identified the issue. API Call is missing id parameter of the route.
Reason: Not being rendered properly from the BE

Raised #6 as a fix for this


fix: implement db reset using pgx

What kind of change does this PR introduce?
enhancement
What is the current behavior?
db reset runs psql inside a container to apply migration scripts.
reset failure will wipe out existing data in local postgres
What is the new behavior?
Opens a pgx connection to apply migrations

more efficient than copy + exec seed file in container
supports line specific debug output

Also applies migrations to a separate clean database before switching over from postgres.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 85 (0.0%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.7%) to 27.891%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/db.go
0
2
0.0%

internal/db/reset/reset.go
0
83
0.0%

Files with Coverage Reduction
New Missed Lines
%

cmd/db.go
1
44.44%

internal/db/reset/reset.go
2
0%

Totals

Change from base Build<PHONE_NUMBER>:
0.7%

Covered Lines:
1370

Relevant Lines:
4912

💛  - Coveralls


Estimated function is not consistent with data

Hi,
I found this weird bug when plotting the estimated curve against the observed data. Any idea why this is happening? I only adapted your code to include my data (matrix X). Thank you!
`for i, term in enumerate(gam.terms):
if term.isintercept:
continue
XX = gam.generate_X_grid(term=i)
pdep, confi = gam.partial_dependence(term=i, X=XX, width=0.95)

plt.figure()
plt.plot(XX[:, term.feature], pdep)
plt.plot(XX[:, term.feature], confi, c='r', ls='--')
plt.scatter(X.iloc[:,i], y, facecolor='gray', edgecolors='none')

plt.title(repr(term))
plt.show()`

Ah wow this is a weird case indeed!
But at first glance it seems to me that the model has produced a good representation of the data. [1]
Can you tell me what you were expecting?
Also, could you share some details about the model setup?

distribution (normal, gamma, poisson...)
link function (linear, log, etc)
lam smoothing value and any other penalties

And then share a histogram of your Y-data / dependent variable?
[1] my interpretation:

It is hard to tell from the plot (due to alpha/transparency), but it seems like there is a lot of density close to S2=0 at least for X2<40.
It seems like the model has correctly identified this pattern (although with an offset) and has set the response of S2 to a pretty constant and low number.
At X2 > 40 there is little data, and the few samples have large response, so the model uses its flexibility to assign larger S2 values there.

I see some undesirable qualities of this model:

which is the "ringing" towards the right side (in order to assign a high response to the single point at  X2=45, the model needs to correct and add an overly negative response to the neighboring splines
the obviously different model response between X2 < 40 (mostly 0) and X2 > 40 (large and positive).

to deal with point 1 i think we could add more flexibility to the model (more splines) so the neighboring splines have more freedom to react independently without needing to correct for the extreme values of their neighbors. You could also increase the lam smoothness penalty for this dimension in order to enforce more similar values (so that one isnt large and positive while its neighbor is large and negative)

to deal with point 2 you could simplify the model by changing this response of X2 from spline to linear, or alternatively keep splines functions for this dimension while increasing smoothness penalty.

Let me know what you think!


Sync VSO - GitHub

Sync VSO - GitHub.
npm package version 0.5.3

Hi @ignatvilesov, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Vilesov Ignat (Akvelon)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;